build: move back to 999-SNAPSHOT

At this point, the release of Timefold Solver Community Edition is ready to be published.
Artifacts have been uploaded to a staging repository.
Release branch has been created.
To finish the release of Timefold Solver Community Edition,
please follow the steps below in the given order:

[ ] Release the staging repository on OSSRH.
[ ] Wait for the artifacts to reach Maven Central.
[ ] Undraft the release on Github. This will cause the release tag to be published, enabling the downstream release scripts.
[ ] Merge this PR. (Only do this after undrafting the release, otherwise the tag will point to the final SNAPSHOT commit.)
[ ] Delete the branch that this PR is based on. (Typically a button appears on this page once the PR is merged.)
[ ] Start release automation for Timefold Quickstarts.
[ ] Start release automation for Timefold Solver Enterprise Edition.

Note: If this is a dry run,
none of the above applies and this PR should not be merged.

Dry run, closing.


Premium Storage + GRS

As it says in the documentation : "Premium Storage supports only locally redundant storage (LRS)."
I'm not sure I understand why. I want fastest possible access to files with an 'automatic backup'.
Where can I find a better technical analysis of the unsupported combination of Premium + GRS.

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

ID: f618f40d-ae53-5c3b-d8bf-0e80614f66d3
Version Independent ID: 93083407-8f1e-b3e2-6a62-e30be5e1fbcb
Content: Data replication in Azure Storage
Content Source: articles/storage/common/storage-redundancy.md
Service: storage
GitHub Login: @tamram
Microsoft Alias: tamram

@roygara - FYI

@wmgries thanks for the additional information. Actually I was aware that this referred to 'SSD' in this context. I was just trying to get as fast as possible disk for my storage plan with replication.
I can see why for a VM disk it has to be LRS - it's just not practical to replicate it, but what I really wanted was just replication of all my blobs. We're living in crazy times and sometime in the next 5, 10, 50 years a datacenter is just going to disappear off the map for whatever reason. That's what I wanted to protect against here, but while having the fastest possible access.
I think perhaps a better solution is installing Azure file sync to my VM's temp SSD and revert the storage back to GRS. But would also be great if in the relevant places a link to an explanation of why it is like this - as well as links to alternative solutions (like file sync).

@simeyla - Thanks, it's a good suggestion. I've made a note for us to clarify that in the content going forward.
#please-close

@simeyla We will now close this issue. If there are further questions regarding this matter, please reply to @femsulu and we will gladly continue the discussion.


Column classification is wrong: It uses predicates to classify columns

Develop branch as of 7.9. Testing file:
Books-v1.csv.zip, the classification for "series" is wrong
For the "series" column,

I got:

But these suggested resources are predicates, not classes. SO they may appear in Step 3 (where relations are suggested), but not in Step 2 (where we are looking for the classes).
In general: Classification (Step 2) must not use predicates as resources classifying the columns
Note: To recognize predicate, it is instance of rdf:Property

The full DBpedia returns Category, but the limited DBpedia returns the correct DBpedia link. Why?

@brodecva I have finally completed the refactoring of SPARQL queries and added the restriction to exclude properties from the search. But I am getting some weird server error with no information in the log. All the test executions are working, are there any problems with the tasks currently?

@JanVana Everything works just fine for me, even after merging your contribution.

@JanVana So what was the root cause of that issue?

@tomas-knap There was no restriction on the type of the searched items before. I have added one, so the query looks like this now:
SELECT DISTINCT ?Subject WHERE
{
?Subject rdfs:label ?Label .
?Label bif:contains '"Steven Erikson"' .
?Subject a ?Class .
?Class a owl:Class .
}

Ok, but please make it customizable (the part: ?Class a owl:Class), so that it reads from the KB which resource is used for expressing "a class". I think there is such a property in the KB already, right? In most cases it is "owl:Class", but could be also just "skos:Concept" in certain cases.

If it is already done in that way, please ignore

Class, label and bif:contains are all customizable in the current version. The rdf type is not, but it could be done.


Make logo appear clearly on retina displays

Images like jpg or png should have source images twice the size of the display area in order to display clearly on retina displays.
Before:

After:

Thanks so much! How did you make the existing logo file larger?

You’re welcome! You had the original png in the public folder.

@fritz-c Hi. I just checked out the new master, and the logo isn't showing up for me at all with your new logo code. Any idea why that might be? I'll go in and debug later, but wondered if you might have an idea since it was your code.
Thanks!

I'm sorry to say I'm not sure what the problem might be. Please keep me updated if you find out the cause.

When you clone down master, do you see the new logo?
On Tue, Oct 17, 2017 at 8:41 PM, Fritz<EMAIL_ADDRESS>wrote:

I'm sorry to say I'm not sure what the problem might be. Please keep me
updated if you find out the cause.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/align-capstone/align/pull/225#issuecomment-337423533,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANNkKmYcT3orE9rc_2mDBNF5nHpet7xKks5stUkygaJpZM4P5UFH
.

--
Sophia Ciocca
http://www.sophiaciocca.com

Yeah, it's working from the latest master branch. Do you get any console errors, like a 404?
Could it be that your bundle code isn't up to date, and you need to run npm run build?

Ok, I got it working (it was a browser-caching problem!) but the new logo's actually coming out kind of fuzzy on my non-retina display. Here's the old logo vs. the new logo:
old logo:

your new retina display logo:

Thoughts on this?


Flexible callbacks for clients and services

We should apply the technique from https://github.com/ros2-rust/ros2_rust/pull/260 to client and service callbacks.
Specifically, we could allow taking the request/response in a Box, and to optionally take a ServiceInfo argument (corresponding to rmw_service_info_t).

@esteve You said you were interested in this?


Remote testing dialog not accurate

The drop down shown on this page does not exist.  Instead there is a test "filter" that does not do what this page explains.

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

ID: 90ec9455-ef64-8854-94bc-9be28887a6a4
Version Independent ID: 9adeea5e-7ba1-065f-5de0-aae9147f9c85
Content: Remote Testing in Visual Studio - Visual Studio (Windows)
Content Source: docs/test/remote-testing.md
Product: visual-studio-windows
Technology: vs-ide-test
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@michaeljunkin What version of Visual Studio are you using? @kendrahavens for awareness.

Visual Studio 2022.

@michaeljunkin The drop down only appears if you have a testenvironments.json in the root of your solution where you define the environments that you have set up for remote testing. https://docs.microsoft.com/en-us/visualstudio/test/remote-testing?view=vs-2022#set-up-the-remote-testing-environment

@kendrahavens -- Kendra, thank you for clarifying.
I am closing this issue now.  You are welcome to @ mention me for any followup.

I had the same issue - that being that no environment dropdown was showing. I had to restart VS 2022 in order for it to show up the first time.

Ah, @GaTechThomas Thank you for mentioning that! I'll follow up with the team and figure out if we always need a restart (and change the documentation) or if we can have it appear as soon as the file is added.

@GaTechThomas I heard back from the engineers. A restart should never be required, we definitely built it to automatically detect the file and update the Test Explorer so we aren't seeing this normally. If you see this again or are able to repro at all, would you mind filing a bug? https://developercommunity.visualstudio.com/
Again, thank you for mentioning the issue!


Unable to activate knife-solo-0.5.1

