XLA Reshape_test failing for 3rd party platforms

I have opened a discussion on the XLA dev mailing list, but I think that this counts as an issue since if it isn't fixed then potentially all XLA tests will migrate to being unusable for 3rd party devices.
https://groups.google.com/forum/#!topic/xla-dev/9cY21Hi0s_s
The issue is that the code in reshape_test.cc assumes that any devices which are not CPU/CPU_PARALLEL/GPU will use the bfloat16 format.  This isn't true for the Graphcore device.

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

Hi,
No problem.
OS Platform and Distribution:

all platforms and distrubutions

Tensorflow installed from:

source

Tensorflow version:

master branch of repo

Bazel version:

unknown / not relevent

CUDA/cuDNN version:

none

GPU model and memory:

none

Exact command to reproduce:

bazel test --test_env TF_CPP_MIN_VLOG_LEVEL=1  --verbose_failures --test_output=all --nocache_test_results --config=opt tensorflow/compiler/xla/tests:reshape_test_poplar

this only works if you have the Graphcore extensions which you will not.  see issue on the XLA dev board.

Hi David, thanks for reporting this issue. We have committed a workaround to disable bfloat16 tests for unknown backends by default:
https://github.com/tensorflow/tensorflow/commit/557e0ce1ca949abb0ffdcccc6ab0480e65ea9fe1

Thakns @ukoxyz . @DavidNorman I think that should solve your problem. Feel free to reopen if it doesn't.


Construct an all-encompassing admin interface for Lynbrook App

After some thinking, I've realized a core few steps which absolutely should be taken into account:
Currently, for announcements to be made, one will have to manually insert them into a database. For non tech-savvy people, it is a huge burden for them to learn Linux commands or the like. Also, for event postings, and other stuff, we have to do something similar.
This is all about usability. Therefore, I propose that we also build a website for the app powered by Github Pages, while connected to Oliver's database (Firebase). In effect, all announcements can be added through this website. At a high level standpoint, the website is a platform from which clubs and other admins can control the database without having to learn complicated bash commands.
This is I believe absolutely mandatory for this app, and something that the Monta Vista app lacks. We wish to require usability above all else, and this would be an important step towards accomplishing that.

@mzhao80 @oliver-ni  I think this is pretty important, and should be prioritized. We need to be alleviate the burden for the user, and avoid as much as possible having them  to go through the hassle of doing all these manual database insertions.

We probably can extends this to creating surveys on here, as well as include some features for users and the Lynbrook student community in general. (Not necessarily restrict this to administratros)

Note: This is already partially complete with this repository:
https://github.com/lynbrookhs/lynbrook-app-admin
We just need to enable creation of surveys and other features.

please ask about the project status before making issues like this


GH-14992: [Packaging] Make dev/release/binary-task.rb reusable from other project

I'll use dev/release/binary-task.rb from apache/arrow-adbc.

Closes: #14992

@github-actions crossbow submit -g linux

@github-actions crossbow submit -g linux

@github-actions crossbow submit -g linux

@github-actions crossbow submit -g linux

+1


Training with my own data - Input Image size

Hi @lkeab
Thanks for your work. It is expected to be helpful for my work.
Btw, I have a question for you. According to your paper, input size is (600, 900).
Is it fixed? So then, I have to prepare my data fitting to it?
Sincerely
Dek

I don't think you need to fit your images to a specific size, like coco has various image sizes. During training, the image sizes will be adjusted by the data loader automatically.


Removed huge duplication of ansible_user and become settings across hosts

Hi @devproof / MapR
Thanks very much for this repo - we are starting to use it at a large financial client of MapR and would like to start fixing up some of the things to make it easier to use across environments.
This first pull request is merely a clean up of a lot of duplicated settings in host inventory files. It doesn't seem necessary to have all these redundant ansible_user/become/become_method settings as there shouldn't be any difference between cluster hosts, although if there were then it could still be overridden on a per host basis.
Thanks
Hari

Thinking about this more, I'm wondering if you are putting all these duplicate ansible_user in the hosts inventories to keep them out of group_vars... but I think since the user has to edit groups_vars/all anyway to set the correct repo URLs and other settings it should be ok, otherwise you can comment them out and just enable for your use or inherit from environment.


are labels in YAML config file ACL specifications in the latest docker image

I just started using docker_auth today - worked quickly out of the box, that is until I tried to use the labels as exampled in https://github.com/cesanta/docker_auth/blob/master/docs/Labels.md, but I could not get them to work.  I tried both the stable (:1) and :1.3 tags of the docker image, but neither seemed to work.  I didn't dig any further, such as trying a source build, but will if someone can tell me that it should be working.  I can of course provide more details if needed as well.

Can you post your config?
On Nov 6, 2017 8:35 PM, "Eric Engstrom"<EMAIL_ADDRESS>wrote:

I just started using docker_auth today - worked quickly out of the box,
that is until I tried to use the labels as exampled in
https://github.com/cesanta/docker_auth/blob/master/docs/Labels.md, but I
could not get them to work. I tried both the stable (:1) and :1.3 tags of
the docker image, but neither seemed to work. I didn't dig any further,
such as trying a source build, but will if someone can tell me that it
should be working. I can of course provide more details if needed as well.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cesanta/docker_auth/issues/202, or mute the thread
https://github.com/notifications/unsubscribe-auth/AC1fSUvZFpphxaV1SyYNTN8mXhHmBY9Wks5sz8_7gaJpZM4QUN8n
.

Config:
server:
  addr: ":5001"
  certificate: "/certs/cert-bundle.pem"
  key: "/certs/cert.key"

token:
  # Issuer *MUST* match issuer in the Registry config.
  issuer: "Docker Auth Server"
  expiration: 900

users:
  # Password is specified as a BCrypt hash.
  # Use `htpasswd -B` to generate.
  # For 'labels' (e.g. groups/projects) see also
  # -- https://github.com/cesanta/docker_auth/blob/master/docs/Labels.md

  "test":
     password: "$2y$05$WuwBasGDAgr.QCbGIjKJaep4dhxeai9gNZdmBnQXqpKly57oNutya"  # 123
     labels: {
        group: [ "group1" ],
        project: [ "proj1", "proj2" ],
        tier: [ "foo", "bar" ]
     }
     # Also tried this:
     #labels:
     #  group:
     #   - "group1"
     #  project:
     #    - "proj1"
     #    - "proj2"

  # Allow anonymous (no "docker login") access
  #"": {}

acl:
  # First match counts, so put most specific first

  # --- CATALOG - Allow catalog access (for docker-ls, e.g.)
  - match: {account: "/.+/", type: "registry", name: "catalog"}
    actions: ["*"]
    comment: "Logged in users can query the catalog."

  # --- hello-world (testing)
  # ANY (incl. anonymous) pull of hello-world (only if anon access allowed).
  - match: {name: "hello-world", account: ""}
    actions: ["pull"]
    comment: "Anonymous in users can pull \"hello-world\"."

  # --- Project access, by project, and account (regexp)
  # Want to be able to use labels like this:
  #
  # Allow project access for ALL users
  - match: {name: "${labels:project}/*"}
    actions: ["push", "pull"]
    comment: "Users can push/pull to any project to which they are assigned."

  ## Allow user/account access in a group access to user-specific images:
  - match: {account: "/.+/", name: "${labels:group}/${account}/*"}
    actions: ["*"]
    comment: "Logged in users have full access to images that are in their personal 'group/namespace'"

  #ADMIN
  - match: {account: "admin"}
    actions: ["*"]
    comment: "Admin has full access to everything."

  # DEFAULT: Access is denied by default.

@carsonoid go ahead with the PR, i can make a point release with the fix

Sounds good. I'll work on that feature add.
On Wed, Nov 15, 2017 at 12:52 AM, Deomid Ryabkov<EMAIL_ADDRESS>wrote:

@carsonoid https://github.com/carsonoid go ahead with the PR, i can
make a point release with the fix
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cesanta/docker_auth/issues/202#issuecomment-344512689,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC1fSXbc8T50eNk2nC2bWvGOqn7-8BDsks5s2pgygaJpZM4QUN8n
.

@rojer PR #206 fixes this. and PR #199 fixes another small bug when using labels in regex

i pushed -latest, will let it brew there for a while before cutting 1.3


Style/MultilineMethodCallIndentation and IndentationWidth

#2560 was recently fixed with "Indentation width can be overridden for individual cops", among these also Style/MultilineMethodCallIndentation (@bbatsov thanks a lot!). This specific cop does not work properly, it seems to use a IndentationWidth of zero:
Style/MultilineMethodCallIndentation:
  IndentationWidth: 4

$ cat tmp/indentation_test.rb
class IndentationTest

  def new
    @something = Someclass
                     .where(kind: :hello)
                     .includes(:other_class)
                     .order(created_at: :desc)
  end

end

$ bundle exec rubocop --only Style/MultilineMethodCallIndentation tmp/indentation_test.rb
Inspecting 1 file
C

Offenses:

tmp/indentation_test.rb:5:22: C: Align .where with Someclass on line 4.
                     .where(kind: :hello)
                     ^^^^^^
tmp/indentation_test.rb:6:22: C: Align .includes with Someclass on line 4.
                     .includes(:other_class)
                     ^^^^^^^^^
tmp/indentation_test.rb:7:22: C: Align .order with Someclass on line 4.
                     .order(created_at: :desc)
                     ^^^^^^

1 file inspected, 3 offenses detected

You are using the default aligned style rather than indented.

Adding a validation check which will prevent others from making the same mistake.

Oh, okay. But when using:
Style/MultilineMethodCallIndentation:
  EnforcedStyle: indented
  IndentationWidth: 4

…I end up with:
$ bundle exec rubocop --only Style/MultilineMethodCallIndentation tmp/indentation_test.rb
Inspecting 1 file
C

Offenses:

tmp/indentation_test.rb:5:22: C: Use 4 (not 17) spaces for indenting an expression in an assignment spanning multiple lines.
                     .where(kind: :hello)
                     ^^^^^^
tmp/indentation_test.rb:6:22: C: Use 4 (not 17) spaces for indenting an expression in an assignment spanning multiple lines.
                     .includes(:other_class)
                     ^^^^^^^^^
tmp/indentation_test.rb:7:22: C: Use 4 (not 17) spaces for indenting an expression in an assignment spanning multiple lines.
                     .order(created_at: :desc)
                     ^^^^^^

1 file inspected, 3 offenses detected

Which setting would I need to get rubocop to accept the style in my initial post? Thanks.

The problem now is that with an assignment statement, indented style indents relative to the LHS of the assignment.
We have an issue open to add a style like the one you want. I think it's the very last issue in the tracker right now....

OK, it's #1646.

