pio_pinchange.py example pin change to high

Hi all
I have this function below from this PIO example https://github.com/raspberrypi/pico-micropython-examples/blob/master/pio/pio_pinchange.py
@rp2.asm_pio()
def wait_pin_low():
    wrap_target()

    wait(0, pin, 0)
    irq(block, rel(0))
    wait(1, pin, 0)

    wrap()

That example (pio_pinchange.py ) using PIO wait for a pin change to LOW and raise an IRQ. I would like to do exactly that, but with pin change do HIGH. I readed this doc https://docs.micropython.org/en/latest/library/rp2.html about the commands of PIO to understand a bit, and changed to that code to this:
@rp2.asm_pio()
def wait_pin_high():
    wrap_target()

    wait(1, pin, 0)
    irq(block, rel(0))
    wait(0, pin, 0)

    wrap()

But not works. I did others modifications, but not success.
Could you please provide this function to works to pin change to high?
I'm using RPICO with MicroPython version 1.8:
import os
>>> os.uname()
(sysname='rp2', nodename='rp2', release='1.18.0', version='v1.18 on 2022-01-17 (GNU 11.2.0 MinSizeRel)', machine='Raspberry Pi Pico with RP2040')
>>>

Thank you very much!

The RP2040 MicroPython port has been merged upstream to the main MicroPython repo, see https://github.com/micropython/micropython/pull/6791. So going forward you should report problems with the port there, https://github.com/micropython/micropython/. Technical support can be found on the MicroPython forums, https://forum.micropython.org/viewforum.php?f=21.


New package: Blosc2 v0.1.0

Registering package: Blosc2
Repository: https://github.com/waralex/Blosc2.jl
Created by: @waralex
Version: v0.1.0
Commit: 67158dda44a0019bed164cbc03e76c2cb3cce50a
Git reference: master
Release notes:

initial version

[noblock]

Package name similar to 1 existing package.
Similar to Blosc.

The reason for this is the similarity of the names of the underlying c libraries. Blosc is the interface to https://github.com/Blosc/c-blosc. Blosc2 is the interface to https://github.com/Blosc/c-blosc2


Disable cuda.init when cuda is disabled

related to #54

Thanks! I missed them in the review.

Thank you very much! I could not notice it.


[IA-4000] Hide terminal entirely when disabled(in azure)

Terminal should no longer be visible when in azure
https://user-images.githubusercontent.com/123021647/216117616-0382967d-b330-4492-9c18-d65bf609fe3e.mov

looks good on PR site


Dynamic committer routing based on language

I'm trying to figure out a way to route a crawled document to a specific Elasticsearch index based on the language of the document. I am using the com.norconex.importer.handler.tagger.impl.LanguageTagger as a pre-parse tagger in my importer config, and I can store the language easily as a field in an elasticsearch index; But the structure I am trying to achieve is to have many Elasticsearch indices (index-en, index-fr, etc...), one index per language. So my question is: Is there any way I can dynamically route the document to it's corresponding Elasticsearch index based on its language (determined by Apache Tika, not any meta tag within the document) using the current version's configuration options?

Right now there is no way to do "dynamic routing" like you suggest.  I am flagging this as a feature request.  In the meantime, you can write you own committer (or extend an existing one) that will perform that custom routing for you.  Without writing code, the only way I can think of to achieve this through configuration alone would be to crawl your site once for every language you support, having a crawler defined for each, with their own committer.  You use import filters to reject documents not in the proper language for each crawler.   Not a pretty solution.
As a side-note, I would suggest you use the LanguageTagger as a post-parse tagger to make sure detection is performed on text only and not text + markup or binary files.  You should get better results.

Thanks for the reply, and the tip, I will do that.

It just occurred to me there is a tricky situation with the approach you want to take you probably will have to account for: deleted documents.
When a URL gets re-crawled, and it no longer exists (e.g. 404), then its URL will be sent for deletion.  At that point the content was not downloaded again so language detection is not possible.  That means you will not have the language to tell you which index to use to delete the document.
The not-so-nice approach of having one crawler per language seem to be the only way to capture deletions properly.
I am un-marking this as a feature request unless we can thing of a reasonable alternative.  Using a custom solution, you could probably just send the URL to be deleted to all indices you have, but I am not sure we want to make this a default practice.
Let me know if a custom committer did it for you.

As I started to think through how I would go about solving this with a custom committer, I also realized there would be several tricky edge-cases I would have to account for. I realized that having more indices is really no different (performance-wise) internally, than simply having more Elasticsearch shards in 1 single index; so I decided to just use a field to store the language instead, since the benefit of creating a custom collector wouldn't really be worth the effort.
If you did still want to add this as a feature, regarding the situation you describe of the collector not knowing which index the soon to be deleted document resides, I think this could be solved by having the delete function search the entire Elasticsearch cluster (assuming the language-specific indices are all in the same cluster) rather than a specific index in that cluster; essentially a wildcard search of every index, which is just as easy to do as an index-specific search (but with added latency of course). And this would really only be necessary when using the DELETE orphan strategy, right? Nonetheless, the feature is no longer required for what I am building, but I think it can certainly be accomplished in the event that you wanted to add it as a feature for others. Thanks for helping!

I am glad you have found an approach that works for you.  Having a solution where it searches for it first would prevent from making this a generic solution (available to other committers as well).  Since you are OK for now, I will close this ticket and we will re-address it if this demand occurs more often.  In the meantime, I added this idea to the Committer Core project internal TODO.txt file so it is not completely forgotten.
Thanks for your input!


Utilizing Patterns

What language is used when utilizing patterns?  For example I want to harvest a folder but I don't want it to harvest from a specific subfolder called "Archive" what pattern would be required?
This would be useful info for a future Wiki.

The pattern language used here is called a "glob" syntax. More about it can be found here. Basically, it is a pattern very similar to the one which is used to filter files using 'dir' or 'ls' command on various operating systems. I doubt you can come up with any pattern which would skip a particular folder. Keep in mind that 'pattern' feature on WAF broker has been designed to filter individual files by it's extension rather than by folder it belongs to (by default the pattern is '**.xml' which only allows broker to grab xml files).
It doesn't mean it is not possible, but it requires a little bit more explanation. All it harvester does is executes or schedules for execution something called: a task. Task represents a workflow of a data from the time it is acquired from the input broker and output broker(s). In general it is as simply as that: take something from input and publish into the output. However, there is already existing concept of filters and transformers. Each of this entity is a little piece of code used to filter incoming data by some sort of predicate, while transformer can transform data from one form to another.
Both filters and transformers can be chained together to perform more complicated operations, moreover, they can be executed in some sort of parallel manner. For example, one could create task which would select only PDF files and publish to the local folder, at the same time it would select xml files then regardless what kind of metadata it is (FGDC, ISO, Dublin Core) it would normalize it to ISO only and publish to the instance of Geoportal Catalog 2.0, yet another thread would select only CSV files, publish it to the instance of 'koop' to create Feature Service based on the data from CSV, then register URL of each feature service to Geoportal Catalog 2.0 AND to the ArcGIS Online.
Such framework does already exist; what is missing is a rich collection filters and transformers and a sophisticated UI alowed to build such tasks. At this moment only REGEX filter is available and one transformer which uses XSLT to transform one metadata format into another.
It's worth no mention that API harvester is being kept intentionally simple so such entities like filters, transformers, brokers can be developed as needed easily.
I hope my explanation shines some light on what is possibly coming in the future releases of harvester.


Forgot the transparency of the images

To fix the issue where the mouth images are not transparent, I need to modify the material setup in your add-on code. Specifically:

Set the "Blend Mode" of the material to Blended (for transparency rendering).
Connect the Alpha output of the texture to the Alpha input of the Principled BSDF shader.

Probably something like:
# Add material and set transparency
mat = bpy.data.materials.new(name=f"Material_{image_obj.name}")
mat.use_nodes = True
mat.blend_method = 'BLEND'  # Enable transparency blending
mat.shadow_method = 'HASHED'  # Avoid shadow artifacts
mat.use_backface_culling = False  # Optional: disable backface culling

# Access nodes
nodes = mat.node_tree.nodes
links = mat.node_tree.links

# Create Texture Image Node
tex_image = nodes.new(type="ShaderNodeTexImage")
tex_image.image = bpy.data.images.load(image_obj.data.name)
tex_image.location = (-400, 0)

# Access the Principled BSDF shader
principled_bsdf = nodes["Principled BSDF"]

# Link texture color to Base Color and Alpha to Alpha
links.new(tex_image.outputs['Color'], principled_bsdf.inputs['Base Color'])
links.new(tex_image.outputs['Alpha'], principled_bsdf.inputs['Alpha'])

# Ensure the material output node is set up
material_output = nodes.get("Material Output")
if not material_output:
    material_output = nodes.new(type="ShaderNodeOutputMaterial")

# Connect BSDF to Material Output
links.new(principled_bsdf.outputs['BSDF'], material_output.inputs['Surface'])

# Assign material to plane
plane_obj.data.materials.append(mat)

Fixed in pre-release v0.1.0-alpha.


fix(plugins): avoid using gocheck=2

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
What this PR does / why we need it:
Supposedly fixes the following error from CodeQL C++:
fatal error: cgocheck > 1 mode is no longer supported at runtime. Use GOEXPERIMENT=cgocheck2 at build time instead.

Which issue(s) this PR fixes:
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 99ca59089915d492955f93871e987ab268acfb94


.asp -> .aspx

I adopted the code to ASP.NET (.aspx) --  I'm using a Mono / XSP / Apache web server and classic ASP is not supported there. If you want to use the code you're welcome to do so. Just replace Default.asp, webfeed.asp, and Web.config with the new Default.aspx, webfeed.aspx, and Web.config. Should work exactly the same way. Code can be found on the fork https://github.com/surfchris/raweb

You should make a pull request for this!  That is a nice feature to add to this so it will work as aspx which, if I am not mistaken, still works in IIS too.

@surfchris it even works on raspberry pi 2.

@surfchris This is great!
I also have a PHP version that someone sent me a few years ago.
Do you guys think it would be better to include the different versions in one repo?
Otherwise, I could modify the Readme.md here and link to the ASPX/PHP versions?
Other ideas?

I can see multiple ways to break it up for aspx and php.  My preference would be to have a separate folder for each type in a single project.  The reason for this is it would remind me (if I was working on the code) that any changes I make in the ASP version, I should add to the ASPX and PHP version to ensure they are all feature compatible.
My thoughts on having different versions in different repos is it will be very easy for one of the 3 versions to become stale.
Plus, when a pull request is made if they are all in a single project, it is easy to reject it if only 1 or 2 of the versions are updated and not all 3.  Well, presuming the bug exists in all 3.  If it is a bug in a specific version of PHP for example, the asp and aspx one wouldn't need updating.
Thoughts?

