Support Msys2 platform

Support not only Cygwin but Msys2 platform (sys.platform -> 'msys').
File not found otherwise.
Traceback (most recent call last):
  File "/usr/bin/gsutil", line 11, in <module>
    load_entry_point('gsutil==4.39', 'console_scripts', 'gsutil')()
  File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 489, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2843, in load_entry_point
    return ep.load()
  File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2434, in load
    return self.resolve()
  File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2440, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/usr/lib/python2.7/site-packages/gslib/__main__.py", line 69, in <module>
    from gslib import wildcard_iterator
  File "/usr/lib/python2.7/site-packages/gslib/wildcard_iterator.py", line 44, in <module>
    from gslib.third_party.storage_apitools import storage_v1_messages as apitools_messages
  File "/usr/lib/python2.7/site-packages/gslib/third_party/storage_apitools/storage_v1_messages.py", line 22, in <module>
    from apitools.base.py import encoding
  File "/usr/lib/python2.7/site-packages/apitools/base/py/__init__.py", line 23, in <module>
    from apitools.base.py.credentials_lib import *
  File "/usr/lib/python2.7/site-packages/apitools/base/py/credentials_lib.py", line 44, in <module>
    import fasteners
  File "/usr/lib/python2.7/site-packages/fasteners/__init__.py", line 23, in <module>
    from fasteners.lock import locked  # noqa
  File "/usr/lib/python2.7/site-packages/fasteners/lock.py", line 24, in <module>
    from fasteners import _utils
  File "/usr/lib/python2.7/site-packages/fasteners/_utils.py", line 39, in <module>
    from monotonic import monotonic as now  # noqa
  File "/usr/lib/python2.7/site-packages/monotonic.py", line 169, in <module>
    raise RuntimeError('no suitable implementation for this system: ' + repr(e))
RuntimeError: no suitable implementation for this system: OSError('No such file or directory',)

I don't have a Windows system that I could test this on. Would you like to send a pull request?

msys2 ships with python 3.7, so I don't see a lot of value in this. It has also not been requested in the five years or so this package has been around, which I take as a signal that this is a pretty niche platform.


Add SkeletonView as example

Hi @leogdion! thanks for this repo. I think it's very useful for everyone 👍🏼
I have a project and IMHO it would be a good example of a README file. The project is SkeletonView.
What do you think? Could we include it?
Thanks! 🙂

Sure! It looks nice.
Go ahead and put in a PR. Add it under the Examples section.
Thank you!


Log swallowed exceptions in RedissonRemoteService

Log exception when can't process the remote service request, like when codec can't unmarshall the RemoteServiceRequest (eg: when using Typed/JsonJacksonCodec).
Closed bug #4163 has root on this, not entering this whenComplete section.

Thanks for contribution!


Number of clients shown in device overview is always zero

The number of clients shown in the device overview table is always zero, i.e. a dash '-' is shown instead of the actual value.
This definitely affects v3 devices. We currently don't have any v4 devices in testing, so I cannot say anything about that.

hi @mbecks - this is already open internally and being fixed currently. thanks for raising


feat: disable max button after setting max balance

disable max button when max balance is already set in the coin input
fix network fee being applied to DAI token
added tests to helper function which calculates maximun balance
added tests to make sure max button is disabled after clicked

Closes #357
Closes #359

Max balance is never shown here.


A transitive dependancy error comes when building the project using docker container for SLAlpha4

Description:
When a ballerina HTTP service is created and build using ballerina SLAlpha4 docker container it gives an error as
error: compilation failed: Transitive dependency cannot be found: org=ballerina, package=task, version=2.0.0-alpha8

The screenshot of the terminal is shown below.

Steps to reproduce:

Create a new ballerina project
Create a simple HTTP service inside that. BBE example
Build the project using the SLAlpha4 docker container

Affected Versions:
Ballerina SLAlpha 4
OS, DB, other environment details and versions:
Ubuntu 20.04 LTS
Ballerina SL Alpha 4
JDK 11
Other references:
Find the link for the failed GitHub action of a similar build here.

@keizer619 , could you please prioritise this as it is a blocker for Alpha4 releases of the connectors?

Closing the issue as this issue is fixed in the central manually.


[FEATURE REQUEST] 3-Month View: Last, Current, Next

Is your feature request related to a problem? Please describe.
Paid once a month near end of month and want to see upcoming month's expected expenses easily.
Describe the solution you'd like
Instead of a choice to view last three months, I'd like a choice to select three months: last, current, next
Describe alternatives you've considered
Currently need to use the clunky month selection drop downs.
Additional context
Older version could show upcoming month.

I've often wished for this! Right now, I have to manually update my account views at the beginning of each month to accomplish this.

I've wanted this for a long time! I asked the YNAB team about this and they said they'd put some thought behind it but no 3-month feature yet... Would love to see a toolkit option for this.


Seeking new maintainer(s)

As I am not actively working with Neo4j anymore and because I would like to have time for other projects, I'm not able to dedicate time to the Neo4j.rb gems (neo4j, neo4j-core, and neo4j-rake_tasks).
I have enjoyed the past 4.5 years helping to build these gems and the community around them.  If somebody is interested I would ask them to contact me in Gitter (either in the support channel or in a DM).
On top of the issues / PRs currently outstanding in the GitHub repos, I have a personal list of things I've wanted to work on as well as a sense of some things that are more urgently requested by users of the gem.  Off the top of my head these are:

Support for SSL in the Bolt protocol (this is probably almost finished.  I did some work to refactor the Bolt adaptor to support this in the neo4j-core gem and also included some performance improvements.  I think it is largely working with one report of a strange issues)
Support for id as an ActiveModel attribute in ActiveNode the neo4j gem (there could also be a larger refactoring of ActiveNode / ActiveRel attributes here)
Refactoring the Query class in the neo4j-core gem to not re-order Cypher clauses on your behalf (this was inspired by ActiveRecord's arel gem before I had a better understanding that Cypher is not the same as SQL in this way)

I also can provide background of the gems which would be helpful in making sense of the code.

@cheerfulstoic I think I speak for many, many people who have/are making use of this gem when I say THANK YOU for the hundreds thousands of volunteer hours you have donated to help us!!!
In the time I've used this gem / watched this repo, you've proven yourself to be tirelessly friendly and welcoming, and gone way beyond what I would ever expect in making yourself available to answer questions ASAP on Gitter and via issues. While I've never met you in real life, I'm confident in saying that you're an amazing and kind human being. I'm definitely very sad to hear of your transition, but it is 1000x earned! And congratulations on the new job 😄.
While I'm not prepared to offer myself up as a potential replacement, I will endeavor to increase my participation answering the occasional questions which pop up. If someone does feel they can step into the maintainer role, I can potentially help out. This might be a situation where multiple people can try and fill pieces of the work you were, largely single handedly, doing before.

We ❤ you @cheerfulstoic

Thanks very much for all of the kinds words ;)
Question answering on Gitter / GitHub would definitely be great, thanks!
Another thing that I just thought of that I should clarify: the minimum requirement (aside from answering questions) which I've been doing for a little while now is working through incoming PRs.  I've been inspired by the C4 protocol on maintaining communities which basically says to do not much more than a sanity check on PRs and if somebody wants something different they can make their own PR.  I don't think I don't that exactly and I usually give feedback on various changes that can be made, but I try to let the community drive when I can.  Dealing with PRs also involves doing releases to rubygems.  It's generally a lot easier for me to release every time there's a fix / feature.  Bigger projects like Rails have a team and regular meetings where they can plan out collecting a number of features in a single version change, though when you're just maintaining something by yourself it's a lot less overhead to just release it immediately (and it has the side effect of making people happy ;) ).  But I hope that the project can have more than just the one maintainer in the future.

Just spotted this: https://neo4j.com/blog/this-week-in-neo4j-getting-started-with-neo4j-desktop-and-browser-graphileon-personal-edition-intuitive-detections-research-with-neo4j/#featured-community-member
👍

:+1: Thanks for the heads up!  Hopefully there will be some interest


Rename some JSX-specific rules to include jsx- prefix

Fixes #668, addresses #686.

Rebased and conflicts resolved!

LGTM

Merges, thanks!

Finished the cleanup of this rule in 2f7a462. Since it was never released in a non-alpha, non-rc build, there's no need to keep the old one around in any form.


added polygon utility example

Added a visual example of the polygon utility function clipPolygon to the documentation.

You need to change the Doc here: https://github.com/Leaflet/Leaflet/blob/81dc24892a3ffd4b53158bb0b5533ae55c1931c1/src/geometry/PolyUtil.js#L7-L14

@mourner do we want this in the docu? If yes I will update the docs


Length one dict saves but wont read?

While trying to generate a minimal example for another bug I ran into this:
In [1]: from asdf import AsdfFile

In [2]: import numpy as np

In [3]: arr = np.random.random((11))

In [4]: with AsdfFile({'wave': arr}) as ff:
   ...:     ff.write_to("test.asdf")
   ...:

In [5]: with AsdfFile("test.asdf") as ff:
   ...:     ff.resolve_and_inline()
   ...:     a = ff.tree
   ...:

---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-5-efc4a1846873> in <module>()
----> 1 with AsdfFile("test.asdf") as ff:
      2     ff.resolve_and_inline()
      3     a = ff.tree
      4

~/Git/asdf/build/lib.linux-x86_64-3.6/asdf/asdf.py in __init__(self, tree, uri, extensions, version, ignore_version_mismatch, ignore_unrecognized_tag, copy_arrays)
    108             self.find_references()
    109         else:
--> 110             self.tree = tree
    111             self.find_references()
    112         if uri is not None:

~/Git/asdf/build/lib.linux-x86_64-3.6/asdf/asdf.py in tree(self, tree)
    279     @tree.setter
    280     def tree(self, tree):
--> 281         asdf_object = AsdfObject(tree)
    282         self._validate(asdf_object)
    283         self._tree = asdf_object

ValueError: dictionary update sequence element #0 has length 1; 2 is required

I've had this bite me before too, and it's possible that we should have a better error message, but here's what you need to do to open the file:
import asdf

with asdf.open('test.asdf') as ff:
    ff.resolve_and_inline()
    a = ff.tree

However, when I do this, I get another error when attempting to access the array. This appears to be related to the use of resolve_and_inline within a context handler, but is not related to the issue you reported.
Try doing this instead just to see if it works in the meantime:
import asdf

with asdf.open('test.asdf') as ff:
    a = ff.tree


Update travis configruation

Update travis configruation to use common conda environment configuration file - environment.yml

I tend to "won't fix" this issue as its scope is too narrow. To be closed in favour of #139.

Fixed while working on #139


Fixed issue with un-resolvable 'components' directory.

These commits should fix issues with the src/components directory being unresolvable as noted in issue #4.
I have also added a webpack config variable (bail: true) that should prevent this issue from being overlooked in the future. This will allow CI (if implemented) to fail and not push a new build to npm if an error occurs during the build process.

How it should be configured with webpack /ES6 ?
It did not work at all for me, as all components were not recognized..


Rules: Add the match_not predicate (release v0.2.0)

There are situations where we might need to negate the outcome of a rule match test. One such example is updating the summary endpoint for non-wiktionaries. This PR adds the functionality via the match_not predicate. If it is present, its condition is evaluated and negated and the message is accepted only if it doesn't match it. This works rather well in conjuction with the match predicate: if both are specified, they both need to be evaluated positively in order for the message to be accepted. Note that match_not is used only for testing messages, it cannot be used for rule execution expansion.
cc @wikimedia/services

May be better not_match?

I'm wondering how far we should push this custom match DSL, rather than supporting functions / expressions as in regular request handlers.

Coverage increased (+0.1%) to 95.259% when pulling 0d8346557772f7c494587dfcbd36d490855e2bd6 on d00rman:rules/match_not into a8486a9edee5af3ca970cd80b393a4c9b298f1f9 on wikimedia:master.

May be better not_match?

I'm not really happy with match_not either, I was in a dilemma between match_not and dont_match but went for the former because it has a note of poetry in it while still being an imperative :)

I'm wondering how far we should push this custom match DSL, rather than supporting functions / expressions as in regular request handlers.

I don't find match_not too sexy either, but I think it's more convenient than having functions / expressions for that (IMHO, it's more readable like this). But in general I agree that if start adding stuff like crazy to it. Maybe revisit this in the future if/when we find features that are needed but lacking?

👍


Don't mutate class attribute

Don't mutate class attribute
Comment out unused imports

@dandavison - the CI issue is caused by macos-latest now pointing to macos-14 which is arm instead of intel. The fix is probably to use actions/setup-python@v4 instead of actions/setup-python@v1, set macos-intel and macos-arm as macos-12 and macos-14 respectively, and disable 3.8 for macos-arm. See https://github.com/temporalio/sdk-python/pull/506. Can be done in this PR or in a separate one your choice (or work can just be put on the backlog if time is limited).

the CI issue is caused...

Thanks! Opened https://github.com/temporalio/samples-python/issues/117 for that.


Feature autocorrect

I'm using VSCode 1.21.
@abe33 I'm getting this in the console (running a debug instance of VSCode from the main instance of VSCode). This was shown after saving a file with simple content.
I've executed npm install in the plugin's git repository before running VSCode.
C:\Program Files\Microsoft VS Code\Code.exe --debugBrkPluginHost=18724 --debugId=7744b279-76bf-4ae5-a864-8969bd120bae --extensionDevelopmentPath=C:\Users\jansorg\source\vscode-plugin
TypeDefinition null
(node:6208) DeprecationWarning: os.tmpDir() is deprecated. Use os.tmpdir() instead.
Error: Error 501: not implemented

	at module.exports.getAutocorrectData.promisifyRequest.then.promisifyReadResponse.then.data (C:\Users\jansorg\source\vscode-plugin\src\kite-editor.js:187:17)
	at <anonymous>
	at Immediate._tickCallback (internal/process/next_tick.js:109:7)
	at runCallback (timers.js:672:20)
	at tryOnImmediate (timers.js:645:5)
	at processImmediate [as _immediateCallback] (timers.js:617:5)
TypeError: Cannot read property 'requested_buffer_hash' of undefined
	at module.exports.onWillSave.Promise.all.getAutocorrectData.then.data (C:\Users\jansorg\source\vscode-plugin\src\kite-editor.js:56:17)
	at <anonymous>
	at Immediate._tickCallback (internal/process/next_tick.js:109:7)
	at runCallback (timers.js:672:20)
	at tryOnImmediate (timers.js:645:5)
	at processImmediate [as _immediateCallback] (timers.js:617:5)

@abe33 This is looking great! :)
I'm still seeing the newline issue here. Let me know how I can help to debug this.
This is what is shown after I saved the file:

(@mwichary I think it's a bit difficult to see whether the switch next to "Enable Error Rescue" is on or off. )

@mwichary Maybe a colored know or background when enabled?

@jansorg I think the disappearing of the entire bottom section + the graying out of the “Any time code is fixed” below in the disabled state should help in communicating what’s going on, so I’m not personally too worried. But if you’d like to experiment with the colour or the shape of the toggle, or have other ideas, please explore this further and we can chat!

@abe33 I've updated the vscode plugin to the latest commit and started it in the VSCode host app (Run->Debug).
I'm getting this error on save:

@jansorg That's now fixed, the issue that while wanting to debug that stuff, the log statement was falsely making the promise resolves instead of getting rejected, resulting in the second error.


psi: initial Psi4 Mol extensions

Description
Molecule JSON exported from Psi4 has several extra fields already checked for self-consistency so that the Mol may henceforth be used without reconsulting a periodic table. This defines the fields of that main psi4mol extension. There are also two more extensions, one for EFP fragments and one for transmitting ZMat mol specifications.
Questions

[ ] I tried to follow guidelines here. Properly there should have been a secondary repo. Here I just made an extensions folder. I'm sure that won't do for structuring a complex schema. I'll investigate further.
[ ] The minimal required input fields for EFP are actually fragment_files, hint_types, geom_hints, and units. All other fields are valid after and EFP system has been populated by an EFP-reading program into a consistent, along-side "viz" representation. These alternate sets of "required" fields have not yet been programmatically imposed.

Status

[ ] Ready to go

This is interesting as it is a formalization of the considered extensions. I would be interested in @ghutchis's (and others) thoughts on this.

This original molecule extension PR will be split.

what remains in the PR is the bit that should be incorporated into moleucle_schema proper, which is 4 new optional keys
from offline discussions it sounds like https://github.com/MolSSI/QCSchema/pull/44/commits/9cf3dffe9afd79696ef2fa7b8d9377870387eaf3#diff-fb40813be873a1ea003de289cb1f9373R6 and following line should be deleted. and the whole schema bumped from 1-->2. My opinion is that, while we shouldn't go overboard on segmenting the schema, having a few separate pieces doesn't hurt, especially when there are whole software projects that only deal with one piece (e.g., molviz and basissetexch)
remaining pieces of original PR now missing in this PR are zmat and efp extensions. Are separate or combined extension PR(s) wanted for that?

Can you make an issue about z-mat and EFP and discuss the extensions there? I don't think we have a good framework of where to put these yet.


feat: sigma-compact measure zero sets are meagre

Should this rather be moved to its own file?

[x] depends on: #7576
[x] depends on: #7730

Note: I have pushed an update to the lean toolchain because this PR was on a buggy version of the toolchain. WARNING: checking out old commits of this PR using v4.2.0-rc2 or v4.2.0-rc3 can cause lake clean to delete your mathlib folder! If you need to do so, make sure to delete lakefile.olean manually before running any lake commands.

Thanks for the comments! I'm still getting used to the mathlib naming convention; thanks for your patience.
I have addressed all comments.

Thanks again; updated.
In the process, I realised two of my lemmas were already in that file.

That build failure is due to lean4checker - I cannot imagine what would have caused this. Probably a false positive.

Thanks for another round of good comments; I have updated the code again.

Thanks! :tada:
bors merge


Support for OAS 3.1?

https://apisyouwonthate.com/blog/openapi-v3-1-and-json-schema/

I do not like posting only URL to a random blog.

There was a title, and the title contains the entirety of the issue:

Support for OAS 3.1?

The URL was just for reference.  Sorry, next time I'll leave out the URL.


[No QA] Handle Moderated Messages in UI

cc @thienlnam
Details

This SHOULD work but right now you have to manually adjust the state to make the messages "flagged" to test it because we don't have moderationDecisions yet.
IOS was broken for me on dev - both on main and in this branch, so it's unrelated - so I was unable to get a screenshot. Please check this platform extra carefully. (I tried for like 30 min to fix it but haven't yet.)
Fixed Issues

$ https://github.com/Expensify/App/issues/18509
Tests

For now, please edit this line to be default true
and this line to be default the status you're attempting to test. (functionality to actually moderate these messages isn't quite there but this needs to go out soon.
The effects should be:

APPROVED and PENDING - no visible change from existing behavior
PENDING_HIDE and HIDDEN should be hidden as in the screenshots, with option to reveal

These will be sent to the frontend via themoderatorDecision in the message array, it will look like:
moderatorDecision: [{
	decision: "pending"/"pendingHide"/"removed"/"hidden"/"approved",
	timestamp: timestamp
}]

