Support for PostgreSQL

This PR adds schema generation for PostgreSQL and the JDBC driver to the runtime configuration of genie-web.
The original 2.0.0 schema was generated by running Genie 2 against PostgreSQL 9.5.1 and taking a schema only dump using psql. The permissions and ownerships weren't dumped so they could be used by anyone.
Then took the current mysql upgrade script and tried to modify it as closely as possible to support upgrading the 2.0.0 schema to 3.0.0. Will try to keep them in sync going forward.
Tested by successfully running Genie 3.0.0-SNAPSHOT on top of PostgreSQL database. Have no large data set internally to test against for the migration. May try to generate one later or look for outside help.
The Postgres jdbc driver version is defined by the Spring Platform IO managed version.

Current coverage is 81.49%

Merging #196 into develop will increase coverage by +0.02% as of 029438a

@@            develop    #196   diff @@
=======================================
  Files           104     104
  Stmts          2948    2951     +3
  Branches        297     296     -1
  Methods           0       0
=======================================
+ Hit            2402    2405     +3
  Partial          71      71
  Missed          475     475

Review entire Coverage Diff as of 029438a

Powered by Codecov. Updated on successful CI builds.


Unable to Adjust the Order of Merged Comics

Steps to reproduce

Merge any comic books,
Adjust the order of merged comics in 'Merge settings'.

Expected behavior
The order of merged comics can be adjusted normally.
Actual behavior
Unable to adjust.
Crash logs
No response
Komikku version
1.11.3-9318
Android version
Android 14
Device
POCOF4 (munch)
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an official extension, I should be opening an issue in the extensions repository.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 1.11.2.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Did it ever work before?

Did it ever work before?

I just reverted to the 'Komikku-arm64-v8a-r9312.apk' version, and I am unable to adjust the order.

There has never been any ability to adjust merged entries order yet. Why do you need to adjust order?

There has never been any ability to adjust merged entries order yet. Why do you need to adjust order?

The symbol in front that looks like ‘=’ generally indicates that you can press and hold it to adjust the order.
I usually use the ‘merge’ function to combine serialized manga, either by parts (such as Part 1, Part 2) or by seasons (such as Season 1, Season 2).

The order matters for the "Dedupe by priority" mode, and so it lets you set up the "priority". I just checked this and it is orderable only after selecting the "Dedupe by priority" mode and not the others. It is not very intuitive because there are no visual clues of this activating option, so this could be improved. The function though works, as in latest preview build.

There has never been any ability to adjust merged entries order yet. Why do you need to adjust order?

When adjusting the order of comics, the chapters are rearranged as well. For example, the first and second chapters of comic "a" and the first and second chapters of comic "b" are originally ordered as a1, a2, b1, b2. After the order is adjusted, they become b1, b2, a1, a2.

@RejectVanity please read my comment above if you did not and you find out it is working.

@RejectVanity please read my comment above if you did not and you find out it is working.

That is only to adjust the priority of removing duplicates, not to adjust the order of comics (in other words, chapters).

@RejectVanity as I understand the Merge function has been designed to fuse sources together to fill the missing chapters void, and so it makes to have this option for deduplication, why would you want the same chapters of more sources stacked like you described instead of just having multiple entries?

It's like doubling the number of chapters for a Manga.. Imagine to track this entry.. It's also against every sorting logic currently implemented.

It's like doubling the number of chapters for a Manga.. Imagine to track this entry.. It's also against every sorting logic currently implemented.

My idea is to merge the entire series into one book (e.g., Season 1, Season 2...), as some works are serialized in parts. I believe this should be considered a feature request, not a "bug" report.

It's like doubling the number of chapters for a Manga.. Imagine to track this entry.. It's also against every sorting logic currently implemented.

Here is the translation:
In the case where 'Allows deduplication' is disabled, I would like to allow adjusting the order of the comics as I mentioned.

Alright @RejectVanity I get you and I agree. The Merge function can be even more useful after this, so the point for merging won't be to just fill the voids but to merge more separated chapters of a series into one, then making the library less disordered.
So like I said, the current issue is that only if we activate the deduplication we can adjust the order of the chapters somehow, so not yet possible to have a1, a2, b1, b2 if we don't wanna deduplicate.
Besides, it would be very useful too if we could just drag-and-drop single chapters of an entry.


Origin file has no clip Info

Dear blogger, hello:
During the process of using your script, he parsed the original file information as an XML file. However, I searched through all the files and did not see the part of the file clip he mentioned. In theory, I should be able to see which file started using a few minutes and seconds, and how many frames were used, but I couldn't find anything
only thing i can find is file path,
i wanner ask is this no info ,or encrypt with something?

From my understanding the information is contained in the prproj:
	<VideoClip ObjectID="120" ClassID="9308dbef-2440-4acb-9ab2-953b9a4e82ec" Version="11">
		<Clip Version="18">
			...
                        <Source ObjectRef="59"/>
			<InPoint>0</InPoint>
			<OutPoint>23191660800000</OutPoint>
		</Clip>
	</VideoClip>

The referenced (ObjectRef="59") Source object contains a reference to the Media object (ObjectURef="151212b9-24e9-41ed-8a3c-78f7fe563c04"):
E.g:
	<VideoMediaSource ObjectID="59" ClassID="e64ddf74-8fac-4682-8aa8-0e0ca2248949" Version="2">
		<MediaSource Version="4">
			<Content Version="10">
			</Content>
			<Media ObjectURef="151212b9-24e9-41ed-8a3c-78f7fe563c04"/>
		</MediaSource>
		<OriginalDuration>23191660800000</OriginalDuration>
        </VideoMediaSource>

With the Media object containing the file information:
	<Media ObjectUID="151212b9-24e9-41ed-8a3c-78f7fe563c04" ClassID="7a5c103e-f3ac-4391-b6b4-7cc3d2f9a7ff" Version="27">
                 ...
		<Title>some_video.mp4</Title>
		<ActualMediaFilePath>C:\some\path\some_video.mp4</ActualMediaFilePath>
	</Media>

How to convert the timestamps into seconds is not entirely clear to me but i think it is done here

Hey,
Sounds like an interesting project and reminds me of the efforts by Devon Crawford: https://github.com/DevonCrawford/Video-Editing-Automation
And his YouTube video: https://youtu.be/0ZeO0IQaJ-A?si=uvJHaq5kIl3SJaJ0
I would like the discussion to be visible in the open web and not hidden in some telegram chat. Do you have some code to share or same specific questions?
Cheers,
Runinho

Hi! Frame information is actually stored in frame count format, amount of frames, independently of the frame rate being used.

Thanks @sergeiventurinov !
Can you point to the part of the code that does that math? Thanks in advance!

Is this the correct location?
https://github.com/sergeiventurinov/PRPROJ-READER/blob/main/PRPROJ-READER.py#L747
Where does the constants come from? Have you found some documentation or did you guess?
Thanks,
Runinho

It's been a long time since I've calculated myself.
Sorry, I'm an amateur.
I deduced it practically. I think this factors are multiples both of 24 (framerate) and 44100 (audio rate)  in a way the Premiere programming needed to use only one factor for both calculations, but i dont know. They work, anyway.

Hi,
I have found your code very helpful! No need to excuse yourself 😀
Just wanted to know if you found some documentation. But the values also worked great for me :)
Thanks and have a great day!

Hello! I make project about automatization video editing right now! And I need more understading about tags in prproj file. If you want to join - please contact with me (telegram: @dl_hello)

You can check this documentation i've made from digging into the prproj files: https://github.com/sergeiventurinov/PRPROJ-READER/blob/main/PRPROJ STRUCTURE.pdf


[wip] Dropdown Button Fix

This will fix a bug that deals with a Right Margin missing so some buttons do NOT connect to others.

This gives buttons space between each other.

Can you send me a before & after screenshots? Because I still don't see anything in visual.

This is before, the other is after.

This fix also messes up the discussion list: https://flightsimtr.tk/

I will find a fix.

Please include your fix in the fixes branch so it will be included in this PR.
After you apply the fix, ping me here so I'll review it again.

I think I did something but, that should fix it.


return not leader error with real leader address

client may send command to not leader pd, now we just close it violently, but we should return an error first and then close it gracefully.

Now we have already logged it in pd, i think that is enough to trace this info.

ok, we can close it now.


Is this project actually abandoned?

I can see that the author did not accept pull request to fix the Gradle crashes, and the Github repo hasn't been updated for about a half year.
I have just saw/wrote some alternatives here:

https://github.com/alann-maulana/flutter_beacon Just googled this, going to pull it and check if this will work.
https://github.com/michaellee8/flutter_blue_beacon Spinned up my own library wrapping flutter_blue. Scanning iBeacon in iOS is not going to work therefore, since Apple require native scanning for iBeacon. No support to background scanning right now.

@michaellee8 Im also searching for an alternative. Have you found a good package?

@yveswehrli I have searched hard but right now only the above 2 exists. Actually this one is the best through.

@michaellee8 Thanks for your help 👍

I've made the relevant changes to support AndroidX and Kotlin 1.3 here
If the maintainer comes back online then we can pull it in.

@duncanhoggan your contribution seems really great, maybe we will have to start consider forking this project to an organization.

it's almost a year since beacons weren't updated by the author. I think this should be archived. create another package as an organization to maintain any updates to the native realm(android & iOS).

I am heavily using this on my flutter apps, let's move it to an organization. I can help a lot in native android side.

Created a request here, https://github.com/fluttercommunity/community/issues/21
Please upvote the issue.

Keep it active so it'll get merged

It seems this project is well and truly abandoned since we've gone 6 more months with no activity.  Is anyone in this thread still interested in maintaining a beacon plugin for Flutter?
I'd love to hear from @lukaspili since he seems to still be active in other Loup projects like streams_channel.  Lukas are you willing to turn this plugin over to someone else so it can be maintained?
How about @awazgyawali and @duncanhoggan?  Are you guys still interested in this plugin and willing to help maintain it?
Beacons are not one of my life passions, but I think they're interesting and I do have a work project right now that is using them.  So I'm willing to spend some time to get this plugin updated with current dependencies, figure out a new home for it (if possible), and put a little time into maintaining it going forward (at least for the time being).
I have zero experience in publishing Flutter plugins, but I'm willing to learn and would love some help.  Anyone else?

I have been tweaking and configuring this project on my fork to fill my requirements. I initially planned to merge my changes here but sadly, the maintainer was not active thus didn't get a chance to merge them. Currently, my fork has gone way too far and has been tweaked to fit my organizational needs.
But I can help with the project if we are again planning to refactor or to maintain this. Let's
get posted here.

@awazgyawali Thanks for the reply. Are your changes to the plugin not generally useful then?  Has this plugin worked for you otherwise?  Or are there some important bug fixes that you have done to it on your fork?
I've had some success with the flutter_beacon plugin, but have also seen some crashes.  This one seems to support background monitoring, whereas after a quick search I don't see anything dealing with that in the code or docs of flutter_beacon.
As was mentioned on the issue you opened on the community site, it would be best if there was one great plugin rather than several good ones.
So it seems the first step is determining a starting point.  There seems to be a lot of good code in this plugin.  On the other hand, flutter_beacon does seem to be maintained and there's a good chance that the maintainer would welcome the help.

Hello,
As @awazgyawali said, his fork is too different and too specific to be merged here.
We plan to restart support for beacons in the near future, but our other libraries have a higher priority right now, so I'm not sure about the timeline for beacons.
I'm open to include additional contributors and/or merge this effort with something else. Let me know, and sorry for keeping you all in the dark for so long.

