updated web frameworks to include others

What kind of changes does this PR include?
x Minor content fixes (broken links, typos, etc.)
x New or updated content
Description

Closes #2066
What does this PR change? This pr updates the content-focused tip section to include other community frameworks besides Next.js.

Thanks @brittneypostma! Chatted with the team a bit about this and we're going to keep it to a single framework for now for simplicity. However, I went ahead and updated the text to be a little less "picking favorites" of Next.js over other frameworks and more "suggesting one alternative" which should address the concern a bit that we were unintentionally suggesting Next.js is better than SvelteKit or others.


GASNet-enabled Realm compiles should use GASNET_* compile/link flags

It appears that Legion is not currently using the GASNet-provided compiler flags for compiling modules that #include<gasnet.h>, or for linking libgasnet.
With a debug-mode GASNet, the lack of GASNet's compiler flags leads to a large number of warnings that we intentionally suppress (mostly notably -Wunused) when compiling with gcc < v6.0. The GASNet makefile fragments and pkgconfig files provide $(GASNET_CXXCPPFLAGS) $(GASNET_CXXFLAGS) that include these warning suppression flags, as well as other flags that are needed on some platforms (eg to enable inline assembly support, C99 support or POSIX support in some exotic compilers). There are likely cases that will fail to compile at all in the absence of these configure-detected flags.
GASNet also provides linker flags (in GASNET_LDFLAGS and GASNET_LIBS) that include everything (notably -l and -L options) needed to correctly link the current conduit and related network libraries. These often include things like custom pathnames to network driver installs provided by the user when he configured GASNet (eg when building against an IB verbs or OFI library that was installed in a custom location, ie not /usr/lib).  These variables also do not currently appear to be used by Realm links. Currently runtime/runtime.mk appears to be hard-coding values for several conduits (ibv, gemini, aries, psm, mpi and udp), however this list excludes several conduits (pami, ofi, mxm) and in general is a fragile approach as it discards any settings detected or provided by the user when configuring GASNet. This is very likely to cause link failures on many configurations.
These flags are documented in the section "Basic Usage Information" of the GASNet README. We provide the settings via Makefile fragments and equivalent pkgconfig data files.
I see some code in cmake/FindGASNet.cmake that appears to be querying these flags, but they don't currently seem to make it to the compile line.
We'd like to encourage the Realm team to find a way to use our provided flags, and are willing to help if there's anything we can do on our side to make this integration work as intended.
CC: @phhargrove

Fixing this issue should also address this warning that we're currently getting on GASNet builds:
mpicxx -o /home/mebauer/legion/runtime//realm/activemsg.cc.o -c /home/mebauer/legion/runtime//realm/activemsg.cc -DENABLE_REMOTE_TRACING -DCTRL_REPL_TRACING -DUSE_DISK  -march=native -DUSE_ZLIB -O0 -ggdb  -Wall -Wno-strict-overflow  -I/home/mebauer/legion/runtime/ -I/home/mebauer/legion/runtime//mappers -I/home/mebauer/legion/examples/circuit -I/usr/local/gasnet-1.32.0-openmpi/include -I/usr/local/gasnet-1.32.0-openmpi/include/ibv-conduit
In file included from /usr/local/gasnet-1.32.0-openmpi/include/gasnet.h:177:0,
from /home/mebauer/legion/runtime//realm/activemsg.cc:31:
/usr/local/gasnet-1.32.0-openmpi/include/ibv-conduit/gasnet_core_fwd.h:24:0: warning: "GASNET_CONDUIT_IBV" redefined [enabled by default]
#define GASNET_CONDUIT_IBV       1
^
In file included from /home/mebauer/legion/runtime/realm/realm_config.h:23:0,
from /home/mebauer/legion/runtime//realm/activemsg.cc:16:
/home/mebauer/legion/examples/circuit/realm_defines.h:16:0: note: this is the location of the previous definition
#define GASNET_CONDUIT_IBV
^

The above warning has now been fixed.


[gym] skip_archive option leads to No output path received from gym

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description

I am using Fastlane Gym to build my app for my Detox tests and for my ad-hoc distribution builds. I use fastlane for both so that I re-use the data in derived_data directory (allowing me to easily cache it in my CI).
The problem is that once I have the flag skip_archive: true, I systematically get the error No output path received from gym.
Command executed

platform :ios do
  lane :build do |options|
    gym(
      configuration: "ReleaseStaging",
      output_directory: "ios/build",
      output_name: "MyApp.ipa",
      workspace: "ios/MyApp.xcworkspace",
      export_method: "ad-hoc",
      derived_data_path: "ios/derived_data",
      skip_archive: ENV['DETOX'] ? true : false,
    )
  end
end

Complete output when running fastlane, including the stack trace and command used

bundle exec fastlane ios build
[...]
[17:54:45]: ▸ Running script '[CP] Copy Pods Resources'
[17:54:45]: ▸ Running script '[CP] Embed Pods Frameworks'
[17:54:46]: ▸ Touching MyApp.xctest
[17:54:46]: ▸ Signing derived_data/Build/Products/ReleaseStaging-iphoneos/MyApp.app/PlugIns/MyApp.xctest
[17:54:46]: ▸ Signing derived_data/Build/Products/ReleaseStaging-iphoneos/MyApp.app
[17:54:47]: ▸ Build Succeeded
[17:54:47]: No output path received from gym
+------+-------------------------------------+-------------+
|                     fastlane summary                     |
+------+-------------------------------------+-------------+
| Step | Action                              | Time (in s) |
+------+-------------------------------------+-------------+
| 1    | Switch to ios load_env_file lane    | 0           |
| 2    | Switch to ios check_env_exists lane | 0           |
| 3    | gym                                 | 64          |
+------+-------------------------------------+-------------+
[17:54:47]: fastlane.tools finished successfully 🎉

Environment

✅ fastlane environment ✅
Stack

Key
Value

OS
12.0

Ruby
3.0.3

Bundler?
true

Git
git version 2.30.1 (Apple Git-130)

Installation Source
~/.rbenv/versions/3.0.3/bin/fastlane

Host
macOS 12.0 (21A344)

Ruby Lib Dir
~/.rbenv/versions/3.0.3/lib

OpenSSL Version
OpenSSL 1.1.1l  24 Aug 2021

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
13.1

Swift Version
5.5.1

System Locale

Variable
Value

LANG
fr_FR.UTF-8
✅

LC_ALL

LANGUAGE

fastlane files:
`./fastlane/Fastfile`
fastlane_require 'dotenv'

import('../../common/fastlane/FastfileConfig')
import('../../common/fastlane/FastfileCheck')
import('../../common/fastlane/FastfileEnvironment')
import('../../common/fastlane/FastfileIOSSigning')
import('../../common/fastlane/FastfileCommon')

No Appfile found
fastlane gems

Gem
Version
Update-Status

fastlane
2.199.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.5.0

bundler
2.2.32

pathname
0.1.0

rake
13.0.6

rexml
3.2.5

CFPropertyList
3.0.5

concurrent-ruby
1.1.9

i18n
1.8.10

minitest
5.14.4

tzinfo
2.0.4

zeitwerk
2.5.1

activesupport
<IP_ADDRESS>

public_suffix
4.0.6

addressable
2.8.0

httpclient
2.8.3

json
2.6.1

algoliasearch
1.27.5

artifactory
3.0.15

atomos
0.1.3

aws-eventstream
1.2.0

aws-partitions
1.540.0

aws-sigv4
1.4.0

jmespath
1.4.0

aws-sdk-core
3.124.0

aws-sdk-kms
1.52.0

aws-sdk-s3
1.109.0

babosa
1.0.4

claide
1.0.3

fuzzy_match
2.0.4

nap
1.1.0

netrc
0.11.0

ffi
1.15.4

ethon
0.15.0

typhoeus
1.4.0

cocoapods-core
1.11.2

cocoapods-deintegrate
1.0.5

cocoapods-downloader
1.5.1

cocoapods-plugins
1.0.0

cocoapods-search
1.0.1

cocoapods-trunk
1.6.0

cocoapods-try
1.2.0

colored2
3.1.2

escape
0.0.4

fourflusher
2.3.1

gh_inspector
1.1.3

molinillo
0.8.0

ruby-macho
2.5.1

nanaimo
0.3.0

xcodeproj
1.21.0

cocoapods
1.11.2

colored
1.2

highline
2.0.3

commander
4.6.0

declarative
0.0.20

digest-crc
0.6.4

unf_ext
0.0.8

unf
0.1.4

domain_name
0.5.20190701

dotenv
2.7.6

emoji_regex
3.2.3

excon
0.89.0

faraday-em_http
1.0.0

faraday-em_synchrony
1.0.0

faraday-excon
1.1.0

faraday-httpclient
1.0.1

faraday-net_http
1.0.1

faraday-net_http_persistent
1.2.0

faraday-patron
1.0.0

faraday-rack
1.0.0

multipart-post
2.0.0

ruby2_keywords
0.0.5

faraday
1.8.0

http-cookie
1.0.4

faraday-cookie_jar
0.0.7

faraday_middleware
1.2.0

fastimage
2.2.5

jwt
2.3.0

memoist
0.16.2

multi_json
1.15.0

os
1.1.4

signet
0.16.0

googleauth
1.1.0

mini_mime
1.1.2

trailblazer-option
0.1.2

uber
0.1.0

representable
3.1.1

retriable
3.1.2

webrick
1.7.0

google-apis-core
0.4.1

google-apis-androidpublisher_v3
0.14.0

google-apis-playcustomapp_v1
0.6.0

google-apis-iamcredentials_v1
0.9.0

google-apis-storage_v1
0.10.0

google-cloud-env
1.5.0

google-cloud-errors
1.2.0

google-cloud-core
1.6.0

google-cloud-storage
1.35.0

mini_magick
4.11.0

naturally
2.2.1

optparse
0.1.1

plist
3.6.0

rubyzip
2.3.2

security
0.1.3

simctl
1.6.8

terminal-notifier
2.0.0

unicode-display_width
1.8.0

terminal-table
1.8.0

tty-screen
0.8.1

tty-cursor
0.7.1

tty-spinner
0.9.3

word_wrap
1.0.0

rouge
2.0.7

xcpretty
0.3.0

xcpretty-travis-formatter
1.0.1

generated on: 2021-12-13

I had a look at the code, and it seems to be because of this:

