Resize editor pane left/right/up/down keybindings

Similar to #46100 but specifically requesting resize left/right/up/down key bindings.
It would also be useful if we could set increment sizes as the default is quite small.
Context: now that vscode's in-editor-pane terminal is reliable I use it a lot more, almost treating vscode as a multiplexer. So being able to resize panes with more fine-grained control from the keyboard would be very useful.

cc @sbatten for layout

uh, I don't think 20 people will upvote this in the next 60 days, but it should definitely not be closed completely just because it's not high priority. This is a basic keyboard control completeness feature

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_smiling_face: This feature request received a sufficient number of community upvotes and we moved it to our backlog. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

@bpasero is there a way to keep this in the backlog for re-evaluation later? Maybe in the event that related layout code gets reworked? Or maybe just get overriding the increment size for Increase/decrease view on the roadmap?
It would be nice to know some of the thought process here, like if there is a fundamental design disagreement, or if the pane management logic is just too much of a tangled mess for this feature to be feasible?

I might be able to, yes – especially with a bit of insight from a prior contributor who's worked in that neck of the woods. So maybe help-wanted would be applicable. Though I'm not sure when I'll get to it exactly so I don't want to over-promise

It seems vscode has this feature now. Increase/Decrease editor Width/Height.

@BABA983 I think that's been around for a bit – this FR is similar but about being able to specify left/right/up/down.
Increase/decrease width works well when there are at most 2-by-2 panes, but once you have a left, middle, right layout and want to see more of middle and less of left it starts to get awkward (have to explicitly shrink left because otherwise you might shrink right). With 4+ panes it gets impossible.
However I do have to admit that after a year there is little chance I'll actually get around to contributing this feature.
Although I still believe "this is a basic keyboard control completeness feature," it can be closed as it seems on one else has come out of the woodwork to advocate and my advocacy and potential contribution is the only reason it's alive.

If no one is working on this, is it fine if I take this up @bpasero ?


Document brand work for workshop

After concluding our website workshop on Feb 14th 2023, we need to capture the findings from those meetings to provide an update to the rest of the team who couldn't attend the workshop.
To-Do

[ ] Document brand work from the workshop
[ ] Report out to the Compiler team in #websites channel

@ segacy1 When will you be able to send out this report from the website workshop? @o-ram and I were thinking of getting the roadmap out by mid to late next week, and we could do this together as one big update.

Sounds good, @AnthonyRollins. I'll circle back tomorrow and let you know if I can wrap this up before my OOO starts on Monday!

Here's a link to the page in Figma or you can view it as a prototype. I'll flesh this out a bit more when I get back, but I think it's okay for you to send along with your roadmap update if you'd like!

We will get this added to our report; thank you!


theano support for 5d tensor in placeholder and rnn

Added the possibility to create 5D tensor in theano,
correction of a bug in rnn which prevented output shape to
be different from input shape. (for 5D tensors)

Tensorflow was not limited to 4d tensors for the placeholder
I modified rnn for tensorflow to make it works with 5d tensor.
I didn't tested the mask option for rnn for more than 3d tensor  in
both backend.

Regarding the TensorFlow changes: I'm worried about performance regressions, since you are doing heavier operations. Can you comment about that? For instance, a 15% performance regression had been previously introduced while fixing masking (which I just solved).

I think the code can be improved:
if  mask is None, I think we can avoid doing any switch operation.
We can keep the loop by creating a mask = [True] * timestep, and
then remove all the switch operation by testing with a if .
if mask is not None, I can get out of the loop the exand_dims and tile operation,
but it will probably require a little bit of extra memory.
Right know for me the operation tf.cast(mask, tf.bool) does not work so I can not really check
the performance.
I will do a new version af rnn that implement these changes today.

So I separated the masked and regular loop. like that we are sure that
the mask operation does not slow down the regular inputs.
For the masked operation, we have to tile the mask on all tha dimensions,
so the only extra operation introduce with respect  to the original implementation
that was working on three dimensions is a squeze  which remove all the dimension equal
to one. I don't think it consume some time. I can remove this if I knew how to make a copy
of a tensorflow tensor.

This PR needs to be rebased to master. But more importantly, can you remind me why it's important to allow for a mask with a different shape from the input? Masks are generated either by Embedding (3D, same as input) or by a Masking layer (nD, same as input).

This pull request can be clause because fchollet, made same kind of modification


rpcport does not work correctly when launching

rpcuser and rpcpassword seem to be functional but rpcport is not. It defaults to port 10229
To reproduce:
Start raptoreumd with rpcport=xxxx in command line or $HOME/.raptoreumcore/raptoreum.conf file
raptoreum-cli does not attempt to use specified port either and will default to port 10229, but will correctly use login credentials allowing for control of the daemon as normal with r-cli but nothing else (like curl).

Yes it switches to up a port per testnet instance so 10229 is correct, it's locked to this for the testnet.

Yes it switches to up a port per testnet instance so 10229 is correct, it's locked to this for the testnet.

Gotcha, did not know that.


Auto grow column with row contents

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

21 of 25 (84.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.3%) to 67.667%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/hooks/useColumnWidths.ts
20
24
83.33%

Totals

Change from base Build<PHONE_NUMBER>4:
1.3%

Covered Lines:
908

Relevant Lines:
1228

💛  - Coveralls


Update NOC code to accept 5 digits instead of 4

Describe the task
Federal government's new format for NOC codes is now 5 digits long and currently, our forms only accept 4 digits.  Update to accept 5.
Acceptance Criteria

[x] Make necessary changes to accept and store 5 digit NOC codes.

Changed NOC code to accept exact 5 digits.
It shows a validation error for 4 or 6 digits.

No errors when 5 digits are provided.


Add option for adding two or more branches to single action/end step

Describe the Feature
Currently, there is only option for adding the branches in one action/end step, or create for every branch it's action or end step.
The attached screenshot is an proposal, how the connected steps could look like. It's not not doable with the current UI.
Screenshots or Videos

Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [latest]

https://github.com/KaotoIO/kaoto-ui/issues/2095#issuecomment-1621614803

This is not... doable in Camel, right? Why do we want to add a feature that is not supported by Camel?

This can be achieved with "direct" component, further specifying the routes as separate ones. So you are right, this is not supported in camel in the way I proposed, so closing


[Enhancement] Estimate scan row bytes dynamically in time

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] refactor
[ ] others

Problem Summary：

The current mechanism of estimating the scanning column bytes is to directly give a fixed estimated value. If the estimated value deviates greatly from the actual value, the parallelism will be too low.
For example, with a Bitmap estimate of 1MB, query_mem_limit of 32GB would limit the parallelism to 1.
Therefore, this PR adds the strategy of "adaptive row bytes estimation".
During the scan tasks running, the estimated_row_bytes is adjusted in real time according to the read chunk bytes, and then the parallelism is adjusted.

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

@Mergifyio backport branch-2.3


Cargo build in resctl root dir fails

$ cargo build
    Updating git repository `https://github.com/gimli-rs/addr2line.git`
    Updating git repository `https://github.com/llogiq/bytecount`
    Updating git repository `https://gitlab.redox-os.org/ahornby/chashmap`
    Updating git repository `https://github.com/fbsource/const-random`
    Updating git repository `https://github.com/krallin/gotham.git`
    Updating git repository `https://github.com/mozilla/sccache`
    Updating git repository `https://github.com/KapJI/rust-openssl`
    Updating git repository `https://github.com/gabrielrussoc/prost`
    Updating git repository `https://github.com/jsgf/r2d2-sqlite.git`
    Updating git repository `https://github.com/jsgf/rustfilt.git`
    Updating git repository `https://github.com/bolinfest/tokio-core`
    Updating git repository `https://github.com/jsgf/toml-rs`
    Updating crates.io index
    Updating git repository `https://github.com/facebookexperimental/rust-shed.git`
    Updating git repository `https://github.com/facebook/fbthrift.git`
error: failed to load source for a dependency on `fbthrift`

Caused by:
  Unable to update https://github.com/facebook/fbthrift.git#81db842b

Caused by:
  Could not find `Cargo.toml` in `/home/agartrell/.cargo/git/checkouts/fbthrift-abf000ee5c7fcc50/81db842/common/rust/shed/codegen_includer_proc_macro`

Can you show the output of rustup show? I suspect it's a toolchain issue. I'm on the same fbthrift rev and it builds.

Can you show the output of rustup show? I suspect it's a toolchain issue. I'm on the same fbthrift rev and it builds.

rustup update was definitely necessary, and allowed me to move a lot further. I can't tell you why.
rustup default nightly was also necessary to get past the dependencies using new features (the one I got blocked on was features_ext)
It's also worth noting that below build in debug mode is, practically speaking, unusable due to the latency.
I'll mark this closed, but it'd be nice to include something in the README about the intended way to build these projects.

rustup update was definitely necessary, and allowed me to move a lot further. I can't tell you why.
rustup default nightly was also necessary to get past the dependencies using new features (the one I got blocked on was features_ext)
It's also worth noting that below build in debug mode is, practically speaking, unusable due to the latency.
I'll mark this closed, but it'd be nice to include something in the README about the intended way to build these projects.

@alexgartrell Yeah, most of that is documented in https://github.com/facebookincubator/resctl/blob/master/resctl/below/docs/building.md .
I'll update it with a note aboug debug/release mode.

@alexgartrell Yeah, most of that is documented in https://github.com/facebookincubator/resctl/blob/master/resctl/below/docs/building.md .
I'll update it with a note aboug debug/release mode.


Going through README.rst impressions

Just wanted to provide some feedback regarding going through the short tutorial in the README.

I started a new Django project, and DEBUG = True.  Since deferred is on by default, it took me a while to figure out why I wasn't seeing anything.
I finally got the deferred content to show, so I turned DEBUG = False to watch it bundle.  It 404'ed because I did not run bundle_media.  It would be nice to be reminded of this at the end of the README.

Some ideas:

Include an examples Django project that is all setup in the repo.  I really just wanted to see the thing work and it wasn't necessary to create everything from scratch to do this.

Otherwise, great project.  Your PNG sprite stuff is the best I've seen.

I did some changes, particularly adding pip and showing the variable names that could have helped you out (DEFER_JAVASCRIPT and USE_BUNDLES) as I had the same problems:
https://github.com/Leftfield/django-media-bundler/commit/2c9ad893b2077f8b2f51cd2b9ee167522646afe9


Typo on documentation

