[$500]  Emoji – Web- Delay when change default skin color

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

Version Number: 1.4.31-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Navigate to staging.new.expensify.com
Open any conversation
Click on Emoji button> Change default skin color
Select other skin color than default one

Expected Result:
Selected skin color should be instantly changed when user select it
Actual Result:
There is a delay when change default skin color
Workaround:
Unknown
Platforms:

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

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/67b5e4da-461e-4662-872a-52a20dd70fa5
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0162088f4af81577d8
Upwork Job ID:<PHONE_NUMBER>674102272
Last Price Increase: 2024-01-24

We think that this bug might be related to #vip-vsp
CC @quinthar

Proposal
Please re-state the problem that we are trying to solve in this issue.
There is a delay when changing the default skin tone.
What is the root cause of that problem?
We are first hiding the emoji skin tone selector, and then it gets updated async:
https://github.com/Expensify/App/blob/4db7b095495476cb45a6c9216027b66d7cfcb338/src/components/EmojiPicker/EmojiSkinToneList.js#L25-L33
What changes do you think we should make in order to solve the problem?
When updating the skin tone, we should hide the skin tone selector after it's been set (after interactions):
    function updateSelectedSkinTone(skinToneEmoji) {
        setHighlightedIndex(skinToneEmoji.skinTone);
        setPreferredSkinTone(skinToneEmoji.skinTone);
        InteractionManager.runAfterInteractions(() => {
            toggleIsSkinToneListVisible();
        });
    }

What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
delay in emoji toggle
What is the root cause of that problem?
The delay is attributed to the use of the usePreferredEmojiSkinTone hook.
https://github.com/Expensify/App/blob/4db7b095495476cb45a6c9216027b66d7cfcb338/src/components/EmojiPicker/EmojiSkinToneList.js#L19
What changes do you think we should make in order to solve the problem?
just change usePreferredEmojiSkinTone to useState hook will solve the problem
Result
https://github.com/Expensify/App/assets/60073279/9d00914f-eaa0-43ec-b27e-2fb34fb37d9c

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01a5c57bf278a77749

I added. this issue to #vip-vsb as a low priority item, but I do think we should fix it now if we can.
@rushatgabhane what do you think of the proposals we're gotten so far?

reviewing

I like @tienifr's proposal
🎀 👀 🎀 c+ reviewed

@youssef-lr you're probably right that it's low-impact. But we've been fixing other delay issues, and there's already a C+ recommended proposal, so I think we should fix it.

Sounds good!

Waiting for a PR from @tienifr

payment request here - https://staging.new.expensify.com/r/323761154428333

@sakluger could you please attach payment summary 🙇

Awaiting payment summary from @sakluger

I reached out to him via DM.

Sorry, everyone - the payment summary is below.
Summarizing payment on this issue:
Contributor: @tienifr $500, paid via Upwork
Contributor+: @rushatgabhane $500, please request on Newdot

$500 approved for @rushatgabhane


Fix !== codegen for arm when destination operand is same as both source operands

When destination of CmpSrNeq_A is same as both its sources, we need to generate instructions that saves the value of both the sources in newly created Opnds.
Current codegen did not check the case when destination can be equal to both the sources. This change fixes it.

Hi @meg-gupta, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Meghana Gupta). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

@satheeshravi  Can you please review ?

@dotnet-bot : can you please test windows arm_release ?

:shipit:

@dotnet-bot Please test windows x86_release


Looking for contributors ?

Hey, if I ever get time to do this, would you accept contributions to the efiboot tool ? In particular, I'd like to add a way to delete and add new UEFI boot entries. Maybe even get on feature parity with efiboomgr ?

@vtavernier Hi, I see you are reviewing my PRs right now, so I wanted to take this opportunity to ask you something.
I'd like to fork this repository to continue working on the project from there
My reasons are:

