save the max_datagram_frame_size transport parameter for 0-RTT

Otherwise, it's easy to violate the size requirement described in section 3 of RFC 9221.

This is not as trivial as just adding the max_datagram_frame_size parameter to TransportParameters.MarshalForSessionTicket. We need a way to tell if the DATAGRAM extension was enabled in the first place.
It looks like the only way to do this is to pass a negotiatedDatatramExtension bool to cryptoSetup.GetSessionTicket, and only include the max_datagram_frame_size in that case.

Hi, I am wondering why we need the negotiatedDatatramExtension parameter when generating the session ticket.  Since max_datagram_frame_size is a unidirectional limit, I guess the server can just put max_datagram_frame_size in the session ticket without any negotiation if EnableDatagrams is set in its config?

@tanghaowillow I just merged #3985. Are you still planning to submit a patch to resolve this issue?

@marten-seemann Yes, but I still have the question mentioned above. I would appreciate if you could give an answer or more guidance.

Hi, I am wondering why we need the negotiatedDatatramExtension parameter when generating the session ticket. Since max_datagram_frame_size is a unidirectional limit, I guess the server can just put max_datagram_frame_size in the session ticket without any negotiation if EnableDatagrams is set in its config?

Yes, that sounds correct. To be clear, this is the server's value of that parameter.


Add uperf 'nthr' param to allow the option to run multi-threaded uper…

…f instance inside the same pod

Oh just realized I didn't update the test CRs, those are separate from resources/crds?

/rerun all

Ah since I added another nthr value to test, it doubled the test matrix so the timeout is just a bit too short, let me increase that.


Allow for group comparisons of samples

New datasets are profiling spatial biology across different sample groups, for example responders vs non-responders. Within the context of spatial biology, these comparisons can be complex, f.e. subgraph topology.
Tagging @MohammedZidane

Is this redundant with #784?

Depends on where we draw the line in "healthy vs control" vs "perturbation screen" imho? What do you think? The first could basically be "simple" matrix comparisons, the latter must probably be more sophisticated

With perturbation analysis I was rather referrring to "treatment vs control", so still a simple comparison. But it could obviously also be used for observational studies.

With @Zethson we're currently scoping out something that goes more in the direction of a perturbation screen, so I feel like there'll need to be tools for both. I don't have an intuition yet on how strong their overlap would be, but I'd tentatively think of them as 2 related but different topics 🤔 Wdyt also @giovp ?

let's rename both and make this one about the screen then


ValueError: ClassSelector parameter None value must be an instance of (function, tuple), not <function size at ###>

Thanks for contacting us! Please read and follow these instructions carefully, then delete this introductory text to keep your issue easy to read. Note that the issue tracker is NOT the place for usage questions and technical assistance; post those at Discourse instead. Issues without the required information below may be closed immediately.
ALL software version info
(this library, plus any other relevant software, e.g. bokeh, python, notebook, OS, browser, etc)
hvplot == 0.7.3
holoviews = 1.14.9
bokeh == 2.4.3
python == 3.11
Description of expected behavior and the observed behavior
Cannot import hvplot
Complete, minimal, self-contained example code that reproduces the issue
import hvplot

Stack traceback and/or browser JavaScript console output
ValueError                                Traceback (most recent call last)
Cell In[16], line 2
      1 from autoviz import AutoViz_Class
----> 2 import hvplot.pandas
      3 import numpy as np
      4 np.bool = bool

File ~\Python\venvs\eda311\Lib\site-packages\hvplot\__init__.py:12
      8 import holoviews as _hv
     10 from holoviews import Store
---> 12 from .converter import HoloViewsConverter
     13 from .util import get_ipy
     14 from .utilities import save, show # noqa

File ~\Python\venvs\eda311\Lib\site-packages\hvplot\converter.py:25
     18 from holoviews.core.util import max_range, basestring
     19 from holoviews.element import (
     20     Curve, Scatter, Area, Bars, BoxWhisker, Dataset, Distribution,
     21     Table, HeatMap, Image, HexTiles, QuadMesh, Bivariate, Histogram,
     22     Violin, Contours, Polygons, Points, Path, Labels, RGB, ErrorBars,
     23     VectorField, Rectangles, Segments
     24 )
---> 25 from holoviews.plotting.bokeh import OverlayPlot, colormap_generator
     26 from holoviews.plotting.util import process_cmap
     27 from holoviews.operation import histogram

File ~\Python\venvs\eda311\Lib\site-packages\holoviews\plotting\bokeh\__init__.py:40
     38 from .graphs import GraphPlot, NodePlot, TriMeshPlot, ChordPlot
     39 from .heatmap import HeatMapPlot, RadialHeatMapPlot
---> 40 from .hex_tiles import HexTilesPlot
     41 from .path import PathPlot, PolygonPlot, ContourPlot
     42 from .plot import GridPlot, LayoutPlot, AdjointLayoutPlot

File ~\Python\venvs\eda311\Lib\site-packages\holoviews\plotting\bokeh\hex_tiles.py:22
     18 from .selection import BokehOverlaySelectionDisplay
     19 from .styles import base_properties, line_properties, fill_properties
---> 22 class hex_binning(Operation):
     23     """
     24     Applies hex binning by computing aggregates on a hexagonal grid.
     25
     26     Should not be user facing as the returned element is not directly
     27     useable.
     28     """
     30     aggregator = param.ClassSelector(
     31         default=np.size, class_=(types.FunctionType, tuple), doc="""
     32       Aggregation function or dimension transform used to compute bin
     33       values. Defaults to np.size to count the number of values
     34       in each bin.""")

File ~\Python\venvs\eda311\Lib\site-packages\holoviews\plotting\bokeh\hex_tiles.py:30, in hex_binning()
     22 class hex_binning(Operation):
     23     """
     24     Applies hex binning by computing aggregates on a hexagonal grid.
     25
     26     Should not be user facing as the returned element is not directly
     27     useable.
     28     """
---> 30     aggregator = param.ClassSelector(
     31         default=np.size, class_=(types.FunctionType, tuple), doc="""
     32       Aggregation function or dimension transform used to compute bin
     33       values. Defaults to np.size to count the number of values
     34       in each bin.""")
     36     gridsize = param.ClassSelector(default=50, class_=(int, tuple))
     38     invert_axes = param.Boolean(default=False)

File ~\Python\venvs\eda311\Lib\site-packages\param\__init__.py:1367, in ClassSelector.__init__(self, class_, default, instantiate, is_instance, **params)
   1365 self.is_instance = is_instance
   1366 super(ClassSelector,self).__init__(default=default,instantiate=instantiate,**params)
-> 1367 self._validate(default)

File ~\Python\venvs\eda311\Lib\site-packages\param\__init__.py:1371, in ClassSelector._validate(self, val)
   1369 def _validate(self, val):
   1370     super(ClassSelector, self)._validate(val)
-> 1371     self._validate_class_(val, self.class_, self.is_instance)

File ~\Python\venvs\eda311\Lib\site-packages\param\__init__.py:1383, in ClassSelector._validate_class_(self, val, class_, is_instance)
   1381 if is_instance:
   1382     if not (isinstance(val, class_)):
-> 1383         raise ValueError(
   1384             "%s parameter %r value must be an instance of %s, not %r." %
   1385             (param_cls, self.name, class_name, val))
   1386 else:
   1387     if not (issubclass(val, class_)):

ValueError: ClassSelector parameter None value must be an instance of (function, tuple), not <function size at 0x0000020F5A342CB0>.

Screenshots or screencasts of the bug in action

[ ] I may be interested in making a pull request to address this

Hi @RNarayan73, I replied in https://github.com/AutoViML/AutoViz/issues/102#issuecomment-1873874818 and am closing this issue as this issue has already been fixed in HoloViews and released. Feel free to comment if you think there's another way we could approach this in which case I'd be happy to re-open the issue.


Enhancement: Conditional foreign key test to support data vault

