Updates to OAuth2Request regarding alternative values for the access token's parameter name

…f the optional argument 'accessTokenParameterName';
Updated OAuth2Request.GetAuthorizationHeader() to accept an optional argument 'accessTokenParameterName' to create auth headers from non-standard access token value names.

Hi @MrXemiu, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

can you check portable-bait-and-switch branch.
I think this issue was fixed there a while ago.
This branch is working branch currently and will be mereged to master soon.


ModelAdmin, usability issue with search filters applied

Since ModelAdmin was revised to have the search filter form appear by sliding down from the top, there doesn't seem to be any obvious way to indicate to the user that a filter is currently active.  After clicking "Apply Filter", the ModelAdmin interface refreshes (via GET) and once again the search filter form is collapsed.  I'd imagine this may be confusing for some users, who may switch to other tabs, attend to other tasks etc., then come back to the CMS without realising there is a filter currently active.
I think it makes sense for the search filter form to collapse after applying the filter and showing the filtered results, but perhaps a class could be applied to the GridField or the EditForm, e.g. filter-applied, that would allow a visual indicator that a filter is currently active on the results?  Or perhaps, a class applied to the filters-button in the top right of ModelAdmin to show that a filter has been applied?  Thoughts?

@clarkepaul how do you feel about a "filter active" indicator on the model admin sections?

Yeah that is a huge UX failure. I would replicate the pattern from pages as much as possible. The message could be integrated into the same position as where the filter would have previously been, allowing the user to amend or clear filter.
Having the search fullwidth at the top was actually a bit of a temporary thing, the direction for the UX of the search and filter panel will look more like this https://projects.invisionapp.com/boards/FAX68PE52Y8W/ but that doesn't really concern the issue mentioned above.
There still needs to be a indication the filter has been applied, as per pages area or as you have mention @colintucker.

Given that this "bug" involves designing something that doesn't exist, I'm re-marking this as an enhancement.

We done some rethinking around how the search mechanics worked, in regards to user experience. Using the selected colour state ($brand-primary: #29abe2;) to give a visual cue when "search mode" is active and inactive. Prototype: https://invis.io/VYBT5JOBM#/234846520_Home

Please note that mockup uses a much older version of the CMS so the filter bar is black when opened. These are the visual changes from what I can tell according to that mockup are:

Filter bar changes colour to blue when filters are set
The background of contents of the gridfield turns light blue when a filter is set
The contents of the column which has been filtered uses bold text rather than a normal body weight

@clarkepaul @colintucker CSV-Import is also hard to find. And if you import, the Success Message like "Imported 20 records." gets hidden in the collapsed Search, since "Import from CSV" collapses Search Panel.

@lerni that would be more relevant on this issue https://github.com/silverstripe/silverstripe-framework/issues/6431
I will put a comment form you there.

Thanks for your efforts with this @clarkepaul - the current filter/import stuff in ModelAdmin is usually the only thing clients complain about to me in the admin.  😉

Agreed this is high priority. Note that I'm also keen to make this "advanced filter" part of GridField components eventually, so it can be applied to any GridField. So ideally we find a design solution where the active filter indication is part of the GridField (e.g. also works if the GridField is halfway down a "edit page" form with other form fields). Maybe that's too much of a compromise, but just keep reusability in mind, and try to minimise the design variations between different uses of GridField. ModelAdmin is really just a glorified GridField.
This is the current view searching for a page called "Home". It's not ideal either, doesn't indicate what search filters are applied, or how to find out more (search icon is very separate from the "showing search results" indicator).

We done some rethinking around how the search mechanics worked

@newleeland The mockup you linked to shows the inline search for GridField (on column headers). That's different from the ModelAdmin filters, which are a dedicated search panel. This issue is about ModelAdmin search, not about the inlined GridField search.
@clarkepaul Moving this to blocked until there's a design

Jared had done a mockup above, but it's with V3 so would be good to update. His mockup https://invis.io/VYBT5JOBM#/234846520_Home

We have been through this for the Files area (with the same/similar issues) but Pages is still using the old style search area and should be updated to the newer one which is present after searching.

I propose we don't replace the actual search box in areas other than Files (as part of this issue) but improve some of the other differences like "Search results with TERM" and colours etc.

@chillu seeing as the Pages doesn't have any React yet, is it too early to open an issue to update its search/filters to the newer pattern pictured above from the files area?
We could show more of the filters applied in the notice also e.g. Search results with keywords “xxx”, with page type “xxx”, and with the status of “xxx”. were applicable.

Yep, the UX should be consistent with the "Files" area, and agree that adjusting the "Pages" area is out of scope - I've created a separate card for it: https://github.com/silverstripe/silverstripe-cms/issues/1891

ModalAdmin search should model itself as much on Pages search results designs in https://github.com/silverstripe/silverstripe-cms/issues/632 (mockup https://invis.io/EACM3LEHK#/244049027_Pages_Search_Results_3_Results).

@clarkepaul @newleeland Can you confirm what you want for the variables in the template You searched for [Keywords]: Mobil, [Category]: Telecommunications? It looks like it should be the label of the form element, and the value provided. The form element label might be a bit more verbose than needed, e.g. "Keywords (feel free enter multiple words separated by spaces)". Using the name attribute of the form element is equally problematic. That may not be semantic at all.
For the values, a dropdown or checkbox selection likeCategory is likely to be an ID, so we'll need a way of resolving search params to a human-readable label.

Sorry that first field should read "Name:Mobil" not Keywords:Mobil because for the Modaladmin filters there isn't a generic overall search keyword which searches over all of the columns (to my knowledge).

Ok, fixed!


Update link to byte-unixbench

See #5.

forgot to adapt the tar command, see 32efe00

@pniedzielski yeah, 32efe00 has the correct command.


COUNTER API (formerly sushi)

As part of our efforts to use plain language, the Code has been updated to reflect correct terminology: COUNTER_API (formerly sushi) in place of COUNTER_SUSHI_API.

For this type of change it might be better to have separate pull requests for renaming directories and files and changing the references, and for making the wording changes.

The changes in this pull request have been broken up into three separate pull requests for changing the file and directory names (#275), changing the labels and references (#276) and changing the terms (#277).


"empty" page

https://github.com/pybee/remarkable is a 404 page when you click on "Source Code" on this web page:

@ArcTanSusan Thanks for the report - This is a known issue; I'm working on cleaning up the code for a public release.

Closing; the page is no longer empty. Can't say it does anything yet... but I'm working on that. :-)


HDRP Support

it seems this tools not supported on HDRP projects, any chance to support HDRP Projects?

What is it that doesn't work on HDRP?


timers created inside the poll loop are never triggered

See carllerche/mio#926 too. Not sure if this is an issue for mio-extras or base mio.
Sample code here:
https://gist.github.com/pusateri/58ea591219ce2a90166daee811c3dbbb

I'm afraid I'm unlikely to do much with this myself - I mostly forked mio-extras to meet my own needs, and this isn't one of them! - but if you do decide that it's a bug in this repository then I'm definitely open to taking a fix.

This turned out to be a problem with the timer being dropped invalidating it.


Live cluster outage

Describe the bug
It looks like our live platform instance has been down for a while.
To Reproduce
Try browsing to kitt4sme.collab-cloud.eu/auth or shelling into the master node e.g.
$ ssh<EMAIL_ADDRESS>
In both cases the operation will time out and you won't be able to access the cluster.
Expected behavior
The live instance should be up and running and accessible through ssh.
Additional context
N/A

@karikolehmainen brought the cluster back to life :-)


Patch already existing versions on channel sync

This PR adds am owner label to the managedOSVersions created by a channel. This allows to easily fetch owned versions later on for a given channel. Having a map of owned versions allows to easily patch already existing versions.
Note this PR does not apply the label to already existing versions, I have not considered this case for simplicity and because the workaround is as simple as deleting the versions and re-sync again, or deleting and recreating channels. More over this is not a regression in any case.
Deleting ManagedOSVersions is not implemented as I am not sure we have a clear criteria to do so. In any case most of the logic is already there and it could be easily added later on if required.
Fixes #513

I find the entire process a bit hard to read and follow.
I see now that we do create a Pod to "load" the managed os version channel image and dump the json content into the logs, so that we can read them to extract such json and patch it to all managed os versions.
I wonder if we can find maybe a simplified approach for this, like maintaining a static .json file somewhere. Even though that would most likely complicate a bit any airgap scenario (since it will require a blob storage or anything else to serve the file)
Regarding this patch however I have doubts regarding deleting old managed os version. If the updated channel does not include existing (and owned) versions, should they be deleted or not? I assume not for safety, but long term this may be a problem. I guess users can delete them manually.
Still approving despite the questions because I think the PR is ok.

The patch looks good. Anyway, wondering... wouldn't be more straightforward to add a owner reference instead of an owner label? Having the child resources of the owner deleted is something I would expect in any case (would make no sense to keep the generated managedOSVersion resources with a label meant to reference the ownership of a deleted resource).

I think the owner reference is already there, the label is to filter search only.

The patch looks good. Anyway, wondering... wouldn't be more straightforward to add a owner reference instead of an owner label?

We already have the owner reference, we always had it. But we can't query owned resources in k8s, for that you have to list them all and find in references, which are in turn another array. It is a O(n^2) operation. We could argue we will never have such a big amount of references, but still, the implementation without the labels would have been uglier.

Having the child resources of the owner deleted is something I would expect in any case (would make no sense to keep the generated managedOSVersion resources with a label meant to reference the ownership of a deleted resource).

This is already the case.

I find the entire process a bit hard to read and follow.
I see now that we do create a Pod to "load" the managed os version channel image and dump the json content into the logs, so that we can read them to extract such json and patch it to all managed os versions.

This out of scope of this PR. This has been always the case with channels, in #529 this got refactored, but the pod was already there dumping content into stdout. The logic just moved from syncer package directly to the controller, this way the controller owns the whole process, it was just hidden in an insane Sync method before.

I wonder if we can find maybe a simplified approach for this, like maintaining a static .json file somewhere. Even though that would most likely complicate a bit any airgap scenario (since it will require a blob storage or anything else to serve the file)

That's already possible, that what the JSON syncer does. And we used to have it downloaded within the reconcile loop which I consider a really bad choice (that was also fixed in #529). Sandboxing the synchronization within a separate process (Pods are like a good extensible and scalable approach in a k8s env) is also a matter of performance and security. The reconcile loop shouldn't ever block downloading stuff from internet.


Convert Earth Engine JavaScripts to Python scripts

Environment Information
I have tried to convert js to python with enabling qgis feature with this following code:
import os
from geemap.conversion import *

# Create a temporary working directory
work_dir = os.path.join(os.path.expanduser('~'), 'geemap')
# Get Earth Engine JavaScript examples. There are five examples in the geemap package folder.
# Change js_dir to your own folder containing your Earth Engine JavaScripts,
js_dir = r'D:\Code\kode\NFold'
# js_dir = get_js_examples(out_dir=work_dir)

