hass-gs-alarm is not recording violation logs

My G90B alarm triggers in zones that have been violated and the time is not reported in Home Assistant and I don't know how to have a record of the day and time of violation of the zones.

Do you have the alarm panel changing the state to Triggered when alarm fires?
Do you see corresponding event in integration event: Settings -> Devices & Services -> Golden Security Alarm -> gs_alarm (or whatever name it has in your setup) -> Logbook?
Any errors/exceptions logged in HASS logs (Settings -> System -> Logs, we are looking for anything related to pyg90alarm or gs_alarm)
Do you have your alarm panel configured with IP <IP_ADDRESS> on your WiFi network (https://github.com/hostcc/hass-gs-alarm?tab=readme-ov-file#description)? That'd make a difference if the integration uses events from panel or does polling instead

Just in case it'd be influential - what is the model/firmware of your device? You could look that up in Device Info section of the device (Settings -> Devices & Services -> Golden Security Alarm -> gs_alarm, or whatever name it has in your setup)?

Does the alarm panel change the Triggered state when the alarm goes off?
Yes, it does;
Do you see the corresponding event in the integration event: Settings-> Devices & Services-> Golden Security Alarm-> gs_alarm (or whatever name it has in your configuration) -> Logbook?
No, I don't, I can only see it in the Android mobile app "Ennio Security 2.0";
Any errors/exceptions recorded in the HASS logs (Settings-> System-> Logs, we are looking for anything related to pyg90alarm or gs_alarm)
I didn't find the HACS log;
Do you have your alarm panel configured with IP <IP_ADDRESS> on your WiFi network ( https://github.com/hostcc/hass-gs-alarm?tab=readme-ov-file#description )? It would make a difference if the integration used dashboard events or polling instead.
No, it has a fixed IP on my local network which is <IP_ADDRESS> so my network is <IP_ADDRESS>;
Just in case it's important - what is the model/firmware of your device? Can you look this up in the Device Info section of the device (Settings->Devices & Services->Golden Security Alarm->gs_alarm, or whatever it's called in your configuration)?
For Golden Security it's 0.0.0 and for TSV018-C3SIA it's Firmware: MCU: 207, WiFi: 207

Interesting thanks @Gaio-Christiano ! I use mine with <IP_ADDRESS> IP address so it delivers notifications to HASS instead of polling (as in your case). I'll try to simulate and see if anything is broken in that scenario.

In my opinion, it would communicate with any IP it receives from the local network.
In this time period I had several door openings, zones that had alarm triggers and no triggers were registered in Home Assistant.
So I don't know what's wrong or if there is a script to make the records.

@Gaio-Christiano , I've looked over the details captured during initial implementation - unfortunately, without notifications from the device no changes to sensors' state will be reflected. Simply put, sensors will always remain in inactive state (doors always closed etc.) and no details will be available when alarm fires (e.g. there will be no information on what sensor triggered that). Unfortunately, that is the limitation of the device and its firmware - I found no method of polling for sensor state, those are only delivered via notifications the device sends.
Additional side effect of the limitation above - changes in panel's state are reflected with somewhat a minute delay (basically, HomeAssistant's refresh interval for sensors). It means the alarm will only be noticed by HomeAssistant in a minute or so, though naturally most of us would stop it sooner, hence the HomeAssistant won't even notice the condition.
This limitation is mentioned in the https://github.com/hostcc/hass-gs-alarm/blob/master/README.md, along with <IP_ADDRESS> IP address trick. It might be possible that panel still sends the notifications even if configured for another address, though my experimentation didn't confirm that - to verify you'd need to ensure the HomeAssistant could receive broadcast network packets from the panel. That means no intermediate routers between those two, including a Docker network except host one.
For my setup I just decided to change my WiFi address and had panel set to <IP_ADDRESS>. Depending on your network setup it might be possible to add another segment (typically via a VLAN) with above subnet and configure DHCP reservation for the panel. However, that'd be an advanced scenario, since you'll have to expose HomeAssistant to such additional segment as well, which highly depends on how your HomeAssistant setup looks like.

@Gaio-Christiano , also, I'd check if door open/close alerts are enabled in mobile application - options screen, Alarm Alert (at least that is how it is named in Carener 2 application I'm using). Potentially those might have a positive effect.

Yes, in the mobile application, as previously reported, everything works OK and shows me the trigger records in up to 50 records, if I remember correctly.
Now in Home Assistant, it does not record any trigger records and only records activation and deactivation records.

@Gaio-Christiano , I have positive news - I've released 1.14.1 version that supports simulating device alerts from history. It helps to workaround inability to receive device alerts via network broadcasts or device not having <IP_ADDRESS> IP Address.
Just update the integration, enable Simulate device alerts from history in the device configuration in Home Assistant (Settings -> Devices & Services -> Golden Security Alarm -> gs_alarm (or whatever it is named in your setup) -> Configure), and enable alerts of interest in mobile application.
Please see more details in the documentation
I'd be very interested to hear on results, please

Good afternoon!
It's still the same, it doesn't record the date and time of the trigger or inform where it heard the trigger, it only records the information when it is triggered or deactivated.
Only in the mobile application is this information recorded in only 100 records.

Yes, I enabled it and it's still the same, I updated it but the version appears to me as 0.0.0

Yes, I enabled it and it's still the same, I updated it but the version appears to me as 0.0.0

Thanks, @Gaio-Christiano - 0.0.0 is fine (I have not yet updated it for a current one).
Did you enable relevant alerts in mobile application, like door close or door open? Alarms if triggered should be logger anyways, but without those settings door sensors' activity won't be reflected.
Finally, any errors you might see in Settings -> System -> Logs -> Load full logs, related to gs_alarm or pyg90alarm, please?

No, I didn't set up those closed and open door things.
It was all automatic.
3 errors appeared in the log and I'm loading them

@Gaio-Christiano , sorry for not being clear - I was speaking re: alert types enabled for the panel, not sensor.
This is how it looks in my mobile app (the screen below is activated by swiping right from the central one):

And then alert configuration:

good morning!
This part on my cell phone is all marked and I only use the G90B in WiFi mode, it has no GSM chip or 4g

Thanks, @Gaio-Christiano ! If you'd update to version 1.15.1 or later it adds support for diagnostic information - you should be able to do that via Download diagnostics on device page. Once completed would be great if you'd attach the resulting file here - it has all sensitive information removed (such as sensor names and alike), so it should be safe to share.

aí está meu arquivo de log gerado pelo registro foi do dia 22/08/2024 para o dia de hoje 23/08/2024.
Continuo sem ter os registros de disparos das zonas
home-assistant_gs_alarm_2024-08-23T12-52-58.614Z.zip

Good morning!
I'm sorry, I sent it in my native language
Here is my log file generated by the registry from 08/22/2024 to today 08/23/2024.
I still don't have the zone trigger records

Good morning! I'm sorry, I sent it in my native language
Here is my log file generated by the registry from 08/22/2024 to today 08/23/2024.
I still don't have the zone trigger records

Absolutely no worries, @Gaio-Christiano ! The log is very useful, though it is system log with debuglevel (not diagnostic information for the alarm panel), hence it contains non-redacted data - please remove the attachment from the issue, just in case.

I hope the log helps to find the fault and improve the system.
I have already removed the sent log.

@Gaio-Christiano , I've reviewed the log you've provided and found the component functioned as expected - it properly generated changes to alarm panel in HomeAssistant. For example:
Panel was armed away:
2024-08-22 20:13:15.097 DEBUG (MainThread) [custom_components.gs_alarm.alarm_control_panel] Received arm/disarm callback: 1

