failed to join/concatenate datasets for model CESM2-FV2

I use the following code to load CESM2-FV2
from cmip6_preprocessing.preprocessing import combined_preprocessing

url = "https://raw.githubusercontent.com/NCAR/intake-esm-datastore/master/catalogs/pangeo-cmip6.json"
col = intake.open_esm_datastore(url)
model = 'CESM2-FV2'

query = dict(experiment_id=['historical'], table_id='Omon',
             variable_id='tos', grid_label=['gn'], source_id=model)
cat = col.search(**query)
print(cat.df['source_id'].unique())
z_kwargs = {'consolidated': True, 'decode_times':False}
tos_dict = cat.to_dataset_dict(zarr_kwargs=z_kwargs, preprocess=combined_preprocessing)

and I get the following error:
AggregationError:
        Failed to join/concatenate datasets in group with key=CMIP.NCAR.CESM2-FV2.historical.Omon.gn along a new dimension `member_id`.
        *** Arguments passed to xarray.concat() ***:
        - objs: a list of 3 datasets
        - dim: <xarray.DataArray 'member_id' (member_id: 3)>
array(['r1i1p1f1', 'r2i1p1f1', 'r3i1p1f1'], dtype='<U8')
Dimensions without coordinates: member_id
        - data_vars: ['tos']
        - and kwargs: {'coords': 'minimal', 'compat': 'override'}
        ********************************************

Thank you for using cmip6_preprocessing and bringing up this issue.
I can reproduce this but am not sure yet, why this is happening. Currently I am having some issues with the pangeo cloud, but I will follow up later.

Ok I think that I have a better idea what is happening. If I try to not concat in the member_id dimension:
tos_dict_pp = cat.to_dataset_dict(
    zarr_kwargs=z_kwargs,
    preprocess=combined_preprocessing,
    aggregate=False
)

and then try to combine them manually
# try to manually concat preprocessed datasets
xr.concat([a for a in tos_dict_pp.values()], dim='member_id')

The error is more informative:
ValueError: cannot reindex or align along dimension 'y' because the index has duplicate values

@andersy005 I think in general it would be nice to have a way to display these errors from within intake-esm. Is that possible currently?
The specific problem here is that the lon/lat values in this particular model contain very high values (which should be masked out)!
If we load the model without preprocessing and look at the latitude field
tos_dict = cat.to_dataset_dict(
    zarr_kwargs=z_kwargs,
    preprocess=None,
    aggregate=False
)
list(tos_dict.values())[0].lat.plot(vmax=200)

we get this:

The land points have values of 1e37. In the newest version this should be fixed here.
@sckw could you confirm which version you are using? You can do that by running
import cmip6_preprocessing
cmip6_preprocessing.__version__

@jbusecke I get 'earthcube_commit+58.g1d57184'

That is an older version, but I just tested with the current version from github and the bug is still there. Ill see how I can fix this and once this is resolved Ill release a new version, which you can then upgrade to with conda install -c conda-forge cmip6_preprocessing.

Thank you!

@andersy005 I think in general it would be nice to have a way to display these errors from within intake-esm. Is that possible currently?

We are already doing this. It's just that @sckw didn't post the first 2/3 of the traceback which includes the ValueError: cannot reindex or align along dimension 'y' error:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
~/devel/intake/intake-esm/intake_esm/merge_util.py in join_new(dsets, dim_name, coord_value, varname, options, group_key)
     55         concat_dim = xr.DataArray(coord_value, dims=(dim_name), name=dim_name)
---> 56         return xr.concat(dsets, dim=concat_dim, data_vars=varname, **options)
     57     except Exception as exc:

~/opt/miniconda3/envs/intake-esm-dev/lib/python3.8/site-packages/xarray/core/concat.py in concat(objs, dim, data_vars, coords, compat, positions, fill_value, join, combine_attrs)
    190         )