When I install chef dk on windows, I try to run "knife solo" :
D:\Users\Matthieu\Documents\Devstack>knife solo
D:/App/Opscode/chefdk/embedded/lib/ruby/site_ruby/2.1.0/rubygems/specification.rb:2288:in `raise_if_conflicts': Unable to activate knife-solo-0.5.1, because net-ssh-3.1.1 conflicts with net-ssh (< 3.0, ~> 2.7) (Gem::ConflictError)
        from D:/App/Opscode/chefdk/embedded/lib/ruby/site_ruby/2.1.0/rubygems/specification.rb:1408:in `activate'
        from D:/App/Opscode/chefdk/embedded/lib/ruby/site_ruby/2.1.0/rubygems.rb:200:in `rescue in try_activate'
        from D:/App/Opscode/chefdk/embedded/lib/ruby/site_ruby/2.1.0/rubygems.rb:193:in `try_activate'
        from D:/App/Opscode/chefdk/embedded/lib/ruby/site_ruby/2.1.0/rubygems/core_ext/kernel_require.rb:126:in `rescue in require'
        from D:/App/Opscode/chefdk/embedded/lib/ruby/site_ruby/2.1.0/rubygems/core_ext/kernel_require.rb:40:in `require'
        from C:/Users/Matthieu/AppData/Local/chefdk/gem/ruby/2.1.0/gems/knife-solo-0.5.1/lib/chef/knife/cook.rb:1:in `<top (required)>'
        from D:/App/Opscode/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.9.41/lib/chef/knife/core/subcommand_loader.rb:100:in `load'
        from D:/App/Opscode/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.9.41/lib/chef/knife/core/subcommand_loader.rb:100:in `block in load_commands'
        from D:/App/Opscode/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.9.41/lib/chef/knife/core/subcommand_loader.rb:100:in `each'
        from D:/App/Opscode/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.9.41/lib/chef/knife/core/subcommand_loader.rb:100:in `load_commands'
        from D:/App/Opscode/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.9.41/lib/chef/knife/core/subcommand_loader.rb:110:in `load_command'
        from D:/App/Opscode/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.9.41/lib/chef/knife/core/subcommand_loader.rb:124:in `command_class_from'
        from D:/App/Opscode/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.9.41/lib/chef/knife.rb:153:in `subcommand_class_from'
        from D:/App/Opscode/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.9.41/lib/chef/knife.rb:214:in `run'
        from D:/App/Opscode/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.9.41/lib/chef/application/knife.rb:148:in `run'
        from D:/App/Opscode/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.9.41/bin/knife:25:in `<top (required)>'
        from D:/App/Opscode/chefdk/bin/knife:61:in `load'
        from D:/App/Opscode/chefdk/bin/knife:61:in `<main>'

I've the problem since 0.10. If I install knife-solo gem out from chefdk, it's work. I think just a dependency problem.

I'm also running into this issue. OS X 10.11.4

I'm also running into this issue. Any helps would be appreciated.
Thanks.

Same here, OS X 10.11.4
griller:test jkulak$ chef --version
Chef Development Kit Version: 0.13.21
chef-client version: 12.9.41
berks version: 4.3.2
kitchen version: 1.7.3

Tried to install net-ssh with version 2.7.0, but it produced same errros:
sudo gem install net-ssh -v 2.7.0

It' a knife-solo issue, that is being addressed here: https://github.com/matschaffer/knife-solo/commit/3b3fa546f24fcee291ef73315e0e9729894f6500
It's already merged in the upcoming https://github.com/matschaffer/knife-solo/tree/release/0.6.0

In the meanwhile, I downgraded chefdk to 0.9 and it works (any version above doesn't work at all).

Version 0.6.0 of knife-solo has been released, and it works with the Chef Development Kit Version: 0.14.25.
This issue can be closed.

A new chefdk should be released which includes this, it may be a knife-solo issue, but it breaks knife entirely.

uninstall knife-solo and keep knife-solo_data_bag, conflict issue will be resolved. It worked for me.

The 0.6.0 release of knife-solo now conflicts with most current chefdk versions due to net-ssh 4.x pinning.
That is strictly an issue in knife-solo, which has actually been fixed in master:
https://github.com/matschaffer/knife-solo/commit/d2a4d6e0fc2aad5f6bcdcea1fb4269c1e6b58721#diff-b30df53cb156a27ee8ae67413fa1d6db
That has not been released to rubygems in a 0.7.0 release, so the only alternative right now is installing the 0.7.0.pre3 knife-solo gem.


Rows removed while user is scrolling aren't always removed.

A user submits an ajax request to remove row data to the backend. Once the row data removal is confirmed in the response, a new data source is constructed with the relevant rows removed/omitted. If the user is scrolling while the grid data source is changed, not all of the rows from the previous data source are removed from the screen.

That sounds somewhat complex to replicate.
Firstly, what version are you using - 2.x or 3.x?  Secondly, is it possible to create a plunker that demonstrates the issue?

No response in 2 weeks. @roguezedd if this is still a problem with rc.20 please file a new issue with some steps to reproduce.  I think it should be possible to check by using a $timeout that removes rows from the data set and just manually scrolling during it.


Fix bug with wrong mutation behaviour

I want to merge this change because it fix bug with inappropriate mutation behaviour.

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-2674 Insufficient stock error on setting quantity to 0


⌨ Keyboard shortcuts

[ ] ALT + 1,2,3 (switch tabs)
[ ] ALT + CMD + left/right (tabs relative, left / right)
[x] <, > (cell value + / -)
[ ] arrows (table cells navigation)
[x] = (change cell values)
[x] CMD + \ (toggle sidebar)
[x] arrows in logger

Should be implemented in the app https://github.com/speedy-tuner/speedy-tuner


iFRAME(s)

x-frame-options: Allows ADFS admins to allow specific relying parties to embed iFrames for ADFS interactive login pages. This should be used with care and only on HTTPS hosts.
Why are you using plural when you can only add 1 iFrame ??? Can you please explain ? Is it finally possible to allow multiple iFrames ? Thanks.

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

ID: 7899bfcb-2e9c-2738-5e7f-468a6eb643ce
Version Independent ID: 482cc911-c7ef-c680-b6ff-f634530b1424
Content: What's new in Active Directory Federation Services for Windows Server 2016
Content Source: WindowsServerDocs/identity/ad-fs/overview/whats-new-active-directory-federation-services-windows-server.md
Product: windows-server-threshold
Technology: identity-adfs
GitHub Login: @billmath
Microsoft Alias: billmath

#assign:IngridAtMicrosoft


Feature/CSS refactoring

Description
My original intention was to introduce a dark mode for webumenia.sk. However I found out that the project CSS was due to an update and I attempted to do one.
List of changes

add current Tailwind CSS to the project, update its build
remove all old and unused LESS and static CSS files
replace legacy Tailwind and Tachyons CSS with Tailwind only classes
replace some Bootstrap classes with Tailwind
optimize style and admin CSS builds

Todo

use npm versions of plugins (eg. Bootstrap, CKeditor, etc.), remove its code from the repository
introduce CSS properties
dark mode

Type of change

Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] I have updated the CHANGELOG
[ ] I have requested at least 1 reviewer for this PR
[ ] I have made corresponding changes to the documentation

Wow 🤩 . This PR looks like a miracle. We were thinking about doing all of these things before... but nobody really wanted to start. I appreciate your effort a lot.
We need some time to walk through all your changes (which are huge!). And double-check any possible issues.
We will definitely give you feedback soon.
Thanks a lot for this. You made my day.... ✨

Thank you for the feedback. I am glad you like the direction of my efforts.
This phase is IMO done, I would like to eventually meet with you someday and talk about testing this PR and future development.
I read something about dropping / replacing the Bootstrap (in the commit messages): this is the task I would love to help you with. I also like to 'atomize' (Tailwind-ize 😄) the CSS and to work on overall refactoring.
Btw I love all your work in lab.sng. I checked the Warhol microsite in the meantime and it blew my dekel off; splendid work!

Hi @adamplesnik 👋!
Thank you very much for your effort. It'll take a while for us to review all the changes, but from skimming the code, it looks like a major step in the right direction.
Right now I'm having trouble compiling the project assets (see e.g. here). Are you able to run npm run dev locally?
Note the project uses NodeJS v14 right now. That's is not a hard requirement AFAIK but if you're using a different version, please let us know.

I am sorry about this commit mess, I reverted the initial build and then reapplied it in a hurry.

I tried to fix all your findings. I think it is safe to enable Tailwind preflight, feedback is welcome.

@adamplesnik Thanks for keeping this branch up to date and sorry for the radio silence. We're busy with preparations for digital installations right now but I'll get back to you in two weeks latest.

@eronisko Thank you for your feedback. No need to hurry, good luck with other projects ;)

@adamplesnik just so you know, I've resumed working on this pull request. I'm putting together an automated visual test suite (Playwright) that'll help us verify the changes.

Hi @adamplesnik
I've created a Playwright report comparing screenshots between develop and this branch: https://drive.sng.sk/d/s/upCsGTpy3fF0UdjPFTuo96fHfNOqRB9O/hQLWVT0_5z_yzUXQYchKpONmqhAPAkUp-nL_gZLLzrAo
Please don't be discouraged by the 'failures' it contains. We're not expecting to be perfectly matched, this is generated to give us a starting point.
I believe most of the issues can be resolved by better matching the Bootstrap and Tailwind classes. There might be some trickier issues that we'd be happy to discuss with you.
Please go over the pages and see what needs to be fixed. The report also provides a slider to help you compare the two versions:

If you give us a shout, we can generate a new report for you. Alternatively, you merge your changes into the WEBUMENIA-1998-pr-832-playwright-feature branch and then run npx playwright test to generate a new report.

@eronisko I tried to fix all major issues. Would you please run the tests again? Thank you very much.

Thanks @adamplesnik. I'll get you the test results first thing on Thursday.

New test run, looks better I think:
https://drive.sng.sk/d/s/vaLZ4vSLp5IDI9nlKHPQ6Nqm6HoHXKOE/iFmH5PIBeQoKKeE3pXNF3u1uv9BuJao9-LbagJQeF0Qo


Change some preferences to be per-device using LocalStorage

Some user preferences like selected theme should be set for each user device. This way if users want one experience on mobile and a different on desktop, there's no hassle.

I'll revisit this another time, for now I think storing separate desktop/mobile preferences in the cloud will be sufficient.


Reactotron 1.5 - Sagas, Timeline Filter + More 💃

For people with existing Ignited apps:

upgrade existing reactotron dependencies in your package.json to 1.5.2
npm i --save-dev reactotron-redux-saga
register reactotron-redux-saga plugin within ReactotronConfig.js
configure createSagaMiddleware() to use a custom sagaMonitor in your CreateStore.js

And now you get this:

You can download the Reactotron app with Homebrew on macOS:
brew update && brew cask install reactotron

To upgrade from an old version:
brew update && brew cask upgrade reactotron

To just download the binaries like a savage.

gonna have to review and merge this at end of week, but so far looks good!

Just refreshed this PR to:

rebase master
upgrade redux-saga to 0.13.0 (no changes required in the app)
bring in latest Reactotron libraries (1.5.2)
added more sample usage in startup saga
update this PR description on the upgrade path for existing Ignited apps + how to get Reactotron


Update default.py

Having the sha1 & sha256 hashes by default would actually be pretty useful I think. This change would add those.

Ha, interesting. Initially, we've decided to minimize the output for the default mode and only include the other hash values in the more verbose outputs. Does using these modes work for you or do you consider these values to be crucial here?

thanks @threathive !


mobile view not instantly loading

when I load an MDL site in mobile, it will show the desktop view for 1 second and changes to mobile view after that.
steps to reproduce:
load https://techmastary.net/fg in mobile
observe change

This is because the layout JavaScript hasn't executed yet to assign the right classes for the mobile view to change.


Goroutine-safe?

Hello,
I'm currently using this library to do write kafka events into HBase. After skimming the code, there's a few questions that I hope you can help clarify for me.

Is the libary thread safe? Looking at https://github.com/tsuna/gohbase/blob/51c9c644410cad081633813d39a10cdba1c527e8/client.go#L559, it seems to me that if multiple goroutines are doing the same call, there's no guarantees that they'll receive the hrpc.Result associated with the hrpc.Mutate call.
The parameters rpcQueueSize and flushInterval seems like it's there to batch up requests. Are there any tips to tweaking these numbers for high throughput? (I'm currently looking at 10k events/second from kafka--but this number will increase to roughly 200k/second)

Thanks for your time!

Yeah the client is trying to be thread-safe.  If it's not, it's a bug.  I don't see any issue with the line you're referring to, can you clarify why you think that the hrpc.Result received would not be the one associated with the hrpc.Mutate?
For higher throughput, a longer queue / longer flush interval would be good, but that comes at the expense of higher latency and more RPC loss when the process gets killed uncleanly.  I don't have a good rule of thumb on how to set them for your use case, you'd have to experiment and see for yourself (please share results if you do!).
Please note that gohbase isn't considered v1 / stable yet.  There may still be API changes, and there are known and unknown bugs.  Please file issues / send pull requests if you want to help!

Hey tsuna, thanks for the quick response.
I might've misunderstood the code regarding point 1, my apologies!
The issue I was trying to track down was this:

(Blue line: runtime.MemStats.StackInuse, Purple line: runtime.MemStats.HeapAlloc from https://golang.org/pkg/runtime/#MemStats)
It seems like the culprit at least in my case was a combination of:

Not passing a timeout context (I was just passing context.Background())
Relying on the default rpcQueueSize (it seems like the default of 100 per 20ms was too low for our use case)

Tuning the rpcQueueSize to 10k per 20ms, as well as passing context.WithTimeout(context.Background(), 30*time.Second) seems to do the trick for me and the application is no longer crashing.

Hi @Taik , thanks for ur sharing. I'm doing the same thing, consuming kafka events into hbase. At first I think the hbase put latency wiil be an issue. So I make a channel between my consumer and writing handler. Consumer is responsible for consuming events and handler is responsible for formatting and writing into hbase. Both of them are goroutines. But when running at production environment, the lag of kafka consumer offset is becoming larger and larger whatever I increase the handler count or not. I feel really confusing about this situation. Does this situation ring a bell?  What's ur architecture?
Thank you!


Atom deems regexes with atomic groups or possessive quantifiers invalid

Atomic grouping and possessive quantifiers are extremely useful in regular expression. Supporting them can only speed up regex evaluation, especially for grammars that use them, as well as reduce the number of Find and Replace hangs that kill an Atom window.
Consider the regex \b(integer|integral|integrity|insert|int|in)\bmatched against integers. The engine will match the first alternative, fail at the \b, then backtrack into the group and repeatedly match, fail, and backtrack through every single alternative, even matching the group in the last two cases only to fail at the \b again. This is quite a lot of work just to figure out integers isn't in our list of words. On the other hand, the atomic group \b(?>integer|integral|integrity|insert|int|in)\b will immediately give up matching the group at the second \b and never backtrack.
Consider also the regexes "[^"]*" and "[^"]*+" matched against "aaaaaaaaaaaaaaaa. Both fail, but the former backtracks through the entire string, while the second fails immediately thanks to possessive quantifiers. When such behavior is desired (often for order-dependent parameters), this makes a huge difference for performance.
Both of these are simplistic examples, but it's easy to imagine more realistic cases where the patterns getting backtracked through contain multiple alternatives, subgroups, character classes, lookaheads, or all of these.
In particular, I remember that switching to possessive quantifiers when writing a parser for Foxboro's ICC format reduced the parse time for a complete CP280 program from ~10min to a few seconds, merely by eliminating backtracking. Unfortunately I can't publish this as an Atom grammar, because Atom doesn't support atomics or possessives (or string anchors).
If this issue is due to Atom's reliance on the crippled javascript regex spec, then it's time to junk it for one suited to the 21st century (or even the 20th).

@alexchandel I'm fairly certain Atom grammars support both the use of atomics and possessives, as I've used them myself in a number of grammar rules.
Like I said in your other issue, grammars and find-and-replace use different regex engines.  Feel free to create an issue on atom/find-and-replace to suggest switching to Oniguruma.


intellij does not use our settings when we create a new project within intellij

Expected behavior
As an intellij user I expect intellij to automatically load/import our settings when I create a new project within intellij.
Actual behavior
Currently if the user creates a new project within intellij. intellij will create a new configuration within the new project folder, ignoring our settings.
Steps to reproduce (bug) / Use Case of feature request (enhancement)

Install a fresh devonfw-ide
install intellij using: devon intellij
Start intellij (should load main workspace with our settings)
Create a new project within intellij in a subfolder of main
Check the .idea and .intellij folders for our settings

Related/Dependent Issues
Comments/Hints:
A possible fix for that could be the "New Projects Setup" feature of intellij where we can set how new projects should be pre-configured.
These settings can be exported and probably be added to our settings.
A workaround for this issue would be to manually create a new folder in workspaces and run: devon intellij create-script within the new folder e.g. (devonfw-ide/workspaces/mynewworkspace).
The devonfw-ide will create a new start script inside the devonfw-ide folder named intellij-mynewworkspace.bat then.
Make sure to disable INTELLIJ_DISABLE_OPEN_WORKSPACE within the .devon.properties to ensure that a new workspace will be opened in a new instance of intellij.
Affected version:

devonfw-ide 2023.09.001

INTELLIJ_DISABLE_OPEN_WORKSPACE should be active by default as IDEasy is a new product and should behave like this from the start. We can even decide to drop this property/variable and always have this behavior.

To export IntelliJ IDEA settings from one project and import them into a new project, you can follow these steps:

Export Settings from the Original Project:
Install intellij using: devon intellij
Start intellij (should load main workspace with our settings)
Open the original project in IntelliJ IDEA.
Go to File > Export Settings.
Choose the settings you want to export. For our case all.
Select the destination folder where you want to save the settings archive.

Create a New Project:
Close the original project if it's still open.
Restart intellij
Create a new project by going to File > New > Project.
Choose the project type and settings appropriate for your new project.

3. Import Settings into the New Project:
Once the new project is created, go to File > Import Settings.
Navigate to the folder where you saved the settings archive in step 1.
Select the settings archive file (typically with a .zip extension).
Click OK to import the settings.
Restart intelliJ.
After importing the settings, it's recommended to restart IntelliJ IDEA for the changes to take effect!
4. Verify Settings:
Check that the imported settings are applied correctly.

@ndemirca Thanks for your description of the export/import of IDEA Settings. This looks like the manual process though, so now we need to check if these steps can be automated somehow.

@jan-vcapgemini Or New Projects Setup>Save Project As Template and the same approach like above would be also a similar solution but to automate this hole process there is more investigation needed.


[libodb] Build error

Host Environment

OS: Linux Ubuntu 20.04 LTS
Compiler:
vcpkg-tool version: 2022-11-10-5fdee72bc1fceca198fb1ab7589837206a8b81ba
vcpkg-scripts version: cf1fda544 2022-12-09 (2 days ago)
Lobodb version : libodb-2.4.0.tar.gz

To Reproduce
Steps to reproduce the behavior:
git clone https://github.com/Microsoft/vcpkg
cd vcpkg
git pull
./bootstrap-vcpkg.sh
./vcpkg update
./vcpkg integrate install
./vcpkg install libodb-sqlite

Failure logs
For example :
vcpkg/buildtrees/libodb/src/libodb-2-59eaf93c2b.clean/odb/details/shared-ptr/base.hxx:38:49:
error: ISO C++17 does not allow dynamic exception specifications
   38 | operator new (std::size_t, odb::details::share) throw (std::bad_alloc);

Additional context
Libodb max c++ version is : C++14 version, because of dynamic exception
i haven’t found in documentation a CLI argument to select C++ standard version

Thanks for posting this issue. Could you please provide the full log?

The full log of result of commandes below
install-x64-linux-dbg-out.log

This is a upstream issue: https://www.codesynthesis.com/pipermail/odb-users/2018-March/003972.html. Could you test the new version 2.5.0-b.3 of libodb? If this can install succeed on C++17, I will submit a PR to update libodb.
The attached file is new version of libodb. You can download it and unzip it, then use these file instead of your local file and rebuild libodb.
libodb.zip

Reply of version 2.5.0-b.3 : it seems to work

this still doesn't work out of the box.
the temporary workaround I found was to override
CXXFLAGS=--std=c++14 ./vcpkg install libodb

be warned that al dependencies will be built using this flag, but it doesn't seem to break anything.


Angel1254mc/late night fixes

PR adds mac addy info to recognize.py (so that we know whose data to update and whose picture to run the prediction on) (In a world of many mac addresses only one may thrive (at a time)) Upon prediction, the script now also updates the user object based on whether the disposed item was trash or recyclable.
also adds crazy crazy stepper code turned dc motor code turned continuous rotation servo code that has been haunting me for the past 2 months or so. Turns out what was causing the esp to not connect to my laptop was my wifi.

LGTM 🔥🔥🔥

Who is this guy that keeps reviewing my PRs bro what


implement loyalty points

Looks good. Remember to update user guide and tests.

Reopened at https://github.com/cs2103-ay1819S2-w11-2/main/pull/75


How to make the make test not download package everytime?

I build a module with test file, so I need make test often to make sure the code is right.
But It installs the same packages everytime, waste a litte time:
make test
make -C test
[INFO] Installing 2 packages
[INFO] Installation complete.
.vessel/.bin/0.6.5/moc --package base .vessel/base/927119e172964f4038ebc7018f9cc1b688544bfa/src --package matchers .vessel/matchers/v1.2.0/src -wasi-system-api -o Test.wasm Test.mo && wasmtime Test.wasm
All tests passed.
rm -f Test.wasm

fixed now when I use vessel


chore: update toml lib to 0.7.6

The parsing of my config looks still OK after using last upstream toml version instead of the fork.

Looks good indeed, and I remember in particular the fork was needed to parse this kind of config:
[als.time]
thresholds = { 0 = "night", 7 = "dark", 9 = "dim", 11 = "normal", 13 = "bright", 16 = "normal", 18 = "dark", 20 = "night" }

Which is now parsed correctly.
Finally, this PR even fixes #47  :wink:
Thanks!


make problem

/1/CameraAiDetector/src/uid_utils.h:5:10: fatal error: format: No such file or directory
5 | #include
|          ^~~~~~~~
compilation terminated.
make[2]: *** [src/CMakeFiles/CameraAiDetector.dir/build.make:90: src/CMakeFiles/CameraAiDetector.dir/core.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:98: src/CMakeFiles/CameraAiDetector.dir/all] Error 2
make: *** [Makefile:91: all] Error 2
Debian GNU/Linux 12 (bookworm)
cmake version 3.25.1
c++ (GCC) 13.3.0
please advise how to solve this problem
thank you

Looks like something related to c++20 headers, I am not sure how to fix this on Debian. Complete working set of commands for Ubuntu can be found in .github/workflows/cmake-multi-platform.yml, maybe that will help.

Switched to ubuntu - but there are new problems :(

Maybe you've checked out tgbot-cpp manually, not with --recurse-submodules?

I don't think so, no.
only 2 actions :)
cmake .
make

Update from master, I just updated the code to reflect latest tgbot-cpp changes

without error
thank


Watchdog interrupt error

When I run the example, I see this output:
The distance is: 3 cm
The distance is: 69 cm
The distance is: 5 cm
The distance is: 2 cm
The distance is: 3 cm
The distance is: 4 cm
2022-03-05T18:44:21.971214Z:
WATCHDOG INTERRUPT error.
0: watchdog_                 /core/exceptions.toit:202:1
1: Driver.read_              pkg:toit-hs-sr04/driver.toit:91:3
2: Driver.run_               pkg:toit-hs-sr04/driver.toit:47:24
3: Driver.start. pkg:toit-hs-sr04/driver.toit:41:9
Typically I see some readings, which make sense. And then after some time the watchdog kills the program.

That's interesting. I thought that the driver was missing a yield somewhere, but it does not look like it.
FYI Toit is not great for timing sensitive drivers like this one as the timings may be a bit off because of interruptions (the garbage collector may start) and overhead (there is a bit of overhead to every primitive call). However, I am working on support for the RMT controller supported by ESP32 which is well suited for timing sensitive applications like this one.

Fixed by 4d27ab2e41139ed9815e0e6ba2950ab470d97019

Thank you for approving the PR so quickly. What is the RMT controller?
By the way I have an US-100 sensor, which can act like HS-SR04, but it can also perform measurements itself and communicate via UART. Maybe I should go for that option, timing should not be an issue then.


Cannot convert 1 drop expressed in XRP to drops

I want to convert 0.000001 XRP to drops (=1 drop) with xrp_to_drops but it raises the XRPRangeException.
When I increase the amount to 0.000002 XRP it converts the amount to drops correctly.
Also when I am using drops_to_xrp to convert one drop to XRP it returns 0.000001 XRP correctly.

This appears to be some sort of bug in Decimal:
>>> from decimal import Decimal
>>> one_drop = Decimal("0.000001")
>>> 0.000001 == one_drop
False
>>> 0.000001 < one_drop
True
>>> float(one_drop)
1e-06
>>> float(one_drop) == 0.000001
True

It should work if you use Decimal(0.000001)

Yes that works for now. But it seems to be important that you specify that drop as string not float.
Do:
xrp_to_drops(Decimal("0.000001"))
Not:
xrp_to_drops(Decimal(0.000001))


bugs when connecting arbitrary nested graph output to output proxy

create nested graph, edit nested graph
insert output proxy, leave unconnected
in main graph, insert output proxy, insert scene renderer
in main graph, connect nested graph node output to output proxy node
expected: nodes connect
actual: nodes signal incompatible, no connection is made
connect nested graph output to scene.
grab connection from scene end and reconnect to proxy output node
expected: result consistent with above
actual: nodes connect OK

I think this works as expected now.


[WIP] Add claims payments flow

Context
We want to add a support flow to pay out the claims made by sending a CSV of those claims to ESFA.
Changes proposed in this pull request

This commit adds the "Payments" flow to the Claims support page.

Guidance to review
I initially broke down the commits into multiple stages, but ongoing design changes have made me change the initial commits too frequently to the point where rebasing against this PR's own commits was creating too frequent conflicts. To assist with ease of development and clean non-contradictory commits within a single PR, I've squashed this commits in a single, large commit.
Link to Trello card

Screenshots

This has gone through a lot of back and forth. I'll close it down for now and re-open a fresh new PR later this week. @ollietreend


Views Support

internal notion doc

[x] grammar for syntax highlighting
[x] preview feature flag
[x] auto completion and suggestion of view block name: https://github.com/prisma/language-tools/issues/1346
[ ] ...

@Druue it looks like everything is completed here with https://github.com/prisma/language-tools/pull/1353
Anything else to add here or should you create another issue for extras?

I need to update this with some more issues as I've noticed more things that don't cross to views.

rename
quickfix för spelling


Puneet/993 feature service update attributes

@doneill can you please review

https://github.com/Esri/arcgis-runtime-samples-android/pull/137

@PuneetEsri Please create a new PR against quartz-beta-dev branch.


Full implementation of badges

Achieving a new badge level shows up on tasting cards:

Badges have dedicated pages, but theres nothing there yet:

Badges show on profiles

Admin is barebones, but works:

Reworked this a bit but its mostly still the same.
Made badges => checks, so all badges can have multiple checks. Currently only supports AND across them all, but that's something we could expand in the future.


await FlutterFreshchat.identifyUser() is returning empty string when restoreId is not passed

await FlutterFreshchat.identifyUser(
    externalID: accountId,
    restoreID: null,
  );

returns an empty string every time instead of restoreId generated by Freshchat.
PS: I am using the free plan of FreshChat

This was tested on Android

@daadu try passing an empty string rather than null to restoreId

Strange. The Activity gets killed or something at times. Anyways, still, the same issue returning an empty string.

@daadu I looked through the issue it doesn't generate a restoreId unless you start a conversation after that if you identify the user you should get a restoreId. I think this has to with Freshchat, not with this plugin

Also, you can pass null to the identifyUser method that should work too.

Strange! anyways I will implement that way and test.Till then keep this issue open.

In my current flow, I identify -> then add user detail -> then start conversation. I guess I should add user detail -> then start conversation -> then identify.

This will also wont work if you are trying to sync conversations so I think the best way would be to check if the user has a restoreId stored locally or the server if he has identify the user or if not start the conversation and then identify.

@daadu going to close this issue if the problem persists feel free to open the issue again

@fayeed Not working. Sorry for responding late.
Empty string returned by FlutterFreshchat.identifyUser even after a conversation is initiated and responded.

I have re-opened this issue again but I don't have time to work on it if you can fix this please do create a pull request or wait till someone else or I fix this.

@daadu I just checked this issue I see no problem, It might be because you also need to update the user info.
Here I tested it and its working:

getting an empty string from identifyUser for my credentials. My freshchat account is on the free plan (not trial). I am guessing the issue could be related to this.

I am also facing the same issue...did you find a fix?

No, it looks like the issue because of free plan. Although not sure.

As per docs. Restore Id is asynchronous and typically only initiated when conversion starts for first time.
So as per their sample app and docs. We have to create Broadcast receiver on android so when it's created for first time. It will be passed by broadcast receiver. (I'm not sure of ios though)
And then from native android we can call a flutter method which will give restore id when it becomes available.

by changing restoreId == "" to restoreId.equals("") in FlutterFreshchatPlugin empty restoreId issue will be resolved

by changing restoreId == "" to restoreId.equals("") in FlutterFreshchatPlugin in android module, empty restoreId issue will be resolved

How about iOS?? I am having issues with the FlutterFreshchat.identifyUser. Prints this error  Could not cast value of type 'NSNull' (0x7fff86b92b38) to 'NSString' (0x7fff86bb7bb0). And crashes when it tries to identify user. What values do I use for externalID  and restoreID to prevent crash..

by changing restoreId == "" to restoreId.equals("") in FlutterFreshchatPlugin in android module, empty restoreId issue will be resolved

How about iOS?? I am having issues with the FlutterFreshchat.identifyUser. Prints this error  Could not cast value of type 'NSNull' (0x7fff86b92b38) to 'NSString' (0x7fff86bb7bb0). And crashes when it tries to identify user. What values do I use for externalID  and restoreID to prevent crash..


Eliminate global GEOS context and release GIL

Inspired by #80 I took another take at releasing the GIL. I think we do not need to have 2 inner loops as in the get_geom func, only static attributes are accessed and the CPython API is not used (except for the exception handling)
However, both this PR and #80 suffer from the same condition: the GeometryObjects (and contained GEOSGeometry objects) could be deallocated while not holding the GIL. I added a unittest that shows this issue.
Luckily, the GeometryObjects are immutable, so we only have to be wary of them being deleted. The way out of this seems: making a copy of the ndarray (and make sure nobody can access it) before releasing the GIL. This would add some overhead, so I guess we want to have some heuristics on the array size before doing this. Or we leave it up to the user.

Thanks for looking into this! (I was wondering the same based on your comments in the subclass PR)

The way out of this seems: making a copy of the ndarray (and make sure nobody can access it) before releasing the GIL. This would add some overhead, so I guess we want to have some heuristics on the array size before doing this. Or we leave it up to the user.

Another alternative I have been thinking about is setting the writable flag to False before passing the arrays into the pygeos.lib ufunc (a user could still change this again to modify the array while the ufunc is running, but then they are clearly on their own)
We would need a way to reset the flag afterwards, also when an error get raised, but that might be possible to solve quite cleanly with a context manager.
I am also fine with leaving it up to the user for now.

I didn't know that existed! It solves this issue nicely. Now you get a ValueError when trying to edit the ndarray while concurrently computing the ufunc.

@jorisvandenbossche  I finished this PR, to enable multithreading on all functions that do not produce geometries.
Also, I eliminated the global geos context. At the end I realized it might be a good idea to have a global context after all that is only useable if you have the GIL. For instance, that would save a call to GEOSInit in the deallocation of geometries. At the other hand, it is good good to follow one pattern everywhere.

@caspervdw thanks for your huge effort here! I will try to take another look one of the coming days

@brendan-ward  Thanks for your review, I addressed all of your comments but one, see above for specifics.

@jorisvandenbossche  Thanks for your review, I addressed your points.
@brendan-ward  @chaitan94  I touched the Frechet distance code in this PR.
Because the functions are eligible for GIL release, we can't set the PyError state from within the loop. So instead of implementing a new C-error code for the specific "out of range" error, I left it to GEOS after all. Please check if you agree on my refactored implementation.

Just went through https://github.com/pygeos/pygeos/commit/0f4082dac4d4af53991af18f2bfc6b30ebf07535. It looks much better now. From this it also seems - in regards to https://github.com/pygeos/pygeos/issues/150 - that to rely on GEOSException whenever we can is a better choice than rethrowing them unnecessarily as ValueError. This keeps our code simple, and we can have generic functionality like we did here with YYd_d_func.

Woohoo ;) We should probably do a new minor release now this is merged.

Woohoo ;) We should probably do a new minor release now this is merged.

Indeed! Do you have time to wrap up https://github.com/pygeos/pygeos/pull/117 for the release?


Missing circe-generic-extras in 0.14.3

See in Maven Central: https://search.maven.org/artifact/io.circe/circe-generic-extras_2.13 The latest published version is 0.14.2.

https://github.com/circe/circe-generic-extras/issues/242


Add basic elements of a good software community

Add the following

[x] Code of Conduct
[ ] Contributing guidelines (including a style guide)
[x] License

Pull request #5 accidentally closed but a style guide is still needed


Installing mlconjug

I'm trying to install mlconjug into the latest offical python docker container, but it fails.
Here is the command that fails:
Command "/usr/local/bin/python -u -c "import setuptools, tokenize;file='/tmp/pip-install-90fg7720/scikit-learn/setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record /tmp/pip-record-abm6cqy_/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /tmp/pip-install-90fg7720/scikit-learn/
Steps to reproduce the behavior:

docker pull python
docker run -it python bash
pip install mlconjug
See error

OS: Win10
Docker <IP_ADDRESS>, linux VM

I also had problems on my host machine installing it, so this is why I switched to the docker container :)
I've also tried to clone the github repository on my host machine (Win10) and install it from there with python setup.py install
But I noticed that it tries to fetch scikit-learn 0.19.x instead of 0.20.2.
So I changed this requirement and now it works, but something is not right.
I ran the following command: mlconjug -l en help and it gave me this result:
{'imperative': {'imperative present': OrderedDict([('2s :', ''),
('1p :', ''),
('2p :', '')])},
'indicative': {'indicative past tense': OrderedDict([('1s', 'ed'),
('2s', 'ed'),
('3s', 'ed'),
('1p', 'ed'),
('2p', 'ed'),
('3p', 'ed')]),
'indicative present': OrderedDict([('1s', ''),
('2s', ''),
('3s', 's'),
('1p', ''),
('2p', ''),
('3p', '')]),
'indicative present continuous': OrderedDict([('1s 1s', 'ing'),
('2s 2s', 'ing'),
('3s 3s', 'ing'),
('1p 1p', 'ing'),
('2p 2p', 'ing'),
('3p 3p',
'ing')]),
'indicative present perfect': OrderedDict([('1s', 'ed'),
('2s', 'ed'),
('3s', 'ed'),
('1p', 'ed'),
('2p', 'ed'),
('3p', 'ed')])},
'infinitive': {'infinitive present': OrderedDict([('', '')])}}

Hi @vectomon how are you?
Thank you for contributing to mlconjug by submitting this bug report 👍 .
I just published mlconjug v3.3.1 which solves your installation issues and scikit-learn versions discrepancies.
Please try using mlconjug v3.3.1 and let me know if it solves your issues.
Cheers!
SekouD.

Thanks for the quick reply and fix, now it's easy to install :)
One more question:
Am I missing something, or the English regular verbs are not generated properly?

Hi @vectomon. There was indeed a regression bug when I updated the model to use scikit-learn 0.20.2 preventing English regular verbs to be properly populated.
I created a new issue #75 to fix this problem.
I should release a new verion of lyrics master fixing this issue by tomorrow or Friday.
Thanks a lot for your feedback, and keep provindung bug reports, enhancement requests, etc...
Cheers,
SekouD.


DNN-8450 Allow override of prefixing the service name for OAuth providers

This is a continuation of #1346 (after rebasing and fixing conflicts)
DNN-8450

The fix for DNN-4133 was to add a prefix for the service to the beginning of the username. This broke our custom OAuth provider and I believe that we should have the ability to override whether we want to prefix the service to the Username in OAuth providers. In addition to allowing that overriding, this also implements the option to automatically associate logins to existing users, based on email.
Testing of this API change will require a custom OAuth provider implementation which sets the new PrefixServiceToUserName and AutoMatchExistingUsers properties in that provider's implementation of OAuthClientBase to false and true, respectively. That custom provider may be an existing provider (e.g. Facebook), tweaked to override those two properties.
When configured as mentioned above, the username of users who log in via that provider should be the user's email (as provided by the OAuth provider). Additionally, if a user with that email address as their username already exists, that OAuth login would be associated with the existing account, rather than trying to create a new account.

@zyhfish you're right, I'm not sure why those changes were made.  I've reverted them.  Let me know if you see anything else

I'm bypassing the CLA requirement since @bdukes already has one signed and this PR was created before the CLA bot was added.


Error creating or renewing a Let's Encrypt certificate

Checklist

Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?

I pulled the last image, 2.9.18 and found no errors in the image

Are you sure you're not using someone else's docker image?

docker pull jc21/mariadb-aria:latest

Have you searched for similar issues (both open and closed)?

Yes

Describe the bug
When I go to create a new certificate or renew it only appears "internal error". It's been more than a month since I created or renewed a certificate, I have more than 30 assets and I wasn't having a problem, I was using version 2.9.9, but now I'm not able to create it.
I didn't change anything on the server that affects this, ports 80 and 443 are open for npm, as I've used it before for domains, but now it's not working.
Nginx Proxy Manager Version
v2.9.18
To Reproduce
Steps to reproduce the behavior:

SSL Certificates
Add SSL Certificates, Let's Encrypt
add domain in Domain Names
I Agree to the Let's Encrypt...
Save

Error: Command failed: certbot certonly --config "/etc/letsencrypt.ini" --cert-name "npm-83" --agree-tos --authenticator webroot --email<EMAIL_ADDRESS>--preferred-challenges "dns,http" --domains "nutrica.com.br"
Saving debug log to /var/log/letsencrypt/letsencrypt.log
Some challenges have failed.
Ask for help or search for solutions at https://community.letsencrypt.org. See the logfile /var/log/letsencrypt/letsencrypt.log or re-run Certbot with -v for more details.

    at ChildProcess.exithandler (node:child_process:399:12)
    at ChildProcess.emit (node:events:526:28)
    at maybeClose (node:internal/child_process:1092:16)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:302:5)

LOG: https://pastebin.com/Sqe4tdnV
Expected behavior

Screenshots

I tested another time and it started working again by itself, it could be a problem where the certificates are created.

I have actually the same Problem but i will create a new Cert.
Error: Command failed: certbot certonly --config "/etc/letsencrypt.ini" --cert-name "npm-20" --agree-tos --authenticator webroot --email "EMAIL" --preferred-challenges "dns,http" --domains "blog.rollenspiel.monster"
Saving debug log to /var/log/letsencrypt/letsencrypt.log
Some challenges have failed.
Ask for help or search for solutions at https://community.letsencrypt.org. See the logfile /var/log/letsencrypt/letsencrypt.log or re-run Certbot with -v for more details.

    at ChildProcess.exithandler (node:child_process:399:12)
    at ChildProcess.emit (node:events:526:28)
    at maybeClose (node:internal/child_process:1092:16)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:302:5)

how did you fix this issue please help im having the same issue

After one day it seems to have settled, either Letsencrypt had a problem or I got into some limit.


Adds some onbuild packages

adds onbuild packages similar to the ones available for the language docker repos
these allow easy experimentation with gRPC as long as docker is available
these can be used to provide an alternative installation path for the Quick Starts

LGTM, did a round of testing building these docker images


project initialization with svelte adders

This afternoon I had a though: How would it be, if we could immediately add some of those awesome adders while running npm init svelte@next
I found that this is quite an interesting question as this would help people setting up stuff really quickly. So there we are:
npm init svelte-extended

This command initializes a new sveltekit project, reads all available repos of this org, and gives the users the possibility to immediately install them. For who does not have a terminal at his hands, here is an image:

The repo for this stuff currently is here: https://github.com/manuel3108/create-svelte-extended (for anyone who wants to have a look). It's currently not tuned and probably has many problems. (Under the hood, this all just executes npm / npx commands)
What are your thoughts about that? Any ideas, problems etc?
PS: I'm writing this here, because it uses all the templates of this org. Depending on your thoughts and the problems you see, we could maybe also transfer this to this org.

I will absolutely explore this in the future!
What I'm expecting it to look like is
npm init @svelte-add/kit  # Create a SvelteKit project (their prompts) then launch the Svelte Adders selection menu
# or
npm init @svelte-add/vite  # Create a Vite-powered Svelte app then launch the Svelte Adders selection menu

Related: https://github.com/svelte-add/svelte-adders/issues/12

Closed by 09c11d5c5f9117eeca8fc5120943439a3b3e30c3. There may be some problems, so I'm leaving this issue open for a while.

@babichjacob I think you either referenced the wrong commute or wrote in the wrong issue, or I don't understand the commit ;)

@babichjacob I think you either referenced the wrong commute or wrote in the wrong issue, or I don't understand the commit ;)

You're right, this is a better commit to look at:
https://github.com/svelte-add/svelte-add/commit/34876aaed8f90463c6c8475179719857e8e7adda#diff-956c9fec778dee35effe338392c21cd77daa31ca3391ae168d6d7d29efd6a0bdR252-R485

Hey, just got some time to test it out. First of all, very well done!!! The process is smooth, the questions and selections are very clear and overall it "just works".

Thank you!!! I really appreciate you taking the time to check it out.

* First thing that i saw is that there is probably a problem while adding the demo files of the bulma adder (or at lease something other creating an issue to be displayed)

I still need to rewrite the Bulma adder. After making this project initializer tool, it is now at the top of the to-do list for the big rewrite.

* Its prompting me to `pnpm run dev -- --open  # start developing with a browser open` but apparently this command does not work at the moment. It gives the following error: ` ERROR   ERROR  Unknown option: 'open' For help, run: pnpm help run`

