Then today when I went to code again, vscode was gone. It was like it had been completely uninstalled.
Anyone else had this happen?

Yes, about 30 minutes ago my Windows 10 vscode disappeared after it failed to update itself when I closed the application. I did not shut down my computer, and looked but could not find the vscode executable or installation.  This is beyond bad.

Can you try disabling Windows background updates? Set update.enableWindowsBackgroundUpdates to false.

Yes I will try setting update.enableWindowsBackgroundUpdates to false, however I first need to re-install VSCode (and all my extensions!!), as it deleted itself last night during its failed auto-update.  Maybe the VSCode installer can first make a backup of the existing VSCode installation, and revert back to that if the update fails...just a thought.


Ignore x.ts (typescript) files when amplify push happens.

Note: If your question is regarding the AWS Amplify Console service, please log it in the
AWS Amplify Console repository

Which Category is your question related to?
Function
Amplify CLI Version
4.49.0
What AWS Services are you utilizing? AWS Lambda
Provide additional details e.g. code snippets
Ignore files when amplify push, specifically lambda function files. Example: Add .amplifyignore to not send some .ts (typescript) files to AWS.

Hi @Efraa we don't currently have a mechanism for this but it's a good suggestion. I'm going to mark this as an enhancement and track it in our function build improvements project


ThreadMessageLike.tsx (Line 66) --> TypeError: part.text.trim is not a function. (In 'part.text.trim()', 'part.text.trim' is undefined)

I tried to make assistant ui work with a sample repo from Langgraph (memory-agent) and it doesnt work. The issue seems to be that when I bin a tool to the model in Langgraph the response content of the assistant message looks like this:
content: [{text: "Sounds", type: "text", index: 0}] (1) --> that causes the error you see in the title of this issue.
BUT when I use the same exact graph but with a model without tools like this: llm.invoke(...) instead of llm.bindTools(...).invoke, the response content looks like this:
content: "Sounds great, I'd love to hear about your upcoming travel plans! As an AI assistant without a physical form, I don't actually have trave…" --> NOT causing any error in assistant ui
I don´t know why the content is an array after binding tools to the model, but could you please just test assistant ui with any sample Langgraph repo using tools!

Thank you for reporting this @GregorBeuster
Please upgrade to the latest version of @assistant-ui/react and @assistant-ui/react-langgraph for the fix


Drawer doesn't reposition after blurring text input

Repro - from the docs: https://vaul.emilkowal.ski/inputs
Current behavior (Safari, iPhone 15, latest iOS)
https://github.com/user-attachments/assets/88b6328f-5997-4ba2-9f79-a8026417c841
Expected behavior
After blurring the text input (in this case, pressing "Done" on the keyboard), the drawer should reposition correctly.

Same on Android.

Drawer opens
focus an input field
keyboard appears on screen
Drawer repositions

There's also a odd magical sized :after element that interferes with visiblity of the textarea
input fields never properly position onscreen

unfocus field, drawer is now somewhat 50% of the size in height it started off with


Support drag-and-drop or paste in well-defined UI state only

This is an overdue patch to clean up when we allow drag-and-drop or image paste operations. In my opinion, they should be supported only in albums or album views, when there are no dialog windows/menus/etc. being displayed. Otherwise there's a good chance that the operation was performed accidentally.

Did not test, but the change sound rather sane if I step through it in my mind.

My thinking was basically that upload via drag-and-drop or paste should be allowed only in situations when you can access the "+" menu to do it manually. The code was doing complicated things such as closing the open dialog windows and such. Better to simply ignore the event in such cases so that the user can finish what they were doing and then they can retry the upload if that's what they actually intended.


Sails transaction name issues with New Relic

I'm trying to dig into this more but I wanted to post it here in case any one has some insight. It seems that sails > 0.10.5 is causing issues when teamed with new relic app monitoring. The problem is that transaction URLs are being malformed during reporting.
For example, I'm seeing the following reported in the dashboard

The reported routes are being repeated including garbage * chars for some reason. However, everything works as expected for a separate app running sails v0.10.0-rc9.

The newrelic library supports express and should be pulling everything it needs from the request but is failing to report property. My only thought is that sails performs some modifications to the request object that are not expected by this library. It sounds like a few other sails users have hit this based on this post from the New Relic forums. The workaround provided does work but I believe doing so would cause you to lose some nice functionality like router introspection.
Does anyone have any ideas on express related changes that may have happened between v0.10.0-rc9 and v0.10.5 which  could cause issues with an integration like this?

Hi Jason,
I am seeing the same thing here. As I am currently trying out newrelic,
they reached out to see how were things going and I asked the question.
Here is the answer:
I looked at the transactions and segments in their RPM # which is 907247.
The data does seem kind of messed up, and I'm not sure why. One possibility
is that he might be running on a version of node.js that we don't support.
But the Environment section does not say what version he is using.
Unfortunately, I think he is going to have to open a support ticket. Note
that we did just release significant new version of our node.js agent which
includes better reporting of database metrics. So, you might suggest he
upgrade.
I haven't tried to upgrade the library yet.
Manuel
On 27 February 2015 at 16:26, Jason Sims<EMAIL_ADDRESS>wrote:

I'm trying to dig into this more but I wanted to post it here in case any
one has some insight. It seems that sails > 0.10.5 is causing issues when
teamed with new relic app monitoring. The problem is that transaction URLs
are being malformed during reporting.
For example, I'm seeing the following reported in the dashboard
[image: screen shot 2015-02-27 at 1 11 04 pm]
https://cloud.githubusercontent.com/assets/1484253/6421292/3b055eb2-be82-11e4-9153-460421d8402e.png
The reported routes are being repeated including garbage * chars for some
reason. However, everything works as expected for a separate app running
sails v0.10.0-rc9.
[image: screen shot 2015-02-27 at 1 11 33 pm]
https://cloud.githubusercontent.com/assets/1484253/6421293/4319131e-be82-11e4-8571-2c16d1ed9ade.png
The newrelic https://github.com/newrelic/node-newrelic library supports
express and should be pulling everything it needs from the request but is
failing to report property. My only thought is that sails performs some
modifications to the request object that are not expected by this library.
It sounds like a few other sails users have hit this based on this post
https://discuss.newrelic.com/t/strange-transaction-names-in-new-relic-for-a-sails-js-app/15754
from the New Relic forums. The workaround provided does work but I believe
doing so would cause you to lose some nice functionality like router
introspection
https://github.com/newrelic/node-newrelic#router-introspection.
Does anyone have any ideas on express related changes that may have
happened between v0.10.0-rc9 and v0.10.5 which could cause issues with an
integration like this?
—
Reply to this email directly or view it on GitHub
https://github.com/balderdashy/sails/issues/2698.

Also have this on sails 0.10.5 & Node 0.10.32

Thanks @mdarveau! Unfortunately, I had opened a support ticket w/ new relic prior to this github issue and their response was "Sorry, we do not support the sails framework". I should be on their latest agent at v1.17.

I was just told to look into this.
https://github.com/newrelic/node-newrelic#newrelicsettransactionnamename
And I was going to use req.route.path but I'm trying to figure out where I should execute that in sails? Thoughts?

Yeah you can use that within some express middleware if you want. That seems more like a patch though. Since sails is based on express it should just work. I'm interested to see what specifically is breaking the newrelic integration.

but sails has its own router so that may be the issue.

Sails has it's own router? I thought it just dynamically built express routes based on model definitions.

@jasonsims you can try this:
sails.on('router:route', function(route) {
  if (route.req.options && route.req.options.controller) {
    newrelic.setControllerName(route.req.options.controller, route.req.options.action);
  }
})

Sails has it's own router? I thought it just dynamically built express routes based on model definitions.

Yea, sails uses express underneath

Looks like NewRelic found a bug in their support for Express 3--see here:
https://docs.newrelic.com/docs/release-notes/agent-release-notes/nodejs-release-notes/node-agent-1173
Are folks still seeing this issue after that patch?


Cannot convert blob

inserting images from Tinymce filemanger and save it to DB , when I call the page or article the image path converted to blob:http://domain.test/7b3f1fb5-aa35-4edc-b64a-ed402b4d6c6f and can not show the image

Did you figure out what caused the issue? I have the same issue occurring on a website running laravel 5.8 (I know it needs a upgrade :) )


Updating the versions for helm chart 0.0.3 release

Signed-off-by: Vivek Lohiya<EMAIL_ADDRESS>Description:  Updating the versions for helm chart 0.0.3 release
General Checklist

[x] Updated Added functionality/ bug fix in release notes

/azp run

/azp run


Use custom layout for toolbar

Hi I used wasabeef's richeditor (your base) for my project. since that doesn't have any built in toolbar, I made a custom one for my app which matches the design of my app. Now I want to switch to your library but I want to keep my toolbar. I dug a little but wasnt able to find a way to use my previous toolbar. Is there any way to do that?

Depends on what you are planning to do.
You can create a custom toolbar with commands to your liking as described here:
https://github.com/dankito/RichTextEditor#create-custom-editor-toolbar
You also can set e.g. toolbar's background color there.
If you also want to set a command's icon, you can do it like this:
toolbar.addCommand(new BoldCommand(new AndroidIcon(R.drawable.ic_format_bold_white_48dp)));

Does this answer your question?

Thank you for your answer!
Although I wanted to reuse my whole toolbar layout (It has 4 style commands and two extra button for app functionalities) but it seems it wont be possible. Since the icons and colors are changeable, I'll try to find a work around.

Sorry, i still don't fully understand what you are planning to do.
But may this helps you:
As EditorToolbar is derived from HorizontalScrollView you can add any view - standard Android views and custom views - to your liking:
toolbar.addView(<your view>);


Removed OAuth2 topics

Removed OAuth setup and user settings as now is not open to public

[CI] Staging environment is at http://cap-staging-docs.codefresh.io.s3-website.us-east-1.amazonaws.com/csdp-ouath-revoke/docs/


Rename package/groupId to org.apache and apply rat plugin.

This PR handles https://issues.apache.org/jira/browse/ZEPPELIN-12.