Here, we check if a path is not null https://github.com/fastlane/fastlane/blob/605d2618da27cd63a088a0bdcbec78d7e0e5def1/fastlane/lib/fastlane/actions/build_app.rb#L69
But in the runner that returns this path, we explicitely say "if skip_archive is enabled, then return nil" https://github.com/fastlane/fastlane/blob/605d2618da27cd63a088a0bdcbec78d7e0e5def1/gym/lib/gym/runner.rb#L22

But after second thought... I just realised that this was actually expected ✅!
My problem was that Detox was not finding my .app. But Gym was actually generating it... I was simply not giving the proper path to Detox. Oops.
Should we document this as expected?
If not, we may close the issue!
In any case, I hope someone else will find this helpful.

I have kind a same issue with SharedValues::IPA_OUTPUT_PATH. It seems gym stopped exporting these envars in one of the latest fastlane versions

It seems that I have similar query, did you get the solution for this?

It seems that I have similar query, did you get the solution for this?

As I said, in my case it was not really a problem. Gym is just warning you that nothing useful was generated, which is true if you skipped the archiving of your .app to a .ipa.
In my case, I needed the .app to install it on a simulator, but this is not common at all. Fastlane's usage is meant for distribution, and a .app on its own is of no use for distribution.

It seems that I have similar query, did you get the solution for this?

I just rolled back to the old fastlane version (2.121.1)


Parse.File location retrieved is different from what is saved on the server

Note
I have moved recently from parse to heroku with mongolab
I'm running Parse version 2.0.3 (heroku doesnt let me install any version after 2.0.4, 2.0.4 doesnt run but install )
I'm aware I should run at least 2.0.7 but not sure if this is fixed so I/m asking you to check
Steps to reproduce:

Save a Parse File (an image in my case)
Return the url from the file

Results:
http://files.parse.com / myapidecrypted / f39283984902380942_myfilename.jpg
Expected:
Here I'm not sure at this point what should I get
is it http://myapp.herokuapp.com/1/files/myfilename.jpg   or something different ?    (this is what I get when I save the file via javascript client sdk )
Thanks in advance for checking this

+1 I'm facing the same hurdle. In my case, the localhost ParseServer results in
http://localhost:[port]/[parse-mount]/ [appId] / [file-name.extension]

Yeah, I solved that too, hacking actually the url, stripping out the "http://files.parse.com / myapidecrypted  and replacing with my info : " http://myapp.herokuapp.com/1/files/myapidecrypted/f39283984902380942_myfilename.jpg"
Done.
Case close until they have an official fix.
You cna do the same, via web and via ios/android etc.

Just want to note ( for anyone landing here with this problem ) that there was an official fix in v2.1.0


First Class Support For Finch

Hi, can you provide first class support for finch in addition to Docker as Docker has license issues now?

Hi @bestickley , we are currently working on Finch support. Will update soon.

Awesome. Thanks @begimher!


Add policy change output support

We now properly output results from apply and replace policy invocations

@dividedmind CR ready again


Set sympy.evaluate(False) on frequent code paths for phase tracking.