Good thinking, MrBrianGale. I checked the PHP version and it's out of date - might update it at a later date, but won't include in the meantime.
Included the ASPX files. Thanks, surfchris!


Add default search values

Lets keywords be defined with default values that will be used if the keyword is not included in the incoming search string.  We have a use case where we always want a search to include a default search term of "some_flag:false" if "some_flag" is not otherwise specified.  The README changes have more specifics.

Travis passed on Ruby 2.0.0, fails on Ruby 1.9.3.  When I run in 1.9.3 all specs pass, so I'm unsure what the problem is.


Basics of Linux Commands

content is redundant. someone, already assigned to work on this
Closing your PR


Direction cosine

In the ReadMe, it says:

direction cosine cx, cy in x and y direction of the incident angle of the photon

Is the meaning of cx, cy = u,v?

From the corsika manual, Table 11:

u direction cosine97 to x-axis
v direction cosine97 to y-axis

https://web.ikp.kit.edu/corsika/usersguide/usersguide.pdf

Maybe we should rename this to folow the corsika notation

To cite uncle bob again: http://www.itiseezee.com/?p=83
1.) Avoid mental mapping: so yes one should follow existing notations.
2.) Choose good variable names: So no, one should change the notation in Corsika from the unintuitive single letter name "u" to something longer and easier  to understand, "cx" is also not good.
He also says somewhere (cannot find the page at the moment), that one should not workaround broken software, but fix it. In this sense we talk about fixing corsika, just a tiny little bit by renaming "u" into "dir_cos_x" or so .. any name is better than "u" ... apart from "i" maybe.