# Convert all Earth Engine JavaScripts in a folder recursively to Python scripts.
js_to_python_dir(in_dir=js_dir, out_dir=js_dir, use_qgis=True)
print("Python scripts saved at: {}".format(js_dir))

Description
error show like this.
use_qgis and import_geemap cannot be both True. Please set one of them to False.
What I Did
I tried to add import_geemap as False
but the result showed
TypeError: js_to_python_dir() got an unexpected keyword argument 'import_geemap'
# Convert all Earth Engine JavaScripts in a folder recursively to Python scripts.
js_to_python_dir(in_dir=js_dir, out_dir=js_dir, use_qgis=True, import_geemap=False)

Thank you for reporting. The bug should be fixed now. Please run geemap.update_package() and restart the kernel


How to install software (dependency) on OpenBSD?

Hello,
I'm trying to use this action for OpenBSD build. I need to install dependency inside OpenBSD.
As I can see, it runs as runner user. This user is not allowed to install packages.
pkg_add package_name doesn't work.
We get an error pkg_add: pkg_add must be run as root. This is correct. Normally we execute pkg_add as a root user or via doas.
doas pkg_add package_name doesn't work.
We get an error doas: doas is not enabled, /etc/doas.conf: No such file or directory
So, my question is: how to install software inside OpenBSD?

