How to Automatically Include Styles?

hi there, apologies if this is a bit of a noob question, but how does one automatically have styles included in the built html file? i can't seem to get them included with the following config:
module.exports = {
  entry: [
    'webpack/hot/dev-server',
    './src/app/main.js',
  ],

  output: {
    path: './dist/',
    filename: 'app.js',
    hash: true
  },

  module: {
    loaders: [
      { test: /\.js$/, loader: 'babel-loader', exclude: /node_modules/ },
      { test: /\.js$/, loader: 'jsx-loader' }, // required for react jsx
      { test: /\.jsx$/, loader: 'jsx-loader?insertPragma=React.DOM' },
      { test: /\.(woff|ttf|eot|svg)$/, loader: 'file-loader' },
      { test: /\.css$/, loader: 'style!css!autoprefixer' },
      { test: /\.sass$/, loader: 'style!css!autoprefixer!sass?indentedSyntax' }
    ]
  },

  resolve: {
    root: [
      path.resolve(__dirname, './src'),
      path.resolve(__dirname, './node_modules')
    ]
  },

  plugins: [
    new HtmlWebpackPlugin(),
    new webpack.HotModuleReplacementPlugin(),
    new webpack.NoErrorsPlugin(),
  ],
};

and this is the structure of my src/app/ directory which contains my entry and my stylesheets:
src/app/
├── index.html
├── main.js
├── static
└── styles
    └── app.sass

do i need to reference that stylesheet somewhere beforehand?

Do you require your app.sass from your main.js?

turns out i was missing style-loader in my deps, sorry for the trouble!


Support for Studio 5.10 and Nuget 2.8.7

We cannot use this addin with Studio 5.10 b/c is uses Nuget 2.8.7.
can you update this to support it?