Summary
Resolves #5405 by disabling sympy evaluation on code paths that are hit frequently for parameterized global phase tracking.
Details and comments
I attempted to apply this more broadly (in ParameterExpression._apply_operation and ParameterExpression._call) which resulted in several failures for the gradient tests. Maybe this can be looked into more after the release.
N.B. This PR also introduces the potential for bugs/complications as the ParameterExpression._symbol_expr for .global_phase will become an unevaluated sympy expression, so some sympy behaviors which we may have come to depend on (e.g. commutativity of addition/multiplication) will no longer behave as they did previously. ~(The most likely consequence that comes to mind is false negatives for global phase equality.)~ (@levbishop confirmed that the changes to ParameterExpression.__eq_ in #5760 should make this a non-issue.)
Evaluating the global phase expression after the fact (via .doit()) resolves these issues but nullifies the performance fix. Likewise for collecting global phase contributions in the BasisTranslator and summing them in a single expression.
This PR is a short term fix at best, and will need some follow up. Would welcome any suggestions or alternatives.

I think this makes sense, since I think we should prioritize the performance of binding parameters over showing a nice expression for the global phase (of course if we can have both in the future that would be even better). Could you add a release note pointing out that the global phase is not simplified?

I think this makes sense, since I think we should prioritize the performance of binding parameters over showing a nice expression for the global phase (of course if we can have both in the future that would be even better). Could you add a release note pointing out that the global phase is not simplified?

Modulo performance differences(at what point the sympy simplification happens), this shouldn't have a user facing impact (unless someone is directly accessing ParameterExpression._symbol_expr). e.g.
>>> th = qk.circuit.Parameter('theta')
>>> qc = qk.QuantumCircuit(1)
>>> qc.rz(th, 0)
>>> out = BasisTranslator(sel, ['p'])(qc)
>>> out.global_phase
-0.5*theta
>>> out.global_phase == -0.5 * th
True
>>> out.draw()
global phase: -0.5*theta
     ┌──────────┐
q_0: ┤ P(theta) ├
     └──────────┘

I think this makes sense, since I think we should prioritize the performance of binding parameters over showing a nice expression for the global phase (of course if we can have both in the future that would be even better). Could you add a release note pointing out that the global phase is not simplified?

Modulo performance differences(at what point the sympy simplification happens), this shouldn't have a user facing impact (unless someone is directly accessing ParameterExpression._symbol_expr). e.g.
>>> th = qk.circuit.Parameter('theta')
>>> qc = qk.QuantumCircuit(1)
>>> qc.rz(th, 0)
>>> out = BasisTranslator(sel, ['p'])(qc)
>>> out.global_phase
-0.5*theta
>>> out.global_phase == -0.5 * th
True
>>> out.draw()
global phase: -0.5*theta
     ┌──────────┐
q_0: ┤ P(theta) ├
     └──────────┘

If you add another qc.rz(-th, 0) before BasisTranslator, won't the global_phase in the drawn circuit change from 0 without this PR to -0.5*theta + 0.5*theta with this PR? I guess can put a doit in the visualizations?

If you add another qc.rz(-th, 0) before BasisTranslator, won't the global_phase in the drawn circuit change from 0 without this PR to -0.5*theta + 0.5*theta with this PR? I guess can put a doit in the visualizations?

That's a good point (this came up in discussions with @Cryoris as well). Given that the simplification is not behavior we're entirely settled on, I can add a note pointing out that a parameterized global phase might no longer be simplified after transpilation.

Regarding the introduction of potential bugs, if that happens, it would be frustrating not being able to recover the original behavior. Why not adding a parameter (i.e. evaluate_symbolic_global_phase) to force the previous behavior?


Avoided switching on and off shift register pin

Found out that the buzzing problem on the 16ch board was due to the frequent switching on and off of the shift register pin.
Avoiding that practice makes the board completely silent and hopefully saves the relays from early death. On the 8ch board should be the same, but since the frequency is way higher the problem might be not easy to spot.

@kecajtop I tested the changes made by @AlessandroTischer and can confirm that the buzzing sound is gone.
cheers

What is the cause of the noise based only on switching the swift register?

Found out that the buzzing problem on the 16ch board was due to the frequent switching on and off of the shift register pin. Avoiding that practice makes the board completely silent and hopefully saves the relays from early death. On the 8ch board should be the same, but since the frequency is way higher the problem might be not easy to spot.

What is the cause of the noise based only on switching the swift register?
I Tested on 8ch and worked. No more noise

What is the cause of the noise based only on switching the swift register?

The fact that the register output was turned off before updating, so the relays were continuously switched too. It was very bad for the relays health actually

I Tested on 8ch and worked. No more noise

Glad to read this!

What is the cause of the noise based only on switching the swift register?

The fact that the register output was turned off before updating, so the relays were continuously switched too. It was very bad for the relays health actually

I Tested on 8ch and worked. No more noise

Glad to read this!

Your changes also work well if you remove all the delays.
This turns the output change faster (in case of multiple inputs changing at same time)


isabelle: 2022 -> 2023

Description of changes
Changelog file
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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 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.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2684

@SuperSandro2000 can this be merged?

In the meantime a release of the linter was tagged, updated the commit accordingly

Would it make sense to also include the AFP as a component? I defined this derivation for it: https://github.com/lukasstevens/dotfiles/blob/a971a4c0bee283d27351413e1cc47e239812ad17/nix/isabelle/components/afp.nix

@SuperSandro2000 can we merge this now?

@SuperSandro2000 sorry for the ping, but this has been sitting for a while now

@ofborg build tlaps tlaps.passthru.tests

Thanks!


.Net: ADR - Agent Architecture

Motivation and Context

A proposal for how to support agents of different modalities.
Description

In progress / open for comments
Contribution Checklist

[ ] The code builds clean without any errors or warnings
[ ] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[ ] All unit tests pass, and I have added new tests where possible
[ ] I didn't break anyone :smile:

Will revisit when synching to agent framework


Remove the "explanation" sentences from the slo report tabbed view

cf #169

PR is closed, closing  the issue


rename master branch

Should rename the default branch to "main" and update the developer guide.
Remove CONTRIBUTING.md file to and move content to wiki page.
References

Rename default branch how-to

renamed default to "main"
move CONTRIBUTING.md to wiki & deleted


How to tear down?

I want to know how to teardown/destroy this component.
EDIT:
Apparently clearing the highest timeout id does it. You might want to consider adding a destroy method likes this:
Sub.prototype.destroy = function() {
  clearTimeout(this.highestTimeoutId);
};

Definitely good to add.


Merge in GIMONS 1.50 changes

GIMONS recently released version 1.50 of the Japanese version. The 68kmla version of RaSCSI was based upon version 1.47.
1.50 appears to have some bug fixes and added "Burst" support.
Version 1.50: http://retropc.net/gimons/rascsi/rascsi.zip
Version 1.47: http://retropc.net/gimons/rascsi/rascsi147.zip

Something that seems to have an implied answer here, but good to ask specifically.  Is there going to be a plan to have a streamlined common core code base or architecture that well supports both projects?  Maybe even possibility for new collaboration between the two?  It's good to see GIMONS sites are back on the modern Internet for one thing.

Currently, there are no plans. I'm definitely open to the idea. But, I have a limitation that I don't speak or read Japanese. The original code was all in Japanese.
I'm hoping that the two projects can share feature/bug fixes/etc, but for now, I think its going to be a manual process.

version 1.52 is out - http://retropc.net/gimons/rascsi/rascsi152.zip

@akuker Would it help this effort if you had a translated diff between 1.47 and 1.52?  I'd be happy to volunteer and make use of my mostly useless degree in Japanese. ;)
I glanced through a diff just now, and it seems to be less than a weekend's worth of translation between documentation and code comments.

This would be super awesome. Just to add, GIMONS is very responsive via email if you have questions. They are happy for the projects to co-exist, and I wouldn't hesitate to reach out if additional information was needed.

@phrax0 Any preferred diff format, before I dig into it?  Linux unified diff?

@rdmark I don't know enough to answer that question.  @akuker is the better person to ask.

Alright so digging a little deeper, this is my plan:
bin/ -> out of scope
doc/ -> translate a diff; only a few paragraphs in total
src/raspberrypi/.h|.cpp -> translate all files (a diff was just too noisy / too low in context to be translatable)
src/raspberrypi/BAREMETAL/.h|.c -> translate main.c which is the only file that seems to have Japanese code comments
src/x86k/* -> translate all files (very little content)

Some progress today:
I realized disk.cpp was just too ridiculously long to retranslate in full, so I went with a diff with 5 lines of context instead.
Here is a complete first pass of disk.cpp: https://github.com/rdmark/RASCSI-translate/blob/master/src/raspberrypi/disk.cpp.diff
It contains the implementation of "burst mode" which might interest y'all.
Also picked off some low-hanging fruit:
Fully translated BAREMETAL: https://github.com/rdmark/RASCSI-translate/blob/master/src/raspberrypi/BAREMETAL/main.c
Fully translated x68k/RASCTL: https://github.com/rdmark/RASCSI-translate/tree/master/src/x68k/RASCTL
Please let me know if this is helpful!

Translated the header file for disk too:  https://github.com/rdmark/RASCSI-translate/blob/master/src/raspberrypi/disk.h.diff
@uweseimet Since you're actively working in this part of the code, do you have an opinion on the best way to tackle evaluating / merging these upstream changes?

@rdmark I don't think there are any untranslated texts anymore in disk.h. Merging something from old sources into the current develop tree is almost impossible, because there were really a lot of changes.

@uweseimet Well noted, this is also the impression I got from going through Gimon's code while glancing at the state of develop. In parts hardly recognizable as a fork of the other.
Regardless, I completed the translation of all code changes, just in case there are some valuable refactors, fixes or new functionality that can benefit this project. (e.g. Burst mode)
When comparing 1.52 with 1.47, I noted these major code organizational changes:

cfilesystem.cpp|h renamed to fsdriver.cpp|h
ctapdriver.cpp|h renamed to netdriver.cpp|h
xm6.h renamed to rascsi.h
Code in log.h, scsi.cpp|h have been folded into disk.cpp|h

As for doc/, I plan to retranslate those from scratch in https://github.com/akuker/RASCSI/issues/208

As for doc/, I plan to retranslate those from scratch in #208

Something to consider…. This project has evolved and the original Japanese documentation might not match the current implementation.
@uwesiemet has been keeping the Man pages up to date. Is there anything in the originally documentation that isn’t talked about in the man pages (or on the wiki somewhere)

@akuker Eyeing through the Japanese text of the three txt files, I'd say there is a good amount of X68000 specific documentation that I don't see in the wiki.  Apart from that, this project has either diverged enough for it to be inapplicable, or it's basic stuff that's in man pages or the wiki.

One more step to resolve this ticket:  a PR to merge the C code and Makefile that builds a Human68k 'RASCTL.X' executable for X68000. Untested since I don't have a X68000 development environment set up. I still think it's worth to add this to the repo for completeness sake.
https://github.com/akuker/RASCSI/pull/255

@akuker With the work done here at this point, do you think there anything else to be done in this ticket?

Adding rasdump/sasidump documentation in https://github.com/akuker/RASCSI/pull/295

Closing this ticket, with remaining unmerged feature "burst mode" as a Won't Fix: It is impossible to merge code as-is, and there unknown impact on various timing-sensitive platforms out there by introducing batched GPIO operations.


Admin Panel Cannot add user or anything, Gives error on console.

Uncaught Unknown button type: export
admin.js:sourcemap:29229

can you tell us to reproduce this issue?

This link works perfectly the user site: http://cms.simple.local:8888/public/
Then I went here at admin site: http://cms.simple.local:8888/public/auth/login
username<EMAIL_ADDRESS>password: 123456
Admin Panel showing successfully. Pop show that Google analytics code should be applied then disappear.
Then I added a new user. It did added in the database, but not showing in the admin user table.
Error in console is: Uncaught Unknown button type: export admin.js:sourcemap:29229
Want some images? contact me at<EMAIL_ADDRESS>

@ozdemirburak How about this, you think you can help me?

Probably admin.js is not being loaded properly, that's why.
If you followed the instructions, then it should've already installed and compiled assets needed. But this error tells you that, it can't find the buttons component of datatables, that's why, it's not being rendered.

@kenntinio Sorry, you were right, if the vendor files of the datatables package don't get published, then one may come across with this error.
I've updated the webpack file now, so server side buttons are included whether you publish the datatables assets or not.
Please reopen the issue if the error still persists.


Consider making redux-saga a store enhancer.

Sagas must be registered with run, but that's only exposed through the instance of saga middleware.  While it's pretty trivial if you're registering all of your sagas when you set up the store
const sagaMiddleware = createSagaMiddleware();
const store = createStore(reducer, applyMiddleware(sagaMiddleware));
sagaMiddleware.run(mySaga);
sagaMiddleware.run(myOtherSaga);

it gets complicated if you want to register them dynamically; you have to either hold onto the instance of middleware
// sagaMiddleware.js
export default createSagaMiddleware();

// store.js
import sagaMiddleware from './sagaMiddleware';
const store = createStore(reducer, applyMiddleware(sagaMiddleware));

// somewhereThatHasAccessToStoreAndWillDynamicallyRegisterASaga.js
import sagaMiddleware from './sagaMiddleware';
sagaMiddleware.run(mySaga);

or patch run into the store.
// store.js
const sagaMiddleware = createSagaMiddleware();
const store = createStore(reducer, applyMiddleware(sagaMiddleware));
store.runSaga = sagaMiddleware.run;

// somewhereThatHasAccessToStoreAndWillDynamicallyRegisterASaga.js
store.runSaga(mySaga);

The latter option is what boilerplates like react-boilerplate seem to do, so it seems that, to use it dynamically, users have to do a little work to upgrade it to an enhancer.
So, like, how about just making it an enhancer?

Could u propse exact API that you have in mind? Also please read this discussion to see about different approaches that were implemented previously and had some serious flaws we were trying to get rid off.

Something like
// store.js
const enhancer = sagaEnhancer();
// enhancer adds `runSaga` property to store.
const store = createStore(reducer, enhancer);

// somewhereThatHasAccessToStoreAndWillDynamicallyRegisterASaga.js
store.runSaga(mySaga);

I am not sure how that conversation relates, so I might be missing something.

And here's a sample implementation I've been using
import {emitter} from 'redux-saga/lib/internal/channel';
import {runSaga} from 'redux-saga/lib/internal/runSaga';
import {ident} from 'redux-saga/lib/internal/utils';

export default function sagaEnhancer({context = {}, ...options} = {}) {
  const {sagaMonitor, logger, onError} = options;
  // Validation removed for brevity.
  return createStore => (reducer, preloadedState, enhancer) => {
    const store = createStore(reducer, preloadedState, enhancer);
    const {dispatch, getState} = store;
    const sagaEmitter = emitter();
    sagaEmitter.emit = (options.emitter || ident)(sagaEmitter.emit);
    const run = runSaga.bind(null, {
      context,
      subscribe: sagaEmitter.subscribe,
      dispatch,
      getState,
      sagaMonitor,
      logger,
      onError,
    });
    const newDispatch = action => {
      const result = dispatch(action);
      sagaEmitter.emit(action);
      return result;
    };
    const setContext = props => {
      Object.assign(context, props);
    };
    return {...store, dispatch: newDispatch, runSaga: run, setContext};
  };
}

If this looks okay, I can file a PR or something.

This proposal is to mutate the store object, and I think we shouldn't do it for that reason, we should try to favor immutability where possible.  Also it would just be extra code to maintain and by implementing this proposal we might implicitly suggest that this is a recommended use case, and I don't think we should do that, especially now that so much documentation is out there explaining the current setup method (which I think is better for the immutability reason stated above).

It doesn't mutate the store object, though.  It returns an enhanced store.

It doesn't mutate the store object, though. It returns an enhanced store.

ah good point, I see that now.

I think this might be a step backwards in http://redux.js.org/docs/advanced/Middleware.html to attempt 3 since dispatch is being money patched, although I guess that is not required for what you are suggesting.

const store = createStore(reducer, enhancer);

Also I'm not sure how this is intended to work with other middleware.

Also I'm not sure how this is intended to work with other middleware.

Same way you'd do it for any other store enhancer
const store = createStore(
  reducer,
  compose(
    applyMiddleware(otherMiddleware),
    sagaEnhancer(),
  ),
);

I dont see the real benefit of doing this now, by being more explicit we force users to compose this behaviour on their stores however they want and its only a few lines of code anyway. I would prefer users being more conscious about this and enhance their stores only when really needed (for the ease of usage).
It could also potentially mess with people's TS/flow types.
Also benefit of saga being a middleware is also an ability to put it into whatever position in the middleware chain you like, although it being 'last' is preferable and the easiest to maintain in the long run.
Im still open for a discussion about this though, maybe others will have something interesting to add, but personally I think its more flexible the way it is now.

by being more explicit we force users to compose this behaviour on their stores however they want and its only a few lines of code anyway

How else would they want to do it?  It's just boilerplate that amounts to the same thing.

It could also potentially mess with people's TS/flow types.

As long as the store enhancer returns a valid StoreCreator, nothing changes.

it will now not be possible to implement a gate keeper middleware checking users permission levels in a way that the 'blocked' action would be skipped and would not hit the sagas.

Sure it is
const store = createStore(
  reducer,
  compose(
    // Wraps dispatch from `sagaEnhancer`.
    applyMiddleware(gatekeeperMiddleware),
    // Wraps original dispatch.
    sagaEnhancer(),
  ),
);

How else would they want to do it? It's just boilerplate that amounts to the same thing.

1 line of code (store.runSaga = sagaMiddleware.run;) hardly can be called as boilerplate imho.
gatekeeper example
In the simplest scenario its possible and fine, but most of the times people use middlewares to hook into dispatch as it was literally their design goal. By spreading this responsibility between middlewares and enhancers we lose ordering flexibility, cause they cannot intermit each other.

After some thinking I havent found enough compelling arguments to change the way how the redux-saga is used at the moment, we might revisit this in the future.


Show Join/Parts message - ircStatusNotices

Trying to get join/parts messages to show in discord but doesn't appear to work;
ircStatusNotices is set to true.  Bot works as normal with running config.
Is this still in development?

ah solved it! the ircStatusNotices was in the wrong part of the config.json.  Moved it to it's own section and it now works.
Good work developers keep up the good work!

ah solved it! the ircStatusNotices was in the wrong part of the config.json.  Moved it to it's own section and it now works.
Good work developers keep up the good work!


URIs are open()ed for no reason

I found this while working through more of the common test suite failures. The draft4/optional/format.json test currently pauses during a URI validation test because it's trying to make a request to load data from the URI. You can reproduce this with:
pry(main)> data = 'http://foo.bar/?baz=qux#quux'
=> "http://foo.bar/?baz=qux#quux"
pry(main)> schema = '{ "type": "string", "format": "uri" }'
=> "{ \"type\": \"string\", \"format\": \"uri\" }"
pry(main)> JSON::Validator.validate(schema, data)

The final line will eventually return true (which is correct), but if you wrap it in Timeout.timeout(15) { ... } or whatever, it reliably throws an exception.
I've tracked this down to one of these open() calls inside Validator#initialize_data, but haven't yet had a chance to dig in any further to figure out when those are necessary and when they aren't:
https://github.com/hoxworth/json-schema/blob/8daf3de35ab834bc9c2325a967be25322b909cdd/lib/json-schema/validator.rb#L584-L598

I got alerted to this via a pen test.
We are using this library to validate user input against a schema. If the user puts in a URI as a value, then this will make a request to the URI.


Wrong Package Name on Bower

Hi,
On README, it is said the name of the Bower package is cytoscape-navigator.
If you try with it, there is no package found.
Instead, it should be "cytoscape.js-navigator".
Hope it would be helpful!

It should be as quoted in the readme.  It should be renamed on the bower side

Published as proper name.  Looks OK now:
$ bower info cytoscape-navigator
bower cached        https://github.com/cytoscape/cytoscape.js-navigator.git#1.1.3
bower validate      1.1.3 against https://github.com/cytoscape/cytoscape.js-navigator.git#*

{
  name: 'cytoscape-navigator',
  description: 'Bird's eye view pan and zoom control for Cytoscape.js',
  main: 'cytoscape-navigator.js',
  dependencies: {
    cytoscape: '^2.6.0',
    jquery: '^1.4.0 || ^2.0.0'
  },
  repository: {
    type: 'git',
    url: 'https://github.com/cytoscape/cytoscape.js-navigator.git'
  },
  ignore: [
    '**/.*',
    'node_modules',
    'bower_components',
    'test',
    'tests'
  ],
  keywords: [
    'cytoscape',
    'cyext'
  ],
  license: 'MIT',
  homepage: 'https://github.com/cytoscape/cytoscape.js-navigator',
  version: '1.1.3'
}