--> 191     return f(
    192         objs, dim, data_vars, coords, compat, positions, fill_value, join, combine_attrs

~/opt/miniconda3/envs/intake-esm-dev/lib/python3.8/site-packages/xarray/core/concat.py in _dataset_concat(datasets, dim, data_vars, coords, compat, positions, fill_value, join, combine_attrs)
    382     datasets = [ds.copy() for ds in datasets]
--> 383     datasets = align(
    384         *datasets, join=join, copy=False, exclude=[dim], fill_value=fill_value

~/opt/miniconda3/envs/intake-esm-dev/lib/python3.8/site-packages/xarray/core/alignment.py in align(join, copy, indexes, exclude, fill_value, *objects)
    339         else:
--> 340             new_obj = obj.reindex(copy=copy, fill_value=fill_value, **valid_indexers)
    341         new_obj.encoding = obj.encoding

~/opt/miniconda3/envs/intake-esm-dev/lib/python3.8/site-packages/xarray/core/dataset.py in reindex(self, indexers, method, tolerance, copy, fill_value, **indexers_kwargs)
   2545         """
-> 2546         return self._reindex(
   2547             indexers,

~/opt/miniconda3/envs/intake-esm-dev/lib/python3.8/site-packages/xarray/core/dataset.py in _reindex(self, indexers, method, tolerance, copy, fill_value, sparse, **indexers_kwargs)
   2574
-> 2575         variables, indexes = alignment.reindex_variables(
   2576             self.variables,

~/opt/miniconda3/envs/intake-esm-dev/lib/python3.8/site-packages/xarray/core/alignment.py in reindex_variables(variables, sizes, indexes, indexers, method, tolerance, copy, fill_value, sparse)
    549             if not index.is_unique:
--> 550                 raise ValueError(
    551                     "cannot reindex or align along dimension %r because the "

ValueError: cannot reindex or align along dimension 'y' because the index has duplicate values

The above exception was the direct cause of the following exception:

AggregationError                          Traceback (most recent call last)
<ipython-input-4-84f1708f03d1> in <module>
      4 print(cat.df['source_id'].unique())
      5 z_kwargs = {'consolidated': True, 'decode_times':False}
----> 6 tos_dict = cat.to_dataset_dict(zarr_kwargs=z_kwargs, preprocess=combined_preprocessing)

~/devel/intake/intake-esm/intake_esm/core.py in to_dataset_dict(self, zarr_kwargs, cdf_kwargs, preprocess, storage_options, progressbar, aggregate)
    925             ]
    926             for i, task in enumerate(concurrent.futures.as_completed(future_tasks)):
--> 927                 key, ds = task.result()
    928                 self._datasets[key] = ds
    929                 if self.progressbar:

~/opt/miniconda3/envs/intake-esm-dev/lib/python3.8/concurrent/futures/_base.py in result(self, timeout)
    430                 raise CancelledError()
    431             elif self._state == FINISHED:
--> 432                 return self.__get_result()
    433
    434             self._condition.wait(timeout)

~/opt/miniconda3/envs/intake-esm-dev/lib/python3.8/concurrent/futures/_base.py in __get_result(self)
    386     def __get_result(self):
    387         if self._exception:
--> 388             raise self._exception
    389         else:
    390             return self._result

~/opt/miniconda3/envs/intake-esm-dev/lib/python3.8/concurrent/futures/thread.py in run(self)
     55
     56         try:
---> 57             result = self.fn(*self.args, **self.kwargs)
     58         except BaseException as exc:
     59             self.future.set_exception(exc)

~/devel/intake/intake-esm/intake_esm/core.py in _load_source(key, source)
    911
    912         def _load_source(key, source):
--> 913             return key, source.to_dask()
    914
    915         sources = {key: source(**source_kwargs) for key, source in self.items()}

~/devel/intake/intake-esm/intake_esm/source.py in to_dask(self)
    244     def to_dask(self):
    245         """Return xarray object (which will have chunks)"""
--> 246         self._load_metadata()
    247         return self._ds
    248

~/opt/miniconda3/envs/intake-esm-dev/lib/python3.8/site-packages/intake/source/base.py in _load_metadata(self)
    124         """load metadata only if needed"""
    125         if self._schema is None:
--> 126             self._schema = self._get_schema()
    127             self.datashape = self._schema.datashape
    128             self.dtype = self._schema.dtype

~/devel/intake/intake-esm/intake_esm/source.py in _get_schema(self)
    173
    174         if self._ds is None:
--> 175             self._open_dataset()
    176
    177             metadata = {

~/devel/intake/intake-esm/intake_esm/source.py in _open_dataset(self)
    230         n_agg = len(self.aggregation_columns)
    231
--> 232         ds = _aggregate(
    233             self.aggregation_dict,
    234             self.aggregation_columns,

~/devel/intake/intake-esm/intake_esm/merge_util.py in _aggregate(aggregation_dict, agg_columns, n_agg, nd, mapper_dict, group_key)
    238         return ds
    239
--> 240     return apply_aggregation(nd)
    241
    242

~/devel/intake/intake-esm/intake_esm/merge_util.py in apply_aggregation(nd, agg_column, key, level)
    194             agg_options = {}
    195
--> 196         dsets = [
    197             apply_aggregation(value, agg_column, key=key, level=level + 1)
    198             for key, value in nd.items()

~/devel/intake/intake-esm/intake_esm/merge_util.py in <listcomp>(.0)
    195
    196         dsets = [
--> 197             apply_aggregation(value, agg_column, key=key, level=level + 1)
    198             for key, value in nd.items()
    199         ]

~/devel/intake/intake-esm/intake_esm/merge_util.py in apply_aggregation(nd, agg_column, key, level)
    216         if agg_type == 'join_new':
    217             varname = dsets[0].attrs['intake_esm_varname']
--> 218             ds = join_new(
    219                 dsets,
    220                 dim_name=agg_column,

~/devel/intake/intake-esm/intake_esm/merge_util.py in join_new(dsets, dim_name, coord_value, varname, options, group_key)
     69         """
     70
---> 71         raise AggregationError(message) from exc
     72
     73

AggregationError:
        Failed to join/concatenate datasets in group with key=CMIP.NCAR.CESM2-FV2.historical.Omon.gn along a new dimension `member_id`.

        *** Arguments passed to xarray.concat() ***:

        - objs: a list of 3 datasets
        - dim: <xarray.DataArray 'member_id' (member_id: 3)>
array(['r1i1p1f1', 'r2i1p1f1', 'r3i1p1f1'], dtype='<U8')
Dimensions without coordinates: member_id
        - data_vars: ['tos']
        - and kwargs: {'coords': 'minimal', 'compat': 'override'}

        ********************************************

Ah ok, thanks for following up. I have to admit I also did not see it in there. Gotta look closer next time.
@sckw I am almost ready with #62 (which also confirms this issue for other variables of the same model), and will work on a fix as soon as that is done.


files with unicode characters are not exported

needs a bit more testing, but user submitted issue can be seen with a PDF linked in the source as such:
<a href="/contents/data/2018/06/2018MayQUIROPRÁCTICA_Y_BIENESTAR.pdf" data-elementor-open-lightbox="" target="_blank">
Which encodes to https://quiropractica.org/contents/data/2018/06/2018MayQUIROPR%C3%81CTICA_Y_BIENESTAR.pdf

same happening to thai characters inside folder and urls


isso.dispatch does not work with the single site

Hello!
I've configured isso to render comments for single site - using latest isso/gunicorn installed (pip --user install)  on the Debian-10 server and launching with systemd's user.
Since I'd like to use isso as commenting engine for few sites, I've tried to invoke it with the isso.dispatch, but as soon as I switch the comments disappear and after restarting the service invoking isso.run the comments do reappear.
Any hint?

Setting up isso in multi-site mode adds a subdir in the isso URL for each site, did you adapt your website configuration accordingly?
See also https://posativ.org/isso/docs/setup/multiple-sites/

Ahh, thanks for the pointer!
Now I modified webconfig  and I can e.g. access:  https://tld/foo/admin which e.g. does list my comments as Valid, but on the web site there is only empty comment form, but no old comments are rendered.
Any (further) hint?


External files or multifile app does not work

Hi. Just started testing shinylive and I have a quick question. I need either a small json file for my app.py or an external module (not pip install just a local wordlist.py file). When I wanted to use shinylive for my app (and files) inside app/ folder and I want to export the outputs to docs/ folder. I use the following command.
shinylive export app docs

When I serve it on the localhost with python3 -m http.server --directory docs --bind localhost 8008 I get the following result.

I know it can be done due to this example: https://shinylive.io/py/examples/#regularization
Perhaps I missed a documentation for multi-file shinylive apps. I can do a workaround with a bloated app.py but knowing how to work with multiple files will be incredible.
Thanks for such an awesome package. It will get better and better I presume.

+1
I'm having a hard time modularizing an app. I have the server and ui broken into parts like this:
server.R
server <- function(input, output) {

  # choose columns to display
  diamonds2 = diamonds[sample(nrow(diamonds), 1000), ]
  output$mytable1 <- DT::renderDataTable({
    DT::datatable(diamonds2[, input$show_vars, drop = FALSE])
  })

  # sorted columns are colored now because CSS are attached to them
  output$mytable2 <- DT::renderDataTable({
    DT::datatable(mtcars, options = list(orderClasses = TRUE))
  })

  # customize the length drop-down menu; display 5 rows per page by default
  output$mytable3 <- DT::renderDataTable({
    DT::datatable(iris, options = list(lengthMenu = c(5, 30, 50), pageLength = 5))
  })
}

ui.R
library(shiny)
library(ggplot2)  # for the diamonds dataset
print('hi')
ui <- fluidPage(
  title = "Examples of DataTables",
  sidebarLayout(
    sidebarPanel(
      conditionalPanel(
        'input.dataset === "diamonds"',
        checkboxGroupInput("show_vars", "Columns in diamonds to show:",
                           names(diamonds), selected = names(diamonds))
      ),
      conditionalPanel(
        'input.dataset === "mtcars"',
        helpText("Click the column header to sort a column.")
      ),
      conditionalPanel(
        'input.dataset === "iris"',
        helpText("Display 5 records by default.")
      )
    ),
    mainPanel(
      tabsetPanel(
        id = 'dataset',
        tabPanel("diamonds", DT::dataTableOutput("mytable1")),
        tabPanel("mtcars", DT::dataTableOutput("mytable2")),
        tabPanel("iris", DT::dataTableOutput("mytable3"))
      )
    )
  )
)

and ideally it would be simple to source both into a qmd like this
---
title: modular shiny
---

# App
```{shinylive-r}
source("server.R")
source("ui.R")

shinyApp(ui,server)

I've tried other methods, like using the `#| file: server.R` param in a code chunk, wrapping the code in special knitr comments `## ---- link` and calling them into the code chunk, and using the include shortcode, but no luck. Looks like for now we just need to put the entire app into one code chunk?

I've tried other methods, like using the #| file: server.R param in a code chunk,

Hi @DOH-FAA3303, I think I see your issue: the special comment should be ## file: server.R (note the initial ## and not #|). That likely explains all of the problems you listed, but if not can you please open an issue over in https://github.com/quarto-ext/shinylive (the repo for the Quarto shinylive extension). Thanks!

I've tried other methods, like using the #| file: server.R param in a code chunk,

Hi @DOH-FAA3303, I think I see your issue: the special comment should be ## file: server.R (note the initial ## and not #|). That likely explains all of the problems you listed, but if not can you please open an issue over in https://github.com/quarto-ext/shinylive (the repo for the Quarto shinylive extension). Thanks!

I'm trying to modularize my shiny app and keep the app from being one giant file. the ##: file function doesn't appear to call in a specific file in my directory.
So i mean i have a tree like this
├── index.qmd
├── src
│   ├── server.R
│   └── ui.R

where i want to call in different files from the src folder into the index.qmd and not have my entire app within the shinylive-r code chunk

nvm I see that you answered this here and you made a shortcode for it! https://youtu.be/8SHTj5QYQuc?si=3lxrejhsyHde9_uI&t=816

@DOH-FAA3303 glad you found that comment! We need to update the shinylive extension docs. In your example, the include shortcode should be enough:
# index.qmd

```{shinylive-r}
#| standalone: true
#| viewerHeight: 600

## file: ui.R
{{< include src/ui.R >}}

## file: server.R
{{< include src/server.R >}}
```


Is there any AutoBackup Roles?

Is your feature request related to a problem? Please describe.
Everything actually runs very well over a year for me, is there a solution for auto-backup for the Synology?
Describe the solution you'd like
Is there a Role to auto backup to ZFS? Just for the storage share
Describe alternatives you've considered
I did manual copy , end up with lot of permission error here and there
Additional context
I scew up my NAS with ran out the space error

I'm not clear what you're looking to back up to where - feel free to reopen with more info.


config.sh: cannot import name 'ComposeResult' from 'textual.app'

Debian's python3-textual packages version 0.1.13
ComposeResult was introduced in https://github.com/Textualize/textual/commit/4090d351684342b8e28ef9d5451c7c821e18d1ae (later released in v0.2.0 (The CSS Release) in October 2022).
The latest Textual release is v0.76.0. Debian Trixie/Sid package v0.52.1

ACK, agreed this is a problem.
We explored the possibility of using a later Debian-packaged Textual - but that is a no-go for example-derived code.
As such, I think we need to explore putting this stuff into a venv, and that might mean we need to drop nfpm in place of debuild to gain access to dh_virtualenv.


No raw data for Unimarkt

For other other shops we return (and keep) all the raw data in fetchData and extract relevant information in getCanonical.
The Unimarkt code already extracts in fetchData (thus, the raw data is thrown away).
I guess this is not on purpose?

Definitely not on purpose. Will fix tonight.

I forgot that the Unimarkt data is an HTML scrape, not an API call. As such, there's no real raw data. We'll have to live with the raw data being the canonical data.


[5.1] Replace stdlib and test/stdlib 9999 availability. (#26108)

Replace stdlib and test/stdlib 9999 availability.

macOS 9999 -> macOS 10.15
iOS 9999 -> iOS 13
tvOS 9999 -> tvOS 13
watchOS 9999 -> watchOS 6

Restore the pre-10.15 version of public init?(_: NSRange, in: __shared String)

We need this to allow master to work on 10.14 systems (in particular, to allow PR testing to work correctly without disabling back-deployment tests).
Cherry-pick of https://github.com/apple/swift/pull/26108

@swift-ci test

Build failed
Swift Test OS X Platform
Git Sha - b4f38b82d4e6d22b5d0cdc0e3546d2cb40e75801

Failure is in LLDB/ExclusivityREPL.test, which should not have anything to do with these changes, but also did not effect https://github.com/apple/swift/pull/26145. Investigating.

@dcci and I are unclear on why this failed. Going to re-run test and see what happens.

@swift-ci please test

@swift-ci nominate

@swift-ci nominate


Udated Readme.md

The documentation was not covering an important  use-case regarding uploading artifacts to already existing releases.
I believe that would help a log new users of the action

This is a very useful chunk of documentation and it's a shame I had to browse open PRs to find it.


51 need end of session signal

related to https://github.com/pollen-robotics/gRPC-WebRTC-pybridge/pull/43

This shouldn't impact the monitoring AFAIK so it's fine.


Count globals rebased

@Wout4 Review finished, let me know what you think!

I added a change to the bounds_calucation pr to calculate a.ub appropriatly for gcc
Also fixed a bug equivalent to the one in table decomposition


One specific virtual device not reachable

I agree to the following

[X] I have read the documentation right now
[X] I have read the FAQs right now
[X] I am aware of the latest release notes
[X] The Backend (CCU/Homegear/...) is working as expected and connected devices are controllable by its own UI.
[X] I am running the latest version of the custom_component (and Home Assistant)

The problem
A virtual device (INT0000005) is not available. All other virtual devices are available.
The blueprint reactivate_single_device does not work.
What version of HomematicIP (local) has the issue?
1.52.0
What was the last working version of HomematicIP (local)?
No response
What type of installation are you running?
Home Assistant Container
What type of installation are you running for your homematic backend?
CCU2
Which version of your homematic backend are you running?
2.61.7
What hardware are you running for your system?
Pi4
Which config details do you use

[X] Username and password
[ ] TLS
[ ] callback data (see)

Which interfaces do you use?

[X] Homematic IP
[X] Homematic (Bidcos-RF)
[X] Groups (Heating-Group)
[ ] BidCos-Wired (HM-Wired)

Diagnostics information
home-assistant_homematicip_local_2024-01-07T15-08-11.824Z.log
Anything in the logs that might be useful for us? The log (full log) is the best source to support trouble shooting!
No response
Additional information
No response

Don't switch to DEBUG level if not requested.

Manchmal Frage ich mich, warum ich das ins Issue Template überhaupt reingeschrieben habe.
Bitte ein Log ohne DEBUG Level anhängen.

Nur zum Verständnis:
Fehler und Warnungen  sind immer im Log. Erst wenn man mit diesen Informationen nicht weiter kommt, dann kann man selektiv für einige Module Debug aktivieren.


Is there a wrap around terminal feature?

Problem:
Since my markdown file has LONG lines, a word wrap would enable me see lines. I see lines going out of screen now.
Solution : Possible Word wrap around 80 characters or width of terminal
May be this feature is already present and I could not figure it out, if so, Let me know! :)

Hey! We're possibly going to add word wrapping here: https://github.com/maaslalani/slides/pull/84

Nice!

@maaslalani  sorry for the question (cause I didn't check myself), but does slides have support for centered and block text, in addition to the left aligned text? Any plans? Even possible?

Yea I want to add center and right aligning in the future.
https://github.com/maaslalani/slides/issues/56#issuecomment-869192906

@TwistingTwists Word wrapping should be enabled now 😄

awesome! Will check.
Thanks!


https://github.com/ephread/Instructions/issues/90 fix

The pull request is to address an issue mentioned in https://github.com/ephread/Instructions/issues/90
When user quickly open and close ViewController, the reference of CoachMarksViewController could be lost.
In order to rectify this, add a weak self reference and perform a check before executing the detachFromWindow() method.

Codecov Report

Merging #161 into master will increase coverage by 1.31%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #161      +/-   ##
==========================================
+ Coverage   71.94%   73.25%   +1.31%
==========================================
  Files          32       32
  Lines        1757     1458     -299
  Branches      116        0     -116
==========================================
- Hits         1264     1068     -196
+ Misses        488      390      -98
+ Partials        5        0       -5

Flag
Coverage Δ

#ios
73.25% <100%> (+1.31%)
:arrow_up:

Impacted Files
Coverage Δ

...ces/Instructions/Managers/Public/FlowManager.swift
69.84% <100%> (+1.68%)
:arrow_up:

...ns/Helpers/Internal/Extensions/UIView+Layout.swift
76% <0%> (-5.25%)
:arrow_down:

...urces/Instructions/Views/OverlaySnapshotView.swift
66.66% <0%> (-3.93%)
:arrow_down:

...ons/Managers/Internal/SkipViewDisplayManager.swift
87.67% <0%> (-1.99%)
:arrow_down:

...ns/Managers/Internal/CoachMarkDisplayManager.swift
80.37% <0%> (-0.3%)
:arrow_down:

...rlayStyleManager/BlurringOverlayStyleManager.swift
48.78% <0%> (-0.16%)
:arrow_down:

...Public/Extensions/CoachMarksController+Proxy.swift
72.41% <0%> (-0.09%)
:arrow_down:

...re/Internal/CoachMarksViewController+Regular.swift
100% <0%> (ø)
:arrow_up:

...gers/Internal/SkipViewDisplayManager+Regular.swift
0% <0%> (ø)
:arrow_up:

Sources/Instructions/Views/DummyView.swift
0% <0%> (ø)
:arrow_up:

... and 23 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 13b278d...a35bfb3. Read the comment docs.

Looks good, thanks for the work you've put in!


Better error message when entering wrong mnemonic

The error is key not supported, suggest Invalid mnemonic, Please make sure you entered a valid mnemonic

https://github.com/threefoldtech/tfchain_client_js/commit/ecc132a9c8ad38b0486b43908c1a51364047fa9a


WiP: update our singularity docker shim to fixup tests etc

Unfortunately we do not have (yet) cron job running tests, so at some point things went awry and I am not sure when.  This PR is WiP toward resolving them.
Part of the problem precluding update of our shim is also that singularity docker images are flawed - have no suid set, reported to @vsoch .
When done, it should close #40

@yarikoptic let me know how I can help. I can manually push the base with setuid for that particular slim tag, but it still remains a mystery why it wasn't there via the automated build.

Thank you @vsoch!  I think with ad-hoc chmod as in the PR that issue is a non-issue for us ;)  yet to figure out what other things have changed


HasMagicField not working with forms

I am fresher in rails , I am not confirmed whether the issue is with has magic field gem or with my understanding. I would like to get help on following issues
I am using cocoon gem for nested form
I have a model abc
class Abc < ActiveRecord::Base
  HasMagicFields::Extend
   has_magic_field
end

_form.html.slim -- to create an object
= simple_form_for(@abc, wrapper: :horizontal_form, validate: true) do |f|
=  f.simple_fields_for :magic_fields do |form|
        = render 'magic_fields', f: form
      .links
        = link_to_add_association '', f, :magic_fields, class: 'fa fa-plus-square'

inside partial _magic_field_fields
= f.input :name
= f.simple_fields_for :magic_attributes do |form|
  = render 'magic_attribute_fields', f: form

inside partial _magic_attribute_fields
= f.nput :value

inside controller permitted all paramters
I am getting an error of
ActiveRecord::AssociationTypeMismatch at /controllername

MagicAttribute(#95887670) expected, got Array(#72503940)

I found out other many easy techniques for nested association, and since no one was available to solve my doubt , i am closing this issue


Move tests into an "ops" subdir

Purpose

Moves tests into a subdir so that merging repos is easier

Changes

[x] move all of the tests into an ops subdir
[x] update any hardcoded references that get broken by the change

Checklist

[x] Update CHANGELOG.md file

[x] Merge in main so the most recent CHANGELOG.md file is being appended to
[x] Add description within the Unreleased section in an appropriate category. Add a new category from the list at the top of the file if the needed one isn't already there.
[x] Add a link to this PR at the end of the description with the PR number as the text. example: #1

[ ] Applicable documentation updated (guides, quickstart, postman collections, tutorial, fidesdemo, database diagram.
If docs updated (select one):

[ ] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
[ ] documentation issue created (tag docs-team to complete issue separately)

[ ] Good unit test/integration test coverage
[ ] This PR contains a DB migration. If checked, the reviewer should confirm with the author that the down_revision correctly references the previous migration before merging
[ ] The Run Unsafe PR Checks label has been applied, and checks have passed, if this PR touches any external services

Ticket
Fixes #934

reordering the datastore list works 100% of the time 70% of the time!

@ThomasLaPiana looks good, I'll merge once the conflicts are resolved

@eastandwestwind appreciated!


add annotations for setting cpu/memory limits on sidecar

When a limitrange is active, not setting the limits will result
in an error. This patch will allow setting limits for the sidecar
and when not set, use the requests value.

@rolandkool: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Hi @rolandkool. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Fix bare images

Change bare images (e.g. image: busybox) to consume from quay.io, to avoid docker.io rate limits and have more predictable setup.

@nirs requires a rebase before merge...

@nirs requires a rebase before merge...

Fixing...

@ShyamsundarR fixed


Limit is not converted to integer

I have this function:
exports.lastCovers = function(page, pageSize) {
  console.log(typeof pageSize)
  console.log(typeof page)
  return Cover.collection().query(function(qb) {
    qb.orderBy('registration_date', 'desc').limit(pageSize).offset((page - 1) * pageSize);
  }).fetch();
}

It prints:
string
string

And happens this error:
{ __cid: '__cid8',
  method: 'select',
  options: undefined,
  bindings: [ '20', 0 ],
  sql: 'select `cover`.* from `cover` order by `registration_date` desc limit ? offset ?' }
{ [Error: ER_PARSE_ERROR: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ''20' offset 0' at line 1]
  code: 'ER_PARSE_ERROR',
  errno: 1064,
  sqlState: '42000',
  index: 0 }

The offset is converted to integer but limit is not.

Hi, @bendrucker ,
why value for limit function parse String to Int, and offset function not?
	  // Only allow a single "offset" to be set for the current query.
	  offset: function offset(value) {
	    this._single.offset = value;
	    return this;
	  },

	  // Only allow a single "limit" to be set for the current query.
	  limit: function limit(value) {
	    var val = parseInt(value, 10);
	    if (isNaN(val)) {
	      helpers.warn('A valid integer must be provided to limit');
	    } else {
	      this._single.limit = val;
	    }
	    return this;
	  },

In Postgres, at least, one can do "LIMIT ALL" in a query.  It would be nice to be able to send in "ALL" (or an equivalent) into limit() and have it not emote a warning everytime.

@VincentVazzo Please open a new issue with this feature request. This issue is closed.


Asterisk 12 support

https://issues.asterisk.org/jira/browse/ASTERISK-21470
I would like to add support for asterisk 12 but would like some direction.  Creating the new event classes would possibly be a breaking change for backward compatibility, catching the events and populating the sub-events seems a little hacky.  Any suggestions?

@wormling What's the status on your changes? I need to work with Asterisk 12+, and I wonder if I can use your changes?

It didn't fit well with ARI, your code should still work, but if you need ARI you can use the wormling/phparia project.  It does not include menu logic, but it's very simple.

Ahh... you'll still need to switch to 'AsyncAGIStart', 'AsyncAGIEnd', 'AsyncAGIExec', 'AGIExecStart', 'AGIExecEnd', there might be something it that clone of PAGI on my github

Something else new in Asterisk 12 is ARI. This nicely separates the command/response traffic from the event listening, with the latter being done via WebSockets.


Add Exclude Annotation to Realtime Databases for Android & iOS

As it exists in Realtime Databases and FireStore in the original SDKs, we need a way to exclude fields from serialization in KMM.
Ref: https://firebase.google.com/docs/reference/android/com/google/firebase/database/Exclude

Can you use @Transient here? https://github.com/Kotlin/kotlinx.serialization/blob/master/docs/basic-serialization.md#transient-properties

@nbransby the problem is that @transient is also for preventing the java serialization, and I need to send some of those fields through intents inside a bundle, I just need to exclude the serialization for Firebase, I think that's is why the @exclude annotation exists.

I dont believe it is, there are two transient annotations, maybe you are confusing it with kotlin.jvm.Transient?
A property can be excluded from serialization by marking it with the @Transient annotation (don't confuse it with kotlin.jvm.Transient). Transient properties must have a default value

Yes, you are right, sorry for the confusion, I will close the issue, thanks for the help!


Expanding official bus endpoint

We plan to have a meeting with a representative from Strætó - will post further news here

@baldurthoremilsson Still interested in laying the groundwork for this?


ENH: Add is_ci field to add_project resolver

To monitor whether usage is "real"

pushing this to staging server


When the mouse constantly slides on PieChat, the section goes off.

When the mouse constantly slides on PieChat, the section you are sliding fly off.
LiveCharts.WPF 0.7.11

PS: there is any suite to record screen and save as gif, I saw it a lot of times here (gif creation)

does it comes back to its original position?

Nope, , it tries to get back to original position but stop on the mid of path, so the next time it will start from there and keeps going....

I will take a look.
About the recorder I use LICEcap

Hello, are you using 0.7.11?, I can not reproduce this error, I similar issue was fixed with #272

Yes, I´m using version v0.7.11

Hello, I´m using now version 0.7.13 the problem persists.
Here is how I´m doing it.
private` PieSeries freeMemory = new PieSeries();
private PieSeries usedMemory = new PieSeries();
public SeriesCollection DonutCollection { get; set; }

DonutCollection = new SeriesCollection
            {
               freeMemory,
               usedMemory
            };

Then I have a method to update the values from another class that have an Instance of my class.
public void ReceiveData(string message)
        {
            MemoryModel model = JsonConvert.DeserializeObject<MemoryModel>(message);

            double free = double.Parse(Regex.Match(model.MemFree, @"\d+").Value);
            double used = double.Parse(Regex.Match(model.Active, @"\d+").Value);

            Dispatcher.Invoke(() =>
            {
                freeMemory.Values.Cast<ObservableValue>().First().Value = free;
                usedMemory.Values.Cast<ObservableValue>().First().Value = used;
            });

        }

No, that I update the values each 300ms interval, sometimes faster.

I see the error, an alternative for now is disable the hovering in your chart
 <lvc:PieChart Grid.Row="1" Hoverable="False">
    </lvc:PieChart>

This is already fixed and will be included since 0.7.14, thanks for the report


Check and Disable  DOTNET_CLI_TELEMETRY_OPTOUT if really enabled

Is your feature request related to a problem? Please describe.
It seems that Microsoft enables telemetry by default on the installation of the SDK via the installer script.
Describe the solution you'd like
Telemetry should be disabled by default
Additional context
How to disable telemetry

Correct me if I'm wrong, but as far as I know, different shells have different ways of setting global variables, but the vast majority of them support adding export NAME=VALUE to the configuration file. However, it's not so simple, because the configuration file is also different for different shells and can be located in different places. I doubt that disabling telemetry can be automated with a universal shell script. I would just add a warning to README.md. This seems to me to be the simplest and most straightforward solution.

Thanks for you contribution @oleksbabieiev , I'll accept your PR as for now, it is the simplest solution to this.
However, I'm failling to imagine a scenario where adding the variable to the environment would not be enough. I mean, we already do that for the DOTNET_* variables on the set-dotnet-env.{fish,zsh,bash} files, wouldn't that be the same case here?

Okay, my "interesting" idea didn't live up to expectations 😂. Perhaps it would have been better to do what you said, just making small adjustments to the existing scripts set-dotnet-env.*, and not reinvent the wheel.


Documentation search includes results from zwavejs2mqtt docs

Checklist:

[X] My problem is not within Home Assistant or the ZWave JS integration. Or: a Home Assistant developer has told me to come here.
[X] My problem is not within ZWaveJS2MQTT. Or: a ZWaveJS2MQTT developer has told me to come here.
[X] I have checked the troubleshooting section and my problem is not described there.
[X] I have read the changelog and my problem was not mentioned there.

Describe the bug
When searching the docs website, the results include docs from the zwavejs2mqtt project. Clicking on a result produces a 404 page. I would expect only node-zwave-js docs.
To Reproduce
Steps to reproduce the behavior:

Open a new private/incognito window so the browser cache is empty
Go to the zwavejs2mqtt docs
Search for hass in zwavejs2mqtt docs
Go to the node-zwave-js docs
Search for hass in the node-zwave-js docs
See results that are from the zwavejs2mqtt docs
Click on a result
The result is a 404 page

Additional context
Screenshot of mixed results from above steps:

The problem only occurs after you've visited the zwavejs2mqtt docs. If you go to node-zwave-js docs first and search, the results are fine.
Result with fresh cache, no zwavejs2mqtt:

Grml. Looks like the docsify search plugin does not respect the namespace.

https://github.com/docsifyjs/docsify/issues/1591


Missing Texture causes thread 'main' panic

While running the example I got the message:
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Internal("user texture 0 not found")', src\main.rs:172:22
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

[process exited with code 101 (0x00000065)]

The problem happened after clicking on the buttton "Color test".
I'm guessing there are some images from the official demo witch are missing.
OS:

Windows 11

Rust
rustup show
rustup 1.24.3 (ce5817a94 2021-05-31)
stable-x86_64-pc-windows-msvc (default)
rustc 1.60.0 (7737e0b5c 2022-04-04)

I'm getting same issue, however on startup.
in my case it's me trying to copy it to a custom engine, did everything line by line and still got that


Update docs links for new URL structure

Part of https://github.com/Qiskit/documentation/issues/1461. Even though these pages have redirects, it's better to go straight to the page we want.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

0 of 0   changed or added relevant lines in 0 files are covered.
18 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 89.749%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
6
92.48%

crates/qasm2/src/parse.rs
12
97.15%

Totals

Change from base Build<PHONE_NUMBER>8:
-0.02%

Covered Lines:
67343

Relevant Lines:
75035

💛  - Coveralls


Could you please make this for 1.18.1 and 1.18.2 ?

Thanks!!!

Version 3.0.0 is now on CurseForge with support for 1.18.1 and 1.18.2.


Support arbitrary PBS endpoints

Currently load-cookie.html only supports 'appnexus' and 'rubicon' endpoints. It's unfriendly for other PBS hosts to have to modify the cookieSync.js script.
Here's what the load-cookie invocation looks like:
https://PROVIDED_BY_HOSTCOMPANY/load-cookie.html?endpoint=HOSTCOMPANY

The proposal here is that we expand the values that the 'endpoint' parameter can accept:

appnexus
rubicon
a url-encoded PBS URL. e.g. https%3A%2F%2Fprebid-server.example.com%2Fcookie_sync

validations:

if the endpoint is not on the VALID_ENDPOINTS object, it must start with https

perhaps its better to have the "endpoint"  value be custom and then support an additional optional param to pass the end point url. Further not that it matters but should we care if the url doesnt have cookie_sync in it ? if it loads the sync correctly it should be fine either way ?

Dropped the cookie_sync validation. I don't feel strongly about this interface. If you write the code, it can be whatever way you prefer. :-)

This was fixed in the PUC load-cookie.html a while back. Just documented the usage on https://docs.prebid.org/dev-docs/show-prebid-ads-on-amp-pages.html#user-sync


Prev and Next arrows appear on top and bottom respectively

Why the arrows prev and next appears on the top and the bottom of the carousel?
I need that they appear on the sides but i haven't any luck with that.
I'm using the JS code like this:
$('#slick-carousel').slick({
        infinite: true,
        autoplay: true,
        arrows: true,
        centerMode: false,
        mobileFirst: false,
        variableWidth: false,
        prevArrow: '
<button type="button" class="tc-twitter-arrows tc-arrow-prev"></button>
',
        nextArrow: '
<button type="button" class="tc-twitter-arrows tc-arrow-next"></button>
',
        dots: false,
        responsive: [
        {
          breakpoint: 768,
          settings: {
            arrows: false,
            centerMode: true,
            centerPadding: '40px',
            slidesToShow: 2
          }
        },
        {
          breakpoint: 480,
          settings: {
            arrows: false,
            centerMode: true,
            centerPadding: '40px',
            slidesToShow: 1
          }
        }
      ]
    });

This is the CSS for the buttons:
.tc-twitter-arrows
{
    background-image: url('http://welovestyles.com/wp-content/uploads/loook/carousel-arrows-333333-50.png');
    background-repeat: no-repeat;
    background-color: transparent;
    border: none;
    margin: 0;
}

.tc-twitter-arrows.tc-arrow-prev{
    width: 40px;
    height: 40px;
    background-position: 0 0;
}
.tc-twitter-arrows.tc-arrow-next{
    width: 40px;
    height: 40px;
    background-position: -40 0;
}

Also, i have a question about the basic "must" css styles that need to have the slides and the container (i'm having issues with the widths and heights).
Kind regards,
Gonzalo.

I have this same problem.

I have the same problem

Same here!

so what's the solution ?

This tends to happen when I'm trying to style .slick-prev and .slick-next  Actually slick-next tends to stay where it should but prev keeps moving to seemingly to the top of the containing element.

use settings below :
arrows-container : '.any-selector'
Then add the selector element on any position of your code


Add a wrapper for cc_library: tf_cc_library

I'll replace every cc_library under //tensorflow with tf_cc_library

Can one of the admins verify this patch?

@meteorcloudy any thoughts here? I'd really like to avoid having is_external around every contrib cc_library() rule. In theory, couldn't we just default is_external to True and then add it to the _pywrap_tensorflow_internal rule only?

Not every contrib cc_library() rule. Some modules under tensorflow/contrib is part of _pywrap_tensorflow_internal.so, and we need to set is_external=False in these dirs.
Another thing: we may disable --config=monolithic on windows. So every kernel is external to the framework shared library. It would be eaiser for us to set CFLAGS correctly.

I have a suggestion:

remove the is_external arugment
manually add "-DTF_COMPILE_LIBRARY" to every cc_libraries who is part of the _pywrap_tensorflow_internal.so
That's sounds more straightforward, because we may have multiple so libraries in the future. We'll add -DTF_COMPILE_AAA to the libraries of libaaa.so , -DTF_COMPILE_BBB to the libraries of libbbb.so, etc.

I am really not fond of either this solution, or the previous one where we added tf_copts to all the cc_library rules.
I have two alternate suggestions:

Setting the flag we need to pass in windows build through the windows Crosstool.
Writing the flag we need to pass in windows build in our bazelrc file during configure. We already write windows-specific things into our bazelrc anyway.

@meteorcloudy WDYT?

Close it as I don't have a good solution right now.


Roll Flutter Engine from 1cdbebee1901 to 45cf05f7a580 (10 revisions)

https://github.com/flutter/engine/compare/1cdbebee1901...45cf05f7a580
2024-06-07<EMAIL_ADDRESS>[web] add test for inefficient overlay allocation (flutter/engine#53284)
2024-06-07<EMAIL_ADDRESS>Update Chrome to 125. (flutter/engine#53282)
2024-06-07<EMAIL_ADDRESS>Roll Dart SDK from 4b693b16eec1 to d4f17e0bf28b (1 revision) (flutter/engine#53283)
2024-06-07<EMAIL_ADDRESS>Fixes a few issues in flutter_js (flutter/engine#53231)
2024-06-07<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from aVohW_hnfDaE0smBX... to zpBZmUB_JC5AjG-f4... (flutter/engine#53279)
2024-06-07<EMAIL_ADDRESS>Roll Dart SDK from 7b239d7f4578 to 4b693b16eec1 (1 revision) (flutter/engine#53277)
2024-06-07<EMAIL_ADDRESS>Roll Skia from b7f51dfcc342 to ad3c9f203f4e (1 revision) (flutter/engine#53276)
2024-06-07<EMAIL_ADDRESS>Roll Skia from 5fc83884a619 to b7f51dfcc342 (4 revisions) (flutter/engine#53275)
2024-06-07<EMAIL_ADDRESS>Revert "Widen CPU affinity set." (flutter/engine#53274)
2024-06-07<EMAIL_ADDRESS>Roll Skia from e36110f8b451 to 5fc83884a619 (1 revision) (flutter/engine#53273)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from aVohW_hnfDaE to zpBZmUB_JC5A
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC matanl@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Reason for revert: Regression affecting integration_test/test/web_extension_test.dart
  PathNotFoundException: Cannot open file, path = '/chromium/canvaskit.wasm' (OS Error: No such file or directory, errno = 2)

(see https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8745735238400976001/+/u/run_test.dart_for_web_long_running_tests_shard_and_subshard_1_5/stdout)
It looks like the presubmit run of web_long_running_tests may have been using the wrong engine commit hash.
The hash in the flutter config --clear-features step in https://ci.chromium.org/ui/p/flutter/builders/try/Linux web_long_running_tests_1_5/63894/overview does not match the hash rolled in this PR.


In the case of parallel execution, one succeeds, but all others fail.

While provisioning a container app is in progress, other provision reques will be fail.
We need guidance or solution to this issue.

Cloud you please comment on how to do it, as well as any logs or screen shots?

If the Container App is in the process of provisioning, the following error will occur when the GitHub action is executed.

Looks like retry logic is needed, further investigation is needed to decide which layer the logic should be implemeted in.

I think the workaround would be to implement retry. In addition, I think that it is necessary to decide what it should be by discussing it separately.

@azure/core-rest-pipeline already has retry logic as described at the following documentation.
https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/core/core-rest-pipeline/documentation/pipeline.md
But the status code 409 in this case is not included in retry-target-response-codes as far as reading the following code.
https://github.com/Azure/azure-sdk-for-js/blob/82996230773ab8295f06b17a8b6f449f9d9f2a8c/sdk/core/core-rest-pipeline/src/retryStrategies/exponentialRetryStrategy.ts#L81-L94

Ref: https://github.com/Azure/azure-sdk-for-js/issues/23298

As guidance, at a minimum, the following guidance should be guided.

Run only one workflow at a time
Ref: Workflow syntax for GitHub Actions

The guidance is about GitHub Actions workflows' concurrency.
On the other hands, this issue can be happened when the action runs during any updates from Azure portal, CLI, PowerShell or REST API.

@k-in
Could you please try an action<EMAIL_ADDRESS>that retries 10 times at maxiumum and let me know the result?

Released<EMAIL_ADDRESS>that exposes max retry count as an input max-retries.
https://github.com/horihiro/aca-preview/blob/5d75583459e1a53ed6ebf3f8d956e0144bb7eb68/src/main.ts#L127-L137

By design, currently,


Would there be interest in a Postman collection with the endpoints and tests?

I'm making a public Postman collection that hits the endpoints, authenticates, and then requests for data, in a practical way, akin to what your documentation library refers to.

Would you be interested in having this added to your documentation, so your users can easily test the endpoints you describe? The greatest advantages of integrating the documentation with Postman are:

Very easy testing of endpoints;
Straightforward testing samples of expected outputs for each endpoint;
The ability to easily convert requests to code in any popular language;

PS: Thanks for the library, I'm doing my best to not fanboy too much. 😅

Hi @TheYuriG
I'm glad you've found this useful, and thanks for your suggestion.
I certainly like the idea of it. The only reservation I have around linking to it would be a concern around the authentication side of it and wanting to make sure that a users SSO token couldn't potentially be stolen by - for example - the Postman collection being altered at a later time to add in an extra step/endpoint which sends that information off to a malicious web server.
Do you know if is it possible to link to a specific version of the collection, or will it always link to the latest version? If it were possible to link to a specific version that could potentially mitigate this concern.
I don't know if my concern is even possible (and I'm also not wanting to suggest you would do such a thing if it is!), but for as convenient as this could be for allowing someone to access the API when handling/automating the authentication for the user I'd want to make sure there's no risk of someone compromising the security of their PSN account by using it.

Do you know if is it possible to link to a specific version of the collection, or will it always link to the latest version?

I'll look into it and let you know!

I don't know if my concern is even possible (and I'm also not wanting to suggest you would do such a thing if it is!), but for as convenient as this could be for allowing someone to access the API when handling/automating the authentication for the user I'd want to make sure there's no risk of someone compromising the security of their PSN account by using it.

Yeah, I understand the concern, there are tons of ways it could go bad like my account getting compromised and the Collection stolen, etc. I'll keep building for myself for now (since I actually really need this to test the endpoints anyway) and I'll get back to you in a few days about how we can safely share this.

Thanks, I look forward to seeing what you come up with.

All the current endpoints are there, but I'm unable to test any of the old endpoints (the V1 ones with .net).
Are you still able to connect to those just fine?
Dunno if I'm doing something wrong or all these legacy V1 endpoints died when this happened because all I get are 403 FORBIDDENs.


Add the check for -mfpu=neon

For 64bit armv8-a there's no need to use "-mfpu=neon" to enable NEON.
But for 32bit system "-mfpu=neon" is required.
This patch adds the detection for -mfpu=neon flag.
Change-Id: Iec4f9e29f78806ae0e7021187038bb0b99eec30b
Signed-off-by: Richael Zhuang<EMAIL_ADDRESS>

@mtl1979 Any comments?

@Dead2 Obviously we should always test if some flag is actually supported on the current toolchain. Assuming is real bad habit.

I said that for 64bit armv8-a there's no need to use "-mfpu=neon" to enable NEON.
But for 32bit armv8-a system and  armv7 system, "-mfpu=neon" is required. I both verified on our 64bit armv8-a  server and the 32bit  environment.  I add the check because I think it's a simple way to distinguish whether it's a 64bit armv8-a or a 32bit one. Of course I have thought that there's other way to distinguish them. This's just a choice.

This needs to be rebased due to a conflict in configure.

@Dead2 With most cmake variables, you need to explicitly append or prepend to them if you want to preserve the old contents.

@Dead2  Since we use “ set (CMAKE_REQUIRED_FLAGS  ”new-flag”) “ here， the  ”new-flag“ will replace the old  CMAKE_REQUIRED_FLAGS. It's different from this way: " set (CMAKE_REQUIRED_FLAGS  " ${CMAKE_REQUIRED_FLAGS}  new-flag") "   . And if  we want to clear the last CMAKE_REQUIRED_FLAGS，”set (CNMAKE_REQUIRED_FLAGS) " can do it.

@richael02 That's what I expected then. So if there are two compile-checks after each other, and the first one sets a flag, then the next one also gets that flag, changing the way that test works. So I think we'd better start resetting these flags after each test, before we get bitten by a bug.

@richael02 Yes, but what if the second test does not set the flag at all, and expects it to be empty?
For example first there is a test checking for compiler flag, and the second test is to check for a supported intrinsic. If the compiler flag does not get reset after the first test, the second test will run with both the compiler flag and the intrinsic. So if the compiler flag fails, the intrinsic also will fail, even though it should have passed.

@Dead2 Should I open another pull request to do this, for it's not related to "neon" issue?

You could add a reset after the check in this PR, but resets for the other checks should be in a separate PR, yes please 👍

Ok, no problem.

Thanks 👍


Admin UI authentication throws an error and prevents other authentication methods from being used

Describe the bug
When I run amplify init, select an existing environment  and choose Amplify Admin UI as my authentication method it will throw the following errors (Failed to refresh tokens: Missing required parameter auth parameters. Failed to get credentials: Missing required parameter auth parameters)
After encountering this error ampliy init will no longer allow me to select a different authentication method, meaning i'm stuck in a broken state and can't attempt to authenticate my environment another way (Access key works fine for me but now I can't switch to use that).
I'm not sure what's causing this, i'm sure i'm missing a setting somewhere but have had no luck getting to the bottom of this yet, I have had Admin UI authentication work for me in the past.
I've also tried re-running amplify configure and going through that process again but ampliy init still gives the same error
This also happens with amplify env checkout live
Amplify CLI Version
4.42.0
To Reproduce
Run amplify init
Select an existing environment
Select Amplify Admin UI as the authentication method
If you get the same error you should then be unable to change authentication methods for your environment
Expected behavior
I'd expect Admin UI to work as an authentication method, not sure what i'm doing wrong to get it into this state
Also amplify init should allow me to select a a different authentication method if one has failed, instead of locking me into my first selection
Screenshots

Desktop (please complete the following information):

OS: Windows
Node Version: v12.18.4

I've managed to sort of work around this by setting up a fresh clone of my repo and then re-running amplify configure and then amplify init where i could authenticate by access key.  Not an ideal fix but it gets things unblocked for now

@JackPenney if you run into this again, you can run amplify logout --appId <yourAppId> and then you'll be able to choose a different authentication method as expected when you run commands.

I just ran into a similar issue and amplify logout --appId <yourAppId> worked for me. Is there any detailed document for these commands and when to use them?

I just ran into a similar issue and amplify logout --appId <yourAppId> worked for me. Is there any detailed document for these commands and when to use them?

amplify help is an easy way to get a list of commands and some context.
However, this is a workaround so you won't find documentation for this scenario currently.

I had the same issue after upgrading to amplify-cli 4.42.0 and then again to 4.43.0, although I do not recall ever selecting the admin ui as my default authentication method.

I had the same issue after upgrading to amplify-cli 4.42.0 and then again to 4.43.0, although I do not recall ever selecting the admin ui as my default authentication method.

Did running amplify logout --appId <yourAppId> fix the issue for you?

The fix for this has been released in version 4.44.1 of the CLI. Please upgrade to the latest version to try it out.

I deleted everything in my amplify folder (except what was not gitignored) and re-ran amplify init. Worked for me


CPR Port | unresolved external symbol  _Cnd_timedwait_for

Describe the bug
Whenever I install the cpr port I can import it but not use it.
See https://github.com/libcpr/cpr/issues/1080
Environment

OS: Windows 10
Compiler: Not sure I'm using Visual Studio Preview 2022

To Reproduce
Steps to reproduce the behavior:
vcpkg install cpr:x64-windows

Create a Console C++ project
Set it to either C++17 or C++20 (I use C++20)

To make it a very easy reproduce just use the code under, I use ixx modules from C++20 but the this issue happen even in the minimal example.
#include <cpr/cpr.h>
#include <iostream>

int main()
{
    std::cout << "Hello World!\n";
    cpr::Get(cpr::Url{ "https://api.github.com/repos/whoshuu/cpr/contributors" },
        cpr::Authentication{ "user", "pass", cpr::AuthMode::BASIC },
        cpr::Parameters{ {"anon", "true"}, {"key", "value"} });

    return 0;
}

Expected behavior
The program should compile.
Failure logs
Build started at 5:37 PM...
1>------ Build started: Project: CPRTest, Configuration: Release x64 ------
1>Scanning sources for module dependencies...
1>cpr.lib(threadpool.cpp.obj) : error LNK2001: unresolved external symbol _Cnd_timedwait_for
1>CPRTest.exe : fatal error LNK1120: 1 unresolved externals
1>Done building project "CPRTest.vcxproj" -- FAILED.
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========
========== Build completed at 5:37 PM and took 00.385 seconds ==========

Severity	Code	Description	Project	File	Line	Suppression State	Details
Error	LNK2001	unresolved external symbol _Cnd_timedwait_for	CPRTest	C:\Users\TheHellTower_THT\source\repos\CPRTest\CPRTest\cpr.lib(threadpool.cpp.obj)	1
Error	LNK1120	1 unresolved externals	CPRTest	C:\Users\TheHellTower_THT\source\repos\CPRTest\x64\Release\CPRTest.exe	1

Additional context
I opened an issue on the cpr repository here

Using the latest version of vcpkg for local installation does not reproduce the issue.
Could you please try these codes?
command that %your_vcpkg_dir% need modified
cmake -Bbuild_dir -DCMAKE_BUILD_TYPE=Debug -DVCPKG_TARGET_TRIPLET=x64-windows -DCMAKE_TOOLCHAIN_FILE=%your_vcpkg_dir%/scripts/buildsystems/vcpkg.cmake

cmake --build build_dir --config Debug

main.cpp
#include "cpr/cpr.h"

#include <iostream>

int main() {

    std::cout << "cpr" << std::endl;

    cpr::Get(
        cpr::Url{ "https://api.github.com/repos/whoshuu/cpr/contributors" },
        cpr::Authentication{ "user", "pass", cpr::AuthMode::BASIC },
        cpr::Parameters{ {"anon", "true"}, {"key", "value"} }
    );

    cpr::Response response = cpr::Get(cpr::Url{ "https://example.com" });

    std::cout << response.status_code << " " << response.reason << std::endl;

    return 0;
}

CMakeLists.txt
cmake_minimum_required(VERSION 3.25.1)
project(Proj VERSION <IP_ADDRESS> LANGUAGES CXX)

add_executable(main main.cpp)
target_compile_features(main PRIVATE cxx_std_23)

find_package(cpr CONFIG REQUIRED)
target_link_libraries(main PRIVATE cpr::cpr)

Using the latest version of vcpkg for local installation does not reproduce the issue.

I'm not using the CMake compiler myself but MSVC one and I never got this issue before it happened after I completelly renewed my vcpkg install.

_Cnd_timedwait_for comes from the C runtime libraries. It sounds like some parts were built with different versions of MSVC and the linker and standard libraries being used are not newer than the newest used in the link.

_Cnd_timedwait_for comes from the C runtime libraries. It sounds like some parts were built with different versions of MSVC and the linker and standard libraries being used are not newer than the newest used in the link.

Does this mean anyone installing the lib now would get this issue ? I'm confused because I think I only have the default version of MSVC..

Bump

Bump

So I seen on the readme that CPR got a NuGet package and:
NuGet Package: Compile
VCPKG Build: LNK2001 unresolved external symbol _Cnd_timedwait_for
So I think the it can be a VCPKG issue 🤔

I know nothing about cpr project. However, every time I build any project from source, I run all the unit tests.

Some screenshots (Windows 11 x64):

I know nothing about cpr project. However, every time I build any project from source, I run all the unit tests.  Some screenshots (Windows 11 x64):

You are using cmake.

This is happening for me as well, on a fresh install. Don't have a repro-case though

This is happening to me as well.

@bamo-volue @Teles1
Could you please try those code https://github.com/microsoft/vcpkg/issues/39829#issuecomment-2222220964 ?
If it fails, please append error logs.

@bamo-volue @Teles1
Could you please try those code #39829 (comment) ?
If it fails, please append error logs.

I'm not using cmake though. I wish :(

i had the same problem with cpr 1.11.0 get from vcpkg.

I discovered my problem and solved it. I need to install the appropriate Visual C++ Redistributable to run the code in other windows.

i had the same problem with cpr 1.11.0 get from vcpkg.

i had the same problem with cpr 1.11.0 get from vcpkg.

Could you please help to give some reproduce steps and demo?

@bamo-volue @Teles1
Could you please try those code #39829 (comment) ?
If it fails, please append error logs.

We changed VM after running into that issue, so it hasn't been an issue since after setting it up on a new system. Sorry for no repro

I'm also seeing this issue, "error LNK2019: unresolved external symbol _Cnd_timedwait_for", building with MSVC via CMake, using cpr 1.11.0.  I'll try to boil it down to a minimal repro.

It‘s solved by upgrade the visual studio to the latest version（17.12.2）Sent from my iPhoneOn Dec 3, 2024, at 20:34, Bård Sigurd Møller @.***> wrote:
@bamo-volue @Teles1
Could you please try those code #39829 (comment) ?
If it fails, please append error logs.
We changed VM after running into that issue, so it hasn't been an issue since after setting it up on a new system. Sorry for no repro
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

I discovered my problem and solved it. I need to install the appropriate Visual C++ Redistributable to run the code in other windows.

What redistributable exactly did you install to solve it?


Any plans to support native MacOS builds?

Right now the package (and therefore appoptics-apm-node) only support linux. This is mostly fine (our production env is all linux) but it would make testing easier if MacOS was supported. Are there any plans to support MacOS or is Linux a limitation of liboboe?

The pain is understood, but his is not going the happen. If anything, and this is also very unlikely, platform independency will be achieved by removing the dependency on the bindings altogether.
All development of both packages is currently done on Mac inside docker containers. You can find the configuration inside the dev directory of both repos. It might, maybe, help a little.
Sorry.


[stable/prometheus-operator] Upgrade kube-state-metrics to v1.6.0

kube-state-metrics v1.5.0 is installed by the current prometheus-operator chart. kube-state-metrics v1.6.0 was released 2019-05-06. The current chart version is 1.6.3. Please upgrade the promethus-operator chart to the 1.6 version.
Is your feature request related to a problem? Please describe.
I would like to take advantage of the new features and bugfixes in the new version.
Describe the solution you'd like
AFAIK, all that is needed is a one character change to requirements.yaml
-     version: 1.1.*
+     version: 1.6.*

Describe alternatives you've considered
Setting kube-state-metrics.image.tag: v1.6.0 in values.yaml does not work because the new version requires new RBAC permissions. The new permissions are installed with the new chart, but not by simply updating the image version.

@Nuru Would you be interested in submitting a PR to fix the issue?

@vsliouniaev I do not want to submit a PR to fix this issue because I do not want to publicize my email address, as is required to submit a PR to this repo. I'm still getting spam on an email address I used for Mozilla 20 years ago. It's a 1-line change; someone who is already a part of the project can easily make it.


Structuring New York City boroughs

Thinking about marking the cities for the NYC boroughs as
New York - Bronx
New York - Brooklyn
New York - Manhattan
New York - Queens
New York - Staten Island
That'll place the alphabetization into more-traditional order within NYS.
Ok to munge that way?

Implemented it -- happy to revert if desired.


Mindmap R3 is up

@afomi

Mindmap 3D R3

Add placards with vendor name and product.
Add more fields
Add text entry

links to web sites
A bit more text about each vendor

Add file open and file save
Add second data file

Data points for energy analysis apps (random positions for now)

Wish list

Sliding menu - so works on phone
Auto-rotate
Coming next with UI:

Adding data to new cubes / saving data to file for reuse
Polylines between data points

Coming next with the data

Data points for file types
Data points for viewers

Add data to new cubes
Edit positions of existing cubes with pointing device
And much more ;-)

😮 awesome!
pulling this down to play with it!

no need to pull it just runs:
https://rawgit.com/opentecture/mindmapping/master/mindmap-3d/r3/mindmap-3d.html

ya, I notice that. I noodled through the PushMe/PullYou file. Interesting how it work. I couldn't get it to run locally fully because it relies on rawgit, but it is useful and I'll learn more about how to use it more effectively.
I created Issues out of the items in this Issue's "Wish list".
--
In the image attached in this Issue, I see a number of new nodes with Product names, but I was unable to replicate this.  I was able to create a new node, but all fields were undefined and I was unable to edit them.  Perhaps I am missing something.

@afomi
Thank you for project managing. I will try to get used to this. ;-)

In the image attached in this Issue, I see a number of new nodes with Product names, but I was unable to replicate this.

Somehow I lost the button

Anyway 3.1 is up and 2nd JSON is callable from the button. Also you can use file dialog to open,

I was able to create a new node, but all fields were undefined and I was unable to edit them.

Yup, no way to fill up those new boxes yet. This will happen.
But I hope to find sites where interesting things can be scraped automatically,

@theo-armour - I'll add this back. My mistake.
I kept rotation, but though building-analysis-software-tools was superfluous, or a leftover.
As for sites that can be scraped, let me know what you find. My go-to for scraping would be Ruby + Nokogiri.
What'd you have in mind? What tools do you go to for scraping?

@theo-armour - I noticed the button has been added back as of https://rawgit.com/opentecture/mindmapping/master/mindmap-3d/r5/mindmap3d.html.
Accordingly, I'll close out this issue.


IP problem

Hello,
my server has 2 IPV4 addresses. With the first IP I can connect to OpenVPN, there are no problems.
But if I do the setup with the second IP, I can not connect to this IP.
If I want to connect via the Openvpn client, I get the following error message:
Client Log:
Sat Dec 09 15:13:55 2017 OpenVPN 2.4.4 x86_64-w64-mingw32 [SSL (OpenSSL)] [LZO] [LZ4] [PKCS11] [AEAD] built on Sep 26 2017
Sat Dec 09 15:13:55 2017 Windows version 6.2 (Windows 8 or greater) 64bit
Sat Dec 09 15:13:55 2017 library versions: OpenSSL 1.0.2l  25 May 2017, LZO 2.10
Sat Dec 09 15:13:55 2017 MANAGEMENT: TCP Socket listening on [AF_INET]<IP_ADDRESS>:25340
Sat Dec 09 15:13:55 2017 Need hold release from management interface, waiting...
Sat Dec 09 15:13:55 2017 MANAGEMENT: Client connected from [AF_INET]<IP_ADDRESS>:25340
Sat Dec 09 15:13:55 2017 MANAGEMENT: CMD 'state on'
Sat Dec 09 15:13:55 2017 MANAGEMENT: CMD 'log all on'
Sat Dec 09 15:13:55 2017 MANAGEMENT: CMD 'echo all on'
Sat Dec 09 15:13:55 2017 MANAGEMENT: CMD 'hold off'
Sat Dec 09 15:13:55 2017 MANAGEMENT: CMD 'hold release'
Sat Dec 09 15:13:56 2017 Outgoing Control Channel Authentication: Using 256 bit message hash 'SHA256' for HMAC authentication
Sat Dec 09 15:13:56 2017 Incoming Control Channel Authentication: Using 256 bit message hash 'SHA256' for HMAC authentication
Sat Dec 09 15:13:56 2017 TCP/UDP: Preserving recently used remote address: [AF_INET]XXX.XX.XXX.XX:1194
Sat Dec 09 15:13:56 2017 Socket Buffers: R=[65536->65536] S=[65536->65536]
Sat Dec 09 15:13:56 2017 UDP link local: (not bound)
Sat Dec 09 15:13:56 2017 UDP link remote: [AF_INET]XXX.XX.XXX.XX:1194
Sat Dec 09 15:13:56 2017 MANAGEMENT: >STATE:1512828836,WAIT,,,,,,

here are the configs:
server.conf:
port 1194
proto udp
dev tun
user nobody
group nogroup
persist-key
persist-tun
keepalive 10 120
topology subnet
server <IP_ADDRESS> <IP_ADDRESS>
ifconfig-pool-persist ipp.txt
push "dhcp-option DNS <IP_ADDRESS>"
push "dhcp-option DNS <IP_ADDRESS>"
push "redirect-gateway def1 bypass-dhcp"
crl-verify crl.pem
ca ca.crt
cert server.crt
key server.key
tls-auth tls-auth.key 0
dh dh.pem
auth SHA256
cipher AES-128-CBC
tls-server
tls-version-min 1.2
tls-cipher TLS-DHE-RSA-WITH-AES-128-GCM-SHA256
status openvpn.log
verb 3

client.ovpn:
client
proto udp
remote XXX.XX.XXX.XX 1194
dev tun
resolv-retry infinite
nobind
persist-key
persist-tun
remote-cert-tls server
auth SHA256
auth-nocache
cipher AES-128-CBC
tls-client
tls-version-min 1.2
tls-cipher TLS-DHE-RSA-WITH-AES-128-GCM-SHA256
setenv opt block-outside-dns
verb 3

/etc/network/interfaces
# The primary network interface
allow-hotplug eth0
iface eth0 inet static
        address XXX.XX.XXX.XXX     **<---- openvpn works**
        netmask <IP_ADDRESS>
        network XXX.XX.XXX.X
        broadcast XXX.XX.XXX.255
        gateway XXX.XX.XXX.1
        # dns-* options are implemented by the resolvconf package, if installed
        dns-nameservers <IP_ADDRESS> <IP_ADDRESS>
        dns-search example.com

auto eth0:1
allow-hotplug eth0:1
iface eth0:1 inet static
        address XXX.XX.XXX.XX     **<---- openvpn does not work**
        netmask <IP_ADDRESS>

I replaced the IP with XXX for Github.
Both IP addresses are reachable.
Everything works with the main IP. only the second IP makes problems.
can someone help?

I reverted the PR since, but could you try to use the local option with your secondary IP like in https://github.com/Angristan/OpenVPN-install/pull/91?

I hope I got it right. My English is not good.
I wrote "local secondary IP" to server.conf, but it still does not work. I am still connected to my main IP when I make an IP check.
in the client.ovpn is the secondary IP in it.

Well the goal of the script is not really to support this kind of setup...

Would it be possible to support multiple externally facing IP addresses?  It would be somewhat nice to be able to cherry pick various IP's from your VPS.
A valid usage case would be:
Imagine you use your VPS while you travel.  You don't want to necessarily associate your traffic while abroad to the usual IP address so you decide to use a normally unused IP address so as to leave some degree of ambiguity between the varying traffic.

In this case you would still link your metadata to the old IP if you weren't careful to avoid using accounts that you don't want associated but if you think about it this way it wouldn't be out of the ordinary to imagine a valid usage case for something like this.

@Angristan, it's not entirely necessary but it would be good to be able to change both if it's that flexible but I guess ingress IP for travel would make the most sense. But separate accounts would want egress to be changeable.


Mods relying on Guava's ClassPath are broken.

Describe the bug
Same as title.
Steps to reproduce

Create ClassPath from current ClassLoader.
No class files discovered.

Logs
Silent failures.
Additional context
I'm aware that ClassPath doesn't really work on Forge by default, as it requires a URLClassLoader somewhere in the hierarchy, which neither TransformingClassLoader nor ModuleClassLoader are, but some mods on Fabric use it.

Can you list an example mod that uses this for testing purposes?

My own Andromeda (I'm so sorry 😅) to discover modules and mixins and Fabrication, in the past, also to discover mixins. Now they use an auto-generated classpath file, but it will fallback to ClassPath if the file is removed from the jar.
Btw, I don't think it can be easily fixed, since even if you extract URLs from the ModuleClassLoader, those URLs use the UnionFileSystem.

Unfortunately I can't think of any way to avoid this on our side, sorry. Forge and Neo both use UnionFS down to the very core, and it's likely not gonna change anytime soon. Even if we were to process the union URLs and try to get the primary source, we can't account for nested JARs from which mods are loaded. If Neo decides to drop UnionPaths in the future, I'll make sure to post and update here.


Move code toward java 9 + modularisation.

Move code toward java 9 + modularisation. Reduce poor encapsulation. Further segregate transaction manager from party info related objects.

Codecov Report

Merging #827 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##             master    #827      +/-   ##
===========================================
+ Coverage     99.49%   99.5%   +<.01%
- Complexity     2021    2035      +14
===========================================
  Files           304     305       +1
  Lines          5970    6000      +30
  Branches        298     298
===========================================
+ Hits           5940    5970      +30
  Misses            5       5
  Partials         25      25

Impacted Files
Coverage Δ
Complexity Δ

.../quorum/tessera/grpc/p2p/PartyInfoGrpcService.java
100% <100%> (ø)
10 <7> (+1)
:arrow_up:

...quorum/tessera/partyinfo/PartyInfoServiceImpl.java
100% <100%> (ø)
17 <2> (+2)
:arrow_up:

...rum/tessera/partyinfo/PartyInfoServiceFactory.java
100% <100%> (ø)
1 <1> (-1)
:arrow_down:

...quorum/tessera/partyinfo/PayloadPublisherImpl.java
100% <100%> (ø)
5 <1> (+1)
:arrow_up:

...tessera/partyinfo/PartyInfoServiceFactoryImpl.java
100% <100%> (ø)
8 <8> (?)

.../main/java/com/quorum/tessera/sync/SyncPoller.java
100% <100%> (ø)
11 <1> (+1)
:arrow_up:

...om/quorum/tessera/partyinfo/ResendManagerImpl.java
97.56% <100%> (+0.12%)
8 <1> (+1)
:arrow_up:

...om/quorum/tessera/core/api/ServiceFactoryImpl.java
100% <100%> (ø)
12 <3> (ø)
:arrow_down:

.../com/quorum/tessera/partyinfo/PartyInfoPoller.java
100% <100%> (ø)
9 <5> (+1)
:arrow_up:

Continue to review full report at Codecov.

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


Acode Preview Port Issue

Description:
Issue Summary:
When attempting to use Acode with a React project running on localhost:5173 and setting the preview port to 5173, the code editor automatically previews on 5174.
Expected Behavior:
The Acode preview should correctly reflect the specified port (in this case, 5173) to match the local development environment.
Steps to Reproduce:
Run a React project on localhost:5173.
Open Acode and set the preview port to 5173.
Observe that the preview automatically uses port 5174.

Acode version?

Acode version?

Acode latest version 1.8.7(330)


2.0.5 build on jitpack is broken and can't be fetched

2.0.5 build on jitpack is broken and can't be fetched
Please check it here https://jitpack.io/#ihsanbal/LoggingInterceptor

same

same

https://jitpack.io/com/github/ihsanbal/LoggingInterceptor/2.0.5/build.log

See here: https://jitpack.io/#ihsanbal/LoggingInterceptor

Greater versions are available like 2.0.6 - 3.0.0 with fixes and improvements. Please use one of the greatest version.


[9.x] Added encoding in limit and large negative number fix

Added encoding in limit()
Previously Str::limit('Laravel', -10); would cause an error due to out of range but now it will show the ending of the limit() rather showing error. Just like Str::limit('Laravel', -10); would result in full string (Laravel).

This is a breaking change in parameters' order

This is a breaking change in parameters' order

So, what should be our steps?

@ankurk91 Updated the files, won't break the parameter sequence. Can you confirm?

It looks good, please update the doc block with new argument and its type

I think we are ready to roll.

Just a note for consideration: This is a potentially breaking change for any application that has extended this class and re-implemented this function as the argument count is changing.

Just a note for consideration: As the argument count is changing, this is a potentially breaking change for any application that has extended this class and re-implemented this function.

Indeed! Didn't think about that. So, in such case, what should be our procedure?

You can just define your own limit function if the current one doesn't meet your needs.


Fix CIP-48 query (id 4017925)

The recent changes to the CIP-48 query seem to have introduced an error. This is because the cow_protocol_ethereum.VouchRegister_evt_Vouch table uses camel case for the bondingPool and CowRewardTarget columns. This PR reverts those columns in the CIP-48 query to a naming format that is compatible with the table.

Oh good catch, and sorry for introducing this error!


Update the new Planetscale logo

Signed-off-by: Alkin Tezuysal<EMAIL_ADDRESS>Pre-submission checklist:
Please check each of these after submitting your pull request:

[ ] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[ ] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[ ] Have you picked the single best (existing) category for your project?
[ ] Does it follow the other guidelines from the new entries section?
[ ] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[ ] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[ ] Does your project/product name match the text on the logo?
[ ] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~15 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

:heavy_check_mark: Deploy preview for landscape ready!
:hammer: Explore the source changes: 920522272bae63d299e132146991fe172f5d6cd0
:mag: Inspect the deploy logs: https://app.netlify.com/sites/landscape/deploys/5fe20d154687030007b3bb2c
:sunglasses: Browse the preview: https://deploy-preview-1941--landscape.netlify.app

LGTM


API seems down

API responds with 502 on all endpoints.
Website does not list the endpoints anymore
Website landing page returns 404

Fixed, oracle downtime. Consolidating DB and API into same server with a failover soon since oracle has failed me too many times.


private room confusion

Hi guys I'm a bit confused.
Creating a private room with password (master branch) makes the room visible by anyone but not accessible without a password. Should it be PRIVATE instead (so not visible?). Whats the difference with a normal passworded room?

@0vermind a private room let's you manage which users can go into it. You can also add a password and have people get into the room that way as well.
We should make this more clear in the UI..

I have not figured out how to invite or add someone to a private room. Where is this functionality documented?


fix: exclude dep version

Fixes #85

PR-Codex overview
This PR updates the eo-parser and snakeyaml dependencies in pom.xml.
Detailed summary

Update eo-parser to version 0.29.3.
Update snakeyaml to version 2.0.

@yegor256 This pr is ready for review

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 16min)


sanitize_test.go compile errors?

Maybe I'm doing sth wrong, but I'm not able to run sanitize tests...
go test src/github.com/shurcooL/github_flavored_markdown/sanitize_test.go
src/github.com/shurcooL/github_flavored_markdown/sanitize_test.go:51: undefined: renderer
src/github.com/shurcooL/github_flavored_markdown/sanitize_test.go:89: undefined: Markdown
src/github.com/shurcooL/github_flavored_markdown/sanitize_test.go:99: undefined: Markdown
src/github.com/shurcooL/github_flavored_markdown/sanitize_test.go:107: undefined: Markdown
src/github.com/shurcooL/github_flavored_markdown/sanitize_test.go:115: undefined: Markdown
src/github.com/shurcooL/github_flavored_markdown/sanitize_test.go:123: undefined: Markdown
src/github.com/shurcooL/github_flavored_markdown/sanitize_test.go:132: undefined: Markdown
FAIL	command-line-arguments [build failed]```

Could this possibly be an issue?

The Travis build is passing, so that is an indicator that tests are passing.
It appears you're providing incorrect parameter to go test. It takes a Go package import path, not a filename path. Try this:
go test -v github.com/shurcooL/github_flavored_markdown

You can read more about the go test command at http://golang.org/cmd/go/#hdr-Test_packages.

@mr-ma, have you tried the above command? Did it work?

Yes, sorry man bad. newbie in go world ;)

No worries. :)


Implements JSON API services

•	Diagram use case services
•	Add rustless framework
•	test rustless
•	merge rustless with iron
•	implements first services structure with JSON API

Post Diagram here.


added XSIMD_VERSION_* defines back to config

the config file is missing XSIMD_VERSION_*  which are still needed in the CMakeLists.
Not sure if 8.0.0 is the correct version of master

I'd like to avoid these defines. Why are they needed?

Note: before the refactor, it was probably in a top-level header file that we parsed from CMake. (See e.g. an example of that in the xtensor project).
This allowed to

have the information in C++ macros for downstream packages including the headers.
still define the version only in one place.

This probably went away with the refactor.

parsed from CMake. (See e.g. an example of that in the xtensor project).

I don't see the xtensor reference, can you point me at it?

The parsing in CMake is still there as it should be  so I think everything is fine
https://github.com/xtensor-stack/xsimd/blob/d4bf79fec205f4bad71d1ad4521400f29d65b59d/CMakeLists.txt#L20

Yes what I meant is that it should be similar in xsimd


Date config

So the extension to the config would be:
...
"timeline": {
  "dateFormat": "MMMM Do",
  "label": "On this day"
},
...

Whole section is optional.

Ah super cool future proofing! LGTM


Plugin variables do not update once they are modified by micro.

Inside of plugins, the tabs variable only references the initial tabs that were created from the command line. If any new tabs are created using the tab command inside of micro, the Lua plugin will not see them.
I don't know much about golang or lua, but I was able to very poorly fix the problem by adding another L.SetGlobal() method into the function NewTab whenever a new tab is created. This would also have to be done whenever tabs is changed. I'm assuming there is a better method that I don't know about due to my lack of knowledge with golang.
I'm not sure if the lua library is supposed to store references to the slice, but for one reason or another, the tabs variable in lua scripts doesn't stay up to date with the global tabs variable in go.
A simple lua script that counts the number of open tabs. Open micro, then open a few tabs. The count will be incorrect.
-- $HOME/.config/micro/plugins/tabs/tabs.lua
function count()
	messenger:Message("Number of Tabs: " .. #tabs)
end

MakeCommand("tabs", "tabs.count", 0)

Specifications
Commit hash: 26da85d (and older versions)
OS: Debian 9
Terminal: xfce-terminal

@sum01, Check out #1154, added GetTabs to return tabs in Lua. Can use CurView().TabNum in place of curTab. Anything else affected by this looks like it shouldn't change during app lifecycle.
@zyedidia, maybe this would be a good place to discuss any further changes to tabs API in Lua. Fixing #1152 should be enough to close this, would you agree @davidofferman? Should further changes be made to deal with weirdness of golang's 0-based collections vs Lua's 1-based collections? I'm thinking maybe something like:
GetCurrentTabIndex() -> int
GetTabCount() -> int
GetTab(int) -> tab
GetCurrentTab() -> tab

This might be a bit more efficient in some situations than having to return the entire table with GetTabs. What do you guys think?

Since micro 2.x removed globals, it's not possible to access tabs and it's not possible to figure out how many tabs exist.


Call servicerequest from Typescript

Dear @volkanceylan,
probably still fighting with typescript, but here I get a "Uncaught TypeError: this.getSaveEntity is not a function" error:
protected getToolbarButtons() {
        let buttons = super.getToolbarButtons();

        buttons.push({
            title: Q.text('Site.TicketDialog.AcceptButton'),
            hint: "Accept Ticket",
            cssClass: 'accept-ticket-button',
            onClick: () => {
                if (!this.validateForm()) {
                    return;
                }

                var box = bootbox.prompt({
                    title: "Please enter a reason?",
                    value: "",
                    callback: function (result) {
                        if (result === null) {
                            Q.notifyWarning("test");
                        } else {
                            Q.serviceRequest(TicketService.baseUrl + "/Accept", { Entity: this.getSaveEntity() }, () => {
                                Q.notifySuccess('Case ' + this.form.SourceSystemId().get_value() + ' has been successfully assigned to ' + this.form.OwnerId() + '.', '', null);
                                Serenity.SubDialogHelper.triggerDataChange(this);
                                this.dialogClose();
                            });
                        }
                    }
                });
                box.zIndex(1200);

                // remove tabindex attribute for bootbox compatibility
                this.element.closest('.ui-dialog').removeAttr('tabindex');
            }
        });
......

I already checked Script.js file to see how code is translated, but with no luck...
Any idea?
Thanks for your support
Best Regards

"this" changes in events to clicked button etc. Saltaralle handles it with ss.mkDel by wrapping this in a closure variable, so you dont notice it. Typescript offers lambda functions that provides same functionality. So instead of onClick: function() { this.xx } do onClick: () => { this.xx }

Here problem is in callback

Solved!
Thank you very much


pkg/k8s: fix golang formatting

Fixes: 0cd1be0e64dc ("operator: GC nodes from existing CNPs")
Signed-off-by: André Martins<EMAIL_ADDRESS>

This change is

test-me-please

over seed by https://github.com/cilium/cilium/pull/7814

Coverage decreased (-0.02%) to 41.212% when pulling 8383e53ccf1dd4b4b224f6977de790bee8c66733 on pr/fix-gofmt into b0d25a8f57fcca978b8286cc9e86d6ddcb3e30d9 on master.


References are duplicated in neovim/helix

Describe the bug
To begin, this is not a duplicate of https://github.com/microsoft/pyright/issues/201
When using pyright with neovim (via the lspconfig project), I have been getting duplicates when I request textDocument/references. I had originally assumed that this was some issue with lspconfig's pyright configuration (see my issue there https://github.com/neovim/nvim-lspconfig/issues/2765 where I have links to my LSP logs), or even some upstream error with neovim itself, but I tested it today with the helix text editor and had the exact same issue.
One peculiar quirk is that this only seems to happen inside of a function definition. If I just have a single script with no functions, it's no issue, but as soon as I put that script into a main() function, they are duplicated.
Code or Screenshots
This works fine in neovim and helix
x = 5
x += 1
x = 3
print(x)

neovim:

helix:

This causes duplicates
def main():
    x = 5
    x += 1
    x = 3
    print(x)

neovim:

helix:

Hi,
Any progress on this?

Sorry but no, this isn't in our list of high priority things to look at. It doesn't have a very high number of up votes.
The fix would likely be in Pyright though, so it could be submitted by somebody outside of the Pyright/Pylance team if somebody external wanted to submit a PR.

@rchiodo, if you don't plan to fix this, you can send it back to the pyright repo, and I'll take it. I figured the pylance team would want to address it since you're the most familiar with the language server functionality and generally own its implementation.

It would likely be months before we'd look at it. Sending back.


Add display_name to mirror hostnames

These hosts were added at about the same time the display_name property was added so it wasn't configured on these 4 hosts.
If the display_name isn't provided it defaults to the FQDN of the monitoring machine which makes our monitoring quite confusing.

Merge when ready.


play audio in-line, use AwesomeFont

v0.1.9

play audio in-line in supported browsers
use AwesomeFont "play" button for audio links in other browsers
added tests for Wav, Ogg files

running locally, I get this error in the browser console:
          GET http://localhost:8000/favicon.ico 404 (Not Found)

puzzled since I see no reference to a favicon and also cannot repro in main

I'm testing it using an npm package https://github.com/lwsjs/local-web-server
ws -d _website

This should really be multiple commits - one to add in-line playing of audio, one to add FontAwesome, one to add CI/CD pushing of demos to GitHub pages.  @ultrasaurus - Your call as to how important git cleanliness is at this early stage in the development process!


Step 1 GitOps changes

What this PR does / why we need it

Towards https://github.com/giantswarm/giantswarm/issues/30971
Things to check/remember before submitting

If you made content changes

Run make lint dev to render and proofread content changes locally.
Bump last_review_date in the front matter header if you reviewed the entire page.

@mproffitt I moved to new PR that points to a new branch where all temp PRs will land before merging. So we can ensure tests are green (PR links not broken). I fixed your comments


Added the ability to add new entries to a select list when addNewEntries is enabled.

Can you make me understand the scenario

In my app I need the user to be able to add a value that doesn’t exist in
the list.
On Sun, Nov 20, 2022 at 3:41 AM Danish Amin Dar @.***>
wrote:

Can you make me understand the scenario
—
Reply to this email directly, view it on GitHub
https://github.com/danish1658/react-native-dropdown-select-list/pull/30#issuecomment-1321071223,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI24FMIJWD37ND3OPADOK43WJHP2VANCNFSM6AAAAAASDMPF7I
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

This would be extremely useful!


feat(forge): rename master branch to main

Motivation
The motivation comes from the Issue #4344.
Forge inits git repo with a "master" branch, however "main" name is used in GitHub name convention.
It creates inconsistency, and in the end people need to rename the branch themselves.
Solution
In the #4344 I left a comment how I can implement this, however new abstraction Git was added;
I added function rename_branch()  in utils.rs module:
pub fn rename_branch(self) -> Result<()> {
    self.cmd().args(["branch", "-M", "main"]).exec().map(drop)
}

and inserted this rename after creating commits in the init.rs module.
Tested this feature locally by running forge.

Thanks for attention!

The forge init default branch name is based on the user's git config. You can test this for yourself by editing your global gitconfig. First change your global gitconfig to look like this and run forge init
[init]
	defaultBranch = main

You'll see the default branch is now called main. Then change it to this:
[init]
	defaultBranch = dev

and you'll see it's called dev.
So I don't think we need to make any changes here

Thanks a lot guys, gonna close the PR then.


Allow linking to Intel OpenMP runtime without MKL

I would like to propose this change that allows linking against the Intel OpenMP runtime (if found) even if mkl-dnn is not built with MKL.

Any update on this? Thanks.

Hi @guillaumekln , Thanks for the PR. Indeed we were not allowing to link to libiomp without linking to MKL. I made a few changes to make it more robust when using windows compiler and clang. Let me know if it is ok for you.
Thanks.

Sure, thanks for the improvements!

Thanks for the contribution!


Fix pre-opening HelloController.java

I'm quite interested in gitpod, but not sure we want to do it quite like this. And the "pre-open" thing isn't making sense yet. If there are 2 changes here I recommend breaking them out and rebasing both onto master.

This looks like someone maybe just using the guide to practise or learn about github. Closing. Please open a new one if there is something you needed to fix.


webContents.sendInputEvent event argument broken

This definition is completely broken:
class WebContents extends EventEmitter {
    ...
    sendInputEvent(event: Event): void;
    ...
}

the DefinitelyType version looks like this:
class WebContents extends EventEmitter {
    ...
    sendInputEvent(event: SendInputEvent): void;
    ...
}

interface SendInputEvent {
    type: 'mouseDown' | 'mouseUp' | 'mouseEnter' | 'mouseLeave' | 'contextMenu' | 'mouseWheel' | 'mouseMove' | 'keyDown' | 'keyUp' | 'char';
    modifiers: ('shift' | 'control' | 'alt' | 'meta' | 'isKeypad' | 'isAutoRepeat' | 'leftButtonDown' | 'middleButtonDown' | 'rightButtonDown' | 'capsLock' | 'numLock' | 'left' | 'right')[];
}

interface SendInputKeyboardEvent extends SendInputEvent {
    keyCode: string;
}

interface SendInputMouseEvent extends SendInputEvent {
    x: number;
    y: number;
    button?: 'left' | 'middle' | 'right';
    globalX?: number;
    globalY?: number;
    movementX?: number;
    movementY?: number;
    clickCount?: number;
}

interface SendInputMouseWheelEvent extends SendInputEvent {
    deltaX?: number;
    deltaY?: number;
    wheelTicksX?: number;
    wheelTicksY?: number;
    accelerationRatioX?: number;
    accelerationRatioY?: number;
    hasPreciseScrollingDeltas?: boolean;
    canScroll?: boolean;
}

@miniak The docs only have type and modifiers documented.
https://electron.atom.io/docs/api/web-contents/#contentssendinputeventevent
Not sure where we could pull the large interfaces from, we would probably have to define them as structure in the Electron docs 👍

@MarshallOfSound Even though the type is defined with those 2 properties at least, it points to the global Event instead.

@miniak Yeah, missed out the bit of my comment that should have said "Wonder where they get lost" 😆

Any update on this? Is this still being worked on?


[Go Live][Feb 03]

Push to live

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

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/enable-and-configure-your-app-for-teams-meetings.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/deploy-and-publish/appsource/prepare/Test-preview-for-monetized-apps.md
:white_check_mark:Succeeded
View

msteams-platform/mstdd-landing.yml
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

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