Minor enhancement to accept conditional filtering on the child side of the foreign key test (similar to dbt_utils.relationships_where test
The specific scenario is testing of referential integrity between data vault Hub and Satellite tables where ghost records are added to Satellite tables on initial creation but never to Hubs. All other surrogate keys present in a Satellite table must also be present in the parent Hub
I appreciate the desire to adhere to "referential constraint" as specified in ANSI SQL 92 however I am keen to

get the benefit of applying FKs (Snowflake)
apply ref integrity tests
retain ghost records in satellite tables

Two approaches I can see:
child predicate param (similar to dbt_utils.relationships_where)
#SATELLITES
   - name: sat_reliability_engineering_functional_location_eca
     description: SATELLITE of Functional Location - attributes related to ECA data
     tests:
      - dbt_constraints.foreign_key:
          fk_column_names:
            - DV_SK_HUB_FUNCTIONAL_LOCATION
          child_predicate: DV_SK_HUB_FUNCTIONAL_LOCATION != repeat('0', 64)::BINARY
          pk_table_name: ref('hub_functional_location')
          pk_column_names:
            - DV_SK_HUB_FUNCTIONAL_LOCATION
          tags: ["data_test"]

nullif condition
  #SATELLITES
   - name: sat_reliability_engineering_functional_location_eca
     description: SATELLITE of Functional Location - attributes related to ECA data
     tests:
      - dbt_constraints.foreign_key:
          fk_column_names:
            - DV_SK_HUB_FUNCTIONAL_LOCATION
          fk_nullif: repeat('0', 64)::BINARY
          pk_table_name: ref('hub_functional_location')
          pk_column_names:
            - DV_SK_HUB_FUNCTIONAL_LOCATION
          tags: ["data_test"]

Happy to contribute if agreeable on the request

The new always_create_constraint feature should provide the functionality you require. It will allow you to define your tests to exclude the ghost records with dbt Core's config: where: feature and still generate a constraint. It would also allow you to define a threshold so you could also use dbt's config: warn_if: and config: error_if: feature to ignore the one FK violation.


Prediction TypeError in Let’s code a Neural Network in plain NumPy

I could use
params_values = train(np.transpose(X_train), np.transpose(y_train.reshape((y_train.shape[0], 1))), NN_ARCHITECTURE, 10000, 0.01)

but there is an error when I implement next step
Y_test_hat, _ = full_forward_propagation(np.transpose(X_test), params_values, NN_ARCHITECTURE)

Here is it

TypeError                                 Traceback (most recent call last)
1 # Prediction
----> 2 Y_test_hat, _ = full_forward_propagation(np.transpose(X_test), params_values, NN_ARCHITECTURE)

in full_forward_propagation(X, params_values, nn_architecture)
8
9         activ_function_curr = layer["activation"]
---> 10         W_curr = params_values["W" + str(layer_idx)]
11         b_curr = params_values["b" + str(layer_idx)]
12

TypeError: tuple indices must be integers or slices, not str

But I think params_values is a dictionary, Why is it? I have copied all the necessary code.
btw, I use Juypter 6.7.0 in Anaconda3 & python 3.6.7

Would you have a chance to share your notebook somewhere on GitHub? It's hard for me to debug your problem when I can't see the source code you're using.

Here is my repository NumpyNN
I've tried many times, but the error still occurs.

Super. I will try to find out what may be the problem.

@CreatureOX  I think I already know what your problem is. The error occurred because some of the code you used was taken from an article on Medium and some from the repository. More specifically, the train function you are using returns a three-piece tuple.
def train(X, Y, nn_architecture, epochs, learning_rate):
    params_values = init_layers(nn_architecture, 2)
    cost_history = []
    accuracy_history = []

    for i in range(epochs):
        Y_hat, cache = full_forward_propagation(X, params_values, nn_architecture)

        cost = get_cost_value(Y_hat, Y)
        cost_history.append(cost)
        accuracy = get_accuracy_value(Y_hat, Y)
        accuracy_history.append(accuracy)

        grads_values = full_backward_propagation(Y_hat, Y, cache, params_values, nn_architecture)
        params_values = update(params_values, grads_values, nn_architecture, learning_rate)

    return params_values, cost_history, accuracy_history # <---TUPLE

Note that in the code version used in the repository, this function returns only one of these elements.
return params_values

How can you solve this problem? In two ways:

Correct the return of the train function so that it returns only one item as it is in the repository.
OR
Correct the code at the point of use of the train function. Unpack the tuple in place, like this:

params_values, _, _ = train(np.transpose(X_train), np.transpose(y_train.reshape((y_train.shape[0], 1))), NN_ARCHITECTURE, 10000, 0.01)

I hope that everything is clear. Let us know if any further problems have occurred. Greetings.

thx for your detailed reply, It works!

Great! It's very nice to hear that.


Reorganize the structure of architecture documents

What is the purpose of the change
Reorganize the structure of architecture documents
Brief change log
(for example:)

Reorganize the structure of architecture documents

Verifying this change
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.

Codecov Report

Merging #186 (866176f) into master (1de000d) will increase coverage by 0.15%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #186      +/-   ##
==========================================
+ Coverage   60.18%   60.33%   +0.15%
==========================================
  Files         179      181       +2
  Lines       13328    13464     +136
  Branches     1991     2018      +27
==========================================
+ Hits         8021     8124     +103
- Misses       4884     4911      +27
- Partials      423      429       +6

Impacted Files
Coverage Δ

...ow_plugins/blob_manager_plugins/s3_blob_manager.py
76.38% <0.00%> (-1.39%)
:arrow_down:

lib/notification_service/setup.py
0.00% <0.00%> (ø)

...notification_service/cli/commands/event_command.py
63.26% <0.00%> (ø)

...n_service/notification_service/cli/simple_table.py
67.74% <0.00%> (ø)

...otification_service/notification_service/client.py
78.05% <0.00%> (+0.55%)
:arrow_up:

...ication_service/notification_service/util/utils.py
83.56% <0.00%> (+0.95%)
:arrow_up:

...w_plugins/blob_manager_plugins/oss_blob_manager.py
81.01% <0.00%> (+1.26%)
:arrow_up:

..._service/notification_service/base_notification.py
76.81% <0.00%> (+1.44%)
:arrow_up:

...ion_service/notification_service/cli/cli_parser.py
96.69% <0.00%> (+2.24%)
:arrow_up:

...tion_service/notification_service/event_storage.py
89.01% <0.00%> (+2.31%)
:arrow_up:

... and 1 more

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 1de000d...866176f. Read the comment docs.


Support custom favicons

Proposal
I'd like to be able to set a custom favicon. Based on the website configuration docs I'd expect the configuration to look something like this:
# See docs at: https://mystmd.org/guide/frontmatter
version: 1
project:
  # title:
  # description:
site:
  template: book-theme
  # title:
  # options:
  #   logo: my_logo.png
       favicon: my_favicon.png

It will have to wait for the next mystmd release to work on github pages, but works locally now. ⭐

Should also work with svg and png.

@rowanc1 I think this may still not work on static html-generated sites...is that possible?

Yes! Not until the next release of mystmd (probably in a few days):

It will have to wait for the next mystmd release to work on github pages

ohhh I see that's what you meant by "work on github pages"

Thanks for the new feature! I'm commenting just to tell you that in order for the new favicon to show up I had to delete the _build folder content. Maybe it's worth pointing this out somewhere in the docs?

Yes, we don't yet have a notification on the new templates being released. There is a note in the update instructions here:
https://mystmd.org/guide/quickstart

We are also tracking the issue here: #854.


Make responsively embedable e.g. using pym.js

Include an example somewhere - perhaps linked from the footer.

as a fallback, pym can poll the window size. But ideally, make it update upon user actions that can resize it.

https://github.com/keep-the-receipts/keepthereceipts-frontend/pull/23

still need to help people find this


Correct setup instructions typo for choose_background.sh

The correct filename is choose_background.sh, not choose-background.sh.

Thanks!


Add a classic mode for the LuckPerms editor

I recently upgraded to the latest version of LuckPerms and the new editor looks ugly. The old editor looked so much cleaner and didnt have so many buttons in weird places. The classic one was far easier to use and didnt have stuff that you can do in game, like parent groups. Just an option in the config or a command to go back to classic mode would be amazing.

As you can see from the big red BETA at the top, it's not in a fully finished state and constructive feedback may help make it look a lot better.
I will not work on the old editor code, and some would need to be done on it to have it work with current versions of LuckPerms. Feel free to fork it and make it happen.
If you hate it that much, you can always downgrade to LuckPerms v4 so you can use it again.


Connection reset by peer with AWS NAT Gateway and Keep Alive

Set up following scenario to reproduce the issue:
backend side:
CloudFlare is proxying to AWS ELB.
ELB is proxying to Nginx.
client side:
Create private subnet and attach AWS NAT Gateway to it.
Launch EC2 instance in private subnet and use a script below to reproduce the issue.
Request path is:
EC2 instance(originating) -> AWS NAT Gateway -> CloudFlare -> AWS ELB -> Nginx.
I created a script which reproduces the issue:
import requests
import time

def request_with_timeout(url, minutes):
    session = requests.Session()
    session.get(url, allow_redirects=False)
    time.sleep(minutes * 60)
    session.get(url, allow_redirects=False)

request_with_timeout('https://coins.ph', 6)

After executing - requests raises an exception:
>>> request_with_timeout('https://coins.ph', 6)
/opt/app/accounting/venv/local/lib/python2.7/site-packages/requests/packages/urllib3/util/ssl_.py:315: SNIMissingWarning: An HTTPS request has been made, but the SNI (Subject Name Indication) extension to TLS is not available on this platform. This may cause the server to present an incorrect TLS certificate, which can cause validation failures. For more information, see https://urllib3.readthedocs.org/en/latest/security.html#snimissingwarning.
  SNIMissingWarning
/opt/app/accounting/venv/local/lib/python2.7/site-packages/requests/packages/urllib3/util/ssl_.py:120: InsecurePlatformWarning: A true SSLContext object is not available. This prevents urllib3 from configuring SSL appropriately and may cause certain SSL connections to fail. For more information, see https://urllib3.readthedocs.org/en/latest/security.html#insecureplatformwarning.
  InsecurePlatformWarning
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "<stdin>", line 5, in request_with_timeout
  File "/opt/app/accounting/venv/local/lib/python2.7/site-packages/requests/sessions.py", line 480, in get
    return self.request('GET', url, **kwargs)
  File "/opt/app/accounting/venv/local/lib/python2.7/site-packages/requests/sessions.py", line 468, in request
    resp = self.send(prep, **send_kwargs)
  File "/opt/app/accounting/venv/local/lib/python2.7/site-packages/requests/sessions.py", line 576, in send
    r = adapter.send(request, **kwargs)
  File "/opt/app/accounting/venv/local/lib/python2.7/site-packages/requests/adapters.py", line 426, in send
    raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', error(104, 'Connection reset by peer'))

Software versions:
Python 2.7.6
requests==2.9.1

But also reproduced on Python 2.7.12 and requests 2.12.4

Attaching tcpdump
tcpdump-log.txt

The issue does not appear when URL has http scheme.

Also everything works fine when there's no AWS NAT Gateway on the traffic path.

I also noticed that error does not happen if requests are passing-by cloudflare.

So, this looks like you're encountering a race condition.
It seems like, one way or another, there is a server or CDN that has a 6-minute timeout on their connection, such that when it is idle for 6 minutes they will close it. If you start another request while the RST packet is in-flight, requests will see the RST as a connection abort. This is indistinguishable from the server closing the connection in our face, so we have to report it as an error condition.
To resolve this, I recommend adding retry logic. Your code should, in general, be resilient to failure in the face of network instability. You can do this either by mounting custom transport adapters, or by changing the value of requests.adapters.DEFAULT_RETRIES to something other than zero before you create your session.

I also stumbled upon this behaviour recently.
Being resilient to failure in the face of network instability is good advice, but sometimes it may be expensive to implement this.
After some research I found that turning on TCP keepalive helps preventing reset errors.
For example like this:
class KeepaliveAdapter(requests.adapters.HTTPAdapter):
    def init_poolmanager(self, *args, **kwargs):
        kwargs['socket_options'] = HTTPConnection.default_socket_options + [
            (socket.SOL_SOCKET, socket.SO_KEEPALIVE, 1),
        ]
        super(KeepaliveAdapter, self).init_poolmanager(*args, **kwargs)

keepalive_aware_session = requests.Session()

adapter = KeepaliveAdapter()
keepalive_aware_session.mount('http://', adapter)
keepalive_aware_session.mount('https://', adapter)

And then use session as usual.
But for this to work it is also neccesary to tune keepalive timers (i.e. set it less than 6 min, I used 180 seconds):
sysctl -w net.ipv4.tcp_keepalive_time=180

Default value for this param on Ubuntu is two hours.


Allow SpanExporters to be added.

Resolves #40.
This introduces a new method called addSpanExporter() on the OpenTelemetryRumBuilder. This is a convenience method for users who are not bringing their own preconfigured SDK and want to include their own preconfigured exporter for spans.

I think there's a lot of benefit to following the pattern(s) already used in https://github.com/open-telemetry/opentelemetry-java/blob/main/sdk-extensions/autoconfigure/src/main/java/io/opentelemetry/sdk/autoconfigure/AutoConfiguredOpenTelemetrySdkBuilder.java where possible

Ok, lemme see about changing away from addSpanExporter() to addSpanExporterCustomizer(). We clearly need more nouns in here. 🙃 I can't say it feels cleaner, but I guess it's more consistent.


Newest Even Newer New Crystal Hollow Items, this time with a new amazing weird pet that needs to be forged for way too long

This will prob be the last new stuff for crystal hollows sadly, so no more epic titles of it after this

The pet takes 12 days to forge as you may see so don't merge yet please mr. IRONM00N, I will add all rarities and stats hopefully once thats done.


[match] Add ability to include all development certificates to the development provisioning profile

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
We want to automate adding devices and development certificates to the development provisioning profiles.
Our limitations:

No one (except CI) has access to the match repo.
We don't want to share the single Apple Development certificate across developers.

So, we become with a solution:

All devices are listed in the file stored in the project repo. With is perfectly handled by the register_devices.
Each run we add missing devices to the development provisioning profile.

The thing we were missing is how to automate adding new Apple Development Certificates to the provisioning profiles in the same manner.
Description
New include_all_certificates option in sigh & match, which will allow us to include all certificates (only for development provisionong profile type).
Additionally, introduce a new option to match force_for_new_certificates, that will work exactly in the same way as the existent optionforce_for_new_devices but for certificates: if the count of development certificates differs (account vs profile) - update the profile.
It still uses the same account and have one code signing identity for distribution and access to the App Store Connect and Developer portal as described in the codesigning.guide, but additionally allows the use of multiple identities/certificates for development provisioning profiles (w/o including them in the repo) as an option.
Testing Steps
    sync_code_signing(
      app_identifier: [
        'com.your.bundle.id',
      ],
      type: 'development',
      readonly: false,
      include_all_certificates: true,
      force_for_new_certificates: true,
    )

This looks great! Just one little suggestion about adding a comment line 😇
👍
Thanks. Done.

@joshdholtz , appveyor failed to DL some dependency, and there is a strange new CLA request after rebasing. idk 🤷

Will take a look when I can! Got LASIK a few days ago so eyes are a bit sensitive to light and computer screens still 🤷♂

Get well soon ❤🩹.

@googlebot I consent.

@nekrich i need so much this option!!  great work 👍


Typing effect by just using css

Creating typing animation using css

@ismeet010 pls assign this issue to me

I have already created this effect using CSS, please assign me under hacktoberfest tag.

I have also done made this animation not this fair that i get the issue

@ismeet010 made the pr too plz check


should s3 key in backend configure for each module be different?

Hello,
I just read again the book terraform up and running to apply in my pet project and the author said that we need to have different the s3 key value in the backend config over each module to prevent overwriting the config. (page 86 | Chapter 3: How to Manage Terraform State)
But as I know, we just need to have a single s3 key value in the backend config in the main file (root folder) and it will apply for all (The main file in the root folder will call these modules). Terraform should consider them as a whole project. I just supposed that a project has a different s3 key for each env, not modules. When running deploy for each project, everything will be put in a single terraform.tfstate file
If we need to have different s3 key value in each module, how about using the module library from Github (we don’t edit this module)
Take the below repo as an example. I don’t see any backend config in the module folders
https://github.com/finleap/tf-ecs-fargate-tmpl/blob/master/alb/main.tf
I supposed that perhaps the author wants to deploy separate modules. If like that, what are the benefits?
The author also repeats his statement here
https://terragrunt.gruntwork.io/docs/features/keep-your-remote-state-configuration-dry/
So anyone can help me to clarify it? I don’t have much experience in terraforming, so perhaps there are some misunderstandings.

I just read the doc in here https://www.terraform.io/language/modules#the-root-module
It seems to be that:
A project must have multiple root modules (What I thought before is that one project should have only one root module).
The root module must have a different s3 key
»The child module, published module use same configure with the root module
So what the author mention is about the root module?


use nocaptcha

https://www.google.com/recaptcha/intro/index.html
Use used to use recaptcha but ditched it since it proved too much of a barrier to sign up. It's gotten a lot better since then and our spam problems have gotten worse, so let's give it another try.

show recaptcha on web signup form
don't prevent signups via the API, i.e. via our own mobile apps

This is fixed in 25feb02, bb352f8, and 9dd7add


Skip questions on matching passed options

Closes microsoft/playwright#31416

@pavelfeldman thanks for the merge! Would it be possible to get this published as a new version?
From the version history, I would guess that it would land in a new patch version:

<EMAIL_ADDRESS>


Menu: left align title when type is card

Please make sure these boxes are checked before submitting your PR, thank you!

[ ] Make sure you follow Element's contributing guide (中文 | English | Español | Français).
[ ] Make sure you are merging your commits to dev branch.
[ ] Add some descriptions and refer relative issues for you PR.

需要设计确认样式


GUI hangs indefinitely

Filing this bug for others.
Steps taken:

Install new gui on Mac
Run installed program
Experience GUI hanging

Fix

Upgrade mint-amazon-tagger with pip3 install --upgrade mint-amazon-tagger
Run mint-amazon-tagger from cli
See new interface working

@glassdimly I see you closed the issue; did the console output help determine the issue? I do have a few bug fixes coming out shortly; there's a known issue with the amazon csv gui code (That will be fixed shortly)


[FUSE] Always use enhanced pragmas

Consolidate the pragma logic:

Ensure we always use an enhanced line pragma via assert
Handle mapping the token and fallback padding as part of building the pragma
Fixup Inject and TagHelper properties where we were still using regular line directives

@dotnet/razor-compiler for review please.
//cc @ryzngard


Add instructions when using on Travis CI

If one is using Travis CI to build a project that depends on nodegit, the build will fail because it can't compile.
These steps will keep this from happening and will let nodegit compile happily.

This is useful information if the build does fail, although with latest Node 0.12 or 4.1 it should download the prebuilt binary that we publish.

It doesn't though, see the log for the latest build I did:
395.37s$ npm install
><EMAIL_ADDRESS>install /home/travis/build/Coaxial/coverall/node_modules/nodegit
> node lifecycleScripts/install
[nodegit] Fetching binary from S3.
[nodegit] Failed to install prebuilt binary, building manually.
[nodegit] Regenerating and configuring code

I'm not sure why this happens but it makes build take forever because it has to recompile every time.

Can you give us your platform (nix/osx?), arch, and node/npm versions?

Sure, this occurs on Travis CI when testing against node 0.10, 0.11 and 0.12. I believe they are using Ubuntu 12.04 in Docker containers.
On 4.0 and 4.1, it fails with the following, no matter what I tried:
Message:
    /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.17' not found (required by /home/travis/build/Coaxial/coverall/node_modules/nodegit/build/Release/nodegit.node)
Stack:
Error: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.17' not found (required by /home/travis/build/Coaxial/coverall/node_modules/nodegit/build/Release/nodegit.node)
    at Error (native)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)
    at Object.<anonymous> (/home/travis/build/Coaxial/coverall/node_modules/nodegit/lib/nodegit.js:8:12)
    at Module._compile (module.js:434:26)
    at Object.Module._extensions..js (module.js:452:10)
    at Object.Module._extensions.(anonymous function) [as .js] (/home/travis/build/Coaxial/coverall/node_modules/gulp-istanbul/node_modules/istanbul/lib/hook.js:109:37)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)
    at Object.<anonymous> (/home/travis/build/Coaxial/coverall/lib/archive.js:9:269)
    at Module._compile (module.js:434:26)
    at Object.Module._extensions.(anonymous function) [as .js] (/home/travis/build/Coaxial/coverall/node_modules/gulp-istanbul/node_modules/istanbul/lib/hook.js:107:24)