@lukaspili It's great to hear that you're planning to continue support for this plugin!  That would be the best outcome by far.
I guess the main question is how can we help?  It seems like the highest priority would be to merge in some of the PRs for build issues and get a new version published.
I'm sure there are others who are willing to help, and I'm dealing with a Flutter beacon project right now.  So I'm highly motivated. :-)
What level of assistance would you be comfortable with?

@deanriverson Happy to assist with maintenance. I ended up writing my own version from scratch. I needed a lib just to do beacon scanning and ranging for iBeacon and Eddystone. Its deployed and working quite nicely.
I agree that having a lib specifically for beacon interaction is really nice. We gotta make sure from the outset that the area of responsibility/feature set is clearly defined.

I'm sure there are others who are willing to help, and I'm dealing with a Flutter beacon project right now. So I'm highly motivated. :-)

🤣

Thanks @duncanhoggan.  Does your version support background monitoring?  I didn't see the source on your GitHub, but I'd love to read through it if it's available somewhere.
@lukaspili It looks like we have a lot of experience and willingness to help in this thread.  We just need a plan and some permissions.  Would you be willing to appoint some contributors with permissions to approve PR requests?  Or would you prefer something else be done?


Wrong path to generated lambda function src displayed after amplify function add

Description of bug
After using amplify function add to generate a new Lambda function, included in the output is the following:
"Check out sample function code generated in project-dir/amplify/backend/lambdaname/src"
However, the actual directory is located at: project-dir/amplify/backend/function/lambdaname/src
To Reproduce
Here is a full verbatim transcript highlighting the error:
amplify function add
Using service: Lambda, provided by: awscloudformation
? Provide a friendly name for your resource to be used as a label for this category in the project: photoprocessor
? Provide the AWS Lambda function name: photoprocessor
? Choose the function template that you want to use: Hello world function
? Do you want to edit the local lambda function now? true
Please manually edit the file created at /home/ec2-user/environment/photo-albums/amplify/backend/function/photoprocessor/src/index.js
? Press enter to continue
? Press enter to continue
Successfully added resource photoprocessor locally.

Next steps:
Check out sample function code generated in <project-dir>/amplify/backend/photoprocessor/src
"amplify function build" builds all of your functions currently in the project
"amplify function invoke photoprocessor" enables you to test a function locally
"amplify push" builds all of your local backend resources and provisions them in the cloud
"amplify publish" builds all of your local backend and front-end resources (if you added hosting category) and provisions them in the cloud

Expected behavior
It should show <project-dir>/amplify/backend/function/<mylambdaname>/src
Amplify version
0.1.27

The fix for this is merged in the latest version of the CLI -> 0.1.28


Improve Tab Props

Tab is a nice addition.  The problem I'm having is that I want to pass props to my tab panes (render functions).  To do this I'm doing <Tab panes={panes} foo="foo" />, which returns Unknown prop foo on <div> tag. Remove this prop from the element. For details, see.  Can we have a new prop, like paneProps that we use just for the render props?
Also, can we have  take in Tab.Pane children in addition to a panes list prop?

You have this warning because unhandled props are passed to the root element of Tab, it's a div in this case. Can you clarify your needs? Why you can't use this approach?
const paneProps = { foo: 'foo' }
const panes = [
  { menuItem: 'Tab 1', render: () => <Tab.Pane {...paneProps}>Tab 1 Content</Tab.Pane> },
  { menuItem: 'Tab 2', render: () => <Tab.Pane {...paneProps}>Tab 2 Content</Tab.Pane> },
]

const TabExampleBasic = () => (
  <Tab panes={panes} />
)

I'm pretty sure that the new prop isn't required.

I had actually considered this as well. Most often, every pane will be rendered with mostly the same props.
I'm still not completely convinced one way or the either, so I had defaulted to the minimal case.

I'm closed this for housekeeping, however the issue can be reopened after @rostero1's answer.

I'm very confused, I just don't get it, how do I pass data to the panes?

@EdsonAlcala create the const panes = [] within Profile component.  This way, you can pass any of the connected data to the panes.

I am confused about this const panes array.
What if my Tab is receiving some event handlers and values as props from parent, which should be distributed to each pane child? How can I pass those props to the child pane?

It uses menuItem as key. Should you have two items with same menuItem name you get a React error like
Encountered two children with the same key

Please open new issues with complete reports for the best help.


AMLOGIC-1457: Remove Unnecessary videoPort from llama

Reason for change:
Remove Unnecessary videoPort from llama
Test Procedure: None
Risks: Low
Change-Id: I53f51cac3c1bac82ff9edc984375a47cf6f06bd0
Signed-off-by:Anooj Cheriyan<EMAIL_ADDRESS>

Seems good.

New one created: https://github.com/rdkcentral/rdkservices/pull/1267


[Bug] Pass trust_remote_code to transformers.AutoTokenizer

Addressing issue #909. It looks like some transformers tokenizers require passing the trust_remote_code kwarg.
Mirroring the way we pass the kwargs of TransformersEngine down to _model, this PR passes the kwargs of TransformersTokenizer down to _tokenizer.
Furthermore we are now getting the value of trust_remote_code from the engine kwargs and passing it to TransformersTokenizer -- this is the one bit that gives me pause because it treats this kwarg as "special".
It's not in general straightforward to determine which kwargs belong to _model and which belong to _tokenizer unless we want users passing explicit dictionaries for each (or just the tokenizer).

My effort in #912 passes everything. Testing with a 'definitely not there' argument results in general failure.

Closing as a dupe of simultaneously submitted #912


Error when editing messages, not overrides componentes field

Before updating discord.js version, when the components array was undefined the discordMessageManager deleted previous components on the original message.
After updating messages keep showing components like menus or buttons.

Fixed on 3.4.0-alpha.2


Optional stream argument for pprint

Add an optional stream argument to any pprint methods, for consistency with pprint.pprint, cf #804.

http://www.nltk.org/_modules/nltk/tree.html#Tree.pprint done
http://www.nltk.org/_modules/nltk/probability.html#FreqDist.pprint done
http://www.nltk.org/_modules/nltk/corpus/reader/verbnet.html#VerbnetCorpusReader.pprint returns string
http://www.nltk.org/_modules/nltk/corpus/reader/conll.html#ConllSRLInstance.pprint returns string
http://www.nltk.org/_modules/nltk/corpus/reader/conll.html#ConllSRLInstanceList.pprint returns string
so this is done i think.

@Dobatymo actually I think there's still quite some work that needs to done to make pprint consistently prints to stdout and pformat return a string.
From docs:

pprint.pformat(object, indent=1, width=80, depth=None, *, compact=False)
Return the formatted representation of object as a string. indent, width, depth and compact will be passed to the PrettyPrinter constructor as formatting parameters.
pprint.pprint(object, stream=None, indent=1, width=80, depth=None, *, compact=False)
Prints the formatted representation of object on stream, followed by a newline. If stream is None, sys.stdout is used. This may be used in the interactive interpreter instead of the print() function for inspecting values (you can even reassign print = pprint.pprint for use within a scope). indent, width, depth and compact will be passed to the PrettyPrinter constructor as formatting parameters.

There's also quite some pretty_print() in NLTK that does various things =(


Share your work with Mattermost community?

Thanks for including Mattermost Desktop App in the Homebrew Cask manager :)  https://github.com/caskroom/homebrew-cask/blob/master/Casks/mattermost.rb

We'd love to help share it with the Mattermost community. Are there any specific guidelines we should follow to appropriately promote your project?

If you ever mention the cask file (the file in your screenshot), be sure to call it a cask, not a formula (different things). It’s also “Homebrew-Cask”, not “Homebrew Cask”. “homebrew-cask” is also fine.
Those are just for correctness, though, we don’t really have any mandatory requirements or guidelines. However you want to do it is fine.

Got it, will do. Thanks @vitorgalvao!


Suggest: let json.dumps keeps non-ASCII characters when sending json request

Summary
When I send a json request, the json.dumps does not keeps non-ASCII characters.
Reproduction Steps
cat <<MM | python3 - &
from flask import Flask,request
app = Flask('')

@app.route('/', methods=['POST', 'GET'])
def test():
    return(request.data)

app.run()
MM

cat <<MM | python3 -
import time
import requests as r
time.sleep(1)
print(r.post('http://localhost:5000/', json={'bar':'程序员'}).text)
MM

Expected Result
I expect result keeps non-ASCII request:
{"bar": "程序员"}
Actual Result
{"bar": "\u7a0b\u5e8f\u5458"}
Suggest
Modify the models vim modles.py +458 to keep the non-ASCII characters.
For example:
body = complexjson.dumps(json, ensure_ascii=True)
System Information
$ python -m requests.help

{
  "chardet": {
    "version": "3.0.4"
  },
  "cryptography": {
    "version": ""
  },
  "implementation": {
    "name": "CPython",
    "version": "3.6.1"
  },
  "platform": {
    "release": "15.6.0",
    "system": "Darwin"
  },
  "pyOpenSSL": {
    "openssl_version": "",
    "version": null
  },
  "requests": {
    "version": "2.18.1"
  },
  "system_ssl": {
    "version": "100020cf"
  },
  "urllib3": {
    "version": "1.21.1"
  },
  "using_pyopenssl": false
}

Does

cat <<MM | python3 -
import json
import time
import requests as r
time.sleep(1)
print(r.post('http://localhost:5000/', data=json.dumps({'bar':'程序员'}), headers={'Content-Type': 'application/json'}).text)
MM

Do the right thing?

Also, looking at our source code: https://github.com/requests/requests/blob/b66908e7b647689793e299edc111bf9910e93ad3/requests/models.py#L454
I see:
        if not data and json is not None:
            # urllib3 requires a bytes-like body. Python 2's json.dumps
            # provides this natively, but Python 3 gives a Unicode string.
            content_type = 'application/json'
            body = complexjson.dumps(json)
            if not isinstance(body, bytes):
                body = body.encode('utf-8')

Which is unclear if that's what you're complaining about. Furthermore, you haven't given us any of the information we kindly requested. So we'll need that before we can move forward.
Please provide us with the output of
python -m requests.help

If that is unavailable on your version of Requests please provide some basic information about your system (Python version, operating system, etc).

@sigmavirus24 I've updated this issue with more info.
I hope the json request could keep non-ASCII.
For the purposes of:

Decrease the size of data
Convenient for debugging and reading.

@ahui132 please answer my question. Your issue now includes the baseline information but is still unclear. Are you suggesting adding ensure_ascii=True to every call?

Hi @ahui132, I think you may be suggesting we set ensure_ascii to False, not True. If you look at the method signature in both Python 2 and Python 3, you'll see it's already set to True by default.
It looks like we are doing something a bit off here. When you use dumps with the json module, the ensure_ascii flag escapes all unicode strings. When we call encode('utf-8'), we are escaping the escape strings which means we send \\u7a0b\\u5e8f]\u5458 instead of \u7a0b\u5e8f\u5458. I'm pretty sure that's not what we want, since the receiving json decoder with think the string is literally \u7a0b\u5e8f\u5458 not 程序员 without building in a custom decoder.
So setting ensure_ascii to False means we won't double escape and do actually send what we're expecting. I'm not sure how this change would affect existing users, or how we've had this functionality around so long without it being called out. I've run out of investigation time today but if anyone wants to build some tests around this, that would be a great start.