For this project one could also think about this:
The user (a physicist I presume) is interested in information about particles, like position, momentum, energyormassor whatever. So instead of outputtingxandy, and let the user deduce z, which is implicitly given by something called "observation plane", we could output something called position`. We still have to explain in which coordinate system we give the position and which units we use, but at least we give a full position. Similar things come to mind, when we think about the momentum, which is given by these direction cosines...
What do you think? should this software help the user to go one layer of abstraction up or not?

I'd say we should keep it simple here.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/13 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-yml-archive-non-versioning-archive.git

Docs Build status updates of commit aede896:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Generalize metadata schemas for wawaka common contracts

Currently, the contract code metadata focuses on validating the code hash on the ledger. However, it would be useful to include more general metadata about contracts such as contract type (wasm vs native), or CDI-type metadata.
This issue may also be related to #294 .

@marcelamelara do you have a more specific request? i would expect any application/contract that wants to use the additional data to extend this function. It would be rather constraining to force all applications to use the additional metadata.

And I think this issue is completely independent of #294. One occurs in the enclave and is part of the fundamental trust relationship between the contract, the ledger and the user. This code is about establishing run-time trust between two separate contracts.


Programatically change grafana variable

Hi all,
Is it possible to change the variable programatically?
I just want to change one variable in the same page.
Then I wouldn't want to reload it or even open a new tab.

https://community.grafana.com/t/how-to-change-a-dashboard-variable-value-from-a-plugin/10107
one way would be to check if relative path is the current dashboard then swap only the variables

another use case to programatically refresh diagram without needing to refresh the whole page.
when using url mapping it opens on a new tab.
even tried <buttononclick=""> but it looks like there's some sanitization

I would love to see this feature as well


Make 'getIndentString' just use a loop.

CC @jasongore

@typescript-bot pack this
@typescript-bot test this
@typescript-bot user test this
@typescript-bot run dt
@typescript-bot perf test this

Seems to fix the following repro:

/**
                                                                                * This is a comment with dumb indentation for some auto-generated thing.
 */
export class SomeAutoGeneratedThing {}

I was going to post Milton + "I was told there would be tail call optimizations" but then it wasn't a tail call 😑


add description for exclude_server_identity_headers

Proposed changes
Add description for exclude_server_identity_headers in UCP configuration docs. (Some customers want to config it but don't know the config item name because it is not in current UCP document)
For details, please see internal PR: https://github.com/docker/orca/pull/16207

Unreleased project version (optional)

Related issues (optional)

Deploy preview for docsdocker ready!
Built with commit 538131a6666773eb82bcbc32583ccb925442018f
https://deploy-preview-8940--docsdocker.netlify.com


CI: add dockerfile hadolint checker

Signed-off-by: Boris Glimcher<EMAIL_ADDRESS>

Codecov Report

Merging #193 (472014e) into main (b644c5e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #193   +/-   ##
=======================================
  Coverage   57.94%   57.94%
=======================================
  Files          14       14
  Lines        1612     1612
=======================================
  Hits          934      934
  Misses        678      678

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 b644c5e...472014e. Read the comment docs.


Install dev tools to find memory leaks

We should install (django-debug-toolbar)[https://github.com/jazzband/django-debug-toolbar] to find performance issues within the backend.
There is this plugin (pympler)[https://pympler.readthedocs.io/en/latest/django.html] which tracks memory usage and integrates with django which should make it easier to find memory issues.
This way, we can address speed and memory concerns more effectively.

Added Silk instead of django-debug-toolbar.

Use https://bloomberg.github.io/memray/ instead of pympler

Or just using docker stats also does the trick


python3-podman: can't utilize rootless podman

Is this a BUG REPORT or FEATURE REQUEST?:
[//]: # kind bug
Description
I really like the python interface you have built. Unfortunately it's not possible to be used with rootless podman due to the varlink limitation. Is there a way to solve it? (have two sockets maybe?)
srw-------. 1 root root 0 Nov  1 14:45 io.podman
srw-------. 1 root root 0 Nov  1 14:45 podman/io.podman

$ id
uid=1000(tt) gid=1000(tt)
context=unconfined_u:unconfined_r:unconfined_t:s0-s0:c0.c1023

$ ipython3
Python 3.7.1 (default, Oct 23 2018, 18:19:07)
Type 'copyright', 'credits' or 'license' for more information
IPython 6.4.0 -- An enhanced Interactive Python. Type '?' for help.

In [1]: import podman

In [2]: with podman.Client() as client:
   ...:   list(map(print, client.images.list()))
   ...:
---------------------------------------------------------------------------
PermissionError                           Traceback (most recent call last)
...

/usr/lib/python3.7/site-packages/varlink/client.py in open_unix()
    459                 s = socket.socket(socket.AF_UNIX)
    460                 s.setblocking(True)
--> 461                 s.connect(address)
    462                 return s
    463

PermissionError: [Errno 13] Permission denied

$ podman images
REPOSITORY                                   TAG                       IMAGE ID       CREATED             SIZE
localhost/this-is-my-image                   latest                    d49960b9dbcd   29 minutes ago      83.2MB
<none>                                       <none>                    d3b925049e8d   29 minutes ago      83.2MB
localhost/this-is-my-image-20181701-161756   latest                    2a566e4de41f   29 minutes ago      83.2MB
docker.io/library/python                     3-alpine                  408808fb1a9e   7 days ago          82MB
registry.fedoraproject.org/fedora            29                        a5cc8ccd230a   2 weeks ago         306MB

Can we set up a socket for rootless in the systemd user instance? That would probably do everything we need for rootless.
Would still need to update the bindings to optionally connect to the separate rootless socket.

Can we set up a socket for rootless in the systemd user instance?

I think that would need to be baked directly in podman, since afaik you can't control user instance from the root environment.
Or maybe just provide docs and mention this in the error message when raising permission denied.

I actually think we should raise an issue for this.  libpodman could launch a podman version locally when running as non root. and then communicate with the podman listening at a socket in the users homedir.

For systems using systemd user sessions (e.g. Fedora for quite some time), the system setup with socket activation could be directly duplicated.


How can cparse be compiled by Visual Studio in Windows Environment

How can cparse be compiled by Visual Studio in Windows Environment?

Hello @jojogh,
We have had an issue earlier about this same issue #55. The user found a way to accomplish it by adding the 4 files of the core-shunting-yard.o library to his project, i.e.:

shunting-yard.cpp
packToken.cpp
functions.cpp
containers.cpp

He also tried compiling the builtin-features.o library but it did not work properly probably because of the static initialization used by this library. He overcame it by using the new build option of initializing this library manually:
#include "cparse/builtin-features.inc"

int main() {
  cparse_startup();

  // ... Your code goes here ...
}

Note: I should already have added this to the docs, my mistake, I will add it right now.

This should do the trick. If it still does not work I could summon the previously mentioned user for help, he would probably be able to give you better tips than me since I am not used to working with Visual Studio.
I hope it helps.

Hello @jojogh, I have added some documentation for this, but since this is not tested it might need some refining:

Download and Setup

Tell me if there are any parts of it that are confusing.

Many thanks, Vinícius Garcia ! I would take a look!
2018-05-10 22:50 GMT+08:00 Vinícius Garcia<EMAIL_ADDRESS>
Hello @jojogh https://github.com/jojogh, I have added some
documentation for this, but since this is not tested it might need some
refining:

Download and Setup
https://github.com/cparse/cparse/wiki/Download-and-Setup#for-visual-studiowindows-users

Tell me if there are any parts of it that are confusing.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cparse/cparse/issues/57#issuecomment-388076933, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAIspXh6e9jTgO2QwLS8gihq1qhXq-T_ks5txFOkgaJpZM4T5Xi9
.

Hi, Vinicius,
I have successfully run the example following your instructions, here is
some things I want to share:

seems the project only support VS 2017, that should be C++ 11's problem;
If you build the code in Visual studio, you have to include the project
path;

I would test the project, many thanks!
Alex Zhang
2018-05-11 10:40 GMT+08:00 Yuanhao Zhang<EMAIL_ADDRESS>
Many thanks, Vinícius Garcia ! I would take a look!
2018-05-10 22:50 GMT+08:00 Vinícius Garcia<EMAIL_ADDRESS>
Hello @jojogh https://github.com/jojogh, I have added some
documentation for this, but since this is not tested it might need some
refining:

Download and Setup
https://github.com/cparse/cparse/wiki/Download-and-Setup#for-visual-studiowindows-users

Tell me if there are any parts of it that are confusing.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cparse/cparse/issues/57#issuecomment-388076933, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAIspXh6e9jTgO2QwLS8gihq1qhXq-T_ks5txFOkgaJpZM4T5Xi9
.

That's good @jojogh, I have updated the Download and Setup page with your feedback. If you can please read it to see if I got it right =].
I hope it works well for you. Regards.

Sure, no problem. I would take time to study and use this project, if I
meet any problem, I would seek help from you.
thanks
Alex Zhang
2018-05-14 10:49 GMT+08:00 Vinícius Garcia<EMAIL_ADDRESS>
That's good @jojogh https://github.com/jojogh, I have updated the Download
and Setup page https://github.com/cparse/cparse/wiki/Download-and-Setup
with your feedback. If you can please read it to see if I got it right =].
I hope it works well for you. Regards.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cparse/cparse/issues/57#issuecomment-388682329, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAIspXZyHSyLClpvQ5PEEOKopKpUU9K0ks5tyPCugaJpZM4T5Xi9
.

:+1: Then I am closing this issue.
Thanks @jojogh for the feedbacks and feel free to report any new problems.


[ISSUE #8213] Optimize some code style in remoting module

Which Issue(s) This PR Fixes

Fixes #8213
Brief Description

How Did You Test This Change?

LGTM


Clean up description in CSV

@JoanneOFlynn2018 just added you in case you have any better suggestions :)

Replaced with #53


Asset Management

Define asset types (e.g., hardware, software, peripherals).
Implement CRUD operations for assets.
Associate assets with users or departments.
Track asset status (e.g., in use, in repair, retired).
Implement asset location tracking.
Develop a system for asset disposal or retirement.
Implement asset scanning and discovery mechanisms.
Create reports for asset inventory.
Integrate with external asset management systems if required.
Implement preventive maintenance for assets.

duplicate


Update Laravel application detection

Fixes https://github.com/pestphp/pest/issues/501
This should work, tested locally in Laravel projects and some packages.
If someone has a better option, I can modify this PR.
Another way to test package install is to use composer-runtime-api and \Composer\InstalledPackages

Question: if a package developer (incorrectly) registers their Package\\Tests namespace under autoload instead of autoload-dev in their composer.json file, then won't Orchestra's TestCase be present in the autoloader? And if that is the case, won't this cause the same issue for a different set of circumstances?

@lukeraymonddowning I don't think so. Orchestra's TestCase won't be present in autoloader because orchertra/testbench won't be installed.
However, this new implementation could fail if a package defines orchestra/testbench in its require instead of in require-dev. This could happen but does not makes sense. I don't know if pest should consider these edge cases caused by third parties.

Hi @nunomaduro
I just saw you closed this PR, maybe because you are working on V2.
In the meantime, the issue still exists. Are you opened to a new PR targetting V2 ?

Yes. Yet, Pest v2.x is not yet ready for Parallel Testing. You can still do the PR if you want. cc @lukeraymonddowning


Server can not bind to port 5683

Hi,
Please see this post.
I am not sure about why it gives me that error.
Thanks,

Hi~
It seems there might be something incorrect with the IPv6 network interface on your OS.
The log says:
DEBUG - Starting CoAP server
Warn - Cannot start endpoint at :::5683
DEBUG - Stopping endpoint bound to :::5683
Warn - Could not start endpoint :::5683
System.Net.Sockets.SocketException: The descriptor is not a socket

The server tries to bind a socket to an IPv6 address of [::] (System.Net.IPAddress.IPv6Any) and fails.
Please try adding the following code before starting a server:
server.AddEndPoint(IPAddress.Any, server.Config.DefaultPort);

It will bind to the IPv4 address (<IP_ADDRESS>), instead of the default IPv6 address.
You can use any other network address on your OS if you'd like to.

Hi, longshine
I have tried your suggestion. It still has problem with binding.
Here is the error message.
Warn - Cannot start endpoint at <IP_ADDRESS>:5683
DEBUG - Stopping endpoint bound to <IP_ADDRESS>:5683
Warn - Could not start endpoint <IP_ADDRESS>:5683
System.Net.Sockets.SocketException: The descriptor is not a socket
  at System.Net.Sockets.Socket.IOControl (Int32 ioctl_code, System.Byte[] in_value, System.Byte[] out_value) [0x00033] in /build/mono/src/mono-4.4.0/mcs/class/System/System.Net.Sockets/Socket.cs:3219
  at CoAP.Channel.UDPChannel.SetupUDPSocket (AddressFamily addressFamily, Int32 bufferSize) [0x00009] in <filename unknown>:0
  at CoAP.Channel.UDPChannel.Start () [0x00117] in <filename unknown>:0
  at CoAP.Net.CoAPEndPoint.Start () [0x00040] in <filename unknown>:0

It can not even bind to ipv4 localhost.
The only distinction my server program has is that I have not added all the resources but HelloworldResource only. But I think this shall not affect the binding, right?
In order to rule out the possibility that my system has some problem, I have compiled a C program, and server can bind correctly.
Thanks,

Hi~
That's very odd. I googled and found some other projects have the same issue too, but no solution. See rabbitmq-dotnet-client and sockets-for-pcl.
CoAP resources should have nothing to do with this issue.
I found some messages in the log:
  at System.Net.Sockets.Socket.IOControl (Int32 ioctl_code, System.Byte[] in_value, System.Byte[] out_value) [0x00033] in /build/mono/src/mono-4.4.0/mcs/class/System/System.Net.Sockets/Socket.cs:3219
  at CoAP.Channel.UDPChannel.SetupUDPSocket (AddressFamily addressFamily, Int32 bufferSize) [0x00009] in <filename unknown>:0

There are some preparation steps when setting up a UDP socket in UDPChannel.SetupUDPSocket(), which might not be well supported on Mono.
I will make a patch later. Are you using the binaries or the source codes?

Fixed in commit 7ffdb9420d282d6cd4c1abd0eea04b0995157ed9.

Hi, longshine
This bug has been resolved.
I can get the following result.
DEBUG - Starting endpoint bound to <IP_ADDRESS>:5683
CoAP server [RFC 7252] is listening on <IP_ADDRESS>:5683

Even if server.AddEndPoint(IPAddress.Any,server.Config.DefaultPort); is commented out.
Thanks,


Prevent replay attacks with Update proposals

Update proposals are subject to a potential replay attack where:

A proposal is rejected (and thus removed from the set of active proposals) without a subsequent/contrary proposal being accepted (and thus bumping the protocol/application version in question).

This is not an especially harmful replay, since it can only be replayed once every 10,000 slots (the update proposal TTL on mainnet). We could prevent it by storing rejected proposals until a new proposal is accepted, at the cost of carrying some extra state around.

Our tentative consensus is that though this is clearly a design flaw in the original system, it is not a serious security threat because it only allows replay of rejected proposals (& corresponding votes), and only every 10k slots, and only until the end of the Byron era. The Shelley ledger has a comprehensive replay attack prevention mechanism, using the UTxO.
So our tentative plan is to analyse this further to confirm or refute our analysis, and not implement any further mitigation, and live with the possibility of very slow replays of rejected proposals.

Seems reasonable. What’s the time period for 10k slots?


Ensure async disposal threads are quiet before disposing game

PRing this in a haphazard state for comment, and because I've lost most of my sanity while fixing up test issues.
Had this manifest in weird ways:

Collection modified exception inside UpdateTransforms internals
osu! FilterControl having null fields in orders that should be impossible

I'm having issues reproducing now that I'm actually trying, unfortunately. A good path forward would probably be checking my changes here and stating beyond doubt that it is not required as we have enough safety to ensure disposal of drawables can't happen while a load is still in progress. I don't believe it is, and as such resolved components may be ripped out from beneath still-loading-drawables.
Note that this is a very rare occurrence and does not deserve priority attention in review/testing, just wanted to get this PR'd for next time it comes up.

Hasn't happened recently. Most likely fixed in the last 4 years.


Cache bugzilla queries.

Filenames collide:
…
Caching data for: "https://bugzilla.mozilla.org/rest/bug?id=1208038" in "dist/cache/httpsbugzillamozillaorgrestbugid12.json"

Caching data for: "https://bugzilla.mozilla.org/rest/bug?id=1202580" in "dist/cache/httpsbugzillamozillaorgrestbugid12.json"
…

Would it make sense to have the various caches in-memory, as most development is done during watch (except changes in engine/, sorry)?

Would it make sense to have the various caches in-memory, as most development is done during watch (except changes in engine/, sorry)?

We could, I personally don't run watch much though.


cmake does not check for readline.h header package

I installed libreadline-dev package in ubuntu 22.04 to fix the problem but that would be better to catch it during cmake.
[  6%] Building CXX object third-party/abc/CMakeFiles/libabc.dir/src/base/main/mainUtils.c.o
cd /home/titan/OpenROAD/build/third-party/abc && /usr/bin/c++  -I/home/titan/OpenROAD/third-party/abc/src -O3 -DNDEBUG -Wall -Wno-array-bounds -Wno-nonnull -Wno-maybe-uninitialized -Wno-format-overflow -Wno-unused-variable -Wno-unused-function -Wno-write-strings -Wno-sign-compare -Wno-deprecated -Wno-register -Wno-format -DLIN64 -DSIZEOF_VOID_P=8 -DSIZEOF_LONG=8 -DSIZEOF_INT=4 -DABC_NAMESPACE=abc -fpermissive -DABC_USE_CUDD=1 -DABC_USE_READLINE -DABC_USE_PTHREADS -Wno-unused-but-set-variable -std=c++17 -MD -MT third-party/abc/CMakeFiles/libabc.dir/src/base/main/mainUtils.c.o -MF CMakeFiles/libabc.dir/src/base/main/mainUtils.c.o.d -o CMakeFiles/libabc.dir/src/base/main/mainUtils.c.o -c /home/titan/OpenROAD/third-party/abc/src/base/main/mainUtils.c
/home/titan/OpenROAD/third-party/abc/src/base/main/mainUtils.c:25:10: fatal error: readline/readline.h: Aucun fichier ou dossier de ce type
25 | #include <readline/readline.h>
|          ^~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make[2]: *** [third-party/abc/CMakeFiles/libabc.dir/build.make:2109 : third-party/abc/CMakeFiles/libabc.dir/src/base/main/mainUtils.c.o] Erreur 1
make[2] : on quitte le répertoire « /home/titan/OpenROAD/build »
make[1]: *** [CMakeFiles/Makefile2:1134 : third-party/abc/CMakeFiles/libabc.dir/all] Erreur 2
make[1] : on quitte le répertoire « /home/titan/OpenROAD/build »
make: *** [Makefile:139 : all] Erreur 2

@titan73 the issue was with the third-party ABC submodule. We have disabled the use of readline for ABC (since we use the C++ API) as per their documentation. If the problem persists, please file an issue with the upstream repo. Thanks.

Morover defining the variable must apparently be done before "add_subdirectory(abc)".
This works:
if (NOT USE_SYSTEM_ABC)
set(ABC_USE_NAMESPACE "abc")
abc does not have "platform" support for macos so force it to use stdint.h
if (CMAKE_OSX_SYSROOT)
set(ABC_USE_STDINT_H 1)
endif()
set(READLINE_FOUND FALSE)
add_subdirectory(abc)
endif()

Morover defining the variable must apparently be done before "add_subdirectory(abc)".

Can you try on your system and if it works, please file a PR.

Is the upstream abc repo "https://github.com/berkeley-abc/abc"?

Yes. If there are changes upstream to be added to OpenROAD, please file an issue.

Yep sure. Will do.


Term change suggestion SDG2.b - [importbeskyttelse]

SDG: 2
Target & phrase: 2.b, python
Suggested term/change: Consider importbeskyttelse / verdikjeden
Reason: "Verdikjeden for mat - importbeskyttelse eller konkurranse?"

I am not sure that these terms are really covered by the target interpretation, not yet implemented.


changes dynamic nav links to route to an absolute path

Fixes #39

thanks, does this also work when pushState is false?

Thanks @theogravity, it sure doesn't. I'll patch it up, should be a quick fix. Just have to check for window.config.hashbang.

npm username: joefraley
I'm not sure how you feel about depending on the window.config variable here instead of passing it down as props. It feels like the sort of variable that should be set as state, but this check for window.config.hashbang is consistent with other global expectations in the file.

i'm indifferent - 🚢
see for merge instructions - or ask @bendman  since you seem to be working with him(?)
https://github.com/theogravity/react-styleguide-generator-alt/pull/35

Yup, we work together at Idealist.org 😄

Nice! looks like a good alternative to linkedin jobs for non-tech

Have you explored getting this to work in the 2x version?

not really mainly because it works for us on our end with the current 2x ver


Problems creating an S3 database service on OM 0.13.0

Affected module
Backend
Describe the bug
Unable to connect to a non-AWS S3 service on OM 0.13.0. This problem did not exist in previous versions of OM.
I get the following error: https://files.slack.com/files-pri/T02BVTLN3G8-F04E8HQCEFP/image.png
This error suggests that OM ignores the value entered into Endpoint URL.
According to @TeddyCr , OpenMetadata never passes the endpoint URL when creating the client: https://github.com/open-metadata/OpenMetadata/blob/601b9cd38ae24088e68f0217f582f96f58b737d5/ingestion/src/metadata/clients/aws_client.py#L49-L69
To Reproduce
Try to create any Datalake database service pointing to an non-AWS S3 service.
Version:

OpenMetadata version: 0.13.0

Thanks for the details. This is the same issue reported at #7841 & #6682.
As this one has some more details, I'll keep this open and close the duplicates.
Thanks


NPM install requires more dependencies

When building on a completely empty ubuntu install, I have to run these commands as well to get the dependencies working:
# apt-get install build-essential
# apt-get install python
# apt-get install libpq-dev

Otherwise node-gyp does not compile. Maybe add to readme?

Hi,
Thanks for reporting. I think you are referring to Node.js installation. I have just followed the official installation instructions on a fresh system and it works out of the box.
Can you please give me more details?

Hi Faruk,
I actually used the same method, I guess it should be enough to use install the build tools and was quite puzzled I needed python for some node dependency (sometimes this is preinstalled).
Anyway, maybe a mention that native addons are required and that you should install also the optional part is enough.

Added info to README. Closing the issue.


When with literal object argument not working

Whenever i call "when" with primitive type argument it gets work without any problems and the function return the desired value inside the test.
But when calling "when" with argument of reference type variables like literal object or a Map the function doesn't get mocked and it doesn't return the desired value from when.
 // it works fine and the function resolves true inside my test
when(browserNode.getStateChannel('some string')).thenResolve(true);

 // it doesn't work and the function doesn't resolve true inside my test
when(browserNode.getStateChannel({ channelAddress: 'some channel address'})).thenResolve(true);

is there any way to get "when" mocking works with literal objects arguments like above!

TLDR: You can use deepEqual or objectContaining i.e.
when(something.method(deepEqual({ ... })).thenResolve(true)

Without knowing the internals of ts-mockito this is likely due to the fact that object equality is checked by reference in JS. Being able to make sure a method is called with or returns an exact instance of an object is useful. For example (perhaps a bit contrived) consider an implementation of the identity function. You'd most likely not want the following implementation to pass.
const identity = x => ({...x})

Otherwise you'd end up with weird behavior like
const obj = { prop: 5 }
identity(obj) === obj // returns false

The following test would fail with this implementation of identity
const obj = { prop: 5 }
expect(identity(obj)).to.equal(obj) // fails

That being said in most situations like the one you mention, you probably don't care whether the object passed as argument matches exactly by reference. The actual behavior we want in most cases is probably closer to deepEqual or objectContaining but these matchers may also lead to unexpected behavior. Let's say you have some procedural style code and your client is supposed to call a method modifyAndReturnTrueOnSucess that modifies the object given as argument and you test looks like this
...
when(mock.modifyAndReturnTrueOnSucess(deepEqual(obj)).thenReturn(true))
...

but your client looks like this (it calls modifyAndReturnTrueOnSucess with a copy of the object)
...
thing.modifyAndReturnTrueOnSucess(copy(obj))
...
// at this point the code may assume obj was modified even though it was not - a copy of the object was

Your test may still pass even though modifyAndReturnTrueOnSucess was not called with the exact instance of the object that was supposed to be modified. This is because deepEqual essentially matches the argument by value.

Thanks @EmilEriksen! Would be great to have deepEqual in the README docs! 🙂

Looks like a PR has already been created #94. @NagRock could we get this merged in please?

Thanks guys, Can you merge the PR please so i can close this one as well


Script failures

there should be a script command similar to corral_set to report an error and cancel the creation of a corral.

Corral should check command exit codes, in addition to this.

In addition to the command we should also be checking exit codes on command

@paynejacob I'm thinking something like corral_exit code=1 message="some message", defaulting to 1 and script exited with code <code>, thoughts?

I think we can rely on the scripts exit codes the I was thinking of the corral_exit command as more of a log.Fatal.  It allows you to send a message to the user.


fix: refactor approach for px-to-rem conversions

This PR suggests an approach to pxToRem that would solve the following problems we are currently struggling with:

ensure that there is no 'race condition' when global font size styles are applied (styles that change default font size for the page)
allow each theme to define the value that should be used as a base one for pxToRem calculations - as each theme might be opinionated about this value (e.g. Semantic is built with base value of 10px being considered, Teams - with value of 14px)
eliminate necessity for theme to change global base font size value - as this loaded theme might have its own "opinion" about what the base size should be for these conversions, and with two or more themes trying to fight for this global font size value the unexpected rendering results might happen (like the ones that we see now for Semantic and Teams themes being loaded on the same page)

I've checked Button docs page, we run pxToRem() there 1313 times. We can reach up to 5x/27x boost with memoization, a little perf test that confirms this: https://jsperf.com/stardust-ui-pxtorem
This is out of scope of this PR, just a note.

@layershifter, absolutely agree - actually, this topic was discussed long ago, essentially the same strategy has been suggested (momoise data that was used as part of styles calculation - there are lots of opportunities for us to utilize cache system). The reason it wasn't implemented is that we've decided to reach correct and flexible implementation first (which should be proved by real app's usage), and after that optimise.

@kuzhelov seems we have a visual regressions, please check them before merge.


solve issue running redis_setup.sh

fixes #80
Checklist

my branch is upto date with upstream/develop branch
Everything works and tested for Python 3.5.2 and above

Description
While running the shell file redis_setup.sh to download docker and other requirements. It gave an error and do not download the docker even when if docker was not installed. It was due to an error in argument of if-else statement . So if statement is never executed and always else is executed.
Change logs
fix error in if-else statement in redis_setup.sh

the repository shall use docker-compose to run. This script has to be removed.


Ownership and maintenance of the aws-naip s3 bucket

Unlike the landsat-pds bucket and other AWS Public Datasets, there is little information online about aws-naip. Is it an official bucket? Who owns it?

@robert-claypool A few of us at AWS maintain s3://aws-naip for our own internal testing and demo purposes. We're happy to let others use it but don't make any guarantees about its availability or the data in it.

@jedsundwall I am trying to use these naips for this project: https://github.com/trailbehind/DeepOSM
I am able to download one NAIP, but I can't figure out how to list the files of the bucket with boto or the aws CLI. Any pointers on how to enumerate the files?

@andrewljohnson I have not tested this, but apparently you can mount the bucket using danilop/yas3fs. See https://gist.github.com/mwkorver/1ef45abac3871360f2b1
You are right that object listing is disabled; compare to landsat-pds...
$ aws s3 ls s3://aws-naip
A client error (AccessDenied) occurred when calling the ListObjects operation: Access Denied

$ aws s3 ls s3://landsat-pds
                           PRE L8/
                           PRE landsat-pds_stats/
2015-01-28 12:13:53      23764 index.html

@jedsundwall please grant List permissions on this bucket for Everyone if possible.

@robert-claypool @jedsundwall FWIW, I was able to enumerate the files with s3cmd 1.6.0, but not aws CLI or boto. It'd be cool if it worked in boto so I don't have to subprocess s3cmd.
s3cmd ls --recursive --skip-existing s3://aws-naip/md/2013/1m/rgbir/ --requester-pays


Fetch api

1- Use the following external API: API Ninjas
2- Create a new component for displaying a quote
3- Fetch data inside of a new component
4- Show a loading state
5- Show an error state

Hi @DeliceLydia
Thank you so much for taking the time to review my code.


The launcher cannot open the game

Describe the bug
Click the start game button, but cannot open the game.
System Information :

macOS: [ 15.2 ]
Device: [MacbookPro 14inch 2021 ]
CPU: [Apple M1]
RAM: [16G ]
Wine Distribution: [9.2 DXVK]

Logs (if any)
2024-12-19 00:05:30.203 wine-preloader[5375:499984] +[IMKClient subclass]: chose IMKClient_Modern
10441.953:0114:err:kerberos:kerberos_LsaApInitializePackage no Kerberos support, expect problems
10443.193:0114:err:winediag:ntlm_check_version ntlm_auth was not found. Make sure that ntlm_auth >= 3.0.25 is in your path. Usually, you can find it in the winbind package of your distribution.
10443.193:0114:err:ntlm:ntlm_LsaApInitializePackage no NTLM support, expect problems
10448.504:0154:err:module:import_dll Library WDFLDR.SYS (which is needed by L"C:\windows\system32\HoYoKProtect.sys") not found
10448.507:0154:err:ntoskrnl:ZwLoadDriver failed to create driver L"\Registry\Machine\System\CurrentControlSet\Services\HoYoProtect": c0000142

Double check if you installed correctly by reading the README. Also, switch wine distribution to DXMT. It says experimental but it is perfect for Genshin. Keep in mind that it does not have shader cache so it might look laggy every time you open the game, but after a while it will run smoothly.


Replicate MST (2018) and MT (2018)

Hi @a-torgovitsky, this is ready for your review.
Let me know if you have any questions or thoughts.
The goal of this PR, together with a-torgovitsky/MarginalTreatmentEffectsWithMultipleInstruments.jl#1, is to extend this package to allow for the replication of:

"Using Instrumental Variables for Inference About Policy Relevant Treatment Parameters"
Mogstad, Santos, and Torgovitsky (2018, Econometrica)
"Identification and Extrapolation of Causal Effects with Instrumental Variables"
Mogstad and Torgovitsky (2018, Annual Review of Economics)

The organization of this project is largely inspired by MarginalTreatmentEffectsWithMultipleTreatmentEffects.jl.
I am looking forward to hearing your feedback about everything, including (but certainly not limited to):

documentation and comments

The more-involved functions (i.e., the ones in plotting-main.jl) uses Julia's documentation system.
The rest use comments before the function.

style / formatting of code
README instructions
organization of functions in src/
opportunities to avoid duplication

Producing the Plots
@a-torgovitsky, here are the steps to reproduce the figures in both the projects:

Clone the develop branch of this repo to your computer.git clone https://github.com/a-torgovitsky/MarginalTreatmentEffects.jl.git
git checkout develop

Clone the develop branch from my fork of MarginalTreatmentEffectsWithMultipleTreatmentEffects.jl.git clone https://github.com/omkarakatta/MarginalTreatmentEffectsWithMultipleInstruments.jl.git
git checkout develop

Install/load the dependencies as described in the README. Note that you will need to use the local version of MarginalTreatmentEffectsWithMultipleTreatmentEffects.jl that you obtained from the previous step.cd MarginalTreatmentEffects.jl
using Pkg
Pkg.activate(".")
Pkg.rm("MarginalTreatmentEffectsWithMultipleTreatmentEffects")
Pkg.add("/path/to/my/fork/of/MarginalTreatmentEffectsWithMultipleTreatmentEffects.jl")

Produce the plots as described in the README.using MarginalTreatmentEffects
menu("/path/to/where/you/want/to/save", compile = true)

Check the plots to see if everything looks alright. Please let me know if you spot any inconsistencies between these plots and the original.

Structure
Here is the source code in reverse-dependency order, i.e., file 1 depends on file 2, so on and so forth.

toplevel.jl -- user-facing function
mst2018econometrica.jl -- functions that produce the figures for MST (2018)
mt2018review.jl -- functions that produce the figures for MT (2018)
plotting-main.jl -- functions used produce the tex files used to create the TikZ plots
plotting-aux.jl -- auxiliary functions used by plotting-main.jl
utils.jl -- utility functions for communicating with file system
weights.jl -- functions to compute the average weights for target parameters and IV-like estimands
MarginalTreatmentEffects.jl -- module

The TikZ templates can be found in the tex/ folder.
They follow similar naming conventions to the original templates from the Matlab/Python code.
These templates rely heavily on the iteration feature of Mustache.jl, which is why they are much shorter and more general than the original templates.
Possible typos in the original plots
I didn't fix any typos that exist in the original plots to make it easier to compare.
Do you want me to go ahead and fix these mistakes?
Let me know if these actually aren't mistakes:

The labels for the left y-axis don't mention that these weights are averaged across the instruments. We had a discussion about this early on. Should I add the word Average to these labels?
In the legend of Figures 4, 5, 6, and 7 of MST (2018), the support of Z is not used (e.g., 3 is not in the support of Z). Instead, the indices of the support are used. Should I fix this? For example, I would change of 𝟙[Z = 1] to 𝟙[Z = 0].
In Figure 7 of MST (2018), the title suggests we are using polynomials of order 9. However, we are using polynomials of degree 9 (which means the order is 10). Should I change the title so it reads "9th degree polynomial bounds"? Or perhaps "Degree 10 polynomial bounds"?
In Figure 3 of MT (2018), the top label reads $p(1) - p(2)$. Should I change it to $p(2) - p(1)$?

Questions
Within the code, I start certain comments with the following tags.

FIX -- these point to mistakes in the original figures that I didn't fix in the reproductions. Should I fix them? (I list them above.)
Q(a-torgovitsky) -- these are questions I have for you. Could you take a look at them?
TODO(omkarakatta) -- these are todo's that I have for myself. They are primarily notes to myself to consider possible refactoring.

Aesthetics
I am pretty sure I matched the aesthetics of the Econometrica paper.
However, the figures in the Review paper use a different style than the typical TikZ figures.
Do you want me to try and replicate the formatting used in the Review paper?
If so, some of the changes I'd need to explore are:

fonts: The Review paper uses a sans serif font for the plots.
colors: My version and your original version uses green and teal, which looks similar to one another in my opinion. For example, see Figure 9 and compare the IV slope weights and the Wald weights.
line width: The Review paper uses thinner lines so the space between the weights/curves is more prominent
tick marks, e.g., the stars for the horizontal line at the ATT in Figure 6

For a future PR
As I wrote this package, I wondered if it made sense to merge this package with MarginalTreatmentEffectsWithMultipleInstruments.jl.
This removes one dependency from this package, and the infrastructure for this package makes it easy to combine the two packages.
If I envision what I need to do correctly, I just need to copy-paste files from the original package to this one.
Do you think it's worthwhile to merge the two?
Or, would you prefer to have them separate.
To-do

Once  a-torgovitsky/MarginalTreatmentEffectsWithMultipleInstruments.jl#1 is merged, I will modify Project.toml to use your package instead of my local fork.
I tried warm-starting the programs to encourage the solvers to choose the desired maximizing MTRs. However, nothing seemed to change. For now, I am adding additional constraints to achieve the desired maximizing MTRs (see Figure 4 of Econometrica paper). However, I personally prefer warm-starting so I'll try looking into this further.

Ok sounds good, an hour isn't long at all and perhaps this will be useful
to someone in the future
On Fri, Jul 15, 2022 at 2:43 PM Omkar A. Katta @.***>
wrote:

Maybe, what's the cost?
For the simple tests I have in mind, the cost is an hour of my time at
most.
I have several assert statements in a scrap directory where I was playing
around with the code.
(I put the directory in .gitignore so it won't be visible. Most of the
development took place in this directory actually.)
All I'd need to do is modify the syntax to fit with Julia's testing
framework.
These assert statements are quite simple and straightforward:
results = compute_bounds(...)
true_lb = 0 # the true bounds were taken from the figure titles in the papers
true_ub = @.*** round(results[:lb], digits = 4) == @.*** round(results[:lb], digits = 4) == true_ub
I did this primarily for the same reason you added tests to the original
package:
I only put in the unit tests that I did because I was extending the code,
so wanted to make sure I didn't make any "breaking" changes.
—
Reply to this email directly, view it on GitHub
https://github.com/a-torgovitsky/MarginalTreatmentEffects.jl/pull/1#issuecomment-1185852895,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEATCRAAZ2NH2AMLLVXY45LVUG5PVANCNFSM53HNIMLA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Whoah...weird, so I guess I had remembered replying correctly :).
No I didn't send anything new.
I think this is a cautionary lesson about using email-reply to GitHub issues...


feat(firebase): support renaming deployed firebase functions

🔗 Linked issue

#266
❓ Type of change

[x] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[x] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

Adds a new option to nitro config that allows giving a custom name to deployed firebase functions.
This is done using rollup replace. I changed the exported name for the server in the firebase entry to __firebaseServerFunctionName__ to ensure it is unique to the firebase server export. The placeholder name is then replaced with firebase.serverFunctionName in the config, which is given a default value of "server".

This change gives an intuitive way to deploy multiple servers to the same firebase project without having them be overwritten. The workaround given in #266 is not very intuitive, and could potentially break other deployments in the future.
I would also recommend adding the same option to V2 functions in #1142, since there doesn't appear to be a way to change the deployed function name in the V2 HttpOptions (I don't think labels does this, but correct me if I am wrong)

Resolves #266
📝 Checklist

[x] I have linked an issue or discussion.
[x] I have updated the documentation accordingly.

I have tested building my own project and the function renaming appears to work as expected. The only difference is that export { s as server } from './chunks/nitro/firebase.mjs' becomes export { _ as server } from './chunks/nitro/firebase.mjs' since the export now starts with _ before the placeholder gets replaced. I don't think this will cause any issues, but I am not 100% sure.

This is looking good thanks ! Let's wait until https://github.com/unjs/nitro/pull/1142 is merged before moving forward with that one.


Confusion about license for PropertyGridViewModel.cs

The LICENSE.md file in the project's root very licenses the code under MIT, yet the file MessageCommunicator.TestGui/_Util/_View/_PropertyGrid/PropertyGridViewModel.cs has a dedicated licensing header mentioning GNU Lesser General Public License. It looks like that file was just copied from SeeingSharp2. Is the ambiguous licensing situation intended for that file? It makes re-using the PropertyGrid code as described in your blog post a bit awkward.

Oh, yes. This is a copy-paste error. Thank you for opening this issue.
I removed the license header from PropertyGridViewModel.cs with commit 01e9be378b8ca4aef11648bc8b14875f45b2e6de


Improve Generator Fidelity

This commit works to reduce the differences between the data supported by CCDA parsing vs generation and to enumerate the differences that do exist. It fixes a few accuracy errors in the parser, adds support for additional data to the generator, and introduces a test that passes JSON through the generator then the parser and verifies that the JSON comes out (close to) unchanged. In the process, it also deletes the AMD tests, which have never seemed to fail in a unique way and are annoying to keep in sync.

@thetylerhayes Want to give this a look before merging? Inspired by #125

Going to go ahead and merge this one


chore: upgrade to redis 6

Signed-off-by: John Pitman<EMAIL_ADDRESS>What type of PR is this?
/kind enhancement
What does this PR do / why we need it:
Upgrades the operator for Redis 6.  The change to actually pull in the Redis 6 images was made in the argocd-operator https://github.com/argoproj-labs/argocd-operator/pull/517, and the gitops-operator has already been updated to pull in that change.  This PR updates the script and config templates used when running in HA mode.
Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes #?
Test acceptance criteria:

[ ] Unit Test
[ ] E2E Test

How to test changes / Special notes to the reviewer:

Install an Argo CD instance with HA enabled.

apiVersion: argoproj.io/v1alpha1
kind: ArgoCD
metadata:
  name: argocd
spec:
  ha:
    enabled: true

Ensure all the pods start.  It may take many minutes for the redis-ha-server pods to start

% kubectl get po
NAME                                       READY   STATUS    RESTARTS   AGE
argocd-application-controller-0            1/1     Running   0          22m
argocd-dex-server-7bc45bf5b6-bht2c         1/1     Running   0          22m
argocd-redis-ha-haproxy-6d474497f7-cgvd4   1/1     Running   0          22m
argocd-redis-ha-server-0                   2/2     Running   0          22m
argocd-redis-ha-server-1                   2/2     Running   0          14m
argocd-redis-ha-server-2                   2/2     Running   0          13m
argocd-repo-server-96c5789dd-jfqct         1/1     Running   0          22m
argocd-server-5c79fbf448-4kgs9             1/1     Running   0          22m

/retest

/lgtm

/approve


[bug]: Overlapping with Table components shadcn

Describe the bug
I run into a issue that the dropdown section overlapping with the shadcn table

Affected component/components
CommandList
How to reproduce

Create a multi selector and a table below. 2. Will see the command clash with the table

Codesandbox/StackBlitz link
No response
Logs
No response
System Info
No response
Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

add z-index for the cmdk dropdown


Enhanced APP.js And APP.css

The renderCell function now includes a key prop, a best practice when rendering lists in React.
Winner Message:
The winner message now accounts for a draw, enhancing user feedback.
A soft background color (#E3F2FD) creates a calming environment.
The main container (.app) has a white background with rounded corners and a subtle shadow for a card-like appearance.
Grid Layout:
The game board is organized into a grid using CSS Grid. This ensures that the cells are evenly spaced and responsive.
Cell Styling:
Cells have a soft background, rounded corners, and a border that changes on hover. This enhances interactivity.
A scaling effect on hover gives a responsive feel to the cells.
Winner Message:
The winner message is emphasized with a larger font size and color, making it clear to the player.
Buttons:
The restart button is styled with a matching color scheme and a hover effect that darkens the button, enhancing user experience.
Typography:
The Poppins font is used for a clean, modern look.

The renderCell function now includes a key prop, a best practice when rendering lists in React.
Winner Message: The winner message now accounts for a draw, enhancing user feedback.
A soft background color (#E3F2FD) creates a calming environment.
The main container (.app) has a white background with rounded corners and a subtle shadow for a card-like appearance.
Grid Layout: The game board is organized into a grid using CSS Grid. This ensures that the cells are evenly spaced and responsive.
Cell Styling: Cells have a soft background, rounded corners, and a border that changes on hover. This enhances interactivity. A scaling effect on hover gives a responsive feel to the cells.
Winner Message: The winner message is emphasized with a larger font size and color, making it clear to the player.
Buttons: The restart button is styled with a matching color scheme and a hover effect that darkens the button, enhancing user experience.
Typography: The Poppins font is used for a clean, modern look.

Go ahead  , please put ss of before and after pr

The renderCell function now includes a key prop, a best practice when rendering lists in React.
Winner Message: The winner message now accounts for a draw, enhancing user feedback.
A soft background color (#E3F2FD) creates a calming environment.
The main container (.app) has a white background with rounded corners and a subtle shadow for a card-like appearance.
Grid Layout: The game board is organized into a grid using CSS Grid. This ensures that the cells are evenly spaced and responsive.
Cell Styling: Cells have a soft background, rounded corners, and a border that changes on hover. This enhances interactivity. A scaling effect on hover gives a responsive feel to the cells.
Winner Message: The winner message is emphasized with a larger font size and color, making it clear to the player.
Buttons: The restart button is styled with a matching color scheme and a hover effect that darkens the button, enhancing user experience.
Typography: The Poppins font is used for a clean, modern look.

Go ahead , please put ss of before and after pr

please fork new updated repo and make changes in it and put ss before pr so i can verify


RenderListDataAsStream & RenderExtendedListFormData suddenly failing due to Serialization Error

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Developer environment
Windows
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[X] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

browser version Version 114.0.5735.199 (Official Build) (64-bit)
SPFx version 1.12.1
Node.js version 14.15.5

Describe the bug / error
We are using 'RenderListDataAsStream' and 'RenderExtendedListFormData' from SharePoint REST API for our extensions, for allowing users to edit and create list items.
Starting last week we have encountered that these endpoints return the following error:
{
    "error": {
        "code": "-1, System.InvalidOperationException",
        "message": {
            "lang": "en-US",
            "value": "Error during serialization or deserialization using the JSON JavaScriptSerializer. The length of the string exceeds the value set on the maxJsonLength property."
        }
    }
}

This happens for lists where we have 7 - 9 lookup fields configured so we should be well below the threshold for lookups (12 for SharePoint Online).
Bear in mind that these lists settings have not been changed for at least a year.
It is happening in three different tenants, but could be more due to holiday inactivity.
The following settings give us the error:

A basic 'RenderExtendedListFormData' call, irregardless of options
'RenderListDataAsStream' with the following options: ClientFormSchema = 64

We of course validated our findings by comparing the built-in SharePoint edit form (Modern Experience) which also fails:

Steps to reproduce

Perform a REST call for the following endpoints on a list with 7-9 lookup fields:
A basic 'RenderExtendedListFormData' call, irregardless of options
A 'RenderListDataAsStream' with the following options: ClientFormSchema = 64

Expected behavior
We expect the endpoints to return data as they did before they started failing end of last week

@MortenGuldbaek - could you please specify where you're performing the REST request?
The screenshot doesn't look like SPFx application but it is marked as SPFx in the dev model.

Sure thing.
The provided screenshot was just the built-in edit list item form in the modern experience, to show that it was not only our customizations that were failing.
We have a list view command set that enables editing list items in a React form where we have custom a validation flow.
This relies on 'RenderListDataAsStream', using the SPHttpClient from the SPFx list view context.
We got reports that this was failing late last week and could replicate on our own tenants this week.
Checking this morning I can see that our solution is now working as expected again, with no change from our side, but I'm still waiting for confirmation from our partners if they are experiencing the same.

We have confirmations from our partners that they do not encounter the bug anymore, so that's great.
Though I'm a bit hesitant on celebrating too early, not knowing what causes the issue.
@AJIXuMuK do you know what could have impacted the API?


fix: Updated ConfirmArtworkModal deps to system instead of artsy

Updated deps for ConfirmArtworkModal to use System instead of Artsy as per this comment.

🙏


Add Rust implementation for The Barnsley Fern

I've based my code on #755 to make things easier and more coherent.
If this gets approved, I would suggest waiting until that PR gets resolved before applying mine. As it would have the same constraints about Cargo.toml.

Without testing if the code is correct (which i'll assume for now and test later) the code itself looks good to me 👍

2 important things to note here:

As per #691, we are thinking about adding build instructions for certain languages. This just means that rust should probably have a Cargo.toml to go with it so people can just run cargo run.
The code produces the following result:

It should produce:

I noticed that the operators were a little off, but maybe there is a bigger issue here because julia is column-major while rust is row-major? I did not dig too much deeper into the code, but I can in a bit.

So I've added a Cargo.toml by creating a project with cargo new rust inside the code directory. Let me know what you think.
I've changed the constants, and I'll have a look at what's wrong with my generation.

I've found the bug. It was a silly one :grimacing: :sweat_smile:
You were right about the constants being off, but the issue was with the matrix multiplication.
Basically this was the issue:
p.x = rhs[0].x * self.x + rhs[0].y * self.y + rhs[0].z * self.z;
p.y = rhs[1].x * self.x + rhs[1].y * self.y + rhs[1].z * self.z;
p.z = rhs[2].x * self.x + rhs[2].y * self.y + rhs[2].z * self.z;

because p.x gets updated fine, but then p.y gets updated with a new p.x and old p.y and p.z. I guess this is why some things still sort of worked by coincidence.
I've moved the code to a method in Point3 called matrix_mul but it could also have been solved by doing
let x = rhs[0].x * self.x + rhs[0].y * self.y + rhs[0].z * self.z;
let y = rhs[1].x * self.x + rhs[1].y * self.y + rhs[1].z * self.z;
let z = rhs[2].x * self.x + rhs[2].y * self.y + rhs[2].z * self.z;
p.x = x;
p.y = y;
p.z = z;

[lang: rust]


fix(vuex-store): implement shared store for navigation and stored objects

We have changed the design of the vuex-store. The navigation and stored objects are now stored
separately for each used language. This allows to change the language without throwing away the
already saved data for the former used language. Besides of that, there is now a possibility to use the
browsers back button after a language switch. For that a query param is added to the url when
triggering a route change but before the new url is visible.
additional information:
This PR is based on the alpha version, since in the alpha version are a lot of changes in the vuex file and this PR also includes a lot of changes in the vuex file. I hope that we can avoid unpleasant merge conflicts with this approach.
known Bugs:

when opening the site with a route that is not in the navigationData of the defaultLocale, it won't be displayed, because the wrong navigationData has been loaded.

possible improvements:

instead of the query parameter, you could use the navigation-service to ask if this route exists and which language it can be assigned to.

the changes are outdated


@alexbelgium I'm not sure if this is the right place to ask, but it feels like it might be. I followed these two YouTube videos (Vid-1 and Vid-2) to set up my external hard drive as a mount and Nextcloud. It worked until about two or three weeks ago. When the power goes out and comes back, my Home Assistant system won't start. I think it's stuck because Nextcloud can't start on time, which prevents Home Assistant from booting up. To fix this, I have to remove the hard drive from the thin client, start Home Assistant, reconnect the USB, and restart Nextcloud. This process is too time-consuming and power outages are common here. Any help or workaround would be greatly appreciated.

          @alexbelgium I'm not sure if this is the right place to ask, but it feels like it might be. I followed these two YouTube videos ([Vid-1](https://www.youtube.com/watch?v=CHvBzX41-Ag) and [Vid-2](https://www.youtube.com/watch?v=NHQ5KJf0IiE)) to set up my external hard drive as a mount and Nextcloud. It worked until about two or three weeks ago. When the power goes out and comes back, my Home Assistant system won't start. I think it's stuck because Nextcloud can't start on time, which prevents Home Assistant from booting up. To fix this, I have to remove the hard drive from the thin client, start Home Assistant, reconnect the USB, and restart Nextcloud. This process is too time-consuming and power outages are common here. Any help or workaround would be greatly appreciated.

I'm using:
OS Version: Home Assistant OS 12.3
Home Assistant Core: 2024.6.3
Nextcloud: Current version 29.0.2-2
It just stuck here for long:
Updating permissions...
... Config directory : /data
... data directory detected : /media/NEXTCLOUD/nextcloud
... setting permissions
Originally posted by @iGaneshKumar in https://github.com/alexbelgium/hassio-addons/issues/1429#issuecomment-2175547769

Mmh sorry you're not using my own hdd mount system so not sure how I can really help... I have my own system to mount hdd drives where you put in the localdisk the disk name (physical such as "sda1" or label sur as "NAS") and then it mounts it to /mnt/sda1 or /mnt/NAS.
What it is doing is going at this stage is go through all your Nextcloud files and adapt permissions, so it can be normal that it takes so long ?
On the other hand if the issue is "only" that Nextcloud start too fast ; you could therefore disable the start at boot and create an automation instead ?

The addon is already configured to start after boot so I can't do anything from my side

Hi, here is a very simple one https://github.com/alexbelgium/hassio-addons/wiki/Add-ons-feature-:-mounting-drives

This can be closed. I'll address it when I have time. For now, I've set up an automation. Thanks for your help!


Support coming for Amazon Elasticsearch 5.x Service?

Hi,
AWS just announced support for Elasticsearch 5 in Amazon Elasticsearch Service
https://aws.amazon.com/about-aws/whats-new/2017/01/elasticsearch-5-now-available-on-amazon-elasticsearch-service/
Are you planning to make this connector support the latest version? If so roughly when?
BR
Masi

The elastic2-doc-manager should be able to handle Amazon Elasticsearch 5.x Service. Simply install  mongo-connector first and then install the doc manager with the Elasticsearch 5.x client and the AWS dependencies:
$ pip install mongo-connector 'elastic2-doc-manager[elastic5,aws]'

Then follow the instructions for using Elasticsearch on AWS. Your config file should look something like:
"docManagers": [
    {
      "docManager": "elastic2_doc_manager",
      "targetURL": "https://something.us-east-1.es.amazonaws.com/",
      "args": {
        "aws": {
            "region_name": "us-east-1",
            "aws_access_key_id": "ACCESS_ID",
            "aws_secret_access_key": "SECRET_KEY"
        }
      }
    }
  ]

Let me know if you run into any issues.

Thanks! Seems to start running - however I stumble upon request size limits on ES Service's side (http://docs.aws.amazon.com/elasticsearch-service/latest/developerguide/aes-limits.html).
Is it possible to tweak the batch sizes for the initial dump? It's very easy to hit those limits (10/100Mb) with the initial dump. Does the "batchSize" config affect also this way or does it only apply to how frequently the timestamp file is updated?

The bulk size defaults to 1000 documents. Try lowering it via the bulkSize option, see the above config as an example of setting it to 500.
A future release of the elastic-doc-managers could support setting the max_chunk_bytes parameter for the streaming_bulk function. That would allow setting the max size to 10 or 100 Mb without worrying about the number of documents.

What's the status with this? I think the readme needs updated, as
$ pip install mongo-connector 'elastic2-doc-manager[elastic5,aws]'

warns that mongo-connector 2.5.1 does not provide the extra 'aws'. It only looks like it's possible to do Elastic 2.x with AWS for now?

I don't see any warnings:
pip install mongo-connector 'elastic2-doc-manager[elastic5,aws]'
Collecting mongo-connector
  Downloading mongo_connector-2.5.1-py2.py3-none-any.whl (58kB)
    100% |████████████████████████████████| 61kB 1.8MB/s
Collecting elastic2-doc-manager[aws,elastic5]
  Downloading elastic2_doc_manager-0.3.0-py2.py3-none-any.whl
Collecting pymongo>=2.9 (from mongo-connector)
  Downloading pymongo-3.5.1-cp27-none-macosx_10_12_x86_64.whl (270kB)
    100% |████████████████████████████████| 276kB 2.7MB/s
Collecting boto3>=1.4.0; extra == "aws" (from elastic2-doc-manager[aws,elastic5])
  Downloading boto3-1.4.7-py2.py3-none-any.whl (128kB)
    100% |████████████████████████████████| 133kB 7.4MB/s
Collecting requests-aws-sign>=0.1.2; extra == "aws" (from elastic2-doc-manager[aws,elastic5])
  Downloading requests_aws_sign-0.1.4-py2.py3-none-any.whl
Collecting elasticsearch<6.0.0,>=5.0.0; extra == "elastic5" (from elastic2-doc-manager[aws,elastic5])
  Downloading elasticsearch-5.4.0-py2.py3-none-any.whl (58kB)
    100% |████████████████████████████████| 61kB 3.5MB/s
Collecting s3transfer<0.2.0,>=0.1.10 (from boto3>=1.4.0; extra == "aws"->elastic2-doc-manager[aws,elastic5])
  Using cached s3transfer-0.1.11-py2.py3-none-any.whl
Collecting jmespath<1.0.0,>=0.7.1 (from boto3>=1.4.0; extra == "aws"->elastic2-doc-manager[aws,elastic5])
  Using cached jmespath-0.9.3-py2.py3-none-any.whl
Collecting botocore<1.8.0,>=1.7.0 (from boto3>=1.4.0; extra == "aws"->elastic2-doc-manager[aws,elastic5])
  Downloading botocore-1.7.28-py2.py3-none-any.whl (3.7MB)
    100% |████████████████████████████████| 3.7MB 341kB/s
Collecting requests>=2.0.0 (from requests-aws-sign>=0.1.2; extra == "aws"->elastic2-doc-manager[aws,elastic5])
  Using cached requests-2.18.4-py2.py3-none-any.whl
Collecting urllib3<2.0,>=1.8 (from elasticsearch<6.0.0,>=5.0.0; extra == "elastic5"->elastic2-doc-manager[aws,elastic5])
  Using cached urllib3-1.22-py2.py3-none-any.whl
Collecting futures<4.0.0,>=2.2.0; python_version == "2.6" or python_version == "2.7" (from s3transfer<0.2.0,>=0.1.10->boto3>=1.4.0; extra == "aws"->elastic2-doc-manager[aws,elastic5])
  Using cached futures-3.1.1-py2-none-any.whl
Collecting docutils>=0.10 (from botocore<1.8.0,>=1.7.0->boto3>=1.4.0; extra == "aws"->elastic2-doc-manager[aws,elastic5])
  Using cached docutils-0.14-py2-none-any.whl
Collecting python-dateutil<3.0.0,>=2.1 (from botocore<1.8.0,>=1.7.0->boto3>=1.4.0; extra == "aws"->elastic2-doc-manager[aws,elastic5])
  Using cached python_dateutil-2.6.1-py2.py3-none-any.whl
Collecting certifi>=2017.4.17 (from requests>=2.0.0->requests-aws-sign>=0.1.2; extra == "aws"->elastic2-doc-manager[aws,elastic5])
  Using cached certifi-2<IP_ADDRESS>-py2.py3-none-any.whl
Collecting chardet<3.1.0,>=3.0.2 (from requests>=2.0.0->requests-aws-sign>=0.1.2; extra == "aws"->elastic2-doc-manager[aws,elastic5])
  Using cached chardet-3.0.4-py2.py3-none-any.whl
Collecting idna<2.7,>=2.5 (from requests>=2.0.0->requests-aws-sign>=0.1.2; extra == "aws"->elastic2-doc-manager[aws,elastic5])
  Using cached idna-2.6-py2.py3-none-any.whl
Collecting six>=1.5 (from python-dateutil<3.0.0,>=2.1->botocore<1.8.0,>=1.7.0->boto3>=1.4.0; extra == "aws"->elastic2-doc-manager[aws,elastic5])
  Using cached six-1.11.0-py2.py3-none-any.whl
Installing collected packages: pymongo, mongo-connector, futures, docutils, jmespath, six, python-dateutil, botocore, s3transfer, boto3, certifi, chardet, idna, urllib3, requests, requests-aws-sign, elasticsearch, elastic2-doc-manager
Successfully installed boto3-1.4.7 botocore-1.7.28 certifi-2<IP_ADDRESS> chardet-3.0.4 docutils-0.14 elastic2-doc-manager-0.3.0 elasticsearch-5.4.0 futures-3.1.1 idna-2.6 jmespath-0.9.3 mongo-connector-2.5.1 pymongo-3.5.1 python-dateutil-2.6.1 requests-2.18.4 requests-aws-sign-0.1.4 s3transfer-0.1.11 six-1.11.0 urllib3-1.22

Can you post the full install command with all its output? And run pip freeze to show what packages you have installed?

Here
C:\WINDOWS\system32>pip install mongo-connector[elastic5,aws]
Collecting mongo-connector[aws,elastic5]
  Using cached mongo_connector-2.5.1-py2.py3-none-any.whl
  mongo-connector 2.5.1 does not provide the extra 'aws'
Requirement already satisfied: pymongo>=2.9 in c:\program files\python36\lib\site-packages (from mongo-connector[aws,elastic5])
Collecting elastic2-doc-manager[elastic5]; extra == "elastic5" (from mongo-connector[aws,elastic5])
  Using cached elastic2_doc_manager-0.3.0-py2.py3-none-any.whl
Collecting elasticsearch<6.0.0,>=5.0.0; extra == "elastic5" (from elastic2-doc-manager[elastic5]; extra == "elastic5"->mongo-connector[aws,elastic5])
  Using cached elasticsearch-5.4.0-py2.py3-none-any.whl
Requirement already satisfied: urllib3<2.0,>=1.8 in c:\program files\python36\lib\site-packages (from elasticsearch<6.0.0,>=5.0.0; extra == "elastic5"->elastic2-doc-manager[elastic5]; extra == "elastic5"->mongo-connector[aws,elastic5])
Installing collected packages: elasticsearch, elastic2-doc-manager, mongo-connector
  Found existing installation: elasticsearch 2.4.1
    Uninstalling elasticsearch-2.4.1:
      Successfully uninstalled elasticsearch-2.4.1
Successfully installed elastic2-doc-manager-0.3.0 elasticsearch-5.4.0 mongo-connector-2.5.1

and pip freeze output
awsebcli==3.12.0
boto3==1.4.7
botocore==1.7.28
cement==2.8.2
colorama==0.3.7
docker-py==1.7.2
dockerpty==0.4.1
docopt==0.6.2
docutils==0.14
elastic2-doc-manager==0.3.0
elasticsearch==5.4.0
jmespath==0.9.3
mongo-connector==2.5.1
pathspec==0.5.0
pymongo==3.5.1
python-dateutil==2.6.1
PyYAML==3.12
requests==2.9.1
requests-aws-sign==0.1.4
s3transfer==0.1.11
semantic-version==2.5.0
six==1.11.0
tabulate==0.7.5
termcolor==1.1.0
urllib3==1.22
websocket-client==0.44.0

For now, I'm just doing pip install mongo-connector[elastic2-aws] and using ES 2.3 on AWS which seems to be working fine.

Instead of pip install mongo-connector[elastic5,aws], run pip install elastic2-doc-manager[elastic5,aws]. The later will install mongo-connector with the Elasticsearch doc manager for Elastic 5 and all the extra libraries for authenticating to AWS.

Wow, I can't believe I misread the package name every time. Tired eyes :/
Thanks. So is ES 5.x fully supported? Any reason this issue's being left open?


[6.0] Enable 'BorrowingSwitch' feature.

Staging PR awaiting approval of SE-432.

Please test with following pull request:
https://github.com/apple/swift/pull/72984
@swift-ci Please test

Please test with following pull request:
https://github.com/apple/swift-syntax/pull/2597
@swift-ci Please test

Please test with following pull request:
https://github.com/apple/swift-syntax/pull/2597
@swift-ci Please test

Please test with following pull request:
https://github.com/apple/swift-syntax/pull/2597
@swift-ci Please test

@swift-ci Please test Linux

Please test with following pull request:
https://github.com/apple/swift-syntax/pull/2597
@swift-ci Please test Linux


[pyright updated to 1.1.224] Update Version

Auto-generated by create-pull-request

There you go! Thank you @artificial-aidan!


#1 set version info into LNB using ENV

Related Issue #1.
I (optionally) set version information into bottom of LNB.
Version is loaded from maven property metatron.app.version.
So, you can use command line parameter like

mvn clean install -Dmetatron.app.version=3.0.1

If not set, nothing will be displayed.

I changed reviewer to @eltriny

run build
deploy to 2


colourise oper names in log output

makes a big log scrollback much easier to skim

been running in live for a while now. works well, might revisit the available colours at a later date


Compiler: fix autocasting case for crossed types

Fixes a couple of scenarios with autocasting. It's a big refactor but it's a good one.

@asterite Why did close this? Is it wrong/too hard to fix?

@straight-shoota This might fix some things but break others. I didn't fully think about the algorithm. I'll leave others tackle these difficult problems from now on, hopefully with a more mathematically and proofy way.


[BUG]: drizzle-kit push:sqlite fails to copy table data after adding a column

What version of drizzle-orm are you using?
<EMAIL_ADDRESS>What version of drizzle-kit are you using?
0.19.13
Describe the Bug
After adding a column to a table in schema, and executing drizzle-kit push:sqlite, it fails with error:
SqliteError: table posts has 4 columns but 3 values were supplied
In the database there's now __old_push_posts table, that the original posts table was renamed into, with all the data.
There's also now a new posts table with 4 columns, but no data.
The error message suggests that drizzle-kit fails to copy the data over after the change, because of the different number of columns.
Running push again, warns that it's going to delete the __old_push_posts, but I know that it contains the original data, not copied to the new posts table.
Expected behavior
drizzle-kit should be able to add a new column to a table in an sqlite db, with all the data intact, after adding a column to a table schema.
Environment & setup
Apple M1 Pro, Sonoma, Node v19.7.0

happens also here with turso ^^

In the meanwhile, I'm manually restoring the data from the temporary column with something like
INSERT OR IGNORE INTO `posts` (`id`, `content`, `created`)
SELECT `id`, `content`, `created`
FROM `__old_push_posts`;

drizzle-kit should be doing something like that.

I am also having this error, and its very annoying :(

Facing this issue with Turso. Added extra column to table and get this error
LibsqlError: SQLITE_UNKNOWN: SQLite error: table listings has 13 columns but 12 values were supplied
    at mapHranaError (file:///Users/shrihari/sponsors/node_modules/.pnpm/@libsql+client@0.4.0-pre.2/node_modules/@libsql/client/lib-esm/hrana.js:257:16)
    at HttpClient.execute (file:///Users/shrihari/sponsors/node_modules/.pnpm/@libsql+client@0.4.0-pre.2/node_modules/@libsql/client/lib-esm/http.js:53:19)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async TursoSqlite.run (/Users/shrihari/sponsors/node_modules/.pnpm/drizzle-kit@0.20.6/node_modules/drizzle-kit/bin.cjs:25497:9)
    at async Command.<anonymous> (/Users/shrihari/sponsors/node_modules/.pnpm/drizzle-kit@0.20.6/node_modules/drizzle-kit/bin.cjs:63284:9) {
  code: 'SQLITE_UNKNOWN',
  rawCode: undefined,
  [cause]: [ResponseError: SQLite error: table listings has 13 columns but 12 values were supplied] {
    code: 'SQLITE_UNKNOWN',
    proto: {
      message: 'SQLite error: table listings has 13 columns but 12 values were supplied',
      code: 'SQLITE_UNKNOWN'
    }
  }
}

"drizzle-kit": "^0.20.6",
 "drizzle-orm": "^0.29.0",

is there any workaround until this is fixed?

any updates?

don't understand what's happening. I got this same error after adding two columns to my schema and using drizzle push:sqlite

same problem

After reading the docs, I can see that the push command is not intended to be used with production db's. It is only for local development, where you are expected to be able to delete the database and restart anytime you make a change.

is there any update on this?

any update?

This is issue is still occuring. I am using turso

+1

Also happening here.

Any updates for this issue? Specifically using turso in dev mode. Issue also occurs on tables that have a foreign key constraint. It is currently not possible to add a new table using the drizzle kit generate and migrate command.

Woo still this issue, Suddenly everyone is using Turso,
I just copy paste the generate sql into Turso SQL runner
like this
ALTER TABLE setting ADD enable_warning integer DEFAULT true;

I'm also facing this issue with pglite

should not be an issue on latest(0.30.0+), if it's still an issue - please reopen


Fix the client secret requirement issue

The lastest standard, the client-secret are no longer required in
PasswordGrant and RefreshToken Grant, thus there will be a circumstance
when client_secret and redirect_uri both are null

Great, been waiting for this

Waiting for it!

@saga64 do you have the link to the section in the spec?

@vinkla Check this link https://github.com/thephpleague/oauth2-server/pull/391

In terms of https://tools.ietf.org/html/rfc6749, since v2-16,
PasswordGrant and RefreshGrant are no longer require client secret validation
by default. Pleaseread section 4.3.1 and section 6.

Thanks @whitefoxy, then we'll wait on https://github.com/thephpleague/oauth2-server/pull/391 to get merged. I'll leave this open for now.

Any news on this?

@vinkla **alexbilbie ** has added it to the code. https://github.com/thephpleague/oauth2-server/pull/391

This will be implemented in the next major version https://github.com/thephpleague/oauth2-server/pull/391#issuecomment-185692796


Add 'yarn options' to list cli options.

Summary
yarn options : A nice-to-have only, b/c I always forget what the options are.
$ yarn options
yarn run v1.22.4
$ node -r esm src/shared/cli/cli-args.js -h
Options:
  --version           Show version number                              [boolean]
  --date, -d          Generate data for or start the timeseries at the provided
                      date in YYYY-M-D format                           [string]
  --endDate, -e       The date after which to stop generating timeseries data
                                                                        [string]
  --location, -l      Scrape only the location provided by full name, i.e City,
                      County, State, Country                            [string]
  --country, -c       Scrape only the country provided, like AUS        [string]
  --id, -i            Scrape only the location provided by id           [string]
  --skip, -s          Skip the location provided by full name, i.e City, County,
                      State, Country                                    [string]
  --outputSuffix, -o  The suffix to add to output files, i.e. passing TEST will
                      produce data-TEST.json etc                        [string]
  --quiet, -q         Suppress logs                                    [boolean]
  --findFeatures      Include feature information in output data       [boolean]
  --findPopulations   Include population information in output data    [boolean]
  --writeData         Write to dist folder                             [boolean]
  --onlyUseCache, -x  Only use cache (no http calls)                   [boolean]
  --help, -h          Show help                                        [boolean]
✨  Done in 0.71s.

Docs updated.

Beautiful!


add opencv

Coincidentally I have also been working on this code recently to incorporate face detection using different OpenCV algorithms (Haar Cascade and DNN). I've also updated the code to closer match the current smartcrop.js algorithm. I will submit this as a PR when I get the chance.


Updated Readme

Pull Request
Proposed Changes

Added myself as a contributor in Readme
List the dependencies used.
[ Fixes #1 ].

Types of changes made

[ ] Fixed Bug
[ ] Added new Features
[ ] Edited Documentations
[ ] Enhanced the working of code
[ ] Additional Changes
Mention any Additional Change here

Additional Info
Provide additional information(if any) you want to convey.
Viability of Code
Describe the tests that you ran to verify the changes made. Mention the tests for easy reproducibility.

Test 1
Test 2

Mention(if any) specific Test Configurations used:

Filmware version:
Hardware:
Toolchain:
SDK:

Screenshots or GIFs (if any)
Attach screenshots of the changes made for easy reference.

Original
Updated

Attach_Original_Screenshot
Attach_Updated_Screenshot

Checklist:

[x] My code follows the Mentioned Guidelines for the project
[x] I have self-reviewed the changes made
[ ] I have made the changes easy to understand by adding comments
[x] My changes does not generate new warnings/errors
[ ] I have attached relevant screenshots to display the changes made
[x] I have made the required changes to the documentations

Hey @Jyotika999 we are using the all-contributors bot for adding contributors :sweat_smile: can you add any other small change to the readme as you like so I can get that merged and add you for docs. Thanks :v:

@all-contributors Add @Jyotika999 for docs


Add fast forward option to command

Added option to fast forward to the fix_subscription_lifecycle management command.
We are now able to send a message or fast forward to the selected end date.

@rudigiesler @KaitCrawford please review, thanks.


Feature: Remember multiple application windows for "Continue where I left off"

What feature or improvement do you think would benefit Files?
Currently, "Continue where I left off" will remember tabs on a primary window and reopen them when re-launching Files. However, secondary windows are not remembered, even those opened by the CTRL+N hotkey or respective "New window" command.
This likely is will relate to #14192 at some point, though that feature can likely be implemented separately.
Requirements

This proposal will allow Files to open multiple windows on its launch and restore the appropriate tabs to each
This proposal will initially make most sense within the existing "Continue where I left off", though may make sense as a separate option down the road (or a variation of the option? E.g. some folks may want only one window restored).
This proposal may require delineating between new windows that are launched as a separate process/instance (compared to any that would be managed by the primary/initial instance).

Files Version
<IP_ADDRESS>
Windows Version
Windows 11 Pro; 10.0.22621 Build 22621
Comments
High level, this will likely involve evaluating the process of spawning secondary windows compared to relaunching Files as a separate instance. From there, determining what is affected by this feature's behavior would be easier to understand.
Edit: Builds on discussion at the end of #12624

#12689 is a prerequisite.


docs: update pop function document

update pop function document

Thanks for the PR. However, the API is documented on the action's page. I don't think it makes sense to document only pop's arguments here. So I'm closing this.