There will only be one decision max.

[X] Verify that no errors appear in the JS console

Offline tests

Same as online
QA Steps

No actual QA for now - just change the default status of messages as noted above to see the effects.

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[X] iOS / Safari
[X] MacOS / Chrome / Safari
[X] MacOS / Desktop

[X] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[X] I followed proper code patterns (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[X] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[X] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I followed the guidelines as stated in the Review Guidelines
[X] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] I verified that if a function's arguments changed that all usages have also been updated correctly
[X] If a new component is created I verified that:

[X] A similar component doesn't exist in the codebase
[X] All props are defined accurately and each prop has a /** comment above it */
[X] The file is named correctly
[X] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[X] The only data being stored in the state is data necessary for rendering and nothing else
[X] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[X] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[X] All JSX used for rendering exists in the render method
[X] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[X] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

Just wondering, is the backend not set up yet? And if so, why is the frontend going out first?

@MariaHCD So it's all in process, the actual flagging is set up in the backend but the ability for concierge to make the decisions on it isn't quite yet. Basically we're trying to get this out quickly (for Thursday) so pushing the pieces we can, when we can. Definitely not ideal, of course, but we'll get there!

@thesahindia Could you please do a code review for this and verify it doesn't break the existing app reportActions?

Over to you, @thesahindia! :)

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.3.21-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Fix invalid plan with precomputed hash optimization in scalar subquery

A query that includes an aggregation, join or other operation at the top-level
of a scalar subquery breaks when the pre-computed hash optimization kicks in
due to the additional column containing the hash.
When processing EnforceSingleRow, add a projection to restore schema
expected by that node.

looks good


chip-control app heap usage increased drastically in TE7 branch

Problem
We had an issue on the TE7 chip-tool build, the heap size increased from 2MB to 151MB comparing to TE6 version.
TE7 build on RPI4:
==5551== HEAP SUMMARY:
==5551== in use at exit: 121,468 bytes in 1,312 blocks
==5551== total heap usage: 18,031 allocs, 16,719 frees, 151,176,146 bytes allocated
TE6 build on RPI4:
==4382== HEAP SUMMARY:
==4382== in use at exit: 114,475 bytes in 1,241 blocks
==4382== total heap usage: 12,362 allocs, 11,121 frees, 2,023,257 bytes allocated
*Expected behavior:
We should be conscious about memory usage even for a controller implementation. The heap usage should be tracked between releases.
*Actual behavior:
Due to this memory jump, we cannot run it on our controller implementation due to limited RAM
*Steps to reproduce
Run with valgrind for pair a IPv4 device using chip-tool.
valgrind-TE6.txt
valgrind-TE7.txt

System configuration: RPi4 with TE release images

Proposed Solution
Identify the cause of heap usage jump and reduce it
Include a validation step to check the heap usage in each release

This may also be a result of moving some pools onto heap instead of static memory. I am not convinced this is something we should fix even for 1.x based on the given report.

A big memory jump like this cannot be ignored or pushed away without detailed analysis report. There are controllers running on devices other than phone or pad. 150MB memory is not something like a TV or a STB can easily give away.


Add docs for Authorization Configuration

Placeholder docs for https://github.com/kubernetes/enhancements/tree/master/keps/sig-auth/3221-structured-authorization-configuration
ref: https://github.com/kubernetes/enhancements/issues/3221

/sig auth

This will also need a change to https://kubernetes.io/docs/reference/config-api/

Hi, @palnabarun! v1.29 Docs Lead here. Please take a look at Documenting for a release - PR Ready for Review to get your PR ready for review before Tuesday 14th November 2023. Thank you!

Hi, @palnabarun! Just dropping by to remind you that the deadline for docs PRs to be ready for review is tomorrow, November 14. Is there anything we can help you with to get this PR ready?

As discussed in Slack, your extended deadline to have this PR ready for review is Friday, 17 November

This will also need a change to https://kubernetes.io/docs/reference/config-api/

@sftim -- could you please point me to the steps to regenerate those? I was trying the process here: https://kubernetes.io/docs/contribute/generate-ref-docs/kubernetes-api/, but it looks the steps are supposed to be run after a release tag is cut.

This will also need a change to https://kubernetes.io/docs/reference/config-api/

@sftim -- could you please point me to the steps to regenerate those? I was trying the process here: https://kubernetes.io/docs/contribute/generate-ref-docs/kubernetes-api/, but it looks the steps are supposed to be run after a release tag is cut.

We're not asking for you to run generation (either in this PR, or during the release). Actually the ask is to help revise https://github.com/kubernetes-sigs/reference-docs/ to cover the changed config formats.

Hi, @palnabarun! We are now past your extended deadline to provide docs for this enhancements. What can I do to get this across the line?

@katcosgrove added the documentation now. I missed committing the changes.

@sftim ack for clarifying. The initial ask wasn't clear. So, I guess no-op on this PR related to your comment.

Hi, @palnabarun! Just a reminder that the deadline to have this PR reviewed and merged is Tuesday, 28 November. Let me know if you need any help!
CCing KEP owners: @deads2k @liggitt @ritazh

/triage accepted

/lgtm

/assign @nate-double-u


Log and Return error-message on function-stats failure

Motivation
Right now, for some-reason when get-function-status api call fails then , app-log doesn't log exception with function reference and it doesn't give correct failure-response code.
Modifications
Fix error-logging and response code on function get status ap call.

rerun tests

rerun cpp tests

rerun java8 tests


ADR Deduplicated blob garbage collection with Bloom Filters

Should we add information about the limitation of BloomFilter?
It is a risk the blob is unreferenced but is not delete.

**Bloom filters** are probabilistic data structures. Here the reference prediction can produce false positives: we might
skip some non referenced blobs that should have been garbage collected. However, the associated probability can be tuned
and by adding a salt we can ensure subsequent runs will have different sets of false positives and thus that all blobs is
eventually garbage collected.

Explains just that.


Is there any image augmentation used in this implementation?

I have just started to use this implementation and I wondered whether we have to do the augmentation on our own.

There is more then just flipping and cropping.
In trainval_net.py roibatchLoader calls get_minibatch from lib/roi_data_layer/minibatch.py
which calls prep_im_for_blob from lib/model/utils/blob.py
which subtracts PIXEL_MEANS from every image.
Afterwards, in the same function, the image is resized using cv2.resize. (As explained in the blog)
The values for PIXEL_MEANS and the default resized dimensions can be found in /lib/model/utils/config.py:
# Scale to use during training (can list multiple scales)
# The scale is the pixel size of an image's shortest side
__C.TRAIN.SCALES = (600,)
# Max pixel size of the longest side of a scaled input image

__C.TRAIN.MAX_SIZE = 1000

Also mentioned there are the default sizes to which an image is then cropped:
# Trim size for input images to create minibatch
__C.TRAIN.TRIM_HEIGHT = 600
__C.TRAIN.TRIM_WIDTH = 600

@jwyang I wonder: where do the annotation bounding box positions get adjusted to fit these new scaled, cropped and/or flipped images?

@RafaelMostert Do you find where the bounding box is adjusted?

There is more then just flipping and cropping.
In trainval_net.py roibatchLoader calls get_minibatch from lib/roi_data_layer/minibatch.py
which calls prep_im_for_blob from lib/model/utils/blob.py
which subtracts PIXEL_MEANS from every image.
Afterwards, in the same function, the image is resized using cv2.resize. (As explained in the blog)
The values for PIXEL_MEANS and the default resized dimensions can be found in /lib/model/utils/config.py:
# Scale to use during training (can list multiple scales)
# The scale is the pixel size of an image's shortest side
__C.TRAIN.SCALES = (600,)
# Max pixel size of the longest side of a scaled input image

__C.TRAIN.MAX_SIZE = 1000

Also mentioned there are the default sizes to which an image is then cropped:
# Trim size for input images to create minibatch
__C.TRAIN.TRIM_HEIGHT = 600
__C.TRAIN.TRIM_WIDTH = 600

@jwyang I wonder: where do the annotation bounding box positions get adjusted to fit these new scaled, cropped and/or flipped images?

Hi, would mind telling me where is the crop function in the codes? I only know how to set flip in the codes, but I haven't find crop.

There is more then just flipping and cropping.
In trainval_net.py roibatchLoader calls get_minibatch from lib/roi_data_layer/minibatch.py
which calls prep_im_for_blob from lib/model/utils/blob.py
which subtracts PIXEL_MEANS from every image.
Afterwards, in the same function, the image is resized using cv2.resize. (As explained in the blog)
The values for PIXEL_MEANS and the default resized dimensions can be found in /lib/model/utils/config.py:
# Scale to use during training (can list multiple scales)
# The scale is the pixel size of an image's shortest side
__C.TRAIN.SCALES = (600,)
# Max pixel size of the longest side of a scaled input image

__C.TRAIN.MAX_SIZE = 1000

Also mentioned there are the default sizes to which an image is then cropped:
# Trim size for input images to create minibatch
__C.TRAIN.TRIM_HEIGHT = 600
__C.TRAIN.TRIM_WIDTH = 600

@jwyang I wonder: where do the annotation bounding box positions get adjusted to fit these new scaled, cropped and/or flipped images?

__C.TRAIN.TRIM_HEIGHT = 600
__C.TRAIN.TRIM_WIDTH = 600
those 2 parameters are actually not used in image cropping

# Trim size for input images to create minibatch __C.TRAIN.TRIM_HEIGHT = 600 __C.TRAIN.TRIM_WIDTH = 600
those 2 parameters are actually not used in image cropping @RafaelMostert