Panel triggered alarm from sensor `xxx 04 movimento (I've obfuscated the real sensor name):
2024-08-23 09:52:21.316 DEBUG (MainThread) [custom_components.gs_alarm.alarm_control_panel] Received alarm callback: xxx 04 movimento (idx=103), entity id: binary_sensor.xxx_04_movimento

The Received ... callback message is logged right where the component tells HomeAssistant the panel's state has changed.
There were no door open/close alerts in the history, so sensor changes weren't generated.
To understand why panel didn't send those, could you please do Download diagnostics this time, please?

gs_alarm-01J3MTR2EM9RDCWPN6YABXJX61-GA1801A9Y000941-c8b87053876ff397e496baf23b8244ad.json
gs_alarm-01J3MTR2EM9RDCWPN6YABXJX61-gs_alarm_GA1801A9Y000941-90d54eab916ea6fd693fc043d85c741c.json
gs_alarm-01J3MTR2EM9RDCWPN6YABXJX61-gs_alarm_GA1801A9Y000941-e28cd862c08838895d9d29f209971d65.json
Estão aí

gs_alarm-01J3MTR2EM9RDCWPN6YABXJX61-GA1801A9Y000941-c8b87053876ff397e496baf23b8244ad.json
gs_alarm-01J3MTR2EM9RDCWPN6YABXJX61-gs_alarm_GA1801A9Y000941-90d54eab916ea6fd693fc043d85c741c.json
gs_alarm-01J3MTR2EM9RDCWPN6YABXJX61-gs_alarm_GA1801A9Y000941-e28cd862c08838895d9d29f209971d65.json
Estão aí

gs_alarm-01J3MTR2EM9RDCWPN6YABXJX61-GA1801A9Y000941-c8b87053876ff397e496baf23b8244ad.json
gs_alarm-01J3MTR2EM9RDCWPN6YABXJX61-gs_alarm_GA1801A9Y000941-90d54eab916ea6fd693fc043d85c741c.json
gs_alarm-01J3MTR2EM9RDCWPN6YABXJX61-gs_alarm_GA1801A9Y000941-e28cd862c08838895d9d29f209971d65.json
Estão aí

Good afternoon!
I changed the IP of my network and things got worse, where Home Assistant can no longer connect to the G90B alarm

eu coloquei o MAC dele no DHCP Address Reservation e reservei o IP <IP_ADDRESS> para o alarme G90B e agora ele não é reconhecido no home assistant e ele fica lento no celular

I returned Home Assistant and the G90B alarm to the old network with IP <IP_ADDRESS> and it started working again.
In the new network that I created with IP <IP_ADDRESS> it did not work in Home Assistant.

I put its MAC in DHCP Address Reservation and reserved the IP <IP_ADDRESS> for the G90B alarm and now it is not recognized in Home Assistant and it is slow on the cell phone

While it might be out of scope for the current issue I do remember the panel starts to timeout on WiFi if it has no Internet connection - the 10.10.10.x subnet might be lacking NAT or alike enabled in your network configuration. Alternatively, HomeAssistant not able connecting to the panel over <IP_ADDRESS> might indicate missing routes between 192.168.0.x and 10.10.10.x subnets, or a firewall blocking connections from 192.168.0.x to 10.10.10.x
Anyways, 1.17.0 should give you al least sensors state change during the alarm condition.

No, the Home Assistant had IP <IP_ADDRESS>, the modem had IP <IP_ADDRESS> and the G90B had IP <IP_ADDRESS>.
I changed the IP range of the modem from the old NET to the current Claro from Arris and then reverted to the old configuration with the backup.

@Gaio-Christiano , network related aspects might be tricky to resolve sometimes.
Anyways, let's capture what is going on - please do the following:

Update the component to 1.17.0 is not done already
I noticed redundant device for the component (due to some changes in previous versions) - please delete both and re-add the integration setting up the panel. Existing automations (if any) for the panel should remain intact (I did same for mine and it worked)
Ensure Simulate device alerts from history configuration option is enabled in the configuration of the panel in Home Assistant
Please do Enable debug logging on the device in Home Assistant, trigger the alarm and activate wireless door sensor a few times, then disable debug logging
Please do Download diagnostics on the device in Home Assistant
Then please attach both files here, I'll have look

Thanks in advance!

Good afternoon!
1 - Yes, the component was already in version 1.17.0, as can be seen in the image above;
2 - Since it was in a new IP network <IP_ADDRESS>, I had to add a new one and left the old one previously added with the network <IP_ADDRESS>;
3 - Yes, this option was enabled;
4 - I will do this on Sunday when I get home;
5 - I will do this on Sunday when I get home;

Hi, @Gaio-Christiano - could you please share the latest on the issue? Is that still affecting your setup, please?

Good morning!
I haven't updated to this new version yet.
The previous one is working fine without any problems and I created some routines in Home Assistant to turn off the alarm in less time and reset it.
So I was also able to implement it by having Alexa announce the location of the alarm, send a message to Telegram and WhatsApp telling the location of the alarm.
I also programmed it to turn on two RGB lamps and depending on the color I already know where it went off in my house.
I closed it because I was afraid of updating and it stopping working lol, an update came out on Alexa that stopped Home Assistant from opening.

Good morning!
I haven't updated to this new version yet.
The previous one is working fine without any problems and I created some routines in Home Assistant to turn off the alarm in less time and reset it.
So I was also able to implement it by having Alexa announce the location of the alarm, send a message to Telegram and WhatsApp telling the location of the alarm.
I also programmed it to turn on two RGB lamps and depending on the color I already know where it went off in my house.
I was afraid of updating and it would stop working lol, an update came out for Alexa that stopped Home Assistant from opening.

Thanks, @Gaio-Christiano! Happy to hear the integration is working for you - I think there were no breaking changes between 1.17.0 and 1.19.1, so you might try updating or keep the older version, whatever works best for you.
Unless there is anything pending for the issue I'm going to close it - thanks for great cooperation!


chore: remove console.log

Thanks!

Hi, can you publish a new NPM package with this fix ?
Thanks!

Done @pqml, thanks for the heads up

Thanks a lot!


Upgrade CoreDNS to v1.6.3

For https://github.com/rancher/k3s/pull/743#issuecomment-528312510

LGTM


Novelbin is not displaying chapters for Immortality Cultivation: I Farm in the System Space

Source information
Novel Bin
Steps to reproduce
Got to Immortality Cultivation: I Farm in the System Space in app with sorce of Novel Bin. Click on any chapter from the last 3 days and you will not see the whole chapter. If you click on web view the full chapter will display.
Expected behavior
Expect to see the full chapter displayed
Actual behavior
See the report to GitHub message
LNReader version
LNReader v2.0.0-beta.2
Android version
13
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 1.1.19.
[X] I have updated all installed extensions.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

I am also having the same issue.
Some of the chapters in some novels are not displayed.

I am also facing the same issue


Link to doc is broken

It's totaly minor but the link to your doxygen documentation is broken, there's only a 1.6.1 version on http://argsparse.livna.org

Sorry about that. :) It's fixed

http://argsparse.livna.org is broken again

Fixed again. :p

Thank you so much sir !


ion-checkbox: binding with ngModel doesn't set checkbox on initial loading

Bug Report
Ionic Info
Ionic:

   ionic (Ionic CLI)             : 4.3.1 (C:\Users\dst\AppData\Roaming\npm\node_modules\ionic)
   Ionic Framework               : @ionic/angular 4.0.0-beta.15
   @angular-devkit/build-angular : 0.10.3
   @angular-devkit/schematics    : 7.0.3
   @angular/cli                  : 7.0.3
   @ionic/angular-toolkit        : 1.0.0

Cordova:

   cordova (Cordova CLI) : not installed
   Cordova Platforms     : not available
   Cordova Plugins       : not available

System:

   NodeJS : v10.13.0 (C:\Program Files\nodejs\node.exe)
   npm    : 6.4.1
   OS     : Windows 10

Describe the Bug
When I bind a value / model to a ion-checkbox using [(ngModel)] the initial value is not set. So if the value is true when the control gets loaded, the checkbox is not checked. I have to add [checked]="value" in order to make that work. In my opinion, the checked value should be set automatically when ngModel value is set to true initially.
Related Code
// when myModel.hasAdditionalAddress is true when the form loads, the checkbox is not checked. After toggling once, it works normally.
<ion-checkbox color="secondary" [(ngModel)]="myModel.hasAdditionalAddress">
// it only works, when checked is set to myModel.hasAdditionalAddress
<ion-checkbox color="secondary" [(ngModel)]="myModel.hasAdditionalAddress" [checked]="myModel.hasAdditionalAddress">

I have a sample app where the [ngModel] binding works fine for ion-toggle and ion-checkbox with beta.15 but does not work at all with beta.16.
https://github.com/kensodemann/test-toggle
Probably tied to this change: https://github.com/ionic-team/ionic/commit/bfbbeca
Here is my ion info:
Ionic:

   ionic (Ionic CLI)             : 4.4.0 (/usr/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 4.0.0-beta.16
   @angular-devkit/build-angular : 0.10.6
   @angular-devkit/schematics    : 7.0.6
   @angular/cli                  : 7.0.6
   @ionic/angular-toolkit        : 1.2.0

System:

   NodeJS : v10.13.0 (/home/kensodemann/.nvm/versions/node/v10.13.0/bin/node)
   npm    : 6.4.1
   OS     : Linux 4.15

@kensodemann I tested [(ngModel)] for checkbox after my commit https://github.com/ionic-team/ionic/commit/429e08344afb8a0e2b4c43c031d04fceced462e0 and it works as expected, so will be fixed in next release.

YEp, this should be fixed in beta.17!

I found the same problem exists in beta.17 for ion-toggle (for reactive forms)


Update clean

minor change to enable cleaning of tmp_mpi files

Great, thanks for the bugfix @jfbabb  and for the review @mortenpi.


Add converter entry point

Add a command line interface for mapdl.convert_script.
Close #1469
Note
I couldn't make it work for [project.entry-points.(...)], not sure if because I was in editable mode (pip install -e .) or because I was doing something wrong.
I end up using [project.scripts] as ansys-templates library.
Result
$ convertscript.exe --helpconvertscript.exe --help
Usage: convertscript [OPTIONS] FILENAME_IN

  PyMAPDL CLI tool for converting MAPDL scripts to PyMAPDL scripts.

  USAGE:

  The main usage is:

      $ convertscript mapdl.dat -o python.py

      File mapdl.dat successfully converted to python.py.

  The output argument is completely optional, in that case, it will just
  change the extension to "py".

      $ convertscript mapdl.dat

      File mapdl.dat successfully converted to mapdl.py.

  You can use any option from ``ansys.mapdl.core.convert.convert_script``
  function:

      $ convertscript mapdl.dat --auto-exit False

      File mapdl.dat successfully converted to mapdl.py.

      $ convertscript.exe mapdl.dat --filename_out mapdl.out --add_imports
      False

      File mapdl.dat successfully converted to mapdl.out.

Options:
  -o TEXT                        Name of the output Python script.
  --filename_out TEXT            Name of the output Python script.
  --loglevel TEXT                Logging level of the ansys object within the
                                 script.
  --auto_exit BOOLEAN            Adds a line to the end of the script to exit
                                 MAPDL. Default ``True``
  --line_ending TEXT             When None, automatically is `` .``
  --exec_file TEXT               Specify the location of the ANSYS executable
                                 and include it in the converter output
                                 ``launch_mapdl`` call.
  --macros_as_functions BOOLEAN  Attempt to convert MAPDL macros to python
                                 functions.
  --use_function_names BOOLEAN   Convert MAPDL functions to
                                 ansys.mapdl.core.Mapdl class methods.  When
                                 ``True``, the MAPDL command ``K`` will be
                                 converted to ``mapdl.k``.  When ``False``, it
                                 will be converted to ``mapdl.run('k')``.
  --show_log BOOLEAN             Print the converted commands using a logger
                                 (from ``logging`` Python module).
  --add_imports BOOLEAN          If ``True``, add the lines ``from
                                 ansys.mapdl.core import launch_mapdl`` and
                                 ``mapdl = launch_mapdl(loglevel="WARNING")``
                                 to the beginning of the output file. This
                                 option is useful if you are planning to use
                                 the output script from another mapdl session.
                                 See examples section. This option overrides
                                 ``auto_exit``.
  --comment_solve BOOLEAN        If ``True``, it will pythonically comment the
                                 lines that contain ``"SOLVE"`` or ``"/EOF"``.
  --cleanup_output BOOLEAN       If ``True`` the output is formatted using
                                 ``autopep8`` before writing the file or
                                 returning the string. This requires
                                 ``autopep8`` to be installed.
  --header BOOLEAN               If ``True``, the default header is written in
                                 the first line of the output. If a string is
                                 provided, this string will be used as header.
  --print_com BOOLEAN            Print command ``/COM`` arguments to python
                                 console. Defaults to ``True``.
  --help                         Show this message and exit.

I couldn't make it work for [project.entry-points.(...)], not sure if because I was in editable mode (pip install -e .) or because I was doing something wrong.
I end up using [project.scripts] as ansys-templates library.

Pinging @RobPasMue @jorgepiloto for the note.

convertscript is quite a generic name, the user should know that it's from pymapdl, right?

Very good point. ConvertMAPD2Python??

@akaszynski a preference?  If we do this more in the future across pyansys libraries, should there be a convention?

@akaszynski a preference? If we do this more in the future across pyansys libraries, should there be a convention?

We should follow bash conventions. Google has a guide for this. I'd go underscores, not CAPS, avoid numbers.

@akaszynski I was thinking more about the name and discovery of the command line program.  SHould it be:
convertmapdlscript -h
ansys_mapdl_convert_script -h
python -m ansys.mapdl.core convert_script -h

pymapdl_convert_script -h

pymapdl_convert_script

Changed to this.

@koubaa could you review this PR?


Increase size of blurb cards to account for longer blurbs/like button big enough to hold bigger like counts/room for all buttons

Blurb cards are fixed


Option to apply SSAO during lighting instead of during postprocessing

Fixes https://github.com/playcanvas/engine/issues/4844
Try it here: https://engine-4u63tegbq-playcanvas.vercel.app/#/graphics/ambient-occlusion
Until now, the SSAO was applied as a post-processing effect. The problem is that the ambient occlusion then incorrectly darkens even the areas hit by lights.
For the render pass architecture, an option to apply AO during lighting in forward passes has been added.
Before - notice how uniformly the SSAO looks

Now - almost no AO is visible in the lit areas, but ambient areas receive AO as before

Detail before

and now

Detail before

and now

Looks fantastic, there was always something "off" about the ssao that irked me and this is a huge improvement.
I've noticed a bug though if you apply scale its no longer aligned with the scene

this is a huge improvement

Totally agree, the impact is small but large at the same time.

I've noticed a bug though if you apply scale its no longer aligned with the scene

And I know exactly why .. will fix, thanks!


WIP Parallel Rubocop

An attempt to resurrect #175 and #117. I'm not sure why exactly they were closed.
Timings on rubocop's own codebase.
master
$ time bin/rubocop --cache false
Inspecting 825 files

825 files inspected, no offenses detected

real	0m31.175s
user	0m30.929s
sys	0m0.199s

parallel
$ time bin/rubocop --cache false
Inspecting 825 files

825 files inspected, no offenses detected

real	0m7.727s
user	0m54.805s
sys	0m0.629s

The difference is huge when running on a large codebase. For an example, @github has 10,000+ Ruby files. It takes about 4 minutes in serial and 55s in parallel on a cold cache. Even with a warm cache, parallelism still helps. 10s serial vs 2s parallel on @github's codebase.
Problems
Theres a number of problems with this as is:

Something is up with offense correction. Thats why the tests are failing. But it does seem to be working when running standalone from CLI. Still trying to figure that out.
I'm not sure what other memory side effects there are when checking for offenses. These object mutations would happen in a worker fork and wouldn't be propagated back up to the main process.
Read/write to cache probably needs to flock or be coordinated by the main process.

Before submitting the PR make sure the following are checked:

[ ] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[ ] Used the same coding conventions as the rest of the project.
[ ] Feature branch is up-to-date with master (if not - rebase it).
[ ] Squashed related commits together.
[ ] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[ ] All tests are passing.
[ ] The new code doesn't generate RuboCop offenses.
[ ] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[ ] Updated cop documentation with rake generate_cops_documentation (required only when you've added a new cop or changed the configuration/documentation of an existing cop).

CC: @aroben

An attempt to resurrect #175 and #117. I'm not sure why exactly they were closed.

I recall that the main problem with the old efforts was that the people working on the changes couldn't get reporting to work right (and I think this worked super slow on JRuby). I think we also have some cops that depend on the order other cops are executed and a few cops that work on all processed files that were added afterwards - I'll have to double check this. So, in brief - the PRs were closed because they never got to a working state and aimed for simpler solution that would preserve the single thread/process nature of RuboCop - notably a single pass of the AST tree for all cops and the cache.

I guess #175 had most of this discussion. In the end the PR was closed simply because it wasn't moving anywhere.

I really like the idea of this. I thought about trying to implement something like this around the time that caching was introduced.

I recall that the main problem with the old efforts was that the people working on the changes couldn't get reporting to work right

Take a look at how Minitest solves this issue. Their reporter implements a synchronize method that makes use of a Mutex to ensure that only 1 reporting operation can happen at a time.

Well, I think I answered my own question about why this PR originally was abandon. I don't have time myself to keep investigating it. If someone else wants to take it over, please do!!!

I told you - this feature is cursed. :D

For anyone interested, here's a wrapper script I use to speed up RuboCop execution, especially running ./bin/rubocop on RuboCop's own source.
#!/usr/bin/env ruby
#
# Usage: warmup <rubocop command...> [-- <rubocop args...>]
#
# Examples:
#   warmup rubocop
#   warmup bundle exec rubocop -- -F

require 'facter'

index = ARGV.index('--')
if index
  binary = ARGV[0...index]
  command = (binary + ARGV[(index + 1)..-1]).join(' ')
else
  binary = ARGV
  command = binary.join(' ')
end
binary = binary.join(' ')

CPU_COUNT = Facter.value('processors')['count']
FILES =
  `#{binary} -L`.split($RS).each_with_index.group_by { |_, i| i % CPU_COUNT }