Please upgrade to the latest version of pnpm and let me know if that fixes it. I believe it was a recent change to align with how npm 7 works.

* In the message `Your TypeScript + mdsvex + ESLint + Prettier SvelteKit app is ready!` its somehow missing out the bulma part (+ scss eventually)

It deliberately doesn't mention Bulma and SCSS because they had error messages above. I can make it explicitly say "There was a problem setting up Bulma + SCSS (see above)." though.

* The generated site is a bit messed up
  ![grafik](https://user-images.githubusercontent.com/30698007/132943909-90a774fa-5805-4e5b-a545-2629116b56d9.png)
  Could have to do with the bulma demo files eventually

The current Bulma adder replaces the __layout.svelte file, removing the app.(s)css import. This will be solved by me rewriting it.

* It looks like its not asking for the which type of svelte-project we want to use (skeleton or demo). If thats the intendet way i would at least expect that it takes the skeleton app

Saying "Yes" to "Do you want example and demonstration code added to your app to demonstrate how to use the tools you've selected?" will choose the demo app. Saying "No. I am very familiar with the application framework and all the tools I've selected, so I fully understand how to use them without any guidance." will choose the skeleton template. This isn't entirely clear, but I'm also not sure how to make it more clear.

Please upgrade to the latest version of pnpm and let me know if that fixes it. I believe it was a recent change to align with how npm 7 works.
Worked flawlessly now!

It deliberately doesn't mention Bulma and SCSS because they had error messages above. I can make it explicitly say "There was a problem setting up Bulma + SCSS (see above)." though.
I think that would be good

Saying "Yes" to "Do you want example and demonstration code added to your app to demonstrate how to use the tools you've selected?" will choose the demo app. Saying "No. I am very familiar with the application framework and all the tools I've selected, so I fully understand how to use them without any guidance." will choose the skeleton template. This isn't entirely clear, but I'm also not sure how to make it more clear.
Oh ok. I thought this question was primarly about adding the demos from the differnent adders, that we included a while ago.
I think it would be good to seperate the adder demos and the sveltkit-demo into two differnent questions

(I might find some time over this weekend to have a look at a few of this issues)

Yes, now that I understand your point, I agree. Also the question (from the cli) clearly describes what you were saying.
I'm going to close this right here, because your cli is up and running, and i have seen you rewriting a lot of adders the last days, so I would assume this is pretty much done.


feat(sdk): add banner in billing screen to show upcoming plan change

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 41.43%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5987

Relevant Lines:
14451

💛  - Coveralls


Update generate & Add template for GitHub container action

This PR is a follow-up of a discussion with @Zimmi48, and of the release of docker-coq-action:
https://github.com/coq-community/docker-coq-action/releases/tag/v1.0.0
see the commit messages for more details

@liyishuai As this PR takes us even further away from your own proposal to use a Makefile instead of the generate.sh script, maybe you want to participate to reviewing it and weight your own concerns in?

My main purpose for a Makefile was to avoid copying files or commands. Whatever approach that better automates the generation process is good to me.

@liyishuai

Whatever approach that better automates the generation process is good to me.

:+1: BTW note that from the current version of the PR, it'd be easy to add other yes/no (or open) questions within the script, e.g. to ask whether the user would like to get a Travis CI configuration or not, etc. (although such extra changes would deserve a separate PR, maybe)

FTR, I also tried to install the npm "mustache" package but it doesn't support YAML, only JSON (like the OCaml version IIUC).

FTR, I also tried to install the npm "mustache" package but it doesn't support YAML, only JSON (like the OCaml version IIUC).

Yes but this is not a issue at first sight, the mustache ecosystem is not very uniform, but I guess that the current script should be OK with any mustache supporting YAML (cf. the remark in the README anyway:

We suggest you rely on a YAML-compatible mustache implementation such as ruby-mustache (available for instance as an Ubuntu or Debian package).
)

(FTR that README remark had been added in https://github.com/coq-community/templates/pull/17 )

I confirm the current script works well with mustache-go.

@erikmd What do you think of removing the prompt and making the generation of files entirely automated based on whether some variables are set or not in the meta.yml file? For instance, right now, we already have one such variable for travis because we put the badge in the README only if it is set. We could do the same for the GitHub Action (and include a badge as well).

One major interest of a fully automated generation process (without any prompt) would be that we could add some more automation: CI checks that the generated files correspond to the ones checked in the repository / a call to @coqbot to regenerate the files based on the latest version of the template and the meta.yml in a PR...

One inspirational example is the automation that is used to manage https://github.com/conda-forge (see in particular https://github.com/conda-forge/conda-smithy).

@erikmd What do you think of removing the prompt and making the generation of files entirely automated based on whether some variables are set or not in the meta.yml file? For instance, right now, we already have one such variable for travis because we put the badge in the README only if it is set. We could do the same for the GitHub Action (and include a badge as well).

Yes, I fully agree with your idea :-)
(But I guess I won't be able to work on this before the evening.)

One major interest of a fully automated generation process (without any prompt) would be that we could add some more automation: CI checks that the generated files correspond to the ones checked in the repository / a call to @coqbot to regenerate the files based on the latest version of the template and the meta.yml in a PR...

:+1: Looks great (IINM you suggest to incorporate in the Travis/GitHub actions config a job that would run on the target project repo, clone the coq-community templates repo, and do a checkup?), anyway that feature itself would probably deserve a separated PR ;-)

:+1: Looks great (IINM you suggest to incorporate in the Travis/GitHub actions config a job that would run on the target project repo, clone the coq-community templates repo, and do a checkup?), anyway that feature itself would probably deserve a separated PR ;-)