toggle the ruler by hotkey

Hi, thanks for sharing this awesome package.
I'd like to request a feature if possible. I don't want to see rulerz all the time. Only when I want to check if my code is lining up.
I'd really appreciate if rulerz added a function that can be activated by hotkey.
Here's an example
'atom-text-editor':
  'ctrl-q': 'toggle-quotes:toggle'
  'ctrl-r': 'rulerz:2000'
  'ctrl-shift-r': 'rulerz:0'

Where the number provided to rulerz is a timeout in milliseconds. So if I provide a 2 second timeout, then the ruler will disappear after 2 seconds, if I provide zero then it toggles on or off, permanently.
If it's difficult accept an unknown number of milliseconds as a parameter, then options for 1-9 seconds would be a great plan B, like this: 'ctrl-r': 'rulerz:2' meaning 2 seconds, 'ctrl-r': 'rulerz:3' meaning 3 seconds, etc.

Hi @lope. Thanks for using Rulerz, I'm glad you find it useful. I don't have much time to add new features to the plugin, but if you're passionate about adding a toggle feature (I suspect many other people would use this as well) I'd be happy to work with you on that.


Escape Crystal Teleport Notification

Allows user to get notified if the escape crystal would activate upon taking damage

You must ensure the added infobox is removed when your plugin is turned off; currently no shutDown() method exists, so this is not happening.

Closing due to inactivity.


[Bug Report] DatePicker changes V-model to trigger component display

Element UI version
1.4.4
OS/Browsers version
win10， chrome60
Vue version
2.4.2
Reproduction Link
https://jsfiddle.net/xj80vxsr/
Steps to reproduce
1、点击日期组件
2、点击按钮
What is Expected?
点击按钮不会触发日期组件出现，而是直接改变v-model的值
What is actually happening?
修改v-model导致picker出现

I hava met the same question.
I tried a lot and found that when clicking out site, handleClose function worked but the input component status is still 'focused'. When I change the value of the V-model, the watcher of value in date-range.vue will work and show the pickerpanel.
I add a variable and bind a click.native event on the input element to fix this, but it's actually a temporary solution.

Fixed in 2.0, Closing this issue.


Upgraded versions to use Meteor1.7 and Angular6

Hi @Urigo and @ardatan,
I have upgraded the versions of all-in-one example of MeteorCLI to use Meteor <IP_ADDRESS> and Angular 6.
Fixed the problems of rxjs6 in server/main.ts and imports/app.component.ts
The code is working fine with --production and AOT=1 flag.
Hope to see your comments on this PR.
Arslan.

thank you @ansaries , could you please also check out this PR and maybe work together on fixing the CI?
https://github.com/Urigo/angular-meteor/pull/1927#issuecomment-399703226

Hi @ansaries and @ardatan this has been fixed in #1927

Thank you @ansaries and @georgyberdyshev for your contribution!


Fix: IOU - No error message when changing amount of a paid request

Details

https://github.com/Expensify/App/issues/33799
Fixed Issues

$ https://github.com/Expensify/App/issues/33799
PROPOSAL: https://github.com/Expensify/App/issues/33799#issuecomment-1873879156
Tests

User A - Request money from User B
User A - Open that request amount page
User B - Pay the request came from User A
User A - Update amount and save in that amount page
Verify that it shows failure error for this operation

[X] Verify that no errors appear in the JS console

Offline tests

N/A
QA Steps

User A - Request money from User B
User A - Open that request amount page
User B - Pay the request came from User A
User A - Update amount and save in that amount page
Verify that it shows failure error for this operation

[X] Verify that no errors appear in the JS console

PR Author Checklist

[X] I linked the correct issue in the ### Fixed Issues section above
[X] I wrote clear testing steps that cover the changes made in this PR

[X] I added steps for local testing in the Tests section
[X] I added steps for the expected offline behavior in the Offline steps section
[X] I added steps for Staging and/or Production testing in the QA steps section
[X] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[X] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[X] I included screenshots or videos for tests on all platforms
[X] I ran the tests on all platforms & verified they passed on:

[X] Android / native
[X] Android / Chrome
[X] iOS / native
[X] iOS / Safari
[X] MacOS / Chrome / Safari
[X] MacOS / Desktop

[X] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[X] I followed proper code patterns (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[X] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[X] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I followed the guidelines as stated in the Review Guidelines
[X] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] I verified that if a function's arguments changed that all usages have also been updated correctly
[X] If a new component is created I verified that:

[X] A similar component doesn't exist in the codebase
[X] All props are defined accurately and each prop has a /** comment above it */
[X] The file is named correctly
[X] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[X] The only data being stored in the state is data necessary for rendering and nothing else
[X] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[X] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[X] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[X] All JSX used for rendering exists in the render method
[X] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[X] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/14358475/0dc056f7-ff13-4543-8748-cd910ae603ef

Mobile Web - Chrome

Mobile Web - Safari
https://github.com/Expensify/App/assets/14358475/59797fbe-41f2-493f-9e60-366d27a157ec

Desktop

iOS
https://github.com/Expensify/App/assets/14358475/a7df346a-f0f4-4c55-b69e-4e85b1183fcf

Android

cc @Pujan92 There is still missing screenshot for Mobile Web - Chrome.

Thanks, Added!

🚀 Deployed to staging by https://github.com/nkuoch in version: 1.4.22-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
cancelled 🔪

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/nkuoch in version: 1.4.22-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/thienlnam in version: 1.4.22-6 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Panels Programmatically

Is there a way to build Panels without the need for Storyboard?

but I'm working to make Panelable nicer :)

"Trying to instantiate something that does not conform Panelable :("
import UIKit
import AVFoundation
import Panels
class ViewController12: UIViewController, Panelable {
@IBOutlet var headerHeight: NSLayoutConstraint!
@IBOutlet var headerPanel: UIView!
}
My storyboard name is "Main". Help me, I do not understand what the problem is. Thank!

Thank! Already figured out.

Thank! Already figured out.

What was your solution? I have the same problem doing programmatically.  Please help.

Good morning! I just deleted and installed again Panelable. Nothing else helped.

Hi,
Its my code.. what is the problem?
let panelManager = Panels(target: self)
let panelConfiguration = PanelConfiguration(size: .oneThird)
panelManager.show(panel: self, config: panelConfiguration, view: myView)
myVIew is just existing UIView
self is UIController & Panelable
runtime error
Panels.swift: 38 Line
parentViewController?.addContainer(container: panel)


Make extra LN gateway route hints optional

While testing I found out that CLN apparently has problems with seeing itself as the first hop of a route hint and won't route (complains that it can't find itself). This sounds similar to the limitation that CLN can't pay itself.
To work around that in small testing networks with only two Lightning nodes we should add a config option or command line flag to the gateway that makes it start as if no extra route hints were available.
A separate issue is why did this not occur in CI so far? The CLN version in the test setup was v0.12.1. Or do we start the gateway so quickly that the channel isn't open yet, so not returned as a route hint?

This was fixed with https://github.com/fedimint/fedimint/pull/2956


Added IP lock and auto mode

Thanks so much.  There are conflicts, so I'll resolve and merge the PR.

What have I done...
This whole thing is so dumb. Please let us have it closed. Thank you.


A tag don't work

"a Tag" works fine in react-markdown, but not in react-md-editor's preview.

Maybe I use this component.
import ReactMarkdownWithHtml from 'react-markdown/with-html';

Is there any way I can extend it?
I hope the problem is solved well, and thank you for making a nice app :)

https://github.com/uiwjs/react-md-editor/issues/56#issuecomment-717892235
@MoonSupport  You can try to setting props?: ReactMarkdownProps;

transformLinkUri - function|null Function that gets called for each encountered link with a single argument - uri. The returned value is used in place of the original. The default link URI transformer acts as an XSS-filter, neutralizing things like javascript:, vbscript: and file: protocols. If you specify a custom function, this default filter won't be called, but you can access it as require('react-markdown').uriTransformer. If you want to disable the default transformer, pass null to this option.
escapeHtml - boolean Setting to false will cause HTML to be rendered (see notes below about proper HTML support). Be aware that setting this to false might cause security issues if the input is user-generated. Use at your own risk. (default: true).
skipHtml - boolean Setting to true will skip inlined and blocks of HTML (default: false).

Thank you for your answer. :)
But sorry, I don't understand your propose. 😞
First, escapeHtml  and skipHtml should be false ( default value ) for activating a tag
And, transformLinkUri seems to be something like middleware used to block specific protocols.
But I can't solve my problem just by understanding above it.


Support cast from varchar to day-time interval  type

From the SQL spec:
20) If TD is interval, then
    [...]
    b) If SD is character string, then SV is replaced by

        TRIM ( BOTH ' ' FROM VE )

    Case:
        i) If the rules for <literal> or for <unquoted interval string> in Subclause 5.3, “<literal>”, can be applied to SV to
           determine a valid value of the data type TD, then let TV be that value.
        ii) Otherwise, an exception condition is raised: data exception — invalid interval format.

and
<unquoted interval string> ::= [ <sign> ] { <year-month literal> | <day-time literal> }

<day-time literal> ::=
    <day-time interval>
  | <time interval>

<day-time interval> ::=
   <days value> [ <space> <hours value> [ <colon> <minutes value>  [ <colon> <seconds value> ] ] ]

<time interval> ::=
    <hours value> [ <colon> <minutes value> [ <colon> <seconds value> ] ]
  | <minutes value> [ <colon> <seconds value> ]
  | <seconds value>