groupId at pom.xml file is changed from com.nflabs.zeppelin to org.apache.zeppelin
package name is changed from com.nflabs.zeppelin to org.apache.zeppelin
apache-rat plugin is applied (license header is added to every file) and NOTICE is updated (https://www.apache.org/legal/src-headers.html)
removed sphinx doc. because of doc was out dated (it was for 0.3.0)

Please, review the changes. Especially, check NOTICE file if there're something i missed.

It should be merged as fast as you can for another PR.

I'm merging it if there's no more discussion

+1


When the addBatch method encounters an error, the writePosition of ClickHouseDataProcessor is not reset to 0, causing the next execution of addBatch to start from an incorrect writePosition.

Describe the bug
When the addBatch method encounters an error, the writePosition of ClickHouseDataProcessor is not reset to 0, causing the next execution of addBatch to start from an incorrect writePosition.
Steps to reproduce

Make an error in a loop of addBatch

Expected behaviour
When the addBatch method encounters an error, the writePosition of ClickHouseDataProcessor should reset to 0.
Code example
for (int i = 0; i < 2; i ++) {
    try {
        ps.setTimestamp(1, new Timestamp(value.getTime()));
        ps.setString(2, "1");
        ps.setString(3, i == 0 ? null : "1");
        ps.addBatch();
    } catch (SQLException e) {
    }
}

The first error of Cannot set null to non-nullable column #3 is caught. We expect the second addBatch should run successfully. However, there is an error in the second addBatch.
Error log
java.time.format.DateTimeParseException: Text '1' could not be parsed at index 0
at java.time.format.DateTimeFormatter.parseResolved0(DateTimeFormatter.java:1949)
at java.time.format.DateTimeFormatter.parse(DateTimeFormatter.java:1851)
at java.time.LocalDateTime.parse(LocalDateTime.java:492)
at com.clickhouse.data.value.ClickHouseStringValue.asDateTime(ClickHouseStringValue.java:208)
at com.clickhouse.data.format.BinaryDataProcessor$DateTime64SerDe.serialize(BinaryDataProcessor.java:291)
at com.clickhouse.data.ClickHouseDataProcessor.write(ClickHouseDataProcessor.java:593)
at com.clickhouse.jdbc.internal.InputBasedPreparedStatement.addBatch(InputBasedPreparedStatement.java:345)
Configuration
Environment

Client version: 0.4.6
Language version: Java 8
OS: Mac

ClickHouse server

ClickHouse Server version:
ClickHouse Server non-default settings, if any:
CREATE TABLE statements for tables involved:
Sample data for all these tables, use clickhouse-obfuscator if necessary

Thanks @ibufu for reporting the issue. Yes, resetDataProcessor() should be able call. Will raise a PR to fix this along with a few more tests.

Hi, @zhicwu ! Any suggestions for handling above exceptions before you fix the bug? Thanks!


Fix UARTE

Related to https://github.com/nrf-rs/nrf52-hal/pull/43

https://github.com/nrf-rs/nrf52-hal/pull/43 is now merged. Restarted the Travis build, let's see what comes of that.
@jamesmunns I'm starting to prepare for the initial crates.io release now. I'm not sure when that will happen, as I haven't decided which issues to close first, but if you want this pull request to be included, it needs to be merged soon-ish.


registration

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/89523
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 50872154a3b2b0d5349476736b88c5b4ab316be3
git push origin v0.1.0


ERROR: Unable to find the converted log file: C:\Users..l\Temp\75ca7034-adb2-4ed5-9afb-07bdc9cfe71d_FilteredPerfmonLog.csv. Relog.exe failed to process the log.

Hi, trying to process a log counter file and got an error i the very beginning.
Unable to find the converted log file: C:\Users..l\Temp\75ca7034-adb2-4ed5-9afb-07bdc9cfe71d_FilteredPerfmonLog.csv. Relog.exe failed to process the log.
Please contact the PAL tool team with this error by posting it to GitHub.com/clinthuffman/PAL. Thank you!

I managed to find the issue and I am writing here directly. If in the path to the counter file there is a symbol which is language specific different than EN, then you will get this error message. Try to change the path, copy the counter file to a different folder


Add security biometric to the app for privacy case.

So, in Settings create a card with a switch so on enabling the switch and re-opening the app the biometric dialog will pop up in the Splash Screen.

Sure @swapnil-musale go head 👍

@swapnil-musale please check CONTRIBUTING.md before contributing.
Thank you.

@aritra-tech currently develop branch is behind master by 2 commits

Okay, my bad sorry for the inconveniences. You can create your branch from master only as of now.

Ok

Any update @swapnil-musale?

@aritra-tech feature development is in progress expecting to be finished in next 2 days

Okay @swapnil-musale


Is Tweet pre-processing needed?

I notice that a previous work BERTweet uses a normalization of Tweets that translate emojis and URLs into specific tokens.
Since you have an emoji prediction task, I guess emojis are not just simply translated into a fixed token. Are they stored in the vocabulary as Unicode characters, or translated into fine-grained tokens? And, how are URLs tokenized, URLs can be sometimes long and meaningless, but sometimes informative.
I didn't find related descriptions in your paper, and I'd like to know if I need a pre-processing of Tweet before using TwHIN-BERT, or just feed a raw Tweet text into the model.
And, of course, thank you very much for the great multilingual pre-trained model!

We did not preprocess emojis and URLs. They are stored as unicode characters, the format you'd get from Twitter API. No preprocessing for tokenization is needed.

Thank you for the reply!


Don't do !important on the button width

We shouldn't do !important on the button width cause that prevents us from custom styling it.
Example case, if you want your buttons to be 100% width on mobile devices but auto width on regular screens? With the current styling you can't achieve this because the scss is parsed to inline css with width: 100% !important which has the highest priority when rendering.

According to this chart, the :not pseudo class doesn't work in a lot of clients: https://www.campaignmonitor.com/css/
This is probably why the !important is on the button class. If you have a test that shows that removing it works across all the supported clients, we'd love to drop it!

@rafibomb The :not doesn't really have anything to do with the email clients. It is there just to tell the sass compiler and css inliner on which elements to attach or not to attach the styles.
The !important is there because the th.column table has higher precedence
than table.button, and therefore we use :not to tell the compiler "hey apply 100% to all tables except the button table, cause we don't need it there", instead of doing that with the !important. :)

@rafibomb And you have gulp test that you can check and see that the rendered code is ok.

Thanks! I'll be testing this out then merging!

This looks good, thanks! Working on adding .small-expaned option to buttons. Expanded buttons need the center tag wrapped around the inner td. Here is the working branch if you have an suggestions: https://github.com/zurb/foundation-emails/tree/small-expanded-button
Visual test: https://github.com/zurb/foundation-emails/blob/small-expanded-button/test/visual/pages/button-inky.html


Minimize open editors at the side bar as a single line

Is your feature request related to a problem? Please describe.
We already have opened files on the top bar, the side bar one seems unnecessary most of times
Describe the solution you'd like
We can let Open Editors which at the sidebar can be minimize as a single line
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
The Open Editors takes me 25% space in the sidebar

VSCode lets you hide this entirely by right-clicking on it. Seems like a decent option:

This is related to #1398.

This is related to #1398.

No it isn't, 1398 is a bug for lapce, this is only a feature request

I'm not saying it's a duplicate, just that someone looking at this may see similar comments in that issue, or someone in that issue may see this one as a mechanism to resolve the issue.

I'm not saying it's a duplicate, just that someone looking at this may see similar comments in that issue, or someone in that issue may see this one as a mechanism to resolve the issue.

In that case, you should comment in 1398, not this issue. Because 1398 cannot affect this issue


Multi-Array support

Multi-array support for bjoern andres marray was added with
4f90a13f218ea41ff18dc0afd36e11611bb50089
Additional multiarrays can be added by reimplementing the methods readSubarray and writeSubarray from https://github.com/constantinpape/zarr_pp/blob/master/include/zarr%2B%2B/multiarray/marray_access.hxx.


Consider delombok before GA

Lombok is pretty sweet but it does add an extra layer that users need if they want to contribute. It might be worth considering if it should be removed before GA.

It only means they have to add an agent to the IDE if they use one, and the build works out of the box (no-one complained yet). I'm not sure I want to do this to be honest (look at how many properties there are in EurekaClientConfigBean).


Add Contact Form to Index Page

Pull Request Type
What kind of change does this PR introduce?
[x] Solved an issue/task
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:
What is the current behavior?

Issue Number: Fork, Commit, Merge - Medium Issue (HTML) #1534
A contact form has been added to the index page, allowing users to submit their name, email, and message. This form is placed inside the container and is styled using Bootstrap CSS for a clean and user-friendly appearance.
Other Information:

The contact form includes a field for the user's name (text input), email (email input), and message (textarea).
The form is accessible and includes a legend for the fieldset.
The submit button is styled using Bootstrap CSS.
The form uses HTML5 input types for improved user experience and validation.
The form is wrapped in a fieldset for better accessibility and organization.

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out our forkcommitmerge.io website repository. The repository contain real issues that you can help to resolve or create some of your own if you see any bugs or want to add new features to the site, etc. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


bug: Localleader of plugin(s) is remapped to leader

Did you check docs and existing issues?

[X] I have read all the which-key.nvim docs
[X] I have searched the existing issues of which-key.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
0.8.3
Operating system/version
Manjaro
Describe the bug
I use the https://github.com/jalvesaq/Nvim-R and https://github.com/lervag/vimtex plugins in combination with https://github.com/LazyVim/starter
Both plugins define \ as a localleader for their mappings.
When I am in a corresponding buffer (.r or .tex) the key bindings of both plugins are mapped to  (Space) instead of local leader.
Running :WhichKey \ also returns nothing.
I am aware of #150 as well as #35 and tried to explicitly set vim.g.maplocaleader in the config of the plugins.
However, I am not sure whether it is the expected behavior that  is re-mapped to  or not.
Steps To Reproduce

Setup Nvim according to https://www.lazyvim.org/installation
Configure/install both or one of https://github.com/jalvesaq/Nvim-R and https://github.com/lervag/vimtex plugins with default settings (i.e., no custom config or key mapping applied)
Open a .tex or .r buffer and press \ to use the key mappings of the plugins.

Expected Behavior
Pressing \ in an .r or .tex buffer should show the default mappings defined by the vimtex or nvim-r plugin.
Repro
No response

Both plugins define \ as a localleader for their mappings.

Plugins don't define a localleader, a user does.
In LazyVim, localleader is set to <space>, see here https://github.com/LazyVim/LazyVim/blob/main/lua/lazyvim/config/options.lua#L4
You can set it to anything else in your local ~/.config/nvim/lua/config/options.lua


Debug.log could be more intuitive for beginners and less misuse prone

Problem
It's  easy to be tricked by a partial application of Debug.log that does not log anything:
let
    _ = Debug.log stringValue
in