Yes, that's just an idea, but worth being discussed further of course. The problem of automatically checking compatibility with the latest templates would be that it would force people to manually update at every iteration of the templates, even if that wasn't needed for them. A possible solution could be:

Doing releases of the templates repo, at a rate TBD, so that updates happen only at a new release;
Providing a way to automatically migrate meta.yml files (if a variable is renamed, or if one is introduced whose value was implicit previously);
Having @coqbot open PRs on every coq-community repository with a meta.yml file to update the meta.yml and the auto-generated files.

Just food for thought for now. Related to #20.

Hi @Zimmi48

What do you think of removing the prompt and making the generation of files entirely automated based on whether some variables are set or not in the meta.yml file? For instance, right now, we already have one such variable for travis because we put the badge in the README only if it is set. We could do the same for the GitHub Action (and include a badge as well).

done, feel free to react if you'd like that I refactor the generate.sh in another way.

[…] Just food for thought for now. Related to #20.

BTW this new version of the PR maybe reduces a bit the "discoverability" of the GitHub action template, as the selected examples do not contain action: true, anyway this just amounts to say this is related to #24 :-)

BTW this new version of the PR maybe reduces a bit the "discoverability" of the GitHub action template, as the selected examples do not contain action: true, anyway this just amounts to say this is related to #24 :-)