@nateprewitt I'm not sure I follow. Which of the following do you feel is correct:
>>> import json
>>> json.dumps({'bar':'程序员'})
'{"bar": "\\u7a0b\\u5e8f\\u5458"}'
>>> json.dumps({'bar':'程序员'}).encode('utf-8')
b'{"bar": "\\u7a0b\\u5e8f\\u5458"}'
>>> json.dumps({'bar':'程序员'}, ensure_ascii=False)
'{"bar": "程序员"}'
>>> json.dumps({'bar':'程序员'}, ensure_ascii=False).encode('utf-8')
b'{"bar": "\xe7\xa8\x8b\xe5\xba\x8f\xe5\x91\x98"}'

>>> print(requests.post('https://httpbin.org/post', headers={'Content-Type': 'application/json'}, data=json.dumps({'bar':'程序员'}).encode('utf-8')).text)
{
  "args": {},
  "data": "{\"bar\": \"\\u7a0b\\u5e8f\\u5458\"}",
  "files": {},
  "form": {},
  "headers": {
    "Accept": "*/*",
    "Accept-Encoding": "gzip, deflate",
    "Connection": "close",
    "Content-Length": "29",
    "Content-Type": "application/json",
    "Host": "httpbin.org",
    "User-Agent": "python-requests/2.18.4"
  },
  "json": {
    "bar": "\u7a0b\u5e8f\u5458"
  },
  "origin": "<IP_ADDRESS>",
  "url": "https://httpbin.org/post"
}

>>> print(requests.post('https://httpbin.org/post', headers={'Content-Type': 'application/json'}, data=json.dumps({'bar':'程序员'}, ensure_ascii=False).encode('utf-8')).text)
{
  "args": {},
  "data": "{\"bar\": \"\u7a0b\u5e8f\u5458\"}",
  "files": {},
  "form": {},
  "headers": {
    "Accept": "*/*",
    "Accept-Encoding": "gzip, deflate",
    "Connection": "close",
    "Content-Length": "20",
    "Content-Type": "application/json",
    "Host": "httpbin.org",
    "User-Agent": "python-requests/2.18.4"
  },
  "json": {
    "bar": "\u7a0b\u5e8f\u5458"
  },
  "origin": "<IP_ADDRESS>",
  "url": "https://httpbin.org/post"
}

As far as I can tell they both have the same result.

😅This is what I get for writing quick and dirty tests. I was using the data field which does change based on ensure_ascii, but the json modules handles this the same (\==\\, \\\==\\\\, etc). I was comparing strings rather than the loaded JSON objects which caused the inconsistency.
@ahui132, this is working as expected. Your server is receiving json-escaped strings which are valid according to the spec. You'll also note that the spec requires all JSON data transferred between systems to be represented this way. If you want the json contents to return to their "expected" form, you'll find calling r.content.decode('unicode_escape') should do the trick.

@nateprewitt @sigmavirus24
Let me make a summarization
json string's format: Non-ASCII vs ASCII
The json-escaped raw string "{\"bar\": \"\\u7a0b\"}" is different from "{\"bar\": \"\u7a0b\"}".

The first one is pure ASCII format, and the second is non-ASCII format
Both of them are valid json escaped format.

>>> json.dumps('\u5458', ensure_ascii=True) # ASCII
'"\\u5458"'
>>> len(json.dumps('\u5458', ensure_ascii=True).encode('utf8'))
8
>>> json.dumps('\u5458', ensure_ascii=False) # non-ASCII
'"员"'
>>> '"\u5458"' # non-ASCII
'"员"'
>>> len(json.dumps('\u5458', ensure_ascii=False).encode('utf8'))
5

How to send json with non-ASCII format
I we want to send json with non-ASCII via requests, we have to use post method with Content-Type: appliaction/json explicitly(It's too complex):

print(requests.post('https://httpbin.org/post', headers={'Content-Type': 'application/json'}, data=json.dumps({'bar':'员'}, ensure_ascii=False).encode('utf-8')).text)

I think the pythonic way is:

Add an ensure_ascii option: print(requests.post('https://httpbin.org/post', json={'bar':'员'}, ensure_ascii=False)).text)
Set ensure_ascii=False by default in vim modles.py +458

This project is not adding new keywords that are related to serialization. If you need this specific handling, then the first way that is explicit is the only right way to do it.
It's not too complex, as json is API sugar for the 98% use-case. The last 2% simply will not get API sugar around that. That's the design credo of this project.


Sun & directional light do not match-up.