Oh I see! Yes, I think you're describing the style I've been looking for (which I think Rubymine started using in recent versions per default, but maybe it was always like this and I didn't notice it).


Updated the room.py log for clarity

@chuckhays PTAL

LGTM.

Couldn't all these be a bit more concise like,
'remove_room_direct: ' + client_gcm_id + ' not in room ' + room_id
I'm happy to take a stab at it.
@wzh-- normally you should squash these commits and force push to your branch. I can show you how in person if you want to know but you have to make sure you're pushing to your repo and not the apprtc one :smiling_imp:

Or it could be,
'Can\'t remove room: ' + client_gcm_id + ' not in room ' + room_id
Anyways, just a suggestion.

I think we need to do some logging work overall to standardize what we report and how/where, probably building on what you've done @milkypostman.

@milkypostman it'd be great if you can show me how to squash these commits, when you have time. thx.

@milkypostman @chuckhays Chuck showed me how to do it. Thanks.

Merge the quick fix for now. It's useful for the client/server debugging. As @chuckhays said, a more complete solution might be needed for a clean logging.


djxl: Netpbm output requires specific file extension for each pixel format

When decoding to Netpbm image formats, djxl requires user to specify different file extensions for RGB, grayscale and images with alpha. Specifically, .ppm is required for RGB and  .pgm is required for grayscale. Additionally, .pam can only be used for images with alpha channel.
djxl should allow .pnm file extension to support both RGB and grayscale images. For comparison, .pfm extension can be used for both RGB and graysdcale images.
The behavior when using .pam extension seems to be a bug, .pam should support RGB and grayscale, with or without alpha.
Here are test files for convenience:
c0.jxl.gz Grayscale, 8-bit
c2.jxl.gz RGB, 8-bit
c4.jxl.gz Grayscale+Alpha, 8-bit
c6.jxl.gz RGB+Alpha, 8-bit
Here are outputs of some tests:
Grayscale image can be decoded using .pgm and .pfm, other file extensions will fail.
d:\>djxl c0.jxl c0.ppm
JPEG XL decoder v0.9.0 136cd81 [SSE4,SSSE3,SSE2]
SelectFormat failed
DecompressJxlToPackedPixelFile failed

d:\>djxl c0.jxl c0.pnm
JPEG XL decoder v0.9.0 136cd81 [SSE4,SSSE3,SSE2]
can't decode to the file extension '.pnm'

d:\>djxl c0.jxl c0.pam
JPEG XL decoder v0.9.0 136cd81 [SSE4,SSSE3,SSE2]
Decoded to pixels.
Encode failed

d:\>djxl c0.jxl c0.pgm
JPEG XL decoder v0.9.0 136cd81 [SSE4,SSSE3,SSE2]
Decoded to pixels.
320 x 240, 11.382 MP/s [11.38, 11.38], 1 reps, 4 threads.

d:\>djxl c0.jxl c0.pfm
JPEG XL decoder v0.9.0 136cd81 [SSE4,SSSE3,SSE2]
Decoded to pixels.
320 x 240, 11.120 MP/s [11.12, 11.12], 1 reps, 4 threads.

RGB image can be decoded using .ppm and .pfm, but .pnm and .pam will fail.
d:\>djxl c2.jxl c2.pnm
JPEG XL decoder v0.9.0 136cd81 [SSE4,SSSE3,SSE2]
can't decode to the file extension '.pnm'

d:\>djxl c2.jxl c2.pam
JPEG XL decoder v0.9.0 136cd81 [SSE4,SSSE3,SSE2]
Decoded to pixels.
Encode failed

d:\>djxl c2.jxl c2.ppm
JPEG XL decoder v0.9.0 136cd81 [SSE4,SSSE3,SSE2]
Decoded to pixels.
320 x 240, 8.676 MP/s [8.68, 8.68], 1 reps, 4 threads.

d:\>djxl c2.jxl c2.pfm
JPEG XL decoder v0.9.0 136cd81 [SSE4,SSSE3,SSE2]
Decoded to pixels.
320 x 240, 8.377 MP/s [8.38, 8.38], 1 reps, 4 threads.

.pam file extension works well for images with alpha channel.
d:\>djxl c4.jxl c4.pam
JPEG XL decoder v0.9.0 136cd81 [SSE4,SSSE3,SSE2]
Decoded to pixels.
320 x 240, 9.644 MP/s [9.64, 9.64], 1 reps, 4 threads.

d:\>djxl c6.jxl c6.pam
JPEG XL decoder v0.9.0 136cd81 [SSE4,SSSE3,SSE2]
Decoded to pixels.
320 x 240, 8.812 MP/s [8.81, 8.81], 1 reps, 4 threads.

Thanks for bringing this up!
After the two changes #2869 and #2878, the following should work:

File Format
Without Alpha

With Alpha

Grayscale, 8-bit
RGB, 8-bit
Grayscale+Alpha, 8-bit
RGB+Alpha, 8-bit

png
✔
✔
✔
✔

apng
✔
✔
✔
✔

jpeg
✔
✔
✔
✔

pnm
✔
✔
✔
✔

ppm
❌
✔
❌
✔

pgm
✔
❌
✔
❌

pfm
✔
✔
✔
✔

pam
✔
✔
✔
✔


Error Payment with Stripe

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
2.1.2
Issue Description
Used Codenteq – Stripe Payment Gateway with testing card.
After paid an error
Class "Webkul\Sales\Transformers\OrderResource" not found
Preconditions
Install, use, etc
Steps to reproduce
Install, use, etc
Expected Result
Class "Webkul\Sales\Transformers\OrderResource" not found
Actual Result
Callback to Shop
Screenshots

Additional context
No response

This issue is not related to this repo. You should report this issue in the following repo:
https://github.com/codenteq/stripe-payment-gateway


Link export modal from delete-datasets modal

Once #1658 is done, the delete-dataset modal should be updated with the missing features from the original sketch file:

[ ] Change cancel-button in delete-dataset modal to "export dataset" and point to this new modal
[ ] Add original text w/ encouragement to export data before delete dataset
[ ] If opened from delete-items dialog should return to that dialog once data is succesfully exported

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.


Update API docs for v3.0.3

Add new technique to poll specific profiles, new bootstrap response, and fun documentation of one of the advertising APIs.

Thanks and sorry about the delays.


Change Github review comment mode to "REQUEST_CHANGES"

Issue
https://github.com/prontolabs/pronto/issues/319
Implements
Just changed event from 'COMMENT' to 'REQUEST_CHANGES',  which is submitted with github review comment.

@rnitta love it! Could you also make it configurable as per this suggestion? 😄

@mmozuras @rnitta Hi, since this PR seems like it lost its traction, I took the liberty of creating a replacement PR that also implements the configurability: #334


Improve Intel Graphics Drivers Distinction

Summary
Intel's graphics portfolio is diversifying and expanding and it seems its drivers are as well. The documentation needs to be updated to reflect this.
Details
There used to be a single Intel Graphics Driver for 6th-12th gen processors from UHD to Xe graphics. Very simple. Within the past few months, there are now three distinct drivers for UHD to Xe 6th-11th gen, Xe and UHD for 12th gen, and most recently Arc graphics. (Welcome to the graphics industry Intel.) The documentation needs some clarification as to what driver is being used and when you should use a different driver.

Lol, intel significantly changed their graphics support this week. Intel 6-10th gen is no longer supported. I understand, but I think dropping the 10th gen is too soon. Regardless, it seems to not change much except for making things worse.
They are bundling 6-10th gen drivers with 11th-12+ gen in the same exe, even though they are different. This means the driver package is now 1 GB in size, which is obscene. Anyway, this issue will be moving along soon.


Project list structure completed

All the requirements have been implemented:

Set up a new project with webpack that is based on the webpack exercise you have already completed.

Create an index.html file and write your HTML markup here. Create an empty To Do List placeholder ( or  element). The index.html file must be set as a template using the HTML Webpack Plugin.

Create an index.js file and set an array of some simple to do tasks (array of objects). Each task object should contain three keys:

description [string].
completed [bool].
index: [number].

Write a function to iterate over the tasks array and populate an HTML list item element for each task.
On page load render the dynamically created list of tasks in the dedicated placeholder. The list should appear in order of the index values for each task.
Create a style.css and set rules for the To Do List. CSS must be loaded by Webpack Style/CSS Loader. Your list should be a clone of the part of the minimalist project captured in the video below.

Video

All your source files (index.html, index.js and style.css) must be located in /src directory and your distribution files will be generated by webpack and served by webpack dev server from /dist folder.

Hi Yuriy,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request

[ ] There is a merge conflict between the base branch and the feature branch, there are also no commit messages related to the update in this project, in order to get your project reviewed make sure that there are no conflicts, the linters are passing and there are descriptive commit messages related to this part of the project. Please send your project for review as soon as these issues are solved.

Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Migrate 'components/analytics/table_chart.jsx' and associated tests to TypeScript

Summary

Migrate 'components/analytics/table_chart.jsx' and associated tests to TypeScript
Ticket Link

Fixes https://github.com/mattermost/mattermost-server/issues/12476

@karlmarxlopez Let me know when this is ready for review

@hanzei Ready for review

Hi @jespino, I think you reviewed the wrong PR? autosize_textarea is here

Sorry. I've pushed to the wrong PR

Hi @jespino, it's weird that my local test pass, but fails on CI. Any idea why?

@karlmarxlopez The build failed. Could you update to master and see if it will pass. Thanks


Use full Go module path

Uses full path to the module itself in its go.mod file, conventional in Go and compatible with common tooling
Thank you for sharing cliProxy! 🙇

I have no idea what I'm doing with anything related to go mod, sum, pkg, or otherwise, and I appreciate your contribution.

No worries @djhohnstein 😄 👍


IDE Protocol

Summary
As part of the implementation of the new runtime, the Luna CLI binary should support operating with some kind of IDE protocol. For Luna Studio's use this will likely be more rich than LSP, but it should also support operation in LSP mode to allow for ease of interoperation with other editors.
Value
Currently, the implementation of Luna Studio uses code from the Luna compiler directly. This increases the implementation burden as LS is inherently tied to any internal changes in the compiler. This is an undesirable state of affairs to say the least.
In an ideal world, Luna Studio will be written purely in (some variant of) JavaScript, and will not have to have any Haskell components. This will be achieved by delegating all of the heavy lifting for the Luna language to the Luna compiler binary and its language server prototcol, thereby removing any true coupling between the two.
Specification
The IDE protocol implementation must support the following features:

A textual format for communication with IDE protocol clients (JSON?).
Performance such that its use is seamless when compared to services running as part of the editor process.

It may be possible to implement the additional features required by LS as extensions to LSP, hence reducing the total implementation effort required to provide this functionality. Furthermore, the IDE Protocol implementation should be suppported by comprehensive tests and benchmarks such that it is easy to detect and fix regressions should they happen.
One of the main sticking points in supporting LSP and a richer IDE protocol is that we want to be able to do things that require attaching semantic metadata to the file in question. LSP has little to no support for rich compiler output, with deep assumptions that code is plain-text, or at most markdown. This means that it becomes much harder to easily support things such as semantic highlighting, doc summaries with interactive examples, and interactive error messages (amongst other highly-interactive features).
A particularly interesting point is the lack of support for semantic highlighting, something that the existing hacky implementation for Luna already does. We definitely need to support it going forward, which likely means being divorced from LSP or at least creating many extensions on top of it.
The upshot of this is that even if we do support LSP, we’ll likely have to have ‘LSP Mode’ and ‘full mode’. This begs the question of whether LSP support is even worth it. If we need all these extensions to provide our ideal interactive editing experience for Luna, then arguably every client of the IDE protocol should do also.
It must support the following requests/commands:

Completion Requests: Get completion items from the language server at a given cursor position.
Hover Information: Sends current textual position to get information (e.g. documentation) about the item under the cursor position.
Inspect Type: Gets the type of the item under the cursor.
Insert Type: Inserts the type for the item under the cursor in a relevant place.
Go to Declaration: Returns the location of the declaration of the symbol at the provided position.
Go to Definition: Returns the location of the definition of the symbol at the provided position.
Find References: Find references in the current project to the item at the provided position.
Highlight: The Language Server should provide highlighting information for each atom of text, allowing the editor to greatly simplify highlighting.
List Symbols: Should list all symbols in a given file.
Code Action: Apply changes (e.g. hotfixes) to code.
Format Code: Apply a formatting style to the provided range (can be a selection or the whole document). This may also work for live formatting while the user is typing.
Rename Symbol: Tell the language server to perform a project-wide symbol rename.
Get Folds: Find ranges that can be folded within a document.
Get Hints: Hints for the document or provided location, which can then be applied as quick fixes. This should account for any alterations to syntax.
Get Fits for Holes: Luna wants to support holes that can be matched by type.
Get Documentation: Get and display the documentation for the object at the given position.
REPL: The IDE protocol needs to have flexible support for interacting in a programmatic fashion with the Luna REPL, allowing for interactive development. This should include ‘evaluate under cursor’, but also functionality to trivially go back from REPL to buffer if need be.
Debugging: The user should be able to place break-points, and easily inspect values during execution of a program. This requires REPL support and IDE protocol support. This debugging functionality should allow for hot-reloading of code, changing of values within a live program (though this is already mostly supported by Luna Studio’s graph), and various other debugger functionality (step over, step in, step out, continue, etc). In future, this could also be extended to support hardware watchpoints and other low-level details as necessary.

From an architectural standpoint, all this functionality needs to be implemented as a separate library, as a REPL would be, as part of the Luna project. While it should be decoupled as much as possible from the implementation details of the compiler/core/type checker/etc, it inherently will need to utilise those APIs.
Acceptance Criteria & Test Cases

The above listed features have been implemented.
Performance of the IDE protocol is indistinguishable from in-process language servers.
The IDE Protocol implementation comes with comprehensive tests and benchmarks.

There has been some discussion (via email) regarding the design of a unified IDE protocol for dependently-typed functional languages that both Idris and Luna (as a start) could support. Thoughts will be included here as those discussions develop.

This should also include visual syntax support

A brief note about how support for the visual syntax could be handled.

The compiler knows type information about all functions.
It can also track node and visualisation state internally.
The protocol message can, in addition to the positions of the nodes, can send the types of controls for expanded nodes.


Opaque token handler

Fixes #174

Stale since we're using generics in a different way in the released version. Will create new PR for opaque tokens.


Scandinavian letters not shown properly in name

in my config.toml I have this line
name = "æøå"
yet when I send an email to someone, (at least in Outlook) it looks all janky. Like this: "Ã¦Ã¸Ã¥". I guess it has to do with utf-8? These are scandinavian letters if it helps.

Possible duplicate of #363?

I will try to fix the issue ASAP, I let you know!


Entire home page fixed

Closes
Close #939
Description
Fixed the entire home page as it was distorted earlier by some pull request
Screenshots
Before

After
https://github.com/OSCode-Community/OSCodeCommunitySite/assets/133862694/1cdab26d-c4c2-4aa2-8387-619d43d36659
Checklist:

[X] I have linked the PR to the correct issue.
[X] I have read the Contribution Guidelines
[X] I have built and tested the changes, and they do not break or show any errors.

@Vaishnavi-Patil2211 @Divesh2003 Please merge this. This was a major issue, since the home page is completely distorted, and this PR fixes that

@Vaishnavi-Patil2211 Can you please label it level 3? It was a major issue since the entire page was distorted.🙂
You forgot to add the label


Missing yield and generators

Missing yield keyword and generators
Advanced example :- Tree walking
function * elements (tree) {
  yield tree.label;
  if (tree.children)
    for (const child of tree.children)
      yield * elements(child);
}

[...elements(tree)]
  //=> [ 1, 2, 4, 5, 3, 6]

http://raganwald.com/2016/11/30/anamorphisms-in-javascript.html

You meant yield :)