sudo works for me (https://github.com/OpenMPT/openmpt/blob/91768c03cdca42dc0a293576c351008d39bd26df/.github/workflows/OpenBSD-7.4-Makefile.yml#L30)

So, my question is: how to install software inside OpenBSD?

@alexander-naumov as mentioned, sudo is available. Perhaps not that visible, but it's mentioned in the openbsd-builder repository [1]. I'm not that familiar with OpenBSD, is doas what's expected to be used instead of sudo?
[1] https://github.com/cross-platform-actions/openbsd-builder#openbsd-builder

Thank you guys! It works.

BTW I had some similar issues and to unify builds of my projects I have created build-actions script, that does a preparation by installing cmake and a C/C++ compiler. I didn't really intend to promote this as it's mostly for my use cases, but if you want to take a look and get some inspiration:
https://github.com/build-actions/build-actions
It's written in python, so it also has a prepare-environment.sh script, which installs python.
This works for me on Windows, Linux, Mac, and BSD VMs. If anyone wants to use this or extend that I'm interested in making it better.


moment().format(String): YYYY should be yyyy and DD should be dd

The tokens accepted by moment().format(String) are somewhat different with what most languages and libraries support.
I focus here on YYYY (2014) and DD (10).
Comparable with Moment.js:

AngularJS: yyyy dd instead of YYYY DD
Qt: yyyy dd instead of YYYY DD
C#: yyyy dd instead of YYYY DD
Java: yyyy dd instead of YYYY DD
Objective-C: yyyy dd instead of YYYY DD
Dart: yyyy dd instead of YYYY DD

They all seem to follow the specifications from the Unicode Consortium: http://www.unicode.org/reports/tr35/tr35-31/tr35-dates.html#Date_Format_Patterns
Not comparable with Moment.js:

Python: % based
PHP: no yyyy nor YYYY
C/C++: % based
Ruby: % based

Interpreted languages are mostly based on C function strftime() (so they are % based) since the interpreters are written in C.

Any news on this?

FYI Next date-fns version (2.0) supports UTR#35, see https://github.com/date-fns/date-fns/issues/520


Make hosted services implement IDisposable

Fixes #2929

I had to manually take care of this due to all the changes.


Angular 10 not yet integrated smoothy

Problem
Does not install smoothly with angular 10, because it requires a peer of angular 9.
To Reproduce
Steps to reproduce the behavior:

Make an angular 10 project
Try to use ngx-svg as described in README

Expected behavior
works.
Instead:
You get an "npm WARN … requires a peer of … but none is installed. You must install ..."
with a peer requirement of angular 9.

Hi! Thanks for creating the issue. I've upgraded the library, now the latest version uses Angular v11 and all other dependencies were upgraded as well. Please give it a try.

Hi! Thanks for creating the issue. I've upgraded the library, now the latest version uses Angular v11 and all other dependencies were upgraded as well. Please give it a try.


Expand faraday version support

Any reason this is locked to v0.9.x?  Causing all kind of compatibility issues with other gems.

@jwg2s It shouldn't be locked to 0.9.x. It's dependency is "faraday", "~> 0.9", which allows use of all later versions of faraday up to and including the latest 0.13.1. It just wouldn't pick up a faraday 1.0 (if that ever happens).

@dazuma "faraday", "~> 0.9" looks like it'll only allow use of later versions of 0.9, i.e. 0.9.1, 0.9.9 etc.
Also, looked this up cause we're in the same boat :)  Want to use this gem but it has a lot of version-locked dependencies which would cause us to not be able to update.

@TempestMia Actually, ~> 0.9 does allow all later 0.x. The "twiddle-wakka" operator means, the furthest-right field must be greater than or equal to, but the second-furthest-right field must be exactly equal. So in this case, it means "0.x[.y]" where x >= 9, i.e. the major version must be "0" but the minor version can be >= 9 (and any patch version can be whatever). Two fields like this is commonly used for semantic versioning because it matches the compatibility semantics of major and minor versions.
What version dependency is failing for you?

@dazuma  I didn't realize it was depending on how far to the right you specify. TIL! Thanks!


Update travis configuration.

explicitely set sudo to false, since we don't need it and it
slows-down builds.
use the newest go toolchain.

👍


flutter run --profile -d web-server requires hard-refresh between recompiles

Steps to Reproduce
In Chrome with DevTools closed, or DevTools open but "Disable Cache" on the "Network" tab unchecked, do the following:

Run any Web app using flutter run --profile -d web-server.
Make a change in your code.
Hit R for hot restart
Watch the tool recompile the code then say Recompile complete. Page requires refresh.
Soft-refresh the page (by hitting the refresh button of Ctrl + r). Make sure is not a hard refresh (e.g. Ctrl + Shift + r).

Expected results:
Code changes should apply.
Actual results:
Code changes do not apply. You are looking at the old app.
Details
This is happening as of version 9bc0e6a991cbbe2bfe8328eacb62f6320b8d6812.

/cc @jonahwilliams

The issue here is that the tool is not setting the correct headers, so the browser is happy to cache main.dart.js forever. This should be an easy fix....

Here's a temporary solution：

Make sure you have NodeJS
In your flutter project root dir, run this:

$ npx flutter_build_web --release

Only do this:

Build main.dart.js -> main_[hash].dart.js
Change index.html main.dart.js -> main_[hash].dart.js

that seems like a much worse fix than fixing the headers


Layout issue on NzTableComponent when using nzCustomFilter and nzShowSort at the same time

Reproduction link
https://stackblitz.com/edit/angular-t4z2kr
Steps to reproduce
Add nzShowSort and nzCustomFilter to a <th> at the same time.
What is expected?
The search-icon should be placed on the right edge of the column header.
What is actually happening?
The search-icon is placed between title and sorting arrow icons.

Environment
Info

ng-zorro-antd
8.0.1

Browser
Firefox Developer Edition 69.0a2

This sample is based on "Customized filter panel" demo in documentation

duplicate of #3721


High RTP deltas in wireshark, intermittent sound on recording in Asterisk.

Hi!
I'm not a voip engineer, coder, or native speaker, but I will try to explain correctly.
I have troubles with transmitting audio when audio_txmode  poll
Wireshark show periodic, not uniform, high RTP deltas, and call records are damaged on Asterisk end (or even call quality goes down).
Dumps for wireshark was made on client side, e.g this one with ausrc_srate 48000:

Reverse stream looks better:

With ausrc_srate 8000 deltas will be around 60ms, but still here.
When audio_txmode thread is used, then all is fine!
I ask your advice in search of a possible reason. It's hardware or software problem? Below is my setup:

name
value

Board
Raspberry Pi 3 B

Sound
Jabra Evolve 20

Distro
Yocto (sumo)

Linux
4.4.68

Baresip
1.6.3 (2894490)

libre/librem
0.6.0

Codec
PCMA;ptime=20

baresip config:
poll_method         epoll # poll, select, epoll ..
sip_trans_bsize     128
call_local_timeout  120
call_max_calls      4
audio_path          /usr/share/baresip
audio_player        alsa,default
audio_source        alsa,default
audio_alert         alsa,default
ausrc_srate         48000
auplay_srate        48000
audio_txmode        poll # poll, thread
rtp_tos             184
rtcp_enable         yes
rtcp_mux            no
jitter_buffer_delay 5-10 # frames
rtp_stats           yes
module_path         /usr/lib/baresip/modules
module              alsa.so
module              g711.so
module_app          auloop.so
module_app          menu.so
module_app          syslog.so
module_tmp          uuid.so
module_tmp          account.so

What else can I do? Thanks!

the difference between poll and thread is basically:

poll - send packets based on timing from sound card

thread - send packets using a dedicated thread and a timer

please test the auloop command on your hardware:
(apply this patch first:
https://gist.github.com/alfredh/473a4ea2db44733786dd4d564846933c)
$ ./baresip -e "/auloop 8000 1"

and let is run for 10-20 seconds before doing "quit".

here is the timing info from my Debian 10 computer (Intel Soundcard):
/auloop 8000 1
Audio-loop: 8000Hz, 1ch, S16LE
~~~~~ Audioloop summary: ~~~~~
8000 Hz 1ch S16LE

* Source
  module      alsa
  samples     89120
  duration    11.140 sec
  real ptime  0.03 / 19.95 / 30.18 ms (min/avg/max)
  avg.sampc   20.000 ms

* Player
  module      alsa
  samples     90080
  duration    11.259 sec

**** List of CAPTURE Hardware Devices ****
card 0: PCH [HDA Intel PCH], device 0: ALC283 Analog [ALC283 Analog]
  Subdevices: 1/1
  Subdevice #0: subdevice #0

the average ptime should be around 20ms.

/auloop 8000 1
Audio-loop: 8000Hz, 1ch, S16LE
~~~~~ Audioloop summary: ~~~~~
8000 Hz 1ch S16LE

* Source
  module      alsa
  samples     192960
  duration    24.120 sec
  real ptime  0.02 / 19.95 / 64.04 ms (min/avg/max)
  avg.sampc   20.000 ms

* Player
  module      alsa
  samples     193440
  duration    24.179 sec

/auloop 48000 1
Audio-loop: 48000Hz, 1ch, S16LE
~~~~~ Audioloop summary: ~~~~~
48000 Hz 1ch S16LE

* Source
  module      alsa
  samples     1160640
  duration    24.179 sec
  real ptime  0.02 / 19.95 / 63.97 ms (min/avg/max)
  avg.sampc   20.000 ms

* Player
  module      alsa
  samples     1169280
  duration    24.359 sec

thanks,
could you please test with a different sound card and see how is the timing ?

Creative LIVE sound card
~~~~~ Audioloop summary: ~~~~~
8000 Hz 1ch S16LE

* Source
  module      alsa
  samples     175680
  duration    21.960 sec
  real ptime  0.02 / 20.07 / 22.16 ms (min/avg/max)
  avg.sampc   20.000 ms

* Player
  module      alsa
  samples     176640
  duration    22.079 sec

**** List of PLAYBACK Hardware Devices ****
card 0: U0x41e0x30d3 [USB Device 0x41e:0x30d3], device 0: USB Audio [USB Audio]
  Subdevices: 1/1
  Subdevice #0: subdevice #290

With Creative LIVE sound card similar picrure:

Reverse:

Hmm, I tend to believe this is not baresip problem at all.
I made these tests on other equipment (Atom Z8350). Same distro but with x64 4.15.18 kernel.
Tried two soundcards, es8316 and Creative LIVE. Results are excellent. Low jitter and normal average ptime. I guess the problem may be that raspberrypy does not have any hardware RTC or so.
Anyway, thank you for you attention and help!

@alfredh
Interesting, I found if I change default ALSA device from alsa,default to alsa,plughw:0,0 all works as intended. Deltas are gone. :confused:


Context and hidden state

Hi, I'm a little confused regarding the notation in your paper and your code and hope you could provide some clarification. In your paper, the aggregation function has two outputs, one is c_t, and the other one is the hidden state. In this part of your code (in DPC/model_3d.py):
pred = []
for i in range(self.pred_step):
      # sequentially pred future
      p_tmp = self.network_pred(hidden)
      pred.append(p_tmp)
       _, hidden = self.agg(self.relu(p_tmp).unsqueeze(1), hidden.unsqueeze(0))
      hidden = hidden[:,-1,:]
pred = torch.stack(pred, 1) # B, pred_step, xxx
del hidden

where the variable hidden is fed into function phi(.). Then hidden and phi(hidden) are fed into the aggregation function. Does it mean c_t is just the hidden state? What might be the 'context representation' mentioned in the paper in that case?

Hi, you are right, what we meant c_t in the paper is just the hidden state of GRU.
Precisely, in paper Fig 2, two horizontal arrows before x_{t+1} and x_{t+2} are not necessary.

Thanks for the prompt answer! That clarifies a lot of confusion. Just one quick follow-up question: as I understand it, the context in model_3d_lc.py is the concatenated hidden states of each time step correct?
feature = F.avg_pool3d(feature, (self.last_duration, 1, 1), stride=1)
feature = feature.view(B, N, self.param['feature_size'], self.last_size, self.last_size) # [B*N,D,last_size,last_size]
context, _ = self.agg(feature)
context = context[:,-1,:].unsqueeze(1)
context = F.avg_pool3d(context, (1, self.last_size, self.last_size), stride=1).squeeze(-1).squeeze(-1)

But since context = context[:,-1,:].unsqueeze(1) which is effectively the same as hidden correct? Is it the same as projecting the last hidden state to the action space?

Yes, you're right. They refer to the same thing, just different names between pre-training and evaluation codes.


Blank display

Hello,
I finally got around to building your project (thank you), everything seemed to go ok with the build, I have managed to program the board using an FTDI controller I already had lying around using Arduino IDE and the Arduino Uno board profile. It said that the upload was successful so I have no reason to think it wasn't.
I just get a blank display, it lights up but nothing on there, I have tried powering the unit externally and its the same. I didn't use the exact 16x2 display but the pin configuration was the same as the one you have used.
Any ideas what I can try to diagnose?

I’d start by checking the voltage supplied to the panel. If the panel has backlighting, is it on? Does the LCD panel support 3.3V?
Next, I’d try to see if the microcontroller is working, by checking if the load is switched on and off. The measurement cycle is slow, so a multimeter should be enough.

Great places to start troubleshhting, can you confirm the load is on pins 25&26?

Not exactly the load, but related to it. Pin 25 and 26 on the MCU read the voltages at the end of the precision 1Ohm current sense resistor, so you should see a voltage across them, from 0V to ~0.8V.

Turns out if was some faulty ribbon cable! It appears to work now. One last question, the 8mhz crystal, does it have to be 8mhz or will a 16mhz one suffice?

8MHz is within the operating spec for the microcontroller when powered at 3.3V. 16MHz may be a bit too much for 3.3V, for that frequency 5V would be more “comfortable”.

Thank you so much for all of your advice, the project is great and will be very useful for me.

Glad to hear that!


Add service sort order

Thoughts on updating service definition attribtue so that it follows the same convention as the Service order? Otherwise it will mean a fair bit of duplicate code.

Yes, absolutely agree. Along with Groups as well. Since Attributes are limited to just one Multivalue list for MMP, the ticket related to this PR is specifically scoped to just Services.

Thoughts on updating service definition attribtue so that it follows the same convention as the Service order? Otherwise it will mean a fair bit of duplicate code.

Yes, absolutely agree. Along with Groups as well. Since Attributes are limited to just one Multivalue list for MMP, the ticket related to this PR is specifically scoped to just Services.

My understanding is that we are limiting it to only allowing service attributes of type multivaluelist but that there can be multiple for one service


No description

No description, website, or topics provided.

FIRST ISSUE!

Thanks for reporting the issue. Description should be added now.

I think this is the first time someone whose handle I don't know has suggest an issue related to the CDC open practices, https://github.com/CDCgov/template/blob/master/open_practices.md
Thanks @tseemann for finding this, and thanks @jiangweiyao for adding a description. Please consider adding topics as well to help people find your project, especially when they are indexed up at code.gov.

@leebrian excited to see this CDC policy for open source software!


Audits CI should be able to push to public forks

If people make changes to the audits in the repo and open a PR, the CI should be able to push the reports updates.
Issue appeared here: https://github.com/graphql/graphql-http/pull/27.

Closed with https://github.com/graphql/graphql-http/commit/7362b22e53d4e50cd712abeb7e35eb10b7b66d57.


ExternalModules::isSystemSetting() and ExternalModules::isProjectSettingDefined() do not consider nested settings

As in the title. These functions should recursively search through sub_settings.
Or alternatively renamed to something like isRootSystemSetting / isRootProjectSetting.
Stumbled upon this while experimenting with file fields in the config for building the enhanced module configuration dialog.
As a side:

isProjectSettingDefined is not used anywhere in the EM framework.
isSystemSetting is used in hasProjectSettingSavePermission() and in the save-file and delete-file ajax calls.

As a consequence of this bug, nested file settings do not work at all. What would happen there if this bug is fixed is a whole other question though.
The enhanced configuration dialog aims to support nesting of any field any level deep and I guess file handling will have to be handled by using temp files. Furthermore, any changes to the module settings table will only ever occur after the dialog is closed with "Save". Currently, file deletions take effect immediately and there is the possibility to get orphaned files in edoc.

This may relate to #171, #186, #188, and/or #205.

Hmm, #171 should work (on the root level). At least I remember this working for me in my recent "exploration". It definitely does not work at deeper nesting levels.


Support custom cachedir classes

This PR allows a user to create a custom CacheDir and have SCons use it for controlling cacheing behaviors.
This PR captures just the changes to SCons to allow MongoDB to use a compressed CacheDir tool. Downstream (MongoDB is currently using a local copy of SCons 3.1.2) changes can be seen here: https://github.com/dmoody256/mongo/pull/1
Contributor Checklist:

[x] I have created a new test or updated the unit tests to cover the new/changed functionality.
[x] I have updated CHANGES.txt (and read the README.rst)
[x] I have updated the appropriate documentation

re custom cachedir class.. how many methods does such class have to implement? Typically python is about duck typing, not class inheritance.

re custom cachedir class.. how many methods does such class have to implement? Typically python is about duck typing, not class inheritance.

it does not need to implement any. The only reason for required issubclass from CacheDir is to help the user from shooting themselves in the foot. This way is leading them in to least likely way to cause confusing errors for themselves.
Duck typing is already an available option anyways, but you would need to set the last_CacheDir on the env, which I assume the '' in _last_CacheDir means 'private', also when you set it can have different results.

@bdbaddog bump, any other issues left? I would like to move forward with similar code in mongodb's vendored scons.

@bdbaddog bump, any other issues left? I would like to move forward with similar code in mongodb's vendored scons.

Not sure how your comment about reinstanctiating answers my response about using isinstance?

@bdbaddog bump, any other issues left? I would like to move forward with similar code in mongodb's vendored scons.

Not sure how your comment about reinstanctiating answers my response about using isinstance?

isinstance would not cause scons to reinstantiate, but the user would want it to because the user has passed a different class to use, that happens to be a subclass of some other class they passed first. It's a bit contrived, but it is a possible use case, and checking types is a valid use of python, so I don't think it should not be handled due to pylinting warning.


legacyio.py contains no executable code.

All of the code in this file has been commented out. I suggest we delete the file.
https://github.com/sandialabs/pyGSTi/blob/ee2158575f4a6a9cffa7465983054be13772e0f8/pygsti/io/legacyio.py#L1-L385

Agreed. I think the next release we are anticipating is 0.9.13, which will have other deletions/deprecations (i.e. Qibo). So we can go ahead and delete this as part of that.

Note: At time of commenting, PR #451 includes an incidental change that would resolve this issue. However, I just opened PR #452 that's dedicated to the broader problem of removing commented-out code.

Closed with #451


<EMAIL_ADDRESS>download via aria2 failed

Scoop Configuration

//# Your configuration here

@808834 Have you tried to use regular package typora instead of typora-stable

Yes, I tried the latest version and can install it successfully. But the crack dll does not work on the latest version, so I want to try this   stable verison.

Yes, I tried the latest version and can install it successfully. But the crack dll does not work on the latest version, so I want to try this stable verison.

The typora-stable wont work either:
Look at the notes property here, it is basically the same.. 15 day trial


TextEditingController was used after being disposed.

I'm using this package and providing my own TextEditingController into RichTextEditor. But when I Navigate out of the page I get the error below;
════════ Exception caught by hooks library ═════════════════════════════════════
The following assertion was thrown while disposing _Entry<HookState<dynamic, Hook<dynamic>>>:
A TextEditingController was used after being disposed.

Once you have called dispose() on a TextEditingController, it can no longer be used.
When the exception was thrown, this was the stack
#0      ChangeNotifier.debugAssertNotDisposed.<anonymous closure>
package:flutter/…/foundation/change_notifier.dart:147
#1      ChangeNotifier.debugAssertNotDisposed
package:flutter/…/foundation/change_notifier.dart:154
#2      ChangeNotifier.dispose
package:flutter/…/foundation/change_notifier.dart:302
#3      _TextEditingControllerHookState.dispose
package:flutter_hooks/src/text_controller.dart:90
#4      HookElement.unmount
package:flutter_hooks/src/framework.dart:493
#5      _InactiveElements._unmount
package:flutter/…/widgets/framework.dart:1917
#6      _InactiveElements._unmount.<anonymous closure>

I'm using the TextEditingController from flutter_hooks which disposes automatically once it is not used.
My widget looks like this.
import 'package:flutter_hooks/flutter_hooks.dart';

Widget build(BuildContext context){
  final textEditingController = useTextEditingController();

  RichTextEditor(
    controller: textEditingController,
    onChanged: (value) {
        //I'm using the textEditingController here
    },
  ),
}

Looks like the issue is in the text_editor.dart file. The controller that is passed to the RichTextEditor is assigned to TextEditingController that is inside the _RichTextEditorState class.
late TextEditingController controller;
late SuggestionController suggestionController;

@override
  void initState() {
    super.initState();
    controller = widget.controller ?? TextEditingController(text: widget.initialValue ?? '');
    suggestionController = (
      widget.suggestionController ?? SuggestionController()
    )..addListener(() {
      setState(() {});
    });
  }

And then once you pop out of the page the _RichTextEditorState's controller gets disposed and the error is thrown.
@override
  void dispose() {
    //This is what throws the error.
    controller.dispose();

    suggestionController.dispose();
    super.dispose();
  }

Also getting this issue, have you solved this one master @inc16sec ?

Also getting this issue, have you solved this one master @inc16sec ?

I delete the controller.dispose(); line inside dispose() method. I don't know if it's the right thing to do, but for now it got rid of the error.
The controller.dispose() inside the text_editor.dart file.

@nelsonweze basically same thing should be done for SuggestionsController as it is can be set outside TextEditor and should be disposed only if it is not existed. If nor - there will be same error about disposing but for suggestions controller.
Sent a PR if you are interested with this fix: https://github.com/nelsonweze/rich_text_view/pull/38
Regards,
Anton


404 on HTTPs

It works fine on http but with https enabled all assets processed by pagespeed are 404.
This is a test setup on nginx, same website with letscrypt ssl works fine on apache modspeed.
Tried latest stable and release,
pagespeed installed as dynamic module.
This is my pagespeed conf, kindly suggest a solution.
` pagespeed on;
pagespeed HonorCsp on;
pagespeed RespectXForwardedProto on;
#Same issue with these enabled and loadfrom file disabled.
#pagespeed SslCertDirectory /etc/pki/tls/certs;
#pagespeed SslCertFile /etc/pki/tls/cert.pem;
pagespeed LoadFromFile "https://test.site.com" "/home/test/public_html/";
pagespeed RewriteLevel CoreFilters;
pagespeed XHeaderValue "Powered By ngx_pagespeed";
pagespeed FileCachePath /var/cache/nginx/;
location ~ ".pagespeed.([a-z].)?[a-z]{2}.[^.]{10}.[^.]+" {
add_header "" "";
}
location ~ "^/pagespeed_static/" { }
location ~ "^/ngx_pagespeed_beacon$" { }`

Either LoadFromFile is not matching with the incoming urls, or it does match but the module isn't able to get direct access the mapped static files. You have pagespeed RespectXForwardedProto on;, which makes me think chances are the incoming urls do not match. Presumably there is a reverse proxy running in front that terminates https. What happens when you test the server directly?

I was just testing "RespectXFor.." , didn't help.
No proxy, just nginx/php-fpm.
Pagespeed cache is getting populated.
Ex. there is a file under ../v3/site/https,3A/..  "xfavicon-16x16.png.pagespeed.ic.JI0K0aTlw4.webp"
but it is 404 when directly called in browser , https://site.com/xfavicon-16x16.png.pagespeed.ic.JI0K0aTlw4.webp
Directly every thing works,  https://site.com/favicon-16x16.png is accessible.

I reinstalled & everything working great now.
Must be a config issue somewhere.


Train network with recent data augmentation methods

Hi..
i have 2 Qst:
The first is: what should we expect after running this command:
python3 train.py --input data/imdb_db.mat
because it took a lot of time without giving any output.
the seconde one is: i ran this command 3 days ago and it's still running..
knowingly that our computer RAM & CPU are suitable!

I recommend to use a GPU in training.
You might be able to use GPU with Google Colaboratory.
In training, it might be better to use ADAM optimizer using the following arguments --opt adam --lr 0.001.


Scan

The compilation time is down from 600s (depth=32, qrnn_frequency=8) to 70s (depth=4, qrnn_frequency=8 -> total_depth=4*8).
Baseline: https://wandb.ai/homebrewnlp/gpt/runs/2ar16xdkjoz57mdvgipffnng335nik6t
New: https://wandb.ai/homebrewnlp/gpt/runs/4hqwsrq7o9ic8x2bhp3wgu7ipdhk1h3y
Once #7 is finished and we run QRNN at every step, the compile time will drop as we don't have to manually unroll 8 steps anymore.

Stable


[Queue] Backoff is ignored if a job is retried after it was killed due to timeout

Laravel Version: 9.x
PHP Version: 8.1
Database Driver & Version: MySQL

Description:
Basically a duplicate of https://github.com/laravel/framework/issues/30509
We have a job that uploads data to an S3 bucket. Sometimes the S3 API we're using stalls due to a temporary networking issue.
We'd like to be able to specify a different retry time via the backoff function available on the job interface.
The backoff parameter gets ignored if the worker has a timeout inferior to the stall time of the job.
This is due to the different path a worker uses to be killed when a job times out (see here and here).
In order for the worker to be killed correctly after a timeout and the job to use the backoff parameter correctly we applied this patch, which introduces a new Exception type, handles the timeout exception and then kills the worker.
diff --git a/src/Illuminate/Queue/JobTimeoutException.php b/src/Illuminate/Queue/JobTimeoutException.php
new file mode 100644
index<PHONE_NUMBER>..c65073766d
--- /dev/null
+++ b/src/Illuminate/Queue/JobTimeoutException.php
@@ -0,0 +1,19 @@
+<?php
+
+namespace Illuminate\Queue;
+
+use RuntimeException;
+
+class JobTimeoutException extends RuntimeException
+{
+    /**
+     * Create a new exception instance.
+     *
+     * @param  string|null  $message
+     * @return void
+     */
+    public function __construct($message = null)
+    {
+        parent::__construct($message ?: json_last_error());
+    }
+}
diff --git a/src/Illuminate/Queue/Worker.php b/src/Illuminate/Queue/Worker.php
index 039cbebb30..48616824c4 100644
--- a/src/Illuminate/Queue/Worker.php
+++ b/src/Illuminate/Queue/Worker.php
@@ -223,7 +223,7 @@ protected function registerTimeoutHandler($job, WorkerOptions $options)
                 );
             }

