feat(search-box): implement SearchInputPlaceholder

Implements https://github.com/empathyco/x/issues/385.
Introduces the SearchInputPlaceholder component that shows a placeholder on top of the SearchInput and works as follows:

Shows a static placeholder (passed in as the default slot) when the SearchInput is not focused.
Shows an animated placeholder that loops through the given placeholders when the SearchInput is focused and empty.
Accepts an animation as a prop.

<SearchInputPlaceholder
  :animatedPlaceholders="placeholders"
  :animation="placeholderAnimation"
>
  Find anything
</SearchInputPlaceholder>

Here it is in action:

Motivation and context
Implements an animated placeholder to make the search box more fun.

[ ] Dependencies. If any, specify:
[x] Open issue. If applicable, link: https://github.com/empathyco/x/issues/385

Type of change

[ ] Bug fix (non-breaking change that fixes an issue)
[x] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that causes existing functionality to not work as expected)
[ ] Change requires a documentation update

What is the destination branch of this PR?

[x] Main
[ ] Other. Specify:

How has this been tested?

Tests performed according to testing guidelines:

Unit tests.
Manual tests.

Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review on my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] New and existing unit tests pass locally with my changes.

Good morning @alonsogarciapablo !!
This PR was without action for the last weeks and the issue has been resolved iteratively in these PRs:
https://github.com/empathyco/x/pull/771
https://github.com/empathyco/x/pull/772
https://github.com/empathyco/x/pull/773
https://github.com/empathyco/x/pull/774
https://github.com/empathyco/x/pull/777
Thanks anyway for the effort!

Thank you @herrardo and @mnavarroespinosa and @CachedaCodes for the CR. Sorry I finally didn't invest more time into finishing what I started 😞 . Glad it's already implemented! 🚀


Fix uploading large files from stream

Uploading bigger files uses an upload session to upload the file in smaller chunks. This session is initialized using the DropboxFile path.
When we create a new DropboxFile from a stream (using the "createyByStream" method), the path wasn't correctly set. The filename was set instead, so when we tried to upload a bigger file from a stream, the upload session tried to retrieve the file using the file name, resulting in an error.
This commit fixes this by setting the stream URI as the DropboxFile path.
Note: This'll only work for a stream that you get from tmpfile. For a regular file pinter, it might only give you the basename.

Thanks!


Prompt for entering non defined secret

--no-prompt flag should be passed to surpass the prompt. ( resolves #566 )

Hello @arshul! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file cli/popper/commands/cmd_run.py:

Line 71:80: E501 line too long (111 > 79 characters)
Line 77:51: E231 missing whitespace after ','
Line 85:50: E231 missing whitespace after ','
Line 88:46: E231 missing whitespace after ','
Line 92:49: E231 missing whitespace after ','
Line 93:64: E231 missing whitespace after ','

In the file cli/popper/gha.py:

Line 16:63: E231 missing whitespace after ','

@ivotron while the internal tests run, they are not using --no-prompt flag. What I can do is change the occurrences of all popper run commands in the test/ folder  to use no-prompt flag or we can change the functionality, i.e pass a flag prompt which when passed shows the prompt and not by default.
What are your thoughts on this?

@ivotron any idea why tests do not run on new commits now??

thanks!


Blocks Skype Messages and Calls

I went to add these IPs to my host files (both x64 versions win10 and win7 x64) after using it, I have an issue with my Skype. I cannot receive some of my contact's messages and group calls. The Skype on my mobile seem to have no problem receiving the messages and calls at the same time having the phone connected to the same network. And on top of that, my friend also have experienced it after using the same set of IPs on his Mac.

Are you sure @gonkyouka?
I take several Skype calls every day, and Skype-message all day, for the past couple of weeks at least, using a machine that has the most locked-down version of these hosts files.
Can anybody confirm?

@StevenBlack Have you tried group calls and/or group messaging? I have no luck in both.

add s.gateway.messenger.live.com to your whitelist

@berrythesoftwarecodeprogrammar how do you actually do that?

there should be a file called whitelist in your directory of this script. if there isnt just create a blank file and add that line. when you use updateHostsFile.py it wont add hosts in the whitelist to the final hosts file

Should this whitelist item be added to any documentation?

maybe. there were other issues related to microsoft hosts and i think they were all caused by the tyzbit hosts file. i still think its a useful host file but people should be told what to whitelist to unbreak things maybe

@berrythesoftwarecodeprogrammar oh great thanks!

@Gitoffthelawn the whitelist is documented here.

@tyzbit can I get you to remove s.gateway.messenger.live.com ?
@gonkyouka @berrythesoftwarecodeprogrammar ya I've only been doing 1-on-1 calls so that explains it?

@StevenBlack the only spot where that exists is in the someonewhocares.org host list, and it's commented out at that.

@tyzbit so it is.  Sorry I should have checked that.  Carry on 😄 .
Alright does anyone have more insight on this, cause I don't experience problems with Skype, and surely it ain't s.gateway.messenger.live.com.

@StevenBlack Sorry, I meant perhaps add to the documentation which specific hosts users may want to whitelist in order to allow specific software to work.
Sometimes a host is worthy of being blocked for most people, but blocking it will cause a specific piece of software to stop working.

@Gitoffthelawn  so far I'm not aware of anything to document, including what's discussed here.
The wiki would be a good place for that...

@StevenBlack Yes, your comment about Skype working for you even with s.gateway.messenger.live.com blocked indicates that there could be something else going on for @gonkyouka and @berrythesoftwarecodeprogrammar .  It's a curious issue.  Perhaps everyone involved could specify the version of Skype they are using.

for me, group chats just wouldnt work until i whitelisted s.gateway.messenger.live.com. my messages wouldnt send and i wouldnt recieve any messages. so i dont know

But @berrythesoftwarecodeprogrammar the domain s.gateway.messenger.live.com currently isn't active in this hosts file.  It's commented out.  Everywhere.
Please try with the latest version.

oh, im not even having the problem anymore. i was just saying thats what i had to do in the past for my skype problems. OP needs to tell us if they are still having the issue and if they are using the latest version

Closing this.


11ty plugin seemingly not doing anything

I'm trying the Eleventy integration, and having followed the setup instructions I'm finding that my code blocks are completely unaffected. For example:
    ```scss
    .item {
        --accent-colour: #{$green};
    }
    ```

Will end up as:

.item {
        --accent-colour: #{$green};
    }

I'm not getting any errors.
I'm using 11ty v1.0.2.
Tried Node v12 and v16.

Ignore this. There was a typo in my .eleventy.js. I'm now getting the error in this issue.


Local depository path setting is expecting a filename as opposed to folder

hence the path cannot be changed.

This one has been fixed


Comment Upvotes and Best Answer Feature

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Comment Upvotes and Best Answer Feature
Allow users to upvote comments and display the highest-voted comment at the top to encourage quality engagement.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

we  honestly do not need these things for comments , as a blogging website our current focus is to have more blog related functionalities , we arent a social media website ,thanks but since it not needed , i will be closing this issue


Update denols to support new LSP handler signature

This PR updates the denols config to support the new LSP handler signature in recent Neovim 0.6 nightlies. It maintains compatibility with the signature used in Neovim 0.5 and previous versions of 0.6 nightly.

Can you just add this: https://github.com/neovim/neovim/issues/14090#issuecomment-913198455
to utils and wrap the old handler?

That snippet wraps a new handler to work with v0.5.0 calls. This PR updated the existing v0.5.0 denols handler to work with the new call format. I could update the deno handler to use the new format by default and wrap it.
However, I think the deno handler might need to know whether it's being called with the new or old arguments, because it calls another handler. Simply an updated handler doesn't work in both 0.5 and the nightly because the handler being called by deno (lsp.handlers[method](err, method, result)) may or may not itself be configured to handle both call formats.

Updating the deno handler is easy. The issue I ran into when doing that is that the deno handler calls another handler. That handler isn't wrapped -- if the deno handler calls it using the 0.5.0 arguments, it works in neovim 0.5.0 but breaks in the nightly. If it's called with the new arguments, it breaks in 0.5.0 but works in the nightly.

Well, then that handler should be wrapped, too ;)

I haven't really dug into it, though, so I may be missing something obvious.

It should, yes. 😄

I believe the handler deno is calling, by default, is the one in neovim's runtime, in which case wrapping isn't really an option.

I updated denols_handler to use the new format and added the wrapper to lspconfig.util. I modified the wrapper slightly; it adds an is_legacy_call field to the context if it had to convert the call format. This lets receivers that care, like denols, know how they should call other handlers.

There is already a wrapper in lspconfig.util; please use that instead of adding yet another one.

(You need to rebase on master.)

D'oh, I was rebased on master, but I didn't see the handler wrapper was already there 🤦

I'm using the existing compat_handler, but I added the is_legacy_call to its context. I'm definitely open to not doing that if there's a better way, but I think that somehow the deno handler will have to change its behavior based on what call syntax is used in the current environment.

Wouldn't it be cleaner instead of doing the "is legacy" checks to just check if the user has nvim-0.5.1?

Version checking would work, as long as it's clear what version the new signature should come in at. Checking how the handler is actually being called removes the need to know that, though, so it's less fragile.

In any case, I'm fine with doing a version check if that's preferable.

As I don't use deno, testing and saying this fixes your issue is the fastest way to get this merged :)

@mjlbach 0.6 master  appears to be working fine without it, @jason0x43 am I missing something?
Nvim master made a very important fix adding codeAction/resolve that makes Deno LSP now work properly, so it is very important all is well when using master :)

I haven't checked with the most recent 0.6. I'll take a look...

This PR is necessary to adapt the custom handler overrides that deno uses. You probably aren't invoking a command that would use them

This is still an issue as of NVIM v0.6.0-dev+dd7812ec6 (from Oct 5) with the current master of lspconfig. I'll update neovim to the latest commit and check that.

