Strict size checks for conversions to fixed size containers

The current implementation accepts conversion of JSON arrays to fixed size containers like std::array, std::pair or std::tuple, as long as there are sufficient elements in the JSON array to fill the container.
The conversion only implicitly fails if there are to few elements via the out of range exception when trying to access further elements.
I suggest to strictly check for matching JSON array size when converting to fixed size containers instead.
In my opinion a mismatch in size should be considered a type error, as in the sense, that i.e. a std::array<int, 4> is a different type than a std::array<int, 5>.
Silently discarding extra JSON array elements might hide problems with mismatching serialized data and result in unexpected behavior.
If a fixed size container is extracted, it explicitly expresses that a certain number of elements is expected, if this expectation is not met by the JSON object, throwing a type error seems to be the safest way to handle the situation.
Though according to the test cases, the current behavior seems at least partially intentional, so I guess, this is up for debate, which approach is preferable.
Anyway, the commits in this pull request add strict size checks for the conversion to std::array, std::pair, std::tuple and c-style arrays.
There are also explicit JSON type checks for more expressive exceptions when the JSON type is not an array at all.
Test cases are adapted accordingly and some extra test cases are added to check for the new size restrictions.

Pull request checklist
Read the Contribution Guidelines for detailed information.

[X]  Changes are described in the pull request, or an existing issue is referenced.
[X]  The test suite compiles and runs without error.
[X]  Code coverage is 100%. Test cases can be added by editing the test suite.
[X]  The source code is amalgamated; that is, after making changes to the sources in the include/nlohmann directory, run make amalgamate to create the single-header file single_include/nlohmann/json.hpp. The whole process is described here.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.7 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Specifically, I am aware of compilation problems with Microsoft Visual Studio (there even is an issue label for these kind of bugs). I understand that even in 2016, complete C++11 support isn't there yet. But please also understand that I do not want to drop features or uglify the code just to make Microsoft's sub-standard compiler happy. The past has shown that there are ways to express the functionality such that the code compiles with the most recent MSVC - unfortunately, this is not the main objective of the project.
Please refrain from proposing changes that would break JSON conformance. If you propose a conformant extension of JSON to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

The change will break existing code, so it cannot be introduced in the 3.x.x versions. I would propose to guard the test/throwing code with a preprocessor symbol so that the feature can be switched on if needed. What do you think?

Coverage remained the same at 100.0% when pulling 748f4ec2279ab7acebf17d38e4f42e3013c68021 on domsoll:conversion_checks into a8398a7151df6fd22691afc80fdc0f8af4d63829 on nlohmann:develop.

Thank's for your input, you are right, an introduction in 3.x.x is problematic, I guess the preprocessor symbol is the best option then.
I've added the according preprocessor conditionals and a cmake option to define the symbol which is disabled by default.
The behavior without the strict container size option should now be unchanged, including the type of exceptions thrown in case of errors.
Code coverage is 100% and all test cases pass with and without the option.
I left the extra test cases for c-style arrays active without the new option enabled and extended them to check for the non-strict variant also, I guess that makes sense?

@domsoll Can you please update to the latest develop version to resolve the conflicts?

(Related: #3010)

The pull request is now updated to the latest develop version.

Thanks! Can you please run the amalgamation (make amalgamate)?

I did update amalgamation before with make amalgamate.
Though without astyle/make pretty there is one closing brace placed differently (which the check complained about) and with astyle/make pretty I get a huge diff in the amalgamation and the source files.
For some reason my version of astyle seems to behave differently to what was used on the sources before.
Anyway, I reverted the brace placement manually again, so the check should hopefully pass now.

@domsoll This PR has conflicts again.
@nlohmann Anything else outstanding here?


SAMZA-2354: Improve UDF discovery in samza-sql.

This patch is comprised of following changes:

Currently to plugin a udf implementation into samza-sql engine, users have to set the fully qualified class name of their UDF implementation in samza.sql.udfClasses configuration.  Any user error made in defining the fully-qualified name of udf-class configuration can only be identified  at run-time. This patch addresses this problem by using reflection to find all the samza-sql UDF implementations available in the class-path of the samza-sql application.
Removes the ConfigBasedUdfResolver class and it's associated usages.
Adds  unit-tests for the newly introduced UDFResolver implementation.

@atoomula @srinipunuru
Please take a look when you get a chance.


iOS 13: UITableView in modal presentation conflicts with dismiss gesture

Here is a new one coming with iOS 13 - when a UITableView is being presented as a modal, the index can't be dragged because it causes rubber banding as that modal's dismissal takes priority.
Just starting to digging into this, but it looks like there's going to need to be some sort of callback to hook into to disable that gesture recognizer during MYTVI touches (see: https://stackoverflow.com/questions/56718552/disable-gesture-to-pull-down-form-page-sheet-modal-presentation).

#32 will solve this :)


Late bind XDG_CACHE_HOME and HOME in ~/bin/hermit

When hermit creates ~/bin/hermit, ensure that $XDG_CACHE_HOME and
$HOME are not expanded in it, but rather use those variables. This
allows the user to change XDG_CACHE_HOME/HOME and have hermit adapt to
those changes, and also accommodates different contents of those
variables on a NFS shared home directory or similar.
It also fixes a subtle bug where the installer script downloaded would
run and put hermit in $HERMIT_EXE but then go to run it from a hardcoded
location that may be different to $HERMIT_EXE.

Looking into the integration test failures...

Thanks!


Toomany Connections : 121 connections created while maximumPoolSize=10?

121 connections created while config as below
spring.datasource.hikari.pool-name=DTE HikariConnectionPool
spring.datasource.hikari.connectionTimeout=30000
spring.datasource.hikari.maximumPoolSize=10
spring.datasource.hikari.minimumIdle=8
spring.datasource.hikari.idleTimeout=600000
spring.datasource.hikari.maxLifeTime=1800000
spring.datasource.hikari.autoCommit=true
spring.datasource.validationQuery=SELECT 1
spring.datasource.hikari.leakDetectionThreshold=10000
Please help?
Getting too many connections error while connecting.

Please provide debug logs for the hikari package. Are you sure all connections originate from your application? How many pool instances?

Now out of 130 from application only 18 but localhost root user connection count is  96 out root user from outside 8. Any idea about how came localhost root user count is 96?
below is th log.
by: com.zaxxer.hikari.pool.PoolBase$ConnectionSetupException: java.sql.SQLNonTransientConnectionException: Data source rejected establishment of connection,  message from server: "Too many connections"

I have configured wait_timeout=2400000 in /etc/mysql/mysql.conf.d/mysqld.cnf. Now commented out that. Its working fine for now. Exactly 10 connections. That is the max-pool-size. If anything coming wrong later i will comment.

No its still growing.

This following is the next try. Update max_user_connections for root user from local.
SELECT max_user_connections FROM mysql.user WHERE user='root' AND host='localhost';
UPDATE mysql.user SET max_user_connections = 2
WHERE user='root' AND host='localhost';
FLUSH PRIVILEGES;
After restart it's not growing more than two connections for root user from the localhost. Thinks it working.

I just got a report on my app using HikariCP 3.4.2 with Oracle that it's using hundreds of Oracle connections instead of the 10 configured connections. We compared the PIDs between DB and Web server, and they matched the single process running on their server.
My database code was unchanged in my latest update (really), but I had upgraded HikariCP from 3.4.1. I'll try downgrading the dependency now, maybe something happened between 3.4.1 and 3.4.2?

@eischet Very strange.  What does netstat -an show on the application machine and the db server in terms of connections?  I recommend enabling debug logging.  HikariCP logs every physical connection close (and acquisition).
Assuming a complete log, you should be able to:
grep "Added connection" your.log | wc -l
and
grep "Closing connection" your.log | wc -l
and the difference between those two numbers should not be greater than maximumPoolSize.

Thanks, I‘ll stage a test ASAP. I can’t directly access the customer‘s machine, so I have to believe when they say that over 700 connections were initiated, with max size set to 10. We’re down to the previous version now, since yesterday, and the issue hasn’t appeared again (yet).

In my case, it's not HikariCP's fault but my own. My app is able to reconfigure data sources at runtime, and these are configured by the user/admin. Shutting down a data source fails to close all connections, and that's something I need to fix in my own code. Thanks for the debugging hints, Brett!

@eischet Whew! Glad you found the cause.

@namjitharavind Any update?  Any chance your code is also recreating the datasource without evicting connections and shutting down the datasource?

@brettwooldridge After my above root user max_user_connections=2 database update. It's only growing up to the pool size. It's working fine now. I have created another user for the application instead of the user root.


VB.NET Windows Forms App (.NET Core) Project Issues after Visual Studio Update

Hello,
I've been experiencing an issue with Visual Studio ever since updating to the latest version. When I open a VB.NET Windows Forms App (.NET Core) project, the Design View interface disappears and the Solution Explorer shows "0 projects", as if no project is open.
Interestingly, when I open a VB.NET Windows Forms App (.NET Framework) project, everything functions as expected. The Solution Explorer displays the project contents and I can use the Design View interface.
Prior to the update, no matter what type of project I opened, the Solution Explorer would correctly display the project content, and the Design View interface was accessible. However, this issue arises when I attempt to open .NET Core version of the Windows Forms project.
To solve this problem, I've tried the following:
Downgrading Visual Studio to a previous version.
Completely uninstalling and reinstalling Visual Studio.
Updating to the latest version again.
Resetting all settings in Visual Studio.
Checking for updates and ensuring both Visual Studio and .NET Core SDK are up-to-date.
Reinstalling .NET Core SDK.
Creating a new .NET Core Windows Forms project.
Unfortunately, none of these steps have resolved the issue. Interestingly, I installed Visual Studio on another machine following the same steps, and all the functions there work as expected, with the Design View interface and the Solution Explorer functioning properly regardless of the project type.
Any insights or assistance you can provide would be greatly appreciated. Thank you.

You are asking in the wrong place. Visual Studio Code is not Visual Studio.


Misconfigured config leads to using all the default settings

In Biome 1.5.1, useConsistentArrayType was moved back to the nursery group, which made the Biome cli complain about it, and is how I discovered this issue.
While the cli doesn't do anything beside throwing an error, it seems like the vscode extension instead reverts to use all the defaults for the formatter.
So formatting on save will change all space indentation to tabs for example.
Ideally a misconfiguration in the linter should not affect the formatter's configuration. Or the linter's config itself for that matter.

I'll move this in the main repository later.
This is a bug, and we should error if the configuration has errors. The LSP should not change any code.
That's how the CLI behaves.


Kanban Setup

Please review our Kanban project.
Thank you.
https://github.com/fortuneonyeka/space-traveler
We are a team of 2 people working on this project.

Hi team,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


ScheduledEvent.type is undefined but is documented as "scheduled"

Which Cloudflare product does this pertain to?
Workers
Existing documentation URL(s)
https://developers.cloudflare.com/workers/runtime-apis/scheduled-event/#syntax-es-modules
What changes are you suggesting?
The documentation for ScheduledEvent's properties has a type field which is expected to be defined and always be set to "scheduled". However, this is not the case when run in a Worker.
From the documentation,
event.type

The type of event. This will always return "scheduled".

Not sure if this is a Workers regression or a documentation clarification.
FYI @irvinebroque
Additional information
Other fields like event.scheduledTime and event.cron are working as expected and is documented correctly.

https://github.com/cloudflare/cloudflare-docs/pull/9671/files appears to have resolved this issue. Closing as complete.


add batchMultiEdits logic to episode rating updates

big rewrite to the logic for episode rating mass operations
Description
Now the code will loop through all episodes and all operations and build up a list of the edits to be made, once it's been through all episodes it will send batch requests to Plex to update the ratings.
Type of Change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist

[x] My code was submitted to the nightly branch of the repository.

I have tested the batchMultiEdits logic of episode ratings (both setting & removing the rating) and can confirm it works without issue.


Added missing data directory and options (3ds, threadsafe). - Update

Original pull request by Thomas Moeller (Bitbucket: TheHubbit, GitHub: TheHubbit).

Updated pull request (again) after accidental deletion of CMake branch.

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

Thomas,
I can see the reasoning behind the additional options, but are we simply ignoring the elements in 'src\setup.h.cmake.in', lines 84-98 for COIN_THREADSAFE and lines 99-110 for HAVE_3DS_IMPORT_CAPABILITIES?
Of course, this also begs the question about the other options in this file, namely HAVE_VRML97, HAVE_NODEKITS, HAVE_DRAGGERS, HAVE_MANIPULATORS, HAVE_SOUND and HAVE_JAVASCRIPT. Only the last two have external dependencies, so I think those should be added. The others are more debatable. What do you think? Should we even consider dispensing with this file altogether and rely just on the CMake options?
Roy

Original comment by Thomas Moeller (Bitbucket: TheHubbit, GitHub: TheHubbit).

Roy,
Very good point! The setup.h file is the correct place where those options should be set. It becomes a lot easier too. I noticed that the naming of the options is a little inconsistent (e.g. *COIN_*THREADSAFE, *COIN_HAVE_*JAVASCRIPT, *HAVE_*SOUND). However, I would start harmonizing them only after CMake is merged into the default branch. Otherwise we may end up complicating the merge.
Another related question to everyone: Would it be OK not to have COIN_BUILD_SINGLE_LIB and COIN_BUILD_SHARED_LIBS under the advanced settings? I feel those are quite important settings, especially as I prefer separate projects for easier navigation ;-).
Thomas

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