puts "Running `#{command}` on #{CPU_COUNT} CPUs..."
CPU_COUNT.times do |cpu|
  fork { `#{command} #{FILES[cpu].map(&:first).join(' ')}` }
end

Process.waitall
puts 'Done'

system command

The idea is to use rubocop -L to get a list of files to inspect, then divide the list in equal parts and fork processes to share the work. No output is collected from these runs. The last stage is running rubocop again an all files, taking advantage of the results from the previous runs being cached. The gem facter is needed to figure out how many CPUs (cores) are available.
I typically get the excution time down to a third of the original when running on 8 cores.


Problem with PubSub when referencing BouncyCastle 1.8.1

Hi.
I am using the Google.Pubsub.V1 1.0.0-beta02 in an app and all works perfect, but now I am integrating it in another app and suddently it fails (just launching the app).
What is the difference between both apps? The difference is that in the new app it is used another package that forces to reference the BouncyCastle package with version 1.8.1 intead of the 1.7 (used in the first app and required for the Pubsub.V1 internal references).
The thrown error is:
An exception of type 'System.InvalidOperationException' occurred in Google.Api.Gax.dll but was not handled in user code
Additional information: Error reading credential file from location C:\x\x\xx.json: No se puede cargar el archivo o ensamblado 'BouncyCastle.Crypto, Version=1.7.4137.9688, Culture=neutral, PublicKeyToken=a4292a325f69b123' ni una de sus dependencias. La definición del manifiesto del ensamblado no coincide con la referencia al ensamblado. (Excepción de HRESULT: 0x80131040)
Please check the value of the Environment Variable GOOGLE_APPLICATION_CREDENTIALS
I can't use the version 1.7 of BouncyCastle because the other package does not allow it, so I have to use the 1.8.1.
Do you know how to solve this problem? Is there any solution?
Thank you.
Jose

@chrisdunelm You know more about the crypto side than I do. Any thoughts? It sounds like an assembly binding redirect should do the right thing here...

Hi.
Any news about this?
Thanks.

@jose3m I've reproduced this in a simple test-case. I'm now investigating why this occurs, and how to fix it.

@jose3m Hmmm, actually I'm getting a different error message, but it may be related.
I've created a library that depends on BouncyCastle 1.8.1
And an application that depends on the above library and on Google.Pubsub.V1 1.0.0-beta02
When a call is made into the library, where the callee calls into BouncyCastle, I get a FileLoadException
Additional information: Could not load file or assembly 'BouncyCastle.Crypto, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0e99375e54769942' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)
Looking at the BouncyCastle.Crypto.dll file in the bin directory, it is v1.7, where I would have expected the 1.8.1 dll to be present.
However, in your case it's saying it can't load v1.7, so presumably v1.8.1 is present.

@chrisdunelm
You are wright. In my bin directory it is present v1.8.1 and it is looking for v1.7.
The only way to "solve" it has been to remove the reference to BouncyCastle v1.8.1 and change its use by an alternative code. I didn't want to do it this way but I am not able to find a solution.