By the way, why not use bluebird's promisify and bluebird's promises rather than node-promisify?

@Coaxial someone tried to use Bluebird in the past, but it broke wayyyy too much stuff. I don't remember the specifics, but you can crawl through the history log for the revert.
We do not support 0.10 for precompiled builds, which means it will compile in those cases. Node 0.12 and Node 4.1 on Travis should not be compiling, can you verify this and/or show us where that's not the case?

This might no longer be needed after #761

We're having a similar issue. Travis fails to install the precompiled binary and eventually dies during build on v0.12.
When installing nodegit from master (npm i --save nodegit/nodegit#master), which includes @johnhaley81's #761, it still fails to install the precompiled binary but successfully builds and everything is A-OK.

@contolini installing nodegit from the master branch won't download pre-built binaries since there are none published for master. We're looking to release again soon and when we do we'll have the pre-built binaries up for downloading during install.

I think we finally git rid of the glibc requirement correct? If so this can just be closed
CC @johnhaley81

Yeah, we can close this one down. You don't need that anymore to build. If that changes for any reason we can open this back up. Thanks @Coaxial

@johnhaley81 this is still an issue

Reopening due to #853

I've since added instructions that are easier and don't require rebuilding the module (prebuilt works fine), so closing this.


updated Accumulo and Hadoop version numbers

Interim fix for the out of data Accumulo download URL.  Interim because it would be better to switch to using URLs that are known to be stable at least whilst the vagrant file refers to them.
See also pull request https://github.com/apache/incubator-rya/pull/27 (which I hadn't noticed before I did my own updates.
Also:

adds download URL for base box
checks that downloads work and fails the script if they don't
uses symbolic version numbers throughout

Wow, this is awesome!  Love the download function.
This issue has gotten a lot of attention, as it should.  I'll have to merge my changes into yours that I did for RYA-54  https://issues.apache.org/jira/browse/RYA-54.
I made it handle restarts, and re-provisioning.
I used the approach of adding    "|| exit 150"   to the ends of the lines that can fail.
david.

Thanks!  I merged this PR.  Please close it :)


Move user data from etcd to another db

why do we need this?

At least, we shouldn't leverage on the etcd of k8s.

we decide to use k8s secret to store user info.


Uncaught TypeError: Cannot read property 'setAttribute' of null

Probably related to #911.
Steps to reproduce