Exactly!

done, thanks @Zimmi48 for the review! in passing I fixed the condition related to travis: / action: so that the CI badge in the README will be inserted iff the .yml CI file is generated.

sorry wait a minute before merging, I did a too aggressive rebase -i…

@Zimmi48 done :)  −  sorry for the noise…

In particular, we might want to add support for building with Nix using GitHub Actions as well.

Feel free to merge by yourself BTW :wink:

@Zimmi48 indeed; updated as suggested.


Invalid selection rectangle size for ListBoxItem when using keyboard up/down keys

Version: 0.5.2-build4248-alpha
Repro:

I think this is from the AdornerLayer not passing the clip rectangle to the adorning element. So for elements that extend out of bounds (possibly clipped by ClipToBounds), their adorner extends to their actual bounds.

This should have been fixed by #1350. Closing, please comment if it's still a problem and I'll reopen.


Clicking subscriber counts should open Subscribers tab

In channel settings, clicking the subscriber count for a channel (including a reasonably-sized click target around it) in the left-side list of channels should open the "Subscribers" panel for that channel.

@zulipbot claim

@zulipbot claim

@zulipbot claim


Callback and data

fix for #78, closes #79.
@psalaets, @javiercr here is an attempt at fixing #78. Would you mind verifying that it solves your issue?