Hi,
You have a typo on documentation, on reading example you are using databse as argument of ReadFromSQL.
Regards!

Thanks for the review. Really appreciate.


Parse SEO URLs & SEO Suffix

Description
Cloudinary allows the ability to create SEO friendly URLs using the SEO Suffix option
This changes the asset type for example /image/upload => /images
This PR allows for the parsing of this and adds the ability to pass in an SEO suffix.
This also updates version management in how it parses and now allows someone to pass in a version
options: {
   seoSuffix: 'my-friendly-url',
  version: 1234
}

Issue Ticket Number
Fixes #18

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Fix or improve the documentation
[ ] This change requires a documentation update

Checklist

[ ] I have followed the contributing guidelines of this project as mentioned in CONTRIBUTING.md
[ ] I have created an issue ticket for this PR
[ ] I have checked to ensure there aren't other open Pull Requests for the same update/change?
[ ] I have performed a self-review of my own code
[ ] I have run tests locally to ensure they all pass
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes needed to the documentation

BREAKING CHANGES: "version" is returned as a number from parseUrl. "id" is now named "publicId" when returned from parseUrl

@all-contributors please add @colbyfayock for code, doc


[Bug] Fix VPS version selector

Elicend — Yesterday at 10:14 PM
Hi it seems this solve the issue ! Thanks!
Hi one detail I know that my backglass is a small screen but I am not sure that I can chose all table version it seems there are some one more at the top

Fixed with 3.10.2


Issues with showing game lengths of Single player only games

I noticed this with Single player only titles such as Mass effect Legendary edition where it shows Co. Op & Vs. with no game length.

But in howlongtobeat its only single player with a range of game length!

Please fix this if possible!

Thanks for the report!
My original logic was just checking to see if HLTB reported the game_type as game, if not then display it as a multiplayer game.
Mass Effect Legendary Edition was classified as a compilation game on HLTB, so I added that to the logic.
Should be fixed now!

Ohh makes sense! Thanks!


Tests/fix backupdbadatabase

dir is part of lab setup so better to leave it
this is until we have a proper "lab initialization script"

sounds good, passes checks 🏁


docs: Update the message of Gateway API 'Programmed'

According to the commit 8a985fa6361c ("Implement programmed flag"), update the 'Programmed' message text.
The message text "Gateway successfully configured" is for 'Ready', which was deprecated by the commit 1055d5988f32 ("Remove deprecated ready status").

docs: Update the message of Gateway API 'Programmed'

/test

/test

/test

/test

/test


Add community catalog validation tests

This pull request will add test cases for the following catalog templates:

wordpress
alfresco
zookeeper
asciinema
consul
etcd
fbctf
ghost
gogs
janitor
jenkins
jenkins_swarm
mongodb
nuxeo
liferay
kafka
minecraft
galera
odoo
pxc
prometheus
puppet
rabbitmq
rancher_bench_security
gocd_agent
rocket_chat
sysdig
taiga
weavescope
wekan
xpilot
elasticsearch
logstash
kibana
gocd_server
gitlab
owncloud

The pull request will add three DO hosts 4GB each and perform the tests.

LGTM


Docs: Quality Controls

Which documentation could use some work?

mux-video
general mux-elements documentation
hls