Thomas,
I am only a beginner with CMake, but, having had a play with checking and unchecking 'advanced' I think I would prefer to see them all the time. Checking 'advanced' adds a lot entries and these could easily be missed. So yes, I agree, don't have them as advanced settings.
Roy

Original comment by Bastiaan Veelo (Bitbucket: veelo, GitHub: veelo).

Would it be OK not to have COIN_BUILD_SINGLE_LIB and COIN_BUILD_SHARED_LIBS under the advanced settings?

OK

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

Thomas,
It's looking good to me. Are you ready for a merge now?
Roy

Original comment by Thomas Moeller (Bitbucket: TheHubbit, GitHub: TheHubbit).

Yes, this can be merged. Currently I'm looking into usage of simage but need a bit more time for that. I'll create another pull request then.
Thank you!

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

Excellent. Thanks again Thomas.
I use simage myself.
Roy

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

AHH - I tried to merge and I failed - with the attached message. Then it closed the branch again!!!

Roy

Original comment by Thomas Moeller (Bitbucket: TheHubbit, GitHub: TheHubbit).

Strange. Now that it closed the source branch, can you maybe bring in the CMake branch from my pull request again? I guess one can roll back the commit too but I'm also not so familiar with Mercurial myself.

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

No, same error message.

Original comment by Thomas Moeller (Bitbucket: TheHubbit, GitHub: TheHubbit).

Should I try creating a fresh pull request of the CMake branch? It seems to recognize that CMake needs to be created:

Original comment by Bastiaan Veelo (Bitbucket: veelo, GitHub: veelo).

Roy,
I think you could contact support with this.
In the mean time the best would probably be that you make an innocent commit to get the branch back, for example with a change in INSTALL. Then Thomas may have to update the pull request to the new state of the CMake branch before a new try.
Bastiaan.

Original comment by Bastiaan Veelo (Bitbucket: veelo, GitHub: veelo).

My guess would be no, we don't want a new branch, we want the old one.

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

After a little time out, including a bath ... and a small Eureka moment  ...
Retracing my steps ...

Coin has two active branches - 'default' and 'CMake'.
Thomas raises pull request #218 on 'CMake'.
Thomas raises pull request #219 on 'default'.
Pull request #36 is ready first so I merge it, closing the source branch.
I sync the 'default' branch into the 'CMake' branch to propagate the update
I try to merge pull request #218 into the 'CMake' branch, closing the source branch.
I get the error message and the 'CMake' branch is closed.

So, my mistake was step 5. I shouldn't have sync'ed with the outstanding pull request on the 'CMake' branch. That made Thomas's source not up to date, as the pull request was now not to the head of the branch. That explains the error message. (I recall that when using subversion I couldn't commit unless the working directory was fully up to date.) The closing of the branch is, I believe, a bug with the repository. I will raise that with support.
Edit: Support issue raised.
Roy

Original comment by Bastiaan Veelo (Bitbucket: veelo, GitHub: veelo).

Glad we know what happened. But mistake or not, I would have expected Bitbucket to mark pull request #218 out of date after step 5, and not offer the merge option...
Edit Could it be that pull request #218 was open in another browser tab, and wasn't refreshed after step 5, so "it" didn't know it was out of date? That would be fragile.

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

I've had an initial response from Atlassian. They are already aware of this sort of problem and are working on it.
https://bitbucket.org/site/master/issue/10242/unintentional-branch-deletions-from-merge
Roy

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

I've tried pushing a small change to the CMake branch and it won't do it. I have even tried re-cloning to a new repository. Still no joy. I have submitted the errors to support and will have to wait till they get back to me. I have also enquired about what would happen if I created a new 'CMake' branch, and whether it would give me the option to re-use the old branch.
Roy

Original comment by Bastiaan Veelo (Bitbucket: veelo, GitHub: veelo).

This doesn't look promising. It's been known since mid August. Let's all vote on that issue.

Original comment by OgreT (Bitbucket: OgreT, GitHub: OgreT).

Maybe you could add a COIN_ prefix for all coin related options?

Original comment by Thomas Moeller (Bitbucket: TheHubbit, GitHub: TheHubbit).

Yes, we should clean this up as mentioned in a comment earlier. I believe it will be easier after we merged back to default as a few source files have to be touched for this too. Especially as BitBucket seems to have issues with non-trivial merges!

Original comment by Thomas Moeller (Bitbucket: TheHubbit, GitHub: TheHubbit).

Ouch! This is bad.
Roy: I had good experience using the SourceTree client. Maybe you find this a good alternative to the web interface for performing merges until the issue is resolved? Not sure though if or how it supports processing pull requests.

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

I am using SourceTree. And it is looking like it is lost and gone forever. I am still communicating with support, though - not quite given up yet.
Please make sure you keep a copy of all the CMake branch stuff, just in case. I have them all on my system too, so we should be OK to regenerate one way or the other.
Roy

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

Thomas. I seem to have finally managed to reopen it. However, when I clone the repository I don't see the branch on my local system. How is it looking for you?
I see three options here:

If the reopened branch looks OK we could try a) I deny this pull request b) You ensure you have a backup of the CMake files c) Update your code to pick up my dummy commit d) Generate a new pull request to replace this one.
Forget this risky branch altogether. a) Close it intentionally. b) We ensure we both have a backup of the CMake files c) I create a new CMake2 branch d) I update with all the previous CMake changes so far. e) You update your code f) You create a new pull request to replace this one.
Forget the CMake branch altogether and just do it in the main 'default' branch.

Roy
EDIT: I should have waited. It took a while for SourceTree to tell me there were 32 changes to pull. Now that I have I can see the branch and my system has the CMakeList files as expected. So I think perhaps we should go with my first option above.

Original comment by Thomas Moeller (Bitbucket: TheHubbit, GitHub: TheHubbit).

I do see the newly created CMake branch and pulled it into my fork. Then I merged your dummy commit into "my" CMake branch and pushed it. Now I will create a new pull request. Let's hope this works...
... done but it simply updated the existing pull request instead of creating a new one...

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

I tried to merge. It failed again! Just like last time. Now it has lost CMake again. I'll see if I can recover it.
Roy
EDIT: Well - at least I have the recovery process sorted now. It's back!

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

Thomas,
Would you be able to reclone a new copy of the repository to a new directory - don't overwrite your existing one. Make sure after recloning you pull all the updates (I had 32 - but could be more now - it only needed one pull). Then copy into the new repository the files you changed for this pull request. Then commit and create a new pull request from the new repository.
Roy

Original comment by Thomas Moeller (Bitbucket: TheHubbit, GitHub: TheHubbit).

Roy,
I tried a few things in a fresh clone now but run into a several issues including two heads of CMake being there, errors that the head of CMake changed and missing merges. I think I finally managed using the force option to switch to the newly created CMake branch and add the changes there again. The pull request has been updated and you may give this another shot. Otherwise I can just email you the changed files. Also, in one merge attempt I undid your change of INSTALL by accident.
Thomas

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

No problem with the INSTALL - it was just a dummy insignificant change to provide a tool to reopen the branch.
I thought I would try the merge without setting the 'Close Source Branch' checkbox to eliminate yet another unwanted closure if it failed. But this time the merge was successful! Hooray!
Roy

Original comment by Thomas Moeller (Bitbucket: TheHubbit, GitHub: TheHubbit).

Great! Thanks for your time and effort getting this pull request merged.

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

Thanks for your patience while I slowly figured out what was going wrong and how to correct it.


65C02 Bitwise Branch Op-Codes Don't Use Labels for Both Operands

I ran into an issue disassembling 65C02 with bitwise branch opcodes.  Here is an example:
                   ; 6502bench SourceGen v1.7.3
                   SHADOW_4112_READ .eq    $11
                   SHADOW_4112_WRITE .eq   $12
                   CPU2_REG_4112   .eq     $4112