I've reproduced this with no Google dependencies at all.
Simply create two libraries, one depending on BouncyCastle 1.7.0, the other depending on BouncyCastle 1.8.1, each with a method that calls into BouncyCastle.
Then create a application that depends on both libraries, and call each function, and it crashes with exactly the same error.
Interestingly, trying this with a different nuget package (I used Json.NET) does not show this problem.  The redirect appears to happen correctly, without anything required in App.config
I suspect this may be because the public key token changes between BouncyCastle version 1.7.0 and 1.8.1
Whereas the public key token is identical for all versions of Json.NET
If this is the cause, then I'm not sure there's anything we can do :(

I tried to force the asembly reference binding in App.config and it didn't work.
I agre with you that the main problem is in the different public key token associated to the different versions of the BouncyCastle, so the only solution for me at this moment was to have all references only to one version, the used in Google packages.
Thank you very much for your help.


#69 Removed the resource default used check as this can very easily b…

…e introduced as an XPath rule if people want to track the usage of this type of grammar.

@racodond I have removed the check, its test, the html description and the test resource file.
Thats everything yeah?


Switching to an unsupported network crashes the app

Describe the bug
On the publish screen, switching to an unsupported network causes an unrecoverable error "Application error: a client-side exception has occurred (see the browser console for more information)."
Also, the error is written in light gray text on a white background, only visible if you highlight it, as I did in the screenshot. In case it's relevant, I'm using dark mode.
To Reproduce

Navigate to Publish screen
Switch to unsupported network (In my case, I used Optimism)
Observe error.
Attempt to switch back to supported network, page doesn't recover. Either manual refresh, or navigating away and back is required.

Preferred behavior
Display an error saying "Unsupported network" without crashing. Also, it would be awesome if publish progress wasn't lost either.
Desktop

OS: Ubuntu 20.04.4 LTS (Focal Fossa)
Browser: Brave
Version:  Version 1.36.119 Chromium: 99.0.4844.83 (Official Build) (64-bit)
Kazam_screencast_00007.mp4

https://images.zenhubusercontent.com/259586079/e8fa1a4f-6843-40a6-b56d-956aa633d18c/kazam_screencast_00007.mp4

Not that big of an problem to require two issues 😆  https://github.com/oceanprotocol/market/issues/1263

😆 Fair point


Clear selected time slot after selection - suggestion

Hey, cool component 👍
Wanted to see if you guys can add clearing of the selected time slot after clicking Confirm
Maybe even do it conditional, like return boolean from onStartTimeSelect and if it's true clear the selection - setSelectedItemIndex(-1). Or clear it always 🤷♀.
Could be useful... At least in my case
Thanks

I guess the UI's assumption is that once you click confirm, the schedule component would be taken away or hidden. I suppose i can see a case for where you might want to keep it but it wouldn't be practical UX to keep the schedule component visible with nothing selected even though the external state would have a time selected - In that scenario it would be better if the schedule actually showed the time selected maybe without the "confirm" text but even still, wouldn't it be better to display a completely alternative UI component showing that something has been selected?
Can you show me your use-case where the schedule needs to remain visible with a time slot already selected? I'm not understanding what the benefit would be

Unless, are you using the schedule component to register multiple time slot selections?

I was thinking about use case like you have when scheduling a job interview, when you asked to select multiple optional time slots.
Right now, i'm using only 1, so i can hide the component after selection. I'm actually updating the time slots after selection by removing the selected slot, and it creates a case where there is another slot (next one) is selected on the screen.
But i want to give an option of multi selection
If it makes sense 😀
And thanks for a quick reply

Gotcha yea I can add a way to unset the "confirm" state

cool. thanks

Okay its available in 0.4.0 Commit
Here's an example: Example & Usage


doc: explain what custom pprof profiles are and why they are useful

Diagnostics page is briefly mentioning custom profiles but failing to give any introduction for those who doesn't know why custom profiles are useful. Explain custom profiles and share a few sample use cases.

Hi @rakyll
I am new to the Go community and would like to work on this issue.
Please let me know if there are any specific points you'd like me to focus on or any additional details to consider before I start working on the changes.
Thank you!


How to call raxml?

@sdwfrost Could you tell me how to call raxml?

In R? You'd use system.call, or if you're running an RMarkdown document in RStudio, you can execute shell commands rather easily. You may also be interested in some smaller, but more up to date courses:
https://github.com/sdwfrost/mevr-course-ghana/blob/gh-pages/index.ipynb
https://github.com/sdwfrost/kemri-phylodynamics

Also, do you look at the actual course notes:
http://sdwfrost.github.io/mevr-course-2016/
Much easier to go through!

@sdwfrost In your R Markdown: https://github.com/sdwfrost/mevr-course-2016/blob/master/ml/raxml.Rmd:
myalignment.rax.gtr <- raxml (myalignment,
m="GTRGAMMAIX", # model
f="a", # best tree and bootstrap
p=1234, # random number seed
x=2345, # random seed for rapid bootstrapping
N=100, # number of bootstrap replicates
file="ray2000", # name of output files
exec="raxmlHPC-PTHREADS-SSE3", # name of executable
threads=2
)
Can the above code aotumate call  "raxmlHPC-PTHREADS-SSE3"  or I have to put "raxmlHPC-PTHREADS-SSE3.exe" in the respective file, thanks.


python3Packages.testpath: fix build reproducibility

Description of changes
Fixes #223583
This strips the exe files when testpath is used on non-windows platforms.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

nix-build --arg pkgs "import ./. {}" ./lib/tests/release.nix is fine locally, ofborg was probably just having a bad day.


Missing Layout

Create the initial layout for the blog.
Should include:

[x] About me page
[x] A template for posts

Currently using the Minima Jekyll Theme which is the default.


Transmission stream resolution

I'm getting the rtsp stream correctly but it's in SD quality, is there any way to get the stream in HD quality?

I suppose you are talking about Tuya, right? Unfortunately, no. This would be an issue to the Tuya integration for Home Assistant or for the Tuya API itself.
You are not the first one to ask this by the way, lol.
Another possible solution would be for go2rtc to support Tuya directly: https://github.com/AlexxIT/go2rtc/issues/315


Can't install 4058 Error

Describe the bug
I am getting a 4058 error when trying to install this package for a project I am working on.
npm WARN deprecated<EMAIL_ADDRESS>Support has ended for 9.x series. Upgrade to @latest
npm ERR! code ENOENT
npm ERR! syscall spawn git
npm ERR! path git
npm ERR! errno -4058
npm ERR! enoent Error while executing:
npm ERR! enoent undefined ls-remote -h -t<EMAIL_ADDRESS>npm ERR! enoent
npm ERR! enoent
npm ERR! enoent spawn git ENOENT
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent
To Reproduce
Steps to reproduce the behavior:

Running 'npm install --save tui-editor'

Expected behavior
Installation of the package
Desktop (please complete the following information):

OS: Windows 10
Node Version: v14.16.0

Trying to install the latest version(1.4.10) of tui-editor, I encountered similar problems at version 1.4.4-alpha.1. My problem was fixed in version 1.4.10

@AbrahamZak It is due to GitHub's protocol problem, and it is currently a bit difficult to support version 1. If possible, it would be nice to update to version 2 and use it.

https://docs.npmjs.com/cli/v7/configuring-npm/package-json#git-urls-as-dependencies


ld64 needs libdispatch on Linux

Fixes #79

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Can you add
conda_build:
  error_overlinking: true

to conda-forge.yml ?


New-horizons branch issue: make test fails

Two root reasons for this issue: one, the makefile doesn't do enough checking (issue #150) which enabled this issue to go undetected unless one was paying careful attention. Second, the psgeodb object tests for language objects in the assertions, but now that the language object was changed, the test failed. This preventd any test from psgeodb ever being run in the new branch.
The tests need to be run now, which probably uncovers issues, since psgeodb was the main user of the psgeolang module.
Also, the testipenkki version of the new-horizons thing doesn't work if one clicks on a cave pin. Possibly due to this issue.
Work needed...

Works now after some fixes.


Run additional CI pipeline steps with latest Python version

Passing

@ulgens I think the default python version should be the minimum, not the maximum.

Makes sense.


[UWP] Set runtime class name on ElementTagContent

Related Issue
Fixes #3836
Description
ElementTagContent didn't have it's runtime class name set, causing Visual Studio's Live Property Tree viewer to Assert and fail to show element properties. Set the runtime class name using the InspectableClass macro via our AdaptiveRuntimeStringClass macro.
How Verified
Verified Live Property Tree now working as expected.
Microsoft Reviewers: Open in CodeFlow

Great catch! Isn't this class used anywhere for this to have been caught earlier? Also is this one a recent addition to our interface?

This class is used to associate various content (such as the card element) with the xaml objects. It's used on most of our xaml objects, and functionally for what we've been using it for it works fine without this declaration. For whatever reason, the only place we're hitting this assert is via the usage of the live visual tree. We've had this class around for a while (since May of last year), but as this seems to be the only error caused by this bug, and because the cause and effect wasn't immediately obvious, we didn't find this until i really needed the Live Visual Tree to work to debug something else.

In reply to: 597902793 [](ancestors = 597902793)

/azp run

/azp run


it would be nice if can add the buff  "Nephalem Glory"

I think it provides a very insignificant bonus - see the Nephalem Glory section in http://us.battle.net/d3/en/forum/topic/8770117237
I.e. a chance to deal 20% weapon damage on hit, and up to 600% explosion when the buff expires.

Isn't the 10% movement speed buff important for barbarians using Standoff?

I dont know the detail of damage  , but  I  saw  many   20-40 million when its active(the yellow flash).  about 300% weapon damage just can reach 20-40 million with my equipment.    There were several  yellow explosions  I have not seen the damage number.

I'll add that NG has 2 derivatives - the PC version and the console version.
Console version is VERY different - basically, PC players get minor movement speed buffs, etc, on console, we do HALF PC damage UNTIL we pick up a NG globe.  Then our damage matches the PC version of the game.  This hampers console players rather badly.  note: if we die, we lose the NG buff.  When pushing GRs to the limit, this is very painful, and PC players don't have to put up with this silly mechanic at all.
If you do include a NG mechanic for d3planner, please make sure to state that it's designed to emulate the PC version of the game so as to not confuse inexperienced console players!


Add Michael and Mark to committers list

The OpenTitan Technical Committee is happy to announce that Mark
Branstad and Michael Munday are now committers!
Thanks for your great work on OpenTitan so far, we're looking forward to
more of it!

@asb can you add the two to the right GH ACL?

@asb can you add the two to the right GH ACL?

Done


Add a functionality to adjust font size of the editor

SQL online editor uses React-ace code editor. Implement a dropdown to change the font size of the editor.
Describe the solution you'd like

Add a dropdown beside the theme dropdown.
use reference from react ace how the font size dropdown is implemented.

for example -

@janvi01 pls assign this issue to me under jwoc

Go ahead @prasukjain07 feel free to drop in any doubts.


Is there a choice to use ignore_mismatch_sizes in PeftModel.from_pretrained like AutoModel in Transformers?

Feature request
Add ignore_mismatch_sizes in PeftModel.from_pretrained.
Motivation
When I try to add some special tokens into an LM and use an adapter, I find though I can set ignore_mismatch_sizes = True when I load the base model, I have no similar choice to ignore the mismatch parameters when I load the adapter.
I have searched a possible way to first load the adapter and then add the base model. However, in my case, my base model has been some trained parameters that I cannot resize the parameters after loading the adapter.
Have I miss some args? Or is it a necessary choice?
Your contribution
I don't know what can I do.

Do you have some simple reproducer for the error you mentioned? Then we can check if there is something we can do on the PEFT side.
From what I understand, this shouldn't require a trained adapter, so it you could just show how you created and saved the adapter, then tried to load it, it should be sufficient.

@BenjaminBossan Sorry, my statement was unclear. I did not encounter a bug, but I wish to submit a feature request. When I try to add new tokens to a pre-trained model as well as a corresponding adapter (which means the size of lm_head and token embed needs to change), I find that the from_pretrained function in transformers can conveniently ignore those modules with the same name but mismatched size by using the ignore_mismatch_size option. (See here)
This is useful when I want to further fine-tune these models. However, I noticed that peft does not offer this option, meaning I cannot load a partially matched adapter into a base model that has loaded some parameters through the above method (with another part of the parameters being reinitialized due to size change). I hope Peft could offer a similar functionality.
In my case, I have trained a Llama with an adapter on my task, I use the trainer.train to automatically save the adapter, and the adapter has been saved successfully. But I want to use this model as well as the trained adpater to finetune on another task, which requires two special tokens, so I resize the base model (here, it means the Llama), and I can successfully load the pretrained Llama (except for the resized lm_head and embed_layer, which I will manually process them). However, I find I cannot load the adapter because I have changed the size of these two modules. I hope I have an option to just ignore them.

I forgot to mention that for these two layers, I set them in modules_to_save, so they are both fully trained and saved in the adapter.

Thanks you for clarifying, I think I got it now. So you would like something like this to work:
import torch
from transformers import AutoModelForCausalLM
from peft import LoraConfig, PeftModel, get_peft_model
from peft.utils.other import ModulesToSaveWrapper

model_id = "facebook/opt-125m"
model0 = AutoModelForCausalLM.from_pretrained(model_id)
peft_config0 = LoraConfig(
    task_type="CAUSAL_LM", target_modules=["q_proj", "v_proj"], modules_to_save=["embed_tokens"], init_lora_weights=False
)
peft_model0 = get_peft_model(model0, peft_config0)

# embedding layer uses ModulesToSave:
[module for module in peft_model0.modules() if isinstance(module, ModulesToSaveWrapper)]

peft_model0.save_pretrained("/tmp/peft/issue-1605")

model1 = AutoModelForCausalLM.from_pretrained(model_id)
model1.resize_token_embeddings(50272 + 2)
# this currently raises an error:
peft_model1 = PeftModel.from_pretrained(model1, "/tmp/peft/issue-1605", ignore_mismatched_sizes=True)
# RuntimeError: Error(s) in loading state_dict for PeftModelForCausalLM:
# 	size mismatch for base_model.model.model.decoder.embed_tokens.modules_to_save.default.weight: copying a param with shape torch.Size([50272, 768]) from checkpoint, the shape in current model is torch.Size([50274, 768]).

Is that right?

@gary-young I created a PR: #1620. Maybe you can check if it works with your use case.


Inline asm types

This most likely supersedes #372.
Various SPIRV instructions have operands that can require more operands when they're specified. Image instructions are one example, OpDecorate is another.
Todo:

[ ] Add tests

Adding a test for OpImageSampleProjExplicitLod also to show how to use BitEnums.


Switched Imager_ImagePathsModel::__construct() to instanceof for Craft\AssetFileModel checks

Switched Imager_ImagePathsModel::__construct()'s use of get_class() to instanceof when checking for \Craft\AssetFileModel.

This has been merged (better late than never), thanks for contributing!


Support displaying address in the pre or post text of the popup

Agree with this suggestion -- would be great if the pop up could be configured to say "Search results for 123 Main St."


[AutoPR hanaonazure/resource-manager] Fixing operation id to be named consistently

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5666

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/4725


Node: timeout / deadline on calls

Is it possible to set a timeout / deadline on calls in Node?
Failed to connect to channel, retrying
Retry in<PHONE_NUMBER>84 seconds

I read through the code and found some references to a deadline but couldn't figure out how to set it.

When you make a call, you can pass the deadline as an optional parameter before the callback. The deadline should just be a date object.

Am I blind or are there no docs on this?

No, you're right. At the moment, the only documentation for this is in the source files themselves.

The API for setting deadlines on calls has been documented as part of the CallOptions type.


Add a script for automating production deploys

Summary of changes
Asana Ticket: 🔧 Use a script to automate production deploys
Add a script for automating production deploys
Add python as a dependency.
Add boto3 and docopt as python requirements.

Before getting review, please check the following:

[x] Does frontend functionality render and work correctly in IE?
[x] Have we load-tested any new pages or internal API endpoints that will receive significant traffic?
[x] Are interactive elements accessible to screen readers?
[x] Have you checked for tech debt you can address in the area you're working in?
[x] If this change involves routes, does it work correctly with pertinent "unusual" routes such as the combined Green Line, Silver Line, Foxboro commuter rail, and single-direction bus routes like the 170?
[x] Are the changes organized into self-contained commits with descriptive and well-formatted commit messages?

@thecristen and @alicia-mbta, I asked Ian for a review knowing he has experience with these types of python scripts as well as our deploy process, but I'm of course interested in any feedback you have as well.

I forgot to mention that I successfully ran our recent deploy using this.

This is great! Does the deployment happen just by running this new script or would there be other steps involved?
On that note, could you add a sentence or so to the README indicating that?

@alicia-mbta Great point, I added a bunch of documentation.

One other thought: As written, the Python script could be replicated with a single AWS CLI call:
aws elasticbeanstalk update-environment --environment-name "$dotcom-prod" --version-label "$sha"

Granted, you have the option to add a lot of sugar to the Python script – you could use describe_application_versions() to pull a list of most recent versions and choose which one you want to deploy, for example. Or poll to watch the update happen. But if you don't want to do any of that, you could rewrite this as minimal a shell script, FWIW.

Oh yeah! That’s exciting 😄


samples/net/wifi fails to build on cc3220sf_launchxl

In file included from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:51:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:21:2: error: redeclaration of enumerator 'Reset_IRQn'
  Reset_IRQn                    = -15,
  ^~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/include/arch/arm/cortex_m/cmsis.h:17,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/cortex_m/stack.h:27,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/kernel_arch_data.h:53,
                 from /home/apboie/projects/zephyr2/zephyr/kernel/include/kernel_structs.h:68,
                 from /home/apboie/projects/zephyr2/zephyr/include/sys/mutex.h:113,
                 from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:19:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:21:2: note: previous definition of 'Reset_IRQn' was here
  Reset_IRQn                    = -15,
  ^~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:51:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:22:2: error: redeclaration of enumerator 'NonMaskableInt_IRQn'
  NonMaskableInt_IRQn           = -14,
  ^~~~~~~~~~~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/include/arch/arm/cortex_m/cmsis.h:17,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/cortex_m/stack.h:27,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/kernel_arch_data.h:53,
                 from /home/apboie/projects/zephyr2/zephyr/kernel/include/kernel_structs.h:68,
                 from /home/apboie/projects/zephyr2/zephyr/include/sys/mutex.h:113,
                 from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:19:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:22:2: note: previous definition of 'NonMaskableInt_IRQn' was here
  NonMaskableInt_IRQn           = -14,
  ^~~~~~~~~~~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:51:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:23:2: error: redeclaration of enumerator 'HardFault_IRQn'
  HardFault_IRQn                = -13,
  ^~~~~~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/include/arch/arm/cortex_m/cmsis.h:17,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/cortex_m/stack.h:27,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/kernel_arch_data.h:53,
                 from /home/apboie/projects/zephyr2/zephyr/kernel/include/kernel_structs.h:68,
                 from /home/apboie/projects/zephyr2/zephyr/include/sys/mutex.h:113,
                 from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:19:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:23:2: note: previous definition of 'HardFault_IRQn' was here
  HardFault_IRQn                = -13,
  ^~~~~~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:51:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:24:2: error: redeclaration of enumerator 'MemoryManagement_IRQn'
  MemoryManagement_IRQn         = -12,
  ^~~~~~~~~~~~~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/include/arch/arm/cortex_m/cmsis.h:17,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/cortex_m/stack.h:27,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/kernel_arch_data.h:53,
                 from /home/apboie/projects/zephyr2/zephyr/kernel/include/kernel_structs.h:68,
                 from /home/apboie/projects/zephyr2/zephyr/include/sys/mutex.h:113,
                 from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:19:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:24:2: note: previous definition of 'MemoryManagement_IRQn' was here
  MemoryManagement_IRQn         = -12,
  ^~~~~~~~~~~~~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:51:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:25:2: error: redeclaration of enumerator 'BusFault_IRQn'
  BusFault_IRQn                 = -11,
  ^~~~~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/include/arch/arm/cortex_m/cmsis.h:17,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/cortex_m/stack.h:27,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/kernel_arch_data.h:53,
                 from /home/apboie/projects/zephyr2/zephyr/kernel/include/kernel_structs.h:68,
                 from /home/apboie/projects/zephyr2/zephyr/include/sys/mutex.h:113,
                 from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:19:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:25:2: note: previous definition of 'BusFault_IRQn' was here
  BusFault_IRQn                 = -11,
  ^~~~~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:51:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:26:2: error: redeclaration of enumerator 'UsageFault_IRQn'
  UsageFault_IRQn               = -10,
  ^~~~~~~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/include/arch/arm/cortex_m/cmsis.h:17,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/cortex_m/stack.h:27,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/kernel_arch_data.h:53,
                 from /home/apboie/projects/zephyr2/zephyr/kernel/include/kernel_structs.h:68,
                 from /home/apboie/projects/zephyr2/zephyr/include/sys/mutex.h:113,
                 from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:19:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:26:2: note: previous definition of 'UsageFault_IRQn' was here
  UsageFault_IRQn               = -10,
  ^~~~~~~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:51:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:27:2: error: redeclaration of enumerator 'SVCall_IRQn'
  SVCall_IRQn                   =  -5,
  ^~~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/include/arch/arm/cortex_m/cmsis.h:17,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/cortex_m/stack.h:27,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/kernel_arch_data.h:53,
                 from /home/apboie/projects/zephyr2/zephyr/kernel/include/kernel_structs.h:68,
                 from /home/apboie/projects/zephyr2/zephyr/include/sys/mutex.h:113,
                 from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:19:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:27:2: note: previous definition of 'SVCall_IRQn' was here
  SVCall_IRQn                   =  -5,
  ^~~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:51:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:28:2: error: redeclaration of enumerator 'DebugMonitor_IRQn'
  DebugMonitor_IRQn             =  -4,
  ^~~~~~~~~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/include/arch/arm/cortex_m/cmsis.h:17,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/cortex_m/stack.h:27,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/kernel_arch_data.h:53,
                 from /home/apboie/projects/zephyr2/zephyr/kernel/include/kernel_structs.h:68,
                 from /home/apboie/projects/zephyr2/zephyr/include/sys/mutex.h:113,
                 from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:19:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:28:2: note: previous definition of 'DebugMonitor_IRQn' was here
  DebugMonitor_IRQn             =  -4,
  ^~~~~~~~~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:51:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:29:2: error: redeclaration of enumerator 'PendSV_IRQn'
  PendSV_IRQn                   =  -2,
  ^~~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/include/arch/arm/cortex_m/cmsis.h:17,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/cortex_m/stack.h:27,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/kernel_arch_data.h:53,
                 from /home/apboie/projects/zephyr2/zephyr/kernel/include/kernel_structs.h:68,
                 from /home/apboie/projects/zephyr2/zephyr/include/sys/mutex.h:113,
                 from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:19:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:29:2: note: previous definition of 'PendSV_IRQn' was here
  PendSV_IRQn                   =  -2,
  ^~~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:51:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:30:2: error: redeclaration of enumerator 'SysTick_IRQn'
  SysTick_IRQn                  =  -1,
  ^~~~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/include/arch/arm/cortex_m/cmsis.h:17,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/cortex_m/stack.h:27,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/kernel_arch_data.h:53,
                 from /home/apboie/projects/zephyr2/zephyr/kernel/include/kernel_structs.h:68,
                 from /home/apboie/projects/zephyr2/zephyr/include/sys/mutex.h:113,
                 from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:19:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:30:2: note: previous definition of 'SysTick_IRQn' was here
  SysTick_IRQn                  =  -1,
  ^~~~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:51:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:31:3: error: conflicting types for 'CMSIS_IRQn_Type'
 } CMSIS_IRQn_Type;
   ^~~~~~~~~~~~~~~