🖊 Describe what you'd like to see a change in the documentation
HLS is great, but it feels a little odd that there is no way to natively select video quality. For example, if the content of the video is quality-critical, IMHO the user should be able to select the quality explicitly.
Currently, I have not been able to find a way how to enable the quality controls neither in mux-player-react nor make plyr work with Mux source :( This is a major pain point, and I hope that I'm simply missing something. Am I?

This is definitely not available yet. However, we're working diligently to create such a menu. Unfortunately, we don't have a timeline for that yet, but it's definitely a highly requested feature.

@gkatsev question related to this item: when the quality selector is released in the future, if someone selects a quality level above what their connection can support, wouldn't they have a "worse" viewer experience?
Example: someone wants to lock their resolution at 1080p, their connection can only reliably support 480p, as a result they would experience buffering...right?

@aminamos yes, theoretically, they could have a worse experience. It would depend on how much can get preloaded ahead of time (often not much with MSE, which is why offline download could be helpful, but that's a whole other 🥫🪱). At the same time, users often know that their internet isn't that good and expect the extra buffering for the higher quality.

If I may, I'd like to add to this - while indeed it may require more pre-buffering/waiting which would result in a worse UX, I would argue in certain cases the trade-off is worth it when the ability to see detail is critically important, I'll spare y'all the list of use-cases that comes to mind. 😃

resolved by #734 (should be coming soon thanks to @luwes's valiant efforts, @ScottAgirs!)


Bugfix/issue 1100 missing bounds

Summary
Add tests and fix lpdfs to close #1100.
Tests
test/unit/math/prim/mat/prob/multi_normal_test.cpp
test/unit/math/prim/mat/prob/multi_normal_cholesky_test.cpp
test/unit/math/prim/mat/prob/multi_normal_prec_test.cpp
test/unit/math/prim/mat/prob/multi_student_t.cpp
Side Effects
None.
Checklist

[X] Math issue #1100
[X] Copyright holder: Peter Wicks Stringfield
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)
[X] the basic tests are passing
[X the code is written in idiomatic C++ and changes are documented in the doxygen
[X] the new changes are tested

I also verified that the actual Stan program that showed the bug can sample with this fix.

(and I also verified that it correctly kicks out when there's an array of length 1 of a vector that matches in size!)


What's the hacs repo?  The readme link disappeared?

System Health details
N/A
Checklist

[X] I have enabled debug logging for my installation.
[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of any previous issues..

Describe the issue
Sorry if I am missing something basic, but I cant even install this because the HACS installation steps say install "THIS" repo, but there's no link in the word "THIS".  What's the repo to get at this package?
Reproduction steps

Have Home Assistant
Have HACS
Try to install this package
Visit this page to get instructions
Get sad when you realize there's no actual repo listed
...

Debug logs
N/A

Diagnostics dump
N/A

There is a link for repo in the head:

Thank you!

Spoke too soon...  When I enter https://github.com/andr83/hacs-anova-oven in the "Add repository" part of HACS, I get no repositories found...

Because you add a wrong link. You should add to custom repositories not link to git project but copy git link like here:

100%  You were right.
Thanks amigo!


Apply clang-tidy fixes for performance-move-const-arg.

Note that a few of the proposed fixes were bad, resulting
in code that wouldn't compile, so I'm not adding this
check to WarningsAsErrors in .clang-tidy
Signed-off-by: James Synge<EMAIL_ADDRESS>

Please trigger a re-run of the failed tests. Those may be infrastructure problems, rather than with this change.

It is already on as a warning, but not as an error. I would have made this (performance-move-const-arg) an error, except there were several suggestions that were erroneous, so such a .clang-tidy change would presumably, and ironically, fail in "ci/circleci: clang_tidy".
The reason for swatting this one-by-one was to get familiar with the process and ease of applying these.

@jamessynge do you have an example for the erroneous suggestions?


Support for meeting APIs (JS)

See parent

@clearab please assign an owner one the parent issue is ready to be worked on.

@joshgummersall I'm not sure what type of owner you're looking for? @munozemilio is assigned to the parent, I'm not sure who is going to implement each individual language. This issue is definitely ready to work on.

@munozemilio we're assigning this to you just for coordination for now. I can help coordinate the work on the JS side.

closing as duplicate


Heading rendering problems  in markdown

I found a heading rendering problem in rendering markdown file using hugo that confused me. Hugo will render heading in markdown like this:
# some heading ->
some heading

It's content becomes it's id and so are the other heading tags (h1-h6).
This may cause rendering confusion if I have a style with the same name to a heading content. eg.
# header ->
header

This issue won't be fixed currently with the reasons listed as follows:

Currently we only have four built-in IDs: wrapper, header, main and footer. Chances that these words are used as a heading is very low.
There is a compatibility problem with hugo since the default markdown renderer changed from blackfriday to goldmark since v0.60 if I am not mistaken. The two renderers' approaches to handle auto heading IDs are different.

We shall come back to this issue if it arises in some scenarios in the future.


Added ManifestV2Digest to work with V2 API

This PR is based on #51
This adds a correct implementation to getting the manifest 'Docker-Content-Digest' from the header section. This is needed to delete stuff from docker registry using registry Api v2 (which should be at least all of them!). This PR sets the Accept in the header directly to V2.
Please get this merged! PR #51 was contributed in 2018! Thanks!

Never mind. This project seems to be drad.

Never mind. This project seems to be drad.


Document limitation or expand implementation of pathify

It's not clear that pathify only works on circles/rectangles/lines. Is there a technical reason why it doesn't work e.g. on polylines, polygons or ellipses? This limitation should be either documented or the missing functionality should be implemented. I can tackle these, but I need to know which is the case.

No technical reason. I just only bothered implementing it for the shapes I happened to be using.


/dev/ttyUSB0 is starting like /dev/ttyS0

I'm using it with USB to RS485 converter which is present like /dev/ttyUSB0, but when I'm starting mbusd it is opening ttyS0 (in my case it is different port!)
sudo mbusd -d /dev/ttyUSB0 -s 9600 -m 8N1 -t -P 5002
07 Jan 2022 10:54:35 mbusd-0.5.1 started...
07 Jan 2022 10:54:35 tty: trying to open /dev/ttyS0 (speed 9600 mode 8N1)
I have fixed it by remove ttyS0 and symbolic linc it to USB0 but it is not like should be :(
When I added this port to /etc/mbusd/ttyUSB0 and started it like a service then everythig was fine.

Aren't you suppose to use -p /dev/ttyUSB0 since -p is the param for the device? 🤔


remove shop if empty

hello, first of all thank you for the awesome plugin!
In my server we have a market and it frequently happens that it runs out of free chests due to players not removing or refilling them.
It would be really interesting to have an option to automatically remove a shop when it is out of supplies (maybe after a certain time).
Thanks again.

Yes, my server is in the same situation
He can cause server to be slow


Remove requiredOption

We should remove requiredOption, in case the value is provided as .env
https://github.com/pinax-network/substreams-sink/blob/main/src/commander.ts#L47-L49
The idea is you don't need to provide any CLI arguments and everything can be set via .env
CLI arguments would be additional overrides, but not required
We're already checking if the field is required or not here:
https://github.com/pinax-network/substreams-sink/blob/main/src/config.ts#L46

Done in commit: https://github.com/pinax-network/substreams-sink/commit/9ef575b9aeaf1f10f405d4ca0a2d44ebd88eb666 with changes to use new Option(...)


Parse sObjects in the landing page and return them in an array

You're going to need to rebase against main.

@khawkins On resolving the conflict now. I was talking with @dbreese but after resolving we will need a refinement in how we render the table as we'll now have both what sObjects that we are generating for and what sObjects' QAs already exist. So this may not come clear after my merge conflict, just a heads up.

@khawkins On resolving the conflict now. I was talking with @dbreese but after resolving we will need a refinement in how we render the table as we'll now have both what sObjects that we are generating for and what sObjects' QAs already exist. So this may not come clear after my merge conflict, just a heads up.

Yeah we need to resolve, for sure. Looking at both implementations, I don't think the results of your functionality (finding the sObjects represented in the landing page) should be going back to the UI. That's functionality that the LWC generation code needs to invoke, to understand what it needs to generate.  To me, that's what should be (and currently is) going back to the UI. I think the sObject interrogation in the landing page should just be represented as functionality internal to the process.
Cc: @dbreese

@dbreese @khawkins I am now working on exactly that, making interrogation as a part or internal to finding which sobject's QA to generate(view/edit/create).  The way it is set up now window.load will need to make to separate calls for finding what sobjects there are on the landing page and which QA's are there for those sobjects. Mashing them into one is the way to go.

@khawkins Made the search to simple look for all objectApiName. I will do prettier update in another PR.

I think the last commit could be the final dagger, @khawkins .

I think the last commit could be the final dagger, @khawkins .

Pretty close! 😆 I think all that's left is to just remove those now-redundant tests in uiUtils.test.ts. Otherwise 🧑🍳😚 looking good to me!


Properly handle deeply nested at-rules inside of @responsive

#642 pointed out a bug in our @responsive handling that didn't account for child rules being at-rules.
That PR fixed the case of nested at-rules being a single level deep which was better than what we were doing, but there was still a bug if at-rules were nested deeper than one level.
This PR solves that issue and simplifies the code path dramatically by using a PostCSS root instance to hold all of the responsive rules, and manipulates all of the selectors using walkRules to recursively alter every rule instead of trying to determine the depth ourselves and handling regular rules differently than at-rules.

Your PR descriptions are 👌


CASMCMS-9225: Add paging to component list requests; use context managers for all requests

Opening this as a draft. I will open it for formal review once I've done more testing and filled out this template

I'm gonna split this into multiple PRs for review purposes.


First window does not restore its former size and location

VSCode Version: 1.38.1 (also couple versions before)
OS Version: (X)Ubuntu 18.04 (XFCE 4.12)

Steps to Reproduce:

Start vscode
Resize and/or shift vscode window
Restart vscode (close and open)

Does this issue occur when all extensions are disabled?: Yes

Can you try to reproduce with our nightly insider builds? You can give our preview releases a try from: https://code.visualstudio.com/insiders/

VSCode Version: 1.38.1 (also couple versions before)
OS Version: (X)Ubuntu 18.04 (XFCE 4.12)

Steps to Reproduce:

Start vscode
Resize and/or shift vscode window
Restart vscode (close and open)

Does this issue occur when all extensions are disabled?: Yes

Same issue

Version
版本: 1.39.0-insider (user setup)
提交: d157c2e941cae17acb79bf26a70bbfad732bdc83
日期: 2019-09-16T06:06:10.647Z
Electron: 6.0.9
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Windows_NT x64 10.0.18362
Current Feature
The restore-button does not work after the editor opened!

Same issue.

@bihshan do thinks improve once you configure "window.titleBarStyle": "native" in settings?

Can you try to reproduce with our nightly insider builds? You can give our preview releases a try from: https://code.visualstudio.com/insiders/

@bpasero  Same issue with nightly insider build.

do thinks improve once you configure "window.titleBarStyle": "native" in settings?

No.

This issue seems to originate from an upstream component and is not something VSCode can fix by making a change to the code. As such, I will go ahead and close it. If you can identify the component that is causing the issue, please report it upstream:

for UI, OS integration: https://github.com/electron/electron
for Rendering: https://bugs.chromium.org/p/chromium/issues/list
for file IO: https://github.com/nodejs/node

Thanks for understanding.


Add verified header type

Add MessageClientHeaderVerified which is like MessageClientHeader. MessageClientHeader is what you send and get from the server. MessageClientHeaderVerified is what you get after a successful unbox, all its fields come from verified signed stuff.
Compared to MessageClientHeader, MessageClientHeaderVerified is currently missing MerkleRoot and will forever be missing Supersedes and Deletes. Supersedes and Deletes are just hints to the server, that can be derived from the message body.

Should we think about folding ClientHeader into ServerHeader or something like that?

ClientHeader is entirely written by the client. ServerHeader is written by the server. Could be good. What would we gain, and would it be worth it?

LGTM if @mmaxim likes it.

The commits 97af84f3802c...076ef26e9991 introduce visual changes on linux. :mag_right: 2 changed

Changed: Attachment_Message-Basic_Preview_Image_Downloading_Preview (before) (after)

Changed: Attachment_Message-Basic_Not_loaded (before) (after)

The commits 9a846ca050e1...775e68a3b81c introduce visual changes on linux. :mag_right: 2 changed

Changed: Attachment_Message-Basic_Preview_Image_Downloading_Preview (before) (after)

Changed: Attachment_Message-Basic_Not_loaded (before) (after)

The commits 9a846ca050e1...775e68a3b81c introduce visual changes on linux. :mag_right: 2 changed

Changed: Attachment_Message-Basic_Preview_Image_Downloading_Preview (before) (after)

Changed: Attachment_Message-Basic_Not_loaded (before) (after)

The commits 3dad37e5a563...775e68a3b81c introduce visual changes on linux. :mag_right: 2 changed

Changed: Attachment_Message-Basic_Preview_Image_Downloading_Preview (before) (after)

Changed: Attachment_Message-Basic_Not_loaded (before) (after)


unit testing for Delta Sigma

Test Delta Sigma calculation (in some way to be determined) and include in repo.
Also include a convenience function wrapper around the mocks.
CC @duncandc

This is resolved by #425


CAMEL-15199: RestDefinition relies on Class.getCanonicalName instead of Class.getName for in/out types

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] If you're unsure, you can format the pull request title like [CAMEL-XXX] Fixes bug in camel-file component, where you replace CAMEL-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean install -Psourcecheck in your module with source check enabled to make sure basic checks pass and there are no checkstyle violations. A more thorough check will be performed on your pull request automatically.
Below are the contribution guidelines:
https://github.com/apache/camel/blob/master/CONTRIBUTING.md

@davsclaus I've reworked the PR to have a transient field to hold the class but this has some impacts in to the APIs and in various places, mind having a look ?

LGTM

@davsclaus @oscerd mind dogin one more review ?


Fix error, tracing, and cache control propagation in schema stitching

Right now, errors, tracing info, and other stuff doesn't always make it across the stitching boundaries. This means sometimes using schema stitching can cause a degradation of developer experience compared to a non-stitched schema.
We should fix that, so that we can recommend stitching without reservation as a first-class way of developing schemas. I'm going to close issues related to this and point them here as a master list.

Hate to be this guy... But are there any plans for this feature?

checkResultAndHandleErrors is really messing up schema stitching for us
We delegate and get a perfectly nice graphql error back from a downstream service
A transformer that we can't disable transforms that into an unusable error by concatenating the body into a string?

I'll go ahead and piggyback off @krisread since we're experiencing the exact same issue. It'd be great to forward the initial error to clients so that downstream services can handle errors on their own..

Is this the reason we might not see info.cachControl in resolvers we build and schema stitch together using graphql-tools? If so this is a pretty big let down because we use schema stitching to organize our larger schema and to stitch in remote schemas.

@cgatian, @krisread, @RickyV33, this should be fixed in graphql-tools-fork, I wonder if you might be able to still show a breaking example.
@flipxfx, my guess is that is a separate issue related to multiple wrapping resolvers working well together. My next fix planned for graphql-tools-fork is to fix default fields, #1121, but if you open a separate issue for working well with Apollo server cache, that seems like another good candidate. If you can include a small code reproduction that would be very helpful.

@yaacovCR i tried graphql-tools-fork (v7.2.3) ,but cache-controls in the remote schema are not passed

@macrozone if you can put together a minimal reproduction of what you expect to be available where I would be happy to take a look.

@yaacovCR will do if i find time. To give you some more information:
i have a remote schema that defines cache hints via @cacheControl(maxAge: 300) on a Query.resolver
i have another ApolloServer in front of that that does schema stitching using makeRemoteExecutableSchema. The resulting graphql server does not respect the cacheHints on the remote schema

Did you set mergeDirectives to true btw?
https://github.com/apollographql/graphql-tools/pull/1003

@yaacovCR thx! did not know that option. However it does not solve the issue. it just throws an error: "Error: Duplicate directive name(s): include, skip, deprecated"

I think the duplicate directives error was from graphql-tools, should have been fixed in the fork. I do not use Apollo engine so not sure exactly the workflow you are describing. I would be happy to learn, but probably best would be a minimal reproduction.
In terms of the cacheControl directive being respected, i imagine you may have to reannotate the fields on the merged schema with extend.
This is a bit heavy as related to https://github.com/graphql/graphql-js/issues/1343#issuecomment-459156658 and lengthy discussion there.

Although I am reminded that mergeSchemas I believe copies all the underlying config ast and extensions except for root types (as those are merged), are you talking about cacheControl on root fields?

A minimal reproduction might help. :)

In terms of the cacheControl directive being respected, i imagine you may have to reannotate the fields (if possible?) on the merged schema with extend.

that's what i want to avoid at all cost. The remote schema is huge and already defines cache hints on some Query.resolvers
For the moment, i opened a discussion on spectrum: https://spectrum.chat/apollo/apollo-server/caching-remote-schemas-result~0b0f5d90-36a1-4b39-a231-a16e5db29d9f
i think its a topic that is not covered properly yet by the apollo team. Its a bit a chaos right now with the deprecated schema-stitching and federation. So i think i will step back and no longer try and wait what the community has to say on the topic.

I disagree a bit. We are the community, waiting has been the strategy employed so far with little progress. I am volunteering to contribute some unpaid time to this issue -- I think you stand the best shot of a working approach in the foreseeable future by adding a failing test to graphql-tools-fork or by sharing a repository with a minimal reproduction.

Looks like read directly from astNode here:
https://github.com/apollographql/apollo-server/blob/d5015f4ea00cadb2a74b09956344e6f65c084629/packages/apollo-cache-control/src/index.ts#L81

@macrozone, Are you asking about cacheControl hints defined in schema (which should be working) or those defined within the resolver functions?

@macrozone, my current understanding is that directives are lost when getting a remote schema via introspection, but that this should work if you have access to original sdl. Will try to prove that in test in fork.

@yaacovCR sorry, i did not want to slam your efforts on that, i really appreciate your time! I am more concerned about the direction the apollo team is taking schema stitching... (deprecating it).
Anyway.
yes, if you have access to the original schema, you can of course have these directives. But the scenario is, that the remote service is not under control, only introspectable.
i think i also made a misstake: In the end, the stitching server makes a http call to the remote call. Here caching could already be leveraged. I just dont know if the apollo server itself honors cache headers of the remote server. If so, the stitched schema does not need to know abot the remote's cache annotation

Ahah!
I think you can get this working using a node fetch polyfill with cache support like https://github.com/zkat/make-fetch-happen as the fetch option in your schema stitching link.
You also must use GET for queries and maybe get apollo-link-persisted-queries working on server side, but looks like a great idea!

Closing for now, please reopen as necessary, error support rolled into #1306

I don't think there is a separate issue that discusses tracing.

I think we can close this stale issue -- from lack of interest about tracing.
We can track caching improvements at #1380 and we should be good on errors since merging of fork.
Tracing should work just fine for the gateway schema, although additional data would be necessary to determine if any bottlenecks are from the underlying subschemas. A plugin could be developed to extract that data, but the Apollo community and those interested in that feature seems to be moving toward Federation, with interest in this issue waning.
The community can reopen a fresh new issue about tracing if they have additional concerns.


CSP Errors in the console when viewing report

@hugo-vrijswijk you changed something with the CSP a while ago, did this get released (or was this a fix for this)?

potentially something missing here https://github.com/stryker-mutator/stryker-dashboard/blob/08ae65d77108808889c8cd44d15dada057901c68/packages/website-backend/src/index.ts#L58

Oops, yeah this is from my changes. I didn't test the real-time report. Though I don't understand what particular part of the real-time report uses eval.

I am not sure too why this happened, it could be that by building the mutation testing elements, it includes eval somehow? I have seen it before when building angular for production, might be worth to also look at that.

I just used a different browser where the issue is not present (perhaps it is one of my extensions..?), but real-time reporting did not work anymore. I am thinking it has to do with this PR: https://github.com/stryker-mutator/stryker-dashboard/pull/524
I'm hoping I will have a fix for this soon

Hmm, I did another test with curl: (https://stryker-dashboard-acceptance.azurewebsites.net/api/real-time/github.com/xandervedder/Wrappit/main?realTime=true) and it showed me the events. I will roll out your change once again.. 😅

Curl does not do CSP checks, so that makes sense

I did some tests using git bisect and the offending PR is #533

Perhaps we could disable compression for all real-time endpoints?
It does make sense compression would break SSE 😮


[Feature Request] 1.21.1 Port

Problem
There's no 1.21.1 version.
Feature
A 1.21.1 version
Other Solutions
No response
Additional Information
Here's what I can see as a to-do from face value:

[ ] Crowdin needs to be dropped
[ ] Smart recipes needs to be ported, it's on 1.20.4
[ ] Gradle wrapper
[ ] JDK updated
[ ] Mappings migrated
[ ] Cloth config, Modmenu, and fapi need to be bumped
[ ] Energy and Satin need to be checked for compat - they both have 1.21 versions, but no 1.21.1
[ ] Compatibility - I haven't researched the five compat mods yet
[ ] Packets need to be updated to 1.21.1
[ ] AWs probably need updating
[ ] All the rest of the jobs

This is not a small job, but it's also a cool mod

I'm surely glad that I invested my time into setting up the forms, as now I get to see gems like this :D

Problem
There's no 1.21.1 version.
Feature
A 1.21.1 version

Anyways, yes, I want to update this mod. However, as you mentioned, there's a lot of work to be done to achieve that, not even considering all the refactoring needed to clean up some of the... questionable design decisions I made along the way. At the moment, I just don't have the time or energy to take on such a daunting task. That doesn't mean I won't, though. I think I'm going to skip a few versions and update the mod directly to 1.21.x or whichever version Mojang settles on after their internal mega-refactoring.

Crowdin needs to be dropped

Why tho?

Smart recipes needs to be ported, it's on 1.20.4

I think I'm going to drop that mod entirely. With all the recent changes Mojang introduced to items, crafting, and other relevant game mechanics, updating it would require significant efforts. Besides, almost nobody but me really uses it. So, while I still like the concept, I only have so many hours in a day, and I'd rather not spend them on something no one needs.

Energy and Satin need to be checked for compat - they both have 1.21 versions, but no 1.21.1

teamreborn:energy shouldn't be an issue, as it rarely changes (if at all). So unless it's been completely deprecated while I was away, everything should work fine.
As for ladysnake:satin, I'm just going to remove it. I had working shaders before I brought it in, hoping that using a specialized library would help with compatibility issues. However, it turned out to cause much, much more problems than my simple, homebrewed approach. Thankfully, it's just a matter of reverting a couple of commits.

Compatibility - I haven't researched the five compat mods yet

Honestly, everything except trinkets and origins will be dropped. I won't be adding and even accepting compatibility layers for non-library mods on my side anymore since they don't offer any semblance of a stable API to rely on, whereas this mod did from the very start.

Awesome! I thought crowdin needed to be dropped because it's deprecated. IMS212's fork is updated to 1.20.4. @cassiancc and I have ported from there to 1.21.1, working out the kinks at the moment. Not sure who I should put the PR into tbh xD
We've dropped hema and requiem as they're not updated, and IMS212 dropped crowdin, but just by commenting it out as far as I can tell. It wouldn't be hard to restore.
The branch is currently very much in an in-dev state, since I haven't figured out how to get origins and its deps properly, and smart recipes is via a file rn (Cassian updated it, I don't know how much testing has been done). Also origins is crashing a lot :P.

I thought crowdin needed to be dropped because it's deprecated

Oh, I remember now. Crowdin integration has two parts to it: a Gradle plugin that downloads updated translations during compile-time, and a mod that can update them during runtime. Honestly, the latter hasn't proven to be particularly useful, so I wouldn't mind if it's gone.

The branch is currently very much in an in-dev state, since I haven't figured out how to get origins and its deps properly

Yeah, I remember spending quite a bit of time getting Origins to work as well. Also, Origins doesn't really have a built-in functionality to store and restore traits separately from the player, so the whole integration thing has been shaky - to say the least - from the very beginning. I'd love to drop support for it too, but it's a library mod and quite a popular one, so unfortunately, that's not an option.

and smart recipes is via a file rn (Cassian updated it, I don't know how much testing has been done)

Props to them! :D That said, as I've mentioned before, it might be better to let smart-recipes rest in peace. Even here, it only manages one recipe at this point, so what's the real benefit there? Since Minecraft is so data-driven nowadays, a more sensible approach might be to abandon the original built-in auto-switcheroo to a more demanding recipe idea for Hardcore players. Many casual players (why are they playing Hardcore, tho?) have requested to revert this behavior, and I provided a datapack for them. So, perhaps we should consider doing a complete 180 and offer a datapack for more involved players instead.

A few notes from my work on the port:
Satin
I would love to see Satin removed, personally. I've been hoping to do an Architectury port of this at some point, as I need a version of Sync on Forge/NeoForge (version dependent, really), and not having to work with/port Satin would make my life easier.
Origins
I've just fixed the issue with the Origins deps, and will continue to try and fix that integration as much as I can in the meantime.
Smart Recipes
My testing has been pretty limited, but the port definitely isn't fully finished, as its failing to load the hardcore recipe at all. I believe it has issues with the new 1.21 data paths, and if we're dropping it anyway, it may not be worth the time to fix.

I've been hoping to do an Architectury port of this at some point

Same here! There are already a lot of things that need changing, along with plenty of player feedback that needs to be addressed. All of this would likely lead to a major refactoring or even a complete rewrite of the mod, so why not make it multiplatform along the way? I'm not sure how I feel about Forge, but I definitely wouldn't mind supporting NeoForge. I just need a bit more free time and/or to get my medication sorted, which will hopefully bring my productivity back from the level of a depressed turtle stuck in a sloth's body xD

as its failing to load the hardcore recipe at all

Welp, this means the mod doesn't work at all since the hardcore recipe is super basic.

I believe it has issues with the new 1.21 data paths

It does. That's why I said porting it would be a pain in the arse - it's heavily entangled with Minecraft’s crafting system, and Mojang has completely overhauled that in recent updates. Basically, the old implementation has been thrown out the window, so smart-recipes obviously no longer work.


Add example cci project

As a contributor to the project, I want an example cci project so I can easily spin up new eda-based or npsp-baed scatch orgs for testing and developing snowfakery recipes.

It is possible to add an example cci project, but its files must be located in the repos root, which clutters things up. I tried moving it to a sub-folder, but cci complained with The file cumulusci.yml was not found in the repo root: /Users/mcneal/src/DataGenerationToolkit. Are you in a CumulusCI Project directory?

@umeditor @samanthashain @acrosman
We talked about this on Day 2 briefly that everyone was creating a cloned local repo in order to load the recipe to a test org. Grassroot Mobile Survey team has more than 1 repo for their project linked through Readme.md and Wiki page.  Potentially we can discuss this in our next meeting. I love to have a repo for cci project init

I agree this should be part of our agenda for our next meeting. A second repo mid-Sprint with a bunch of open pull requests and more expected would have been a major challenge, but between Sprints is the perfect time to make such a move if we are in agreement (which I expect we'll find we are, but we should talk it through to be sure). Having that repo be a usable CCI project of some kind (easily modified for a variety of use cases) would be useful.  How exactly we make that happen isn't clear to me today – but that's why we have a whole group to draw on for ideas and insights.

The easiest way to do this is to make a CumulusCI.yml with multiple flows like:
eda_org
npsp_org
pmm_org
So you'd run
cci flow run eda_org --org qa

@prescod We resolved this by offering flows installing NPSP or EDA dependency to a scratch org. I just want to make sure we are okay to close this issue.


HTTPError: 500 Server Error: Internal Server Error

Hello, I'm trying to use Angus SDk, but I got an HTTP Error 500.
I tried the "Hello World" example (with face_detection, gaze_analysis and others) and I got the following error:
"Traceback (most recent call last):
File "test.py", line 7, in
job = service.process({'image': open('.\denis.jpg')})
File "C:\Python27\lib\site-packages\angus\rest.py", line 185, in process
resource_type=Job)
File "C:\Python27\lib\site-packages\angus\rest.py", line 118, in create
r.raise_for_status()
File "C:\Python27\lib\site-packages\requests\models.py", line 831, in raise_for_status
raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 500 Server Error: Internal Server Error"
If I try the "Hello World" example with dummy I don't have any error though. So it seems like I'm connecting correctly to the servers.
Could it be client side (firewall, libs...) or server side ?
I'm using python and windows 10.
Best regards, Denis.

Hi,
First, so sorry for getting back to you so late.
This comes from different default behavior of the open() function on Windows vs Linux/Mac.
The file needs to be opened in binary mode. Adding 'rb' to open() seems to solve the problem:
job = service.process({'image': open('.\denis.jpg', 'rb')})
We've just updated our doc accordingly.
Let me know if it is ok now.
Best regards,
Gwenn

Hi,
I faced the same problem.
Unfortunately, that does not solve the problem. I get the same error in Linux and also in Windows when opening file in binary mode.
Best regards,
Anu

Hi Gwenn,
Thanks for the update, I tried again the exact same test that I was running when I got the issue and had a different error:
{u'url': u'https://gate.angus.ai/services/gaze_analysis/1/jobs/a729c6bc-c823-11e6-9602-0242ac110002', u'status': 201, u'uuid': u'a729c6bc-c823-11e6-9602-0242ac110002', u'Error': u'Input does not appear to be valid, please check that you send well formated png or jpg images.'}
So it seems that the connection works now without any problem, but still could not open the image without 'rb' mode.
I then added 'rb' to open() like you suggested and it worked like a charm!
Tried with Windows 10, not tested with Linux.
(I though changed my office and connexion settings since last time I tried)
Anyway it's working for me now, thank you again.
Denis


Added more language support to wordCount

closes #9507

Changed the utils.wordCount implementation to the one used by simpleMDE
Added extra À-ÿ to the regex to support diacritics characters
Added corresponding text with Chinese text mentioned in the issue

LGTM 👍

Nice work~


Updating social icons

Social connections in the footer section can be hyperlinked to the respective social media icons.
@GarimaSingh0109 I would like to work on this issue.
Please assign this issue to me. Also assign the corresponding labels to this issue.

please assign me this ,i would like to work on it

please assign this to me under Gssoc extended


Web Uploader: Warn users if they are submitting an observation without an ID

From https://forum.inaturalist.org/t/warn-me-about-missing-ids-when-uploading-observations-on-website/13597
Currently, we have two warning pop-ups if a user submits observations which are lacking attributes:
You are submitting observations without photos and taxon names. Observations without this information are often impossible to identify and will not be visible by default in observation searches.
and
You are submitting observations without dates or locations. Observations without this information are often impossible to identify and will not be visible by default in observation searches.
If the user submits observations which meet both of those cases, they see both warnings, one after the other.
Users have requested that we add a warning if they have not added an identification to any of their observations to prevent them from the purgatory of taxonomic limbo. I'm loathe to add a third attribute-related warning, so my suggestion would be to change "and" to "or"  for the first pop-up and have it appear if any observation is missing a photo or a taxon name/placeholder.
You are submitting observations without photos or taxon names. Observations without this information are often impossible to identify and will not be visible by default in observation searches.
I think this makes sense...

As mentioned in the forum:

(the) error message only appeared for observations missing IDs and media evidence like a photo

and a quick glance shows that is the logic in place.
It shouldn't be difficult to add a check for taxon id (or species guess) on each observation just on the web uploader. I can start working on that.
To confirm @tiwane, display a warning "You are submitting observations without photos or taxon names. Observations without this information are often impossible to identify and will not be visible by default in observation searches." if 1) the observation does not have any files attached, or if 2) the observation does not have either a taxon id or species guess. Sound right?

@todtb's PR is working as you described, Tony, but now the error message is misleading, particularly the part about observations not being "visible by default in observation searches," which is not technically true for observations without media. How would you feel about something more verbose but accurate like this?
Some observations are missing media or identifications. Observations without media will not be visible by default in searches, and those without any identification may not be seen by experts looking for specific taxa. Even a very coarse identification like "plants" can help a plant expert find your observation.

Sounds good, thanks.


Wrong aspect ratio for VLMPlanner screenshot

I noticed that VLMPlanner is consistently doing screenshot at a wrong aspect ratio for my secondary monitor

My monitor is a 3440 x 1440 ultrawide, so this may be a rare edge case.
ShowUI, on the other hand, is doing the screenshot at the correct aspect ratio. The mouse coordinate calculation seems to be fine for the next step and the VLMPlanner screenshot issue doesn't seem to affect the model action, at least for my particular test case.

Anyways, great work Showlab team! I'm really enjoying seeing the development and just logging this minor issue here in case it is useful to you!

Hi @fayeli,
Thanks for your feedback! You're right, we actually want the screenshots passed to vlm_planner to be after resized, e.g. to 1920x1080, in order to control the inference time and token usage. We will consider visualize the image before resized (i.e. its original size) in chat interface so that it doesn't confuse the user. Thanks again for your valuable findings :)


libwacom-surface: 2.7.0 -> 2.8.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/linuxwacom/libwacom/releases.
meta.description for libwacom-surface is: Libraries, configuration, and diagnostic tools for Wacom tablets running under Linux
meta.homepage for libwacom-surface is: https://github.com/linux-surface/libwacom-surface
meta.changelog for libwacom-surface is: https://github.com/linuxwacom/libwacom/blob/libwacom-2.8.0/NEWS
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.8.0 with grep in /nix/store/6g7kd2804hgg87p83h44kgcsjdah3vc5-libwacom-surface-2.8.0
found 2.8.0 in filename of file in /nix/store/6g7kd2804hgg87p83h44kgcsjdah3vc5-libwacom-surface-2.8.0

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

4759 total rebuild path(s)

4759 package rebuild(s)

First fifty rebuilds by attrpath
AusweisApp2
ChowKick
CuboCore.coreaction
CuboCore.corearchiver
CuboCore.corefm
CuboCore.coregarage
CuboCore.corehunt
CuboCore.coreimage
CuboCore.coreinfo
CuboCore.corekeyboard
CuboCore.corepad
CuboCore.corepaint
CuboCore.corepdf
CuboCore.corepins
CuboCore.corerenamer
CuboCore.coreshot
CuboCore.corestats
CuboCore.corestuff
CuboCore.coreterminal
CuboCore.coretime
CuboCore.coretoppings
CuboCore.coreuniverse
CuboCore.libcprime
CuboCore.libcsys
EmptyEpsilon
MIDIVisualizer
OSCAR
PageEdit
SDL2
SDL2_gfx
SDL2_image
SDL2_image_2_0_5
SDL2_mixer
SDL2_mixer_2_0
SDL2_net
SDL2_sound
SDL2_ttf
SDL_audiolib
SDL_compat
SDL_gpu
Sylk
_1oom
_20kly
_7kaa
_86Box
_90secondportraits
aaxtomp3
abbaye-des-morts
abracadabra
activitywatch

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/6g7kd2804hgg87p83h44kgcsjdah3vc5-libwacom-surface-2.8.0
ls -la /nix/store/6g7kd2804hgg87p83h44kgcsjdah3vc5-libwacom-surface-2.8.0/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @dotlambda for testing.

@ryantm How can we make sure @r-ryantm updates libwacom rather than libwacom-surface?


posible to multiple account?

how to run multiple account in file .env
thanks

how to run multiple account in file .env
thanks

I think the best way is to run multiple times the script in parallel. You can have probably a folder for each IG account

how to run multiple account in file .env
thanks

@anjingadalahasu Currently, the script only accept one target from .env. However, you could run the script parallelly as suggested by @ap0l0 . Accepting multiple users from .env/command line should be added in the next version.

Closing this issue as this feature is available since v.3.0


When switched to Hierarchical Veiw, the focus of the Project View does not change when the Editor changes

Environment:

OS: Win 10/Mac/Linux
Vscode: 1.51.1
Vscode java project manager: 0.15.0

Actual:

Verify in this build. This issue is fixed, so close this issue


testfork for comments

[x] I have read the guidelines above.
[x] This PR adds a new recipe.
[ ] This PR updates an existing recipe.
[ ] This PR does something else (explain below).

@dmaticzka Thank you very much for the detailed renicing of the files. I changed them and they are now concidered part of the package.


Symlinks in modules are depricated

When the module is installed from the puppet forge, the following warning message comes up:

puppet module install jhaals-osx_notify

Warning: Symlinks in modules are unsupported. Please investigate symlink jhaals-osx_notify-0.0.4/spec/fixtures/modules/osx_notify/lib->../../../../lib.

Warning: Symlinks in modules are unsupported. Please investigate symlink jhaals-osx_notify-0.0.4/spec/fixtures/modules/osx_notify/manifests->../../../../manifests.

I don't maintain this module anymore. I would gladly accept PR's if you'd like to fix this issue.


Task25_ExtractDataFromYoutube

Extract the information from Youtube(Automate with Uipath Channel), TItle, view and month and print the result in excel

@abishekp1991
Task -25
code verified.


hidapi error: Failed to open a device with path '/dev/hidraw4': Permission denied

On Ubuntu 22.04.3 LTS, I am using a micro:bit v1 and am here: https://github.com/rust-embedded/discovery/blob/master/microbit/src/03-setup/verify.md
When I ran lsusb | grep -i "NXP ARM mbed", I got
Bus 001 Device 010: ID 0d28:0204 NXP ARM mbed
and when I ran ls -l /dev/bus/usb/001/010, I got
crw-rw-rw-+ 1 root plugdev 189, 9 Jan  2 17:22 /dev/bus/usb/001/010
so I think I did the previous instructions correctly.
From, ~/projects/discovery/microbit/src/03-setup$, I run cargo embed --target thumbv6m-none-eabi and I get
Finished dev [unoptimized + debuginfo] target(s) in 0.01s
  Config default
  Target /home/simon/projects/discovery/microbit/target/thumbv6m-none-eabi/debug/rtt-check
   Error Probe could not be created

         Caused by:
             0: hidapi error: Failed to open a device with path '/dev/hidraw4': Permission denied
             1: hidapi error: Failed to open a device with path '/dev/hidraw4': Permission denied

I found this: https://github.com/rust-embedded/discovery/issues/490 so I renamed the file to 70-microbit.rules (I'm not sure of the significance of the prefix) and changed the content as described but I got the same errors as shown above.
Update
I bought a micro:bit v2 and tried it on the same machine and it worked okay.
I also took a Raspberry Pi 4, and with a new SD card image (Debian GNU/Linux 12 (bookworm)), installed Rust and followed the instructions up to chapter 3.  The micro:bit v2 also worked on the Pi and the original v1 micro:bit also failed, this time with:
Finished dev [unoptimized + debuginfo] target(s) in 7.88s
  Config default
  Targe    Finished dev [unoptimized + debuginfo] target(s) in 7.88s
  Config default
  Target /home/simon/projects/discovery/microbit/target/thumbv6m-none-eabi/debug/rtt-check
   Error Probe could not be created

         Caused by:
             0: hidapi error: Failed to open a device with path '/dev/hidraw0': Permission denied
             1: hidapi error: Failed to open a device with path '/dev/hidraw0': Permission denied

Running ls -l /dev/hidraw0
gives:
crw------- 1 root root 242, 0 Jan  9 17:53 /dev/hidraw0

I have a micro:bit v1 only, encountered the same issue. Had to use a more general rule (which should work for all CMSIS-DAP devices) based on probe_rs rules file:
$ cat /etc/udev/rules.d/69-microbit.rules
# CMSIS-DAP for microbit
ATTRS{product}=="*CMSIS-DAP*", MODE="666", GROUP="plugdev", TAG+="uaccess"


Radio button placement to be rectified.

I don't have these screens

https://xd.adobe.com/view/ebae18ce-2577-4d53-9b75-cb9ae5317eb1-befd/
@Pawan2792

done


feat(ux): add a direct feedback link on all pages

so that readers are a click away from providing us a feedback on documentation pages

Hypothesis: this CTA will increase the amount of typos notifications and help us iterate on the doc (content or organization)
It opens the Intercom Messenger with a fallback on our support email address.

Suggestion / question: it would be possible to trigger the intercom widget using a small script apparently, I wonder if that woudl be any good ?

Yes, that's what the .intercom-launcher class does since #330!


REL: more work on release action

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

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 91.899%

Totals

Change from base Build<PHONE_NUMBER>9:
0.0%

Covered Lines:
5343

Relevant Lines:
5814

💛  - Coveralls


webpack-dev-server seems to ignore the change of ejs partial template which is "nested" in another ejs partial template.

What happened? :
I found this problem yesterday, I have an index.ejs , a footer.ejs, and a svg-sprite.ejs just like below.
index.ejs:
<!DOCTYPE html>

<%- include('./src/template/head.ejs',{title:'index'})%>

      <%- include('./src/template/footer.ejs') %>

footer.ejs:

<%- include('./svg-sprite.ejs')%>

svg-sprite.ejs:
<svg width="0" height="0" class="svg-sprites" style="display:none">
  <symbol xmlns="http://www.w3.org/2000/svg" viewBox="0 0 38 38" id="checkboard">
    <path fill-rule="evenodd" clip-rule="evenodd"
      d="M15.8334 4.9165C14.1765 4.9165 12.8334 6.25965 12.8334 7.9165H11.875C9.65131 7.9165 8.53944 7.9165 7.74073 8.45018C7.39497 8.68122 7.09809 8.9781 6.86705 9.32386C6.33337 10.1226 6.33337 11.2344 6.33337 13.4582V28.4998C6.33337 31.4854 6.33337 32.9782 7.26087 33.9057C8.18836 34.8332 9.68114 34.8332 12.6667 34.8332H25.3334C28.3189 34.8332 29.8117 34.8332 30.7392 33.9057C31.6667 32.9782 31.6667 31.4854 31.6667 28.4998V13.4582C31.6667 11.2344 31.6667 10.1226 31.133 9.32386C30.902 8.9781 30.6051 8.68122 30.2593 8.45018C29.4606 7.9165 28.3488 7.9165 26.125 7.9165H25.1667C25.1667 6.25965 23.8236 4.9165 22.1667 4.9165H15.8334ZM16.4167 7.9165C16.4167 7.36422 16.8644 6.9165 17.4167 6.9165H20.5834C21.1357 6.9165 21.5834 7.36422 21.5834 7.9165C21.5834 8.46879 21.1357 8.9165 20.5834 8.9165H17.4167C16.8644 8.9165 16.4167 8.46879 16.4167 7.9165ZM14.25 17.9998C13.6978 17.9998 13.25 18.4476 13.25 18.9998C13.25 19.5521 13.6978 19.9998 14.25 19.9998H23.75C24.3023 19.9998 24.75 19.5521 24.75 18.9998C24.75 18.4476 24.3023 17.9998 23.75 17.9998H14.25ZM14.25 24.3332C13.6978 24.3332 13.25 24.7809 13.25 25.3332C13.25 25.8855 13.6978 26.3332 14.25 26.3332H20.5834C21.1357 26.3332 21.5834 25.8855 21.5834 25.3332C21.5834 24.7809 21.1357 24.3332 20.5834 24.3332H14.25Z"
      fill="currentColor" />
  </symbol>
</svg>

You can see that svg-sprite.ejs is actually  "nested" in footer.ejs, but not in index.ejs(Here index.ejs is the template I use as an entry template for htmlWebpackPlugin).
The problem is that when I saved the change in svg-sprite.ejs,  webpack-dev-server just didn't get that change.
BTW It seemed NOT a problem about hot module reload(HMR), because even I turned off the hot option in devServer, this still happened, and I also set the watchFiles option for those partial template , but it was still not working.
Expected behavior:
Just need webpack-dev-server to watch the change of those nested partial template.

Any idea about how to fix this ?
If you need a reproducible repository for testing, feel free to ask for it :)
PS. Actually I know a little bit Japanese, I guess it's ok to read comments written in Japanese.

