[PyTorch] Fix FP8 logic related to FA2/FA3

Description
This PR fixes the backend selection logic when FA2 is present but FA3 is not. It fixes tests test_mha_fp8_vs_f16, test_dpa_fp8_vs_f16 and test_custom_mha_fp8_vs_f16 in tests/pytorch/fused_attn/test_fused_attn.py.
It also enforces preference for FusedAttention over FA3 when FP8 execution is required. FA3 currently assumes all scaling factors to be 1, while FusedAttention allows the use of more customized scaling factors (to each tensor and each iteration) in fp8_meta.
Type of change

[ ] Documentation change (change only to the documentation, either a fix or a new content)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Infra/Build change
[ ] Code refractor

Changes
Please list the changes introduced in this PR:

Sets use_flash_attention=False when FA3 is not available, for FP8 tests
Avoids FA3 tests when it's not installed
Prefers FusedAttention over FA3 for FP8 attention, when both FA3 and FusedAttention are eligible backends

Checklist:

[x] I have read and followed the contributing guidelines
[x] The functionality is complete
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/te-ci pytorch

/te-ci pytorch

Hi @cyanguwa, I feel the warning of missing FA3 is a bit annoying, as users may be using GPUs other than Hopper or just want to use cuDNN fused attention, but they still see these warnings.
https://github.com/NVIDIA/TransformerEngine/blob/4ddb0a7bea787294282d0fe0715adf5ea4a39779/transformer_engine/pytorch/attention.py#L91-L98

Hi @cyanguwa, I feel the warning of missing FA3 is a bit annoying, as users may be using GPUs other than Hopper or just want to use cuDNN fused attention, but they still see these warnings.
https://github.com/NVIDIA/TransformerEngine/blob/4ddb0a7bea787294282d0fe0715adf5ea4a39779/transformer_engine/pytorch/attention.py#L91-L98

I added some restriction to the warning (e95ef54). Thanks.

/te-ci pytorch

mask_9_0 failure was due to cuDNN regression. cuDNN 9.3 passes that test.


Trying to use --bootstrap option

Not sure if I am missing something but I am passing " --bootstrap /my/file/path " into the command line options. When running I get unrecognised option error.
Any help would be appreciated.

I set my path to bootstrap.php in the required phpunit.xml file. This is, of course, relative to the project root, so no leading slash.
<?xml version="1.0" encoding="UTF-8"?>
<phpunit
    bootstrap="test/bootstrap.php"
    colors="true"
    strict="true"
    convertErrorsToExceptions="true"
    convertNoticesToExceptions="true"
    convertWarningsToExceptions="true">
    <testsuites>
        <testsuite name="Unit Tests">
            <directory suffix="Test.php">test</directory>
        </testsuite>
    </testsuites>
</phpunit>


Do not start signaling until signaling channel is registered

@marwahvikas
Fixing a connection setup issue on staging (or head of Master).

LGTM.

lgtm


Add support for translating DateTimeOffset.LocalDateTime and DateTimeOffset.UtcDateTime

Undecided: Should I do something else for other providers? SqlServer.Compact

SQL Server Compact only has the datetime data type, so nothing to do there

@ErikEJ sorry for my English. Did I fix correct thing?

I actually meant: DATEIME2 > DATETIME2

@ErikEJ oops thanks

Merged. Thanks!


Runtime error when installing in gVim Portable on Windows 7

Steps to reproduce:

Unzip gVim Portable to any folder (let's assume D:\portable\gVim here)

Clone Vundle to D:\portable\gVim\Data\settings\vimfiles\bundle\Vundle.vim

Add the following to the end of D:\portable\gVim\Data\settings\_vimrc:
filetype off  " Required

" Set the runtime path to include Vundle and initialize
set rtp+=$VIMRUNTIME/../../../Data/settings/vimfiles/bundle/Vundle.vim
call vundle#begin('$VIMRUNTIME/../../../Data/settings/vimfiles/bundle')

" Let Vundle manage Vundle, required
Plugin 'gmarik/Vundle.vim'

" Add all your plugins here (note older versions of Vundle used Bundle instead of Plugin)

" All of your Plugins must be added before the following line
call vundle#end()          " Required
filetype plugin indent on  " Required

Start gVim and execute :PluginInstall

I get the following error dialog:

How did you clone Vundle to your portable drive? I'm trying to do the same thing so that I can run install Vundle on my portable Vim that I have installed on my USB. I also have portable Git. I have been trying forever to figure this out and nothing seems to work. I tried installing the portable MySysGit and still haven't been able to get it to work.

@arturogarza I used Git shell and don't recall doing anything special. For example, to clone to D:\portable I do cd /d/portable followed by git clone some_repo.

@thdoan Hey, I finally ended up being able to install Vundle. I just got the Vundle "autoload" folder and inserted it's files "vundle and vundle.vim" into the .vim folder that I created under my gVimPortable files. It looks like this gvimPortable/Data/settings/.vim/bundle/Vundle.vim/autoload. The folder "vundle and file vundle.vim" are both in the "autoload" folder.
I then opened up gVim and I used the :PluginInstall and it worked. Finally.
I used rtp+=~/.vim/bundle/Vundle.vim and Plugin 'VundleVim/vundle.vim'
It is a little more confusing on Windows to clone from Git as most of the clone code is meant for Linux. We have to manually go in and change stuff.
I tried to install Fugitive using the same process however I keep getting an install error. Any ideas? Do you have Fugitive?

@arturogarza No, I've never tried Fugitive, but after reading its readme it's something I should probably try :).

@thdoan Let me know if you get it working?


separate predict step from evaluation step

🔬 Background
We want to structure our framework in a way that the whole benchmarking pipeline is built upon modular functions. Modular functions can be called individually, which means outside of the benchmarking procedure. That offers the user the option to set up a customized experiment from scratch by using the modular function. To allow a modular function structure the predict step and evaluation step needs to be separated.
🔮 Key changes

introduce new modular function evaluate_forecast().
Create a wrapper method in the experiment class for predicting _make_forecast() and for evaluating _evaluate_model().
Additionally, save the forecast of each experiment in the benchmark output.

📋 Review Checklist

[x]  I have performed a self-review of my own code.
[x]  I have commented my code, added docstrings and data types to function definitions.
[x] No pytests needed.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@33005db). Click here to learn what that means.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main      #37   +/-   ##
=======================================
  Coverage        ?   90.41%
=======================================
  Files           ?       18
  Lines           ?     1325
  Branches        ?        0
=======================================
  Hits            ?     1198
  Misses          ?      127
  Partials        ?        0

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


Disable autocomplete in SavedDataSelector

@mattbell tested it, so gonna merge.

@mattbell tested it, so gonna merge.


[ArPow] Re-enable sourcelink

https://github.com/dotnet/sourcelink/issues/629 is an issue for the current source-build plan, worked around by disabling sourcelink in https://github.com/dotnet/source-build-reference-packages/pull/133.
When fixed and uptaken by Arcade, we can enable sourcelink in 5.0.
(Filing this a little early, to make sure it ends up tracked.)

Sourcelink is enabled in .NET 6 and 7, which is why we discovered issues like https://github.com/dotnet/source-build/issues/2903


Fix typos, grammar and spelling mistakes in docs

I’ve fixed some typos, grammar and spelling mistakes I found in the docs.
What’s the policy regarding rephrasing the docs? I think they are already of great quality and I’m not a native English speaker, but I would sometimes change the sentences slightly to make them more clear (well—in my opinion). Should I just open pull requests and they can be accepted or not, or are these sort of changes not desired anyways?

That's great, @silllli, thank you very much 👍
I'm 100% open if you wish to propose some improvements to the docs! I'm not a native English speaker neither, so there is probably some room for improvement. Feel free to make PR so we can discuss about it 😄

@all-contributors please add @silllli for doc


Nord broken C++ highlighting

Turns out it's an issue with TreeSitter

Hi @jay-patel-cognoai :wave:, thanks for your contribution :+1:
Looks like this is related to #269 which is currently in review and might be fixed in #270.

Resolved in #270.


fix: make sure necessary directories exist

- What I did
fix: make LocalSshKeyUtil constructor (1) create, if they don't already exist, any directories that are expected to exist and (2) set strict directory permissions
Fixes #1138
- How I did it
- How to verify it
Tests pass

@cpswan @cconstab can you build sshnpd from this branch and verify it resolves #1138 please?

Confirmed that the fix works.
In terms of releasing I'd suggest that we don't do a 5.3.1, but rather just move ahead with 5.4.0
We can add to the 5.3.0 release a note saying that the .sshnp directory needs to be created if not present


Re-write README

Been a while and there are major updates available. Include how DMQC gets done.

Completed in #60


Fix stdlib builds

This PR resolves localization targets only if the frontend is being built.
In https://github.com/apple/swift/blob/master/localization/CMakeLists.txt#L9, there's an inherent dependency on the frontend. If another target, like only stdlib, is built, "swift-frontend" doesn't seem to resolve.
Attempt to fix by just moving the include into where "tools" is resolved, which I think also implies "swift-frontend".
Addresses rdar://65993232

Hey @xedin, can you take a look since you added localization/? I'm definitely not certain how reasonable a PR like this is.

I am not so sure “tools” implies frontend, @edymtt could you please take a look?

@xedin the tools includes swift-frontend - its all the executables put into bin effectively.  The localization information only makes sense if the compiler is built, so this seems reasonable.

@swift-ci please test

@swift-ci please test Windows platform

Thanks for fixing this @Ktwu !


Unclear documentation and behavior for sampler in Tensorflow

For the samplers implemented in tensorflow, e.g. tf.nn.fixed_unigram_candidate_sampler. The behavior is not well-defined in the document. For instance, I would expect the labels specified in true_classes will be excluded from the sampling pool, and the sampling will be conducted for each batch. But according to my experiments, neither of above is true.
Consider the following code:
`import tensorflow as tf
labels_matrix = tf.reshape(tf.constant([1, 2, 3, 4], dtype=tf.int64), [-1, 1])
sampled_ids, _, _ = tf.nn.fixed_unigram_candidate_sampler(
true_classes = labels_matrix,
num_true = 1,
num_sampled = 1,
unique = True,
range_max = 5,
distortion = 0.0,
unigrams = range(5)
)
init = tf.initialize_all_variables()
with tf.Session() as sess:
sess.run(init)
print sess.run([sampled_ids])
`
The output can be 3, which actually belongs to the set of true classes. - Also, the output has the dimension [1], which basically means that the sampling is only conducted once, not for each batch.
Can someone help to clarify this?

Is this still current?

Yes, this is probably still a problem in the docs, and it will be useful for the TF writer to keep the issue for reference.

any solutions for this issue? I also find this problem in version1.2


Need to support remote tfstate

Currently only supports tfstate locally which is not always available or best practices.

Also related to #8


UI improvements to membership benefits page

图片大小模仿小程序，特别注意手机观看效果

[ ] Use category images instead of words (BEAUTY, RESTAURANT, OTHER, SWEETS), use old ones for now: https://drive.google.com/drive/folders/1s5HKVKHqSNpcP2roFK69Bsu1usTJ1sV2?usp=sharing