e4f2: 0f 11 0a     CHECK_4112_0    bbr0    $11,IRQ_DONE      ;SHADOW_4112_READ
e4f5: f7 12                        smb7    SHADOW_4112_WRITE
e4f7: a5 12                        lda     SHADOW_4112_WRITE
e4f9: 8d 12 41                     sta     CPU2_REG_4112
e4fc: 20 78 e6                     jsr     IRQ_UPDATE_7
e4ff: 6c 06 01     IRQ_DONE        jmp     (IRQ_PTR)

If you refer to the line CHECK_4112_0, you can see that it does not insert SHADOW_4112_READ for $11.  I am not sure this would be possible without some sort of major overhaul though, it seems the disassembler is sort of designed to have 1 label per opcode.  It is not a huge problem for me to work around (I just insert a comment for these) and I am very happy with 6502bench SourceGen.

This is mentioned in the README, under "limitations":

The zero-page argument for the Rockwell BBR/BBS instructions cannot be formatted.

(That's a little disingenuous since it has other limitations, e.g. it doesn't appear in cross-reference lists either.)
The reason for it is exactly what you suspect: the design only allows for a single operand per instruction.  The only other instruction with more than one operand is 65816 block move (MVN/MVP), where it's less of a problem because both args are bank numbers.  I was aware of the existence of the instructions from an appendix in the Eyes & Lichty 65816 book, but thought they were only present on some weird Rockwell variant, so didn't pay much attention to them until issue #87.
Making it work right would require hackery in a bunch of different places.  The basic code/data analysis doesn't really change, but cross-refs, local variables, and the Edit Operand UI would need some effort.

Sorry I had not read the readme for a while or very carefully. I think it is just fine the way it is and pretty obscure for most people to ever even run into that.  Anyway, thanks for the prompt explanation and I really like your work on this disassembler.  I actually found a 65C02 inside the Famicom Network System and it is my first time working with these extra instructions.
I am closing the issue, it definitely seems like more trouble than it is worth, and really actually works totally fine the way it is.


lodash: added _.callback() method (alias iteratee)

https://lodash.com/docs#callback
https://lodash.com/docs#_ (description of the chainable wrapper)

@chrootsu thanks mate!


[react-hooks] 1.12.0 minor broke import in Next.js

Overview
The 1.11 to 1.12 minor update changed the way react-hooks is authored and broke the import for a client's next.js app. I have downgraded for now. It appears to be due to the .mjs file.
This is likely just growing pains with support for .mjs and bundlers catching up (webpack is used internally by next) however I think either way this change could have been a major.
Likely no action required, but thought I would open in case anyone else runs into this. Some searchable stuff below:
Can't import the named export '<hook>' from non EcmaScript module (only default export is available)

Consuming repo

What repo were you working in when this issue occurred?
Private repo, can provide reproduction repo at request

Area
Not sure :thinking:
Scope
react-hooks

cc @BPScott @ismail-syed - I think you're looking into this?

Yes, I've been seeing related reports. I'm going to spend some time next week to look into this deeper.

Agreed, growing pains of .mjs support. There's no real action here to take. In retrospect, this should have been a breaking change. For adding support in nextjs configs, see https://github.com/Shopify/quilt/issues/1722#issuecomment-756963544.


Presence for federated users in outlook desktop does not work

Hi Team,
Customer reported Presence for federated users in outlook desktop does not work. It looks there is no statement for this in below docs.
https://docs.microsoft.com/en-us/microsoftteams/communicate-with-users-from-other-organizations
After confirming with teams product engineering, this feature is not supported. Could you please kindly help to document this？
Please contact me via<EMAIL_ADDRESS>[Enter feedback here]

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

ID: 5f080a5c-4268-1db4-566f-f1edc27920b5
Version Independent ID: ac556a92-58af-3e05-f53b-0fc00afc96c0
Content: Call and chat with users from other organizations - Microsoft Teams
Content Source: Teams/communicate-with-users-from-other-organizations.md
Service: msteams
GitHub Login: @serdars
Microsoft Alias: serdars

@XixiHuang333  Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hi @XixiHuang333, thank you for your feedback.
If you have a Microsoft alias, you can link your accounts in GitHub so that a blue "Microsoft submitter" label appears to any PR's or issue you submit. Guidance on it here: https://docs.opensource.microsoft.com/tools/github/index.html
Here is the site you can use to see the mappings between GitHub and Microsoft aliases: https://repos.opensource.microsoft.com/people?q=XixiHuang333
I tested it in the following version of Outlook, and it works, I can see presence for federated accounts:

Hi @XixiHuang333, thank you for your feedback.
If you have a Microsoft alias, you can link your accounts in GitHub so that a blue "Microsoft submitter" label appears to any PR's or issue you submit. Guidance on it here: https://docs.opensource.microsoft.com/tools/github/index.html
Here is the site you can use to see the mappings between GitHub and Microsoft aliases: https://repos.opensource.microsoft.com/people?q=XixiHuang333
I tested it in the following version of Outlook, and it works, I can see presence for federated accounts:

Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Crash after updating VS

Issue Type: Bug
Help to review.
1_ I click on menu update vs.
2_ I saw small popup tell me install now or not. I hit button No. I dont want update right now.
3_ I click on close to quit the Application.
4_ I meet issue cannot open VS.
Thank you and sorry for my bad english.
VS Code version: Code 1.22.2 (3aeede733d9a3098f7b4bdc1f66b63b0f48c1ef9, 2018-04-12T16:38:45.278Z)
OS version: Windows_NT x64 6.1.7601

I have similar problem with the new v. 1.22.2 (Mac OS 10.12.6) and had to roll back to v. 1.22.1

@thosanmaytinh What's the issue when you try to open VS Code?

My crashing occurs when I have 2 tabs with ".js" files opened(with split editor or not)


[MBL-1207] Pledge Button on Project Page

this applies to PCP campaigns that backers have already pledged to

📲 What
Updates the pledge button on project pages to say "Back this project" when a campaign has Post-Campaign Pledging (PCP) enabled and is actively accepting late pledges.
In addition, I also made sure that the correct pledge button shows on rewards when in a PCP state. Right now, if a project isn't live or has already been backed, we don't show anything. Now, the pledge button will show when in an active PCP state if the user hasn't already backed.
If the user has already backed it, they'll see the normal "Manage" CTA that presents users with their pledge info.
🤔 Why
If PCP is disabled and a user has backed the campaign, we show a “View Rewards” button. For this feature, we want backers to be able to pledge post a successful campaign. It's important to note that backers should never be able to pledge more than once to any campaign
🛠 How
Let’s use the feature flag and the new PCP properties on the project fragment to gate this new behavior.
👀 See
Designs
✅ Acceptance criteria

[x] If a campaign has PCP enabled and is actively accepting late pledges and users have not backed already, then the project should show the "Back this project" CTA
[x]  If a campaign has PCP enabled and is actively accepting late pledges and users have already pledged, then the project should show the "manage" pledge CTA that shows users their pledge info.
[x] If a campaign has PCP disabled or is not actively accepting late pledges, then the normal pledge flow should be active

@ifosli, your suggestions also pointed out that we'll probably want to show the manage pledge button and update the manage pledge view as well. Posted in Slackfor some team/design feedback and will finish this PR once I get confirmation on that.


InheritedModel dependencies not cleared on rebuild

Consider a widget that conditionally depends on an InheritedModel aspect:
if (foo) {
  return Text("Foo", MyModel.of(context, aspect: MyAspect));
}

It seems that if foo requirement is met once. Then even if that condition fails in the future; the widget will stay dependent on the MyModel.
This leads to unnecessary builds.  I'd expect the list of dependencies to be cleared before build call

@Hixie This is blocking for the release of a state management library I'm working on.
I can work on this if needed, but what can be done here?

We looked into this when designing this feature, but it seemed at the time like the cost to clear the dependencies would far outweigh any gains from not redundantly calling a few build methods. However, the best way to determine this would be to benchmark it.
If you are interested in addressing this, the place to look in would be framework.dart. I'm not sure what the best way to do this would be. We would definitely want to carefully examine benchmarks as part of any such PR.

Is this a duplicate of #12992 ?

It is, although I started some work on the topic.
I do not know how to benchmark the performance impact, so I'm kind of stuck here.


Invalid Request Error: str type expected

Hi, I am trying to use the extension with my local OpenAPI-compatible LLM, but I am facing an "str type expected error".
This is the error response:
{
    "object": "error",
    "message": "[{'loc': ('body', 'messages'), 'msg': 'str type expected', 'type': 'type_error.str'}, {'loc': ('body', 'messages', 0, 'content'), 'msg': 'str type expected', 'type': 'type_error.str'}]",
    "type": "invalid_request_error",
    "param" null,
    "code": null
}

Upon inspecting the requests, I noticed that the plugin is sending the request in this structure:
{
    "messages": [
        {
            "role": "system",
            "content": [
                {
                    "type": "text",
                    "text": "You are a programmer assistant called ..."
                }
            ]
        }
    ],
    ...
}

However, the OpenAI API reference shows this structure instead:
{
    "messages": [
        {
            "role": "system",
            "content": "You are a programmer assistant called ..."
        }
    ],
    ...
}

May I know what could be the issue? Thanks! 🙏

Hi @leejunweisg,
I made tests and found the problem. The lasts plugin's versions came with this bug and another ones that I found now related if the "content" object.
I rollbacked the plugin version and now all are working fine. I will publish a new release soon.


feat: improve build process

#4

What do you think of being able to include the SearchBar component just by including the plugin? I tried getting this to work but kept running into a compilation error. I know Docusaurus supports overriding components in such a manner.

Ok... I don't have a lot of knowledge on that .. I will look into it ...


Crash on startup

Hello!
Material Maker version:
Latest windows release: 1.3
OS/device including version:
Windows 11, AMD Vega 64 latest drivers
Issue description:
I can see the splash screen, then the app just closes
Steps to reproduce:
Open the app and wait
Console output:
`Godot Engine v3.5.2.stable.custom_build.170ba337a - https://godotengine.org
OpenGL ES 3.0 Renderer: Radeon RX Vega
Async. shader compilation: OFF

Cannot open user://export_targets
Failed to open user://library/user.json
Failed to open user://library/user_brushes.json
WARNING: Ignoring unsupported header information in HDR: GAMMA=1.
     at: ImageLoaderHDR::load_image (modules\hdr\image_loader_hdr.cpp:50)
WARNING: Ignoring unsupported header information in HDR: PRIMARIES=0 0 0 0 0 0 0 0.
     at: ImageLoaderHDR::load_image (modules\hdr\image_loader_hdr.cpp:50)
2
ERROR: Index p_index = -1 is out of bounds (data.children.size() = 2).
   at: Node::get_child (scene\main\node.cpp:1362)
ERROR: Index p_idx = -1 is out of bounds (items.size() = 8).
   at: PopupMenu::set_item_disabled (scene\gui\popup_menu.cpp:871)

`