@dc7290
I found this problem is reproducible in your another repository.
https://github.com/dc7290/ejs-wepack-template
Just take a try :  include a new ejs file(let's call it example.ejs) inside /layout/header.ejs,  then  npm run dev, then try change something in example.ejs, you will find that webpack dev server just ignore that change.


Steam GameMode system update hanging

In game mode there appears to be a update under system apparently a "OS" Update  attempt to install just hanging at "Applying 99% / 00:01 remaining" any way to fix/remedy this? Already tried restarting the deck same issue. On Bazzite-deck:latest

The progress is faked, it's not paused at 99%, merely taking longer than our guess. Leave it and it'll finish.

The progress is faked, it's not paused at 99%, merely taking longer than our guess. Leave it and it'll finish.

Oh ok i didn't know it was faked that explains it. Thanks


[$500] Android - Chat - Unable to upload pdf file

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.44-0
Reproducible in staging?: y
Reproducible in production?: n
Issue reported by:
Action Performed:

Open the app and log in
Open any chat
Tap the "+" button > Add attachment > Choose document
Select a pdf file

Expected Result:
The preview of the PDF is displayed
Actual Result:
User is returned to the chat, the preview of the PDF is not displayed
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/7a2a01c4-c409-49a2-83b8-5ff0b67547fb
View all open jobs on GitHub

We think that this bug might be related to #vip-bills
@davidcardoza

Confirmed that this isn't reproducible on iOS staging (1.4.44-0), will test on Android dev to see if it's reproducible there.
@davidcardoza do you know of any PRs related to PDFs that were recently merged/deployed?

Hmm, I'm unable to reproduce on dev so far and don't have an android device to confirm that it's not reproducible on prod.
I don't see any changes to the AddAttachment logic that's still on staging and would be a culprit for this.
@kbecciv what account did you use to send the attachment, I'll check the logs to see if there's anything suspcious

Looks like it's coming from https://github.com/Expensify/App/pull/35164 where we are trying to get image size even though it's a PDF.

@NikkiWines Tester's account -<EMAIL_ADDRESS>

Oh yeah missed this since that PR was created back in January 😅 does indeed look like that's the likely culprit

this is fixed now! 🎊 thanks @dukenv0307 @parasharrajat

Adding a CME to see if there's any payment to be handled here 🙇

@parasharrajat Thank you for helping review this PR, I am not available at that time

Will pay out @parasharrajat for the review here. @dukenv0307 will get paid on the original issue.

$500 approved for @parasharrajat


Resolve required NODE_ENV

When starting the app in the terminal:

I think we just need to tell everyone in the channel - about https://github.com/FightPandemics/FightPandemics/commit/fb51532d0e1722437aeda9fa61de7b8515a82ce4
Unless we want to make the default to 'dev' here - https://github.com/FightPandemics/FightPandemics/blob/master/backend/config.js - (or is this risky)?

@joshmorel Ah, you're right. I forgot to implement the changes that Manny talked about. That should fix it.
We shouldn't add a default dev as mentioned here

)