[ ] Reduce border size when viewing on mobile

[ ] On the detail popup, make image scrolling smoother like on the 小程序

Resolved by #37.

Closed by mistake, reopening.

After scanning QR code the page title will show "cssa-frontend" for a very brief moment


Only refresh apt and install git when dark theme is enabled

Would it be possible to only conditionally refresh the apt cache and install git if the dark theme is enabled?
Additionally, iff the two tasks that change the repos changes anything, then attempt updating the apt cache, to ensure it works?

Thanks for the suggestions! I've implemented them in release 1.3.0.


Add AppStream metainfo file

This patch adds an AppStream metainfo file to Spyder, providing metadata to make Spyder visible in the software centers of Linux distributions, like KDE Discover and GNOME Software (the default software center in Ubuntu and Fedora).
At time, we process the .desktop file in Debian/Ubuntu to make Spyder show up, but that's not ideal - with this patch, you gain control over the information that is shown and a screenshot gets displayed as well.
The metainfo XML file can be translated, so you can hook it up into your l10n system if you have one and want that.
More information on AppStream can be found here, and there is a quickstart guide for application maintainers too.
This patch makes the Python3 version of Spyder the only version to be shown in software centers - the other versions are not shown. I think that is a good and future-proof decision, but if you think otherwise, let me know and I will adjust the patch.
You can validate the metainfo file using the command appstreamcli validate spyder3.appdata.xml.
Additionally to adding this metadata, you might also consider the following:

Rename the .desktop files to follow the reverse-DNS scheme as recommended by the specification. - This will kill warnings when AppStream validates the metainfo file.
Ship more icon sizes (or an SVG icon) in the canonical locations in /usr/share/icons/hicolor/<size>/apps to make icon handling easier (and save desktops from resizing stuff a lot).

Cheers,
Matthias

Thanks a lot for your contribution! I see no problem with this.
Would you mind to redo this PR against the 3.1.x branch, so that we can add it to one of our bugfix releases? The master branch is for Spyder 4, which will come in a year or so.

Rename the .desktop files to follow the reverse-DNS scheme

I don't understand this. What should be the name of our desktop file?

Ship more icon sizes

We are in the process of remaking our icon, but our current icon is an SVG one ;-)

Rename the .desktop files to follow the reverse-DNS scheme
I don't understand this. What should be the name of our desktop file?

Something like org.pythonhosted.spyder3.desktop or com.github.spyder3.desktop - those IDs are used because they are unique on the system, which allows apps to be D-Bus activated on Wayland and also makes it easy to map icons to applications on Wayland.

Ship more icon sizes
We are in the process of remaking our icon, but our current icon is an SVG one ;-)

This of course makes this issue way less relevant - since using /usr/share/pixmaps is discouraged, one could place the svg icon in /usr/share/icons/hicolor/scalable/apps directory though.

Would you mind to redo this PR against the 3.1.x branch

Should I do a separate PR for that?

Something like org.pythonhosted.spyder3.desktop or com.github.spyder3.desktop

Ok, we'll think about it.

This of course makes this issue way less relevant - since using /usr/share/pixmaps is discouraged, one could place the svg icon in /usr/share/icons/hicolor/scalable/apps directory though

I think leaving it in pixmaps is fine for now (for old distros).

Should I do a separate PR for that?

Yep, you need to do it against the 3.1.x branch.

PR for the 3.1.x branch is filed as https://github.com/spyder-ide/spyder/pull/4123

Btw, this is how the data looks like in KDE Discover.

Really nice!! Thanks!
I'm closing this one because it's not necessary anymore ;-)


Outdated signal onboarding instruction

Closed via https://github.com/tactilenews/100eyes/pull/1981


[Issue when testing web app] IModelReadRpcInterface.openForRead() is not available

Hi All,
I followed steps on https://github.com/imodeljs/imodeljs-samples/blob/master/README.md to build interactive app devleopment setup.
After I run npm run start:servers, the test web popped up and It worked well for me to log in. However, after I clicked open sample iModel, it shows  "IModelReadRpcInterface.openForRead() is not available"
Could you help me with this issue?
Thanks

@tciuan it sounds like the backend may not have properly started. Can you try to launch the frontend and backend in two separate terminals?  For the backend run "npm run start:backend" and for the frontend try "npm run start:webserver".
This should let you better see the logs for both of the processes.

I ran into this exact same problem yesterday.  As a pure guess, I deleted everything and re-cloned the git repository from scratch.  Somehow that fixed the problem and I can run all three of the UI samples OK now.  Perhaps there was a problem with an npm dependency that was corrected when I started over?

I can confirm this is an issue as well. After deleting basic-viewport-app directory and then re-cloning the issue went away.

A number of others have run into this issue, it appears to be there's an inconsistency in how Rush is installing the dependencies. The workaround for now would be to do a rush purge && rm -rf common/temp && rush install && rush clean && rush build --to simple-viewer-app.  This completely cleans up the local installation and build to start from a clean slate.
We're working on making it easier to consume these samples by separating them into multiple repositories and not using Rush.  Those will be rolling out over the next several weeks so keep an eye out for them.
I'm going to close this issue but feel free to re-open, or create a new one in the imodeljs-samples repo, if the workaround does not work. Thanks!


More advanced cron samples

Is there any guidance on what cron stuff is supported and what is not? My requirement is for instance to have a reminder sent to people two days before the month ends. I'm not a cron expert but I did find some samples, just wondering what is supported and what's not.

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

ID: b5527c3b-a5da-c73d-66c2-20d96003ff7c
Version Independent ID: 42154c7b-b0e0-26db-79bc-3f1673c48e40
Content: Timer trigger for Azure Functions
Content Source: articles/azure-functions/functions-bindings-timer.md
Service: functions
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

@jsiegmund Thanks for the question! We are investigating and will update you shortly.

@jsiegmund can you elaborate a bit more? What exactly are you looking for past the Cron Examples listed in this doc?

@jsiegmund will close this out for now. Please reopen when you are ready to continue.


Rendering a Pore Network with color bar

Hi, Marco.
Thank you for vedo, it is a really nice and convenient lib.
I have been trying to render Pore Networks with vedo inside jupyter notebooks. I have successfully plotted Pore Networks without color bars, but unfortunately I'm struggling to add color bars (to colorize according to pressure or solute concentration for Fluid Flow in the PN).
I feel like I'm missing something very basic here. Would you mind to give me some advice please? I'm providing a MWE below (next comment).
Cheers,
Diego

Hi Diego! thanks for reporting the issue.
When I run the code above I get this

(which btw looks pretty cool) but I'm not sure what you are trying to colorize. When you say "color bars" you refer to colorizing some mesh based on some poit/cell data, or the actual scalarbar Assembly object?
Sorry I should be able to guess all this from the example but i'm feeling a bit lost :)

..oh I see a few weird things there...

being a physicist I'm immediately suspicious of the pore_concentration and throat_concentration spanning so many orders of magnitude... does that really make sense?
you cannot assign a single float as a color input of Sphere and Cylinder, because vedo will try to map this onto the default jet scale in range [0,1]. You should explicitly map the scalar to a [r,g,b] color using colorMap()

maybe this is better?

throat_coordinates = list()
throat_connectivity = throat_data['throat_connection']
for throat_connection in throat_connectivity:
    pore_i = throat_connection[0]
    pore_j = throat_connection[1]
    pore_i_coordinate = pore_data['center'][pore_i]
    pore_j_coordinate = pore_data['center'][pore_j]
    throat_coordinates.append((pore_i_coordinate, pore_j_coordinate))

pores_rendering_list = list()
for idx in range(len(pore_data['center'])):
    pore_rendering = vedo.Sphere(
        pos=pore_data['center'][idx],
        r=pore_data['radius'][idx],
        # c=pore_data['color'][idx] /50
        c=vedo.colorMap(pore_data['pore_pressure'][idx], name='jet', vmin=0, vmax=10)
    )
    pores_rendering_list.append(pore_rendering)

# just create a temporary Points and extract a scalarbar
sb = vedo.Points(pore_data['center']
                 ).cmap('jet', pore_data['pore_pressure']
                        ).addScalarBar(c='k', title='pore\npressure').scalarbar

cylinder_radius_scale_factor = 0.5
throats_rendering_list = list()
for idx, throat_coordinate in enumerate(throat_coordinates):
    cylinder_radius = cylinder_radius_scale_factor * throat_data['radius'][idx]
    throats_rendering = vedo.Cylinder(
        pos=(throat_coordinate[0], throat_coordinate[1]),
        r=cylinder_radius,
        c=vedo.colorMap(cylinder_radius, name='bone', vmin=0, vmax=5),
    )
    throats_rendering_list.append(throats_rendering)

vedo.show(*pores_rendering_list, *throats_rendering_list, sb, axes=1)

Thank you for your quick reply!

Hi Diego! thanks for reporting the issue.
When I run the code above I get this

Ohh! Beautiful! I was expecting something like that! And btw, we can add it to vedo gallery if you want! There is a community of a software called OpenPNM which works on analyses like this one, maybe they would find vedo useful.

(which btw looks pretty cool) but I'm not sure what you are trying to colorize. When you say "color bars" you refer to colorizing some mesh based on some poit/cell data, or the actual scalarbar Assembly object?

Ah, OK, my bad. Let me give you some information. The pore network is composed by two kinds of geometric entities: Pore and Throats. Pores are some prisms depicted as Spheres (yeah, this is misleading because the spheres are representing prisms), and Throats are also prisms, but represented by cylinders. The idea of the visualization is to see the topology and network arrangement. Moreover, I am interested in some properties (as you have noticed), like the pressure of a fluid filling each Pore or solute concentration in such fluid. So this is what I would like to "colorize" and attached an associated "colorbar" as you did (and looks great). Both Pressure and Concentration are indexed for each Pore and Throat, so this is not exactly a "mesh" as we have in Finite Elements and etc. For vtk, IDK if I'm right to guess that for Pore Networks we have the same logic for Point/Cells, right?

Sorry I should be able to guess all this from the example but i'm feeling a bit lost :)

Yeah, I just put the example here all of sudden! I hope the above explanation helps.

..oh I see a few weird things there...
1. being a physicist I'm immediately suspicious of the `pore_concentration` and `throat_concentration` spanning so many orders of magnitude... does that really make sense?

Thank you for your observation! Indeed, this example does not make sense, it is just a toy problem to check if I can generate the visualization in notebooks. Maybe I can provide a more realistic example to add to vedo's gallery (I have to check it, because it is generated by a proprietary software).

2. you cannot assign a single `float` as a color input of Sphere and Cylinder, because `vedo` will try to map this onto the default jet scale in range [0,1]. You should explicitly map the scalar to a [r,g,b] color using `colorMap()`

Hmm.. so this is the proper incantation, I see! Thanks!!

maybe this is better?

A lot better!

IDK why I can't have the same result as you. Maybe this is related to the vtk version I have to use (my dev environment has a lot of version pins for dependencies). I have tried your code and it raises an error for me:
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-18-92600978fbfc> in <module>
----> 1 vedo.show(*pores_rendering_list, *throats_rendering_list, sb, axes=1)