Ah, what @mjlbach said 😁
Also, I updated the PR to do a version check since 0.5.1 does include the new handler signature (I wasn't aware that was definitely going to be the case when you mentioned it earlier).

Thanks, I just approved the CI run. I'll merge after this, and we'll remove the compatibility shims once 0.6 is released.


registry/proxy: build scripts for raspberry pi

Running modprox on a raspberry pi is fun, provide scripts to build the components for raspian on pi 3's

quick and dirty deploy script for deploying the two components (assumes you've set them up to run in systemd)
https://gist.github.com/shoenig/879fd7f4b01263233959a23f360b254a


Added a Rebus

Associated Issue: #
Summary of Changes

Added a rebus

Pull Request Test Coverage Report for Build 811

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

Totals

Change from base Build 514:
0.0%

Covered Lines:
158

Relevant Lines:
180

💛  - Coveralls


rename lease_time to current_time

Signed-off-by: linning<EMAIL_ADDRESS>
What have you changed?

rename lease_time to current_time.
record current_time after calling RawNode::ready_since to reduce the call to
monotonic_raw_now()

What is the type of the changes?

Improvement (a change which is an improvement to an existing feature)

How is the PR tested?
No
Does this PR affect documentation (docs) or should it be mentioned in the release notes?
No
Does this PR affect tidb-ansible?
No
Refer to a related PR or issue link (optional)
#5440

/run-integration-test

/run-integration-tests

/merge


Support x-accessibility

https://github.com/Azure/autorest.csharp/issues/1134

Will revisit this with CADL in mind


Orient package is incorrect

See https://github.com/philip-wernersbach/nim-orient - the code is in the prerelease branch which is NOT the default, also it's not specified in Nimble (can we even specify a branch in packages.json? If so, I'll make a PR to fix this one).
Running nimble install orient you'll get:
~/P/n/pkgraph ❯❯❯ nimble install orient
Downloading https://github.com/philip-wernersbach/nim-orient using git
       Tip: 1 messages have been suppressed, use --verbose to show them.
     Error: Specified directory (/tmp/nimble_5457/githubcom_philipwernersbachnimorient) does not contain a .nimble file.

P.S.: Found when working with my pkgraph "project".

CC @philip-wernersbach

Still an issue

feel free to fix it

Not an issue with the package list and it's unlikely the author will ever fix this.


[API-3485]:: PUB_SUB gem: AWS::SNS fails to publish when OS username has a '.' in it

Issue:
API-3485
Proposed Changes:
Following are the required changes:

In lib/pub_sub.rb, update the env_suffix method to use the sanitize_for_aws method to ensure only AWS allowed characters are passed

Impact:

Will this introduce any externally facing changes(endpoint, fields, validation, query)? No
Any special deployment coordination needed(env changes, data/db migrations, etc.)? No
Urgency(low, medium, high)? Medium

Requirements:

Test the behaviour in UAT after it's been merged, then add the "Tested in UAT" label.
Test the behaviour in production if possible, then add the "Tested in Production" label

@adamcohen, @gwshaw, @kislay100

Can I be taken off these threads please?

Please update pub_sub/lib/pub_sub/version.rb to 1.5.1 before merging.

@gwshaw updated the version.


Test shape of output from to_rna

Hi, I'm a beginner to J and tried the rna-transcription exercise. My first imperative attempt started with an empty string and did a while loop appending individual transcribed characters to the output.
transcribe =: monad : 0
out =. ''
select. y
case. 'G' do. out =. 'C'
case. 'C' do. out =. 'G'
case. 'T' do. out =. 'A'
case. 'A' do. out =. 'U'
end.
out
)

to_rna=: monad : 0
out =. ''
count =. # y
i =. 0
while. i < count do.
 out =. out , transcribe i { y
 i =. i + 1
end.
out
)

Visually this prints expected results but the shape is not preserved when the input is only one character.
   $ 'C'           NB. empty print is expected https://www.jsoftware.com/help/primer/shape.htm

   $ to_rna 'G'
1

It took me a while to understand that this is the reason tests 2, 3, 4, and 5 failed. Would it help if there was a test for preserving shape of input?

@eNascimento178 Something for you to consider.

I will choose the -:&, option for this case and, at a later moment, include a message in the analyser highlighting the issue with the shapes after the student creates the solution plus some hints If stuck.


Custom order query

resolves #115

Coverage decreased (-2.0%) to 93.777% when pulling 7a88f5d5d144c749afda814de33d16f127db9e80 on custom-order-query into a748e25d6f89fc3fbadf9520e500a2b2dc0b834d on main.


Question about texture of inserted objects

I have a model of a chair, and I am using the object insertion capability to add the model to a test scene.  The texture of the synthetic object does not look very good.  Basically, there is almost no texture at all.  I am wondering if I am doing something wrong.  Maybe I need to add a lighting setting or something?  Attached is a screenshot of the glb file in an online viewer and the image generated by Habitat for the same object.

I also noticed that the feet on the chair are not being rendered.

Hey Daniel, you are correct that there is a missing shader issue here. We have a feature in the works to address this. PR #152 shows the current state of that effort, but we intend to improve on this to support additional shaders and settings in the coming months. For now, if you need non-trivial shading via illumination you can check out some of the changes in the clutter_nav branch.
Specifically: src/esp/gfx/GenericDrawable.cpp shows lighting setup in the shader draw.
Happy to chat more about this and help you get a working solution while we get this feature locked down.

Related to Issue #319

I have a new prototype branch to address this issue: phong_shaded_physics_objects. With this branch all scenes are flat shaded via textures (as in master), but objects you add with PhysicsManager are rendered with a Phong illumination shader. The default setup is 8 lights placed at the scene bounding box corners. You can also set the position, color, and intensity of the lights in python (see example of this functionality). As a note, adjusting light intensities [0,1] for all 8 lights is likely the best course of action to immediately use this functionality for new scenes (default is 0.2).
A more complete implementation to come, but this should do what you need presently. Let me know how it goes!


how to leaflet add wms layer ?

An example of leaflet on javascript library is avialable for your reference at http://jsfiddle.net/devmgs/t3u7xszy/14/

I second this request, as all my available map services are either WMS or WFS services, and very rarely WMTS (tiled). This would be an invaluable addition to this plugin - and unfortunately currently a deal breaker for my application. Please plan to support WMS services. Thank you very much in advance.

@nesnejretep we accept pull requests! If you would like to try to implement this I am happy to provide guidance. It looks like you could extend the TileLayer class and make the modifications in the file linked by @devmgs


ExpressJS backend added for RESTful CRUDs.

The title basically explains the changes. An Express server was added along with some routing rules, so now all CRUDs on the models are through RESTful requests.

Very cool, thanks for contributing!


Fix sparse dot according to the new RSP definition

@eric-haibin-lin Please review.

csr.T, dns => rsp is fine. This is used for backward gradient.
For forward, it should be csr dns -> dns

Could you also complete the fallback call for DotForwardEx / DotBackwardEx?

also could you move NDARRAY_IDX_TYPE_SWITCH to a common place, not in ndarray.h?

Yes we need a workaround, maybe just explicitly copy the nnr value back to
CPU for Alloc. But we do need such function
On 2017年5月18日 周四 at 20:45 reminisce<EMAIL_ADDRESS>wrote:

@reminisce commented on this pull request.
In src/operator/tensor/elemwise_unary_op.h
https://github.com/eric-haibin-lin/mxnet/pull/35#discussion_r117398483:

 if (0 == nnr) {

   rsp->SetAuxShape(rowsparse::kIdx, TShape(mshadow::Shape1(0)));

   return;  // zero matrix

 }

 rsp->CheckAndAllocData(mshadow::Shape2(nnr, num_cols));

 // TODO(junwu): single thread for compressing row_idx and copying data

 // from dns to rsp, might be a bottleneck.

 auto in_tensor = dns.FlatTo2D<xpu, DType>(s);

 auto out_tensor = rsp->data().FlatTo2D<xpu, DType>(s);

 int last_nnr_id = -1;  // last non-zero row id

 for (index_t i = 0; i < num_rows; ++i) {

   if (row_idx[i] < static_cast<RType>(num_rows)) {  // non-zero row found

     row_idx[++last_nnr_id] = row_idx[i];

     mshadow::Copy(out_tensor[last_nnr_id], in_tensor[i], s);

   }

 }

But this code needs access to TBlob data, which does not work when those
data are on GPUs.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/eric-haibin-lin/mxnet/pull/35#discussion_r117398483,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFSeqMAtivfclMjLBjNl0P15JVLchtv7ks5r7RBHgaJpZM4NetIU
.

--
Best Regards,
Haibin Lin
Department of Computer Science
School of Computer Science
Carnegie Mellon University


Chromedriver executable path

Hi,
selenium.common.exceptions.WebDriverException: Message: 'chromedriver.exe' executable needs to be in PATH. Please see https://sites.google.com/a/chromium.org/chromedriver/home
Wanted to apply the fix with this [https://stackoverflow.com/questions/29858752/error-message-chromedriver-executable-needs-to-be-available-in-the-path]
P.S. Thx for adding the example IG. Will try to submit other PRs for examples

1- Go to https://sites.google.com/a/chromium.org/chromedriver/home
2- Click latest stable release
3- Download win32 or use another version for your operating system.
4- Copy chromedriver.exe or chromedriver executable file to suitable path. if you use Windows, default path is C:/bin/chromedriver.exe. Else default path is /usr/local/bin/chromedriver.
You can change chromedriver path with set_chromedriver_path in code.
from tweetcapture import TweetCapture
import asyncio
tweet = TweetCapture()

tweet.set_chromedriver_path("/usr/bin/chromedriver")

asyncio.run(tweet.screenshot(
    "https://mobile.twitter.com/jack/status/20", "test1.png", mode=3, night_mode=2))

Thx for the quick responses :)
Still doesn't work.

I've downloaded the right version of ChromeDriver added to both the main folder of the script and tweetcapture folder. I also added it to PATH in windows - still doesn't work.
From the StackOverflow post it seems the only workaround is the webdriver manager.
Also in Windows there is no C:/bin, I think that's in in Linux. And where are you using set_chromedriver_path

Make sure you have the latest version of Google Chrome installed. Chromedriver needs Google Chrome.
C:/bin/chromedriver.exe is default path for this program. Windows doesn't have it. Create folder C:/bin and copy chromedriver.exe to the folder. If you will use default path, don't use set_chromedriver_path in your code.
If you won't use default path, use set_chromedriver_path in your code and define custom path. But be careful when using slashes. C:\Users\user\chromedriver.exe is incorrect. Use / or \ as slashes.
e.g.
C:\Users\user\chromedriver.exe
C:/Users/user/chromedriver.exe

I cd into the examples to run the insta script tho it is posting the generic jack tweet.
Back to the previous comment, how to connect this to another tweet URL and  for every new tweet (based on.)...
Thx


argparse -- try to resolve overload ambiguity

Signed-off-by: Larry Gritz<EMAIL_ADDRESS>

abandoning


Allow writing an empty tag

This just allows writing an empty tag. There isn't really a reason to deny this is there?

Actually, would it make more sense to just do remove_from and nothing more?


Move CODE_OF_CONDUCT and CONTRIBUTING community health files

These are now at 1. Individual projects may override e.g. with their
own CONTRIBUTING.md file, like 2.
Signed-off-by: Sebastian Schuberth<EMAIL_ADDRESS>

And should it say "Remove" instead of "Move"?

I was considering that, but wanted to emphasize that the files were not completely removed, but actually moved to a different repository.


Pull request for gmsh

For travis-ci/travis-ci#4427.
Ran tests and found no setuid bits.
See https://travis-ci.org/travis-ci/apt-whitelist-checker/builds/72062165

Sorry, but this test will have to be re-run. Will open a new one later.


using a getter result for mat-select options causes an infinite loop

What are you trying to do?
When I use a getter in some component to provide the options for a mat-select in a child component it causes an infinite loop. The child component ngOnChanges is being called all the time.
What troubleshooting steps have you tried?
I didn't expect to create an infinite loop because if I use a simple select it works as expected.
Reproduction
https://stackblitz.com/edit/components-issue-civfkw?file=app%2Fchild.component.ts
Steps to reproduce:

Uncomment the block to use the mat-select.
The application became unresponsive 😞

Environment

Angular: ^8.0.0
CDK/Material: ^8.0.0
Browser(s): Google Chrome
Operating System macOS

That's because the options getter returns a new array every time change detection is run so Angular thinks that the data has changed and it'll keep updating the view. You put the array into a regular property.

@crisbeto I understand the Angular behavior you mentioned but I don't get why mat-select and select behave differently. The problem here is the mat-select is triggering the change detection somehow while the select is not doing that. Do you have any idea why this could be happening?


ROX-17926: Extract common code into sub template

Description
Extract PG filter generation.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
CI – generated files were not changed

/retest


ion-card as directive

Ionic version:  (check one with "x")
[x] 3.x
I'm submitting a ...  (check one with "x")
[x] feature request
Feature Request:
Just as ion-item can be used as a directive on a button:
<button ion-item *ngFor="let item of items" (click)="itemSelected(item)">
  {{ item }}
</button>

I'd like to be able to use ion-card as a directive:
<button ion-card *ngFor="let card of cards" (click)="cardSelected(card)">
  <ion-card-header>{{card.title}}</ion-card-header>
  <ion-item>{{card.someProperty}}</ion-item>
  <!--etc.-->
</button>

Then we could use cards without worrying about adding tappable, but my hope would be that the ripple effect would work across the whole card area.

Actually, if you want to check if the user has clicked, tapped or done something on the card, you can use the event (click) directly on the card as shown bellow:
<ion-card class='card'  class="movie-info" (click)='itemSelected(item)'>
</ion-card>

Checkout my example add for more details:

https://github.com/vitorsilvalima/ionic-tmdb

Thanks for the suggestion, but I am aware that I can put (click) on the card. That's why I mentioned the tappable attribute, which should be added to your example to remove the 300ms delay.
<ion-card tappable (click)='itemSelected(item)'>
</ion-card>

The point of this request is for "first class" support of cards as buttons. As I mentioned, the ripple effect would be nice. We would gain improved accessibility as well.

Hello everyone! Thanks for the feature request. I'm going to move this issue over to our internal list of feature requests for evaluation. We are continually prioritizing all requests that we receive with outstanding issues. We are extremely grateful for your feedback, but it may not always be our next priority. I'll copy the issue back to this repository when we have begun implementation. Thanks!

This issue was moved to ionic-team/ionic-feature-requests#276


Improve the control interface usage examples

Description
The Control Interface usage examples currently do not show how to deal with the response message of an update request.
Additionally one method called "write.. " is updating and reading the current state.
We should update the examples, s.t. it is clear how to use the control interface. Additionally the example in the documentation has to reflect reading the update request response message. It would be also good to show kind of in a sequence that each request has its response and that ids are matching.
Goals

Update all Control Interface usage examples:
** to reflect the request response model
** to show better the reading of the complete state by separating it into its own method.
User documentation:
** Review the user documentation regarding the control interface usage
** Reflect the request-response model there too
** Add a sequence diagram showing how an update is performed
** add something about the IDs and that they shall be unique in case of parallel calls

Final result

Summary
To be filled when the final solution is sketched.
Tasks

[ ] Task 1
[ ] Task 2
[ ] ...

This should be done after the SDKs are available #263  and #304.


with_clblas seems broken

When running with with_clblas flag enable I get the following error:
/nix/store/gl7h4kwkjh2vzqi8y87gd4nsymd2wah9-local-ai-2.9.0/bin/local-ai: error while loading shared libraries: libOpenCL.so.1: cannot open shared object file: No such file or directory
This might be user error, I'm on NixOS and have these options in my config:
  hardware.opengl.extraPackages = with pkgs; [
    rocmPackages.clr.icd
    rocmPackages.rocm-runtime
  ];
  hardware.opengl.enable = true;
  systemd.tmpfiles.rules = [
    "L+    /opt/rocm/hip   -    -    -     -    ${pkgs.rocmPackages.clr}"
  ];

Thanks, I fixed the issue with commit:
https://github.com/ck3d/nix-local-ai/commit/ea9d3e735103c5582ae7301a797bc7af61938557


Upgrade toolchain channel to 2022-01-05

Fixes https://github.com/hermitcore/libhermit-rs/issues/321.

bors try

bors r+


[Fix #4106] Make Style/TernaryParentheses unsafe autocorrect detector smarter

This cop, despite checking for unsafe autocorrects, did not recognize methods sent to constants or literals, leading to inappropriate omission recommendations.
For example, this was correctly identified as being unsafe to correct:
(foo.bar baz) ? 1 : 2

but none of these were:
(Foo.bar baz) ? 1 : 2

("foo".bar baz) ? 1 : 2

This change fixes that.

Before submitting the PR make sure the following are checked:

[X] Wrote good commit messages.
[X] Commit message starts with [Fix #issue-number] (if the related issue exists).
[X] Used the same coding conventions as the rest of the project.
[X] Feature branch is up-to-date with master (if not - rebase it).
[X] Squashed related commits together.
[X] Added tests.
[X] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[X] All tests are passing.
[X] The new code doesn't generate RuboCop offenses.
[X] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[X] Updated cop documentation with rake generate_cops_documentation (required only when you've added a new cop or changed the configuration/documentation of an existing cop).

👍


Naming of Dotnet variables Business Central AL Language Source Control

The naming of  DotNet Variables on prem is very strange
example:
MyDateTime: DotNet MyDateTime; //AL(AA0072) Warning
"System.DateTime": DotNet MyDateTime; //Not warning
As it seems wrong to me to declare the variable in this way ("System.DateTime"), I would give it the same name as the alias (MyDateTime).

This might explain what https://github.com/microsoft/AL/issues/6422 was trying to determine, i.e. what the AA0072 CodeCop currently considers a "valid" name for .Net objects.

The only way to bypass the warning at the moment is by using the name of the assembly, it seems that the code analyzers don't use the alias as the expected name.
There is an extended example:
dotnet
{
    assembly(mscorlib)
    {
        type(System.DateTime; MyDateTime){}
    }
}

codeunit XXX MyCodeunit
{
    trigger OnRun()
    var
        MyDateTime: DotNet MyDateTime; //This is showing the AL(AA0072) Warning
        "System.DateTime": DotNet MyDateTime; //This is not showing any warning
    begin
    end;

}

Thanks for that info. I'd propose that #6422 be closed as a dupe of this, since this thread got the answer on what the current preferred naming rule is and what it should be instead.

Agreed. This sucks

Naming should follow the regular rules and not be special for DotNet

In that case I would just #pragma out the warning for sections of code using .Net variables, rather than follow the (accidental?) convention with protest ;-)

Hello dnpb,
can you please write a code sample, how to use #pragma to suppress the warning in a special region?
Thanks
S-Keim

#pragma warning disable AA0072
// code for which you don't want / care about warnings here
#pragma warning restore AA0072

Thanks for reporting this issue. Sorry we haven’t completed it yet, but we’ve had to prioritize elsewhere. We’re planning to give the CodeCop engine and its rules an overhaul in a future major release. Thanks for your patience.
For this case consider using Pragma Warning Directive.


Fix issue #155 "Travis-ci configuration outdated"

Migrate from legacy to container-based infrastructure for faster builds adding sudo: false
Resque the android/debug.keystore from Iosched 2014 for CI usage using wget.
Update tools.
Travis-ci runs ./gradlew build connectedCheck by default, qualityassurance buildType requires new proguard rules and server tests run and fail on build, so I suggest only assemble by now.

Fixes 1..5 points of issue #155,  included to use default android-wait-for-emulator script. The custom script used by the 2014 version was removed.


Possible to load .dll files?

I realize that the environment is likely in sandbox mode, but is it possible? I'm trying to run (narrator  which requires lpeg but it doesn't seem to search for dlls.

Hi,
I remember trying to import some other lua files from the scripts folder, and if i remember correctly that might work, but i am not sure about luarocks packages,
I would post an issue directly on rlua's repo since I basically just host the scripts directly with their library, so if it's possible there it's possible here!

with mlua, I can vouch that loading shared libraries works. I made a dll (.so on linux) with rust (using C abi) and successfully required it with a mlua script.
the lua environment technically can't be sandboxed anyway :D there's a lot of unsafe things happening behind the scenes between luajit + rust + lua combo. it is sort of possible with luau backend (roblox version of lua) of mlua as that is explicitly designed for sandboxing.
maybe with a wasm based lua vm or https://github.com/danii/hematita we can get proper safe sandboxing in future.
you can use (and also create) luarocks packages, but you need to manage them externally and add the path + cpath of luarocks install target location to the mlua vm.

I am planning on swapping to mlua anyway, or rather tealr for documentation generation features, so if rlua does not allow for external modules this won't be a problem for too long. I'll investigate this once i am done working on the bevy script facing API!

both mlua and rlua don't allow loading C libraries by default. They also don't allow loading the debug module by default.
You can enable both of them to allow this, but this requires unsafe functions to create the lua instance as both C libraries and the debug module can result in UB due to circumventing Rust's borrow rules and/or messing with state that mlua and rlua depend on.
So, in both cases it is less about sandboxing, but more as an attempt to make sure that no UB happens through their safe api. I believe there is an issue on rlua about it, questioning if this design is not more harmful than useful.

Hmm in this case I might put that functionality behind a feature to make it opt-in. Thanks for the info @lenscas!

May I suggest another name for that feature? Considering that enabling it presumably also enables the debug module. It is probably also good if the feature name itself makes it clear that ub can happen if it is enabled.
unsafe_lua_instance at least makes the fact that it uses unsafe clear but stays a bit vague. Though that might be a good thing to hopefully get people to read the docs so they know why it is unsafe?

That's probably a good idea, i will name it unsafe_lua_modules  though to imply clearly what functionality is enabled with the feature


Please reconsider AGPL - weights are Apache 2.0 but code is AGPL?

You have an incredible project; however, the licensing needs clarification. The weights on HuggingFace are under Apache 2.0, but you marked the code for CharacterGen as AGPL? Is this your intention?
This makes using the code for CharacterGen legally tricky for developers that want to use your weights.
CharacterGen seems most useful as a library in other larger 3D projects. ComfyUI-3D-Pack includes it. However, technically this is a violation of the AGPL as ComfyUI-3D-Pack is MIT, which is incompatible. AGPL is a viral license so CharacterGen would force ComfyUI-3D-Pack to be AGPL. Since the author of ComfyUI-3D-Pack licensed his project as MIT and has incorporated your code, he has technically violated your license.
It would be better for the ecosystem as a whole for you to license CharacterGen under MIT or Apache 2.0; however, if you do decide to keep the code under AGPL, please clarify the license of the weights.

Thanks for your advice. We want this project can be used in more projects. So we change the license in commit 02453103ecee16a8fd367c193e23bec1c163dfc7. :)


Possible 'ifBlock' helper

Basically, I'd like to include extra HTML in my layout conditionally, based on whether a particular {{#content}} has been included on a page.
I wrote a little helper that extends your helpers:
// Include Assemble & Helpers
var assemble = require('assemble');
var handlebars = require('handlebars');
var layouts = require('handlebars-layouts')(handlebars);

handlebars.registerHelper('ifBlock', function (names, options) {
  actions = this._layoutActions || {};
  names = names.split('||');
  for (var i=0; i < names.length; i++) {
    var name = names[i].trim();
    var action = actions[name];
    if (action) {
      return options.fn(this);
    }
  }
});

a use example would be:
layout--a.hbs
---
title: layout--a
---
{{#extend "master"}}
  {{#content "master__head" mode="append"}}
    {{#block "head"}}{{/block}}
  {{/content}}
  {{#content "master__body"}}

        {{#ifBlock "region--a"}}

        {{#block "region--a"}}{{/block}}

        {{/ifBlock}}
        {{#ifBlock "region--b"}}

        {{#block "region--b"}}{{/block}}

        {{/ifBlock}}

  {{/content}}
  {{#content "master__foot" mode="append"}}
    {{#block "foot"}}{{/block}}
  {{/content}}
{{/extend}}

page.hbs
---
title: Page
---
{{#extend "layout--a"}}
  {{#content "region--a"}}

content

  {{/content}}
{{/extend}}

...The output of page.hbs would not include the
 markup as I did not use {{#content "region--b"}}
I wondered if this might be worth including as part of your helpers?

As discussed in #17, I've added support for {{#if @content.main}} into the develop branch. It will be released as soon as I have time to update the documentation.

Released as 2.0.

All seems to be working great with this code:
---
layout: layout--a
---
{{#extend "master"}}
  {{#content "master__head" mode="append"}}
    {{#block "head"}}{{/block}}
  {{/content}}
  {{#content "master__body"}}

      {{!-- {{#or @content.region--a @content.region--b @content.region--c}} --}}
      {{#or @content.region--a @content.region--b}}

        {{#if @content.region--a}}

            {{#block "region--a"}}{{/block}}

        {{/if}}
        {{#if @content.region--b}}

            {{#block "region--b"}}{{/block}}

        {{/if}}
        {{#if @content.region--c}}

            {{#block "region--c"}}{{/block}}

        {{/if}}

      {{/or}}

  {{/content}}
  {{#content "master__foot" mode="append"}}
    {{#block "foot"}}{{/block}}
  {{/content}}
{{/extend}}

I just need to find a helper that allows to to do {{#or}} on more the two items
any suggestions @shannonmoeller ?

@spacedawwwg The syntax is a little gross, but you can actually nest helpers using subexpressions.
{{#or @content.region--a (or @content.region--b @content.region--c) }}
    ...
{{/or}}

Seems or is not playing nicely with subexpressions. I'll keep looking, but thanks!

You could write your own. :)
// helpers/compare.js
module.exports = {
    both: function(a, b) {
        return a && b;
    },
    either: function(a, b) {
        return a || b;
    }
};

{{#if (either @content.region--a (either @content.region--b @content.region--c)) }}
    ...
{{/if}}

@spacedawwwg Try ifAny: https://github.com/assemble/handlebars-helpers/blob/v0.6.0/lib/helpers/comparisons.js#L282

Thanks @shannonmoeller , that is exactly what I needed!!
...Though, seems to be broken in Assemble.
BUT, this guy sorted it:
https://github.com/assemble/handlebars-helpers/pull/165
so for now I'm simply using:
handlebars.registerHelper('eitherOr', function () {
  var options = arguments[arguments.length - 1];
  for (var i = 0; i < arguments.length - 1; i++) {
    if (arguments[i]) {
      return options.fn(this);
    }
  }
  return options.inverse(this);
});

Which will get me by. Thank you again though!!

Awesome!

@spacedawwwg An issue was submitted related to the @content variable. The resolution was to switch back to a helper-based solution. As such, the content helper is now a boolean getter when used as a subexpression. So the above ifAny example becomes:
{{#ifAny (content "region--a") (content "region--b") (content "region--c")}}
    ...
{{/ifAny}}

It's a breaking change that I released as 3.0. Since you're one of the two people I know are using this feature, just wanted to give you (and any googler's of this thread) a heads up.


[DeepSpeed Inference] init_inference() fails for diffusers with the latest stable versions

Enviornment

CUDA: 11.6
Python: 3.8.10
PyTorch: 1.13.1
DeepSpeed: 0.8.1
Transformers: 4.26.1
Diffusers: 0.13.0
GPU: T4

Code
"""
Simple script to benchmark Stable Diffusion optimized with DeepSpeed inference.

References:
	(1) https://github.com/microsoft/DeepSpeed-MII/blob/main/mii/models/load_models.py
	(2) https://github.com/huggingface/diffusers/pull/2303 (benchmarking function)
"""
import os

import deepspeed
import torch
import torch.utils.benchmark as benchmark

from diffusers import StableDiffusionPipeline

def benchmark_torch_function(f, *args, **kwargs):
    t0 = benchmark.Timer(
        stmt="f(*args, **kwargs)", globals={"args": args, "kwargs": kwargs, "f": f}
    )
    return round(t0.blocked_autorange(min_run_time=1).mean, 2)

# general parameters
world_size = int(os.getenv("WORLD_SIZE", "1"))
prompt = "A photo of an astronaut riding a horse on marse."
weight_dtype = torch.float16
num_inference_steps = 50
batch_size = 2

# deepspeed specific parameters
ds_kwargs = {
    "mp_size": world_size,
    "dtype": weight_dtype,
    "replace_method": "auto",
    "enable_cuda_graph": True,
    "checkpoint": None,
    # "config": None,
    "training_mp_size": 1,
    "replace_with_kernel_inject": False,
}

# load original pipeline
model_id = "CompVis/stable-diffusion-v1-4"
pipeline = StableDiffusionPipeline.from_pretrained(
    model_id, torch_dtype=weight_dtype, safety_checker=None
).to("cuda")
pipeline.set_progress_bar_config(disable=True)

# benchmark
print("Running benchmark with vanilla pipeline...")
f = lambda: pipeline(
    prompt, num_inference_steps=num_inference_steps, num_images_per_prompt=batch_size
).images
time_vanilla = benchmark_torch_function(f)

# optimize the pipeline with deepspeed
engine = deepspeed.init_inference(getattr(pipeline, "model", pipeline), **ds_kwargs)

if hasattr(pipeline, "model"):
    pipeline.model = engine

# benchmark
print("Running benchmark with DeepSpeed pipeline...")
f = lambda: pipe(
    prompt, num_inference_steps=num_inference_steps, num_images_per_prompt=batch_size
).images
time_ds_inference = benchmark_torch_function(f)

# log information
print(f"Vanilla Stable Diffusion:         {time_vanilla} s")
print(f"DeepSpeed Stable Diffusion:       {time_ds_inference} s")

Error
[2023-02-20 06:28:39,725] [INFO] [logging.py:75:log_dist] [Rank -1] DeepSpeed info: version=0.8.1, git-hash=unknown, git-branch=unknown
[2023-02-20 06:28:39,726] [WARNING] [config_utils.py:74:_process_deprecated_field] Config parameter replace_method is deprecated. This parameter is no longer needed, please remove from your call to DeepSpeed-inference
[2023-02-20 06:28:39,726] [WARNING] [config_utils.py:74:_process_deprecated_field] Config parameter mp_size is deprecated use tensor_parallel.tp_size instead
[2023-02-20 06:28:39,727] [INFO] [logging.py:75:log_dist] [Rank -1] quantize_bits = 8 mlp_extra_grouping = False, quantize_groups = 1
Traceback (most recent call last):
  File "benchmark_sd_ds_inference.py", line 58, in <module>
    engine = deepspeed.init_inference(getattr(pipeline, "model", pipeline), **ds_kwargs)
  File "/opt/conda/envs/deepspeed/lib/python3.8/site-packages/deepspeed/__init__.py", line 311, in init_inference
    engine = InferenceEngine(model, config=ds_inference_config)
  File "/opt/conda/envs/deepspeed/lib/python3.8/site-packages/deepspeed/inference/engine.py", line 137, in __init__
    parser_dict = AutoTP.tp_parser(model)
  File "/opt/conda/envs/deepspeed/lib/python3.8/site-packages/deepspeed/module_inject/auto_tp.py", line 68, in tp_parser
    assert AutoTP.supported(model), "Automatic policy not supported for model. Please provide policy."
  File "/opt/conda/envs/deepspeed/lib/python3.8/site-packages/deepspeed/module_inject/auto_tp.py", line 35, in supported
    if key.group(1).lower() in unsupported:
AttributeError: 'NoneType' object has no attribute 'group'

What am I missing out on?

Any update on this?

Hi @sayakpaul,
Our latest DeepSpeed release fixed Stable Diffusion, which now works with the latest Torch, diffusers, and Triton versions:
https://github.com/microsoft/DeepSpeed/releases/tag/v0.12.3
Can you please try using this release and see if your issue persists? We also have a Stable Diffusion example here:
https://github.com/microsoft/DeepSpeedExamples/tree/master/inference/huggingface/stable-diffusion
Thanks,
Lev

Hi @sayakpaul,
Our latest DeepSpeed release fixed Stable Diffusion, which now works with the latest Torch, diffusers, and Triton versions: https://github.com/microsoft/DeepSpeed/releases/tag/v0.12.3
Can you please try using this release and see if your issue persists? We also have a Stable Diffusion example here: https://github.com/microsoft/DeepSpeedExamples/tree/master/inference/huggingface/stable-diffusion
Thanks, Lev

The latest version now (deepspeed and diffusers), reports errors when inference

ing with multiple cards:'StableDiffusionPipline objects has no attribute  chrildern'


Unwanted markdown conversion in Section title

Description / Beschreibung
We have a user that set a numbering on the section titles of a catalogue, eg. "1. Title".
This is now with markdown converted in to a
 list element and appears strangely in the interview.
Section and other element titles don't need a list element in their markdown conversion I would guess.
Should we specify and strip certain html elements depending on which rdmo element field does the conversion?
Otherwise, can we add an ignore string to the conversion method?
Any other suggestions?
Expected behaviour / Erwartetes Verhalten
Have a section title with text  "1. Title" be displayed normally .
Steps to reproduce / Schritte zum Reproduzieren

add a section title with text  "1. Title"
go to interview and see markdown converted
 in the section title

Context / Kontext
Version 2.2.1
References / Verweise

rdmo/projects/templates/projects/project_questions_head.html

rdmo/projects/templates/projects/project_questions_navigation.html

https://github.com/rdmorganiser/rdmo/blob/1eadee51947399601d85e476b261391826e0f95a/rdmo/core/utils.py#L371

https://daringfireball.net/projects/markdown/syntax#list

Ordered lists use numbers followed by periods:

1.  Bird
2.  McHale
3.  Parish

It’s important to note that the actual numbers you use to mark the list have no effect on the HTML output Markdown produces.
The HTML Markdown produces from the above list is:

Bird

McHale

Parish

Hmm, it makes no sense to have ol (or ul) in the titles or the text of a question, but in a help text it makes sense. I am a bit hesitant to create different markdown workarounds for different fields and different markdown features. The easiest solution would be to escape the syntax: 1\. Title.


Typings

hey @andyrichardson thanks for putting this together. Any reason why the Verify function is typed as any? Also, mind if I put up a PR to try and fix that typing as well as export Verify and Options?

@andyrichardson put up a PR. Just a heads up, when I run npm i on master, I get changes to the package-lock.json. You might need to run npm i and then commit that change.

Marking as done :+1: Thanks for the contribution


Would it be possible to add [System.Collections.SortedList] as a valid type for -Entity

Hi,
Would the type [System.Collections.SortedList] break stuff if allowed as Entity type ?
Rationale being to NOT having to cast to [hashtable] when calling.

Great question! I've been tempted to have another look at the casting logic, it's definitely a point of improvement. Good idea 👍


Fixed README.md example program

The example program does not currently run because there is no function called Create(). Change Create() to ShowCreation() which runs on the pip version.
Changelog / Overview

Reviewer Checklist

[ ] The PR title is descriptive enough
[ ] The PR is labeled correctly
[ ] The changelog entry is completed if necessary
[ ] Newly added functions/classes either have a docstring or are private
[ ] Newly added functions/classes have tests added and (optional) examples in the docs
[ ] Newly added documentation builds, looks correctly formatted, and adds no additional build warnings

We changed ShowCreation to Create via #1124 -- it just doesn't work in the latest released version yet. We'll release v0.5.0 in a few hours though, then this will be resolved.

Sorry, for the quick close -- and thank you for your contribution nonetheless! In this case, the current version of the README is intentional though; it reflects the version of the library on the development branch.

No problem.  I was just happened to come across this project today and couldn't seem to get the example running, so I dug around a bit. Thanks for the quick response.


Misleading and out of date information

This documentation relating to legal entity information being bootstrapped onto entities is now out of date with the new Duel Write set up solution packages provided in GA since April and needs updating to reflect the current solution.

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

ID: 0891e384-80e8-5d40-8bde-24081e06876a
Version Independent ID: b6ecc0cb-e827-80aa-934d-ef03d674c97e
Content: Bootstrap with company data FAQ - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/data-entities/dual-write/bootstrap-company-data.md
Service: dynamics-ax-applications
Product: ****
Technology: ****
GitHub Login: @RamaKrishnamoorthy
Microsoft Alias: ramasri

Thanks for the feedback. @RamaKrishnamoorthy can you please take a look?

BUG 473607 created to track the issue.

Company bootstrapping is required for initial data a CE system contains before enabling dual-write. So, this document is not out -dated. However, we are coming with more bootstrapping options and guidelines by mid August.


Translate: refs-must-have-owner && invalid-aria-prop && dont-call-proptypes

refs-must-have-owner
invalid-aria-prop
dont-call-proptypes

Relates to https://github.com/reactjs/es.reactjs.org/issues/4

We're closing this issue to reduce the noise in the repo. The place to claim an issue is #4
Thanks! 👍


Issue test

try to know how to use zhe ge po wan yier

try to know how to use zhe ge po wan yier


Adapt colorscheme  for pydata-sphinx-theme>=0.9

pydata-sphinx-theme 0.9 has been released recently and it has changed it's default styling, which breaks our custom styling, in particular the heading colors.
We've now pinned to pydata-sphinx-theme>=0.9, so we need to adapt our customization to this.
Please also check whether there are any other obivous changes in the new theme version.

I'm not sure we want to use 0.9 yet, to be honest.  I am not sure how to wrap it with this theme, and it has a bunch of css-theme related stuff that is all marked beta and subject to change.  https://pydata-sphinx-theme.readthedocs.io/en/stable/user_guide/. Maybe @choldgraf has some suggestions for how we should proceed.

OK, Maybe not as bad as I thought.  I'd misunderstood how the inheritance works...

Closed by #33.


RN: unable to resolve react-dom on 4.0.0-rc3

are you requiring from mobx-react/native ?
Op vr 18 nov. 2016 13:26 schreef Vojtech Novak<EMAIL_ADDRESS>
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mobxjs/mobx-react/issues/162, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABvGhN_hRBthbcEsu-nByni3LhnuPxcPks5q_ZldgaJpZM4K2beB
.

Yes, I am. I did npm i of mobx-react rc3 and got this error. I looked at the native.js file and saw "react-dom" directly in there, around line 140 if i remember correctly.

@vonovak ok thanks for reporting, I'll investigate it! It should be stubbed :(  https://github.com/mobxjs/mobx-react/blob/83e7e93fb202b90b813184e60ee567f1026b22a7/webpack.config.js#L26

Could you try 4.0.0-rc.4?


ID is not returned

Describe the bug
Code segment:
	res, _ = api.GetChildPages("327703")
	for _, r := range res.Results {
		fmt.Println("ID: ", r.Content.ID)
	}

This yields empty IDs for the child pages. So does GetContentByID -> which returns an empty bodied content, only the title is visible even though I'm specifying body.storage.
To Reproduce
Run the above code.
Expected behavior
The ID of the child pages is visible.
Release version
Master.
Confluence version: 7.1.1. running in a docker image.

Also body.storage isn't working even though this:
http://localhost:8090/rest/api/content/327705?expand=body.storage

Does return the desired content of the page.

FWIW this, works:
	search, _ := api.GetContentByID("327705", goconfluence.ContentQuery{
		Expand: []string{"body.storage"},
	})
	fmt.Println("Maybe: ", search.Body)

But I have to find the ID for this.

Which arguable I could parse out from the URL maybe. But URL is also empty.

I'm not sure what's going on but nothing except the title is returned.
{Content:{ID: Type: Status: Title: Ancestors:[] Body:{Storage:{Value: Representation:}} Version:{Number:0} Space:{ID:0 Key: Name: Type: Status:}} Excerpt: Title:Development URL:}

code:
	res, err := api.GetContent(goconfluence.ContentQuery{
		SpaceKey:      "DEV",
		Expand: []string{"children.page,history,body.storage,body.view"},
	})
	if err != nil {
		log.Fatal("error getting content: ", err)
	}

	//fmt.Println("ID:", res)
	for _, v := range res.Results {
		fmt.Printf("%+v\n", v)
         }

The returned json structure is no longer the way you describe it.  The ID is now inside the result.

hi @Skarlso ill take a look at this and fix the id, ty for finding the bug

@Skarlso i found the Problem you are right the GetContent returns a Search Object which does not fullfill the correct criteria, now GetContent will return an object of type ContentSearch that holds the correct structures.

@Skarlso i fixed this in v1.0.2 can you test if everything is working properly.

Sure will do as soon as I can. 😊


Fix issue with TypeORM loading entities from the dist folder

In some scenarios, when running the api natively, TypeORM will load the "compiled" entities (.js file from the dist folder) and thus will fail to load some entities.

I also don't know why it magically loads things from /dist instead of the "normal" folder. as to why this works, the /dist folder will have a similar structure to the "normal" folder, just that is has a .d.ts and a .js file for each .ts file. This is why the change works.
This sort of issue with typeorm is quite ubiquitous in stackoverflow and the relevant gh repos, with multiple people reporting this solution as one that works for them. I understand this is an issue only I've experienced so far, but I also don't see any downside to the solution I'm providing, which is why I'm pitching it. If you see any inconvenient, I'm happy to drop this

Aaaah ok, I understand now - I had misread your original problem statement as if the problem was that transpiled entities were actually being loaded and you wanted to avoid that. Ok, sure. Now I understand the fix :)
And it will actually be needed if we run the Marxan API from transpiled source - in other projects we settled on using ts-node after drinking the kool aid that it doesn't introduce noticeable overhead, but it's good to keep options open.


Overlapping text under "Invoice"

Overlapping text ( Fee 0.005 symbol.xym, 0 symbol.xym ) under "Invoice"
(linux version 0.14.0 - http://desktop-build.symboldev.com/#/invoice)

Fixed.


GLV framework + precomp scripts

The final piece of the puzzle, GLV. Patent expired 2 days ago.
Recommended to review after rest of algorithmic optimisations have been merged.
A description of the approach taken for GLV and background knowledge can be found in this hackmd article.

3/3 of a series of PRs to scipr-lab/zexe.
Features:

Batch Affine ops
Batch and ordinary w-NAF
GLV framework, GLV precomputation script.
GLV impl for both batch affine and projective mul (only for BW6)
Batch bucket addition tree
Batched MSM (1.4-1.7x bump)
Batch subgroup verification based on addition to random buckets
2D test matrix feature-gating with axes (curve, test type)
Refactor tests to use common template
Code timing instrumentation with fine grained control of functions and callers

Combined speedup (with w-NAF) for

Batched mul (with GLV): 2.3-2.7x.
Non-batch projective mul (with GLV): 2.1x.
Without GLV: 1.65-1.8x and 1.4x resp.

Speedup for subgroup verification

Over old impl 25-80x.
Over new batch mul: 10-30x.

Future PRs:

Improved MSM scaling (free scaling to 100s of threads for problem sizes > 2^25). (Need to investigate if held back by Amdahl's law in serial portion of code).
Recursive batch verification that gives ~1.5x speedup for large n.
Accel-based CUDA scalar mul + load-balancing code/framework to balance between CPU and GPU. Performance is neck to neck between weak GPU (GTX 1650 GPU) and a powerful CPU (16T Ryzen 4800H that clocks a sustained 4.2GHz). Probably better than most laptop Intels.

Future work:

Extend GLV to all applicable curves https://github.com/scipr-lab/zexe/issues/267

Hi @jon-chuang do you mind re-opening this PR against arkworks-rs/algebra and against arkworks-rs/curves?
(It'll probably be quite a rework, sorry for the inconvenience!)


Make histogram cumulative

ebpf_exporter currently does not conform to Prometheus's histogram specifications because its buckets are not cumulative (higher buckets don't contain lower ones). This causes issues when using Prometheus histogram processing functions and Grafana's heatmap visualization. This PR fixes that and makes the buckets cumulative.

I'm not sure I follow. The values are already cumulative (they are never reset). Perhaps you mean contiguous buckets, i.e., all buckets present from the start, with no gaps in them. Is that what is causing the issue with Grafana?

Ah, ok now I understand. I was not aware that Prometheus histograms are cumulative (unlike the BPF histograms). Brian Brazil explains it well at https://www.robustperception.io/why-are-prometheus-histograms-cumulative/

Yeah, I should've linked to some documentation of what it's doing. I even thought of linking that blog post but I forgot.
I also removed my Kubernetes spec file (was an artifact from an interactive rebase from an internal repo) since it has nothing to do with this PR.

@lorenz I refactored my code fairly heavily, so that the parsed values are written to an order-preserving array, and then summed later when passed the bucket map to MustNewConstHistogram(). I've also ensured that all buckets are exposed, as I found that Grafana's stacked bar chart was showing bogus negative values when the buckets had gaps in the sequence. This should resolve the problems you experienced with it.


Integrate a list of compromised stores that contain Javascript malware

https://gitlab.com/gwillem/public-snippets/snippets/28813

@kowalcj0 thanks.  Interesting.  So's the accompanying blog post.
How often is this list updated?  Is it actively curated?
Issue #180 applies here since this list isn't in host file format.

I am closing this now since it appears that this source, while interesting, is not actively curated into the future.

Hi @StevenBlack
Sorry but I missed your first response, it looks like the list not abandoned as it was edited on 2017-01-10 11:08:11 UTC
Let's see if the owner would like to contribute to this project.


rearming ai from arsenal issue?

Version: latest
Mods: CBA, TFAR, ACE(no-medical)
Environment: SP
.rpt attatched? no
have you edited the missionfile?: No
Issue:
I have an issue with rearming ai in squad, with prev version 17G I was able to take control of the ai for a limited time and had no issues on rearming. Now I have no control over the ai, unless I missed something.  I have selected the ai to rearm gone though the list of options, and selected arsenal, the ai runs to the location, but instead of opening his inventory to it it opens mine instead. is this a bug or have I missed something? Thanks

You can control AI in this version, just using Y menu.


dts: bindings: pwm: stm32: Remove deprecated property

The st,prescaler, has been deprecated for at least 2
releases.  So we can now remove it.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

Merging with approval from @FRASTM in absence of @erwango


[FEEDBACK] Add Parsing ability

Version
0.6.0-pre.4
Feedback
Add parsing ability so that helpers can help and make sure people are meeting reqs in runs, includes probably using a script to use a ss of the /who and checking their realm eyes to make sure that their currently played char is on par with the reqs of the run as specified
Submitter Information
Discord Tag: Karma's Guide#6864
Discord ID:<PHONE_NUMBER>69041665
Date/Time Submitted: 7/22/2020, 11:47:54 AM
Disclaimer: PLEASE READ
This issue was NOT written by the author of the repository OR the account that posted this issue. This issue was submitted through the program hosted in this repository, via Github's API, by someone that uses the program ("user"). The user can be found above.

This will be added to a separate bot.

This will be on hold until the new API comes out.


MacOS support

Just copied in libsdl from brew and it appeared to work out of the box. All the examples loaded.

Tests also passed:
andy@Andys-MacBook-Pro-2 SDL_ts % deno task test                                                                                (git)-[macos-support]
Task test deno test --unstable --allow-ffi
Check file:///Users/andy/src/SDL_ts/src/boxes.test.ts
running 10 tests from ./src/boxes.test.ts
I8 can be boxed ... ok (3ms)
U8 can be boxed ... ok (4ms)
I16 can be boxed ... ok (3ms)
U16 can be boxed ... ok (3ms)
I32 can be boxed ... ok (4ms)
U32 can be boxed ... ok (3ms)
Int can be boxed ... ok (4ms)
F32 can be boxed ... ok (3ms)
F64 can be boxed ... ok (3ms)
Pointer can be boxed ... ok (4ms)

ok | 10 passed | 0 failed (50ms)

Hey first off thanks for this and I'm glad that you could get it working on Mac even though you're having problems on Windows. I'm glad someone else is taking interest other than myself.
The only issue I have with the pull request is that I would actually like to move away from having the SDL binaries inside this repo. I did it for Windows as a crutch to get the project going but long term I plan to replace that with a script that will download the binaries for you. Long term it wouldn't be sustainable to have binaries in the repo for every OS / architecture.
I don't own a Mac so I'm flying completely blind here but on Linux after installing the binaries they can just be referenced by their filename. On Debian I actually change the filename for libSDL2_image.so and libSDL2_ttf.so. See here:
https://github.com/smack0007/SDL_ts/blob/SameGame/src/deno/_library.ts#L48
Can you please try that out?

The problem with mac is there isn't a straightforward installation path. You can install via brew, macports, or from source. Actually, I originally tried building from source but couldn't figure out how to produce the required .dylib files, but this is definitely due to my lack of understanding. Brew installs the actual dynlib files, but it's stored in the typical brew path (/opt/homebrew/**) which is not a typical posix location for libraries. I could certainly reference that path but that will only work if others install via brew.

getLibraryPaths returns multiple paths where the binary should be looked for so there be multiple places to look wouldn't be a problem.

That makes sense, how's this? https://github.com/smack0007/SDL_ts/pull/2/commits/0fecd4fb65c083aa34a8090746c8c4ed144adc37
(All tests passed)

Hi sorry got sick for a few days. This looks good to me now.
Suggestion:
In the Readme there is a credits section. Maybe you would like to add your information under something like "Initial MacOS Support". Let me know what you think and then after that I'll merge it.

No worries - done!


[WIP] Add hardhat-zksync-toolbox to plugins documentation

I'd move the toolbox plugin to be the beginning of the page, so users can scroll down to see the specific configs of each of the plugins that it includes.


Add a Back button in Project.html page so that navigation become easy

Hi @lovishprabhakar,
I added this "Back" button in the Project.html page for proper navigation between the home page. Please merge this pull request with the Hacktoberfest merge label.
Thanks

Good work @Jai-Marothiya!


Screen randomly whiteouts if in local home

I have noticed that if I'm in my local world (eg by using -DontAutoOpenCloudHome) and am using this mod my entire screen will randomly turn white, preventing me from being able to interact with anything. I have not been able to find a way to clear the screen other than already be in the process of joining a world when it happens or restarting Neos.  Even then, if I go back to my local world the screen will still turn white, requiring me to restart Neos. If I leave the local world then come back later, there seems to be a really good chance that upon switching to it my screen will turn white near instantly.
I'm on desktop and I haven't been able to test if this also affects VR yet because my headset is being RMA'd
This happens with the latest version of the 3DDashOnScreen mod and the last two versions.
Running Win 10, tested without mods and just this mod.

As discussed in Discord before, this might be a Neos bug. Since there was no new info for a long time, I will close the issue for now. If new intel is obtained, it will be re-opened.


Error Handling Strategy

Strategy for error handling in OpenDP, especially across the FFI boundary:

[x] Develop general approach for signaling errors from library functions.
[x] Define a way to expose this to FFI in safe manner.
[x] Write some example code for how this should be applied in the library.

The final element was to incorporate backtraces. After trying a few different approaches, we settled on an Error structure like this:
pub struct Error {
    pub variant: ErrorVariant,
    pub message: Option<String>,
    pub backtrace: _Backtrace
}

Also, for convenience, using an err!() macro to generate these.

This was covered by the work in #31 & #32.


Double precision on WebGL for mapping centimetric points

Hello,
I have a use case where I need to plot a very large number of centimetric points.
As mentioned on this blog, WebGL projected points are jittering on a grid when the zoom level is too important.
I managed to emulate double types on webGL to pass the exact point position, and avoid this jittering. An example here
Is this in the scope of this library ?
I will try to add this functionality (if I get some time) but I am not sure where to add it.
Is it useful only for Points, or also for polygons ?
Would a centimetric boolean on the settings be the right place for this ?
As explained here, it is also possible to add double precision only on the viewport position. This removes the jittering, but points are still placed on a grid (because their precision is still float). But it has the advantage of needing only one more uniform value to the program, instead of 2 more floats per point.
Would a halfCentimetrical setting for this be useful too ?
I will try to propose a pull-request for this if some people have this problem too.

If it isn't to heavy, and can be turned on/off, I'd be up for it.

If it isn't to heavy, and can be turned on/off, I'd be up for it.

Is there any news about ths issue? I really need too this centimetrical precision, but my skills in javascript are far from optimal. In fact, I am using leaflet.gligy from R coding, through the leafgl port by Tim-Salabim...

I finally gave up because of limited time and because this function didn't seem to interest a lot of people. This function also changed a lot of stuff on the shaders handling and it seemed like it was not worth it :-/
It's not simply a on/off functionnality. The way points are sent to the graphic card is completely different.
If you want to dig into how to display centimetric points in webGL, I did a demo here
Otherwise, just try to use standard leaflet points on a leaflet canvas :-/
I finally resolved my own problem by pre-computing images on the server side with all points combined.
I'll try to resurrect this project if I get a little time for this. Sorry in advance if I don't.

I have realized that this kind of errors in displayed positions is over 1 meter.

I do have interest

I have just found the following package:

64-bit Floating Point package (luma.gl)

I thing that the mathieugalle's global emulation for double-precision webGL (https://github.com/mathieugalle/faistos18.github.io) for leaflet.js seems promising.

@RayLarone thanks for pointing out that #156 may "possibly" close this. Can you confirm that this is resolved? If so I can go ahead and close it.


signaturePad.toData(); is not a function

const data = signaturePad.toData();
TypeError: signaturePad.toData is not a function

toData() is not a function and also in not included in signature_pad.js
I have installed signature_pad by Bower.
how should I get the drawn signature as an array of points?
as I understood till the latest version available by npm and bower (v 1.5.x) this function is not included

This function is only available in 1.6.0 beta version, which has not been and most likely won't be released to Bower. If you need this feature, you can install it using npm i signature_pad@beta or yarn add signature_pad@beta. Keep in mind that it's still beta and some things might (and most likely will) change in the final version.


Adds documentation for the Filter Chain entity available in Kong Gateway 3.4

Description
Adds documentation for the Filter Chain entity available in Kong Gateway 3.4 when WebAssembly support is enabled.
Draft status:

[x] Admin API docs
[ ] OpenAPI updates
[ ] review labels

Testing instructions
Netlify link:
Checklist

[ ] Review label added
[x] PR pointed to correct branch (main for immediate publishing, or a release branch: e.g. release/gateway-3.2, release/deck-1.17)

I renamed this for a better changelog entry!


Update README.md

Added virtualfabric's newly registered domains (NRD) and ongoing and emerging threats domains.

Thanks for sharing, great additions.
If you have the time could you next time also perform the other contribution actions? https://github.com/Bert-JanP/Open-Source-Threat-Intel-Feeds?tab=readme-ov-file#contributions
I will complete the other actions.

Sorry, @Bert-JanP! Missed this!


September DAME sprint

Coverage increased (+0.3%) to 26.28% when pulling 93a6251e9504938183c6120509a3b1ee58afdcc1 on september-dame-sprint-staging into a865f6d42df2b75068eed51cdbfcd9b95054b155 on master.


Add virtual metabolic human data

Also, don't forget to run the linter (./run_tests.sh -f from the root of the repo).


Remembering User Settings on Results Page

Astonishing work you've done in this program.  It's really quite impressive how you are not only good at producing a quality checker (the best there is), but you have an excellent knack for UI design.
One proposal i would like to make.  When the results page comes up, the user is allowed to select items on/off to filter the results.  For example, inputting the proxy ports to see in the results.  It would be nice if it would remember the user's settings on this page so they don't have to re-input each time.
I see you already do this on the front page of the program, but it is not carried over into the results page.
Once again, brilliant development!

Hi thanks for appreciating my work, i add this in v1.6.0 patch.


SVG files not converted

In the last days, the workflows for building our paper have been failing (example 1 and example 2). This issue seems to be related to this warning hinting that rsvg-convert is missing:
Warning:  Could not convert image /tmp/tex2pdf.-9cc0189afa7c7022/piglot.svg: check that rsvg-convert is in path.
Converting the SVG file to another format allows the paper to compile (example).
The timeline seems to point to #52, which is also consistent with the docker images used in our first failing and the last successful builds.

I can confirm, the build succeeds now. Thanks for the quick fix!


Display error message for service port-fowarding if user is logged out

Issue
Currently, if a user is logged out and a user performs a service port-forwarding, he/she is misled to think it is working, while it is not. This can be noticed once the connection with db is attempted
Related issue: https://github.com/epinio/epinio/pull/2366
Steps to reproduce
Note: these steps can also be found here: Connecting service port-forwarding to a database

Install Epinio
Create a service (ie. MySQL)
Log out
Port-forward it (observe how no error seems to be displayed

Connect to DB
Observe error

Request
Previous check to API to ensure the user is logged in should be done, displaying some message inviting the user to log in

Fix verified here: https://github.com/epinio/epinio/pull/2444#issuecomment-1630542097


Added child control type casting using generics

Added generic to the FormControlCollectior.control(String name) in order to cast the control (the generic must extends AbstractControl).
I also added the same thing to ReactiveForm.of(control).
If no generic is specified, it will use the extended class (AbstractControl<dynamic>), just like before.
I think this makes it cleaner to access a control of a certain type without having to explicitly cast it.
For example:
ReactiveForm.of<FormGroup>(context).control<FormControl<int>>("someControl")...

...

final control = formGroup.control<FormControl<bool>>("someControl");

instead of
((ReactiveForm.of(context) as FormGroup).control("someControl") as FormControl<int>)...

...

final control = formGroup.control("sameControl") as FormControl<bool>;

Hi @Jtplouffe
You can check https://github.com/artflutter/reactive_forms_generator
it generates all the control operation methods based on your model and makes life easier

Hi @Jtplouffe You can check artflutter/reactive_forms_generator
it generates all the control operation methods based on your model and makes life easier

Hi, thanks for the reply!
The generator would in fact help with removing the need for casting controls, but I still think it would be nice to have the generics to directly cast the controls

Try to look at casting from different perspective.
The type casting with "as" is definitely not the best approach of writing code. But if we will move casting deeper into the code this will not help us to avoid any possible mistakes. We just hiding casting inside the package)
It is the same as between
i++
and
i += 1

I'm not in charge of merging PRs
It's just my thoughts as far as I had the same casting issues which forced me to write generator package)

Try to look at casting from different perspective. The type casting with "as" is definitely not the best approach of writing code. But if we will move casting deeper into the code this will not help us to avoid any possible mistakes. We just hiding casting inside the package)
It is the same as choosing between
i++ and i += 1

I agree that it doesn't solve the problem, but I think it makes a nicer api
I think it looks better to do
ReactiveForm.of<FormGroup>(context).control<FormControl<int>>("myControl")...

than
((ReactiveForm.of(context) as FormGroup).control("myControl") as FormControl<int>)...

The generator would of course fix the need for this, but I really don't like the way Dart does code generation. Maybe it will be better when there is proper static metaprogramming

Hi @vasilich6107,
In case we include this casting feature in the library, does it affect the implementation of the "Reactive Form Generator"?
I will study deeper if this change is possible or not, but I would like to know first if you thing it will bring troubles to the generator.

Hi @joanpablo
You can add this feature to the package if you feel it more practical.
I will update the generator if there will be any conflicts)

Hi @joanpablo
We are approaching to migration to Flutter3, it would be good to add this PR also while making major release

Yes you are right!!!! 👍

Hooray!

Thank you!

Hi @Jtplouffe and @vasilich6107
There are a few things that I found (that I remembered) that pull me back to release these changes.
I think I rushed to merge it without testing it exhaustively.
I now remember why I didn't merge this before and why I didn't implement it many many many months ago.
I need more time and rethink this PR.
The first inconvenience is the Code Completion of IDEs (AndroidStudio and VSCode) none of them are able to give a nice code completion about the Generic type of the method "control()".
And secondly, there are some other methods that continue with the previously "AbstractControl" type that were not migrated to a generic method, for example, findControl() and findControlInCollection(). This creates inconsistency in the API.
And lastly, I still don't feel comfortable with the Documentation about the Exception that is raised if the cast is failing. It does not feel clear enough to me in the "control()" method documentation.
Sorry for the inconvenience but I'm not prepared yet to Release this PR.
Have a nice weekend!

No problem for me)

Are you going to revert this or and release the flutter 3 migration?

The Flutter 3 will be released. Your PR @vasilich6107 will be released. Only this PR will not be released. I will Reset the Master Branch to your PR (the PR before this one) and release a version from that point.


Upcoming deprecation of URL.createObjectURL

The console in chrome shows the following warning:
"[Deprecation] URL.createObjectURL with media streams is deprecated and will be removed in M68, around July 2018. Please use HTMLMediaElement.srcObject instead. See https://www.chromestatus.com/features/5618491470118912 for more details."
Are you planning on releasing a new version that uses the HTMLMediaElement.srcObject to prevent Instascan from working after July 2018?

It seems this was already fixed quite a while ago but there was never a new release after this (check bb59b1e1bd0271d0641ea9663bbad42bab217a96).
When I clone the project, install the dependencies and run gulp release it creates a new release in the project's dist folder. Replace the release you use with the newly created release and the warning should be gone.

In my case, camera window is not there but camera led is on.  No scanning.  Ubuntu 16.04.  This problem starts today.  It was working few days ago.
Press F12 on FireFox, the console shows the error.
However, Demo page works well.  And I compare my instascan.min.js and Demo's one,
$ ls -al instascan.min.js /var/XXXXXXX/instascan.min.js
-rw-rw-r-- 1 XXXXXX XXXXXX 1566037 Sep 12 19:37 instascan.min.js  => Demo
-rw-rw-r-- 1 XXXXXX XXXXXX 1561845 Mar  5  2018 /var/XXXXXX/instascan.min.js  => MY one
Different size.   But both of them contain URL.createObjectURL .

Using the direct download worked for me ^


Refactor Glob() and FindFile()

This issue was originally created at: 2008-08-28 01:29:24.
This issue was reported by: gregnoel.
gregnoel said at 2008-08-28 01:29:25

Glob() and FindFile() have a number of common logical elements, yet they have completely independent implementations.  They should be refactored to share code where possible.
One possibility would be to add a dirs= optional parameter to Glob() so that it acts the same as FindFiles.  Then FindFiles could be deprecated in favor of Glob(), which would simpify things all around.

gregnoel said at 2008-09-02 14:33:47

Bug party triage.

gregnoel said at 2009-04-15 20:28:30

Copied from issue #2398; it's not obvious from the comments above that this issue is intended to fix the problem described there.
Russel Winder sez:
Glob follows glob.glob and allows globbing -- which is not surprising given the name of the function.  So for example ~ or x__y.chpl.
Ant allows for a recursive glob pattern, for example **/*~ -- the ** is not treated as two instances of the glob character but as a special token.  The basic globbing is the same but the ** instructs there to be a recursive tree walk.  >
Adding this Ant-style extension to the Glob function would be incredibly useful and should be fairly easy to implement -- and it would save having to explicitly write a wrapper to the standard Python tree walk function to achieve the goal.

gregnoel said at 2009-04-15 20:29:27

*** Issue 2398 has been marked as a duplicate of this issue. ***

garyo said at 2012-09-01 10:04:01

de-assigning all tickets assigned to Greg Noel (no longer working on SCons)

gregnoel said this issue is duplicated by #2398 at 2009-04-15 20:29:27.

The #2398 stuff copied here was not really a duplicate - but note #3851 now exists to track the recursive-glob request.


Grammar fixes for auth

This is me nitpicking, but having to edit these manually every time I use the package gets old after a while.
Thanks!

Thanks!


App Crashes when using TakePhoto or TakeVideo

If you are creating an issue for a BUG please fill out this information. If you are asking a question or requesting a feature you can delete the sections below.
Failure to fill out this information will result in this issue being closed. If you post a full stack trace in a bug it will be closed, please post it to http://gist.github.com and then post the link here.
Bug Information
Version Number of Plugin: 3.0.1
Device Tested On: Android 23
Simulator Tested On:
Version of VS: 2017
Version of Xamarin:4.5
Versions of other things you are using: Support Libraries 25.+
Steps to reproduce the Behavior
Call  var file=await CrossMedia.Current.TakeVideoAsync()
Expected Behavior
Open Camera
Actual Behavior
Crashes with error
Attempt to invoke virtual method 'android.content.res.XmlResourceParser android.content.pm.ProviderInfo.loadXmlMetaData(android.content.pm.PackageManager, java.lang.String)' on a null object reference
Code snippet
var file = await CrossMedia.Current.TakeVideoAsync(new StoreVideoOptions()
{
PhotoSize = PhotoSize.Medium,
DesiredLength = new TimeSpan(0, 1, 0),
                                CompressionQuality = 70,
                            });

                            if (file == null) return;

Screenshots

I fixed it from text on the Readme.md about editing your Manifest file. My Bad


get challenge api

The API endpoint according to this PR is something like /roadmap-site/challenges/ which seems pretty long to me. Can we have it shortened?

If you could give in some shortened names, may be I can take a look


Polish the tessellation API and fix GL backend

Fixes #1088

@homu r+

:pushpin: Commit 035a558 has been approved by kvark

@homu r+

:bulb: This pull request was already approved, no need to approve it again.

:pushpin: Commit 035a558 has been approved by kvark

:bulb: This pull request was already approved, no need to approve it again.

:zap: Test exempted - status


Fix: validate input or throw when using MessagesPlaceholder

Closes #2871
Throw when input passed to MessagesPlaceholder.formatMessages is not of correct type

Thank you!


Add ability to use yaml config

This is to research the ability to handle both .json and .yaml/.yml files for configuration.

Also looked at this a few days ago.
Definitely possible and something we should consider for v2.


[Docs]: Upgrade Docusaurus to v3.

Is there an existing issue for this?

[X] I have searched the existing issues

Engineering Ticket Link
NA
Release Date
NA
Release Number
NA
First Draft
No response
Loom video
No response
Discord/slack/intercom Link if needed
No response
PRD
No response
Test plan/cases
No response
Use cases or user requests
Upgrade to the latest docusaurus version

closed - https://github.com/appsmithorg/appsmith-docs/pull/2032


Check if a tag name to load is a valid digest.

Fixes #12968.
/cc @stevvooe, @dmcgowan because I don't really know what I'm doing.
Signed-off-by: David Calavera<EMAIL_ADDRESS>

Maybe I should put that check somewhere else?

@stevvooe I've changed the implementation to only check the digest when we load an image. I think this is more correct than checking in every case we use the ValidateTagName function.

LGTM

LGTM


correct path in package.json

Was checking issue #15, and it seems to be the case that vs code can not detect the extension as the path to extension.ts in package.json is incorrect

@tanmaymunjal can you please share the steps of how you're running the extension?
The package.json points to the js file created in /out directory when we run the project and looks correct.

Got it already, really sorry for starting an issue on it, it seems that I need to run npm run compile first. A little beginner in vs code extensions, but yeah my bad there.


Compile Jade templates for AJAX requests - Client vs Server-Side

Hi guys,
I'm new to jade and was wondering if you had any advice for rendering jade templates on the fly with a simple ajax call.
I've been looking around and see that you can do it both server-side or client-side and am wondering if you've already included these capabilities somewhere that I'm not seeing. Right now I've included all my partials/includes in my public folder in HTML, but i would like to move everything to jade and am having some difficulties.
Thanks for any help in advance!
~ Kyle

It took a little bit of research and trial & error, but I came up with a system that I am truly ecstatic with.

Im doing the Jade compiling client-side during development with Grunt Jade compiler, which compiles all of my Jade partials from 'templates > views > partials' to 'public > partials_compiled'. This may seem excessive, but the site I'm building currently has about 40 partials and will likely get up to +100 by the end of it. This allows me to access the pages as static HTML with AJAX right now, but if I want to create custom routes and manipulate data client side, while maintaining the asynchronous behavior of the HTML, then its extremely easy to do so because the file structure is identical in each 'partials directory'. I think this solution is also likely as good as a I can do performance wise, because the other options are...

a) client-side compiling, per AJAX transaction
b) server-side compiling, per AJAX transaction
c) mass server-side compiling to JSON objects at initial load, in some monstrous file
None of these options seemed very efficient, manageable or scalable in the long-term. this solution also allows my to minify all my HTML in production getting even more speed and easily move the code to other platforms in HTML for different non-Jade/Node projects. I'd still like to hear how other peoples opinions on the matter though.

To move all my previous HTML into Jade I used [html2jade] (https://github.com/donpark/html2jade), which I also run with grunt. I takes a little bit of clean-up after compiling to Jade, but is by far the best tool I found. Its also the only tool I found that allows you to designate nesting with 'Tabs' vs 'Spaces', which I think is a million times easier to read and maintain; as well as other options like eliminating commas between attributes.

I'm coming from building in Laravel with Blade, which is just as verbose as PHP, and the horrors of Wordpress for CMSs (I was moving to OctoberCMS before I left, but still doesn't even compare to the power and flexibility of Keystone). Even though the transition has taken some work and there's still a much smaller community/resources with Keystone & Node, I'm completely thrilled with everything about the transition. The entire development process is unbelievably smooth, no more VMs, dependency issues and massive DB migrations. Best of all my HTML has never been more beautiful and maintainable with Jade; I literally stared at it for an hour last night.
So thats my St. Pattys day induced rant. Thanks for the the great theme you guys have put together and to everyone supporting the Node community!
~ Kyle


popover.js accessibility issue on generated html template

When Popover default template is used within a page, document structure is somewhat odd :

It would be better to leave h level parametrized, or use a different way to style it like an h3.
Many thanks

When you say popover box code where do you see the H1? If you could link to where you are seeing this or provide a JSBin of the problem that would be appreciated.

Comment updated, the h1 was somewhere else.  As for the hierarchy, this is the reference.
bootstrap.js, line 1582
Popover.DEFAULTS = $.extend({}, $.fn.tooltip.Constructor.DEFAULTS, {
placement: 'right',
trigger: 'click',
content: '',
template: ''
})
Maybe the use of h3 in the default template should not be necessary, and using a div will also work. That said, bottom of the backlog then.

I'm not 100% sure I follow the issue.
In this code, what specifically is the issue?
Popover.DEFAULTS = $.extend({}, $.fn.tooltip.Constructor.DEFAULTS, {
    placement: 'right',
    trigger: 'click',
    content: '',
    template: '

'
  })

Would I be correct that you feel the
 is not semantically correct and should instead be a styled ,
 or similar element?

That's correct. Thanks!

@maia-n For future reference, you need to either enclose HTML in backticks or use fenced code block when posting HTML in GitHub comments.

Thanks again, and my apologies for not being aware of working guidelines. Next time I will use them.

hm i'm struggling to see how the h3 is invalid – @patrickhlauke i'm going to close this issue, unless you have some strong feeling that a title shouldn't be a heading element?


Can't open HidHide Configuration at all. Crash on line 91 of the file "FilterDriverProxy.cpp".

[x] I have searched open and closed issues for duplicates
[x] I am submitting a bug report for existing functionality that does not work as intended
[x] I have read https://vigem.org/Community-Support/
[x] This isn't a feature request or a discussion topic

Bug description
Simply i can't open HidHide, i tried different versions but i always get the same error: "Something unforseen has happened which cannot be recovered from. Program execution has to be terminated. Sorry for the inconvenience."
I understand that's a problem that only I have, but i dont know ho to resolve it.
Steps to reproduce

Install HidHide.
Try to open it.

Actual result: "Something unforseen has happened which cannot be recovered from. Program execution has to be terminated. Sorry for the inconvenience."
Expected result: It should open normally.
Machine info
CPU-Architecture: x64 AMD Ryzen 5 2600x
Windows version: Windows 10 21H2
Software/driver version(s): 1.1.50 - 1.2.98
Any other helpful information
I don't know much about programming and this sort of stuff, only basics, but i tried debugging with Visual Studio 2019 and i can tell that the Debug of HidHideClient crash after a breakpoint on line 91 of the file "FilterDriverProxy.cpp".
Line 91: if (FALSE == ::DeviceIoControl(device, IOCTL_GET_WHITELIST, nullptr, 0, buffer.data(), static_cast(buffer.size() * sizeof(WCHAR)), &needed, nullptr)) THROW_WIN32_LAST_ERROR;

Not really fixable.


added KiDs DR4 lenses

@lukavujeva From Li+2021, it looks like there are two galaxies next to each other, so there are genuinely two lenses with that RA and DEC

More human checking/vetting:
             name                 RA        DEC    zlens   type   grading                               ref
                                 deg        deg
            str29              float64    float64  str15   str7     str9                               str171
----------------------------- ---------- --------- ----- ------- --------- --------------------------------------------------------------
<PHONE_NUMBER>44    129.889    -1.679  0.28 cluster confirmed                               https://arxiv.org/abs/2002.01611
KiDSDR4 J083933.372-014044.81 129.889052 -1.679115    --  galaxy  probable https://ui.adsabs.harvard.edu/abs/2020ApJ...899...30L/abstract

From HSC:

From KiDS

and the reported photometric redshift is 0.62 for this system

             name                 RA       DEC    zlens  type  grading                                    ref
                                 deg       deg
            str29              float64   float64  str15  str7    str9                                    str171
----------------------------- ---------- -------- ----- ------ -------- -----------------------------------------------------------------------
<PHONE_NUMBER>26   188.5909  -0.1572  0.41 galaxy probable https://arxiv.org/abs/2002.01611 LinKS https://arxiv.org/abs/2312.07333
KiDSDR4 J123421.817-000925.99 188.590905 -0.15722    -- galaxy probable          https://ui.adsabs.harvard.edu/abs/2019MNRAS.484.3879P/abstract

@lukavujeva We should replace Links with this explicit reference: https://ui.adsabs.harvard.edu/abs/2019MNRAS.484.3879P/abstract, and these two, I believe, are duplicates and should be merged into just one entry.

Hi, do you want my input on any of these cases?
This:
KIDS J112610.75+033847.89<PHONE_NUMBER> 3.646635465    -- galaxy probable
KIDS J112610.68+033848.05<PHONE_NUMBER> 3.646635465    -- galaxy probable
must be a typo, as the KiDS ID is constructed from coordinates , so one of these has wrong coords.

Hi, do you want my input on any of these cases? This: KIDS J112610.75+033847.89<PHONE_NUMBER> 3.646635465 -- galaxy probable KIDS J112610.68+033848.05<PHONE_NUMBER> 3.646635465 -- galaxy probable must be a typo, as the KiDS ID is constructed from coordinates , so one of these has wrong coords.

Hi @MBilicki
Yes. I tried to search for KIDS J112610.68+033848.05 in Li+2021 and actually could not find it being mentioned in the paper at all. Should we ping the corresponding author (Nicola I believe?), or maybe there is some internal communication channel for KiDS people that you can bring this up for us?
Thanks!

Also @lukavujeva we should also add KiDS DR4 high quality strong lens candidates in README.md

Hi, do you want my input on any of these cases? This: KIDS J112610.75+033847.89<PHONE_NUMBER> 3.646635465 -- galaxy probable KIDS J112610.68+033848.05<PHONE_NUMBER> 3.646635465 -- galaxy probable must be a typo, as the KiDS ID is constructed from coordinates , so one of these has wrong coords.

Hi @MBilicki
Yes. I tried to search for KIDS J112610.68+033848.05 in Li+2021 and actually could not find it being mentioned in the paper at all. Should we ping the corresponding author (Nicola I believe?), or maybe there is some internal communication channel for KiDS people that you can bring this up for us?
Thanks!

Yes, the best is to write to Nicola as we don't have a quicker way of communicating.

@lukavujeva Changes will need to be re-based because of https://github.com/lenscat/lenscat/pull/24.


Feature request: disable asterisk that denotes time adjusted for Daylight Savings Time

Some cities (Santiago, Adelaide, etc.) have an asterisk next to their name indicating the time displayed has been adjusted for DST. Can an option be added to disable the asterisk?

This could probably be closed. I used your edit workaround so Santiago no longer displays an asterisk next to the time.

This could probably be closed. I used your edit workaround so Santiago no longer displays an asterisk next to the time.

I actually like the idea, so let's not close it just yet and we'll see how much support it gets.
On a related note, I was wondering about having an option of showing a + or - instead of the day if the city's time is for the next or previous day. If you have an opinion on that, let me know.

I actually like the idea, so let's not close it just yet and we'll see how much support it gets.
On a related note, I was wondering about having an option of showing a + or - instead of the day if the city's time is for the next or previous day. If you have an opinion on that, let me know.

Personally I like seeing the day. I have added cities that have time zone three hours apart starting with mine. So as the day progresses, so does the day displayed for each city.

Personally I like seeing the day. I have added cities that have time zone three hours apart starting with mine. So as the day progresses, so does the day displayed for each city.

The option to enable/disable the asterisk works great Phil, thank you!