English translation

Hi! Needed help in translation. Can't find how to change info above each file and title of storage/emulated/0. These values I want on English.

Hi! Needed help in translation. Can't find how to change info above each file and title of storage/emulated/0. These values I want on English.

First remove the "io.github.molihuan:pathselector" dependency from your project.
Then download the source code, unzip the package, open "build.gradle" under the pathselector folder, remove the (->id 'com.kezong.fat-aar'<-) and (->apply from: "${getRootDir().absolutePath}/gradles/publish- maven.gradle"<-), modify (->embed(name: 'XXPermissions-13.6', ext: 'aar')<-) to (->implementation fileTree(dir: "libs", includes: [".aar", ".jar"] ) <-) .
then open your project and pass the pathselector folder into your project by way of a module, then making sure that project build.gradle (no app or pathselector build.gradle) contains the remote repository:
allprojects {
repositories {
...
mavenCentral()
maven { url 'https://jitpack.io' }
}
}
Finally sync Now Get the dependencies
After that you can translate to English by modifying the source code
For example, you search for "内部存储" and change it to "storage"

I thought about changing source code, but hoped that there is another way. Thank for your answer!

I thought about changing source code, but hoped that there is another way. Thank for your answer!

There is an English translation now


ProgressBar: Enhancement ideas

