error since bower update to 1.8.11

Hi,
Since bower is updated to 1.8.11,
I have this error:
 [exec] module.js:550
 [exec]     throw err;
 [exec]     ^
 [exec]
 [exec] Error: Cannot find module 'qs'
 [exec]     at Function.Module._resolveFilename (module.js:548:15)
 [exec]     at Function.Module._load (module.js:475:25)
 [exec]     at Module.require (module.js:597:17)
 [exec]     at require (internal/module.js:11:18)
 [exec]     at Object.<anonymous> (/usr/lib/node_modules/bower/lib/node_modules/request/lib/querystring.js:3:10)
 [exec]     at Module._compile (module.js:653:30)
 [exec]     at Object.Module._extensions..js (module.js:664:10)
 [exec]     at Module.load (module.js:566:32)
 [exec]     at tryModuleLoad (module.js:506:12)
 [exec]     at Function.Module._load (module.js:498:3)

I have the same problem.
Sorry by question but, a fix to this problem will be merged still today?

I have the same problem.
Sorry by question but, a fix to this problem will be merged still today?

Another related issue: https://github.com/bower/bower/issues/2590

Another related issue: https://github.com/bower/bower/issues/2590

Another related issue: #2590

Thanks :)

Another related issue: #2590

Thanks :)

My team have been facing this issue since the latest update, looks to be a missing dependency. If anyone needs to solve this urgently before a fix is pushed, simply install the "qs" package globally via npm i qs -g.

My team have been facing this issue since the latest update, looks to be a missing dependency. If anyone needs to solve this urgently before a fix is pushed, simply install the "qs" package globally via npm i qs -g.

I'm workingon resolving this. In the meantime 1.8.8 is marked as latest version so it should install by default

I'm workingon resolving this. In the meantime 1.8.8 is marked as latest version so it should install by default

It should be fixed in 1.8.12 which is now marked as latest. Sorry for the trouble, we need to fight against new npm versions each time we upgrade becase we bundle all dependencies

It should be fixed in 1.8.12 which is now marked as latest. Sorry for the trouble, we need to fight against new npm versions each time we upgrade becase we bundle all dependencies


Are you sure you want to leave this page?

Hi. Thanks for the great plugin.
I've run into a minor issue on content types that feature a CK editor. To reproduce the issue:

Add or edit an entry that features a CK editor
Don't make any changes to the content
Navigate to a different page

What happens
CK editor thinks that content has been modified and a browser alert asks 'Are you sure you want to leave this page?'
What should happen
If no content has been modified, it should be possible to navigate away without an alert.
Strapi version: v4.3.6
Node vserion: v14.20

I faced this bug when I was modifying collection types. Generally, all works fine, and the alert occurs when you do some changes to your collections in the content-type builder, if you save your entry after it it will continue to work fine. I'm not sure why it happens and what causes the issue but I presume that something from the hot-reload mechanism may cause it


Achievement 4.2

Wonders of the World is abnormal, please check and correct it
The statistical quantity at Achievement 4.1 is normal

Duplicate #488


[Feature Request] Implement number of uppercase character rule

Hello,
I am using your library and I would like to request a feature if possible.
I would like to be able to specify the number of required uppercase characters. So , in the specific, having the method requireUpperCase to accept an $amount (default = 1) and having the corresponding validation in the check method.
Would it be possible?
Thanks for considering this, and have a nice day :)

Hi @gioialorusso,
Sure - That sounds like a good feature for me to add. I'll work on adding it on Friday and will do a release then.
I'm also thinking about bumping to a new "major" version release and upgrade the package requirements to PHP 7 - Would this cause you any issues?
Cheers,
Bobby

This has now been implemented. I've updated the unit tests too and all seem to be fine. Please update your ballen/plexity package in composer to 1.0.3 to add this functionality 👍

Hi @allebb ,
Thanks for the new feature! I have tried it and everything is working perfectly fine.
Regarding the major version upgrade, no problem in requiring PHP 7.
BR,
Gioia


LayoutLM using roberta as initialization

Hi, I think the released LayoutLM v1 model in huggingface is initialized with BERT weights. I do see in the paper it states that initializing with roberta weights gives better results. IS there a reason for not releasing LayouLM v1 model with roberta initialization?

https://huggingface.co/microsoft/layoutlm-base-cased

Thanks for the clarification! Is there also an large model LayoutLM based on roberta initialization?


Support SwiftPM

Hi Christian, I stumbled across this package a few days ago and wanted to take it for a spin through SwiftPM 😊

Hi Kilian,
thank you for your time and effort :-) Merged!


Animations: prefers-reduced-motion workaround breaks animations

Context
Small regression in  prefers-reduced-motion approach breaking whoosh in
Summary
All elements are using css vars if user has motion enabled like so:
            @media (prefers-reduced-motion: no-preference) {
                .animation-wrapper {
                  opacity: var(--initial-opacity);
                  transform: var(--initial-transform);
                }
              }

Issue was stemming from element hierarchy like this:
<wrapper1 --initial-transform: scale(2)>
  <wrapper2 --initial-opacity: 0>
    <wrapper3 --initial-transform: translateX(100px)>

In this scenario wrapper2 would end up having these styles:
.wrapper1 {
  transform: scale(2);
}

.wrapper2 {
  opacity: 0;
  transform: scale(2);
}

.wrapper3 {
  opacity: 0;
  transform: translateX(100px)
}

Relevant Technical Choices
Reset initial vars at every level of the hierarchy so it only sets the initial style of the property that's being animated

To-do
NA

User-facing changes
Fixes whoosh in animation in preview

Testing Instructions
Check out whoosh in animation in the preview. should work as expected

QA
Check out whoosh in animation in the preview. should work as expected

[ ] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:

UAT
Check out whoosh in animation in the preview. should work as expected

[ ] UAT should use the same steps as above.

This PR can be tested by following these steps:

Reviews
Does this PR have a security-related impact?

Does this PR change what data or activity we track or use?

Does this PR have a legal-related impact?

Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[x] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[ ] This PR contains automated tests (unit, integration, and/or e2e) to verify the code works as intended (docs)
[x] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Fixes #8046

Ah, a good reminder that the C in CSS stands for cascading 🤦
Tests need updating but otherwise looks good 👍
Would be good to add some bullet points in the QA instructions about testing with and without "Reduce motion" :-)

Ah, a good reminder that the C in CSS stands for cascading 🤦

lol you and me both. my bad for not catching this earlier.
Awesome. updating tests rn and will make the QA section here a little nicer

Thanks for the quick fix!


Automatic retry failed integration test on buildkite

Pull Request Test Coverage Report for Build 400

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 75.163%

Totals

Change from base Build 396:
0.0%

Covered Lines:
8670

Relevant Lines:
11535

💛  - Coveralls


Error 'HuggingLanguageModel' object has no attribute 'generate_solution'

/opt/conda/lib/python3.10/site-packages/tree_of_thoughts/treeofthoughts.py:481
solution = self.model.generate_solution(initial_prompt, best_state)
AttributeError: 'HuggingLanguageModel' object has no attribute 'generate_solution'

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

@rkuo2000 try the new script on the readme

https://www.kaggle.com/code/rkuo2000/llm-tot
please check out the error messages on Kaggle
Error: '>=' not supported between instances of 'int' and 'str'
Setting pad_token_id to eos_token_id:50256 for open-end generation.
AttributeError: 'HuggingLanguageModel' object has no attribute 'generate_solution'
what script should be runned ?

I am trying to run with HuggingFace free models, so Huggingface Language Model is no longer supported ?

@rkuo2000 check the example on the readme md we make big new update

https://www.kaggle.com/code/rkuo2000/llm-tree-of-thoughts/notebook
HuggingfaceLLM could not generate text because of error: Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu! (when checking argument for argument index in method wrapper_CUDA__index_select), try optimizing your arguments
2024-02-25T12:46:11.196489+0000 Error running agent: local variable 'response' referenced before assignment


Addressed docs gap around user provisioning

Based on documentation feedback, addressed a docs gap around getting new users into a Mattermost instance and adding existing users to teams and channels.

Thanks Carrie! I will leave the review in Laney's hands. One thought is that it might be good to mention this functionality in the groups doc itself, 0/5

Thanks, @esethna, and great idea! I'll get that added as part of this PR.

Note to Carrie: incorporate a link to the Manage channel members documentation into this PR.


Fullscreen in 10.10 (with multiple monitors) has a black bar obscuring the the top of the window.

What steps will reproduce the problem?

open macvim in osx 10.10, toggle the native osx fullscreen function.

What is the expected output? What do you see instead?

Expected that the application would transition to the native OSX fullscreen
functionality and that the entire window would be visible. Instead, the top of
the window is obscured by a solid black bar.

Attached before and after screenshots.

What version of MacVim and OS X are you using (see "MacVim->About MacVim"
and  "Apple Menu->About This Mac" menu items, e.g. "Snapshot 40, 10.5.6
Intel")?

Custom Version 7.4 (73), Installed via homebrew.
10.10, Intel

Please provide any additional information below.

I had a similar issue in 10.9, however the bar there was gray instead of black,
and it was not consistent. This seems to happen every time.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 17 Oct 2014 at 10:49
Attachments:

before_fullscreen.png
after_fullscreen.png

This issue was moved to macvim-dev/macvim#14


Requesting a pictures results in a 0 byte file.

I tried to download a file with the following code snippet. The code snippet themselfes is the first example on the project landing page.
var request = require('request');
var fs = require('fs');
var staticUrl = "http://farm9.staticflickr.com/8358/8337105913_b41013426f_s.jpg";
request(staticUrl).pipe(fs.createWriteStream("/path/to/my/file.jpg"));
Even though the the static file is available on the url defined above, the created file on my harddisk has a filzesize of 0 bytes.
I run Ubuntu 12.04LTS and Node.JS v0.6.12

@joshtwist  can you share code of how you limited concurrent requests. I'm also getting empty image files


use with ReactCSSTransitionGroup

Hello, I would like to be able to use ReactCSSTransitionGroup with a Preact Component. Is this possible?
I've tried to set it up with Browserify and Aliasify to manage the aliases but it doesn't seem to be working.
I've set the aliases in my package.json like this:
"aliasify": {
    "aliases": {
      "react": "preact-compat",
      "react-dom": "preact-compat"
    }
  }

There must be something I'm missing. Would you have an idea?

Hi @ld0rman - I haven't yet attempted to use ReactCSSTransitionGroup with Preact. Looking through the source I'm not seeing anything that is incompatible, so it's possible this is related to #15.  If you have any error messages or if your project is public, any additional details would be super helpful. Thanks!

It's just a little POC I was working on, I've been wanting to test out React but it's too heavy to justify using at my work so I wanted to take a look at Preact. I'll put it on GitHub.
When you said you checked out the code did you mean the npm package react-addons-css-transition-group ?
Because now I'm looking further into the code I'm not sure how it would work - so would be grateful for your insight on that.

Yes, the npm package. It's really just a module ripped out of React's core, so the dependencies spider pretty wildly all over their codebase (perhaps slightly defeating the purpose of exposing it as a separate module).  preact-compat should still work with it, but there might be additional code in the bundle that we can't get rid of.   If that's the case, I think a better approach would be to reimplement this as preact-transition-group or some more obvious name.  The API and implementation is actually not that complex, it would just be a matter of making sure the semantics are the same so we're not locking people into their choice long-term.

Yes I saw that was what made me question it being able work - if we're aliasing preact-compat for react, there isn't actually a call anywhere in the code to react so I don't think it's going to work. Like I said, I'm still feeling my way around React so I'll have to think about reimplementing it myself...

Added you to the gitter room if you want.

Closing as this is now solved by installing and aliasing rc-css-transition-group:
resolve: {
  alias: {
    'react-addons-css-transition-group': 'rc-css-transition-group'
  }
}


[backport] Update examples for current version of CuPy

Backport of #4999

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit bee11445bad3606dc63fbd09856ef632c5cc5ccc, target branch v9) failed with status FAILURE.