switch to .json file with JSON linter package enabled (in my case, this was not a package.json like in #911)

on editing the same or any other JSON file further, the error does not reappear until restarting atom
when restarting atom, the error appears immediately on startup when the cursor is in a JSON file on startup

type one character (" in my case)
error message popup appears (see below)

Atom Version: 1.0.19; installed just beforehand via rpm -U, no problems in previous installed version (1.0.9, I think)
System: Fedora 22, x86_64, Linux 4.1.7
Thrown From: linter package, v1.7.0
Stack Trace
Uncaught TypeError: Cannot read property 'setAttribute' of null
At /home/timo/.atom/packages/linter/lib/ui/message-element.js:21

TypeError: Cannot read property 'setAttribute' of null
  at linter-message.updateVisibility (/home/timo/.atom/packages/linter/lib/ui/message-element.js:41:209)
  at BottomPanel.setMessages (/home/timo/.atom/packages/linter/lib/ui/bottom-panel.js:56:24)
  at LinterViews.render (/home/timo/.atom/packages/linter/lib/linter-views.coffee:36:12)
  at /home/timo/.atom/packages/linter/lib/linter.coffee:29:14
  at Emitter.module.exports.Emitter.emit (/usr/share/atom/resources/app.asar/node_modules/event-kit/lib/emitter.js:82:11)
  at MessageRegistry.updatePublic (/home/timo/.atom/packages/linter/lib/message-registry.js:116:20)
  at UpdateMessages (/home/timo/.atom/packages/linter/lib/message-registry.js:42:17)

Commands
     -0:56.8.0 core:move-right (atom-text-editor.editor.is-focused.autocomplete-active)
  4x -0:56.5.0 editor:select-to-beginning-of-word (atom-text-editor.editor.is-focused)
  3x -0:55.4.0 core:select-left (atom-text-editor.editor.is-focused)
     -0:54.5.0 core:delete (atom-text-editor.editor.is-focused)
     -0:54 editor:move-to-beginning-of-word (atom-text-editor.editor.is-focused)
     -0:53.8.0 core:save (atom-text-editor.editor.is-focused)
  2x -0:22.7.0 editor:move-to-beginning-of-word (atom-text-editor.editor.is-focused)
     -0:21.9.0 core:move-right (atom-text-editor.editor.is-focused)
  2x -0:21.5.0 editor:select-to-beginning-of-word (atom-text-editor.editor.is-focused)
     -0:20.5.0 core:select-left (atom-text-editor.editor.is-focused)
  2x -0:19 core:delete (atom-text-editor.editor.is-focused)
     -0:18.3.0 editor:move-to-end-of-screen-line (atom-text-editor.editor.is-focused)
     -0:17.8.0 core:backspace (atom-text-editor.editor.is-focused)
  3x -0:17.2.0 core:move-left (atom-text-editor.editor.is-focused)
     -0:16.5.0 core:save (atom-text-editor.editor.is-focused)
     -0:01.7.0 core:move-left (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "disabledPackages": [
      "preview-inline",
      "python-isort",
      "atom-save-all",
      "open-html-in-browser",
      "python-iresolve",
      "indent-guide-improved",
      "live-doc-viewer"
    ],
    "destroyEmptyPanes": false,
    "openEmptyEditorOnStart": false,
    "themes": ["atom-material-ui", "one-dark-syntax"]
  },
  "linter": {"showErrorPanel": false}
}

Installed Packages
# User
atom-beautify, v0.28.14
atom-html-preview, v0.1.11
atom-jinja2, v0.5.0
atom-material-ui, v0.6.1
atom-smart-tabs, v0.1.0
autoclose-html, v0.19.0
autocomplete-bibtex, v0.6.1
autocomplete-haskell, v0.5.1
autocomplete-html-entities, v0.1.0
autocomplete-paths, v1.0.2
autocomplete-php, v0.3.7
autocomplete-python, v0.5.4
autocomplete-ruby, v0.1.0
color-picker, v2.0.12
colorful-json, v1.0.0
compare-files, v0.6.2
emmet, v2.3.13
emmet-snippets-compatibility, v1.0.0
fonts, v0.6.0
haskell-ghc-mod, v0.9.12
ide-haskell, v0.7.2
indentation-indicator, v0.6.0
language-diff, v0.3.1
language-haskell, v1.4.9
language-latex, v0.6.1
language-z80asm, v0.34.0
latex, v0.25.1
latexer, v0.2.7
linter, v1.7.0
linter-chktex, v1.1.0
linter-csslint, v1.0.6
linter-htmlhint, v0.1.1
linter-js-yaml, v1.1.2
linter-jsonlint, v1.0.1
linter-python-pep257, v0.0.1
linter-python-pep8, v0.2.0
minimap-git-diff, v4.1.8
mixed-indent-warning, v0.2.0
pdf-view, v0.27.0
pretty-json, v0.4.1
python-autopep8, v0.1.3
python-tools, v0.6.5

# Dev
No dev packages

Restart the editor and let me know if this happens again

I do have this issue happen from time to time. I have to restart the editor for the linter to work again, but the error still pops up at times.

I have been running master for weeks, didn't get this error, I'll try to repro

Here's what my report if that helps:
Atom Version: 1.0.19
System: Mac OS X 10.10.5
Thrown From: linter package, v1.7.0
Stack Trace
Uncaught TypeError: Cannot read property 'setAttribute' of null
At /Users/thibault.ml/.atom/packages/linter/lib/ui/message-element.js:21

TypeError: Cannot read property 'setAttribute' of null
    at linter-message.updateVisibility (/Users/thibault.ml/.atom/packages/linter/lib/ui/message-element.js:21:59)
    at BottomPanel.setMessages (/Users/thibault.ml/.atom/packages/linter/lib/ui/bottom-panel.js:36:22)
    at LinterViews.render (/Users/thibault.ml/.atom/packages/linter/lib/linter-views.coffee:36:12)
    at /Users/thibault.ml/.atom/packages/linter/lib/linter.coffee:29:14
    at Emitter.module.exports.Emitter.emit (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/event-kit/lib/emitter.js:82:11)
    at MessageRegistry.updatePublic (/Users/thibault.ml/.atom/packages/linter/lib/message-registry.js:86:18)
    at UpdateMessages (/Users/thibault.ml/.atom/packages/linter/lib/message-registry.js:31:16)

Commands
     -0:01.5.0 core:save (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "themes": [
      "flatland-dark-ui",
      "flatland-dark"
    ],
    "openEmptyEditorOnStart": false
  },
  "linter": {}
}

Installed Packages
# User
build, v0.47.0
build-cargo, v0.2.0
flatland-dark, v0.1.6
flatland-dark-ui, v0.2.4
language-rust, v0.4.5
linter, v1.7.0
linter-clang, v3.2.3
linter-erb, v0.1.1
linter-pep8, v1.0.1
linter-rust, v0.2.9
minimap, v4.13.4
racer, v0.17.2

# Dev
No dev packages

I have very little knowledge of JS, none of the linter or CoffeeScript, but looking at the source code, I believe the second line of this screenshot might be the problem.
It's trying to get linter-message-link, but there is no link associated with that error. If I comment the code that generates that error, then the error doesn't appear. If I uncomment the code that generates that error, then I do get that error.

Here's the actual compiler issue:
src/main.rs:3:24: 3:27 warning: unused import, #[warn(unused_imports)] on by default
src/main.rs:3 use std::sync::{Mutex, Arc};
                                     ^~~
src/main.rs:9:1: 11:2 warning: struct is never used: `Table`, #[warn(dead_code)] on by default
src/main.rs: 9 struct Table {
src/main.rs:10     forks: Vec<Mutex<()>>,
src/main.rs:11 }
src/main.rs:10:5: 10:26 warning: struct field is never used: `forks`, #[warn(dead_code)] on by default
src/main.rs:10     forks: Vec<Mutex<()>>,
                   ^~~~~~~~~~~~~~~~~~~~~

I believe linter-rust groups the last two errors as one, which is probably why the line is not displayed.
I would say that while the rust linter is the one grouping them, I don't believe linter should error because of the absence of link.

I have a fix up for this

A patch release is up for download.

Thanks, that was quick :-) I see my assumption was the right one!

@naixn Yes it was sir :)

Also fixing the scenario where multi line messages don't have a link when they should.


(Bug) #942 user from w3 tries to register after deleting his account

Description
Remove w3Token from AsyncStorage after setting isLoggedIn to true in register flow.
About #942
How Has This Been Tested?
You need to somehow initiate the error in SignUpState finisgRegistration function. So after you reach the last step of registration you will see an error.
Try to register a new user.
After you will see an error - reload the page.
The w3Token should not be deleted from AsyncStorage and the registration process will be continued with w3Token, the data fetched from w3Site and email verified successfully.
Checklist:

[x] PR title matches follow: (Feature|Bug|Chore) Task Name
[x] My code follows the style guidelines of this project
[x] I have followed all the instructions described in the initial task (check Definitions of Done)
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have added reference to a related issue in the repository
[x] I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
[ ] I have added screenshots related to my pull request ( for frontend tasks)
[ ] I have pasted a gif showing the feature.
[x] @mentions of the person or team responsible for reviewing proposed changes

just finish review. but u already merged


pod completion status-1

Changes to get pod completion status propagated back to app wrappers.

In this first cut we check if all the running pods have successful completion for an app wrapper to be completed.

Thanks @asm582, could you also add a test case here: https://github.com/IBM/multi-cluster-app-dispatcher/blob/master/test/e2e/queue.go

Thanks, Adding test case


Is SlimPruner only available for PyTorch?

from nni.compression.tensorflow import SlimPruner
results in the error
ImportError: cannot import name 'SlimPruner' from 'nni.compression.tensorflow'
so I'm wondering: is it only available for PyTorch?
Short summary about the issue/question:
Brief what process you are following:

How to reproduce it:

nni Environment:

nni version:
nni mode(local|pai|remote):
OS:
python version:
is conda or virtualenv used?:
is running in docker?:

need to update document(yes/no):
Anything else we need to know:

@snippler currently yes. The support of tensorflow on model compression is still ongoing, it is highly likely to be included in v1.6 release.

Closing the question as it has been answered. @snippler thanks for supporting and watching NNI, we have continuous plan on model compression development, you are warmly welcome to provide your feedbacks in recent iteration plan: https://github.com/microsoft/nni/issues/2299 or the backlog: https://github.com/microsoft/nni/issues/1917. Feature requests or feedbacks to the model compression features are highly appreciated.


HadoopInputFile to pass down FileStatus when opening file.

In the current version of the HadoopInputFile implementation:
https://github.com/apache/parquet-java/blob/apache-parquet-1.14.0/parquet-hadoop/src/main/java/org/apache/parquet/hadoop/util/HadoopInputFile.java