A lot of beginners try Debug.log "I got here" logs and are asking on slack why this does not work.   Even experienced developers are occasionally tricked by mute _ = Debug.log value, wasting some precious debug time.
Requirements
Based on the feedback of developers and those helping regularly beginners, it seems that to cover most use cases without being error prone, the log function must:

Be able to log any value
Return the logged value
Be able to log just a string
Be able to be added into a pipeline without changing the result
Not be easily partially applied

One solution
Consequently, an alternative to the current design would be to have two functions, the default one just logging the value, another one prefixing a string before the value like the current design:
log : a -> a

logWith: String -> a -> a

It's then expected to not have any log with a partial application, as nothing is logged:
let
    _ = Debug.log
in

It's easy and intuitive to log a simple string:
Debug.log "I got here!"

It still works in pipelines:
thing
    |> Debug.log
    |> ...

Moreover Debug.log allows to log a value without the cognitive load of having to find a tag name, which is a very common use case if you need only one log during a debug session.
Lastly if the log is found confusing, it's easy to consciously prefix a tag with logWith:
thing
    |> Debug.logWith "tag"
    |> ...

Possible Improvements
A later improvement could be to add automatically a tag for the Debug.log function based on compiled code information, for example the function name or source code file and source code line.
Impact
This would be a major change in semantic versioning. However the fact that Debug.log is not permitted with --optimized should significantly limit the actual impact on users source code.

Thank you @jessta, @pd-andy and others for the inputs and feedback.

Is a hack but something alone these lines might work:
const log = name => {
    const p = Promise.reject("you must pass this function two arguments!");
    return value => {
        p.catch(x => x);
        console.log(`${name}: ${value}`);
    }
}

If you call log('a') you get a warning but calling log('a', 5) works as expected. Tested in node and in firefox console.

I agree with you, and I think log/logWith should be formatted as a tree by default to be inspected/expanded, currently it is displayed as a big unreadable string.

Just to tell that this issue is important. I spent 3 hours until 03:00 am yesterday because of that, only to find today that the compiler is not enforcing the condition of having two arguments.
Since we usually can trust the compiler, I never ever imagined this time was an exception, just to find out that I was wrong :)

I agree that the usage in the Debug.log function could be made less error prone, especially for beginners. Since we're ignoring the data coming from it, it's common to forget the currying when printing strings and get the outcome pointed to in this issue.
One pattern I adopted on my elm usage that avoids this issue is trying to use the Debug.log to print data that is already in the data flow of my application, avoiding the _ =  definition. In that way, I try to always log some data in-place and actually using the return type of the Debug.log function, which makes the compiler actually able of telling me when I forgot a parameter.
Anyway, it isn't a fix and I think elm could benefit from the suggestion of "log/logWith", maybe just avoiding the breaking change, but I wanted to share as a mindset change that helps me avoid this mistake.

I think the compiler should just tell you to read the docs for the Debug module when you get an error with using Debug.log

Actually maybe the type of Debug.log should become this in order to prevent accidental partial application:
( String, a ) -> a

Actually maybe the type of Debug.log should become this in order to prevent accidental partial application:
( String, a ) -> a

It would not be possible anymore to insert it in the middle of a pipeline without composing with Tuple.pair, which would not be as practical. This is why I put this as a requirement in my orinal post above.

I think having Debug.log and Debug.logWith is the best solution. But Debug.logWith should instead be named Debug.logAs

Maybe Debug.log should just instead take a { label : String }

Maybe Debug.log should just instead take a { label : String }

I see there is a "breaking" label, with this option: https://github.com/elm/core/issues/1082#issuecomment-615234194 it doesn't have to be a breaking change.


Is there an offline harp.js file ?

Is there an offline harp.js file?
How to deploy offline？

@15375296005 sorry for the delay, I don't quite understand the question. You could download locally and use it offline.

If you want to generate it locally, you can run yarn build-bundle, then go to the @here/harp.gl/dist folder and search for harp.js

Closing as no response from @15375296005


DonorsChoose success message copy change

What's this PR do?

Changes hardcoded "Teacher says" copy in 2nd success message sent upon completing a DonorsChoose donation.

How should this be reviewed?
Locally:
Post required params to /v1/chatbot?bot_type=donorschoose to confirm copy change.
Staging:
Text in keyword for staging DonorsChoose campaign upon deploy to staging server.
Any background context you want to provide?
Hardcoding this copy was one of the tradeoffs made when we made the move from configuring our message content from editing Mobile Commons OIP Conversations  into the Gambit Jr. API.  We could potentially add this to our own internal {{postal_code}} Liquid-esque logic we're doing here, but it also seems easy enough to keep hardcoded for now, since we don't run multiple DonorsChoose campaigns at a time.
Relevant tickets
Fixes #601
Checklist

[ ] Documentation added for new features/changed endpoints.
[x] Tested on staging.

Updated the help text in Gambit Jr. edit form via editing the donorschoose_bot Pod and making the change in msg_donation_success description:


Add help to qshell

Fixes issue #5111

@ericye16: Can you please this on PX4/master and push it again. You'll have to force push using -f to your branch. This will then run the CI checks again. Thanks.

I think you a word ;-P Did you mean rebase? As far as I can tell, this branch is already based on PX4/master...

Ha! Yep rebase :)

Jenkins: test this please

Fetching upstream changes from https://github.com/PX4/Firmware
> git --version # timeout=10
> git -c core.askpass=true fetch --tags --progress https://github.com/PX4/Firmware +refs/heads/:refs/remotes/origin/
ERROR: Timeout after 10 minutes
ERROR: Error cloning remote repo 'origin'
ERROR: Error cloning remote repo 'origin'

Is the infra down maybe?

Applied via rebase, thanks!


Correct build command for Linux/Mac

./build.sh instead of build.sh

Thanks!

You're welcome.
Anyway, at the moment, it doesn't work on my mac. Problems with ttx: command not found and I followed your istructions.

Other people had similar issues. Try this from another comment.
https://github.com/kiliman/operator-mono-lig/issues/22#issuecomment-335505346


CVRP gives different result in Python and Java

Or tools version:7.5.7466
Python: 3.7
We developed a capacitated vehicle routing model both in python and java platforms. We use the same version of or-tools packages (7.5.7466) and all inputs and searching parameters are the same. But the results are not same.  For instance we have 10 demands and the capacity of each vehicle is 6 demands.  In python 1st vehicle has 5 demands and  2nd vehicle has 5 demands, In java 1st vehicle has 6 demands and 2nd has 4 demands.
Is it possible due tue heuristic approach of or-tools or we have to check some another points.
Regards

Can you share the model?

this is the  Python Code
`
def create_data_model():
data = {}
data['distance_matrix']=a

data['demands'] = [int(x) for x in talep_ister_zaman2['Tasiyici_Uzunluk']]

data['demands'][0]=0
data['num_vehicles'] = len(binpacking.to_constant_volume(data['demands'] ,capacity_per_vehicle) )
data['vehicle_capacities'] = [capacity_per_vehicle]*data['num_vehicles']
data['depot'] = 0
return data

def print_solution(data, manager, routing, assignment):
"""Prints assignment on console."""
total_distance = 0
total_load = 0
#sira=0
konvoy_max_element=len(konvoy)
for m in range(data['num_vehicles']):
    vehicle_id=konvoy_max_element+m
    konvoy[vehicle_id]=pd.DataFrame()
    index = routing.Start(m)
    plan_output = 'Route for vehicle {}:\n'.format(vehicle_id)
    route_distance = 0
    route_load = 0
    sequence=0
    while not routing.IsEnd(index):
        node_index = manager.IndexToNode(index)
        route_load += data['demands'][node_index]

        plan_output += ' {0} Load({1}) -> '.format(node_index, route_load)
        previous_index = index

        index = assignment.Value(routing.NextVar(index))
        route_distance += routing.GetArcCostForVehicle(
            previous_index, index, m)
    plan_output += ' {0} Load({1})\n'.format(manager.IndexToNode(index),
                                             route_load)
    plan_output += 'Distance of the route: {}m\n'.format(route_distance)
    plan_output += 'Load of the route: {}\n'.format(route_load)
    total_distance += route_distance
    total_load += route_load

def main():
data = create_data_model()

manager = pywrapcp.RoutingIndexManager(len(data['distance_matrix']),
                                       data['num_vehicles'], data['depot'])

routing = pywrapcp.RoutingModel(manager)
def distance_callback(from_index, to_index):
    from_node = manager.IndexToNode(from_index)
    to_node = manager.IndexToNode(to_index)
    return data['distance_matrix'][from_node][to_node]

transit_callback_index = routing.RegisterTransitCallback(distance_callback)
routing.SetArcCostEvaluatorOfAllVehicles(transit_callback_index)

def demand_callback(from_index):
    """Returns the demand of the node."""
    from_node = manager.IndexToNode(from_index)
    return data['demands'][from_node]

demand_callback_index = routing.RegisterUnaryTransitCallback(
    demand_callback)
routing.AddDimensionWithVehicleCapacity(
    demand_callback_index,
    0,
    data['vehicle_capacities'],
    True,
    'Capacity')

search_parameters = pywrapcp.DefaultRoutingSearchParameters()
search_parameters.first_solution_strategy = (
    routing_enums_pb2.FirstSolutionStrategy.PATH_CHEAPEST_ARC)
assignment = routing.SolveWithParameters(search_parameters)

if assignment:
    print_solution(data, manager, routing, assignment)

if name == 'main':
main()
`

To recreate this I'm sure the data will be needed.

Here is the data as below.
`data['distance_matrix']=[[0, 351, 291, 344, 291, 399, 248, 245, 399, 258, 167],
[533, 57, 150, 190, 124, 46, 31, 9, 97, 9, 141],
[364, 204, 57, 121, 30, 100, 155, 137, 49, 102, 86],
[540, 139, 233, 57, 207, 36, 114, 72, 14, 72, 224],
[364, 230, 83, 147, 57, 126, 182, 163, 76, 128, 113],
[485, 376, 254, 294, 228, 57, 135, 93, 6, 93, 245],
[407, 323, 176, 240, 149, 219, 57, 256, 168, 221, 205],
[423, 340, 434, 257, 407, 236, 314, 57, 186, 263, 247],
[485, 427, 304, 344, 278, 107, 185, 144, 57, 144, 296],
[397, 621, 485, 538, 459, 517, 366, 338, 467, 57, 208],
[488, 469, 334, 387, 307, 366, 215, 186, 315, 173, 57]]
data['num_vehicles'] = 2
data['demands']=[0,1,1,1,1,1,1,1,1,1,1]
data['vehicle_capacities']=[6,6]
data['depot'] = 0`