I'm on it, planned for today delivery :wink: thanks for the suggestion

hey,  I new

Oops, didn't had time to do this as I planned 😅, personal stuff happened. If someone wants to PR on this, do it 👍 otherwise I will do it when I can.


Add Data Platform to FAST

Add Data Platform example to FAST

Lore, gave it a quick glance. There are minor things to check (mainly extra variables added to stage 2s), apart from that it looks good. Will spend some time on it tomorrow.

Thanks Ludo for the initial comments, I have some TODO I would like to check with you on Monday. For example: I added 2 ways to handle VPC-SC, I'd like to check with you which is the more suitable one (or identify a third one :-) ).


NEXT-10898 - Empty State at customers->Orders optically defective

In the Administration under Customers in the tab Orders the display of the Empty State is broken if you do not have an order yet.

We've looked into this and are currently planning to implement new empty states. This will be fixed then.


How should I use my local model when build an Embeddings instance?

Thank you for the question but I'm not sure on the exact question.
If the question is how to load a custom transformers model to vectorize documents.
from txtai.embeddings import Embeddings
embeddings = Embeddings({"path": <path to custom transformers model>})

If the question is loading a saved embeddings index
from txtai.embeddings import Embeddings
embeddings = Embeddings()
embeddings.load(<path to previously saved embeddings index>)

Got it! Thank you.


Remove default max token limit

This should solve suprising issues like #816 where the default limit of 1000 tokens is unexpected.

no reason to keep it anymore, I agree. merging.


[pulse] PulseFormula fails to detect simple UNSAT, leading to false positives in Pulse-ISL

Infer version v1.1.0-ed1af6a90
(i.e. built from commit HEAD ed1af6a90b034d6b1c6362aad53a7923c872fda8)
MacOS Monterey 12.5.1, on ARM M1
A minimal example is below. Of course the null-pointer dereference is not actually reachable. Pulse (and Pulse-ISL) think that it is. PulseFormula is seemingly not detecting that the path condition is UNSAT, after the summary for test_func is applied at the callsite in main.  Note that, as the comments say, the issue appears to be an interaction of the prior if-condition and the resulting addition to the path condition.
#include <stdio.h>
#include <stdlib.h>

void test_func(int m, int b){
  /* commenting out this if-statement makes the false bug disappear */
  if (b >= 34) {
    abort();
  }

  /* uncommenting an additional if-test like this also makes it disappear/
  if (b+1<b){
    abort();
  }
  */

  if (m < b * 2) {
    printf("Against all the odds, I'm going to dereference a null-pointer\n");
    int *p = NULL;
    (void)*p;
  }else{
    printf("Of course I'm not going to dereference a null-pointer\n");
  }
}

int main(){
  test_func(64,32);
  return 0;
}

Infer command (the same false bug is also reported regardless of whether --pulse-isl is present, so this seems to be an issue for Pulse in addition to Pulse-ISL).
./infer/bin/infer  --pulse-only --pulse-isl --no-pulse-report-latent-issues -- cc -c  test.c