Support Height > 1
Support a Text property that would be centered. Clients could do  pc.Text = "${pc.Fraction:F0}%".
Support a SegmentCharacter property that would allow caller to specify the character drawn for each segment (currently hard coded to Driver.Stipple).
Add a marquee mode (like https://docs.microsoft.com/en-us/dotnet/api/system.windows.forms.progressbarstyle?view=netcore-3.1). Text would scroll in marquee fashion.
It should render more detail depending on the height. 1 line, current design; 2 progress plus percentage; 3 framed; 4 frame with progress; 6 frame wothnprogress padded (via #52).

Support a SegmentCharacter property that would allow caller to specify the character drawn for each segment (currently hard coded to Driver.Stipple).

Would be very great so we can actually draw opaque progress bar then.

Guys, what do you think about this innovation?

The continuous and opaque one is the one that I actually needed that will be excellent!

All styles are good and I like them all!

I intend to added options to print fraction text and framed before submit a PR.

* Support a `Text` property that would be centered. Clients could do  `pc.Text = "${pc.Fraction:F0}%"`

I don't agree with this be a Text property because the ProgressBar already have the fraction value and must be him to handle this and print automatically, case for e.g. a FractionText enum property (None, Inside, Outside) is set to Inside or Outside. What yours opinions about this?

* Support a `SegmentCharacter` property that would allow caller to specify the character drawn for each segment (currently hard coded to `Driver.Stipple`).

I added support to this but I do not guarantee that will work with all the SegmentCharacter choosed.

* 6 frame wothnprogress padded

I didn't understand this part. Can someone explains, please?

* 6 frame wothnprogress padded

I didn't understand this part. Can someone explains, please?

Originally from @migueldeicaza at https://github.com/migueldeicaza/gui.cs/issues/52

Originally from @migueldeicaza at #52

Yes I know that, thanks. But is the same thing. I'll ignore this part.

Done folks. #546

Glorious!


Your error stating that comingSoon isn't defined on https://orion.emca.app

I don't usually use github so pardon me for doing this under the issues thing. When I was going through and pushing this through debug that's when I found that it wasn't defined, I went to the melee.js file and it seems you just forgot to define the comingSoon variable although you probably already know but here's the definition "var comingSoon = [];"

#115 Created this pull request @C0deS3rp3nt

Fixed by #115. Thanks for reporting! 😄


[Feature Request] Menu Builder Route Prefixes

After playing around with everything since the last time I took a look, I wanted to check and see if this would be something that people might want.
As it stands, installing Voyager will drop a small bit of code into the Routes file with a nice Prefix that a developer can go in and change at their leisure. The problem is that after installation if somebody wants to change this prefix, all seeded admin menu items won't reflect the new change:

Install Voyager
Change the route prefix in routes/web.php to something else
Log into Voyager
Click a Menu Item

The only way a developer can change the route prefix is by first changing each menu item route by hand (before changing the prefix), and then adjusting the prefix in the routes file.
For new users, this approach may not seem apparent and could cause confusion when changing the prefix first and being presented with a NotFoundHttpException after clicking any of the admin menu items.
This can be remedied fairly easily by a few approaches:

Add an additional prompt during the installation process to allow the user to install Voyager on a nonstandard route with admin being the default
Add an optional prefix input under the Menu Editor that will prefix all (?) menu items with the designated menu prefix (should one exist)
Add an additional artisan command that will change the routes/web.php file as well as the seeded menu items to reflect the new change (ie: voyager:routes:prefix)

I see Item 1 being a good start, and perhaps the easiest in terms of getting Voyager set up with the desired routes.
I see Item 3 being more complicated to implement due to rewriting the routes/web.php reliably, but would ultimately streamline the prefixing process nicely
The only one I'm unsure about is Item 2, as this may have unintended consequences for users who want to include prefixed routes with external URLs in their menus.
Thoughts?

This has been discussed a few times in the past.  The method we've discussed in the past is to give menu items the option of storing a static URI or a route name.  So you could store the link as "admin.users.index" as your link, and it would render as "/admin/users".
I'm not sure if @marktopper has made any progress on this, so I'll let him respond.

The menu builder already supports routes, however we still need to make the seeder for the menu items to use routes as well in order to solve this.


[dappnode-dao] Add dappnode-dao strategy

Adds dappnode-dao strategy:

3 unipool contracts in target network: 2 with LP token, 1 with same token
1 unipool contract in xDAI with same token

@dapplion there is some error while running the strategy:
Test strategy "dappnode-dao" › Strategy should run without any errors

    call revert exception (method="totalSupply()", errorArgs=null, errorName=null, errorSignature=null, reason=null, code=CALL_EXCEPTION, version=abi/5.3.1)

      731 |                 case 2:
      732 |                     _a = _b.sent(), res = _a[1];
    > 733 |                     return [2 /*return*/, res.map(function (call, i) { return itf.decodeFunctionResult(calls[i][1], call); })];
          |                                                                                ^
      734 |                 case 3:
      735 |                     e_2 = _b.sent();
      736 |                     return [2 /*return*/, Promise.reject(e_2)];

Hey @dapplion, I think the error is because your fork branch is outdated from the master branch (many big changes pushed recently), can you please update your branch with those, that should fix this weird issue :)

@dapplion this strategy works for you in your local? Im getting this error
    call revert exception (method="totalSupply()", errorArgs=null, errorName=null, errorSignature=null, reason=null, code=CALL_EXCEPTION, version=abi/5.3.1)

try running npm run test --strategy=dappnode-dao and make sure all test cases are passing,
I think we are getting this error from unipoolUniv2LpStrategy which is being used in this strategy, maybe something is not being passed correctly, please check :) Thanks