This fixed it for me

@kasperp this also fixes the issue for me, thanks! :D

Was also having this issue.  Any chance for an npm push?  Thanks!

@joemcbride we'll do a npm push soon

:clap: Will be nice to have this plus the watch group pushed.

@davidchang :smile:

@javiercr @joemcbride @psalaets @kasperp my apologies for the delay - 0.1.6 is released: https://github.com/davidchang/ngReact/releases/tag/0.1.6 (thanks @psalaets for the issue and writing failing tests! Thanks Kasper as always :D )


Add test for checking digital signaturedigestCheck

Please merge with main so this github's ci can use these already merged PRs:

https://github.com/test-network-function/cnf-certification-test/pull/508 : adds "common" as default label if no others provided. This will avoid your TC to run in CI.
https://github.com/test-network-function/cnf-certification-test/pull/512 : fixes important issue regarding the daemonset.

@aabughosh , I'd like to test this change in our labs to confirm it's fine, but I cannot test it, it looks like the change need to be sync'ed with current main branch, can you update that?

@ramperher  i did rebase u can run the test

@ramperher i did rebase u can run the test

I cannot test it, anyway I've been checking the code and it should be fine with the current status, I'll approve.


[4anime.to] cannot download i get big error

03/23 13:38:26 [ERROR] CUID#7 - Download aborted. URI=https://mountainoservo0002.animecdn.com/Mob-Psycho-100/Mob-Psycho-100-Episode-12-1080p.mp4
Exception: [AbstractCommand.cc:351] errorCode=3 URI=https://mountainoservo0002.animecdn.com/Mob-Psycho-100/Mob-Psycho-100-Episode-12-1080p.mp4
-> [HttpSkipResponseCommand.cc:219] errorCode=3 Resource not found
this is the error i get

@Anon-Exploiter please help me if you cannot i would just like  a sorry or smthing

@Anon-Exploiter please help me if you cannot i would just like a sorry or smthing

What pack do you smoke? What's the brand seriously? Hahahaha, dude what am I supposed to apologize for xD? :rofl:
The script is open-source you can use it or not, or create your own xD

Also, It was working at the time I implemented the feature, they might have added some check again, will probably have to debug requests to find the fix.
You can go for gogoanime.ai for the meanwhile, pickup a anime from there and try it xD

@Anon-Exploiter please help me if you cannot i would just like a sorry or smthing

What pack do you smoke? What's the brand seriously? Hahahaha, dude what am I supposed to apologize for xD? :rofl:
The script is open-source you can use it or not, or create your own xD

Also, It was working at the time I implemented the feature, they might have added some check again, will probably have to debug requests to find the fix.
Meanwhile you can go for gogoanime.ai, pickup an anime from there and try it xD

Try the following:
python3 anime-dl.py --url https://gogoanime.ai/category/mob-psycho-100 -p 5

The issue is with old animes sometimes with 4anime.to since it has different urls for handling new and old series. My script is trying to fetch the latest one and it doesn't downloads.
I'll see if I can add an logic or just hardcode both starting urls since ending paths are same, anyways, 4anime.to is too shit to use, try gogoanime.ai or gogoanime.be
Will keep this issue open, trying to write a bypass now, but the backend server hosting other episodes is down.

Working now, will add statement to download from both:

Fixed in https://github.com/Anon-Exploiter/anime-dl/commit/1208a693ac276a668aea9b670e3d61853f384beb.
git pull the latest version and rerun and try again. Should work fine now.


onInstallConversionData not returning query param in response

Report
Hi, we've been using OneLink and AppsFyler to automatically pre-fill a referral code when a user clicks on the oneLink, install the app and signs up. An example of this OneLink is https://join.runna.com/lKmc/refer?code=LETSDOTHIS. As you can see we add the query parameter to the end of the oneLink. Previously when using 6.9.2 of this library the code would appear in the appsFlyer.onInstallConversionData response. Here is an example response:
{
  "status": "success",
  "type": "onInstallConversionDataLoaded",
  "data": {
    "adset": null,
    "click_time": "2023-01-13 16:39:54.721",
    "is_incentivized": "false",
    "esp_name": null,
    "is_branded_link": null,
    "is_retargeting": "false",
    "fbclid": "XXXXX",
    "shortlink": "refer",
    "af_siteid": null,
    "agency": null,
    "http_referrer": "https://l.instagram.com/",
    "af_sub2": null,
    "af_status": "Non-organic",
    "af_dp": "XXX://",
    "af_cpi": null,
    "af_sub1": null,
    "campaign": "Runna Affiliate",
    "cost_cents_USD": "0",
    "is_universal_link": null,
    "install_time": "2023-01-13 16:48:26.949",
    "adgroup": null,
    "code": "JP2",
    "iscache": true,
    "af_lp_src": "XXXXX",
    "af_sub5": null,
    "af_click_lookback": "7d",
    "adset_id": null,
    "adgroup_id": null,
    "media_source": "referral_link",
    "engmnt_source": null,
    "af_sub4": null,
    "is_first_launch": true,
    "campaign_id": null,
    "redirect_response_data": null,
    "orig_cost": "0.0",
    "match_type": "probabilistic",
    "af_sub3": null,
    "retargeting_conversion_type": "none"
  }
}