Ok my bad. I installed latest AMD drivers (it seems my version wasn't the latest...) and it fixed the problem. Sorry!


Unable to write extrinsic

Getting "BadProof" and "InvalidTransaction" upon attempting to submit an extrinsic that writes value to a custom storage map in Substrate.
Full details: https://stackoverflow.com/questions/61005569/how-to-write-in-the-storage-of-substrate-using-substrate-api-client
I couldn't understand as to why there would be a bad sign when I am repeating signature steps from balance transfer example.

cc @brenzi
Thanks in advance!

what node version are you running against?

answered here. https://stackoverflow.com/questions/61005569/how-to-write-in-the-storage-of-substrate-using-substrate-api-client/61006857#61006857


mysql-utilities-1.6.4

Checklist

[x] The commit message includes the cask’s name and version.

[x] brew cask audit --download {{cask_file}} is error-free.

[x] brew cask style --fix {{cask_file}} left no offenses.

Addresses caskroom/homebrew-cask#24076

Well; I thought that it was working however there seems to be an installer issue once again.
https://bugs.mysql.com/bug.php?id=82749

Can you try installing the pkg manually, and see if that succeeds?

Is this OK to merge, @mengesb?

Sorry @adidalal ; I did try and install it manually but it didn't install manually or via the generated command line... seems there might be another issue (hence the bug i filed)

Looks like the bug opened by @mengesb has been marked as a duplicate of https://bugs.mysql.com/bug.php?id=81048, which has been open since April.

Yes it has... And I'm not seeing any updates unfortunately
On Sep 5, 2016 00:11, "Michael Ford"<EMAIL_ADDRESS>wrote:

Looks like the bug opened by @mengesb https://github.com/mengesb has
been marked as a duplicate of https://bugs.mysql.com/bug.php?id=81048,
which has been open since April.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/caskroom/homebrew-cask/pull/24077#issuecomment-244673489,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUHF0oKFp5XXhRgCcp0MMZz0C-BJX9Jks5qm8CNgaJpZM4JubnK
.

Marking as an upstream issue.

Will remove the cask, since it has been broken upstream for a good while. If/when a fix is released, we can readd it.

https://github.com/caskroom/homebrew-cask/pull/25061.


Syntax

bad comma in https://github.com/kaustubh-sadekar/OmniCV-Lib/blob/master/paper/paper.md : to, simulate virtual
other minor syntax issues

cc https://github.com/openjournals/joss-reviews/issues/2533

Thank you very much for the suggestions. I have updated paper.md file.


Translate scheduling/kube-scheduler.md in Korean

This is a Feature Request

What would you like to be added

Translate concepts/scheduling/kube-scheduler.md in Korean.
Why is this needed
There is no documents with concepts/scheduling/kube-scheduler.md in Korean.
Comments

https://kubernetes.io/docs/concepts/scheduling/kube-scheduler/

/language ko
/assign


Button

A simple button that reads a click from the user to perform some action.
Rough mockup:

Button component completed.


No error information is displayed after parse error from assert

Trying to test the vscode extension with the PCAP DFDL schema.
I clobbered the magic number in the first bytes of an icmp1.cap file. This fails an assert at line 97 of pcap.dfdl.xsd, which is the first element of simple type in the schema.
I step through with the debugger, and nothing displays anything about the assert failure. Execution ends. A little pop-up dialog says it wrote my infoset out, which it didn't. There's nothing showing that anything went wrong. All panels on the left are blank.
I was testing 1.3.0-RC2 version of the vscode extension.

I created an icmp1.bad.cap where I hammered the first byte to be 0x00. Using that as the data file causes the failure.

@mbeckerle Thank you for this information this helped me reproduce the issue! I will get working on a fix, thank you again

Testing 1.3.0-RC3
You got this output, which would be great:
Schema context: MagicNumber Location line 87 column 14 in file:/Users/sdell/apache/sampleWorkspace/dfdl-pcap/src/main/resources/com/tresys/pcap/xsd/pcap.dfdl.xsd
Data location was preceding byte 4

Alas, my debug console window is empty, and the terminal window does not contain the characters "assert" (case insensitive). So I am simply not getting anything to indicate an error occurred.

@mbeckerle are you on the terminal tab? By default I think it ends and goes to console or debug console but those won't have the output. There should a tab called Terminal that you can click on and see all the information from the debugger

I am testing 1.3.0-RC3.
Complete output from terminal is attached in a file. Note there is a backtrace.
terminal-output.zip

@mbeckerle can you try removing this folder: /home/mbeckerle/.local/share/daffodil-dap/ and run the debug again? I believe you are using an old version of the debugger and that is why you are not seeing the output you should.

@mbeckerle If the above comment fixes the issue, we have another GitHub issue about making testing local changes to the debugger easier. If this is the issue you see I think it would be in the best interest to figure out another way of extracting the debugger or bundling the debugger so that you always use the latest and won't hit this issue anymore

Uninstalling the debugger should delete ~/.local/share/daffodil-dap if it installs it there.
I do indeed get an assert-related error to the terminal now. But then a big noisy backtrace due to exception in initializer error.
Can we turn down the Logging level from INFO to remove some of the noise from the terminal?
java.lang.Error: Parse Error: Assertion failed: Magic number was not 0xA1B2C3D4 (for bigEndian) or 0xD4C3B2A1 (for littleEndian).
Schema context: MagicNumber Location line 87 column 14 in file:/home/mbeckerle/dataiti/opensource/DFDLSchemas/PCAP/src/main/resources/com/tresys/pcap/xsd/pcap.dfdl.xsd
Data location was preceding byte 4
        at org.apache.daffodil.debugger.dap.Parse$$anon$1.$anonfun$run$2(Parse.scala:113)
        at as @ com.monovore.decline.effect.CommandIOApp$.addVersionFlag(CommandIOApp.scala:67)
2023-06-22 16:54:18,313 [io-compute-3] INFO  o.a.d.d.d.DAPSession - R> #16 disconnect {"restart":false}
2023-06-22 16:54:18,315 [io-compute-7] INFO  o.a.d.d.d.DAPSession - <R #16 disconnect success null
2023-06-22 16:54:18,316 [io-compute-blocker-5] DEBUG o.a.d.d.d.DAPodil - launched: Succeeded
2023-06-22 16:54:18,316 [io-compute-blocker-5] DEBUG o.a.d.d.d.Parse - signalling a stop
2023-06-22 16:54:18,317 [io-compute-blocker-5] DEBUG o.a.d.d.d.Parse - parse done, no interruption
Exception in thread "io-compute-6" java.lang.ExceptionInInitializerError
        at com.sun.xml.bind.v2.runtime.reflect.opt.AccessorInjector.prepare(AccessorInjector.java:81)
        at com.sun.xml.bind.v2.runtime.reflect.opt.OptimizedAccessorFactory.get(OptimizedAccessorFactory.java:179)
        at com.sun.xml.bind.v2.runtime.reflect.Accessor$FieldReflection.optimize(Accessor.java:285)
        at com.sun.xml.bind.v2.runtime.property.ArrayProperty.<init>(ArrayProperty.java:68)
        at com.sun.xml.bind.v2.runtime.property.ArrayERProperty.<init>(ArrayERProperty.java:88)
        at com.sun.xml.bind.v2.runtime.property.ArrayElementProperty.<init>(ArrayElementProperty.java:100)
        at com.sun.xml.bind.v2.runtime.property.ArrayElementNodeProperty.<init>(ArrayElementNodeProperty.java:62)
        at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
        at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77)
        at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
        at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:499)
        at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:480)
        at com.sun.xml.bind.v2.runtime.property.PropertyFactory.create(PropertyFactory.java:128)
        at com.sun.xml.bind.v2.runtime.ClassBeanInfoImpl.<init>(ClassBeanInfoImpl.java:181)
        at com.sun.xml.bind.v2.runtime.JAXBContextImpl.getOrCreate(JAXBContextImpl.java:503)
   ...

The back trace is a Java 17 issue. If I use Java 11 I do not get this backtrace.

With a caveat that 1.3.0-RC3 still doesn't work with Java 17 (release note), this issue can be closed.

While this fix adds the error to the (terminal) console, we could also handle it by writing it to the "debug console":

or we could create a special event that causes a dialog/popup, like the current behavior when there is a compilation error in the schema.
What do you think @mbeckerle @shanedell?

I also saw the comments in #666 about feedback mechanisms. Instead of ending the debug session, we could raise an exception which would pause the debugger and the UI would show the assertion failure. Any subsequent "next" action would end the parse.
In contrast the current mechanism sends the error info, which we then log or show a dialog, but the debug session ends concurrently.

@arosien I like displaying it in the "Debug Console". I wouldn't do the pop up or dialog because we had that before and it was removed  as @mbeckerle said it didn't need to be a blocking dialog


Null description in case of no missing value

Validity mask is a missing value representation that depends on the Column in the protocol. If describe_null() is meant to describe missing values at the column level for a given dtype.
In the case there is no missing values, shall we still provide a validity array with 1 (valid) at all entries or
shall we raise an exception ?
From my perspective, the later is better because we can just check null_count == 0  without allocating and filling the whole array with the same value. That is how it works in cudf dataframe. If there is no missing value, accessing the attribute nullmask (which holds the validity array) raises an exception with message: "Column has no null mask".

If the native representation does not have a mask - either because there are no missing values or because it encodes them as nan in the data - then it should not be necessary to create an "all ones" buffer for it. However, rather than checking null_count, I believe this should come from describe_null()  - if that says there's a bit mask or byte mask, then it should actually be there independent of if there's zero or more null values.
An optimization path is still possible then right? The implementation can put the null_count check inside describe_null.

ok, thank you for your explanation.

An optimization path is still possible then right? The implementation can put the null_count check inside describe_null.

I agree so describe_null should behave as follow:
if there is no missing value,  we should return value = 0 meaning the column is non-nullable thus
do not have any mask array. Otherwise, we return value = 3 as bit mask
is universally used for any dtype in cudf.
def describe_null():
     if self.null_count == 0:
          null = 0
          value = None
     else:
          null = 3
          value = 0
    ...
    return null, value

Actually we had a conversation about this issue last night, and the preference was for having None be a valid return value when accessing the validity buffer in case no values are missing.

I think this is already the case when null == 0. From the pandas implementation, in _get_validity_buffer, we have:
elif null == 0:
     msg = "This column is non-nullable so does not have a mask"
else:
     raise NotImplementedError("See self.describe_null")
raise RuntimeError(msg)

And, in get_buffers, we have:
try:
     buffers["validity"] = self._get_validity_buffer()
except:
     buffers["validity"] = None


[fix] [m] Make changes to existing pages inline with new specs - fixes #422