Killed jenkins, blocked by https://github.com/chainer/chainer-test/pull/649

I think this should be a blocking.

pfnCI, test this please.

Jenkins CI test (for commit bee11445bad3606dc63fbd09856ef632c5cc5ccc, target branch v9) failed with status FAILURE.

Jenkins, test this please.

Jenkins CI test (for commit bee11445bad3606dc63fbd09856ef632c5cc5ccc, target branch v9) failed with status FAILURE.

Merging as Jenkins failure is irrelevant


holoviews / bokeh doesn't like cftime coords

Code Sample, a copy-pastable example if possible
Consider a simple working example of converting an xarray dataset to holoviews for plotting:
ref_date = '1981-01-01'
ds = xr.DataArray([1, 2, 3], dims=['time'],
                  coords={'time': ('time', [1, 2, 3],
                                   {'units': 'days since %s' % ref_date})}
                  ).to_dataset(name='foo')
with xr.set_options(enable_cftimeindex=True):
    ds = xr.decode_cf(ds)
print(ds)
hv_ds = hv.Dataset(ds)
hv_ds.to(hv.Curve)

This gives
<xarray.Dataset>
Dimensions:  (time: 3)
Coordinates:
  * time     (time) datetime64[ns] 1981-01-02 1981-01-03 1981-01-04
Data variables:
    foo      (time) int64 ...

and

Problem description
Now change ref_date = '0181-01-01' (or anything outside of the valid range for regular pandas datetime index). We get a beautiful new cftimeindex
<xarray.Dataset>
Dimensions:  (time: 3)
Coordinates:
  * time     (time) object 0181-01-02 00:00:00 0181-01-03 00:00:00 ...
Data variables:
    foo      (time) int64 ...

but holoviews / bokeh doesn't like it
/opt/conda/lib/python3.6/site-packages/xarray/coding/times.py:132: SerializationWarning: Unable to decode time axis into full numpy.datetime64 objects, continuing using dummy cftime.datetime objects instead, reason: dates out of range
  enable_cftimeindex)
/opt/conda/lib/python3.6/site-packages/xarray/coding/variables.py:66: SerializationWarning: Unable to decode time axis into full numpy.datetime64 objects, continuing using dummy cftime.datetime objects instead, reason: dates out of range
  return self.func(self.array[key])
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
/opt/conda/lib/python3.6/site-packages/IPython/core/formatters.py in __call__(self, obj, include, exclude)
    968
    969             if method is not None:
--> 970                 return method(include=include, exclude=exclude)
    971             return None
    972         else:

/opt/conda/lib/python3.6/site-packages/holoviews/core/dimension.py in _repr_mimebundle_(self, include, exclude)
   1229         combined and returned.
   1230         """
-> 1231         return Store.render(self)
   1232
   1233

/opt/conda/lib/python3.6/site-packages/holoviews/core/options.py in render(cls, obj)
   1287         data, metadata = {}, {}
   1288         for hook in hooks:
-> 1289             ret = hook(obj)
   1290             if ret is None:
   1291                 continue

/opt/conda/lib/python3.6/site-packages/holoviews/ipython/display_hooks.py in pprint_display(obj)
    278     if not ip.display_formatter.formatters['text/plain'].pprint:
    279         return None
--> 280     return display(obj, raw_output=True)
    281
    282

/opt/conda/lib/python3.6/site-packages/holoviews/ipython/display_hooks.py in display(obj, raw_output, **kwargs)
    248     elif isinstance(obj, (CompositeOverlay, ViewableElement)):
    249         with option_state(obj):
--> 250             output = element_display(obj)
    251     elif isinstance(obj, (Layout, NdLayout, AdjointLayout)):
    252         with option_state(obj):

/opt/conda/lib/python3.6/site-packages/holoviews/ipython/display_hooks.py in wrapped(element)
    140         try:
    141             max_frames = OutputSettings.options['max_frames']
--> 142             mimebundle = fn(element, max_frames=max_frames)
    143             if mimebundle is None:
    144                 return {}, {}

/opt/conda/lib/python3.6/site-packages/holoviews/ipython/display_hooks.py in element_display(element, max_frames)
    186         return None
    187
--> 188     return render(element)
    189
    190

/opt/conda/lib/python3.6/site-packages/holoviews/ipython/display_hooks.py in render(obj, **kwargs)
     63         renderer = renderer.instance(fig='png')
     64
---> 65     return renderer.components(obj, **kwargs)
     66
     67

/opt/conda/lib/python3.6/site-packages/holoviews/plotting/bokeh/renderer.py in components(self, obj, fmt, comm, **kwargs)
    257         # Bokeh has to handle comms directly in <0.12.15
    258         comm = False if bokeh_version < '0.12.15' else comm
--> 259         return super(BokehRenderer, self).components(obj,fmt, comm, **kwargs)
    260
    261

/opt/conda/lib/python3.6/site-packages/holoviews/plotting/renderer.py in components(self, obj, fmt, comm, **kwargs)
    319             plot = obj
    320         else:
--> 321             plot, fmt = self._validate(obj, fmt)
    322
    323         widget_id = None

/opt/conda/lib/python3.6/site-packages/holoviews/plotting/renderer.py in _validate(self, obj, fmt, **kwargs)
    218         if isinstance(obj, tuple(self.widgets.values())):
    219             return obj, 'html'
--> 220         plot = self.get_plot(obj, renderer=self, **kwargs)
    221
    222         fig_formats = self.mode_formats['fig'][self.mode]

/opt/conda/lib/python3.6/site-packages/holoviews/plotting/bokeh/renderer.py in get_plot(self_or_cls, obj, doc, renderer)
    150             doc = Document() if self_or_cls.notebook_context else curdoc()
    151         doc.theme = self_or_cls.theme
--> 152         plot = super(BokehRenderer, self_or_cls).get_plot(obj, renderer)
    153         plot.document = doc
    154         return plot

/opt/conda/lib/python3.6/site-packages/holoviews/plotting/renderer.py in get_plot(self_or_cls, obj, renderer)
    205             init_key = tuple(v if d is None else d for v, d in
    206                              zip(plot.keys[0], defaults))
--> 207             plot.update(init_key)
    208         else:
    209             plot = obj

/opt/conda/lib/python3.6/site-packages/holoviews/plotting/plot.py in update(self, key)
    511     def update(self, key):
    512         if len(self) == 1 and ((key == 0) or (key == self.keys[0])) and not self.drawn:
--> 513             return self.initialize_plot()
    514         item = self.__getitem__(key)
    515         self.traverse(lambda x: setattr(x, '_updated', True))

/opt/conda/lib/python3.6/site-packages/holoviews/plotting/bokeh/element.py in initialize_plot(self, ranges, plot, plots, source)
    729         if not self.overlaid:
    730             self._update_plot(key, plot, style_element)
--> 731             self._update_ranges(style_element, ranges)
    732
    733         for cb in self.callbacks:

/opt/conda/lib/python3.6/site-packages/holoviews/plotting/bokeh/element.py in _update_ranges(self, element, ranges)
    498         if not self.drawn or xupdate:
    499             self._update_range(x_range, l, r, xfactors, self.invert_xaxis,
--> 500                                self._shared['x'], self.logx, streaming)
    501         if not self.drawn or yupdate:
    502             self._update_range(y_range, b, t, yfactors, self.invert_yaxis,

/opt/conda/lib/python3.6/site-packages/holoviews/plotting/bokeh/element.py in _update_range(self, axis_range, low, high, factors, invert, shared, log, streaming)
    525             updates = {}
    526             if low is not None and (isinstance(low, util.datetime_types)
--> 527                                     or np.isfinite(low)):
    528                 updates['start'] = (axis_range.start, low)
    529             if high is not None and (isinstance(high, util.datetime_types)

TypeError: ufunc 'isfinite' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''

Similar but slightly different errors arise for different holoviews types (e.g. hv.Image) and contexts (using time as a holoviews kdim).
Expected Output
This should work.
I'm not sure if this is really an xarray problem. Maybe it needs a fix in holoviews (or bokeh). But I'm raising it here first since clearly we have introduced this new wrinkle in the stack. Cc'ing @philippjfr since he is the expert on all things holoviews.
 Part of it is that we to add the new type(s) to holoviews.util.datetime_types. Secondly it seems like bokeh's date conversion code will also have to be made aware of this new type somehow (I haven't investigated that yet).

I encountered this problem right now with the xarray built-in plotting. Does anybody know a workaround for the xarray plotting by any chance?

@jbusecke I recently stumbled upon https://github.com/SciTools/nc-time-axis.  You can install it through conda-forge:
$ conda install -c conda-forge nc-time-axis

However, for now you won't be able to use xarray's built in plotting, since xarray will raise an error if you try to plot anything with coordinates that aren't numeric or of type datetime.datetime or np.datetime64 (but nc-time-axis at least gives you matplotlib support).  You'll also need to convert your dates to nc_time_axis.CalendarDateTime objects; you can do that with a simple list comprehension (see the example in their README).

Darn. Just when I thought the time stuff was sorted. This is (yet another) deal breaker as far as recommending mass adoption goes.
Is there an estimate when, or if, cftime indexes will be supported by xarray's .plot() method?

Darn. Just when I thought the time stuff was sorted. This is (yet another) deal breaker as far as recommending mass adoption goes.

@aidanheerdegen -- support for unconventional time coordinates (via cftime) is not a trivial problem--there are many special cases and complex logic to deal with. Rather than being discouraged by this bug, I encourage you to take a longer view to see how much this support has improved over the past year. One year ago, xarray could not have decoded this time coordinate at all!
I'm curious what you are referring to with your comment about "mass adoption." To whom are you making recommendations about adoption of xarray? And what do you consider the numerous other dealbreakers to be? I hope you see that these blanket comments could be read as quite discouraging and negative to the volunteer developers working hard on xarray.

Is there an estimate when, or if, cftime indexes will be supported by xarray's .plot() method?

The work on cftimeindex is being done entirely on a volunteer basis by graduate students like @spencerkclark (👏 👏 👏). In order to make xarray development progress faster, we need more contributors to the project. In my view, the ideal place to recruit such contributors is from within the paid computational support staff from major climate modeling centers. These people are ideally poised to understand what features are most important to users, and they are actually paid to help develop tools (like xarray) to help users be more productive. We recently created a contributor guide to make it easier for new contributors to come on board. If you can think of anyone who might be interested in contributing to xarray, please let us know, either here or via a private channel.

Hi @rabernat,
I apologise if what I said is discouraging. I didn't intend it that way. It was the result of exasperation, as I think array is a fantastic tool, and I thought with the development of cftime support all barriers to widespread adoption had pretty much been overcome. When I said "another" it was in reference to the previous barrier of not supporting long time, or old time, indices which had since been overcome
As far as recommending, it is to the researchers at the centre of excellence where I am one of the people who is paid to support climate models and the support infrastructure to run and analyse their outputs. I guess I've outed myself as one of those paid computational support staff you referred to.
My initial comment above was a clumsy attempt to highlight what I thought was an important feature to support to further increase xarray adoption. From my perspective as someone who has to support users I'm often having to decide what I think the majority of users will be able to use efficiently, taking into account very wide levels of expertise and motivation. Before the cftime upgrades I did not wholeheartedly evangelise for xarray adoption because I knew there were many cases where it was not simple and easy to use. For every edge and corner case I have to support users when they encounter them. In some ways, having a tool that can do such amazing things as xarray, but which don't work in some circumstances for some datasets is very frustrating for users. It can take a lot of work to find out what doesn't work.
Having said which, we're currently doing half way through a 2 hour training session for xarray for researchers in the CoE who are interested, but not being able to easily plot cftime datasets will harm adoption, and all those who are volunteering their time developing xarray want it to be adopted as widely as possible right?
Thanks for the pointer to the contributor guide, I did read it, and I will try and find some time to make a positive contribution to xarray. I had started down that path already (https://github.com/pydata/xarray/issues/2244)

Thanks for your follow up comment. I understand much better where you are coming from now! We do sincerely appreciate your feedback and contributions. Yes: we definitely want xarray to be adopted widely!
There is no doubt that these weird calendars are a continuous source of frustration. The challenge is compounded by the fast that, basically, only the climate community needs them. All of the fancy time indexing stuff in pandas is most likely there because of its value to the finance community.
Maybe a good path forward would be for @spencerkclark to outline what steps might be needed to get xarray's built in plotting to work with cftimeindex. That way, anyone who urgently needs this feature has some sort of roadmap for starting to implement it themselves.

Sounds like a great idea.

I think a quick and dirty fix could be to add nc-time-axis as an optional dependency, and convert any cftime.datetime objects encountered in plotting routines to nc_time_axis.CalendarDateTime objects before passing them to matplotlib.
But I wonder if there is potentially room for improvement upstream in cftime?  Should we need to convert to these proxy objects before plotting or might there be a way to make cftime objects themselves more friendly to use with matplotlib?  That could have benefits for multiple libraries (not just xarray).
@pelson @lbdreyer @ocefpaf (developers of nc-time-axis and Iris) -- it seems you have grappled with this problem a fair bit already.  Have you thought about those questions before?

From a cursory examination, it sure looks (to me) that nc_time_axis could be made to directly support plotting of cftime.datetime classes. This could probably either be done in a separate package or upstream in cftime.

This could probably either be done in a separate package

Why a separate package and not in nc-time-axis? (Or alternatively, just in cftime, as you say)
FWIW https://github.com/SciTools/nc-time-axis/issues/16 tried to get the ball rolling on clarifications around the CalendarDateTime object. That was necessary in the days when there was a single PhoneyDateTime object, that didn't have calendar information associated with it. It seems that some refactoring is very plausible in nc-time-axis at this point.

Why a separate package and not in nc-time-axis? (Or alternatively, just in cftime, as you say)

Yes, of course, that would probably be preferred. 👍

Thanks @pelson, that's good to hear that you would be open to someone doing some refactoring in nc-time-axis.
@rabernat @aidanheerdegen @jbusecke I think the cleanest approach for someone interested in fixing the built-in plotting issue then would be to engage with the nc-time-axis folks to see if there is a way to enable plotting with cftime.datetime objects directly (that they would feel comfortable with).  Then in xarray, modulo some logic to handle the optional imports of cftime and nc_time_axis, I think it could just be a matter of adding cftime.datetime as one of the allowed types for plotting (within here), adding some tests, and maybe a documentation example.

For those interested in this topic, see https://github.com/SciTools/nc-time-axis/pull/42.
I'll also note that @philippjfr has already added support for plotting cftime types in holoviews, https://github.com/ioam/holoviews/pull/2728, making use of the nc_time_axis.CalendarDateTime object in its matplotlib backend.


Hypercore fails if random-access-storage is read only.

As per quick discussion of gitter with @mafintosh.

We should make hypercore check if storage.writable === false and not try the write in that case.

Closing cause old and lots have changed


Compatible with phonegap?

Thanks a lot Tarlepp for sharing!! :)
I'm relatively new to Sails and Angular, and trying to create a hybrid app that I can use with phonegap and put on the app store. The documentation around is lacking, and a lot of people seem to be confused about how they all fit together as well. Your solution could be a huge help!!
I just wanted to ask a couple of questions: Is this readily usable with phonegap, (as in you just use the frontend/dist folder)? And in this case would all the functionality (authentication, websockets, etc.. ) still work?

First of all, thanks for positive feedback! It's always nice to hear that this really help.
I haven't tried phonegap with frontend code, so I cannot be sure if that works or not. But basically after you have make dist code it should work just standalone, and if I remember right phonegap consumes basic html+css+js so you should be ok there (I haven't used phonegap in years, maybe I should take another look at somepoint).
If you get frontend code work with phonegap it would be very nice, if you could provide some pointers about that.

@tarlepp I using ionic as frontend for my. I was wondering if I could only use your backend codes.

I don't see any reason why you cannot use backend side code for your app. Closing this for now.


How can I make a query having Common Table Expression

Can you please let me know how can I make a Common Table Expression query using gorp?
To explain I have query something like following:
`query := With base_data as(select
OCC.id as occurence_id,
PROD.retailer_id,
PRO_DET.id as product_detail_id,
PROD.id as product_id,
RET.name as retailer_name,
OCC.imported_at as imported_at,
RET.currency,
PRO_DET.updated_at,
PRO_DET.reference_code,
ROW_NUMBER()OVER(partition BY PROD.retailer_id ORDER BY RET.id,PRO_DET.updated_at DESC)
FROM products  PROD,retailers  RET,occurences AS OCC
JOIN product_details AS PRO_DET ON PRO_DET.id = OCC.product_detail_id
WHERE PROD.id =   PRO_DET.product_id and RET.id = PROD.retailer_id  AND PRO_DET.reference_code = (SELECT reference_code from product_details where sku_code = '285-1004-014'))
SELECT
COALESCE(to_char(BD.imported_at, 'YYYYMMDD'), '') AS date,
COALESCE(to_char(BD.imported_at, 'Dy'), '') AS day_of_the_week,
COALESCE(BD.retailer_id, 0) AS retailer_id,
COALESCE(BD.retailer_name, '') AS retailer_name,
COALESCE(PROD.brand, '') AS brand,
COALESCE(PROD.category, '') AS category,
COALESCE(PRO_DET.model_number, '') AS model_number,
COALESCE(PRO_DET.product_code, '') AS product_code,
COALESCE(PRO_DET.sku_code, '') AS sku_code,
COALESCE(PRO_DET.reference_code, '') AS reference_code,
COALESCE(PROD.title, '') AS product_name,
COALESCE(PRO_DET.weight,'') AS weight,
regexp_replace(COALESCE(PROD.description, ''), E'[\n\r]+', ' ', 'g' ) AS product_description,
COALESCE(PRO_DET.barcode, '') AS barcode,
COALESCE(PRO_DET.color, '') AS color,
COALESCE(PRO_DET.status, '') AS status,
COALESCE(PRO_DET.size, '') AS size,
COALESCE(PROD.gender, '') AS gender,
COALESCE(PRO_DET.no_of_gears, '') AS no_of_gears,
COALESCE(PRO_DET.suspension, '') AS suspension,
COALESCE(PRO_DET.thumb_image_url, '') AS image_url_thumbnail,
COALESCE(PRO_DET.preview_image_url  , '') AS image_url_preview,
COALESCE(BD.currency, '') AS currency,
COALESCE(PRO_DET.original_price, 0) AS original_price,
COALESCE(PRO_DET.selling_price, 0) AS sale_price
FROM base_data AS BD
LEFT OUTER JOIN product_details AS PRO_DET ON PRO_DET.id = BD.product_detail_id
LEFT OUTER JOIN products AS PROD ON PROD.id = BD.product_id
LEFT OUTER JOIN shopper_types AS SHOP_TYPE ON SHOP_TYPE.id = PRO_DET.shopper_type_id
where bd.row_number =1`
Following is how I configured gorp in my project:
`import (
"database/sql"
_ "github.com/lib/pq"
"log"
"fmt"
"io/ioutil"
"path/filepath"
"gopkg.in/yaml.v2"
"gopkg.in/gorp.v1"
)
// Gorp Implementation
var dbmap *gorp.DbMap
var Dbmap *gorp.DbMap
var Db *sql.DB
var err error
var config Config
type Config struct{
Database Database
}
type Database struct{
Username string
Password string
Database string
Port string
}
func InitDb() {
setValuesConfig()
Db,err = sql.Open("postgres",getFormat())
if err != nil {
log.Fatalln(err)
}
dbmap = &gorp.DbMap{Db: Db, Dialect: gorp.PostgresDialect{}}
Dbmap = &gorp.DbMap{Db: Db, Dialect: gorp.PostgresDialect{}}
}
func setValuesConfig(){
path, err := filepath.Abs("./database/config.yml")
source, err := ioutil.ReadFile(path)
if err != nil{
log.Fatal(err)
}
err = yaml.Unmarshal(source, &config)
if err != nil{
log.Fatal(err)
}
}
func getFormat()string{
return fmt.Sprintf("user=%s password=%s dbname=%s port=%s", config.Database.Username, config.Database.Password, config.Database.Database, config.Database.Port)
}`
and following is how I was trying to fire the query the I shared in the very beginning:
 _, err = dbmap.Select(&products_data, query)
I am stuck at making this query and have no Idea why its not working. I also checked the query in DB terminal and the query as such works fine but it is not working in gorp somehow.

Apologise for opening this issue.
I found that that the problem was in the query. the sub query was returning multiple records which I didnt check properly.


If BattOr power is cut while writing to flash or if wrote to end, SD card open fails

I left a BattOr running all night on a phone with a full battery, and in the morning I could not run a SD card store on the BattOr. It went into a reset loop with an SD_OPEN halt each time it rebooted.

I seem to have encountered this bug again, this time I was able to capture an image of the first 1GB of the problematic SD card.
https://drive.google.com/file/d/0B9CX-VNv-IpicjEycGRpcHY4VDQ/view?usp=sharing
Now someone (probably me) needs to write a little python script to read the metadata from this image so we can see what the bug is.

After investigating many SD card images and seeing that they are just fine (and making the fs_dump.py script to make this easier), I realized that this bug is actually due to misreading the capacity of the SD card. This is now fixed in 41b294b7f8c41b44bcc1264ebe560306c5734b78.


Footnote/margin note workaround

Grabbed Ian's workaround here: https://bugs.chromium.org/p/chromium/issues/detail?id=1499290

Yes


Angular 7 Intergartion

It is not issued just I wanted to know how to integrate angular-flip-clock with Angular 7. Any ref for that

@tusharmestri This library was created for AngularJS 1.x and I don't know if there is Angular 7 version of http://flipclockjs.com/.
The code is pretty simple, so you can even try to write your own version ;)

Also, after v1 of FlipClock.js is released (that doesn't have jQuery as dependency) it will be easier to integrate it with Angular 7: https://github.com/objectivehtml/FlipClock/issues/312

please close the issue


Indicate why detailed data is empty in UI

🐣 Is your feature request related to a problem? Please describe.
It's confusing that Ackee displays no data for systems, devices, browsers, or sizes and doesn't explain why.
💡 Describe the solution you'd like
The empty state of detailed data widgets should be updated to include a notice that this feature needs to be enabled.
📋 Additional context
https://github.com/electerious/Ackee/issues/190
https://github.com/electerious/Ackee/issues/127

Hey, can I work on this?

Thanks for offering your help! I'm currently refactoring a lot of the UI for #40. I guess it's better to wait until all the changes are done.

Explanation has been added! It will be part of the v3 release.

Explanation has been added! It will be part of the v3 release.


Update Wayland libraries

This is pretty much a rewrite of the Wayland client code for wayland-client and wayland-protocols v0.30.0, and smithay-client-toolkit v0.17.0.
Also includes fix for #41 and some minor related tweaks/refactorings.

Looks like there are a few regressions. Holding off merging this until I've had a few days to thoroughly test.

#86 is only a partial fix. Want to get that properly sorted before merging.


Sample Code of Beebop: Accelerometer Data

Hello, I'm trying to use the ARDrone SDK3 with a Bebop Drone for a map reconstruction project and I need to know the drone position (without GPS)·
Does anyone know if the SDK offers an option to read the IMU data (accelerometer)?
I read the SDK documentation but I can't find it.
Thanks.

Hi tau90,
You won't get access to the IMU data through the SDK, sorry.
Best regards,
Djavan

Hi,
Instead of the IMU, you can have access to the attitude of the drone on the tree axes.
These data are sent through an ARCommand at around 5hz (this rate is not fixed and may change in the future).
You receive these values in an ARCOMMANDS_Decoder_ARDrone3PilotingStateAttitudeChangedCallback_t .
Regards,
Djavan


checkup: Add VirtualMachineInstance creation

Create the VirtualMachineInstance object to be used as a test subject.
A follow-up PR will also wait for the VMI to boot.
The VMI object is created with a random suffix, in order to enable execution of multiple checkup instances simultaneously.
The e2e test was manually executed against an OpenShift 4.11 cluster, with a RT configured node.
The VMI had booted successfully.

Change: Answered review comments.

Change: Take the KubeVirtChecker and ServiceAccount names from the objects and not from the constants.

Change: Rebase

Change: Fixed a small nit.

Change: Removed explicit instantiation of VMI.Spec.
Removed redundant white space before cloudInit data.


pageserver: remove ability to run without remote storage

Right now, it's possible to run pageserver without remote storage.
A lot of python tests use this, but we don't use it in prod.
Further, having to account for optionality of remote storage being configured or not induces headaches in pageserver implementation.
Proposal: make remote storage mandatory, switch tests to use local_fs / mock_s3 by default.
👍  if you agree, comment if you don't.

+1 from my side, as long as local_fs / mock_s3 are used for dev and testing setups

so all tests will require more disk space when ran locally? we rarely run all tests locally, because they take a long time, but feels like working towards this is not a great idea. I didn't check how much the test_output would grow.
I don't see the cost of optionality that bad.

well, localfs could use hardlinks, macs probably support those as well. oki, objection withdrawn.

can we have an cli argument for remote_storage_path or something, where to offload data instead of s3? pretty useful for testing or maybe someone has big enough disks to run their neon and they would put data in this folder, without having s3 setup

Note that by default, our tests delete all the large data files after each test, so that you don't run out of disk space when you run the tests locally. See PR #3090.

if needed to actually test s3 during integration tests we can use https://github.com/localstack/localstack

if needed to actually test s3 during integration tests we can use https://github.com/localstack/localstack

minio is more than enough (it's already there in docker compose), but my point is to keep minimal installation of neon as simple as possible, right now it does not require remote storage and would be nice if it won't require in the future.
Maybe we can limit functionality somehow if there is no space left and no remote storage is connected? Instead of not allowing to run neon without it.. for example, if there is no remote storage, than behaviour can be the same as if remote storage started responding with errors (just dummy remote storage, which should never be called if there is enough space)

for example, if there is no remote storage, than behaviour can be the same as if remote storage started responding with errors (just dummy remote storage, which should never be called if there is enough space)

that is a good idea -- such event can totally happen. IIUC pageserver own disk consumption will just grow in that case

Marked my earlier comment invalid but looking that my test_output is quater of my disk (250GB), I think after this change it would be more but I am too lazy to write a bash script to find me the non-remote storage using tests and double them to see their effect. Of course, the problem is that we rarely run successful tests locally.
But I do sympathize with the tech debt in $subject, the current remote storage interface has not been evolved with the evictions work and we are thinking and have been thinking alternative approaches to RemoteLayer... Haven't yet seen where we would need more than the option check or "null object" pattern, or witness of it being configured. Because it's currently in an Option, it's hard to see how one would fail to handle the two cases.


Change how status type is encoded in the protocol

Not to be merged until the client can handle the new protocol. We might need to also give bot developers a heads up as well? Discussion is all on #5590

Hm, looks like you'll also need to change:

/cmd userdetails https://github.com/Zarel/Pokemon-Showdown/blob/master/server/chat-commands.js#L4485-L4486 (it's already broken anyway, because targetUser.away is not a valid property in User...) to use targetUser.userMessage in the status key because getStatus doesn't return the status anymore.

PROTOCOL.md to reflect the newest changes.


Consider runtime CPU feature detection

I've tried the multiversion crate on top of #513 just to see what would happen.
In unfiltering, only Paeth benefits. On stable the only important change is bpp=4 improving by 15%, while 3 and 1 are unchanged. Also 2 regresses while 6 and 8 improve, but those are for 16-bit images only so they don't matter as much.
Explicit SIMD benefits a bit more, with +7% on 3bpp and +17% on 4, 6 and 8. Same regression on 2, but 2 is rare, so who cares.
There is no benefit to using AVX or AVX2 - SSE4.1 is sufficient to get optimal performance in unfiltering. This is not surprising since we literally never use 256-bit vectors in unfiltering. AVX might still be of use in filtering because it does operate on 256-bit wide chunks, but we don't have any benchmarks for those at present, so I haven't tried that yet.

Full unfiltering benchmark results on #513, stable
unfilter/filter=Paeth/bpp=1
                        time:   [7.5264 µs 7.5265 µs 7.5265 µs]
                        thrpt:  [519.00 MiB/s 519.00 MiB/s 519.01 MiB/s]
                 change:
                        time:   [-0.2168% -0.2042% -0.1927%] (p = 0.00 < 0.05)
                        thrpt:  [+0.1931% +0.2046% +0.2173%]
                        Change within noise threshold.
Found 5 outliers among 100 measurements (5.00%)
  3 (3.00%) high mild
  2 (2.00%) high severe

unfilter/filter=Paeth/bpp=2
                        time:   [12.690 µs 12.704 µs 12.724 µs]
                        thrpt:  [613.99 MiB/s 614.96 MiB/s 615.66 MiB/s]
                 change:
                        time:   [+28.839% +28.938% +29.086%] (p = 0.00 < 0.05)
                        thrpt:  [-22.532% -22.444% -22.384%]
                        Performance has regressed.
Found 8 outliers among 100 measurements (8.00%)
  2 (2.00%) low mild
  2 (2.00%) high mild
  4 (4.00%) high severe

unfilter/filter=Paeth/bpp=3
                        time:   [12.801 µs 12.801 µs 12.802 µs]
                        thrpt:  [915.42 MiB/s 915.44 MiB/s 915.46 MiB/s]
                 change:
                        time:   [-0.0576% -0.0543% -0.0509%] (p = 0.00 < 0.05)
                        thrpt:  [+0.0509% +0.0543% +0.0576%]
                        Change within noise threshold.
Found 6 outliers among 100 measurements (6.00%)
  1 (1.00%) low mild
  3 (3.00%) high mild
  2 (2.00%) high severe

unfilter/filter=Paeth/bpp=4
                        time:   [11.142 µs 11.144 µs 11.147 µs]
                        thrpt:  [1.3689 GiB/s 1.3692 GiB/s 1.3695 GiB/s]
                 change:
                        time:   [-13.170% -13.092% -13.026%] (p = 0.00 < 0.05)
                        thrpt:  [+14.977% +15.064% +15.168%]
                        Performance has improved.
Found 5 outliers among 100 measurements (5.00%)
  1 (1.00%) low severe
  1 (1.00%) low mild
  1 (1.00%) high mild
  2 (2.00%) high severe

unfilter/filter=Paeth/bpp=6
                        time:   [11.292 µs 11.292 µs 11.292 µs]
                        thrpt:  [2.0270 GiB/s 2.0270 GiB/s 2.0270 GiB/s]
                 change:
                        time:   [-11.799% -11.784% -11.772%] (p = 0.00 < 0.05)
                        thrpt:  [+13.342% +13.359% +13.377%]
                        Performance has improved.
Found 6 outliers among 100 measurements (6.00%)
  3 (3.00%) high mild
  3 (3.00%) high severe

unfilter/filter=Paeth/bpp=8
                        time:   [11.263 µs 11.267 µs 11.271 µs]
                        thrpt:  [2.7077 GiB/s 2.7087 GiB/s 2.7095 GiB/s]
                 change:
                        time:   [-25.694% -25.606% -25.533%] (p = 0.00 < 0.05)
                        thrpt:  [+34.287% +34.419% +34.578%]
                        Performance has improved.
Found 9 outliers among 100 measurements (9.00%)
  2 (2.00%) low mild
  2 (2.00%) high mild
  5 (5.00%) high severe

Full unfiltering benchmark results on #512, nightly
unfilter/filter=Paeth/bpp=1
                        time:   [7.4692 µs 7.4893 µs 7.5095 µs]
                        thrpt:  [520.18 MiB/s 521.58 MiB/s 522.98 MiB/s]
                 change:
                        time:   [-0.3953% -0.2773% -0.1736%] (p = 0.00 < 0.05)
                        thrpt:  [+0.1739% +0.2780% +0.3969%]
                        Change within noise threshold.
Found 13 outliers among 100 measurements (13.00%)
  9 (9.00%) low severe
  2 (2.00%) high mild
  2 (2.00%) high severe

unfilter/filter=Paeth/bpp=2
                        time:   [12.572 µs 12.585 µs 12.599 µs]
                        thrpt:  [620.10 MiB/s 620.80 MiB/s 621.44 MiB/s]
                 change:
                        time:   [+29.756% +30.235% +30.720%] (p = 0.00 < 0.05)
                        thrpt:  [-23.501% -23.215% -22.932%]
                        Performance has regressed.
Found 3 outliers among 100 measurements (3.00%)
  2 (2.00%) high mild
  1 (1.00%) high severe

unfilter/filter=Paeth/bpp=3
                        time:   [10.142 µs 10.147 µs 10.153 µs]
                        thrpt:  [1.1272 GiB/s 1.1278 GiB/s 1.1284 GiB/s]
                 change:
                        time:   [-7.2193% -7.1641% -7.1059%] (p = 0.00 < 0.05)
                        thrpt:  [+7.6494% +7.7169% +7.7810%]
                        Performance has improved.
Found 7 outliers among 100 measurements (7.00%)
  2 (2.00%) low mild
  5 (5.00%) high mild

unfilter/filter=Paeth/bpp=4
                        time:   [8.9340 µs 8.9373 µs 8.9433 µs]
                        thrpt:  [1.7062 GiB/s 1.7073 GiB/s 1.7080 GiB/s]
                 change:
                        time:   [-14.910% -14.813% -14.733%] (p = 0.00 < 0.05)
                        thrpt:  [+17.279% +17.389% +17.523%]
                        Performance has improved.
Found 7 outliers among 100 measurements (7.00%)
  2 (2.00%) low mild
  1 (1.00%) high mild
  4 (4.00%) high severe

unfilter/filter=Paeth/bpp=6
                        time:   [9.7180 µs 9.7236 µs 9.7307 µs]
                        thrpt:  [2.3522 GiB/s 2.3539 GiB/s 2.3552 GiB/s]
                 change:
                        time:   [-14.878% -14.698% -14.543%] (p = 0.00 < 0.05)
                        thrpt:  [+17.017% +17.231% +17.478%]
                        Performance has improved.
Found 6 outliers among 100 measurements (6.00%)
  1 (1.00%) low mild
  4 (4.00%) high mild
  1 (1.00%) high severe

unfilter/filter=Paeth/bpp=8
                        time:   [8.9602 µs 8.9612 µs 8.9624 µs]
                        thrpt:  [3.4051 GiB/s 3.4055 GiB/s 3.4059 GiB/s]
                 change:
                        time:   [-14.968% -14.708% -14.522%] (p = 0.00 < 0.05)
                        thrpt:  [+16.990% +17.244% +17.603%]
                        Performance has improved.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) low mild

The drawbacks of multiversioning are slightly increased binary size and some trivial unsafe code in a dependency, but we get to keep our #![forbid(unsafe_code)].
I'm considering adding an optional feature for multiversioning.
@anforowicz would this be of interest to Chromium? Right now Chromium only mandates SSE3, while we get improved performance from SSE 4.1, so it should be beneficial. Are these gains meaningful? Would this kind of transitive unsafe be trivial enough to enable?

I'm not thrilled about introducing proc-macro generated unsafe code. At the moment we've reduced the (non-std) transitive uses of unsafe to only alder32 and CRC computations, and the former at least may plausibly by replaced by std::simd usage in the not too distant future.

I think multiversioning is low-risk as far as unsafe code goes because unlike checksum computation it never operates on attacker-controlled values.

Assuming the performance is compelling enough, I think multiversion is fine as an optional disabled-by-default feature.
I have no reason to believe multiversion has any soundness issues, but to explain a bit why I feel slightly uncomfortable with it: It doesn't operate on attacker-controlled program inputs which certainly reduces risk, but IMO generating the unsafe blocks via proc macros adds its own risks. Or perhaps increases the difficulty of auditing for the remaining risks. Like consider this unsafe block. You can't simply look at the immediate surrounding context to figure out if it is OK, but rather need to have a broad understanding of the rest of the crate to convince yourself.
Another approach is inspecting the output after macro expansion, but that doesn't tell you whether seemingly innocuous changes to the macro expansion site could result in different and unsound code being produced.

@anforowicz would this be of interest to Chromium?

I don't have a strong opinion.

I think that the Chrome Security team would not be fundamentally opposed to unsafe if it can be somehow encapsulated and audited (*).  It seems that the multiversion crate does encapsulate the unsafe bits, but I don't know how comfortable we would be with auditing it (/cc @danakj).  It is also not entirely clear to me if mutliversion relies entirely on auto-vectorization (potentially with a more powerful/capable target) VS if multiversion allows using architecture-specific SIMD intrinsics without unsafe (I heard that @veluca93 has been exploring an approach that utilizes some nightly-rustc features for target detection so that thin wrappers over SIMD intrinsics can be used without unsafe;  I don't know what the current status of that is).
I don't have a good sense of benefit/cost here.

On one hand, having a good first impression (e.g. from great codec performance) would go a long way toward making people sympathetic toward Rust adoption.  OTOH, it is not clear how much of an impact codec performance would have on end-to-end metrics like time to first contentful paint, etc.  And we don't really have robust measurements of the current performance delta in real world scenarios (I hope to get those within a quarter or two).
It seems difficult to predict which optimizations can give the best returns.  Unfiltering performance is certainly on the radar, but I am not sure how much of an impact unfiltering/SIMD can have on small images (say, less than 32 pixels wide) which AFAIK are where Rust is at a disadvantage today.

(*) I understand that auditability is not a black/white property of a crate, but it really helps when:

unsafe blocks can be proven to be sound locally (without having to consider the whole crate and/or code outside of the crate)
unsafe blocks come with explicit safety comments that explain why the safety requirements are always met (e.g. requirements of unsafe functions called from the unsafe block; or requirements of whatever other unsafe operations are done within the block)

And the example linked above by @fintelia above illustrates how an audit may be tricky.

I understand the concern about auditability, but I don't really see optional dependencies as a particularly good way of addressing it.
We do include the standard library. Consequently it's clear that the presence of unsafe code itself is not cause for concern about a library, but a dependencies willingness and ability to ensure sound implementations of the unsafe usage. I like some unsafe code. For instance bytemuck is perfectly filling the blank (temporal) space between the a usable and highly useful abstraction now and some future builtin safe transmutation. I don't want to operate under a requirement of it becoming a full replacement and really the semantic difference will be slim if any. The ability to encapsulate unsafe behind sound usage is a large selling point. Inclusion of the encapsulation into std or the compiler mainly gets more eyeballs in review and sometimes a better understanding of under-specified boundaries.
Runtime feature detection as in multiversion seems very similar to the safe-transmute issue. The (necessary) soundness requisites are clear. It's also quite obvious why core would be cautious in providing intrinsic support: the design space for additional function attributes is just too large.
As a counter-example, I'd very much like to avoid dependencies such as lexical-core<1.0. The amount of unsafe was a concern, but not the problem per-se. They were playing it loose with regards to verification. We should somehow be able to qualify that instead of the mere existence of unsafe. And at the same time the consideration of necessity was not posed, a bunch of reactions to the RUSTSEC seem to be able to remove the dependency outright.
I'm not in favor of optional dependencies as a solution. It's a weak mitigation in itself against the purely potential threat of unsafe code, imo. What would downstream code do in case an unsound code is discovered? Removing the feature is a patch but then again we or the underlying dependency should patch, too. So this sounds quite redundant. And is a patch guaranteed to work? We've extended the libraries configuration matrix considerably if we do each of them. Is everything still guaranteed to work?
Alternative to consider:
A team of unsafe reviews, aka. cargo-crev or similar for unsafe dependencies, with a CI bot to trigger re-reviews whenever dependencies introduce / change additional unsafe code. Try not to repeat my mistake. And since this is additional effort, I'd like to find out if the Chrome Security team can somehow contribute to such reviews as they may be shared (not as much to avoid a monoculture if we're considering resilience though). For instance we should track each release of bytemuck, if possible pro-actively. Identifying the dependencies that do need reasoning reviews is half the battle here.

Google already publishes their security team's audits through cargo vet: https://opensource.googleblog.com/2023/05/open-sourcing-our-rust-crate-audits.html

Okay, so I've experimented with this a bit in #515. This is what the trade-offs look like:
In decoding, multiversioning gets up to 4% better end-to-end performance on large images that use Paeth filter, on some CPUs. On CPUs that do not benefit, there is a potential for a ~5% end-to-end improvement from multiversioning something other than the filters (source).
I'd say we shouldn't be pursuing multiversioning in decoding while other opportunities for performance improvements are available, such as https://github.com/image-rs/fdeflate/pull/31. The development effort is better spent there, not on multiversioning.
There are promising results for encoding, but not in multiversioning filters. This might be worth following up on, but AFAIK Chromium doesn't use this crate for encoding so this will not be relevant to Skia/Chromium use cases.

I've tried compiling the entire decode+encode roundtrip with RUSTFLAGS='-C target-cpu=x86-64-v3' and got a 10% end-to-end speedup when using fast compression and adaptive filtering from it, but it's not localized to any single function. The profiles look almost identical, barring a slight reduction in filtering time that cannot account for the difference on its own.
Baseline: https://share.firefox.dev/4dzkdW8
V3: https://share.firefox.dev/4dCiWhc
So it seems we won't be able to get these gains via multiversioning, or at least with anything short of cargo multivers.
Therefore, given the concerns about the unsafe code generated by proc macros, I don't think there is a sufficiently strong performance benefit to justify multiversioning at this time.


plugin list save function

hi, some time recover operator system, re-install vscode.
^-^ now, want to quickly install before plugin. hope can store plugin information.
heihei~

account sync plugin information.
or have a json config file, can store, after import it can auto install config's plugin json list.

== it is idea~thanks vscode.

Maybe this helps? https://marketplace.visualstudio.com/items?itemName=golf1052.code-sync


Translates practically empty challenges d3, react, sass

Closes #6512
Closes #6511
Closes #6507

:+1:


rm -rf !!!

It looks a little bit like the follwing line cmd := exec.Command("/bin/sh", "-c", "sudo rm -rf "+dir) deleted my system ?! Not sure, i have to investigate a little bit more but my desktop closed after i had executed freshgo and now it doesent boot with error that things like /sbin/init is missing.

I had the same issue, deleted my /usr/local/bin (I built it for darwin..). I was kicking myself for not code reviewing it before using it. This cleanup function needs to be much safer. Thankfully it was easy for me to recover from it.

@jonoden could you please provide me your GOPATH and your os version?

The only way this could be reproduced is if the GOROOT was set to /usr/local which is wrong. I added checks for this case in this commit so I keep it cloased: https://github.com/nikfot/freshgo/commit/5ac7c3919fc301497bf81be1f88a2675029ded1d

I don't have any hard evidence of what it was set to at the time as I have since recovered and reinstalled Go, but looking over the code (the version I still have pulled at any rate, I haven't pulled from origin since) it does a which go then removes /bin/go from that path to determine the directly to delete, so it seems there was perhaps a link in /usr/local/bin for some reason that took precedence in the PATH.

Yes, as I said above the previous version of Go was probably installed in /usr/local/. I have added more failsafes to this and I will be working on it in the next commits.


Redactor 3

All tests succeed, even the system/capybara tests (well, most of the time. Racing conditions seem to be racing).
Biggest changes for version 3 of Redactor (please review @GBH, I don't know if this breaks something):

lots of additional plugins (5 of them bring a little CSS which I always included because it's so little)
language/i18n files have been massively reduced (were the extra ones, e.g. "az", "ba", "bg", user contributed?) and have been partly renamed (e.g. "zh-CN" -> "zh_cn"), bringing them out of sync with the rails locales -> needs work

Let's finalize this together @GBH :) Do you have a real world project we can throw this into?

I just noticed this HowTo for a Reactor upgrade:
https://github.com/comfy/comfortable-mexican-sofa/wiki/HowTo:-Upgrading-Redactor-Wysiwyg
I did not see this before, sorry. We'll have to go through that as well I guess.

@GBH fyi: we'll be using it in a project, so we're currently fixing everything that the update broke. :)

@gr8bit 👍 Rails 6 final release probably will happen this fall. I'll jump in sometimes then to merge everything in.


Transform Manager Improvements

Do what we did in #52 but for TransformManager. The big issue here is that unlike in the other managers transforms have to move around in memory sometimes (i.e. when re-parenting the children change rows). I think at this point we should look into building a more robust scene graph system since fixing this problem is already going to require that we rebuild a large chunk of the transform system. It would be good to look at what Ogre3D's scene graph code looks like for 2.1 since it allows for client code to hold pointers directly to graph nodes yet still manages to be performant (and it's MIT licensed so we can duplicate what they're doing without licensing issues).

This change just got merged into master as part of me lazily merging all of my working changes, so this is done for the time being. We'll need to do more optimization improvements in the future but this handled the bulk of the work.


Don't upload small tensors as uniforms to packed op programs

Trivial: modify the check to prevent upload to uniform when
texData.isPacked == true and program.usesPackedTextures == true.
BUG
Description
I got this reproduced from batchnorm tests with packed binary PR on and this modification of batchnormalization4D tests.
This test would run existing batchnormTests with WEBGL_PACK flag on.
although texData.isPacked == true, and program.usesPackedTextures == true. the logic would still upload the tensor to uniform.

For repository owners only:
Please remember to apply all applicable tags to your pull request.
Tags: FEATURE, BREAKING, BUG, PERF, DEV, DOC, SECURITY
For more info see: https://github.com/tensorflow/tfjs/blob/master/DEVELOPMENT.md

This change is

@annxingyuan PTAL.
I'm aware that you have a patch in queue that enables sampling from uniforms.

LGTM - thanks a lot @astojilj for catching this! Feel free to merge.

Thanks. I don't have write access required to merge it.


Update Travis Keys

See: https://github.com/google/clasp/pull/246#issuecomment-401626948
Travis should pass afterwards.

This should have been updated with https://github.com/google/clasp/commit/43e63994c51c112a9f95ddc5efae890cae4ce431#diff-1a862d4222b65fd1f893b41b978a145b


chore: refine for clinic

// TODO

Codecov Report

Merging #1552 (17db171) into master (618eb20) will increase coverage by 2.51%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1552      +/-   ##
==========================================
+ Coverage   24.73%   27.25%   +2.51%
==========================================
  Files         168       93      -75
  Lines       14998    10097    -4901
==========================================
- Hits         3710     2752     -958
+ Misses      11017     7165    -3852
+ Partials      271      180      -91

Flag
Coverage Δ

backend_integration
?

backend_ut
27.25% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report in Codecov by Sentry.

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


cv verification result topics

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[x] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

LGTM. Per slack we chatted about removing a duplicate topic with these new topics.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


[BUG] Ghost categories?

Expected Behaviour
After the upgrade, pre-existing products and categories should not be changed
Actual behaviour
After the update, some categories are ghosted: they do not appear from the list of categories, but they still appear on the site and can be assigned to products

Version
v.3.0.4-beta.26

Looks like they arent tagged to a location - have you switched between single and multi location mode at some point?

Looks like they arent tagged to a location - have you switched between single and multi location mode at some point?

@ryanmitchell It was a previous test environment so it is possible that I have used that switch at some point.
We should have a migration table/routine that reassign all categories and item to the default location when you switch from single to multi?
And we should make mandatory to assign a location to menuitem and categories and so on?
Or maybe the best approach is to assign location even in single store mode when you create a menuitem or a category

@ryanmitchell It was a previous test environment so it is possible that I have used that switch at some point.
We should have a migration table/routine that reassign all categories and item to the default location when you switch from single to multi?
And we should make mandatory to assign a location to menuitem and categories and so on?
Or maybe the best approach is to assign location even in single store mode when you create a menuitem or a category

Right, its the missing location issue.
@BreakSecurity do you want to have a go at making locations selection required throughout the system to avoid issues like this one? Also the demo data on a fresh install, those should be assigned to the default location.

Right, its the missing location issue.
@BreakSecurity do you want to have a go at making locations selection required throughout the system to avoid issues like this one? Also the demo data on a fresh install, those should be assigned to the default location.


Use env address() instead of caller()

ext.env().caller() - origin AccountId which calls contract
ext.env().address() - contract AccountId which calls chainextension
This PR changes allows contract AccoutId to have ownership of collections, nfts etc.
Corss contract call case, for example solidity, msg.sender is caller contract address. So, This (caller of chain extension is contract address) should be expected behavior.

Is there any way to have this configurable from the ink side?
For example if I want to call RmrkCore::Send on behalf of a user (ie. for an escrow), I'm not sure if it's possibly when only using env.ext().address().


[fix][broker][branch-2.9] Fix no value present

Fixes #14877
Motivation
Fixes #14877
Modifications
cherry pick from #13950
Documentation
Check the box below or label this PR directly (if you have committer privilege).
Need to update docs?

[ ] doc-required
(If you need help on updating docs, create a doc issue)

[x] no-need-doc
(Please explain why)

[ ] doc
(If this PR contains doc changes)

/pulsarbot run-failure-checks

@codelipenghui  PTAL


Add missing youtube live stream from Cody

title: Add new youtube live stream from Cody

Discord Username: @
What type of PR is this? (select all that apply)

[ ] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 🚧 Breaking Change
[ ] 🧑💻 Code Refactor
[ ] 📝 Documentation Update

Description

Related Tickets & Documents

Related Issue #
Closes #

QA Instructions, Screenshots, Recordings

UI accessibility concerns?

Added/updated tests?

[ ] 👍 yes
[ ] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

[optional] Are there any post deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

thank you!


fix(ci): jobs are canceled on submitted review

we are using pull_request_review.submitted trigger to have CIs running for release PRs and releases
the issue is that when someone submits a review on ordinary PR, it creates a bunch of jobs which are immediately skipped, but they still cancel already running jobs because they are in the same group

I was already wondering where this behavior was coming from, nice that you already found that out

I was already wondering where this behavior was coming from, nice that you already found that out

Yeah, I was thinking what is causing it and went down the rabbit-hole about cancel-in-progress and that it can take expressions, so I was thinking how to cancel only if the new job was not skipped ... and then it hit me - let's just change the concurrency.group.


Getting java.lang.IncompatibleClassChangeError

When I use the method:
new PegDownProcessor().markdownToHtml("Some **markdown**")
I'm getting the error:
java.lang.IncompatibleClassChangeError: class org.objectweb.asm.tree.ClassNode has interface org.objectweb.asm.ClassVisitor as super class
I found out that the issue https://github.com/sirthias/pegdown/issues/66 brings the same error for the version 1.2 of pegdown.
But this time I'm using pegdown 1.6 from maven. My dependencies are:

pegdown 1.6.0
parboiled 1.1.7

And the parboiled have the following:

parboiled-core 1.1.7
asm 5.0.3
asm-tree 5.0.3
asm-analysis 5.0.3
asm-util 5.0.3

Any clues? Thank you.

It seems I have problems with the Atlassian SDK, which my application is based on. The Atlassian SDK is using ASM 3.3.1. Because the problem is not with pegdown, I'm closing this.


New version: IanWalton.JellyfinMPVShim version 2.8.0

Pull request has been created with komac v2.2.1 :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:
(Automated response - build 886.)


Implemented: mobile view support for the inprogress page of fulfillment app (#244)

Related Issues

Related Issue: #244
Short Description and Why It's Useful

Implemented the mobile view for the inprogress page in the fulfillment app.

Implemented the EditPackagingModal and ReportIssueModal so as to display order details and items in the modal.
Added the functionality to the modal's popovers and buttons.

Screenshots of Visual Changes before/after (If There Are Any)

Before

After

IMPORTANT NOTICE - Remember to add changelog entry
Contribution and Currently Important Rules Acceptance

[x] I read and followed contribution rules

Closing as the apps UI is getting changed and there is no immediate requirement of mobile support.


network output

Using this implementation for 2 class problem (I'm using 1 channel target image with binary(0,1) value), but my output is not binary (0,1). did I miss something? should I add some activation function at the last layer?

Yes, you need to apply sigmoid to the model output as in the example:
# Predict
pred = model(inputs)
# The loss functions include the sigmoid function.
pred = F.sigmoid(pred)
pred = pred.data.cpu().numpy()

The reason behind this is to use F.binary_cross_entropy_with_logits, which combines BCE+sigmoid in one layer and applies the log-sum-exp trick for numerical stability.


use of enum 'lightStatus_t' without previous declaration

I keep running into this error and searching the web for a solution but to know avail.
I am using Arduino IDE 1.8.7 (Windows Store 1.18.15) and the latest version of Aquacontrol32.
Below is the error message log.
Arduino: 1.8.7 (Windows Store <IP_ADDRESS>) (Windows 10), Board: "LOLIN D32, Default, 80MHz, 921600, None"
aquacontrol32:114:6: error: use of enum 'lightStatus_t' without previous declaration
enum lightStatus_t
  ^

aquacontrol32:119:39: error: 'lightStatus_t' does not name a type
const char lightStatusToString( const lightStatus_t status )
                                   ^

aquacontrol32:119:60: error: two or more data types in declaration of 'lightStatusToString'
const char lightStatusToString( const lightStatus_t status )
                                                        ^

aquacontrol32:115:1: error: expected unqualified-id before '{' token
{
^
aquacontrol32:119:39: error: 'lightStatus_t' does not name a type
const char lightStatusToString( const lightStatus_t status )
                                   ^

C:\Users\Tom\Documents\Arduino\aquacontrol32\aquacontrol32.ino: In function 'const char lightStatusToString(int)':
aquacontrol32:123:10: error: 'LIGHTS_OFF' was not declared in this scope
 case LIGHTS_OFF:   return PSTR(" LIGHTS OFF");

      ^

In file included from C:\Users\Tom\Documents\ArduinoData\packages\esp32\hardware\esp32\1.0.0\cores\esp32/WString.h:29:0,
             from C:\Users\Tom\Documents\ArduinoData\packages\esp32\hardware\esp32\1.0.0\cores\esp32/Arduino.h:150,

             from sketch\aquacontrol32.ino.cpp:1:

C:\Users\Tom\Documents\ArduinoData\packages\esp32\hardware\esp32\1.0.0\cores\esp32/pgmspace.h:36:25: error: invalid conversion from 'const char*' to 'char' [-fpermissive]
#define PSTR(s)       (s)
                     ^

C:\Users\Tom\Documents\Arduino\aquacontrol32\aquacontrol32.ino:123:31: note: in expansion of macro 'PSTR'
 case LIGHTS_OFF:   return PSTR(" LIGHTS OFF");

                           ^

aquacontrol32:124:10: error: 'LIGHTS_ON' was not declared in this scope
 case LIGHTS_ON:    return PSTR(" LIGHTS ON ");

      ^

In file included from C:\Users\Tom\Documents\ArduinoData\packages\esp32\hardware\esp32\1.0.0\cores\esp32/WString.h:29:0,
             from C:\Users\Tom\Documents\ArduinoData\packages\esp32\hardware\esp32\1.0.0\cores\esp32/Arduino.h:150,

             from sketch\aquacontrol32.ino.cpp:1:

C:\Users\Tom\Documents\ArduinoData\packages\esp32\hardware\esp32\1.0.0\cores\esp32/pgmspace.h:36:25: error: invalid conversion from 'const char*' to 'char' [-fpermissive]
#define PSTR(s)       (s)
                     ^

C:\Users\Tom\Documents\Arduino\aquacontrol32\aquacontrol32.ino:124:31: note: in expansion of macro 'PSTR'
 case LIGHTS_ON:    return PSTR(" LIGHTS ON ");

                           ^

aquacontrol32:125:10: error: 'LIGHTS_AUTO' was not declared in this scope
 case LIGHTS_AUTO:  return PSTR("LIGHTS AUTO");

      ^

In file included from C:\Users\Tom\Documents\ArduinoData\packages\esp32\hardware\esp32\1.0.0\cores\esp32/WString.h:29:0,
             from C:\Users\Tom\Documents\ArduinoData\packages\esp32\hardware\esp32\1.0.0\cores\esp32/Arduino.h:150,

             from sketch\aquacontrol32.ino.cpp:1:

C:\Users\Tom\Documents\ArduinoData\packages\esp32\hardware\esp32\1.0.0\cores\esp32/pgmspace.h:36:25: error: invalid conversion from 'const char*' to 'char' [-fpermissive]
#define PSTR(s)       (s)
                     ^

C:\Users\Tom\Documents\Arduino\aquacontrol32\aquacontrol32.ino:125:31: note: in expansion of macro 'PSTR'
 case LIGHTS_AUTO:  return PSTR("LIGHTS AUTO");

                           ^

C:\Users\Tom\Documents\ArduinoData\packages\esp32\hardware\esp32\1.0.0\cores\esp32/pgmspace.h:36:25: error: invalid conversion from 'const char*' to 'char' [-fpermissive]
#define PSTR(s)       (s)
                     ^

C:\Users\Tom\Documents\Arduino\aquacontrol32\aquacontrol32.ino:126:31: note: in expansion of macro 'PSTR'
 default:           return PSTR(" UNDEFINED ");

                           ^

C:\Users\Tom\Documents\Arduino\aquacontrol32\aquacontrol32.ino: At global scope:
aquacontrol32:177:1: error: 'lightStatus_t' does not name a type
lightStatus_t           lightStatus;
^
C:\Users\Tom\Documents\Arduino\aquacontrol32\aquacontrol32.ino: In function 'void setup()':
aquacontrol32:234:33: error: no matching function for call to 'Adafruit_ILI9341::begin(int, SPIClass&)'
tft.begin( TFT_SPI_CLOCK, SPI );
                             ^

In file included from C:\Users\Tom\Documents\Arduino\aquacontrol32\aquacontrol32.ino:7:0:
C:\Users\Tom\Documents\Arduino\libraries\Adafruit_ILI9341/Adafruit_ILI9341.h:131:17: note: candidate: virtual void Adafruit_ILI9341::begin(uint32_t)
     void    begin(uint32_t freq=0);

             ^

C:\Users\Tom\Documents\Arduino\libraries\Adafruit_ILI9341/Adafruit_ILI9341.h:131:17: note:   candidate expects 1 argument, 2 provided
C:\Users\Tom\Documents\Arduino\aquacontrol32\dimmertask.ino: In function 'void dimmerTask(void*)':
dimmertask:42:3: error: 'lightStatus' was not declared in this scope
lightStatus = LIGHTS_AUTO;
^
dimmertask:42:17: error: 'LIGHTS_AUTO' was not declared in this scope
lightStatus = LIGHTS_AUTO;
             ^

C:\Users\Tom\Documents\Arduino\aquacontrol32\dimmertask.ino: In function 'void lightsOn()':
dimmertask:137:3: error: 'lightStatus' was not declared in this scope
lightStatus = LIGHTS_ON;
^
dimmertask:137:17: error: 'LIGHTS_ON' was not declared in this scope
lightStatus = LIGHTS_ON;
             ^

C:\Users\Tom\Documents\Arduino\aquacontrol32\dimmertask.ino: In function 'void lightsOff()':
dimmertask:148:3: error: 'lightStatus' was not declared in this scope
lightStatus = LIGHTS_OFF;
^
dimmertask:148:17: error: 'LIGHTS_OFF' was not declared in this scope
lightStatus = LIGHTS_OFF;
             ^

C:\Users\Tom\Documents\Arduino\aquacontrol32\dimmertask.ino: In function 'void lightsAuto()':
dimmertask:153:3: error: 'lightStatus' was not declared in this scope
lightStatus = LIGHTS_AUTO;
^
dimmertask:153:17: error: 'LIGHTS_AUTO' was not declared in this scope
lightStatus = LIGHTS_AUTO;
             ^

C:\Users\Tom\Documents\Arduino\aquacontrol32\tfttask.ino: In function 'void showMenu()':
tfttask:155:10: error: 'lightStatus_t' does not name a type
static lightStatus_t displayedLightStatus;
      ^

tfttask:168:5: error: 'displayedLightStatus' was not declared in this scope
 displayedLightStatus = lightStatus;

 ^

tfttask:168:28: error: 'lightStatus' was not declared in this scope
 displayedLightStatus = lightStatus;

                        ^

tfttask:181:8: error: 'displayedLightStatus' was not declared in this scope
if ( displayedLightStatus != lightStatus )
    ^

tfttask:181:32: error: 'lightStatus' was not declared in this scope
if ( displayedLightStatus != lightStatus )
                            ^

C:\Users\Tom\Documents\Arduino\aquacontrol32\tfttask.ino: In function 'void showStatus()':
tfttask:233:10: error: 'lightStatus_t' does not name a type
static lightStatus_t displayedLightStatus;
      ^

C:\Users\Tom\Documents\Arduino\aquacontrol32\tfttask.ino: In function 'void drawMenuButtons()':
tfttask:431:24: error: 'lightStatus' was not declared in this scope
tempButton.color = ( lightStatus == LIGHTS_ON ) ? ILI9341_RED : ILI9341_BLUE;
                    ^

tfttask:431:39: error: 'LIGHTS_ON' was not declared in this scope
tempButton.color = ( lightStatus == LIGHTS_ON ) ? ILI9341_RED : ILI9341_BLUE;
                                   ^

tfttask:435:39: error: 'LIGHTS_OFF' was not declared in this scope
tempButton.color = ( lightStatus == LIGHTS_OFF ) ? ILI9341_RED : ILI9341_BLUE;
                                   ^

tfttask:439:39: error: 'LIGHTS_AUTO' was not declared in this scope
tempButton.color = ( lightStatus == LIGHTS_AUTO ) ? ILI9341_RED : ILI9341_BLUE;
                                   ^

C:\Users\Tom\Documents\Arduino\aquacontrol32\webservertask.ino: In lambda function:
webservertask:293:67: error: 'lightStatus' was not declared in this scope
   response->printf( "%s\n%s\n", timeStr, lightStatusToString( lightStatus ) );

                                                               ^

C:\Users\Tom\Documents\Arduino\aquacontrol32\webservertask.ino: In lambda function:
webservertask:462:68: error: 'lightStatus' was not declared in this scope
   return request->send( 200, HTML_HEADER, lightStatusToString( lightStatus ) );

                                                                ^

webservertask:462:82: error: return-statement with a value, in function returning 'void' [-fpermissive]
   return request->send( 200, HTML_HEADER, lightStatusToString( lightStatus ) );

                                                                              ^

webservertask:468:68: error: 'lightStatus' was not declared in this scope
   return request->send( 200, HTML_HEADER, lightStatusToString( lightStatus ) );

                                                                ^

webservertask:468:82: error: return-statement with a value, in function returning 'void' [-fpermissive]
   return request->send( 200, HTML_HEADER, lightStatusToString( lightStatus ) );

                                                                              ^

webservertask:474:68: error: 'lightStatus' was not declared in this scope
   return request->send( 200, HTML_HEADER, lightStatusToString( lightStatus ) );

                                                                ^

webservertask:474:82: error: return-statement with a value, in function returning 'void' [-fpermissive]
   return request->send( 200, HTML_HEADER, lightStatusToString( lightStatus ) );

                                                                              ^

Multiple libraries were found for "SSD1306.h"
Used: C:\Users\Tom\Documents\Arduino\libraries\ESP8266_and_ESP32_Oled_Driver_for_SSD1306_display
Not used: C:\Users\Tom\Documents\Arduino\libraries\esp8266-oled-ssd1306-3.2.5
Not used: C:\Users\Tom\Documents\Arduino\libraries\arduino_880236
Multiple libraries were found for "WiFi.h"
Used: C:\Users\Tom\Documents\ArduinoData\packages\esp32\hardware\esp32\1.0.0\libraries\WiFi
Not used: C:\Program Files\WindowsApps\ArduinoLLC.ArduinoIDE_<IP_ADDRESS>_x86__mdqgnx93n4wtt\libraries\WiFi
exit status 1
use of enum 'lightStatus_t' without previous declaration
This report would have more information with
"Show verbose output during compilation"
option enabled in File -> Preferences.
Please advise.

I will look into this.

This issue should be fixed with 12f425399c4a6f3f1bfe6d100e0aef7247627ed6.
Still no W10 pc to test this, so some feedback would be welcome.

I will get to it later today.
On Fri, Sep 28, 2018 at 8:46 AM Cellie<EMAIL_ADDRESS>wrote:

This issue should be fixed with 12f4253
https://github.com/CelliesProjects/aquacontrol32/commit/12f425399c4a6f3f1bfe6d100e0aef7247627ed6
.
Still no W10 pc to test this, so some feedback would be welcome.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CelliesProjects/aquacontrol32/issues/9#issuecomment-425424383,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Aph18BWudpAPXcoE1cUp9fgWpcSG68dfks5ufho3gaJpZM4W27WE
.

Just tested it on a borrowed W10 laptop and https://github.com/CelliesProjects/aquacontrol32/commit/12f425399c4a6f3f1bfe6d100e0aef7247627ed6 seems to work fine.

The error
C:\Users\Tom\Documents\Arduino\aquacontrol32\aquacontrol32.ino: In function 'void setup()':

aquacontrol32:234:33: error: no matching function for call to 'Adafruit_ILI9341::begin(int, SPIClass&)'

tft.begin( TFT_SPI_CLOCK, SPI );

is because the ILI9341 library you are using is not the correct version.
Check  your included libs.

Thank you I am going to test it as soon as I can get to it.
On Thu, Oct 11, 2018 at 10:56 AM Cellie<EMAIL_ADDRESS>wrote:

The error
C:\Users\Tom\Documents\Arduino\aquacontrol32\aquacontrol32.ino: In function 'void setup()':
aquacontrol32:234:33: error: no matching function for call to 'Adafruit_ILI9341::begin(int, SPIClass&)'
tft.begin( TFT_SPI_CLOCK, SPI );
is because the ILI9341 library you are using is not the correct version.
Check your included libs
https://github.com/CelliesProjects/aquacontrol32/blob/master/libraries.md
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CelliesProjects/aquacontrol32/issues/9#issuecomment-428983508,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Aph18NQzGWBRg_cUo7eUHZI0qyaKRxXfks5uj1wvgaJpZM4W27WE
.


Support ONNX SAME_UPPER

Add support for SAME_UPPER. Looks like Caffe2 does not support SAME_LOWER by default.

Fix the bug in C++ ONNX C2 backend. If an attribute is renamed to empty string, we should delete it.

Looks like we have some CuDNN fall back issue, will fix it.

https://github.com/caffe2/caffe2/pull/2362 removed the python code. We can just focus on C++ code. :)


FEATURE: Responsive Ads Code

UPDATE: this needs to be added in the ad format for responsive
data-ad-format="auto"

it will cater all add formats
here is the difference of the code in terms of the inline style ads vs responsive ads
responsive ads code

720 x 90 ads code

Hey @rnovino
Thanks for the feedback! Considering that Adsense does have size-specific and responsive-type ads, we should definitely get to supporting both options. Thanks for bringing this point up, it's great to hear from users on what could be improved!

Hi @rnovino,
Thanks for your issue.  Based on the following guidance from google adsense: https://support.google.com/adsense/answer/3213689?hl=en, responsive ads are:

"Auto" as the size, they adjust size for a preexisting container within the page.
Adjust for different views (mobile/desktop).

With discourse and specifically this plugin:

There is no preexisting container for an ad, the container is set by the size of the ad that a user would like, so auto wouldn't really work here.  Even if someone wanted auto, they'd still have to set the size of the container here.
We have mobile sized ads for account for the mobile view.

We'll close this issue for now but if enough people request this feature on Discourse meta, it's something we'll definitely look at reopening.

I don't understand, the code should inject a DIV with 100% width and the adsense code in it, why it shouldn't work, I do it for all of my websites.


Frame Props dialog shows BT.709 Primaries as Unspecified

Gotta admit this is a weird one, because I checked the code and the primaries are indeed in the correct order in the list vspreview has.
The issue I'm seeing is that, when opening the Frame Props dialog for a script where _Primaries is set to 1, the Frame Props dialog shows "Unspecified" instead of the expected "BT.709". Matrix and Transfer properties display correctly, and all other Primaries except BT.709 also display correctly.

I don't know what's up with the props lately, I also got people telling me that limited color range is displaywled as full.
I will fix all of these issues once I switch to the new package I,ve written, vstools, which has got all the enums needed and helper functions to get the values from frame props _prop_erly.

Fixed in https://github.com/Irrational-Encoding-Wizardry/vs-preview/commit/eafdde3a6d89f573c9485aced1f8f0b831f3e771


whereNotIn() must be of the type array

Hello,
So I was thing to do the following query:
SELECT `orig_id` FROM `RPTC` WHERE `id` NOT IN (SELECT `id` FROM `PCT`)

My code was:
// Sub Query
$query2 = $db->table("PCT")->select("id");

// Main Query
$query = $db->table("RPTC")
			->select("orig_id")
			->whereNotIn("id", $db->subQuery($query2));

However I got this error message:
 Catchable fatal error: Argument 2 passed to Pixie\QueryBuilder\QueryBuilderHandler::whereNotIn() must be of the type array, object given, called

Is there any specific reason why Pixie doesn't allow for a sub-query as the whereIn / whereNotIn second parameter? How can this be accomplished? Apparently a simple where() accepts sub-queries.
Thank you.

Side Note: I'm aware I could use LEFT JOIN in combination with WHERE PCT.id IS NULL to accomplish the same result, however the ability to include the sub-queries can become useful in other situations...

Use
$query = $db->table("RPTC")
->select("orig_id")
->where($db->raw('id NOT IN (SELECT id FROM PCT)'));

@marc937 shouldn't subQuery() actually work?

Yes it should. This issue could be marked as enhancement.

For now please use Raw.

@usmanhalalit thanks, that's what I've been doing.

Have same problem

@TCB13 can you please update and check if the solution works?
I just merged #99

@usmanhalalit the issue seems to be fixed on whereIn() however the problem is still present on whereNotIn().


Disable Event Tracker on all Windows Servers

Operating System Version: Windows Server
Provider (VirtualBox/VMWare): All
Vagrant Version: latest
Packer Version: latest
Is the issue reproducible or intermittent?
reproducible

Description of the issue:
On Windows Server there exist a "Shutdown event tracker" and it displays a popup window after operating system reboot and user logins when we do "vagrant-reload".

I suggest powershell script that add registry setting to disable it.
I think it unnessesary setting on test lab vms.
Here an example powershell script block that do this.
# Purpose: Disable Server Event tracker. that spawns a dialog window on Windows Servers after accidentally restart

#Lets test that Parent key is exist in registry. if it is not exist, lets create it
if ( -Not (Test-Path 'registry::HKEY_LOCAL_MACHINE\SOFTWARE\Policies\Microsoft\Windows NT\Reliability'))
 {
  New-Item -Path 'registry::HKEY_LOCAL_MACHINE\SOFTWARE\Policies\Microsoft\Windows NT' -Name Reliability -Force
  }
#Lets add registry setting
Set-ItemProperty -Path 'registry::HKEY_LOCAL_MACHINE\SOFTWARE\Policies\Microsoft\Windows NT\Reliability' -Name ShutdownReasonOn -Value 0

We must add this script in VagrantFile.
winsrv_config.vm.provision "shell", path: "scripts/disable-event-tracker.ps1", privileged: true

Another option is to disable it with local or group policy, but i think it will add unnessesary complexity
Another suggestion is to add this setting in packer scripts.

Added this in #149 to the Packer build scripts


Upload a zip archive of all songs for a bust

So that people who want to archive the songs can do so easily.
We wouldn't be able to host the songs on Discord, but if we end up integrating with google services as part of #12, then uploading to google drive may be an option.
Perhaps either attaching it as a link to the !list command output, or sending it as a message at the end of a bust.
A request from Toasterless.

I'm against attaching it to the end of !list as the compress+upload time would greatly increase latency of the command.
I might be okay with sending it in the "thanks for coming" message. I don't think this is a feature I'd implement myself as it doesn't to save that much time over just clicking every single direct link in the pinned message in sequence and downloading that way.

Perhaps an !archive command that would need to be run after !list.
We could just post these at the end of an archive bust channel.


Set CSS property font-family

This PR fixes fonttbl parsing when the individual fontinfo segments are not separated by curly braces and sets the specified fonts on CHPs.

Have you had time to look at this PR?

I have looked at it but haven't had time to read the relevant parts in the spec.  Do you happen to have a file for testing this?  At first glance, I think it's fine, but I'd like to play with it ;-)

The two files  used for testing are http://thewalter.net/stef/software/rtfx/sample.rtf and http://www.scottlynch.us/files/rsurs-sample.rtf
The second one has the fontinfo without brackets


Set personal goals with Jack and Mike

Set personal Development goals for Chris in order to improve efficiency and accountability.

https://www.notion.so/tonomy-foundation/2023-03-30-Chris-and-Jack-ca7cef8352c543bd9b6cd7724f7b110a?pvs=4

https://www.notion.so/tonomy-foundation/Chris-s-SMART-Goals-Summary-f553e745f3044908a259e6266fd29a29