(only the <day-time literal> part of <unquoted interval string> applies to day-time intervals.

Hey! Is this still open? I'm new to open-source, if you could give me a head-start, I'd like to submit a PR for this issue.

Yes, this is still open. Go for it!

@martint Thanks! Gimme a day to get a hang of this codebase 😊

@martint Hey, I was going through the code base, and if i understood the issue statement properly, I should be doing what the TODO is saying and use timeStampWriteFunction() instead of return WriteMapping.longMapping("datetime", timestampWriteFunctionUsingSqlTimestamp(TIMESTAMP_MILLIS)); here:

in the io.prestosql.plugin.mysql.MySqlClient.java class.
Please let me know if I'm on the right track, as I'm still new to huge code-bases like this one. 👍


Discord.NET Audio Join

Hi,
I would like to connect my Bot to the Vocal Lounge however, even following the documentation FindServers returns in all cases null. I tried with the name of it or its identify but same result.
I use the version 0.9.6 available on NuGet. The code is :
await discord.GetService().Join(discord.FindServers("Gamers Addict").FirstOrDefault().VoiceChannels.FirstOrDefault());
Thank you for your reply.

When are you calling this code? If it's directly after connecting to Discord, FindServers() will return null because not all server data has been received yet. Use await Task.Delay() to add a delay of 3 seconds in between connecting and calling that code and it shouldn't return null. If it still does, double check the name or try using GetServer() using your server's unique ID.
As a side note, these types of questions are better suited for the Discord.Net channel on the Discord API server. You'll get more immediate help there.

Hi,
The link to salon discord expired. I followed your advice about Delay, but it still does not work. The connection causes an error.
    Task.Run(async () =>
        {
            await discord.Connect("#", TokenType.Bot);

            await Task.Delay(TimeSpan.FromSeconds(4));
            try
            {
                Channel voiceChannel = discord.GetChannel(94889398826434560);
                //_vClient = await voiceChannel.JoinAudio();
                _vClient = await discord.GetService<AudioService>().Join(voiceChannel);
            }
            catch(Exception e)
            {
                Debug.Write(e + " Exception caught. [ALERT]");
            }
        });

The Exception is :

    System.DllNotFoundException: Unable to load DLL 'opus': The specified module could not be found. (Exception from HRESULT: 0x8007007E)

    at Discord.Audio.Opus.OpusConverter.UnsafeNativeMethods.CreateEncoder(Int32 Fs, Int32 channels, Int32 application, OpusError& error)
    at Discord.Audio.Opus.OpusEncoder..ctor(Int32 samplingRate, Int32 channels, Int32 frameLength, Nullable`1 bitrate, OpusApplication application)
    at Discord.Net.WebSockets.VoiceSocket..ctor(DiscordConfig config, AudioServiceConfig audioConfig, JsonSerializer serializer, Logger logger)
    at Discord.Audio.AudioClient..ctor(DiscordClient client, Server server, Int32 id)
    at Discord.Audio.AudioService.<Join>d__29.MoveNext()
    --- End of stack trace from previous location where exception was thrown ---
    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
    at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()

Thank you for your reply.

You need those audio libraries in the execution path of your program. Here's a link where you can download them, and here's a permanent link to the Discord channel. My bad, I should've made that permanent in the first place.

@MarkusGordathian
when i downloaded your file  i had the error:
System.BadImageFormatException: Poging om een programma te laden met een onjuiste indeling. (Uitzondering van HRESULT: 0x8007000B)    bij Discord.Audio.Opus.OpusConverter.UnsafeNativeMethods.CreateEncoder(Int32 Fs, Int32 channels, Int32 application, OpusError& error)    bij Discord.Audio.Opus.OpusEncoder..ctor(Int32 samplingRate, Int32 channels, Int32 frameLength, Nullable1 bitrate, OpusApplication application)
bij Discord.Net.WebSockets.VoiceSocket..ctor(DiscordConfig config, AudioServiceConfig audioConfig, JsonSerializer serializer, Logger logger)
bij Discord.Audio.AudioClient..ctor(DiscordClient client, Server server, Int32 id)
bij Discord.Audio.AudioService.d__29.MoveNext()`
is there something wrong with those files or do i need to do something else first?

That means you got the wrong version of 'libsodium' or 'libopus'

Audio is in a really weird place in a moment. many people are getting the same exception. I'm personally of the opinion that the issue is something with D.Net and/or fox needs to update the binaries. Not to mention that one of those binaries has incorrect checksums in its info.txt.

wrong version of 'libsodium' or 'libopus'

I tired every combination and still got the same thing. Stop propagating this "advice" when that s not the issue. =/

This is the first I'm hearing of such an issue, as well as the fact that one binary is screwy.

If anyone else comes across this.

So first check that you are really using the right binaries: 32 bit or 64 bit.
If you need 32 bit or 64 bit depends on
a) the architecture your operating system is for
b) your compile settings (CPU target. If it is AnyCpu be sure that you disable prefer 32 bit if you are on 64 bit)
c) The prefer 32 bit option
d) The dotnet runtime in your path. Check your PATH environment variable if it lists the 32 or 64 bit version of Dotnet first and change it if neccessary.
e) If you still have problems try targeting 64 or 32 bit in the CPU target.
f) If this does not fix your problem ask again.

Hi, I'm having issues with the libraries. It tells me that it wan't find the libsodium.dll when it IS in the run directory :/ I tried everything  @Obbi89 mentioned but nothing changes :/ Do you know where I can get a DLL that has been tested and works ?

Can this issue be locked with a notice saying what people should check? This thread has been kept alive for too long


Using HW Sync on D400 series with photoswitch to sync single shot capture

I have a D415 camera which I would like to sync a single frame capture with a photoelectric switch triggering (on a conveyor, tripped by an object).  How can I configure my D415 to listen on the HW sync pin header for the signal and to NOT take any frames (such that wait_for_frame() will not return with any data) until the signal indicates an object has arrived?
I want my image capture thread to block until a single frame is received, and that the frame exposure time is constant (with auto exposure turned off) so that I get very repeatable depth image captures of these objects on a conveyor..
I understand that there is new firmware available to enable the HW sync functionality on the cameras? my cameras are from the first public batch released in January 2018.
A secondary question to this is - is there a firmware update tool which runs on Linux? I looked recently (month ago?) and saw only windows based firmware update tools. I have cameras in embedded enclosures and it's not going to be super easy to get the cable out and plug into a windows laptop to perform a firmware update. The camera is normally connected to an embedded linux micro PC.

One approach that Intel took with a multi-camera capture demo this year was to "use a commercial grade flash unit to signal the start and end of a capture, allowing us to use that spike in data to make sure our camera footage was properly synchronized together".
I do not have any further technical details of how that trigger was set up.  One of the Intel guys on this GitHub can likely provide guidance on that.
In regard to your second question: yes, there is a Linux firmware updater tool.
https://www.intel.com/content/dam/support/us/en/documents/emerging-technologies/intel-realsense-technology/Linux-RealSense-D400-DFU-Guide.pdf

The multi-camera sync white paper says about sync triggering that whilst you can have a master and slave setup with the cameras, "Alternatively an external signal generator can also be used as the master trigger with all cameras set to slave mode.  When applying an external sync pulse, the HW SYNC input requires a 100 microsecond positive pulse at the nominal camera frame rate, 33.33 ms for a 30Hz frame rate for example".
"Inputs are high impedance, 1.8V CMOS voltage levels.  However, it is important to make sure to use a high resolution signal generator.  The frequency of the signal generator needs to exactly match the sensor frame rate.  For example, if the sensor is set up as 30 fps, the real frame rate may be 30.015 fps.  You may need to use an oscilloscope to measure the real frame and configure the signal generator to the same frequency.  For this reason, it may be better to just using one additional camera as master sync signal generator"

From the sounds of that, it's isn't what i'm looking for in terms of functionality :(
I am after a SINGLE frame, with a re-trigger rate of probably 1->0.5Hz (1 second or more, before triggers).  I'll read that white paper, thank you Marty

[Realsense Customer Engineering Team Comment]
Ticket being closed due to inactivity for 30+ days

@kyranf , were you able to get this to work? I am trying something similar, but the realsense doesn't seem to wait for my external signal before capturing an image. Any help is appreciated

@oafolabi  sorry, I haven't had the time to properly investigate this, and I was able to make my industrial part scanning just work off the leading edge detected in the depth data, referenced well enough for the robot with encoder tracking and the shared photo switch to do the job.
I think it's really important that they add this easy 'external frame trigger' for one-shot captures, so you don't have to worry about precise timestamps or the frame jitter if it's streaming.  In my situation I just wanted it to wait until asked to take a single snapshot with an external trigger input.


Linux安装文档的步骤增加lxml安装

在一个新的环境安装QUANTAXIS的时候，由于 QUANTAXIS 依赖了tushare，而tushare在pip安装之前需要提前安装lxml包，所以可能导致如下的报错，建议在文档的安装步骤上提示一下..
rainx@ubuntu170:~/dev/QUANTAXIS$ sudo pip3.6 install -e .
The directory '/home/rainx/.cache/pip/http' or its parent directory is not owned by the current user and the cache has been disabled. Please check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
The directory '/home/rainx/.cache/pip' or its parent directory is not owned by the current user and caching wheels has been disabled. check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
Obtaining file:///home/rainx/dev/QUANTAXIS
Collecting pandas>=0.20 (from quantaxis==0.3.9b2.dev53)
  Downloading http://pypi.doubanio.com/packages/fe/6f/5733658857dffb998afa2120027171c263384ada0487a969e5ecd5bf9ac9/pandas-0.20.3-cp36-cp36m-manylinux1_x86_64.whl (24.5MB)
    100% |████████████████████████████████| 24.5MB 6.3MB/s