In file included from /home/apboie/projects/zephyr2/zephyr/include/arch/arm/cortex_m/cmsis.h:17,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/cortex_m/stack.h:27,
                 from /home/apboie/projects/zephyr2/zephyr/arch/arm/include/kernel_arch_data.h:53,
                 from /home/apboie/projects/zephyr2/zephyr/kernel/include/kernel_structs.h:68,
                 from /home/apboie/projects/zephyr2/zephyr/include/sys/mutex.h:113,
                 from /home/apboie/projects/zephyr2/zephyr/lib/libc/newlib/libc-hooks.c:19:
/home/apboie/projects/zephyr2/zephyr/soc/arm/ti_simplelink/cc32xx/soc.h:31:3: note: previous declaration of 'CMSIS_IRQn_Type' was here
 } CMSIS_IRQn_Type;
   ^~~~~~~~~~~~~~~
zephyr/lib/libc/newlib/CMakeFiles/lib__libc__newlib.dir/build.make:62: recipe for target 'zephyr/lib/libc/newlib/CMakeFiles/lib__libc__newlib.dir/libc-hooks.c.obj' failed
make[2]: *** [zephyr/lib/libc/newlib/CMakeFiles/lib__libc__newlib.dir/libc-hooks.c.obj] Error 1
CMakeFiles/Makefile2:1050: recipe for target 'zephyr/lib/libc/newlib/CMakeFiles/lib__libc__newlib.dir/all' failed
make[1]: *** [zephyr/lib/libc/newlib/CMakeFiles/lib__libc__newlib.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....

@andrewboie Submitted fix for this build issue.


feat: add links to build site [SRED-199]

Added the links, let me know if design and code is acceptable. Tried not to spend too much time on it.
Scroll to bottom to see the links:
https://wpds-ui-rd2pvo5ys.preview.now.washingtonpost.com/

@ebgranger See the design spec in the ticket.


design: each expression

@chrisrpatterson

Could we do this in $[ ] expressions, too?
variables:
  NODE_VERS: ['6.x', '8.x', '10.x']
  TS_VERS: ['2.8', '2.9', '3.0']
strategy:
  matrix:
    $[ each nodeVer in variables.NODE_VERS ]:
      $[ each tsVer in variables.TS_VERS ]:
        $[ concat(nodeVer, '_', tsVer) ]:
          NODE_VERSION: $[ nodeVer ]
          TS_VERSION: $[ tsVer ]
steps:
- script: echo Node $(NODE_VERSION) against TypeScript $(TS_VERSION)

It's not the prettiest way to create a matrix, for sure. I predict people will try to do it, though, and the same thing is even grosser if you're forced into a template for it :)