@mr-vara all content looks good. small comments on commit message
Should be something like:
[blog][s]: Update frontmatter of various blog pages to use description rather than summary inline with recommendations in https://github.com/frictionlessdata/project/issues/422.

If you have a first item it should be descriptive. [fix] is not ... Something like [blog] would have been.
is this really a fix for 422 - i don't think. I would be wary of closing issues from commits in general esp if it is a multi-item issue
422 is another repo so you can't do #422 - you need the actual liunk
be specific this is about frontmatter change


Catch Invalid Response

Created a quittable loop and logic to catch an invalid response, instead of just crashing the program for a nameless issue.
I... only edited AreYouOk/AreYouOk.py... how did the others get edited? I'm not sure. They're under my commit, through my computer, yet I don't even know any programming language but Python fluently enough to make those changes.
It looks like GitHub is also registering the changes made by other people recently to me. I didn't edit those, and I'm not sure what happened here.
Any ideas how that could have happened?

Also, you can see exactly who made those changes - if you view the individual file.
Sorry for the confusion.
-Avi C. ~

this also fixes #968.

This also fixes #944.


[Renderer] Dynamic light not lighting certain textures

Map name: ze_blackmesa_dp
Notice that it doesn't light up only when being directly aimed on.
https://user-images.githubusercontent.com/3608544/194762630-691b478a-5798-41cb-ad61-6e2e7292e06a.mp4

That is probably because the origin of dynamic light in under the floor.

Bad map design?

Bad map design?

I don't know if the dynamic light is from map entity or amxx plugin or what but the dynamic light should always stay above floor because the lighting is calculated in "normal of surface" dot "light direction" which gives negative value in this situation.

I noticed that it happens the same for the default flashlight, but only on certain(older?) server versions.
If i create a new game or join an up to date server the bug doesn't happen. If you need video proof I'll show you.

Idk if this helps with anything, but i've noticed something even on listenserver:
https://user-images.githubusercontent.com/3608544/197361138-94caaad8-a6c2-4bab-b820-68f9a6cc1f74.mp4

This might be helpful: when connected to a dedicated, and in first person, it doesn't draw correctly but when switching to 3rd person it works.
https://user-images.githubusercontent.com/3608544/198355276-797eeb82-8526-467b-9596-69d1a054f67e.mp4

Why two dynlight at the same time?

I don't know. The server is default and doesn't change the flashlight in any way, so everything you see is from metahook.

Issue still present as of build https://github.com/hzqst/MetaHookSv/releases/tag/v20240514a
Can be tested on <IP_ADDRESS>:27015
Default flashlight is being used + a small fix for vanilla client that sets a DLIGHT on player aim origin when aiming on another player entity.


Replace README with Wiki

Issue #, if available:
https://github.com/partiql/partiql-ir-generator/issues/120
Description of changes:

Breaks up the README into a wiki
Fixes some typos
Formatting fixes
Fix .ftl indent

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Rendered — https://github.com/partiql/partiql-ir-generator/blob/wiki/README.md


HttpTrigger local func start keys issue

HttpTrigger local func start gives the following error - anonymous function vs function keys – there was issue in the Kudu console.
Tried to use anonymous functions without keys and deployed using BYOF - the trigger did not work in azure.
Worked locally. Interchanging with and without keys caused errors in Kudu. - This was reproducible during serverless openhack.

@priyaananthasankar can you provide the repro steps? Also, what version of the CLI were you using?

I can't repro this on the latest. I'll close this but let me know if you are still seeing issues.

Reopening this issue as it is observed even today. A simple get method with func new works locally, but gives 401 unauthorized after func core tools deploy.

I followed the instructions at https://github.com/Azure/azure-functions-python-worker/wiki/Create-Function-(CLI).  I could get the function to work locally at http://localhost:7071/api/HelloCherry.  After publishing it to Azure, I could not get the function to work.  I initially received an HTTP 401 response.  I changed the 'authLevel' in function.json to 'anonymous' and republished.  That resulted in an HTTP 500 error.
I'm using Python version 3.6.5 and Azure Function Core Tools version 2.0.1-beta.28.

@mcollier  Can you share your function.json file, app name and the endpoint you're calling to invoke your function?

App name: https://mcollier-func-py.azurewebsites.net/
Function name: HelloCherry
I've tried calling via https://mcollier-func-py.azurewebsites.net/api/HelloCherry and https://mcollier-func-py.azurewebsites.net/HelloCherry.
The function.json file I used is as follows:
{
  "scriptFile": "__init__.py",
  "disabled": false,
  "bindings": [
    {
      "authLevel": "function",
      "type": "httpTrigger",
      "direction": "in",
      "name": "req"
    },
    {
      "type": "http",
      "direction": "out",
      "name": "$return"
    }
  ]
}

I've tried changing the authLevel to 'anonymous'.  That resulted in an HTTP 500 error.

Have you tried the below?

Quick update - @mcollier and I synced offline. The issue is not a 404 but a 500.
There is no repro on our end but I'll investigate and come back here.

/cc @ahmelsayed

@pragnagopa can you investigate this?

The issue is caused due to a breaking change in the v2 runtime (preview). Deployments are in progress. We'll send an update to this thread once the release is done.

This should now be resolved. @priyaananthasankar and @mcollier  go ahead and try again.
Please make sure that you upgrade the Azure Functions Core Tools to version 2.0.1-beta.33
If you're not starting over from scratch, check that the requirements.txt file contains the following list of dependencies:
azure-functions==1.0.0a1
azure-functions-worker==1.0.0a3
grpcio==1.12.1
grpcio-tools==1.12.1
protobuf==3.6.0
six==1.11.0


fix(utility): use "dump" instead of "_dump" in "AttrsMinix"

Pull Request Test Coverage Report for Build 859006093

2 of 4 (50.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.004%) to 80.8%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tensorbay/utility/attr.py
2
4
50.0%

Totals

Change from base Build 857354006:
-0.004%

Covered Lines:
4242

Relevant Lines:
5250

💛  - Coveralls


Add File Summaries Functionality in Executor

We need to add the ability for Executor to fetch file summaries of all the code files. This will aid agents in writing more efficient code.

[x] Create file summaries and store them in a centrally shared place across multiple sessions. We can use Llama 3 8b for fetching summaries.
[x] Use file summaries in the agent's system prompt.

Fixed by #146


Implement database from database concept

Describe the current situation / your motivation for the change
Hera currently stores all of its data in property files or not at all.
Describe the solution you'd like
Setup a database based on the database concept created in #59 using MySQL through Heras web server.

Database can be found in the web server control panel --> phpMyAdmin.
The aaronkromer prefixes were necessary because it is the name of the web disk account.

Relations and tables look good so far. They need specifictation of nullable modifiers though.

Fields role#parent and module_settings#roleFK are now nullable.


ERROR_NOT_DOS_DISK on exe signature scan

Greetings!
I was hoping to find a solution on this issue which is basically blocking me from the final steps of my project: ERROR_NOT_DOS_DISK.
This error appears in the yr_rules_scan_file function call, by using LibYara.
If I scan ANY file, I wouldn't have any problem, but if I scan this large .exe (PE) file (I can't release it, sorry!) I receive this error.
What I noticed during the debugging of LibYara to figure out where the error might happen is that sometimes the scanner does hit a match on many of my rules on this target, but these doesn't get reported back to the callback at all.
I'm sorry I can't help you more than the details I have exposed, the debugging is a bit of a problem due to the scanner timeout.
To understand if I made a mistake or if it is a Yara problem, I used yara64.exe compiled under the Debug profile and tested it with the same target & set of rules: in this case, it does work wonderfully.
What could the problem be?
This is how I initialize the library:
if (yr_initialize() != ERROR_SUCCESS)
	return false;

if (yr_compiler_create(&g_compiler) != ERROR_SUCCESS) {
	return false;
}

FILE* _handle = nullptr;
fopen_s(&_handle, _pathToYaraDb.c_str(), "r");
if (!(_handle != NULL))
			return false;

if (yr_compiler_add_file(g_compiler, _handle, NULL, NULL) != 0) {
	return false;
}

if (yr_compiler_get_rules(g_compiler, &g_rules) != ERROR_SUCCESS) {
	return false;
}

The initialization function (above) returns true, meaning no errors are reported during the init process.
The scanning function:
auto _result = yr_rules_scan_file(g_rules, _pathToFile.c_str(), NULL, _defaultCB, (void*)&__lastFile, 0);
if (_result != ERROR_SUCCESS)
	return _result;

Do you mean that you are getting ERROR_NOT_DOS_DISK from YARA? It's yr_rules_scan_file returning that error?

So little update after my sickness, sorry for the wait.
I fixed the problem by passing 0 and disabling the debugging performance monitor (heap monitor and CPU profiling), this does not happen on my personal PC but happens at work, so it may be a specific situation.


Generate and upload SBOM

Using the SBOM generator tool detailed here:
https://www.1eswiki.com/wiki/ADO_sbom_Generator#YAML_Pipelines

I'm only uploading the sbom on linux, since I don't think we need to upload an sbom for each OS.
I'm uploading the contents of the _manifest directory as an artifact named "sbom".

I'm only uploading the sbom on linux, since I don't think we need to upload an sbom for each OS.

I doubt we even need to run it on Windows/Mac...


Investigate tuples with same field names but different order

Currently the new value tuples contain the first parameter with CqlTupleMetadata which contains the tuple item types and names. Equality of tuples are equal for tuples with exactly the same CqlTupleMetadata and the remaining values in the tuples.
But should we consider that the order of fields names should not matter? After all, it should be considered as a dictionary. So (a:1, b:"b") should be equal to (b:"b", a:1)
This needs to be investigated

The order of tuple elements does not matter.  I had to fix a similar bug in HEDIS in which we had functions like this:
define function Foo(t Tuple{ x: Integer, y: String })
define function Foo(t Tuple{ y: String, x: Integer })

Those are the same function and an ambiguity error must be raised.
My solution to this was always to sort the tuple's elements alpha ascending before creating it as a type.


Not able to set up environmental variables in PowerShell

[Enter feedback here]
setx SPEECH_KEY does not set up environmental variable

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

ID: 1370a1c1-3686-3810-ccd2-648780aa69c9
Version Independent ID: 93ff06b1-e74b-f680-18f9-d9b8dd283e24
Content: Azure OpenAI speech to speech chat - Speech service - Azure Cognitive Services
Content Source: articles/cognitive-services/Speech-Service/openai-speech.md
Service: cognitive-services
Sub-service: speech-service
GitHub Login: @eric-urban
Microsoft Alias: eur

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

Duplicate of #108614


Survey updates models if they exist

When a survey is approved, before creating new records, check whether matching records exist and update as described below.
I'm using the word 'record' to mean the pre-existing record in the CIC database, and 'survey' to mean information that was entered in the survey form.
Person:

locate a matching Person record based on ORCID, or email if ORCID is not available
First Name, Last Name: Don't change existing content in the record
Email: Set the record's email field to the union of the addresses already in the record and the addresses from the survey
ORCID: Add to the record if it was not present already
Keywords: Set the record's keywords to the union of the existing keywords in the record and keywords from the survey

Grant:

locate a matching Grant record based on the combination of Funder and Award ID (both must match)
Title: Don't change existing content in the record
Keywords: Set the record's keywords to the union of the existing keywords in the record and keywords from the survey

DOIs:

for each DOI, check whether it exists in CIC as a Publication or Dataset
If it exists, leave it alone
If it does not exist, create a new record

@ryscher

Can you include the rules for adding/updating Co-PI when the form is approved? I think we talked about not approving the form if the grant doesn't exist in our system in case the form is submitted by a Co-PI, correct?
If the grant exists, and the form is submitted by Co-PI, do we follow the same rules as you described?


Update NHM v1.1 byHW model output to extend time period (1979-01-01 to 2021-12-31)

Current NHM model output covers 1979-01-01 to 2019-12-31. The gridmet forcing product for the NHM has been extended through 2021-12-31. The on-prem NHM byHW model output should be updated to include the additional time steps.

The on-prem NHM v1.1 byHW model output has been extended to cover 1979-01-01 to 2021-12-31.


⚠ BCR Bot has degraded performance

In 59dbb14, BCR Bot (https://bcr.alternativasinteligentes.com/BCRApi/web/chatbot) experienced degraded performance:

HTTP code: 200
Response time: 2242 ms

Resolved: BCR Bot performance has improved in 8786681.


WIP: bos-tasks-j3s26: Check for NPM dependency updates

Check for any outstanding update of point version of any outstanding NPM package dependency

[ ] npm-update

npm-update in progress on bos-tasks-j3s26.
Log: http://fedorapeople.org/groups/cockpit/logs/npm-update-577-20190304-071717/


Please authenticate yourself now to create a credential for

I got this error when I try to run install.sh

just download Microsoft visual c++ 2015 https://www.microsoft.com/en-us/download/details.aspx?id=52685


Add >> and << to Eff so they can be used applicatively

Similarly to *> and <*.

I would like to keep *> for the applicative case and >> for the sequential one because I think they both have their uses. I redefined *> directly on Eff because it was otherwise implemented in terms of flatMap and was forcing sequential evaluation.

This makes sense to me.


[MONIT-34586] Fix Filebeat integration throwing an exception

Looks like it was caused by this PR: https://github.com/wavefrontHQ/wavefront-proxy/pull/839

Should address #850


Deprecation warnings

There's a number of deprecation warnings that make elm-pages fail to build a release build:
DeprecationWarning: 'createNumericLiteral' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createPropertyAssignment' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createNull' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createObjectLiteralExpression' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'updateVariableDeclaration' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createParameterDeclaration' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createReturnStatement' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createBlock' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createFunctionExpression' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createVariableDeclaration' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createCallExpression' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'updateCallExpression' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: '' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.

elm-optimize-level-2 -V
DeprecationWarning: 'createIdentifier' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
0.3.5

Noticed this as well, when optimizing my project.


Use styled path for files copy and check

When storing the generic proxy files, Indy use hashed path instead of raw path. This fix is for promotion-by-copy. It turns the raw path to hashed path before the copy (and existence check) so the storage can find the right files. Otherwise, the storage try to 'copy' the raw path and failed on 'file not found'.
This fix also remember the mapping between raw path and hashed path, so it can report the promotion results to users by raw path. The users do not see any hashed paths in results or error messages.
The promotion service get the path_style from repo service. It does not parse the whole json but only get the path_style property. This avoids using whole lot of extra model classes.

LGTM, let's verify it by tests.


Add possibility to track errors in datepicker

Summary
Currently the date picker does not have support to check if an invalid date is entered in the component. Adding the validation ourselves is not so easy as we need to know what the format of the date is. This is determined via the configured Locale of the Angular app in an internal Clarity service.

What is the change?
Create a component that implements the Angular ControlValueAccessor interface. This allows users bind the value to a FormControl. The FormControl should be marked as invalid if the an invalid value was entered for the date.

Why should it go in Clarity?
This seems to be a common use case that probably many people can benefit from.

Does this change impact existing behaviors? If so how?
This should normally not break existing features.

If this change introduces a new behavior, is this behavior accessible?
Yes.

Use case
Add a date picker on a form group. The save button of the form should not be enabled if the date picker contains an invalid date (e.g wrong format or a date in the past, or ...).
Examples
Workarounds
Checking on the interval value of the date picker. If this is null and the HTML input has a text value, the value can be considered as invalid.

This should be completed with the forms refactorings done so as long as you mark a field as invalid with Angular, there is error support.


Uncaught TypeError: Cannot read property 'length' of undefined

On default for yesod project Application.hs file  I get Uncaught TypeError.
Atom Version: 0.180.0
System: Windows 7
Thrown From: haskell-ghc-mod package, v0.4.1
Stack Trace
Uncaught TypeError: Cannot read property 'length' of undefined
At C:\Users\Bet\AppData\Local\atom\app-0.180.0\resources\app\node_modules\text-buffer\lib\text-buffer.js:391

TypeError: Cannot read property 'length' of undefined
  at TextBuffer.module.exports.TextBuffer.lineLengthForRow (C:\Users\Bet\AppData\Local\atom\app-0.180.0\resources\app\node_modules\text-buffer\lib\text-buffer.js:391:29)
  at TextBuffer.module.exports.TextBuffer.clipPosition (C:\Users\Bet\AppData\Local\atom\app-0.180.0\resources\app\node_modules\text-buffer\lib\text-buffer.js:943:53)
  at TextBuffer.module.exports.TextBuffer.clipRange (C:\Users\Bet\AppData\Local\atom\app-0.180.0\resources\app\node_modules\text-buffer\lib\text-buffer.js:925:20)
  at MarkerManager._results.push._this.(anonymous function) [as clipRange] (C:\Users\Bet\AppData\Local\atom\app-0.180.0\resources\app\node_modules\delegato\lib\delegator.js:67:61)
  at MarkerManager.module.exports.MarkerManager.markRange (C:\Users\Bet\AppData\Local\atom\app-0.180.0\resources\app\node_modules\text-buffer\lib\marker-manager.js:84:20)
  at TextBuffer.module.exports.TextBuffer.markRange (C:\Users\Bet\AppData\Local\atom\app-0.180.0\resources\app\node_modules\text-buffer\lib\text-buffer.js:645:27)
  at DisplayBuffer.module.exports.DisplayBuffer.markBufferRange (C:\Users\Bet\AppData\Local\atom\app-0.180.0\resources\app\src\display-buffer.js:1272:41)
  at TextEditor.module.exports.TextEditor.markBufferRange (C:\Users\Bet\AppData\Local\atom\app-0.180.0\resources\app\src\text-editor.js:1309:59)
  at EditorController.module.exports.EditorController.showMessage (C:\Users\Bet\.atom\packages\haskell-ghc-mod\lib\editor-controller.coffee:51:30)
  at C:\Users\Bet\.atom\packages\haskell-ghc-mod\lib\editor-controller.coffee:1:1
  at C:\Users\Bet\.atom\packages\haskell-ghc-mod\lib\editor-controller.coffee:104:7
  at C:\Users\Bet\.atom\packages\haskell-ghc-mod\lib\ghc-modi-process.coffee:114:9
  at Socket.<anonymous> (C:\Users\Bet\.atom\packages\haskell-ghc-mod\lib\ghc-modi-process.coffee:62:9)
  at Socket.g (events.js:202:16)
  at Socket.emit (events.js:138:20)
  at readableAddChunk (_stream_readable.js:164:16)
  at Socket.Readable.push (_stream_readable.js:127:10)
  at Pipe.onread (net.js:530:20)

Commands
  3x -2:44.6 core:backspace (atom-text-editor.editor.mini)
     -2:41.8 core:confirm (atom-text-editor.editor.mini)
  4x -2:28.2 core:move-down (atom-text-editor.editor.mini)
     -1:54.9 editor:select-to-first-character-of-line (atom-text-editor.editor.mini)
     -1:52.4 core:confirm (atom-text-editor.editor.mini)
     -1:05.0 haskell-ghc-mod:type (atom-text-editor.editor)
     -1:03.5 pane:reopen-closed-item (atom-text-editor.editor)
     -0:54.1 git-plus:add (atom-text-editor.editor)

Config
{
  "core": {
    "themes": [
      "atom-dark-ui",
      "monokai-dark"
    ],
    "disabledPackages": [
      "build-systems"
    ]
  },
  "haskell-ghc-mod": {
    "ghcModPath": "C:\\Users\\Bet\\AppData\\Roaming\\cabal\\bin\\ghc-mod.exe",
    "ghcModiPath": "C:\\Users\\Bet\\AppData\\Roaming\\cabal\\bin\\ghc-modi.exe"
  }
}

Installed Packages
# User
autocomplete-glsl, v0.1.3
autocomplete-haskell, v0.2.0
autocomplete-paths, v1.0.2
autocomplete-plus, v2.1.6
build, v0.23.4
color-picker, v1.4.4
editor-stats, v0.17.0
editorconfig, v0.3.0
git-clone, vundefined
git-plus, v4.4.2
git-projects, v1.7.1
haskell-ghc-mod, v0.4.1
ide-haskell, v0.3.3
language-glsl, v1.0.1
language-haskell, v1.0.0
linter-hlint, v0.3.1
monokai-dark, v1.0.0

# Dev
No dev packages

Can't reproduce. Could you test with Atom 0.181.0? (since it looks a lot like bug in Atom 0.180.0) If it still happens, could you list actions needed to reproduce, or show a minimal example?

I update atom to 0.181.0. The error still.
Minimal example is yesod default scaffolded site. Are you have yesod installed?
On small programm (< 200 code rows) I can't reproduce error.

Tested with default yesod 1.4.1 site (created with yesod init with simple db). Still couldn't reproduce.
Would you mind running
atom.packages.getActivePackage('haskell-ghc-mod').mainModule.process.getType(atom.workspace.getActiveEditor().getText(),atom.workspace.getActiveEditor().getSelectedBufferRange(),function(range,type){console.log(range.start); console.log(range.end); console.log(type);})

in dev console with your haskell source open and text cursor on symbol you can't get type of, and sending me its output? (it may take a couple seconds before any meaningful output)
Please also include output from
atom.workspace.getActiveEditor().getBuffer().getRange().end

If you set cursor on import statement something like this:
import Text.Jasmine (minifym)
And then press Ctrl+Alt+I
Or run your command in Dev mode:
atom.packages.getActivePackage('haskell-ghc-mod').mainModule.process.getType(atom.workspace.getActiveEditor().getText(),atom.workspace.getActiveEditor().getSelectedBufferRange(),function(range,type){console.log(range.start); console.log(range.end); console.log(type);}) undefined events.js:94 Uncaught Error: This socket has been ended by the other party atom.workspace.getActiveEditor().getBuffer().getRange().end Point {row: 150, column: 0, copy: function, negate: function, freeze: function…}
It is another error, if I press Ctrl+Alt+T
got something strange from ghc-mod type: ["ghc-mod.exe: : cannot satisfy -packa…d-static-<IP_ADDRESS>-37b95737a41b298f05cbaf886276d038", unique: function]0: "ghc-mod.exe: : cannot satisfy -package-id yesod-static-<IP_ADDRESS>-37b95737a41b298f05cbaf886276d038"length: 1
The error "Cannot read property 'length' of undefined" show me after first start of Atom. After 10-20 seconds, I see another error "This socket has been ended by the other party"

Ok, it makes some semblance of sense now. First, half those errors are from ide-haskell. Second, try running ghc-mod check Application.hs in your project directory. You'll likely find that ghc-mod spits out errors. Try correcting those.
Note that cabal sandboxes have very limited support both in ide-haskell and this package, so if you're using them, you may be out of luck for now.
You may also want to uncheck some ticks in haskell-ghc-mod settings, since you're using ide-haskell anyway.
Please also note, that ghc-modi does not support spaces in project path. This is a bug in ghc-mod, so nothing I can do.

Closing due to no activity (probably solved)

FWIW, I'm seeing this error consistently today.
While I have experience with programming in general, I'm new to Haskell, but if you want more details, I'll be happy to supply them, if only you tell me what I need to do to obtain them.
This error message appears in the Atom editor, where I'm using these packages:

language-haskell (version 1.4.1)
haskell-ghc-mod (version 0.8.13)
ide-haskell (version 0.7.1)
autocomplete-haskell (version 0.4.5)

I don't expect that this in itself is enough information, but I'll be happy to provide more data upon request.

@ploeh, if you could create an isolated test case for this, this would be best. But I would guess that's not easy, if at all possible. If you could start with providing Atom version you're running, OS and stack traces for this error, that would probably be good enough. You can copy Atom's error report to clipboard (click little clipboard icon above "View this Issue" button on error message), which would provide this information.

I'll see if I can chase down a simpler repro, but I'm currently working my way through Real World Haskell, having now reached chapter 5. In that chapter, the authors are explaining how to develop a library, and I'm typing in the code as I read along.
Here are some steps that seem to reproduce the issue on my machine.
Add the following files to the same directory:
SimpleJSON.hs
module SimpleJSON
  (
    JValue(..)
  , getString
  , getInt
  , getDouble
  , getBool
  , getObject
  , getArray
  , isNull
  ) where

data JValue =
    JString String
  | JNumber Double
  | JBool Bool
  | JNull
  | JObject [(String, JValue)]
  | JArray [JValue]
    deriving (Eq, Ord, Show)

getString :: JValue -> Maybe String
getString (JString s) = Just s
getString _ = Nothing

getInt (JNumber n) = Just (truncate n)
getInt _ = Nothing

getDouble (JNumber n) = Just n
getDouble _ = Nothing

getBool (JBool b) = Just b
getBool _ = Nothing

getObject (JObject o) = Just o
getObject _ = Nothing

getArray (JArray a) = Just a
getArray _ = Nothing

isNull v = v == JNull

PrettyStub.hs
module PrettyStub where

import SimpleJSON

data Doc = ToBeDefined
  deriving (Show)

-- string :: String -> Doc
-- string str = undefined

text :: String -> Doc
text str = undefined

double :: Double -> Doc
double num = undefined

(<>) :: Doc -> Doc -> Doc
a <> b = undefined

Open the folder with these two files in Atom.
Open the PrettyStub.hs file.
Save the file (Ctrl+s, or just use the File -> Save menu). This triggers the error. It's not necessary to edit the file first.

Atom Version: 1.0.7
System: Microsoft Windows 10 Pro
Thrown From: haskell-ghc-mod package, v0.8.13
Stack Trace
Uncaught TypeError: Cannot read property 'length' of undefined
At C:\Users\mark\.atom\packages\haskell-ghc-mod\lib\util.coffee:100

TypeError: Cannot read property 'length' of undefined
    at Object.module.exports.Util.pointWithTabsRev (C:\Users\mark\.atom\packages\haskell-ghc-mod\lib\util.coffee:142:32)
    at C:\Users\mark\.atom\packages\haskell-ghc-mod\lib\ghc-modi-process.coffee:331:28
    at Array.forEach (native)
    at queueCmd.callback (C:\Users\mark\.atom\packages\haskell-ghc-mod\lib\ghc-modi-process.coffee:321:17)
    at C:\Users\mark\.atom\packages\haskell-ghc-mod\lib\ghc-modi-process.coffee:166:11
    at BufferedProcess.exit (C:\Users\mark\.atom\packages\haskell-ghc-mod\lib\ghc-modi-process-base.coffee:124:20)
    at triggerExitCallback (C:\Users\mark\AppData\Local\atom\app-1.0.7\resources\app.asar\src\buffered-process.js:213:47)
    at ChildProcess.<anonymous> (C:\Users\mark\AppData\Local\atom\app-1.0.7\resources\app.asar\src\buffered-process.js:235:18)
    at emitTwo (events.js:87:13)
    at ChildProcess.emit (events.js:169:7)

Commands
     -7:27.2.0 core:save (atom-text-editor.editor.is-focused)
     -7:27.1.0 ide-haskell:check-file (atom-text-editor.editor.is-focused)
     -7:27.1.0 ide-haskell:lint-file (atom-text-editor.editor.is-focused)
     -0:35.2.0 pane:show-next-item (atom-text-editor.editor.is-focused)
     -0:31.9.0 core:move-to-bottom (atom-text-editor.editor.is-focused)
     -0:31.4.0 editor:newline (atom-text-editor.editor.is-focused)
     -0:21.8.0 editor:move-to-end-of-screen-line (atom-text-editor.editor.is-focused.autocomplete-active)
     -0:12.8.0 editor:newline (atom-text-editor.editor.is-focused)
     -0:04.1.0 autocomplete-plus:confirm (atom-text-editor.editor.is-focused.autocomplete-active)
     -0:03.3.0 core:save (atom-text-editor.editor.is-focused)
     -0:03.3.0 ide-haskell:check-file (atom-text-editor.editor.is-focused)
     -0:03.3.0 ide-haskell:lint-file (atom-text-editor.editor.is-focused)

Config
{
  "core": {},
  "haskell-ghc-mod": {}
}

Installed Packages
# User
autocomplete-haskell, v0.4.5
haskell-ghc-mod, v0.8.13
ide-haskell, v0.7.1
keyboard-localization, v1.4.7
language-haskell, v1.4.1

# Dev
No dev packages

Mea culpa. I didn't think a fix for another issue trough. Reverted that for now, 0.8.14 should be good.

That was fast :open_mouth:
Initial testing seems to indicate that this has resolved the problem. :+1:
Thank you


Fixed python wrapping to use qt5 with PythonQT qt5 version from sourcefo...

Our application depends on Qt5 and uses the ctk python wrapping with PythonQt. We propose a fix to use the new PythonQT 3.0 from sourceforge with ctk and qt5 fixes for the ctkPythonWrapping. If the PYTHONQT_INSTALL_DIR is set during superbuild CTK will try to use an external PythonQT else the build process will remain the same.

@jcrf could you please review this. It would help us to move on with the MITK-Qt5-Python integration. Currently the patch only supports an externally built PythonQt 3 for CTK, but I suggest we postpone the full superbuild integration until PythonQt 3 supports CMake, currently it only provides a qmake pro file.

@jcfr @jcrf Sorry, mixed some letters ...

+1 this would be great.  I'd love to see slicer support Qt5 as well (not
sure what else would need changing, but this would be a great start)
On Wed, Feb 11, 2015 at 12:43 PM, Marco Nolden<EMAIL_ADDRESS>wrote:

@jcfr https://github.com/jcfr @jcrf https://github.com/jcrf Sorry,
mixed some letters ...
—
Reply to this email directly or view it on GitHub
https://github.com/commontk/CTK/pull/538#issuecomment-73926885.
The information in this e-mail is intended only for the person to whom it
is
addressed. If you believe this e-mail was sent to you in error and the
e-mail
contains patient information, please contact the Partners Compliance
HelpLine at
http://www.partners.org/complianceline . If the e-mail was sent to you in
error
but does not contain patient information, please contact the sender and
properly
dispose of the e-mail.

@eric-h Thanks for your contribution.  Few minor issue to address and it should be good.
On last thing, could you also add a message(FATAL_ERROR "To build CTK with Qt >= 5 and wrapping enabled, you are currently required to provide your own PythonQt by re-configuring CTK with option -DPYTHONQT_INSTALL_DIR:PATH=/path/to/PythonQt-30-install").
This should be added here: https://github.com/commontk/CTK/blob/59dfc7f24b0355be907a030a351fbd9f065794bf/CMakeExternals/PythonQt.cmake#L35

The Qt5::Widgets dependency is needed, since there are C++ classes generated by the ctkWrapPythonQt.py script that  include QWidget.

Thanks for the quick answer.
Since ctkWrapPythonQt.py generates the decorators based on the list of classes available in each Libs folder (Libs/Core, Libs/Widget, ... ), do you know which generated classes require Qt5::Widget ? I just want to make sure to understand what lead to that requirement.

The generated headers with QWidget are:
org_commontk_CTKCore.h
org_commontk_CTKDICOMCore.h
org_commontk_CTKPluginFramework.h

@eric-h I just pushed a new topic including your changes. See https://github.com/commontk/CTK/compare/master...jcfr:fix-python-qt5
Would be great if you could test and let me know how it goes.

@jcfr I just tested your branch. It works fine for us.

Great, thanks for the follow up :+1:  Before integrating, could  you also comment on https://github.com/commontk/CTK/compare/master...jcfr:fix-python-qt5#commitcomment-10113444 ?

Adding the ${PYTHONQT_INSTALL_DIR}/src dir was needed in the search path since the PythonQt 3.0 doesn't build with cmake and had no standard install/include folder, depending on if the QtCreator project is used on Linux or Visual Studio on windows the PythonQt.h header can end up in build/src rater than in build/include where it should be.

Et voila.
Fixed in 9ad3253f123ceec5247a147db1c9d1ee688fdbdb


dgl.copy_edge deprecated

change to dgl.copy_e() https://docs.dgl.ai/en/0.8.x/generated/dgl.function.copy_edge.html#