Collecting numpy>=1.12.0 (from quantaxis==0.3.9b2.dev53)
  Downloading http://pypi.doubanio.com/packages/59/e2/57c1a6af4ff0ac095dd68b12bf07771813dbf401faf1b97f5fc0cb963647/numpy-1.13.1-cp36-cp36m-manylinux1_x86_64.whl (17.0MB)
    100% |████████████████████████████████| 17.0MB 11.5MB/s
Collecting tushare==0.7.4 (from quantaxis==0.3.9b2.dev53)
  Downloading http://pypi.doubanio.com/packages/e6/86/ad0b88b85119292a4503a43355091988c25a234d0e89511b782464c4ff46/tushare-0.7.4.tar.gz (89kB)
    100% |████████████████████████████████| 92kB 13.6MB/s
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-build-mbu_8l9t/tushare/setup.py", line 4, in <module>
        import tushare
      File "/tmp/pip-build-mbu_8l9t/tushare/tushare/__init__.py", line 6, in <module>
        from tushare.stock.trading import (get_hist_data, get_tick_data,
      File "/tmp/pip-build-mbu_8l9t/tushare/tushare/stock/trading.py", line 13, in <module>
        import lxml.html
    ModuleNotFoundError: No module named 'lxml'

还有pandas
Collecting tushare==0.7.4 (from quantaxis==0.3.9b2.dev53)
  Downloading http://pypi.doubanio.com/packages/e6/86/ad0b88b85119292a4503a43355091988c25a234d0e89511b782464c4ff46/tushare-0.7.4.tar.gz (89kB)
    100% |████████████████████████████████| 92kB 14.8MB/s
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-build-68tu_t9z/tushare/setup.py", line 4, in <module>
        import tushare
      File "/tmp/pip-build-68tu_t9z/tushare/tushare/__init__.py", line 6, in <module>
        from tushare.stock.trading import (get_hist_data, get_tick_data,
      File "/tmp/pip-build-68tu_t9z/tushare/tushare/stock/trading.py", line 15, in <module>
        import pandas as pd
    ModuleNotFoundError: No module named 'pandas'

晕，还有requests
Collecting tushare==0.7.4 (from quantaxis==0.3.9b2.dev53)
  Downloading http://pypi.doubanio.com/packages/e6/86/ad0b88b85119292a4503a43355091988c25a234d0e89511b782464c4ff46/tushare-0.7.4.tar.gz (89kB)
    100% |████████████████████████████████| 92kB 9.6MB/s
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-build-zlte071g/tushare/setup.py", line 4, in <module>
        import tushare
      File "/tmp/pip-build-zlte071g/tushare/tushare/__init__.py", line 99, in <module>
        from tushare.trader.trader import TraderAPI
      File "/tmp/pip-build-zlte071g/tushare/tushare/trader/trader.py", line 12, in <module>
        import requests
    ModuleNotFoundError: No module named 'requests'

是的 tushare坑死我了

弄得我travis ci至今是failed

我好想pypi上的quantaxis没更新 得从github上下
pip install git+https://github.com/yutiansut/quantaxis

或者 这样  https://github.com/yutiansut/QUANTAXIS#部署问题


[WFLY-11145]: multicastPrefix and anycastPrefix cannot be set

Adding extra parameters on connectors and acceptors when constructing
TransportConfigurations.
Jira: https://issues.jboss.org/browse/WFLY-11145

I don't fully agree with you @jmesnil : we don't support those parameters as parameters but as extra parameters and as such we currently don't "list" them, they could be anything as there is no defined list (like for connectors / acceptors) from Artemis.

@ehsavoie fair enough, these extraParameters are "get-out-of-jail" parameters that could be anything...
However from WildFly perspective, they are passed to the params attribute of the connector so we end up with config using 2 different naming conventions:
 <http-connector name="http-connector-throughput"
                            socket-binding="http"
                            endpoint="http-acceptor-throughput">
   <param name="batch-delay" value="50"/>
   <param name="multicastPrefix" value="jms.topic"/>
</http-connector>

This is inconsistent and will be difficult to document and maintain.
Maybe we could offer conversion for some well-known and documented parameters such as multicast-prefixand unicast-prefix.
If we can not find such a mapping, so be it, we pass the names as they are to Artemis.
wdyt?

retest this please

retest this please


List of supported document tags

Hello,
I just had sharpDox create some pretty slick docs for a new project I'm working on, but I'm wondering if there is a list of supported XML comment tags anywhere. I noticed a few tags either didn't get pulled in when generating the docs or they didn't get exported when generating the HTML.
Microsoft recommends using these: http://msdn.microsoft.com/en-us/library/5ast78ax.aspx
It would also be awesome to support the <inheritdoc /> tag that other doc gen applications use.
PS, awesome project so far!

Fixed in the latest release.

Doesnt work for me, throws error in DocumentationParser line 36.

Could you give me an example how you are using the tag? Thank you!

example
using System;

namespace console03.Test1
{
	/// <summary>
	/// Description of ITest1.
	/// </summary>
	public interface ITest1
	{
		/// <summary>
		/// Description of Method1
		/// </summary>
		void Method1();
	}
}

using System;

namespace console03.Test1.Impl
{
	/// <inheritdoc />
	public class Test1 : ITest1
	{
		#region ITest1 implementation

		/// <inheritdoc />
		public void Method1()
		{
			throw new NotImplementedException();
		}

		#endregion
	}
}

example project

Thank you! The problem seems to be, that the  is just allowed on member level at the moment. If you remove the tag from your class, it should work. Will look into this problem.

I removed tags from classes, now it works fine. Thanks.


Workflow reports UI tweaks

Small tweaks to the workflow reports UI, as discussed in standup this morning.
See commit log for details.

Thanks for the quick review @zerolab!
The path can't be a sub-path of /workflows because this will make both menu items display as active when someone is looking at workflow tasks.

The path can't be a sub-path of /workflow because this will make both menu items display as active when someone is looking at workflow tasks.

Ah, I knew I was missing something obvious! :shipit:


feat(cli): add user and group commands to create users and groups

This PR:

Introduces two new commands user and group and starts the pattern for creating new commands in the future like dataset to work with entities in the metadata model.
Creates a simple yaml based format for specifying users and groups.

Companion docs PR: #7574
Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

I do feel a bit uneasy with this much code going in with no tests

Added smoke tests for the user and group commands

@anaghshineh for deletes, you can use the datahub delete command!


Breaks apps on iOS 12

The library seems to prevent apps from running on iOS 12 (and possibly below) except when being run via Xcode.
Nether TestFlight nor Ad-Hoc build works when installed to a iOS 12 device. However, the same build works on a iOS 13 device.
I tested this with a newly created blank project only adding SwiftyUserDefaults via CocoaPods. Removing the pod makes the app work again.

@ultimate-deej so you are able to run the app on iOS 12 but only when built with Xcode?

Yes it only works when built and run with Xcode.
It just doesn't start, closing as soon as it opens. No errors or anything.

@ultimate-deej So the TestFlight or Ad-Hoc builds are built by a CI? I'm trying to figure out how to quickly reproduce the issue. Can you give me your environment details? (e.g. macOS version, Xcode version)

I didn't use a CI. The first couple of builds were built with fastlane, then I switched to manual export from Xcode Organizer.
My environment is macOS 10.14, Xcode 11, CocoaPods 1.8.0. The library versions I tried were 5.0.0-beta.2 and 5.0.0-beta.4.

@ultimate-deej I was able to confirm that archiving with CocoaPods indeed links it strongly, not weakly. Still investigating the issue.

Same here, crashed as well. Make build on apple enterprise account with CI. iOS 13 working fine. Just having crash on 12.4.1

@nthtrung09it @thihaaung6245 it seems like there might be a problem with Combine framework itself. We have the same problem in Moya (also Kingfisher). You can take a look at this mega-thread for more details, but I will be removing the Combine support because of that in both Moya and Swifty and maybe reintroduce it later on. Sorry for the problem.

Thanks @sunshinejr , just let you know. Happy to help.

But, @sunshinejr , one thing weird is, when I tried to install in debug mode with my iOS 12.4.1 devices with the same 5.0.0-beta.4, nothing happened. It's only happening at distribution mode. Which still weird. Now, I am downgrading back to 4.0.0 and see it's work or not.

@thihaaung6245 yeah exactly that's the problem. In the thread I described what we can do to avoid it, but it's really not that easy and we would break compatibility by doing so. Unfortunately it seems like if you want to use a framework that uses Combine and your min. target is lower than 13, then it's a big risk.

@sunshinejr Yeah, looks like need to downgrade for a while. Appreciate the support. Cheers.

@thihaaung6245 yup good idea. New beta version without Combine should be up soon, in the meantime use 4.* or just don't make release builds and you can use 5 betas.


strict types is ignored when mapping or filtering arrays due to the usage of PHP builtin functions

This bug was introduced in #169 by mistake, the solution would be to wrap all callbacks provided by the end user in another closure to ensure that strict types is used.
Environment (please complete the following information):

OS: *
PHP version: 7.4, 8.0
Version: 1.6, 1.7

Additional context
see https://github.com/Roave/BackwardCompatibilityCheck/pull/306#discussion_r635004153

we have to benchmark which solution is better:

wrap callbacks in a closure
re-implement array_* functions in userland.

re-implement array_* functions in userland.

I think we'll likely need to suck it up and keep using array_ functions here: assuming 100% type coverage on sources, we can assume that the given values are within the type boundaries of the provided callbacks.
The array_* functions are massively outperforming any iterative approach, as they can operate at zval level.

fixed and released as 1.6.1 and 1.7.1


Altair --> View Compiled Vega

We have written the needed data into your cIn general, I'm vey satisfied with Jupyter Notebook for VS Code Extension.
What I found as a bug (I guess) is that in working with Altair module / Python, I'm getting empty file when do Chart --> [...] --> View Compiled Vega.
Expected behavior: Open file that contains JSON schema for the Altair chart.
Jupyter Notebook opened with conventional way (Anaconda terminal --> Jupyter Lab) handles the task (it opens a new tab in browser with JSON schema of Altair chart)lipboard because it was too large to send. Please paste.

sorry, it was intended for Jupyter Lab Extension for VS Code


Fix #1057 missing zosversion in zos.json

Fixes #1057
According to https://github.com/zeppelinos/zos/issues/1008#issuecomment-503707380 the entry zosversion will be renamed to manifestVersion. But as long as we go with zos 2.4, this PR is required to  make this example work.

As mentioned in https://github.com/zeppelinos/zos/pull/1058:
We are in the middle of a renaming migration in the master branch (See https://github.com/zeppelinos/zos/issues/925 and this). I would suggest you to work with the same examples but in the release/2.4 branch, which does not have these naming changes.

Closing as this was fixed on a different PR


Nuxt does not work when generated with nuxtServerInit in the Vuex Store

Version
v2.11.0
Steps to reproduce
With a serverNuxtInit in the Vuex Store, nuxt does not generate the application
export const actions = {
  async nuxtServerInit({ dispatch }) {
    await dispatch('captions/get')
  }
}

ERROR   /                                                                                17:57:31

Error: Request failed with status code 401
    at createError (/Users/fran/Projects/xxxx/node_modules/axios/lib/core/createError.js:16:15)
    at settle (/Users/fran/Projects/xxxx/node_modules/axios/lib/core/settle.js:17:12)
    at IncomingMessage.handleStreamEnd (/Users/fran/Projects/xxxx/node_modules/axios/lib/adapters/http.js:236:11)
    at IncomingMessage.emit (events.js:333:22)
    at IncomingMessage.EventEmitter.emit (domain.js:485:12)
    at endReadableNT (_stream_readable.js:1201:12)
    at processTicksAndRejections (internal/process/task_queues.js:84:21)

ERROR   /tracking                                                                                17:57:31

Error: Request failed with status code 401
    at createError (/Users/fran/Projects/xxxx/node_modules/axios/lib/core/createError.js:16:15)
    at settle (/Users/fran/Projects/xxxx/node_modules/axios/lib/core/settle.js:17:12)
    at IncomingMessage.handleStreamEnd (/Users/fran/Projects/xxxx/node_modules/axios/lib/adapters/http.js:236:11)
    at IncomingMessage.emit (events.js:333:22)
    at IncomingMessage.EventEmitter.emit (domain.js:485:12)
    at endReadableNT (_stream_readable.js:1201:12)
    at processTicksAndRejections (internal/process/task_queues.js:84:21)

What is expected ?
I should compile with the nuxtServerInit
What is actually happening?
Currently not compiling, showing the errors that I have put in the previous comment

This bug report is available on Nuxt community (#c10322)

The HTTP request you are performing inside captions/get is returning a 401 status code and an exception is raised by Axios. You are most likely missing the authentication method.

The HTTP request you are performing inside captions/get is returning a 401 status code and an exception is raised by Axios. You are most likely missing the authentication method.

For authentication I am using @nuxtjs/auth

You are passing wrong username/password or api token

You are passing wrong username/password or api token

He just answered me when I was editing the other message.
Thank you for helping me realize my mistake. Apologies


Signature uniqueness checking not precise enough for decoding events

I can't create a wildcard event listener on both the below two events, on the same topic because the signature is the same per ethereum standard semantics Transfer(address,address,uint256)...
...but the encoding into the event is not the same because in one the 3rd parameter is indexed: true.
ERC-20 transfer
    {
        "anonymous": false,
        "inputs": [
            {
                "indexed": true,
                "name": "from",
                "type": "address"
            },
            {
                "indexed": true,
                "name": "to",
                "type": "address"
            },
            {
                "indexed": false,
                "name": "value",
                "type": "uint256"
            }
        ],
        "name": "Transfer",
        "type": "event"
    }

ERC-721 transfer
    {
        "anonymous": false,
        "inputs": [
            {
                "indexed": true,
                "internalType": "address",
                "name": "from",
                "type": "address"
            },
            {
                "indexed": true,
                "internalType": "address",
                "name": "to",
                "type": "address"
            },
            {
                "indexed": true,
                "internalType": "uint256",
                "name": "tokenId",
                "type": "uint256"
            }
        ],
        "name": "Transfer",
        "type": "event"
    }

@peterbroadhurst I fixed this one as part of the multiple listeners, closing


Fail to download prebuilt module 5.4.0-89-generic_100.ko

Describe the bug
I wanted to install Falco as a Helm Chart on my civo.com K3S Cluster, but after installing the Falco Pods panic because they are not able to download the generic module for the Kernel, as the download source is currently not available.
How to reproduce it
Spin up a Kubernetes / K3S Cluster with the "latest" Version on civo.com.
Then install Falco as a Helm Chart with the latest Application Version (0.30.0)
helm repo add falcosecurity https://falcosecurity.github.io/charts
helm repo update
helm upgrade falco --install --namespace falco --create-namespace falcosecurity/falco

# Falco pods will fail now immediatley
kubectl -n falco get pods
NAME          READY   STATUS   RESTARTS   AGE
falco-xljwb   0/1     Error    1          8s
falco-ktjks   0/1     Error    1          8s
falco-qdcnh   0/1     Error    1          8s

# Get events
kubectl -n falco describe pod falco-xljwb
[...]
Events:
  Normal   Pulled     14s (x3 over 43s)  kubelet            Container image "docker.io/falcosecurity/falco:0.30.0" already present on machine
  Normal   Created    14s (x3 over 42s)  kubelet            Created container falco
  Normal   Started    14s (x3 over 42s)  kubelet            Started container falco
  Warning  BackOff    12s (x4 over 37s)  kubelet            Back-off restarting failed container
[...]

kubectl -n falco logs -f falco-xljwb
* Setting up /usr/src links from host
* Running falco-driver-loader for: falco version=0.30.0, driver version=3aa7a83bf7b9e6229a3824e3fd1f4452d1e95cb4
* Running falco-driver-loader with: driver=module, compile=yes, download=yes
* Unloading falco module, if present
* Trying to load a system falco module, if present
* Looking for a falco module locally (kernel 5.4.0-89-generic)
* Trying to download a prebuilt falco module from https://download.falco.org/driver/3aa7a83bf7b9e6229a3824e3fd1f4452d1e95cb4/falco_ubuntu-generic_5.4.0-89-generic_100.ko
curl: (22) The requested URL returned error: 404
Unable to find a prebuilt falco module
* Trying to dkms install falco module with GCC /usr/bin/gcc
DIRECTIVE: MAKE="'/tmp/falco-dkms-make'"
* Running dkms build failed, couldn't find /var/lib/dkms/falco/3aa7a83bf7b9e6229a3824e3fd1f4452d1e95cb4/build/make.log (with GCC /usr/bin/gcc)
* Trying to dkms install falco module with GCC /usr/bin/gcc-8
DIRECTIVE: MAKE="'/tmp/falco-dkms-make'"
* Running dkms build failed, couldn't find /var/lib/dkms/falco/3aa7a83bf7b9e6229a3824e3fd1f4452d1e95cb4/build/make.log (with GCC /usr/bin/gcc-8)
* Trying to dkms install falco module with GCC /usr/bin/gcc-6
DIRECTIVE: MAKE="'/tmp/falco-dkms-make'"
* Running dkms build failed, couldn't find /var/lib/dkms/falco/3aa7a83bf7b9e6229a3824e3fd1f4452d1e95cb4/build/make.log (with GCC /usr/bin/gcc-6)
* Trying to dkms install falco module with GCC /usr/bin/gcc-5
DIRECTIVE: MAKE="'/tmp/falco-dkms-make'"
* Running dkms build failed, couldn't find /var/lib/dkms/falco/3aa7a83bf7b9e6229a3824e3fd1f4452d1e95cb4/build/make.log (with GCC /usr/bin/gcc-5)
Consider compiling your own falco driver and loading it or getting in touch with the Falco community
Tue Nov  2 12:29:22 2021: Falco version 0.30.0 (driver version 3aa7a83bf7b9e6229a3824e3fd1f4452d1e95cb4)
Tue Nov  2 12:29:22 2021: Falco initialized with configuration file /etc/falco/falco.yaml
Tue Nov  2 12:29:22 2021: Loading rules from file /etc/falco/falco_rules.yaml:
Tue Nov  2 12:29:23 2021: Loading rules from file /etc/falco/falco_rules.local.yaml:
Tue Nov  2 12:29:23 2021: Unable to load the driver.
Tue Nov  2 12:29:23 2021: Runtime error: error opening device /host/dev/falco0. Make sure you have root credentials and that the falco module is loaded.. Exiting.

curl -IL https://download.falco.org/driver/3aa7a83bf7b9e6229a3824e3fd1f4452d1e95cb4/falco_ubuntu-generic_5.4.0-89-generic_100.ko
HTTP/2 404
content-type: application/xml
date: Tue, 02 Nov 2021 12:30:19 GMT
server: AmazonS3
x-cache: Error from cloudfront
via: 1.1 29f7132906866b79866659848b3a3b69.cloudfront.net (CloudFront)
x-amz-cf-pop: FRA56-P3
x-amz-cf-id: DJxtoQF-AeXs3j4euhRm8TF2GjUy2MaF5Yuq9u2Vd9sMZAMXCVAqSw==
age: 5

Expected behaviour
Falco should just either download the fitting module or compile the right one by itself.
Environment
K8S/K3S VERSION: v1.20.2+k3s1
OS-IMAGE: Ubuntu 20.04.3 LTS
KERNEL-VERSION: 5.4.0-89-generic
CONTAINER-RUNTIME: containerd://1.4.3-k3s1

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale


Broken link

Hi, the link to Standard Encoder formats and codecs is broken.

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

ID: 0217f580-4696-c67a-b3cb-6df8040236d1
Version Independent ID: 269cc4d1-fe05-e3bd-f0dd-b4850e131be8
Content: Upload and index videos with Azure Video Indexer (formerly Azure Video Analyzer for Media)
Content Source: articles/azure-video-indexer/upload-index-videos.md
Service: azure-video-indexer
GitHub Login: @Juliako
Microsoft Alias: Juliako

@Juliako Could you please help fix the broken link in the documentation?

@RohitMungi-MSFT , working on it

The link is fixed, #please-close


Is it possible add callback function after releasing on edge for the mousewheel?

Hi
I set the mousewheel parameter value to {releaseOnEdges: true}, then I wonder if any method to add the callback function after releasing on edge? For example, I want to change the section css styles after releasing on edge, but I haven't found the method that where I can add the callback in the documentation.
Thanks

Nevermind, I found reachBeginning, reachEnd method.
Just close the ticket.


parse/parse_serialize needs explicit encoding metadata

This eval should print US-ASCII and not UTF-8:
#coding: US-ASCII

p eval("__ENCODING__")

parse_serialize needs explicit encoding metadata added so we can pass in that the bytes are US-ASCII.

It's stored in the Loader: https://github.com/ruby/prism/blob/3b70a9dfceb86d66dc46ba355fe8ec4e9c1e39a6/templates/java/org/prism/Loader.java.erb#L82
If you subclass it, e.g. to implement bytesToName() then you can access it there.
And it's already part of the serialization format: https://github.com/ruby/prism/blob/main/docs/serialization.md#structure

Ah but the field is private, my bad, we could change it to protected of course, could you make a PR if that's enough for what you need?

This issue is that the eval parse must pass in that its encoding is explicitly US-ASCII. I do not see a mechanism for that. I think we can just pass in another section to metadata for that.

Right that makes sense, it's indeed missing from https://github.com/ruby/prism/blob/main/docs/serialization.md#apis.
The issue description was rather sparse, it would be nice to update it with that.

@eregon I tried rearranging that sentence to hopefully be more clear.


chore(ui): Import consistently from lodash

Description
Recent work to delete unused dependencies reminded me of an inconsistency that the team had discussed, and then forgotten. I became curious whether rewrite named imports as default imports from lodash subfolder affects either bundle size or yarn.lock file.

In ui/apps/platform folder:find src \( -name '*.js' -o -name '*.ts' -o -name '*.tsx' \) -exec fgrep -h "'lodash" \{\} >> somewhere/lodash.txt \;

Find in File 'lodash' 6 named out of 150 total

import { isEmpty } from 'lodash';
import { debounce } from 'lodash';
import { uniq } from 'lodash';
import { uniq } from 'lodash';
import { uniqBy } from 'lodash';
import { sortBy } from 'lodash';

Although I do not have a preference between named import from 'lodash' versus default import from 'lodash/whatever' a small number of exceptions to a pattern sometimes defeats my old school style to search the code.
Checklist

[ ] Investigated and inspected CI test results
[x] ~Unit test and regression tests added~

Testing Performed
TL;DR You don’t know unless you measure.

yarn lint in ui
yarn in ui
yarn build in ui and then compute branch - master for the following

wc build/static/js/*.js
main.js -108 = 4231761 - 4231869
total -76834 = 10235754 - 10312588
ls -al build/static/js/*.js | wc
files -1 = 79 - 80

yarn start in ui

Manual testing

Visit /main/dashboard and while there analyze page with aXe DevTools to assess accessibility (pardon pun)

Continue to visit every link in left navigation and ditto

Nice and thorough. Was curious if you either missed an import in /ui/apps/platform/cypress/helpers/scopeSelectors.js or just decided it could be an exception to the rule?
The import in question:
import { isString, isPlainObject, isFunction, mapValues } from 'lodash';

Oh, good eyes! I forgot to search in the cypress folder.


DOC: add a fractal example

🤦🏻 sorry for pushing to upstream, forgot to fork and GH's automatic forwarding kept my remote working.

might be fun to scale the number of iterations inversely proportional to area of the axes window
that would make you able to zoom in even further with better resolution (at the cost of draw time going up)


[BUG] Major wounds and damage calculator — "The Mystery of 1"

The Mystery of 1
Hi there, creating this issue by the Nick Coffin, PI#8616's request.
Originally I've noticed some fudging in Damage Calculator's work — DC(Damage Calculator) made you roll twice for major wound to an eye, both without any modifiers. While in reality victim must take a single roll with -10 for the eye injury.
By exploring more, I've noticed a pattern. Here's what I've found out:
1) I've experienced the same issue with Eye, Face and Vitals hit locations.
2) Under certain circumstances, DC calculated Major Wound modifiers Properly.
3) Originally I was using [1d imp] OTF and the issue persisted IF I was applying 1 imp damage. But then, I've tried the same thing with [2d imp] OTF and the issue mostly persisted if I rolled 1 on both dice, giving a net of 2 imp damage. So my guess is that the cause lies near the number 1 — if the result is the lowest of all possible, then it happens. At least I know that you may try to simulate the issue with the result of 1 imp on [1d imp] OTF.
4) In my case, the issue persisted only if all the dice I've rolled gave the lowest possible result. But interesting enough, I was actually able to simulate the issue with the result 2 on the die on [1d-1 imp] ← this formula. After calculations I've got only 1 damage to apply and the issue took it's place.

Tests
So here is some tests.
Damage was dealt to the Eye, Face and Vitals hit locations and the DR was 0 in all cases, thus everything had penetrated DR. The results for major as follows:
1 die

Eye → No modifier, two rolls instead of one.
Face → No modifier, one roll.
Vitals → No modifier, one roll.

2 dice

Eye → Modifier -10 for both rolls, two rolls instead of one.
Face → No modifier, one roll.
Vitals → No modifier, one roll.

1 die of imp damage, 1 imp damage from this die
Here is the photos of DC calculations

2 dice of imp damage, 1 imp damage from each dice for a net of 2
Here is the photos of DC's calculations:

Here is the Link to a Discord message report on GURPS Discord server


Idea: custom data attributes on components

Would you be open to adding the ability to set custom data attributes on components? We're using data-test attributes for Cypress tests, and it would be great to be able to set them on formbuilder components. I would be happy to write a PR if that helps.

Yeah, this is definitely something that should be supported consistently across the helpers. Some already support this behaviour by adding any extra keyword args as HTML attributes, but a couple of the mimic Rails' built-in html_attributes.

I had a quick play with this and think the best route is to create a Traits::HTMLAttribute module that can be dropped in to any of the classes that would support this behaviour, which will probably be most (or all) of the Elements.
There's a slight inconsistency because #govuk_select mimics Rails' html_options arg instead of just using the keyword args for HTML attributes. I guess at this point the key choice is on the direction we take, add a html_options arg to the other helpers or allow #govuk_select to use the keyword args (and eventually deprecate html_options).

I suspect keyword args will lead to lower maintenance burden in the long run


corrects minor mistakes in names

Changes committed:
modified:   config/config.go
Signed-off-by: Abhishek Kashyap<EMAIL_ADDRESS>

Codecov Report

Merging #27 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #27   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines          36     36
=====================================
  Hits           36     36

Impacted Files
Coverage Δ

config/config.go
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


PHP Icon Very Small

Maybe it is just me, but the PHP icon looks much smaller than the rest. Could we increase it's size?

Hi @TheColorRed, it's not jus you. I have to say that I took the icons from Sublime Text editor so I guess they somehow decided that this was the correct size for php files... and honestly, I don't know really why. 😁
I'll try to fix this and I'll let you know once it's done! 👍

I've uploaded a new icons.zip file with a new php icon. You will have to update the extension. 😄


refactored instanceof to .isKind()

Depends on https://github.com/wapc/widl-js/pull/6 which also includes details of why this refactor is important.

This LGTM but perhaps we should add the github build/release to wapc/widl-js so you can get a semver for this PR.


Deploy English Auction Frontend to gh pages

We need to deploy the frontend for users to view.  The corresponding contracts should be deployed to the latest testnet.

https://github.com/FuelLabs/sway-applications/issues/71 needs to be merged first


1.21.2-rc1 is incompatible

Hi there! Although 1.21.2 isn't fully released now, I figured that I'd make an issue regardless. When using Server Translations API (2.3.1+1.21-pre2), the server crashes with the error:
Receiver class xyz.nucleoid.server.translations.impl.ServerTranslations does not define or inherit an implementation of the resolved method 'abstract java.util.concurrent.CompletableFuture reload(net.minecraft.resource.ResourceReloader$Synchronizer, net.minecraft.resource.ResourceManager, java.util.concurrent.Executor, java.util.concurrent.Executor)' of interface net.minecraft.resource.ResourceReloader.

Fixed with 2.4.0+1.21.2-rc1!


Does not add using "vapor xcode"

tried: .Package(url: "https://github.com/vapor/mustache-provider.git", majorVersion: 0, minor: 10)
the documentation seems out of date after move to the Vapor account.
The package.swift file refers to a Vapor version: 0.14.0 that does not exist - perhaps 1.4 is what would be better.

Mustache is no longer supported since we have https://github.com/vapor/leaf.