Available versions:
  - 1.1.3
  - 1.1.2
  - 1.1.1
  - 1.1.0
  - 1.0.2
  - 1.0.1
  - 1.0.0

You can request info for a specific version with 'bower info cytoscape-navigator#<version>'


Fix/recommenders handle 404 with message

This PR introduces

fixed behavior for a UI message on errors, e.g. 404.
streamlined function getRecommendedUsers with corrected state handling to use asyncFetch only.
fixed behavior for low metric user data, e.g. likes, followers, following when either of them should be null.
removed/fixed EngagementTracker response handling, only logging errors, since OK is ok.
additional functionality to get next users onFollow click in ActivityPage > LatestPeople > Recommended: followed user is removed, subsequent user is added.
general minor cleanups

Hey @scopalaffairs, thanks for the PR! I recently refactored some of the same components to improve performance and fix a couple bugs along the way. Unfortunately that means there's a big merge conflict now. Can you pull from develop and review the updated code? Then you can re-apply any of your changes that you still feel are needed. Thanks!

Hey @calebjacob, thanks for the heads up and the good work there.
Main thing I had to resolve is how the returnProfileForUser function you introduced lately is called.
It looks good on my side, PeoplePage and especially the ActivityPage > LatestPeople > hover on profiles, that's very much improved and smooth now! Great!

@calebjacob, returnProfileForUser would also make sense to have on the other components that do the same.
Do you think, it's a good idea to add that in this PR?

hey @calebjacob, sorry for making any post-approval changes.
I had the branch tested with @TiffanyGYJ  and we adjusted the TrendingUsersView error message to make more sense.
Also, I reverted the width for the components in PeoplePage to be 100vw again, since 100% does not use the full space and looks weird.


Introducing OpenSearch Guru on Gurubase.io

Hello team,
I'm the maintainer of Anteon. We have created Gurubase.io with the mission of building a centralized, open-source tool-focused knowledge base. Essentially, each "guru" is equipped with custom knowledge to answer user questions based on collected data related to that tool.
I wanted to update you that I've manually added the OpenSearch Guru to Gurubase. OpenSearch Guru uses the data from this repo and data from the docs to answer questions by leveraging the LLM.
In this PR, I showcased the "OpenSearch Guru", which highlights that OpenSearch now has an AI assistant available to help users with their questions. Please let me know your thoughts on this contribution.
Additionally, if you want me to disable OpenSearch Guru in Gurubase, just let me know that's totally fine.

Is this a commercial product? If so I don't think a badge is appropriate.
FYI, my corporate network blocked the gurubase.io website.

Is this a commercial product? If so I don't think a badge is appropriate.

No, it's not.

FYI, my corporate network blocked the gurubase.io website.

Do you know why this might be happening?

FYI, my corporate network blocked the gurubase.io website.

Do you know why this might be happening?

Probably something from this list:  https://internetsecure.org/?domain=gurubase.io

Hi everyone,
Have you had a chance to finalize the decision for this PR?
Additionally, I’d like to update you on the release of the Maintainer Panel feature on Gurubase. With this panel, you can add, remove, or update data sources, as well as change the logo and more. You can find the details about how to claim a Guru here.
In the near future, I’m planning to include analytics insights in this panel, such as the number of questions asked, the most frequently asked ones, and more. I’ll be sure to update this thread once it’s available. However, in case I miss it, I highly recommend joining our Discord channel to stay updated.
If you’d prefer that I don’t update this thread anymore, please let me know.

I don't see a ton of enthusiasm from maintainers or users for this. Feel free to leave this open and drum up some interest, but in the meantime I think it's a no go.


Question: Detect IPV6 Cameras

Could this be used to detect IPV6 cameras? Our cameras only gives up IPV6 and not IPV4. Thanks

Unfortunately, the node-onvif does not support IPv6 for now.

Cheers!


[C++][Parquet] Do not assume that perl is found in /usr/bin

Some leaned down Linux environments may not have Perl in /usr/bin (required by ctest / run-test.sh)
Reporter: Wes McKinney / @wesm
Assignee: Wes McKinney](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=wesm) / @wesm
Note: This issue was originally created as PARQUET-661. Please see the migration documentation for further details.

Wes McKinney / @wesm:
Issue resolved by pull request 138
https://github.com/apache/parquet-cpp/pull/138


Can't apply multiple args to mpv using --append-arg=

Trying to use with pipe-viewer with mpv and it won't append more than one arg. I'm trying to use :
mpv --no-border --geometry=0%:100%
I setup an alias
alias pvm="pipe-viewer --resolution=480p $1 --append-arg=--geometry=0%:100%"
So I can just type "pvm ltt" which works but only for one arg ,it doesn't take multiples
Tried multiple args, arg array. It only takes one arg.

Quote the arguments as:
--append-arg='--geometry=0%:100% --no-border'


feat: Add perf benchmark app for Griffel

Takes core benchmarking code from
https://github.com/necolas/react-native-web/tree/master/packages/benchmarks
and create a benchmarking app for Griffel. Includes by default a griffel
(aliased) implementation and another for inline-styles.
The app is built with esbuild for speed 🏎🏎
Ran the benchmarks against the original to make sure the numbers were similar

I think that there is something wrong with this benchmark app.
Original

This PR

Numbers cannot be different 4x times. Is the implementation different?

Numbers cannot be different 4x times. Is the implementation different?

I realized that my copy is tweaked to render more elements, can we increase it a bit there to make difference more visible, too?
getComponentProps: () => ({ breadth: 2, components, depth: 7, id: 0, wrap: 1 }),

getComponentProps: () => ({ breadth: 2, components, depth: 8, id: 0, wrap: 1 }),

It also would be nice to have at least Emotion/plain CSS in the comparison (we can do in follow up PR).

@layershifter  I added emotion since it was the easiest to do


PelicuLatino

Addon Manifest URL
https://app.strem.io/shell-v4.4/#/addons/community/all?addon=https%3A%2F%2Fpeliculatino.herokuapp.com%2Fmanifest.json
Addon Description
¡Películas en excelente calidad con audio latino!
Language of Content
Spanish; Castilian (es-MX)
Choose Labels

[ ] anime
[ ] asian drama
[ ] bollywood
[ ] cartoons
[ ] debrid support
[ ] http streams
[ ] live tv
[ ] metadata
[ ] misc
[X] movies
[ ] music
[ ] nsfw
[ ] podcasts
[ ] radios
[ ] subtitles
[ ] torrents
[ ] tv shows
[ ] video games

this is not the way to submit working addons, they need to end with /manifest.json


doc: fix broken link on README

They moved the files again.

Merging directly. Don't wanna spam anyone with this.


Proposal: Out-of-the-box support for Closure Compiler

Recently came across this cool project from Google called Closure Compiler. From some testing it seems like it reduces code size quite a bit, an example that was recently posted on the svelte subreddit shows some pretty significant gains (from 1.3 to 0.8 time to interactive and a much smaller bundle size).

There's also a Rollup plugin for it.

After some tests on elderguide.com scripts it doesn't look like there's much benefit. Might be worth investigating more, though.

@kevmodrome Tested this out when working on #42. It crashes mysteriously with any non-trivial implementation.
On the trivial implementations, almost no size benefit.


apply Echdna to see what it says