When performing a newStream, the reference to the FileStatus is lost, which has already been previously consulted to create this class. This means that when you go to the implementation of each FileSystem, it will surely have to be requested again, since you have requested the reference of whether the file exists, when the file is weighed or relevant information to be able to open the file.
Hadoop's openFile() builder API does support this, but it is not on older releases, so until Parquet moves to Hadoop 3.2.0+ only it cannot use the API. And because its a complex and extensible design, it's very hard to use reflection.
HADOOP-19131 adds reflection-friendly entry points for this and other operations, so for releases with the new class, Parquet can pick up the speedup.
Reporter: Oliver Caballero Alvarez
Related issues:

Improve Parquet IO Performance within cloud datalakes (blocks)
WrappedIO to export modern filesystem/statistics APIs in a reflection friendly form (depends upon)
Reduce the number of accesses to S3 object storage (depends upon)
Include FileStatus when opening a file from FileSystem (is depended upon by)

Note: This issue was originally created as PARQUET-2493. Please see the migration documentation for further details.

can someone assign to me; I have a WiP PR for this

@steveloughran Thanks, assigned!


[Bug] edTech word list too small

We are against scammy edTech startups. We want to avoid people talking about it in the community. The warning system is implemented but the library is very small. The task is to add more words to list so it becomes more flexible.

hey, @kaiwalyakoparkar I have successfully set up the project and I would like to contribute and work on this issue. can you please clarify how we will be adding more words for eg: adding words in the array or any better approach for it?

I found out the file restrictedWords.js for the words so do we have to add more words in that array?
@kaiwalyakoparkar

Hey @ItsMeIshan Can you tell me how to set up the project?

Hey @ItsMeIshan Can you tell me how to set up the project?

hey @jayesh-srivastava I have created a discussion for setting up the project and its pretty easy lets hop in!!

@ItsMeIshan Thanks you so much helping other folks and willing to contribute. Do one thing add words as you think as of now. And create a PR. and as soon as I confirm the list proposed by you I will merge it. So I can give suggestions and you can edit on the go. If you want you can also create draft PR for the same.
Go for it. I will assign this issue to you :) If you need any help let me know.

@ItsMeIshan Thanks you so much helping other folks and willing to contribute. Do one thing add words as you think as of now. And create a PR. and as soon as I confirm the list proposed by you I will merge it. So I can give suggestions and you can edit on the go. If you want you can also create draft PR for the same.
Go for it. I will assign this issue to you :) If you need any help let me know.

that's really awesome I am getting started now!!

hey @kaiwalyakoparkar i have selected some words but as we start adding more words in array the linear search implemented will make our code slow so what i have thought that before using array we can use .sort() method and then i can implement binary search to get a Big oh complexity of log n . Do tell me your views on it!

If that works correctly. I don't have any problem you can go for it. Just check every possible scope

hey, @kaiwalyakoparkar please check the list of the words and suggest if more is to be added.


Add a markdown linter

Perhaps would be good if on pull-request, the source branch must pass some markdown lint tests

I can take a stab at opening a PR for this work :)


fix: format table name to include backticks

Fixes #47

Thanks for reviewing the PR. May i ask why SQL Injection is a concern to you in this app when it is just a SQL client?.
I totally understand that i should use prepared statements, i plan to make a huge refactor on all of the queries, but i want to solve the problem with a quick solution first. Does it makes sense to you?

The table name formatting is really dirty, but it is a quick solution. That should be refactored too.

Thanks for reviewing the PR. May i ask why SQL Injection is a concern to you in this app when it is just a SQL client?.

I tend to always plan the worst. Often, I'm not disappointed and something close happens 😅

I totally understand that i should use prepared statements, i plan to make a huge refactor on all of the queries, but i want to solve the problem with a quick solution first. Does it makes sense to you?

Not necessarily prepared statement, but using Exec the right way should be always preferred.

Also please consider a table could have a varchar primary key, and that this field may contain a quote
Something like john "babayaga" wick
It would be a bad design to rely on a varchar PK for this, but your app will be unable to work with this row

May i ask why SQL Injection is a concern to you in this app when it is just a SQL client?

My SQL injection I was more thinking about a source of bugs, than a attack.
Maybe it's what you were asking for

May i ask why SQL Injection is a concern to you in this app when it is just a SQL client?

My SQL injection I was more thinking about a source of bugs, than a attack.
Maybe it's what you were asking for

I understand. I appreciate the feedback. For now, i'll merge this PR just to resolve the connection problem and open another PR addressing your feedback.
Thank you very much.

Sure


Fix for new SDCC calling convention

Apparently SDCC 4.2.0 changed the default calling convention, which broke several GBSDK lib functions. I've fixed the offending functions here. This change will break compatibility with older SDCC versions, so 4.2.0 will need to be the new minimum.

Oh, that looks a lot more compact. But I think it will break SGB. The SGB function was tested on real hardware and is timing sensitive. Emulators do not emulate the timing requirements.

Oh, interesting. I don't have an SGB to test on, so for now I suppose I'll just pad it out with nops so the timing is the same as before

nops would work, or you can do a push&pop, which are the instructions that have a high cycle count per byte. Adding some more delay there isn't an issue. Less most likely is.
I also don't have a SGB to test on. Someone else tested these for me.

Maybe it would also good to have some check somewhere on the SDCC version? No idea if it sets some kind of version define we could check on.


fixed typo in readme -- from colledges to colleges

Hi @ramkakarala ！First of all, we want to express our gratitude for your significant PR in the mmdetection project. Your contribution is highly appreciated, and we are grateful for your efforts in helping improve this open-source project during your personal time. We believe that many developers will benefit from your PR.
We would also like to invite you to join our Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us ：https://discord.gg/raweFPmdzG
If you have WeChat，welcome to join our community on WeChat. You can add our assistant ：openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends：）
Thank you again for your contribution❤


add more desgin variables to vector X

DVs P_max, B_p, and w_n have no effect
Most thicknesses are hitting a lower bound, indicating either a problem with structures, forces, or lack of constraint


munin: fix build