change to dgl.copy_e() https://docs.dgl.ai/en/0.8.x/generated/dgl.function.copy_edge.html#

Thank you for your advice on this problem.
By the way, have you met the nan problem?
complex_names where Nan encountered
I suspect this results from the data process part, because a lot of warnings has been reported like "rdkit coords could not be generated without using random coords. using random coords now". I do not know how to solve it.

Is there anything I can do to avoid this nan error? Thank you !


Safari 16: Add note about Push only being available in macOS Ventura

And changed iOS to n as it's not coming until 2023.
https://webkit.org/blog/12945/meet-web-push/

Good points!

Thanks!


Add option to use tiers.

Barter has an option for users to tag games in tiers.
(low medium high highest)
please add an option to ignore certain tiers.

Currently this is not possible. I need barter to add this information to your tradable list JSON/API.


Improvement and Feature request

Hi Keijiro,
First of all this is really awesome works man.
and yeah i know that this version are prototype and you will improve it again in the future, so i just want to request some feature that maybe you can added in the future.
Mostly it would be cool if we can adapt the guerilla games volumetric rendering tech into this cloud

1 RGBA 2DTexture noise for cloud curling/distortion can be used perchannel
1 RGBA gradient for perchannel falloff for height & distance & and curl value
Descend the cloud into horizon for seamless transition
not quite sure about this
distance coverage multiplier
:)

Just in Case :)
https://www.guerrilla-games.com/read/the-real-time-volumetric-cloudscapes-of-horizon-zero-dawn

if you don't mind posting here, some inspiration here also :)
Reset Atmosphere Tech FINAL
https://www.youtube.com/watch?v=FnmWuCtxKw8


fix: getting clientWidth on null object

@bokuweb Sorry about the fail. Maybe you could fix this by your self?

@edgesoft thx, ok.

@bokuweb Would be great to see the fix. When resizing a get a lot off error messages. The same goes for removing callback in setState. Also produces errors.

@bokuweb You still have the callback in setState. This will produce error.

@edgesoft An error does not reproduce my env. callbacks is needed to open modal when  isOpen default state is true.Please tell us about your environment and error log.

@bokuweb
invariant.js?4599:38 Uncaught Invariant Violation: setState(...): Expected the last optional `callback` argument to be a function. Instead received: Event (keys: isTrusted).

When I remove callback from setState the error goes away.

SetDefaultState can still contain callback.

@edgesoft  componentDidMount need this.resize and callbacks.


Adding tags to the stackoverflow command

Description

The proposal is that we allow tags to be specified as another argument for the stackoverflow command. Stack Exchange API allows filtering by tags with the query, so if the user would like to specify the tags for a certain query, i,e. .so JSON [tags], the stackoverflow command would add the additional tagged parameter with the Stack Exchange API with the query 'JSON'.
Reasoning

The current stackoverflow command will search for the top 5 results, but it can't filter by tag. If someone wanted a specific query specific to a certain topic, they can filter by tags with this feature to get the tags for their query relating to said specific topic.
Proposed Implementation
Using the parameter tagged={} when using the StackExchange API, you can filter by tags. Below are some screenshots of filtering by tag in usage:
This shows an example of filtering by one tag, with 'JSON' being the query and 'python' being the tag to filter by.

This shows an example of filtering by multiple tags, with 'JSON' being the query and 'python' and 'mongodb' being the two tags (separated by the delimiter ',') that must be in the answer along with other tags for it to show.

Do note that the tags only go up to the first 3 tags, so while sometimes the tag you filtered by may not show, the tag is in the overall count of tags, just not the first three tags.
Would you like to implement this yourself?

[x] I'd like to implement this feature myself
[ ] Anyone can implement this feature

I would guess a flag would be the best way to do this? Maybe -t
Then a user would do .so  -t

I would guess a flag would be the best way to do this? Maybe -t
Then a user would do .so  -t

That's true. I can update it to take a flag instead.

This is my first shot at it, definitely could clean up the string combination as well as having an embedded error message for more than 5 tags.
    async def stackoverflow(self, ctx: commands.Context, *, search_query: str, ) -> None:
        """Sends the top 5 results of a search query from stackoverflow."""
        def tag_parse(query: str) -> str:

            list_query = query.split()
            if "-t" in list_query:
                stop_index = list_query.index("-t")
                if len(list_query) - stop_index < 5:
                    query = " ".join(term for term in list_query[:stop_index])
                    tags = " ".join(term for term in list_query[stop_index+1:])
                else:
                    raise ValueError(f"Too many tags were passed to the command: {query}")
            else:
                query = query
                tags = None

            return query, tags

        search_query, tags = tag_parse(search_query)
        search_query += f"tagged={tags}"

I had an idea in the original PR, maybe you could allow people to search using the stackoverflow search syntax:

and use a regular experession to get and pass in the tags, authors, etc. This would not only make it neater, but also provide support for other filters like authors and whether or not the question is answered.

I had an idea in the original PR, maybe you could allow people to search using the stackoverflow search syntax:

and use a regular experession to get and pass in the tags, authors, etc. This would not only make it neater, but also provide support for other filters like authors and whether or not the question is answered.

That would be pretty cool to be honest, I can work on adding other features too, I presume the Stack Exchange API allows all of these parameters in one API request?

I had an idea in the original PR, maybe you could allow people to search using the stackoverflow search syntax:

and use a regular experession to get and pass in the tags, authors, etc. This would not only make it neater, but also provide support for other filters like authors and whether or not the question is answered.

That would be pretty cool to be honest, I can work on adding other features too, I presume the Stack Exchange API allows all of these parameters in one API request?

Yup, here are the parameters allowed:
q - a free form text parameter, will match all question properties based on an undocumented algorithm.
accepted - true to return only questions with accepted answers, false to return only those without. Omit to elide constraint.
answers - the minimum number of answers returned questions must have.
body - text which must appear in returned questions' bodies.
closed - true to return only closed questions, false to return only open ones. Omit to elide constraint.
migrated - true to return only questions migrated away from a site, false to return only those not. Omit to elide constraint.
notice - true to return only questions with post notices, false to return only those without. Omit to elide constraint.
nottagged - a semicolon delimited list of tags, none of which will be present on returned questions.
tagged - a semicolon delimited list of tags, of which at least one will be present on all returned questions.
title - text which must appear in returned questions' titles.
user - the id of the user who must own the questions returned.
url - a url which must be contained in a post, may include a wildcard.
views - the minimum number of views returned questions must have.
wiki - true to return only community wiki questions, false to return only non-community wiki ones. Omit to elide constraint.

Ah alright, thanks!

Hey are you still planning on working on this? Someone sent me this link for Discord Flags: https://pypi.org/project/discord-flags/might be helpful to use these for all of the parameters? Could default most of them to None.

Hey are you still planning on working on this? Someone sent me this link for Discord Flags: https://pypi.org/project/discord-flags/might be helpful to use these for all of the parameters? Could default most of them to None.

Hey @brad90four, I was thinking that there is no need for using flagged arguments, as I have mentioned here.

Hey are you still planning on working on this? Someone sent me this link for Discord Flags: https://pypi.org/project/discord-flags/might be helpful to use these for all of the parameters? Could default most of them to None.

Hey @brad90four, I was thinking that there is no need for using flagged arguments, as I have mentioned here. This would even add functionalityfor more filters.

So I presume we would be using Regex?

Planning can continue, if possible.


Better support for OBS

Need to add:

an HTML output which can be CSS skinned with auto refresh
instructions
way to display the link to the user

Consider it done, although it's somewhat basic. But, it works!


[ 4º ] - Hotfix/v2.1.1: Solved vulnerabilities and frontend bugs

Why we create this PR?
There is one vulnerability in the application related to the axios library. There several bugs found in the E2E tests in the frontend component, related with the translations.
What we want to achieve with this PR?
Resolve the bugs with the frotend and prepare the release v2.1.1
What is new?
Added

Added missing german translations to the transmission pass
Added missing german translations to the policy selection feature

Updated

Updated axios library from v1.6.0 to v1.6.7 to fix vulnerability

Issues Fixed

Fixed tabs bugs related to translations
Fixed translations of battery pass and digital product pass
Fixed missing additional data visualization
Fixed missing total sources visualization

Me, @dsrparracho and @saudkhan116 tested this version and everything is ok!

Ok done with the testing everything is fine!


_ variable in for loop causes segfault

The following code
actor Main
  new create(env: Env) =>
    for _ in ["foo"].values() do "" end

results in compilation error
Segmentation fault: 11

The key point is variable's name _. Changing it to a valid name makes a program compile successfully.

Mac OS (10.10.5). Pony: 0.2.1-473-gd45c520 (d45c520b3d6ee078fff299e7c299ec4c615dc018)

Yep that's a bug, thanks for the report.
It should be a simple fix, we're just producing the wrong AST for "don't cares" in for iterators (and with variables as well).

Fixed

There seems to be a new, likely-related problem (using ponyc built today (9a3a8a524fb2711fb9fd2b6cf7175c223ddd7390)):
$ ponyc --version
0.2.1-672-g901983a
$ cat temp.pony
actor Main
  new create(env: Env) =>
      for _ in ["foo"].values() do "" end
$ ponyc .
Building builtin -> /Users/hugo/Projects/ponyc/packages/builtin
Building . -> /Users/hugo/Projects/langbench
Error:
/Users/hugo/Projects/langbench/temp.pony:3:11: the don't care token can only appear in a tuple, either on the LHS of an assignment or in a pattern
      for _ in ["foo"].values() do "" end
          ^
Error:
/Users/hugo/Projects/langbench/temp.pony:3:11: left side must be something that can be assigned to
      for _ in ["foo"].values() do "" end
          ^

That looks entirely correct to me.
_ is not a legal variable name, it is the "don't care" symbol. The first error tells you where "don't care" can appear, which isn't here.
The second error is correct, although it's not very helpful, so maybe we should get rid of it. But I wouldn't call it a problem.
This issue was originally raised because this erroneous code caused a crash rather than giving a sensible error message, which it now does.

Thanks, that makes sense. I understood that _ was don't-care ... I wanted to be able to be able to ignore the loop variable. Would you consider amending the syntax to permit don't-care for loops? It's quite common to need to repeat something N times, but not need access to the loop variable. (e.g. for _ in Range(0,5) do ... instead of for dummy in Range(0,5) do ...). There doesn't seem to be a better control flow structure for that.

Sorry, I misunderstood what you were saying.
We certainly could allow for loops to take _ for unused index variables. But is it worth it? Is:
for x in Range(0,5) do ...

(where x is any variable name you don't use) actually any clearer than:
for _ in Range(0,5) do ...

I'm not sure really.
If you think it is a big enough deal then please open a new issue requesting it.