Some time ago, Mythril, solhint and Solium have been applied on the contract sources: https://github.com/raiden-network/raiden-contracts/issues/224
This issue keeps track of applying Echdna, and recording the setup and the results.

This depends on #713.


rename and update gha workflow to sync boostlook.css across repos

renamed workflow file from update-website-v2.yml to sync-boostlook-css.yml
simplified commit process to push changes directly to develop branch if updates are detected
added steps to trigger ui-release and publish workflows in website-v2-docs repository
ensured pre-commit hooks are run before committing updates

fixes #44

Hi @pdimov! Could you approve the token requested for website-v2-docs? Thanks


python3Packages.transitions: 0.8.7 -> 0.8.8

Motivation for this change
https://github.com/pytransitions/transitions/releases/tag/0.8.8
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 118720 at 6d49bb23 run on aarch64-linux 1

12 packages built successfully:

python38Packages.consonance
python38Packages.dissononce
python38Packages.hbmqtt
python38Packages.junos-eznc
python38Packages.roombapy
python38Packages.transitions
python38Packages.yowsup
python39Packages.consonance
python39Packages.dissononce
python39Packages.junos-eznc
python39Packages.transitions
python39Packages.yowsup

Result of nixpkgs-review pr 118720 run on x86_64-linux 1

12 packages built:

python38Packages.consonance
python38Packages.dissononce
python38Packages.hbmqtt
python38Packages.junos-eznc
python38Packages.roombapy
python38Packages.transitions
python38Packages.yowsup
python39Packages.consonance
python39Packages.dissononce
python39Packages.junos-eznc
python39Packages.transitions
python39Packages.yowsup


Dummy change

Previously, the error msg xxx, in this commit, we change it into dummy.
Task: BABEL-3455
Signed-off-by: Xiaohui Fanhe<EMAIL_ADDRESS>Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

fix the syntax


Closes #100

commit 19836b781ac7c711a95b8acf5b07c2cdc9630ef5 adding cwd to browserify spawn was the issue.
I have added a check for the existence of the module in node_modules.
Now core and non core modules work
It will also fix #101

Oh word, that makes total sense.
I'll merge this tonight when I get home from work.

Great!

Could we also uncomment the test that covers this?

Done and done (and deployed). Thanks!!


ScalametaParser: keep delims around body

Partial undo of #2941. Turns out it's very hard to handle enclosed trees when their enclosing parens belong to a parent.

@kitbellew can we release a new version without this PR? Or is it blocking for scalafmt?

@kitbellew can we release a new version without this PR? Or is it blocking for scalafmt?