Motivation for this change
fallout from perl 5.24 -> 5.28 update
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: munin
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: munin
Partial log (click to expand)
/nix/store/dn3p0i7j3k1igcs5fb94mjkw108bvms3-munin-2.0.37/lib/plugins/digitemp_: interpreter directive changed from "" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/dn3p0i7j3k1igcs5fb94mjkw108bvms3-munin-2.0.37/lib/plugins/perdition: interpreter directive changed from "" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/dn3p0i7j3k1igcs5fb94mjkw108bvms3-munin-2.0.37/lib/plugins/if_: interpreter directive changed from "/bin/bash" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/bash"
/nix/store/dn3p0i7j3k1igcs5fb94mjkw108bvms3-munin-2.0.37/lib/plugins/vserver_loadavg: interpreter directive changed from "/bin/bash" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/bash"
/nix/store/dn3p0i7j3k1igcs5fb94mjkw108bvms3-munin-2.0.37/bin/munin-check: interpreter directive changed from "/bin/bash" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/bash"
/nix/store/dn3p0i7j3k1igcs5fb94mjkw108bvms3-munin-2.0.37/bin/munin-cron: interpreter directive changed from "" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/dn3p0i7j3k1igcs5fb94mjkw108bvms3-munin-2.0.37...
moving /nix/store/dn3p0i7j3k1igcs5fb94mjkw108bvms3-munin-2.0.37/sbin/* to /nix/store/dn3p0i7j3k1igcs5fb94mjkw108bvms3-munin-2.0.37/bin
Removing references to /usr/{bin,sbin}/ from munin plugins...
/nix/store/dn3p0i7j3k1igcs5fb94mjkw108bvms3-munin-2.0.37

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: munin
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnfree = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnfree = true; }
to ~/.config/nixpkgs/config.nix.


Widget does not display

Environment
Operating System: Ubuntu version 18.04.4
Python Version: Python 3.7.8
How did you install ipysheet: conda
Python packages:
py37_rdkit.txt
Description of Issue
I don't seem to be able to display any widget. How should I solve it?
Thank you!

I have exactly the same issue, could you tell me if a solution was found?

maybe you need to install the ipysheet first and enable the widget to jupyter notebook using following cmd:
jupyter nbextension enable --py --sys-prefix ipysheet

Will be fixed by https://github.com/QuantStack/ipysheet/pull/200


Add a PR template the reminds one to write good commit messages

Signed-off-by: Sebastian Schuberth<EMAIL_ADDRESS>

@mnonnenmacher @fviernau how about doing this as an experiment?

@mnonnenmacher @fviernau how about doing this as an experiment?

I like that idea, but we should also add a link to the CONTRIBUTING.md because it contains more guidelines about PRs.

I like that idea, but we should also add a link to the CONTRIBUTING.md because it contains more guidelines about PRs.

Done.

I like that idea, but we should also add a link to the CONTRIBUTING.md because it contains more guidelines about PRs.

Done.

What I actually meant was to add a link to the template that points to CONTRIBUTING.md, to remind people to read that first. But we can add that later as well.

@sschuberth  If we add that template, will GitHub still use the commit message as PR message for single commit PRs?

What I actually meant was to add a link to the template that points to CONTRIBUTING.md, to remind people to read that first. But we can add that later as well.

Ah, I though we'd rely on GitHub's feature to automatically do this, at least for first-time contributors. But you're right, it seems a bit arbitrary to only quite parts of CONTRIBUTING.md but not the file itself.
So, should we always link to CONTRIBUTING.md only?

@sschuberth If we add that template, will GitHub still use the commit message as PR message for single commit PRs?

That's one of the things I wanted to try out after the merge. But I guess I could also create a dummy PR with this PR as the target branch...

What I actually meant was to add a link to the template that points to CONTRIBUTING.md, to remind people to read that first. But we can add that later as well.

Ah, I though we'd rely on GitHub's feature to automatically do this, at least for first-time contributors. But you're right, it seems a bit arbitrary to only quite parts of CONTRIBUTING.md but not the file itself.
So, should we always link to CONTRIBUTING.md only?

Yes, I think that would be best, then we don't have to repeat ourselves.

Yes, I think that would be best, then we don't have to repeat ourselves.

Done.

@sschuberth If we add that template, will GitHub still use the commit message as PR message for single commit PRs?

Yes, it will look like this:


Double click on a folder didn't open it (editor)

Same behavior as #36684.
Removed by mistake in #36426.
Fixes #36757.

Ups. I didn't see this PR and I resolved this in the same way. Hehe, so this is working by double check.

Thanks!


stanford parser setup error

Hi,
i am using the following code to parse a sentence using stanford parser and for setting the variables
import os
java_path = "C:/Program Files/Java/jdk1.8.0_60"
os.environ['JAVAHOME'] = java_path
os.environ['CLASSPATH'] ='C:\Users\ASUS\Downloads\stanford-parser-full-2015-12-09\stanford-parser-full-2015-12-09'
os.environ['STANFORD_MODELS'] = 'C:\Users\ASUS\Downloads\stanford-parser-full-2015-12-09\stanford-parser-full-2015-12-09'
from nltk.parse.stanford  import StanfordDependencyParser
dep_parser = StanfordDependencyParser(model_path='edu/stanford/nlp/models/lexparser/englishPCFG.ser.gz' )
sentences = dep_parser.raw_parse_sents(["""
The product was old but it was good.
An excellent product packed very well with a very prompt delivery !
"""])
print (str (sentences))
for line in sentences:
for sentence in line:
sentence.draw()
But I am getting the following error
Traceback (most recent call last):
File "C:/Users/ASUS/PycharmProjects/untitled2/parse.py", line 11, in
dep_parser = StanfordDependencyParser(model_path='edu/stanford/nlp/models/lexparser/englishPCFG.ser.gz' )
File "C:\Python34\lib\site-packages\nltk\parse\stanford.py", line 61, in init
key=lambda model_name: re.match(self._MODEL_JAR_PATTERN, model_name)
TypeError: unorderable types: NoneType() > NoneType()
Please help me
thanks

@sat230594 please take a look at: https://gist.github.com/alvations

@sat230594 are you still having problems after following the instructions from the github gist?


Create dataset with JSON file

Is your feature request related to a problem? Please describe.
Now we support dataset creation with excel, csv and txt files.
Let's also support json files.
Describe the solution you'd like
Allow Json file when making file type imported dataset.
Describe alternatives you've considered
Additional context
We should support json file type snapshot soon.

Serve side job: JSON parsing and JSON to CSV converting logic.
Clinet side job: Allow json file type upload and change some phrases.

During the 2nd integration test, JSON was not able to import.
Maybe there was some problem while merging # 908.
@Ksparknot is working on it.
PR will be created with a new branch.
2번째 통합테스트 중에 JSON이 import가 안되는 문제가 발생했습니다.
아마 #908을 merge하는 중에 문제가 생긴 것 같습니다.
관련해서 @ksparknot 님께서 작업 중입니다.
새로운 branch로 PR이 만들어질 예정입니다.


[HOLD for payment 2024-03-06] [$500] Web - Added amount remains when refresh page and start Manual request flow again

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

Version Number: 1.4.42-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
Action Performed:

Navigate to chat> +> Money request> Manual
Add amount and click Next
Refresh the page
Dismiss the RHN by clicking outside the modal
Navigate to chat> +> Money request> Manual

Expected Result:
Amount field should be empty
Actual Result:
Added amount remains on Request money after refreshing the page
Workaround:
unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/38435837/53fd610d-fa3c-402d-acd9-ed3eaa0a6a9b

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0113bb7d40c5152c91
Upwork Job ID:<PHONE_NUMBER>095025664
Last Price Increase: 2024-02-15
Automatic offers:

bernhardoj | Contributor | 0

We(applause) think that this might be related to wave #vip-vsb
cc @quinthar

Proposal
Please re-state the problem that we are trying to solve in this issue.
The issue is that after navigating to the manual money request flow, adding an amount, and then refreshing the page, the previously entered amount persists when the manual request flow is initiated again. This behavior is unexpected, as the amount field should be reset and empty when starting a new money request - especially after refreshing the page.
What is the root cause of that problem?
The root cause may involve the application state not being reset properly after the page refresh. I'm guessing the amount value is being cached or stored in a persistent state that is not cleared out on page refresh or when the money request modal is dismissed.
What changes do you think we should make in order to solve the problem?

Ensure that the component state managing the amount field in the money request flow is reset to its initial state when the modal is dismissed or when the page is refreshed.
Add logic to clear any relevant fields or cached values when the modal closes or during the unmounting phase of the component lifecycle.
Conduct tests across all supported platforms to confirm that the amount field is indeed empty when initiating a new manual money request after a page refresh.

What alternative solutions did you explore? (Optional)
Another solution could be to implement a check upon re-opening the manual money request flow to determine if a refresh event has occurred and reset the state accordingly.

Proposal
Please re-state the problem that we are trying to solve in this issue.
The previously filled data on the Chat > + > Request money > Manual page is not cleared when dismissing the RHN after a page refresh on the Chat > + > Request money > Manual > Confirmation page. The expected result when starting a new Request Money (after dismissing the RHN) is the form should be blank.
What is the root cause of that problem?
The root cause of the problem is the IOU.clearMoneyRequest call is never made in the problematic scenario. IOU.clearMoneyRequest is only called when IOURequestStartPage is unmounted but when refreshing url https://dev.new.expensify.com/create/request/confirmation/1/1111111111111011, IOURequestStartPage is not mounted, instead IOURequestStepConfirmation is mounted. Therefore the IOU.clearMoneyRequest is never called.
What changes do you think we should make in order to solve the problem?
Call IOU.clearMoneyRequest when IOURequestStepConfirmation is unmounted.
What alternative solutions did you explore? (Optional)
We could call IOU.clearMoneyRequest when dismissing RHN instead of the IOURequest components unmounting

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01e0203f76b7eed76a

PR is ready
cc: @thesahindia

PR is on staging

This was deployed to prod on 2/28, adjusting for 7 day hold.

Summarizing payment on this issue:

Contributor: @bernhardoj $500 PAID via Upwork
Contributor+: @thesahindia $500, please request via ND

Upwork job is here: https://www.upwork.com/jobs/~0113bb7d40c5152c91

$500 approved for @thesahindia


Refactor state library structure

This PR contains a refactor of the organization of modules related to state (Consensus.*, Blockchain_state, Protocol_state, Snark_transition, Internal_transition, and External_transition). These modules have been moved into a new library dependency pattern in order to help remove them from more functors moving forward. In particular, this has been done in order to enable proper integration of the External_transition_validation stuff in a way that is outside of the current signature pattern so that it doesn't make signature type equality any more painful than it already is.
Of primary note is that the Consensus library has been changed in order to decouple the Consensus definition from the definition of state that depends on Consensus related types as much as possible. This has been achieved by adding one small functor to consensus that provides consensus hooks that require access to blockchain state and protocol state separately. This functor is instantiated at the same layer that the states it requires are defined.
The transition objects are not defunctored in this PR, but the groundwork has been layed for them to be fully defunctored in the near future.

Can we remove Coda_intf? It doesn't look like it's being used anywhere

Can we remove Coda_intf? It doesn't look like it's being used anywhere

Yes, sorry. That was accidentally checked in. I ended up just keeping everything in coda_pow for this refactor.


terraform nomad guide

From this great Nomad guide by @xmonader on the TF Forum, this PR proposes to add a Nomad cluster guide to the Advanced section of the Terraform guides.
Some things to notice:

Added WireGuard access, along with the Planetary access
Set environment variables file to ease the process
Tested on main net and it works 100%
Added disk storage available on all client and server nodes

Thanks to @AbdelrahmanElawady for the two FLists used in this guide!

@xmonader Hi Thabet!
do you think you can have a look at this PR? It's very close to your Nomad terraform file.
Thanks!


TripleO - Deployment configurations

It would be nice if in each step you add in witch node should be executed the commands
like
01 - Create the toor user. (installer node)
02 - Prepare the hypervisor node. (computes nodes)

Done! I just added some clarification notes. Hope they help you.


The difference between "storage per Partition" and "Partition size" is unclear.

[Enter feedback here]
There are "storage per Partition" and "Partition size" columns on  Storage limits Article. This difference is very unclear.
I think that "Storage per partition" shows this limit  because Free tier has 50 MB as service limit. But all other values are the same  between "storage per Partition" and "Partition size". Because there are no explanations or annotation,  it is difficult to understand it.
Could you please revise to be clear this columns?

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

ID: 05e60837-26b9-1f5a-7a44-88ae8c9a7bc3
Version Independent ID: eaa6fabe-3d51-d626-3a51-d2e46efca454
Content: Service limits for tiers and skus - Azure AI Search
Content Source: articles/search/search-limits-quotas-capacity.md
Service: cognitive-search
GitHub Login: @HeidiSteen
Microsoft Alias: heidist

@y10e
Thanks for your feedback! We will investigate and update as appropriate.

Your feedback has been shared with the content owner for further review. Thanks again for the feedback!

Thanks @y10e for the feedback. I've made the updates, but waiting for tech review before signing off.  I removed the duplicate row and added a footnote that reaffirms the free tier has storage (50 MB) but no partition.

The PR is merged and you should see the updates tomorrow, 1/3, around 10:00 a.m. PST.  Thanks again for the feedback.
#please-close


Fixed issue caused by multiple dots in file name

Having more than one dot in file name will cause situation when references will not be found in html files.
Example (debug info on screenshots just to make it easier to understand):
with default settings seems like there is no problem at all

but if we change options.length to 8 (witch is exactly length of word "prefixed") we will get this

Improving your RegEx a little we can assure that we getting the right spot for our hash, just before last dot.
P.S. Unfortunately my changes breaks some of your tests witch i personally don't see any use of. :(

Thanks for notifying me. That regex really does need updating to account for a few more use cases.
some of your tests which i personally don't see any use of

Which ones in particular?

You can see them in travis log. I'm not familiar with nodeunit so its hard to understand a real use cases for those tests. Maybe i'm wrong...

I'm struggling to find a good solution to this problem - ultimately it's best to use a long hash value to avoid this.
I have implemented my own solution at https://github.com/hollandben/grunt-cache-bust/pull/105/files?diff=unified#diff-c8079eb1d55a5ec5db4c263df129a086R46

I cant see how is current solution is different from original. Would you be so kind to explain the difference?

The first regex is wrong as it still contains the escaped strings. Here is the correct version - http://regexper.com/?#%2F\.([a-zA-Z0-9]{8})(\.\w%2B)%24%2Fgi
The newer implementation I merged earlier is better than what was there but still not great. It will now check for a series of characters matching the length of the hash provided, followed by a period and then more text till the end of the string. It covers the case where you are re-busting an asset. Here is an example imagining that the length is set to 4
// Before, this would have replaced the `path` with nothing before applying the new hash
/assets/file.path.1234.js // <= input
/assets/file1234.js // <= result

// Now it will remove the hash before applying the new hash
/assets/file.path.1234.js // <= input
/assets/file.path.js // <= result

Unfortunately I can't figure out a way to deal with the following case:
/assets/file.path.js // <= input
/assets/file.1234.js // <= result

Hopefully that makes more sense

Oh, right.
Since your solutions is basically equivalent to mine i think we can close this PR now :)


Linking to other peoples stuff?

Hey Scott,
This seems like a really cool idea! :smile: What do you think about linking to other peoples "tutorials" for distributed systems tools?
e.g.

https://github.com/kelseyhightower/kubernetes-fleet-tutorial
https://github.com/bketelsen/coreos-kubernetes-digitalocean

@JKhawaja I certainly don't have a problem adding links to the README.md to point people to other useful projects, but I don't have time to maintain such a curated list of useful links. I would be open, however, to merging a PR that added such links, if you're interested in contributing. I'd suggest adding a new section to the README.md file to hold links to other useful projects.

That sounds simple enough. I'll try to remember to do that soon! Thanks! :)

@JKhawaja If you need/want any help in actually creating and submitting the PR, let me know. Happy to help walk you through the process. (If you're familiar with the process, then just ignore this.) :-)
I'll leave this issue open until such time as you (or someone else) gets around to creating the PR, at which point you can reference this issue number in the commit message for reference.

@lowescott would you like me to just name the section in the readme 'Other Useful Projects'?

@lowescott  please let me know if there is something about the PR that you want me to change.

@JKhawaja Just merged your PR. Thanks! Closing this issue now.


Responsive Design

Perma | TNW
Original Page link
Requirements
• Using sementic tags ✅
• Using positioning elements ✅
• Inserting Flexbox and Grid systems ✅
• Build with responsive design ✅

@sinansevgi thank you so much for your comments and suggestions, they helped us a lot.
We made the changes you suggested for the project in general and also to the Highlights, Latest Funding, and Footer sections.
Have a good day.


Weird behavior of Sccoda

Report
When I ran this snippet code:

It gave me the unexpected MuData: obs of coda was 'gender', not 'condition'.
condition has 2 categorical variables: 'T2D' and 'non-diabetic. This dataset has four batches: 'batch 1', 'batch 2', 'batch 3', 'batch 4'.
Version information
pertpy 0.8.0

It gave the unexpected MuData: obs of coda was 'gender', not 'condition'.
condition has 2 categorical variables: 'T2D' and 'non-diabetic. This dataset has four batches: 'batch 1', 'batch 2', 'batch 3', 'batch 4'.

Sorry, I don't quite understand the problem. Did you get a traceback or what is the issue?
Thanks!

In your tutorial, when you assign ['condition'] to covariate_obs, coda object will have 'condition' obs. But when I followed the tutorial, I received a coda object with 'gender' obs instead of 'condition' obs.
This is my adata.obs:

Okay, thanks. I'll look into it as soon as possible

My Anndata object only has one value for gender obs: 'male'. I want to compare the cell composition between two groups: T2D and non-diebetic of 'condition' obs.

Okay, thanks. I'll look into it as soon as possible

Thank you very much.

Would it be possible for me to have your dataset or would you be able to create a minimal working example to reproduce the bug, please?

This is the dataset.
I ran these code lines: sccoda_model_draft = pt.tl.Sccoda()
sccoda_data_draft = sccoda_model_draft.load(
adata_1,
type="cell_level",
generate_sample_level=True,
cell_type_identifier="celltype",
sample_identifier="batch",
covariate_obs=["condition"],
)
sccoda_data_draft
MuData object with n_obs × n_vars = 3803 × 4013
2 modalities
rna:	3799 x 4000
obs:	'gender', 'age', 'condition', 'batch', 'celltype'
layers:	'counts'
coda:	4 x 13
obs:	'gender'
var:	'n_cells'

I can not upload the data on here. Can I send the data to you via your email?

Yes you can! You can find my email on my Github profile. You can also send me a link to a file hoster there.

And by the way, this dataset composes of only four batches. Do you know the way to increase the statistical significance of each cell type in the dataset. Now I have to set the fdr equaling to 0.5, If I set it to a lower value,  I will not get any information about the changes of the cell types. I mean that when I ran:
sccoda_model_male.credible_effects(sccoda_data_male_final, modality_key="coda")
It return all False values.

I'll check the issue out tomorrow. On my TODO list.

And by the way, this dataset composes of only four batches. Do you know the way to increase the statistical significance of each cell type in the dataset. Now I have to set the fdr equaling to 0.5, If I set it to a lower value, I will not get any information about the changes of the cell types. I mean that when I ran:
sccoda_model_male.credible_effects(sccoda_data_male_final, modality_key="coda")
It return all False values.

You can consider testing healthy versus one condition separately instead of testing vs all non healthy conditions jointly. If this is not an option for your dataset, I am afraid that you may not have any significant shifts in composition. But maybe @johannesostner wants to chip in?

No, the condition variable of the dataset only comprises two values: non-diabetic and T2D. I know that if the model uses the Gaussian distribution as null model, the dataset must contain many batches to use this assumption according to the central limit theorem.

Regarding significance:
Even though scCODA does not use the Gaussian as a null distribution (it's a Bayesian model, so there is no "Null model", but rather the posterior gets informed by the prior and the observations), the concept of more samples -> higher power still holds.
Therefore, there is no way to increase statistical significance without violating some foundational statistical principles.

Regarding significance:
Even though scCODA does not use the Gaussian as a null distribution (it's a Bayesian model, so there is no "Null model", but rather the posterior gets informed by the prior and the observations), the concept of more samples -> higher power still holds. Therefore, there is no way to increase statistical significance without violating some foundational statistical principles.

So scCODA can only be applied for the data which composes of many batches or can we create batches by ourselves (just randomly creating samples for the statistical test)?

This depends on the effect strength. For very strong effects, small sample sizes are sufficient to detect them also at lower FDR levels. For weaker effects, you will need more samples to find them with high credibility.
Randomly creating samples (e.g. through bootstrapping) is a known practice, but must be done correctly to not violate statistical principles. Nevertheless, this will not improve your statistical power if done correctly

If I randomly assign each cell to one of a list from 'batch 1' to 'batch 10', will this technique enhance the statistical power?

I identified the issue and pushed a fix to main. However,
adata.obs.groupby("batch").nunique() shows that your batches do not only have single conditions but several.
The expectation is:

But your dataset results in:

You identify that by a warning.
Could you please test with main?

What do you mean "test with main"?

@ngvananh2508 sorry for my brievity

pip install git+https://github.com/pertpy/pertpy.git@main

@ngvananh2508 sorry for my brievity

pip install git+https://github.com/theislab/pertpy.git@main

Thank you. I will test it and let you know the result ASAP.

@ngvananh2508 sorry for my brievity

pip install git+https://github.com/theislab/pertpy.git@main

When I install this pertpy on a new env and import it, it raise this error:
ModuleNotFoundError: No module named 'arviz'
ModuleNotFoundError: No module named 'pydeseq2'
ModuleNotFoundError: No module named 'formulaic'
ModuleNotFoundError: No module named 'toytree'
I can get around these issues by installing them, but I still report in case you want to modify your library

I identified the issue and pushed a fix to main. However,
adata.obs.groupby("batch").nunique() shows that your batches do not only have single conditions but several.
The expectation is:

But your dataset results in:

You identify that by a warning.
Could you please test with main?

So you mean that each batch only consists of one condition?


Composer: update dev dependencies

Signed-off-by: Roman Ondráček<EMAIL_ADDRESS>

It requires to work on it Roman. ;-) Could you please fix it? @Roman3349

@Roman3349 Hey, could you rebase it? Thx.

@f3l1x Done


Add sample data for practice PE

Add sample data with the correct dates so that the Today section is populated with data

This includes tags, some weekly repeatable events so testers can test all the features


Add vercel support - update to nextjs 12

Adds vercel support.
Current state
This app wont deploy to vercel.com
Desired state
This app will easily deploy to vercel.com
The problem
The problem was probably caused by vercel not supporting older versions of nextjs - im no sure.
Whatever, the build was not passing anyway, and after my changes - it does pass.
Error from vercel deployment
Export successful. Files written to /vercel/path0/out
--
21:14:52.682 | Done in 37.26s.
21:14:52.693 | Error: The file "/vercel/path0/.next/routes-manifest.json" couldn't be found. This is normally caused by a misconfiguration in your project.
21:14:52.693 | Please check the following, and reach out to support if you cannot resolve the problem:
21:14:52.693 | 1. If present, be sure your `build` script in "package.json" calls `next build`.  2. Navigate to your project's settings in the Vercel dashboard, and verify that the "Build Command" is not overridden, or that it calls `next build`.  3. Navigate to your project's settings in the Vercel dashboard, and verify that the "Output Directory" is not overridden. Note that `next export` does **not** require you change this setting, even if you customize the `next export` output directory. Learn More: https://err.sh/vercel/vercel/now-next-routes-manifest

The netlify build is failling, because it runs on older version of node
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">=12.22.0". Got "12.18.0"


Feature/#17 angular imeplement routing

Added material module
Implemented routes between home and template selection

I think that in slack it was said that, for now, @brauliodiez would review the angular project. Maybe you should add him as a reviewer.


Trev8939/wms layer

Adds wms layer url sample

Thanks @TADraeseke - it's working OK for me now. Given your comments about intent of sample and external service above, worth adding a note to the readme to this effect before merging I think. Apart from that looks good to merge.


Update prompt code example to be a prompt

Fixxes this isue #150

Hi @vuchl
Thanks a lot for this fix. I've merged and I will release this ASAP in the website. :)


Camel-xslt-quarkus: add extension functions support

Camel-xslt-quarkus uses hardcoded Xalan org.apache.xalan.xsltc.trax.TransformerFactoryImpl to compile XSL at the build-time.
Also feature FEATURE_SECURE_PROCESSING is enabled by default. Which means that Xalan extension functions cannot be used.
Related to JVM and Native modes.
Proposal:

Add setting quarkus.camel.xslt.transformerFactory
Add setting quarkus.camel.xslt.extensionFunctions

Example:
package my.extentionfunctions;

public class Functions {
    public static XObject customFunc() throws Exception {
        return new XObject(...);
    }
}

<xsl:stylesheet version="1.0"
        ...
        xmlns:myfunc="xalan://my.extensionfunctions.Functions"
        ... >
        <node>
            <xsl:value-of select="myfunc:customFunc()"/>
        </node>
</xsl:stylesheet>

Proposal:

Add setting quarkus.camel.xslt.transformerFactory
Add setting quarkus.camel.xslt.extensionFunctions

I guess those should be new configuration parameters, right? Could you please specify their expected range of values, defaults, and explain what they should actually do?

Hi @ppalaga, to be able to use extension functions in JVM mode (as well as in Native mode), these configuration steps have to be done:
1. Set FEATURE_SECURE_PROCESSING to false
2. Specify extension function classes, so they will be registered for reflection to run in Native mode
Step 1. can be achieved in different ways:

add new setting quarkus.camel.xslt.secureProcessing with values true/false
add new setting quarkus.camel.xslt.features with comma separated features, that should be enabled by default. For example,  quarkus.camel.xslt.features=http://javax.xml.XMLConstants/feature/secure-processing,http://xml.apache.org/xalan/features/optimize,...
add new setting quarkus.camel.xslt.transformerFactory to specify custom class, where developer can override features that are enabled by default. For example quarkus.camel.xslt.transformerFactory=my.namespace.MyTrasformerFactory

Step 2. can be achieved by specifying classNames, so xslt component will register them automatically for reflection using new ReflectiveClassBuildItem(...).
Example: quarkus.camel.xslt.extensionFunctions=my.extensionfunctions.Functions1,my.extensionfunctions.Functions2,...
To summarize:
Taking into the account existing Camel setting transformerFactoryClass I think that it would be reasonable to have these 2 settings:

quarkus.camel.xslt.transformerFactory=my.namespace.MyTrasformerFactory
Also it would be usefult to be able to override transformerFactoryConfigurationStrategy
quarkus.camel.xslt.extensionFunctions=my.extensionfunctions.Functions1,my.extensionfunctions.Functions2,...

But maybe I'm missing something and it could be implemented in simpler way.

Thanks for the explanation @DenisIstomin, it is much clearer to me now!
Step 1:
I think giving users the option to set their own TrasformerFactory could awake the impression that any transformer factory implementation would work. In fact our extension expects this particular Xalan TrasformerFactory. So I'd be in favor of allowing end users to configure this factory via config params rather than passing their own class.
Now how could we make the factory configurable. I'd vote for having a generic map of parameters, something like:
quarkus.camel.xalan.feature."http://javax.xml.XMLConstants/feature/secure-processing" = true
quarkus.camel.xalan.feature."http://xml.apache.org/xalan/features/optimize" = true

They have something similar in Quarkus Agoal config https://github.com/quarkusio/quarkus/blob/main/extensions/agroal/runtime/src/main/java/io/quarkus/agroal/runtime/DataSourceJdbcRuntimeConfig.java#L138-L139
The defaults could be mainitained in a Map containing the current only feature javax.xml.XMLConstants.FEATURE_SECURE_PROCESSING, true
Step 2.:
I do not think we need something special only for the reflection registration. Given that those functions will mostly be provided by end users themselves, can't they simply add @io.quarkus.runtime.annotations.RegisterForReflection to the classes?

If you like the above, please feel free to send a PR.

Thanks @ppalaga, I'm working on providing PR.
I verified that extension functions work with my local changes that disable FEATURE_SECURE_PROCESSING.
There is also no need for any improvements to make them work in Native mode.
There is only one issue: extension functions work only when they are defined in a separate jar, otherwise QuarkusClassloader is not able to load class from target folder when you run tests. Because all classes from ApplicationPath are marked as banned and cannot be loaded.
I'll address this issue in the upcoming PR.

Fixed via #3526


Wrong repo path leads to unclear error

When you include more than one {} in your repo path you will get an unclear error:
DEBUG    inmanta.module:module.py:286 could not clone repo
Traceback (most recent call last):
  File "/home/bart/dev/bics_ce/env/lib64/python3.6/site-packages/inmanta/module.py", line 279, in clone
    url = self.baseurl.format(name)
IndexError: tuple index out of range

This can happen for example when you forget the the repo separator is   and not :
The indexerror can only happen in such a case and should be transformed in a clear error message.

use replace '{}' instead of 'format'
allow only one occurrence of '{}'

Hey team! Please add your planning poker estimate with ZenHub @arnaudsjs @bartv @FloLey @Hugo-Inmanta @sanderr

Hello there! May I know if I can work on this issue? I'm a beginner in Open Source and would like to contribute. :)

Welcome @Aditya1402 ,
You can work on this issue, but of course you fist have to know what Inmanta is and how to run the tests.
For some introductory materials into Inmanta, take a look at the quickstart guide: https://docs.inmanta.com/community/latest/quickstart.html
For running the tests, you need to have postgresql installed on your system (because we use pytest-postgresql and some form of 'make'
You can then run the test with make install make test. If this work, you can pick up this ticket, but don't be disappointed if someone else resolves the ticket first.
When you create a Pull Request, we will send you the CLA (contributor license agreement).
Wouter


Fix unsupported PNPM error message

What are you trying to accomplish?
Give good error messages.
Anything you want to highlight for special attention from reviewers?
PNPM 9 is now supported.
How will you know you've accomplished your goal?
New error reads properly.
Checklist

[ ] I have run the complete test suite to ensure all tests and linters pass.
[ ] I have thoroughly tested my code changes to ensure they work as expected, including adding additional tests for new functionality.
[ ] I have written clear and descriptive commit messages.
[ ] I have provided a detailed description of the changes in the pull request, including the problem it addresses, how it fixes the problem, and any relevant details about the implementation.
[ ] I have ensured that the code is well-documented and easy to understand.

Hey hey 👋 what kinds of errors are you seeing currently @deivid-rodriguez ?

I'm not seeing any errors, I just saw this in the code base and noticed the message needed an update :)
I do suspect there's something not quite working with corepack version switching because I noticed that https://github.com/dependabot/dependabot-core/commit/176339f93709a24ca78d8af9d8192c3c20d584cb was already present before upgrading PNPM at #10073. So my understanding is that corepack should've been switching to PNPM 9 when finding an modern PNPM lockfile and not revert the lockfile format, even without #10073.
So I will dig a bit more into that, in case there's something else that needs fixing!


Update Pinta PPA with recent versions

Description
Make available Pinta 2.1.1 and further versions as debs, in Pinta PPA https://launchpad.net/~pinta-maintainers/+archive/ubuntu/pinta-stable, for Jammy and further Ubuntu versions.
Additional context
From Ubuntu Jammy, 22.04 LTS, and further versions (Mantic; Noble future 24.04 LTS...), "dotnet-sdk-8.0" is available as a package from Ubuntu repository. See https://pkgs.org/download/dotnet-sdk-8.0
This makes possible to have Pinta 2.1.1 available from a launchpad PPA, since all the dependencies can be found in Ubuntu repository and since the installation of .Net 8.0 from Microsoft is no longer required.

Yep, having dotnet available definitely helps. I think the additional problem is that the PPA builder doesn't have internet access, so Pinta's dependencies on Nuget packages involve some extra complexity to get those bundled up and working
The maintainer of this PPA (https://xtradeb.net/apps/pinta/) worked out how to solve those issues, so similar changes could be ported over to the official PPA in the future. I'd suggest using that PPA for now though!

any news for 24.04 ?
23.04 ppa don't work

There are no immediate plans since I don't have the development time to work on maintaining more packages. I'd suggest using the PPA linked in the previous comment, or using the snap / flatpak packages

I can't understand why people do this to themselves, it's so bad with snap, and it adds so much unnecessary and heavy dependency :/.

I would suggest contacting the maintainers of https://xtradeb.net/apps/pinta/ to ask them to update their PPA for 24.04 (there are already packages for 23.10 and 22.04 by the look of it)

22.04 use python 3.10
23.04 use python 3.11
so i can't use old ppa :/

Their webpage has links to their contact email and bug tracker

The Pinta snap is one of the best snaps going!
It's also the smallest version of Pinta, at just 34.4 MB installation space used. The flatpak for example is 100MB. The Windows installer is 50MB download and 220MB once installed. (N.B, this assumes we're removing duplicated packages, which both the Flatpak and Snap do).
From Ubuntu 22.04 where you'd already have the Gnome snap and the rest installed, it literally really is 34.MB installed.
If it isn't a satisying experience, I'll personally confront the guy who made it ;).


[Carry 17414] Added additional container information to "docker info".

Carry #17414.
WAIT BEFORE MERGE: This PR is blocked by docker/engine-api#36

Vendoring is happening in #19243.

two really tiny nits, LGTM otherwise

fixed.

LGTM
moving to "merge", taking previous LGTM's from https://github.com/docker/docker/pull/17414  into account

:sob: janky, nooooo
01:17:45 Build timed out (after 120 minutes). Marking the build as failed.


Proper TCP socket shutdown; Generic TCP timeout utils; built-in HTTP server timeouts; update docu

This PR implements:

A TcpShutdown trait with proper socket close and abort functionality for both STD and embassy-net. Needs testing for embassy-net
Generic TCP "timeout" decorators in edge_net which can be used by anyone
Built-in timeouts (both for each IO operation, as well as for the whole request-response) for the HTTP server

I tested with esp-wifi + embassy-net and I can confirm that this fixes the issues I've been having with closing connections:
Here's a trace for a request issued with "Connection: Close" and following the redirection.

Here's a trace for a request issued with "Connection: Keep-Alive" and following the redirection.

LGTM.
Thank you very much for taking care of this.
If some additional optional parameters are ever added to run(), it may become more user friendly to switch to a builder pattern, but at this point, this is more about UX than usability, and we should focus on usability for now.

If some additional optional parameters are ever added to run(), it may become more user friendly to switch to a builder pattern, but at this point, this is more about UX than usability, and we should focus on usability for now.

Agreed. A builder pattern can wait a bit.


How do you run this?

Hi, thanks for all the effort that you have put into this. From the description this seems to be the only DukasCopy downloader that can download OHLCV Seconds Data in csv format. The only issue that I have is: How do I use it? Would you be kind enough (again), to state how a user may go about using this?
Thank you.

Hi,
dukas.net -s "2020-01-01 00:00:00" -e "2020-12-31 23:59:59" -p DEUIDXEUR,GBRIDXGBP,USA30IDXUSD,AUSIDXAUD,LIGHTCMDUSD -o "e:\temp" -q Both --major-scale Min --minor-scale 1 --file-scale Day --header -g
Please follow the
All is described in readme file. Please let me know, if it's unclear or wrong. I'll update accordingly.

Hi thanks for the reply. The question was in relation to installation instructions. There was no installation instructions. I googled and figured it out. Thanks


Add name attribute to BeamPage

Sorry for yet another tiny pull request, I know the overhead of releases adds up.
This adds the ability to pass through page names that navigator observers can use. Again, it's quite useful for debugging and error reporting to follow through the navigation paths that users have taken.
For example,

No worries. I appreciate your interest in this project.