Unable to flash LSC spiral filament bulb

Hello :)
About 2 years ago, I successfully flashed 3 of these filament bulbs. I used a Pi 3 B for that.
Today, I tried to flash one of these but was not successful. This time, I used a raspberry pi 4, removed the network info from wpa_supplicant.conf and put my phone and a computer in the vtrust network. After turning on an off the bulb 6 times in a row, it started blinking quicker then when the wifi setup is active. The bulb only blinks 12 times and is turned on afterwards. The ./start_flash.sh had the dots running over and over again without finding the device.
Since this bulb is not on the list, I'm not even sure if it has the ESP. And how about software patches by tuya? Does this work with newer devices at all?
Best
Jan

https://github.com/ct-Open-Source/tuya-convert/issues/999#issuecomment-968182311
Same story.


Add examples into GP 101

Add non-mathy examples into "What is a GP?"  Not for version 0.1.0

Added examples and pushed update.  Open to feedback.

Nice, that looks good to me.

in the posynomials, should it maybe be 0.56 instead of .56?
Should it be mentioned that all monomials are also posynomials?

0.56 should be updated now.
Yea, I can do that.
On Tue, Jan 27, 2015 at 5:26 PM, bqpd<EMAIL_ADDRESS>wrote:

Nice, that looks good to me.

in the posynomials, should it maybe be 0.56 instead of .56?
Should it be mentioned that all monomials are also posynomials?

—
Reply to this email directly or view it on GitHub
https://github.com/convexopt/gpkit/issues/150#issuecomment-71741332.

Added a one sentence description, should be up shortly.  @whoburg any more thoughts before we close the issue?

Looks great. I made a couple minor tweaks in f2af087957f188d7c5f12a9816852629c5cb4236. Closing the issue, but feel free to continue tweaking/improving.


Listen to Pump.fun or Photon

Anyway to change the bot to look at pump.fun or photon? I'm seeing a lot more coins with LP locked/burned then on raydium.

Pump fun doesn't use marketplaces, solana-trading-bot listens for major solana market places only.


Store whole forwardable_headers in Task

Issue https://github.com/RedHatInsights/topological_inventory-persister/issues/58
Insights::API::Common::Request.current_forwardable is saved in Task instead of x-rh-insights-request-id. To be able to resend it through Kafka in persister

dependent https://github.com/RedHatInsights/topological_inventory-persister/pull/59
dependent https://github.com/RedHatInsights/topological_inventory-api/pull/302

Gem version: 1.1.4

[ ] released to RubyGems

@lindgrenj6 please review


fix: cannot input new line using custom input control

For https://github.com/flutter/flutter/issues/125875 and https://github.com/flutter/engine/pull/45522
According to this discussion, Just added forceMultiline to JSON on _configurationToJson. @LongCatIsLooong @Renzo-Olivares

@Satsrag Happy new year! It appears that the PR is failing the google testing check, it's likely because the commits are a bit behind. Could you rebase the branch to upstream/main and squash the commits to rerun that test?

Thanks for all of this work. Happy new year!


vscode ipython text not shown in terminal (remote ssh)

VSCode Version: 1.52.0-insider (user-setup)
Version: 1.52.0-insider (user setup)
Commit: bc13785d3dd99b4b0e9da9aed17bb79809a50804
Date: 2020-12-07T14:46:48.305Z
Electron: 9.3.5
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.17763
OS Version: Windows 10 Enterprise (1809)

Steps to Reproduce:

Open remote ssh session - host Ubuntu 18.04
Open terminal and type ipython and Enter.
The version information text will not show. (image 1) If the text is shown, Press Ctrl-D or type exit()
Type ipython and Enter again.
If you close the terminal view and open again. Those text will show now. (image 2)

image 1:

image 2:

Does this issue occur when all extensions are disabled?: Yes
Remote SSH extension is needed. So, this can't be tested with extensions disabled.

can you try changing the renderer type? or zoom in and out?

This looks like an issue with our old terminal backend "winpty", going forward we are focusing our efforts on the new backend "conpty" which is build by the Console team at Microsoft and should fix many issues with the terminal under Windows. You can read more about getting ConPTY here.
/duplicate https://github.com/Microsoft/vscode/issues/45693

@meganrogge Tried ConPTY by setting the suggested parameter in settings still have issues with terminal. Only appeared when updated to 1.52.1
Are there any other suggestions on how to remedy?


Support for Full text search

It is known that Dexie does not support full text search as they do not have much use for index.
However there should a configuration that if enabled would allow the same, it can be very easily be achieved by using filter function (behind a config flag) along with StartsWithIgnoreCase used for search

I have started working on a major update that would also fix this problem, but since Microsoft released its own clipboard manager for Windows 10 and there are more mature alternatives for MacOS I haven't really used Olden, therefore, I'm not sure there is still a necessity for it.
If you want to implement this feature, I'll be more than happy to merge that change, but I probably won't be working on this project any longer.


[socks] Add support for SOCKs5 proxies

Initial commit adds support for:

proxying outgoing peer connections
Proxying HTTP(s) tracker connections

UDP tracker support is currently not proxied.
Incoming connections are not proxyable.

Migrated here: https://github.com/alanmcgovern/monotorrent.socks5/releases/tag/release-v0.1.0


Ideal Image Size for IRIS detection of SINGLE-IMAGE DEPTH ESTIMATION on desktop CPU version

Hi, is there any ideal image size for IRIS detection of SINGLE-IMAGE DEPTH ESTIMATION on desktop CPU version?

Hi @Gul11, Could you please elaborate your use case with complete details. Thanks!


units::quantity :

Use Concept names rather than class-template names for units::Quantity operator function signatures.

The resulting function signature is shorter and easier to read and understand.
The functions are not restricted to quantity<...> type, but can be reused by other models of Quantity.

As long as I agree with making math.h functions more generic I do not think changing operators in quantity.h to a generic concept is a good idea. I believe that arithmetic and comparison operators should be provided by every type separately for itself anyway (there should not be one generic implementation that cover all the libraries out there). Moreover, it will probably not work if we define those as Hidden Friends as it will be found only via ADL and it is highly probable that the Committee will ask us to do so (more info in the #42). Actually, you reminded me to push the other issue a bit....

So to addrees the friend issue
in this push I have used a non template base class for quantity and put the relevant functions as friend functions in there which solves https://github.com/mpusz/units/issues/42 quite nicely, so lookup is now using the friend method.
To address the only on template type issue
I made a implementation concept ll_quantity_concept (wharever) which applies only to quantity and used that then to define the friend functions, so not part of the interface now, but still nice and terse.  You could use is_base_of to constrain more tightly maybe

This actually does not change anything as the Quantity concept is defined as:
https://github.com/mpusz/units/blob/5dd9eaac87a50adc62b170987ebac2d1344d63c5/src/include/units/concepts.h#L218-L232
so the definition is exactly the same. Actually, I thought that is your idea for the extension as everyone can partially specialize this type trait for his/her own quantity class and those functions will just start to work. Now I am a bit puzzled what was your intention with using a Quantity concepts instead of the quantity class template?

I dont really understand why the operations need to be restricted to one class template. If that is the case, what is the rationale of Quantity concept at all? ( Hint : The answer is "less typing, which increases expressiveness". I follow that idea through in the operator function definitions)
Beyond that Ideally the library should be 'lifted' so that is can work with many types ( The issue then is ultimately you must output a type, but you your inputs only have to conform to the concepts, not a particular type so you have gained much freedom. A simple example would be a transpose view of a matrix just takes a ref to the matrix and outputs the elemnst in a different order),
... but I have done as you wished in restricting to one class template. I said previously that you could further restrict using is_base_of, so in latest pull I changed the ll_quantity_concept definition to use is_base_of so now only units::quantity is a model. The beauty of that is it is possible with one line, so that is one advantage of this approach over using explicit class-templates, just like using a typedef for a long winded type.
"The aim of generic programming is to generalize a useful algorithm or data structure to its most general and useful form."  - Bjarne Stroustrup.

I dont really understand why the operations need to be restricted to one class template

It depends which operations. As I said math.h is fine as it might work with everything. However, operators should be closely related to a specific type (i.e. other quantity type might have one additional member but still pass Quantity concepts).

what is the rationale of Quantity concept at all? ( Hint : The answer is "less typing, which increases expressiveness"

No, it is not about less typing. For example, how would you specify a function that returns a quantity of unspecified dimension, unit, or rep?
Quantity make_quantity(args...);

It is not possible with a class template, right? Also, how would you constrain a variable to be of quantity type having only a class template?
Quantity auto q = make_quantity(...);

Concepts are not at all about less typing.

Beyond that Ideally the library should be 'lifted' so that is can work with many types

I do not believe so. This library or a C++ Standard will have one concrete quantity type and we should stick to it in this library. We can think about creating a new library of library-independent tools like math or random (or keep such tools here for now) but it is about generic tools using quantities and not about operators which as I said should be closely related to a concrete type.

you could further restrict using is_base_of

It is not possible as we never inherit from a quantity in this library (it would lose constructors and many operators).

I changed the ll_quantity_concept definition to use is_base_of so now only units::quantity is a model

Not true. If in your library you will inherit your quantity from this base class it will also match those operators.
"KISS" :-D

To me this is a viable and elegant solution given the requirements, and the objections presented seem to be merely stylistic - class_template v implementation concept, but
From my experience implementation concept ( which I have used with SFINae for many years ) is easier to maintain.
The hidden friend part could be used with or without implementation concept part
The hidden friends part is precised here using implementation concept, but could be as easily changed to class template:
https://github.com/mpusz/units/issues/42#issuecomment-635337703
I will await the more elegant solution

As I wrote I liked your idea of using a Quantity concept in math.h. If you want to submit a PR for it I will be happy to accept it. Otherwise, please let me know and I will still your idea and do it by myself ;-)

No problem.  "Success has many parents, but failure is usually an orphan.."   ;-)

I dont really understand why the operations need to be restricted to one class template