Here is a patch to the examples/cycle.rs file to show the issue.  Shadows are turned on to help see the different angle of the directional light and the rendered Sun.
diff --git a/examples/cycle.rs b/examples/cycle.rs
index e52a5f1..4ca88ce 100644
--- a/examples/cycle.rs
+++ b/examples/cycle.rs
@@ -56,6 +56,10 @@ fn setup_environment(
     // Our Sun
     commands
         .spawn_bundle(DirectionalLightBundle {
+            directional_light: DirectionalLight {
+                shadows_enabled: true,
+                ..default()
+            },
             ..Default::default()
         })
         .insert(Sun); // Marks the light as Sun
@@ -94,7 +98,7 @@ fn setup_environment(
     // Spawn our camera
     commands
         .spawn_bundle(Camera3dBundle {
-            transform: Transform::from_xyz(5., 0., 5.),
+            transform: Transform::from_xyz(5., 0., 5.).looking_at(Vec3::new(0., 1., 0.), Vec3::Y),
             ..default()
         })
         .insert(AtmosphereCamera(None)) // Marks camera as having an atmosphere that isn't on a specific render layer

It seems the looking_at on the camera isn't causing the problem, since removing it doesn't sync-up the Sun and directional light.  Still need to have shadows on to easily see the direction of the light vs the rendered Sun.
This issue might be related to the change to default value for sun_position in the 0.4.0 release:
-            sun_position: Vec3::new(1.0, 1.0, 1.0),
+            sun_position: Vec3::new(0.0, 1.0, 1.0),

Thanks for pointing this out. It should be fixed with 58d5ccf12c9120a985d6413ecb5cf6e0f6c8120a.


Enable provider forks to easily implement their own revalidate route logic

Currently /app/api/revalidate/route.ts is hardcoded to support Shopify's collection and webhook model. This means a provider can't focus only on the lib folder and will cause overhead over time to keep the fork up to date with changes Vercel makes to the core.
Would be nice to have this logic moved to be inside the lib folder as well, maybe even separated out under lib/hooks/revalidate.ts so it's clear what it handles.

Choosing Shopify as the standard is really the strangest thing about V2. Shopify already has a React-powered framework in Hydrogen that is actively maintained and is updated nearly daily. Why would anyone who uses Shopify choose this repo over Hydrogen? So who exactly are they building this for? Personally, I think they should have built an agnostic repo with products in a JSON file, a cart powered by Vercel KV (Redis), and checkout with Stripe. This would allow headless CMS's to support the repo for the product part, and the other features can be augmented via different 3rd party packages.


Updating package.json

so that different versions of power select can be used in conjunction with this addon.
https://github.com/cibernox/ember-power-select-with-create/issues/55

Also need this.  Nice!

Superceded by #59


Add Ratepay

Add Ratepay option into the openinvoice payment method

todo: add devicefingerprint in HPP request
https://docs.adyen.com/developers/revenueprotect#gettingthedevicefingerprint

DeviceFingerprint is added see: https://github.com/Adyen/adyen-magento/commit/8ae927967b73ab0634a2edbac79c36d94bd5059c


Getting Name Error for "ffmpeg_streaming"

Getting a Name Error
NameError: name 'ffmpeg_streaming' is not defined

Obviously I have installed ffmpeg and python-ffmpeg-video-streaming

I also tried it on a Ubuntu Instance, still getting the same error.

Add import ffmpeg_streaming line at the begging of your code


[Deck] Tofu Issue with CJK Languages in Steam

Only affects Steam, in both Game Mode and Desktop Mode.

More info: https://github.com/ValveSoftware/steam-for-linux/issues/9418
Workaround (for some reason it does not work for Korean and also the update screen before entering Game Mode):
$ rpm-ostree override remove google-noto-sans-cjk-vf-fonts
$ rpm-ostree install google-noto-sans-cjk-fonts

Thank you!
https://github.com/ublue-os/bazzite/commit/c87d9feab4705f5befff556d01534f9f41fa639c

Located the fix for the tofus on Steam update screen as well - I believe Valve has hardcoded the path to the font files:
$ strings $HOME/.local/share/Steam/ubuntu12_32/steam | grep "font"
/data/src/common/uifontfile.cpp
fontFile.encrypted_contents().size() <= INT_MAX
fontFileDecrypted.opentype_font_data().size() <= INT_MAX
iIndex >= 0 && iIndex < m_pFontPackagePB->encrypted_font_files_size()
/tenfoot/resource/fonts/tenfoot.uifont
/usr/share/fonts/truetype/ttf-dejavu/DejaVuSans.ttf
/usr/share/fonts/opentype/noto
/usr/share/fonts/noto-cjk
uifontfile_format.proto
uifontfile_format.proto"C
font_file_name
opentype_font_data
encrypted_font_files

Working PoC:
$ sudo ostree admin unlock
$ sudo ln -s /usr/share/fonts/google-noto-sans-cjk-fonts /usr/share/fonts/noto-cjk

https://github.com/ublue-os/bazzite/commit/7dd9580f709369844d80d67485f256808264187a

@KyleGospo
Things are broken again in F39. I believe patches applied to F38 in ublue-os/main#334 needs to be applied to F39 as well.

On F39, whatever font gets picked by the Game Mode UI seems fine with Chinese and Japanese but still broken with Korean (I cannot locate the font file used at the moment, Valve probably made some changes)
Symlink I proposed is broken on F39 as google-noto-sans-cjk-fonts is not installed, causing the tofus on the Steam Bootstrapper screen again

Fixed by rpm-ostree install google-noto-sans-cjk-fonts


Opbinaryop and other minor optimizations

Nothing like the last optimisation but but it does add up to around 4% improvement. I've done some micro optimizations on the v.ip increments. While individually they don't amount to much, they do seem to add up on a fib(35) and likely on any long running script.
OLD:

fibonacci(35)
Result:  9227465
Go:      45.674312ms
Parser:  32.976µs
Compile: 104.373µs
VM:      2.597270621s
fibonacci(35) (tail-call #1)
Result:  9227465
Go:      62.044356ms
Parser:  20.597µs
Compile: 71.31µs
VM:      2.692990659s
fibonacci(35) (tail-call #2)
Result:  9227465
Go:      447ns
Parser:  37.322µs
Compile: 69.521µs
VM:      13.112µs
NEW:

fibonacci(35)
Result:  9227465
Go:      47.159815ms
Parser:  34.767µs
Compile: 115.81µs
VM:      2.44406141s
fibonacci(35) (tail-call #1)
Result:  9227465
Go:      62.935803ms
Parser:  19.642µs
Compile: 64.569µs
VM:      2.52746129s
fibonacci(35) (tail-call #2)
Result:  9227465
Go:      251ns
Parser:  17.535µs
Compile: 49.706µs
VM:      14.56µs

Using opcodes as a parameter to another opcode doesn't feel right. Can we just add 1-byte operand to OpBinaryOp (for operator token) and delete all other binary opcodes (OpAdd, OpSub, ...)?

Using opcodes as a parameter to another opcode doesn't feel right. Can we just add 1-byte operand to OpBinaryOp (for operator token) and delete all other binary opcodes (OpAdd, OpSub, ...)?

I did try that originally but for some reason that I can't remember now I changed to this. Let me try it again and see how it goes.

Also realized that we were failing to set v.errOffset properly.
filePos := v.fileSet.Position(v.curFrame.fn.SourceMap[v.ip-v.errOffset]) // <- this will not work

We can probably find a better way to handle this. I will think about this a little bit today.

One solution to IP offset is to add the following method to CompiledFunction:
// SourcePos returns the source position of the instruction at ip.
func (o *CompiledFunction) SourcePos(ip int) source.Pos {
	for ip >= 0 {
		if p, ok := o.SourceMap[ip]; ok {
			return p
		}
		ip--
	}
	return source.NoPos
}

, and, change error handling code in VM to:
err = v.err
if err != nil {
	filePos := v.fileSet.Position(v.curFrame.fn.SourcePos(v.ip - 1))
	err = fmt.Errorf("Runtime Error: %s\n\tat %s", err.Error(), filePos)
	for v.framesIndex > 1 {
		v.framesIndex--
		v.curFrame = &v.frames[v.framesIndex-1]

		filePos = v.fileSet.Position(v.curFrame.fn.SourcePos(v.curFrame.ip - 1))
		err = fmt.Errorf("%s\n\tat %s", err.Error(), filePos)
	}
	return err
}

Then, we don't have to track IP offset (.errOffset) and remove it.

Also, if we keep the proposed panic-recover approach, we can remove stack overflow checks too:
defer func() {
	if r := recover(); r != nil {
		if v.sp >= StackSize || v.framesIndex >= len(v.frames) {
			v.err = ErrStackOverflow
			return
		}

		if v.ip < len(v.curInsts)-1 {
			if err, ok := r.(error); ok {
				v.err = err
			} else {
				v.err = fmt.Errorf("panic: %v", r)
			}
		}
	}
}()


Revert "cybperunkred-raycw: Update fields automatically for cyberpunkred-raycaw"

Reverts Roll20/roll20-character-sheets#9914
Formatting is off due to the recently merged PR.  Requesting revert to previous known-good version.

Character Sheet Info Roll20 Internal Use only.

Found the issue and will submit a new PR to fix.


Free radius Feature Request

Hi, i was spoke with mimugmail in forum.
as he says, there is some complexity for add this feature, i want to add Free-radius  functionality for all the VPN connections, can you guide me for researching and help you?
thanks

is there any plan to add this functionality? or is there any radius appliance for use for radius server?

@rout3rx with 18.7 you can use radius for openvpn and also assign IPs or set routes ..

Close?

Yes, @rout3rx can reopen if missing something


Update Plurals

Closing as we don't need to wrap the plural list in a class as it's added to the load path of I18n already.


bug: 🐍 error on library decoding

Issue
Error on executing module without --allow-all flag.
(try to repoduce the bug on different plateform)
Context
The lib files are cached before runtime with static imports in "./deps.ts" via ejm.sh. As ejm.sh serves b64 encoded string of the binary data and Deno.dlopen only accepts filesystem path the method is to use the cached base64 string, decode  it and write it to a file in the user temp directory and use it as Deno.dlopen path.
Description
Seems to only fails on windows.
When trying to run the module with the smallest restrictions (deno run --allow-read --allow-write --allow-ffi --unstable "...WEBUI" ).
error: Uncaught (in promise) NotFound: Le specified filed was not found. (os error 3): writefile 'C:\Temp\webui-2-x64.dll'
    Deno.writeFileSync(libPath, libBuffer);
         ^
    at Object.writeFileSync (ext:deno_fs/30_fs.js:771:7)
    at writeLib (file:///.../deno-webui/src/utils.ts:31:10)
    at loadLib (file:///.../deno-webui/src/lib.ts:34:34)
    at new WebUi (file:///.../deno-webui/src/webui.ts:54:47)
    at file:///.../deno-webui/examples/file_explorer/main.tsx:7:16
    at eventLoopTick (ext:core/01_core.js:183:11)

But works fine with (deno run -A --unstable "...WEBUI" ), note that no other permissions are requested.
Need to be investigate. Maybe an error due to "os_paths" dependency or from Deno runtime, in that case need to report the issue.

What -A does?
And does the file C:\Temp\webui-2-x64.dll exist?

-Astands for --allow-all.  The issue came maybe from os_paths used to write cache to the temp file of the user (only give the temp user path depending of the os).
Normally it should return C:/Users/<username>/AppData/Local/Temp on windows but it don't work properly without --allow-all flag, so maybe a permission issue but no one ask. I don't known the situation on other os.
Some investigation is needed but a good hot fix can be to use Deno.makeTempFile instead.
I don't use it at first because as the name of the file is random I can't reuse the uncompressed lib between launchs but the startup time is not really impacted as the lib is already cached as base64 in ram at the script execution (and in deno cache disk cache between runs) due to import ... 'https://esm.sh/... .


Job name parsing is skipping the first plus (+)

If you look at the attached email generated by runbld, the job name isn't parsed quite correctly into its parts...my understanding is the jobs should now be user + project + branch + meta-data.

JOB NAME
    elastic+elasticsearch+master+multijob-intake

EXPECTED
    user:      elastic
    project:   elasticsearch
    branch:    master
    meta-data: multijob-intake

ACTUAL
    user:      elastic+elasticsearch
    project:   master
    branch:    multijob-intake
    meta-data: <none>

Is my assumed format no longer correct? I am actually using the metadata field for establishing the different phases of a single top-level job, and there's a regex being used to create the various Jenkins "views" to keep things organized.
BTW, the new output is amazing! The summary and parsed out/attached errors is awesome.

It's a bug, fixed but not yet pushed!
And thanks :grin:

Reopening, want to make sure it closes from the commit....

Doh, sorry for jumping the gun.


Docker image fixes

For local development, uses a local CCC server.
All docker computes are done in CCC (with the npm CCC client lib).
Refactored root *js files into a separate folder ('main') to allow mounting server files for faster local development.
Added tests and test routes for testing workflow0.
Partial migration to storing less uploaded files, and moving them to CCC.
Bunyan logging added. Fluent not yet hooked up, but little configuration to do so.
Errors are JSONable.

NB workflow/0 still has some issues that I cannot identify. The format of the final output is not clear to me.

Also, you should set up a linter in your editor if you can.  There is a .eslintrc file in the project that should give us a common styleguide.  I want to add it to a CI check at some point.


Change language RX1R

Hello,
I want to change the language of my camera RX1R  from Japanese to English.
I tried pmca-gui V 0.17 and I have this error:
Get camera info :
Using drivers Windows-MSC, Windows-MTP
Looking for Sony devices
Querying MTP device
Sony Corporation DSC-RX1R is a camera in MTP mode
Model:              DSC-RX1R
Product code:<PHONE_NUMBER>
Serial number:      06016825
Firmware version:   1.00
Start Tweaking :
Querying MTP device
Sony Corporation DSC-RX1R is a camera in MTP mode
Getting device info
Using firmware for model DSC-RX1R
Unknown device: DSC-RX1R
Install App
Switching to app install mode
Traceback (most recent call last):
File "C:\projects\sony-pmca-re\pmca-gui.py", line 76, in do
File "C:\projects\sony-pmca-re\pmca\commands\usb.py", line 274, in installCommand
File "C:\projects\sony-pmca-re\pmca\commands\usb.py", line 37, in switchToAppInstaller
File "C:\projects\sony-pmca-re\pmca\usb\sony.py", line 306, in switchToAppInstaller
File "C:\projects\sony-pmca-re\pmca\usb\sony.py", line 244, in sendCommand
File "C:\projects\sony-pmca-re\pmca\usb\sony.py", line 98, in sendSonyExtCommand
File "C:\projects\sony-pmca-re\pmca\usb_init.py", line 81, in _checkResponse
pmca.usb.InvalidCommandException: MTP error 0x2006
My computer is with Windows 10, I have try with a virtual machine with windows 7, it does not work. Same error MTP 0X2006
I have changed my cables. But I cant update.
Is it possible to update my camera? has anyone the same problem.
Thanks

Problem solved
With the last version https://ci.appveyor.com/project/ma1co/sony-pmca-re/build/artifacts
My RX1R is supported.
And now in French!!!
Thanks

please let us know how did you do it  rx1r from japanese to english
step by step please


Bad grammar in react.dev tictactoe documentation.

I know this is very minor, but I'm just starting to learn react and I've noticed that on the tictactoe site there's bad grammar near the section about:
"
Here is why this doesn’t work. The handleClick(0) call will be a part of rendering the board component. Because handleClick(0) alters the state of the board component by calling setSquares, your entire board component will be re-rendered again. But this runs handleClick(0) again, leading to an infinite loop:```
Console
Too many re-renders. React limits the number of renders to prevent an infinite loop."
Under that's there's a text that says "You could fix by creating a function like handleFirstSquareClick that calls handleClick(0)"
And honestly saying "You could fix by creating a function..." doesn't sound too good. It should instead be something like:
"You could fix THIS by creating a function like..."
I know this is a very small issue but I still wanted to point it out ^^

ok , i will correct this

@Vinqueire please check this #6126

Makes sense, merged the fix!


Update content on supplier details page

Some suppliers didn't realise the gravity of changing these details.
This updated content should help to clarify.

Is there a functional test PR to go along with this? Might make sense to review them together?

Yep, I'm putting it together now, about to push


Update Dependencies and Refactor Code for zip Crate Breaking Changes

Pull Request: Update Dependencies and Refactor Code for zip Crate Breaking Changes
This pull request updates the project's dependencies to their latest versions, addressing breaking changes introduced by the zip crate. Additionally, the codebase has been refactored to accommodate these changes and align with current Rust practices.
Key Changes

Dependency Updates

hashbrown: 0.14.5 → 0.15.2
quick-xml: 0.36.2 → 0.37.1
zip: 1.1.4 → 2.2.1

Refactoring

The WriterManager struct and related methods have been updated to work with a mutable reference to ZipWriter instead of consuming it directly. This improves flexibility and reduces unnecessary ownership overhead.
Adjusted Cells logic to replace insert_unique_unchecked with HashMap::insert, improving safety and aligning with standard practices.

Added resolver = "2"

The new dependency resolver was enabled in Cargo.toml to resolve compatibility issues caused by dependency version conflicts.

Known Issue: wasm-bindgen Bug Prevents Build
Currently, the code does not build due to a critical bug introduced in the wasm-bindgen crate. The issue stems from a dependency chain as shown below:
wasm-bindgen-macro v0.2.97 (proc-macro)
└── wasm-bindgen v0.2.97
    ├── iana-time-zone v0.1.61
    │   └── chrono v0.4.38
    │       └── umya-spreadsheet v2.1.2
    ├── js-sys v0.3.74
    │   └── iana-time-zone v0.1.61 (*)
    ├── rav1e v0.7.1
    │   └── ravif v0.11.11
    │       └── image v0.25.5
    │           └── umya-spreadsheet v2.1.2
    └── v_frame v0.3.8
        ├── av1-grain v0.2.3
        │   └── rav1e v0.7.1 (*)
        └── rav1e v0.7.1 (*)

To fix this issue manually, the following change is required:
diff --git a/Cargo.toml b/Cargo.toml
index db40a10..ff7756f 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -7,6 +7,7 @@ keywords = ["excel", "spreadsheet", "xlsx", "reader", "writer"]
 license = "MIT"
 readme = "README.md"
 description = "umya-spreadsheet is a library written in pure Rust to read and write xlsx file."
+resolver = "2"

 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

Additional Notes
This wasm-bindgen bug also affects the main branch as it stands, so enabling resolver = "2" is necessary for both the main branch and this PR.
Once the upstream issue in wasm-bindgen is resolved, this branch should build correctly without further intervention.

The internal refactoring was necessary because the finish() function in the zip crate was updated to consume the ZipWriter, as introduced in this commit. To accommodate this change, WriterManager now holds a mutable reference to the ZipWriter instead of consuming it directly.

The additional commit db119108d53124c92fbca69963d11bfc2746d979 removes the hashbrown dependency completely.
Reason:

Since Rust 1.36, this is now the HashMap implementation for the Rust standard library. However you may still want to use this crate instead since it works in environments without std, such as embedded systems and kernels.

Source: https://github.com/rust-lang/hashbrown

@mxsrm
Thank you for the PR.
We have no problem with the fixes and will merge them.
We also found out about the problem with wasm-bindgen.
If it has not been fixed by the time the new version of umya-spreadsheet is released, we will address it with resolver = “2”.


add 'subdir-objects' to AM_INIT_AUTOMAKE, quiet warnings about having…

… source / object files in subdirectories.

tanked on travis. will reopen after root-causing and figuring out next steps.

AFAIK, you won't be able to fix it. It was a bug in automake, which is why we eventually ended up with keeping the annoying warning.


maintain: Chapter-2-2

Resolve #534

This change is

@enitrat  tell me what you think of the ''Compound'' section, with references for arrays and dict , to have a clear chapter with :

1 -> Scalar types
2 -> Compound types

I didn't really know the difference, so after a quick search:
The difference between a compound type and a collection in programming lies in their structure and intended use:

Compound Type: A compound type, also known as a composite type, is a data type that is composed of two or more primitive or other compound types. These types are structured and have a fixed schema. Examples include:

Structures (Structs): Combine different types of elements (e.g., integers, strings) into a single entity. Each element of a struct is called a field.
Arrays: Hold a fixed number of elements of the same type and are indexed.
Tuples: Contain a fixed set of elements that may be of different types. Each element in a tuple can be of a different type, and the tuple has a fixed size.
Classes (in object-oriented programming): Define objects that encapsulate data and behavior together. A class is a blueprint for creating instances (objects), each with its own state and behavior.

Compound types are typically used to represent more complex data structures and are integral to the data modeling aspect of programming.

Collection: A collection, on the other hand, is a more general term that refers to any grouping of multiple items, often referred to as elements or members. Collections can grow or shrink dynamically and are primarily used to store, retrieve, and manipulate data. Common types of collections include:

Lists: Ordered collections that can contain duplicate elements and whose size can change dynamically.
Sets: Unordered collections that do not contain duplicate elements.
Maps or Dictionaries: Collections of key-value pairs, where each key is unique and is used to retrieve its corresponding value.
Queues: Collections that operate on a first-in, first-out (FIFO) principle.
Stacks: Collections that operate on a last-in, first-out (LIFO) principle.

Collections are more about managing groups of data and providing mechanisms for dynamically handling these groups (like adding, removing, and searching elements).

In summary, compound types are structured and more rigid in their composition, combining multiple data types into one, while collections are more about handling groups of data, often with flexible sizes and dynamic capabilities. Compound types are a way to structure and relate data, whereas collections are a way to manage and manipulate groups of data.
From this definition:

A span is a sort of compound type

A tuple is a compound type

An array is a collection (a mutable data structure, non-fixed size of elements)

A dictionnary is a collection

I think there is a plan to support later on const arrays which would be basically arrays of constant size known and declared at compile time - which would fit in the compound type section.
So, TLDR: let's keep only tuples there for now.

very clear, thanks for this.
will modify and go back to tuple only then

@enitrat  updated


Execution count logging should be fixed

I tried to apply all the steps and run but the count in logging didn't increase, then I found out that changing {{Count:#,0}} to {Count:#,0} solves the problem. Not a big deal but could be confusing for some people.
private void DoWork(object? state)
    {
        int count = Interlocked.Increment(ref _executionCount);

        _logger.LogInformation(
            "{Service} is working, execution count: {Count:#,0}", //it was {{Count:#,0}} before
            nameof(TimerHostedService),
            count);
    }

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

ID: 8b247969-9dc8-e710-ca64-745596cc3082
Version Independent ID: 483cf588-2b7b-73cd-f9f9-a231a9c8dd7f
Content: Implement the IHostedService interface
Content Source: docs/core/extensions/timer-service.md
Product: dotnet-fundamentals
GitHub Login: @IEvangelist
Microsoft Alias: dapine

Hi 👋 @umutcankarakas
Thank you so much for posting this issue. We really appreciate it. I will work on fixing this issue right away.


Add contributors documentation

I probably should have one... This should have criteria

Welcoming to encourage creating issues, pull requests
Informative about ways of contribution i.e. test driven development, issue summary, etc...
Assertive about behavior and communication

Resolved.


Model shows value without country code

Hi,
When I enter value model does not contain the country code.

It can be corrected by adding the mode="international" probe.  In this case, the phone number will always be converted to +123 123 123...
<vue-tel-input               mode="international"               :enabledCountryCode="true"               id="phone"               inputClasses="form-control"               v-model="data.phone"               :validCharactersOnly="true"             >

Thanks @batuhanoztrk!

Thanks @batuhanoztrk!


Add workout images

Yo it's been half an hour where they at???

Progress update please??? Hello??

stfu

Unprofessional will be reporting to HR!

Replaced with youtube video functionality


ICloneable2

Is the interface ICloneable2 meant to be public? Even if it is meant to be public, could someone explain its usefulness to me?
Screenshot

No, it is not. Somehow, I never noticed.

Does that answer your question?

It answers my question. Thanks.


Conditions for SelenideProxy

Hello and a big respect to everyone involved in creating Selenide.
There has long been a need to write a convenient wrapper for SelenideProxy, which would allow waiting for and checking the content of requests and responses in the browser in a concise form. I would like to see something like the following:
getSelenideProxy()
.responses()
.filter(contain(post("/payment/submit")))
.first()
.should(haveStatusCode(200))
.should(haveHeader("Content-Type", "application/json"))
.should(containBody("expected content"))
.withTimeout(ofSeconds(13));

OR
getSelenideProxy().responses()
  .should(contain(post("/payment/submit")
  .should(haveStatusCode(200).withTimeout(ofSeconds(13));

Good idea.


sketch -L doesn't work

The -L parameter for the sketch subcommand does not work as advertised.  I get:
sketch: invalid option -- 'L'

Followed by usage message.  If I replace -L with the long version of the parameter, --sketch-size-l2, it works fine.  I took a brief look at the code and found the getopt struct for this option but didn't immediately see what might be wrong with it.

Found the problem - missing "L" in the getopt string. I've prepared its fix with a CLI sanity check (making sure there are no unexpected flags)


Bot Builder SDK v3.5?

Is ex_microsoftbot currently compatible with the authentication changes made for botV3.5? Just wondering since the botframework web site claims bots won't work after 07/31/2017 if not compatible.
https://blog.botframework.com/2017/05/07/Bot-Framework-Auth-Change/
https://docs.microsoft.com/en-us/bot-framework/rest-api/bot-framework-rest-connector-authentication
Also, thanks for the very useful module.

@fooqri The library hasn't been updated to support the changes from botV3.5. Looking at the docs there seems to be few changes but the good thing is that it still is using JWT so I believe the changes involved shouldn't be huge.
I currently am not able to make the changes but feel free to send a pull request if you make the changes to support it :). Thanks.

Thanks for the response. I will look into it next week. If you have any additional experiences/info/thoughts on what needs to change or gotchas, it might speed the process and would be appreciated.


[WIP] feat: Self-serve Manage Form Ownership

Summary | Résumé
Self-serve Manage Form Ownership

Access from Responses page
Manage form access dialog
Invite form

Re-invite
Cancel invitation

Remove existing owner
Adding invitees + validation

Accept or decline an invitation

Test instructions
To come...

Fixes #3127

superceded by: https://github.com/cds-snc/platform-forms-client/pull/4386 and https://github.com/cds-snc/platform-forms-client/pull/4392

superceded by: https://github.com/cds-snc/platform-forms-client/pull/4386 and https://github.com/cds-snc/platform-forms-client/pull/4392


Is there a complete installation tutorial for MAC?

Is there a complete installation tutorial for MAC?
The commands in README file don't work, I believe it's for Linux and I only found the link for Windows.
And to install on an online server, which one do you recommend?

No, there is not. You can use one of AI tools to get help on that.
And to install on an online server, which one do you recommend?
* I don't have any recommendation. You should do your own research.


Use matplotlib widget backend instead of notebook

The notebook backend doesn't work on JupyterLab.

Should this be closed/postponed, since we don't have Lab for now?

Yes, postponed


Patch/rc 131 dep vsets

closes: #119
resolves: https://github.com/ehn-dcc-development/ehn-dcc-valuesets/issues/5
Removes valuesets from this repo. This will become the release/1.3.1 branch.
I updated the DCC JSON Schema version to 1.3.1 (and corresponding README.md) although this is debatable, which is why I did it in a separate commit.
In removing the valuesets from this repo (https://github.com/ehn-dcc-development/ehn-dcc-schema), there is absolutely NO change to the DCC JSON schema. However, we need a change in this DCC Schema repo to finally remove the valuesets from this repo (where they should never have been in the first place!). The default view for this repo is "latest release". Thus I will generate a new branch release/1.3.1 where the valueset files are no longer present and which can become the default view.
However, the JSON Schema version could stay (technically) at 1.3.0 since there is no change to the JSON Schema per se - but then we would have JSON Schema v1.3.0 for both release/1.3.0 and release/1.3.1 branches. So I updated the JSON Schema version to 1.3.1 to match the release branches. Thus also why I did this as a separate commit since you could argue that the JSON Schema version should remain unchanged.

@ryanbnl Could you update the tests? We have a choice:

move the valuesets to the test folder (shift one level down), or
(better) refer to the "real" valuesets @ https://github.com/ehn-dcc-development/ehn-dcc-valuesets

Obvs (2) is better since we then a SPOT for the valuesets, but I'll leave the final decision to you.

@ryanbnl Could you update the tests? We have a choice:

Didn't happen after a week, so did it myself.
See latest commit for Makefile change to ensure valuesets from remote repo ehn-dcc-valuesets.git are available for tests

some "scribble notes" off the top of my head:
See the Prerequisites section in the README.md, but here sth of a
summary for a debian/ubuntu etc family Linux distro you would probably
want (if you don't already have them):
/sudo apt install build-essential/
/sudo apt install nodejs (which is almost sickeningly massive)/
from that you can run at a cmd prompt:
/make install-ajv (if you don't have ajv already)/
and then e.g.
/make value_sets_current_latest/
which should get them locally for you, and
/make validate-examples/
will ensure that the examples run with the valuesets etc
This gives you at least a functional check (which the github checks
already do, BTW) and some confidence in the changes even if you don't
feel up to analyzing the Makefile changes themselves.
HTH,
Gaby
Op 02-08-2021 om 14:51 schreef Hynek:

@.**** approved this pull request.
I agree with the way how @gabywh https://github.com/gabywh resolved
the problem with dual VS location. However, I was not able to verify
chnges of the makefile (sorry, not my domain).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ehn-dcc-development/ehn-dcc-schema/pull/120#pullrequestreview-720166129,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASHCPHMCKB3BZ4LHDYKG64LT22IF3ANCNFSM5A3BTIOA.

Would be good to get this merged in if possible. The outdated value sets in this repo have caused some confusion amongst colleagues. Makes sense to remove them from here to avoid any further confusion.
Thanks

The value sets here should have been removed a long time ago. I already
have a pr for 1.3.1 which I did last July (!) but since end July l am no
longer working on this. Gaby.
Op do 24 feb. 2022 12:55 schreef andrewkay18981 @.***>:

Would be good to get this merged in if possible. The outdated value sets
in this repo have caused some confusion amongst colleagues. Makes sense to
remove them from here to avoid any further confusion.
Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/ehn-dcc-development/ehn-dcc-schema/pull/120#issuecomment-1049784422,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASHCPHKZPHN3RBPAKE6QHQLU4YMEPANCNFSM5A3BTIOA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

@dslmeinte could you approve this PR and finalize it?

@dslmeinte could you approve this PR and finalize it?

@kruzikh we've released a 1.3.1 in the meantime, so we need to rebase this and bump to 1.3.2. Will get on it, and clean up the other open PRs whilst I'm at it.

@kruzikh we've released a 1.3.1 in the meantime, so we need to rebase this and bump to 1.3.2. Will get on it, and clean up the other open PRs whilst I'm at it.

@kruzikh we've released a 1.3.1 in the meantime, so we need to rebase this and bump to 1.3.2. Will get on it, and clean up the other open PRs whilst I'm at it.

** NOTE ** base branch should be ** MAIN **
The policy is upstream first https://docs.gitlab.com/ee/topics/gitlab_flow.htmlhttps://docs.gitlab.com/ee/topics/gitlab_flow.html  and e.g. https://www.chromium.org/chromium-os/chromiumos-design-docs/upstream-first/ :
Merging into main and then cherry-picking into release is called an “upstream first” policy, which is also practiced by [Google](https://www.chromium.org/chromium-os/chromiumos-design-docs/upstream-first) and [Red Hat](https://www.redhat.com/en/blog/a-community-for-using-openstack-with-red-hat-rdo).

The changes for the now released 1.3.1 should all already be on main after I had to fix the branches
You should keep all changes on main and then create a branch from main when you are ready to make a new release


Laser Cut Outlines Overlap

First of all - THANK YOU! This project is just what I needed, so cool.
Currently exporting the default object as SVG causes overlap of the arranged slices, like so (look at the left):

This isn't really a problem because you can move them around in Inkscape or whatever, but it'd be nice if this just didn't overlap the layers - even better, if you could supply a ratio/bed size and fill to that to minimize sheets needed!

This problem seems to happen mostly with the small bits.

You've really done a super job with this software, I'm very impressed.

thanks! this shouldn't be too hard to fix.

found and fixed.  please let me know if it works for you.  you will have to reload the page. hovering your mouse over the "help" button should reveal a version of "1.0.67" or greater.  if not, you're getting a cached/old copy.

Wow, that was fast! You are awesome!


Icon issue

If we are going to upload image then video and image button are not visible.
Similar to all other cases.
I expect all the button should hide when we are uploading image/video/audio.

I kept in a  way,  lets say user selected an image so without uploading he/she can also have a option for audio or video at same time.
I will change it if everyone does not like it.

I kept it in such a way, lets say user selected an image so without uploading he/she can also have a option for audio or video at same time. I will change it if everyone does not like it.

I suggest shifting the discussion to the GD to determine the percentage of people who are in favor and those who are against the proposed idea. If the majority are against it, then I am open to making changes.


use small lock in following file

Summary
use small lock in following file
arch/or1k/src/mor1kx/mor1kx_serial.c
arch/risc-v/src/bl602/bl602_serial.c
arch/risc-v/src/common/espressif/esp_lowputc.c
arch/risc-v/src/common/espressif/esp_lowputc.h
arch/risc-v/src/common/espressif/esp_tickless.c
arch/risc-v/src/esp32c3-legacy/esp32c3_idle.c
arch/risc-v/src/esp32c3-legacy/esp32c3_lowputc.c
arch/risc-v/src/esp32c3-legacy/esp32c3_lowputc.h
arch/risc-v/src/esp32c3-legacy/esp32c3_rtc_lowerhalf.c arch/risc-v/src/fe310/fe310_gpio.c
Impact
arch/or1k/src/mor1kx/mor1kx_serial.c
arch/risc-v/src/bl602/bl602_serial.c
arch/risc-v/src/common/espressif/esp_lowputc.c
arch/risc-v/src/common/espressif/esp_lowputc.h
arch/risc-v/src/common/espressif/esp_tickless.c
arch/risc-v/src/esp32c3-legacy/esp32c3_idle.c
arch/risc-v/src/esp32c3-legacy/esp32c3_lowputc.c
arch/risc-v/src/esp32c3-legacy/esp32c3_lowputc.h
arch/risc-v/src/esp32c3-legacy/esp32c3_rtc_lowerhalf.c arch/risc-v/src/fe310/fe310_gpio.c
Testing
ci

[Experimental Bot, please feedback here]
This PR description does not adequately meet the NuttX requirements. Here's why and how to fix it:
Summary Deficiencies:

Missing "Why":  It states what was changed (using smaller locks) but not why.  Was this for performance?  Deadlock avoidance?  Reduced memory usage? This is critical information.
Missing "How exactly":  Saying "use small lock" is vague.  What kind of lock was used before? What kind of lock is used now?  Were spinlocks replaced with mutexes? Was a custom lock implementation introduced?  Provide specifics.
Missing Related Issues:  If this addresses a reported bug or implements a requested feature, link the relevant issue(s).

Impact Deficiencies:

Repeating file list is unhelpful:  Listing the affected files again doesn't explain the impact of the changes.  Address each point individually:

New Feature? Existing Feature Changed?:  Specify which.  Likely an existing feature changed (serial driver, tickless operation, GPIO).
Impact on User:  Will applications behave differently?  Likely no, but state that explicitly.
Impact on Build:  Are there new configuration options?  Probably not, but confirm.
Impact on Hardware: Does this affect specific architectures, boards, or drivers? Yes, specifically the listed architectures. Mention them clearly.
Impact on Documentation: Does the documentation need updating to reflect these changes?  If not, say so explicitly. If yes, has the documentation been updated in this PR?
Impact on Security:  Are there any potential security implications from changing the locking mechanisms? (e.g., potential for race conditions if not done correctly). Carefully consider this.
Impact on Compatibility:  Does this break backward compatibility?  Does it affect compatibility between different NuttX configurations?

"Anything else to consider?":  This is for unusual edge cases.  If none, you can omit this or state "None."

Testing Deficiencies:

"ci" is insufficient:  Just stating "ci" doesn't demonstrate testing.  Provide:

Detailed build host information: OS version, compiler version (including minor version), etc.
Detailed target information:  Specific architectures and boards tested (e.g., sim:nsh, riscv:bl602-iot-sdk, etc.).
Relevant log snippets: Don't just dump entire logs.  Show specific before-and-after logs that demonstrate the issue being fixed or the new functionality working.  Focus on the relevant parts.  What was the problem before? How is the behavior different now?

Example of an Improved Summary:
This PR replaces mutexes with spinlocks in the serial driver, tickless timer, and GPIO driver for the OR1K, BL602, FE310, and ESP32C3 architectures. This change aims to improve performance by reducing the overhead of mutex acquisition and release in these frequently accessed code paths.  This addresses the performance regression reported in [Issue #XYZ].
By providing specific details, you make it much easier for reviewers to understand and evaluate your changes, which leads to faster reviews and a higher likelihood of your PR being merged.

@hujun260 please verify this error:
Configuration/Tool: esp32c3-devkit/ble
2024-12-20 14:23:20
------------------------------------------------------------------------------------
  Cleaning...
  Configuring...
  Building NuttX...
chip/esp32c3_lowputc.c: In function 'esp32c3_lowputc_disable_all_uart_int':
Error: chip/esp32c3_lowputc.c:718:29: error: passing argument 1 of 'spin_lock_irqsave' discards 'const' qualifier from pointer target type [-Werror=discarded-qualifiers]
  718 |   flags = spin_lock_irqsave(&priv->lock);
      |                             ^~~~~~~~~~~
In file included from chip/esp32c3_lowputc.c:30:
/github/workspace/sources/nuttx/include/nuttx/spinlock.h:511:55: note: expected 'volatile spinlock_t *' {aka 'volatile unsigned int *'} but argument is of type 'const spinlock_t *' {aka 'const unsigned int *'}
  511 | irqstate_t spin_lock_irqsave(FAR volatile spinlock_t *lock)
      |                                  ~~~~~~~~~~~~~~~~~~~~~^~~~
Error: chip/esp32c3_lowputc.c:735:26: error: passing argument 1 of 'spin_unlock_irqrestore' discards 'const' qualifier from pointer target type [-Werror=discarded-qualifiers]
  735 |   spin_unlock_irqrestore(&priv->lock, flags);
      |                          ^~~~~~~~~~~
/github/workspace/sources/nuttx/include/nuttx/spinlock.h:674:54: note: expected 'volatile spinlock_t *' {aka 'volatile unsigned int *'} but argument is of type 'const spinlock_t *' {aka 'const unsigned int *'}
  674 | void spin_unlock_irqrestore(FAR volatile spinlock_t *lock,
      |                                 ~~~~~~~~~~~~~~~~~~~~~^~~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:144: esp32c3_lowputc.o] Error 1
make[1]: Target 'libarch.a' not remade because of errors.
make: *** [tools/LibTargets.mk:170: arch/risc-v/src/libarch.a] Error 2
make: Target 'all' not remade because of errors.

@hujun260 your recent modifications broken the CI (stm32f4discovery), see LVGL PR from @JorgeGzm
I think RISC-V also was affected:
Configuration/Tool: esp32c3-generic/tickless
2024-12-20 13:56:24
------------------------------------------------------------------------------------
  Cleaning...
  Configuring...
  Building NuttX...
common/espressif/esp_tickless.c: In function 'up_timer_start':
Error: common/espressif/esp_tickless.c:359:23: error: passing argument 1 of 'up_timer_cancel' from incompatible pointer type [-Werror=incompatible-pointer-types]
  359 |       up_timer_cancel(&g_esp_tickless_lock);
      |                       ^~~~~~~~~~~~~~~~~~~~
      |                       |
      |                       spinlock_t * {aka unsigned char *}
common/espressif/esp_tickless.c:268:48: note: expected 'struct timespec *' but argument is of type 'spinlock_t *' {aka 'unsigned char *'}
  268 | int IRAM_ATTR up_timer_cancel(struct timespec *ts)
      |                               ~~~~~~~~~~~~~~~~~^~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:144: esp_tickless.o] Error 1
make[1]: Target 'libarch.a' not remade because of errors.
make: *** [tools/LibTargets.mk:170: arch/risc-v/src/libarch.a] Error 2
make: Target 'all' not remade because of errors.
/github/workspace/sources/nuttx/tools/testbuild.sh: line 385: /github/workspace/sources/nuttx/../nuttx/nuttx.manifest: No such file or directory
  Normalize esp32c3-generic/tickless

@hujun260 please verify this error:
Configuration/Tool: esp32c3-devkit/ble
2024-12-20 14:23:20
------------------------------------------------------------------------------------
  Cleaning...
  Configuring...
  Building NuttX...
chip/esp32c3_lowputc.c: In function 'esp32c3_lowputc_disable_all_uart_int':
Error: chip/esp32c3_lowputc.c:718:29: error: passing argument 1 of 'spin_lock_irqsave' discards 'const' qualifier from pointer target type [-Werror=discarded-qualifiers]
  718 |   flags = spin_lock_irqsave(&priv->lock);
      |                             ^~~~~~~~~~~
In file included from chip/esp32c3_lowputc.c:30:
/github/workspace/sources/nuttx/include/nuttx/spinlock.h:511:55: note: expected 'volatile spinlock_t *' {aka 'volatile unsigned int *'} but argument is of type 'const spinlock_t *' {aka 'const unsigned int *'}
  511 | irqstate_t spin_lock_irqsave(FAR volatile spinlock_t *lock)
      |                                  ~~~~~~~~~~~~~~~~~~~~~^~~~
Error: chip/esp32c3_lowputc.c:735:26: error: passing argument 1 of 'spin_unlock_irqrestore' discards 'const' qualifier from pointer target type [-Werror=discarded-qualifiers]
  735 |   spin_unlock_irqrestore(&priv->lock, flags);
      |                          ^~~~~~~~~~~
/github/workspace/sources/nuttx/include/nuttx/spinlock.h:674:54: note: expected 'volatile spinlock_t *' {aka 'volatile unsigned int *'} but argument is of type 'const spinlock_t *' {aka 'const unsigned int *'}
  674 | void spin_unlock_irqrestore(FAR volatile spinlock_t *lock,
      |                                 ~~~~~~~~~~~~~~~~~~~~~^~~~
cc1: all warnings being treated as errors
make[1]: *** [Makefile:144: esp32c3_lowputc.o] Error 1
make[1]: Target 'libarch.a' not remade because of errors.
make: *** [tools/LibTargets.mk:170: arch/risc-v/src/libarch.a] Error 2
make: Target 'all' not remade because of errors.

done

@hujun260 I'm having similar issue on esp32c3-generic on this commit.
Could you please take a look?

./tools/configure.sh esp32c3-generic:nsh
Set CONFIG_SPINLOCK
Make

common/espressif/esp_lowputc.c: In function 'esp_lowputc_disable_all_uart_int':
common/espressif/esp_lowputc.c:215:29: error: passing argument 1 of 'spin_lock_irqsave' discards 'const' qualifier from pointer target type [-Werror=discarded-qualifiers]
  215 |   flags = spin_lock_irqsave(&priv->lock);
      |                             ^~~~~~~~~~~
In file included from common/espressif/esp_lowputc.c:40:
/home/fdcavalcanti/nuttxspace1/nuttx/include/nuttx/spinlock.h:511:55: note: expected 'volatile spinlock_t *' {aka 'volatile unsigned int *'} but argument is of type 'const spinlock_t *' {aka 'const unsigned int *'}
  511 | irqstate_t spin_lock_irqsave(FAR volatile spinlock_t *lock)
      |                                  ~~~~~~~~~~~~~~~~~~~~~^~~~
common/espressif/esp_lowputc.c:232:26: error: passing argument 1 of 'spin_unlock_irqrestore' discards 'const' qualifier from pointer target type [-Werror=discarded-qualifiers]
  232 |   spin_unlock_irqrestore(&priv->lock, flags);
      |                          ^~~~~~~~~~~
/home/fdcavalcanti/nuttxspace1/nuttx/include/nuttx/spinlock.h:674:54: note: expected 'volatile spinlock_t *' {aka 'volatile unsigned int *'} but argument is of type 'const spinlock_t *' {aka 'const unsigned int *'}
  674 | void spin_unlock_irqrestore(FAR volatile spinlock_t *lock,
      |                                 ~~~~~~~~~~~~~~~~~~~~~^~~~
IN: boards/libboards.a -> staging/libboards.a cc1: all warnings being treated as errors
make[1]: *** [Makefile:144: esp_lowputc.o] Error 1
make[1]: *** Waiting for unfinished jobs....
CC:  misc/reboot_notifier.c make: *** [tools/LibTargets.mk:170: arch/risc-v/src/libarch.a] Error 2
make: *** Waiting for unfinished jobs....

common/espressif/esp_lowputc.c: In function 'esp_lowputc_disable_all_uart_int':
common/espressif/esp_lowputc.c:215:29: error: passing argument 1 of 'spin_lock_irqsave' discards 'const' qualifier from pointer target type [-Werror=discarded-qualifiers]
215 |   flags = spin_lock_irqsave(&priv->lock);
|                             ^~~~~~~~~~~
In file included from common/espressif/esp_lowputc.c:40:
/home/fdcavalcanti/nuttxspace1/nuttx/include/nuttx/spinlock.h:511:55: note: expected 'volatile spinlock_t *' {aka 'volatile unsigned int *'} but argument is of type 'const spinlock_t *' {aka 'const unsigned int *'}
511 | irqstate_t spin_lock_irqsave(FAR volatile spinlock_t *lock)
|                                  ~~~~~~~~~^~
common/espressif/esp_lowputc.c:232:26: error: passing argument 1 of 'spin_unlock_irqrestore' discards 'const' qualifier from pointer target type [-Werror=discarded-qualifiers]
232 |   spin_unlock_irqrestore(&priv->lock, flags);
|                          ^
/home/fdcavalcanti/nuttxspace1/nuttx/include/nuttx/spinlock.h:674:54: note: expected 'volatile spinlock_t *' {aka 'volatile unsigned int *'} but argument is of type 'const spinlock_t *' {aka 'const unsigned int *'}
674 | void spin_unlock_irqrestore(FAR volatile spinlock_t *lock,
|                                 ~~~~~~~~~~~~~~~~~~~^~
IN: boards/libboards.a -> staging/libboards.a cc1: all warnings being treated as errors
make[1]: *** [Makefile:144: esp_lowputc.o] Error 1
make[1]: *** Waiting for unfinished jobs....
CC:  misc/reboot_notifier.c make: *** [tools/LibTargets.mk:170: arch/risc-v/src/libarch.a] Error 2
make: *** Waiting for unfinished jobs....

https://github.com/apache/nuttx/pull/15350


Export custom normals correctly

Use custom normals if available during export

Thanks a lot for fixing this. Just gave it a quick try and it fixes everything that currently forces us to use Maya to export our stuff (coming from Blender via FBX because of the vertex normals).

@SteffenDuenner fantastic! Always happy when another Mayan bites the dust and converts to our beloved Blender 😄

Hahaaaaa! The funny thing is: I'm not even a "Mayan", nothing could be further away from truth. I'm a Softimage refugee, and also I'm a long term Blender afficionado, I was only forced to use Maya in a current project just because of the faulty vertex normal export. These times are now over thanks to you! :D

@SteffenDuenner Ahhh, XSI - definitely one of the best commercial 3D software. I actually also used Maya a lot, back in the day, until I took the jump to Blender full on.


Verifying Power Factor values

I believe with the Microchip IC used, all ADC values are captured simultaneously. This would imply that power factor calibration is largely unnecessary unless the AC-AC transformer is introducing some phase delay. I have a very low power factor seemingly correlated to the output of my PV microinverters. It even is close to zero for nearly 30 minutes.
Has anyone else reported similar experience with solar inverters?
What has been your experience with phase shift settings? Suggestions for calibration/validation?

Some questions...

Are you measuring the use from your PV system?
Do you have a split single phase electrical system?
If so, are you measuring both phases?

It makes sense that PF for mains would go negative if you're exporting power to the grid. When it's at 0 you aren't importing or exporting power - only the PV power is being used. PF slowly increases as you use less PV power, and more grid power.

Seems like everything is working as intended then.
I am not measuring the PV system yet (I have the second board, I just haven't installed it yet)
Yes, I have a split single phase 240v 60hz (North American houshold) with 200A service.
Yes, I am measuring both legs of the split phase.
I understand that the PF would go negative on "export" of PV power, but I don't follow why the PF gradually transitions through 0 as the PV generation goes down. Sounds like I need to do more research.

A bit of research showcases why the Power Factor drifts through 0 as PV generation goes up. The key is the PF is the relationship between real and apparent power. Changing the "real" power value with a device with a PF near 1 (i.e. water heater) or -1 (PV inverter) can significantly impact PF without any affect on apparent power.
If apparent power stays the same (is not corrected) and real power is reduced (either by PV generation with a PF of 1 or by unplugging a device which is consuming real power with a PF of 1), the ratio of real to apparent power changes, reducing power factor. The triangle diagram vs the "phase" presentation helped me understand it.

https://www.sma-sunny.com/en/bad-power-factor-a-reason-to-oversize-your-inverter/
While there are PV inverters which can also actively correct PF, I do not have a model which supports that function.

Thanks for posting that! Very good explanation.


Should we add default /api/ prefix to all url's

Or as a configurable prefix

In our project we don't use /api/ prefix, because the domain name is the API access. But a feature like this would be great, for API versioning, like https://example.com/v1/endpoint.

I like the idea of versioning, but wouldn't that be something to put on your controller with ie an attribute? Something like this:
[RestController(Version:=1)]

I still believe that a separate prefix will be useful.

I think an attribute on controller makes it very difficult. When you have a bigger API, you have many Controllers with possibly different versions. In practice you mark with an API version update the complete API as a new version. So I think a prefix is very useful. Many big API player use this kind of versioning. By the way: On apigee you can find a short eBook with best practices for API Design.

But if you have controllers with different versions in one project, how would the prefix help? If you want to store the version there, you will only have one version number for the complete service. Cause my idea was to have a prefix for the entire service, not a prefix for one controller.

Prefix is available on the constructor of RestWebServer


[SPARK-37937][SQL] Use error classes in the parsing errors of lateral join

What changes were proposed in this pull request?

In the PR, I propose to use the following error classes for the parsing errors of lateral joins:
INVALID_LATERAL_JOIN_RELATION, LATERAL_JOIN_WITH_NATURAL_JOIN_UNSUPPORTED, LATERAL_JOIN_WITH_USING_JOIN_UNSUPPORTED, and UNSUPPORTED_LATERAL_JOIN_TYPE
These new error classes are added to error-classes.json.
Why are the changes needed?

Porting the parsing errors for lateral join to the new error framework should improve user experience with Spark SQL.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Added new test suite

cc @MaxGekk @cloud-fan @karenfeng

@imback82 Could you fix the JIRA id in PR's title. Seems SPARK-37858 is not correct.

Updated, thanks!

+1, LGTM. Merging to master.
Thank you, @imback82 and @cloud-fan for review.


Data Fix: Great Club cannot be infused

Tom,
Thank you for putting this tool together, it is super useful!
As far as I know, the Great Club  cannot be infused, but the weapon calculator offers it with infusions. It does upgrade with standard stones which is a bit odd.
Anyway, no urgency coming for me, mostly just wanted to let you know you are appreciated.

Thanks, will take a look

Fix is deployed: https://github.com/ThomasJClark/elden-ring-weapon-calculator/runs/10128640451
The Great Club will no longer show up if you pick affinities other than special


Update http4s-circe, http4s-dsl, ... to 0.22.12

Updates

org.http4s:http4s-circe
org.http4s:http4s-dsl
org.http4s:http4s-ember-client
org.http4s:http4s-ember-server

from 0.22.6 to 0.22.12.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, early-semver-minor, semver-spec-patch, commit-count:1

Superseded by #73.


⚠ Bofrost.lu has degraded performance

In 20c5f3f, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2483 ms

Resolved: Bofrost.lu performance has improved in b24ab2f after 20 minutes.


how to use rsmpeg

macOS m2 install ffmpeg 7.0 by brew: /opt/homebrew/opt/ffmpeg
in project Cargo.toml
rsmpeg = "0.15"

Thank's for reporting.
I am unable to reproduce this issue. Could you try importing rsmpeg like this:
rsmpeg = { git = "https://github.com/larksuite/rsmpeg", branch = "master", features = ["ffmpeg7", "link_system_ffmpeg"] }

Let me know if the issue still persists.


readability enhancement

I am learning webpack by reading documentation. So will propose suggestion if I feel something can improve readability

[x] Read and sign the CLA. PRs that haven't signed it won't be accepted.
[ ] Make sure your PR complies with the writer's guide.
[ ] Review the diff carefully as sometimes this can reveal issues.
[ ] Do not abandon your Pull Request: Stale Pull Requests.
Remove these instructions from your PR as they are for your eyes only.

Thanks but I don't think those changes are right or necessary.


Update README

Add questions/suggestions section
Add "About thoughtbot" section

Looks good, merge at your own discretion

ping @thoughtbot/design

LGTM


Error

Unexpected error: 'NoneType' object has no attribute 'get'

Same problem here!

Hey I think I fixed it in the newest version - can pull you check?
Else please provide short code example

The problem only happens to me when I try to use it via litellm, which I'm using the free version of gpt 4o from github, now if I use it directly via the chatgpt API, I don't have this problem.

Hey I think I fixed it in the newest version - can pull you check?
Else please provide short code example

pulled and reinstalled from source still getting this:
python test.py "find me a recipe for pancakes" --provider openai
DevTools listening on ws://<IP_ADDRESS>:62898/devtools/browser/ff0a7099-8614-41fd-b986-e9c07e51dc9e
INFO     [root] Anonymized telemetry enabled. See https://github.com/gregpr07/browser-use for more information.
INFO     [agent] 🚀 Starting task: find me a recipe for pancakes
INFO     [agent]
📍 Step 1
INFO     [agent] 🤷 Evaluation: Unknown
INFO     [agent] 🧠 Memory:
INFO     [agent] 🎯 Next Goal: Search for a pancake recipe online.
INFO     [agent] 🛠 Action: {"search_google":{"query":"pancake recipe"}}
ERROR    [agent] ❌ Result failed 1/5 times:
Unexpected error: 'NoneType' object has no attribute 'get'
INFO     [agent]
📍 Step 2
INFO     [agent] ⚠ Evaluation: Failed: The page is empty and there are no interactive elements to search for a recipe.
INFO     [agent] 🧠 Memory:
INFO     [agent] 🎯 Next Goal: Open a new tab to search for a pancake recipe.
INFO     [agent] 🛠 Action: {"open_tab":{"url":"https://www.google.com"}}
ERROR    [agent] ❌ Result failed 2/5 times:
Unexpected error: 'NoneType' object has no attribute 'get'
INFO     [agent]
📍 Step 3
INFO     [agent] ⚠ Evaluation: Failed: The page is empty and there are no interactive elements to search for a recipe.
INFO     [agent] 🧠 Memory:
INFO     [agent] 🎯 Next Goal: Open a new tab to search for a pancake recipe.
INFO     [agent] 🛠 Action: {"open_tab":{"url":"https://www.google.com"}}
ERROR    [agent] ❌ Result failed 3/5 times:
Unexpected error: 'NoneType' object has no attribute 'get'
INFO     [agent]
📍 Step 4
Created TensorFlow Lite XNNPACK delegate for CPU.
INFO     [agent] ⚠ Evaluation: Failed: The page is empty and there are no interactive elements to search for a recipe.
INFO     [agent] 🧠 Memory:
INFO     [agent] 🎯 Next Goal: Open a new tab to search for a pancake recipe.
INFO     [agent] 🛠 Action: {"open_tab":{"url":"https://www.google.com"}}
ERROR    [agent] ❌ Result failed 4/5 times:
Unexpected error: 'NoneType' object has no attribute 'get'
INFO     [agent]
📍 Step 5
INFO     [agent] ⚠ Evaluation: Failed: The page is empty and there are no interactive elements to search for a recipe.
INFO     [agent] 🧠 Memory:
INFO     [agent] 🎯 Next Goal: Open a new tab to search for a pancake recipe.
INFO     [agent] 🛠 Action: {"open_tab":{"url":"https://www.google.com"}}
ERROR    [agent] ❌ Result failed 5/5 times:
Unexpected error: 'NoneType' object has no attribute 'get'
ERROR    [agent] ❌ Stopping due to 5 consecutive failures

I will fix it this evening.
Quick fix:
comment out self._update_usage_metadata(response['raw']) in get_next_action  let me know if that helps

It works for me. I am using the github gpt-4o. Thanks

trying to run using the github models and I get the same error as above, using the base URL from github
if i use my normal OpenAI API key it works fine

@lucasvnd so you mean you use the exact same code - only different API keys ?

Im using base_url so I can use the free github API.
like
def get_llm(provider: str):
if provider == 'anthropic':
return ChatAnthropic(
model_name='claude-3-5-sonnet-20240620', timeout=25, stop=None, temperature=0.0
)
elif provider == 'openai':
return ChatOpenAI(base_url='', model='gpt-4o-mini', temperature=0.0)
else:
raise ValueError(f'Unsupported provider: {provider}')

I will fix it this evening.
Quick fix:
comment out self._update_usage_metadata(response['raw']) in get_next_action  let me know if that helps

it works for me even I use openai in github

I'm getting 'NoneType' object has no attribute 'action'

same
I'm getting 'NoneType' object has no attribute 'action'


no "notes" section, when there is a s Doc section inherited from a extends_documentation_fragment

Tried out the tool as your recommended some issue is the tool doesn't check the doc_fragments.
All our collection use the extends_documentation_fragment for comment notes, and params
So for instance this module while has no note
https://github.com/ansible-collections/netapp/blob/master/ansible_collections/netapp/cloudmanager/plugins/modules/na_cloudmanager_aggregate.py#L22
Will inherited the note here
https://github.com/ansible-collections/netapp/blob/master/ansible_collections/netapp/cloudmanager/plugins/doc_fragments/netapp.py
When the documentation is built.
You can see this here in the .rst file that ansible-build build-docs will generate.
https://github.com/ansible-collections/netapp/wiki/na_cloudmanager_aggregate

@carchi8py , very good spotted, thanks! will see what i can do


Listed component crashes or throws an error during configuration

After dragging and dropping the component, it responds as expected initially. However, when attempting to configure the component’s properties, the system crashes or throws an error, preventing further configuration.

DataTable: Crashes when adding columns then save
EntityPicker: The crashes occurs after configuring columns then click within the component to select
EntityPicker: Clicking an existing configuration and selecting a value results in an error
Wizard: Crashes when adding steps then save
Tab: Does not crashes, however the tabs configured do not show
Columns: Crashes when adding columns
SizableColumns: Crashes when adding columns
KyInformationBar: Crashes when adding columns
ButtonGroup: Crashes when adding buttons/groups then save
Dropdown: Crashes when adding values then click save
Radio: Crashes when adding values then click save
CheckboxGroup: Crashes when adding values then click save
Chevron: Crashes after specifying the referenceList property
Khaban: Crashes after specifying the referenceList property
When you console something on formSettings: https://functional-adminportal-test.azurewebsites.net/shesha/forms-designer?id=c4a6dec7-5ee3-4d8c-9d6d-b452b05ff56e
DateField: The component doesn't crash, however when you submit the console logs an error of e.d.gettime is not function

Most of the errors are similar

Hi @AlexStepantsov the following issues are still persisting. Let me know if they are not related so I can create a separate card

EntityPicker: Clicking an existing configuration and selecting a value results in an error
DateField: The component doesn't crash, however when you submit the console logs an error of e.d.gettime is not function

Here's a link: https://functional-adminportal-test.azurewebsites.net/dynamic/TestModule/crash-table

Click on the Add dialog
Open your console
Enter the details including tab 3
See the errors logged after selecting address and when submitting


Is gitlab-runner necessary in the worker VM image?

The README says

You should upload a special image with gitlab-runner and container runtime installed in it.

We are using an image for the worker VMs which does not have the gitlab-runner package installed, and don't see any obvious misbehaviour as a result. The GitLab docs https://docs.gitlab.com/runner/executors/docker_autoscaler.html#configure-docker-autoscaler don't mention the requirement either. What situations require gitlab-runner to be installed in the workers? What things should we watch out for if we don't have gitlab-runner in the VM image? Thanks!

One of idea, they probably just pulls runner as docker image too. Older code simply starts runner via SSH. So why not to use container infra for that?
Just as advise try to setup caching registry proxy.

@davebiffuk i had some time to see and test new things. So I can prove that now only container runtime is necessary on worker.
More over i took special container linux, a Flatcar, and recreated whole farm. Gitlab Runner container runs on the manager node, then just connects to remove docker.sock via SSH.
https://github.com/sardinasystems/fleeting-plugin-openstack/tree/main/heat


Installtion is still broken with devscripts directory

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2022.08.14 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
Installation is still broken with devscripts under Ubuntu/Debian
The really best is to change the devscripts name. Why not to changes in dev-scripts ?
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
yt-dlp (1:2022.08.14-dmo1) over (1:2022.08.08-dmo2) ...
dpkg: error processing archive /srv/dak/ftp/pool/main/y/yt-dlp-dmo/yt-dlp_2022.08.14-dmo1_all.deb (--unpack):
 trying to overwrite '/usr/lib/python3/dist-packages/devscripts/__init__.py', which is also in package devscripts 2.22.2
dpkg-deb: error: paste subprocess was killed by signal (Broken pipe)
Errors were encountered while processing:
 /srv/dak/ftp/pool/main/y/yt-dlp-dmo/yt-dlp_2022.08.14-dmo1_all.deb

Ask the package distributor to fix this issue. We don't provide debian packages.

We already have a discusion about this isue in https://github.com/yt-dlp/yt-dlp/issues/4603
Why devscipts files are installed when they are only needed when yt-dlp is installed ?

Ar you currently trying to build the yt-dlp package? devscripts/ must not be contained in production package.

Yes I'm packaging yt-dlp since the beginning.
Then why these files are installed ?

To be sure, the debian package don't install the devscripts directory.
These files are installed by pybuild

Maybe setup.py is declaring itself to include devscripts/...
Can you try this patch then rebuild it?
$ git diff
diff --git a/setup.py b/setup.py
index 03f413081..81a701fe4 100644
--- a/setup.py
+++ b/setup.py
@@ -28,7 +28,7 @@

 def packages():
     if setuptools_available:
-        return find_packages(exclude=('youtube_dl', 'youtube_dlc', 'test', 'ytdlp_plugins'))
+        return find_packages(exclude=('youtube_dl', 'youtube_dlc', 'test', 'ytdlp_plugins', 'devscripts'))

     return [
         'yt_dlp', 'yt_dlp.extractor', 'yt_dlp.downloader', 'yt_dlp.postprocessor', 'yt_dlp.compat',

Thanks, this fix this issue.