@vtbassmatt I would much rather have a specific function for doing a cross product a variable array and apply some exclusions.  Seems cleaner.

my comments got deleted, eh?

@4c74356b41, looks like your comments were added as a part of mmitche's review, which was resolved but is still available.

can you provide a link, i cant seem to find those. thanks. @chcosta

https://github.com/Microsoft/azure-pipelines-yaml/pull/2#discussion_r223173555

ok, why is it resolved than? I dont mean to say I'm the most important guy out here, but comments are there for a reason. I work with these yaml definitions and they are pretty horrible for anything remotely clever. why cant I use concats? or other common string\object manipulation functions?

mmitche's review was resolved (likely via a GitHub push).  If you have feedback, you should do your own review or start another comment thread so that the topics don't get lost in the mix.

What is meant by "cross-product matrix"?  @vtbassmatt , is that the same request as https://dnceng.visualstudio.com/7ea9116e-9fac-403d-b258-b31fcf1bb293/_workitems/edit/59 or a different scenario?

@chcosta: It looks like https://dnceng.visualstudio.com/7ea9116e-9fac-403d-b258-b31fcf1bb293/_workitems/edit/59 is the same as https://github.com/Microsoft/azure-pipelines-yaml/issues/4.  The ability to use a variable (in this case from a matrix) to set vmImage.
By "cross-product matrix", I mean the scenario where you want to use the matrix strategy on a cross-product of multiple dimensions.  For example, say you want to run tests on 2 OS (Linux, Windows) and 2 versions of Python (3.6, 3.7).  Currently, you'd need to manually create a matrix with all the combinations:
strategy:
  matrix:
    Linux_Python36:
      VM_IMAGE: 'ubuntu-16.04'
      PYTHON_VERSION: '3.6'
    Linux_Python37:
      VM_IMAGE: 'ubuntu-16.04'
      PYTHON_VERSION: '3.7'
    Windows_Python36:
      VM_IMAGE: 'vs2017-win2016'
      PYTHON_VERSION: '3.6'
    Windows_Python37:
      VM_IMAGE: 'vs2017-win2016'
      PYTHON_VERSION: '3.7'

While this isn't too bad with a small number of dimensions and a small number of variables in each dimension, it can quickly become unmaintainable as the dimensions and variables grow.  It would be nice if there was a way to express this more succinctly, something like:
strategy:
  cross-product:
    PYTHON_VERSION: [ '3.6', '3.7' ]
    VM_IMAGE: [ `ubuntu-16.04`, `vs2017-win2016` ]

@vtbassmatt: Separate issue for cross-product: #20

Hi Team
Sub: Variables in YAML pipeline are not allowing to define array
Error :  A sequence was not expected,
Please advise how to define variables in YAML pipelines are not allowing to define array.
Code:
variables:

name: MyVariableName
value: ['Value1','Value2']

I want to use/pass MyVariableName to template.yml where i want to iterate array
parameters:
MyVariableName: [] #defining array here to expect multiple values.
steps:

${{ each hostname in parameters.MyVariableName}}:
logic

@andrewsuiter sorry, my example was wrong.
Here is a better example:
# my-template.yml
parameters:
steps:
- ${{ each project in parameters.project }}:
  - task: PublishBuildArtifacts@1
    displayName: 'Publish ${{ project }}
    inputs:
      PathtoPublish: '$(Build.ArtifactStagingDirectory)/${{ project }}.zip'

# ci.yml
steps:
- template: my-template.yml
  parameters:
  - test1
  - test2

That way, one task is added for each item in the array parameters.project. Does that work? If not, can you paste the cotents of both files (the template, and the caller of the template)

@ericsciple - FYI the quote is missing in the displayName of your example:
displayName: 'Publish ${{ project }}**'**


Coded pending todo for adding support for additional paramters to ZendService\Twitter\Twitter::statusesUpdate

Added support for additional parameters in ZendService\Twitter\Twitter::statusesUpdate($status, $parameters)
Retrocompatible: The signature has slightly changed, but the old one is still supported.

These changes are invalid, starting from the composer.json ones (package rename)


how to set mask_value for multi-feature data