It depends which operations. As I said math.h is fine as it might work with everything. However, operators should be closely related to a specific type (i.e. other quantity type might have one additional member but still pass Quantity concepts).

what is the rationale of Quantity concept at all? ( Hint : The answer is "less typing, which increases expressiveness"

No, it is not about less typing. For example, how would you specify with a class template a function that returns a quantity of unspecified dimension, unit, or rep?
Quantity make_quantity(args...);

I have been doing Concepts for many years without real concepts
https://github.com/kwikius/quan-trunk/blob/master/quan/concepts/fusion/sequence.hpp
It is entirely possible and still necessary for many compilers I have to use.

It is not possible with a class template, right? Also, how would you constrain a variable to be of quantity type having only a class template?
Quantity auto q = make_quantity(...);

As above. Not So.  The old way was using SFINAE which was the foundation of Concepts even back to c++03 I think.
https://github.com/kwikius/quan-trunk/blob/master/quan/is_model_of.hpp
backed up by STATIC_ASSERT
https://github.com/kwikius/quan-trunk/blob/master/quan/static_assert.hpp

Concepts are not at all about less typing and should not be perceived this way (otherwise someone int the ISO C++ Committee will object against using those in this library).

I would say that Concepts are still way too new for anyone to make a sweeping statement ;) As to the C++ committee, the only ones I know much about are Bjarne Stroustrup and Daveed Vandevoorde. The rest I don't really . These days many seem surprisingly young though!

Beyond that Ideally the library should be 'lifted' so that is can work with many types

I do not believe so. This library or a C++ Standard will have one concrete quantity type and we should stick to it in this library. We can think about creating a new library of library-independent tools like math or random (or keep such tools here for now) but it is about generic tools using quantities and not about operators which as I said should be closely related to a concrete type.

Not quite. With operators the result has to be a concrete type but the arguments only need to be models of the specified concept  as I said before. (The basis of generic Programming)
https://github.com/kwikius/quan-trunk/blob/master/quan/basic_matrix/basic_matrix.hpp#L370

you could further restrict using is_base_of

It is not possible as we never inherit from a quantity in this library (it would lose constructors and many operators).

I changed the ll_quantity_concept definition to use is_base_of so now only units::quantity is a model

Not true. If in your library you will inherit your quantity from this base class it will also match those operators.

Another class deriving from the impl base class is I think a theoretical rather than practical problem?
Like unnecessary type punning, someone may try it, but so what? Anyway it is trivial to solve

C++ makes everything possible. There are many hacky ways to make things work. However, at least for the C++ Standard Library, we are trying to behave ;-) and not implement widely strange and uncommon interfaces.
I think we will not agree here 😉 My point is that operators are and should be closely related to a specific type or class template and doing this for a family of types may be dangerous and strange. Although, it would work for most cases...


Possibility to blacklist events for influxdb

Possibility to blacklist some events from being send to influxdb based on the entity_id

Looks good. Thanks!


Resource Type Pattern doesn't match Ec2NodeClass custom resources

I use the bats-detik library to run functional tests on my EKS cluster. I would like to test the existence and status of a custom resource ec2nodeclass. But all tests on a ec2nodeclass resource type fail with "Invalid expression: it does not respect the expected syntax."
It's because the resource type pattern, in the utils.bash, does not accept numeric characters.
# The pattern for resource types
# (pod, PODs, complex.crd.resource, etc.)
 resource_type_pattern="[a-z][a-z.]+"

Is it possible to modify this regex to accept those numeric characters ?

Hello @kassof.
Sure, we can update the pattern to support this resource name. I will tell you once it is done.

I have just pushed a PR: https://github.com/bats-core/bats-detik/pull/49
Once it gets an approval, we will tag a new version.

The tag v1.3.3 has been released and fixes it.


Openpilot build (sons -u -j$(nproc)): "Deprecated tool 'qt' renamed to 'qt3'. "

Describe the bug
Hi,
when trying to build openpilot with:
$ scons -u -j$(nproc)
I get the following:
scons: Readig SConscript files ...
scons: ** Deprecated tool 'qt' renamed to 'qt3'. Please update your build accordingly. 'qt3' will be removed entirely in a future release. File ".../openpilot/SConstruct", line 314, in module.
Could somebody please assist?
Thank you.
OS Version
Ubuntu 20.04
openpilot version or commit
No response
Additional info
No response

This is an issue with newer scons versions. Ensure you're installing the pinned version from the poetry.lock file.


Release tag for v0.4.2

Hi there! Would you please do a GitHub release/tag for the 0.4.2 bump?

Hiya, I pushed a tag for 0.4.2 on sunday but didn't bother with a release for a bugfix version.
Any particular reason for a full release?
Alex
On Tue, 8 Feb 2022, at 01:16, Emily Trau wrote:

Hi there! Would you please do a GitHub release/tag for the 0.4.2 bump?
—
Reply to this email directly, view it on GitHub
https://github.com/lmas/opensimplex/issues/28, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBDCXMTUMTMXOXTLJ4IQTLU2BOG7ANCNFSM5NY57TZQ.
You are receiving this because you are subscribed to this
thread.Message ID: @.***>

Nope, all good. Cheers!


[FEATURE] Support parsing of VCS requirements of the form SomeProject<EMAIL_ADDRESS>

Currently this library fails to parse requirement lines of the form:
SomeProject<EMAIL_ADDRESS>This feature should parse the above example to a Requirements as follows:
{
   "line": "SomeProject<EMAIL_ADDRESS>   "editable": false,
   "local_file": false,
   "specifier": false,
   "vcs": "git+https",
   "revision": "1.3.1",
   "name": "SomeProject",
   "uri": "git+https://git.repo/some_pkg.git",
   "subdirectory": null,
   "path": null,
   "hash_name": null,
   "hash": null,
   "extras": ["extra"],
   "specs": []
}

Update: Removed version from specs as @1.3.1 might be a branch or tag name, so this should only map to the revision.

Hello,
I think this should be:
SomeProject[extra]<EMAIL_ADDRESS>
And the parser should work with spaces too:
SomeProject[extra]<EMAIL_ADDRESS>
See my example there: https://github.com/madpah/requirements-parser/issues/75#issuecomment-1131416862


Jupyter Notebook Support for Legion Python

The last remaining TODO for #726 is to add support for Jupyter notebooks for Legion Python.

Work on new equivalence sets has delayed this to the next milestone.

Jupyter notebook support for single node execution or multi-node execution without control replication has been added to the master branch.

Still leaving this issue open so we can discuss multi-node execution with control replication, but definitely very good progress.

Is there anything to still do here?

Yes, there is still the "multi-node" execution with control replication feature which we should probably do for scalability reasons (and because Legate might need it).

Documenting the control replication feature here for @manopapad. The idea is to implement a custom Jupyter notebook kernel that rather than directly forwarding commands and data directly to the Python interpreter will instead distribute this using a tree broadcast out to however many top-level tasks are running in a control replicated fashion across a large machine. Each top-level task will effectively be listening for these commands and then interpret them in the Python interpreter on its local node in a control-replicated sense. This will be much more scalable than running in a non-control-replicated fashion and having tasks dispatched from a single controller node. This will require creating a custom Jupyter notebook kernel. It would also be good to use Realm directly to do the broadcast rather than relying on slow messaging layers like ZeroMQ.


Git graphs unreadable due to colors

Just installed the plugin (v0.1.1) and tried rendering Git graphs. My Obsidian uses the Default theme. Regular top-down graphs look ok and readable, but the Git graph is unreadable:

I tried setting another theme as per the library's documentation (see docs), but I'm getting an error, seems the CSS syntax they define is not supported:
Error parsing Mermaid diagram!

Unsupported color format: "var(--background-primary-alt)"