But since upgrading to 6.9.4, the code (i.e. code: JP2) is not getting returned in the response anymore. Is this expected?
Plugin Version
6.9.4
On what Platform are you having the issue?
iOS

We just released a new version of the app with 6.9.2 and can confirm that the query parameter is getting sent through in the response for appsFlyer.onInstallConversionData(). So therefore 6.9.4 is broken. Would someone be able to take a look please as we wont be able to upgrade until this is fixed/
{
  "status": "success",
  "type": "onInstallConversionDataLoaded",
  "data": {
    "redirect_response_data": null,
    "code": "JP2",
    "adgroup_id": null,
    "engmnt_source": null,
    "retargeting_conversion_type": "none",
    "is_incentivized": "false",
    "orig_cost": "0.0",
    "is_first_launch": false,
    "af_click_lookback": "7d",
    "af_cpi": null,
    "iscache": true,
    "click_time": "2023-03-10 17:33:56.182",
    "is_branded_link": null,
    "match_type": "gp_referrer",
    "adset": null,
    "campaign_id": null,
    "shortlink": "refer",
    "af_dp": "xxxx",
    "install_time": "2023-03-10 18:03:04.278",
    "media_source": "referral_link",
    "agency": null,
    "af_siteid": null,
    "af_status": "Non-organic",
    "af_sub1": null,
    "cost_cents_USD": "0",
    "af_sub5": null,
    "af_sub4": null,
    "af_sub3": null,
    "af_sub2": null,
    "adset_id": null,
    "esp_name": null,
    "campaign": "Runna Affiliate",
    "http_referrer": null,
    "af_lp_src": "xxxxx",
    "is_universal_link": null,
    "is_retargeting": "false",
    "adgroup": null
  }
}

it happens on iOS? Android? both?

i've noticed that on android it only works if i reset my advertising id. but then, only the first installation have the deeplink data. reinstalling the app doesnt  bring the data unless i add my device as test device in appsfyler's dashboard

@walterholohan the last payload that you showed here is from 6.9.4? (because i can see the code key). The behavior is the same between 6.9.2 and 6.9.4 so it should work the same. It would be better if you would open a ticket to our support team at<EMAIL_ADDRESS>for further investigation of your issue.

The last payload is from 6.9.2. After we reverted 6.9.4 back to 6.9.2 'code' re-appeared in the payload which confirmed to us that 6.9.4 was the culprit. We added logging to our app which has ~5000 downloads a day and was able to confirm our assumptions that 6.9.2 was working perfectly.
We opened a support ticket, however they requested debug logs that are impossible to retrieve. I.e. as the use case is: user clicks on OneLink, link navigates to App Store, user downloads app, onInstallConversion listener returns 'code' in payload.
Can you keep this issue open and pass onto the relevant product team for investigation? It must be to do with the iOS SDK upgrade.

@amit-kremer93 any update on above? I've seen you released a new version yesterday. Will this resolve the issue?


Dubbo 3.2.0 broadcast + injvm not work as expected

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Environment

Dubbo version: 3.2.0 / 3.2.1
Operating System version: Windows 10
Java version: OpenJDK 17.0.6

Steps to reproduce this issue
public interface RemoteCacheSyncService {

  void flushCache();

}

@DubboService
public class RemoteCacheSyncServiceImpl implements RemoteCacheSyncService {

  @Override
  public void flushCache() {
	  // some code
  }

}

We have 3 microservices, referred to as A, B, and C.
Each service contains the above code.
But, just C has a @DubboReference to the interface only.

Actual Behavior
However, since there is a service implementation of the interface in the same application ( C ),
Dubbo will use the injvm protocol. Only the service implementation located in C will be invoked.
Same issue with https://github.com/apache/dubbo/issues/6842 , however, it came back again.

@PhoenixBackendon PTAL

I see what you mean. The current solution is configured as forced remote in the settings, which should be able to solve it, or we can provide a rule to adapt to the requirements based on this broadcasting mechanism, and I will finish it later.

I see what you mean. The current solution is configured as forced remote in the settings, which should be able to solve it, or we can provide a rule to adapt to the requirements based on this broadcasting mechanism, and I will finish it later.

https://github.com/PhoenixBackendon/dubbo/blob/35da167f6a921f9c0c0799b807442f72d0fefaff/dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/support/wrapper/ScopeClusterInvoker.java#L132
logger.debug() should consider use placeholder {} ?


optimization debian package manager tweaks

By default, Ubuntu or Debian based "apt" or "apt-get" system installs recommended but not suggested packages .
By passing "--no-install-recommends" option, the user lets apt-get know not to consider recommended packages as a dependency to install.
This results in smaller downloads and installation of packages .
Refer to blog at Ubuntu Blog .
Signed-off-by: Pratik Raj<EMAIL_ADDRESS>Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@jpeddicord @aghecenco @andreeaflorescu @rn  requesting review

@Rajpratik71 thank you for these suggestions. I wasn't aware of these options are they are certainly useful.
This repository is mostly a read-only repository as a record of the experiments and data we used for the NSDI paper we published end of last month. It's certainly not perfect but this is the setup we used. I'm considering to archive this repository.


Fix esp_reset_reason_t constant names

The esp_reset_reason_t ESP_RST_PG and ESP_RST_CPU constant names don't seem to exist. Instead, the actual names (added in esp-idf 5.2.2) are ESP_RST_PWR_GLITCH and ESP_RST_CPU_LOCKUP.
The incorrect names caused the second-to-last match arm to be interpreted as a binding name (that is, a catch-all), causing incorrect behavior and build warnings when using esp-idf 5.2.2 and up.

See https://github.com/espressif/esp-idf/blame/v5.2.2/components/esp_system/include/esp_system.h#L39-L40
These were added in https://github.com/espressif/esp-idf/commit/e930ff0b1f527591b16e8f94f62ab2027b1c40fb -- I'm not sure where the incorrect names (used in #444) came from.