If no one gets to this before the weekend I'll give it a look.

my 2 cents,
https://github.com/google/or-tools/blob/45770b833997f827d322e929b1ed4781c4e60d44/ortools/constraint_solver/constraint_solver.cc#L89-L92
did you test on the same machine ?

We do not aim for determinism across platforms.


AutoBogus.Conventions - Does StateAbbr work?

Hi. I've been using AutoBogus on a new project and I love it. I have one issue though I was hoping was maybe something I did wrong on my end or maybe this is an actual issue. I'm using .NET Core 3.1, AutoBogus 2.12.0 and AutoBogus.Conventions 2.12.0
This simple test (using NUnit and FluentAssertions fails:
[TestFixture]
public class ValidateAutoBogus
{
    public class TestClass
    {
        public string State { get; set; }
    }

    [SetUp]
    public void SetUp()
    {
        AutoFaker.Configure(builder =>
        {
            builder.WithConventions(conventionConfig =>
            {
                conventionConfig.StateAbbr.Aliases("State");
            });
        });
    }

    [Test]
    public void StateAbbrShouldBeUsed()
    {
        var result = AutoFaker.Generate<TestClass>();

        result.State.Should().HaveLength(2);
    }
}

The error from the test is: 'Expected result.State with length 2, but found string "Texas" with length 5.'
Any ideas how I can get a state code out of the conventions?

Hacking around, I found that explicitly setting
conventionConfig.State.Enabled = false;
got the test to pass. Maybe something related to trying to use a convention for a field name that normally has a different associated convention?

It seems like this happened because State was first in an IEnumerable of generator configs... It found State first before StateAbbr and used State as the generator instead of StateAbbr... Maybe just some simple documentation callout would be helpful... saying to always explicitly disable conventions when trying to do some override like this?

Hey @er1k-peterson
I have taken a look at your issue, and you are correct that State is being checked before StateAbbr. There are a load of auto generated convention handlers built from the Bogus library. When a matching being made, it follows a greedy approach in that the first match is used. Because your property name and type (State and string) match the requirements for State it is diving stright in to use it before getting to StateAbbr.
The suggestions here are  to use the Enable flag like you have or rename your property to something like StateCode.
Nick.

Thank you.


Add body dimension to fhir transformer

@brandonpollett,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

After #15 is pulled in, this will need a bit of refactor to take advantage of the date and abbreviation work done there.


yolof demo ？

How to use yolof to detect a single photo? Are there any sample programs like detectron2's demo?
@chensnathan

Any progress on this issue?

How to use yolof to detect a single photo? Are there any sample programs like detectron2's demo?
@chensnathan


Race condition when waiting repo state

During release there was error in logs(thought all worked fine):
Staging Repo Release
Released repo test-1078
Waiting repo test-1078 state released
Repo state closed
(node:2840991) UnhandledPromiseRejectionWarning: Error: Request failed with status code 404
    at createError (/home/ghrunner/atest7/_work/_actions/jvalkeal/nexus-sync/gh3-work1/dist/index.js:105:15)
    at settle (/home/ghrunner/atest7/_work/_actions/jvalkeal/nexus-sync/gh3-work1/dist/index.js:6088:12)
    at IncomingMessage.handleStreamEnd (/home/ghrunner/atest7/_work/_actions/jvalkeal/nexus-sync/gh3-work1/dist/index.js:6570:11)
    at IncomingMessage.emit (events.js:215:7)
    at endReadableNT (_stream_readable.js:1184:12)
    at processTicksAndRejections (internal/process/task_queues.js:80:21)
(node:2840991) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 2)
(node:2840991) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
Post job cleanup.

Need to check where it's coming from.

Might be when release-auto-drop is set to true, repo goes away while we're trying to get released state. Probably need to assume in this case that repo release succeed if it's automatically dropped by nexus.


Permission Dialog always showing

Show App permission dialog only for first time

In my phone it never shows  that to me.. (Sigh)

since i was side loading the app i think app permissions automatically granted or
may be app is targeting minimum sdk version that doesn't require user interaction(means targeting sdk version 22) so permissions granted automatically

You mean AlertDialog that shows rationals

Yes

This should have been fixed from release 0.9.8.


How to stop a conversion?

Rasa Core version:
latest
Python version:
py3.5
Operating system (windows, osx, ...):
ubuntu
Issue:
i use the api to start a conversion,but i don't know how to stop?the function in the api has PAUSE(rasa_core.events.ConversationPaused),but i don't find STOP, is the conversion effect the performance of the server if not stop?
Content of domain file (if used & relevant):

what do you mean? you don't need to stop a conversation, the user can simply stop replying to the bot :)

@akelad Is there any way to break conversation flow from middle of a story and drop in another story?

You would have to create a story for this

Hi Guys,
On this topic, I have 1 question is that when a user asks a bot a question, the bot will answer the question in order to gather more information about the user's question, but then the user does not respond to the bot. What will bot do? As far as I understand, the bot will keep the status of this chat / story forever, right?
Is there any idea to make the bot to automatically take an action to proactively stop the conversation like: 'I've been waiting for your response for 15 minutes, please check and answer soon :)'

Yes, you can use the ReminderScheduled event. I'm gonna close this issue as this question isn't relevant to the original topic of conversation. If you need more help on how to use this event, then please post in our forum


New version: CCPGames.EVEOnline version 1.2.2

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

/AzurePipelines run

/AzurePipelines run

@wingetbot waivers Add Validation-Unattended-Failed

You should just have forced merger the pr

Sorry for the delay, but our process is to add a waiver and re-run Automatic Validation.


CB-7902 add /hive2 and /cliservice to support PowerBI driver to conne…

…ct through Nginx
PowerBI drivers use 2 default hard-coded paths to contact HiveServer2: /cliservice and /hive2.
CDW forwards all these paths to the corresponding Hive install through Nginx, because the
cluster name is encoded in the hostname. Knox encodes the endpoint information in the HTTP path
and it is not possible to override it in PowerBI drivers.
A proxy_pass through nginx will result in Knox rejecting /cliservice or /hive2 and it does not
work out of the box.
See detailed description in the commit message.

Looks good on the surface. Might want to have @t3rmin4t0r try it on a cluster manually updating the nginx config to confirm.

Yep, good idea. @t3rmin4t0r can you do that?


convert slime template to heex

this pr is an implementation for #11

I think you need to run mix deps.clean --unused --unlock to clean up the following unused deps, which should get CI to pass:
** (Mix) Unused dependencies in mix.lock file:

  * :neotoma
  * :phoenix_slime
  * :phoenix_slime_live_view_collocated_template
  * :slime

Potentially, we can do the conversion in batches, however, given the number of files that changed, I feel it might be easier to do in one swoop.


SSJS libraries are cut off around 32K

Most likely, the export code isn't reading "this is followed by X structures" properly and is only dealing with the first one.

Though my test exported the whole file, I hit an exception when compiling:
Caused by: java.lang.RuntimeException: Input length = 1
	at com.ibm.commons.util.FastStringBuffer.append(FastStringBuffer.java:792)
	at com.ibm.commons.util.FastStringBuffer.load(FastStringBuffer.java:778)
	at com.ibm.commons.util.io.StreamUtil.readString(StreamUtil.java:205)
	at org.openntf.nsfodp.commons.odp.util.DXLNativeUtil.getJavaScriptLibraryData(DXLNativeUtil.java:53)
	at org.openntf.nsfodp.commons.odp.JavaScriptLibrary.getCompositeData(JavaScriptLibrary.java:47)
	at org.openntf.nsfodp.commons.odp.JavaScriptLibrary.attachFileData(JavaScriptLibrary.java:52)
	at org.openntf.nsfodp.commons.odp.AbstractSplitDesignElement.getDxl(AbstractSplitDesignElement.java:71)
	at org.openntf.nsfodp.compiler.ODPCompiler.lambda$12(ODPCompiler.java:635)
	at org.openntf.nsfodp.compiler.ODPCompiler$$Lambda$137/0x0000000000000000.apply(Unknown Source)
	at java.util.stream.Collectors.lambda$toMap$58(Collectors.java:1321)
	at java.util.stream.Collectors$$Lambda$170/0x0000000000000000.accept(Unknown Source)
	at java.util.stream.ReduceOps$3ReducingSink.accept(ReduceOps.java:169)
	at java.util.stream.ReferencePipeline$2$1.accept(ReferencePipeline.java:175)
	at java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1384)
	at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:497)
	at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:487)
	at java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:708)
	at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:241)
	at java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:566)
	at org.openntf.nsfodp.compiler.ODPCompiler.importFileResources(ODPCompiler.java:631)
	at org.openntf.nsfodp.compiler.ODPCompiler.compile(ODPCompiler.java:367)
	at org.openntf.nsfodp.compiler.ODPCompiler.compile(ODPCompiler.java:301)
	at org.openntf.nsfodp.compiler.servlet.ODPCompilerServlet.lambda$3(ODPCompilerServlet.java:213)
	at org.openntf.nsfodp.compiler.servlet.ODPCompilerServlet$$Lambda$39/0x0000000000000000.call(Unknown Source)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at lotus.domino.NotesThread.run(Unknown Source)

That trouble comes from malformed unicode sequences that happen a good chunk of the way through the exported file. My hunch is that this comes from improper reading of the data - that could be misc. struct garbage being read out as strings.

If I replace the content on disk with a clean >32K file from elsewhere, the import works, but the export is oddly chomped in the middle: the start and end both exist, but the middle is missing a bunch of text. It could be that iteration within each item breaks silently.

It seems like this might be encoding-related, either on reading or writing. The first chunk of data, which is compiled in 20,000 bytes at a time, is read back at 20,000 but read back as 7233 bytes when read back through LMBCS. It's not just a matter of not needing the LMBCS path, though, since I believe that is indeed the proper encoding. It could be that there's a null character in there somewhere... it being at 7233 rules out chunked CD records, which is helpful, at least.

Modifying the example to output functions numbered from 1 - 499 in the name makes it clear that both troubles seem to be happening: output goes from 1-140, then 303-337, then stops.


fix(android): check null activity

If activity is null at initialization, the app crashes. This change checks for null activity and throw react native error, which prevents app crash.