@barbu110 I have no control over (or at least don't plan to interfere with) Obsidian's Mermaid support. All the plugin does is create a convenient way to add diagrams as text. Perhaps you should address the question to official forums?


Avoid ParquetDataset API when appending in ArrowDatasetEngine

This is a small follow-up to comments made in #7527 (Note that I'd like that PR to get in first).  Most of these changes are just moving around existing code.
ParquetDataset will likely be deprecated in the nearish future.  This PR ensures that we will avoid using it when the "pyarrow-dataset" engine is specified.

This is related to #7871 and should be ready after #7879 goes in

Merged main since https://github.com/dask/dask/pull/7879 has been merged


Move visiblefacets to ShapeModel

Delete struct Facet
Create visiblefacets in ShapeModel

Codecov Report
Patch coverage: 90.14% and project coverage change: +10.93% :tada:

Comparison is base (129cb18) 77.41% compared to head (f774a84) 88.35%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff             @@
##             main      #68       +/-   ##
===========================================
+ Coverage   77.41%   88.35%   +10.93%
===========================================
  Files           7        8        +1
  Lines         465      481       +16
===========================================
+ Hits          360      425       +65
+ Misses        105       56       -49

Files Changed
Coverage Δ

src/AsteroidThermoPhysicalModels.jl
100.00% <ø> (ø)

src/thermophysics.jl
60.63% <57.97%> (-5.20%)
:arrow_down:

src/TPM.jl
98.08% <97.29%> (-1.92%)
:arrow_down:

src/shape.jl
98.36% <97.61%> (+48.36%)
:arrow_up:

src/facet.jl
93.13% <100.00%> (+21.98%)
:arrow_up:

src/non_grav.jl
100.00% <100.00%> (+23.52%)
:arrow_up:

src/roughness.jl
100.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Support ColorStateList and Drawable

In some places only raw color is supported. It would be nice to support ColorStateList as a filter color and Drawable as a CardView's background.

ColorStateList as a filter color of SVGView is now supported


Add jsdoc_insert_unknown_return_type setting

Currently, DocBlockr will insert a return tag if it hasn't "ruled out" the possibility of a function return. If it hasn't determined a return type, but hasn't ruled out a return, then it adds a @return tag with no type.
This isn't always desirable behavior, as it is often easier to add a new @return tag on the fly, then to have to do a complicated delete that doesn't break the formatting (you either have to do a shift-home delete backspace backspace backspace, or select with a mouse, neither of which are fast).
I added a setting (defaulted to not change the current behavior) that will change whether or not a @return tag is inserted if no return type could be determined.

jsdoc_insert_unknown_return_type
Return Confirmed
Return Unknown
Return Ruled Out

true (default - old behaviour)
add tag
add tag
don't add tag

false
add tag
don't add tag
don't add tag

then to have to do a complicated delete that doesn't break the formatting (you either have to do a shift-home delete backspace backspace backspace, or select with a mouse, neither of which are fast).

Ctrl+Shift+K is the default binding for delete line. (Personally, I rebound it to Ctrl+D since I use it so much). I think it's preferable to use that rather than add a very specific configuration flag to the package.


Per default refetch the chain tip at most every 20 seconds

At the current state, every access to protocol params causes at least one chain tip fetch. Due to the potential delay of this request, often another protocol params fetch is triggered as well.
This significantly (and mostly unnecessarily) slows down transaction building. This fix caches the chain tip for a default of 20 seconds (the average slot time), making sure that subsequent calls to _is_chain_tip_updated return without any delay, drastically speeding up tx building.

Using the opshin-starter-kit as an example
Before
(opshin-starter-kit-py3.8) ➜  opshin-starter-kit git:(main) ✗ time python3 src/off_chain/make_vest.py owner beneficiary
transaction id: 4a26f76b7354d0d0f1c232dc187822701a1fe3faa565ffdc94a08ff9282eb51f
Cardanoscan: https://preview.cardanoscan.io/transaction/4a26f76b7354d0d0f1c232dc187822701a1fe3faa565ffdc94a08ff9282eb51f
python3 src/off_chain/make_vest.py owner beneficiary  0.60s user 0.08s system 1% cpu 43.842 total

After
python3 src/off_chain/make_vest.py owner beneficiary  0.28s user 0.05s system 5% cpu 6.408 total
(opshin-starter-kit-py3.8) ➜  opshin-starter-kit git:(main) ✗ time python3 src/off_chain/make_vest.py owner beneficiary
transaction id: 7042a915046c87c8cd1d7b953f09e3738fb3c93c585730f9546b7000fc32d018
Cardanoscan: https://preview.cardanoscan.io/transaction/7042a915046c87c8cd1d7b953f09e3738fb3c93c585730f9546b7000fc32d018
python3 src/off_chain/make_vest.py owner beneficiary  0.27s user 0.04s system 3% cpu 7.921 total

This is a speed up from 40s to 7s and the script only builds a simple transaction!

Codecov Report

Merging #211 (8b8269a) into main (6ec8623) will decrease coverage by 0.07%.
The diff coverage is 50.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #211      +/-   ##
==========================================
- Coverage   85.60%   85.53%   -0.07%
==========================================
  Files          26       26
  Lines        2882     2889       +7
  Branches      693      694       +1
==========================================
+ Hits         2467     2471       +4
- Misses        305      308       +3
  Partials      110      110

Impacted Files
Coverage Δ

pycardano/backend/ogmios.py
58.79% <50.00%> (-0.06%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

An optimal design would have a few threads that asynchronousmy fetch the chain tip, the protocol params, the epoch and such and fetch it. Then access to those values is instant and they are almost never outdated.

The test failed. I had a fix but can't push to your fork. Could you pull the fix into your fork?

Neat, multi-player PR


Cannot read property 'Consumer' of undefined

Hi I have started using this module been getting this issue
the functionality breaks in DynamicModuleLoader.render function where it tries to find .Consumer
return (React.createElement(react_redux_1.ReactReduxContext.Consumer, null, function (reactReduxContext) { return (React.createElement(DynamicModuleLoaderImpl, __assign({}, _this.props, { reactReduxContext: reactReduxContext }))); }));
i am using react-redux version 5.0.7 Please Suggest How to Proceed ?

Thanks @webor please send codesandbox for a minimal repro.

@webor, Which version of react-redux are you using? We have deprecated 5 and below, that might be causing this issue

Can confirm issue is fixed updating react-redux to 6.x.
Worth bumping peerDep? https://github.com/microsoft/redux-dynamic-modules/blob/master/packages/redux-dynamic-modules/package.json#L47 ?


initial commit

I have created the cfd's rust module. Using the cfd library linked with cfd-sys on the rust.
I'm new to Rust, so if you know Rust, please add it to reviewer. :pray:
The current degree of perfection is as follows. I will add more in the future.

src

common.rs: 95% (low document comment)
key.rs: 90% (low test, low document comment)
address.rs: 80% (low test, No comments)
hdwallet.rs: 80% (low test, No comments)
transaction.rs: 80% (low test, No comments)
script.rs: 75% (low test, No comments)
descriptor.rs: 50% (No test, No comments)
confidential_address.rs: 50% (No test, No comments)

test: 10% (minimum. (Focus on part of common and key.))
cfd-sys: 90% (The document cannot be scrutinized yet)

usage: https://github.com/cryptogarageinc/cfd-rust/blob/feature/initial-commit/README.md
note:
I have confirmed that this library can be used with rust-bitcoin by creating a simple application that uses this library.

@Tibo-lg
I fixed it all!
Although it seems like it can be optimized more, it will take time to scrutinize the logic, so I will leave it as it is.
Please review. :pray:

Thank you for reviewing! After this, I will continue to add implementations, tests, and documentation. (All features will be implemented within a month or so)
Finally add the cfd-sys releases asset creation in CI and merge!


fixes #2817 container run with the empty label

Ⅰ. Describe what this PR did
fixes the issue #2817 Failed to run a container with empty label
Ⅱ. Does this pull request fix one issue?

fixes #2817
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
I have modified the labels_test.go and add some new unit tests.
Ⅳ. Describe how to verify it
pouch run --label k without value can run successfully
Ⅴ. Special notes for reviews

LGTM


Request for a major version release and restore v4 without breaking changes

Fold is not a function
Last changes introduced in this library, specifically the ones introduced in this commit in particular: https://github.com/mplatt/fold-to-ascii/commit/4cf57c75f61396e421255ac26c2e69dcb6472497#diff-e9ef0426c130f92ae3cda5a4b3d0356fL32. Must be considered as a breaking change and probably should be managed as a major release of the package. v5.0.0
If I do this:
{ fold } = require('fold-to-ascii'),

I end up with an error that says fold is not a function.

this broke my production API!

For the 4 line, fold could just be added back as an alias for foldMaintaining, with perhaps a depreciation warning.

Re-introduced fold(str, replacement) in 4.1.2 to appease semver wildcard users and removed it in a subsequent 5.0.0 release.


Add skeletons loaders

Description
It can be discussed, but it could be great to have loaders that are using skeletons. However, Skeletons were available in MagnusUI but no more in NativeBase

can i join


%near-gateway added, %gossip implemented

Working on issue #17 .
%near-gateway agent created, %gossip implemented

Closes #17
Closes #20


Non filtered samples getting lost

Somehow a sample that doesn't get filtered after encoding is getting lost (doesn't show up in the logs subsequently).
08:35:09 DEBUG sc2ts.inference Encoded ERR5054123 B.1.1 missing=150

Doh - had max-daily-samples still set


Add availability zone names to info endpoint response

Towards https://github.com/giantswarm/giantswarm/issues/7105
This PR adds the capability to let the info endpoint (GET /v4/info/) inform about the distinct availability zone names usable in a region.

@tuommaki PTAL


Better validation output

Is this a BUG REPORT or FEATURE REQUEST?:
FEATURE REQUEST
What happened:
When my template is wrong I get not too informal output which is ugly in long workflows
What you expected to happen:
Will be good if validation can display me as minimum line number where something is wrong, in best option also what's wrong.
How to reproduce it (as minimally and precisely as possible):
This is example error i have:
2019/02/01 11:47:41 Failed to parse workflow: error unmarshaling JSON: while decoding JSON: json: cannot unmarshal string into Go struct field Container.args of type []string.
And in situation where i have 10 different containers with args is hard to recognize what happen.
Especialy if on first look everything looks good.

@kivio could you (or anyone else who thumbs up this ticket) give an example please?

OK. So this needs to be fixed in either lint.go (which does not "lint" anything :() or 'validate.go'.

I've done a cursory investigate into this and it is possible to get the line number and character position of JSON errors from json.Unmarshal. We use this is many places in the code, sometime when we get it we know which file we have, sometimes not. Sometimes we have the text of the section of the file (YAML with many manifests), but not the starting line number of the section we're examining.
None of this is complex, but there is a lot of it in different places, so I think we need to get specific as to where we want to return line number.

I don't have this file - my bad.
But simplest example ever - even in simple example from main Argo repository. Put some unknown variables into code.
Like I do - I have put "mountPath" into container definition. Which is invalid of course. But when you get this file for example from the internet and it has 10 "mountPath's" for a different reason - it will be a little hard to recognize which one is bad.
argo lint image-builder.yaml
WARN[0000] image-builder.yaml failed to parse: error unmarshaling JSON: while decoding JSON: json: unknown field "mountPath"
FATA[0000] Invalid workflow/workflows found

In this example number of line will be very helpful.

After further research, this is easy enough for JSON, but not really possible for YAML.

@alexec Is it atleast possible to include the problematic param name in the error output? With a lot of parameters it is very difficult to debug these type mismatch errors.


[TypeScript Resolvers] How to use contextType?

I have tried almost everything but contextType still does not work as expected. Is there any example?
Inside my types.tx Context is properly imported, but in the resolver, there is a plain object.

Could you provide an example / small reproduction?
https://github.com/dotansimha/graphql-code-generator/blob/7154c52e9e80664a8f1806d1e3cb1381d0254bba/packages/plugins/typescript-resolvers/tests/typescript-resolvers.spec.ts#L204-L280

I got my implementation working. Important to note that the import of the .ts file containing your context should be relative to the "generate" output .ts file specified in your codegen.yml.
overwrite: true
schema: 'src/schema.graphql'
documents: null
generates:
  src/generated/graphql.ts:
    config:
	  # ensure this path is relative to "src/generated/graphql.ts"
      contextType: ../context#MyContext

And then your context file should be of similar format:
// src/context.ts
export interface MyContext {
  username: string;
}

Alternatively, you can also point to the exact exported type as a value of the config.contextType property, without having to use the add plugin:
generates:
  ./schema.ts:
    config:
      contextType: ./Context#MyContext
    plugins:
      - typescript
      - typescript-operations
      - typescript-resolvers

Given then there's a Context.ts file relative to the generated schema.ts:
// Context.ts
export interface MyContext {
  foo: 'bar'
}

Now in your resolvers you can access a type-safe context:
getResources(root, args, context) {
  context.foo // "bar"
}


the fact that it exists

This is an extremely massive issue. This should not exist.

term please fix this

sorry it exists and it will forever stay that way because it does

so therefore this issue will stay open and forever be unresolved

the world may never know then

i'd probably fix it (with friends)


The code for ImageNet experiments is missing

Hi,
In your paper, you do experiments on ImageNet, but the repository that your share does not contain code for ImageNet experiments. Could you please share it?

Hi again,
I was wondering whether I should expect any reply here. There are experiments on ImageNet in your paper, so it should be just a matter of sharing the code.


STM32 Flash fix and optimizations

Fix in STM32l4 driver that could be vulnerable to unaligned memory access if data is not aligned in flash_write operation.
Additionally, use __ISB to replace abusive usage of flash_stm32_wait_flash_idle

Codecov Report

Merging #14142 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #14142   +/-   ##
=======================================
  Coverage   52.68%   52.68%
=======================================
  Files         307      307
  Lines       45473    45473
  Branches    10530    10530
=======================================
  Hits        23956    23956
  Misses      16640    16640
  Partials     4877     4877

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 48c6548...24a0b5e. Read the comment docs.


Abstract store in client

There are a few cli commands that leverage the .store() accessor on the Client. I think we should remove the store accessor and instead all cli command should go via public methods of the Client.
Here are two instances of where this is done.
https://github.com/0xPolygonMiden/miden-client/blob/1de56fa21973388b1c2cd7e29290d0595d4b6c56/src/cli/input_notes.rs#L62-L75
https://github.com/0xPolygonMiden/miden-client/blob/1de56fa21973388b1c2cd7e29290d0595d4b6c56/src/cli/account.rs#L155-L167

For the account one, this PR has a fix for it: https://github.com/0xPolygonMiden/miden-client/pull/35. If you can take a look @frisitano we can merge it. I'll work on the notes one as well.

Added a commit to that branch in order to also take care of this issue


converting Vec2/Vec3 to static usage

changes to Vec2, Vec3:

rename VecX.neo to VecX.create
replace all usage of new VecX(...) with VecX.create(...)
deprecate the constructors that pass in an object (they aren't used anywhere)

Let me know if I missed anything, or if I'm breaking any patterns/customs/etc.
Future work based on this:

convert the rest of the Vec2, Vec3 functions to static calls
convert all instances of Vec2 -> Vec2Value, Vec3 -> Vec3Value
prototype some ts-morph logic to automate the production of an array notation based implementation of planck.js

Thanks for the PR.
Given the scope of your goal, I wonder what's your opinion about implementing all the changes in a fork, and eventually applying the key changes here? It could also inform our API design for the next major release if scope of changes is not trivial.
Also, as discussed before, Vec2 will be phased out sooner or later, so I prefer not to change it in the meantime.

what's your opinion about implementing all the changes in a fork

Maybe I misunderstand, but isn't that what we're already doing?  I've got my changes in the mreinstein/planck.js fork and this PR is to merge it into piqnt/planck.js.

Also, as discussed before, Vec2 will be phased out sooner or later

I'm confused, I don't recall talking about Vec2 being phased out.  What will it be replaced with?

What will it be replaced with?

From discord: There are new functions in common/Matrix which will replace Vec2 (and other similar classes) in the future. The new functions are already used in many places.

Maybe I misunderstand, but isn't that what we're already doing? I've got my changes in the mreinstein/planck.js fork and this PR is to merge it into piqnt/planck.js.

You are mostly right. On discord you mentioned "something to note is this isn't the full set of changes that would be needed.  I figured for the first PR, I'd keep it to just the constructor related changes", "And I wanted to limit the size of the initial PR to get a sanity check before I go all in on the rest of the changes", and before you mentioned your final goal is being able to use ts-morph to support array notation.
What I meant is to first have your entire solution (all the changes and additions) in mreinstein/planck.js, and then we can figure out what are the necessary changes in the library to support your use-case. This way we can see the big picture and make an informed decision; we can decide a proper release plan (if they are trivial we can release in v1, otherwise postpone until next major release); and you could freely change the code until you achieve your goal.

We won't be able to remove things from the API without major version change, but I guess it makes sense to remove depreciated stuff in your branch.

Alright I will just go full-nuclear in my fork then.  Do you think we can publish this v2.x? Might as well rip off the bandaid.

We will definitely take a look to see what could we merge in

@shakiba I'd like to get some eyes on this please.
still to do:

fix serialize/de-serialize capability
go through every single test bed example and compare my fork to the pinqt/planck testbed to ensure no errors or visible behavior deviation
update all the docs to match the new reality

Am I on the right track with this?

@mreinstein I appreciate your effort; before we could see your array project built using the proposed changes here unfortunately I won't be able to have any opinion.

@shakiba why are these changes predicated on having my array notation idea completed?  These changes are meant to stand on their own as improvements to planck.  Namely, having a consistent, data oriented vector API.

The way I've been envisioning it is a 2 step process:

update the Vec2/Vec3 to use static functions everywhere, which means a simplified API, and having pure data passed around rather than a class implementation. Get this released into a planck v2
using planck@2 as a starting point, develop an automated transform that can replace all Vector usage from object to array notation

I'm planning/hoping that we could get through Step 1 first, because even without Step2 it still improves planck for everyone, even people that continue to use vector object notation.
does that make sense to you too?

These changes are meant to stand on their own as improvements

Without context I neither agree nor disagree with proposed changes. You are welcome to have your opinion, and fork the library and make it better.

Without context I neither agree nor disagree with proposed changes. You are welcome to have your opinion, and fork the library and make it better.

This really doesn't sound like you're willing to work with me on this. Again: I'm trying to split this work into 2 chunks, so that I don't need to put in 60 hours up-front and get to the end of it with an answer that is essentially no.
This PR is chunk 1. tries to improve the following for vec2/vec3:

make the API consistent, so that for example there is not a different behavior for a.add(b) vs  Vec2.add(a, b)
make vectors data only, instead of having to attach dozens of methods and state to every single one
combine some duplicate functions like new Vec2(), Vec2.neo() so there is 1 consistent way to create vectors
add some additional common vector functions, like Vec2.copy and Vec2.scale
pave the way for easier code parsing in some later release (for array notation for example)

Chunk 2 would be the array notation exploration. It is out of scope for this PR.  I'm doing this explicitly to split the work out into releasable pieces, because mega PRs are harder to actually release compared to small targeted pieces.  We should be releasing smaller pieces more often, and delivering value with each release. That is what these 2 chunks attempt to do.


gtk-youtube-viewer can not launch due to "Can't call method "set_active" " error

Hi,
First of all thank you for maintaining such a tool, it's very useful.
I am on Ubuntu 14.04.5 platform. Last year I installed and used gtk-youtube-viewer. Today I cloned this repo an reinstalled it. But it won't launch.
   $ /usr/bin/gtk-youtube-viewer
   [WARN] undefined object: comboboxtext8
   [WARN] undefined object: comboboxtext4
   [WARN] undefined object: comboboxtext9
   [WARN] undefined object: playlists_checkbox
   [WARN] undefined object: comboboxtext1
   [WARN] undefined object: comboboxtext7
   [WARN] undefined object: spinbutton3
   [WARN] undefined object: channel_name_save
   [WARN] undefined object: channel_id_save
   [WARN] undefined object: comboboxtext6
   [WARN] undefined object: comboboxtext13
   [WARN] undefined object: panel_user_entry
   [WARN] undefined object: comboboxtext3
   [WARN] undefined object: comboboxtext2
   [WARN] undefined object: channels_checkbox
   [WARN] undefined object: dash_checkbutton
   [WARN] undefined object: audio_only_checkbutton
   [WARN] undefined object: comboboxtext5
   [WARN] undefined object: videos_checkbox
   [WARN] undefined object: clear_list_checkbutton
   [Sat Oct  8 16:20:34 2016]: Can't call method "set_active" on an undefined value at /usr/bin/gtk-youtube-viewer line 749.
   Can't call method "set_active" on an undefined value at /usr/bin/gtk-youtube-viewer line 749.

How can I fix it?
Regards.
Kenn

Hi. I'm glad you find youtube-viewer useful.
From the error message, it seems that the file gtk-youtube-viewer.glade is located incorrectly. The problem may be related to the Perl directory where the libraries of (gtk-)youtube-viewer are installed. A possible fix would be to execute:
perl -MFile::ShareDir=dist_dir -E 'say dist_dir("WWW-YoutubeViewer")'

...and delete the directory that is printed by the above command.

Thanks a lot, that works!
It's related to previous installation path, because
 perl -MFile::ShareDir=dist_dir -E 'say dist_dir("WWW-YoutubeViewer")'

printed
/usr/local/share/perl/5.18.2/auto/share/dist/WWW-YoutubeViewer

After I removed that directory it runs now.
Best regards.


Key error "UPN" when an user guest is not from AD

Hi there, im back!
In one of our projects we need to enable guest users to login into the app via Azure AD.
With users that belongs to another AD is no problem, it takes the UPN with no problem.
The point is that users that doesnt belong to an AD (for example hotmail.com users) does not have an UPN and ofcourse the UPN is not in the dict that Azure returns.
Could be great that you can give a second USERNAME_CLAIM for those cases where the dict doesnt return UPN but email, for example.
Can I do a PR with this feature?
Thank you!

PRs welcome. 🎉  Keep in mind i added a New setting to block guest users thats in master but not pushed to PyPi yet.


git-town: 15.1.0 -> 16.0.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/git-town/git-town/releases.
meta.description for git-town is: Generic, high-level git support for git-flow workflows
meta.homepage for git-town is: https://www.git-town.com/
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 15.3.0 with grep in /nix/store/1g25mzkcpwsa3sf08q06z0czi337lidx-git-town-15.3.0
found 15.3.0 in filename of file in /nix/store/1g25mzkcpwsa3sf08q06z0czi337lidx-git-town-15.3.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

git-town

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/a4a7d72a73168b0c11447e0659365555b8780daf#git-town

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/1g25mzkcpwsa3sf08q06z0czi337lidx-git-town-15.3.0
ls -la /nix/store/1g25mzkcpwsa3sf08q06z0czi337lidx-git-town-15.3.0/bin

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

1 package built:

git-town

Maintainer pings
cc @allonsy @blaggacao @gabyx for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/git-town/git-town/releases.
meta.description for git-town is: Generic, high-level git support for git-flow workflows
meta.homepage for git-town is: https://www.git-town.com/
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 16.0.0 with grep in /nix/store/gphrdgmds56a1x6lc9m8qxm984ga40ff-git-town-16.0.0
found 16.0.0 in filename of file in /nix/store/gphrdgmds56a1x6lc9m8qxm984ga40ff-git-town-16.0.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

git-town

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/521d3eb8c458c919d06d8cd588be472f3faa7135#git-town

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

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

1 package built:

git-town

Maintainer pings
cc @allonsy @blaggacao @gabyx for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

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

1 package failed to build:

git-town

There was probably an issue on my side. If ofborg says it builds on Darwin, then everything should be fine.


docs: Final edits before ML Challenge

Closes #1312
Preview at https://melissawm.github.io/cryoet-data-portal/

The TOC for the python API is not rendering properly.


[WIP] Move editor actions into editor controller

Move pageflow/entries#partial to pageflow_paged/editor/entries

Rename entries#edit action to editor/entries#show

REDMINE-17267

Coverage decreased (-0.02%) to 99.047% when pulling b9b8674919b45557cff082fdb89b19df349b2ee7 on tf:move-editor-actions into 0fdabc54cab6f3f4aeff2bbe8a7125a6c74cfc10 on codevise:master.


tags.scm should probably recognize functions with string names on the LHS

Like "$<-.data.table" in data.table
https://github.com/Rdatatable/data.table/blob/4d42cdc4f73084390eccc589e726519556c1f8a1/R/data.table.R#L2257-L2264

Not sure exactly what we should tag as the @name, probably just the whole string including the string delimiters? I think if we naively drop the delimiters and use the string_content we could have issues in the case of "name \" with dquote" because you end up with this \ in the captured text that would have to be unescaped to get name " with dquote. Seems a little complicated.
Then again, maybe if it works better in the 99% case with string_content, we should just do that and be ok with problems in the 1% case?

I'm also on board with just using string_content. It also allows us to discount the delimiter, since these all do the same thing:
foo <- 1
"foo" <- 1
'foo' <- 1
`foo` <- 1

Not to mention, raw strings 😅

Yea, raw strings are a huge pain:
R"----------(why is this allowed?)----------" <- \(x) x

`why is this allowed?`(2)
# [1] 2

Probably a separate bug, but I notice there's no support for right-assigned functions:
(function(x) { x }) -> foo
foo(2)
# [1] 2

This is a monstrosity (in my ever-so humble opinion), but does come up in CRAN packages:
https://github.com/cran/symbol.equation.gpt/blob/b9c5722f84da9b3464baef121ebd6c0cd8f26689/R/aaa.R#L15-L22


[2.3.0] Invalid messages block queue processing. nack_errors: false, qos: 100

There were a few invalid indexing message in the queue, that tried to index a parent dependent type with an empty string as parent which threw:
[2014-09-08 04:30:20,824][WARN ][river.rabbitmq           ] [es-master-1] [rabbitmq][rabbitmq_river] failed to execute bulk for delivery tags [[59, 60, 61, 62, 63, 64, 65, 66, 67, 68, 69, 70, 71, 72, 73, 74, 75, 76, 77, 78, 79, 80, 81, 82, 83, 84, 85, 86, 87, 88, 89, 90, 91, 92, 93, 94, 95, 96, 97, 98, 99, 100]], ignoring
org.elasticsearch.action.RoutingMissingException: routing is required for [prod-project-250]/[emailSubscribe]/[null]
at org.elasticsearch.action.index.IndexRequest.process(IndexRequest.java:592)
at org.elasticsearch.action.bulk.TransportBulkAction.executeBulk(TransportBulkAction.java:216)
at org.elasticsearch.action.bulk.TransportBulkAction.doExecute(TransportBulkAction.java:153)
at org.elasticsearch.action.bulk.TransportBulkAction.doExecute(TransportBulkAction.java:65)
at org.elasticsearch.action.support.TransportAction.execute(TransportAction.java:65)
at org.elasticsearch.client.node.NodeClient.execute(NodeClient.java:92)
at org.elasticsearch.client.support.AbstractClient.bulk(AbstractClient.java:159)
at org.elasticsearch.action.bulk.BulkRequestBuilder.doExecute(BulkRequestBuilder.java:164)
at org.elasticsearch.action.ActionRequestBuilder.execute(ActionRequestBuilder.java:91)
at org.elasticsearch.action.ActionRequestBuilder.execute(ActionRequestBuilder.java:65)
at org.elasticsearch.river.rabbitmq.RabbitmqRiver$Consumer.run(RabbitmqRiver.java:415)
at java.lang.Thread.run(Thread.java:744)
I was expecting the invalid messages will be discarded, given nack_errors is set to false, but instead it blocked processing of the queue. I'm guessing this has to do with the latest qos setting

This project is not maintained anymore.


plugin not work

plugin not work

Looked forward to trying this out, since it'd be nice to have a menu automatically created for markdown files... seems like (yet another) feature that GitHub should just support out of the box.
Is the fix in https://github.com/willklein/markdown-menu/pull/19? Looks like it was all ready to be merged, but then never was.
Anyway, it'd be nice to have this, but sometimes life (and priorities) change. 🤷♂

Unfortunately, the extension does not work.

Hey, thanks all for reporting this and sorry I never checked my notifications.
Super cool of y'all! Good news is I can finally retire this extension because the GitHub UI supports it natively! 🥳
Again, thanks friends.

I realize their feature doesn't work on anything except the main page of the repo and for its README only. I might fix this and retire this extension in a working state.


Custom integration errors after first install

First of all thank you for creating this repo, really exited about adding our onesmartcontrol house to homeassistant ....
I have added the 0.30 version in HACS, created a separate userid in smartcontrol, added the integration in HA and used the newly created username and password. Without any authentication errors after adding there are no devices and no entities. The HA logging shows the following:
Error while setting up onesmartcontrol platform for switch
11:00:42 PM – (ERROR) Switch (custom integration)
Error while setting up onesmartcontrol platform for water_heater
11:00:42 PM – (ERROR) Water Heater (custom integration)
Error while setting up onesmartcontrol platform for climate
11:00:42 PM – (ERROR) Climate (custom integration)
Error while setting up onesmartcontrol platform for sensor
11:00:42 PM – (ERROR) Sensor (custom integration)
Details of the first one:
Logger: homeassistant.components.sensor
Source: custom_components/onesmartcontrol/onesmartentity.py:31
Integration: Sensor (documentation, issues)
First occurred: 11:00:42 PM (1 occurrences)
Last logged: 11:00:42 PM
Error while setting up onesmartcontrol platform for sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 281, in _async_setup_platform
await asyncio.shield(task)
File "/config/custom_components/onesmartcontrol/sensor.py", line 92, in async_setup_entry
OneSmartSensor(
File "/config/custom_components/onesmartcontrol/sensor.py", line 127, in init
super().init(hass, config_entry, wrapper, update_topic, source, device_id, name, suffix, icon)
File "/config/custom_components/onesmartcontrol/onesmartentity.py", line 31, in init
self._room = rooms[self._device[OneSmartFieldName.ROOM]]
KeyError: ''
Any idea of what I might have done wrong?

Hmm, interesting... Do you have any rooms setup in the app?

Might be fixed with #37. Will be in release v0.3.1. Let me know if it works.

Working, thanks....


Implement the interface CommunityDatabaseType

This is part of the work required in https://github.com/red-gate/flyway-main/issues/3845

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


[Bug] Database version not found: /dataset/v1.0-mini/v1.0-trainval

Prerequisite

[X] I have searched Issues and Discussions but cannot get the expected help.
[X] I have read the FAQ documentation but cannot get the expected help.
[X] The bug has not been fixed in the latest version (dev-1.x) or latest version (dev-1.0).

Task
I'm using the official example scripts/configs for the officially supported tasks/models/datasets.
Branch
main branch https://github.com/open-mmlab/mmdetection3d
Environment
bevfusion docker
Reproduces the problem - code sample
python tools/create_data.py nuscenes --root-path /dataset/v1.0-mini/ --out-dir /dataset/nuscenes --extra-tag nuscenes
Reproduces the problem - command or script
python tools/create_data.py nuscenes --root-path /dataset/v1.0-mini/ --out-dir /dataset/nuscenes --extra-tag nuscenes
Reproduces the problem - error message
Database version not found: /dataset/v1.0-mini/v1.0-trainval
Additional information
After I downloaded the nuscene-mini dataset, I did not find v1.0-trainval folder inside.

The mini version is called v1.0-mini and the full dataset v1.0-trainval. That's why it's not working.


Add integerValue to IntegerLiteralExprSyntax and floatingValue to FloatLiteralExprSyntax

Resolves https://github.com/apple/swift-syntax/issues/405

StringLiteralExprSyntax has representedLiteralValue property. IMO, we should use the same name. One question is whether we should put them in SwiftParser, WDYT @ahoppen ?

I agree with the naming. Unless there is a reason to put them in the parser (which I don’t think there is, StringLIteralExprSyntax.representedLiteralValue only needs it because it hooks back into the lexer for unicode parsing (eg. \u{1F36D}). But since we don’t need it here, I’d just put it in the SwiftSyntax module.

representedLiteralValue

Renamed to representedLiteralValue

@ahoppen should this also have a release note?

Yes, please. I thought I amended my review with that but looks like I never hit the submit button.

Could this do it?

@swift-ci please test

@swift-ci please test

@swift-ci please test windows

@bnbarham @rintaro can you please review :D
Rebased on latest main

@swift-ci please test

@swift-ci please test windows

@swift-ci please test

@swift-ci please test


Missing Access-Control-Expose-Headers header in the HTTP response

Missing Access-Control-Expose-Headers header in the HTTP response.
This header is crucial for enabling proper access to rate-limiting headers in the browser when making cross-origin requests. Currently, the absence of this header is causing significant functionality issues for users who are attempting to interact with the API from a web application.
Bug Symptoms:
The absence of the Access-Control-Expose-Headers header is resulting in the following symptoms:

Rate Limiting Headers Unavailable: As a result of the missing Access-Control-Expose-Headers header, the rate-limiting headers necessary for monitoring and managing API request limits are not accessible in the browser. This makes it impossible for developers to implement effective rate-limiting strategies and impacts the user experience.

Steps to Reproduce:

Make a cross-origin request to the affected API endpoint/resource from a web application running in a different domain.
Monitor the response headers received in the browser.

Expected Behavior:
The Access-Control-Expose-Headers header should be present in the HTTP response with a value that specifies the rate-limiting headers that should be exposed to the browser, such as:
Access-Control-Expose-Headers: X-RateLimit-Request-Count, X-RateLimit-Last-Reset, X-RateLimit-Reset, X-RateLimit-Max

This would allow developers to access and utilize these rate-limiting headers for effective API usage tracking and management.
Actual Behavior:
The Access-Control-Expose-Headers header is missing from the HTTP response, preventing the necessary rate-limiting headers from being exposed to the browser.
Proposed Solution:
To resolve this issue, I recommend implementing the Access-Control-Expose-Headers header in the API response, specifying the rate-limiting headers that should be accessible in the browser. This will allow developers to retrieve and interpret these headers, enabling them to effectively manage and optimize API requests.

yes i wrote this with chatgpt

BAN! NOW!!


Django 4.1 issue with migrate the database after adding django-axes

After django-axes added to the project, when migrate the database this error raised:
sqlite3.OperationalError: error in index axes_accessattempt_trusted_0eddf52e after drop column: no such column: trusted
django 4.0.7 works just fine.

I think this is the same as https://github.com/jazzband/django-axes/issues/915


Access property default using bracket notation

Using dot notation to access the property default causes an error in Internet Explorer 8:
Expected identifier
~/power-assert/index.js:35:0

because default is a reserved word.
Similar errors:
http://stackoverflow.com/a/14448965
http://stackoverflow.com/a/32780767

@LeshaKoss Ah, good catch! Thanks!

Just released power-assert 1.3.1.
Thank you for your contribution!


Match repo state to ignored files list

There are a number of file types that were added to the root .gitignore list at some point without all of the existing checked-in copies of those files being removed, which is now a pub publish error. This

removes all of the files that are on the .gitignore list but are checked in, and
adds generated_plugin_registrant.h to the ignore list, since all of the other generated registrant files are there; it doesn't make sense to remove the .cc files but not the .h files.

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[x] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

(I'm still not completely sold on having a root .gitignore at all, since project directories should have them, but since I know there have been issues with flutter adding new generated files that aren't on the existing projects' ignore lists, for now we can keep this approach.)

Landing on red to fix the tree. Everything but publishable passed in the last run, and publishable passed this time.


Update privateca API v1beta1 to v1

Updated the cas-gclb-rotation tool from from v1beta1 to v1

Added v1beta1 to v1
Updated config.yaml  from issueCa -> issuingPool
Change public key format to PEM
Added x509 in cert body

Updated the cas-gclb-rotation tool from from v1beta1 to v1
Added v1beta1 to v1
Updated config.yaml from issueCa -> issuingPool
Change public key format to PEM
Added x509 in cert body

Author is OOO, so will merge these changes and follow-up on my comments separately.


Question about the labeling of scannet 2d data

Thank you for your work.
I would like to use the same data as you did for my research, and I need the labels in scannet 2d data.
I read your code, and figured out that you are mapping the labels to 1-40 which is NYU40.
But you are using 20 classes of scannet for 3D evaluation, right?
When I downloaded your preprocessed data and checked the data range of label files, it is 0-255.
How am I supposed to get the 1-40 labeling or 0-19 labeling as the 3D counterpart?
Thanks!

Really sorry for the delayed reply, I did not check the issue. The label range is not 0-255, but 0 to 19 for 20 classes, and 255 means there is no annotation for the corresponding points. For the label mapping, we simply followed the previous work like BP-Nets.


Support for htmlhint.json file

is there a way to tell Visual studio code to use htmlhint.json instead of redefining the content of the htmlhint.json in VSC again?

We use htmlhint together with yarn and webpack like this:
htmlhint --config htmlhint.json "app/scripts/{,}/*.html" "test/{,}/*.html"
and so I have to redefine the rules in VSCode again.

@mnp-mid - see this PR: https://github.com/Microsoft/vscode-htmlhint/pull/35.  Happy to take contributions here.


Signalled faults (SIGSEGV, SIGBUS, etc.) in the target fail to preserve the test case

When running the cargo-fuzz on this project, fuzzing runs as intended for a while but inevitably crashes with Error: Fuzz target exited with signal: 11. This occurs on both macOS and Linux (running in a VM, I can't test running it natively at the moment). I'm using the latest nightly compiler, which at time of writing is rustc 1.46.0-nightly (0cd7ff7dd 2020-07-04).

You will want to try and run the fuzzer with a debugger and then look at the stack trace to understand if the signal 11 (SIGSEGV) is coming from your library or somewhere else.

(Ideally we wouldn't crash without retaining the test case, but the issue report is not super clear whether that's actually true)

I just ran into this. In my case it looks like a stack overflow.
Unfortunately the target doesn't seem to be compiled with debug symbols. Is there any way to check if the input was saved? Or should I assume it wasn't, and try to get it off the stack (which I doubt I'll be able to)?

After debugging it I did find the crash was caused by a stack overflow in my library. The input which caused it wasn't saved and so it was pretty hard to track down. I ended up figuring out the cause using afl.rs which correctly handled the stack overflow.

Getting this with an infinite recursion
Workaround was to println the fuzzed input


Added RXRetrofitDemo

Added this demo which uses generalised model for calling API with Retrofit 2.0 & RxJava 2.0

Thank you @raghavsatyadev and @WilsonChristian94 for putting efforts together, will perform review on it and share!

Thanks @raghavsatyadev for the contribution, looking forward to see more contribution!


Stop bot if detected soft ban

If some kind of soft ban or hard ban is detected, it would be great if we could have an option to terminate the bot process to prevent it from sending further packets to the server

#3902


Validate planning with Lex

A first version of the planning for the coming one month (i.e., till Nov. 2023) was discussed with Lex and Jonas.

One action point (for Prince) is to ask colleagues if a single archimate model can have english as well as dutch version of element names


PHOENIX-6153 Table Map Reduce job after a Snapshot based job fails wi…

…th CorruptedSnapshotException

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
5m 34s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ 4.x Compile Tests _

+1 :green_heart:
mvninstall
10m 43s
4.x passed

+1 :green_heart:
compile
1m 11s
4.x passed

+1 :green_heart:
checkstyle
0m 38s
4.x passed

+1 :green_heart:
javadoc
1m  0s
4.x passed

+0 :ok:
spotbugs
3m 46s
phoenix-core in 4.x has 957 extant spotbugs warnings.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
5m 56s
the patch passed

+1 :green_heart:
compile
1m 17s
the patch passed

+1 :green_heart:
javac
1m 17s
the patch passed

-1 :x:
checkstyle
0m 39s
phoenix-core: The patch generated 6 new + 94 unchanged - 4 fixed = 100 total (was 98)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
javadoc
0m 57s
the patch passed

+1 :green_heart:
spotbugs
4m  0s
the patch passed

_ Other Tests _

-1 :x:
unit
153m 53s
phoenix-core in the patch failed.

+1 :green_heart:
asflicense
0m 38s
The patch does not generate ASF License warnings.

193m  9s

Reason
Tests

Failed junit tests
phoenix.end2end.UpsertSelectIT

phoenix.end2end.SystemCatalogIT

phoenix.end2end.index.IndexVerificationOutputRepositoryIT

phoenix.end2end.ViewMetadataIT

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-902/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/902

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile

uname
Linux 88b7c76073ba 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
4.x / 1df0cc9

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

checkstyle
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-902/1/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-902/1/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-902/1/testReport/

Max. process+thread count
6409 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-902/1/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

https://issues.apache.org/jira/browse/PHOENIX-6153 is resolved and Patch has been committed to master and 4.x


[Feature] Support for clone as a model materialization type

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
We currently "gate" models behind tests by adding a downstream model that does a full table copy via table materialization.  This worked at first but has not scaled and gets expensive on pay-as-you-go compute.
Instead of using a table materialization, we would like to add a clone materialization that does a databricks shallow clone on the upstream table and adds it to a different schema with the same table name.  This will be a very fast operation and will not have an expensive I/O hit.
It should only work with upstream models that are supported by shallow clone (not views, for example)
clone models should run just like any other model, including support for the graph operators, hooks, custom schema macro, custom database macro, etc.
There should not be any dependencies on dbt state artifacts in order to run a model with a clone materialization (unlike databricks-dbt's current implementation of the dbt clone cli linked below)
Describe alternatives you've considered

Post hooks: can't find a way to wait until after tests finish successfully
On-run-success:  can't find a way to wait until after tests finish successfully
Custom logic in Airflow or Databricks jobs: prefer to keep the graph internally to dbt and not add custom logic elsewhere.  It gets too complex to maintain the graph of models in difference codebases/tools.
databricks-dbt custom materialization for clone: not supported by models and only works from the dbt clone cli command.

Who will this benefit?

Anyone who wants to copy a model to a new database schema in a fast and cost-optimized way on a database that supports shallow clone

Are you interested in contributing this feature?
Yes
Anything else?

Related github issue with databricks-dbt package: https://github.com/databricks/dbt-databricks/issues/832

See a potential approach with a code example in the comment here.  This is not fully functioning but is close.

databricks-dbt clone materialization (does not work from a model): https://github.com/databricks/dbt-databricks/blob/main/dbt/include/databricks/macros/materializations/clone.sql

This is a great feature request. I have been trying to fiure out a way to wait until all tests have passed, before a table is cloned to production without extra cost.
What we have implemented so far is to get the result we want is to add a custom schema for the most important models of a model folder. We run our tests in our custom schema during the build command. When everything has finished successfully, we run a macro to clone the tables from the custom schema to the production schema with a macro, that is called repeatedly. This logic only exists on Airflow and not on the dbt repo, so it's difficult to remember and debug in case a test fails. I have this in my backlog since July 2023, to refactor the usage of macros after the flow has finished, and rather use post-hooks. A new type of materialization which could clone the previous model could solve the problem, although it would create yet another model in the model folder.
With our implementation we make sure we never show into production untested tables, as we prefer leaving them outdated.
Any thoughts?

@thomasantonakis I've thought about that too.  We also use Airflow and do a lot of custom parsing of the dbt manifest to hook in other functionality that dbt does not support natively.  However I would love to get away from those patterns and use more native dbt solutions.  I'm okay with the overhead of another model because it reduces complexity in our custom airflow logic.
I got a POC pretty close to working here but it feels a bit hacky.  The model.sql would call the clone() macro instead of adding the config() block.  With some guidance I don't think it would be a big lift to add a proper clone materialization for use by database backends that support cloning.
I am open to contributing to this some more but will need guidance. The dbt-databricks folks suggested that I open a feature request with dbt-core

I also think this is a good feature request. Note that if you want this materialization to be available for any adapter, you'll have to check what zero-copy clone feature each data warehouse offers.
If it's too ambitious to get this approved as a new official dbt materialization, you could instead write a dbt package that provides this materialization.
I have written a dbt package that provides a materialization "acid_clone" that does something like this for Snowflake. In our case, it's important that the clones are created only at the end of the run, if the whole run was successful. The point is to make the dbt run "ACID", as far as these final tables are concerned. We create all the clones in a single Snowflake transaction. The package is not public but I'd be happy to share more information.

I noticed a [clone.sql](that https://github.com/dbt-labs/dbt-adapters/blob/b12c9870d6134905ab09bfda609ce8f81bc4b40a/dbt/include/global_project/macros/materializations/models/clone/clone.sql#L2) in the dbt-adapters repo already but it looks to have the same problem as the dbt-databricks version here.
Why was it implemented with a dependency on the state file?  Most other materializations do not have that dependency.
Could the table type be determined at runtime instead of via the state file?


feat: optimize useModal ts type

I added a generic parameter to useModal and NiceModalHandler, which will affect the type of resolve and the return value type of show.
const { resolve,  show } = useModal<string, number>('id')
/** good */
resolve(10)
/** bad */
resolve('10')

show('test').then((value) => {
/** value is a number type */
})

Thanks @IVLIU for the PR!
However, there have been several attempts to enhance types of APIs like useModal, modal.show, etc, but all failed and rolled back.
So I need to carefully look into code. You can also revisit the code again to make sure it covers all possibilities and is backward compatible with the current type definition.


Video

Hi, I'm from Brazil, could you make a video of how to configure, because I do not know

@nikeyes Thank you

Hi Henrique,  I will try to make a video on monday or tuesday.

Thank you very much, I await the link.

Hi, good morning, did you make the video?
I did according to readme and could not install.
Thank you so much!

Hi Henrique,
Sorry for the delay,  i'm out and i will arrive this afternoon.  I will try to make the video tonight.
Jorge
Enviado desde Yahoo Mail con Android
El mié., ago. 1, 2018 a 15:13, Henrique de Sousa<EMAIL_ADDRESS>escribió:
Hi, good morning, did you make the video?
I did according to readme and could not install.
Thank you so much!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Thank you


Wrong value retained after a period of consumption stability

When the consumption is stable for a while, sometimes this is not the latest value which is published.
Example: EMIT_INTERVAL=10s

01s : 500 VA -> published
02s : 600 VA
03s : 500 VA
04s : 600 VA
05s : 500 VA
06s : 600 VA
07s : 000 VA
08s : 000 VA
09s : 000 VA
10s : 000 VA -> not published because frame[10s] =frame[9s]

So the current value displayed (in home-assistant for example) is 500VA instead of 0VA

Merged in master.
(https://github.com/fmartinou/teleinfo2mqtt/commit/a64090c5543876495ab569bbd4118e952f230c1c)


feat: terms and conditions for onboarding [LESQ-1001]

Description
Music year: 1985

adds text section with links to ts&cs
slightly refactor school input so it doesn't start in an error state
added additional tests and also noticed some act errors in onboarding page tests, not sure how I missed that but fixed them (there's some async actions in react-form that causes these so we have to await/find everything)

How to test

Go to https://deploy-preview-2665--oak-web-application.netlify.thenational.academy/onboarding (feature flag must be active for your distinct id)
You should see text below the form for oak's ts & cs and a link to contact us

Screenshots
How it should now look:

Checklist

[ ] school picker still works as expected and highlights error state / disables continue button
[ ] links in text go to the correct places
[ ] before merging: check feature flag is not enabled on prod except for specific user ids

thanks @RBrownsett should have said that these designs/copy aren't finalised as we're just getting the structure and functionality of the onboarding page set up!

:tada: This PR is included in version 1.503.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Place verifier contract to contracts folder instead of contract

Problem
Hardhat tipically searches the folder contracts for contracts, but nargo codegen-verifier places the default verifier into a contract folder
Proposed solution
The easiest solution would be to add this verifier contract to the contracts folder
Alternatives considered
Using the source configuration param in hardhat, you can have it search for other folder. However, seems like we would want this to be consistent with the more common contracts naming
Additional context
No response

@TomAFrench Is it okay to call this folder contracts given that we changed the contract command?

I have a couple of reservations about this:

It assumes that the noir project will live in the same directory as the hardhat/foundry root. This is unlikely imo.
The contracts package isn't a universal standard as foundry projects tend to use src.
If we have verifiers which don't target smart contract languages then this doesn't make sense.

I'd prefer to add an out arg to the codegen-verifier which defines which directory to write the generated verifier. We can then have a very generic default directory (e.g. verifier) and then add a field in Nargo.toml for users to set up specify where they want it send by default:
[codegen-verifier]
out = "../hardhat_project/contracts/verifier`

Marking this as a good first issue as it should be fairly achievable for without a ton of context on the internals of the Nargo/Noir. That said it's a feature of a pretty decent size.

@TomAFrench whats the status of this issue?


Heuristic for textual file recognition

Request
Add logic for differentiation binary and textual files based on their content
This might possibly be opt-in, as it'd slow down the processing a bit.
Motivation
Currently the textual files are recognized only by their extension. So there can be snapshots which are considered non-textual despite they can be handled as texts.
Textual comparison is preferable to stream comparison for multiple reasons (ability to normalize differences in conventions like newlines, BOMs, etc.; ability to provide more actionable equality failures). So extra efforts in trying to differentiate binary and textual files might bring extra benefit to users
Possible implementation
Unfortunately there is no easy solution :-) Probably someting along the lines of taking a sample of file and doing frequency analysis of bytes/group of bytes and looking for high presence of nonprintable characters.
Existing third party libraries exist: https://github.com/tylerlong/ude, https://github.com/superstrom/chardetsharp etc.

wouldn’t this suffice ? https://github.com/VerifyTests/EmptyFiles#addtextextension

@JanKrivanek ?

Thank you @SimonCropp - I wasn't aware of the addextension - thinking about it bit more it should likely suffice - as heuristic would anyway need to be opt in - so than why not just opt in extensions. In my case I was thinking about a scenario where Verify is used by an intermediate tooling, where final client code doesn't have access to settings - but the extension choices can be simply passed through, or if needed the heuristic can live in the intermediate tooling and then simply just call addextension based on results.
tl;dr;: closing this (can be reopened if there are more asks for it.)


including additional services autoscale now supports

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

@shilpasharmaAM - Hi. This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. Would you make this content update in the private repo? We'll close this PR. Thanks.


Resizer seems to lose it's mind

While using inspect the "resizer" jumped off the screen to 166% left.

Do you have some steps to reproduce this?

Nah, I'll watch for it and reopen if it happens in a reproducible way.


:bug: Refactor taint usage and flow controll in node_reuse e2e test

This PR:

introduces wait loops that wait for all pods to be running at the end of scale up and
scale down cycles to make sure that the workloads of the cluster won't get stuck or kept
in scheduling longer than absolutely needed
disables control plane node tainting during the node_reuse test to get rid of the
complexity of managing tains parallel to scale up/down async processes
noSchedule taints are disabled for control plane nodes because of an other reason also,
namely that during most of the test there are no other nodes than control plane nodes
available for scheduling so it doesn't make sense tainting them

test-centos-e2e-feature-main

/keep-test-centos-e2e-feature-main

/keep-test-centos-e2e-feature-main

/test-centos-e2e-integration-main
/test-ubuntu-integration-main
/test-centos-e2e-feature-main

/test-centos-e2e-feature-main

/test-centos-e2e-feature-main

/parallel-test-centos-e2e-feature-main

/override "test / test-centos-e2e-feature-main"

/override test /test-centos-e2e-feature-main

/override test test-centos-e2e-feature-main

/override test-centos-e2e-feature-main

/test-centos-e2e-integration-main
/test-ubuntu-integration-main
/parallel-test-centos-e2e-feature-main

/test-centos-e2e-integration-main
/test-ubuntu-integration-main
/parallel-test-centos-e2e-feature-main

/parallel-test-centos-e2e-feature-main

/parallel-test-centos-e2e-feature-main

/test-centos-e2e-feature-main

/test-centos-e2e-feature-main

/test-centos-e2e-integration-main

/test-centos-e2e-feature-main

/test-centos-e2e-integration-main

/test-centos-e2e-feature-main

/test-centos-e2e-feature-main

/test-centos-e2e-feature-main

/test-centos-e2e-integration-main

/test-centos-e2e-integration-main

/test-centos-e2e-feature-main

/test-centos-e2e-feature-main

/test-centos-e2e-feature-main

/test-centos-e2e-feature-main

/test-ubuntu-integration-main

Thanks @lentzi90  for the comments, I will address them in a followup, I am also planning to review the some timeouts, especially stuff related to deletion because those are really tight.

/cherry-pick release-1.2


can i use init.lua with this plugin?

yes

My init.lua is not loading on vscode startup, can someone help me with fixing it?


crash

processor Celeron 840 1.1GHz x2 onboard intel gpu HD2000
https://github.com/user-attachments/assets/6aa6b096-bcd0-4196-af7f-3115c5d34154

Hi:
I am new here but i see that you don't have the create marked.


Clang Kits always use the Visual Studio bundled version

    Another related note is the `C` and `CXX` compiler specifications do not appear to do anything when using clang-cl - for example, if I specify:

    "compilers": {
      "C": "C:\\Program Files\\LLVM\\bin\\clang-cl.exe",
      "CXX": "C:\\Program Files\\LLVM\\bin\\clang-cl.exe",
    }

... or specify CMAKE_CXX_COMPILER by hand in the CMakeCache.txt, when using the visual studio generators, cmake will always use the Visual-Studio-bundled clang-cl instead of the one I tell it to.
Originally posted by @fredemmott in https://github.com/microsoft/vscode-cmake-tools/issues/2810#issuecomment-1287437440

For context, the reason I care about this is that the Xaml compiler is currently only supported via msbuild, so I need to use an msbuild-based generator - but I'd like to use clang for the better error messages, especially for templates and C++20 concepts

@fredemmott Does this issue still reproduce for you? Our apologies for how long this issue has been open, but we'd like confirmation about whether this issue is still blocking you before we add it to our backlog for the next milestone. Thanks!

Closing this issue as stale, please open a new issue if you still are facing issues. Thanks!


Error building basic on main branch

I can't seem to build out a basic project on main using the "release mode"
Error Message
error: Unknown variable
   ┌─ /home/behold/projects/dotnet_proj/hello_gleam_dotnet/build/packages/gleam_stdlib/src/gleam/string.gleam:71:3
   │
71 │   do_reverse(string)
   │   ^^^^^^^^^^ Did you mean `reverse`?

The name `do_reverse` is not in scope here.

error: Unknown variable
    ┌─ /home/behold/projects/dotnet_proj/hello_gleam_dotnet/build/packages/gleam_stdlib/src/gleam/string.gleam:689:3
    │
689 │   do_to_utf_codepoints(string)
    │   ^^^^^^^^^^^^^^^^^^^^ Did you mean `to_utf_codepoints`?

The name `do_to_utf_codepoints` is not in scope here.

gleam.toml
version = "1.0.0"

# Fill out these fields if you intend to generate HTML documentation or publish
# your project to the Hex package manager.
#
# description = ""
# licences = ["Apache-2.0"]
# repository = { type = "github", user = "", repo = "" }
# links = [{ title = "Website", href = "" }]
#
# For a full reference of all the available options, you can have a look at
# https://gleam.run/writing-gleam/gleam-toml/.

[dependencies]
gleam_stdlib = ">= 0.34.0 and < 2.0.0"

[dev-dependencies]
gleeunit = ">= 1.0.0 and < 2.0.0"

hello_gleam_dotnet.gleam
@external(fsharp, "System.Console", "ReadLine")
pub fn readline() -> String

@external(fsharp, "System.Console", "WriteLine")
pub fn println(format: String) -> Nil

pub fn main() {
  println("Hello, world!")
}

Yeah until #2 is complete, pretty much any gleam dependencies won't work yet since they won't have the valid standard library yet

Got it! Closing.


Calling prepareData on an actor with owned sheet item with max uses causes crash

env:

Foundry v12.324
dnd5e 3.2.0

expected:
no crash when calling .prepareData; from the docs: "The work done by this method should be idempotent. There are situations in which prepareData may be called more than once."
log:
dnd5e.mjs:10117 Problem preparing the Maximum Uses formula within Test Item. TypeError: formula.replace is not a function
    at replaceFormulaData (dnd5e.mjs:2545:21)
    at FeatData._prepareFinalFormula (dnd5e.mjs:10111:38)
    at FeatData.prepareFinalActivatedEffectData (dnd5e.mjs:10053:10)
    at FeatData.prepareFinalData (dnd5e.mjs:42173:10)
    at Item5e.prepareFinalAttributes (dnd5e.mjs:12251:35)
    at dnd5e.mjs:17534:37
    at EmbeddedCollection.forEach (foundry-esm.js:5020:9)
    at Actor5e.prepareData (dnd5e.mjs:17534:16)
    at <anonymous>:20:12

repro:
(async () => {
	const act = await Actor.create({
		name: "Test",
		type: "npc",
	});
	await act.createEmbeddedDocuments(
		"Item",
		[
			{
				name: "Test Item",
				type: "feat",
				system: {
					"activation.type": "none",
					"uses.per": "sr",
					"uses.max": 1,
				}
			}
		]
	);

	act.prepareData();
})();

For my use-case I can just call .reset() instead; reporting just in case there's some underlying issue which could use visibility.

You will have to call reset anyway, because despite what the official Foundry documentation says, calling prepareData will never be idempotent even in core because it calls applyActiveEffects, which if called multiple times will make cumulative changes to actor data.


Init the FastAPI app inside the fastapi_babel

setting up middleware in the main app file is a little redundant, so:

add Babel.init_app method to setup the middleware inside the fastapi_babel
update Babel.__init__ to accept the app: FastAPI optional first argument to init the middleware
because of this, the configs argument is now keyword only, and it shouldn't be a problem.
update examples
update README

Yes it's a good idea.
And we will add this feature.
Thanks for mention that.


Unable to fetch all data from Apis that implement pagination mechanisms

There are many api's out there that implement pagination mechanisms. To mention some:

Atlassian Jira
Teamviewer Api

It would be great if the plugin would support these apis better by leveraging the pagination and fetching more data. This is not as easy as it might sound. I can't propose a sound technical spec on how to implement this. Therefore I'm highlighting two concrete examples:
Teamviewer Api endpoint https://webapi.teamviewer.com/api/v1/reports/devices
This api sends an optional next_offset item in the json response that should be sent as http argument offset_id to fetch more data. For example the first response to /reports/devices?from_date=2021-08-01T05:14:00Z json may look like:
{
   ...,
   "next_offset": "eyJTdGFydCI6IjIwMjEtMDgtMDFUMDU6MTQ6MDArMDA6MDAiLCJUYWJsZUNvbnRpbnVhdGlvblRva2VuIjoiOGVmYjQyNjctMjM1NS00YzMxLWIyYmYtNDgzMTlkMzIzOTRhIn0="
}

And the following request to fetch more data should now be sent with the parameter /reports/devices?offset_id=eyJTdGFydCI6IjIwMjEtMDgtMDFUMDU6MTQ6MDArMDA6MDAiLCJUYWJsZUNvbnRpbnVhdGlvblRva2VuIjoiOGVmYjQyNjctMjM1NS00YzMxLWIyYmYtNDgzMTlkMzIzOTRhIn0=.
A smart guy would be able to base64 decode eyJTdGFydCI6IjIwMjEtMDgtMDFUMDU6MTQ6MDArMDA6MDAiLCJUYWJsZUNvbnRpbnVhdGlvblRva2VuIjoiOGVmYjQyNjctMjM1NS00YzMxLWIyYmYtNDgzMTlkMzIzOTRhIn0= to:
{"Start":"2021-08-01T05:14:00+00:00","TableContinuationToken":"8efb4267-2355-4c31-b2bf-48319d32394a"}

Atlassian Jira Api endpoint https://jira.example.com/rest/api/2/search
This api endpoint expects numeric http parameters maxResults and startAt while returning startAt, maxResults, total in the json response. For example requesting ?maxResults=2000&startAt=100 the following is returned:
{
    "startAt": 100,
    "maxResults": 1000,
    "total": 1521,
    ...
}

Note that maxResults is capped at 1000.

Hi @dpsenner where you able to connect to Jira using JSON API at all, or did this very issue prevented it from working?
With curl and postman I can fetch the data, but on Grafana it just says: "JSON API: Cannot connect to API"

Yes, I could fetch the data but ended up with partial results. This is not at all useful and confusing at best.


Rewrite then/2 when function is a named function reference

[x] Please sign Adobe's CLA if this is your first time contributing to an Adobe open source repo. Thanks!

:wave: first and foremost, thanks for creating and maintaining styler! We use it in a large codebase where i stumbled over some occurrences of named function references passed into Kernel.then/2. I couldn't figure when that would be useful so I am opening this PR proposing to have it handled by styler. I've tested this change in our codebase but am happy to get any pointers on what i could've missed!
Let me know if you'd consider adding this to styler (or not! :)

shipped in 0.11.7 ❤