Command output:
Capturing in make/cc mode...
Found 1 source file to analyze in /Users/tobiasm1/git/infer/infer-out
1/1 [################################################################################] 100% 22.326ms

/Users/tobiasm1/tmp/libsodium_binhex.c:27: error: Null Dereference
  The call to `test_func` may trigger the following issue: `p` could be null (from the call to `test_func()` on line 18) and is dereferenced in the call to `test_func()`.
  25.
  26. int main(){
  27.   test_func(64,32);
        ^
  28.   return 0;
  29. }

Found 1 issue
             Issue Type(ISSUED_TYPE_ID): #
  Null Dereference(NULLPTR_DEREFERENCE): 1

Thanks for the bug report, it surfaces an interesting weakness in the current way the "tableau" arithmetic domain we use for inequalities interacts (not enough) with the linear arithmetic domain.

Here is another similar example, in case it's helpful. Also tested against ed1af6a90b034d6b1c6362aad53a7923c872fda8
#include <stdio.h>

void
test_func2(int size){
  int len = (size + 31) / 32;
  printf("len: %u\n",len);
  // commenting out the outer if-test removes the spurious NULL dereference report
  if (0 < len){
    if (1 < len){
      int *p = NULL;
      (void)*p;
    }else{
      printf("Of course not.\n");
    }
  }
}

int main(){
  test_func2(2);
  return 0;
}

% ./infer/bin/infer --pulse-only --pulse-isl --no-pulse-report-latent-issues -- cc -c ~/tmp/cttk_int31_ppp.c
Capturing in make/cc mode...
Found 1 source file to analyze in /Users/tobiasm1/git/infer/infer-out
1/1 [################################################################################] 100% 16.645ms

/Users/tobiasm1/tmp/cttk_int31_ppp.c:20: error: Null Dereference
  The call to `test_func2` may trigger the following issue: `p` could be null (from the call to `test_func2()` on line 11) and is dereferenced in the call to `test_func2()`.
  18.
  19. int main(){
  20.   test_func2(2);
        ^
  21.   return 0;
  22. }

Found 1 issue
             Issue Type(ISSUED_TYPE_ID): #
  Null Dereference(NULLPTR_DEREFERENCE): 1

Thanks again for the report @tobycmurray, this led to a nice improvement. I've verified that 74fdf59a3b39f02b3da0baf6c971444209e392f1 fixes the issue in both of your examples and there's a new unit test for the arithmetic domain that captures the crux of the problem.


some sprites are drawn at wrong size

Description
sometimes when examining people it happens
also happens to pinpointer as shown
Screenshots

lil pai

i guess this was fixed (we now have grid inventory anyway)


ML.NET Model Builder Price Prediction 'Start Training' gives error 'Trainer RSquared Absolute-loss Squared-loss RMS-loss Duration #Iteration'

Problem encountered on https://dotnet.microsoft.com/learn/ml-dotnet/get-started-tutorial/train
Operating System: windows
Provide details about the problem you are experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
I have a .NET Core CLI application with "hello world", and I used the ML.NET Model Builder Extention:
Steps to reproduce: I clicked clicked "add machine learning"
Step 1. "Price Prediction".
Step 2. For the data I use a local MSSQL server (test connection succeeded), and then I configured the data. I want to predict the Value (decimal(16,6)) based on 10 input columns/features (7 integer columns, 2 datetime columns, 1 bit column)
Step 3. Train --> it wants to train for 1800 seconds. However, it quickly gives the error "Failed. See more in Output Pane", but the output pane only mentions "'| Trainer RSquared Absolute-loss Squared-loss RMS-loss Duration #Iteration |".
If have also tried setting the duration to 60 seconds, but then it says that 60 seconds is not enough for 127MB, and that I should train for longer. If I click 'try anyway', the same error occurs.
Why does it give this error and how can I fix this?

@perkst Sorry for the late reply. Could you upgrade to the latest modelbuilder and run the same experiment again, this time it should give you some error information (if there's any)
And what's your dataset's PredictedPrice looks like, is it all NULL value, or part of it is NULL while the other is number?

@danielcamps or @perkst Any additional info regarding Xiaoyun's questions above? Does this still repro for you with the latest version?

We don't have enough information to investigate this issue further. If you're still seeing a problem please...

Update to the latest Model Builder
Send us any updated error message.
Send the .tsv file in the %temp% folder produced from the downloaded sql.

Ping me and we'll re-open the issue. Thanks!


Nuke

Checklist
Build system is ancient here - need to bring over the https://github.com/petabridge/Petabridge.Library/ build engine.

@eaba does this work correctly locally?

Thanks for the quick turn around guys! 🙇


FullGroupJoin with null keys

I expected that FullGroupJoin will return all items, even those with null keys, but somehow FullGroupJoin does not return null keys.
I tried implementing custom comparer, but it did not help.
So to fix this, I am currently using "clever hack" for key functions using tuple: p => (p.Id, true).
This treats null keys as non-null. A record would work too I guess.
Is there any more clear way to make FullGroupJoin not ignore null keys?
Null keys do not work: https://dotnetfiddle.net/iefSBj
Null keys work: https://dotnetfiddle.net/QS7ZNK

Whoah what a foot gun! This took me a few hours to track down deep in a codebase.

Would you accept a documentation patch calling this out?

would you accept a PR to add a parameter to control the null key behaviour?

I quickly hacked around it by removing the check, and it seems to work buy I don't quite understand the code enough to be sure:
https://github.com/morelinq/MoreLINQ/blob/03a160ceaa0942f46cf585894a3c6e53df040076/MoreLinq/Lookup.cs#L98

If this is confirmed undocumented behavior of Enumerable.Join, is there a doc bug about it?

@fowl2 Lookup.cs in this project comes from .NET, which has the same guard and behaviour.

Would you accept a documentation patch calling this out?

Sure.

would you accept a PR to add a parameter to control the null key behaviour?

Not really.


Request to Support Create Unit Tests for VB.NET in VS 2017

Per my issue reported in https://developercommunity.visualstudio.com/content/problem/70314/vbnet-create-unit-tests.html VS 2017 does not support Create Unit Tests. Requesting an enhancement to include functionality in later release of software.

Closing as duplicate of https://github.com/Microsoft/testfx/issues/376


Unable to access role history correctly

I am trying to use this package to get records about working group leadership in the present and past.
Getting them for the present seems to work fine. As an example I'm using dnsop.
https://datatracker.ietf.org/group/dnsop/history/
> wg = dt.group_from_acronym("dnsop")
> [dt.person(r.person).name for r in dt.group_roles(group = wg)]
['Warren "Ace" Kumari', 'Benno Overeinder', 'Suzanne Woolf', 'Tim Wicinski']

These are indeed the names of the people currently in leadership of this group.
Now I'm trying to find the people historically but not currently in roles in this working group.
> [dt.person(r.person).name for r in dt.group_role_histories(group = wg)]
['Andrew Sullivan', 'Marc Blanchet']

I don't think these people have ever been in dnsop leadership. I'm expecting the names Peter Koch, Joel Jaeggli, and Ronald Bonica. These names are mentioned in certain GroupEvents about leadership change. Is there link to these other persons maintained in the group history?

The group parameter for group_role_histories should be a GroupHistoryURI, and not a GroupURI. This isn't an intuitive name, but it does match the Datatracker API.
You can fetch the GroupHistory objects for a group like this:
wg = dt.group_from_acronym("dnsop")
group_role_histories = [dt.group_role_histories(group = grp_hist, name=dt.role_name(RoleNameURI("/api/v1/name/rolename/chair/"))) for grp_hist in group_histories]

(Note the use of name to filter to only WG chairs: you can remove this filter, or switch it to other roles (e.g., ad).)
There'll be a GroupHistory object for each time the metadata about a group changes, including for chair changes. This has evolved over time: the first entry I can see is from 2005.
From each GroupHistory object, you can look up the roles held at that point:
for group_role_history in group_role_histories:
    print([dt.person(r.person).name for r in group_role_history])

That should give you the WG chairs for each GroupHistory.

Thank you. That's so helpful!
Is there a way to get the GroupHistoryURI from a known GroupURI, or acronym?

Hmmm. I'm sorry, I think I asked a badly thought through question.
I am seeing how to get group histories from an acronym:
https://github.com/glasgow-ipl/ietfdata/blob/master/ietfdata/datatracker.py#L674
Which may be just what I want!
I'm a bit confused about what you are saying about group_role_histories.
Is it possible that the type signature of the method is off?
https://github.com/glasgow-ipl/ietfdata/blob/master/ietfdata/datatracker.py#L2883
It says that group is Optional[Group' not Optional[GroupHistory]

Yes -- the signature is wrong here; should be Optional[GroupHistory] as you say. Will fix (along with the tests..).


Create get_colors() function

To be consistent with map coloring, create a function that takes at least one character argument and returns a (potentially named) vector of hex colors. Values for the argument could be:

"richness": the 7-9 color palette used for richness maps
"rich-delta": the two endpoints of the delta colors used in richness change maps
"eastwest": the two colors used to distinguish east & west species
"hotspot": single element vector with color to use for richness hotspots
"overlap": four color vector for predicted distribution: absent, host only, insect only, and host+insect
"dist-delta": four color vector for change in predicted distribution: absent, loss, gain, stable
"protected": four color vector for four management types of protected areas.

Function added in e828aa2. Still to be implemented in scripts.


Search Feature Not Working

When I try to search for something in the DEA User Guide, the search page opens, but no results are ever returned.
It looks like something has broken with the build of the site. If I look at the Javascript Console in my browser, there's an error message:
search.html?q=landsat&check_keywords=yes&area=default:146 Uncaught ReferenceError: jQuery is not defined
    at search.html?q=landsat&check_keywords=yes&area=default:146:7

JQuery is a Javascript library that should be included on the page, but for some reason is not. It could be something broken with the theme we use, or an incompatibility between something else and the theme.

Fixed with https://github.com/GeoscienceAustralia/dea-docs/commit/0ed62994ef0b33d46d9775b74debdf9dd3cfd159


Creation of the terrain

As a car driver, I want to be able to manoeuvre the car in a virtual environment of a parking lot so that i can test the functionality of autonomous parking.
Acceptance Criteria:

[ ] Should be able to load in the smartcar into the new SMCE terrain
[ ] Terrain has 6 to 8 parking lots.
[ ] Terrain has at least three DomyCars.

@adityak714 is this done or should we move into sprint2?

The acceptance criteria are met for this beginning issue of the creation of the terrain. More extended requirements could also follow later, like in issue #49, where the plan is to make a few more additions in the terrain to allow, potentially, more types of parking in the system.


Unify ingester and distributor chunks into a single heap merge.

Before this change, chunks returned by streaming ingester queries are put into one heap, and chunks from the chunk store another.  These two heaps are then merged using yet another heap; whilst this should have the same algorithmic complexity as using one big heap, in reality is faster to put all the chunks into one heap.
As part of this PR I completely reorganised the querier package, sorry.   I tried to split them out into a separate changeset.
Signed-off-by: Tom Wilkie<EMAIL_ADDRESS>

I read something somewhere yesterday which just prompted this question: is this the kind of situation where timsort would perform well? I.e. does the data have lots of sequential runs?

I read something somewhere yesterday which just prompted this question: is this the kind of situation where timsort would perform well? I.e. does the data have lots of sequential runs?

Thats not really relevant to this PR - we're not changing how we merge/dedupe chunks here, just ensuring that when we use ingester chunk streaming we put them in the same "batch" of chunks as we got from the chunk store, and then merge them.
That being said, its probably worth investigating timsort for chunk merging.  It should perform better than our recursive binary merge strategy, although it will have the downsides of increase memory usage.

Finally got round to testing this - for a certain class of queries (~100k chunks, 12hr duration, sum(container_memory_rss) by (namespace) this reduced the promqlInnerEval from a reliable 16s to 6s, and the overall query from 20s to 10s.

We've had this running in prod now for ~4 days, and the unit tests are now pretty comprehensive.  Going to merge once its green.


Autocount is breaking on custom SELECTs

The method JSONAPI::Resource#find_count should be changed to:
 def self.find_count(filters, options = {})
  filter_records(filters, options).count(:all)
 end

so the SELECT is cleared. This is how ActiveRecord::Relation#size is implemented, BTW.
This problem emerges as a syntax error on any custom select, specially the ones you make just to please Postgres’s GROUP BY clauses.

:+1:


Allow invalid binaries as input files to ILC

This is a hopefully short-term hack which allows files specified as
inputs to ILC to be actually native binaries that we ignore and proceed
with compilation.  The CoreCLR drops we test against contain a mixture
of managed and native binaries in test folders in some cases.
Also, add a quality of life switch to runtest.cmd (/CoreCLRSingleTest)
that allows specifying a single CoreCLR test to run by executable path.
This downloads the test blob and sets the correct environment variables
first.

@MichalStrehovsky PTAL.


Investigate using translate for window positioning

We should profile this to see if we are spending a lot of time in paint while users are dragging windows around. If they are, we might get an improved experience by using translate which creates a new rendering context and thus can avoid the paint.
https://www.paulirish.com/2012/why-moving-elements-with-translate-is-better-than-posabs-topleft/
cc @durasj who has been looking at our window dragging performance in Electron.

I’ll probably get to this eventually, but it’s not a high priority right now. I’ll comment here if I start looking.

So I did a quick look too since I was quite excited about such "easy gains" but I have mixed feelings. While it does prevent layout recalculations (posabs vs. transform - check bottom right corner) and it does seem to lag less behind my mouse, it feels worse in the actual electron project with the transparency. But that could be also due to the fact that winamp2-js >= 0.0.4 breaks it. But I'll take a look at that over the weekend if I will be able to and also try to do some more objective testing using puppeteer.


Support builtin fail() function

It'd be nice to be able to throw an error and get a stack trace in Starlark. Even if #230 isn't feasible, could a hardcoded fail function be implemented? Something like:
state.Globals["fail"] = starlark.NewBuiltin(
    "fail",
    func(thread *Thread, fn *Builtin, args Tuple, kwargs []Tuple) (starlark.Value, error) {
        var msg string
        err = UnpackPositionalArgs("fail", args, kwargs, 1, &msg)
        if err != nil {
            return nil, err
        }
        return nil, fmt.Errorf(msg)
    },
)

I'm dumb, fail() does work right now

I'm dumb, fail() does work today


Grid Reload does not work properly in Ajax call

I have a page using gijgo grid. As simple as it is, everything works perfect. And then I add event to one column, responding to a clicking, which will trigger a function, do a ajax call to update the data. In the .done() part, it calls the grid.reload(), but nothing happend, no refresh at all. I use a window.location.reload(false) instead, it works.
I can verify the ajax call return successfully and the data has been changed and the window.location.reload works. But not grid.reload({...}).
Can please take a look on this issue and try to fix it?
Attach the codes I am using:

sorry for the mix of code. don't find the way to make it in

This is not a bug in the grid. The ordersgrid variable is local var for document.ready and not exists outside of this method. Declare ordersgrid outside of the document.ready function.


Use ECMA6 or higher for this project

The old browser versions won't work with ECMA but I think almost of user are using the new browser version.
Can we use ECMA6 or higher for this project?
If you want to support the old browser version, we can use babel to transform.

I think it’s fine to constrain on ECMA. This project pretty much only
supports the latest version of chrome anyways :/
On Mon, Apr 22, 2019 at 8:32 PM Hongarc<EMAIL_ADDRESS>wrote:

The old browser versions won't work with ECMA but I think almost of user
are using the new browser version.
Can we use ECMA6 or higher for this project?
If you want to support the old browser version, we can use babel to
transform.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pcottle/learnGitBranching/issues/570, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAIVDHYZZDAYN2CH5VYSV5TPRZ7OPANCNFSM4HHUJDVQ
.

--
-Sent from Gmail Mobile

Last my commit fails with grunt (because I update react).
I am fixing it and it will be easy to fix if we use class instead of function so I ask you to use that.
Thank you for your response.
I will fix it can run first and convert all to ECMA.


What kind of feedback comes via onCallListener?

Hi! Can you explain how onCallListener works? What kind of feedback or response object comes through the listener?
Let's assume a simple scenairo.
I have created the video session for User1 and attached the media to the stream. Now User1 can see himself on the screen. When I star call with session.call, what happens?
Does User2 gets a response via onCallListener? If it is, what kind of response?
Thanks!

Hi, @tahademirer

Hi! Can you explain how onCallListener works? What kind of feedback or response object comes through the listener?
Let's assume a simple scenairo.
I have created the video session for User1 and attached the media to the stream. Now User1 can see himself on the screen. When I star call with session.call, what happens?

One user uses session.call(extension) and sends a signaling message with the session and any data as object (extension) via XMPP chat. Second user receives the signaling message (XMPP's data - a stanza) and the message is parsing to define which listener should be called (ConnectyCube.videochat.onCallListener = (session, extension) => {...} in this case)
https://developers.connectycube.com/js/videocalling?id=initiate-a-call

Does User2 gets a response via onCallListener? If it is, what kind of response?
Thanks!

User2 receives the session and object with data in ConnectyCube.videochat.onCallListener(session, extension)
Then User2 should accept or reject the incoming call using the session from the onCallListener:
ConnectyCube.videochat.onCallListener = (session, extension) => {
  session.accept({}); // the `onAcceptCallListener` will fired on opponent's side
  // or `session.reject({})` - the `onRejecttCallListener` will fired on opponent's side
  // if user won't answer the `onUserNotAnswerListener` will fired on opponent's side after interval.
}

After connection would establish, both users will receives a remote stream from opponent:
ConnectyCube.videochat.onRemoteStreamListener = (session, userID, remoteStream) => {
  console.log(`Remote stream: ${remoteStream}`);
  console.log(`From user with ID: ${userID}`);
}

Hi, @tahademirer

Hi! Can you explain how onCallListener works? What kind of feedback or response object comes through the listener?
Let's assume a simple scenairo.
I have created the video session for User1 and attached the media to the stream. Now User1 can see himself on the screen. When I star call with session.call, what happens?

One user uses session.call(extension) and sends a signaling message with the session and any data as object (extension) via XMPP chat. Second user receives the signaling message (XMPP's data - a stanza) and the message is parsing to define which listener should be called (ConnectyCube.videochat.onCallListener = (session, extension) => {...} in this case)
https://developers.connectycube.com/js/videocalling?id=initiate-a-call

Does User2 gets a response via onCallListener? If it is, what kind of response?
Thanks!

User2 receives the session and object with data in ConnectyCube.videochat.onCallListener(session, extension)
Then User2 should accept or reject the incoming call using the session from the onCallListener:
ConnectyCube.videochat.onCallListener = (session, extension) => {
  session.accept({}); // the `onAcceptCallListener` will fired on opponent's side
  // or `session.reject({})` - the `onRejecttCallListener` will fired on opponent's side
  // if user won't answer the `onUserNotAnswerListener` will fired on opponent's side after interval.
}

After connection would establish, both users will receives a remote stream from opponent:
ConnectyCube.videochat.onRemoteStreamListener = (session, userID, remoteStream) => {
  console.log(`Remote stream: ${remoteStream}`);
  console.log(`From user with ID: ${userID}`);
}

Thanks!

Thanks!


Create dependabot.yml config to only scan core dependencies

By default, dependabot scans all python dependencies and requirements.txt files in the repository. ~95% of these are for examples, which trigger false positives for vulnerable packages that have low potential for exploit. Dependabot should only scan the following for vulnerable dependencies:
-setup.py
-requirements-linters.txt
-requirements-test.txt
-docs/requirements-docs.txt
-openfl-tutorials/experimental/requirements_workflow_interface.txt

Fixed with #1077.


fix mufft crash caused by unaligned input buffer

Fixes https://github.com/etternagame/etterna/issues/832

Works fine on Windows

Thanks for your first pull request on this repository @hexptr !
This seems to be allocating a new buffer every time we execute the callback
Could it be rewritten to instead allocate the buffer here (And free it if it existed, same as the output buffer) and store it in the class? https://github.com/etternagame/etterna/blob/68cf405883df88a81c29b5a9727115e028f54a2a/src/RageUtil/Sound/RageSound.cpp#L780-L781
As an aside, since you're touching this. I think we can just get rid of this line https://github.com/etternagame/etterna/pull/837/files#diff-2a93c97fb469c142670452b11fddef84R365 (We're clearing the vector this writes into right after)
Neither of these is a huge deal and should not stop this from getting merged

Thanks for the suggestion.
I took the liberty to rewrite both input and output buffers as vectors that use a custom allocator which calls into the mufft_alloc and mufft_free functions.
Both buffers along with the FFT plan, are stored in the class now.
I added more includes and declarations into the RageSound.h header to make things work. I don't know how much we're worried about the compile speed regressing because of things like this.

This looks good! I like how we also reuse the plan now. Thank you!
The need for the include in the header is unfortunate, but I don't think it's worth it to try to add some additional indirection (I'm thinking something pimpl like), the mufft header does not look particularly big (About 300 lines and mostly comments)
I compared the CI times (Except windows, which isn't finished yet) and I didn't see any noticeable changes (I imagine CI times are unreliable but it's something)

build times look fine to me


Reuse of PDOStatement when query cache is enabled results in SQLite "21 bad parameter or other API misuse" for file connections

No duplicates 🥲.

[X] I have searched for a similar issue in our bug tracker and didn't find any solutions.

Database
SQLite
What happened?
When query caching is enabled, if a constraint violation is encountered during an insert statement, a subsequent attempt to perform the same insert query will reuse the PDOStatement object stored in the cache, and its reuse will result in SQLite producing the following error:

PHP Fatal error:  Uncaught PDOException: SQLSTATE[HY000]: General error: 21 bad parameter or other API misuse in vendor/cycle/database/src/Driver/Driver.php:444

This situation occurs under two additional conditions:

The SQLite database is a file.
The SQLite file is opened, not created.

Reproduction
I've attached a test script. The output will consist of o, record inserted, or x constraint violated. Execution of this script under different conditions follows.
<?php
declare(strict_types=1);

use Cycle\Database\Config\DatabaseConfig;
use Cycle\Database\Config\SQLite\FileConnectionConfig;
use Cycle\Database\Config\SQLite\MemoryConnectionConfig;
use Cycle\Database\Config\SQLiteDriverConfig;
use Cycle\Database\DatabaseManager;
use Cycle\Database\Exception\StatementException\ConstrainException;
use Cycle\ORM\EntityManager;
use Cycle\ORM\Factory;
use Cycle\ORM\Mapper\StdMapper;
use Cycle\ORM\ORM;
use Cycle\ORM\Schema;
use Cycle\ORM\SchemaInterface;

require __DIR__ . '/vendor/autoload.php';

$driver = new SQLiteDriverConfig(
	connection: new FileConnectionConfig(
		database: __DIR__ . '/blah.db',
	),
	reconnect: true,
	queryCache: true,
);
//$driver = new SQLiteDriverConfig(
//	connection: new MemoryConnectionConfig(),
//	reconnect: true,
//	queryCache: true,
//);

$databaseName = 'default';
$dm = new DatabaseManager(new DatabaseConfig([
	'default' => $databaseName,
	'databases' => [
		$databaseName => ['connection' => 'sqlite']
	],
	'connections' => ['sqlite' => $driver]
]));

$tableName = 'testing';

$table = $dm->database('default')
	->table($tableName)
	->getSchema();
$table->setPrimaryKeys(['a', 'b']);
$table->string('a')->nullable(false);
$table->string('b')->nullable(false);
$table->save();
unset($table);

$orm = new ORM(
	new Factory($dm),
	new Schema([
		$tableName => [
			SchemaInterface::MAPPER => StdMapper::class,
			SchemaInterface::DATABASE => 'default',
			SchemaInterface::TABLE => $tableName,
			SchemaInterface::PRIMARY_KEY => ['a', 'b'],
			SchemaInterface::COLUMNS => [
				'a' => 'a',
				'b' => 'b',
			],
		]
	]),
);

$things = ['123', '123', '456'];
foreach ($things as $value) {
	$em = new EntityManager($orm);
	$record = $orm->make($tableName);
	$record->a = 'static';
	$record->b = $value;
	$em->persist($record);
	print '_';
	try {
		$em->run();
		print chr(8) . 'o';
	} catch (ConstrainException) {
		// 23000: UNIQUE constraint failed
		print chr(8) . 'x';
	}
}
print "\n";

Fresh database file, queryCache: false
$ rm blah.db
$ php testorm.php
oxo

Fresh database file, queryCache: true
$ rm blah.db
$ php testorm.php
oxo

Existing database file, queryCache: false
$ php testorm.php
xxx

Existing database file, queryCache: true
$ php testorm.php
x_PHP Fatal error:  Uncaught PDOException: SQLSTATE[HY000]: General error: 21 bad parameter or other API misuse in /home/whmcs/sync/soylent/vendor/cycle/database/src/Driver/Driver.php:444
Stack trace:
#0 vendor/cycle/database/src/Driver/Driver.php(444): PDOStatement->execute()
#1 vendor/cycle/database/src/Driver/Driver.php(270): Cycle\Database\Driver\Driver->statement()
#2 vendor/cycle/database/src/Query/InsertQuery.php(125): Cycle\Database\Driver\Driver->execute()
#3 vendor/cycle/orm/src/Command/Database/Insert.php(85): Cycle\Database\Query\InsertQuery->run()
#4 vendor/cycle/orm/src/Transaction/Runner.php(61): Cycle\ORM\Command\Database\Insert->execute()
#5 vendor/cycle/orm/src/Transaction/UnitOfWork.php(150): Cycle\ORM\Transaction\Runner->run()
#6 vendor/cycle/orm/src/Transaction/UnitOfWork.php(327): Cycle\ORM\Transaction\UnitOfWork->runCommand()
#7 vendor/cycle/orm/src/Transaction/UnitOfWork.php(375): Cycle\ORM\Transaction\UnitOfWork->resolveSelfWithEmbedded()
#8 vendor/cycle/orm/src/Transaction/UnitOfWork.php(219): Cycle\ORM\Transaction\UnitOfWork->resolveRelations()
#9 vendor/cycle/orm/src/Transaction/UnitOfWork.php(96): Cycle\ORM\Transaction\UnitOfWork->walkPool()
#10 vendor/cycle/orm/src/EntityManager.php(47): Cycle\ORM\Transaction\UnitOfWork->run()
#11 testorm.php(77): Cycle\ORM\EntityManager->run()
#12 {main}

Expectation
Regardless of whether the SQLite database file is created within the same process, a constraint violation in a prepared INSERT statement should not result in the same INSERT query later failing; whether it would encounter another constraint violation or not.
Version
PHP 8.1.23 (cli) (built: Sep  2 2023 06:59:15) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.1.23, Copyright (c) Zend Technologies
    with Zend OPcache v8.1.23, Copyright (c), by Zend Technologies

cycle/database                   2.5.2   DBAL, schema introspection, migration and pagination
cycle/orm                        v2.3.4  PHP DataMapper ORM and Data Modelling Engine

$ php -i | grep -i sqlite
/etc/php/8.1/cli/conf.d/20-pdo_sqlite.ini,
/etc/php/8.1/cli/conf.d/20-sqlite3.ini,
PDO drivers => sqlite
pdo_sqlite
PDO Driver for SQLite 3.x => enabled
SQLite Library => 3.37.2
sqlite3
SQLite3 support => enabled
SQLite Library => 3.37.2
sqlite3.defensive => On => On
sqlite3.extension_dir => no value => no value

I am happy to submit this against the ORM repository if some investigation points to it being the culprit. As my discovery led me to the query cache and discussions on the net regarding reuse of PDOStatement objects, I felt like starting at the driver level might be prudent.
Thank you for your time and work!


Loading SPI instance(singleton )can not use the passed constructor parameters after the first-time call

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

Ⅰ. Issue Description
When I Load SPI with Singleton-Scope,because that the singleton instance will be cached, except for the first time load(), the constructor parameters passed in later call will not be used to create instance.
Ⅱ. Describe what happened
1. Define a SPI implementation using Singleton-Scope.
2. Load by calling:
public static <S> S load(Class<S> service, String activateName, Class[] argsType, Object[] args)
and passing paramterA, get the instance.
3. Load again by calling the same load() and passing parameterB
4. Expect to get an new instance by using parameterB,but will get the cached instance.

Ⅲ. Describe what you expected to happen
We should know that different parameters means the different state of extension instance, so when we use stateful extension points, we should define it as Prototype-Scope.
But the api in EnhancedServiceLoader allows users to load a Single-Scope extension point by passing different parameteres.
In this case ,the user should get an explicit exception to notify them that the parameters after the first time 's call will not be used . In this way ,we can help them to identify where the problem is:

(1) Usage problem, if the singleton load is called multiple times, they should pass the same parameter.
(2) The Scope() of the extension point was wrong. they should use Prototype-Scope
Ⅳ. How to reproduce it (as minimally and precisely as possible)
the same as Ⅱ. Describe what happened
Ⅴ. Anything else we need to know?
Ⅵ. Environment:

JDK version :
OS :
Others:

@ggndnn  ggndnn and I think We can resolve this problem in this way:
1.Record the parameters of a single instance passed by the user.
2.After the first time call, if the parameters passed is not same with the cached, throw an exception to notify the user,instead of directly returning the first cached instance.


Ywt/add nms rotated op

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation and Context
add nms rotated op
to fix problem in PointPillar
Description

Use cases (Optional)

BC-breaking (Optional)

Checklist
Before PR:

[x] I have read and followed the workflow indicated in the Contributors.md to create this PR.
[x] Pre-commit or linting tools indicated in Contributors.md are used to fix the potential lint issues.
[x] Bug fixes are covered by unit tests, the case that causes the bug should be added in the unit tests.
[x] New functionalities are covered by complete unit tests. If not, please add more unit test to ensure the correctness.
[x] The documentation has been modified accordingly, including docstring or example tutorials.

After PR:

[x] CLA has been signed and all committers have signed the CLA in this PR.

综合看下，能不能把cpu的逻辑删了。

综合看下，能不能把cpu的逻辑删了。

已删除


Page need to be updated after 19.10

As title explained.

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

ID: 8c6fff5c-8c9b-cdf9-5df3-4b211c91c0a9
Version Independent ID: 9a2f0e11-0532-3194-423f-84e79f47dd07
Content: Azure Sphere CLI login command
Content Source: AzureSphere/articles/reference/azsphere-login.md
Product: azure-sphere
GitHub Login: @pennyo
Microsoft Alias: azurespheredocs

@xiongyu0523 Thanks for the feedback! We are currently investigating and will update you shortly.

@pennyo Please check this and update the documentation.

@xiongyu0523 Thanks for the feedback. We'll fix this in the next publication of the docs.

@xiongyu0523 As Penny mentioned, this will be fixed in the next documentation updates.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Modified dimension check for new BufferedImage()

Instantiating BufferedImage requires height and width to be greater than 0.
Please refer to DirectColorModel.createCompatibleWritableRaster() which gets called from BufferedImage's constructor which is called at line imgscalr:2037.

Thank you for the pull request!


Is there a good example tutorial on how to use Tensor2Tensor?

Is there a good example tutorial on how to use Tensor2Tensor?

Can you see if https://colab.sandbox.google.com/github/tensorflow/tensor2tensor/blob/master/tensor2tensor/notebooks/hello_t2t.ipynb works for you?
Or maybe the other notebooks here which can be accessed the same way:
https://github.com/tensorflow/tensor2tensor/tree/master/tensor2tensor/notebooks


Use fs.unlinkSync instead of fs.unlink when removing eq.log

Another issue I found when hardening the test infrastructure.
This is necessary because Node.js crashes when fs.unlink is called without a callback function.

Trivial (and verified locally because the bots don't work reliably yet).

In https://github.com/mozilla/pdf.js/blob/master/test/font/ttxdriver.js there's a couple of occurrences of fs.unlink without callback as well. Could fixing those too perhaps help avoid a bunch of (node:5308) [DEP0013] DeprecationWarning: Calling an asynchronous function without callback is deprecated. lines when the font-tests, see e.g. the following (Windows) bot output: http://<IP_ADDRESS>:8877/fd76f7812d1afe5/output.txt

Yes, it looks like it. I think I somehow excluded the test directory in my grep for this patch. I'll take care of fixing this; thanks for bringing it up!


Fix yarn command in the Setup Cookbook

Related Issues
None
closes #
n/a
Short Description and Why It's Useful
The yarn command was incorrect in the documentation.
Screenshots of Visual Changes before/after (if There Are Any)
n/a
Which Environment This Relates To
n/a
Upgrade Notes and Changelog

[x] No upgrade steps required (100% backward compatibility and no breaking changes)
[ ] I've updated the Upgrade notes and Changelog on how to port existing Vue Storefront sites with this new feature

IMPORTANT NOTICE - Remember to update CHANGELOG.md with description of your change
Contribution and Currently Important Rules Acceptance

[x] I read and followed contribution rules

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you!


Adopt abstract base class syntax

NLTK has plenty of abstract base classes. Now that Python has support for these, we need to audit the codebase and adopt this syntax.
cf https://docs.python.org/3.0/whatsnew/2.6.html#pep-3119

Thanks @mrecachinas and @kmike for https://github.com/nltk/nltk/pull/1223. It looks like we have a model that can be applied across the toolkit now.

@stevenbird Is the issue fixed?

Thanks @souravsingh, @gmotzespina!


Looking forward to your code

After reading your paper, I am very interested in your novel ideas and look forward to your code. When will the code be released?

Thanks for your interest. We are going through the code review process and will probably release the code in the next month.

Thank you, and hope you release the code soon~


Re-install all APKs on upgrade

in hosts and thinclient
look in the file /etc/apk/world instead of looking in the tower config
update the npm and pip guides for installation in /home

Done except the npm guide. See https://github.com/towercomputers/toweros/issues/238


feat(demo-playwright): rewrite button(kit) test to playwright

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Refactoring
[ ] Build or CI related changes
[x] Tests related changes
[ ] Documentation content changes

What is the current behaviour?
Button e2e test wrote on Cypress.
Closes #
What is the new behaviour?
Button e2e test wrote on Playwright.

These are screenshots by cypress:

These are screenshots by playwright:

Do I have to reduce quality of images somehow to increase performance?

@waterplea Can you take a look at this pr, please?

Do I have to reduce quality of images somehow to increase performance?

nope, it is not required

@nsbarsukov I fixed everything. Can you take a look, please?

@40oleg
Thanks ❤


When use crystal i with wezterm(a popular linux terminal), Alt + TAB will output controller character like [O[I [O[I into terminal

Hi, i create a new issue there, please have a look.
Thanks.

My fault, sorry.


Updating orbit-db-keystore

I've found the problems here.

I made a change in https://github.com/orbitdb/orbit-db-keystore/pull/60 that i shouldn't have as it wasn't necessary; i had left a comment regarding it before it had been merged.
key.public.marshal() calls in tests need to be wrapped with a Buffer.from call because, with the new libp2p-crypto version, Uint8Arrays are used in place of Buffers. TypedArrays dont have the built in same .toString method as Buffers and don't support encoding to a hex string.

I'll make a pr to the keystore and add the Buffer changes to the tests here.


No code generated for interface inline fragments

I can use objects as inline fragments, but for interface fragments no code is generated. Shouldn't this be possible?
{
... on ClosedEvent {
        # object, works
    }
... on IssueEvent {
        # interface, no code is generated
    }
}

(Using Swift, but saw similar results with the other targets.)

Hey @Anviking thanks for the report, this seems to be an issue with the common code between the targets, we definitely need to handle interfaces better... We'll take a look but we also invite you to try to tackle the problem yourself in the meantime, we'll be happy to help and give you pointers for you to do your first contributions on this project. However, don't worry if you can't, that's something we'll take a look at anyway.

@Anviking @rricard: I'm not sure I would classify this as a bug (and certainly not an easy one 😊), but I can see how the behavior can be confusing. And maybe we should reconsider it.
Looking at the Swift target for now, we only generate as<Type> methods for object types. These do contain fields selected on interfaces however.
So using your example (and assuming it is from the GitHub schema), given that ClosedEvent implements IssueEvent, any fields selected on IssueEvent will be added to the generated fragment for ClosedEvent (and to any other fragments for object types implementing the same interface).

@martijnwalraven Maybe I classified it too fast but there is definitely an issue. The example given, even if we could avoid it should still work: that is a correct graphql document. I thought it could be easy but indeed, before we refactor, and after evaluating again the code, I also think it'll be hard to fix...

@rricard: Hmmm, maybe I don't understand the issue. But I definitely wasn't suggesting we don't support inline fragments on interfaces. The query above should work, but the fields selected on IssueEvent will be added to the fragment generated for ClosedEvent. So to be absolutely clear, when you write the query as above, you will get an asClosedEvent property returning a fragment that also includes all fields selected on IssueEvent (in the ... on IssueEvent part of the query).
I verified this works for Swift with the StarWars schema for what seems to be a similar situation. But if there is indeed a problem, it would really help to have a complete failing example I could use to reproduce it.

I can confirm that using the TypeScript code generation, the fields coming from inline fragments with type conditions are missing from the interface that is supposed to describe the results of the query.

I have an example for this:
Schema:
zoo.graphql
interface Feedable {
    _id: ID!,
    kind: String!
    feedWith: String!
}

interface Reptile {
    scaleSkin: String!
}

type Crocodile implements Feedable, Reptile {
    _id: ID!,
    kind: String!
    feedWith: String!
    scaleSkin: String!

    victims: Int!
}

type Iguana implements Feedable, Reptile {
    _id: ID!,
    kind: String!
    feedWith: String!
    scaleSkin: String!

    age: Int!
}

root.graphql
type Query {
    getZoo: [Feedable!]!
}

schema {
    query: Query
}

And now I setup a very simple query with some fragments:
zooQuery.graphql
query getZoo {
    getZoo {
        ...FeedableFragment
    }
}

FeedableFragment.graphql
fragment ReptileFragment on Reptile {
    scaleSkin
}

fragment CrocodileFragment on Crocodile {
    victims
}

fragment IguanaFragment on Iguana {
    age
}

fragment FeedableFragment on Feedable {
    _id
    kind,
    feedWith

    ... on Reptile {
        ... ReptileFragment
    }

    ... on Crocodile {
        ... CrocodileFragment
    }

    ... on Iguana {
        ... IguanaFragment
    }
}

And now I check the generated interfaces and I have this:
export interface FeedableFragment {
  _id: string;
  kind: string;
  feedWith: string;
  _id: string;
  kind: string;
  feedWith: string;
  _id: string;
  kind: string;
  feedWith: string;
}

Notice the property duplication and the fact that all inline fragment information is missing.

I'm having this issue as well. This should probably create a discriminated union type using the __typename as the discriminator.

Hello @martijnwalraven,  is there a plan to address this issue?

@IgorLipovac: Is this still an issue with the latest version? There have been some changes in this area for the Flow/TypeScript targets recently.

Looks like this is fixed. I just tried it with the zoo example above. If anyone has another case that showcases a bug, please re-open or file another issue :), thanks!


SeisNoise v0.5.3 fails to compile on M1 Mac

due to pinned dependencies in SeisIO. See https://github.com/jpjones76/SeisIO.jl/issues/93 and proposed fix https://github.com/jpjones76/SeisIO.jl/pull/94. Depending on how the SeisIO version is bumped after that fix, compat changes may be needed here too.
In a local instance of SeisNoise.jl that depends on SeisIO.jl with the proposed fix implemented, compilation of this package is successful.

Great, thanks for letting me know! I have not tested SeisNoise on an M1/M2. I'll get in touch with Josh Jones to see if he can merge quickly - should be an easy fix.

I think the comment using my forked version of SeisIO I mentioned in #101 should fix for now. We are working on moving to use SeisBase.jl  in the future.

We moved to using SeisBase.jl, can confirm SeisNoise.jl works on macOS.


能否同时加载Disqus和来必力？

两个插件同时enable

For that moment there is no 2 or more comments system dynamically loading, but it is backloged.


Network hashrate calculation

Hi!
By default, a wallet method "getmininginfo" returns the average hash rate for the number of blocks targeted for 24 hours (if developer coded it correctly, which is not always the case). However, there's also a method "getnetworkhashps" which can accept additional parameter - the number of blocks to take into account, or -1 to look since the last difficulty change onwards.
Can you please tell me which method MPOS uses and/or can it show the network hashrate based on user-configured number of blocks (or -1)?
Thank you!

MPOS wraps the standard RPC call supplied in the bitcoin class/RPC with some "catchall" methods to figure out what the network hash rate is: https://github.com/MPOS/php-mpos/blob/master/include/classes/bitcoinwrapper.class.php#L69-L96
As you can see, we are calling getmininginfo in the RPC since this is the most useful and just decide on the array keys available where to read that information. This means the call getnetworkhashps is wrapper for all coins into that multi-call. If you want to change it to match your specific need, feel free to do so :-)


Update cls.md

@npocmaka : Thanks for your contribution! The author(s) have been notified to review your proposed change.


issue 108, supporting running apiman on http too

Hi @gashcrumb, please take a look. It looks up the service scheme annotation.

Awesome!  Can you do a 'gulp build' and check in the generated hawtio-kubernetes.js also?  Thanks!

@gashcrumb done, btw the css changed too.

Great, thanks Kurt!


Grammar error

Add an apostrophe in line 82 'its' to "it's"

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.


Reduce par threads when latency penalty is known

When tdcc compiles a par block, it allocates a new FSM for each thread:
par { A; B; C; }

Each sub-program gets its own independent FSM to ensure that threads can make progress independently. However, sometime, we can have programs that look like this:
par {
  while cond { B };
  upd_reg;
}

In this case, upd_reg takes 1 cycle and the loop may take thousands. Regardless, we still allocate a whole new FSM for upd_reg. It would be better to just transform this into a seq instead and use exactly one FSM. The challenge is that, in general, we don't know how long a "simple looking" control program will take; after all, a loop is compiled into a group at some point.
Instead, we should use the newly added @promote_static(n) attribute to detect when a group (which wasn't upgraded to a static island), takes a small fraction of the cycle-time of the other threads and instead sequence it with one of the threads. We can expose a compiler knob to decide what the exactly fraction should be but the upshot is that this will enable us to reduce the number of FSMs we allocate.
It also occurs to me that the static inlining pass should annotate the generated group with a @promote_static(n) attribute so that this information can be used to reschedule par threads.

@calyxir/static-calyx this is another example of how the static extensions help the overall compiler pipeline. We should implement this for the camera ready and brag out resource benefits we get.


Add links for SDK and archive page

@SeanKilleen : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Karl-Bridge-Microsoft : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Error with fsevents

Hi. I run this repository on Ubuntu 18.04. When I run the docker-compose build command, an error related to fsevents appeared in the terminal:
Step 6/8 : RUN npm ci
 ---> Running in f46233c1cbc5
npm WARN prepare removing existing node_modules/ before installation

><EMAIL_ADDRESS>postinstall /src/node_modules/ejs
> node ./postinstall.js

Thank you for installing EJS: built with the Jake JavaScript build tool (https://jakejs.com/)

><EMAIL_ADDRESS>install /src/node_modules/fsevents
> node-gyp rebuild

gyp ERR! find Python
gyp ERR! find Python Python is not set from command line or npm configuration
gyp ERR! find Python Python is not set from environment variable PYTHON
gyp ERR! find Python checking if "python" can be used
gyp ERR! find Python - "python" is not in PATH or produced an error
gyp ERR! find Python checking if "python2" can be used
gyp ERR! find Python - "python2" is not in PATH or produced an error
gyp ERR! find Python checking if "python3" can be used
gyp ERR! find Python - "python3" is not in PATH or produced an error
gyp ERR! find Python
gyp ERR! find Python **********************************************************
gyp ERR! find Python You need to install the latest version of Python.
gyp ERR! find Python Node-gyp should be able to find and use Python. If not,
gyp ERR! find Python you can try one of the following options:
gyp ERR! find Python - Use the switch --python="/path/to/pythonexecutable"
gyp ERR! find Python   (accepted by both node-gyp and npm)
gyp ERR! find Python - Set the environment variable PYTHON
gyp ERR! find Python - Set the npm configuration variable python:
gyp ERR! find Python   npm config set python "/path/to/pythonexecutable"
gyp ERR! find Python For more information consult the documentation at:
gyp ERR! find Python https://github.com/nodejs/node-gyp#installation
gyp ERR! find Python **********************************************************
gyp ERR! find Python
gyp ERR! configure error
gyp ERR! stack Error: Could not find any Python installation to use
gyp ERR! stack     at PythonFinder.fail (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/find-python.js:307:47)
gyp ERR! stack     at PythonFinder.runChecks (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/find-python.js:136:21)
gyp ERR! stack     at PythonFinder.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/find-python.js:179:16)
gyp ERR! stack     at PythonFinder.execFileCallback (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/find-python.js:271:16)
gyp ERR! stack     at exithandler (child_process.js:301:5)
gyp ERR! stack     at ChildProcess.errorhandler (child_process.js:313:5)
gyp ERR! stack     at ChildProcess.emit (events.js:198:13)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:246:12)
gyp ERR! stack     at onErrorNT (internal/child_process.js:415:16)
gyp ERR! stack     at process._tickCallback (internal/process/next_tick.js:63:19)
gyp ERR! System Linux 4.19.76-linuxkit
gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /src/node_modules/fsevents
gyp ERR! node -v v10.19.0
gyp ERR! node-gyp -v v5.0.5
gyp ERR! not ok

I solved this problem like this, in app/Dockerfile file:
FROM node:10.15

ENV APP_ROOT /src

RUN mkdir ${APP_ROOT}
WORKDIR ${APP_ROOT}
ADD . ${APP_ROOT}

RUN echo http://mirror.yandex.ru/mirrors/alpine/v3.5/main > /etc/apk/repositories; \
  echo http://mirror.yandex.ru/mirrors/alpine/v3.5/community >> /etc/apk/repositories

RUN apk add --no-cache --virtual .gyp python make g++ \
  && npm ci \
  && apk del .gyp

RUN npm run build

ENV HOST <IP_ADDRESS>

I can't replicate this error, building with ubuntu 18.04. However, the base image node:10.15-stretch-slim can reproduce this error as it is the lighter base image without python. The base image node:10.15 should work. Could you provide something that would replicate the error please?

@wiamsuri Oh, you are right. Now I have changed from FROM node:10.19.0-alpine3.11 to FROM node:10.19.
Do you know how to link nuxt volumes in docker-compose.yml to update the code locally (git pull) and run the docker-compose exec app npm run build command? Now my docker-compose.yml file looks like this, but it does not work:
version: "3"

services:
  nuxt:
    build: ./app/
    container_name: nuxt
    restart: always
    ports:
      - "3000:3000"
    command:
      "npm run start"
    volumes:
      - ./app:/src

  nginx:
    image: nginx:1.17
    container_name: ngx
    ports:
      - "80:80"
    volumes:
      - ./nginx:/etc/nginx/conf.d
    depends_on:
      - nuxt

I'm guessing you want to develop nuxt locally inside docker. That's a great way to go. Here is an example of docker-compose file that can achieve that.
version: "3"

services:
  nuxt:
    build: ./app/
    container_name: nuxt
    restart: always
    ports:
      - "3000:3000"
    command:
      "npm run dev"
    volumes:
      - ./app:/src
      - /src/node_modules

  nginx:
    image: nginx:1.17
    container_name: ngx
    ports:
      - "80:80"
    volumes:
      - ./nginx:/etc/nginx/conf.d
    depends_on:
      - nuxt

Don't forget to change the command to npm run dev.

@wiamsuri No, I want to update my project in production. So, what i did in my server (without nuxt volumes):
cd /var/www
git clone git@gitlab.com:username/projectname.git myproject
cd myproject
docker-compose build
docker-compose up -d
And It works. But, after add volumes in nuxt service in docker-compose.yml (as I wrote above), command docker-compose up -d does not work. This is correct that does not work, because I do not have the node_modules folder locally. Here logs:
nuxt    |
nuxt    | ><EMAIL_ADDRESS>start /src
nuxt    | > nuxt start
nuxt    |
nuxt    | sh: 1: nuxt: not found
nuxt    | npm ERR! code ELIFECYCLE
nuxt    | npm ERR! syscall spawn
nuxt    | npm ERR! file sh
nuxt    | npm ERR! errno ENOENT
nuxt    | npm ERR<EMAIL_ADDRESS>start: `nuxt start`
nuxt    | npm ERR! spawn ENOENT
nuxt    | npm ERR!
nuxt    | npm ERR! Failed at the<EMAIL_ADDRESS>start script.
nuxt    | npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
nuxt    | npm WARN Local package.json exists, but node_modules missing, did you mean to install?
nuxt    |
nuxt    | npm ERR! A complete log of this run can be found in:
nuxt    | npm ERR!     /root/.npm/_logs/2020-04-20T08_28_26_165Z-debug.log
nuxt exited with code 1

And so, how can I update code in production? I thought so, but it does not work:
cd /var/www/myproject
git pull
docker-compose exec nuxt npm run build

Updating code like this in production will lead to down time. It is not ideal. Ideally, you should have a container orchestrator to manage change. However, if you have limited resource, I would say that
git pull
docker-compose up --build -d

would also do the job.

@wiamsuri Thank you very much. After docker-compose up --build -d I would add to delete the old image.
docker image prune -f

cheers


Victoria Australia Gazetteer entry

There are search records attached to the Victoria Australia Gazetteer entry. The need to be moved to the main Australia Victoria entry and the incorrect entry (does not follow Greater to Smaler ruke) needs to be removed.
Geoff

Done


LDAP sync question

Hello,
I'm currently testing Zulip with LDAP as auth backend and have a question: how can users be correctly recognised after their name changed in the directory (like marriage).
My understanding is that Zulip stores the mail address ("real" mail address from LDAP or generated by combining username and LDAP_APPEND_DOMAIN) as delivery_address and uses this to map LDAP users to Zulip user mappings.
This is also what I observed: I changed the username in the directory, logged into Zulip with the new name and got a new account.
Is there a way to prevent this so the user recognized and renamed accordingly in Zulip? Peferably logging in by username or password and use a property like objectGUID, which is unique and does not change, to map Zulip users to LDAP users.

Thanks for the feedback @d-sko, and sorry for the slow reply -- somehow this issue got missed in triage. In general questions are best asked in chat.zulip.org as we document in our community guidelines.
Our long-term plan for this is https://github.com/zulip/zulip/issues/17910; that issue mentions SAML but it should be the same project to do LDAP as well.
In the meantime you can use manage.py change_user_email to make those changes in Zulip at the time you make them in LDAP.


Create an array on the projects databases to link tickets to each project

There's no need to do this, resolving


Tic Tac Toe Game in Python

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
The aim was to build a Tic Tac Toe Game using Minimax Algorithm. It involves recursively picking out the next best possible state for the player and it makes use of backtracking.
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
This PR is with reference to issue #6995.

@HarshCasper @bhav09 @ankitaggarwal23 please review this PR :)

@Manasi2001, are you sure you have written the code because the quality of the written context changed drastically. In your previous PR, there comments that weren't aligned no proper workflow and now suddenly it has everything. I think I don't even have to search it on Google to see if it plagiarised or not.

@bhav09 the code is written by me only. 😂
Actually I had this course named 'AI' in my previous semester, in which we had to make this project. It was built under our teacher's guidance, that's why it's so presentable and all. On the the hand side, the code in my previous PR was casually written at personal end. Still if you want, you may go ahead with checking it in any manner that suits you. 😊
PS: You may want to look at this: https://github.com/TesseractCoding/NeoAlgo/pull/6977


Purpose of WebsocketRPCEndpoint.on_connect

I was trying out the package and I was going to inherit from WebsocketRPCEndpoint and override on_connect.
https://github.com/permitio/fastapi_websocket_rpc/blob/d0f322e362a50f9bb955e5a0ec66c2670c3c5f77/fastapi_websocket_rpc/websocket_rpc_endpoint.py#L110
However in my test example it's never called, and in fact, it seems like it's never referred to at any point; the handlers in the private member _on_connect are passed directly to RpcChannel:
https://github.com/permitio/fastapi_websocket_rpc/blob/d0f322e362a50f9bb955e5a0ec66c2670c3c5f77/fastapi_websocket_rpc/websocket_rpc_endpoint.py#L85
So wonder what is the purpose of on_connect? Is that supposed to be called externally by e.g. FastAPI? Was it supposed to be passed to register_connect_handler?

on_connect is exposed for deriving layers that extended or build on top of the RPC channel - see https://github.com/permitio/fastapi_websocket_pubsub for example (https://github.com/permitio/fastapi_websocket_pubsub/blob/c757bd707622252aab761a59115897dd12413478/fastapi_websocket_pubsub/pub_sub_client.py#L194)

The purpose of the on_connect argument of __init__ (which the example you linked makes use of) is clear.
I was wondering about the on_connect method of WebsocketRPCEndpoint. It creates a task that executes self._on_connect handlers, but it's never called and it duplicates the functionality already offered by RpcChannel.on_handler_event. It looks like it's not called by anything.

Yes, you seem to be right.
I guess it's seem artifact missed in a refactor - this part is 4 years old.

Would you like to open a PR to fix this? :)


[Impeller] On iOS, make Impeller on by default with an opt-out.

Deprecate the --enable-impeller flag and FLTEnableImpeller plist key. Replace them with --disable-impeller and FLTDisableImpeller key.

Tracks capping out the Impeller on iOS milestone.

We also need to update the google3 infra to always specify FLTImpellerEnabled to true or false, so that we can update the default independently there and not deal with it in a roll. FYI @chingjun

A few performance issues were discovered by the temporary devicelab benchmark flag flip:

[ ] https://github.com/flutter/flutter/issues/122408
[ ] https://github.com/flutter/flutter/issues/122406
[ ] https://github.com/flutter/flutter/issues/122405

These aren't necessarily blockers, and given that all tests ran without errors, we should attempt landing the flag-flip in the engine after the g3 infra issue is sorted out.

g3 infra changes have landed as of CL/515454075 (Thanks @chingjun !)

@zanderso May I close this as done?

It is done!

Yes.

Closing the loop here since this issue got attention. IIRC, this was completed after the provisional flip of the flag in this patch stuck its landing. Is that right @zanderso @jonahwilliams? Please correct me if I am mistaken 🙏.

Correct


Avg Attributes and Elements

Added some missing svg attributes and elements by looking at the mozilla docs.
Tests and readme where updated accordingly.

Hi @tevintchuinkam. Thank you for the PR! I'll have a look through it when I've got some spare time. 😊

@tevintchuinkam, I've been thinking about this. I'm a but reluctant to add all this to the core gomponents library, as I don't know so much about SVG myself, and I'm wondering whether it can give problems down the road to include it in an HTML rendering library.
For example, this will break code where you dot-import both HTML and SVG elements, because SVG has its own style.
I've been thinking about creating a gomponents-svg library instead, focused on rendering SVG/XML. I think it may have been a mistake to include the few SVG items I have already.
Thoughts?

@markuswustenberg  I didn't consider that. You're right.


Allow addition of restraints

Currently it's difficult to add restraints to the BLUES simulations without changing the base code. It'd be good to think of a scheme to allow easier additions of restraints.

#117 We talked about implementing the creation of systems so that it's easy to add restraints if needed (reminding @nathanmlim as he's refactoring the code).

Function for restraining has been provided here
Accessed in the SystemFactory.restrain_positions()
    def restrain_positions(cls, structure, system, selection="(@CA,C,N)", weight=5.0, **kwargs):
        """
        Applies positional restraints to the given openmm.System.
        Parameters
        ----------
        system : openmm.System
            The OpenMM System object to be modified.
        structure : parmed.Structure()
            Structure of the system, used for atom selection.
        Kwargs
        -------
        selection : str, Default = "(@CA,C,N)"
            AmberMask selection to apply positional restraints to
        weight : float, Default = 5.0
            Restraint weight for xyz atom restraints in kcal/(mol A^2)
        References
        -----
        Amber mask syntax: http://parmed.github.io/ParmEd/html/amber.html#amber-mask-syntax


Upgrade to logback 1.1.7 breaks this library when additionalFields are used - tests missing?

When the 'additionalFiels' element is used in XML configuration, logback 1.1.7 results in the following configuration error:
java.lang.IllegalStateException: Logback configuration error detected:
ERROR in ch.qos.logback.core.joran.action.NestedBasicPropertyIA - Unexpected aggregationType AS_BASIC_PROPERTY_COLLECTION

The ERROR line is repeated as many times as there are additionalFields.
Basically, logback 1.1.7 is broken: http://jira.qos.ch/browse/LOGBACK-1158, http://jira.qos.ch/browse/LOGBACK-1164 and possible more, all fixed here: https://github.com/qos-ch/logback/commit/2ba396a20e44d2f. The fix will be available in 1.1.8, not yet released.
I guess there is nothing to be done as the only difference between logback-gelf 1.1.8 and 1.1.9 is the upgrade to logback 1.1.7, and people can just force the earlier lib. But, there seems to be a test missing which tries a full configuration with logback, which would catch such a logback regression.

I agree there should be a test, but that test belongs in logback. It should not be this project's responsibility to test another project's implementation.


Black window

Description
After uploading 8 changes, the map windows became completely black (no background image, no geographical object
Version
2.0-alpha
What browser are you seeing the problem on? What version are you running?
Chrome v106.0
Steps to reproduce
No response
The browser URL at the time you encountered the bug
https://mapwith.ai/rapid-v2-alpha#background=Maxar-Premium&datasets=fbRoads,msBuildings&disable_features=boundaries&map=16.00/NaN/NaN

Thanks for the bug report! If you still happen to have that browser window around on your desktop, could you please check to see if there are any debug log traces in it?
Select the three-dot menu in the upper right, then more tools -> Developer Tools, then click the 'Console' tab and copy/paste what you find there.

There's no good way to copy from the console, I just have to select things manually with the mouse:

closing as stale