hey @wjaykim what stack trace were you getting for the native crash? the app i'm working on is seeing quite a bit of null pointer exceptions on android (we're running 6.0.0-alpha.2). this is the trace we're getting from the crashes:
Exception java.lang.NullPointerException: Attempt to invoke virtual method 'void android.app.Activity.runOnUiThread(java.lang.Runnable)' on a null object reference
  at com.brentvatne.exoplayer.ReactExoplayerView$6$1.run (ReactExoplayerView.java:548)
  at java.util.concurrent.ThreadPoolExecutor.runWorker (ThreadPoolExecutor.java:1167)
  at java.util.concurrent.ThreadPoolExecutor$Worker.run (ThreadPoolExecutor.java:641)
  at java.lang.Thread.run (Thread.java:929)

@alekhinen @freeboub I couldn't reproduce crash, in my own device, but crashlytics logged some user's crash. The error trace was same with @alekhinen posted.

@freeboub I haven't reproduced the crash on my end but I'll follow those steps and see if that reliably causes an app crash. Your steps seem to line up with what Google Play is reporting for my app. About 98% of all instances of this crash occur when the app is backgrounded.


Allow backend host and Port configurable

The default_host = "su1.3scale.net".
To work with on-prem version, the host and the port would need to be configurable.

This appears to already be supported by the ServiceApiDriver class.  The following constructors exist:
ServiceApiDriver()
ServiceApiDriver(String::provider_key)
ServiceApiDriver(String::provider_key, boolean::useHttps)
ServiceApiDriver(String::provider_key, String::host)
ServiceApiDriver(String::provider_key, String::host, boolean::useHttps)

The last two variants allow a custom host to be used (rather than the default which is su1.3scale.net.  The port can be passed in as part of the host.  For example, you could do this:
ServiceApiDriver driver = new ServiceApiDriver(KEY, "myserver.foo.com:8080");

Or if you were using https on a custom port you could do this:
ServiceApiDriver driver = new ServiceApiDriver(KEY, "myserver.foo.com:8443", true);

This isn't great, admittedly.  The code just creates a string and then uses that to create a URL, which isn't the best way to connect to a server, but it DOES work.
I could update the documentation (readme) if that's helpful.  Or if major improvements to this code are desired, I could spend some time bettering it. :)

@EricWittmann Thank you very much! yes, a major improvement would be needed. It would be very helpful if you had some time for bettering it :-)
We can take care of the documentation.

OK I'll leave the documentation to you, then.  :)  I'll hold off any any significant changes for later, and only focus on minimal changes required for this upcoming on-prem release.  We should save any breaking changes I might make for a new major version of the plugin.

OK, thanks! @EricWittmann


Transition.styleTween has incorrect signature

The signature of Transition.styleTween is currently:
styleTween(name: string, tween: (datum: Datum, index: number, attr: string) => Primitive, priority?: string): Transition; (line 833)
Note that the tween is said to return a Primitive. This seems incorrect, both in terms of D3 intent and implementation.
The correct version appears to be:
styleTween(name: string, tween: (datum: Datum, index: number, attr: string) => (t: number) => Primitive, priority?: string): Transition;
(This is similar to  similar to Transition.attrTween.)
First, the documentation states:

The return value of tween must be an interpolator: a function that maps a parametric value t in the domain [0,1]
to a color, number or arbitrary value.

Second, the source code of d3 3.5.5 has:
d3_transitionPrototype.styleTween = function(name, tween, priority) {
if (arguments.length < 3) priority = "";
function styleTween(d, i) {
var f = tween.call(this, d, i, d3_window(this).getComputedStyle(this, null).getPropertyValue(name));
return f && function(t) {
this.style.setProperty(name, f(t), priority);
};
}
return this.tween("style." + name, styleTween);
};
Note the line "this.style.setProperty(name, f(t), priority);" where the result f of applying the tween is passed a parameter t.
The only point of discussion might be the type of the return value of the tween's interpolator output. Is it Primitive or any? The documentation quoted above (incidentally the same for attrTween and styleTween) explicitly allows for an arbitrary value. I don't have enough D3 experience to know if this is a practically relevant possibility.
Many thanks for your great work on d3.d.ts!!! Especially the use of tweens and interpolators perfectly illustrates the benefits of Typescript.
Best wishes,
Matthias

d3/d3.d.ts
to authors(@gustavderdrache @borisyankov). could you review this PR?
:+1: or :-1:?
check list

[ ] pass the Travic-CI test?

:+1:
(I picked Primitive here and in other places to try and compromise between being lenient about attributes and being strict about how it's being used, since the DOM APIs use string values exclusively.)

Not sure if your comment applies to the main issue or only to my last remark. To be sure we are talking about the same thing: The style tween must return the an interpolation function (i.e. (t: number) => Primitive), NOT primitive.

Sorry, wasn't clear. Your PR is right, I was just describing why the interpolator's return value should be Primitive and not any.

@vvakame this code is ready to merge whenever you'd like.

@gustavderdrache thanks!
@MatthiasHild thanks mate!


pathping output is mangled

the codeblock with the pathping results is unreable in it's current state.

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

ID: 0eee40cd-e852-11e7-7713-0e8c8033dc53
Version Independent ID: 147206f0-5fb6-edf3-70ee-02bc9b6a2c09
Content: pathping
Content Source: WindowsServerDocs/administration/windows-commands/pathping.md
Product: windows-server-threshold
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

I can add that the markdown looks just fine when viewing it at github but completely broken on https://docs.microsoft.com/en-us/windows-server/administration/windows-commands/pathping

Yep, the markdown just needed some carriage returns to break up the sections. I'm working on it now. Thanks!

I fixed the formatting so you can view the actual example syntax now. Thanks for raising this issue.


[Don't review] CI Test

Thanks for submitting a pull request!
Please provide enough information so that others can review your pull request.
Summary
Explain the motivation for making this change.
What existing problem does the pull request solve?
Are there any linked issues?
Result
Demonstrate the code is solid.
Example: The exact commands you ran and their output,
screenshots / videos if the pull request changes UI.

Hi @aymeric-giraudet, why did you close this PR before all the checks were done?
I communicated this activity with your security team


Hope adding a LoadBalancer plugin

Have you considered adding a LoadBalancer plugin? I can contribute if possible

What do you mean by "LoadBalancer"?

It may not be accurate, it should be resolver, just like Spring Cloud adds the ability to support service discovery for Fegin, just add a Resolver ability to allow users to increase the ability of service discovery to make service calls?

It may not be accurate, it should be resolver, just like Spring Cloud adds the ability to support service discovery for Fegin, just add a Resolver ability to allow users to increase the ability of service discovery to make service calls?

Yeah, that would be useful sometimes. PR is welcomed


add ConfigMap and Service for observability

Changes

this PR adds ConfigMap for configuring metrics and a Service to expose the port

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has Docs included if any changes are user facing
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[x] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
- Metrics can be accessed by exposing the Service `tekton-chains-controller`
- adds a ConfigMap for configuring metrics

/ok-to-test
/lgtm


loosen peer dependency restriction on relay

would like to use this util w/ relay 0.8.0

Thank you!
Will close as a duplicate of #234


[ART-7621] Refactor advisory fetching to handle multiple advisories

The function get_branch_advisory_ids now handles all advisories instead of only the first and second in the list. This change makes the code more flexible.

/lgtm
/approve


Ginkgo: Replace awk usage with jq

The majority of the cilium commands now have support for '-o json'
output, so replace these usages of '... | grep ... | awk ...' with
'-o json | jq ...'.
Fixes: #1725
Signed-off-by: Joe Stringer<EMAIL_ADDRESS>

test-me-please


admin ports view takes too much space

closed in favor of https://github.com/threefoldtech/jumpscaleX_threebot/issues/563


allow client to remove created tasks with PID 0

Fixes #7357
If a container is restored from a checkpoint that has a configuration error, the task for the restored container is created, but fails to start and is left in the state CREATED with a PID of 0.
In the original issue, the induced error comes from call to task.Start:
ctr: OCI runtime restore failed: namespace {"not" ""} does not exist: unknown

The main task created from a restored container will always be PID 0 before the task is explicitly started
https://github.com/containerd/containerd/blob/e1abaeb3862827d3f507c092d5c1c99e0968a8fc/cmd/ctr/commands/containers/restore.go#L107
I had originally intended to make a change on the go-runc side to prevent PID 0 tasks from even being created, but given that restored tasks are assigned a PID at task.Start, that would have been a breaking change.
If the task fails to start in this case, the task will be left in the CREATED state with PID 0. Before this change, the only way to remove this task was to find the PID of the shim monitoring the task and kill that process. Now, ctr t rm <task> will work on tasks that result in the CREATED state with PID 0.
Signed-off-by: Gavin Inglis<EMAIL_ADDRESS>

/ok-to-test


Invalid rendering on Mojave

Reported by @peterhellberg (Thank you!)
2048 example:

#691 for more detail


Unpin the postgresql client version

Alpine remove old versions of the postgres client from their main apt
index when upgrading, meaning this will frequently break. The postgres
client libraries are backward compatible so there's no need to pin.

We have precedence for not pinning Alpine package versions in abbeaa0af4d2cc39de46580dd17bba1051137ab9

The test fails. We should NOT check the version as commented in  abbeaa0
/home/keymon/.rvm/rubies/ruby-2.2.1/bin/ruby -I/home/keymon/.rvm/gems/ruby-2.2.1<EMAIL_ADDRESS><EMAIL_ADDRESS>--pattern psql/\*\*/\*_spec.rb

psql image
  installs the right version of Alpine
  installs required package (FAILED - 1)
  can run psql
  has the right version (FAILED - 2)

Failures:

  1) psql image installs required package
     Failure/Error:
       expect(command("apk info #{PSQL_PACKAGE}").stdout.strip).to \
         include("#{PSQL_PACKAGE}-#{PSQL_PACKAGE_VERSION}")
       expected "WARNING: Ignoring APKINDEX.d3812b7e.tar.gz: No such file or directory\nWARNING: Ignoring APKINDEX.bb2c5760.tar.gz: No such file or directory\npostgresql-client-9.4.9-r0 description:\nPostgreSQL client\n\npostgresql-client-9.4.9-r0 webpage:\nhttp://www.postgresql.org/\n\npostgresql-client-9.4.9-r0 installed size:\n442368" to include "postgresql-client-9.4.6-r0"
       Diff:
       @@ -1,2 +1,11 @@
       -postgresql-client-9.4.6-r0
       +WARNING: Ignoring APKINDEX.d3812b7e.tar.gz: No such file or directory
       +WARNING: Ignoring APKINDEX.bb2c5760.tar.gz: No such file or directory
       +postgresql-client-9.4.9-r0 description:
       +PostgreSQL client
       +
       +postgresql-client-9.4.9-r0 webpage:
       +http://www.postgresql.org/
       +
       +postgresql-client-9.4.9-r0 installed size:
       +442368

     # ./psql/psql_spec.rb:23:in `block (2 levels) in <top (required)>'

  2) psql image has the right version
     Failure/Error: expect(command("psql --version").stdout.strip).to include(PSQL_VERSION)
       expected "psql (PostgreSQL) 9.4.9" to include "9.4.6"

     # ./psql/psql_spec.rb:32:in `block (2 levels) in <top (required)>'