Yes. I noticed that too a month ago and because I was too lazy to open an issue, I forgot about that. :(
Thanks!

BTW: We practice a bad pattern in most "raw-C-bindgen-constants-mapped-to-Rust-enums where if a particular constant is not found in the match, we hit an unreachable! (i.e. we claim to have a bug).
The problem is, ESP IDF regularly and silently extends its C enums with new constants, and while the unreachable! helps us catch this with new ESP-IDFs, this is not OK for production, as the problem might be spotted too late.
Perhaps we need custom handling logic here, where (by default) each such unknown constant is mapped to MyRustEnum::Unknown and then this Unknown is logged at an appropriate place with log::warn.


Support for baseline files

This adds support for baseline files for ignoring bugs.
Fixes #348

@KengoTODA could you look into this PR ? I need this in my project to suppress SB warnings in legacy code, and from #348 I suppose I'm not the only one

Thanks so much, sorry about the kind-of-spammy reminder !


Using setState in a Modal loses input focus

Describe the bug
Whenever I set the state for an input in a modal, the input loses focus. If I use a ref instead, it works fine. If the input is used outside of the modal, the useState doesn't make the input lose focus.
To Reproduce
Steps to reproduce the behavior:

Copy the base example of a "Modal with form elements" found on the modals page
Add a useState hook to the onChange event of either input
Observe that the focus is lost

Expected behavior
The input should not blur on each new added character.
System information:

Device: Macbook Pro
Resolution:
OS: Ventura 13.4.1
Browser: Chrome
Version:

Project information:

tailwindcss: 3.3.2
Flowbite: 1.7.0
Flowbite React: 0.4.10
Type: Next.js w/ React

Additional context
Add any other context about the problem here.

Maybe this is a dup? #701


Allow a customizable warning message with variables

What I'm looking for is rather than the text showing "Warning: This password would take {{ password_strength_time }} to crack", something like "Password Strength: {{ zxcvbn_score }}".
I'd use Django's render_to_string so that I could use template tags. This would allow me to do something like:
PASSWORD_STRENGTH_WARNING = "Password strength: {% if zxcvbn_score < 2 %}Weak{% else %}Strong{% endif %}"

Default implementation would be the current message so this would only apply to users who want to override the warning message.
Is this something you'd entertain? Thanks again for building and maintaining this library!

Hello! Sorry for the delay, I just came back from vacations.
I think I would like that indeed, because I was never quite satisfied with the current messages. If you want to give it a try and send a PR, it will be welcomed :slightly_smiling_face:

Will likely not work on this project anymore.


Issue in deployment to slack marketplace

Description
While filling up the details in the when I add the Direct install URL  'https://<my_domain>/slack/install' in the 'Installing your app' section, I get a error of 'Your application’s redirect URL must return a status code of 302 to a slack.com url'. Is this handled by the bolt package or do we have to address this explicitly?
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[x] question
[ ] documentation related
[ ] example code related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
package version:^3.6.0
node version:^14.7
OS version(s):
Steps to reproduce:

Expected result:
What you expected to happen
Actual result:
What actually happened
Attachments:
Logs, screenshots, screencast, sample project, funny gif, etc.

Hi @shubhamAffinidi, thanks for asking the question!
This had been a missing feature in bolt-js but we're going to add a new option directInstall: boolean since the next minor version (v3.7.0). Here are the issue / pull request for it:

https://github.com/slackapi/bolt-js/issues/977
https://github.com/slackapi/bolt-js/pull/1079

As we are still working on other remaining tasks for v3.7, it make take a while until the release. If you need the feature right now, consider  doing this way as a tentative workaround 🙇 :

Copy the latest HTTPReceiver: https://github.com/slackapi/bolt-js/blob/992b64a767e43a5188b4a1b75e1a18ed50070d23/src/receivers/HTTPReceiver.ts into your app
Manually instantiate HTTPReceiver and pass it as receiver in the App constructor.
In the HTTPReceiver initialization, set installerOptions.directInstall to true(the default is false)

I hope this helps!

Hi @seratch, is there a tentative date for the v3.7.0 release?

@shubhamAffinidi No, there isn't. The timeline depends on how quickly the remaining tasks here are resolved. I think that releasing in this week may be a bit challenging but we can within a few weeks at latest.

Hi @shubhamAffinidi 👋  This feature is now released as part of v.3.7.0!
I'm going to go ahead and close this issue now!


Already Select

Hello,
If i selected 5 photos from gallery and done. and we need to select those photo previous again by default selected.
Is it possible.
Thanks in advance.

Yes it is possible and is documented. Always try to read interfaces for further documentation!
/**
 *  It contains the selected `PHAsset` objects. The order of the objects is the selection order.
 *
 *  You can add assets before presenting the picker to show the user some preselected assets.
 */
@property (nonatomic, strong) NSMutableArray *selectedAssets;


[SPARK-7971] Add JavaDoc style deprecation for deprecated DataFrame methods

Scala @deprecated annotation actually doesn't show up in JavaDoc.

Merged build triggered.

Merged build started.

Test build #33821 has started for   PR 6523 at commit 26da2b2.

Test build #33821 has finished for   PR 6523 at commit 26da2b2.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Jenkins, retest this please.

Merged build triggered.

Merged build started.

Test build #33827 has started for   PR 6523 at commit 26da2b2.

Test build #33827 has finished for   PR 6523 at commit 26da2b2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.


Check that methods have functions in serialize.py

The inspect built-in python module has the following code:
def getclosurevars(func):

    if ismethod(func):
        func = func.__func__

    if not isfunction(func):
        raise TypeError("{!r} is not a Python function".format(func))
    ...

And the function getclosurevars is used in lithops with each element of worklist in SerializeIndependent._module_inspect.
This PR checks each method appended to worklist to be sure that its __func__ are python functions. With these checks, we can be sure that no TypeError will be raised when working with methods.
This bug has been discovered using the following code with lithops 2.6.0 and Sage 9.5 and python 3.10:
from sage.all_cmdline import Integer
import lithops

_sage_const_7 = Integer(7)

def my_map_function(x):
    return x + _sage_const_7

iterdata = [Integer(1) , Integer(2) , Integer(3) , Integer(4) ]
fexec = lithops.FunctionExecutor()
fexec.map(my_map_function, iterdata)
print(fexec.get_result())
fexec.clean()

P.S. I've run the tests, following the contributing guidelines, but it turns out that the current master branch fails on some tests. No more fails are added.
Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

@gfinol Just ran all the tests on a couple of backends (IBM CF & AWS Lambda) and all them ran ok.
Can you provide more information about your backend and the tests that are failing? Are tests failing due this PR?

@JosepSampe Sorry, it was my fault. I was running tests with python 3.10 and some things gone wrong (runtime not supported yet). I just run them all with python 3.9 and no test failed.


Added INotifyPropertyChanged to MapControl

Implement the INotifyPropertyChanged interface in MapControl
Added some comments
Removed some empty lines

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Dirk Weltz seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@charlenni Somehow you did not sign the CLA. I would think you have, might be glitch. Perhaps you can sign again.


[BOT PR] to update dependencies of istio

This PR will be merged automatically once checks are successful.
none

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: istio-testing
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
If they are not already assigned, you can assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
28fbc0b970a6560e33fd331e9c8ac26a2cbef182
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
28fbc0b970a6560e33fd331e9c8ac26a2cbef182
link
/test istio-unit-tests

prow/istio-integ-k8s-tests.sh
28fbc0b970a6560e33fd331e9c8ac26a2cbef182
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
28fbc0b970a6560e33fd331e9c8ac26a2cbef182
link
/test istio-unit-tests

prow/istio-integ-k8s-tests.sh
28fbc0b970a6560e33fd331e9c8ac26a2cbef182
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
28fbc0b970a6560e33fd331e9c8ac26a2cbef182
link
/test istio_auth_sds_e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
28fbc0b970a6560e33fd331e9c8ac26a2cbef182
link
/test istio-unit-tests

prow/istio-integ-k8s-tests.sh
28fbc0b970a6560e33fd331e9c8ac26a2cbef182
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
28fbc0b970a6560e33fd331e9c8ac26a2cbef182
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
28fbc0b970a6560e33fd331e9c8ac26a2cbef182
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Implement the client side of the Addon

I've now included the changes made in: https://github.com/ynput/ayon-shotgrid/pull/12/commits

Made latest revision changes, ensure we do not create a shot grid version unless we have valid representations


I found the Mar 22, 2024 bug. I caused the problem

I think I was the imposter of the Mar 22, 2024 bug of View and edit content bug.
I think it is a security vulnerability. So it's better and much safer if I say it personally through my e-mail.
I don't want to make others know to exploit this security vulnerability.
I didn't intend this incident although it seems as a DoS attack.
Plz send me e-mail to let you guys know about the reason of it.
My email address<EMAIL_ADDRESS>

Oh, holp to fix it quickly.

Hello are you from notion?
2024년 3월 22일 (금) 오후 6:29, sunleo200013 @.***>님이 작성:

Oh, holp to fix it quickly.
—
Reply to this email directly, view it on GitHub
https://github.com/makenotion/notion-sdk-js/issues/496#issuecomment-2014692976,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCJUYI2JAZZRQHHOJDAPFI3YZP2YVAVCNFSM6AAAAABFC7FHQWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJUGY4TEOJXGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

I found the bug
And I need your ID to verify where you are from.
I don't want to make any more fuzz
2024년 3월 22일 (금) 오후 6:31, Jun Byeon @.***>님이 작성:

Hello are you from notion?
2024년 3월 22일 (금) 오후 6:29, sunleo200013 @.***>님이 작성:

Oh, holp to fix it quickly.
—
Reply to this email directly, view it on GitHub
https://github.com/makenotion/notion-sdk-js/issues/496#issuecomment-2014692976,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCJUYI2JAZZRQHHOJDAPFI3YZP2YVAVCNFSM6AAAAABFC7FHQWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJUGY4TEOJXGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Sorry, I'm just a user, finding my App can not use. Holp you contact Notion quickly, they must need you.

I am so sorry to make this incident.
I sincerely apologize what I did in advance.


refactor: create the etcd cluster after create the operator

will remove etcd dependency from greptimedb chart: https://github.com/GreptimeTeam/helm-charts/issues/13


Service Feasibility - Field to save route result expression out

We need the ability to save the result expressions value in a field, just like how the user can save the route length.

@MikeMillerGIS
We are planning to add another option as highlighted in the mockup below. Please confirm if this is ok.

Confirmed

verfied


Expo Version?

Hey @zaragit !
Would love to use this but noticed it's a bare workflow react native app. Does this work with Expo and do you plan to have it work with Expo?

Hi!
Sorry for the deplayd reply. I couldn't check my GitHub for a while due to personal reasons.
Is it not working in Expo?
I think the problem is react-native-handler-gesture, react-native-reanimated if it doesn't work in expo.
Try this.
yarn expo install react-native-gesture-handler
yarn expo install react-native-reanimated

babel.config.js
module.exports = function(api) {
  api.cache(true);
  return {
    presets: ['babel-preset-expo'],
    plugins: [
      'react-native-reanimated/plugin',
    ],
  };
};

yarn start -c


Vickel: Watch stucerts(?!.com)

Vickel requests the watch of the watch_keyword stucerts(?!\.com). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

stucerts(?!\.com) has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


fix router parameters handling by auto url decoding

Codecov Report
Base: 76.55% // Head: 76.56% // Increases project coverage by +0.01% :tada:

Coverage data is based on head (f50e5ca) compared to base (a1b9eca).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2262      +/-   ##
==========================================
+ Coverage   76.55%   76.56%   +0.01%
==========================================
  Files         582      582
  Lines       48420    48423       +3
==========================================
+ Hits        37067    37076       +9
+ Misses       9327     9320       -7
- Partials     2026     2027       +1

Flag
Coverage Δ

go-1.15-386
76.57% <100.00%> (+<0.01%)
:arrow_up:

go-1.15-amd64
?

go-1.16-386
76.59% <100.00%> (+<0.01%)
:arrow_up:

go-1.16-amd64
76.58% <100.00%> (+0.01%)
:arrow_up:

go-1.17-386
76.59% <100.00%> (+<0.01%)
:arrow_up:

go-1.17-amd64
76.60% <100.00%> (+0.02%)
:arrow_up:

go-1.18-386
?

go-1.18-amd64
76.51% <100.00%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

net/ghttp/ghttp_server_handler.go
64.03% <100.00%> (+0.43%)
:arrow_up:

net/ghttp/ghttp_server_router_serve.go
76.92% <100.00%> (ø)

os/gfsnotify/gfsnotify_watcher_loop.go
83.19% <0.00%> (+5.04%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


EgorBot for AndyAyersMS in #109209

Processing https://github.com/dotnet/runtime/pull/109209#issuecomment-2440125268 command:
Command
-intel -arm64 -profiler --envvars DOTNET_JitDisasm:Foreach Count
using BenchmarkDotNet.Attributes;
using BenchmarkDotNet.Running;
using System.Runtime.CompilerServices;

BenchmarkSwitcher.FromAssembly(typeof(Bench).Assembly).Run(args);

public class Bench
{
    string[] s_ro_str_array = new string[512];

    [Benchmark]
    public int Foreach()
    {
        IEnumerable<string> e = s_ro_str_array
        int sum = 0;
        foreach (string s in e) sum += s == null ? 0 : s.Length;
        return sum;
    }

    [Benchmark]
    public int Count() => CountInner(s_ro_str_array);

    [MethodImpl(MethodImplOptions.NoInlining)]
    int CountInner(ICollection<string> c) => c.Count
}

The Foreach test should now devirtualize and stack allocate the enumerator (but not yet promote, needs #109209).

(EgorBot will reply in this issue)

❌ Failed on Arm64: Job failed, see logs.
cc @AndyAyersMS (logs)

❌ Failed on Intel: Job failed, see logs.
cc @AndyAyersMS (logs)


using .make_charts() outside Jupyter Notebook environment

Hi
Is there a workaround for getting the charts as an HTML output? Since, the property does not function outside the Notebook environment.
Kindly let me know.
Thanks.

Hi,
Thanks for reaching out.
An option to render charts in HTML output format is in the pipeline, but this may take a while...