~/Work/miniconda/envs/pore_gui-test/lib/python3.6/site-packages/vedo/plotter.py in show(*actors, **options)
    277                                     bg2=bg2,
    278                                     axes=axes,
--> 279                                     q=q,
    280         )
    281     return _plt_to_return

~/Work/miniconda/envs/pore_gui-test/lib/python3.6/site-packages/vedo/plotter.py in show(self, *actors, **options)
   1846         if settings.notebookBackend:
   1847             if settings.notebookBackend not in ['panel', '2d', 'ipyvtk']:
-> 1848                 return backends.getNotebookBackend(actors2show, zoom, viewup)
   1849         #########################################################################
   1850

~/Work/miniconda/envs/pore_gui-test/lib/python3.6/site-packages/vedo/backends.py in getNotebookBackend(actors2show, zoom, viewup)
    200
    201             #####################################################################Points
--> 202             elif isinstance(ia, Points) or ia.NPoints() == ia.NCells():
    203                 # print('Points', ia.name, ia.N())
    204                 kcols=[]

AttributeError: 'vtkRenderingAnnotationPython.vtkScalarBarActor' object has no attribute 'NPoints'

Any thoughts on this error?

Hi Diego, it's definitely a bug... in any case the jupyter support is very limited (at some point i wondered if i would remove it completely..).
The above picture is from a normal python script with the native vtk rendering window.
From within jupyter you can obtain it with setting either:
from vedo import embedWindow
embedWindow(False)
# or
embedWindow("ipyvtk")

it's very possible that none of these is good enough for jupyter.. I 'll try to fix the issue in all cases..

IDK if I'm right to guess that for Pore Networks we have the same logic for Point/Cells, right?

I dont know much of pore networks but sounds possible :)

we can add it to vedo gallery if you want!

Sure thanks! I hope you can achieve some satisfactory renderings!

Thanks, Marco!
I think we can close this issue since you have worked on it. I will test it in another dev environment I have, and if I find any problem, we can re-open the issue.
Cheers!


xml code completion with schema in Eclipse

OpenUI5 version: 1.34
Steps to reproduce the problem:

Use Eclipse
Manually edit the used schema files and add the missing "schemaLocation" attribute to the xsd:imports e.g. for sap.m.xsd:
<xsd:import namespace="sap.ui.core" schemaLocation="sap.ui.core.xsd"/>
Open the sample file
Goto <f:SimpleForm - make a space and press Ctrl+Space

What is the expected result?
Code completion works. No validation errors (although validation can only work in a limited way)
What happens instead?
Only the default xml templates are displayed because <core:FragmentDefinition /> is not valid. If changed to <mvc:FragmentDefinition /> it works for the code completion / validation but not for the UI5 runtime.
Please correct the schema files and add the schemaLocation so that code completion can work in Eclipse.
Please support  <core:FragmentDefinition /> in the schema file.
Sample Fragment File:
NewDialog.fragment.xml.zip

thanks for reporting this. I forward this to the colleagues responsible. Internal reference is<PHONE_NUMBER>

Hello mkueng1,
thanks for reporting. We have never rolled out official support for the code completion feature. As long as it does not block your development with Eclipse I would suggest to just ignore the validation error or correct the xsd files manually.
Best regards
Benedikt

Thanks for your explanation. Disappointing to hear that - I thought it would be simple for you to add the relative schemaLocation and add the core:FragmentDefinition that's all that would be needed to help a lot.

I hopefully fixed parts of this as well as some other issues in the schema files:

add definitions for sap.ui.core.Fragment and sap.ui.core.FragmentDefinition
allow Fragment in all aggregations (fragments are somehow transparent decorators, they disappear at runtime, only their content remains. So normal type checking doesn't apply to them)
no more duplicate "visible" attribute for Controls that define it on their own (Control since several releases defines a general "visibility", but some other controls redeclare it, e.g. the HTML control. XML Schema doesn't allow to redefine an attribute, so this caused issues when validating the schema itself).
fix transformation of Javascript regular expression literals to XML schema regular expressions. Those expressions are used in some of our property types (e.g. sap.ui.core.CSSSize). Unfortunately,  the feature sets of JS regexp and XSD regexp don't match. For example, unnecessarily escaped characters are not accepted by XSD validators. The generation of XSDs now handles this special case, but we might run into others in future...
add missing attributes for sap.ui.core.mvc.View (e.g. controllerName, resourceBundleName, ...)

What I did not implement is adding the relative schema locations to the imports. There are usages of the schema files where the proposed relative locations wouldn't match -  so I decided against them.
If someone wants to verify it, the latest nightly build already contains the fixes:
https://openui5nightly.hana.ondemand.com/downloads/schemas/sap.ui.core.xsd

Thanks a lot for your changes. I tested the nightly version and the code completion works very fine now. Eclipse cannot handle the substitutionGroups and therefore validatation with the schema files doesn't work - but I'll have to live with that.
It's ok that the schema locations cannot be added - I'm adding them withgrunt after updating.


Updated build.gradle

Changed implementation to compile for react-native dependency
This change is a candidate to solve Issue #13

Version 3 support RN 0.56 with Gradle updated to 4.x and RN 0.57.

I'm closing this. Current and future versions of react native use gradle 4.+, in which compile is deprecated in favor of implementation.


[SNAP-1389] Optimized UTF8String.compareTo

As part of performance testing for SNAP-1389, it was seen that UTF8String.compare could be optimized quite a bit. The upstream code already contains comment "// TODO: compare 8 bytes as unsigned long". This PR completes that TODO as well as adds other changes taking from previous ByteArrayMethods.arrayEquals optimizations.
What changes were proposed in this pull request?

use unsigned long comparisons, followed by unsigned int comparison if possible,
before finishing with unsigned byte comparisons for best performance
use big-endian long/int for comparisons since it requires the lower-index bytes to be in MSB
no alignment attempted since we expect most cases to fail early in first long comparison itself

For a comparison see the code of ByteArrayMethods.arrayEquals. As noted above, alignment is not attempted because we expect most comparisons to fail and within first few bytes so alignment will unnecessarily end up doing byte-by-byte comparisons in most cases. This is also seen in sort micro-benchmarks which see a 20% drop in performance with alignment code. By contrast alignment is very useful for UTF8String.equals since in hashing we expect most cases to be successful match after hashcode has already been matched.
In general the comparison itself is ~2X faster with these changes. Following is the comparison in micro-benchmarks (in snappydata PR).
Random strings
Java HotSpot(TM) 64-Bit Server VM 1.8.0_131-b11 on Linux 4.4.0-21-generic
Intel(R) Core(TM) i7-5600U CPU @ 2.60GHz

Sort num=10M distinct=1K:        Best/Avg Time(ms)    Rate(M/s)   Per Row(ns)   Relative
----------------------------------------------------------------------------------------
Spark                                 4651 / 4674          2.1         465.1       1.0X
Snappy                                3579 / 3599          2.8         357.9       1.3X

Sort num=10M distinct=10M:        Best/Avg Time(ms)    Rate(M/s)   Per Row(ns)   Relative
----------------------------------------------------------------------------------------
Spark                               11225 / 11590          0.9        1122.5       1.0X
Snappy                                9536 / 9852          1.0         953.6       1.2X

The first case where number of distinct values is smaller compared to total number of values sees a bit more gain than the second case where all values are distinct due to large number of bytes required to be compared on an average.
Here the numbers are dominated by cache misses so the gains are smaller. For arrays that are already mostly sorted, the numbers are dominated by comparison performance:
Java HotSpot(TM) 64-Bit Server VM 1.8.0_131-b11 on Linux 4.4.0-21-generic
Intel(R) Core(TM) i7-5600U CPU @ 2.60GHz

Sort(pre-sorted) num=10M distinct=1K:  Best/Avg Time(ms)  Rate(M/s)  Per Row(ns)  Relative
-----------------------------------------------------------------------------------------
Spark                                   1917 / 1935          5.2         191.7       1.0X
Snappy                                   840 /  845         11.9          84.0       2.3X

Sort(pre-sorted) num=10M distinct=10M: Best/Avg Time(ms)  Rate(M/s)  Per Row(ns)  Relative
------------------------------------------------------------------------------------------
Spark                                   1687 / 1753          5.9         168.7       1.0X
Snappy                                   736 /  758         13.6          73.6       2.3X

How was this patch tested?
precheckin, test checking for sorting results of random strings, local tests comparing performance (in snappydata PR)

You mean no aligned access attempted?

Spark 2.1 added alignment comparison to its ByteArrayMethods.arrayEquals. Originally this was added to make it work on archs like Spark that do not support unaligned int/long reads, but as part of that PR, this specific optimization also got added that if both sides have same non-zero offset then consume both byte-wise till the 8 byte boundary, then read as longs. Briefly speaking non-aligned long reads will typically result in 2 long reads hence this is a good improvement. See https://github.com/apache/spark/blob/master/common/unsafe/src/main/java/org/apache/spark/unsafe/array/ByteArrayMethods.java#L52
When merging it to our optimized arrayEquals, had changed it to align at least one side (if required) even if offsets of both sides are not equal, so we get the advantage of aligned reads on at least one side -- see https://github.com/SnappyDataInc/spark/blob/snappy/branch-2.1/common/unsafe/src/main/java/org/apache/spark/unsafe/array/ByteArrayMethods.java#L47 for comparison.
In a general comparison case, we expect the comparisons to mostly fail in first few bytes itself, so its better to not attempt the alignment. This is also seen in micro-benchmarks where a drop of 20-30% is seen when alignment is attempted.
Since arrayEquals is frequently used in hash aggregate/join maps, so we expect the comparison to mostly succeed for those cases once hashcode matches -- for equality filter this does not hold and comparisons can frequently fail but hit there is ~20% while gain for hashmap is more. Still we can re-think if more extensive testing shows different numbers, or we can selectively change for filter comparison in CodegenContext.genEqual

It may depend on data and use case, no? For e.g. dictionary on a column with couple of unique values?

Even with two unique values we expect about half the comparisons to fail while half will succeed in sorting. So attempting alignment will still not provide much advantage. Alignment will mostly help if comparison continues till near the end. For all other cases of larger number of values, the failure cases will be much more and usually in first few bytes.

How did you unit tested the return value of the compare?

Not compare directly but checked result of sorting of 10M random values (including different sizes).


New parameters

Changed parameters to accept new version of XGBoost
Code updated in several notebooks
Added comments

@msalvaris  in the BCI exp, xgboost hist is slower than xgboost. I'm not sure why


Backwards compatability for Django 1.8

The more I think about it, the more I think we should try to use Django's own support rules spelled out here in that green and red table. If we go by that, we should be supporting Django 1.9 and 1.8 right now.

Travis tests are passing so I think we're good on this one.


fix travis build

This would also affect users of older npm versions.
npm can not find gulp-postcss 6.4.1 and so it fails.
See the log of the last Travis build: https://s3.amazonaws.com/archive.travis-ci.org/jobs/252964124/log.txt?X-Amz-Expires=30&X-Amz-Date=20170712T212949Z&X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAJRYRXRSVGNKPKO5A/20170712/us-east-1/s3/aws4_request&X-Amz-SignedHeaders=host&X-Amz-Signature=10152e4ec35485aab4efc26452e96a40a0488b53e332e5f4b26e7356f26df561

Now the PR is correct. The issue is just in the mster branch s we still reference 6.4.1 there instead of 6.4.0 or 7.0.0.


Request to split based on what word is being used for the Gradio app

Can you include the option to split the chunk based on what word is being used? Like say for example, I want the split to start with the word cheese and split from this
I like to eat cheese.

It is like explicitly do chunking?

I thought it could be realized with multi-style generation (to do explicit chunking) but you just pass in same reference audio (for each chunk), lol


UP- 4697 Enhancement (epic) of the Off canvas

https://issues.jasig.org/browse/UP-4697
to test don't forget to switch : <xsl:param name="USE_FLYOUT_MENUS" select="'true'" /> in navigation.xsl
UP-4697 is trying to fix several existing Jiras to give me a branch name...
UP-4622 Add offcanvas stickynav flyout : there is a bug with the optional fly-out menu on iPhone
UP-4670 An horizontal scrollbar appears on narrow display (new mobile nav)
UP-4671 the new mobile nav doesn't have the right background-color (improvement: new LESS variable to handle colors and position in variable.less)
No-Jira Implement a new animated pure CSS Hambuger menu on mobile
No-Jira Use GPU accelerated CSS properties to animate the offcanvas
No-Jira Add support for the offcanvas to ie9, ie10, ie 11
No-Jira Improve uPortal Accessibility, a blind people can navigate, open/close main Navigation, open/close optional subNav (aka optional fly-out menus) and reach directly one portlet in maximize view located into Xth tab of the main Navigation with Voiceover (see https://t.co/1UdCIlfLKY). Improvement on keyboard navigation (tab/enter/space)
No-jira Enhancement of the Touch Area of the sub nav toggle, tabs gripper and tabs delete buttons for Touch devices
No-jira The bootstrap's Font (glyphicons) wasn't update with the last uPortal Bootstrap update : some Bootstrap css classes don't work correctly because glyphicons font wasn't bump to the right version + add font in Wolff2 + security Fix on the fonts.
partly (99%) resolve UP-4695 for breackpoints :765px, 766px 767px, breackpoint 769px, 770px, .... Still a display problem at 768px tue to two differents breackpoints in Bootstrap respondr. but it's usable in 768px now.
Tested on Windows 1O, OSX, Linux on Chrome / FF / Edge /Safari / IE11 / IE10 / IE 9
IOS latest (Iphone 6S / 5 /4) [chrome /safari] / IOS Old (old IPAD 1 not updated) (old chrome / old Safari) / Android
with and without the fly-out enable.

Coverage decreased (-0.01%) to 22.86% when pulling 211547a81dfe9b241e3c38108d86428dd0cbe7c5 on cousquer:UP-4697 into 47247bdb315d3715b5a9f9c840d25ac87af56332 on Jasig:master.

Coverage remained the same at 22.871% when pulling 999aabaeaaac3157aa448bc8907f21a4097fb157 on cousquer:UP-4697 into 47247bdb315d3715b5a9f9c840d25ac87af56332 on Jasig:master.

:+1:

:+1:

Thanks @cousquer!

@drewwills if you need clarification on some points of this pull-request. Feel free to ping me in the comments.
I had to make some choices to stabilize the all possibilities (with / without fly-out menu, mobile / desktop, Nameable,breakpoint's problems on iPad , LESS cascading limitation, etc.)
and, please, correct all my misspellings ... :-)

It would be great to have comments with // that are less comments instead of /* */ that are for css comments.
Is it possible ?
Else a big :+1:  I already adopted it in our dev version ;)

From an email on uportal-user:

This pull request solved our problem. The menu is much better and it seems to be working on all devices I have tested.
I have only tested this for a few hours yesterday. It took some time to implement this into our customized version of uPortal 4.3, but it was working great. I will make a merge request on Monday (on OU's gitlab), so Aaron will be able to take a look at it then.
All version of software on iOS were up to date.
iOS version: 9.3.2
iOS Chrome version: 51.0.2704.104
iOS Firefox version: 4.0
I also noticed this was happening on OSX with the latest version of El Capitan. This issue was happening on Safari, but not the latest version of Chrome. Chrome was working fine on OSX, but not iOS. However, this pull request fixed this issue on as OSX as well. It is working on all devices I have tested.
More testing to be done on Monday.

That's interesting @jgribonvald -- I didn't realize that LESS supports //-style comments (iirc CSS doesn't).

@drewwills @jgribonvald http://lesscss.org/features/#features-overview-feature-comments ;-)

@ChristianMurphy Do-you think that what you have done with #722 (adapted from #722) can be done with #681 on Jasig:rel-4-3-patches ?
I cc @bjagg and @drewwills.

The pull request to master (#723) includes the changes from this pull request (#681), so that is already done.
As far as backporting the changes from this PR (#681) to rel-4-3-patches, it should be doable.
git cherry-pick is what I used to make #722 and #723, it should work for backporting these changes as well.

@ChristianMurphy @drewwills I've cherry-pick #681 into rel-4-3-patches with this PR https://github.com/Jasig/uPortal/pull/734
it was doable :-)


Upgrade the DID registry contract and test deploy upgrade (via proxy) on testnet.

Moving over to blockchain-contracts


Add a read-only state to Switch

Area
UI Components
The problem
Most of the other selection-based components have a read-only state, e.g. Checkbox, Radio Button. Read-only is commonly used to show a non-editable state in an application.
The solution
Add readOnly to Switch.
Alternatives and examples
.
Are you a JPMorgan Chase & Co. employee?

[ ] I am an employee of JPMorgan Chase & Co.

Related

#3941

Business case required to drive priority, if you need this, let us know.


<EMAIL_ADDRESS>: The example is too old and has no effect on 5.0.1

The example is too old and has no effect on 5.0.1

how to use 5.0.1


Can't generate patched IPA

Version used : 0.7
The tool is saying :
Patching finished, the new iPA should now be in your downloads folder!
But in CMD we can see the following :
/Applications/iPAPatcher.app/Contents/Resources/temp/Payload/Parchisi Star.app /Applications/iPAPatcher.app/Contents/Resources/temp/Payload/Parchisi Star.app plutil -convert xml1 -o - /Applications/iPAPatcher.app/Contents/Resources/temp/Payload/Parchisi Star.app/Info.plist|grep -A1 Exec|tail -n1|cut -f2 -d\>|cut -f1 -d\< temp/Payload/Parchisi St mkdir: Star.app: No such file or directory START==== /Applications/iPAPatcher.app/Contents/Resources/libsubstitute.0.dylib /Applications/iPAPatcher.app/Contents/Resources/libsubstitute.dylib /Applications/iPAPatcher.app/Contents/Resources/CydiaSubstrate.framework usage: cp [-R [-H | -L | -P]] [-fi | -n] [-apvXc] source_file target_file        cp [-R [-H | -L | -P]] [-fi | -n] [-apvXc] source_file ... target_directory usage: cp [-R [-H | -L | -P]] [-fi | -n] [-apvXc] source_file target_file        cp [-R [-H | -L | -P]] [-fi | -n] [-apvXc] source_file ... target_directory usage: cp [-R [-H | -L | -P]] [-fi | -n] [-apvXc] source_file target_file        cp [-R [-H | -L | -P]] [-fi | -n] [-apvXc] source_file ... target_directory usage: cp [-R [-H | -L | -P]] [-fi | -n] [-apvXc] source_file target_file        cp [-R [-H | -L | -P]] [-fi | -n] [-apvXc] source_file ... target_directory 2020-10-26 16:41:48.395 iPAPatcher[2692:1366674] array:(     "",     Applications,     "iPAPatcher.app",     Contents,     Resources,     temp,     deb,     Library,     MobileSubstrate,     DynamicLibraries,     "star.dylib" ) error: /Library/Developer/CommandLineTools/usr/bin/install_name_tool: more than one input file specified (Star.app/Dylibs/star.dylib and /Applications/iPAPatcher.app/Contents/Resources/temp/Payload/Parchisi) Usage: /Library/Developer/CommandLineTools/usr/bin/install_name_tool [-change old new] ... [-rpath old new] ... [-add_rpath new] ... [-delete_rpath old] ... [-id name] input 	zip warning: name not matched: St.ipa zip I/O error: No such file or directory zip error: Could not create output file

Should be fixed in release 1.0


Add sample plugin docker config

Description
We need to have an option to spin the sample plugin with docker, this PR adds docker config with option to pass custom variables to it in order to serve the plugin on HCC.

/approve


Allow trusted consumers to change user's password via the API

This picks up from the work done by cyberspice/mgdm

Updated AGAIN to include editing other user account fields, not just the password, so this is now replaced by https://github.com/joindin/joindin-api/pull/169


feat(hide examination timetable link): hide Examination Timetable Link

Hide Examination Timetable Link

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

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


Clean up setup process and simplify DSL implementation

There's a potential to clean up how ROM sets up its environment and all the objects (relations, mappers and commands). This requires to ensure that:

ROM exposes abstract classes (or modules) that can be used in your classes to easily configure them using simple class-level DSLs
It is easy to instantiate objects using classes extended with ROM interfaces
Setup DSL can simply use classes that inherit core DSLs to instantiate objects and store them in the registries

Here's some code to explain this better:
setup = ROM.setup(:memory)

# simple relation class definition extended with rom-sql DSL
class UserRelation < ROM::Relation
  include ROM::SQL::Relation

  one_to_many :posts

  def sorted_index
    order(:id, :name).select(:id, :name, :email)
  end
end

# easy to instantiate a relation object
user_relation = UserRelation.new(setup.repositories[:default].dataset(:users))

# simple mapper class definition
class UserMapper < ROM::Mapper
  attribute :name, from: :user_name

  wrap address: [:street, :zipcode, :city]
end

# easy to instantiate a mapper by passing mapping processor type
user_mapper = UserMapper.build(:transproc) # .new would receive a built header with transformer object

# simple Reader class definition
class UserReader < ROM::Reader
  expose :sorted_index
end

# easy to instantiate a reader
user_reader = Reader.new('users', user_relation, users: user_mapper)

Things to consider:

Should we drop setup DSL completely and use a typical approach with class inheritance and use inherited hook to register all classes so that finalize process knows what to instantiate?
Would it be possible to simply use modules from adapters to extend relation classes (ie include ROM::SQL::Relation - right now setup uses repository object to inject custom interfaces but that might not be needed
Have a strict setup process where we build objects in specific order without delaying it (like it is done now) - this would get rid of the need to have finalization process. It would make it easier to see when things go wrong as it happens immediately close to your code. Probably the only challenge is to figure out how to expose relations to other relations (reminder: right now every relation has access to all other relations).

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

I'm gonna take a stab at this and see how it "feels"...stay tuned


[Feature, Request], dynamic, action, attr, in, upload, component

Existing Component
是
Component Name
upload
Description
Can action attribute be modified when uploading file using upload component? I need to upload files with upload component, but I want to dynamic change the action url , cause there are extra query params I need to append to the url, such as http://test.server/post?tag=xxx, the value of tag is dynamic added.

what the heck is this freaking bot ? translate my question totally wrong... not intelligence enough.

Use a variable: https://jsfiddle.net/rz814e42/


chore: updated to null safety

So the main thing this PR does is convert to null safety.
I ran into a few other issues, which I also fixed while I was at it. They are outlined below.
Changelog:
chore: updated to null safety
fix: ScrollController memory leaks
chore: aligned GridView parameters with Flutter's
chore: converted the plugin to a package instead, and removed the android and ios folders, as they were not being used
chore: version bump to 2.0.0
If anyone needs this PR immediately, they can use the following pubspec:
drag_and_drop_gridview: #^2.0.0
    git:
      url: https://github.com/larssn/drag_and_drop_gridview

Would you like to be a collaborator on this project?

Wont hurt, but I'm sure I'm as busy as you are. So I can't promise I'm always available to review PR's or do other maintenance work. Just so we're clear on that. 😅

Yes. That's why this project needs more contributor. And Thank You for your contribution :smiley:.


Custom IDocumentProvider

I would like to create my own API generation tool.
The reason for this I want to create a custom Map for endpoints for the package MediatR. The mapping works, but I can't generate Swagger docs for it as they are not controllers.
If the IDocumentProvider would be public, I could generate it. As far as I know I only need to implement that. Tell me if I am wrong.
On the other hand I may use ApiDescriptionProvider like this. Can NSwag make use of that?
Is there any other solution to add custom routes to a document? I would like to keep the controllers  while adding my custom generated routes also.

I think you can just copy the interface to your own library and register it in DI the same way and the tool will then pick it up by full type name.


Unkown packet type: 28

I recently got a hold of one Radar_MB_1R2T_V1.5.8 lidar from Aliexpress and I'm getting "Unknown packet type: 28". I haven't looked at the datasheet yet as I just wanted to test out the lidar before. What does this error mean?

Im actually getting a mis of 28 and 93 error codes if that helps.

I don't have a proper datasheet, so this driver is mainly a product of reverse engineering  and a previously created ROS1 driver. If you could find out what that packages mean, you could add these definitions to the driver via a PR. My lidar does not produce this packets or at least it did not while I was testing it.


Critical Region Mutexing

Add mutexes / semaphores to critical areas of kernel

[x] TTY / VGA Printing
[ ] Memory allocation / freeing
[x] Serial IO

Wait, doesn't our heap already have mutual exclusion support due to liballoc? liballoc_lock() and liballoc_unlock() are already implemented. Do we want to add additional mutex to the paging code too?

Yeah I did add a mutex to the liballoc code. We probably should add locking to the paging code, but maybe only the code that actually looks up which pages are free since nothing else (even the free) technically requires a lock. Basically just think about what would happen if the code was preempted at any point in time and whether or not that would have any bad effects.

Updated in fa3c80f


Add config to disable the default File loader

I'm working on a postgres extension using boon, and I'd like to prevent it from loading the default file loader. Right now I'm replacing it with a no-op loader, but it'd be nice to have a feature to enable to just prevent it from being compiled. Something to complement the cfg(not(target_arch = "wasm32")) macro, having the same behavior but not specific to wasm.

fixed in da002ca
Compiler.register_url_loader(...) is removed. now Compiler.use_loader(...) added
FileLoader is now made public.
new struct SchemeUrlLoader added which supports previous style url scheme based loaders
see tests/examples.rs for changes in usages

Nice. So if I want it to have no loaders at all, is there something I can pass to use_loader() so that FileLoader isn't loaded?

you can use SchemeUrlLoader:
let loader = SchemeUrlLoader::new();
compiler.useLoader(Box::new(loader));


Add paused reason to domain states

Up until now if a domain was in a "paused" state,
the reason that was reported in virt-handler log was "Unknown".
This commit adds support for reporting the reason for the pause state.
Fixes #1261
Add support for reporting the reason when a domain is in a paused state.

Signed-off-by: gbenhaim<EMAIL_ADDRESS>Signed-off-by: Yuval Turgeman<EMAIL_ADDRESS>

ok to test

Thanks for your contribution! Looks good to me, but it would be nice to have this for the domain states other than "paused", too. But maybe that could also be done in a follow up PR. Just don't close the issue then please, and add a note to it.

@rmohr I've added a test.
@slintes I've updated the commit message.
Unit/Functional tests failed on unrelated error:
The command "if [[ -n "$(git status --porcelain)" ]] ; then echo "It seems like you need to run `make generate`. Please run it and commit the changes"; git status --porcelain; false; fi" exited with 1.
0.49s$ if diff <(git grep -c '') <(git grep -cI '') | egrep -v -e 'docs/.*\.png|swagger-ui' -e 'vendor/*' | grep '^<'; then echo "Binary files are present in git repostory."; false; fi

Fedora 28 - x86_64 - Updates                    3.6 MB/s |  20 MB     00:05

Error: Failed to synchronize cache for repo 'fedora'

The command '/bin/sh -c dnf -y install procps-ng nmap-ncat     && dnf -y clean all' returned a non-zero code: 1

make[1]: *** [docker] Error 1

make[1]: Leaving directory `/home/jenkins/workspace/kubevirt_kubevirt_standard-check-pr/kubevirt'

Summarizing 2 Failures:

[Fail] Storage Starting a VirtualMachineInstance With VirtualMachineInstance with two PVCs [It] should start vmi multiple times

/root/go/src/kubevirt.io/kubevirt/tests/utils.go:1060

[Fail] Networking should be able to reach [It] the Inbound VirtualMachineInstance with custom MAC address

/root/go/src/kubevirt.io/kubevirt/tests/vmi_networking_test.go:224

@gbenhaim I re-ran unittests and now can see the other problem
Compiling tests...
    compiled tests.test. Please run it and commit the changes
 M pkg/virt-launcher/virtwrap/manager_test.go
The command "if [[ -n "$(git status --porcelain)" ]] ; then echo "It seems like you need to run `make`. Please run it and commit the changes"; git status --porcelain; false; fi" exited with 1.
136.13s$ if [[ $TRAVIS_REPO_SLUG == "kubevirt/kubevirt" ]]; then make goveralls; else make test; fi

Please run make generate and commit.

@cynepco3hahue thanks.


Add service.instance.id for all resource detectors

Is your feature request related to a problem? Please describe
service.instance.id is a recommended attribute, and some resource detectors are already setting the value, but it's still missing from some.
Describe the solution you'd like to see
Set service.instance.id for the missing cloud providers detectors:

[ ] Alibaba
[ ] AWS
[ ] Azure
[ ] GCP

Additional context
I've started the work on this

Closing this, because I couldn't find something better than random for all of them, so instead the SDK itself should be the one setting the value as random, when nothing else is set.


Document how to run Docker containers inside a VM

Running Docker containers inside an Aleph VM should be possible, but how to do so is not documented.
To facilitate deployment, it could be interesting to provide a runtime that runs a Docker-Compose file directly. The user would then only have to take care about specifying the right volumes to mount in the VM according to the containers defined in the compose file.

For performance and reliability reasons, it would be preferable to pull the images upfront in an immutable volume that the VM will mount than pulling them on demand.


9bb2f383-aaf2-40ad-8ede-3f801b1d6aae

this is for automated testing

1dbf16d6-94d5-4865-9aa1-c7556fa9d0bc


Unable to sort by hybrid properties

I have a basic setup
    columns = []
    columns.append(ColumnDT('id'))
    columns.append(ColumnDT('name'))
    columns.append(ColumnDT('address'))
    columns.append(ColumnDT('child.date'))
    columns.append(ColumnDT('created_at'))

When trying to sort on name or address which is a hybrid property, I get
OperationalError: (OperationalError) (1054, "Unknown column 'obj.address' in 'order clause'")

How the hybrid property is defined
    @hybrid_property
    def name(self):
        return utils.join([self.first_name, self.last_name])

    @name.expression
    def name(cls):
        return cls.first_name + ' ' + ls.last_name

Could you possibly create an example here in a branch, so that I can look at it from the same point of view as you?

Example created

@Pegase745, think this can be closed as it should be solved now

@tdamsma I didn't rebase his example onto the latest code evolution in order ton confirm that yet


PRs Passing Build

Hey guys.
If you want to make sure your PRs don't break the build, you can close and re-open them to restart the Travis Check.
It's not that important, but some might find this useful. Cheers.

Thanks for putting this up here for everyone to see.

No prob.


book.json has errors when using 7.x series editors

Category
This is a bug that concerns the 7.x series of Web and Desktop editors.
What is the current behavior?
The book.json file shows as having an unidentified parsing error when using the 7.x series Web editor or Desktop editor (mac)

What is the desired behavior?
That the book.json is validated correctly.
That the book.json file does not error when loaded, which was the behaviour in the 6.5.2 editor.
Additional information
For the GitBook Editor:
- mac0S Latest
- Chrome for Mac latest
- 7.x
For GitBook.com:
- jaredmorgs
- https://www.gitbook.com/book/hubpress/hubpress-knowledgebase/details
NOTE: user @nagim in Slack said the following:
nagim [5:40 PM]
@jaredmorgs Just a tip, I converted only my summary from AsciiDoc to MarkDown, renamed it to SUMMARY.md and it works with that.
aaron [5:42 PM]
@jaredmorgs can you post this in feedback and Cc @SamyPesse samypesse and @Soreine
jaredmorgs [9:15 PM]
Yep, shall do @aaron I"ll also post @nagim's workaround as well. :thumbsup:

I just saw this and @nagim’s workaround. It worked for me. Still can’t preview asciidoc, but at least the book works again. For now …

Please see my comment on issue #372. I don’t think this issue has been resolved.

This should be fixed in the Editor v7.0.10. The Web Editor will be updated during the day.

@nagim Imre is your SUMMARY in AsciiDoc ? I could imagine asciidoctor.js eating up a ton of ram on larger documents.

@AaronO Yes, it is, I thought I would throw it at the 7.0.10 to see if it works.

What is classed as a "big" summary in this context?

--

Sent from Mobile.

~600 lines in my case. Or maybe the problem is the list levels, I have level 6 deep lists. I will test it with only as deep as level 5.

@jaredmorgs @nagim Could you guys share your SUMMARY files ?
See if there's a substantial difference between parsing with asciidoctor.js or a similar SUMMARY in markdown.
We have very little control over asciidoctor.js and since it's ruby code transpiled to JS using opal, I think it's far from optimal, but it would be interesting to test and benchmark both.

OK that is a colossal book. The files I'm dealing with are way smaller.
https://www.gitbook.com/book/hubpress/hubpress-knowledgebase/edit#/edit/master/SUMMARY.adoc?_k=73u4as
--
Sent from Mobile.

@AaronO I guess they have made that more efficient in newer versions (it no longer uses Ruby I think, but @Mogztter could confirm). I will share my summary on a private channel.

Update: the size was the problem. I trimmed it down to about 250-260 lines. It took about a minute to load it, but it loaded eventually. Level 6 deep list items are also not a problem (it was an issue before, so :+1: for solving that). I will experiment further to find out the treshold.

@nagim 600 lines shouldn't be a problem. How can I reproduce this issue ? Are you using the latest version of Asciidoctor.js ?

@mogztter I'm using whatever GitBook Editor 7.0.10 is using :smile:

It took 7 minutes to load it with 400 lines. Around the same time for ~470 lines. 500 lines in 11 minutes seems to be the limit (it fails to load with 520).

Could you share your SUMMARY file ? Did you try to preview the same file in Atom ?
Is the code of GitBook Editor open source ?

Yes, I can preview it in Atom just fine in a couple of seconds. The Editor's code is not public I think.

I think that would suggest that some tooling optimisation on the GitBook
side may need to occur over time.
I know that this is an iterative process, so thanks for looking into it,
team.

--

Sent from Mobile.

So, why is this issue closed, if users are still reporting problems. It certainly hasn’t been resolved for me.


Remove #[doc(hidden)] from struct fields

We've decided to keep the struct fields exposed rather than moving purely to field accessors.

This was done a while ago.


Apply theme to right config when specifying both -c and -t

What current issue(s) does this address?, or what feature is it adding?
When specifying both -c  and -t the theming currently gets applied to the pre-loaded config.
How did you solve this problem?
Apply the theme after the specified config has been loaded instead of before
How did you make sure your solution works?
manual testing
Did you add any tests?
no
Did you check your pycodestyle?
yes - autopep8
Are there any special changes in the code that we should be aware of?
no
Are you making a PR to a feature branch or development rather than master?
development

Coverage remained the same at 49.183% when pulling ca1026485692f02182ab936c441288574e7ed616 on ixje:development into 80e0c6aaaa87112d80c3673cc743309e0c9b771f on CityOfZion:development.

I think I agree with @metachris, would be easier to maintain a single theme.json file

How about a settings.json or configuration.json just to keep it generic and open for any unforeseen configurations we might want to store in there?

those two names are a bit misleading, leading one to think those files would alter how the node functions.  I'd prefer something like preferences.json

+1 for preferences.json

Shall we then also move it completely out of Settings.py and into a
Preferences.py? As the former seems to be dedicated to node functionality.
On Sat, Nov 4, 2017 at 5:52 PM, Chris Hager<EMAIL_ADDRESS>wrote:

+1 for preferences.json
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CityOfZion/neo-python/pull/55#issuecomment-341912055,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGUZAS7_5yION5tnmnR2Tn0d9k0Y23eSks5szJZbgaJpZM4QR9tk
.

Coverage increased (+0.01%) to 49.197% when pulling 3772f094d1777f694436db6bd675afd5067e1e39 on ixje:development into 80e0c6aaaa87112d80c3673cc743309e0c9b771f on CityOfZion:development.

Coverage increased (+0.04%) to 49.228% when pulling f3e3f2abc6cd0034463f3bb2ada5e50376f88a79 on ixje:development into 80e0c6aaaa87112d80c3673cc743309e0c9b771f on CityOfZion:development.

Current behaviour is:

preferences.json is found but has invalid json content -> sys.exit to allow the user to make changes (e.g. when they accidentally removed a " or similar). If they completely borked the file or have thrown it away they'll just have to download it again. I don't think we should be keeping shadow copies.
preferences.json is found but is missing the theme or themes key -> restore defaults and sys.exit. User has to run the command again. Given the way Settings.py is bootstrapped this to me seemed like the easiest and still reasonable approach. Many programs do some recovery and then ask you to try again.


Update EiskaltDC++

addition info in https://github.com/caskroom/homebrew-cask/pull/16683

Thank you for the contribution. It was merged directly as 49efae1fef4e98f7431e9d3dd32ad11814621957 to keep commit history cleaner. Your contribution is still credited to you.


⚠ SceneTime has degraded performance

In 0b9cff9, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1639 ms

Resolved: SceneTime performance has improved in ae335f6 after 9 minutes.


Remove prepared flavor

This removes the prepared flavor from the workflows and configuration as it currently serves no purpose and causes issues like making it hard to know which captions to update. It was initially introduced to archive the output of prepare-av, but that step was removed during the rework f the workflows. This would make #5598 obsolete.
Might still be able to go into OC 16?

I'm in favor of this and it works for new mediapackages. But it might require migration for users with old mediapackages that were created with prepared-av in mind. So if someone with such mediapackages could test this, that would be great :)

Is there a sysadmin meeting this could be brought up at? :)

Adding "official" approval, as I am in favor of this. @KatrinIhler Did anyone speak up against this?

Not so far, no. Brought it up at the SysAdmin meeting and got no comments.

This is all configuration so anybody not liking this can change it back.


Upgrade deps

Upgrade some more node dependencies.
QA: To test, standard regression checks + CI functionality

Regression testing is being carried out as usual in the release process. A couple of issues have been uncovered:
https://maidsafe.atlassian.net/wiki/spaces/TEST/pages/279838876/v0.11.2

QA verified - regression and CI are passing


AI Generative Sample Issue

Section link1

Reasons:
client.tracking_url has a azureml scheme, which is not unsupported.
@rohit-ganguly , @lmazuel  , @luigiw , @needuv , @paulshealy1 , @singankit  for notification.

Hi @diondrapeck, clearing up some older issues with samples/docs and saw that you had the last commit to this sample. Do you mind taking a look?

Closing this issue as this prefix is expected as part of AzureML.

@paulshealy1  Yet by running this sample ai_samples_evaluate.py , an exception raised shows that  azureml is not included in the supported url scheme , could you help check this error？


Add support to run subtasks seperately

I am trying to reduce the build time of my app, so I came up with the idea to split up the BartyCrouch build phase into 3 separate build phases:

one build phase to run update.code with my .swift files as input files, so it will only run when I made changes in my code.
one build phase to run update.interfaces with my .storyboard files as input files, so it will only run when I made changes in the UI.
and finally a build phase to run lint and update.normalize with the .strings files as input files.

Unfortunately this currently doesn't seem to work since the command line tool does not offer any option to run a separate update subtask. Am I missing something?
I think one of these two enhancements would be a very good addition:

add a 'subtask' parameter to the update command, eg. bartychrouch update code
or
add a parameter to specify a different configuration file path, so you could configure your subtasks in separate configuration files

@ndreisg Thank you very much for this suggestion. Sounds reasonable and I think adding an option to specify a path for the configuration file is the way to go here to keep things simple in the code base but still allowing calling bartycrouch in a flexible way.
Please note that I'm currently working on another project in my free time so won't have much time to invest here. But before I start my next iOS project, I plan to make a new major BartyCrouch release, where I would like to improve on several aspects, I think it makes sense to solve this then as well. But that will still take some time, probably in the summer this year. If you need this feature earlier, feel free to post a PR – I'd happily review it and make a minor release! :)

I think the most generic and easiest solution to fix your problem would be if we add an option to BartyCrouch to specify a path for the BartyCrouch config file. Then, you could create multiple config files, where in each only the steps you want to run are turned on. Then you can run them whenever you need to.

Yes that would indeed solve my problem.
I'm sorry I didn't have time yet to look into it, maybe I will find some time on the weekend to make a PR :)

Sounds good! Looking forward to it. 👍

@ndreisg I've decided to implement this myself as it's not a big feature.
The fix for this was just released in 4.2.0 and should be available via Homebrew soon. 🎉

Thank you very much for adding this, I will test it soon. Sorry I didn't find the time to make a PR :/


interfaces aren't exported in 1.7.0-dev.20151006

On master you have exported interfaces, e.g.: https://github.com/Microsoft/TypeScript/blob/master/src/compiler/types.ts
For example, I need to use ClassLikeDeclaration in a project I'm helping maintain, but that isn't exported in 1.5.3, 1.6.2 or 1.7.0-dev.20151006.
Can you release an @next style thing like @master? - Or something?
Thanks!

Can you release an @nextstyle thing like@master? - Or something?
What do you mean by this? We already have typescript@next for our nightly builds on npm.

Okay, well then I'm not sure what's going on. In types.ts ClassLikeDeclaration is exported.
In typescript.d.ts it is not exported. Can we change that?

Not sure if this is what you mean, but in an ambient declaration all top level declarations are exported implicitly. no need for the export modifier..
so ClassLikeDeclaration should still be exported in:
declare namespace ts {
    interface ClassLikeDeclaration extends Declaration {
    }
}

Hmm, then why is this failing with an Module "typescript" has no exported member "ClassLikeDeclaration"?:
/// <reference path="node_modules/typescript/lib/typescript.d.ts"/>

import * as ts from 'typescript';

function foo(node: ts.ClassLikeDeclaration) {
    console.log(node.heritageClauses);
}

You could just remove the import * as ts from 'typescript'; line, and use the ts namespace directly.
If you want to use it as a module with the import syntax then tsc has to correctly map the "typescript" module name to the typescript.d.ts file. It's possible it thinks "typescript" is something else, depending on what the module loading rules are. You could force it by putting this somwhere:
/// <reference path="node_modules/typescript/lib/typescript.d.ts"/>

declare module "typescript" {
    export = ts;
}

are you writing a node app or is this in the browser?

if you are in node, what you are doing is correct.. you do not even need the /// <reference>
so this works for me:

npm install typescript@next

C:\test\sb>npm install typescript@next
<EMAIL_ADDRESS>node_modules\typescript

C:\test\sb>node_modules\.bin\tsc --version
message TS6029: Version 1.7.0-dev.20151006

C:\test\sb>type test.ts
import * as ts from 'typescript';

function foo(node: ts.ClassLikeDeclaration) {
    console.log(node.heritageClauses);
}

foo(undefined);

C:\test\sb>node_modules\.bin\tsc test.ts --m commonjs

C:\test\sb>test.js

C:\test\sb\test.js:2
    console.log(node.heritageClauses);
                    ^

TypeError: Cannot read property 'heritageClauses' of undefined
    at foo (C:\test\sb\test.js:2:21)
    at Object.<anonymous> (C:\test\sb\test.js:4:1)
    at Module._compile (module.js:434:26)
    at Object.Module._extensions..js (module.js:452:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Function.Module.runMain (module.js:475:10)
    at startup (node.js:117:18)
    at node.js:951:3

Please reopen if this is still an issue.


Issue with Frigate script

Please verify that you have read and understood the guidelines.
yes
A clear and concise description of the issue.
During deployment i get the following error
Loading the ONNX file...
Adding yolo_layer plugins.
Adding a concatenated output as "detections".
Naming the input tensort as "input".
Building the TensorRT engine.  This would take a while...
(Use "--verbose" or "-v" to enable verbose logging.)
Available tensorrt models:
ls: cannot access '*.trt': No such file or directory

[ERROR] in line 192: exit code 0: while executing command $STD bash /opt/frigate/docker/tensorrt/detector/rootfs/etc/s6-overlay/s6-rc.d/trt-model-prepare/run
root@z240:~#

I can't log in because I don't know the password.
The system i'm using is a HP Z240 with a Nvidia P2000 gpu.
What settings are you currently utilizing?
Default Settings
Which Linux distribution are you employing?
Debian 11
If relevant, including screenshots or a code block can be helpful in clarifying the issue.

Please provide detailed steps to reproduce the issue.
Execute bash -c "$(wget -qLO - https://github.com/remz1337/Proxmox/raw/remz/ct/frigate.sh)"

from the host shell, simply run pct enter <lxc id> to log in. I'm suspecting a TensorRT issue due to mismatch in CUDA version. There was another issue related to this, where older GPU (such as yours) did not support the default CUDA version that comes with the latest drivers. Look in the logs if you see any error message about CUDA  or TensorRT.
Otherwise, log in and try to see if the trt files are somewhere else. You can also run the trt commands manually from the script and see where it blocks

Cool! didn't know that command. Inside the LXC i have exactly the same output for nvidia-smi as on host. On a different host, cuda 12.4 is working, but with a RTX 3060 card.
There haven't been any *.trt files generated find / -name *trt doens't find anything.
I haven been looking at how to run the command manually. https://github.com/remz1337/Proxmox/blob/remz/install/frigate-install.sh is the file i guess is the one I need to check, but i'm stuck. Honestly wouldn't know what to run.

Yes, I'm pretty sure that's the issue (older GPU). My setup uses Turing architecture Nvidia GPU (T600) and RTX 3060 uses Ampere architecture (more recent), but P6000 has Pascal architecture (older). The right fix would be to rework the script and allow a different CUDA version, but that's a lot of effort I'm not willing to invest. Plus, I could not even test it. But if you want to work on this, I'd be happy to accept a pull request.
As I have suggested previously here (issue https://github.com/remz1337/Proxmox/issues/8), there's potential workaround if you can find a recent driver version (>500) that comes with CUDA 11 instead of 12.
Now, if you want to debug manually, first run the script to create a base debian 11 lxc, then enter the LXC and copy paste each command from the frigate-install script one by one. For the debian LXC, run this from the host bash -c "$(wget -qLO - https://github.com/remz1337/Proxmox/raw/remz/ct/debian.sh)", then you need to go into advanced and select debian 11 and also enable GPU passthrough.


status of current Netopeer2 Implementation in terms of compliance to RFC 6241

is there any document or reference which talks about the implementation status in terms of
RFC 6241 compliance for example replace command and Lock command is not working

Hi,
the whole RPC should be supported, if some operations are not working, please create an issue (the ones you mentioned are quite basic and believed to work properly for quite some time now). But, looking at all the capabilities, URL and Confirmed Commit are currently not supported, they may be later.
Regards,
Michal

Thanks Michal


Design suggestions for JTube.

Suggestions can be updated. Check the date. 05/07/22

The start of the application.
*1. Classic design. Search bar up, down suggestions in list mode.
*2. Search bar up, down suggestions in grid mode.
*3. Search bar up, down suggestions in coverflow mode. Then 1 or 2 buttons can be added for options.
*4. Keep the search bar static in any window.

Shortcuts for phones with keyboard. Each option in the application should be assigned to a key on the phone. 12 phone keys for 12 shortcuts.

Floating windows for options or menu. Currently in Jtube 2.0.5, when accessing the menu or options opens another screen.
*Instead of switching screens, a floating window should appear above the interface.

Improve interface visualization. Change the colors to gold (In dark mode) for: Video name, channel, playlist, views, date, time.
*Or select the color you want.

More information. This was the design of the yotube application in java.
*1. Video - Java app.
-https://www.youtube.com/watch?v=h02LcEJ8Q6o
-https://www.youtube.com/watch?v=o0ljWtG1xOM
*2. Video - Web app.
-https://www.youtube.com/watch?v=5oMzDAsYjt8
*3. Images of the app.
-https://www.google.com/search?q=youtube+mobile+j2me&tbm=isch&hl=en&tbs=rimg%3ACeYq-7rXBgMQYeKuwV05vCSs8AEAsgIMCgIIABAAOgQIABAA
-https://yandex.com/images/search?url=https%3A%2F%2Favatars.mds.yandex.net%2Fget-images-cbir%2F2493848%2FwKcf5xcCMk1ijEYMLN-f0A8306%2Forig&rpt=imageview&cbir_id=2493848%2FwKcf5xcCMk1ijEYMLN-f0A8306&cbir_page=similar

@shinovon


[Question]

[REQUIRED] Please fill in the following fields:

Unity editor version: 2021.1.18f1
Firebase Unity SDK version: 8.1.0
Source you installed the SDK: .unitypackage (.unitypackage or Unity Package Manager)
Problematic Firebase Component: Storage (Auth, Database, etc.)
Other Firebase Components in use: Auth,Database,Firestore (Auth, Database, etc.)
Additional SDKs you are using: None (Facebook, AdMob, etc.)
Platform you are using the Unity editor on: Windows (Mac, Windows, or Linux)
Platform you are targeting: Android (iOS, Android, and/or desktop)
Scripting Runtime: Mono (Mono, and/or IL2CPP)

[REQUIRED] Please describe the question here:
Why does not finish?
await storageReference.Child(firebasePath).GetFileAsync(downloadPath,null,cancellationTokenSource.Token).ConfigureAwait(false);

How can I wait for finishing "GetFileAsync"?
Code:
using System;
using System.Collections;
using System.Collections.Generic;
using System.Threading;
using System.Threading.Tasks;
using UnityEngine;
using Firebase.Storage;

public class StorageManager
{
    private static StorageReference storageReference;
    private static CancellationTokenSource cancellationTokenSource;

    public static void init()
    {
        storageReference = FirebaseStorage.DefaultInstance.GetReferenceFromUrl("gs://********.appspot.com");
        cancellationTokenSource = new CancellationTokenSource();
    }

    public static void getData(string firebasePath,string downloadPath)
    {
        DownloadFile(firebasePath,Uri.UriSchemeFile + "://" + downloadPath).Wait();
    }

    private static async Task DownloadFile(string firebasePath,string downloadPath)
    {
        Debug.Log(String.Format("Download {0} to {1}",firebasePath,downloadPath));
        await storageReference.Child(firebasePath).GetFileAsync(downloadPath,null,cancellationTokenSource.Token).ConfigureAwait(false);
        Debug.Log(String.Format("FinishGetFileAsync {0}",firebasePath));
    }
}

Log:
unity           | Download (firebasePath) to (downloadPath)
NetworkRequest  | No App Check token for request.

Hi @soup-soup605,
Usually, this makes use of the ContinueWIthOnMainThread method where you can check the result of your GetFileAsync function as seen here.
Could you confirm if the code in the documentation applies to your use case? If it doesn't, please provide a minimal, reproducible example of your project along with the complete steps to replicate the issue.


allow variable sequence lengths for stateful rnn

Stateful rnn requires input_batch_shape = (batch_size, timesteps, nb_features), where batch_size is required for the saving the state. however I am not sure why timesteps is required. it would be beneficial to allow timesteps=None, just as this is possible for stateless rnn having input_shape = (None, nb_features), allowing variable sequence lengths among batches (at least in theano)
usecases:

train batches with short seq. length for truncated BPTT vs. validation batches with full seq. length
train and predict with different seq. lengths (in stateful prediction we may want seq len=1 timestep)

batch_input_shape=(100, None, 10)

unfortunately this does not work for stateful rnn in theano

Works for me. Do you have a script?

>>> model=Sequential()
>>> model.add(LSTM(10,batch_input_shape=(100,None,10), stateful=True))
>>> model.compile(loss='mse', optimizer='sgd')
>>> x=np.zeros((100,7,10))
>>> y=np.zeros((100,10))
>>> model.fit(x,y, batch_size=100)
Epoch 1/10
100/100 [==============================] - 0s - loss: 0.0000e+00
Epoch 2/10
100/100 [==============================] - 0s - loss: 0.0000e+00
Epoch 3/10
100/100 [==============================] - 0s - loss: 0.0000e+00
Epoch 4/10
100/100 [==============================] - 0s - loss: 0.0000e+00
Epoch 5/10
100/100 [==============================] - 0s - loss: 0.0000e+00
Epoch 6/10
100/100 [==============================] - 0s - loss: 0.0000e+00
Epoch 7/10
100/100 [==============================] - 0s - loss: 0.0000e+00
Epoch 8/10
100/100 [==============================] - 0s - loss: 0.0000e+00
Epoch 9/10
100/100 [==============================] - 0s - loss: 0.0000e+00
Epoch 10/10
100/100 [==============================] - 0s - loss: 0.0000e+00
<keras.callbacks.History object at 0x000000013FC93940>
>>>

You are right. I was probably hitting another issue and blaming the wrong thing. I can no longer recreate the issue. great news. thanks!


Update nextflow_schema.json for non-integer primer_set_version

[Issue #288] Changed 'type' for primer_set_version to 'number' in order to accept non-integer primer set versions.
Bumped version number to 2.4.1

PR checklist

[ X] This comment contains a description of changes (with reason).
[ X] If you've fixed a bug or added code that should be tested, add tests!

[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] If necessary, also make a PR on the nf-core/viralrecon branch on the nf-core/test-datasets repository.

[ X] Make sure your code lints (nf-core lint).
[ X] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

Thanks @stevin-wilson ! Need to update the CHANGELOG too.


Enforce Content Types

This PR ensures that the correct content type is always returned when the files are requested to be downloaded, no matter how they are requested.
In my case, my endpoints are accessible externally, being routed through Cloudflare -> Nginx Proxy -> internal server. Nginx is configured to return the correct files when I access a standard URL, e.g. https://iptv.domain.com/samsung/playlist or https://iptv.domain.com/samsung/epg. When accessing the files this way, the browser is unaware of the file content type.
After these changes, proxy access with custom URLs works, and internal access still works as it did previously.

thank you :)


(Dart.next) Semantics of throw <not-a-Error|Exception>

Dart allows anything to be thrown, but the style guide recommends just throwing something that inherits from Error or Exception. Can we either:

Make throwing a string literal automatically wrap in an Exception or (Error) class
Disallow throwing anything that isn't an Exception (or Error)

/cc @munificent

I think there's an argument that the language should allow things the style guide discourages. The style guide is sort of implicitly scoped to "real" code that you care enough about to check into source control, share with others, etc.
If you're hacking some local throwaway script, by all means, throw strings, use hacky names for your variables, make everything mutable and public.

I agree with that. How about option 1 then, where every literal is implicitly Error?

http://dart-lang.github.io/linter/lints/only_throw_errors.html

Let's just say that requiring thrown exception classes to extend Exception is not universally considered a reasonable requirement.
IOW: I personally disagree completely with the style guide on that, it's a needless restriction. The non-errors that an API throws is part of the API contract. It can be any class that makes sense.
The Exception class is a marker interface. Marker interfaces are not necessarily a problem, depending on how they are used, but in this case it adds no value at all. There is nothing common to exception objects, it's a role that objects play, but not something that you ever need to check.

There is not enough benefit to break existing programs. For clean programs it's probably a good recommendations, but Dart works fine without requiring the rule, so there is no need to enforce it.
Throwing is used for two purposes: bugs and exceptional cases. If the bug is something that can happen on API boundaries or for other users (like passing in "-1" to an array-index), then you should probably throw an Error. They have the advantage that they automatically capture the stacktrace (and they give your API a more professional feel). If it's just a throw in your code to make sure you didn't forget an enum value (which should just be unreachable code), then it shouldn't really matter what you throw. (This is especially true, if you are not in a widely shared library).
If the throw is used for an exceptional case, then the most important thing is, that the type is unique to handle. Otherwise the on X catch wouldn't work. In general, it's a good idea to have a specific class for that, and (just in case it's not handled by the caller), making it an Exception class makes it clear that this is not an error. As Lasse pointed out: if you are catching within your own library, then you could just forego the additional class. It's sometimes faster, but should happen rarely.
Option 1 (wrapping everything into an Error) would force non-Exception/Error classes to become Errors which is not necessarily the intent.


Monkeypatch res.writeHead instead of res.end

The middleware would fail if res.write() or res.writeHead() are called, with the error:
Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client
This change moves the logic to the last valid moment to make header changes.
However, note that the writeHead method takes an optional headers argument,
that this code will not fix up. If you want to address that edge case, you could add
apply headers argument to the request first, then don't send the headers through
to writeHead().

I also tweaked the regex so that:

Both SameSite and None should be matched case-insensitively.
(The cookies package writes it as all lowercase and that is
valid according to the relevant standards.)
Whitespace is allowed between all tokens.
Cookie headers are not allowed to end with a semicolon (RFC6265
section 4.1.1), the new pattern avoids that.

Thanks. I will have a look.

I actually found a couple more issues in my testing, I'll add to this same PR if you don't mind?

OK it's ready when you are. Thank you for considering this PR.

We are using express session middleware also see a lot of this same error, this patch seems fix the problem.


feat: ffem

PR for #36
Waiting for #43 to be merged before rebasing and running the script in this branch

:+1:


The language expression property 'outputs' doesn't exist

Is this Bicep Module no longer providing outputs?
I reviece the following error
The template output
'result' is not valid: The language expression property 'outputs' doesn't exist, available properties are 'provisioningState, azPowerShellVersion, scriptContent, arguments, retentionInterval, timeout, containerSettings, status, cleanupPreference
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 2a6fccd2-266a-7cfb-f77e-4b219131192a
Version Independent ID: 6512ebbf-772f-1ae5-aa9a-cd2e4b29793d
Content: Use deployment scripts in Bicep - Azure Resource Manager
Content Source: articles/azure-resource-manager/bicep/deployment-script-bicep.md
Service: azure-resource-manager
Sub-service: bicep
GitHub Login: @mumian
Microsoft Alias: jgao

@jlagarda Thanks for your comment. We will review the issue and get back to you shortly.

@mumian  Can you please provide your insights on this?

@jlagarda - sorry for the late reply.  I assume you got the error message from vscode.  Please make sure you have the latest VSCode Bicep extension.  See https://learn.microsoft.com/en-us/azure/azure-resource-manager/bicep/install#vs-code-and-bicep-extension. If you still have the error message with the latest extension, please provide your code. I can take a look at it.

@jlagarda - I haven't heard from you.  If you still have the same issue, you can ping me back to reopen the ticket.
#please-close


ThirdPartyGradleModuleMetadataSmokeTest resources use a hardcoded version of Kotlin but should use AbstractSmokeTest.TestedVersions.kotlin instead

This is a test maintenance issue.
Current behavior
Upgrading the embedded Kotlin to 1.3.70 there were 26 test resource files that required updates. Half of the diff is from updating the resources for the GMM smoke test, starting at https://github.com/gradle/gradle/pull/12424/files#diff-680780de644898975da0785571e8e871
Expected behavior
Updating AbstractSmokeTest.TestedVersions.kotlin should be enough.
Fixing this might be a good oportunity to @Unroll the test on the supported kotlin versions.

Thanks. I missed that when creating the tests. Thanks for updating this time.
I don't think an unroll is necessary here as changes in Gradle are unlikely to change anything with older Kotlin versions only.
This test is here to observe/track/react to changes done on the Kotlin side.

That test is still on Kotlin 1.4. This issue is still relevant

This issue is no longer relevant.
See:

https://github.com/gradle/gradle/pull/26800


Creating WindowStyle from another WindowStyle causes missing stageBackground in the new WindowStyle, possible typo

Issue details
WindowStyle created from another WindowStyle with a non-null stageBackground will have a null stageBackground. From the source code, it seems like there may be a typo.
Class: com.badlogic.gdx.scenes.scene2d.ui.Window.WindowStyle

public WindowStyle (WindowStyle style) {
	background = style.background;
	titleFont = style.titleFont;
	if (style.titleFontColor != null) titleFontColor = new Color(style.titleFontColor);
	background = style.background; // Typo here, should be stageBackground = style.stageBackground?
}

Reproduction steps/code
WindowStyle windowStyle = new WindowStyle();
windowStyle.stageBackground = new TextureRegionDrawable(); // Set stageBackground
WindowStyle windowStyleCopy = new WindowStyle(windowStyle); // Call the constructor above
System.out.println(windowStyleCopy.stageBackground); // -> null

Version of libGDX and/or relevant dependencies
1.11.0
Stacktrace
NA
Please select the affected platforms

[x] Android
[x] iOS
[x] HTML/GWT
[x] Windows
[x] Linux
[x] macOS

Definitely looks like a copy-paste mistake, good catch. @NathanSweet is there anything that we should be careful about when fixing this? scene2d.ui is your area of expertise.

Why is background being assigned twice, though?


Angular NPM WebMWriter not defined

I'm using angular 12 and have tried many different ways to include the webm-writer.
This error occurs when I do ccapture.capture():
WebMWriter is not defined
at new CCWebMEncoder (CCapture.js:283)
at new CCapture (CCapture.js:633)
at visualizer.service.ts:149
at Generator.next ()
at asyncGeneratorStep (asyncToGenerator.js:3)
at _next (asyncToGenerator.js:25)
at asyncToGenerator.js:32
at new ZoneAwarePromise (zone.js:1387)
at asyncToGenerator.js:21
at VisualizerService.exportVisuals (visualizer.service.ts:147)
let ccapturer = new CCapture({
verbose: true,
display: true,
framerate: framerate,
motionBlurFrames: 0,
quality: 100,
format: "webm",
timeLimit: this.audioPlayer.soundDuration?.value,
frameLimit: 0,
autoSaveTime: 0,
});
"scripts": [
"node_modules/ccapture.js/build/CCapture.min.js",
"node_modules/ccapture.js/src/webm-writer-0.2.0.js"
],
using package github:aceslowman/ccapture.js#npm-fix

same with astro/svelte

Did anyone find a solution?


[FEATURE] Make TwitFix bot works with webhooks instead of normal messages

When someone sends a Tweet link, instead of the bot replying to the message with the fxtwitter link, it could simply delete the normal tweet link and send the fxtwitter link using a webhook with the name and avatar of the person who sent the normal tweet link, so it would be a better solution and it would look like the person sent the fxtwitter link

Possibly a good idea, I did intend to do it when originally writing the bot but it's kind of a can of works and the current functionality was 'good enough' to ship, so I got it working good enough to put out the door before I got really busy with IRL stuff. When I get some time again I'll take a look, though


[WIP] Rework Scheduler

This PR returns the EventLoopScheduler to the v1 style construction.
This also adds AsyncSchedulerInterface for operators and observables that require async schedulers.
The README.md still needs to be updated.

Coverage remained the same at 100.0% when pulling 5f5b4f9d565cb486f30ee2bf4dd841b7cbf600f0 on mbonneau:2.x-remove-interop into 7c058d53f6e144ba0851fbff79df5c9643dbfb9c on ReactiveX:2.x.

I'm just curious what happened to Loop and async-interop. I read the discussions in async-interop/event-loop repo and it seems the API is likely to change and it'll take some for the first stable release. Is this the reason we're going back to React\EventLoop?

@martinsik This change is being made because version 2 of RxPHP is taking too long to release with the dependency on the interop loop as the only blocking issue.
When first developing version 2 of RxPHP, it appeared that work on the Async-Interop Loop library had settled a bit and we had hoped that a stable version was close (see: https://github.com/async-interop/event-loop/issues/119). We chose to try and push things forward with adoption of the interop loop standard. Since then, there have been more issues raised with the interop loop and consensus is taking a bit longer than we would like for inclusion into this next release of RxPHP.
Inclusion of the interop loop would have allowed users to remove a small bit of boiler plate code. With this PR, construction of the actual scheduler will be the same as it was in v1 and then you can set the default scheduler in the Scheduler static. RxPHP is loop agnostic (react is a dev-dependency for testing).

Coverage remained the same at 100.0% when pulling 48fefb55950abcc48dbc7fed3936d861420e6ff5 on mbonneau:2.x-remove-interop into 7c058d53f6e144ba0851fbff79df5c9643dbfb9c on ReactiveX:2.x.

I'm going to merge this because the async interop loop project is all but dead.  If and when there is a loop standard we can introduce new interop loop scheduler under 2.x and then bump the version to v3 or what ever the next major is, when we're reading to make it the default.


Test Issue created at Sun, 16 Jan 2022 02:31:19 GMT

Test issue body  Sun, 16 Jan 2022 02:31:19 GMT

Comment on issue at Sun, 16 Jan 2022 02:31:21 GMT


Fix standardrb/standard-ruby-action version in example

Type of PR (feature, enhancement, bug fix, etc.)
Bug fix
Description
I tried using the example workflow, and it fails with:
Unable to resolve action<EMAIL_ADDRESS>unable to find version `v0.1.0`

It was using 0.1.0 which doesn't seem to exist. This changes it to 0.0.5, which works, but I'm not sure it is what should be in the docs.
Please include a summary of the change and which issue is fixed.
Fixes # (issue)
Why should this be added
Easier to people to start using it.
Checklist

[ ] My code follows the style guidelines of this project
[ ] Actions are passing

Seems legit!


⚠ HDArea has degraded performance

In 4f0dd17, HDArea ($HDAREA) experienced degraded performance:

HTTP code: 200
Response time: 1109 ms

Resolved: HDArea performance has improved in 51cdfda after 14 minutes.


./shared.js not found

Thanks for the recent update!!  Sadly something broke in the build and shared.js is not being included in node_modules/remark-mermaidjs:

Thanks for the report! I know what causes this, but I'm currently not on my laptop. I'll fix in in an hour or so.

Fixed in version 4.1.1.


Add Linux Binaries and Intermediate files to FilterPlugin.ini

The Epic Marketplace team requested we do this in order to have Linux binaries correctly published in the Marketplace.

Thanks for the pull request @kring!

:heavy_check_mark: Signed CLA found.

Reviewers, don't forget to make sure that:

[ ] Cesium for Unreal Samples works.


wso2is-km-5.6.0.zip download link gives a zip with  version with RC4

the link to download  wso2is-km-5.6.0.zip on the page
https://github.com/wso2/product-apim/releases/tag/v2.5.0

Fixed the file : https://github.com/wso2/product-apim/releases/download/v2.5.0/wso2is-km-5.6.0.zip