Version 0.10.1 of the NuGet extensions addin should work with Xamarin Studio 5.10. It should be on the [alpha addin repository[(https://github.com/mrward/monodevelop-nuget-extensions#installation).
Alternatively you can download the .mpack and install it using the Install  from file button on the Addin Manager dialog.


[Question] IOverrideFilter analog

MVC5 used to have IOverrideFilter which allowed for authorization overrides on actions. Is there an analog in Core?

Seems like you would ask this on the mvc repo


Refresh loop on latest deployment

When opening https://dex-explorer.testnet.penumbra.zone/lp/plpid1d79ghpr2v6qlaytcky8m57sxjdea99cccxlreca245nnr3qkym6s07yu4c on the current version, I see a refresh loop where the page content flashes several times. However, after loading the pages, subsequent refreshes seem to work, maybe it's a caching issue?

Not reproducible


Inline video audio continues to play after the video is no longer visible

See (has audio) https://streamable.com/ur696 - the audio stops after leaving the comments view.
Also, after returning from the comments view, the red audio icon is not visible when replaying the video: https://i.imgur.com/Lv7BASa.mp4 The audio plays when you click the video to show it fullscreen but remains muted when you dismiss the fullscreen view: (has audio) https://streamable.com/ow86h
I think it would be helpful to have an option to have inline videos play with audio unmuted. Sometimes I'll play a video inline and not realize it had audio until the end of the video. Or maybe there could be a quick animation that alerts the user that the inline video has audio? Maybe an animation like a larger version or the red audio icon starting in the middle of the video view and shrinking to the corner.

Hmm, I am not able to reproduce this. I think what might be happening in your case is the video is playing in the CommentVC which is actually still stored in memory (the last page is stored so you can return if you closed accidentally), but I do destroy the AVPlayer in the viewWillDisappear block. The LinkCellView there is actually completely recreated when returning to the subredditVC so you should not be seeing different behavior when returning (as it's essentially restarting the cell). It's possible that stored AVPlayer in the comment vc is taking the audio focus?

That's weird, maybe the issue is iOS dependent? I notice this issue on an iPhone 6 Plus running iOS 10.3.3 (14G60).
Clean install: https://streamable.com/60ogy
* Note that you can hear the audio doubled after the unmute button is pressed in the comments view.
Clean install with "Play videos on tap" set: https://streamable.com/6ew2x

If the sound is doubled, that means there are two AVPlayer instances running at one time. It must be the one in comments not getting killed somehow 🤔

Actually, I think pausing/muting the video when opening comments from the LinkCellView would fix this issue

I think I fixed it in my latest commit, if not please re-open this!

Okay, I just install the latest version and the main audio issue is fixed. There are still some cases where the unmute button isn't present. I'll see if I reproduce the issue.


Fix error about translation

Fix https://github.com/kiali/openshift-servicemesh-plugin/issues/34
Before

After

At least for init this and remove the error. We don't use i18n in kiali so It make no sense to have this configuration

I see docker-endpoint.sh in that "Before" log - that is old - we don't have that anymore - the new image using the new base image no longer uses that. You should rebuild the server with the latest Dockerfile.


Image change scanning jumps the cursor position

In the latest version of the plugin (I'm on 0.7.2) the new option for image change scanning (default period 3s) keeps jumping the cursor position, but only in certain notes. I haven't yet found what is unique about these notes, but if I set the period for change scanning to 0, the problem goes away. It is not always the start, but it always jumps up...
I'll post more if I find the common denominator for the notes causing this issue!
OS: linux
Joplin ver: 2.4.7
Plugin ver: 0.7.2

Note that another side-effect of this is that the spell-checker seems to re-refresh at this period - i.e. the red underlining of incorrect words vanishes and re-appears with the period set fore the image scanning.

Thanks for reporting this @msbentley. In hindsight that scanning change was a bit half backed...
I've pushed up a fix now, please give it a test when you get the chance and let me know if it actually fixed your issue (it should have!).

Thanks @CalebJohn I just tested the latest and the jumping around seems to have stopped - many thanks! The spell-checker refresh seems still to happen at the polled interval, so for now I've set that to a large number (since zero seems to stop it working).

Setting it to zero does disable the periodic checking, but it will still check for changes after any change in the document. You might find that is enough.
That's strange, I can't duplicate it anymore even in my stress testing notes. I'll keep my eyes open for it and we'll see if it happens for anyone else.

Thanks - I updated my second machine this morning and everything seems OK here, so perhaps it was something odd with my other install. So all good now, many thanks!


WIP - feat: add compressed file support for ORCRecordReader

This is WIP on https://github.com/apache/pinot/issues/9847
The aim of this change is to support gz compressed files for ORCRecordReader and ParquetAvroRecordReader in order to achieve the feature parity with other types like csv/json format.

hi @snleee,
I've made some adjustments. Could you please take a look when you have time?
While I was implementing the solution was a bit concerned about the code duplication, but not sure if extractingunpackIfRequired into, f.e. RecordReaderUtils would make the code clear.  Please let me know what do you think about it.
Also regarding theisGZippedFile  utility method as far as I understood we would like to support only gz and not archive files in general, right?

Regards, Eugene

@etolbakov
Error:      src/test/java/org/apache/pinot/plugin/inputformat/orc/ORCRecordReaderTest.java
Error:          @@ -26,7 +26,6 @@
Error:           import·java.util.List;
Error:           import·java.util.Map;
Error:           import·java.util.zip.GZIPOutputStream;
Error:          -
Error:           import·org.apache.hadoop.conf.Configuration;
Error:           import·org.apache.hadoop.fs.Path;
Error:           import·org.apache.hadoop.hive.ql.exec.vector.BytesColumnVector;
Error:  Run 'mvn spotless:apply' to fix these violations.

Let's try to run mvn spotless:apply to let maven clean up the formatting.


[BUG]

Plug-in won't open using Ableton 10 + 11 or Max/Msp 8.
Says plug-in cannot be opened in both applications.
System Info

What machine/OS you're using: Macbook Pro 10.13.6 OS High Sierra
Your DAW and version: Ableton 10+11 & Max/MSP 8

Would love to use the plug-in. I hope it can be fixed ASAP

Hi, it seems likely this is an issue with the older OS, as in #5
We just released a new version that should work with older versions of macOS (https://g.co/magenta/ddsp-vst-mac) can you try it out and see if it works for you? Thanks!


How to add toolbar items?

Do you have an example of how to add toolbarItems?
http://nhnent.github.io/tui.editor/api/latest/ToastUIEditor.html

Same question...

Please add some example to add toolbarItems.

same

Hey guys, I just found few example
Example 1:
https://github.com/nhnent/tui.editor/pull/124
Example 2:
this.editor.exec('Bold')

https://github.com/nhnent/tui.editor/issues/240
http://nhnent.github.io/tui.editor/api/latest/CommandManager.html

Certainly helps! Many thanks! 😄  @1c7

Thanks @1c7
Look good to close this issue now.


togeojson doesn't work with Electron 17.1.0

Hi, I'm trying to use togeojson on an Electron app using npm i @mapbox/togeojson --save --build-from-source with env variables set for Electron build. However it isn't finding togeojson:
Uncaught Error: Cannot find module 'togeojson'
If someone could help that would be great. Thanks.

I needed to do require('@mapbox/togeojson')


Error: TS2322: Type '{ codec: string; width: number; height: number; }' is not assignable to type 'undefined'.

The expected type comes from property 'video' which is declared here on type 'MuxerOptions<ArrayBufferTarget, undefined, undefined>'

Because now the default is undefined with no way of changing it:
https://github.com/Vanilagy/mp4-muxer/blob/main/src/muxer.ts#L33-L36

Can you show me your code that errors?

This is only an error in 3.0.0
      const {Muxer, ArrayBufferTarget} = await import('mp4-muxer');
      this.videoType = 'mp4';
      this.muxer = new Muxer({
        target: new ArrayBufferTarget(),
        video: {
          codec: 'avc',
          width: image.width,
          height: image.height,
        },
      });

Ah yes, I incorrectly used generics in the declaration file. I released a fix in v3.0.1, can you please verify?

Given your fix, it doesn't work as is.
It seems like I need to modify my definition to:
new Muxer<ArrayBufferTarget, VideoOptions>({

But I can't import it

No you shouldn't have to update your definition, the generics are inferred from what you pass to the Muxer. What is the error now? From the above example, it seems like you're not passing fastStart.

This works for me in v3.0.1. Note how the second generic is properly inferred from what gets passed:

with fastStart, there's no need for explicit typing. thanks!


Complete some artist routes

Completed routes:

GET /artist/details/:artist_id
PUT /artist/profile/:artist_id
POST /artist/event/:artist_id

 Controller and Model Object Artists


[Snyk] Fix for 2 vulnerabilities

Snyk has created this PR to fix 2 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

package.json
package-lock.json

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Heap-based Buffer Overflow SNYK-JS-SHARP-5922108
  909

Regular Expression Denial of Service (ReDoS) SNYK-JS-SEMVER-3247795
  696

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Regular Expression Denial of Service (ReDoS)

👷 Deploy Preview for silly-twilight-a128dd processing.

Name
Link

🔨 Latest commit
f4951ebba06d8bf19718b55afe34453ecd032675

🔍 Latest deploy log
https://app.netlify.com/sites/silly-twilight-a128dd/deploys/66dab243a4a8880008bfd833


Generalize StateTransitionEvent<S> to allow identity transitions

Objective
This PR addresses one of the issues from discord state discussion.
Same-state transitions can be desirable, so there should exist a hook for them.
Fixes https://github.com/bevyengine/bevy/issues/9130.
Solution

Allow StateTransitionEvent<S> to contain identity transitions.
Ignore identity transitions at schedule running level (OnExit, OnTransition,  OnEnter).
Propagate identity transitions through SubStates and ComputedStates.
Add example about registering custom transition schedules.

Changelog

StateTransitionEvent<S> can be emitted with same exited and entered state.

Just waiting on #13626, then this can proceed and I'll need to draft one more for ComputedStates extension

I just need to add a test for ComputedStates propagation, but the bulk of the work is done and ready for review.

Thank you to everyone involved with the authoring or reviewing of this PR! This work is relatively important and needs release notes! Head over to https://github.com/bevyengine/bevy-website/issues/1348 if you'd like to help out.


feat: implement limited context menu and readonly view (no message input) (Issue #389, #431)

Issue #389
Issue #431

/deploy-review

/deploy-review


bugfix | change name of publication task in build.gradle.kts

Why?
After update to Kotlin DSL artifacts of last two versions were not published to maven central.
Run ./gradlew publishToSonatype closeAndReleaseSonatypeStagingRepository
Downloading https://services.gradle.org/distributions/gradle-8.1.1-bin.zip
...........10%............20%............30%............40%............50%............60%............70%............80%...........90%............100%

Welcome to Gradle 8.1.1!

Here are the highlights of this release:
 - Stable configuration cache
 - Experimental Kotlin DSL assignment syntax
 - Building with Java 20

For more details see https://docs.gradle.org/8.1.1/release-notes.html

Starting a Gradle Daemon (subsequent builds will be faster)

SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
> Configure project :
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.

FAILURE: Build failed with an exception.

* Where:
Build file '/home/runner/work/tradukisto/tradukisto/build.gradle.kts' line: [13](https://github.com/allegro/tradukisto/actions/runs/6088432033/job/16535361070#step:4:14)1

* What went wrong:
Task with name 'publishing.publications' not found in root project 'tradukisto'.

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org/

BUILD FAILED in 49s
Error: Process completed with exit code 1.

https://github.com/allegro/tradukisto/actions/runs/6088432033

Codecov Report

Merging #114 (ab8be1d) into master (3cd5170) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.
@@            Coverage Diff            @@
##             master     #114   +/-   ##
=========================================
  Coverage     99.19%   99.19%
  Complexity      425      425
=========================================
  Files            67       67
  Lines          1736     1736
  Branches         50       50
=========================================
  Hits           1722     1722
  Misses            7        7
  Partials          7        7


Problem running edd without a defined gap

Hello,
Edd runs to completion when I specify a gap penalty score, but hangs up with the following error when I don't specify a gap penalty:
Traceback (most recent call last):
File "/usr/local/bin/edd", line 146, in
main(args, config)
File "/usr/local/bin/edd", line 69, in main
df = loader.get_df(args.unalignable_regions.name)
File "/usr/local/lib/python2.7/dist-packages/eddlib/experiment.py", line 201, in get_df
mc_trials=100, pval_lim=0.05)
File "/usr/local/lib/python2.7/dist-packages/eddlib/estimate.py", line 89, in instantiate
util.save_bin_score_file(binscore_df, bedgraph_path)
File "/usr/local/lib/python2.7/dist-packages/eddlib/util.py", line 10, in save_bin_score_file
df['chrom start end score'.split()].sort(['chrom', 'start']).to_csv(
File "/usr/local/lib/python2.7/dist-packages/pandas/core/generic.py", line 3081, in getattr
return object.getattribute(self, name)
AttributeError: 'DataFrame' object has no attribute 'sort'
All dependencies were up-to-date and installed prior to installing Edd. Any ideas on what to do?
Thanks for your time!

Thank you for notifying me. The issue was a non backwards compatible change in the Pandas library EDD depends on. I have updated EDD and version 1.1.18 works with the most recent pandas version.
run pip install -U edd to upgrade

Problem solved! Thanks for the speedy response.


Port to OS X

I successfully ported the code to my Mac yesterday.  I'd like to build a nice toolbar interface and also execute it with the voice accessibility features.  Has anyone every merged apple script and python or objective-c/swift and python?

Since this is 5 months old, I'm closing this. Feel free to reopen or open a PR with a link in README if you have anything active on this.

There is actually #128 open with info, so no need to mourn over this empty issue :-)


File permissions issue if generation fails

Prior to #313, artman failed mid-generation when running in Docker mode (protoc succeeded, but toolkit failed). The artman-genfiles output directory contained files owned by root, making them non-editable by the user. It should be investigated whether the file owner/permission change fails to happen if an exception occurs in the middle of generation, and if so, it should be fixed so that the user is given permissions on output even when generation fails.

Fixed in https://github.com/googleapis/artman/pull/322


[GHSA-xqr8-7jwr-rhp7] Removal of e-Tugra root certificate

Updates

Affected products

Comments
https://play.google.com/store/apps/details?id=cn.tydic.ethiopay&hl=en&gl=US
@Hackedworld36
Application name - Telebirr
Procecer name - Cn.tydic.ethiopay
Uid - 10441
Installer - com.google.android.packageinstaller
Application source - unknown
Target SDK - 33
Target Version - Android 13 Tiramisu
Minimal SDK - 21
Mini version - Android 5.0 Lollipop
Apk path - /data/app/~~T5oFsfyXCNej52z07XvZ5g==/cn.tydic.ethiopay-m1JmF7VEx4SNsEOYMJxunw==/base.apk
Data path - /data/user/0/cn.tydic.ethiopay
Install Location - Internal storage only

Hi there @alex! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our highly-trained Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

This is the 3rd time I've been pinged for a change that does nothing but touch the modified date.
I'm not sure if this is a bug, or a very bizzare form of abuse.

Hi there @alex! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our highly-trained Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

This appears to be purely abusive/spam. Can someone please report/ban this user.


tools: support ids and delete for component command

What problem does this PR solve?
Currently, it's hard for users to know about component ID when using the component command to control configuration.
What is changed and how it works?
This PR adds two more subcommands named ids and delete. ids will show all component IDs with a given component; delete will remove a component ID from config manager which need to be used when offline a store.
Usage:
pd-ctl component ids <component>

>> component ids tikv
[
  "<IP_ADDRESS>:20160",
  "<IP_ADDRESS>:20161"
]

pd-ctl component delete <component ID>

>> component delete <IP_ADDRESS>:20161
[
  "<IP_ADDRESS>:20160"
]

Check List
Tests

Manual test

/merge


fixed bug with "./db -q add" or "./db -q add"

I searched the bug in the code. If we used:
./db -q add or ./db -q add, we would have a Segmentation fault.
The reason of this is that the argument const char *str of function make_item() can be a NULL pointer. In make_item() function you used to call strdup(str). If the argument of strdup() function is a NULL pointer, you will have Segmentation fault.
To fix this I made next changes:

I added new field to enum main_err : EM_NODATA and description for it to static char *errmsg[]  which means that Error: Not avaiable data about person.
make_item() now takes a pointer to struct dbitem as second argument, instead of returning it
make_item() now returns int value, in which means error code(enum main_err). This error code can be EM_OK(0) or EM_NODATA(12)
after call make_item() function I added checking on return value of this function.

I finded one more potentially dangerous place in code.
In function **listdir() **there is a line
_int n = scandir(dirname, &namelist, &onlyfiles_filter, alphasort);
Function scandir() returns -1 if something goes wrong(no enough memory, there isn`t a folder with dirname name and in any more situations).
Ok, so n can be -1.
Next if() block in this situation throw us to the bottom of the function via goto().
And here this danger is situated, in
while (n--)
free(namelist[n]);

This loop will be infinite
free(namelist[n]); will result in an Segmentation fault.

I fixed this, added checking condition that n>0.


[Mobile / Desktop ] Implement SfCollectedProduct #103

Related issue
#103

Scope of work

Mobile and desktop views for the component;
API and reference;

mobile

desktop

Checklist

[x] I followed composition rules for my component
[x] I tested the component in most common device sizes (can be tested in Storybook from viewport addon in top menu)

Hey! Can you please adjust ti as much as you can to the designs please? Especially desktop version.
Also i think it needs specified max width same as SfPoductCard

@filrak just a note, the input is different because it uses SfCounter that's being edited here https://github.com/DivanteLtd/storefront-ui/pull/300, so after the PR merged this SfCollectedProduct component should be updated, right?

@leomp12 yeah I see, that one is not a problem. I was talking more about icon sizes, font sizes, padding etc which seem to be far from the ones on the design
@talissonf please lmk if you could do this this week, otherwise I can help by myself to speed things up ;)

@talissonf should I take this one?

@filrak sorry, I'm doing your proposals now

Things adjusted

max-width is too small - using figma CartPreview/ProductOnHover width
hover effect is not matching hover effect on designs - ok
SfCircleButton is lacking it's hover effect - ok
'x' shouldn't be visible without being hovered (desktop only) - ok
font sizes are wrong and in pixels (should be in rems, take a look at SfProductCard) ok
There is no x on mobile - ok

To adjust

image size - i got confused with unit of measure in figma
Use numeric input instead of Counter (check SfAddToCart) - i tried to use but ran away from figma design

Hey! It's much better now@ Few more tiny things I noticed:

on desktop x should be visible only on hover (this was part of hover not matching designs)
on mobile there is different x
on mobile there is not box-shadow

@talissonf this component looks good. I cleared and adjusted a few things.
@filrak I have a problem with mobile dots, this dots run something like mobile selector but for actions. Can we discuss this behaviour? Now I add just dot's SfIcon

to adjust:

[ ] qty input, basic HTML  is best by SfImput, because SfInput have useless logic
[ ] update story, we cant adjusted it to design :) because wy can't use media queries inside story

@filrak done
design:

web:

design:

web:

@przspa can we make those inputs smaller since there is no arrow? Also number is not fully centred

thanks @talissonf! :)


[DISCUSSION] Check open PRs and tell if someone has already suggested an icon or linked a component

About this discussion
The more open PRs there are, the more time it will take to check which icons have already been suggested.
Theoretically you can check both by svg name and by component names at once. After a person has created a PRs, he can be told that the icon is already there or the component has already been linked. Give a link to the PRs where this happened.
But the actual problem is that there is no common table where contributors can say that they have taken a request into work. This is also superimposed on the fact that nobody updates the table of requests — now at least half of the rows there are irrelevant. The requests themselves consist of clipped component names. Such names are useless for apps that can be installed only on specific phone brands and you always have to search full component names for each request.
Therefore, if we talk about a normal process, then we have a public table, which receives whole components from those who want new icons. Contributors in such a table could label with a date the requests they are working on. Other contributors would see the overall situation, so that they don't waste their time or the reviewer's time on the same requests.
Once a contributor has created a PR, you could mark the requests in the table as ready to review. And after the PR is accepted, you can remove completed requests from the table. But even if it takes a lot of effort to do this automation, leaving the manual ability to mark rows as "in work" and "done" via the comment feature would help to save time. I assume that it is possible to solve this problem in other ways.

GitHub has Projects, you would need an admin to set them up though

GitHub has Projects, you would need an admin to set them up though

Is there an easy way to write responses from the requests form to GitHub Projects?
My guess is that there isn't. And it's easy to change the current requests table for this task.

Given each request is a separate issue, you can comment on them that way
If each one is a separate issue you could also just use labels or milestones to track them

Given each request is a separate issue, you can comment on them that way If each one is a separate issue you could also just use labels or milestones to track them

A lot of users write requests in batches. It looks like this:
nz.co.kiwibank.mobile
nz.co.bnz.droidbanking
com.hjiangsu.thunder
app.symfonik.music.player
nz.co.telecom.smartphone.android
nz.co.partpay.mobileapp
com.github.olga_yakovleva.rhvoice.android

So, if I understand you correctly, in this case it will be necessary to compare requests between each other and see who has the same components to keep information up to date. Or we will have to put the task of sending the request one by one on the users. Or we'll have to force contributors to make requests based on a particular user's list, rather than in the desired order. Either way you lose flexibility for either users or contributors.
In addition, you can tell from the table which requests are the most popular. This allows you to set priorities in a more meaningful way. I'm not sure if it will be possible to see the popularity of requests through Projects without the need for additional actions on the part of users.

Users can :+1: issues pretty easily - but yeah it would still be hard to open each issue separately

Fair, but even if you used a sheet wouldn't someone in the team still have to see if each icon was requested already or not

if you used a sheet wouldn't someone in the team still have to see if each icon was requested already or not

Are you saying that seeing it would be more convenient here than opening a sheet? If I understand you correctly, it would be great if GitHub supported tables.
With development the problem would have a very different solution, I wrote about one option in a past issue.

Bit confused right now - are you proposing a publicly editable table, a privately editable table, or something else?
(Also Github Projects has a table view but :shrug:)

are you proposing a publicly editable table, a privately editable table, or something else?

Right now I'm proposing to do something like that.  "RAW RQ" is actually a requests table, in which a bunch of components are separated using formulas so that there is one component per row. "Popular RQ" is a pivot table that collects components by popularity.
Seems like you could do the same thing with Lawnicons table. And in a pivot table you can allow comments, so that contributors can mark which icons are in progress and which ones should be removed.

https://docs.google.com/spreadsheets/d/10_o0DjhAGfP8ToDDNQ8M7eCZKtbzXLhIvxuF1XZEql0/edit?usp=sharing
Let me know if Sheets didn't let you copy the Apps Script so I need to send it to you
Also are you on Discord? I couldn't find you in the Lawnchair Discord, it would be nice to be able to quickly contact you there.


optimizeImage returns null

How can we use optimizeImage() with an existing file without using an "uploaded" one? At this time it returns null.
Otherwise it will be a good thing to use optimizeImage with an encoded image.

Hi,
encoded image as a filepath or binary data?
For filepath you also use this:
https://github.com/approached/laravel-image-optimizer/blob/master/src/ImageOptimizer.php#L18

At this time I am using filepath, but it returns null.
A good idea is to use encoded image as binary data, so we have not to save temporarily the image, optimize it and then delete.
My process is this:

Upload image through $request->file()
Resize image
Here I need to optimize it, but from step 2 I have only the possibility to retrieve the encoded value of the file or save it.
Upload it on S3

@DaniloPolani
A good idea is to use encoded image as binary data...
Feel free for a pull request 👍
But be know, on php file uploading process, the file will automatically save under /tmp/AD23SAD34

Sure, I know it.
Anyway, how to use optimizeImage()? This was the original question

Similar like this:
https://github.com/approached/laravel-image-optimizer/blob/master/README.md#demo
Just
$imageOptimizer-> optimizeImage('/home/little/boy.jpg');

Ok, but if the method returns null, after $imageOptimizer->optimizeImage() what I have to do? Does it overwrite the image of the given filepath?

Yes, it just overwrite.

@approached we can make a middleware that check for upload file if type of image and optimize it on the fly.
@DaniloPolani for already uploaded images, u can list all files in a path and optimize them through loop.

@ctf0 good idea :-)

@approached here is the middleware https://gist.github.com/ctf0/5beee05601681309dbb9dee80ec6f84a , i didnt have a chance to test it yet so if you can do it from ur part i would appreciate it.

It look's great 👍

@approached PR sent.

PR is merged


[libjpeg-turbo] Fix usage when building static

Since upstream allows building both dynamic and static libraries at the same time, we cannot patch to remove the static library suffix.
The generated cmake configuration file should synchronize the modification of the static library name.
Fixes #23422.

@reynoldsbd Can you please test this PR?
Thanks.

@JackBoosY this works great, thanks!

/azp run


Python 3.11 (and 3.12) support

Currently sheeprl requires Python version >= 3.8 and <3.11 (https://github.com/Eclectic-Sheep/sheeprl/blob/main/pyproject.toml#L27)
Is there a good reason not to support Python 3.11 (which was released 18 months ago) or even 3.12?

The reason was that in the beginning python 3.11 was too new. I'll open a new branch asap

Hi @MischaU8, ca you try out this branch? Python 3.11 should be already supported, for Python 3.12 we have to investigate all the other dependencies and check which is working and which not


Export the utilities from export.ts in excalidraw package

We can start exporting the methods in export.ts.
This will allow the host to use these functionalities out of the box.
cc @dwelle

I think we wanted to export the ones in src/packages/utils.ts, but otherwise yes.

That would be better, so all utilities from utils are always available for excalidraw package


Unable to unpack game made with AGS <IP_ADDRESS>

The notes for v.07 say it supports AGS 3.6.0, but I was not able to unpack a game built with <IP_ADDRESS>, receiving an error message that the exe was not a valid AGS game. Is that too new of a sub-version for AGSUnpacker to handle? If so will the program be updated? (I was able to successfully unpack a game made with AGS version 3.4.1-something.)

Can you verify if this is still valid for AGSUnpacker v08?
If v08 doesn't solve this problem then specify which game you're trying this on and its version also.

After four months, I'm not entirely certain what game it was I was even trying to unpack anymore 😅 But if I'm right by matching the dates of certain files on my computer, this version of AGSUnpacker works on the game in question and I was able to get the text .trs file out of it. That was Stair Quest Winter 2023 version https://decafjedi.itch.io/stair-quest

Ok, I guess we can close this issue now 😃

where can one get the winter '23 version of that? i downloaded the current linux version and the engine says: Compiled with: <IP_ADDRESS>


[cxx-interop] Emit function types as Swift closures in templated params

This is required to make the compiler distinguish between instantiations of std::function.
Previously, when generating a Swift type name for a std::function instantiation, we would always emit _ as the template parameter. If someone referenced two different instantiations of std::function in a Swift module, they would get mangled with the same name, triggering linker errors later.
rdar://103979602

@swift-ci please smoke test

@swift-ci please test source compatibility


Why do jspm and gulp need to be installed globally?

I have gotten into the habit of using only local versions of "exectuables." This makes it much easier to do CI and run consistently across developer machines.
I can then do node_modules/.bin/jspm install -y  and node_modules/.bin/gulp watch
Is there a good reason the docs suggest that you need both a local and a global version?
see also #179 and #134
I would be willing to do a pull request to revise this if we agree it is the best practice

Even if you install globally and locally, the local one will be invoked if you're running gulp/jspm from the root of the web project.

@oising thanks for pointing this out. You are right.
My concern is that

when people try to migrate to CI, they will end up spending a lot of effort trying get the globals working with CI when they don't even need to
relying on up to 2 versions of a package (global and local) is unnecessarily complex.  There will inevitably be some issue that creeps up based on the version combo. The less the global version does before calling the local version, the less the risk, but there is still risk.

I am not convinced the cons above are worth the convenience of jspm over node_modules/.bin/jspm especially when we could "install" a symlink (ln -s node_modules/.bin/jspm ./jspm) instead of the global which gets us nearly the same thing ./jspm install -y.
I am going to close the issue as all public documentation suggests that installing both global and local is the way to go. Consider reopening it if you guys start seeing issues related to CI and version conflicts :)


Philadelphia Meetup

Our first Philadelphia IPFS Meetup will be in mid-April. I created a meetup.com group: https://www.meetup.com/Philadelphia-IPFS-Meetup

duplicate. see #220


Correcting the name of the test.

not to be too picky, but we expect commits to be GPG-signed :)

Ok jim, sorry about that! ;-)


Update Notifications

Allow users to get informed when updates happen.
Type of notification is to be defined. In the past a Twitter feed would have been a fitting choice, but that's no longer an option.
Possibilities:

e-mail
RSS / Atom (this should be fairly straigthforward with static generation)
some other social media - instagram, fediverse?

Publishing a RSS and/or Atom feed is the best place to start. Based on that, people can do their own integrations to get notifications in their own way, without the project explicitly needing to support it. E.g. for chats like slack, discord

RSS available at /rss, enough for now


Added missing instance name in example

Just a minor addendum

@soffensive : Thanks for your contribution! The author, @MikeRayMSFT, has been notified to review your proposed change.

Thank you @soffensive - #sign-off


Node-collector mounts for CNI binaries

What steps did you take and what happened:
Node-collector cannot be deployed to GKE Dataplane V2 by Trivy because it fails to create container:
failed to generate container "150..." spec: failed to generate spec: failed to mkdir "/opt/cni/bin/": mkdir /opt/cni: read-only file system: CreateContainerError
What did you expect to happen:
node-collector runs without error
Environment:

K8s-Node-Collector version: 0.0.5
Kubernetes version (use kubectl version):
Server Version: version.Info{Major:"1", Minor:"25", GitVersion:"v1.25.8-gke.500", GitCommit:"f117e29cb87cfb7e1de32ab4e163fb01ac5d0af9", GitTreeState:"clean", BuildDate:"2023-03-23T10:22:38Z", GoVersion:"go1.19.7 X:boringcrypto", Compiler:"gc", Platform:"linux/amd64"}

Seeing this same problem.

@chauncey-garrett @dejewi you can override volumeMounts and volumes in trivy-operator helm values
Let me know if it will solve your issue.


Text drawn in wrong place on Android

See attached screen dump from my android phone . The channel name has been written one row below where it should have been.

which phone is it?
We could shorten the name a bit as a workaround.

We could shorten the name a bit as a workaround.

If that would avoid it happening on all phones then fine. If that fix would be specific to phones of a certain screen size & it might still happen on other screen sizes then perhaps not.

Looks like the displaysize setting "larger or largest" breaks the layout. We will change it so that the service name is cut to avoid it being drawn incorrectly. (or otherwise adjusted, I'll let the layout master decide the best solution)

Looks OK on my Huawei Mate 10 Pro phone - screen resolution is 1080x2160 pixels at 480dpi

@jpiesing - please comment on whether this has been resolved, i presume that this is HTML 'flow' related

Fixed


Model tests

Tests for model package

I don't know much about testing with frameworks, but looks good to me :+1:
@bradmontgomery you are going to have to merge this one, grey button...

Thanks @damianurbanczyk !


Allow 'refresh_immune_slices'

Ref #2958 , add a 'refresh_immune_slices' property to dashboards. Any slice ids listed will not be included in an automatic dashboard refresh.
I chose to exclude rather than include to be consist with filter_immune_slices.

Also you can make sure you pass the build before sending in the PR by setting up travis-ci against your fork. It's really easy to setup and will run a build for each new push on your repo.

Any additional reviews on this PR? (Apart from don't merge master into your dev branch like that in future :) )

Coverage increased (+0.002%) to 69.3% when pulling db7a64d4e3989ef52992817d1485c263cf718198 on RichRadics:refresh_immune into 90592d3e3d4d2009ae59fcea55c08c932522e154 on apache:master.

Coverage increased (+0.002%) to 69.3% when pulling db7a64d4e3989ef52992817d1485c263cf718198 on RichRadics:refresh_immune into 90592d3e3d4d2009ae59fcea55c08c932522e154 on apache:master.


Adding input sanity checks to Manufacturer APIs

Adding checks to api/v1/rvinfo
Adding checks to api/v1/customers
Adding checks to api/v1/assign

Signed-off-by: Davis Benny<EMAIL_ADDRESS>

Validated


Any interest in joining the trac-hacks organization?

Hi,
I'm an administrator for trac-hacks.org.  We've created an organization on GitHub and are inviting authors of trac-hacks plugins to join the organization.  Your repository could be transferred into the organization as described in the GitHub docs.  You'd retain full control over the repository, but the hope is that it will be easier for users on GitHub to fine trac-hacks plugins, including yours, by viewing the trac-hacks organization.
In the future, if you choose to no longer maintain the plugin, the organization would be able to provide commit access to new developers.  This would only be done with your permission, as has been the policy for plugins hosted on trac-hacks.org.  If you've already decided to no longer maintain the plugin, we would particularly appreciate you transferring the plugin to the trac-hacks organization so that we can help find new maintainers for it.
Finally, creating the trac-hacks organization is a work in progress. I appreciate if you have any suggestions on how to best support the community of trac plugin developers on GitHub. I'll be writing a blog post soon with more information and will follow-up here with a link to that post.
Kind regards,

Ryan

Hi, that sounds interesting, but I'm not sure how "ownership" of organization repositories works.  Would I be assigned by the organization as an "owner" of my repo? Could you take that away for some reason without my agreement?  I'm just not sure how that all works.

You'd be administrator of the repository and in control of who you grant r/w access to. The  3 other admins and I technically have full control to grant and revoke permissions, so in theory we could revoke your permissions without your agreement. I can't think of a valid reason for us taking actions like that, and I hope that the trac community would hold anyone accountable for bad behavior like that. I've spent years working in the trac community and never had a conflict with anyone. It's important for me to retain a good reputation and to forge good relationships with everyone.
I had hoped to come up with a simpler solution that would avoid these "technically yes we can do that" situations, but it does not seem to be possible on GitHub. Repository tags would be the simplest solution, but GitHub doesn't support those. Creating an organization and moving repositories seems to be the only solution.
The motivation for this is based on what Jenkins CI has implemented - an index for browsing plugins. By having an organization it will make it easier to Trac users to locate plugins hosted on GitHub. We'll be linking to the organization from Trac and Trac-Hacks.org. The upside for you is hopefully the possibility of more people using your plugin, and more contributors.

Also, you can take a look at repository access teams. You'd be the only remember of the Admin access team for your repository, unless you choose to add other users. I and 3 other organization admins are in the Owner team.

Is this something you guys are still interested in?
Would you be able to provide some help with making sure binaries are available on PyPI properly?

Is this something you guys are still interested in?

You should receive an invite to join the trac-hacks organization. You'll have general Admin access to the trac-hacks organization which will allow you to execute these steps and transfer the repository. After you transfer the repository I'll remove you from the Admin team, create an Admin group for your repository and make you the Admin for your repository.

Would you be able to provide some help with making sure binaries are available on PyPI properly?

Yeah, I'm happy to help with that. Eventually I hope to have some infrastructure on trac-hacks.org for publishing to PyPI, but for now we can just do that manually.

Okay, transferred the repo.  You can now make the changes to make me an Admin for tracdocs, tracsql, and tracstats.

Thanks, permissions should be set correctly now for tracdocs.


Laravel Requirement Version

Is this also supported with Laravel 5.1?
Thanks!

Currently the minimum version is 5.3.

Oh ok. Thanks a lot!


python_examples: Use ensure_users where appropriate.

We added this function in 8e1a7cfb52aa2df6ff878d27a065f6ddd8778eab
in order to make things more readable in example which hard-code user
ids. The point is to validate that the id indeed refers to the user that
the person writing the example expects, while providing information to
readers of the code so they don't have to do db queries to figure out
the user. As mentioned in the commit referred to above, this is
particularly useful when some db changes cause renumbering of user ids -
because then all these ids have to be adjusted and it's nice to know the
intended user.

Merged, thanks @mateuszmandera!


Nearest swarm fixes, ip update fixes & optimizations

Nearest-swarm logic was broken in various ways when dealing with the wrapping region (i.e. between largest_swarm_id and smallest_swarm_id), and the test suite was vastly under-specified to catch these errors.  (See individual commits for details).
IP updates were broken: the code was specifically preserving old values when new swarm info arrives, which was exactly the wrong thing to do, and would prevent storage server from seeing IP address updates of other nodes (which would very likely lead to onion request connectivity failures).  This fixes it by only preserving values from the old swarm data if the new swarm info has a "0" value and the old data had a non-0 value.
The code around this area was also poorly optimized, with wasteful extra (hefty) allocations and data structure creations on every swarm update.
See individual commit messages for more details.

One thing that confused me is why the IP update problem didn't manifest in swarm testing.  Upon further investigation the reason is that swarm testing works from a separate set of all funded swarms (both active + decommissioned), and the set of funded swarms gets populated from the non-updated swarm list (i.e. the IP updates were happening to the copy thats get put in all_valid_swarms_, but then the funded swarms uses the pre-copy vector.


Depth order question

Hi,
I need not the real depth but the order of the different depths. in your paper you mention ..." For people of different scales, the people with greater scale are in the front. For people of similar scale, the people with greater center confidence are in the front"...
My doubt is how you classify "different scales" & "similar scales".
(Maybe you have already coded that issue when selecting the rendering order of the different  masks/meshs; if so could you point where that code is?)
Many Thanks in advance!

Good question! Thanks for your interest in our work!
As we all know, we estimate the 3-dim parameters of a weak-perspective camera model for 3D-to-2D projection and use 2D pose for supervision. The 3-dim camera parameters correspond to the scale, x-axis and y-axis translation of people in 2D images. We use the estimated scale with a distinguishable threshold to determine the "different scales" & "similar scales". Actually, in real cases, the center confidence is sufficient to determine the depth order of the crowd.

Thanks for the quick response & valuable comments!. Great work!

Hi,
It is quite interesting that the model can give a depth order in this (implicit?) way!  That info is valuable for the reconstruction.
Regarding this same issue. Can we recover the actual 3D scene depth with your model? From this cam parameters can we get the actual (estimated) absolute translation (x,y,z in 3D space) of each person in the scene?

Thanks for your interests!
Given the specific intrinsic and extrinsic matrix of a camera, maybe we can convert the estimated weak-perspective camera parameters into the absolute translation you expected.
We are working on a higher version that will directly give you the absolute translation of each person.
I believe it will fit your request. And we hope it can be released soon.

In the case of images on-the-wild, how could you have the intrinsic camera? Would it work in that case?

Of course, it is realy hard to drive the intrinsic matrix from the in-the-wild image. Therefore, weare able to estimate the 'relative' absolute translation, instead of the real one.
The new version use the perspective camera.

Hi，where can I get the 'relative' translation?


Fix ls colors on MacOS

ls --color=auto does not exist, use ls -G instead when on Mac

No longer using mac, but see https://formulae.brew.sh/formula/coreutilsto get yourself out of trouble.


Tests leave lots of random Docker tags

A quick (not necessarily accurate) skim of my docker image ls shows a whole lot of superfluous tags generated by the test suite:
run-exec-form
run-shell-form
different-user
new-user
cmd-shell-form
cmd-exec-form
cli-invocations
mamba-1322-workaround
multi-env

We should probably be tagging these as micromamba:test-run-exec-form, etc.
I'd submit a PR to fix this, but there are already enough merge conflicts in my current PRs. :stuck_out_tongue_closed_eyes:

I agree. I tend to regularly blow all my images away, so I haven' t really noticed.

Done in #111


fix docs or rename onlyIfMissing

the code: https://github.com/delaneyj/datastar/blob/main/packages/library/src/lib/plugins/backend.ts#L264
the docs: https://github.com/delaneyj/datastar/blob/main/backends/go/site/static/md/reference/plugins_backend.md ( ifmissing false )

Thanks for reporting this! Fixed in https://github.com/delaneyj/datastar/pull/198.


Closes #3264, disable options menu for null media from provider

Description (required)
Fixes NPE in MediaDetailFragment #3264
What changes did you make and why?
Disable options menu for null media from Provider
Tests performed (required)
Tested {ProdDebug} on {One Plus 3} with API level {27}.

Codecov Report

Merging #3265 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #3265      +/-   ##
=========================================
- Coverage    6.54%   6.53%   -0.01%
=========================================
  Files         270     270
  Lines       11613   11626      +13
  Branches      906     906
=========================================
  Hits          760     760
- Misses      10793   10806      +13
  Partials       60      60

Impacted Files
Coverage Δ

...ee/nrw/commons/media/MediaDetailPagerFragment.java
0% <0%> (ø)
: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 7b9e792...2227091. Read the comment docs.


add replicas check

enhancement base on: https://github.ibm.com/IBMPrivateCloud/roadmap/issues/57592#issuecomment-55240538
when we check deployment, both readyReplicas and replicas should be one

test looks good to me:
root@axe1:~/projects/go/src/ibm-common-service-operator/cp3pt0-deployment# ./setup_tenant.sh
(oc -n openshift-operators get deployment cert-manager --no-headers --ignore-not-found -o jsonpath='{.status.readyReplicas}' | grep '1' && oc -n openshift-operators get deployment cert-manager --no-headers --ignore-not-found -o jsonpath='{.status.replicas}' | grep '1')
[INFO] Waiting for Deployment cert-manager to be ready
[✔] Deployment cert-manager is running

needReplicas is a value
replicas and readyReplicas are command

Thank you @qpdpQ
It looks good to me!
Meanwhile, could we remove the wait here? https://github.com/IBM/ibm-common-service-operator/blob/ffd4ac6fe50f0622aaee491455565f4a136b0353/cp3pt0-deployment/setup_tenant.sh#L306


Removing 3rd build framework

CMake is the local build (for the library, CLI and bindings) and QMake in the .pro/.pri files, so as far as I can tell these are redundant.

They are now... I moved embroidermodder over to QT6 using cmake.


Add pycryptodome backend to compute keccak256 hash

What was wrong?
Moving the work here from this PR: https://github.com/ethereum/eth-bloom/pull/12
Refer to this comment to know we are doing this: https://github.com/ethereum/eth-bloom/pull/12/files#r166493846
How was it fixed?
Added a new backend for keccak256 using the pycryptodome package. We need to do this to add support for PyPy (3).
Cute Animal Picture

Thanks! Looks like this repo will get moved to /ethereum/eth-hash soon.

@carver Do you think we can cut a release and deploy to PyPI, so that I can start working on integration of eth-hash with other Ethereum packages relying on keccak256?


async handler fn by macro

The macro https://github.com/tqwewe/kameo/blob/bb4c8ffdbda8bd4f56cde00ffe8bc38e7d557197/crates/kameo/examples/macro.rs#L19
example shows how to define a handler fn which is not async. Is that on purpose and what's the reason for that? I would have expected this to be async or return something impl Future<...>

Ah. nice. Just discovered you support both: sync and async:
https://github.com/tqwewe/kameo/blob/bb4c8ffdbda8bd4f56cde00ffe8bc38e7d557197/crates/kameo_macros/src/actor.rs#L332
Sorry for the noise.

No worries! I'll just clarify anyway why it works with/without async:
When using the macro, you can add async or not and it'll work either way.
This code essentially expands to:
struct Inc { amount: u32 }

impl Message<Counter> for Inc {
    type Reply = Result<i64, Infallible>;

    async fn handle(self, state: &mut Counter) -> Self::Reply {
        state.inc(self.amount) // .await is added here if your method is async, otherwise doesn't
    }
}


Create TakeSurvey button

Summary

[ ] Create a button that opens symptom tracker survey
[ ] Set up a route /symptomsurvey

Motivation
The button should redirect to symptom tracker survey

I'll try to implement this


Rapid Breach Response Pack Reference Enhancement

Status
Ready
Related Issues
fixes: https://jira-dc.paloaltonetworks.com/browse/CIAC-10482
Description
Better understanding of the Rapid Breach Response pack goal and initial configuration.

@altmannyarden Doc review completed.


Adding custom html to the picker

I wanted to add "quick date range options" to the calendar. A small button to trigger specific ranges like: today, last 7 days, this month, etc...

I was able to add them with the onShow callback function.
onShow: function () {
var quickPicks = document.createElement('div');
quickPicks.setAttribute('class', 'container-fluid px-1');
var quickOption = document.createElement('a');
quickOption.setAttribute('class', 'btn btn-sm btn-primary m-1');
quickOption.setAttribute('href', '#');
quickOption.innerHTML = 'Last 30 days';
quickOption.addEventListener("click", function () {
    this.setDateRange(moment().subtract(29, 'days').toDate(), moment().toDate());
    this.hide();
 }, false);
quickPicks.appendChild(quickOption);
document.getElementsByClassName('container__months')[0].appendChild(quickPicks);
}

This works when the picker gets opened. The buttons even work before any other interaction with the calendar.
But as soon as I interact with the calendar (select one date in the calendar, or change months) the quickPicks disappear immediately.
I assume the div's are getting destroyed each time an interaction takes place. (That makes sense when changing months, but that doesn't seem intuitive when only selecting a date range).
How could it be possible to add custom html to the calendar that remains after interactions?

It seems in calendar.ts
this.picker.innerHTML = '';
this.picker.appendChild(months);

totally resets the whole html of the component on each render() call.

I have it "working" by hijacking all onSelect, changeMonth events and adding the custum html there.
Maybe a callback after the render() event would make this easier?

Sure, PR is welcome.

PR created.

Thanks for PR.
v1.0.31 is published.

Hey guys.
I wasted way more time than I wanted to find a lightweight datepicker for Vue – and failed completely. Found out Litepicker and was super excited, but the lack of a Vue version and missing shortcut buttons was very disappointing.
But, kind of by accident, I found this issue and it was super encouraging. @glennvdbroeck did a great job hacking the project and it was the base for me to start  – and now I am happy to say that I could achieve partially what I wanted:

Vue + Litepicker + Shortcut buttons on Codesandbox 🥳
I am not a great Javascript developer, but I thought that maybe I could help this project with a PR of these shortcut buttons.  But the first problem is: how to use the same date library as user? Is that a way to find (and them import) the right package?
For example, @glennvdbroeck is using Moment, but I prefer date-fns. Since we need only basic and few functions, I thought we could ship the package "plug-and-play" for Moment, date-fns and Day.js – without installing any of them by default, just verifying what is already there and using appropriately.


Failed send file if there are spaces in name or path

Describe the problem
Failed to send file when there are spaces in the path.
I am sorry, I know this issue is in cometchat_chat_uikit, but I have search the repository of cometchat_chat_uikit but I cant found it. So I reported this issue here.
What was the expected behavior?
File successfully send or give the documentation how to resolve this issue.
Reproduction

Step 1 : Run the app
Step 2 : send files that have spaces in the path

Exception : Invalid message object received
SS :

Environment
cometchat_chat_uikit: ^4.0.0
flutter version 3.10.6

you can use this
removeUriSpaces(String string) {
return string.replaceAll(' ', '');
}
or if you want to use extension method then
extension UriExtension on String {
String removeUriSpaces() {
return this.replaceAll(' ', '');
}
}

you can use this removeUriSpaces(String string) { return string.replaceAll(' ', ''); }
or if you want to use extension method then extension UriExtension on String { String removeUriSpaces() { return this.replaceAll(' ', ''); } }

Thank you for the response,
But the function of send file is in ui kit or handled by ui kit, and cannot using onSendButtonTap property in messageComposerConfiguration because the file will automatically sending without tap button send.
So should I wait for updates from cometchat_chat_uikit ?
I tried posting the issue but didn't find the repository on Github, nor did pub.dev list their repository

Hello @nandakista,
Thank you for the additional details. We have taken this report and sent the notified our product engineering team who will be investigating.
We will reach out when we have more information on this.
Provesha Pyne
CometChat

Hello @proveshapyne
Any update for this issue ?

Hey @nandakista,
Thank you for enquiring!
Our team has taken up this task but I cannot provide you with a specific timeline at this moment. Nevertheless, I assure you that we will update you as soon as we have more information.
In the meantime, please feel free to submit a new request if you need a hand with anything else.

Hi,
I upgrade to cometchat_chat_uikit: ^4.0.4 and I see in the cometchat_uikit_shared: ^4.0.4 changelog, it solved the issue
I've tested and it really solved the issue.
Thank you for the update and response
Close the issue for now


remoteURL parameter error cause  challenge cache not working which eventually lead to client manifest request timeout

Problem
In file proxyregistry.go remoteURL parameter error cause challenge map cache  not working,critical area consume to much time, which eventually lead to client manifest request timeout(90s) in massive(which i test 80 concurrent) concurrent manifest request situation. this directly lead to docker pull error.
Cause
GetChallenges(key),this key should be remoteURL not r.remoteURL ,because AddResponse(key) key=remoteURL.
this little error cause Challenge cache Map not working ,which the r.Lock() `s critical area consume to much time ,and in massive concurrent situation which will lead to client request timeout.  we observe this timeout in our product environment.
Reproduce

first bound  ekv59auv.mirror.aliyuncs.com to you registry mirror ip in your /etc/hosts
use the following python code to reproduce .

#!/usr/bin/env python
# -*- coding:utf-8 -*-

import threading,logging,sys

import requests,random,uuid

class ConCurrentTask(threading.Thread):
    URL_PREFIX = "https://ekv59auv.mirror.aliyuncs.com/v2/library/"
    IMAGES = [("ubuntu","latest"),("busybox","latest")]

    def __init__(self):
        threading.Thread.__init__(self)

    def run(self):
        return self.do(*self.random())

    def do(self,image,tag,id):
        logging.info("start request manifest: id = %s",id)
        res = requests.get("%s/%s/manifests/%s?id=%s"%(self.URL_PREFIX,image,tag,id))
        if res.status_code == 200:
            logging.info("request manifest finished success : id = %s, image = %s, tag = %s, body = %s"%(id,image,tag,len(res.text)))
        else:
            logging.error("request manifest finished error: id = %s, image = %s, tag = %s, error_code = %s"%(id,image,tag,res.status_code))

    def random(self):
        i = random.randint(0,len(self.IMAGES)-1)
        return self.IMAGES[i][0], self.IMAGES[i][1], uuid.uuid1()

def setup_logging():
    fort = logging.Formatter('[ %(asctime)s %(name)s - %(levelname)s ]:::: %(message)s')
    chand= logging.StreamHandler(sys.stdout)
    chand.setFormatter(fort)

    root = logging.getLogger()
    root.addHandler(chand)
    root.setLevel(logging.INFO)

def main():
    NUM_TASK = 100
    setup_logging()
    tasks = [ ConCurrentTask() for i in range(NUM_TASK)]
    for t in tasks:
        t.start()

    for t in tasks:
        t.join()

if __name__ == "__main__":
    main()

Fix

modify the r.remoteURL to remoteURL,this can fix the timeout problem.
further more, i was thinking to remove r.Lock() and defer r.Unlock(),as we already have a sync.RWLock for challenge map protection,but this still need to be tested,

// tryEstablishChallenges will attempt to get a challenge type for the upstream if none currently exist
func (r *remoteAuthChallenger) tryEstablishChallenges(ctx context.Context) error {
	=======================
	//r.Lock()
	//defer r.Unlock()
	=======================
	context.GetLogger(ctx).Debugf("tryEstablishChallenges: get lock success")
	remoteURL := r.remoteURL
	remoteURL.Path = "/v2/"
	challenges, err := r.cm.GetChallenges(remoteURL)
	if err != nil {
		return err
	}

	context.GetLogger(ctx).Debugf("tryEstablishChallenges: cm.GetChallenges()=Len(%d)",len(challenges))
	if len(challenges) > 0 {
		return nil
	}

Signed-off-by: spacexnice<EMAIL_ADDRESS>

Current coverage is 50.81% (diff: 100%)

Merging #1958 into master will decrease coverage by 10.17%

@@             master      #1958   diff @@
==========================================
  Files           124        124
  Lines         11084      11084
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           6760       5632   -1128
- Misses         3441       4709   +1268
+ Partials        883        743    -140

Powered by Codecov. Last update cb744ef...3c4b551

Thanks for finding this, this change looks correct to me
LGTM

great catch @spacexnice
LGTM


[AutoPR cognitiveservices/data-plane/QnAMaker] fix schema property naming

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6901

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/5627


Add testing functionality to verify compilation for generated unit-tests

This commit adds verification functionality for unit tests generated, which are api-testing, overloaded-api-testing, policy_funcs/api-testing and policy_funcs/overloaded-api-testing under auto_generated. Targets run-api-testing / run-overloaded-testing / run-policy-api-testing / run-policy-overloaded-testing are now available with your favorite RISC-V compiler.
For example: (Under rvv_intrinsic_doc/)
make -C rvv-intrinsic-generator run-api-testing COMPILER=/path/to/llvm-project/build/bin/clang

Closing this PR, the commit will be included in merge of #194.


Keras model card

This PR creates automatically generated model cards for Keras models, requested in #628
The model card contains:

a model plot (I first added summary then swapped it with the plot itself because it was looking bad)
Model hyperparameters like in TF transformers
Model metrics like in TF transformers
Additional sections for limitations, intended use etc. (see below)
if user wants to push a specific task, I added another argument to push_to_hub_keras() called task_name

TF tests are failing because tf.keras.utils.plot_model() requires a package called pydot.
see docs:

Converts a Keras model to dot format and save to a file.

Is it worth having given it's another dependency? @osanseviero

@osanseviero where do I add test dependencies (like requirements file) I've written the is_pydot_available() function

@osanseviero edgiest case ever, seems like they don't get installed for some reason

@osanseviero I honestly don't know what else I have to do so that it would install pydot. I'll look deeper into it.

This is so weird, the tests are failing bcs model plot is not appearing in the repository, probably because the plotting itself is skipped (see below). The test is running when graphviz and pydot are available.
see:
def require_tf(test_case):
    if (
        not is_tf_available()
        and not is_pydot_available()
        and not is_graphviz_available()
    ):
        return unittest.skip("test requires Tensorflow, graphviz and pydot.")(test_case)
    else:
        return test_case

The tests that require pushing has to plot model, and the model plotting part is skipped likely because:
if is_graphviz_available() and is_pydot_available():
        _plot_network(model, repo_path_or_name)

But this shouldn't happen since I'm calling it inside the tests decorated with require_tf, meaning, graphviz and pydot should be available.
When I test on my local push_to_hub_keras() works fine and it pushes the plot as well.
@nateraw can you see if I miss something here?

After you call model.predict() or model.evaluate() the model.history.history goes away. push_to_hub_keras() doesn't have such side effect. Also see this issue
In the notebook @nateraw sent there's a prediction cell in between that's why it happens. (See below, I also saw the same behavior in another Keras example)

If your model is build but not compiled it looks like this.
If model is build + compiled it looks like this.
If model is fit it looks like this
I'm roughly done (also with tests), I just need to solve this merge conflict. (which I'll do tomorrow)

I moved model card creation under save_pretrained_keras().
model.save_pretrained() calls save_pretrained_keras() with model card and plotting arguments with default values (plot_model = True and task_name = None).
The tests check that model card and plot is created when model is built + fit, called under both save_pretrained_keras() and push_to_hub_keras().

I managed to break the card.
It writes every epoch like so
for line in lines:
            model_card += f"\n| {lines.index(line) + 1}|"  # add values

Lines look like this
[{'Train Loss': 0.15567651391029358}, {'Train Loss': 0.1543986201286316}, {'Train Loss': 0.1531306654214859}]
which line looks like this:
{'Train Loss': 0.1531306654214859}
if training loss never changes the indexes will look like 1 1 1 on the table.


xray: 1.8.0 -> 1.8.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for xray is: A platform for building proxies to bypass network restrictions. A replacement for v2ray-core, with XTLS support and fully compatible configuration
meta.homepage for xray is: https://github.com/XTLS/Xray-core
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.8.1 with grep in /nix/store/n2gjwcqhzdhd1g7f6f6kfjr0vpgsfbxy-xray-1.8.1
found 1.8.1 in filename of file in /nix/store/n2gjwcqhzdhd1g7f6f6kfjr0vpgsfbxy-xray-1.8.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
xray

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/n2gjwcqhzdhd1g7f6f6kfjr0vpgsfbxy-xray-1.8.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A xray https://github.com/r-ryantm/nixpkgs/archive/c2ead9bf1dee1ac88a7e96da2e49a8796d452a22.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/n2gjwcqhzdhd1g7f6f6kfjr0vpgsfbxy-xray-1.8.1
ls -la /nix/store/n2gjwcqhzdhd1g7f6f6kfjr0vpgsfbxy-xray-1.8.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

xray

Maintainer pings
cc @iopq for testing.

Result of nixpkgs-review pr 229403 run on aarch64-darwin 1

1 package built:

xray


java.lang.UnsupportedOperationException when use material-dialog.

as following logcat:
Caused by: android.view.InflateException: Binary XML file line #105: Error inflating class
	at android.view.LayoutInflater.createView(LayoutInflater.java:606)
	at com.android.internal.policy.impl.PhoneLayoutInflater.onCreateView(PhoneLayoutInflater.java:56)
	at android.view.LayoutInflater.onCreateView(LayoutInflater.java:653)
	at android.view.LayoutInflater.createViewFromTag(LayoutInflater.java:678)
	at android.view.LayoutInflater.rInflate(LayoutInflater.java:739)
	at android.view.LayoutInflater.rInflate(LayoutInflater.java:742)
	at android.view.LayoutInflater.inflate(LayoutInflater.java:489)
	at android.view.LayoutInflater.inflate(LayoutInflater.java:396)
	at android.view.LayoutInflater.inflate(LayoutInflater.java:352)
	at com.a.a.c.(MaterialDialog.java:104)
	at com.a.a.g.a(MaterialDialog.java:1449)
        ... 11 more
Caused by: java.lang.reflect.InvocationTargetException
	at java.lang.reflect.Constructor.constructNative(Native Method)
	at java.lang.reflect.Constructor.newInstance(Constructor.java:417)
	at android.view.LayoutInflater.createView(LayoutInflater.java:586)
	... 28 more
Caused by: java.lang.UnsupportedOperationException: Can't convert to dimension: type=0x12
	at android.content.res.TypedArray.getDimensionPixelSize(TypedArray.java:463)
	at android.view.View.(View.java:2846)
	at android.view.ViewGroup.(ViewGroup.java:385)
	at android.widget.FrameLayout.(FrameLayout.java:99)
	at android.widget.FrameLayout.(FrameLayout.java:95)
	... 31 more

so, I'm not sure is this issue caused by material-dialogs, but it really seems be. Can anyone help me?
And by the way, it's not always happen for all devices, only one Device(OPPO Finder).

You might need to research this a bit yourself. Unfortunately I have no way to solve this from the logcat.

:( OK, I'll try my best for it. Thanks all the same.

Hi, I have met the same exception on OPPO, have you fixed it?

@shufeike not yet.

Any news on this?

why don't you post the place you use MaterialDialog and its xml?

Does this crash still happen with the latest versions of the library?

@afollestad I'll try it later and tell u the result ASAP.

There's a possibility that release <IP_ADDRESS> fixes this issue, since large internal parts of the library were pretty much redone. Let me know.
https://github.com/afollestad/material-dialogs/releases/tag/v<IP_ADDRESS>

It has been in testing process, let's waiting for the result. @afollestad

@afollestad I have to say: Nice shot! It's fixed.


Centering text

Is there a recommended way to center text within the Text2d mesh?
I see that text is always left-aligned within the Text2d canvas due to this line: https://github.com/gamestdio/three-text2d/blob/master/src/CanvasText.js#L32
I was able to center the text by changing the line to: this.ctx.fillText(text, (this.canvas.width - this.textWidth)/2, (this.canvas.height - this.textHeight)/2); but then there's no way to change that alignment either.
Thanks

Hey @russplaysguitar, using textAlign.center should do it. Example: https://github.com/gamestdio/three-text2d/blob/master/index.html#L37

I think that the problem was that I was scaling-up the text using Text2d.geometry.scale(), but this didn't scale-up the relative positioning of the text to the mesh. Using Text2d.scale.multiplyScalar() fixed that.


Update project config

update the linguist
Remove the global.json
Fix the DDPA.Web reference
Remove win7-x64 run-time identifier
Update the package reference
Update the jquery and chart.js

Downgrade the Automapper to version 7 due to Automapper 8.x disabling dynamic mapping. Create a separate pull request for the updates later.


REGRESSION: TextField Hint Multiline

Problem: TextField Hint is not multiline and there is no way to make it multiline. This is.a regression of https://github.com/flutter/flutter/issues/20941 #20941
Steps to Reproduce

Create a multiline TextField with a Hint Input Decoration.
Verify that the multiline TextField works by setting multiLine: null.
Verify that the hint does not work.

Load https://gist.github.com/searchy2/8ca6f4d92fb4044552928d56a6dc9962 to see example.
Flutter doctor
[✓] Flutter (Channel stable, v1.5.4-hotfix.2, on Mac OS X 10.14.4 18E226, locale en-US)
    • Flutter version 1.5.4-hotfix.2 at /Users/christopher/Development/flutter
    • Framework revision 7a4c33425d (3 weeks ago), 2019-04-29 11:05:24 -0700
    • Engine revision 52c7a1e849
    • Dart version 2.3.0 (build 2.3.0-dev.0.5 a1668566e5)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/christopher/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.2.1, Build version 10E1001
    • ios-deploy 1.9.4
    • CocoaPods version 1.6.1

[✓] Android Studio (version 3.4)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 35.3.1
    • Dart plugin version 183.6270
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

[✓] Connected device (1 available)
    • iPhone X • BAFA7B7D-73DB-45DD-8DC1-F0D8834D01BD • ios • com.apple.CoreSimulator.SimRuntime.iOS-12-2 (simulator)

InputDecorator hint text is rendered with ellipsizing enabled:
https://github.com/flutter/flutter/blob/master/packages/flutter/lib/src/material/input_decorator.dart#L1959
If hintMaxLines is null, then the text will be ellipsized at the first line that exceeds the width constraint:
https://api.flutter.dev/flutter/painting/TextPainter/ellipsis.html
Try setting hintMaxLines to a non-null value that will be large enough for the text in this widget.

@jason-simmons Thanks for the temporary workaround. However, your solution should not be accepted as the desired behavior for the following reasons:

The original solution https://github.com/flutter/flutter/commit/114335dff0e511b6555a05c99511ad07563252c9 specifies that the InputDecoration hintMaxLines inherits from the parent TextField's maxLines. This means the hint behavior should be inherited as well for consistency.
Setting hintMaxLines does not allow the InputDecoration to calculate its layout or ellipsize correctly.

The desired behavior is as follows:

hintMaxLines should correctly inherit the behavior from the parent TextField's maxLines value.
The placeholder hint text should ellipsize if it exceeds the placeholder text bounds.

A similar complaint about this behavior was posted in https://github.com/flutter/flutter/issues/32830 (and a PR was opened https://github.com/flutter/flutter/pull/33017).  @searchy2 What are your thoughts on that and would that PR solve your problem?
I think I agree with Jason that this is working as it was written, but the behavior might not be ideal and we can probably make some improvements.  About your desired behaviors:

hintMaxLines does inherit the value of maxLines if it's not specified. However, the behavior can be slightly different, like you see in your gist, because hint is rendered using Text under the hood and Text just isn't exactly the same as TextField.  Mainly:
I don't think it's currently possible to ellipsize a multiline Text when maxLines is null.  If you specify a number > 1 for maxLines then it will show an ellipsis on the final line, but if you specify null then it will ellipsize on the first line.

So for your gist, it's not currently possible to ellipsize the hint in that TextField with maxLines: null on the final line.  You could work around this by specifying a number for maxLines that effectively looks the same and removing the height on the parent Container.

And lastly, I don't know how you created a TextField where the hint renders outside the top like in your screenshot but that looks like it could be a bug :confused:

@justinmc thanks for taking a look and filling me in on the background of this issue. I hope we can agree that when hintMaxLines is set to null, text should not ellipsize on a single line but continue until it is clipped? If so, then #33017 is sufficient.
I understand that text input is a major problem for Flutter right now that is being worked on. If I could provide a suggestion from my experience working with text on Android, it would be to have two text rendering logic flows controlled by whether the TextField's constraints are fixed or flexible. If the bounds are fixed, then text should be properly ellipsized at the end of the line when the text runs out of room. This would also enable text to be ellipsized in the middle. If the bounds are flexible, then the TextField should expand to wrap text. Of course, having a maxHeight and maxWidth is necessary when using flexible.
For now, the single line PR would be enough. Until we get a smarter text layout engine 🙏

The single line fix seems ok to me after investigating a bit, so I'm just waiting for the author to come back with some tests and hopefully we can get that merged.
Flutter doesn't quite have the first class distinction between flexible/fixed, but does sort of work that way under the hood.  There are several changes to TextField and Text in progress now, so hopefully those will start to clear up things like this.

Thanks @justinmc for your hard work! I'll close this issue for now as a fix is in the works.
For how text should behave, I think the best test case is to support multline ellipsis middle. On Android this is not possible because of the text rendering order. If it was, it would solve so many text layouting issues.
Look forward to the improvements!


Missing space before external document in breadcrumbs

Search terms
space, breadcrumb, missing
Expected Behavior
A space before the document name in the breadcrumbs, e.g:
Name / Package / Folder / Document

Actual Behavior
No space before the document name in the breadcrumbs, e.g:
Name / Package / Folder/Document

Live Example
Take a look here: https://npm.nicfv.com/documents/t6.examples_Assert_Boolean_Values.html

Environment

Typedoc version: 0.26.5
TypeScript version: 5.5.4
Node.js version: 20.11.1
OS: 22.04

If you had a document named "examples" which included child documents for each of the sub pages, typedoc would produce what you are expecting. You could also use the yaml front matter to set the document title to "examples / Assert Boolean Values"

Can you please point me in the direction how to do parent/child documents, as well as the yaml formatting?

https://typedoc.org/guides/documents/ should have everything you need

https://typedoc.org/guides/documents/ should have everything you need

Thank you!

Is it possible to just request there to be a space before and after that /? I think it would look nicer, but I know that's an opinion.

Not including the space is the correct behavior -- perhaps it would be more clear if TypeDoc's UI used something besides / for separating pages in the breadcrumbs view...
TypeDoc is just displaying the document name here, I could maybe see an argument for splitting the document name on / and including non-linked entries similarly to how the page sidebar supports that magic, but just including spaces is definitely wrong.


Support both local installation and hermetic build for Python.

Reasons for those changes

Why setup the @python repository: Python projects with C/C++ extensions/modules require corresponding cpython headers and libraries.
Why add options to build a hermetic Python: setting up a local bazel_python installation is efficient but may confuse new users like the reviewers.
BTW, we want to fix #14.

Changes in bazel_python.bzl
bazel_local_python
bazel_local_python replaces the original bazel_python. Changes:

It now takes new python_version and name='python' arguments.
It will create a local repository @name (@python by default) and expose cpython headers, libraries (@python or @python//:python) and binaries (e.g., @python//:bin/python3).

bazel_hermetic_python
The new bazel_hermetic_python fetches a python source code and builds it using @bazel_python//:external/python*.*.BUILD. It has the same interface as bazel_local_python and gives the build @python repository.
bazel_python_venv
bazel_python_venv no longer takes the python_version argument. Instead, it takes an (implicit) python (tool) argument to specify which Python repository and binary to use. It by default is @python//:bin/python3.

Should README.md also be updated?


Gutter icon not showing

@martomo
I am experiencing the same issues outlined in these 3 tickets.
#58
By @PRR24
#74
By @oddietaco
#116
By @dickiedyce
My current line indicator is not showing while execution has been halted and I step through my code.  The breakpoint indicators are visible and they work, I can step through my code and the stack and watch windows work correctly, I just don't get any indication of what line I am currently on?
Has there been a solution for this implemented yet?  Is there some workaround I could implement?
It's extremely difficult to debug when the current line cursor is not showing.
I have tried uninstalling the package and Sublime Text 3.  I cleared all cache files and did a clean install of Sublime.  I installed only the xdebug package, so I could rule out clashes with other packages, but still no current line indicator.
I am running: -
OS : Mac OSX Sierra 10.12.4
Sublime : Build 3126
Xdebug client : v2<IP_ADDRESS>.15.10

I've got an update.  I noticed my file was named using camelcase.  I needed this to all be lower case for the system to work correctly in my case-sensitive production environment, so i changed the name from partsFactory.php to partsfactory.php using the mv command on CLI.  The cursor now shows in Sublime?
Presumably as the mv command has created a new file with the content of the old file and written the new file over the top of the old one, this has reset some cache somewhere or reset the permissions or something?
Before renaming the file, I added a breakpoint to my index.php bootstrap file and this showed the current breakpoint cursor correctly.
Hope this helps others with the same problem.


Bulletin post with no images not visible

The bulletin post with no images are not visible. The html tag key that we are looking for is not inclusive enough.

Fixed in this commit 24d0b2ecb3f591f48323e71847f48d349b635be6


Fix delete not refreshing: Refetch all quotes after Delete

This change fixes the issue where react continues to display an already-deleted quote.
Deleteing quotes succeeds to the backend, but react doesn't re-render the entire list, so the deleted post remains shown.
(This was why we previously did a window.refresh() which was slow and disruptive to the UI)
Apollo Queries/Mutations should cause its react component to re-render after changes.  So when we call the delete mutation, we can pass it an option to force Apollo to refetch the entire array.
See: https://www.apollographql.com/blog/when-to-use-refetch-queries

cancelling, adding more commits for seed


 Could even suggest what teleport to use to get there quickest.
Could start slow and start with common ones (nechs, gargoyles etc) with settings to determine which one you want to go to.
Would save the hassle in referencing a map constantly and would be super useful for those early obscure slayer assignments.

Duplicate of #3843


JBDS-3657 Installer should set updated default username and password in .cdk

+1 for the change. Not convinced this causes the travis build failure though

This build reports are useless right now. It fails all the time ;)

The travis builds don't run properly because of github rate limit + jspm + travis only allowing to use a github token for branches in this repo. Builds for PRs from forks just won't work as of now, so don't worry about these. However, that does not mean you shouldn't check it locally :)

Applied in master


Feature request: Add conversion from OpenCV to ndarray

I think converting OpenCV to ndarray would be a good feature. As Pyo3 has good support for converting numpy arrays to ndarrays, seems like a good feature to be able to convert OpenCV mat directly to ndarray as that enables python/rust OpenCV usage.
A possible conversion could be
trait AsArray {
    fn try_as_array(&self) -> Result<ArrayView3<u8>>;
}impl AsArray for cv::core::Mat {
    fn try_as_array(&self) -> Result<ArrayView3<u8>> {
        if !self.is_continuous() {
            return Err(anyhow!("Mat is not continuous"));
        }
        let bytes = self.data_bytes()?;
        let size = self.size()?;
        let a = ArrayView3::from_shape((size.height as usize, size.width as usize, 3), bytes)?;
        Ok(a)
    }
}

Though, this only works for continuous arrays.

It is implemented in opencv-to-ndarray branch. You can see the usage in test code. Can you check if it works for you?
To run the test,
cargo test --features ndarray_0-15,opencv_0-63 mat_ref_to_array_view_conversion

To include the new future in your crate, add this to Cargo.toml.
[dependencies]
cv-convert = { git = "https://github.com/jerry73204/rust-cv-convert.git", branch = "opencv-to-ndarray", features = ["ndarray_0-15", "opencv_0-63"] }

Hello,
Thanks for the update! Unfortunately, I get a runtime error. Below is the code I'm running:
    let test_img = cv::imgcodecs::imread(
        "./test_img.png",
        cv::imgcodecs::IMREAD_COLOR,
    )?;
    dbg!(&test_img.size()); // Check image is loaded (size should not be zero)
    let nd_mat: ndarray::ArrayViewD<u8> = (&test_img).try_into_cv().expect("not working");

Error message:
[experiment/src/main.rs:36] &test_img.size() = Ok(
Size_ {
width: 5121,
height: 2880,
},
)
thread 'main' panicked at 'not work.: ShapeError/OutOfBounds: out of bounds indexing', experiment/src/main.rs:37:69
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace

When I'm running the test you created I can see that it always add a 1 dim in the end
[experiment/src/main.rs:48] &mat.shape() = [
    31,
    17,
    8,
    13,
    1,
]
[experiment/src/main.rs:48] &mat.shape() = [
    19,
    10,
    1,
]
[experiment/src/main.rs:48] &mat.shape() = [
    19,
    27,
    24,
    8,
    1,
]
[experiment/src/main.rs:48] &mat.shape() = [
    6,
    27,
    1,
]
[experiment/src/main.rs:48] &mat.shape() = [
    2,
    11,
    8,
    1,
]

The shape for a RGB (BGR) image is
[experiment/src/main.rs:85] &test_img.shape() = [
    2880,
    5121,
    3,
]

This is why the test case doesn't detect it

I tracked the issue to mat.as_slice(). It doesn't return the right number of elements for the RGB case.
For u8, replacing "as_slice()" with "data_bytes()" solves the issue (though this will not be generic). I haven't found a CV function that can extend the mat with singleton dimension (as that would essentially solve the issue as the "as_slice" function will work for last channel 1).

The concept of OpenCV's Mat channel is absent in ndarray, tch and many multi-dimensional arrays. In this case, the channel is treated as the last dimension and thus you see one additional dimension there.
You PR is merged and the crate doc is changed accordingly. If you feel comfortable with this change, I will publish it on crates.io.

Sounds good, thanks for your efforts! Love the package.


Retrieving an image service response

Hi @jgravois ,
Thanks for an excellent plugin, it looks very promising.
Im just trying to work out how to retrieve a raster map service as the response.
http://codepen.io/rowanwins/pen/gpRWxW
Im trying to do similiar this example although I only want the raster layer and not the features.
https://developers.arcgis.com/javascript/jssamples/gp_resultmapservice.html
Im just getting a null response back at the moment. Is there a good way to get meaningful errors back from the requests, I've tried error reponse and I just get "Job failed" which doesnt tell me much. Im guessing perhaps I need to do something about with the result object?
Thanks in advance,
Rowan

right now I pretty much only parse GP responses looking for featuresets to convert them to GeoJSON.  that being said, I'm interested in trying to incorporate support for output map services as well.

Ah ok, thanks @jgravois . Certainly map service outputs would be handy. I'll keep on eye on the plugin as it keep growing.

Alright @jgravois I've taken a bit of a look and I think Im making headway, thanks for the encouragement!
Just a couple of quick q's

Should we add an option related to outputType, which defaults to the current setup (geojson), but also accepts MapService?

Are the MapServers always parallel to the GPServers? Basically Im trying to work out if we need an option to set the output MapServer or if we cn just work it out based on the GPServer address...
eg
http://sampleserver6.arcgisonline.com/arcgis/rest/services/911CallsHotspot/GPServer
and
http://sampleserver6.arcgisonline.com/arcgis/rest/services/911CallsHotspot/MapServer

Cheers
Rowan

Gday @jgravois
I've created a fork where I've pulled together some working code that I think is actually half-reasonable...
What I've done

Created a new Task method for setOutputType which Im setting to 'MapService'
In the checkJob function if the outputType is MapService I slightly tweak what happens. The most important thing is I create a  'this.outputMapService' which gives the user a url to use like any other L.esri.dynamicMapLayer
I've created a sample file called mapService-example.html where you can see how it all fits together.

Anyway take a look and see what you think. Happy for any feedback. If you're happy to accept it let me know what my next steps are, I think I need to create a pull request or something?
Cheers,
Rowan
PS This was a great learning experience, for the first time in my life I've actually started to understand breakpoints and the source tab in devtools, watch out world!

very cool!  i'll take a look in the next couple days.

rowan, i have a few suggestions:
1. your work needs to be passed in the callback
for consistency, instead of setting the property this.outputMapService, the url needs to be made available via the parsed response from the API.
callback.call(error, <yourwork>, response);

2. avoid the need to declare the output type as "MapService"
it seems to me like it would be ideal if the API just handled it.

developer tells the API which outputParams to keep track of by name
after a job is completed successfully, the API introspects the result and passes back the appropriate value (boolean, string, MapService, GeoJSON features) nested in a parsed response

for organization purposes, i think the best idea would be to inject your own code flow within processGPOutput().
3. make sure the new code works with synchronous GP services that output a MapService as well.
i haven't worked with one, but i'm sure its possible to design a GP service this way.
for work going forward
the best thing you could do at this point is to create a feature branch to store your work (rather than commiting directly to master) and submit a Pull Request to this repo so that we can continue the discussion.
last, but not least
i really appreciate your help tackling this.  i know it'll take pretty much a complete refactor, but a GP plugin which is capable of returning multiple output types simultaneously from both sync and async services would be a serious improvement.
let me know if anything i mentioned above doesn't make sense.

Hi @jgravois
Thanks for the feedback. I'll have a shot at incorporating the feedback, I had a feeling you might say something along those lines :)  If you can find a synchronous GP Services that spits out a MapService for testing that would be fab.
Cheers
Rowan

just got confirmation that GP Services must be asynchronous in order to output a Map Service, which is nice for us. :smile:

Gday @jgravois ,
Have hit a bit of a brick wall unfortunantlly and need a hand thinking things through.
Working with this service for example
http://sampleserver6.arcgisonline.com/arcgis/rest/services/911CallsHotspot/GPServer/911 Calls Hotspot
The setOutputParam = Hotspot_Raster    which is equal to the parameter 'name' seen here
http://sampleserver6.arcgisonline.com/arcgis/rest/services/911CallsHotspot/GPServer/911 Calls Hotspot?f=pjson
The problem is that to access the MapService I need a layerID, which in this case equals 1
http://sampleserver6.arcgisonline.com/arcgis/rest/services/911CallsHotspot/MapServer/jobs/j0287a516524544409f4a5e9506961963/1?f=pjson
I could iterate through all the available services but the name of the service does not equal the parameter name set above, for example the name of the Service is 'Hotspot Raster' (see URL above).
It looks like the name of the service is contained back on the original page as 'Display Name' rather than 'name'
http://sampleserver6.arcgisonline.com/arcgis/rest/services/911CallsHotspot/GPServer/911 Calls Hotspot?f=pjson
Urgh its a bit convoluted to explain but hopefully that has made a bit of sense! Any suggestions as to how to best proceed?
Cheers
Rowan

to access the MapService I need a layerID, which in this case equals 1

i'm not sure i follow you.  dynamic map services are accessed without referencing the index position of individual layers.  i see that in your own sample 'mapService-example.html' you are only interested in displaying layer 1, but why is that anything the plugin needs to worry about?

I could iterate through all the available services but the name of the service does not equal the parameter name set above,

again.  i don't follow.  why exactly?  map service job output is accessible via a url dependent on the GP Service name and jobid (as you already demonstrate here)
i think i'm probably missing something fundamental (ie: you are attempting to access an entirely different output type than you were previously) so im curious to hear more when you have the time.

Gday @jgravois
Ok I think Im with you. The thing that still kind of seems weird to me is that to run an asynchronous task you need to define a setOutputParam (eg Hotspot_Raster), without that the query wont run (at least as its currently structured), but we are then effectively disregarding that when it comes to the displaying part on the map which just seems a little odd.
Anyway I've rejigged my code and moved stuff down into the processGPOutput() and have got the MapService URL outputting in the callback although Im not sure I've quite structured that part right, it's too late at night to think clearly!
Feedback welcome of course :)
Cheers
Rowan

fixed in 9b4917ea95bffb8e6dd0d6403b05aef744b822c8


Self loops in Tree-NeighborsMatch datasets

Hi Uri,
I hope you're doing well. Your paper is really nice, congrats.
I just have a small question.  What I don't understand is why you add self loops to all nodes. The illustrations in the paper don't show any self loops. I guess the only difference is that you'll be using the own embedding of each node when performing message passing, isn't it? Or is there something I'm missing?
Thank you very much for releasing the code :)

Hi @jhonygiraldo,
Thank you for your interest in our work and for your kind words.
Adding self-loops just performed slightly better, so we added them.
The purpose of these experiments was to show that this simple-looking Tree-NeighborsMatch benchmark is difficult, no matter what tricks are used. So we added self-loops and experimented with many kinds of batch normalization, layer normalization, nonlinearities, residual connections, etc.
Still, no model could perform well starting from a radius of 5.
Let me know if you have any questions!
Best,
Uri

Great, thanks Uri, I understand now :)
Best,
Jhony.


0.6.1. not loading on Mac with Obsidian v0.12.3

Not sure what else I can say, is there some log I can store from the loading process?
The symptom is, whenever I try to activate the plugin by clicking on the switch, it automatically goes off again and a notification appears saying "Failed to load plugin".

Hade the same behaviour. My issue was that the main.js was missing (probably a sync issue on my side I guess).
You can check the console by opening up the Developer Tools (Menu: View > Toggle Developer Tools; Hotkey: ⌘+⌥+i) And go to the "Console" tab. Watch the console while you try to enable the plugin and you should probably see an error message.

0.6.1 requires Obsidian v0.13.0 or above