Finished in 0.6766 seconds (files took 0.33092 seconds to load)
4 examples, 2 failures

Failed examples:

rspec ./psql/psql_spec.rb:22 # psql image installs required package
rspec ./psql/psql_spec.rb:31 # psql image has the right version

/home/keymon/.rvm/rubies/ruby-2.2.1/bin/ruby -I/home/keymon/.rvm/gems/ruby-2.2.1<EMAIL_ADDRESS><EMAIL_ADDRESS>--pattern psql/\*\*/\*_spec.rb failed


[Viewer] Add proxy to database node for json_render request

It was made for query handler already

https://github.com/ydb-platform/ydb/pull/1872

Merge to stable-24-1 please, it's required for basic charts milestone


chore: refactor AceEditorWrapper to functional component

SUMMARY

Converted AceEditorWrapper class component into functional component.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@jinghua-qa @andrey-zayats Can you test it?

@jinghua-qa @andrey-zayats Can you test it?
What area can this pr affect? (where should we run regression tests?)

SQL Lab editor

SQL Lab editor

Okay, I'll take a look

/testenv up

/testenv up

@andrey-zayats If the issue is already in master then we need to create GitHub issue and fix in a separate PR. Right?

@andrey-zayats If the issue is already in master then we need to create GitHub issue and fix in a separate PR. Right?

Yeap, you're right

PR LGTM, qa verified.


docs: generic asyncs state adapter demo

Generic Async State Adaptor and Factory Service to show how to
use two or more separate async state adapters inside a component scope.

Hi @mfp22, I created this demo to show off how I have implemented state adapter im my project, handling async requests to update state store. The contents inside async-generic folder are generic enought to be consunmed by any projects and don't have any external dependencies.
My intent was allow various differents adapters from same type, inside an same scope.
This is usefull when we deal with differents views, attached to different streams,  from a single entity type.

Let me know what you think about it.

Hey @rodzappa , thanks for the PR!
AsyncEntity was next on my roadmap, actually. There are a couple of differences between how I planned it and how you've implemented it here. You've got a couple of better ideas than the ones I had (for example, having entity as a property instead of spreading the object with the async properties like loading; it seems much cleaner). I am torn on naming though, because @ngrx/entity has createEntityAdapter (singular) and that is actually a list of entities. So I would have to break from their convention if I created both asyncEntity and asyncEntities.
Currently I am working on the core documentation, but my next plan was to add an official library for adapters, which would include an async one, and then use it in the shopping demo app. I am doing documentation first because I have to do it eventually anyway, and it might be even faster for me to do than creating demo apps.
The main issue I see in your PR is that usually with async state you already have an observable from an http request, so the more natural thing to do is to convert it into various sources rather than defining new sources entirely. If you look in the core library files, you'll see a getHttpSources function that is not documented yet. As soon as I finish this documentation I will let you know. My goal is today, but it could take a little longer.
After I am done with this documentation I will give a more detailed feedback on this.
As always, thanks for the help and let me know if you want to talk about anything.

@rodzappa I just published the documentation for sources. Might work on the overview or adapters next. https://state-adapt.github.io/concepts/sources