I'm trying to get the formatter working without this change, it might not be needed -- and perhaps not ideal to depend on positions of parentheses.
but since the next release will not be a patch, should we perhaps finalize the decision -- and implementation -- of the interleaved change (#2929)?

@kitbellew can we release a new version without this PR? Or is it blocking for scalafmt?

I'm trying to get the formatter working without this change, it might not be needed -- and perhaps not ideal to depend on positions of parentheses.

Ok, I can wait for that :+1:

but since the next release will not be a patch, should we perhaps finalize the decision -- and implementation -- of the interleaved change (#2929)?

I don't think we need to hurry with interleaved, we can do another minor later, it's not a problem. As for decisions, did any of my suggestions make sense?

@kitbellew can we release a new version without this PR? Or is it blocking for scalafmt?

I'm trying to get the formatter working without this change, it might not be needed -- and perhaps not ideal to depend on positions of parentheses.

Ok, I can wait for that 👍

thank you, i might be ready next week.

but since the next release will not be a patch, should we perhaps finalize the decision -- and implementation -- of the interleaved change (#2929)?

I don't think we need to hurry with interleaved, we can do another minor later, it's not a problem. As for decisions, did any of my suggestions make sense?

the desire made sense but i haven't tried implementing it -- and thus seeing what transpires. and yes, let's postpone it until next release.


Ruby on rails installation update

DO NOT CREATE NEW ISSUES ON THIS REPOSITORY!
Please create all issues found when running brew install at https://github.com/Homebrew/homebrew-core/issues/new
Please create all issues found when running any other brew command at https://github.com/Homebrew/brew/issues/new
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
Error: Git is unavailable
Error: Failure while executing: /Users/sarahmahmud/rvm use 1.9.2/bin/brew tap homebrew/core -q
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
Error: Git is unavailable
Error: Failure while executing: /Users/sarahmahmud/rvm use 1.9.2/bin/brew tap homebrew/core -q
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
Error: Git is unavailable
Error: Failure while executing: /Users/sarahmahmud/rvm use 1.9.2/bin/brew tap homebrew/core -q
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
Error: Git is unavailable
Error: Failure while executing: /Users/sarahmahmud/rvm use 1.9.2/bin/brew tap homebrew/core -q
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
Error: Git is unavailable
Error: Failure while executing: /Users/sarahmahmud/rvm use 1.9.2/bin/brew tap homebrew/core -q
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
/Users/sarahmahmud/rvm use 1.9.2/Library/brew.sh: line 85: /usr/bin/xcode-select: No such file or directory
Installing requirements for osx.
Updating system....
Error running 'requirements_osx_brew_update_system ruby-1.9.2-p330',
showing last 15 lines of /Users/sarahmahmud/.rvm/log/1460425496_ruby-1.9.2-p330/update_system.log
https://github.com/Homebrew/homebrew/wiki/Common-Issues
and make sure brew update works before continuing.'
++ rvm_pretty_print stderr
++ case "${rvm_pretty_print_flag:=auto}" in
++ case "${TERM:-dumb}" in
++ case "$1" in
++ [[ -t 2 ]]
++ return 1
++ printf %b 'Failed to update Homebrew, follow instructions here:
https://github.com/Homebrew/homebrew/wiki/Common-Issues
and make sure brew update works before continuing.\n'
Failed to update Homebrew, follow instructions here:
https://github.com/Homebrew/homebrew/wiki/Common-Issues
and make sure brew update works before continuing.
++ return 1
Requirements installation failed with status: 1.
Sarah-Mahmuds-iMac:~ sarahmahmud$
Sarah-Mahmuds-iMac:~ sarahmahmud$

Sorry but you have not followed the requested steps on the Troubleshooting page
Please follow (all of) these steps and post the information here so we can help you with your problem.
Also, to help us debug this issue can you explain:

What you were trying to do (and why)
What happened
What you expected to happen
Step-by-step reproduction instructions (with as minimal input data as possible)

Thanks!

Closing for a lack of response/information; happy to reopen once there's some activity here.


Tests?

Given the critical nature of this sort of application, it would be really nice if there were tests that this actually works, which are run in a CI server (see #23).

I'll drop this here. I think it'd be really nice to set it up using tox http://tox.readthedocs.io/en/latest/ it works well locally and it is easy to use with travis-ci.

Check out #41 I'm just waiting or @KristianOellegaard to enable travis-ci and we're ready to write tests.


New version: timschneeb.GalaxyBudsClient version 5.0.1

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Is there a linked Issue?

Resolve #161127
Resolve timschneeb/GalaxyBudsClient#522

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

This package installs and launches normally on a Windows 10 VM.


Add a ROOT path constant for fastlane_core

Continuation of work on concerns raised in #5770

Replace use of Helper.gem_path with FastlaneCore::ROOT
Deprecate Helper.gem_path

👍


New Feature Request: "Latch Only"

I´m ok so far with the Nuki HomeApp Integration. I would be "only" interested in the Latch Mechanism.
Would be cool if there would be an option in the Setting Menu to be able to "disable" the check box for the Lock.
Thank you!

Not sure what you mean by the "check box for the lock"?  I won't be providing an option to expose only the Lock Mechanism service for the latch, if that's what you're asking.  I don't think it's a good idea to combine Homebridge NB with the Smart Lock's native HomeKit integration; neither the Nuki bridge nor the Smart Lock like concurrent connections.
You can unlatch the lock through the native HomeKit integration by "slide from unlock to unlock", see #16.


Addition of a right

A right was missing from the list. Added the basic info.
Problem: https://github.com/MicrosoftDocs/windows-itpro-docs/issues/8551

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

@illfated - Thanks, how did you find this file.

@illfated - Thanks, how did you find this file.

Well, first of all I opened the same directory as the file used in your PR, user-rights-assignment.md

https://github.com/MicrosoftDocs/windows-itpro-docs/tree/public/windows/security/threat-protection/security-policy-settings

Secondly, I searched for some of the keywords in the security setting (SeDelegateSessionUserImpersonatePrivilege)
and your added description ("Obtain an impersonation token for another user in the same session")
By removing the word ending, the shortened word form impersonat led me to the file:

SeDelegateSessionUserImpersonatePrivilege
"Obtain an impersonation token [ ... ]"

Noticing that this keyword might help me find a useful file, and the fact that all the other columns in the table contains links to files in this same directory (https://github.com/MicrosoftDocs/windows-itpro-docs/blob/public/windows/security/threat-protection/security-policy-settings/impersonate-a-client-after-authentication.md), I searched the filename list for the keyword impersonat and found:

impersonate-a-client-after-authentication.md

Well, first of all I opened the same directory as the file used in your PR, user-rights-assignment.md

https://github.com/MicrosoftDocs/windows-itpro-docs/tree/public/windows/security/threat-protection/security-policy-settings

Secondly, I searched for some of the keywords in the security setting (SeDelegateSessionUserImpersonatePrivilege)
and your added description ("Obtain an impersonation token for another user in the same session")
By removing the word ending, the shortened word form impersonat led me to the file:

SeDelegateSessionUserImpersonatePrivilege
"Obtain an impersonation token [ ... ]"

Noticing that this keyword might help me find a useful file, and the fact that all the other columns in the table contains links to files in this same directory (https://github.com/MicrosoftDocs/windows-itpro-docs/blob/public/windows/security/threat-protection/security-policy-settings/impersonate-a-client-after-authentication.md), I searched the filename list for the keyword impersonat and found:

impersonate-a-client-after-authentication.md

#sign-off

#sign-off

Hi, The right is listed, but it links to the wrong content. It links to Impersonate a client after authentication which is a different right altogether.


Feature/cu 8ywha8 reload catalog without restarting container

I think you would usually wait for an approval before you merge the PR

I think you would usually wait for an approval before you merge the PR


Need to add Ellipse drawing to the Canvas API

Need to replace the circle API with an ellipse.

This has been added.


eureka: allows authentication

this helps integration test https://github.com/openzipkin/zipkin/issues/3697

Thanks for the look @reta!


SW-3919 MyAccount language and preferred units selectors should be full width

Looks great thanks! For mobile the inputs use up table width. Unrelated to this PR for sure. Moving the Table out of the top level grid container should fix it.. small change but sure we can do this separately if preferred. Also, do you think we should get Chudo's feedback?

Oh, will fix that now. I think we can merge it when it's ready and if Chudo has any feedback we can follow up later

Looks great thanks! For mobile the inputs use up table width. Unrelated to this PR for sure. Moving the Table out of the top level grid container should fix it.. small change but sure we can do this separately if preferred. Also, do you think we should get Chudo's feedback?

Locally it looks good, that's strange

Looks great thanks! For mobile the inputs use up table width. Unrelated to this PR for sure. Moving the Table out of the top level grid container should fix it.. small change but sure we can do this separately if preferred. Also, do you think we should get Chudo's feedback?

Locally it looks good, that's strange

Hmm.. yes for some reason broken in vercel. We can maybe just create a new ticket if you prefer.
Thanks so much for the changes. Yes we can follw up on any @chudo76 to give feedback later.

Looks great thanks! For mobile the inputs use up table width. Unrelated to this PR for sure. Moving the Table out of the top level grid container should fix it.. small change but sure we can do this separately if preferred. Also, do you think we should get Chudo's feedback?

Locally it looks good, that's strange

Hmm.. yes for some reason broken in vercel. We can maybe just create a new ticket if you prefer. Thanks so much for the changes. Yes we can follow up on any @chudo76 to give feedback later.

But all tables work like that, you have to scroll to the right to see all the data

Looks great thanks! For mobile the inputs use up table width. Unrelated to this PR for sure. Moving the Table out of the top level grid container should fix it.. small change but sure we can do this separately if preferred. Also, do you think we should get Chudo's feedback?

Locally it looks good, that's strange

Hmm.. yes for some reason broken in vercel. We can maybe just create a new ticket if you prefer. Thanks so much for the changes. Yes we can follow up on any @chudo76 to give feedback later.

But all tables work like that, you have to scroll to the right to see all the data

Yes, that is correct. I was referring to the input elements like language selector etc.
We can make that section independent of the table width with Grids. We do that in some other places.

Looks great thanks! For mobile the inputs use up table width. Unrelated to this PR for sure. Moving the Table out of the top level grid container should fix it.. small change but sure we can do this separately if preferred. Also, do you think we should get Chudo's feedback?

Locally it looks good, that's strange

Hmm.. yes for some reason broken in vercel. We can maybe just create a new ticket if you prefer. Thanks so much for the changes. Yes we can follow up on any @chudo76 to give feedback later.

But all tables work like that, you have to scroll to the right to see all the data

Yes, that is correct. I was referring to the input elements like language selector etc. We can make that section independent of the table width with Grids. We do that in some other places.

The problem is unrelated to this PR :) I will create a ticket for us later.

Looks a bit odd with a three-column row on top of a four-column row but I don't have a good alternative so I think we're good for now. The form fields look much better at full-width, though. Thanks!


Problem when run in Burp on Linux

Hi,
I tried your extension last night on a windows host and all was fine, however my main pen-testing system is Linux based (Arch Linux, Jython version is 2.7, never had issue with this with any other extensions that require it, Java is version 8) and I added the extension this morning without issue, until I came to create the users and roles in the extension.
The extension took the names for the new users and roles but displayed nothing, so I checked the log in the extender tab, the output is below;
Traceback (most recent call last):
File "/usr/share/burpsuite/bapps/30d8ee9f40c041b0bfec67441aad158e/AuthMatrix.py", line 327, in getInputUserClick
self._db.getOrCreateUser(newUser)
File "/usr/share/burpsuite/bapps/30d8ee9f40c041b0bfec67441aad158e/AuthMatrix.py", line 568, in getOrCreateUser
self.arrayOfUsers.append(UserEntry(userIndex,
AttributeError: 'java.util.ArrayList' object has no attribute 'append'
Traceback (most recent call last):
File "/usr/share/burpsuite/bapps/30d8ee9f40c041b0bfec67441aad158e/AuthMatrix.py", line 333, in getInputRoleClick
self._db.getOrCreateRole(newRole)
File "/usr/share/burpsuite/bapps/30d8ee9f40c041b0bfec67441aad158e/AuthMatrix.py", line 590, in getOrCreateRole
self.arrayOfRoles.append(RoleEntry(roleIndex,
AttributeError: 'java.util.ArrayList' object has no attribute 'append'
Traceback (most recent call last):
File "/usr/share/burpsuite/bapps/30d8ee9f40c041b0bfec67441aad158e/AuthMatrix.py", line 327, in getInputUserClick
self._db.getOrCreateUser(newUser)
File "/usr/share/burpsuite/bapps/30d8ee9f40c041b0bfec67441aad158e/AuthMatrix.py", line 568, in getOrCreateUser
self.arrayOfUsers.append(UserEntry(userIndex,
AttributeError: 'java.util.ArrayList' object has no attribute 'append'
Traceback (most recent call last):
File "/usr/share/burpsuite/bapps/30d8ee9f40c041b0bfec67441aad158e/AuthMatrix.py", line 333, in getInputRoleClick
self._db.getOrCreateRole(newRole)
File "/usr/share/burpsuite/bapps/30d8ee9f40c041b0bfec67441aad158e/AuthMatrix.py", line 590, in getOrCreateRole
self.arrayOfRoles.append(RoleEntry(roleIndex,
AttributeError: 'java.util.ArrayList' object has no attribute 'append'
If you require any further details please let me know and I'll be happy to help out
Thanks
Dave

Hi,
I may have worked this out
I checked the downloaded version of Jython27 from the website against the installed version of Jython27 and some of the folders that are included in the downloaded version (bin, extlibs), are missing from the Jython27 folder in the package version.
Looks like this might be an ArchLinux package issue.
Will test further and let you know.
Dave

Sounds good.  I suspect you are correct in that its an issuing with your Jython install.  Let me know if that fixes it.
-Mick

Any update to this?

I haven't been able to test it properly just yet, I may have chance this
week.
On 23 Feb 2016 23:33, "mickayz"<EMAIL_ADDRESS>wrote:

Any update to this?
—
Reply to this email directly or view it on GitHub
https://github.com/SecurityInnovation/AuthMatrix/issues/1#issuecomment-187965800
.

So it looks like this was a mistake in the code that just happened to work for certain versions of java/jython (including mine).
I pushed an update, version 0.5.1, that should have fixed this.  If you have a chance could you try it out and close this issue if it no longer produces the error?
Thanks for reporting this.

Hi @mickayz,
I had this same exact issue and corrected it in the same way you did it, yes I didn't look at the reported issues in GitHub before hand :) Thankfully I didn't went to the trouble of creating a pull request ahahah :P Anyway, I can confirm that the fix works.
By the way, did you notice that in the BApp the version available is 5.0 instead of 5.1?
Cheers,
Duarte
PS: Great extension by the way, will be testing/using it further this week.

Not just Linux. Confirm same (well, equivalent) error on Windows:

Window 7
Burp latest (1.6.39)
Jython 2.7.1b

Confirmed that 0.5.1 update (via commit 18cd2b2111b9c751b33cb37f08aaa9a3a6bf85d9) fixes it.
Thanks for creating/releasing this. Really badly needed tool, I think it'll do a lot of good.Excited to be trying it out.

Thank you all for the support! v0.5.1 has been submitted to the BApp store and should be updated there soon.


ins / underline extension doesn't support

import markdown
from markdown.inlinepatterns import SimpleTagPattern

DEL_RE = r"(\~\~)(.+?)(\~\~)"
INS_RE = r"(\+\+)(.+?)(\+\+)"

class DelInsExtension(markdown.extensions.Extension):
    """Adds del_ins extension to Markdown class."""

    def extendMarkdown(self, md, md_globals):
        del_tag = SimpleTagPattern(DEL_RE, 'del')
        ins_tag = SimpleTagPattern(DEL_RE, 'ins')
        md.inlinePatterns.add('del', del_tag, '<not_strong')
        md.inlinePatterns.add('ins', ins_tag, '<not_strong')

When I try this extensions:
>>> import markdown
>>> src = '''This is ++added content++ and this is ~~deleted content~~'''
>>> html = markdown.markdown(src, extensions=['martor.extensions.del_ins:DelInsExtension'])
>>> html
'
This is ++added content++ and this is deleted content
'
>>>
>>> markdown.version
'3.0.1'

I think it should:
'
This is added content and this is deleted content
'

You reused DEL_RE for your insert pattern, you need to use the insert regular expression.

damn.. I forgot it.. thanks @facelessuser


Nitpick securityGroup = "cml"

In code cml should be replaced with iterative

@0x2b3bfa0 did you had a chance to test this ?

@DavidGOrtega, unable to reproduce (yet), but trying again the long way.


chore: rename isthmus to interop on solidity files

Contract side of changes, the client side of them are here: https://github.com/defi-wonderland/optimism/pull/63
CLOSES OPT-228

PR to Optimism repo here: https://github.com/ethereum-optimism/optimism/pull/12132


qemu: add capstone support

Currently using the -d in_asm flag in QEMU does not report any dissassembly. This fixes it.
Description of changes

Things done

Built on platform(s)

[ ] 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

[ ] Fits CONTRIBUTING.md.

This broke virtualbox builds: https://hydra.nixos.org/eval/1795211#tabs-now-fail
Also note that all "breaking changes" are disallowed currently, in preparation for 23.05 release.

@vcunat looks fine now.  False positive?

🤔  I saw at least three failed attempts, but all were executed on the same machine.  I suspect that higher --cores might be the difference here.


add KV tx

Codecov Report

Merging #50 into develop will increase coverage by 0.08%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop      #50      +/-   ##
===========================================
+ Coverage    38.34%   38.43%   +0.08%
===========================================
  Files          120      120
  Lines        10311    10311
===========================================
+ Hits          3954     3963       +9
+ Misses        5957     5947      -10
- Partials       400      401       +1

Impacted Files
Coverage Δ

gemmill/blockchain/pool.go
72.18% <0%> (+3.38%)
:arrow_up:


remove self if no path

Hi @Dev1an , this PR include:

Add self at index 0 if only there is a path, otherwise, just leave the path empty(UnitTest added).

Hi @Dev1an , take this test case, for example, we can know if there is a path between two points by checking the path.count  > 0, otherwise, the checking logic should like this path.count > 1.
In my opinion, the previous one is more intuitive, how do you think about it?
func testEmptyPath() {
    let path = c3.findPath(to: c5)
    XCTAssertEqual(path.count, 0)
}

hi @Dev1an , looking forward for your feedback. thanks

Thanks for all your contributions @ampm !
Yeah it is true that it looks nicer when the path is completely empty rather than a collection of one element. But this is again a breaking change so I will have to do a major version bump again. I don't think this is so much of a problem though because I doubt there are many people using this library.
So if you have other proposals for breaking changes please post them now, and then I can do a big major update with all of them together, rather than having to do major version bumps every day 😉.

And I don't think you need to rename the test target in Package.swift (91a686d3df668f490d8a73e7e01d0cae69473cdd). SPM Reads the directory names, and not the target names from the Xcode project. You can test if the Package.swift file by running swift test in the root directory of this project using the terminal.
When I do this after your renaming, I get:
'AStar' /Users/damiaan/Documents/Projecten/A-Star: error: Source files for target AStarTests_macOS should be located under 'Sources/AStarTests_macOS', or a custom sources path can be set with the 'path' property in Package.swift

Hi @Dev1an, no other thoughts so far.  I'll try to collect them together if i have in the future.
And have revert that commit, not the test should pass.
Thanks very much.

hi @Dev1an, can we merge this PR? thanks.

I merged your commits and added some clean up. You can try it using version 3.0.0-beta-1


Update blog article link

Description

Motivation and Context

How Has This Been Tested?

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

closing in favor of https://github.com/checkr/flagr/pull/474


Option to only load cache, without writing to it at the end of the job

Description
I have a continuous integration workflow that runs multiple jobs in parallel, one for each package in the monorepo. In each of these jobs I am using the @actions/setup-node action, with caching enabled.
The problem is that the action attempts to write to the cache using the same key, at the end of the job exection:

Unable to reserve cache with key node-cache-Linux-yarn-ed73988f6a7bd1a0270433a2d2ff89c15016643cf8f05a2fb87a270786528399, another job may be creating this cache.

Possible Solution
Add a boolean flag in the action with: setting that lets us specify if we only want to load the cache, if it exists, without further writing to the cache at the end of the job. This way, I could install the dependencies in an initialization job, which all other jobs would depend upon.
Example Workflow
See the "ci-amm", "ci-flash-swap", "ci-protocol" and "ci-proxy-target" jobs in workflow execution.

Looks like this could be done after PR https://github.com/actions/cache/pull/489 is merged in the cache action repo.


tokenizer.json required for TEI?

System Info
When trying to use this model (ibm/re2g-reranker-trex) in TEI, it will error because there is no tokenizer.json file. If I call AutoTokenizer.from_pretrained("ibm/re2g-reranker-trex"), there aren't any issues creating the tokenizer.
I have opened a pull request on the model page to include the tokenizer.json file, but I'm wondering if something should/could be done on the TEI side.
Information

[X] Docker
[ ] The CLI directly

Tasks

[X] An officially supported command
[ ] My own modifications

Reproduction
model=ibm/re2g-reranker-trex
volume=$PWD/data

docker run --gpus all -p 8080:80 -v $volume:/data --pull always ghcr.io/huggingface/text-embeddings-inference:1.0 --model-id $model

Expected behavior
No error

@OlivierDehaene ,
This may be an issue with older models on the hub both for the tokenizer and the config.json.
Older Bert models won't have a tokenizer.json file.
SequenceClassification models won't have num_labels, id2label, or label2id in config.json
Should TEI be able to handle these cases, or is it up to the user to create a PR to include these new files?

SequenceClassification models won't have num_labels, id2label, or label2id in config.json

Do you have an example?

Should TEI be able to handle these cases, or is it up to the user to create a PR to include these new files?

For tokenizer.json, TEI will not be able to replace it. For the other case I'm not sure and would like to explore the examples to figure it out.

On second glance, this might be an anomaly. Other older models seem fine:
https://huggingface.co/CAMeL-Lab/bert-base-arabic-camelbert-da-sentiment/blob/main/config.json
https://huggingface.co/jb2k/bert-base-multilingual-cased-language-detection/blob/main/config.json

I have the same issue with not having tokenizer.json with old models. Is there any work around for we to have "tokenizer.json". As far as I know this is from FastTokenizer class? https://huggingface.co/docs/transformers/en/fast_tokenizers

SequenceClassification models won't have num_labels, id2label, or label2id in config.json

Do you have an example?

How about this :
https://huggingface.co/amberoad/bert-multilingual-passage-reranking-msmarco/blob/main/config.json

Also experiencing this issue with vectara


[feature-request] Extract Constants

Thank you for taking the time to file an issue!
In order to route, prioritize, and act on this, please include:

Dart SDK Version: 2.2.0

I would like the Analysis Server to improve the extraction of a constant.
Extraction for class-level constant, local final variable with/without type annotation.
The analysis server needs to support it first in order to help improve the tooling for Dart developers such as the Dart plugin for IntelliJ.
Some background information as it relates to the JetBrains tools:

Extract Constant refactoring is the only one that is done by the IDE itself, without using the Analysis Server. It happened this way for historical reasons. I think we should simply remove this refactoring because it is not consistent with its 'Extract variable' server-driven peer.

So while it may seem like support is there, the IDE did it without the aid of the analysis server and would need the analysis-server to be updated to continue to support this feature.

Closing as stale.

@keertip I've seen other Extract tickets still active. Is it possible this will be looked at in the future?

Maybe. If we do plan to work on it, the issue will be updated.


scripts: add positiveblue key to verify script

Added the commit to #5053 since the key isn't on Keybase so we can't use the "old" way of downloading it on the fly as we currently do on master.


feat(config): add zw141

Imported from OZW without changes.

This is in our todo list. #1600. If you want to do the legwork to get it in shape I'll deal with the conflict, otherwise it'll have to wait until I'm to those devices, which will hopefully be soon. We don't display all of the parameters if they aren't supported so you'll need to run down if there are any changes. And edit the labels to make them more clean. Get rid of unnecessary descriptions. Add units as appropriate. Split bitmasks if any. Etc.

I'm confused. We already support this device. A file already exists.

I'm happy that you are enforcing good config files for zwavejs! There were indeed some weird stuff on OZW files.
I've changed them as suggested. Let me know if I should do anything further.

Thanks! There were a few more minor changes but I just made them myself. Appreciate your help.

I've merged this into my upstream work, so I'm closing it here. This will merge with my template push.

@blhoward2 what is the branch where you've merged this into? Github is not telling me, and I would like to track it to know when this gets release :)

It’s merged into my local branch. It’ll be a few days as I need to finish cleaning up and templating all of the Aeotec devices. You’ll see me push up a PR templating all of Aeotec.

@blhoward2 any plans on getting this into the repo?

@blhoward2 Its been two months already. Can you just merge this into master without the rest of the changes you have on your local branch?

No but the good news is that I'm aiming to merge this one today or tomorrow anyway.


Fix Angular global style imports

Issue: #4378 #5239
This is @yjcxy12 's PR #5525 with the conflicts resolved. I was unable to push the resolution to his branch due to some github bug around github actions (since the merge contained edits to main.workflow ... don't ask). So I branched and resolved conflicts here. Please direct any further discussion to that PR.

This seems like it is a fix for Angular 6 and up with notes of only looking at angular.json?  what about pre angular 6 with using ".angular-cli.json"?

@mikepaggi AFAIK @storybook/angular has peer deps on angular 6.0+?

@mikepaggi AFAIK @storybook/angular has peer deps on angular 6.0+?

Where is that mentioned anywhere in the documentation?  All I see is Angular support with support with using the CLI.  So far I haven't seen anything spit out errors when i was installing initially or when running it.  As much as there is support for Angular...the docs are very lacking :-(


Error: No such config: osmap.csv

Every time lme_update.sh is running, I could see this error:

Updating from git repo
Already up to date.
Removing existing Docker stack Removing service lme_elasticsearch Removing service lme_kibana Removing service lme_logstash Removing network lme_esnet logstash.conf logstash_custom.conf
Attempting to remove legacy LME files (this will cause expected errors if these no longer exist)
Error: No such config: osmap.csv
Sleeping for one minute to allow Docker actions to complete...
Updating current configuration files
Custom logstash config exists, Not creating
Recreating Docker stack

Can I ignore this?

Hello LeitLei,
You can safely ignore that message, as the line above the error in your logs states, the subsequent command is attempting to remove a legacy configuration file from the docker engine.
Thanks,
Duncan

Thanks for clarification!


Cannot run tests with ^ characters in their names

It looks like the regex does not catch tests if they contain ^ character in their name.

Fixed in next EAP


Typo in Flutter Accessibility page

There's a typo on flutter.dev, specifically at https://flutter.dev/docs/development/accessibility-and-localization/accessibility#inspecting-accessibility-support
In the Accessibility Inspector, select Audit in the toolbar, and then select Run Audio to get a report of potential issues
should read
In the Accessibility Inspector, select Audit in the toolbar, and then select Run Audit to get a report of potential issues
(replace "Audio" with "Audit")
Nice documentation btw :)

Hi @vanillabeach,
I think it should read Audio, since theAccessibility Inspector in Xcode is used as a voiceover for the elements and has Play button for it, so I think the statement / word is correct, unless I missed anything that I didn't understand.

Hi @darshankawar ,
Thanks for your quick response :)
I assumed it was Audit based on the context of the instructions, and the option in the toolbar (see tooltip in screenshot).

Either way, feel free to close this issue if you think it does not need changing.
Thanks
// J

@vanillabeach  ah I see :-) It should be Run Audit in the docs.


Rev1 Updates

Rev1 Updates to the following documents:

Mar 14-15
Problem Statement
Development Plan

#289


Content API vs Redux

@lilannie  @ychoy  I have been doing a lot more Context API than redux and it definitely has a lighter learning curve. Should we continue with Redux or move to Context API? We do have time to discuss this since neither is quite needed yet.

@JMStudiosJoe
Are you talking about React's Context API??
I've used React's Context API but never as a substitute for Redux.

Yes I am talking about that API. Interesting I have not needed both either one or the other before but why both? I have found redux to be just too much but that is from my experience whereas context api has been much more manageable and easier to understand.
Sent with GitHawk

I’d prefer Redux because:

it’s more established
already has a bunch of dev tools
is much more common knowledge throughout React devs bc it’s an older
library

I do agree Redux can be a lot. However, I don’t think redux-act-async
reducers a huge chunk of boilerplate.
I would be okay with using the new React Context API, but I’m hesitant for
the reasons above.
About having not used Context API to replace Redux:
Pre-React 16, Context was an experimental feature that you weren’t suppose
to touch it unless necessary. Redux actually uses part of the Context API
under the hood.
On Mon, Oct 8, 2018 at 9:02 AM Joseph Richardson<EMAIL_ADDRESS>wrote:

Yes I am talking about that API. Interesting I have not needed both either
one or the other before but why both? I have found redux to be just too
much but that is from my experience whereas context api has been much more
manageable and easier to understand.
Sent with GitHawk http://githawk.com
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/codeforsanjose/OpenDSJ-2018/issues/8#issuecomment-427889851,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKsqpJOcgnLmBhR_HM1zXbKKjL-WmJ3Kks5ui3cQgaJpZM4XL-bM
.

--
Annie Steenson
B.S. Software Engineering | Iowa State University
Leadership Studies | Minor
<EMAIL_ADDRESS>402-578-3545

All good points. We can stick with Redux for now since it is more established and already in place. I'd say give Context a try sometime for practice since when I went to Context did not feel the need for all of Redux :). Yes it was experimental before but now it is a full API and recommended over Redux but you are right not as established, well known, or as many tools.

@lilannie @ychoy think we can close this issue now


Update readme links

Update Pact Broker and Pact specification links
ps: pact users are getting redirected to a inactive repo for the broker, so thought of updating the readme.

Thanks for the PR


Visit parents of Lambda classes to catch potential inherited random generators

Issue #, if available: Issue #18
Description of changes:

Created a new detector that saves parent classes of lambda handlers so they can be visited by the main detector later to catch potential inherited random generators.
Updated the main detector to also visit parent classes of lambda handlers
Added tests to reproduce the issue and to make sure things are working properly

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

The approach in this PR does not cover cases for multiple level inheritance. We made the decision to only cover lambda handlers direct parent classes for now, while we investigate how we can cover multiple levels inheritance.

Update: added changes to cover multi-level inheritance.


Add kubeadm + containerd e2e test.

Discussed in slack, we should add an e2e test in test infra for kubeadm + containerd.

Maybe we can use https://github.com/kubernetes-sigs/kubeadm-dind-cluster

Yes, results should be posted on testgrid.k8s.io.
Towards this end, I think a deb/rpm would be super helpful https://github.com/containerd/cri/issues/827
cc @timothysc @chuckha @detiber @feiskyer


pkg/validator: fix regexp mistake

The problem is a misplaced "-".
It was interpreted as a range.
And it seems to be the only dangerous symbol now.
Let's blacklist "--" to disable potential SQL injections.

Thanks @dvyukov for noticing coverage for files with "-" doesn't work.


go2rtc autopause

Hello
I'm facing an issue with go2rtc ; Maybe it's not related to it but All my cameras are having this kind of behaviour. They switch to pause during the live
is there an explanation to this ? (Network issue, cpu/ram server issue, misconfiguration ...) ?
That's so sad because go2rtc seems to provide very good quality with low latency :)

Looks like you using MSE technology. Because you have while time line. This is a normal situation for MSE in poor channel conditions. Should be solved on the client side (JavaScript). Perhaps there are others issues about such an improvement.
WebRTC technology works well in poor connection conditions. Without any additional modifications on the client side.


Crash during EU recognizing

Hey,
I use sample application. I changed us to eu but app was crashed during plate recognizing.
String result = OpenALPR.Factory.create(MainActivity.this, ANDROID_DATA_DIR).recognizeWithCountryRegionNConfig("eu", "", destination.getAbsolutePath(), openAlprConfFile, 10);

Log:
12-17 13:38:57.978 2318-2355/com.sandro.openalprsample D/OpenGLRenderer: Use EGL_SWAP_BEHAVIOR_PRESERVED: true
12-17 13:38:57.987 2318-2318/com.sandro.openalprsample D/Atlas: Validating map...
12-17 13:38:58.024 2318-2355/com.sandro.openalprsample I/Adreno-EGL: <qeglDrvAPI_eglInitialize:410>: QUALCOMM Build: 10/28/14, c33033c, Ia6306ec328
12-17 13:38:58.025 2318-2355/com.sandro.openalprsample I/OpenGLRenderer: Initialized EGL, version 1.4
12-17 13:38:58.046 2318-2355/com.sandro.openalprsample D/OpenGLRenderer: Enabling debug mode 0
12-17 13:38:58.149 2318-2318/com.sandro.openalprsample I/Timeline: Timeline: Activity_idle id: android.os.BinderProxy@300604ab time:114794351
12-17 13:39:05.914 2318-2355/com.sandro.openalprsample D/OpenGLRenderer: endAllStagingAnimators on 0xb84db010 (RippleDrawable) with handle 0xb84dadd0
12-17 13:39:35.365 2318-2318/com.sandro.openalprsample W/Settings: Setting airplane_mode_on has moved from android.provider.Settings.System to android.provider.Settings.Global, returning read-only value.
12-17 13:39:35.628 2318-2318/com.sandro.openalprsample I/Timeline: Timeline: Activity_idle id: android.os.BinderProxy@300604ab time:114831830
12-17 13:39:41.153 2318-3779/com.sandro.openalprsample A/libc: Fatal signal 11 (SIGSEGV), code 1, fault addr 0x4 in tid 3779 (AsyncTask #1)
12-17 13:39:42.180 2318-2318/com.sandro.openalprsample I/Choreographer: Skipped 55 frames!  The application may be doing too much work on its main thread.

Probably this is related to #1and should be fixed now, can you please test with the version 1.0.1.

I tested with 1.0.1 version. Issue still exists.

I have the same issue

I tried with a Portuguese licence plate and it worked as expected (please check the screenshot above).
Probably the problem was that the library was not able to recognize any licence plate in the picture. I improved the sample application (https://github.com/SandroMachado/openalpr-android/commit/e1037941f4f7dba1bd74ef24b0833ccfd8fd6e18) to display a toast instead of crashing when the library is not able to recognize any licence plate.

Same issue here with v1.1.0.
Same picture works with "us" and doesn't with "eu". Tested with sample app on two different devices.
Fortunately I was able to get rid of this SIGSEGV! All i did was adding the new folder in assets of the library: main/assets/runtime_data/keypoints/eu I also copied there one of the files from "us" folder.
Already made few tests that were producing the crash before - now wasn't able to reproduce the problem!

Fixed by #9.


Add contextvars to package __init__

Closes #53
I've also added celery_current_id and celery_parent_id.

Thanks @Bobronium!


Dataset

Hi, it seems that url http://tosca.cs.technion.ac.il/data/toscahires-mat.zip is not available.
And I notice that you are using another way to load it by "tosca_data_path = "datasets/tosca/Meshes""
So would you mind providing me with your tosca dataset? Thanks!

You will be able to find TOSCA and SHREC'07 datasets from here- http://www.vovakim.com/projects/CorrsBlended/SurfCorr2.0.benchmarks.bins.zip
The official TOSCA website does not seem to work at the moment

Thank you so much for the url you providing!!! I have searched for a long time because of the unknown closing of official TOSCA website. It is important for me. Thanks for your help!


Rename node binary's key generation command

gevulot generate node-key -> gevulot generate key
The node-key command still exists, but will be phased out. The new command name allows for more ubiquitous use of gevulot container e.g. in prover/verifier development purposes.
 Shout me if you disagree with this and we'll revert.


Remove mrc binning

Userstory:
I want to update the dataset schema, so that mrc files no longer have binning.
Metadata update:

Update metadata generation process to no longer store the binned mrc
Update schema to remove fields for binned mrc
Update writes to database to not include fields for binned mrc

This is complete per the updated standardize_dirs script


Update binary_search.cpp

The index of the element will be given by mid+1, considering the indexes start from 0.

Please star this repository to show your support, in the meantime I am reviewing your code.


Software or Hardware Trigger

Hello,
I am working with three GigE zooming cameras (DFK Z12GP031) connected via a GigE switch to the PC. I have a question regarding the triggering of cameras for synchronization. What would be the best approach for triggering (Hardware or Software)?

I would use a hardware trigger.

Thanks.
Using Aravis I tried to activate the hardware triggering mode and I am getting error: "Could not create image stream"
arv_device_set_string_feature_value(Device, "TriggerMode", "On");
arv_device_set_string_feature_value(Device, "TriggerSource", "Line1");
arv_device_set_string_feature_value(Device, "TriggerActivation", "RisingEdge");
arv_device_set_string_feature_value(Device, "AcquisitionMode", "Continuous");
For triggering, I am generating PWM using Arduino at specific FPS. What can be the reason?

I must admit, I am not able to guess why you get "Could not create image stream" since I do not know, how you open the camera, set video format and frame rate. Does the camera show a live stream whithout handling the trigger parameters?
Stefan

yes I am able to get live stream without handling the trigger parameters.

Ok. If the trigger is enable, then you should get one image, if a trigger pulse occurs.  The trigger signal must be a voltage change from 0V to 5V. I hope your Arduino can provide that.
Stefan

Yes, arduino digital pin gives output of 5 volts. I am not sure either the problem is in hardware or software.
Here is how I calculate the triggering pulse.
T_ON = 12 us.
T_OFF = 1/FPS + Exposure_time + Delay (Td)
T_OFF = (1/15)us + 70000us + 1.4us
Is it correct?

Additionally, Since I am using a GigE switch for POE. I used only trig+ (PWM) and trig- (Ground) pins for triggering. I hope I don't need any other source for the optocoupler.

I must admit, I do not know enough about Arduino to justify, whether your code works.
And no, you do not need an extra power for trigger. The trigger is "powered" by the wires. You could use a battery e.g. 4.5V or 9V for triggering, just to see, whether you get an image.
Stefan

@haseeb-uob Can you share some thoughts on how did you solve this? I'm trying to do the same, I am using the external trigger python example.

I must admit, there no solution in this issue, therefore, I have no thoughts.
However, I wonder, why you got the error message "Could not create image stream", if you set the trigger parameters. From our conversation I see, that if you do not set the trigger parameters, you get a nice image stream.
Do you have some source code or better a very small sample project, that shows, what you do and reproduces the issue? Maybe I can find something in the Aravis programming.
Stefan