Thanks for looking into it! I've been debugging the strategy but I don't see why the multicall fails. I can call totalSupply() on the conctract individually but though your script's multicall it reverts. Have you encounter issues like this in the past?

@dapplion Closing this for now, please feel free to reopen this/to add a new strategy :) Thanks

Thanks, DAppNode dao just uses a multichain strategy right now so this one is not necessary anymore, props to the nice modular design :+1:

Thanks, DAppNode dao just uses a multichain strategy right now so this one is not necessary anymore, props to the nice modular design 👍

@dapplion Cool then thank you :)


Manual Bundle "Drag and Drop" Source File Directory Failure

Xcode can't find multiple source files in CLImageEditor when it's implemented manually. I'm aware that the README instructed to drag and drop the source files from the demo into the Xcode project. (I'm not doing this via Cocoapods for personal reasons). Whenever I drag the project files (including the bundle) I persistently get an error saying that CLImageEditorViewController.h/.m is missing. Any ideas?

Have you checked Create groups?

@yackle sorry, how do you mean by that?

@yackle Thanks, I resolved this issue by adding the files with the "Create Groups" option!


description of the repo contains typo

Yet another space temed idle game -> Yet another space themed idle game

Thanks. Fixed.


Ubuntu 16.10: fsi.exe System.TypeInitializationException on startup

After building on Linux, fsi.exe and fsc.exe immediately exit with a fatal error on the current master (22ca784796d5f0fdd017246b25feca5c397ec2d1).
A full log and uname are below.
The complaint is:
Make sure "FSCore.resources" was correctly embedded or linked into assembly "FSharp.Core" at compile time, or that all the satellite assemblies required are loadable and fully signed.

but this assembly is present:
$  ls release/net40/bin/
default.win32manifest                         FSharp.Core.sigdata
fsc.exe*                                      FSharp.Core.xml
fsc.exe.config                                fsiAnyCpu.exe*
fsc.exe.mdb                                   fsiAnyCpu.exe.config
fsc.xml                                       fsiAnyCpu.exe.mdb
FSharp.Build.dll*                             fsiAnyCpu.xml
FSharp.Build.dll.mdb                          fsi.exe*
FSharp.Build.xml                              fsi.exe.config
FSharp.Compiler.dll*                          fsi.exe.mdb
FSharp.Compiler.dll.mdb                       fsi.xml
FSharp.Compiler.Interactive.Settings.dll*     Microsoft.DiaSymReader.dll*
FSharp.Compiler.Interactive.Settings.dll.mdb  Microsoft.DiaSymReader.PortablePdb.dll*
FSharp.Compiler.Interactive.Settings.xml      Microsoft.FSharp.targets
FSharp.Compiler.Server.Shared.dll*            Microsoft.Portable.FSharp.targets
FSharp.Compiler.Server.Shared.dll.mdb         Microsoft.VisualStudio.Shell.14.0.dll*
FSharp.Compiler.Server.Shared.xml             Microsoft.VisualStudio.Shell.Immutable.10.0.dll*
FSharp.Compiler.xml                           System.Collections.Immutable.dll*
FSharp.Core.dll*                              System.Reflection.Metadata.dll*
FSharp.Core.dll.mdb                           System.ValueTuple.dll*
FSharp.Core.optdata

Repro steps

$ ./build.sh
$ ./release/net40/bin/fsi.exe # or fsc.exe