Hi, @mfp22. I've read the documentantion for sources, but I'm busy mantaining a legacy system (What don't we do for living?). I'll still busy on it until the end of the month, althoug I hope will have some time to include getHttpSources in my implementation of state adapt. Until there, if you have some more docs or examples published, I'll appreciate to know that.
PS: Good luck with your home moving!

@rodzappa Thank you. Things have gone pretty smoothly with the house, so I am happy.
I just published all the docs I was working on. I will add some diagrams here and there, or maybe not, but I think I got everything important in the docs now. I added a section at the bottom of Sources, and may have made a couple of other small changes too. https://state-adapt.github.io/concepts/overview
I am making a change to the API. After writing a few dozen adapters, I have started to hate writing selectors as a function with getSelectors. The docs have the updated syntax. The only reason I started it that way was to match the pattern established by @ngrx/entity and Redux's toolkit. However, I have already broken from @ngrx/entity with the state change functions by having the state be the first argument of all of them (it was necessary), so we cannot just use their entity adapter out of the box anyway. And I imagine people will be writing thousands of adapters in the future, so it seems unnecessary to match the API for their one adapter so closely. The reason they had selectors accessed by getSelectors was so developers could pass in the path selector, but in StateAdapt people are providing that with the state path in .init() so I am just attaching the adapter selectors onto that. We do not need it.
I haven't released 0.38.0 yet but it will have this change in it.
My next priorities are

Publish 0.38.0 (with getSelectors => selectors)
improve the design of the docs site a little bit
start making the adapter creator library

After going over the whole API again while writing those docs, I feel like StateAdapt is getting very close to a 1.0.0 release.

@mfp22, I'll try a new approach after reading those docs and will send new PR after finishing.


Implementation is really slow

I love Elevenlabs, but I mostly use it for real-time text-to-speech so latency is critical. This official API is really slow. Seems a lot of improvements can be made?
Here is a class I created earlier this month before the release of the official API and it is considerably faster:

import os
import pygame
import io
import requests
import os
from io import BytesIO
import threading
import queue
import wave

class TextToSpeech:
    def __init__(self, voice_id='EXAVITQu4vr4xnSDxMaL'):
        self.voice_id = voice_id
        self.base_url = f"https://api.elevenlabs.io/v1/text-to-speech/{voice_id}/stream"
        pygame.mixer.init()

    def synthesize(self, text):
        headers = {
            "xi-api-key": os.environ.get("ELEVEN_API_KEY"),
            "Content-Type": "application/json"
        }

        data = {"text": text,}

        response = requests.post(f"{self.base_url}", headers=headers, json=data)

        if response.status_code == 200:
            return BytesIO(response.content)
        else:
            raise Exception(f"Error: {response.status_code} - {response.text}")

    def synthesize_async(self, text, audio_queue):
        try:
            audio = self.synthesize(text)
            audio_queue.put(audio)
        except Exception as e:
            print(f"Error: {e}")

    def play_audio(self, audio):
        pygame.mixer.music.load(audio)
        pygame.mixer.music.play()
        while pygame.mixer.music.get_busy():
            pass

    def speak(self, text):
        num_sentences = len(text.split("."))

        if num_sentences == 1:
            self.play_audio(self.synthesize(text))

        else:
            first_sentence, rest_of_text = text.split(".", 1)
            audio_queue = queue.Queue()
            # Send the rest of the text to be synthesized in a separate thread
            thread = threading.Thread(target=self.synthesize_async, args=(rest_of_text, audio_queue))
            thread.start()
            self.play_audio(self.synthesize(first_sentence))
            self.play_audio(audio_queue.get())
            thread.join()

    def delete_history(self):
        # Need to implement this
        pass

Thanks for the feedback! Can you please send a code to reproduce the slowness? It should be super fast if you do it like this:
audio_stream = generate(text=30*"this is a very long streaming voice hello hello hello", stream=True)
stream(audio_stream)

it will start playing immediately after receiving audio bytes from audio_stream.

Sorry, I should have clarified, I'm not using the streaming functionality as the MVP dependency is causing a lot of issues from my end (on Windows).

This should now be fixed, there was a problem where when you provided a voice_id as string it would query your profile for all voices instead of just using the one provided to query the API.


Support Lower Python Client Versions

Closes https://github.com/G-Research/armada/issues/1996

Codecov Report
Base: 46.09% // Head: 46.07% // Decreases project coverage by -0.02% :warning:

Coverage data is based on head (0e15bd9) compared to base (eba0b72).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1997      +/-   ##
==========================================
- Coverage   46.09%   46.07%   -0.03%
==========================================
  Files         205      205
  Lines       27712    27712
==========================================
- Hits        12773    12767       -6
- Misses      14061    14064       +3
- Partials      878      881       +3

Flag
Coverage Δ

airflow-operator
89.13% <ø> (ø)

armada-server
44.94% <ø> (-0.03%)
:arrow_down:

python-client
93.83% <ø> (ø)

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

Impacted Files
Coverage Δ

internal/common/pgkeyvalue/pgkeyvalue.go
75.28% <0.00%> (-3.38%)
:arrow_down:

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.


Add token count per message

This PR is picking up on #460 and just doing some final cleanup. For some reason I'm not able to push to that PR like I normally can so I needed to create a new pull request. Copying over Daniel's description:
Implement token counting per message by using the returned usage stats from the LLM API backend. Anthropic sends token stats already by default. As for OpenAI, the stream needs to be first configured to return those.

Add migration for new Message attributes input_token_count, output_token_count
Add concern Message::TokenCount for Message model
Implement capturing of returned token count from Anthropic & OpenAI and set Message attributes inside the async API-specific jobs. For OpenAI, configure the stream to return usage stats
Add basic usage dropdown-menu beneath last message that shows the number of input/output tokens when clicked on the "$" sign. This is just a quick demo for the functionality and will likely be changed to something else

This MR is related to #402
@lumpidu All I did on this one was a little code cleanup and move the message display over to the conversation sidebar. All the update logic "just worked" so I didn't need to do anything fancy to get it to display.

@jlvallelonga I finished testing this and cleaning it up a bit. I'm merging it in now.


AFNetworking 1.0 issue with ios8?

I am using AFNetworking1.0 in my project for making HTTP Calls.
and My app works on ios8.
I have created AFHTTPClient class as an Singleton class.....were i have define all my url into this class.
my Problem is,Sometime when were i make an https: post Or put call, I get an error message "Generic error Message".
By getting this message i am not able to complete my request.
But when i close this message ,and again if i try to complete my request,it does complete the request.
So i am not able to find out what's the exact problem?
Can you plese advice?

Hi @Nilesh2985
At this time AF 1.x is only receiving security fixes. If you are looking for help on the library, I'd recommend taking your question to Stackoverflow.
:beers:


Does not appear to do anything

I am looking for a solution for full-screen swipe detection with an effect. The demo for this library looks pretty good, however it the callback and visual effect never seem to happen. I have a gesture detector with an onTap callback around a PlatformScaffold and other assorted widgets inside. I've tried using SwipeEffect as the outter-most widget. I've tried inside the GestureDetector. I've tried removing the GestureDetector. In all situations, I cannot get the SwipEffect callback to fire, and there are no visual effects.
Implementation brief:
SwipeEffect(       direction: direction,       color: direction == TextDirection.rtl           ? Colors.red.withAlpha(70)           : Colors.green.withAlpha(70),       verticalTolerance: 1.0,       startDeltaPx: start,       callbackDeltaRatio: callbackDelta,       callback: () {         print('face');         setState(() {           start += 10;           callbackDelta += 0.1;         });       },       child: GestureDetector(...
I've tried taking the values from both examples. Target is Pixel 4, API 31.

Dear @epistax it seems that your code works fine, please consider trying a large startDeltaPx value, as it determines at which pixel the swipe effect will start.
Also please consider testing on actual device, since it might be hard to swipe on the emulator (especially if the emulator system listens to horizontal swipe)
Widget build(BuildContext context) {     Size size = MediaQuery.of(context).size;     return GestureDetector(       onTap: () {         print('user clicked somewhere');       },       child: SwipeEffect(         callback: () {           print('user swiped');         },         startDeltaPx: size.width * 0.2,         direction: TextDirection.ltr,         child: Scaffold(           appBar: AppBar(             title: Text(widget.title),           ),           body: Center(             child: Column(               mainAxisAlignment: MainAxisAlignment.center,               children: <Widget>[                 const Text(                   'You have pushed the button this many times:',                 ),                 Text(                   '$_counter',                   style: Theme.of(context).textTheme.headline4,                 ),               ],             ),           ),           floatingActionButton: FloatingActionButton(             onPressed: _incrementCounter,             tooltip: 'Increment',             child: const Icon(Icons.add),           ), // This trailing comma makes auto-formatting nicer for build methods.         ),       ),     );   }

Dear @epistax it seems that your code works fine, please consider trying a large startDeltaPx value, as it determines at which pixel the swipe effect will start. Also please consider testing on actual device, since it might be hard to swipe on the emulator (especially if the emulator system listens to horizontal swipe)

Thank you! Yes a very, very large startDeltaPx will work. size.width * 0.6 to 0.8. However now that I see the behavior I don't think it will work for me. I am looking for multiple swipe directions and I don't want to tell users that they have to tap on the right side to swipe left, or the left side to swipe right. I'll check around for other options.
Thanks again.


Wildcard Stateful Config

I have a case where whitelisting specific domains isn't practical. For example services like Zeit deploy development branches with unique xxxxxxxx.company.now.sh domain names. It would be nice if there was a way to define a wildcard.

This was just added two hours ago: https://github.com/laravel/airlock/releases/tag/v1.0.1


fix for #1939, Druid ParseSpec validator

Made for fun. With this clumsy little codes, you can see the result of parsing. For example,
java -cp "lib/*" io.druid.data.input.ParseSpecValidator -i ~/projects/druid/processing/src/test/resources/simple_test_data_record_parser.json -t  ~/projects/druid/processing/src/test/resources/simple_test_data.tsv

@navis can this be consolidated with https://github.com/druid-io/druid/blob/8199ecf1a401e0f9f9402a864786890903492163/services/src/main/java/io/druid/cli/validate/DruidJsonValidator.java so we can have a command line to verify all json specs? would be useful to a lot of people I think

@fjy Done.

:+1:

Changed vars to final. Thanks.

Looks good to me.

Looks great, thanks!


Using system color

Will you make it possible to set so the accent color matches the host os session color (eg the color of Material You on Android >= 12, accent color on macOS etc.
Is there any Web API supporting getting the user accent colors except the old deprecated ones?
This is one of the key features of Material You

At the moment there is not a Web API for this, it'd need a proposal and buy-in from the major browser vendors.
Closing since this is out of our scope, but I'd encourage you to file feature requests in Chromium and/or Android ! That'd be a good place to start the discussion.


Use system wide installed wkhtmltopdf

Is there any chance to use a system wide installed version of wkhtmltopdf?
Because my hoster already deploys an up to date version on theirs servers. I have multiple services which use pdf rendering with wkhtmltopdf and it would be very nice to use the version provided by the system, instead having every app it's own binary

Hi!, i think you could, without any modification in the library, you could do something like this in the startup
services.AddWkhtmltopdf("PATH_TO_SYSTEM_WIDE_VERSION");

Leider wird vom WkhtmlDriver vorgegeben, dass der Pfad zu wkhtmltopdf eine Bezeichnung des Betriebssystems enthält.
            if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows))
            {
                rotativaLocation = Path.Combine(wkhtmlPath, "Windows", "wkhtmltopdf.exe");
            }
            else if (RuntimeInformation.IsOSPlatform(OSPlatform.OSX))
            {
                rotativaLocation = Path.Combine(wkhtmlPath, "Mac", "wkhtmltopdf");
            }
            else
            {
                rotativaLocation = Path.Combine(wkhtmlPath, "Linux", "wkhtmltopdf");
            }

Yeah, i forgot about those lines, the online way i think it could work is with a few modifications

i think im gonna add a parameter in the setup like "usesystemwide" or something like that....and invoke wkhtmltopdf, the only catch is that the path to wkhtmltopdf will have to be added to the env variable PATH

@fpanaccia I'm planning to implement such feature in my fork. We can collaborate on that. I think user should be able to provide full path to executable, also I suggest removing the static path from settings.

Hi @gurustron! yes, i had planned to do that in the weekend, i have little time available as this moment, if you have the time, please, be my guest


Add testcase for SWDEV-321277

The num_threads() clause does not work

add paths to JIRA ticket .. thx


⚠ Fortnite has degraded performance

In 540005d, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1824 ms

Resolved: Fortnite performance has improved in d16bc1e.


[8.x] Replace view facade with helper in RegisterErrorViewPaths

It is currently not possible to to use error views outside of Laravel today. By changing from the view facade to the helper when registering the view paths this allows us to use the views.
If this gets merged this would allow us to add a this as an example to Torch.

👎 the foundation package is strictly not for use outside of the framework.

@Juhlinus I'm a bit confused as well here. I thought Torch was for using the illuminate components separately from the framework. How is a foundation dependency blocking you?

@Juhlinus I'm a bit confused as well here. I thought Torch was for using the illuminate components separately from the framework. How is a foundation dependency blocking you?

Well, if I wish to catch all the HTTP 5XX and 4XX responses like Laravel does and render them then it runs through this trait looking for the error blade views.
I guess one option would be to just make your own renderable response, but I wanted it to mimic Laravel as closely as possible, and PR'ing this change seemed like a non-invasive contribution to accomplish what I wanted.
Below is the code I have for catching the exceptions and rendering them in my own app.
try {
    $response = (new Pipeline($container))
        ->send($request)
        ->through($globalMiddleware)
        ->then(function ($request) use ($router) {
            return $router->dispatch($request);
        });
} catch (\Exception $e) {
    // Report exception
    $container[ExceptionHandler::class]->report($e);

    // Render Exception
    $response = $container[ExceptionHandler::class]->render($request, $e);
}

it runs through this trait looking for the error blade views.

The file you modified isn't a trait, it's a class. And it's not available if you use the illuminate/view dependency separately. The foundation component isn't available as a composer package.

it runs through this trait looking for the error blade views.

The file you modified isn't a trait, it's a class. And it's not available if you use the illuminate/view dependency separately. The foundation component isn't available as a composer package.
Oh yeah, sorry, I misspoke.

The foundation is and has been available as a package on the laravel-zero organisation since version v5.5.28. https://github.com/laravel-zero/foundation

@Juhlinus we don't maintain that package.

@Juhlinus we don't maintain that package.

I am aware.
My interpretation of what you said here:

The foundation component isn't available as a composer package.

Was whether one is able to use the foundation component in their projects or not. Which they are thanks to the wonderful work done by the laravel-zero people.
If you'd explain why this change is not welcomed, or simply just decide to close this PR I'd be fine with it either way. I just wanted to contribute something that would help me personally.

Ah well, I was mostly trying to figure out why you needed to make an edit here. I still don't really understand why Torch needs the foundation component. It seems to me that the real changes that are needed are somewhere to be made in the view component instead so it doesn't invokes stuff from the foundation component?

Ah well, I was mostly trying to figure out why you needed to make an edit here. I still don't really understand why Torch needs the foundation component.

Oh, I did not mean to imply that this is in any way needed in Torch.
I just meant that this would simply be one more thing that we could add as an example to the Torch repo.

It seems to me that the real changes that are needed are somewhere to be made in the view component instead so it doesn't invokes stuff from the foundation component?

Yeah that might be a better idea.

@Juhlinus gotcha. In that case I think it's best that you investigate what would be needed to decouple this part from the foundation component instead and send a new pr?

@Juhlinus gotcha. In that case I think it's best that you investigate what would be needed to decouple this part from the foundation component instead and send a new pr?

Sounds good. :) Thanks for the feedback!
I'll close this then.


New package: ImGuiGLFWBackend v0.1.0

Registering package: ImGuiGLFWBackend
Repository: https://github.com/JuliaImGui/ImGuiGLFWBackend.jl
Created by: @Gnimuc
Version: v0.1.0
Commit: 7f97e783d839c8e19104f8054d3f6ec8521561f5
Reviewed by: @Gnimuc
Reference: https://github.com/JuliaImGui/ImGuiGLFWBackend.jl/commit/7f97e783d839c8e19104f8054d3f6ec8521561f5#commitcomment-51460653

[noblock]
Hi @DilumAluthge, could you help to merge this and #37772 manually? I don't want to wait 3 days.


Upticking cryptography to address 41.0.5 security vulnerability.

resolves #845
Problem
DDOS vulnerability. Timely fix.
Solution
Uptick it past 41.0.5.
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

I noticed you skipped the changelog. I think we have a type for security fixes. Is there a reason to not use that?
How far back do we need to backport this?

oop! Glad you pointed out the changelog. I'm still coming to our current state of affairs process-wise. Added changelog!


Tracker.trackEvent should default URL to the last viewed

Referring to
https://github.com/piwik/piwik-sdk-android/blob/master/piwik-sdk/src/main/java/org/piwik/sdk/Tracker.java#L321  and  https://github.com/piwik/piwik-sdk-android/blob/master/piwik-sdk/src/main/java/org/piwik/sdk/Tracker.java#L686
The events tracked by Tracker.trackEvent() is likely to be defaulted to "http://HOST/", since trackEvent() does not take URL as argument. This is very likely to be wrong, and this behaviour is inconsistent with Javascript and the iOS SDKs.
We should retain the URL of the last screen view, and use it for all subsequent events. This is implemented in iOS SDK (see https://github.com/piwik/piwik-sdk-ios/blob/0ddb5078f5ed4248bcac424639ff89e5b5c30aa7/PiwikTracker/PiwikTracker.m#L485)

I see what issue you mean.
Isn't that an issue with all methods except maybe the trackScreenView ones?
We can't use the URL of the last screen as the Tracker instance might be used by multiple threads.
I would suggest to change the method(s) to require an url argument or take a  (optional?) TrackMe and then in another PR consider adding some helper class for this.

@dotsbb You did not start on this yet, right?

I didn't start yet. Feel free to go ahead with this one.

Initial issue fixed by #93
Further SDK API changes in #94 to make this easier for devs in the future.

Thank you!

I'm not professionell, only user.
Ich wrote a short Info few month ago on my notice App. May it is important for better working by  back up this App :
---------------- SD Maid ------------------
System Toybox/Burybox install.
Question: Please help me which file i need to open for back up my Android 6.0.1 Samsung Galaxy S6 Edge.
Thank you so much withe nice greets  ...
Mike


Using match with app store and enterprise account when App ID / Bundle ID remains the same

New Issue Checklist

[ x] Updated fastlane to the latest version
[x ] I read the Contribution Guidelines
[x ] I read docs.fastlane.tools
[x ] I searched for existing GitHub issues

Environment
Please run fastlane env and copy the output below. This will help us help you :+1:
If you used --capture_output option please remove this block - as it is already included there.
✅ fastlane environment ✅
Stack

Key
Value

OS
10.12.6

Ruby
2.4.0

Bundler?
false

Git
git version 2.13.5 (Apple Git-94)

Installation Source
~/.rbenv/versions/2.4.0/bin/fastlane

Host
Mac OS X 10.12.6 (16G29)

Ruby Lib Dir
~/.rbenv/versions/2.4.0/lib

OpenSSL Version
OpenSSL 1.0.2k  26 Jan 2017

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
9.0

System Locale

Variable
Value

LANG
en_CA.UTF-8
✅

LC_ALL

LANGUAGE

fastlane files:
No Fastfile found
No Appfile found
fastlane gems

Gem
Version
Update-Status

fastlane
2.62.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.1.0

slack-notifier
1.5.1

CFPropertyList
2.3.5

multipart-post
2.0.0

word_wrap
1.0.0

public_suffix
2.0.5

tty-screen
0.5.0

babosa
1.0.2

colored
1.2

highline
1.7.8

commander-fastlane
4.4.5

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
2.1.0

gh_inspector
1.0.3

mini_magick
4.5.1

multi_json
1.12.1

multi_xml
0.6.0

rubyzip
1.2.1

security
0.1.3

xcpretty-travis-formatter
0.0.4

faraday_middleware
<IP_ADDRESS>

declarative
0.0.9

declarative-option
0.1.0

representable
3.0.4

addressable
2.5.1

mime-types-data
3.2016.0521

mime-types
3.1

jwt
1.5.6

memoist
0.15.0

os
0.9.6

signet
0.7.3

googleauth
0.5.1

httpclient
2.8.3

google-api-client
0.13.4

json
2.1.0

io-console
0.4.6

excon
0.57.0

openssl
2.0.2

plist
3.3.0

faraday
0.12.1

unf
0.1.4

domain_name
0.5.20170404

claide
1.0.2

colored2
3.1.2

nanaimo
0.2.3

xcodeproj
1.5.1

bundler
1.15.1

terminal-table
1.8.0

unicode-display_width
1.2.1

generated on: 2017-10-17

Issue Description
Let us say that I have access to an app store developer account, through which apps are published to the app store. I also access to an enterprise account, through which apps are distributed internally.
Let us say I have an app with an App ID / Bundle ID of com.myapp.myapp. This is setup in the app store account.
I would also want to distribute the app, via the enterprise account using the same App ID / Bundle ID. I know that I cannot create the App ID on the enterprise account as it's unique, and already exists on the app store account. However, I can create a wildcard App ID, and that can be used to generate a provisioning profile and code sign the app without changing the bundle ID.
So what's the problem? Well, I'd quite like to use Match, but it seems that I likely won't be able to do this, as one of the first things I'd need to do after setting up the repo, is to specify an app id for the enterprise setup, and, as I've found with sigh, it's going to fail as it won't be able to find or create the app id on the enterprise account, as it exists in the app store account.
Does anyone have any ideas of how one could get match to work with the setup above, or is it simply the case I'd need to create new App ID's in the enterprise account?
As to why I've not done that already, I believe that there are third party items that are configured with the bundle ID, and it may not be easily possible to add additional bundle ID's to those items.

@sharkusmarkus Unfortunately, I think it will be difficult to get match to support this configuration. Using a wildcard app id to allow an app store and enterprise build with the same bundle identifier isn't really recommended by Apple and unfortunately it breaks a lot of the assumptions made by match. Your best bet is probably creating new App IDs for the enterprise account.  Sorry I don't have better news!

Thanks for the reply. I thought I would ask, but did presume that it'd be the case that new ID's would be needed, which should be ok :)


fix: Don't set localhost on <IP_ADDRESS>

related: #3449, #2180

Codecov Report

Merging #3653 into dev will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##              dev    #3653   +/-   ##
=======================================
  Coverage   99.19%   99.19%
=======================================
  Files          18       18
  Lines        1123     1123
  Branches      302      302
=======================================
  Hits         1114     1114
  Misses          9        9

Impacted Files
Coverage Δ

lib/core/nuxt.js
98.82% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6aaf839...7e3466a. Read the comment docs.


php artisan migrate issue

So looking for laravel 5.2 multi-auth example. So stoked I found your example on github. Just having problems with initial migrate. Total newbie.
What I have done: homestead works well, been building small apps from homestead to tryout laravel. Migrations, db:seed etc works 100%. No problem.
Also tried out other laravel starters from github - all work well.
Just this one after trying to migrate generates the below warnings. As I've said, Newbie, could be just a small thing I'm not seeing. SOS anyone. Thanks!
vagrant@homestead:~/Code/easy1$ php artisan migrate
PHP Warning:  require(/home/vagrant/Code/easy1/bootstrap/../vendor/autoload.php): failed to open stream: No such file or directory in /home/vagrant/Code/easy1/bootstrap/autoload.php on line 17
PHP Stack trace:
PHP   1. {main}() /home/vagrant/Code/easy1/artisan:0
PHP   2. require() /home/vagrant/Code/easy1/artisan:16
Warning: require(/home/vagrant/Code/easy1/bootstrap/../vendor/autoload.php): failed to open stream: No such file or directory in /home/vagrant/Code/easy1/bootstrap/autoload.php on line 17
Call Stack:
0.0004     226032   1. {main}() /home/vagrant/Code/easy1/artisan:0
0.0020     228648   2. require('/home/vagrant/Code/easy1/bootstrap/autoload.php') /home/vagrant/Code/easy1/artisan:16
PHP Fatal error:  require(): Failed opening required '/home/vagrant/Code/easy1/bootstrap/../vendor/autoload.php' (include_path='.:/usr/share/php:/usr/share/pear') in /home/vagrant/Code/easy1/bootstrap/autoload.php on line 17
PHP Stack trace:
PHP   1. {main}() /home/vagrant/Code/easy1/artisan:0
PHP   2. require() /home/vagrant/Code/easy1/artisan:16
Fatal error: require(): Failed opening required '/home/vagrant/Code/easy1/bootstrap/../vendor/autoload.php' (include_path='.:/usr/share/php:/usr/share/pear') in /home/vagrant/Code/easy1/bootstrap/autoload.php on line 17
Call Stack:
0.0004     226032   1. {main}() /home/vagrant/Code/easy1/artisan:0
0.0020     228648   2. require('/home/vagrant/Code/easy1/bootstrap/autoload.php') /home/vagrant/Code/easy1/artisan:16

This is unrelated to the package. Your composer autoload file simply doesn't exist. Run composer install && composer dump-autoload to fix this.
Additionally, you shouldn't use this package. Starting from version 5.2 Laravel has built-in support for multi-auth. Check this blog post for some information


Virtual IPs UI: New VIP form lacks description field for IP Alias, Proxy ARP, and Other; CARP alone shows one.

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
When adding a new IP Alias in the Virtual IP UI (/ui/interfaces/vip) the form field for entering a description is not present.  The form field is shown when the Virtual IP mode is set to CARP.
To Reproduce
Steps to reproduce the behavior:

Go to Interfaces -> Virtual IPs -> Settings
Click the button to add a new virtual IP
Cycle through the different Modes to see a Description form field in the CARP mode and no other.

Expected behavior
It would be nice to be able to set a description on the Virtual IP regardless of the Mode.
Describe alternatives you considered
In this case the IP Alias was saved as a CARP VIP with the target description, then modified to be the expected IP Alias.  The description persisted and now shows in the list of VIPs.
Screenshots

Relevant log files
n/a
Additional context
n/a
Environment
OPNsense 23.1_6 (amd64, openssl)
FreeBSD 13.1-RELEASE-p5

easy to have missed, but was fixed recently in https://github.com/opnsense/core/pull/6304. should be in the next version

Thanks!  Sorry to have missed it.  I was looking through Issues and neglected to search through the existing PRs.

@cyberpuffin-digital no problem, easy to miss this one :)


[HOLD for payment 2024-03-22] [HelpDot] Margins for Info box

Info box looks cramped because it doens't have margin below it.

just a tracking issue

@dubielzyk-expensify please see this comment

Replied 👍

Anything needed from me on this?

no we can close the issue.


add time expression

unlike other shells (bash, zsh, fish), time is an expression.
as a result, measure execution time of arbitrary expressions.
time {
  do-something 1
  do-something 2
}

time expression cannot measure system or user time due to not use wait4


Print using PDF format.

Hi @januslo, thank you for this amazing plugin.
One question in mind is how do I print PDF file using BluetoothTscPrinter functionality ?
Thank you again.

Not using pdf anymore

is Library supports pdf document print.