As much as I love you reviewing my PRs (I'm still relatively new in Rust), I find it a bit annoying to have to wait ~a week each time for my PRs to be merged (This is not in any way a criticism addressed to you, you can do whatever you want with your free time)
I plan to contribute in the long term to this project, and I'd like have more control over its infrastructure, in the case you don't come around anymore and I find myself with no access to continue working

Forking the project itself is not a big problem, but I'd like to have access to the https://crates.io/efivar crate to be able to update it myself (not really efiboot, I plan to change its name to efivarcli since its not limited to boot variables stuff).
Plus, I kinda wanted an excuse to talk to you about this, since I think it would be an asshole move to just fork the project and move on without talking to you about it
So.. would you be okay with my forking the project, and would you be okay with giving me access to publish the efivar crate ?

Hello @iTrooz
Thanks for being upfront about this. I've been busy recently (and away for the past two weeks), hence the delay in reviewing PRs. I haven't caught up yet but looking at the recent code you pushed, I think it makes sense you could become the maintainer for this project.
Since I'm available to figure this out, here's my proposal:

I add you as owner of efiboot and efivar crates on crates.io (there's no point in me keeping access to only one of those)
I transfer this repository from my user to yours on Github (in the Settings > Transfer repository option)

I think the transfer option is preferable to a fork, since it'll automatically redirect users to the new home for the project, as well as keep all issues and open pull requests. What do you think?

I think that would be great to keep issues and PRs, yeah !

I add you as owner of efiboot and efivar crates on crates.io (there's no point in me keeping access to only one of those)

Just to be sure I understand, do you want me to remove your access from the crates once I am an owner ?

I add you as owner of efiboot and efivar crates on crates.io (there's no point in me keeping access to only one of those)

Just to be sure I understand, do you want me to remove your access from the crates once I am an owner ?

Yes, but if you plan to stop maintaining the crates at some point in the future don't hesitate to reach out to me if you don't find another maintainer to replace you then.

Okay !

I've sent you the invites for crates.io on both crates, but on GitHub you'll need to rename your copy of efiboot-rs to something else so I can initiate the transfer

I've just accepted the crates.io invitations, and I've renamed my fork

Ok so it turns out you can't transfer a repository to an organization that contains a fork of the repository itself (iTrooz already has a repository in the vtavernier/efiboot-rs network).
I think you'd have to delete your fork, then we do the transfer and you can push your (local) branches to the new repository since you'll become an admin of the repository and re-open the PRs (they'll close automatically when deleting the fork I assume). Would that be okay with you?

Yep ! I just did it, and it indeed closed the PRs

You should have received the transfer request now!

I have the repository !
Well.. thanks for everything, don't hesitate to come by in the future ^^

Sure thing, thanks for taking on the maintenance!


Doc RichTextLabel push strikethrough and align

Is there a reason why the Align argument is showing a link but this one does not https://github.com/godotengine/godot/pull/22942/files#diff-3b1b61c395a8be4793176830cfd0b61bR75

Thanks!

Cherry-picked for 3.0.7 (the push_align part, the other one is not relevant in 3.0.x).


Testing on Ubuntu

I am currently testing the install on Linux Ubuntu. However I am having issue on port 53 being used by the standard Linux DNS server. So  I get an error message saying that the port is already taken.
How did you make it work on Linux?
Can I change the port of docker dnsmasq to 53 to eg port 54?
I understand that DNSmasq is there so that all new created domains are available outside the docker environment?

What is the reason you are forwarding port 53:53? There is no need for this I think as your Mac resolver (etc/resolver) is forwarding all .test traffic to <IP_ADDRESS>
I've tested it without port fowarding 53:53 and it is also working.

Hey @freeyland,
We've not tested on Linux, so it would be great if you can test it!
Regarding 53 port, it is default configuration from Dnsmasq docker image.

Hey @shkoliar ,
The issue for Linux is that  port 53 is already taken by the default dns server from Ubuntu (or by standard dnsmasq installed on Ubuntu). I  am searching how the Mac /etc/resolver/test is going inside docker dnsmasq on port 53.
I don't think is a good idea to stop standard Ubuntu DNS server as when the docker is not started you won't have any network.

Following issues already found on Linux:

domain check not working
admin path url check not working
During init you do a push of scripts to container (e2c_push_exec). However this should not be done for Linux. The help message is displayed instead and the script is stopped.

I will continue testing....

https://github.com/freeyland/e2c/tree/master
is working  with Linux. I still need to do some smaal tweaks. I will then make the changes on your repo  and make a merge request

https://github.com/freeyland/e2c/tree/master

btw, @freeyland you have to keep the original copyright notice due to MIT License rules.

Ok, will add it

@freeyland As I've noticed you are copying every our commit into your repo with adding your authorship. GitHub has fork and pull requests functionality which may allow you to have less "rewriting" work, of course if your goal is not "re-author" all this work.
It would be great if you can also give us feedback on Linux usage and create pull requests if you have improvements for Linux based systems.

@shkoliar Today I am doing the latest tests on Ubuntu.
As promised, I will then merge all the changes I've made

All working now on Ubunu. See latest merge request


Pydantic Error : A non-annotated attribute was detected: is_crew_class = True | this is not from my code

Traceback (most recent call last):
File "", line 1, in
File<EMAIL_ADDRESS>line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "", line 1204, in _gcd_import
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "", line 940, in exec_module
File "", line 241, in _call_with_frames_removed
File "/Users/jackmullen/jmautogpt/news_agency_1/main.py", line 7, in
from crew import NewsSourceResearcher
File "/Users/jackmullen/jmautogpt/news_agency_1/crew.py", line 12, in
@CrewBase
^^^^^^^^
File "/Users/jackmullen/jmautogpt/news_agency_1/.venv/lib/python3.11/site-packages/crewai/project/crew_base.py", line 12, in CrewBase
class WrappedClass(cls):
File "/Users/jackmullen/jmautogpt/news_agency_1/.venv/lib/python3.11/site-packages/pydantic/_internal/_model_construction.py", line 93, in new
private_attributes = inspect_namespace(
^^^^^^^^^^^^^^^^^^
File "/Users/jackmullen/jmautogpt/news_agency_1/.venv/lib/python3.11/site-packages/pydantic/_internal/_model_construction.py", line 406, in inspect_namespace
raise PydanticUserError(
pydantic.errors.PydanticUserError: A non-annotated attribute was detected: is_crew_class = True. All model fields require a type annotation; if is_crew_class is not meant to be a field, you may be able to resolve this error by annotating it as a ClassVar or updating model_config['ignored_types'].
For further information visit https://errors.pydantic.dev/2.7/u/model-field-missing-annotation
The file that caused that error below:
import os
from crewai import Agent, Crew, Process, Task
from crewai.project import CrewBase, agent, crew, task
from crewai_tools import WebsiteSearchTool
from langchain_community.tools import DuckDuckGoSearchRun# Initialize the tool
search_tool = DuckDuckGoSearchRun()
from langchain_groq import ChatGroq
import sqlite3
@CrewBase
class NewsSourceResearcher(Agent):
def __init__(self, zip=None) -> None:
    if zip == None:
        pass
    # add other models here
    self.groq_llm = ChatGroq(temperature=0, model='llama3-8b-8192')
    self.agents_config = 'config/agents.yaml'
    self.tasks_config = 'config/tasks.yaml'

@agent
def source_manager(self) -> Agent:
    return Agent(config=self.agents_config['source_manager'],
                 llm=self.groq_llm,)
@task
def source_manager_task(self) -> Task:
    return Task(
        config=self.tasks_config['source_manager_task'],
        agent=self.source_manager(),
        tools=[search_tool,WebsiteSearchTool(),sqlite3]
    )

@crew
def crew(self) -> Crew:
    return Crew(
        agents=self.agents,
        tasks=self.tasks,
        process=Process.sequential,
        verbose=2

        )

I upgraded to  install crewai==0.30.0rc6 crewai-tools==0.2.3 but Poetry downgraded both versions.

Should be fixed no on v0.30.8 :D, can you double check

I am sorry -- but it now has a new error -- I upgraded to 0.30.8 and tools 0.2.3
Traceback (most recent call last):
File "", line 1, in
File<EMAIL_ADDRESS>line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "", line 1204, in _gcd_import
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "", line 940, in exec_module
File "", line 241, in _call_with_frames_removed
File "/Users/jackmullen/jmautogpt/news_agency_1/main.py", line 7, in
from crew import NewsSourceResearcher
File "/Users/jm/jmautogpt/news_agency_1/crew.py", line 12, in
@CrewBase
^^^^^^^^
File "/Users/jm/jmautogpt/news_agency_1/.venv/lib/python3.11/site-packages/crewai/project/crew_base.py", line 13, in CrewBase
class WrappedClass(cls):
File "/Users/jm/jmautogpt/news_agency_1/.venv/lib/python3.11/site-packages/pydantic/_internal/_model_construction.py", line 93, in new
private_attributes = inspect_namespace(
^^^^^^^^^^^^^^^^^^
File "/Users/jm/jmautogpt/news_agency_1/.venv/lib/python3.11/site-packages/pydantic/_internal/_model_construction.py", line 343, in inspect_namespace
all_ignored_types = ignored_types + default_ignored_types()
^~~~~~~~~~~
TypeError: can only concatenate list (not "tuple") to list

weird that I can't replicate the problem
but let me try chancing it again

I am using poetry - and not in the case above -- but if run poetry install it will downgrade both crewai and the tools -- I am not in love with poetry - should i remove it ?

I am here now if you wanted to help me out -- strange thing is I have another crew setup with the same structure (config file of yamls and methods as shown in my crew.py, and I got it to work. I duplicated the structure and install all new requirements in the new environment and these issues appear, it would then have to be a result of the upgraded modules and their interactions.  Another question that you could answer -- quickly -- I do not seem to be able to add tools using the Tasks yaml file
tools:
- WebSiteSearchTool()
- search_tool
These do not work.

after figuring out that i had a value for tools in pyproject.toml that prevented crewai .. I have a new error message:
Traceback (most recent call last):
File "", line 1, in
File<EMAIL_ADDRESS>line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "", line 1204, in _gcd_import
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "", line 940, in exec_module
File "", line 241, in _call_with_frames_removed
File "/Users/jackmullen/jmautogpt/news_agency_1/main.py", line 7, in
from crew import NewsSourceResearcher
File "/Users/jackmullen/jmautogpt/news_agency_1/crew.py", line 12, in
@CrewBase
^^^^^^^^
File "/Users/jackmullen/jmautogpt/news_agency_1/.venv/lib/python3.11/site-packages/crewai/project/crew_base.py", line 13, in CrewBase
class WrappedClass(cls):
File "/Users/jackmullen/jmautogpt/news_agency_1/.venv/lib/python3.11/site-packages/pydantic/_internal/_model_construction.py", line 91, in new
config_wrapper = ConfigWrapper.for_model(bases, namespace, kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/jackmullen/jmautogpt/news_agency_1/.venv/lib/python3.11/site-packages/pydantic/_internal/_config.py", line 128, in for_model
raise PydanticUserError('"Config" and "model_config" cannot be used together', code='config-both')
pydantic.errors.PydanticUserError: "Config" and "model_config" cannot be used together


add nimitheme

I made a library for adding additional themes to nimib. The repository is here: https://github.com/neroist/nimitheme
It also provides Dark Reader and highlight.js support (you can easily set the code theme you'd like to use).

the repo is here: https://github.com/neroist/nimitheme

That's really cool! 🤩 Thanks a lot for making this and letting us know!

We should for sure link it either here or in nimib readme or both!

Wow! That's awesome indeed, it must have been a fair bit of work just exploring what themes there are out there and wrapping all of them 🤩
I vote for adding a link to nimitheme in both nimib and nimibex!


[Event Request] Codeunit 90 Purch.-Post - OnCheckPurchLineIsSetItem

Hi Team,
In Codeunit 90 we need possibility to have our additional checks . Having event with handling in procedure InsertReceiptLine would be a perfect solution :

[IntegrationEvent(false, false)]
local procedure OnCheckPurchLineIsSetItem(var PurchLine: Record "Purchase Line"; var IsHandled: Boolean)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Subcategories

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
UI Feature
🔖 Feature description
A way to make for a group on the status page subgroups, I have the problem that I have many APIs and proxies to a group and this looks very messy.
✔ Solution
Create a drop-down menu where you can add the service
❓ Alternatives
No response
📝 Additional Context
No response

Is this the same issue as the one that will be adressed in #2693 (i.e. a duplicate of the other issues mentioned there?)
This reads very similar, please elaborate on the difference or close this issue 😉


Fix AVL rotations

I don't like the way AVL Rotations are explained in the slides. Based off of this image, it seems we can transform a LR or RL case into an LL or LR case and just makes more sense:

This PR changes the rotation code to use this. I've tested it for each of the four cases when the root is re-balanced in the process, and once for a LR where the main pivot point was past the root. We should probably test some more though (unit tests ftw?)

Tests will come in #5.


intermediation between 2 users directly

Is your feature request related to a problem? Please describe.
2 users may want to use the brokerage of the bot to make payments-exchanges between them directly without posting their offer to the others
Describe the solution you'd like
It would be necessary to enable, apart from the general group and the community group, a function in the private channel with the bot of the type: "intermediation between 2 users", where the bot acts as insurance for an exchange. where you would put the name of the user to whom it is intended, in the generation of the order
Describe alternatives you've considered
could, instead of being a new command, use the same "publish order" command but with one more parameter: "concrete user"
Additional context
Add any other context or screenshots about the feature request here.

How about this?

Alice creates an "unpublished" order (does not appear on any channel).
Alices gives orderId to Bob by whatever method they want (tg, wsp, etc)
Bob executes /takeorder <AliceOrderId> (#371)


Plugins don't work. TypeError: plugin is not a function

I tried to create a custom plugin as described here, but as soon as I change the plugins to use [...require("phenomic/lib/loader-preset-default").default], I get a Module build failed: TypeError: plugin is not a function build error.
I digged around a bit and found out that using any functions in plugins doesn't work, they get replaced by nulls. It's not an import error, as the type error happens even when I don't require anything and just hardcode a function there:
loaders: [
   // webpack 2
   // rules: [
     {
       test: /\.(md|markdown)$/,
       loader: phenomicLoader,
       query: {
          context: path.join(config.cwd, config.source),
          plugins: [id => id]

In phenomicLoader webpackInstance.query is an array of nulls, whereas numbers/strings in webpack.config's plugins are correctly received.
So what's the issue with putting functions there?

My bad on this. The current documentation is for webpack 2 and webpack 1 does not accept anything that can't be converted to JSON in query.
With webpack 1, you actually need to pass what you pass as a query in a global object in webpack config (like for postcss for example, at the same level of webpack module)

// ...

export default (config = {}) => {
  // ..
  return {
    phenomic: {
      context: path.join(__dirname, config.source),
      plugins: [
        ...require("phenomic/lib/loader-preset-markdown").default
      ]
    },
    module: {
      loaders: [
        // *.md => consumed via phenomic special webpack loader
        // allow to generate collection and rss feed.
        {
          // phenomic requirement
          test: /\.(md|markdown)$/,
          loader: phenomicLoader,
      },

I will fix the doc and base theme.

Thanks, glad it's just a documentation mistake.
It works when I put the context into the query instead of phenomic:
// ...

export default (config = {}) => {
  // ..
  return {
    phenomic: {
      plugins: [
        ...require("phenomic/lib/loader-preset-markdown").default
      ]
    },
    module: {
      loaders: [
        // *.md => consumed via phenomic special webpack loader
        // allow to generate collection and rss feed.
        {
          // phenomic requirement
          test: /\.(md|markdown)$/,
          loader: phenomicLoader,
          query: {
            context: path.join(__dirname, config.source),
          },
      },

Reopening as it needs docs.


get adventure to link

I had difficulty building discobsd because of how a header file for the game adventure didn't use extern in the way modern gcc expects, so here's a commit that fixed it.
I'm having the same problem with another game, atc. From the output of gmake:
gmake[2]: Entering directory '/home/sam/proj/discobsd/games/atc'
/usr/bin/arm-none-eabi-gcc -mcpu=cortex-m4 -mabi=aapcs -mlittle-endian -mthumb -mfloat-abi=soft -nostdinc -I/home/sam/proj/discobsd/include  -N -nostartfiles -fno-dwarf2-cfi-asm -T/home/sam/proj/discobsd/lib/elf32-arm.ld /home/sam/proj/discobsd/lib/crt0.o -L/home/sam/proj/discobsd/lib -o atc.elf extern.o grammar.o input.o lex.o list.o log.o main.o tunable.o graphics.o update.o -lm -lcurses -ltermcap -lc
/usr/lib/gcc/arm-none-eabi/10.3.1/../../../arm-none-eabi/bin/ld: /home/sam/proj/discobsd/lib/libm.a(sqrt.o):(.bss+0x0): multiple definition of `errno'; /home/sam/proj/discobsd/lib/libm.a(asin.o):(.bss+0x0): first defined here
/usr/lib/gcc/arm-none-eabi/10.3.1/../../../arm-none-eabi/bin/ld: /home/sam/proj/discobsd/lib/libcurses.a(cr_put.o):(.bss+0x18): multiple definition of `_win'; /home/sam/proj/discobsd/lib/libcurses.a(refresh.o):(.bss+0x4): first defined here
/usr/lib/gcc/arm-none-eabi/10.3.1/../../../arm-none-eabi/bin/ld: /home/sam/proj/discobsd/lib/libtermcap.a(tgoto.o):(.bss+0x50): multiple definition of `BC'; /home/sam/proj/discobsd/lib/libcurses.a(curses.o):(.bss+0x104): first defined here
/usr/lib/gcc/arm-none-eabi/10.3.1/../../../arm-none-eabi/bin/ld: /home/sam/proj/discobsd/lib/libtermcap.a(tgoto.o):(.bss+0x4c): multiple definition of `UP'; /home/sam/proj/discobsd/lib/libcurses.a(curses.o):(.bss+0x3c): first defined here
/usr/lib/gcc/arm-none-eabi/10.3.1/../../../arm-none-eabi/bin/ld: /home/sam/proj/discobsd/lib/libtermcap.a(tputs.o):(.bss+0x2): multiple definition of `PC'; /home/sam/proj/discobsd/lib/libcurses.a(curses.o):(.bss+0x10): first defined here
/usr/lib/gcc/arm-none-eabi/10.3.1/../../../arm-none-eabi/bin/ld: /home/sam/proj/discobsd/lib/libtermcap.a(tputs.o):(.bss+0x0): multiple definition of `ospeed'; /home/sam/proj/discobsd/lib/libcurses.a(cr_tty.o):(.data+0x0): first defined here
/usr/lib/gcc/arm-none-eabi/10.3.1/../../../arm-none-eabi/bin/ld: /home/sam/proj/discobsd/lib/libc.a(exit.o):(.bss+0xc): multiple definition of `errno'; /home/sam/proj/discobsd/lib/libm.a(asin.o):(.bss+0x0): first defined here
collect2: error: ld returned 1 exit status
gmake[2]: *** [Makefile:25: atc] Error 1
gmake[2]: Leaving directory '/home/sam/proj/discobsd/games/atc'

I think what this means is that some of the libraries (libm, libcurses, libtermcap, libc) each one has the same symbol declared several times). But I am not sure (yet!) how to fix these up.

Hey there,
What is your development environment? I am guessing it is a Linux distro, but to confirm, please give the output of uname -a and also the output of arm-none-eabi-gcc --version.
Also, please make a new issue for the /games/atc issue you've experienced, as it is cleaner to deal with each problem separately of each other.
I will look into your PR. Thanks again for the cooperation!

please give the output of uname -a

Linux [redacted] 5.15.0-56-generic #62-Ubuntu SMP Tue Nov 22 19:54:14 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

and also the output of arm-none-eabi-gcc --version

arm-none-eabi-gcc (15:10.3-2021.07-4) 10.3.1 20210621 (release)
Copyright (C) 2020 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

I'll try and file an issue for the atc problem soon.

Please update to the latest commit 97d57678f6d2dd48ecb893c4751ad7cc89347406 where -fcommon was enabled for arm, and report back if it fixes the linking issues you have experienced.

Please update to the latest commit https://github.com/chettrick/discobsd/commit/97d57678f6d2dd48ecb893c4751ad7cc89347406 where -fcommon was enabled for arm, and report back if it fixes the linking issues you have experienced.

I can confirm, that adventure builds and some other binaries build that previously didn't
My build does have some errors:
Created filesystem at partition 1 of distrib/stm32/sdcard.img - 102400 kbytes
/home/somebody/discobsd/distrib/obj/destdir.stm32/usr/lib/crt0.o: No such file or directory
/home/somebody/discobsd/distrib/obj/destdir.stm32/usr/lib/libc.a: No such file or directory
/home/somebody/discobsd/distrib/obj/destdir.stm32/usr/lib/libm.a: No such file or directory
/home/somebody/discobsd/distrib/obj/destdir.stm32/usr/lib/libcurses.a: No such file or directory
/home/somebody/discobsd/distrib/obj/destdir.stm32/usr/lib/libgpanel.a: No such file or directory
/home/somebody/discobsd/distrib/obj/destdir.stm32/usr/lib/libreadline.a: No such file or directory
/home/somebody/discobsd/distrib/obj/destdir.stm32/usr/lib/libtermlib.a: No such file or directory
/home/somebody/discobsd/distrib/obj/destdir.stm32/usr/lib/libwiznet.a: No such file or directory

but I believe this is an unrelated error
The commit message https://github.com/chettrick/discobsd/commit/97d57678f6d2dd48ecb893c4751ad7cc89347406 has the wording

Enable -fcommon for arm/stm32 until base is -fno-common clean.

which seems to imply that -fno-common cleanness is something we're aiming for; have I understood this correctly?

My build does have some errors:

Created filesystem at partition 1 of distrib/stm32/sdcard.img - 102400 kbytes

/home/somebody/discobsd/distrib/obj/destdir.stm32/usr/lib/crt0.o: No such file or directory

/home/somebody/discobsd/distrib/obj/destdir.stm32/usr/lib/libc.a: No such file or directory

/home/somebody/discobsd/distrib/obj/destdir.stm32/usr/lib/libm.a: No such file or directory

/home/somebody/discobsd/distrib/obj/destdir.stm32/usr/lib/libcurses.a: No such file or directory

/home/somebody/discobsd/distrib/obj/destdir.stm32/usr/lib/libgpanel.a: No such file or directory

/home/somebody/discobsd/distrib/obj/destdir.stm32/usr/lib/libreadline.a: No such file or directory

/home/somebody/discobsd/distrib/obj/destdir.stm32/usr/lib/libtermlib.a: No such file or directory

/home/somebody/discobsd/distrib/obj/destdir.stm32/usr/lib/libwiznet.a: No such file or directory

but I believe this is an unrelated error

Yes, this error is most likely due to an unresolved issue with FreeBSD's version of BSD make. Try with GNU make and see if your build completes without any errors.
Note also that the built in libraries (/usr/lib/lib*.a) aren't actually usable in DiscoBSD/stm32, as I haven't yet ported an assembler that handles Thumb-2. Those libraries work with DiscoBSD/pic32 though (same as RetroBSD for pic32) and can be used with /tools/virtualmips to compile and run the example code in /usr/share/examples.

The commit message https://github.com/chettrick/discobsd/commit/97d57678f6d2dd48ecb893c4751ad7cc89347406 has the wording

Enable -fcommon for arm/stm32 until base is -fno-common clean.

which seems to imply that -fno-common cleanness is something we're aiming for; have I understood this correctly?

Yes, eventually I would like to set -fno-common as the default. Most base utilities will need to be either fixed or updated to a more modern version from one of the BSDs.
I will merge this pull request, as that makes one less program to update.

I will merge this pull request, as that makes one less program to update.

Brilliant!
Shall I carry on with atc (separate pull request) and whatever else I find, or?


Convert selected chat conversation state into routeTree route

Working towards getting attachment popups as routes, I needed to be able to express the chat you're on as a route.
Chat routes look like this: /tab:chatTab/<conversationId>
It's important to create a route for each conversation. If we store anything in routeState, we'll want separate state for each tab of the conversation list. Thus, routeTree needed to be extended so that the children of a route can be a function. Tests included!
The rest of this is just moving state around. There's now a nothingSelected sentinel value to represent no tab selected yet. Deriving the selected conversation out of the route state is a little terse, but I'm not sure what this code is doing is worth generalizing into a helper.
:eyeglasses: @keybase/react-hackers

:eyeglasses: @keybase/react-hackers bump


Update main.css

Prettied up the formatting.
Simplified the wording of the comments (must they be so complex?).
Simplified the Spin animation.

CJSKA1 is no longer accepting PRs, as the new version is to be released sometime this week.


Statagem timer re-add removes old timer

If you re-add a timer, the old timer should be removed.  This allows "updating" a timer via running add a second time.
Fixes #1099
Signed-off-by: Nathaniel Clark<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1108   +/-   ##
=======================================
  Coverage   72.27%   72.27%
=======================================
  Files           3        3
  Lines         202      202
=======================================
  Hits          146      146
  Misses         56       56

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 cfe0cd4...d13e4df. Read the comment docs.

This approach is basically duplicating the logic in UnconfigureStratagemTimerStep.

It's actually lighter weight than a full UnconfigureStratagemTimerStep because it doesn't do a daemon-reload

Instead, why don't you conditionally add UnconfigureStratagemTimerStep if the timer already exists here:
https://github.com/whamcloud/integrated-manager-for-lustre/blob/d13e4df2260cb4ae45631760ba4d81568b02ee58/chroma_core/models/stratagem.py#L153-L156
That will be more explicit what is happening in the cmd output and will reuse the existing logic.

It seems like an improper separation of layers to check the file existance in the Job, vs in the Step

It's actually lighter weight than a full UnconfigureStratagemTimerStep because it doesn't do a daemon-reload

I don't think there's much cost to running systemctl daemon-reload, is there?

It seems like an improper separation of layers to check the file existence in the Job, vs in the Step

It's certainly a strange way to do it, but it's how It's done within IML. The advantage of doing it this way is it's more explicit what steps were taken when reviewing later on from a user perspective.
In other words, but using an explicit step the user will see that there was an unconfigure step taken in the GUI before configuring again.

This is what shows in the command status output for subsequent runs of configure add.


Authorization postback fails on PostgreSQL backend

When using PostgreSQL as database backend, authorization fails because maximum username max_length is set to 30 characters in User model, and uuid4() length is 32.

Oops, my bad, I'm never running tests in sqlite. =)
I'll cut tail of uuid4 hash; I hope it does not violate unique index for username field.


Renaming tabs doesn't work on Linux

This issue pertains to the following package(s):

[x] GraphQL Playground - Electron App

haven't tried the rest
What OS and OS version are you experiencing the issue(s) on?
Linux
What version of graphql-playground(-electron/-middleware) are you experiencing the issue(s) on?
I downloaded the amd64 asset from here: https://github.com/graphcool/graphql-playground/releases/tag/v1.5.0-rc.5/.
What is the expected behavior?
Double clicking the tab name, then typing something works
What is the actual behavior?
Double clicking the tab name works, the name becomes empty and the cursor becomes invisible. Typing then results in nothing.

Fixed in latest version.


Fix minor formatting nits

Change numbering of UC
Fix formatting for MSS

LGTM. Thanks for making the changes in formatting!


add NOTR

A followup to https://github.com/electron/electron.atom.io/pull/593

@zeke Thank you

Anything else required from my end ?

Nope, all set. This will land on the website once we ship https://github.com/electron/electron.atom.io/pull/616


support code block name with whitespace

defining code blocks (file)names with ```julia:name
breaks syntax highlighting, allowing whitespace like ```julia :name
would fix it.

Hello!
What editor do you use? On Atom the highlighting seems to work fine.
But yes what you suggest is possible, the main change would be to add a single space in the regex that captures these things: https://github.com/tlienart/Franklin.jl/blob/d9f84d45488b4ecbbbdea84ff3d29e4275071d69/src/regexes.jl#L67-L74. So I guess that would make the first line
"```([a-zA-Z][a-zA-Z-]*) ?"

we can test this to see if it breaks anything and, assuming it doesn't, I'm happy to bring this in

Hello @pdubya, not in the current release. It might land in the next one though


Incompatible with newer OpenSSL versions

Knife solo requires net-ssh < 4.0, but that doesn't seem to work with newer openssl versions. Consider updating net-ssh dependency.
Version:
net-ssh-3.2.0
OpenSSL 1.1.0f
Chef: 13.3.42
knife-solo (0.6.0)
Output: knife solo prepare -VV <user>@<host>
Bootstrapping Chef...
/home/<user>/.gem/ruby/2.4.0/gems/net-ssh-3.2.0/lib/net/ssh/transport/cipher_factory.rb:97: warning: constant OpenSSL::Cipher::Cipher is deprecated
/home/<user>/.gem/ruby/2.4.0/gems/net-ssh-3.2.0/lib/net/ssh/transport/kex/diffie_hellman_group1_sha1.rb:118:in `generate_key': undefined method `p=' for #<OpenSSL::PKey::DH:0x000000033ebcc0> (NoMethodError)
Did you mean?  p
	from /home/<user>/.gem/ruby/2.4.0/gems/net-ssh-3.2.0/lib/net/ssh/transport/kex/diffie_hellman_group1_sha1.rb:51:in `initialize'
	from /home/<user>/.gem/ruby/2.4.0/gems/net-ssh-3.2.0/lib/net/ssh/transport/algorithms.rb:362:in `new'
	from /home/<user>/.gem/ruby/2.4.0/gems/net-ssh-3.2.0/lib/net/ssh/transport/algorithms.rb:362:in `exchange_keys'
	from /home/<user>/.gem/ruby/2.4.0/gems/net-ssh-3.2.0/lib/net/ssh/transport/algorithms.rb:206:in `proceed!'
	from /home/<user>/.gem/ruby/2.4.0/gems/net-ssh-3.2.0/lib/net/ssh/transport/algorithms.rb:154:in `accept_kexinit'
	from /home/<user>/.gem/ruby/2.4.0/gems/net-ssh-3.2.0/lib/net/ssh/transport/session.rb:210:in `block in poll_message'
	from /home/<user>/.gem/ruby/2.4.0/gems/net-ssh-3.2.0/lib/net/ssh/transport/session.rb:188:in `loop'
	from /home/<user>/.gem/ruby/2.4.0/gems/net-ssh-3.2.0/lib/net/ssh/transport/session.rb:188:in `poll_message'
	from /home/<user>/.gem/ruby/2.4.0/gems/net-ssh-3.2.0/lib/net/ssh/transport/session.rb:225:in `block in wait'
	from /home/<user>/.gem/ruby/2.4.0/gems/net-ssh-3.2.0/lib/net/ssh/transport/session.rb:223:in `loop'
	from /home/<user>/.gem/ruby/2.4.0/gems/net-ssh-3.2.0/lib/net/ssh/transport/session.rb:223:in `wait'
	from /home/<user>/.gem/ruby/2.4.0/gems/net-ssh-3.2.0/lib/net/ssh/transport/session.rb:88:in `initialize'
	from /home/<user>/.gem/ruby/2.4.0/gems/net-ssh-3.2.0/lib/net/ssh.rb:232:in `new'
	from /home/<user>/.gem/ruby/2.4.0/gems/net-ssh-3.2.0/lib/net/ssh.rb:232:in `start'
	from /home/<user>/.gem/ruby/2.4.0/gems/knife-solo-0.6.0/lib/knife-solo/ssh_connection.rb:42:in `session'
	from /home/<user>/.gem/ruby/2.4.0/gems/knife-solo-0.6.0/lib/knife-solo/ssh_command.rb:153:in `try_connection'
	from /home/<user>/.gem/ruby/2.4.0/gems/knife-solo-0.6.0/lib/knife-solo/ssh_command.rb:194:in `detect_authentication_method'
	from /home/<user>/.gem/ruby/2.4.0/gems/knife-solo-0.6.0/lib/knife-solo/ssh_command.rb:290:in `run_command'
	from /home/<user>/.gem/ruby/2.4.0/gems/knife-solo-0.6.0/lib/chef/knife/solo_prepare.rb:71:in `operating_system'
	from /home/<user>/.gem/ruby/2.4.0/gems/knife-solo-0.6.0/lib/chef/knife/solo_prepare.rb:67:in `bootstrap'
	from /home/<user>/.gem/ruby/2.4.0/gems/knife-solo-0.6.0/lib/chef/knife/solo_prepare.rb:57:in `run'
	from /home/<user>/.gem/ruby/2.4.0/gems/chef-13.3.42/lib/chef/knife.rb:442:in `block in run_with_pretty_exceptions'
	from /home/<user>/.gem/ruby/2.4.0/gems/chef-13.3.42/lib/chef/local_mode.rb:44:in `with_server_connectivity'
	from /home/autarch/.gem/ruby/2.4.0/gems/chef-13.3.42/lib/chef/knife.rb:441:in `run_with_pretty_exceptions'
	from /home/<user>/.gem/ruby/2.4.0/gems/chef-13.3.42/lib/chef/knife.rb:219:in `run'
	from /home/<user>/.gem/ruby/2.4.0/gems/chef-13.3.42/lib/chef/application/knife.rb:156:in `run'
	from /home/<user>/.gem/ruby/2.4.0/gems/chef-13.3.42/bin/knife:25:in `<top (required)>'
	from /usr/bin/knife:22:in `load'
	from /usr/bin/knife:22:in `<main>'

https://rubygems.org/gems/knife-solo/versions/0.7.0.pre should fix this.


usage/

Is this not enough?
I will update the description if that is what you meant.
From the code:
fprintf(stderr, W "\nUsage: takedown requests targets_file victim\n" N);
fprintf(stderr, "\nEx: takedown 10000 wp_pingback www.victim.com\n\n");

@d4rkerNight the usage from this script file, before terminal linux.
this script file, can takedown the website that contains, php.

"website that contains, php" What? Did you understand the code?
Anyway, thank you for bringing this issue to my attention, I guess not many people can read simple C code. When I have time I will update the README.md.

you can make the start menu with bash.sh script .
like .
takeout.sh domain.com
loading.php
sending nodes.....
domain.com down..........
i will keep this silently.
god keeps you one day from one princess, :)
bash script menu. number one.
chmod a+x *
bash takeout.sh
domain: domain.com
etc.etc
domain.com is down.
i will keep this silently. from eyes from the others. @d4rkerNight

apt install make, cmake, clang, gcc, g++
gcc *
g++ *
make
make all
in the bash script
chmod a+x file.php

"you can make the start menu with bash.sh script" lol, why should I use bash when this is C language?!? Please stop!! I can stand anymore to noobs!!


Workspace.getActiveEditor is deprecated.

Posting the deprecation that popped up after I updated to v2.15.10.
Call ::getActiveTextEditor instead
Workspace.getActiveEditor (C:\Users\Chad\AppData\Local\atom\app-0.192.0\resources\app\src\workspace.js:999:12)
<unknown> (C:\Users\Chad\.atom\packages\atom-typescript\dist\main\atomts.js:27:24)

Thanks! Fixed locally. I plan to push by end of day (big refactoring going around generating .d.ts files) :rose:

Np, just wanted to make sure you were aware. Love the plugin, thanks for all the hard work!


Deploy ignite vm using Pulumi

closes #74

[x] Fix random failure of hypervisor-test. Example failing job: https://github.com/mvgijssel/setup/runs/6056430515?check_suite_focus=true
[x] Install Pulumi using rules_python
[ ] Run Ignite VM using Pulumi

[x] SSH connectivity to vagrant machine in dev using https://github.com/pycontribs/python-vagrant
[ ] Use Pulumi CopyFile to copy a declarative ignite vm
[ ] Use https://github.com/innobead/kubefire to deploy a kubernetes cluster using Ignite
[ ] Ensure the kubernetes cluster starts after reboot
[ ] Ensure the vms can be reached from the hypervisor using SSH

[ ] Test running Pulumi against hypervisor machine
[ ] Create custom Bazel rule or macro for Pulumi

Currently the setup copies a yaml file describing an Ignite vm to /etc/firecracker/manifests/. Here ignited picks up the file and launched the vm. After launching ignited updates the .yaml template to include things like network, runtime, kernel, etc.
If we currently make a change to the template and push the changes with Pulumi, ignited replaces the internal state with the new changes in the file, but because the template does not contain any information about network, kernel, etc this information is lost and the vm becomes unusable. Reading the documentation about gitops https://github.com/weaveworks/ignite/blob/e76367e56ad3b84a145926ddb1d0996bcfafbe1b/docs/gitops.md this is expected behavior and gitops normally pushes back the updated state into the repository.
Possible paths to move forward:

Commit the entire template as generated by ignited into git manually and push this using Pulumi
Let Pulumi merge new changes instead of replacing the existing yaml file

Means removing remote.CopyFile and implementing remote.Command or a (dynamic) provider

Don't use ignited but spawn vms directly using ignite providing a config

Is it possible to do updates this way?
How would we remove vms?

Use ignited gitops instead of Pulumi

There also seems to be an issue with ignited removing the vm template (under some circumstances) https://github.com/weaveworks/ignite/issues/668 so might need to move forward without ignited.

https://github.com/innobead/kubefire looks like a light and easy way to verify if it's possible to run kubernetes on firecracker within a vm in the GitHub actions CI.

Investigating the logs of https://github.com/mvgijssel/setup/runs/6206675627?check_suite_focus=true it shows that the Ignite vm after 5 minutes booted up to "target Basic System".
I0428 07:45:16.086886   11645 eventsink.go:59] [  OK  ] Started Discard unused blocks once a week.
I0428 07:45:16.087124   11645 eventsink.go:59] [  OK  ] Started Message of the Day.
I0428 07:45:16.087391   11645 eventsink.go:59] [  OK  ] Reached target Timers.
I0428 07:45:16.087658   11645 eventsink.go:59] [  OK  ] Listening on D-Bus System Message Bus Socket.
I0428 07:45:16.087975   11645 eventsink.go:59] [  OK  ] Reached target Sockets.
I0428 07:45:16.088290   11645 eventsink.go:59] [  OK  ] Reached target Basic System.

Doing the same test locally inside the hypervisor VM it reaches basic system within ~10 seconds.
Reason for the enormous difference in performance is likely to be explained by the three levels of virtualization:

GitHub actions runner is a vm
Hypervisor virtualbox vm in in runner vm
Ignite vm in hypervisor vm

This basically means it's not possible to test the vms running within the hypervisor in the CI.

A potential pivot that needs validation:
Use Nomad as the primary way to run virtual machines within the hypervisor machine. Because Nomad also works on macOS, we can pretend the macOS in the CI is actually the hypervisor basically meaning we can remove the second layer of virtualization to test the provisioning of the infrastructure.
Because Firecracker does not work on macOS (requires KVM (https://github.com/firecracker-microvm/firecracker#overview)), use qemu with the hvf accelerator.

Validated that Nomad will solve the following:

Can act as a vm manager, can use it to schedule qemu machines
Runs in the CI, because works on macOS, Linux and Windows, removing the second layer from https://github.com/mvgijssel/setup/pull/85#issuecomment-1111885973
There is Nomad plugin for Pulumi https://www.pulumi.com/registry/packages/nomad/ making infrastructure as code easy.

Therefore closing the Ignite Hypervisor project and this issue and starting Nomad Architecture.


fix: change dts codegen style to avoid eslint-checked-error

If you are using unplugin-auto-import, it should be fixed in it. I think it's better to keep unimport simple here. Thanks.


Need Capability to specify OPENAI Model Version, temperature and output token length #1

While self hosting this app, we need capability to specify some of the key parameters to make it usable. Currently it only generates 2 sentences stops. SInce while hosting on our own the cost is bourn by the user it make perfect sense to allow the user to use this beautiful app to its true capability.

You can already do that in the API endpoint: https://github.com/steven-tey/novel/blob/main/app/api/generate/route.ts

Closing this for now but lmk if I misunderstood you and we can revisit this issue!


Change header colors based on selected map

This fixes #81 by changing the gradient in the header of the composition form any time you change the map that's selected.

It looks awesome because of your color choices! I'm gonna think of this like when @RobThePM plays a healer and says his heals are on point. 😆

They are indeed, on point! 😄 👈 ☝ 👉 👇


Call redraw_for_cursorline(curwin) in update_topline_redraw()

The redraw_later() call above is no longer needed since patch 8.1.0374.
Fix #9970

I had not looked at this PR when I made my fix.  Would it still be useful to include the test from this PR?  I would think the sign placement and unplace isn't relevant for drawing the cursorline.

Maybe the test is not useful. However, I think the call to redraw_later() in op_yank() call still be removed, as starting from patch 8.1.0374 SOME_VALID isn't required to redraw relativenumber.

I have tests removing that and it doesn't break any checks: https://github.com/zeertzjq/vim/commit/6af0bea2418f0e996b6a04bd1f9075eab1aad69a (https://github.com/zeertzjq/vim/actions/runs/2006794696)


Remove public PhasedBuildActionExecutor type

The type behaves just like a regular build action
except that it can hook into different phases of
the build.
FYI @lsmaira

Cool, thank you for the info :)


Alias

Do not merge. Pushing for @joycelynng  to continue

Coverage decreased (-1.9%) to 77.952% when pulling 02a46bc98e5caff04b0a49fabc71c7d546e52065 on alias into 168e6dfce2c996d33a6620dbbf4d27ddae9e27d3 on master.

Has already been merged into add-alias-command-and-parsing branch.


Bug report [[#]] cannot be used in where close anymore

What happened?
In the previus version i wase able to use [[#]] to filter in a where clause. Those queries return empty now
DQL
table WITHOUT ID
  file.link as Name,
  Role,
  Email,
  Phone

from #contacts
where Company = [[#]]
sort file.name desc

JS
No response
Dataview Version
0.4.21
Obsidian Version
0.12.19
OS
MacOS

For an immediate work around, you can use this.file.link instead of [[#]] in WHERE statements. I'm not sure what changed to cause this specific behavior, though.


iOS Tracker: Getting errors in pod source files after installing 5.0.0

Describe the bug
Hi
When I update the pod version to 5.0.0 and try to run the app I am getting some errors in ConfigurationFetcher and SQLiteEventStore  classes for optional binding .
To Reproduce
Install the 5.0.0 version using cocoa pods and just run the application.
Expected behavior
App should run without any errors in the pod source files.

Xcode version: 14.1

Hi @Utk2911, thanks for the report! Could you please tell us which version of Swift are you using?

I am using swift version 4.2

I'm pretty sure that is the problem, the tracker requires Swift version 5. Could you please try upgrading the Swift version in your project?

Thanks for the help.

No problem! Closing this task, please reopen in case the problem persists.


Not reloading after save

This looks like a really cool feature but I can't seem to get it to trigger no matter what. I've tried the other starter kits as well but no luck there either. For any changes to take effect I need to run npm start again each time.
Arch Linux 64 bit
node v0.10.36
npm 2.3.0

Browser console in Chromium 40.0.2214.93 (64-bit)
[HMR] Waiting for update signal from WDS...
[WDS] Hot Module Replacement enabled.

Output from running npm start.
> node server.js
Listening at localhost:3000
Hash: 42936a5ff1b93df92458
Version: webpack 1.5.3
Time: 922ms
    Asset    Size  Chunks             Chunk Names
bundle.js  780270       0  [emitted]  main
chunk    {0} bundle.js (main) 680118 [rendered]

* MY EDIT BELOW
[0-165] built...etc

webpack: bundle is now VALID.

I change the text in the <h1> in scripts/app.js but nothing happens in the browser. No more output or any error messages.
Please advice on how I can debug this.

Here's what I see in Network tab:
Before saving

After saving

Do you see any of these?

I get the first ones but nothing after saving. It will recognize the server going down though when I ctrl+c it. So it has some kind of connection back to the server. It just doesn't catch any updates.

Please file this in Webpack repo. This has to be some kind of connection problem but I don't know how to debug it.

Will do, thanks!

let us know if you find a solution! I'm collecting those in https://github.com/gaearon/react-hot-loader/blob/master/docs/Troubleshooting.md


TOKEN

I want to report the error that in the part of config.json there is no tocken option also if you try to put it in the file I do not recognize it

You actually have to create a process.env file and set your token there (TOKEN=Your token here).

You actually have to create a process.env file and set your token there (TOKEN=Your token here).

SyntaxError: /home/runner/discord-modmail/config.json: Unexpected token } in JSON at position 157
at parse ()
at Object.Module._extensions..json (internal/modules/cjs/loader.js:1171:22)
at Module.load (internal/modules/cjs/loader.js:985:32)
at Function.Module._load (internal/modules/cjs/loader.js:878:14)
at Module.require (internal/modules/cjs/loader.js:1025:19)
at require (internal/modules/cjs/helpers.js:72:18)
at Object. (/home/runner/discord-modmail/bot.js:3:16)
at Module._compile (internal/modules/cjs/loader.js:1137:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1157:10)
at Module.load (internal/modules/cjs/loader.js:985:32)

SyntaxError: /home/runner/discord-modmail/config.json: Unexpected token } in JSON at position 157
at parse ()
at Object.Module._extensions..json (internal/modules/cjs/loader.js:1171:22)
at Module.load (internal/modules/cjs/loader.js:985:32)
at Function.Module._load (internal/modules/cjs/loader.js:878:14)
at Module.require (internal/modules/cjs/loader.js:1025:19)
at require (internal/modules/cjs/helpers.js:72:18)
at Object. (/home/runner/discord-modmail/bot.js:3:16)
at Module._compile (internal/modules/cjs/loader.js:1137:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1157:10)
at Module.load (internal/modules/cjs/loader.js:985:32)

There was an error in the json config file. I fixed that in the above commit, could you retry?

There was an error in the json config file. I fixed that in the above commit, could you retry?

Looks like a comma was missing, can you retry?

Looks like a comma was missing, can you retry?


OCM-5092 | fix: remove empty line between mode descriptions in -h output

https://issues.redhat.com/browse/OCM-5092

/lgtm

/lgtm


Android: Multiple definition of JNI_Onload

I'm already using a JNI_OnLoad in my app, so since naett defines one, I get a multiple definition error.
So what's needed is some way to pass your jni environment into naett, instead of having it try to grab it itself. Scratch that, I see that naettPlatformInit can already do it. So I only need to get rid of the extra JNI_OnLoad somehow.
Can we have a define maybe? #ifdef USE_EXISTING_JNI_ONLOAD for example.

Hi. That's a remainder from an early version, thanks for finding that.
You are supposed to pass the VM handle in the call to naettInit, the JNI_OnLoad is not needed.

Thanks for getting rid of it!
Might also be good to address #12, although I can't vouch for it as these JNI references confuse me sometimes as well..


Incorrect total block value in publish block log

Describe the bug
The totalBlockValue printed out when publish is wrong
info: Published block slot=7414120, validator=0x8955…5c2c, graffiti=Lodestar-v1.14.0/d7ca290, executionPayloadSource=engine, executionPayloadBlinded=false, executionPayloadValue=0.01908 ETH, consensusBlockValue=0.02732 ETH, totalBlockValue=27329242.01908 ETH, strictFeeRecipientCheck=false, builderSelection=executiononly, api=produceBlockV3
info: Published block slot=7414416, validator=0x86c0…b87c, graffiti=Lodestar-v1.14.0/d7ca290, executionPayloadSource=engine, executionPayloadBlinded=false, executionPayloadValue=0.00628 ETH, consensusBlockValue=0.02464 ETH, totalBlockValue=24646563.00628 ETH, strictFeeRecipientCheck=false, builderSelection=executiononly, api=produceBlockV3

The issue is likely related to https://github.com/ChainSafe/lodestar/pull/6286 as it looks like we missed to remove gweiToWei conversion.
https://github.com/ChainSafe/lodestar/blob/d7ca290d173dfa82b2977f3e94df6e5e8d12ef3e/packages/validator/src/services/block.ts#L225-L226
But it needs to be confirmed that this fixes the issue and if there are other places that need to be updated.
Expected behavior
Should print out correct total block value
Steps to reproduce
Produce a block with produceBlockV3 api
Additional context

https://github.com/ChainSafe/lodestar/pull/6286

Operating system
Linux
Lodestar version or commit hash
d7ca290d173dfa82b2977f3e94df6e5e8d12ef3e

Might also be worth to confirm that consensusBlockValue is  correct, I am seeing a lot lower values on https://goerli.beaconcha.in/ but possibly related to the current network conditions and non-finality.

But it needs to be confirmed that this fixes the issue and if there are other places that need to be updated.

My bad. Took a second look this seems to be the only place I missed. Will do a more thorough testing to confirm.

Might also be worth to confirm that consensusBlockValue is correct, I am seeing a lot lower values on https://goerli.beaconcha.in/ but possibly related to the current network conditions and non-finality.

Not too familiar with the beaconcha.in UI, can you point me where the block proposer reward is? The Tx Reward only counts the payload value.
With the two blocks you mentioned, I looked at the consensus income of their proposers.
For slot 7414120, the proposer has a consensus income of 0.02896 ETH on that day, minus other duties ~0.002ETH so its reward for proposing that block should be around 0.02796ETH which is close to what we have.
For slot 7414416, the proposer has a consensus income of 0.02628 ETH on that day, minus other duties ~0.002ETH so its reward for proposing that block should be around 0.02428ETH which is close to what we have.

can you point me where the block proposer reward is?

I don't think they have a per block consensus reward, I also just looked at the Daily Income panel which shows a different, more correct value now. From their api docs I also don't see a way to get the rewards.
I guess the best way to confirm this is to compare the rewards APIs of clients and see if results are the same. But based on current data I don't see an issue with the calculation we have.


[REQUEST] Store data in ApplicationData directory

Is your feature request related to a problem? Please describe.
Whenever osuplayer is run, the data folder is created in the working directory. This is frustrating because it makes moving to new versions harder.
Describe the solution you'd like
The solution to this problem would be to store the data folder in the AppData/Roaming folder on Windows or the ~/.local/share/ directory on Linux.
Additional context
My C# is a bit rusty, but you could use the code:
Path.Join(Environment.GetFolderPath(Environment.SpecialFolder.ApplicationData), "osuplayer");

to get ~/.local/share/osuplayer on Linux and AppData/Roaming/osuplayer on Windows.

Maybe make it a toggle?

Maybe make it a toggle?

I don't want to add another migration for that, because we have to migrate all current files to the other location, this would be a one time thing, but I don't see this necessary, as it never was in those 7 years. You were always supposed to overwrite your existing installation, otherwise you would end with X folders with different versions.
Making it toggle-able would require more work and makes it even more confusing inside the code base.
TLDR: Don't be lazy, drag & drop and hit replace files

I'll close this for now, as it isn't planned.
However if more people think this should be useful and don't want the simplicity of removing all fines at once instead of splitting the files across the system, then we could open this issue again

I am coming back to this issue because it makes the app near impossible to package because the app is installed in a privileged directory, and cannot create the files it needs:
Unhandled exception. System.UnauthorizedAccessException: Access to the path '/usr/lib/osuplayer-2/logs' is denied.
 ---> System.IO.IOException: Permission denied
   --- End of inner exception stack trace ---
   at System.IO.FileSystem.CreateDirectory(String fullPath, UnixFileMode unixCreateMode)
   at System.IO.Directory.CreateDirectory(String path)
   at OsuPlayer.Extensions.UnhandledExceptionHandler.CreatLogsDirectoryIfItsMissing() in /home/johne/osuplayer/src/osuplayer-2024.802.0/OsuPlayer.Extensions/UnhandledExceptionHandler.cs:line 15
   at OsuPlayer.Extensions.UnhandledExceptionHandler.HandleException(Exception ex) in /home/johne/osuplayer/src/osuplayer-2024.802.0/OsuPlayer.Extensions/UnhandledExceptionHandler.cs:line 24
   at OsuPlayer.Program.Main(String[] args) in /home/johne/osuplayer/src/osuplayer-2024.802.0/OsuPlayer/Program.cs:line 46

Lemme make this clear: You installing the player somewhere, where it does not have the permissions to create data in its own folder? So I assume that the osu!player has the same permissions as the privileged directory in that case not enough.
I don't know why you try to install the osu!player  into lib as you shouldn't install software in your lib folder. As this folder is reserved for system files and managed by package managers.
Either install it in home
Or in /usr/local/bin or /usr/local/lib however /usr/local/bin makes more sense as well.
Using Environment.SpecialFolder.ApplicationData on linux will return /home/,username/.configSo in the end if you install it in /home anyways we don't have to make a huge refactor of the codebase and think about already existing configs, to provide a smooth transition. Please consider that, instead of installing it somewhere where it doesn't belong

Lemme make this clear: You installing the player somewhere, where it does not have the permissions to create data in its own folder?

This is often how software is packaged on archlinux's AUR. Look at osu-lazer's pkgbuild, which installs it to a directory where it does not have the ability to create files. This allows the program to make one config for per user.

Please keep in mind, that osu-player isn't a package. This is just a zip you extract at a place where it is convenient for you. if opt/osuplayer is convenient for you install it there. Yes we could have used the Environment.SpecialFolder.ApplicationData in the first place, but we did not (). I just want to avoid a big refactor of the codebase. Because this is easy fixable from your end, as no other linux had issues with that so far.


[FLINK-29405] Fix unstable test InputFormatCacheLoaderTest

What is the purpose of the change
Fix unstable test InputFormatCacheLoaderTest.
The problem was caused by race condition when we interrupt Full cache reload thread. There is the following logic: when main reload thread is being interrupted, it also interrupts underlying threads, that read from InputSplits (InputSplitCacheLoadTask). Previously, main thread didn't wait for the end of underlying threads, so test sometimes failed, if test thread progressed faster than InputSplitCacheLoadTask threads.
Note, that previously if we put, for instance, @RepeatedTest(1000) on this test, it would be failed almost every time. Now, with  @RepeatedTest it never fails (but I think, it's unnecessary to have this annotation on continuing basis).
Brief change log

Fixed unstable tests
Fixed InputFormatCacheLoader, so now after exception it waits until InputSplitCacheLoadTasks finish

Verifying this change
This change is already covered by existing tests
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

cc @PatrickRen

@zentol Any thoughts?

Waiting forever in production code is super sketchy and should virtually never be done.
The PR is also lacking a sort of problem analysis and explanation for how this fixes the issue.

Hi @zentol, I added timeout on wait after interrupt and updated PR description to explain the problem and proposed solution more precisely. Can you check the latest commit, please?

Hi @XComp, thanks for the review! I answered on your comments and added a commit with fix. Have a look when you will be available, please.

Hi @XComp @zentol, thanks for your careful review! I took into account all your comments and created a commit with following changes:

Now CacheLoader returns future directly in method CacheLoader#reloadAsync(). As a consequence CacheLoader has direct control over the thread which executes cache reload operation.
I no more rely on common ForkJoinPool. CacheLoader creates Executors.newSingleThreadExecutor() in #open method that is responsible for the whole reload operation.
Now CacheLoader can directly shutdown its executor in #close, so I decided to remove close mechanism based on boolean variable checking and keep only the one based on thread interruption. Therefore logic became more clear and straightforward. So now there is only one test on CacheLoader#close
Now InputFormatCacheLoader executes one load task in main reload thread (in Cacheloader#reloadExecutor), so we are saving on creating additional thread for reading from input split.
Improved logging on CacheLoader#close a little bit

Please take a look on the latest commit when you will be available, looking forward for your review!

After some offline chat with @zentol about it, I came to the conclusion that it's really not the right location for such a big change. We should focus on stabilizing the test here. I moved my proposal into FLINK-30354. I will do another review of this PR on Monday.

After some offline chat with @zentol about it, I came to the conclusion that it's really not the right location for such a big change. We should focus on stabilizing the test here. I moved my proposal into FLINK-30354. I will do another review of this PR on Monday.

Yea, I think this MR is already a little bit blowed up, so such big changes should be in separate ticket. About your proposal: I like the idea of reusing the same thread pool, and especially, clarifying the logic of CacheLoader, so it won't deal with interrupts. But I'm not sure about changing CacheReloadTriggers and CacheReloadTrigger.Context, because it touches already public API (although added relatively recently). BTW during the FLIP-221  development I discussed the design with @PatrickRen and @wuchong. So I think it would be good if they also take a look at your proposal in FLINK-30354

Hi @XComp, I'm sorry I've been gone so long. I had a vacation to pass university exams and then got ill 😬. I answered all your comments and made a fix in recent commit (force-pushed only because of rebase). Have a look please, when you will have time.

@XComp @PatrickRen At this time I fixed all comments that was by this MR. If there will be no more remarks, existing commits can be squashed into one and merged to master. After that I will create the same MR, but if Flink 1.16.

Thanks @SmirAlex. May you squash and rebase the PR. This way, we could already get it final hoping that @PatrickRen finds time to have a pass till tomorrow. If not, I would still go ahead and merge the PR tomorrow before feature freeze to make this test instability go away at least. Based on how much you trust your PR, you could also create a 1.16 backport already to have CI run over night. ...up to you (as it might double your amount of work if @PatrickRen still has change requests).

@XComp Big thanks for the review, your comments were very helpful! @PatrickRen Thanks for merging a MR! And sorry for not responding, I was on a vacation without laptop last week, so, unfortunately, couldn't merge it by myself.


Fix paths

fix worker chown path and refactor
by default chown fuse dir

@Kai-Zhang A small fix. PTAL. Thanks!


Add undocumented avoid_static_ip flag

Hi,
As promised my pull request to close #536.
Best,
Chris

Well, the question is, why would anyone do that? You would have to change the IP in your config file everytime, instead of simply connecting to your server. If you have a public ip from your ISP, you can still connect via dynamic DNS services.

@Zeik0s Well, I think my use case is not very common, but maybe some others need it for similar things, so I thought an undocumented flag would be a good solution for this.
My use case:
I don't care about the internal network the Pi is connected to. I only care about the external (American) IP the internal network is routed through. Therefore I would like the Pi to be flexible enough to react to changing local networks/IP address ranges. The clients of the VPN can find the VPN server via a reverse SSH tunnel also running on the Pi. And this IP is the actual static IP for the clients.
Best,
Chris

I don't think this is within the scope of PiVPN: this is for advanced users/a really specific usecase. If users want to uses this, they should just edit the files manually.


Making ReadUidMultiReader.ino works

OS version: Linux
Arduino IDE version: 1.8.13
MFRC522 Library version: 1.4.7
Arduino device: Uno
MFRC522 device: RFID-RC522 from AZ

Hello !
I was trying to make the example ReadUidMultiReader.ino  works with fiability.
It seems that the problem come from
#define SS_1_PIN        10
because it is used as SS for Arduino Uno to known that it is a slave SPI (https://www.arduino.cc/en/reference/SPI)
So I used another pin and just add in the setup:
pinMode(10, OUTPUT);
and it seems to work.
I am not sure if this is correct solution. Can someone can test it and corroborate my observation please ?
Best regards,

there is already a huge topic #290


Datatypes in Python

You have to create a new separate python file (concept-name_yourusername.py) inside the Python3-Learn folder. don't add your code inside Example_Amansharma.py this is for demo okay make these change

sorry, i am new in github.


[FEAT]: Test the ARNEIS customized NN with depthai_demo.py

Description
As a proof of the completion of #138 an independent tester should integrate the customized NN and run it on depthai_demo.py.
Additional Information
Expected deliverables (completion criteria for this task):

A document under /docs/howto which will explain in details the steps needed to perform this task
Some test results produced by running depthai_demo and have it recognized some mignon bottles taken from the available set.

@alv67 based on today conversation would you agree to take this AI?


ejabberd.hrl Does not exist in Ejabberd 18.12

ejabberd.hrl Does not exist in Ejabberd 18.12. How should this module be compiled

@BesatZardosht Did you end up finishing the module?


Am I generating colors correctly?

I would like some advice on extracting a voxel representation for generating colors.
I was able to extract very poor vertex color by editing these lines in the G.sample_mixed loop in gen_videos_proj_withseg.py:
sample_result = G.sample_mixed(...)
sigmas[:, head:head+max_batch] = sample_result['sigma']
color_batch = G.torgb(sample_result['rgb'].transpose(1,2)[...,None], ws[0,0,0,:1])
colors[:, head:head+max_batch] = np.transpose(color_batch[...,0], (2, 1, 0))

If I look for the nearest color on the isosurface mesh, it gives me this:

But when I look at the render I see this:

I realize that the render has a final superresolution pass that makes it so clear, but I feel like I might be missing something.
My understanding of the process is something like:

G.sample_mixed takes the samples (xyz coordinates in a 3d grid) and the transformed_ray_directions_expanded (which is just 0,0,-1) and w (which is the latent vectors of shape (14,512) from the mapping network output, combining latent and camera pose) and then outputs a few results (sigma, rgb, and a copy of xyz).
The rgb is not actually rgb, but it is a 32 dimensional feature vector. So we have to decode it to RGB using the G.torgb network. This is what I find tricky. The network seems designed to process 2D images, but here we only have a bundle of N=10M feature vectors. So I pass it in a 10Mx1 image, and I hope this is ok. Also, torgb expects only a single w from the 14 options. I just picked the first one ws[0,0,0,:1] but I'm not sure if this is correct. Would it be better to run torgb for each w and then average them, or find the median, or something else?
Finally, I convert these resulting colors back to voxel space and then use the mesh vertex locations to lookup the closest color.

My questions are:

Is it ok to give torgb a 10Mx1 image or is this damaging the performance of the feature-to-color conversion?
Is it ok to only use the first ws or should I be using multiple ones somehow? Are each of the ws latents representing a different camera pose, or do they represent something else?

Thanks @SizheAn!

mix colors according to the 'normal' vector. 2 images are enough to create. full back and front.

Cher MustafaHilmiYAVUZHAN,
Je vous remercie de votre email. Je suis heureux de vous aider avec la
génération de couleurs. Si vous avez besoin d'une explication plus
détaillée, n'hésitez pas à me le faire savoir. Je suis à votre disposition
pour vous fournir toutes les informations nécessaires.

—
Reply to this email directly, view it on GitHub
https://github.com/SizheAn/PanoHead/issues/27#issuecomment-1656763006,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7FJOG7P3Y4GXVIXHE5ZLUTXSUWVZANCNFSM6AAAAAA2XUEDVQ
.

@MustafaHilmiYAVUZHAN thanks for your input. Do you have any reference code for this? When you say normal vector, do you mean the w vector? Should I run multiple w vectors through torgb and then take an average or something?

view uv

Hi! Very interesting attempt! Could you please share the code of "look for the nearest color on the isosurface mesh"? Thanks a lot!

Hi, did anyone find a good way to get vertex color?


boilerplate app doesnt run

Describe the bug
I'm using up to date macos and ran npm create etc. to get a svelte javascript project etc.
I can run the app but clicking Greet just generates:
uncaught type error... can not read properties of undefined reading invoke
the tauri cli is installed, intellie sense can see the invoke definition and the code is straight out of the box...
cheers
Reproduction
No response
Expected behavior
I expected a greeting
Full tauri info output
[✔] Environment
   - OS: Mac OS 15.1.1 x86_64 (X64)
   ✔ Xcode Command Line Tools: installed
   ✔ rustc: 1.83.0 (90b35a623 2024-11-26)
   ✔ cargo: 1.83.0 (5ffbef321 2024-10-29)
   ✔ rustup: 1.27.1 (54dd3d00f 2024-04-24)
   ✔ Rust toolchain: stable-x86_64-apple-darwin (environment override by RUSTUP_TOOLCHAIN)
   - node: 20.17.0
   - pnpm: 9.1.0
   - npm: 10.9.1

[-] Packages
   - tauri 🦀: 2.1.1
   - tauri-build 🦀: 2.0.3
   - wry 🦀: 0.47.2
   - tao 🦀: 0.30.8
   - tauri-cli 🦀: 2.1.0
   - @tauri-apps/api : 2.1.1
   - @tauri-apps/cli : 2.1.0

[-] Plugins
   - tauri-plugin-shell 🦀: 2.0.2
   - @tauri-apps/plugin-shell : 2.0.1

[-] App
   - build-type: bundle
   - CSP: unset
   - frontendDist: ../build
   - devUrl: http://localhost:1420/
   - framework: Svelte
   - bundler: Vite

Stack trace
No response
Additional context
No response

ps... I just tried the same boilerplate on another mac (M3) and ai get exactly the same error.

Turns out Im the error... I started the app with nom run dev when it should have been npm run taurie dev...
Time to stop skimming the manual...


The Grading Script Has A Problem.

The "giturl" used in the bash script should be
https://raw.githubusercontent.com/CT-CS5356-Fall2017/cs5356/master/README.md
instead of current one.
Or the returned result of "curl" will be html rather than markdown content, which will cause "awk" and anything after that to fail.

Sorry ... it turned out to be my mistake :P


Bug: Cannot install wrapper with "pip install python-ursa" on Python 3.x

Steps to reproduce

have an environment with Python 3.x and pip installed (I tried Python 3.5, 3.6 and 3.8 with Ubuntu 16.04, Ubuntu 18.04 and Ubuntu 20.04)
check pip[3] --version to make sure it is really working with Python 3
run pip install python-ursa

Expected results
Python Ursa wrapper is installed
Actual results
Python Ursa wrapper is not installed with output like
Collecting python-ursa
  Could not find a version that satisfies the requirement python-ursa (from versions: )
No matching distribution found for python-ursa

despite that pip search python-ursa successfully finds it
$ pip search ursa
python-ursa (0.1.0)  - python wrapper for ursa universal crypto library

Now this is fixed, thanks!


Added derivative and Hessian for Levy No. 13 test function

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

Comparison is base (ce20502) 91.39% compared to head (9eed27f) 91.43%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #448      +/-   ##
==========================================
+ Coverage   91.39%   91.43%   +0.03%
==========================================
  Files         162      162
  Lines       21404    21482      +78
==========================================
+ Hits        19563    19641      +78
  Misses       1841     1841

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


Support message to self

A participant should be able to send a message to them self, e.g. Alice tell Alice "Do something".
Or should this introduce a new keyword?

Yes! ...and I suggest not introducing a new keyword.


Uncaught exception 'MoorContinueException' when Moor::routeNotFoundCallback()

In a simple scenario:
# .htaccess
RewriteEngine On
RewriteRule ^.*$ bootstrap.php [QSA,NS]

<?php
// bootstrap.php
include '../libraries/moor/Moor.php';
Moor::route('/hello', function() {
    echo 'Hello!';
});
Moor::run();

When I try to get / (empty request_uri):
Fatal error: Uncaught exception 'MoorContinueException' in /srv/http/cineforiano/libraries/moor/Moor.php:611
Stack trace:
#0 /srv/http/cineforiano/libraries/moor/Moor.php(1241): Moor::triggerContinue()
#1 /srv/http/cineforiano/libraries/moor/Moor.php(762): Moor::validateMethodCallback('Moor::routeNotF...')
#2 /srv/http/cineforiano/libraries/moor/Moor.php(565): Moor::dispatchRoute(Object(stdClass))
#3 /srv/http/cineforiano/public/bootstrap.php(7): Moor::run()
#4 {main}
  thrown in /srv/http/cineforiano/libraries/moor/Moor.php on line 611

Instead to get a Moor::routeNotFoundCallback()

NOT FOUND

If I define a notFoundCallback() and assign it to Moor with setNotFoundCallback(), it works as expected.
<?php
// bootstrap.php
include '../libraries/moor/Moor.php';
function notFound() {
    echo 'Not Found!';
}
Moor::setNotFoundCallback('notFound');
Moor::route('/hello', function() {
    echo 'Hello!';
});
Moor::run();

this is due to the default not found callback not being valid 1) its not public 2) it is on Moor which doe not extend MoorAbstractController - it is fixed in the dotink fork if you would like to try that.


Rename to "import attributes"

Closes #65

Merging due to positive feedback in #65 and no concerns raised at TC39.


Added base environment support to country & western pods

You could argue that some (not all) of this could be injected in the webhook.
One downside so that approach is that it would be split: we would
still have to have this Env() function in the executor to actually
compute the whole env (docker image vars, vpctool vars, etc). The other
downside is that populating the entire env in the pod would make our pod objects
even bigger, duplicating the arrays N times for N containers.
Some of these variables are not known to the webhook until the last
second, and they are not "container-specific" you might say.
When a user specifies an env variable for the whole job, they probably
expect it to show up on each container. This PR implements that.
It also starts a new variable, TITUS_CONTAINER_NAME, which we did have
on kubelet, so I'm bringing it back.

Rebased.
Just to reiterate, this is a temp solution here to get some environment variables in the C&W containers.
When we have control-plane API support, we'll get some variables injected into our env for free (like NETFLIX_ACCOUNT_ID), Then we'll still have to have some notion of "late" env variables injected at the last minute by the executor (hopefully only the ones we really have to vpc stuff).
In other ways, this naive approach is "wrong" because it will inject variables that come from Image I think?
And, combine this with @rgulewich 's change to use the v1 pod spec and put system and user vars in the pod env, this may be a hint that this is going to get more complicated :)
I would like to revisit this approach after we have just pod spec stuff, no cinfo.


System.ArgumentOutOfRangeException: - Causes Queuing of messages to edgeHub

Describe the bug
We are seeing queuing behavior with OPC Publisher that seems to correlate with this message in our opc publisher logs.
System.ArgumentOutOfRangeException:   The specified timeout must represent a value between -1 and<PHONE_NUMBER>,   inclusive.
This error occurred on 3 different servers with in 1 hour of each other, so might be related to run time.
[13:08:02   INF] OpcPublisher status @ 10/10/2019 13:08:02 (started @ 09/30/2019   20:20:32)
--
[13:08:02   INF] ---------------------------------
[13:08:02   INF] OPC sessions (configured/connected): 1/1
[13:08:02   INF] OPC subscriptions (configured/connected): 1/1
[13:08:02   INF] OPC monitored items (configured/monitored/to remove): 59744/59744/0
[13:08:02   INF] ---------------------------------
[13:08:02   INF] monitored items queue bounded capacity: 250000
[13:08:02   INF] monitored items queue current items: 0
[13:08:02   INF] monitored item notifications enqueued: 222539990
[13:08:02   INF] monitored item notifications enqueue failure: 0
[13:08:02   INF] ---------------------------------
[13:08:02   INF] messages sent to IoTHub: 298515
[13:08:02   INF] last successful msg sent @: 10/10/2019 13:07:47
[13:08:02   INF] bytes sent to IoTHub:<PHONE_NUMBER>1
[13:08:02   INF] avg msg size: 261737
[13:08:02   INF] msg send failures: 0
[13:08:02   INF] messages too large to sent to IoTHub: 0
[13:08:02   INF] times we missed send interval: 0
[13:08:02   INF] number of events: 222539990
[13:08:02   INF] ---------------------------------
[13:08:02   INF] current working set in MB: 4896
[13:08:02   INF] --si setting: 10
[13:08:02   INF] --ms setting: 262144
[13:08:02   INF] --ih setting: Amqp_Tcp_Only
[13:08:02   INF]   ==========================================================================
[13:08:30   ERR] Error while processing monitored item messages.
System.ArgumentOutOfRangeException:   The specified timeout must represent a value between -1 and<PHONE_NUMBER>,   inclusive.
Parameter   name: millisecondsTimeout
Actual   value was -2147483648.
at   System.Collections.Concurrent.BlockingCollection`1.ValidateMillisecondsTimeout(Int32   millisecondsTimeout)
at   OpcPublisher.HubCommunicationBase.MonitoredItemsProcessorAsync(CancellationToken   ct) in   D:\a\1\s\modules\opc-publisher\opcpublisher\HubCommunicationBase.cs:line 1643
[13:09:02   INF]   ==========================================================================

Immediately after this logging occurs, the monitored item queue starts increasing until the limit is hit.
[13:09:02   INF] OpcPublisher status @ 10/10/2019 13:09:02 (started @ 09/30/2019   20:20:32)
--
[13:09:02   INF] ---------------------------------
[13:09:02   INF] OPC sessions (configured/connected): 1/1
[13:09:02   INF] OPC subscriptions (configured/connected): 1/1
[13:09:02   INF] OPC monitored items (configured/monitored/to remove): 59744/59744/0
[13:09:02   INF] ---------------------------------
[13:09:02   INF] monitored items queue bounded capacity: 250000
[13:09:02   INF] monitored items queue current items: 12554
[13:09:02   INF] monitored item notifications enqueued: 222577434
[13:09:02   INF] monitored item notifications enqueue failure: 0
[13:09:02   INF] ---------------------------------

To Reproduce
Steps to reproduce the behavior:
Unsure of replication steps at this stage, appears to occur after a significant running period  possibly every ~ 7 days?
Expected behavior
Any exceptions should be handled gracefully and connections to edgeHub/IoT Hub should be maintained to avoid dataloss.

FYI - I've tried to correlate the edgeHub logs to see if there's anything in those logs that could be related.
Here's what I think might be relevant edgeHub logs (filtered out messages relating other modules)
<7> 2019-10-10 13:08:30.347 +00:00 [DBG] [Microsoft.Azure.Devices.Edge.Hub.Amqp.LinkHandlers.EventsLinkHandler] - EventsLinkHandler processed 1 messages for nzs070iotedg01m/opcpublisher
<7> 2019-10-10 13:08:30.375 +00:00 [DBG] [Microsoft.Azure.Devices.Edge.Hub.Core.Routing.RoutingEdgeHub] - Received 1 message(s) from nzs070iotedg01m/opcpublisher
<7> 2019-10-10 13:08:30.379 +00:00 [DBG] [Microsoft.Azure.Devices.Edge.Hub.Amqp.LinkHandlers.EventsLinkHandler] - EventsLinkHandler processed 1 messages for nzs070iotedg01m/opcpublisher
<7> 2019-10-10 13:08:30.409 +00:00 [DBG] [Microsoft.Azure.Devices.Edge.Hub.Core.Routing.RoutingEdgeHub] - Received 1 message(s) from nzs070iotedg01m/opcpublisher
<7> 2019-10-10 13:08:30.414 +00:00 [DBG] [Microsoft.Azure.Devices.Edge.Hub.Amqp.LinkHandlers.EventsLinkHandler] - EventsLinkHandler processed 1 messages for nzs070iotedg01m/opcpublisher
<7> 2019-10-10 13:08:30.442 +00:00 [DBG] [Microsoft.Azure.Devices.Edge.Hub.Core.Routing.RoutingEdgeHub] - Received 1 message(s) from nzs070iotedg01m/opcpublisher
<7> 2019-10-10 13:08:30.446 +00:00 [DBG] [Microsoft.Azure.Devices.Edge.Hub.Amqp.LinkHandlers.EventsLinkHandler] - EventsLinkHandler processed 1 messages for nzs070iotedg01m/opcpublisher
<7> 2019-10-10 13:08:30.482 +00:00 [DBG] [Microsoft.Azure.Devices.Edge.Hub.Core.Routing.RoutingEdgeHub] - Received 1 message(s) from nzs070iotedg01m/opcpublisher
<7> 2019-10-10 13:08:30.486 +00:00 [DBG] [Microsoft.Azure.Devices.Edge.Hub.Amqp.LinkHandlers.EventsLinkHandler] - EventsLinkHandler processed 1 messages for nzs070iotedg01m/opcpublisher
<7> 2019-10-10 13:11:23.108 +00:00 [DBG] [Microsoft.Azure.Devices.Edge.Hub.CloudProxy.ConnectivityAwareClient] - Received connection status changed callback with connection status Disconnected_Retrying and reason Communication_Error for nzs070iotedg01m/opcpublisher
<7> 2019-10-10 13:11:24.990 +00:00 [DBG] [Microsoft.Azure.Devices.Edge.Hub.CloudProxy.ConnectivityAwareClient] - Received connection status changed callback with connection status Connected and reason Connection_Ok for nzs070iotedg01m/opcpublisher
<7> 2019-10-10 13:13:00.487 +00:00 [DBG] [Microsoft.Azure.Devices.Edge.Hub.Core.IDeviceScopeIdentitiesCache] - Getting service identity for nzs070iotedg01m/opcpublishe
<7> 2019-10-10 13:13:00.487 +00:00 [DBG] [Microsoft.Azure.Devices.Edge.Hub.Core.DeviceScopeAuthenticator] - Client nzs070iotedg01m/opcpublisher in device scope reauthenticated locally.
<7> 2019-10-10 13:13:00.487 +00:00 [DBG] [Microsoft.Azure.Devices.Edge.Hub.Core.Authenticator] - Client nzs070iotedg01m/opcpublisher re-authenticated successfully
<7> 2019-10-10 13:13:00.487 +00:00 [DBG] [Microsoft.Azure.Devices.Edge.Hub.Core.ConnectionReauthenticator] - Reauthenticated client nzs070iotedg01m/opcpublisher successfully

To me It looks like the edgeHub Communication_Error occurs after the OPC Publisher Exception, so looks to me like it might be caused by an OPC Publisher issue rather than the other way around?
After the exception in OPC Publisher logs, no Received 1 message(s) from nzs070iotedg01m/opcpublisher events are shown in the edgeHub log.
Client reauthentication messages keep occuring successfully in edgeHub logs.
Restarting OPC Publisher resolves this issue (at least temporarily)

The code looks potentially problematic in regards to how the milliseconds are calculated, probably causing some overrun on the int when cast.   Investigating.

@marcschier I've managed to compile that code and done some testing with this issue. I haven't been able to replicate it locally without artificially causing the overflow. My best guess is that some how the datetime could be shifting on the server?
However I'm thinking it might be appropriate to limit the millisecondsTillNextSend to the DefaultSendIntervalSeconds if configured?
That would avoid this issue, and at least degrade nicely in this scenario. Would you be interested in reviewing a pull request like this?
HubCommunicationsBase.cs : line ~1630
                            millisecondsTillNextSend = nextSendTime.Subtract(DateTime.UtcNow).TotalMilliseconds;
                            if (millisecondsTillNextSend < 0)
                            {
                                MissedSendIntervalCount++;
                                // do not wait if we missed the send interval
                                millisecondsTillNextSend = 0;
                            }
                            else if(millisecondsTillNextSend > TimeSpan.FromSeconds(DefaultSendIntervalSeconds).TotalMilliseconds)
                            {
                                Logger.Error($"Invalid wait time till next send ({millisecondsTillNextSend})ms, Next Send Time: {nextSendTime}, Now {DateTime.UtcNow}].");
                                //Use Default Send interval seconds as timeout, If millisecondsTillNextSend is > Int32.MaxValue then the TryTake below will fail with an exception
                                millisecondsTillNextSend = TimeSpan.FromSeconds(DefaultSendIntervalSeconds).TotalMilliseconds;
                            }

I'm also concerned that other exceptions could cause a similar problems. Is there a nice way to restart the MonitoredItemsProcessorAsync Task if it fails for some reason?

Hi @jmbrunskill, the overall compare of double against 0 can lead to unexpected results.  It is better to first cast to an integer (probably long) and then check the absolute result, otherwise, this is probably the way to go.   I would probably rather catch the "failure" early on in some way and handle it gracefully.

Hi,
we included this fix in 2.5.2 so I'm closing this issue. Please reopen if the problem persists.
Thanks!
Markus


Question: why install uWSGI in the webapp and no the webserver?

Question: why install uWSGI in the webapp and no the webserver?

I'm not sure if I understand your question correctly. But since uwsgi is the application server that actually loads and runs the webapp, in my opinion it should be packaged along side the webapp. Nginx here serves as a reverse proxy that relays requests to uwsgi. Hope that answers your question, and reiterate what is said in the readme, I don't claim that this is a 100% correct setup, there are always different opinions, considerations, and methods and I advice you to  seek out different setups to see what works best for your use case. Cheers

Thanks I think you are correct it is the only logical place and probably the only option :).


Multi Combobox: when used inside a form, deleting an item submits the form

Describe the bug
After a user selects an item, if they were to delete it by clicking on it, this action would cause the form that the multiCombobox is within to submit. I think the issue is that the button that gets rendered when a user selects an item doesn’t have its “type” set to  “button” and defaults to “submit.”
To Reproduce
Steps to reproduce the behavior:

Go to https://codesandbox.io/s/react-magma-multi-combobox-issue-when-changing-item-list-forked-u2iwzu?file=/example.tsx
If you select the red item, the form gets submitted. This only seems to happen when there is more than one item selected

Expected behavior
Deleting an item does not submit the form

@silvalaura
Where should i test this ticket?

@Uditsingh67 This storybook example is wrapped in a form
https://storybook-preview-dev--upbeat-sinoussi-f675aa.netlify.app/?path=/story/combobox--full-page-example

The issue is fixed.
See Screen-cast : http://somup.com/c3XffAvp7c
Thanks!


Uncaught TypeError: Cannot read property 'isVisible' of null

[Enter steps to reproduce:]

cmd+shift+p
click Status Bar: Toggle to get the status bar to show up
error occurs

Atom: 1.24.0-beta1 x64
Electron: 1.6.15
OS: Mac OS X 10.13.2
Thrown From: Atom Core
Stack Trace
Uncaught TypeError: Cannot read property 'isVisible' of null
At /Applications/Atom Beta.app/Contents/Resources/app/node_modules/status-bar/lib/main.js:51

TypeError: Cannot read property 'isVisible' of null
    at HTMLElement.<anonymous> (/Applications/Atom Beta.app/Contents/Resources/app/node_modules/status-bar/lib/main.js:51:41)
    at CommandRegistry.handleCommandEvent (/Applications/Atom Beta.app/Contents/Resources/app/src/command-registry.js:388:36)
    at Object.didConfirmSelection (/Applications/Atom Beta.app/Contents/Resources/app/node_modules/command-palette/lib/command-palette-view.js:165:35)
    at SelectListView.confirmSelection (/Applications/Atom Beta.app/Contents/Resources/app/node_modules/command-palette/node_modules/atom-select-list/src/select-list-view.js:401:26)
    at SelectListView.didClickItem (/Applications/Atom Beta.app/Contents/Resources/app/node_modules/command-palette/node_modules/atom-select-list/src/select-list-view.js:287:16)
    at ListItemView.onclick (/Applications/Atom Beta.app/Contents/Resources/app/node_modules/command-palette/node_modules/atom-select-list/src/select-list-view.js:222:37)
    at ListItemView.didClick (/Applications/Atom Beta.app/Contents/Resources/app/node_modules/command-palette/node_modules/atom-select-list/src/select-list-view.js:449:16)

Commands
  8x -0:40.9.0 fuzzy-finder:toggle-file-finder (div.panels-item)
  2x -0:30.1.0 core:move-up (div.panels-item)
     -0:20.8.0 command-palette:toggle (div.panels-item)
     -0:17.5.0 status-bar:toggle (div.panels-item)

Non-Core Packages
atom-beautify 0.30.9
atom-ternjs 0.18.3
atom-typescript 11.0.10
busy-signal 1.4.3
color-picker 2.2.5
docblockr 0.13.4
editorconfig 2.2.2
file-icons 2.1.15
git-time-machine 1.5.9
hyperclick 0.1.5
intentions 1.1.5
js-hyperclick 1.12.2
language-babel 2.81.1
language-pegjs 0.5.0
language-rust 0.4.12
linter 2.2.0
linter-eslint 8.4.0
linter-flow 5.6.1
linter-rust 0.8.4
linter-tslint 1.9.0
linter-ui-default 1.6.10
local-history 4.3.1
minimap 4.29.7
platformio-ide-terminal 2.7.0
prettier-atom 0.43.1
project-viewer 1.2.5
sort-lines 0.18.0

Restarting Atom fixed this for me, so feel free to close this out.  I'm not sure I have good reproducible steps.

Restarting Atom fixed this for me, so feel free to close this out. I'm not sure I have good reproducible steps.

Glad restarting fixed things for you :+1:   x-ref an issue with a similar stack trace: https://github.com/atom/status-bar/issues/199.

@ineiti can you reproduce with those steps?  Also can you clarify your step 2 what status means?  Are you toggling the status like in step 3?


SWARM-1371 - Don't resolve snapshots from repo.jboss.org.

Motivation
It's a bad idea.
Modifications
Stop resolving snapshots from repo.jboss.org.
Result
We no longer resolve snapshots from repo.jboss.org.

[X] Have you followed the guidelines in our Contributing document?
[X] Have you created a JIRA and used it in the commit message?
[X] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[X] Have you built the project locally prior to submission with mvn clean install?

retest this please


Theme not displayed

Hello,
Sorry i've a problem with the theme.
I checked to have it installed in the right place and it appears in the themes in redmine but it does not work. I use nginx web server on a centos machine. I'll send you a screenshot of browser view.
I tried with several browsers and I checked the coockies
Thx a lot for your help

FYI I just setup my Server yesterday on AWS using the image by Bitnami (recently purchased by vmware) and all I had to do to installl the theme was to unzip it in: /opt/bitnami/apps/redmine/htdocs/public/themes/
which created the folder PurpleMine2-master under themes.
If the other skins work I suspect a problem with permissions on the files (or it was not unzipped properly? Personally I use midnight commander to deal with archvies, it never fails). If none of the skins work you may want to check your webserver settings. (but dont ask me which)
BTW the bitnami image is free and so is AWS for a full year. Works great on a t3.nano instance which costs about 3.75$ / month after the free year to run. Best of all its all setup ready to run for you with latess patches, updates and so far all settings are set for best practices + security.
Good luck.


react warning props input, meta on  tag in Form.Select

I use Form.Select with Field of redux-form and get warning

Could you complete all the required fields in the issue template, please? We also need to know which version and have a codepen to investigate.

We can reopen upon receiving more info.


nixos/ejabberd: ensure erlang cookie is made

Description of changes
Tries to fix flaky nixos tests, which currently is plaguing hydra. I'm unable to reproduce locally, but https://github.com/NixOS/nixpkgs/pull/257225#issuecomment-1815346071 extracted the salient error from the logs.
Apply the same fix as found in couchdb and rabbitmq.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

The hydra failure can be reproduced with
stress -c 24 -i 2 -m 2 -d 2 change 24 to your cores
nix build ".#nixosTests.ejabberd" --option cores 1 --rebuild

I was able to reproduce it on master 50% of the time with stress, not once with this PR. Did 10 runs