-            $this->kill(static::EXIT_ERROR);
+            $this->handleJobException($job->getConnectionName(), $job, $options, new JobTimeoutException("Job timeout reached"));
         });

         pcntl_alarm(
@@ -473,7 +473,11 @@ protected function handleJobException($connectionName, $job, WorkerOptions $opti
             }
         }

-        throw $e;
+        if($e instanceof JobTimeoutException) {
+            $this->kill(static::EXIT_ERROR);
+        } else {
+            throw $e;
+        }
     }

     /**

If nobody sees anything wrong with this i'd like to open a pull request with these changes.
Steps To Reproduce:
Create a new job with backoff, timeout, retries and in its handle method put a sleep for more than the job's timeout.
Example code:
<?php

namespace App\Jobs;

use Illuminate\Bus\Queueable;
use Illuminate\Contracts\Queue\ShouldBeUnique;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Foundation\Bus\Dispatchable;
use Illuminate\Queue\InteractsWithQueue;
use Illuminate\Queue\SerializesModels;
use Illuminate\Support\Facades\Log;

class BackoffJobDelay implements ShouldQueue
{
    use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;

    public function backoff() {
        return [5, 10, 20];
    }

    public $timeout = 10;
    public $tries = 4;

    public function handle()
    {
        Log::info("PCNTL installed: ". (extension_loaded("pcntl") ? "true" : "false"));
        Log::info("Sleeping for 120 seconds");
        sleep(120);
        Log::info("Resumed from sleep");
    }
}

Start a queue worker:
$ php artisan queue:work

Dispatch the job via tinker onto a mysql db:
$ php artisan tinker
> \Bus::dispatch(new \App\Jobs\BackoffJobDelay());

After the canonical 90 seconds the worker gets killed and the job is dispatched again without waiting for the specified backoff time.

If you feel a change should be made you can open a PR for review.


Delete Bintray code

Remove all code related to Bintray. It no longer works so there's no point keeping it around.
Some of this could arguably be deprecated/disabled first/instead but: I'm not sure I see the sense in keeping stuff around that's known to be broken.

Review period ended.


Web Workers

Add web workers, closes #206.

After fixing the conlicts, this is still broken on the serialzie unknown length issue in bincode.
Need to try and figure out what isn't of known length.

And the dead-end upstream issue.


Tutorial Page btp-free-tier-account.md Issue. PROD

Tutorial issue found: https://github.com/sap-tutorials/Tutorials/blob/master/tutorials/btp-free-tier-account/btp-free-tier-account.md contains invalid tags. Even though your tutorial was created, the invalid tags listed below were disregarded. Please double-check the following tags:

software-product>sap-business-technology-platform; free-tier

Affected server: PROD

I have checked the tags and tutorial>free-tier should be the correct one.

Changed and created pull request


Implement single endianness handling pattern

Is your feature request related to a problem? Please describe.
Multiple ways to handle target endian in various contexts:
fsw/cfe-core/src/es/cfe_es_perf.c:CFE_ES_SetupPerfVariables - determines it at runtime
fsw/cfe-core/src/tbl/cfe_tbl_internal.c:CFE_TBL_ReadHeaders - determines it at runtime
fsw/cfe-core/src/tbl/cfe_tbl_task_cmds.c:CFE_TBL_DumpToFile - determines it at runtime
fsw/cfe-core/src/fs/cfe_fs_api.c:CFE_FS_ReadHeader (and many other functions in here) - determines it at runtime
fsw/cfe-core/src/inc/ccsds.h has macros for conversion (CFE_MAKE_BIG*) who’s implementation depends on SOFTWARE_BIG_BIT_ORDER
There’s an endian flag in the CCSDS header
osal/src/os/inc/common_types.h defines either SOFTWARE_LITTLE_BIT_ORDER or SOFTWARE_BIG_BIT_ORDER based on 8 possible defines
cfe/cmake/sample_defs/cpu1_platform_cfg.h defines CFE_PLATFORM_ENDIAN as either CCSDS_LITTLE_ENDIAN or CCSDS_BIG_ENDIAN, and also has a separately configurable CFE_PLATFORM_TIME_CFG_BIGENDIAN
CCSDS extended header has an endian bit
Then there’s all the different ways the various defines are used.
Describe the solution you'd like
See https://github.com/nasa/cFE/issues/202#issuecomment-536680998 for a compile time suggestion:
#define IS_LITTLE_ENDIAN (((union {unsigned x; char c;}){1}).c)
Describe alternatives you've considered
None
Additional context
Triggered from email discussion on setting another endian flag in a toolchain file for an app to use
Requester Info
Jacob Hageman - NASA/GSFC
Ping - @klystron78 @ejtimmon @wmoleski

Jake,
            This is defined in my sample_defs/cpu1_platform_cfg.h file. However, CF seems to be looking for an additional define.

            Also, I thought the MCP750 was BIG ENDIAN with vXWorks. The above file has LITTLE set in my repo.

Walt
Walt Moleski                  Phone:<PHONE_NUMBER>
NASA/GSFC Code 582   Fax:<PHONE_NUMBER>
Greenbelt, MD 20771
email: Walter.F.Moleski<EMAIL_ADDRESS>From: Jake Hageman<EMAIL_ADDRESS>Sent: Thursday, March 4, 2021 1:42 PM
To: nasa/cFE<EMAIL_ADDRESS>Cc: Moleski, Walter F. (GSFC-5820<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: [EXTERNAL] [nasa/cFE] Implement single endianness handling pattern (#1209)
Is your feature request related to a problem? Please describe.
Multiple ways to handle target endian in various contexts:
fsw/cfe-core/src/es/cfe_es_perf.c:CFE_ES_SetupPerfVariables - determines it at runtime
fsw/cfe-core/src/tbl/cfe_tbl_internal.c:CFE_TBL_ReadHeaders - determines it at runtime
fsw/cfe-core/src/tbl/cfe_tbl_task_cmds.c:CFE_TBL_DumpToFile - determines it at runtime
fsw/cfe-core/src/fs/cfe_fs_api.c:CFE_FS_ReadHeader (and many other functions in here) - determines it at runtime
fsw/cfe-core/src/inc/ccsds.h has macros for conversion (CFE_MAKE_BIG*) who's implementation depends on SOFTWARE_BIG_BIT_ORDER
There's an endian flag in the CCSDS header
osal/src/os/inc/common_types.h defines either SOFTWARE_LITTLE_BIT_ORDER or SOFTWARE_BIG_BIT_ORDER based on 8 possible defines
cfe/cmake/sample_defs/cpu1_platform_cfg.h defines CFE_PLATFORM_ENDIAN as either CCSDS_LITTLE_ENDIAN or CCSDS_BIG_ENDIAN, and also has a separately configurable CFE_PLATFORM_TIME_CFG_BIGENDIAN
CCSDS extended header has an endian bit
Then there's all the different ways the various defines are used.
Describe the solution you'd like
See #202 (comment)https://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnasa%2FcFE%2Fissues%2F202%23issuecomment-536680998&data=04|01|walter.f.moleski%40nasa.gov|c4f7b4ceb01d4bb7fd6b08d8df3d4565|7005d45845be48ae8140d43da96dd17b|0|0|637504801545877402|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=81rVPefvdLbnWukGS1h01UhMe6noOnKC9jMkICEE5oo%3D&reserved=0 for a compile time suggestion:
#define IS_LITTLE_ENDIAN (((union {unsigned x; char c;}){1}).c)
Describe alternatives you've considered
None
Additional context
Triggered from email discussion on setting another endian flag in a toolchain file for an app to use
Requester Info
Jacob Hageman - NASA/GSFC
Ping - @klystron78https://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fklystron78&data=04|01|walter.f.moleski%40nasa.gov|c4f7b4ceb01d4bb7fd6b08d8df3d4565|7005d45845be48ae8140d43da96dd17b|0|0|637504801545877402|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=5Qa2xKjMS31BfpXizhqYDq%2FoKOomX7JF2Y42qfVhXy0%3D&reserved=0 @ejtimmonhttps://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fejtimmon&data=04|01|walter.f.moleski%40nasa.gov|c4f7b4ceb01d4bb7fd6b08d8df3d4565|7005d45845be48ae8140d43da96dd17b|0|0|637504801545887351|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=%2FvM9RWsjp2uJlduFqO6nmZiKxJg3k9xGnMSynIL9XEI%3D&reserved=0 @wmoleskihttps://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fwmoleski&data=04|01|walter.f.moleski%40nasa.gov|c4f7b4ceb01d4bb7fd6b08d8df3d4565|7005d45845be48ae8140d43da96dd17b|0|0|637504801545887351|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=NwRKm6mCzet3j8acQpkYiWODNYor4qwOAJBqqUds%2BuE%3D&reserved=0

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnasa%2FcFE%2Fissues%2F1209&data=04|01|walter.f.moleski%40nasa.gov|c4f7b4ceb01d4bb7fd6b08d8df3d4565|7005d45845be48ae8140d43da96dd17b|0|0|637504801545897311|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Q9fTB5m41SYlQSWu75kjak%2FWX1%2FirnG3ex0Bv1gKqpY%3D&reserved=0, or unsubscribehttps://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAQU7KNKTGRK54XUDMV3MK2LTB7IBHANCNFSM4YTY76AA&data=04|01|walter.f.moleski%40nasa.gov|c4f7b4ceb01d4bb7fd6b08d8df3d4565|7005d45845be48ae8140d43da96dd17b|0|0|637504801545897311|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=eYxuJRhPXcoPAJZ0n5nXlil5QWgD5LcI%2BjjdeNrvF5U%3D&reserved=0.

The challenge here is that the C standard (and library) don't want to expose endianness.  They actively discourage one from writing endian-specific code, and there is no standardized "marker" to say what endianness the current CPU has.  In particular a machine is allowed to arrange the bits however it pleases and still be compliant (see PDP-11).  AFAIK There is even a variant that takes the weird PDP-11 order and reverses it.  While none of these are common, they do exist.
The problem with any "endian" check is that it is usually binary on the common ones (big or little), and does not account for other variations.
Rather than introducing another endianness-check to rule them all, I'd rather see migration away from any sort of endian-specific code/dependencies.
FWIW, the SOFTWARE_LITTLE_BIT_ORDER (a misnomer because its actually byte order) was intended for compile-time checks where the union trick does not work.


Improve the pod moving behavior during the Kubernetes cluster upgrade.

Fix an issue of not waiting for at least one replica to become ready
(if the Statefulset indicates there are replicas) when moving the master
pod off the decomissioned node. Resolves the first part of #279.
Small fixes to error messages.

Coverage decreased (-0.07%) to 11.26% when pulling 7eb3165272912b9c8984242e7cbd0da1fe892096 on wait_for_replicas_on_upgrade into 8f08bef67cac54503c4794145f63421d0446b0c4 on master.

👎 the race condition is apparently still there

👍

As a note, we always wait for the label without checking the status of the pod (it could be that the pod is in termination phase but the label is still there.). This, however, should be a subject of a separate PR to avoid having too many things crammed into this one.

👍


Please help me to run this project

Please help me to run this project
please provide the steps to run this projec

There are some SQL Queries provided in the MySQL Directory. Please execute them after creating a database named "applestore". Make sure you change the credentials in the PHP connections with the credentials for your local machine.


Portals are not generated in the same shape as built

I don't think that's a bug, the same thing happens for normal nether portals

This should be fixed in #65


I want to reset my drawing library

Right now I don't see a way to reset (or clear) my drawing library.  I see a reset in the code here:
https://github.com/excalidraw/excalidraw/blob/5b829772d937262945dc55281e62de468e919776/src/data/library.ts#L11
But don't see it called in the UI.
workaround
You can reset your library manually by clearing it's local storage key excalidraw-library  (eg via chrome dev tools)

Yes, this is definitely needed!

@excalibot try importing heftier libs like this and then try to delete by one :). (But that's also https://github.com/excalidraw/excalidraw/issues/2618.)

I would love to take this up. Should I add a clear button below "Browse libraries" maybe?

@h7y I was thinking we could add a Trash can button to the tools on the drawing library drawer. Like this:

what do you think?

Yes, I think icon makes more sense. As there is no undo in this action.. we might need a confirmation dialog. @h7y I'll assign it to you.


GPU Monitor over 100%

i think the gpu usage was over 100% looks unnatural
Reproduce:

must have 2 gpus (igpu and egpu)
right click on schale folder
suffer

Picture:

update:
i think it was a script error

It definitely looks like an incorrect implementation of the UsageMonitor plugin by me. I do not own a device with multiple GPUs, so this issue went unnoticed. The official UsageMonitor documentation implies a problem with summation vs. average of the resource values, but I will have to rely on you to confirm the below solution works as intended.
Try changing the following value in BlueArchive\template\resources.inc in Line 62 (under a section titled [GPUUsage]):
Index=0 to Index=-1
The new -1 value should fix the overflowing display value by taking the average of all detected GPUs' usage percentage instead of their sum. However, keep in mind that the value might still be different from Task Manager's value according to this thread on the Rainmeter Forums; the difference unfortunately cannot be resolved, but should be small enough to be negligible.
You may want to change the value of Index of the CPU measurement in the same manner as well, located on Line 48 in section [CPUUsage].
Let me know if it works so I know to apply this change in a clean-up patch for the suite.

neat! i fixed the bug!


Android Notification

Android notification is not receiving when app is closed

Hi @rwaidKalka,
Thanks for reaching out. We'd be glad to assist.
Please refer to the following article in our support center for help on this topic:
How can I send notifications to Android devices in Doze / power saving mode?

Thanks for your instructions i already did that one too i disabled the battery optimization and still the notification is not reached to us while the app is terminated
is there anything else i can try to solve this issue ?
i am using flutter framework for the app development and the ios doesn't have any issue only android has this issue.

Hi @rwaidKalka,
I'd be glad to help you investigate this issue further.
If you could please provide the following information about a device experiencing this issue, that would be great:

Device manufacturer
Device model name
Android OS version

Hi @rwaidKalka,
Can you please share your project's main.dart? You may also send it to us privately at<EMAIL_ADDRESS>

Sure i am gonna send it right now but i did the configuration in another file which is app.dart i will send both of them from the main.dart i called tha app.dart file in the runApp.

Hello
I already sent the code to the provided email but i didn't receive any
email.
On Wed, Jun 7, 2023, 8:44 AM Pushy @.***> wrote:

Hi @rwaidKalka https://github.com/rwaidKalka,
Can you please share your project's main.dart? You may also send it to us
privately at @.***
—
Reply to this email directly, view it on GitHub
https://github.com/pushy/pushy-flutter/issues/53#issuecomment-1579940412,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQS3US62VVCHENXVSY77LXTXKAINFANCNFSM6AAAAAAY2E7VLQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @rwaidKalka,
Thanks for sending over your code.

As per the Flutter docs, please ensure the following method is inside your main.dart file (remove it from app.dart), right after the import statements, and outside any Widget class declaration, to process push notifications in the background via a Flutter background isolate:
// Please place this code in main.dart,
// After the import statements, and outside any Widget class (top-level)

@pragma('vm:entry-point')
void backgroundNotificationListener(Map<String, dynamic> data) {
    // Print notification payload data
    print('Received notification: $data');

    // Notification title
    String notificationTitle = 'MyApp';

    // Attempt to extract the "message" property from the payload: {"message":"Hello World!"}
    String notificationText = data['message'] ?? 'Hello World!';

    // Android: Displays a system notification
    // iOS: Displays an alert dialog
    Pushy.notify(notificationTitle, notificationText, data);

    // Clear iOS app badge number
    Pushy.clearBadge();
}

For future readers, it seems I was able to setup the entry point as a static method in an arbitrary class:
class MyClass {
  @pragma('vm:entry-point')
  static void _backgroundNotificationListener(Map<String, dynamic> data) {
    ...
   }
  void init() {
      Pushy.listen();
      Pushy.setNotificationListener(_backgroundNotificationListener);
      ...
  }
}

Seems flutter documentation allows it, and the package awesome_notifications does it this way too
https://pub.dev/documentation/flutter_isolate/latest/


Settlement report amendments

Features

Minor changes to settlement report

Delivers #121503853

✋

Please address with Woj the patches from scrutinizer basket should run clean no issues/ patches etc...

@sped2773 Changes done


DecodeURI on dbname from connection string

Addresses issue described here https://github.com/sequelize/cli/issues/545

Did you want to make a PR to the upstream repo instead?

I have what is probably a better solution that I will do a proper PR for tomorrow :D after I've written tests ect. https://github.com/Xanderite/cli/commit/842a42842ac2bff2da6cf3595fef77e658f58b7d
This branch https://github.com/Xanderite/cli/commits/build/3.0.0 includes the built files if you need to use the package sooner than later, which is what I'm temporarily doing before this gets merged upstream.


Create a proof of concept for onboarding and detecting new releases

Create a simple project* that would produce a release artifact.  Make a 1.0.0 release, a 1.1.0 release, and a 1.1.1 release. Following the notions of what has been described in the overall release process.  Afterward determine sharp edges that can be improved.
*: language of your choice
Acceptance Criteria:

[x] Create the test release project
[x] Create the workflow
[ ] Use the workflow to create version 1.0.0
[ ] Repeat for a minor release, 1.1.0
[ ] Repeat finally for a patch releases, 1.1.1
[ ] Document sharp edges in this comment
[ ] Create follow up issues before rolling out.
[ ] Create final list of features that are needed to roll out the universal release process on https://github.com/opensearch-project/opensearch-build/issues/1234

I've got a basic workflow, https://github.com/peternied/data-prepper/pull/new/uni-poc still need to make sure there is glue on the other side and confirm the jenkins connection.
I've created a fresh CI instance to make sure I'm not depending on our existing logic following https://github.com/opensearch-project/opensearch-ci dev deployment

I have created a proof of concept workflow tested within a jenkins instance:

Implementation for sign, promote, maven, docker, ecr for Data Prepper, PR
Implementation for centrally managed pipeline in OpenSeach-Build, PR
Workflow output, console log

Break down of the work needed to productize this flow:

Create the standard release pipeline + test helpers to mock out functionality
Confirm the shared boiler plate code for the library import, parameters, workflow, and any additional pipeline configuration
Create publishToMaven function
Create publishToArtifactRepositry function
Work backwards on what the distribution manifest should include that is missing, or create additional parameters in the job
Create a DistManifest class
Update Dist manifest in python to follow the new schema

Updated the recent associated PR to include the potential manifest changes, and reviewed with Data Prepper team.


Add a pycall!() macro

This is a draft. The code isn't the most readable in the worlds and it likely contains bugs. It also won't pass CI, as I coded for most recent Python and Rust versions. I am opening this PR to gauge opinions.
I chose the performance over code safety or conciseness. This choice can be reverted, fully or partially. While this PR is very large, I believe we can get into reasonable size if we are ready to give up some of the performance.
The whole new pycall module is a safety boundary. Things aren't marked unsafe even when they should be. Part of this could be fixed, but part is hard because of macro constraints. Users should not be able to use the stuff defined in this module anyway.
This macro allows you to call Python objects with the most convenient syntax and maximum performance.
The implementation is... complicated. The performance requirements mean that we do a lot of type juggling, including clever tricks.
Fixes #4414.

It looks like there are some stylistic choices like "named_argument"=value rather than named_argument=value, and also (*)args and (**)kwargs. Can you elaborate on the choices here: are they draft syntaxes, are they due to limitations in macro parsing, or are they to prevent ambiguities? I think what I'd like to see is an exact mirror of Python call syntax, but we also have the problem of * being the deref operator in Rust, which creates at least one complication.

The syntax is named_argument=value, just like in Python. There is another alternative ("named_argument")=value (which is less efficient), for when you want to provide a single argument whose name is only known at runtime (that is, the expression within the parentheses is evaluated at runtime instead of a single identifier string). This is essentially a shortcut for (**)("named_argument", value),) (i.e. unpacking a single-item tuple of key and value) that will also have the same performance. Of course, we can drop this option if we like.
The syntax for unpacking include parentheses because of the deref ambiguity you mentioned. There are alternatives; for example, we can say if you want to deref you have to parenthesize the value (i.e. (*value)) but I see that as confusing. But in general, the syntax is kind of a placeholder; I focused more on the implementation (since this was more fun).

As per the thread in RFC: py_call! macro #4414 there are maintainers among us who rightly question whether a macro is necessary. Can we use builder functions to achieve the same effect (or indeed just regular functions)? If so, how ergonomic can we make those? My intuition is that because kwargs need to be appended to the args slice, and kwarg names separated out into a tuple, the function forms are non-trivial / high overhead. But it would be good to rule that out rather than proceed based on my hunch.

A builder is possible, but:

It cannot provide the same level of convenience, and
It cannot provide the same level of efficiency. The macro relies on being a macro for two things: validating that keyword argument names are disjoint (when all are known at compile time), and autoderef specialization to choose the most performant implementation. The latter can be replaced by multiple functions (but this will be both less convenient and risk people will accidentally choose the less efficient implementation), but the former cannot. Which means passing kwargs with all-knwon-at-compile-time names will have to be penalized.

The number of combinations of helpers are enormous. Do you think there are natural points where this PR could be broken up so we could review in chunks and possibly extend the design incrementally? We could potentially merge a limited implementation behind an experimental-pycall-macro feature, for example, and extend functionality as we go through the PRs.

Yes. We can provide only the most general combinations (which are unpacking with any iterator or any python iterable/mapping), and provide more efficient specializations as we go. From convenience POV, this will be the same as the last iteration, while from performance POV we can add things progressively. There is a question of how much to "predict the future" (i.e. whether to add things to a PR when they will only be needed for future combinators), but I guess we can deal with that in time.
As a last thing, I was noted on Stack Overflow that some of my work is useless ;P While I hope it will become useful with future enhancements to Python, it currently cannot provide any speedup and maybe even slow things down a bit. So we'll have to decide what to do with that too.

Wow, this is a huge amount of work... and a huge PR. :+1:
I for am +1 for a pycall! macro with nice syntax and potential efficiency, but I agree that it should be scaled down initially, and then expanded over time depending on the review bandwidth and user feedback.
As for syntax, I think this hits the sweet spot of being close enough to Python syntax, while resolving the unfortunate clash with * in a fine way.
It it was up to me, I would probably make one change - add a sigil like = to the "computed names" in brackets. I.e. name=value or (=name)=value and the same for obj.method() and obj.(=method)() - to make it obvious that something is being computed here. (name) and name just are a bit too similar, and it might be hard to remember at a glance that the parens are significant.

FWIW, the linked future enhancements to Python discussion is more about consistent naming for the existing CPython call API, than about performance or behaviour.
If there's some calling style that CPython could add to make calls from Rust faster, let's try to add it now :)

@encukou From what I understand it's also about allowing you to call with tuple and dict without a copy in case you are already holding one, and not providing vectorcall where it's slower (https://github.com/python/cpython/issues/123372). Which are the things I care about.

Can the "performance" talk please be backed up by benchmarks? It's hard to judge if there actually is any performance benefit or, for that matter, whether the extra performance is worth the added complexity[^1]...
From where I am standing the current complexity[^1] of this PR is not worth any extra performance. Even if you managed to pare it down to ~2k LOC it would not just have to be faster, it would have to be significantly faster to justify the added complexity[^1].
But back to the PR; I think this PR is not the best way forward:

it's huge, and looks like it took a lot of effort. We definitely appreciate your enthusiasm (👍 ) but big PRs out of nowhere are bad for several reasons, mostly because they're hard to review and require a big time/effort investment on your part.
it looks like there's some kind of arrayvec impl in here; please just use that instead of reinventing the wheel.

Instead,I'd prefer the following approach:

create a set of benchmarks of realistic scenarios where you can get a significant benefit by using pyo3_ffi api rather than PyAnyMethods. If there is a big difference then we'd like to see improvements, and it would motivate more work. If the difference is small or non existent, you've just saved yourself a lot of work.
change or add to PyAnyMethods to make it better.
if the above api gets big and/or confusing, add a macro to forward calls to it

[^1]: I touched on this slightly in https://github.com/PyO3/pyo3/issues/4414#issuecomment-2267158235 . With "complexity", I don't just mean "this PR is a pain to review". Complexity also involves: Can someone (who is not you) confidently make changes or improvements to it? How easy is it to understand as a user of the library? How much documentation is required? We happen to have a lot of documentation, so adding even more should not be done lightly.


Activity indicator disappears on scroll

Hello,
Thank you for your last update with the activity indicator, it's very useful!
I faced an issue with the loader: if I open a FullScreenSlideshowViewController, the loader appears correctly, but if I scroll to the next picture and then I scroll back immediately to the previous one, the loader is gone.
Is it a bug?
Thanks!
– Thomas

I'm also experiencing this issue. It also seems that when a picture finishes loading, other indicators are hidden.
E.g. I scroll to a picture, and immediately scroll to the next one. When the first picture finished loading the second indicator is hidden but the picture is still loading.

Hi, thanks for the report and sorry for problems. I've attempted to fix the issue in b83bdd9, cloud you apply changes in this commit to your project (or let the CocoaPods get the code from the master branch) to see if it helps?

Yep, it solves the problem in my case.

Me too! Thx! :)

Thanks, guys. I released a new version 1.3.1 with the fix.


Redundant work when using both io::write::FileWriter and io::flight::serialize_* methods.

With my limited understanding (correct me if I'm wrong), apart from their different headers, I'd say that both the Feather file format and the Flight FlightData messages are identical.
In my application, I'm creating a Feather file and the corresponding FlightData messages, both using IPC Zstd compression.
If it is true that the data content is identical, it seems rather wasteful to compress twice the same data. Is there a way to write a Feather chunk from the already-compressed corresponding FlightData messages? If not, could we introduce such an API?

Hey @GPSnoopy ! Yes, it is not possible. Yes it should be possible. :)
I think we would need something like:

expose encode_chunk so that the user can use it instead of calling serialize_batch.
FileWriter::mut_dictionaries
add FileWriter::write_encoded that FileWriter::write would use, but users could also use to write an already encoded chunk.


Issues with QMK configurator when compiling Helix firmware.

When compiling firmware on QMK configurator, repeated errors occur with Helix builds. This prevents the firmware from being downloaded. The following is the error reported in the command window:
|
| .build/obj_helix_rev2_back_oled_default_583be4a/keyboards/helix/rev2/rev2.o: In function matrix_init_kb':  | /qmk_compiler/qmk_firmware/keyboards/helix/rev2/rev2.c:25: undefined reference to is_helix_master'
| collect2: error: ld returned 1 exit status
|
makek:300: .build/helix_rev2_back_oled_default_583be4a.elf] Error 1
make: *** akefile:579: helix/rev2/back/oled:default_583be4a] Error 1
ake finished with errors

Known issue.
If I only need to fix the Helix code, I can fix it right away, but unfortunately I need to change the qmk core and it will take some time.

Does this mean there is currently no way to flash a custom keymap?

There is but you have to build your keymap by hand in something like notepad++ or such then compile with msys2 or linux terminal.

There is but you have to build your keymap by hand in something like notepad++ or such then compile with msys2 or linux terminal.

I posted another thread above, but I found msys2 generated the same/similar errors. I was able to compile successfully on Linux, with the caveat being that using a .json file when compiling would not allow oled/back lighting to function, whereas building my own keymap did not.

I made a workaround for the compile error on QMK configurator.
Currently the following four targets are compilable.

helix/pico/qmk_conf
helix/rev2/qmk_conf
helix/rev3_4rows
helix/rev3_5rows


Istanbul function coverage for arrow function?

I have a module that I would like to have code coverage using Istanbul, but I cant get it working when I use arrow function.
my module compute.js
'use strict';

exports.addition = (a, b) => {
    return a + b;
}

exports.multiplication = (a, b) => {
    return a * b;
}

and my unit test code test.js
describe('Compute', function() {
    describe('addition()', function() {
        it('should add', function() {
            assert.equal(5, compute.addition(2, 3))
            assert.equal(15, compute.addition(2, 13))
        })
    })

    describe('multiplication()', function() {
        it('should multiply', function() {
            assert.equal(6, compute.multiplication(2, 3))
            assert.equal(26, compute.multiplication(2, 13))
        })
    })
})

When i run this command
node_modules/.bin/istanbul cover node_modules/mocha/bin/_mocha test/test.js

I get my coverage summary shown below:
================== Coverage summary =======================
Statements : 100% (4/4)
Branches   : 100% (0/0)
Functions  : 100% (0/0)
Lines      : 100% (4/4)

In my Function row, i am getting 0/0, if i change my compute.js to use function call.
exports.addition = function(a, b) {
    return a + b;
}

exports.multiplication = function(a, b) {
    return a * b;
}

now, i am getting the correct code coverage
================== Coverage summary =======================
Statements : 100% (4/4)
Branches   : 100% (0/0)
Functions  : 100% (2/2)
Lines      : 100% (4/4)

package.json
"istanbul": "^0.4.0",
"mocha": "^2.3.4"

May i know why the arrow function is not working and how can I fix them?

Looks like istanbul is not counting arrow functions as "functions". This should be easy to fix but I'm in he middle of a refactor so I can only get to it later.
Thanks for reporting this issue.

@sweetim I had same error.  I just forgot to remove describe.only from one of my tests! =))) I thought it may help you.

The issue still exists with<EMAIL_ADDRESS>Any update?

Istanbul's inability to cover arrow functions is causing me to do some frustrating refactoring. Do we have any updates on this?

You could use nyc package https://www.npmjs.com/package/nyc
this problem has been resolved with that package! ;)

@sweetim it should be a native function from Istanbul instead using another package.

We seem to be having this same problem with Jest here

Any updates?

Updates on this issue?

Still active

As stated in the readme, this version of Istanbul has been deprecated, 5 years ago already. Use https://github.com/istanbuljs/nyc
@gotwarlost why don't you put this repo into read-only mode?

As stated in the readme, this version of Istanbul has been deprecated, 5 years ago already. Use https://github.com/istanbuljs/nyc @gotwarlost why don't you put this repo into read-only mode?

Sorry!


Inconsistent Date Results on macOS vs. Linux for Swift Code Calculating Current Month Range

The code below produces different results when run on macOS and Linux.
On macOS, it returns 2024-10-01 for the first day of the month and 2024-10-31 for the last day.
However, on Linux, it returns 2024-09-30 for the first day and 2024-10-30 for the last day.
private func getCurrentMonthRange() -> (start: String, end: String) {
    let today = Date()
    let calendar = Calendar.current

    let firstDayComponents = calendar.dateComponents([.year, .month], from: today)
    let firstDay = calendar.date(from: firstDayComponents)!

    var lastDayComponents = DateComponents()
    lastDayComponents.month = 1
    lastDayComponents.day = -1
    let lastDay = calendar.date(byAdding: lastDayComponents, to: firstDay)!

    let dateFormatter = DateFormatter()
    dateFormatter.locale = Locale(identifier: "en_US_POSIX")
    dateFormatter.timeZone = TimeZone(secondsFromGMT: 0)
    dateFormatter.dateFormat = "yyyy-MM-dd"
    return (start: dateFormatter.string(from: firstDay), end: dateFormatter.string(from: lastDay))
}
let (start, end) = getCurrentMonthRange()
print("first day of month: \(start), last day of month: \(end)")

calendar.date(from:) converts the date component into a Date using the given calendar's time zone, so if Calendar.current.timeZone on your Linux machine and mac machine are different, firstDay would be different.
On top of that, I believe there's also a discrepancy that you're calculating a date from Calendar.current.timeZone, but later formatted into GMT. You'd want to use the same calendar and time zone for both calculation and formatting.
Since it looks like you want the eventual result to always be represented in GMT, I think it makes sense to use a fixed calendar instead of .current as well as setting its timeZone so everything is aligned. Something like this
let calendar = Calendar.current

With
var calendar = Calendar(identifier: .gregorian)
calendar.timeZone = .gmt

Or the other way around, i.e. setting dateFormatter's calendar and time zone to the one you used for calculation

calendar.date(from:) converts the date component into a Date using the given calendar's time zone, so if Calendar.current.timeZone on your Linux machine and mac machine are different, firstDay would be different.
On top of that, I believe there's also a discrepancy that you're calculating a date from Calendar.current.timeZone, but later formatted into GMT. You'd want to use the same calendar and time zone for both calculation and formatting.
Since it looks like you want the eventual result to always be represented in GMT, I think it makes sense to use a fixed calendar instead of .current as well as setting its timeZone so everything is aligned. Something like this
let calendar = Calendar.current

With
var calendar = Calendar(identifier: .gregorian)
calendar.timeZone = .gmt

Or the other way around, i.e. setting dateFormatter's calendar and time zone to the one you used for calculation

Thank you, for the reminder. There is a difference between the time zones in Linux and macOS, and I overlooked this issue.


ng attributes added to accordion-header needs to move to panel-heading

any ng attribute added to accordion-header:
Needs to move to the panel-heading when the cotent is moved

Explain why this needs to happen - what is your use case?

So i can use ng-click to trigger an angular function when the accordion
item's header is clicked on. We just need a function to run when an item is
expanded / collapsed
On Tue, Dec 1, 2015 at 2:49 PM Wesley Cho<EMAIL_ADDRESS>wrote:

Explain why this needs to happen - what is your use case?
—
Reply to this email directly or view it on GitHub
https://github.com/angular-ui/bootstrap/issues/4987#issuecomment-160988471
.

Why not watch for the value of the is-open property on the group?

Yes, could do. It still would be handy to have this functionality for other
use cases.
On Tue, Dec 1, 2015 at 2:58 PM Wesley Cho<EMAIL_ADDRESS>wrote:

Why not watch for the value of the is-open property on the group?
—
Reply to this email directly or view it on GitHub
https://github.com/angular-ui/bootstrap/issues/4987#issuecomment-160990663
.

This can be accomplished by creating your own accordion heading template, e.g:
<uib-accordion-heading>

      I can have markup, too!

</uib-accordion-heading>

why should you have to create your own? This is such a simple thing to add
to the default template
On Tue, Dec 1, 2015 at 4:34 PM Rob Jacobs<EMAIL_ADDRESS>wrote:

Closed #4987 https://github.com/angular-ui/bootstrap/issues/4987.
—
Reply to this email directly or view it on GitHub
https://github.com/angular-ui/bootstrap/issues/4987#event-479154689.

Adding support for features that can be extended by the user is probably the largest cause of bloat in libraries, which is problematic if not all users want a particular feature.

okay fair point. Thanks for your help guys anyway
On Tue, Dec 1, 2015 at 4:59 PM Wesley Cho<EMAIL_ADDRESS>wrote:

Adding support for features that can be extended by the user is probably
the largest cause of bloat in libraries, which is problematic if not all
users want a particular feature.
—
Reply to this email directly or view it on GitHub
https://github.com/angular-ui/bootstrap/issues/4987#issuecomment-161031076
.


[Doubt] AppendSlide

Hello, I'm using the swiper to be populated dynamically, the slides are being populated correctly, but when using the fraction type page, the popular one gets 1/0 and is only updated to the correct value when the user moves to the next slide.
Has anyone had this?
Follow my code below:
          .then(response => response.json())
          .then(resp => {
            if(resp.success == true) {
              let data = resp.data;
              let photos = [];

              data.filter( function( result, i, data ) {
                photos.push(that.galleryTemplate(result));
              });

              swiper.appendSlide(photos);

              swiper.update();
            }
          })
          .catch(err => {
            console.error(err);
        });

Can't replicate it. Do you have codepen or jsfiddle with the issue?

Closing due to inactivity and/or no responses from issue author


Compatible with Project v2

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Please wait the CLA until my Employer confirm it.

Please wait the CLA until my Employer confirm it.

No worries! I know it can take some time :D
Happy to answer any questions they may have.

Sorry, I'm late. I signed the CLA.

Sorry, I'm late. I signed the CLA.

No worries! Thanks for the PR!


PHP instanceof word

First of all it is not an issue. Just basic a question. Instanceof word isn't highlighted. What should I do?

I think this is question you should as in atom/language-php

Thank you, your feedback. I create new issue on atom/language-php


Refactoring: keyboard plugin code

This change is just refactoring.
Changes

Rename sorce file name: key_event_plugin_glfw_util.cc/h to keyboard_glfw_util.cc/h
Avoid #define
Use xkb_keymap_mod_get_index

@taka-okayama Will you review the PR?

Thanks.


Unable to install local package repositories on ubuntu 18.04

Issue description
After upgrading to ubuntu 18.04 which bumped the system python from 3.5 to 3.6, pipenv install would not load a local package from the file system, specified using the {file=} syntax.
Expected result
I changed only the python version number in the Pipfile, issued a pipenv --rm followed by a pipenv install.  I would have expected the private pp package to have re-installed.
Actual result
It looks like an additional . was placed in front of the file:/// specification which caused the pipenv to not be able to find the package location on the file system.
This is the output of the command:
production@ip-10-246-198-197:~/apps/joinconversion$ pipenv install
Creating a virtualenv for this project...
Pipfile: /opt/data/production/production-user/apps/joinconversion/Pipfile
Using /usr/bin/python3.6m (3.6.5) to create virtualenv...
⠋Running virtualenv with interpreter /usr/bin/python3.6m
Using base prefix '/usr'
New python executable in /home/production/.local/share/virtualenvs/joinconversion-1uvQApZz/bin/python3.6m
Also creating executable in /home/production/.local/share/virtualenvs/joinconversion-1uvQApZz/bin/python
Installing setuptools, pip, wheel...done.
Setting project for joinconversion-1uvQApZz to /opt/data/production/production-user/apps/joinconversion

Virtualenv location: /home/production/.local/share/virtualenvs/joinconversion-1uvQApZz
Pipfile.lock (74bb87) out of date, updating to (8774f8)...
Locking [dev-packages] dependencies...
Locking [packages] dependencies...
Traceback (most recent call last):
 File "/usr/local/bin/pipenv", line 11, in <module>
   sys.exit(cli())
 File "/opt/data/production/production-user/.local/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 722, in __call__
   return self.main(*args, **kwargs)
 File "/opt/data/production/production-user/.local/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 697, in main
   rv = self.invoke(ctx)
 File "/opt/data/production/production-user/.local/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 1066, in invoke
   return _process_result(sub_ctx.command.invoke(sub_ctx))
 File "/opt/data/production/production-user/.local/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 895, in invoke
   return ctx.invoke(self.callback, **ctx.params)
 File "/opt/data/production/production-user/.local/lib/python3.6/site-packages/pipenv/vendor/click/core.py", line 535, in invoke
   return callback(*args, **kwargs)
 File "/home/production/.local/lib/python3.6/site-packages/pipenv/cli.py", line 435, in install
   selective_upgrade=selective_upgrade,
 File "/home/production/.local/lib/python3.6/site-packages/pipenv/core.py", line 1943, in do_install
   pypi_mirror=pypi_mirror,
 File "/home/production/.local/lib/python3.6/site-packages/pipenv/core.py", line 1285, in do_init
   do_lock(system=system, pre=pre, keep_outdated=keep_outdated, write=True, pypi_mirror=pypi_mirror)
 File "/home/production/.local/lib/python3.6/site-packages/pipenv/core.py", line 1057, in do_lock
   settings['packages'], project, r=False, include_index=True
 File "/home/production/.local/lib/python3.6/site-packages/pipenv/utils.py", line 513, in convert_deps_to_pip
   new_dep = Requirement.from_pipfile(dep_name, dep)
 File "/home/production/.local/lib/python3.6/site-packages/pipenv/vendor/requirementslib/models/requirements.py", line 764, in from_pipfile
   r = FileRequirement.from_pipfile(name, pipfile)
 File "/home/production/.local/lib/python3.6/site-packages/pipenv/vendor/requirementslib/models/requirements.py", line 394, in from_pipfile
   return cls(**arg_dict)
 File "<attrs generated init 0936e6bebbaa40af1d505786c59b44fbe27de130>", line 23, in __init__
 File "/opt/data/production/production-user/.local/lib/python3.6/site-packages/pipenv/vendor/attr/validators.py", line 106, in __call__
   self.validator(inst, attr, value)
 File "/home/production/.local/lib/python3.6/site-packages/pipenv/vendor/requirementslib/models/utils.py", line 123, in validate_path
   raise ValueError("Invalid path {0!r}", format(value))
ValueError: ('Invalid path {0!r}', './file:/opt/share/python/packages/pp/pp-0.4.1.tar.gz')

Steps to replicate
Provide the steps to replicate (which usually at least includes the commands and the Pipfile).

Upgraded from Ubuntu 16.04 to 18.04
Installed pipenv manually using pip install pipenv
editted the Pipfile to adjust environment to 3.6

[[source]]
name = "pypi"
verify_ssl = true
url = "https://pypi.python.org/simple"
[packages]
pp = {version="*", path="file:///opt/share/python/packages/pp/pp-0.4.1.tar.gz" }
joinconversion = {version="*", path="file:///opt/share/python/packages/joinconversion/joinconversion-0.2.tar.gz" }
simple-salesforce = "*"
numpy = "*"
ipython = "*"
pandas = "*"
sqlalchemy-redshift = "*"
sqlalchemy = "*"
"psycopg2-binary" = "*"

[dev-packages]

[requires]
python_version = "3.6"

Run pipenv install

Please run $ pipenv --support, and paste the results here. Don't put backticks (`) around it! The output already contains Markdown formatting.
If you're on macOS, run the following:
$ pipenv --support | pbcopy

If you're on Windows, run the following:
> pipenv --support | clip

If you're on Linux, run the following:
$ pipenv --support | xclip

path is for specifying a local file path. If you want to specify a URL, use file instead. Read the Pipfile spec for examples. https://github.com/pypa/pipfile#examples-spec-v6


[Docs] Best practices for viewing alignments

I've been exploring igv.js along with igv-notebook as a tool to explore our data. It seems like it could be a great option for interactive analysis that we can also share with collaborators.
As I get started, I wanted to check that I'm doing everything I can to make it responsive. I'd like to build a tool for exploring an aligned dataset and visualizing the evidence for different isoforms, etc. If there are particular ways to prepare my BAM file to make igv.js work better, that'd be great to know.
I was looking at the documentation for alignments and saw the section on sorting, but I wasn't clear on what it did. Is that config for re-sorting the track for display, or is it a hint about how the BAM is sorted, or something else? Does it make a difference to performance?
I'm focused on viewing BAMs that are on the local machine, via a jupyter notebook. I was considering whether there's anything I could do on the "server" side to make loading data faster. i.e. perhaps I could pre-filter the BAM file to a certain region based on the user's analysis, before passing that to the viewer. I would want to avoid writing that to disk, though, so it would require some way to pass an in-memory object to the viewer.

Sure:
igv_browser = igv_notebook.Browser(
    {
        "genome": "hg38",
        "locus": "chr1:198,683,443-198,713,056",
        "tracks": [{
            "name": "BAM",
            "url": "/ptprc.bam",
            "indexURL": "/ptprc.bam.bai",
            "format": "bam",
            "type": "alignment",
            "height": "900",
        }],
    }
)

This is a little slower than I would expect, and will look further into it.  Its not loading the bam data though, that is fast, it seems to be with the rendering.    I will use this dataset to look for optimizations, its a bit  unusual, this is the first long-read transcriptome data I've seen so there might be something particular about it that is not optimized for.  In any event there is nothing you are doing wrong.
Leave the issue open for now,  I am optimistic this can be sped up.
The 4-5 seconds is a combination of loading the page itself + the bam rendering.  To time individual parts I split it into 3 cells as follows.
import igv_notebook

igv_notebook.init()
igv_browser = igv_notebook.Browser(
    {
        "genome": "hg38",
    }
)

igv_browser.search("chr1:198,683,443-198,713,050")

igv_browser.load_track({
            "name": "BAM",
            "url": "data/bamsample/ptprc.bam",
            "indexURL": "data/bamsample/ptprc.bam.bai",
            "format": "bam",
            "type": "alignment",
            "height": "900"
        })


fix(cloudrequestlog): handle nil-result of Check correctly

If the (*Logger).Check method returns nil, it means that the entry
should not get logged.

Looks even better now


Faas templates

Add Functions examples for nodeja and python packageing additional modules based on the docs
https://cloud.ibm.com/docs/codeengine?topic=codeengine-fun-create-repo#fun-package-repo

@reggeenr A review would be appreciated :)

I suggest to move this into helloworld-examples/function-nodejs and helloworld-examples/function-python

@reggeenr @kersten1 added your suggestions :)


Adds reminder functionality for collection.NotionDate

This allows reminders to be set programmatically. Demo

Thanks for the PR, and sorry for the slow response!
Looks like this would also be covered by https://github.com/jamalex/notion-py/pull/49 -- do you (or others) have a strong preference of the class-based vs raw-dict-based approach for this? The library has tended to favor class-based, but this does look to be a simple enough data structure that it may be overkill.

Thanks, works for me, using like:
row.date = NotionDate(start=today + datetime.timedelta(days=1), reminder={'time': "9:00", 'unit': "day", 'value': 0})

Can you please provide another example for its usage ? The one mentioned here didn't work for me.

Thanks! Has now been merged in via #239, along with some additional fixes.

Thanks! Has now been merged in via #239, along with some additional fixes.


PHI 1.5 - Realistic model size with incredible performance

🚀 Feature
To support microsoft phi 1.5 : https://huggingface.co/microsoft/phi-1_5
a big mistake would be to consider this as not important, like its completly made for small devices.
Motivation
Small model size on small device is the way to go
Alternatives
None
Additional context
with 1.3 billion parameters (~GPT2) it should run at incredible speed on any small device , and since its benchmark as powerfull as certain 13B models, it is incredibly interesting to implement it for small devices !!

Duplicate of #905.


FEATURE REQUEST : Add TextAlign property to control how the text should be place inside the button

https://github.com/Frezyx/group_button/pull/46


install plugin

installPlugin: (repo: string, manifest: PluginManifest, version: string) => Promise;
the 2 last parameters are inverted
(repo: string, version: string, manifest: PluginManifest)

Sorry for being so late with this, thank you for reporting the issue!


in_tail: Watch only recently modified files

Would you consider adding an option like limit_recent_modified :time ?
The purpose of this option is to solve the following:

I want to skip logs of expiration when Fluentd stops for a long time due to failure or maintenance.

In addition, suppress unnecessary watcher creating to save resources.

ex.
     def expand_paths
       date = Time.now
       paths = []

       excluded = @exclude_path.map { |path| path = date.strftime(path); path.include?('*') ? Dir.glob(path) : pat\
h }.flatten.uniq
       @paths.each { |path|
         path = date.strftime(path)
         if path.include?('*')
           paths += Dir.glob(path).select { |p|
             if File.readable?(p) && !File.directory?(p)
-              true
+              if @limit_recent_modified and File.mtime(p) < (date - @limit_recent_modified)
+                false
+              else
+                true
+              end
             else

It seems good idea.


Region.Op.REPLACE is deprecated in API level 28

When trying to use a clip path on an element, the use of Region.Op.REPLACE is causing an exception Invalid Region.Op - only INTERSECT and DIFFERENCE are allowed. This is due to REPLACE being deprecated as per https://developer.android.com/reference/android/graphics/Canvas.html#clipPath(android.graphics.Path, android.graphics.Region.Op)
I would love to PR but I'm not clear on how to resolve this to be honest.

If you need a quick fix then you can just remove the second argument: https://github.com/msand/react-native-svg/commit/a1097b85942c559ebcef6b93fa2ce601434d9c50#diff-633cc935907e7685222b28cf9a0d7696L268
That's more spec conformant anyway.

@msand Great stuff - this appears to work quite well. Will leave open until this is included upstream.