Suppose I have a multi-variable(multi-dimensional) time series data for classification, input_shape = (batch_size, time_steps, data_dim). some of the samples have different time steps, so I need to pad 0s to keep the time steps same to feed them into LSTMs.
I would like to mask these padding zeros so I also use a masking Layer, but I do not understand how to set mask_value for these multi-feature data( data_dim > 1), I try to refer to the source code but it seems theanoish... and these is also litter explanation in the docs of how mask Layer works. I have no idea.
should I use
model.add(Masking(mask_value = np.zeros((1,data_dim)), input_shape=(max_timesteps,data_dim)))
or
model.add(Masking(mask_value = 0., input_shape=(max_timesteps,data_dim))?
if  I misunderstand the meaning of Masking, please point it out.

Can you post all your code or fake data along with your model?
That will make things easier for people to help you.

OK, I dived into the code of Masking Layer, then referred to some theano & numpy documents, finding that both of these two usages are right(and it also be confirmed in my experiment, the outputs are the same). The core reason is numpy's broadcast mechanism.


Full data profile - Inventory

Make sure all combinations of the ndfc_inventory are tested:
ndfc_inventory.j2

---
# This NDFC switch inventory config data structure is auto-generated
# DO NOT EDIT MANUALLY
#
{% for switch in MD_Extended.vxlan.topology.switches %}
{% if switch.management.management_ipv4_address is defined %}
- seed_ip: {{ switch['management']['management_ipv4_address'] }}
{% elif switch.management.management_ipv6_address is defined %}
- seed_ip: {{ switch['management']['management_ipv6_address'] }}
{% endif %}
  auth_proto: {{ MD['vxlan']['global']['auth_proto'] | default(defaults.vxlan.global.auth_proto) }}
  user_name: PLACE_HOLDER_USERNAME
  password: PLACE_HOLDER_PASSWORD
  max_hops: 0 # this is the default value as it is not defined into the data model
  role: {{ switch['role'] }}
  preserve_config: false
{% endfor %}

To be tested:
 auth_proto: enum('MD5', 'SHA', 'MD5_DES', 'MD5_AES', 'SHA_DES', 'SHA_AES', required=False)

role: enum('spine', 'leaf', 'border', 'boder_spine', 'border_gateway', 'border_gateway_spine', 'super_spine', 'border_super_spine', 'border_gateway_super_spine', 'tor', required=False)

With our current lab configurations only the MD5 authproto, and the spine, leaf, border seem to be available to confirm the configurations are valid.  Will check with the team to see if we need to test all options changing or expanding the lab.

Currently all devices are created with MD5 /type 5 creds, so none of the other authprotos work for our current lab setup.
username admin password 5 $5$GA……
 role network-admin
ip domain-lookup
copp profile strict
snmp-server user admin network-admin auth md5 205F…..

I suppose we could change these or add new additional devices with different auth methods.
As for the roles, if we could get some assistance with the lab setup for those other roles, we can create configs and test all of those in our schema.
@dsx1123

Issue is resolved with the following PRs:
Updates for border gateway role for integration tests
https://github.com/netascode/ansible-dc-vxlan/pull/124
updates for border_spine for full integration tests #129
https://github.com/netascode/ansible-dc-vxlan/pull/129
updates for SHA auth_proto for integration tests #130
https://github.com/netascode/ansible-dc-vxlan/pull/130


About image resolution and optical flow resolution

I noticed that in the code './tool/video_inpainting. Py 'the resolution of the optical flow estimation for low-resolution images is extended, which causes the resolution to not align when the function rf.regionfill() is called.
if imgH < 350:         flowH, flowW = imgH * 2, imgW * 2     else:         flowH, flowW = imgH, imgW
After this code is deleted, the sample is generated accurately. Can the author explain the reason for this?v

That may be a bug after refactor, I will check the details. The usage about these two lines is to estimate accurate optical flows. Since we adopt RAFT for flow estimation, we noticed the performance of deep learning-based optical flow estimation methods is related to the frame resolution. If the resolution is too low, the performance of flow estimation will degrade. Therefore, we enlarge the frame resolution when the resolution is too low.

This bug is fixed, I impose the resize operations to optical flows in calculate_flow function in the code ./tool/video_inpainting.py.


Support concatenated arrays

With this patch, NStepRNN supports concatenated arrays. A user can skip concatenation with this new argument.

Why don't you add document about lengths ?

Why don't you add document about lengths ?

@unnonouno LGTM except comment

ping @unnonouno


can't login to local docker registry (hidden behind nginx with basic auth)

Setup

running docker registry on 5000 (from Fedora repository)

common: &common
    loglevel: _env:LOGLEVEL:info
    standalone: _env:STANDALONE:true
    sqlalchemy_index_database: _env:SQLALCHEMY_INDEX_DATABASE:sqlite:////tmp/docker-registry.db
local: &local
    <<: *common
    storage: local
    storage_path: _env:STORAGE_PATH:/var/lib/docker-registry

nginx with basic auth and ssl in front of

upstream docker-registry {
    server <IP_ADDRESS>:5000;
}

server {
    listen 443;
    server_name ...net;
    access_log /var/log/nginx/docker.access.log;
    error_log /var/log/nginx/docker.error.log notice;

    ssl on;
    ssl_certificate /etc/ssl/certs/docker-registry;
    ssl_certificate_key /etc/ssl/private/docker-registry;

    client_max_body_size 0; # disable any limits to avoid HTTP 413 for large image uploads
    # required to avoid HTTP 411: see Issue #1486 (https://github.com/docker/docker/issues/1486)
    chunked_transfer_encoding on;
    proxy_set_header Host $http_host;   # required for docker client's sake
    proxy_set_header X-Real-IP $remote_addr; # pass on real client's IP
    proxy_set_header Authorization  ""; # see https://github.com/dotcloud/docker-registry/issues/170
    proxy_read_timeout 900;

    proxy_set_header X-Forwarded-Proto "https";
    proxy_set_header X-Forwarded-Protocol "https";

    location / {
        auth_basic "Restricted";
        auth_basic_user_file docker-registry.htpasswd;
        proxy_pass http://docker-registry;
    }
    location /_ping {
        auth_basic off;
        proxy_pass http://docker-registry;
    }
    location /v1/_ping {
        auth_basic off;
        proxy_pass http://docker-registry;
    }

}

Issue
When I try to log in, I get this:
$ docker login ...net/
Username: admin
Password:
Email:
2014/10/06 16:38:26 Error response from daemon: (Code: 404; Headers: map[Server:[nginx/1.4.7] Date:[Mon, 06 Oct 2014 14:38:27 GMT] Content-Type:[text/html] Content-Length:[233] Connection:[keep-alive] X-Docker-Registry-Version:[0.8.1] X-Docker-Registry-Config:[local]])

Server logs:
#access
<IP_ADDRESS> - admin [06/Oct/2014:16:37:42 +0200] "GET /users/ HTTP/1.1" 404 233 "-" "docker/1.2.0 go/go1.2.2 kernel/3.16.3-200.fc20.x86_64 os/linux arch/amd64"
<IP_ADDRESS> - - [06/Oct/2014:16:38:26 +0200] "GET /_ping HTTP/1.1" 200 4 "-" "Go 1.1 package http"
<IP_ADDRESS> - - [06/Oct/2014:16:38:26 +0200] "POST /users/ HTTP/1.1" 401 194 "-" "Go 1.1 package http"
#error
2014/10/06 16:38:26 [error] 1675#0: *8 no user/password was provided for basic authentication, client: <IP_ADDRESS>, server: ...net, request: "POST /users/ HTTP/1.1", host: "...net"

docker version:
Server:
docker-1.2.0
docker-registry-0.8.1

Client:
docker-1.2.0

Am I doing something wrong?

I've had this issue when using mirroring in the docker registry container. Mirroring and basic auth do not work together, so turning off mirroring fixed the issue for me. See
https://github.com/docker/docker-registry/issues/576#issuecomment-56017584

There are lots of issues about this and how to fix https://github.com/docker/docker/issues/8557 https://github.com/docker/docker/issues/8557 https://github.com/docker/docker/issues/8132 and many more, but I think need help with registry setup please open an issue there or checkout #docker on IRC

I have this issue as well.  And it seems it's related to how you access the registry.
In my situation I have a local registry that is made public.  It works when accessed by the public IP.
It has a real cert attached so no need for insecure-registry flag.
However, If you try to access the registry using a re-written DNS record that resolves to a private IP in the RFC1918 range that it stops doing basic auth.


Admin loses rights for moved group

I logged in as admin and then created a top-level group and added a child group to it. Then I edited the child group and set its parent to "[None]". After that I was not able to see this group anymore under passwords. I could also not grant permissions for this group to myself. I had to manually edit the database to grant permissions for this group to the admin user again.

Confirming I see that issue here.


[WIP] Do not store histogram on SplitInfo

Tentative fix for #31.

At this time some tests are broken (need to be updated) and the code runs slightly slower than on master for some reason I do not understand yet.
The memory leak issue should be fixed though.

The fact that we observe a slowdown when we update the packed histograms array in the parallel for loop while we do not observe this issue in master where the packed histograms array is filled sequentially might be a case of False Sharing.

Actually I tried again with master and the various threading backends and this PR is either as fast or faster than master. I must have done something wrong when I reported the initial slow down.
In any case, tbb is significantly faster than the workqueue backend when the number of cores is large (e.g. 12 in my case).

@NicolasHug I have to go, feel free to update the failing tests and merge this PR.

Cool, let's merge as it's already a net improvement.
About the minimal reproduction case, I confirm I get the leak with your code, without any error message or exception. Just memory usage increasing as reported by psutil.

We still have a discrepancy in terms of results with LightGBM though. But there is another issue for that.

I have opened https://github.com/numba/numba/issues/3473 and https://github.com/numba/numba/issues/3472 regarding the leak and some other weird stuff I found.


Do not remove rules, only deprecate them

We had already removed some rules in the past (with some sort of deprecation period). In the future it will be better to 'move' rule to deprecation forever instead removing it. Our documentation will be more consistent (the rules will not 'dissapear' suddenly when switching between versions) and there will no sudden blank space in rules ids:
    "080X": Rule(
        rule_id="080X",

    ),
    "080X+1": Rule(
        rule_id="080X+1",
        deprecated=True,
    ),
    "080X+2": Rule(
        rule_id="080X+2",

    ),

If the rule is marked as deprecate, it will show warning when rule is used in --include/--exclude (same as with current deprecation solution but it will be handled on rule definition level, not on configuration level). The rule implementation could be removed.

Our documentation should also reflect it by some sort of visual mark (grayed out rule etc).

Nice perk is that it will allow users to deprecate their own rules - if company has multiple custom rules and want to deprecate one without breaking Robocop with older configurations (that still uses deprecated rule).


upgrade k8s & adopt client-go

We would love to have client-go for the k8s calls, and also upgrading k8s.
This work is being part of PR #116 so when the PR getting merged we can close this.

Solving at https://github.com/bitnami/kubeless/pull/122

Close as the PR #122 was merged


Created foxglove folder

Created foxglove folder and 2 files. First streams rgb image to foxglove studio, second streams pointclouds to foxglove studio.

Sweet thanks @ZigiMigi !

when testing point cloud in foxglove you will see that the point cloud is flickering, that is not a problem on our side, but on foxglove's side (as they have a problem with their 3D panel)

Fixed argument setting, new PR here: https://github.com/luxonis/depthai-experiments/pull/370


Include example fasta files into github.

Could you include the example files (especially amp_seq_fasta example) into your github ?

Hi @jkbenotmane ,
did you work this out?  if you did could you provide an example run?  I too am having trouble working out exactly what is expected in the --amp_seq_fasta file.
Thanks

The --amp_seq_fasta is a plain text file that gives the full sequence of the expected amplicon (i.e. the sequence between the forward primer and the backward primer, including the primer sequence but excluding the 5' barcode sequence)
A zipped --amp_seq_fasta file is here:
htt_amplicon3.zip
This is a region in the HTT gene and AAAACGAGGGTTGTCAAAGACCCCA is the forward primer and GCAGTAGCTTGCGTTATCAGGTTTGT is the backward primer. The sequence is not case-sensitive. I converted the primer sequence to uppercase so that it is more human-readable.
If you have more questions, please let me know.


Feature Request: Update README "Get Started" to include YARN details along with existing NPM details.

PROBLEM
Currently, the react-slingshot README section "Get Started" provides instruction details which assumes the user to be utilizing NPM as their preferred package-manager for Node. However, if the user follows the instructions as written, but utilizes YARN instead of NPM, then it is not immediately obvious what to do, and more importantly, the final step in the process fails. YARN does not recognize the -s flag to enable silent mode as NPM does, and instead requires a different flag of --silent to successfully enable silent mode on script run. See ADDITIONAL CONTEXT for broader explanation as to why this is a relevant problem that should be fixed.
SOLUTION
Update README section "Get Started" to include the the NPM equivalent YARN scripts alongside the already provided NPM scripts listed.
ALTERNATIVES
Seems like this is pretty straight forward.
ADDITIONAL CONTEXT
React-Slingshot supports YARN already. Can't see why this wasn't just included when support was added.
See: #319 and #292

Closing as per comments in #610.


fix: lost the 'installing dependencies' message on all loads.

Implemented a falsy check to verify if the message is displayed. Could someone please double-check this? 🙏

@mohandast52 works for me 🙏


Event Request: Publisher in DeleteRelatedData on Item Table

In a modification we have a new subsidiary table to the Item table, it holds data similar to how the Item Vendor table works.   If the item is deleted after data is entered into the subsidiary tables, we need the data in the subsidiary tables to be deleted too.   We notice that yMicrosoft has moved the code for deleting subsidiary table data to a local function called DeleteRelatedData in the item table.
We have found that if we add code to a subscriber of the OnAfterDelete publisher on the item table, to delete our subsidiary table data when the item is deleted, that works when you are actually deleting the item.   However, the subsidiary data is also deleted when the item is processed by the Inventory Adjust Cost process, which is not what should happen.
Our observation is that when the code for deleting the subsidiary table data is in the DeleteRelatedData function, the subsidiary data is not accidentally deleted when the Inventory Adjust Cost process is run.   When we add code for deleting data for our subsidiary table into the DeleteRelatedData, then our data is not accidentally deleted because of the Inventory Adjust Cost process.
We are requesting a “OnAfterDeleteRelatedData” publisher be added by Microsoft in the DeleteRelatedData function on the item table, so that we can move our code to a subscriber that uses the new publisher.   We have tested this by creating a publisher in the DeleteRelatedData and it does not delete our data.

I'm curious - how did you bypass the issue templates which show that event requests are no longer accepted on this forum?
This repository is no longer dedicated to handling application bugs, application requests and application enhancements.
Instead please use the following repository: https://github.com/Microsoft/ALAppExtensions


Add types for ref

Adding some types for TextTickerRef for more added typesafety :).
Also minor change was to remove generic variable T, as it wasn't being used.

Big thanks for your PR's! Appreciate the work you've done here and I'll merge them and push out a minor release with them soon.


[BUG] Exponential speed increase issue with Colytra

Version of Advanced Rocketry

1.12.2-1.6.0-180

Have you verified this is an issue in the latest unstable build

[x] Y
[] N
[] N/A

Version of LibVulpes

1.12.2-0.4 1-62

Version of Minecraft

1.12.2

Does this occur with other mods installed

[x] Y
[] N
[] N/A

Colytra  (1.12.2-<IP_ADDRESS>)
Baubles (1.12-1.5.2)

Crash report or log or visualVM (if applicable)
http://pastebin.com is a good place to put them
crash reports that are put in the issue itself are
hard to read

No crash

Description of the problem
Equip a space suit with jet pack module. Equip bauble Elytra. Activate the bauble Elytra and fly.
Speed increases exponentially. I initially travelled over 300k blocks in about a minute. My PC finally gave up loading chunks....
This does not occur when the space suit chest is un-equipped. It does not occur with other armor. It also does not occur if the jet pack is not installed. It does occur regardless of whether the jet pack is enabled/disabled.

Is this with only the jetpack module with no speed upgrades in the helmet?  Also does this occur in hover mode as well or just normal mode?

I didn't even know there were speed modules, lol. The chest was the only piece with upgrades. It had 2 oxygen tanks, the jet pack, and a tank for jet pack fuel.
I was not in hover mode to my knowledge. I simply activated the jetpack (base keybind x, hud turned green), then double space bar while falling to start flying with the elytra.

Here's with the hover upgrade installed.
https://streamable.com/bxdjn
Doesn't really matter what state the chest is in though. If the jet pack is installed, this happens. Only way to stop it is to remove the chest plate/jetpack. Works fine with a space suit chest with no jetpack though.

Hmm.... what version  of forge?  I'm just getting a straight up crash from Colytra on forge-1.12.2-<IP_ADDRESS>60

You should add in the forge version as well.
On Tue, Sep 10, 2019, 10:39 PM zmaster587<EMAIL_ADDRESS>wrote:

Hmm.... what version of forge? I'm just getting a straight up crash from
Colytra on forge-1.12.2-<IP_ADDRESS>60
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/zmaster587/AdvancedRocketry/issues/1696?email_source=notifications&email_token=AGVDBJFWFUTH7WP5LHNPIVLQJBR53A5CNFSM4IVB46V2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6NE23Q#issuecomment-530206062,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGVDBJGUGRQGY37REPVWFQDQJBR53ANCNFSM4IVB46VQ
.

Nevermind, I got it, <IP_ADDRESS>24 works

So it turns out there was an off-by-one where it would give you one invisible speed upgrade.  Speed upgrades work by multiplying player velocity by some factor greater than one and it eventually hits an equilibrium with "friction".
Specifically
		//Apply speed upgrades
		player.motionX *= 1 + speedUpgrades*0.02f;
		player.motionZ *= 1 + speedUpgrades*0.02f;

But it looks like when an elytra is worn it seems the friction isn't calculated so you get infinite speed.

fixed for next build.


Use bulleted list for Emacs links

The links in the HTML tutorial were rather difficult to read. This PR uses a bulleted list for the links, uses the actual names of the sources, and dispenses with the visible URL.
I'm not sure how this will look in the PDF rendering. I see there that links have a visibly different color, so it should be obvious that these are links.

Merged by 4476eda53c4f8c4c51c96392dd72a722d6028ae7


Feature: Scorecard policy that helps evaluate results for GCP Binary Authorization

Current Behavior
Today, scorecard's policies can be used to enable/disable certain checks. As per #1405 , the --policy flag was disabled in favor of implementing a settings-based policy file.
Desired Behavior
One motivation for scorecard policies is to use them to make decisions about the runtime authorization (e.g. Google Cloud Binary Authorization) for an application. For example, an operator may want to use scorecard to implement the rule Don't run this application unless the SCM repo used to build this application is free from Binary Artifacts AND unless the SCM repo mandated two-party review.
We can think of 'policies' as a way of using all of scorecard's data to answer questions an operator may have about an application, as opposed to what is done today, which uses policies to change how scorecard behaves.
One possible approach would take heavy inspiration from the way Allstar policies are written.
This is linked to #1245.

Thanks! I wonder - the yaml-based approach may have some overlap with the proposed "SECURITY-INSIGHTS" document, per https://github.com/ossf/security-insights-spec.

@david-a-wheeler Thanks for bringing that project to my attention! My initial thought is that the threat model of Security-Insights is different from that of a Binary Authorization policy (or other deployment policy). Whereas Security-Insights is the set of insights about a specific repo declared by the repo owner, the Binary Authorization policy would be a set of standards for any repo declared by a cloud deployment gatekeeper. If repo owners controlled the Binary Authorization/deployment policy, then they could upload a policy that would always pass and always allow deployment, even if the Scorecard score was low and the practices followed were poor.
However, if we were to use this schema on the deployer-side, an allstar-style policy would be easier to use because it would expresses security requirements more directly. For example, if I wanted to express "The code in an artifact must require 2-party review in its SCM in order to be deployed," in an allstar-style policy format, this would be a one-line change (ApprovalCount: 2), and the implementation of the policy logic would fall on Scorecard maintainers. Using Security-Insights means Scorecard can be more generic and not maintain as much custom logic for each check, but I think expressing the same requirement as a YAML (even if it was sparse, and we did something like semantic YAML matching) would be a higher bar for the end-users of this feature.

@laurentsimon @naveensrinivasan @justaugustus - Raghav is ready to start work on this. Please give a shout if there are any concerns here.

This work works well on branch protection because each result is "named", but not well for other checks like token-permissions or pinning. What's the plan for the others?

IMO some checks like Branch-Protection, Token-Permissions shouldn't be exposed at all through this policy. I don't see these checks being useful to gate an image built at commit X. The list of checks to expose here IMO are: Bin-Artifact, CI-Tests, Code-Review, Pinned-Deps, SAST and Vulnerabilities (atleast for the initial release).
And for checks that require an ignoreList (Pinned-Deps, Bin-Artifacts) how about something like:
ScorecardPolicy struct  {
  // In YAML, specify:
  // allowedUnpinnedDependencies: nil // Disable this check.
  // allowedUnpinnedDependencies: []  // Enable this check and apply on all.
  // allowedUnpinnedDependencies: // Enable and apply on all except provided exceptions.
  //   - dependency:
  //        filepath: <>
  AllowedUnpinnedDependencies []Dependency{
     Filepath string
     PackageName string
     Version string
  }
  //
  AllowedBinaryArtifacts []FilePath
}

Wdyt?

IMO some checks like Branch-Protection, Token-Permissions shouldn't be exposed at all through this policy.

do you mean Dependency-Pinning or Branch-Protection? (I'm asking because @raghavkaul 's initial description is heavy on the Branch-Protection check.) I understand that the check is not bound to a commit / source code, though. If this is better suited for AllStar, does this mean we'll ask users to install both? Or will AllStar encompass all the policies we declare here?
I don't see these checks being useful to gate an image built at commit X. The list of checks to expose here IMO are: Bin-Artifact, CI-Tests, Code-Review, Pinned-Deps, SAST and Vulnerabilities (atleast for the initial release).
It's not clear why, for example, Token-Permissions could not be exposed. What's the reasoning?

And for checks that require an ignoreList (Pinned-Deps, Bin-Artifacts) how about something like:
ScorecardPolicy struct  {
  // In YAML, specify:
  // allowedUnpinnedDependencies: nil // Disable this check.
  // allowedUnpinnedDependencies: []  // Enable this check and apply on all.
  // allowedUnpinnedDependencies: // Enable and apply on all except provided exceptions.
  //   - dependency:
  //        filepath: <>
  AllowedUnpinnedDependencies []Dependency{
     Filepath string
     PackageName string
     Version string
  }
  //
  AllowedBinaryArtifacts []FilePath
}

Wdyt?

If this is better suited for AllStar, does this mean we'll ask users to install both? Or will AllStar encompass all the policies we declare here?

Scope here does not include defining a generic policy. This policy definition will only be used for expressing a BinAuthz policy.

It's not clear why, for example, Token-Permissions could not be exposed. What's the reasoning?

It could be exposed, I mean they don't need to be because there doesn't seem to be a strong usecase for a BinAuthz client to gate on Token-Permissions.

The main reason to keep Branch-protection out of the checks is that it's point in time, so it makes sense for Allstar because Allstar checks it during merge-time, but for a deployment-time check, Binary authorization could have been enabled post-hoc.
Also agree with the first set of checks implemented into policy to be:
Bin-Artifact, CI-Tests, Code-Review, Pinned-Deps, SAST and Vulnerabilities

This work works well on branch protection because each result is "named", but not well for other checks like token-permissions or pinning. What's the plan for the others?
It's case by case - everything we express in YAML may have a different check format depending on what makes sense for a deployer to gate on


Enable db timeouts and WAL mode

I finally got around to testing LotMan in a multi-process environment with many processes trying to simultaneously read/write to the database. While the underlying sqlite library is technically process safe, it didn't handle the testing well and wound up throwing lots of "SQLITE_BUSY" and "database locked" errors. After a bit of researching, I settled on handling this by adding timeouts that let database connections queue for a certain time before they give up, and by enabling WAL (Write- Ahead-Logging) mode. According to the sqlite docs (here)[https://www.sqlite.org/wal.html], WAL mode provides better concurrency as readers do not block writers and a writer does not block readers.

Normally I'd say -- yes! Let's test!
But in this case, I think this is a particularly hard thing to test adequately. When the database is receiving many requests across processes, setting the timeout depends on everything from processor speeds, to disk writing speed, to the number of processes and what they're trying to do. A test running through github actions might behave differently than a local test.


This is my idea

What would you like to be added?
Crowdsec should detect all the things
Why is this needed?
To be the best

/kind feature

This feature sounds great!
/triage accepted