Expected behavior
An interactive prompt is presented.
Actual behavior
The error below is displayed:
$ release/net40/bin/fsi.exe # or fsc.exe
System.TypeInitializationException: The type initializer for '<StartupCode$FSharp-Compiler>.$Microsoft.FSharp.Compiler.AbstractIL.Internal.Library' threw an exception. ---> System.TypeInitializationException: The type initializer for 'FastGenericComparerTable`1' threw an exception. ---> System.TypeInitializationException: The type initializer for '<StartupCode$FSharp-Core>.$Prim-types' threw an exception. ---> System.Resources.MissingManifestResourceException: Could not find any resources appropriate for the specified culture or the neutral culture.  Make sure "FSCore.resources" was correctly embedded or linked into assembly "FSharp.Core" at compile time, or that all the satellite assemblies required are loadable and fully signed.
  at System.Resources.ManifestBasedResourceGroveler.HandleResourceStreamMissing (System.String fileName) [0x000cf] in <8f2c484307284b51944a1a13a14c0266>:0
  at System.Resources.ManifestBasedResourceGroveler.GrovelForResourceSet (System.Globalization.CultureInfo culture, System.Collections.Generic.Dictionary`2[TKey,TValue] localResourceSets, System.Boolean tryParents, System.Boolean createIfNotExists, System.Threading.StackCrawlMark& stackMark) [0x00110] in <8f2c484307284b51944a1a13a14c0266>:0
  at System.Resources.ResourceManager.InternalGetResourceSet (System.Globalization.CultureInfo requestedCulture, System.Boolean createIfNotExists, System.Boolean tryParents, System.Threading.StackCrawlMark& stackMark) [0x000ad] in <8f2c484307284b51944a1a13a14c0266>:0
  at System.Resources.ResourceManager.InternalGetResourceSet (System.Globalization.CultureInfo culture, System.Boolean createIfNotExists, System.Boolean tryParents) [0x00002] in <8f2c484307284b51944a1a13a14c0266>:0
  at System.Resources.ResourceManager.GetString (System.String name, System.Globalization.CultureInfo culture) [0x0006a] in <8f2c484307284b51944a1a13a14c0266>:0
  at <StartupCode$FSharp-Core>.$Prim-types..cctor () [0x00010] in <583b13cfdff9fae1a7450383cf133b58>:0
   --- End of inner exception stack trace ---
  at (wrapper managed-to-native) System.Object:__icall_wrapper_mono_generic_class_init (intptr)
  at Microsoft.FSharp.Core.LanguagePrimitives.get_StringComparer () [0x00000] in <583b13cfdff9fae1a7450383cf133b58>:0
  at Microsoft.FSharp.Core.LanguagePrimitives..cctor$cont@2335-2[T] (System.Type matchValue, Microsoft.FSharp.Core.Unit unitVar) [0x000b3] in <583b13cfdff9fae1a7450383cf133b58>:0
  at Microsoft.FSharp.Core.LanguagePrimitives+FastGenericComparerTable`1[T]..cctor () [0x000b0] in <583b13cfdff9fae1a7450383cf133b58>:0
   --- End of inner exception stack trace ---
  at (wrapper managed-to-native) System.Object:__icall_wrapper_mono_generic_class_init (intptr)
  at <StartupCode$FSharp-Compiler>.$Microsoft.FSharp.Compiler.AbstractIL.Internal.Library..cctor () [0x00021] in <583b14477f81fe20a745038347143b58>:0
   --- End of inner exception stack trace ---
  at (wrapper managed-to-native) System.Object:__icall_wrapper_mono_generic_class_init (intptr)
  at Microsoft.FSharp.Compiler.Interactive.Shell.evaluateSession@2800 (System.String[] argv, Microsoft.FSharp.Core.Unit unitVar0) [0x00049] in <583b1468046a099ca745038368143b58>:0
  at Microsoft.FSharp.Compiler.Interactive.Shell.MainMain (System.String[] argv) [0x0007d] in <583b1468046a099ca745038368143b58>:0

$ uname -a
Linux 4.8.0-27-generic #29-Ubuntu SMP Thu Oct 20 21:03:13 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux

Just an update that this is fixed in http://github.com/fsharp/fsharp, but the fixes need to be back ported here

The fix was back-ported a while back


HTML: certain tags (e.g. data, meter, time) should have an Emmet abbreviation, while others (e.g. datagrid) shouldn't

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: Version: 1.78.2 (user setup) b3e4e68a0bc097f0ae7907b217c1119af9e03435
OS Version: Windows_NT x64 10.0.22621

Steps to Reproduce:
In an HTML document:

Type data, meter, time and notice none of them have an Emmet abbreviation
Type datag and notice it's an Emmet abbreviation for datagrid, even though that's a nonexistent HTML tag (apparently it was part of a draft version of HTML5)

Emmet does't works

i also have this problem

Emmet is breaking while using

Hi @anthonyvdotbe, I'm wondering what meter should expand to? Should it include all the attributes?
Also, the snippets live in the emmetio/emmet repository. Marking this as an upstream issue.

Hi @anthonyvdotbe, I'm wondering what meter should expand to? Should it include all the attributes?

Hi @rzhao271, I'd prefer if it would just include the value attribute

Also, the snippets live in the emmetio/emmet repository. Marking this as an upstream issue.

Thanks for pointing this out, I've created an issue there

This issue seems to have been resolved as part of https://github.com/microsoft/vscode/pull/180583.


Set provisioning state to read only, and fix force update property in 2015-06-15 version

@hyonholee - Can you fix the issue mentioned here https://github.com/Azure/azure-rest-api-specs/issues/352 at the same time? It should be a really simple fix.

@lmazuel the Travis CI is now failing with this error:
The command "if [[ $MODE == 'python' ]]; then $DOCKER_CMD AutorestCI/azure-sdk-for-python --pr-repo-id Azure/azure-sdk-for-python -o dev -v; fi" exited with 1.
Can you please take a look?

@John-Hart Yes, I answered in the email thread I'm on it :)

Test passed. Can it be merged? Thanks

LKTM

No modification for NodeJS

No modification for Python


Add feature: Keypress events

whenKeyPressed(keyValue)
//returns a Promise that should get resolved whenever that key is pressed

/*Example Usage*/
whenKeyPressed("a").then({
    //do something
})

I've already worked on this. Should be easy for me to do.

Quick question, are we going to wait until the key is pressed and then execute the code, or are we going to skip over it and wait for it like an event?

Not really sure what you mean, but it should be using an event listener.
On Sun, Mar 5, 2017 at 8:55 PM Baran Skistad<EMAIL_ADDRESS>wrote:

Quick question, are we going to wait until the key is pressed and then
execute the code, or are we going to skip over it and wait for it like an
event?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Scratch-JS/Scratch-JS/issues/39#issuecomment-284284370,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGck4l0k7U0NCrwuUlxvVr9IkJ0YOwFgks5ri2eogaJpZM4MTTJ-
.

Nvm, gtg, ttyl

I ran out of things to do, I'm gonna do this too.

Promises could only be resolved once, so I'm gonna implement it differently.
whenKeyPressed {
    //do something
    //anything here can access an object called event
    //for example:
    say(event.key)
}


update electron version

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


Reworked internals and added score query

interface to scout should remain the same
New way to query without scout
added function score query (docs)

The only thing I do not yet understand is how I can use the function score query with Scout. I would expected to be able to use the newCompound() function on the Scout builder, but that is forced to be a Bool Query right now.

The only thing I do not yet understand is how I can use the function score query with Scout. I would expected to be able to use the newCompound() function on the Scout builder, but that is forced to be a Bool Query right now.

Not with scout, you must use your

The only thing I do not yet understand is how I can use the function score query with Scout. I would expected to be able to use the newCompound() function on the Scout builder, but that is forced to be a Bool Query right now.

Not with scout directly you would need to inject the ElasticAdapter and use Search with the SeachCommand. If you want to be able to do it from scout we would need to change the whole compound thing, because most things aren't working on these other query . (a score function doesn't have an must, should ,filter and so on)

The only thing I do not yet understand is how I can use the function score query with Scout. I would expected to be able to use the newCompound() function on the Scout builder, but that is forced to be a Bool Query right now.

Not with scout directly you would need to inject the ElasticAdapter and use Search with the SeachCommand. If you want to be able to do it from scout we would need to change the whole compound thing, because most things aren't working on these other query . (a score function doesn't have an must, should ,filter and so on)

Got it, that's a nice-to-have for the future then. Thanks!


Dollar sign in postgresql identifier causes code generate error

Version
1.11.0
What happened?
When a function name is defined using a dollar sign ($) in the schema, sqlc reports an error:
error generating code: source error: 30:6: missing variable type or initialization (and 4 more errors)

Because the generator emits the identifier into the Go source:
func (q *Queries) Add(ctx context.Context, ) (int32, error) {
	row := q.db.QueryRowContext(ctx, add, )
	var f$n int32
	err := row.Scan(&f$n)
	return f$n, err
}

The PostgreSQL documentation states:

SQL identifiers and key words must begin with a letter (a-z, but also letters with diacritical marks and non-Latin letters) or an underscore (_). Subsequent characters in an identifier or key word can be letters, underscores, digits (0-9), or dollar signs ($).

4.1.1. Identifiers and Key Words
Additional points:

It seems that all identifiers are affected by this, not only function names. So even column names in table definitions are causing this error.
According to the PostgreSQL documentation, the dollar sign is not SQL standard compatible. So in the first place the question if identifiers with dollar signs should be reported as invalid or properly translated to valid go identifiers.
Interestingly the PostgresSQL identifier could also contain non-latin letters (e.g. áäâ...) but this matches the go language specification so these are fine. However I do not know if other language targets are affected by this.
Looking through the code the function names are not really relevant. So the naming of these could be quite simply changed. On the other side the column names are differently, as they are of course part of the public model. So possibly it would make sense to accept function names with dollar signs and reject column names with dollar signs.

Relevant log output
No response
Database schema
No response
SQL queries
CREATE FUNCTION f$n() RETURNS integer AS 'SELECT 1';

-- name: Fn :one
SELECT f$n();

Configuration
{
  "version": "1",
  "packages": [
    {
      "path": "db",
      "engine": "postgresql",
      "schema": "query.sql",
      "queries": "query.sql"
    }
  ]
}

Playground URL
https://play.sqlc.dev/p/5920ab6d1c170a37455d6c5bab505f22c5d660645d96a465feffb559571fd221
What operating system are you using?
Linux, Windows
What database engines are you using?
PostgreSQL
What type of code are you generating?
Go

This has been fixed https://play.sqlc.dev/p/76509f5d006e5efa8d1d4cf4775022955aa5c2c46d45014ac5587cf44ce977a9


Replace Connection.RemoteAddr() by NetConn()

I am implementing TLS in the example server and found that I need access to the underlying net.Conn. I added NetConn() method to access it. Now that we have access to net.Conn, RemoteAddr() is no longer needed, it can be substituted by NetConn().RemoteAddr().
Note: this is a breaking change.

I think the change is good. I would prefer we use the name "Connection" or "NetworkConnection" (seems redundant), rather than match the package/struct name of the connection. I will let you decide.

Just to be clear, you suggest Connection() as the method name instead of NetConn()?

Yes


Is Gnet Framework capable of Multi Port Listen?

I am a user who is paying close attention to the Gnet framework.
We open two ports for security
one for external devices
One wants to use only the inside
Is there a way to listen to the two ports of Gnet?

Start two goroutines and call gnet.Serve() to listen on two ports.

Close this for inactivity.


add prediction and evaluation steps to run.all.sh

It is currently tedious to reproduce the evaluation in the paper because run.all.sh does not invoke run.predict.sh nor run.evaluate.sh.
This can be done as follows:
$ diff --context=1 /tmp/SkillSpan/scripts/run.all.sh .
...
               scripts/run.experiment.sh $model $exp
+
+              for set in dev test
+              do
+                         scripts/run.predict.sh $model $exp $set
+              done
+
+              scripts/run.evaluate.sh $model

note: This will produce the necessary data files needed for #3 (e.g., data/bert/skills.tech.test.2.out).

Thanks a lot again, Tom. It was more intended to do it separately. As shown in the QA of the authors of MaChAmp. However, feel free to keep using your version of the run.all.sh script.