example directory listings are broken

The union example now gives:
Not Found.

and the express example says
Cannot GET /

the core.js example works.
This bug seems to relate to the new logic at lib/ecstatic.js line 40 so that showDir only ever fires when next isn't a function.

This is expected behavior.

I should elaborate:
From a middleware level, I suspect there are cases where you would not want to serve the directory view. However, I tried to write the directory view such that you could reuse it as another middleware.

As an aside: The "showdir" middleware now works as expected and is included in the examples.

Nifty. Perhaps this could be additionally added to an option too? ecstatic(__dirname + '/public', { showDir : true }) or some such.

I'd be down for that.


evaluate docker 1.10 on the test farm

We're still seeing the connection reset by peer which should be fixed in 1.10 http://build.ros.org/view/Kbin_dj_dJ64/job/Kbin_dj_dJ64__mvsim__debian_jessie_amd64__binary/39/console
And 1.13 is out now which should be even better.

The Xenial puppet refactor supports fetching Docker from the upstream repositories and is being tested with the 17.5-ce release.
When #146 merges this can probably be closed by the giant version leapfrog :frog:


Deployment commit for prod-beta

Merged stage-stable branch to prod-beta.
Use latest commit to update namespace ref in app-interface repo. Don't use merge commit, SHAs must be unique when images are created for each branch.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 65.72%. Comparing base (89d73a2) to head (9578d8b).

Additional details and impacted files
@@             Coverage Diff             @@
##           prod-beta    #3894    +/-   ##
===========================================
  Coverage      65.72%   65.72%
===========================================
  Files            515      515
  Lines          10001    10001
  Branches        2357     2198   -159
===========================================
  Hits            6573     6573
- Misses          3180     3424   +244
+ Partials         248        4   -244

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Test Issue created at Mon, 12 Apr 2021 09:31:41 GMT

Test issue body  Mon, 12 Apr 2021 09:31:41 GMT

Comment on issue at Mon, 12 Apr 2021 09:31:46 GMT


DesiredHeight property in Style doesn't exist on Border control

[Enter feedback here]

<Border.BorderBrush>

</Border.BorderBrush>
<Border.Background>

</Border.Background>

<ControlTemplate.Triggers>

</ControlTemplate.Triggers>

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

ID: bdb9feb2-67b1-f850-9acb-38f05f9ce5f1
Version Independent ID: 1ad823ca-dbdf-a145-0265-0cf29056ec98
Content: Expander Styles and Templates - WPF .NET Framework
Content Source: dotnet-desktop-guide/framework/wpf/controls/expander-styles-and-templates.md
Product: dotnet-framework
Technology: dotnet-wpf
GitHub Login: @adegeo
Microsoft Alias: adegeo

Thank you for opening this issue. I'll add it to the backlog. If you would like to contribute, you can submit a fix by editing the original article. Click on the Content Source link at the bottom of your original comment. To learn how to edit, see the Editing files in a repository article from GitHub.
I did a quick test and found that the value Height works.
Thanks again!


Extensions for LLM

the idea. Write a litte app - put it on a raspberry to swith lights on or off
Would start with phi-3 (which works as .net app) extend with whisper..
Would need a feature like embeddings or functions
Had a look at semantic kernel but needs a http Rest endpoint, which makes no sense.
Any advice?

I didn't have experience to run a LLM on raspberry device. How does the perf look like, for example to run a phi3 model?

@hannespreishuber Your app idea sounds interesting! Can you elaborate on the flow of the application a little more?
Did you try to run phi-3 on raspberry pi already?
Fyi, we are in the process of adding support for whisper.

Closing the issue. Please re-open if this is still relevant.


Fix for numpy 1.15

The appveyor failures in #268 piqued my interest even though they were not directly related to that PR.  And it reveals a failure that seems to be due to numpy: it turns out numpy 1.15 (which I can only assume just got picked up by appveyor recently) had a subtle change of behavior: for whatever reason numpy.squeeze no longer accepts integer arrays, and that approach is used in processing blobs in emcee.  But squeeze  seems to behave exactly the same if you just convert the axes argument to tuples...  So this PR just makes that trivial change.
Local testing on 1.15 seemed to do the job.  We shall see what appveyor says, though.

multi-dim array indexing has changed in 1.15  -- only tuples are allowed now. See the second item in deprecations

@manodeep - thanks for pointing that out! A bit annoying that they use the term "deprecation" even though it's not a deprecation but an immediate fail... Ah well.
Anyway, looks to have done the job.

@eteq Agreed. Deprecation should only yield a deprecation warning, and not trigger a test failure. Kind of the point for having deprecation.

Sometimes one wants to have the DeprecationWarning create a CI test failure. Otherwise the deprecations can be missed, only to have code start failing in use because the changes weren't made early enough.

That's may be true in the CI @andyfaff, but in this case it's an actual failure. That is, anyone who updates to numpy 1.15.0 will start seeing this failure if they are using blobs, I think.
Which begs a question for @dfm: are you planning to have a maintenance branch for emcee 2? If so I guess this should be backported there. If not it's a great advertisement for 3: "Now with numpy 1.15 compatibility!" :wink:

Haha! Thanks for all this. I would like to have a maintenance branch so I'll have a go at backporting this.

This actually won't be a problem for v2 because the treatment of blobs was much simpler back in the good ole days :-)
I've merged this in for v3. Thanks!


(docs): add available docker versions

What this PR does / why we need it: adds available docker versions to the cluster definition doc.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
If applicable:

[ ] documentation
[ ] unit tests
[ ] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

(pending @CecileRobertMichon's note :) )

@CecileRobertMichon made formatting changes, thanks


Can't publish bot to Azure with version 1.3.0

Describe the bug
After updating to 1.3.0 the publish to Azure Web App functionality keeps on spinning on the UI but the bot is never published to Azure.
Opening the developer tools I can see that the request to api/publish ends with a 431 (Request header fields too large) and all the following requests to get the status end with a 404.
Version
1.3.0
Browser

[x] Electron distribution
[ ] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[ ] macOS
[x] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Go to Publish and select a bot
Click on "publish selected bots"
The bot is never published

Expected behavior
The bot should be published to the Azure Web App
Screenshots

Additional context

This is a duplicate of https://github.com/microsoft/BotFramework-Composer/issues/5335


Add the Chinese version user guide for Egress feature

Add the Chinese version user guide for Egress feature
Signed-off-by: Wenqi Qiu<EMAIL_ADDRESS>

Codecov Report

Merging #2437 (e8dc248) into main (7469d7e) will decrease coverage by 17.61%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #2437       +/-   ##
===========================================
- Coverage   59.63%   42.02%   -17.62%
===========================================
  Files         284      146      -138
  Lines       22178    18127     -4051
===========================================
- Hits        13226     7617     -5609
- Misses       7527     9827     +2300
+ Partials     1425      683      -742

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
42.02% <ø> (+0.03%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/agent/agent_linux.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/ovs/openflow/default.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/config/node_config.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/cniserver/pod_configuration_linux.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/apis/controlplane/v1beta1/register.go
0.00% <0.00%> (-92.86%)
:arrow_down:

pkg/ovs/openflow/logs.go
9.52% <0.00%> (-90.48%)
:arrow_down:

pkg/apis/controlplane/register.go
0.00% <0.00%> (-90.00%)
:arrow_down:

pkg/agent/nodeportlocal/k8s/annotations.go
0.00% <0.00%> (-83.88%)
:arrow_down:

pkg/agent/client.go
0.00% <0.00%> (-77.42%)
:arrow_down:

pkg/ovs/ovsconfig/ovs_client_linux.go
0.00% <0.00%> (-76.93%)
:arrow_down:

... and 231 more

I'm not sure if it's common and good to maintain localizated document in the code repo itself, would like to hear others' opinions and experience. @antoninbas @jianjuns @salv-orlando @abhiraut
It should be good to have it on the website, I know @antoninbas had a proposal to maintain website source code in another repo.

I agree with @tnqn, I don't think we should start adding translated versions of the documentation to his repo as it doesn't make sense from a maintenance perspective. When the website is moved to its own repo, we should look into how we can provide the Antrea documentation in different languages.

Thanks for feedbacks. I will close this PR, maybe it can be used in the website repo if needed.


Fix address validation for ConversionHost model

The address validation for the ConversionHost model is busted. It mistakenly does not pass in a parameter, which causes it to blow up with an ArgumentError when called.
Inadvertently introduced in https://github.com/ManageIQ/manageiq/pull/18277

@djberg96 Is this not needed in hammer branch?  https://github.com/ManageIQ/manageiq/pull/18277 was hammer/yes(now hammer/backported), so double-checking...

@miq-bot remove_label hammer/no

@miq-bot add_label hammer/yes

@simaishi updated. Can we use the same BZ? (https://bugzilla.redhat.com/show_bug.cgi?id=1695797)

@djberg96 yes, the same BZ is ok. Thanks!


provide a way for applications to request loader protocols

Ideally plaster could ship with a default loader for INI files very easily via the stdlib ConfigParser, and a plaster_yaml and plaster_toml could also be written for .yml and .toml file extensions respectively. Then plaster would be immediately usable for applications that just want to abstract the file format.
However, right now, plaster.get_loader will get upset if more than one loader is defined for the same file format even if the file format is not what's interesting to the loader. For example, with plaster_pastedeploy we are adding extra methods to the loader to load wsgi-specific constructs such as loader.get_wsgi_app. Consider this a loader that supports a "wsgi" loader protocol (effectively it defines a loader with extra methods).
Pyramid would then only care about loaders which supported this protocol. A regular .ini loader would be unseen.
loader = plaster.get_loader('development.ini', protocols=['wsgi'])

Currently, plaster leaves the loading completely up to the user to specify the appropriate loader. For example, if it's ambiguous then a user needs to say ini+pastedeploy://development.ini. The goal here is to hide this from the user and let them continue with development.ini while the application can help disambiguate by specifying the desired feature-set of the loader. The user could always override this by specifying the exact loader like they can now. This is strictly about reducing the number of cases in which they must do this in order to make the syntax easily digestible.
So far I don't have a solid way to do this, but my first thought is to add an extra bit of info to the entry point registration. Alternatively, I could further specify the ILoaderFactory that addons provide (usually just a Loader(uri) class right now such that we can query it at runtime about its supported protocols. This is less efficient because it requires loading the code behind every entry point instead of using simply the entry point name.

I think one option is just to treat the thing on the right side of the + as the protocol. I think this works out fairly seamlessly. We could either support multi-level protocols (multiple + delimiters) or just one. Assuming we just support one then:
get_loader('development.ini', protocol='wsgi') # -> ini+wsgi
get_loader('ini+pastedeploy://development.ini', protocol='wsgi') # -> ini+pastedeploy

Assuming multiple delimiters:
get_loader('development.ini', protocol='wsgi') # -> ini+wsgi
get_loader('ini+pastedeploy://development.ini', protocol='wsgi') # -> ini+pastedeploy+wsgi

I'm leaning toward the first option myself which allows the user to override the selected loader explicitly.

I'm in a bit of a rut thinking about this. I'm not sure if it's really a solvable problem given the ambiguity in using the file extension. The issue is that there's so many different formats for an ini file. Only part of it is the parser and the rest is the higher-level constructs built on top of the format which are left undefined and ambiguous. For example, Python 3's configparser supports several different interpolation mechanisms including zc.buildout-style ${section:item} and legacy %(default_item) (which is what PasteDeploy uses). The decision about which one to use is going to really matter to the user who wrote the ini file. In plaster's api this decision is only relevant to the user, not the app. However the app will care about the api exposed by the loader (whether it supports get_wsgi_app or not).
I think the best we can do is have the application require a particular feature-set on the loader and if the ini file fails to parse properly then the user will need to be more explicit about which loader to use by specifying the protocol.
Here's the use case I'm hung up on:

Pyramid wants a loader that supports wsgi features (get_wsgi_app).
CLI scripts I write don't care.
I want to share the config file between my scripts and Pyramid.
If plaster ships its own ini parser, and Pyramid installs plaster_pastedeploy then we now have two ini parsers available.
Above I discussed how Pyramid would ensure that it always gets plaster_pastedeploy because it would request the loader that supports wsgi features.
It would be silly for the CLI scripts to specifically request a loader with wsgi features.
How does the CLI get the same loader as the Pyramid app such that the file parses the exact same way?

Part of the issue here is the sort order / priority of the entry points I think. For example, maybe plaster's ini support could specify itself as a fallback such that it isn't used unless there is no other option. This may work but has its own problems and doesn't scale past a single fallback.
Anyway, as I said I'm in a rut with this and it's the primary blocker from really endorsing this library.


Add delete device script

@mjabali can you take a look? Thanks!

@AdrianaPineda Can you change this so the API key is passed as part of the header instead of the URL?
e.g. -H "X-Authy-API-Key: $AUTHY_API_KEY"

@josh-authy @mjabali updated code. Can you take a look again? Thanks!


Display my own profile and all the logged in users in the User Profile Screen.

Time estimate: 3h
This task is part of user story #58

e62732c

870668b

3e08e02

a1bc8bf


Getting started titles are off-center

The title appears to not be centered here

It seems the icons are now too low:


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/40991

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/41058

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/58163

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/64478


ModuleNotFoundError: No module named 'github_com.TheThingsNetwork'

I used python3 in China, and used "pip install ttn" to install the package. But, when I run the code provided as exmaple, some errors happend:
Traceback (most recent call last):
File "E:/python_code/LoRa_AT/test.py", line 2, in
from ttn import MQTTClient as mqtt_client
File "C:\Users\caocongcong\AppData\Local\Continuum\Anaconda3\lib\site-packages\ttn_init_.py", line 1, in
from .ttnmqtt import MQTTClient
File "C:\Users\caocongcong\AppData\Local\Continuum\Anaconda3\lib\site-packages\ttn\ttnmqtt.py", line 12, in
import github_com.TheThingsNetwork.api.discovery.discovery_pb2_grpc as disco
ModuleNotFoundError: No module named 'github_com.TheThingsNetwork'

Thanks for your bug report! I can reproduce it on my side. Looks like a regular package import. I'll have a look at it in the next few days!

Version 2.1.1 of the SDK was just released, resolving this issue! 🎉 Thanks for reporting!

I have an issue :
handler = ttn.HandlerClient(app_id, access_key)
AttributeError: 'module' object has no attribute 'HandlerClient'
when using this library with python2.7 . is it compatibility issue ? any solution plz
command: "pip install ttn" working without any error

@zmarwa I had the very same error as you. My problem was, that my scipt was called ttn.py whch go mixed up with the ttn module.


Bin refinerycms to generator

Overhaul of how we used to generate a refinery cms app.
The idea here is that we use Thor to delegate to different generators and just run them in succession instead of shelling out to the command line.

Will review asap

gem install refinerycms --version 2.0.0
---
61 gems installed
15:01:46: ~$ which refinerycms
/Users/parndt/.rvm/gems/ruby-1.9.3-p0@refinery200/bin/refinerycms
15:01:47: ~$ refinerycms foobar
      create
      create  README
      create  Rakefile
      create  config.ru
      create  .gitignore
      create  Gemfile
      create  app
      create  app/assets/images/rails.png
      create  app/assets/javascripts/application.js
      create  app/assets/stylesheets/application.css
      create  app/controllers/application_controller.rb
      create  app/helpers/application_helper.rb
      create  app/mailers
      create  app/models
      create  app/views/layouts/application.html.erb
      create  app/mailers/.gitkeep
      create  app/models/.gitkeep
      create  config
      create  config/routes.rb
      create  config/application.rb
      create  config/environment.rb
      create  config/environments
      create  config/environments/development.rb
      create  config/environments/production.rb
      create  config/environments/test.rb
      create  config/initializers
      create  config/initializers/backtrace_silencers.rb
      create  config/initializers/inflections.rb
      create  config/initializers/mime_types.rb
      create  config/initializers/secret_token.rb
      create  config/initializers/session_store.rb
      create  config/initializers/wrap_parameters.rb
      create  config/locales
      create  config/locales/en.yml
      create  config/boot.rb
      create  config/database.yml
      create  db
      create  db/seeds.rb
      create  doc
      create  doc/README_FOR_APP
      create  lib
      create  lib/tasks
      create  lib/tasks/.gitkeep
      create  lib/assets
      create  lib/assets/.gitkeep
      create  log
      create  log/.gitkeep
      create  public
      create  public/404.html
      create  public/422.html
      create  public/500.html
      create  public/favicon.ico
      create  public/index.html
      create  public/robots.txt
      create  script
      create  script/rails
      create  tmp/cache
      create  tmp/cache/assets
      create  vendor/assets/stylesheets
      create  vendor/assets/stylesheets/.gitkeep
      create  vendor/plugins
      create  vendor/plugins/.gitkeep
/Users/parndt/.rvm/gems/ruby-1.9.3-p0@refinery200/gems/refinerycms-core-2.0.0/lib/generators/refinery/app/app_generator.rb:238:in `read': No such file or directory - foobar/Gemfile (Errno::ENOENT)
	from /Users/parndt/.rvm/gems/ruby-1.9.3-p0@refinery200/gems/refinerycms-core-2.0.0/lib/generators/refinery/app/app_generator.rb:238:in `find_and_replace'
	from /Users/parndt/.rvm/gems/ruby-1.9.3-p0@refinery200/gems/refinerycms-core-2.0.0/lib/generators/refinery/app/app_generator.rb:177:in `generate_rails!'
	from /Users/parndt/.rvm/gems/ruby-1.9.3-p0@refinery200/gems/refinerycms-core-2.0.0/lib/generators/refinery/app/app_generator.rb:61:in `run!'
	from /Users/parndt/.rvm/gems/ruby-1.9.3-p0@refinery200/gems/thor-0.14.6/lib/thor/task.rb:22:in `run'
	from /Users/parndt/.rvm/gems/ruby-1.9.3-p0@refinery200/gems/thor-0.14.6/lib/thor/invocation.rb:118:in `invoke_task'
	from /Users/parndt/.rvm/gems/ruby-1.9.3-p0@refinery200/gems/thor-0.14.6/lib/thor/invocation.rb:124:in `block in invoke_all'
	from /Users/parndt/.rvm/gems/ruby-1.9.3-p0@refinery200/gems/thor-0.14.6/lib/thor/invocation.rb:124:in `each'
	from /Users/parndt/.rvm/gems/ruby-1.9.3-p0@refinery200/gems/thor-0.14.6/lib/thor/invocation.rb:124:in `map'
	from /Users/parndt/.rvm/gems/ruby-1.9.3-p0@refinery200/gems/thor-0.14.6/lib/thor/invocation.rb:124:in `invoke_all'
	from /Users/parndt/.rvm/gems/ruby-1.9.3-p0@refinery200/gems/thor-0.14.6/lib/thor/group.rb:226:in `dispatch'
	from /Users/parndt/.rvm/gems/ruby-1.9.3-p0@refinery200/gems/thor-0.14.6/lib/thor/base.rb:389:in `start'
	from /Users/parndt/.rvm/gems/ruby-1.9.3-p0@refinery200/gems/refinerycms-2.0.0/bin/refinerycms:5:in `<top (required)>'
	from /Users/parndt/.rvm/gems/ruby-1.9.3-p0@refinery200/bin/refinerycms:19:in `load'
	from /Users/parndt/.rvm/gems/ruby-1.9.3-p0@refinery200/bin/refinerycms:19:in `<main>'

Have to modify the find and replace function from the old script then - there's a built in Thor/Rails actions way to do it for generators


⚠ Bofrost.at has degraded performance

In d4510d3, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1495 ms

Resolved: Bofrost.at performance has improved in c0619ef.


add restart random servers, limit time

Delit setup,add restart random servers limit time

change branch for PR to dev pls. just created.


[pl] Update the password string

The default required length for the password is eight digits as of Laravel 5.8.0.

thank you


cloud watch log group tags, filters, actions

closes #2240

there's one wrinkle on universal tag actions, they don't handle correctly the way that api returns errors. namely it returns a 200 status code, but then embeds per resource failures within a json mapping structure that needs parsing and handling. the primary place this comes into play is on the rate limit throttles, which don't handle the retry when its in an embedded status code.

updated pr to handle individual resource retry

last few commits attempting to work through some test failures due to azure dependency conflicts.


FLUTTER DOCTOR UBUNTU(Zorin OS) Summary shows the Problem Dart Path: Consider adding to front of Path

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
When running kamranhccpszorin@kamranhccpszorin-Latitude-3300:~$ flutter doctor
Actual results
Doctor summary (to see all details, run flutter doctor -v):
[!] Flutter (Channel stable, 3.10.5, on Zorin OS 16.2 5.15.0-76-generic, locale
en_US.UTF-8)
! Warning: dart on your path resolves to /usr/bin/snap, which is not
inside your current Flutter SDK checkout at
/home/kamranhccpszorin/snap/flutter/common/flutter. Consider adding
/home/kamranhccpszorin/snap/flutter/common/flutter/bin to the front of
your path.

Logs

Logs

<!-- Paste your logs here -->

Flutter Doctor output

Doctor output

<!-- Paste your output here -->

Hi @kamranhccp
The warning message is quite clear. You should add the new dart/flutter path before the $PATH variable. Please check this comment for more information.
Closing the issue from here.

Hi @kamranhccp The warning message is quite clear. You should add the new dart/flutter path before the $PATH variable. Please check this comment for more information.
Closing the issue from here.

I tried upgrading flutter using
flutter upgrade --flutter
and it's been solved


Polyfills updates

Updated the following for IE11:

Object.getPrototypeOf
Symbol

Added the following to support Symbol.iterator:

String.prototype.codePointAt
String.fromCodePoint

Coverage remained the same at 80.741% when pulling 71d8f21142e8cf8f5a82753e4a6adb9fdb7a436b on scottmo/polyfillsUpdates into d59df2da95ca1eed158d2d6713597170ac68d83a on master.

Coverage remained the same at 80.741% when pulling 71d8f21142e8cf8f5a82753e4a6adb9fdb7a436b on scottmo/polyfillsUpdates into d59df2da95ca1eed158d2d6713597170ac68d83a on master.

Coverage remained the same at 80.741% when pulling 5eae9990c9f9dea7ab657f3c29bdb8c3c75f1f06 on scottmo/polyfillsUpdates into d59df2da95ca1eed158d2d6713597170ac68d83a on master.


Documentation links to the Slack team for flywheelsports which is password protected

The documentation has a link to Slack that requires a sign in to join. It is great to have a slack channel to talk about Hydra, but it would be even better if I could join the discussions ;)

@jedsmithobd not sure how to open Slack up to just allows signup. But if you send me an email at<EMAIL_ADDRESS>I'll send you an invite.

I think I have seen a slack auto invite that a colleague used for his slack channel. I will see if I can find the link for you.

I did see that too, but it seemed to be tied to domain names. @jedsmithobd saw your email, I'll add you manually. You should see an invite soon.

I found an example of what I was looking for. https://github.com/outsideris/slack-invite-automation. It is basically a simple server that lets an individual enter an email and then automatically sends them an invite.
It also looks like it can be deployed to Heroku or Azure to make it even easier to implement.

Closing for now. Until we have time to implement this.


OCMStub an instance mthod not work as expected

Hi there,
I am trying to use OCMock to test my framework.
I have a simple class:
@interface PrivacyPolicy : NSObject <NSCoding>

- (BOOL)hasPermissionFor:(NSString *)biz scene:(nullable NSString *)scene;
@end

- (BOOL)hasPermissionFor:(NSString *)biz scene:(nullable NSString *)scene {
    return [self someVerifyMethod];
}

and my test code:
    id locationMock;
    locationMock = OCMClassMock([CLLocationManager class]);
    OCMStub(ClassMethod([locationMock authorizationStatus])).andReturn(kCLAuthorizationStatusAuthorizedWhenInUse);

    id policyMock = OCMClassMock([PrivacyPolicy class]);
    OCMStub([policyMock hasPermissionFor:OCMOCK_ANY scene:OCMOCK_ANY]).andReturn(YES);

    [privacyManager requestUserPermission];  // which will eventually call hasPermissionForBiz

    // hope hasPermissionForBiz:OCMOCK_ANY return YES

    OCMVerify([policyMock hasPermissionFor:OCMOCK_ANY scene:OCMOCK_ANY]);  // fail
    OCMVerify(ClassMethod([locationMock authorizationStatus]));  // pass

however, I figure hasPermissionFor: is never getting mocked, the real method is always code, and simply return NO, while I tried to mock as YES.
I have no idea what's wrong, because mocking authorizationStatus is successful.
Could someone help why or where I made some mistakes, and maybe give some places I could debug with?

UPDATE:
when I try:
- (void)testMock {
    id policyMock = OCMClassMock([PrivacyPolicy class]);
    OCMExpect([policyMock hasPermissionFor:OCMOCK_ANY scene:OCMOCK_ANY]).andReturn(YES);

    PrivacyConfig *config = [PrivacyConfig currentConfig];
    PrivacyPolicy *policy = [config getPolicyWithPermission:TestPermissionKey];

    BOOL val = [policy hasPermissionFor:KSPKTestPermissionKey scene:@"test"];
    XCTAssertTrue(val);
    OCMVerify([policyMock hasPermissionFor:OCMOCK_ANY scene:OCMOCK_ANY]);  // fail
}

When I tried to explicitly call hasPermissionFor:scene: it seems no matter with or without .andForwardToRealObject().andReturn(YES) the XCTAssertTrue and OCMVerify never pass, indicating the mock is not working at all.
I'm not sure if this is a limitation, or bug? because it seems PrivacyPolicy is a very simple class and nothing special.

This is looks like a problem with using OCMock. What is the object returned by getPolicyWithPermission:? Assuming that this is not the mock object, how would the mock notice that a method is called on that object?
Please do not ask follow-up questions here. Questions relating to the use of OCMock should be asked on StackOverflow. Thank you.

@erikdoe I do hesitated where to ask before opening this thread, but eventually I thought I have read the doc and thinking it's worthy of confirming whether it's a bug or not.
Because in the doc
id mock = OCMClassMock([SomeClass class]);
OCMStub([mock someMethod]).andReturn(myValue);

/* run code under test */

OCMVerify([mock someMethod]);

it seems I can hook a instance method on a Class level mock?
getPolicyWithPermission returns a PrivacyPolicy instance, which I assume
id policyMock = OCMClassMock([PrivacyPolicy class]);
OCMExpect([policyMock hasPermissionFor:OCMOCK_ANY scene:OCMOCK_ANY]).andReturn(YES);

should work for this returned object? Could you confirm this is not a bug?
e.g. how could I mock a instance method for all instance for a specific class? Thanks.

It's not a bug. You have a basic misunderstanding of what the code is doing. OCMClassMock does not return a class level mock, it's returning an "mock" instance of the class. Please follow through with posting to stack overflow so that others can easily find the answer if they have similar questions.


Add Crossfeed scans to SQS process

💡 Summary
Now that P&E scans are functional in the SQS process, add the remaining Crossfeed one's to be run on all CyHy organizations.
Motivation and context
This process is better suited for scaling and management.
Implementation notes
These will be added:

[x] Censys
[x] Xpanse

@aloftus23 - Just clarifying. With this PR - https://github.com/cisagov/XFD/pull/68 is this item closed or additional work needed?

Closed and opened a new ticket for other scans: https://github.com/cisagov/XFD/issues/138


grammar error 'stucked'

Hi thyarles, thank you for this amazing script.  Quick note, "stucked" should be "stuck".

Hi @patrick0057, thank you! I'm looked at the code, there a lot of the words "stucked"... all of them need to be changed? For example:
--delete-resource     Delete resources found in your **stucked** namespaces
--delete-all          Delete resources of **stucked** namespaces and broken API
--force               Force deletion of **stucked** namespaces even if a clean deletion fail

Could you, please, paste here the full sentence?
Thanks.

Hi @thyarles yes all occurrences of "stucked" should be "stuck".  You can just run a search and replace on everything for the word "stucked".  "Stucked" isn't actually a word but I understand what you mean when using it in your script and README.md.  You mean the past tense of "stick" which is "stuck".

Hi @thyarles yes all occurrences of "stucked" should be "stuck". You can just run a search and replace on everything for the word "stucked". "Stucked" isn't actually a word but I understand what you mean when using it in your script and README.md. You mean the past tense of "stick" which is "stuck".

Awesome, @patrick0057!! Done.
Thank you for the lesson. Now my English is a bit better.


Records are treated as case sensitive in PowerDNS-admin

We have a domain, AllStarLink.org in the domains table, and the records are a mix of host.allstarlink.org and host.AllStarLink.org.
When you go in to edit a domain, if the domain part is no stored in the database record as matching the case of the domain record, PowerDNS-Admin will not reconzise the host portion, and save it as host.allstarlink.org.allstarlink.org in the database.  pdnsutil edit-zone doesn't care about case and DNS is case insensitive.  This is a fairly major bug in PowerDNS-Admin as it caused the entire zone to be rewritten and we had to revert by scripting.
I've attached an example screen shot showing this in the domain edit page.

Finding it a bit hard to understand what it is you're trying to do that gets you to this point? PowerDNS-Admin won't let you create mixed case domains, so I guess the database is pre-existing and created with another tool? And you are then trying to add a new record? or edit an existing record? and apply changes corrupts the record you added/changed? or all records?

Hello,
This request is stale for too long. Regarding our policy to clean up this project, we're closing it. Feel free to open a new issue later.
Regards
PowerDNS Admin Team
See https://github.com/PowerDNS-Admin/PowerDNS-Admin/issues/1153 for the policy


EnTT in Action - add to list

I would be happy if you add my project to your "EnTT in Action" list. It is a small engine that I program in my spare time for learning purposes.
https://github.com/stwe/SgOgl

Sure, I'm glad to do that! Thank you for pointing it out.
I'll add it as soon as possible upstream. What version of EnTT have you used? Just for reference.
Since you've used actively EnTT, may I ask you if you've any feedback or feature request that would have made your goal easier to accomplish?

Version: EnTT v3.2.1
Much would be easier for me if you had your own small and simple sample program. This should include the most important things (create entities -> add components -> create and use systems -> events). Other developers do it like this: https://github.com/SuperV1234/ecst/blob/master/CODE_SAMPLE.md
You currently have your wiki and programs from other developers. That's OK. But you have to find everything in different places. I also don't know if what others do is "best practice".


no console output is displayed for tests

Is there something wrong with my operation? Please have a look, thanks！

@insureyang  Click on View tab and select Toggle Developer Tools to see token value. You have not added any tests.


added configuration for hour and minute labels in time picker

Hello,
Most of the labels in the picker are configurable, but the Hour and Minute are not. I've fixed that in this pull request.
I would be grateful for review and merge.
Thanks,
Michał Schielmann

@mominsamir Can we merge this one? Is there anything wrong with it?


Add the original Austin law

Just for fun, I added Austin's original law from [1], and also included the proof that it has no finite models. The proof that it has satisfying infinite models is not done.
[1] A. K. Austin. A note on models of identities. Proc. Amer. Math. Soc., 16:522–523, 1965.

Are you planning to add the infinite model case to this PR in due course, or would you prefer that this PR be merged as is?

I think it should be merged now. I'm not sure if I'll get to the infinite case, and I think it's a nice historical identity to have regardless of that proof.


When is it applicable?

Doesn't work for my import connection. I have read that it only works with the Direct Query.

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

ID: 99521173-b241-25e5-ea78-de782dc25edc
Version Independent ID: 98a21d67-6880-73cd-571b-cfcf4aeac4f2
Content: Change data source connection strings with PowerShell - Power BI
Content Source: powerbi-docs/report-server/connect-data-source-apis.md
Service: powerbi
Sub-service: powerbi-report-server
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

UPD: doesn't work with the Direct Query as well.


[frontend] Create a public user page

Similar to the team-info page we have -- a page with info for each user. (Not sure what might go on here / what's could be considered public)
A couple reasons why:
So that users can individually advertise their own info. (bio, links to other websites like personal website or LinkedIn or github or whatevs)
So that users can be on multiple teams across multiple competitions. Handy if we make a all-version scrimmaging website

Thinking that the inital public-facing user page will consist of an updated userCard and the user's teamCard below it. This will provide basic info about the user while leaving room for future feature-adding.


Unexpected key(s) in state_dict: "aux_heads.tm.linear.weight", "aux_heads.tm.linear.bias"

I am trying to run inference using OpenFold and I found that finetining_ptm_2.pt was missing in openfold/resources/openfold_params. I found a similar file on hugging faces  and used their file. But I am getting the following error :
Traceback (most recent call last):   File "/opt/openfold/run_pretrained_openfold.py", line 391, in <module>     main(args)   File "/opt/openfold/run_pretrained_openfold.py", line 203, in main     for model, output_directory in model_generator:   File "/opt/openfold/openfold/utils/script_utils.py", line 101, in load_models_from_command_line     model.load_state_dict(d)   File "/opt/conda/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1605, in load_state_dict     self.__class__.__name__, "\n\t".join(error_msgs))) RuntimeError: Error(s) in loading state_dict for AlphaFold: 	Unexpected key(s) in state_dict: "aux_heads.tm.linear.weight", "aux_heads.tm.linear.bias".
The huggingfaces file is from 11 months ago. So, is there a more recent file that I can use or should I remove those keys altogether?

you may need to add the config_preset command. In the README this is used in the non-docker example, but not the docker example and resolved this issue in my use.
--config_preset "model_1_ptm" \

I don't use docker to run the code, but get the same issue, and try the config_preset config, it doesn't work. Have somebody solved this problem? please give me a hand.

I found that using the "finetuning_ptm_1.pt" to replace "finetining_ptm_2.pt" can work well.

Hello!
I' ve got such error RuntimeError: Error(s) in loading state_dict for AlphaFold:                                                                     Missing key(s) in state_dict: "aux_heads.tm.linear.weight", "aux_heads.tm.linear.bias".
when I tried to use my own checkpoint after train OpenFold.
 --openfold_checkpoint_path /checkpoints/14my.ckpt  for running run_pretrained_openfold.py script
Who can explain me please how to solve this problem?


Tons of errors in MS documentation

The official Microsoft documentation (https://learn.microsoft.com/en-us/microsoft-365/security/defender-endpoint/mac-install-with-intune?view=o365-worldwide#step-14-download-the-onboarding-package) referencing the files in this repo is awfully messed up. Here are the issues I found:

Step 1 / 1: not in Microsoft Defender portal, but in Endpoint/Intune portal
Step 2 / 9: missing file selection
Step 4 / 6: someone copypasted name from Step 3 (FullDiskAccess), it should say something like BackgroundService
Step 4 / 8: again wrong suggested name (should be background_services.mobileconfig)
Step 5 / 6: same issue as step 4/6
Step 7: wrong github link (needs to be raw, as is downloads a github json)
Step 7: doesn't mention that user should look for ChannelName in the XML to choose one of the channels (default set to Production, which is OK)
Step 8: the "Microsoft Defender portal" link is all wrong and links to some weird place that requires a MS (non-365) account
Step 11: why can't DLP be included in the .mobileconfig file we download in step4, also which .mobileconfig do we update with features/dataLossPrevention/enabled?
Step 15: it would be nice to say we're switching back from Defender portal to Intune portal.
Step 15 / 6+7: why are you callling it Autoupdate? that was Step 7. It should be something like "Licensing" or "Onboarding"
Step 15 / 10: which effing file?? (it's the one in the intune directory inside the zip we downloaded in step 14)

@tmpkn , please leave comments to corresponding pages directly.
This repo is for open sourced tools for Defender for macOS and Linux only


qna authoring tests

API changes have been detected in azure-ai-language-questionanswering. You can review API changes here

API changes have been detected in azure-ai-language-questionanswering. You can review API changes here


Distribute strategy extended broadcast_to broadcasts different tensors when using tf.random

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
TensorFlow version (use command below):1.14
Python version:3.6
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the current behavior
strategy.extended.broadcast_to does not broadcast the same value when using tf.random.uniform.According to the description of the method, it is supposed to mirror a tensor on one device to all worker devices
Describe the expected behavior
Should broadcast the same value to all devices?
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

In order to expedite the trouble-shooting process, please provide a code snippet to reproduce the issue reported here. Thanks!


allow setting token via environment variable for spin cloud deploy

it will be useful to support setting an env variable which can be consumed by spin cloud deploy to do the deployment.

Can you elaborate on the use case @rajatjindal?  Perhaps for scenarios where the spin cloud login action to Fermyon Cloud has been done separate from the current environment (i.e. the json config wouldn't be present locally at spin cloud deploy time?)
Question on precedence: If the env var is non-empty, I'm assuming its value should take precedence over the token stored in the json config, if exists, for the current deployment environment?

Can you elaborate on the use case @rajatjindal? Perhaps for scenarios where the spin cloud login action to Fermyon Cloud has been done separate from the current environment (i.e. the json config wouldn't be present locally at spin cloud deploy time?)

yes, precisely that. if we have a cloud token, we should be able to use that directly without having to write it to a file in a specific format.

Question on precedence: If the env var is non-empty, I'm assuming its value should take precedence over the token stored in the json config, if exists, for the current deployment environment?

yes, I would think so too.
on the other hand, if we want to introduce a flag --token instead of environment variable, that works too.

You'd need an env var for server URL (and arguably "danger_accept_invalid_certs") too, wouldn't you? I wonder if we'd be better off just adding a --token opt to spin cloud login to update the credentials file.

I'd be a big proponent of adding env vars over requiring we write config to a fixed file - esp for CI/CD Workflows where you probably wanna pull those from "some" kind of secret store


Fix delete statement for exclusions

Only stumbled over this recently. Repeated changes to exclusions within a single alt group dont work, because I messed up the SQL for the delete statement

Great catch.


Do not freeze ROT exports

Changed Rollup config so that exports from ROT are not frozen.  This allows extension libraries to add directly to the ROT objects.  For instance:
window.ROT.Util.distance = function(a, b) { ... }

Hmm, this looks somewhat incompatible with the ES6-modules version of rot.js (ES6 exports are immutable by design).

Ok.


Error handling on download and Fedora

Error handling on download failures and support for Fedora remix install (sideloading required).

Now using net.webclient object to download packages instead of original Invoke-WebRequest method. This should make downloads a bit faster. Sideloading support for unsigned appx packages has also been added.


Add RandomUniform op

Here be a RandomUniform op, which generateth numbers according to a uniform distribution.
It takes four arguments and one static parameter.
Arguments:

min_value (scalar, any floating type)
max_value (scalar, same type as min_value)
result_shape (vector of i64)
use_fixed_shape (boolean scalar)

Static parameter:

fixed_seed (size_t)

To do:

[x] ~Add CPU support~ Just updated manifest for now
[x] Update manifests
[x] Add .rst doc

In order to deal with the templating madness, I've changed UniformRNGState to always generate double internally, and the reference casts it to the actual result type (note that RandomUniform is only allowed for FP types currently). I'm not 100% sure this is the right thing to do, but since I don't think we are considering classes in ngraph/state to be part of any public API, we can revise it pretty easily in future PRs should it cause issues.


bug fixes and python 3.5 compatibility

Hello,
Found your diary on linuxfr ;)
I was interested by you work for my own (mpd running on a OPi) but I had to fix few things:

some copy/paste bugs on config file handling
removal of f-strings which are not supported below python 3.6
documentation update

Feel free to merge my fixes if you find them useful ;)
Regards,
Eric

Thank you for your contribution Éric!
That got me thinking this project needs a proper setup.py installation script.
It should help specify the dependencies with their required versions, and it will also handle installation, and interpreter environment.
If you agree, I can have a look into this in a few days, and I think I only need your first commit in that case (I will take care of the spurious print line).


linking -latomic is not portable

This commit which replaces boost atomic with std::atomic automatically links -latomic when testing for std::atomic support and in the main CMake script. Unfortunately this is in general not completely portable and in particular breaks compilation on macos with Clang where there is no libatomic. As I understand it for reasons that are unclear to me gcc chose to implement fallbacks for lock free routines in libatomic instead of of libstdc++ but this is not standardized across compilers. It seems that something like this is necessary to do this portably.

@K-ballo what's your take on this?

Let's add the check.

Added the check to libatomic-check branch. Local builds and CI look ok, but I do not have access to macos to test.
@kleinhenz could you give it a try and report whether it works as needed?

Works for me. Thanks!


Resolve security vulnerability reported below

Resolve security vulnerability reported below
https://github.com/omniauth/omniauth/wiki/Resolving-CVE-2015-9284

Closing and then reopening to put on F19 Kanban board

@ProbablyFaiz looked at this and determined that:

it was unlikely to be  seriously vulnerability unless we were to do some very specific, very stupid thing in our code.
that this was the position of the folks that maintain the library
that GitHub is stubbornly refusing to accept this explanation

So I'm proposing that we document this a bit in the README.md, with some references (via links to urls) to the sources of our info, and some information about what specific stupid thing we should not do, and then we'll just move on and consider this resolved.


Take advantage of gembox

I think that it is easy to use if it is divided according to the type of library and the general purpose.
It is a subdivision from the previous default.gembox.
By type gembox:

stdlib: Add some standard Ruby methods not provided by core.
stdlib-ext: Add some standard Ruby methods that are not provided by stdlib.
stdlib-io: Provides IO, File, Socket, print, etc. Conflict with MRB_NO_STDIO
math: Add a class related to math. Conflict with MRB_NO_FLOAT
metaprog: Adds features related to metaprogramming.

Purpose gembox:

default.gembox: Import the same gems as before
default-no-stdio.gembox: Import the stdlib and math
default-no-fpu.gembox: Import the stdlib only

I think there is room for debate about the gembox name and classification.

Done.

Oops, I made a mistake. Reopen.


BLEScan->setInterval

I am using the the BLE_scan example program to read an iBeacon.
I can use the program as is and get a reading.
Now I am wanting to get a reading every 10 seconds or so. Is using BLEScan->setInterval the best way to do that?
I am not sure how that works but when I put in the line pBLEScan->setInterval(10000); the program stops working.

There is 3 channels for ble and ble is scanning for very short time (few millis to about 10 sec) and usually jump to next channel. That what for is interval and window. Then you have scan time in BLEScan->scan(time).
In you case you can try to play with setInterval() and setWindow() (never been trying it), but i think its better to have loop with delay(10000) and have BLEScan->scan(time) inside this loop.


Enforce JDK 11

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

Thank you


Should be possible to customise day abbreviations in dialog header

Is your feature request related to a problem? Please describe.

In Dutch, days are shorted to two letters, so instead of "din", I need it to say "di".
Describe the solution you'd like
A way to customise the names used at the top. For instance, I'd like to be able to write something like
dialogHeaderDays={{
  monday: "ma",
  tuesday: "di",
  wednesday: "wo",
  thursday: "do",
  friday: "vr",
  saturday: "za",
  sunday: "zo"
}}

Describe alternatives you've considered
A simpler alternative might be to set the length of days at the top, so in my case I would write
headerDayLength={2}

I think this is not preferable though, because other languages might use varying lengths to abbreviate day names. For instance, in Dutch, "woe" is also used for Wednesday.
Additional context
It's not obvious to me if this is an issue with this package, @date-io/date-fns, or date-fns/locale/nl.

I realise now the obvious solution would be to have a prop to specify the date format used in the dialog header.

Nevermind I found how to do it:
import DateFnsUtils from '@date-io/date-fns';

and
DateFnsUtils.prototype.getDatePickerHeaderText = function(date) {
  // wo. 6 nov
  return moment(date)
    .locale('nl')
    .format('dd. d MMM');
};


split one l1 block into multi circuit instance

it's very likely that a circuit cannot contain a full l1 block. So we may need to do this splitting.

@lispc when are we supposed to use the zkEVM 'project' ( https://github.com/orgs/scroll-tech/projects/2/views/1  )?

@lispc when are we supposed to use the zkEVM 'project' ( https://github.com/orgs/scroll-tech/projects/2/views/1 )?

We used it before. Currently we mainly rely on jira.

@lispc when are we supposed to use the zkEVM 'project' ( https://github.com/orgs/scroll-tech/projects/2/views/1 )?

We used it before. Currently we mainly rely on jira.

Gotcha, thx

currently it takes 7000 rows for proving 1540 opcodes with 5 txs(4 succ + 1 fail) in multi-erc20.json.
So 2**18 circuit can prove 187 erc20 transfers.

this design now is called "continuation" widely.  A long term big task. Will revisit later.


how to scrape data about the project creators?

Hi everyone, I am doing academic research about project description and crowdfunding success. i have a dataset of 2000+ projects, and I would like to also get information about their creators (as mentioned in the WIKI), such as the biography of the creators and whether they have posted their facebook accounts, and if yes, how many friends they have. What can I do to scrape those data? Many thanks!!!

I think you can get a little bit of info about project creators through Kickstarter's API... unfortunately it won't be nearly as much as you're looking for, I think. Here's the general idea of what you can do:
project = c.search_projects('whatever').first # or project = c.find_project xxxxxxxxx (kickstarter id number)
puts project.creator.inspect```


Move null checks to two equals

Also updates one test that was failing before I changed anything

hec that's wrong


Automatically triage ASSIGNED and REOPENED bugs too

For ASSIGNED there is less value as a developer has definitely already seen it and so it was likely already triaged.
For REOPENED, it might be more valuable.

Coverage remained the same at 28.401% when pulling 9100ad0180e92b025e31203922ae8eb8b7715301 on marco-c:more_opened into 81549c2e2ebb94c7d7970ea30e682722fd356ef7 on mozilla:master.

For ASSIGNED, maybe we should ni the assignee to ask her/him to set p::c.

For ASSIGNED, maybe we should ni the assignee to ask her/him to set p::c.

We could write a separate script for that, but maybe the number is so low that it is not worth it to do something specific for this case.

Indeed, https://bugzilla.mozilla.org/rest/bug?component=Untriaged&bug_status=ASSIGNED&count_only=true is only 2, so I would just use the component suggestor and let the assignee fix the suggestion if needed.


Set allowBlobPublicAccess to False in all storageAccount provisioning

This is intended to avoid having someone create a public container in those.
TODO:

[ ] fix the diff test
[ ] rebuild biceps

Abandoned, this will be covered as part of the refactoring done in #132


Remove deprecated Text.lineColor attribute in graphical annotations

Text.lineColor (as all other contents of FilledShape) were deprecated in MLS v3.3 and should be replaced by Text.textColor in an MSL based on MLS >= v3.3.
@HansOlsson How easy can you figure out all the occurences of deprecated FilledShape attributes on Text in MSL? There are plenty of them ...

@beutlich We already have logic in Dymola to allow users correct some other annotations (flipped Bitmaps and misuse of LinePattern.None for connection-lines) and it was straightforward to re-use that logic to replace lineColor by textColor and remove the other ones (there are some uses of lineThickness, fillColor, fillPattern, and even pattern).
Shall I make a pull request based on that - or just removing the meaningless attributes, and keep lineColor?Note that some tools do not yet generate the Modelica 3.3 annotations for textColor - since lineColor is more compatible.
BTW: Since I extended the previous routine it modified one connection line in Modelica.Magnetic.FluxTubes.Examples.MovingCoilActuator.Components.PermeanceActuator - I will check if this still happens, and in that case open a separate issue on that. (I could understand that you sometimes want invisible connection lines - but not that one.)

@HansOlsson A PR (=pull request) against master branch is highly appreciated. For now, Keeping lineColor and removing the silly Text attributes should cause no harm. Thank you.

As far as I understand the merged pull-requests closed this issue.

No, this is the task for the MSL based on Modelica > 3.2. As long as we do not know, what the master branch will be based on for the next release, we have to keep it open (and use textColor attribute once we know that master can use Modelica 3.3 features).


Support Minitest's parallel execution

This morning, We encountered a strange issue when integrating this gem with our Minitest test suite. Sometimes, the OpenAPI schema was generated, sometimes not. In the end, we discovered that no schema was generated when running all tests simultaneously, but it worked fine when only running a specific controller test.
Rails offers a helper named "parallelize" to run multiple tests simultaneously. However, this only comes into effect when running a more considerable amount of tests. Single tests are still executed in one process.
My assumption right now, which explains the phenomenon explained earlier, is when Rails starts parallelizing the tests, each process has its copy of RSpec::OpenAPI.path_records. The primary process does not execute any tests, and therefore, its version of the path records remains empty, and no OpenAPI schema is generated.
Parallel test execution is a native feature of Minitest. It would be nice if RSpec OpenAPI could support it.

I will not work on this since I do not use minitest.
PR welcome.


Triage on ostracon#324

https://github.com/Finschia/ostracon/pull/324 adds time some info to info logs. This looks be able to be merged into cometbft and be suggested to upstream.

@loloicci
Do you know the background behind why it became necessary to log this information at info level?


UI - Uncertain status of fix to block publication of dataset

There is a UI update bending (branch = fix-no-sample-publish) that blocks publications of datasets that have no samples or that have no processes. The status of this fix is uncertain, because it was not behaving as expected; and this problem was tracked to some problem with samples not being included in the dataset when a process was added. The feature needs to be tested at the UI and the code and data need to be reviewed.

Ref to commit: https://github.com/materials-commons/materialscommons.org/commit/5d78c45ba3fb03ec17edb9eafc270f09ae9fce1b

Disable public button in this case?


OCPBUGS-2812: Do not skip compliant policies containing status fields

/jira refresh

/assign @Missxiaoguo

With this change, the compliant subscription policy will not be skipped at the beginning. Wondering if you tested in real? Can the subscription policy become NonCompliant before TALM gets into it?

The code looks good to me. With this change, the compliant subscription policy will not be skipped at the beginning. Wondering if you tested in real? Can the subscription policy become NonCompliant before TALM gets into it?

No. I did not. This fix is as good as the annotation workaround. The race condition is tracked by OCPBUGS-2813. In the scale lab, I've seen the catalog source alone taking more than 12 minutes to become ready again: https://redhat-internal.slack.com/archives/C03QX2FG4JW/p1674145146463829

/assign @sudomakeinstall2

/cc @imiller0

/lgtm

/cherrypick release-4.12


Sileo doesn't respring after installing tweaks.

i6, 12.1.2

I'll fix soon

Fixed in latest compat layer update


Update cla.yml

[ ] All tests passed. If this feature is not already covered by the tests, I added new tests.
[ ] I used npm run format for formatting the code before submitting the pull request.


Get a MWE (minimal working exemple) in the README.md to demonstrate basic features

I feel the use of LaTeX should be encouraged with this package enabling KaTeX in Genie.
Also, I would like to try this out, but I am not sure how to use it in my Genie app.
So a MWE would be nice in the README.
(A detailed documentation would simply follow, I guess).

@essenciary
We could paste my example from the latest PR in the MWE, what do you think?

@hhaensel agree
Just a bit concerned about the route section. The global doesn't look like good practice. Would be good to "compact" the code in route and remove the global.

Something like this should work too?
route("/") do
    page(@init(), ui()) |> html
end

Have a look at the README.md

@samusz can we close this?

Sure !
Le lun. 17 avr. 2023, 23:35, hhaensel @.***> a écrit :

@samusz https://github.com/samusz can we close this?
—
Reply to this email directly, view it on GitHub
https://github.com/GenieFramework/StippleLatex.jl/issues/8#issuecomment-1512110218,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACTNSS6DF54GVW2B7NYLK3XBWY6RANCNFSM456TMZHA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Bug 12

The PR contains 3 changes:

travis.yml so bugs like this are prevented in the future
The transfer encoding patch from @rkaw92 which improves caching support in many cases
Closes #12

Could someone agree on this commit and release it on master ?


Bludit v3.9.2 Code Execution Vulnerability in "Images Upload"

A Code Execution Vulnerability in Bludit v3.9.2
There is a Code Execution Vulnerability which allow to get server permissions,the url is /admin/ajax/upload-images
1, login with any account which allows you to edit content

2, click the button of Images to upload a picture

3 change the name and content of the file and then upload

and also upload a.htaccess file

The file will now be under the TMP folder

4 visit evil php  http://<IP_ADDRESS>/bludit-3-9-2/bl-content/tmp/shell.php

PHP version
PHP 7.3.2

Hi,
thanks for share the vulnerability.
I was thinking this was solved with checking the file extension, I will check.

Hi,
When you upload the file, it will check the fle extension,but before this, file will be move to tmp folder.Although .htaccess will limit the path of tmp, you can also uplaod a new file of .htaccess to tmp folder to override the original route.Thus can visit the tmp folder.

I uploaded a fix, I delete the file from the temporary directory.
https://github.com/bludit/bludit/commit/7092697112b8e362703e77de796c05cb61bf63b2

I added check the extension file, if you can try to do the exploit with the version from Github. I will release a new version in a few days, or maybe a beta.

Fixed in Bludit v3.10.0.


REACH-182 Replace using "that = this" with ".bind(this)"

Nothing special, just replacing 'that' with 'this'.

Hi @pbidenko,
This looks great.  It makes the code base a good deal more consistent.
If you can fix the conflicts, I can pretty merge this in.

Hi @pboyer,
I've fixed conflict, please take a look.

Thanks @pbidenko!


Just a few questions

I started off using many of my hours analyzing the way this guy did it from https://www.void.gr/kargig/blog/2016/12/12/firejail-with-tor-howto/
But theres 3 questions i have about this.

I couldnt understand the last cmd he posted:
iptables -t nat -A PREROUTING -i tornet -p tcp -m tcp --dport 9050 -j DNAT --to-destination <IP_ADDRESS>:9050
This is for Applications supporting SOCKS5, Im not sure why this traffic needs to be redirected if we have already created a separate SOCKS5 proxy <IP_ADDRESS>:9040 ??
How could apps that support SOCKS5 still connect through the bridge?

I did a lot of tests, and it seems that if you run for example xterm in firejail using that bridge, if you decide to stop the tor service, and if the app (in this case xterm) is malicious and isnt able to connect to their servers (you can still ping from the jail by the way), it can try to ping their own servers, and they could analyze with tcpdump on their end where the ping is coming from. The ping reveals the users REAL IP address.  I have tried to block icmp to/from the bridge tornet using iptables, but the jail STILL is able to ping outside, even though there is no resolving of any kind.
Does anyone know why this happens here?
How is orjail different from this?  Does it block icmp and any connectivity in the namespace when TOR service is stopped?

How is creating a network namespace different from a bridge when being used with TOR and firejail?

Thank you in advance

3 - Firejail creates a network namespace when a bridge is used with it, the difference is that the firejail netns is a unnamed network namespace.


[DOCS] Documentation Example of how to apply top-down, middle-out, bottom-up approach.

Hi,
In the fpp2 book on the chapter  Forecasting hierarchical or grouped time series, it is possible to easily switch between different methods (top-down, middle-out, bottom-up) when using the hts approach.
prison.gts <- gts(prison/1e3, characters = c(3,1,9),
  gnames = c("State", "Gender", "Legal",
             "State*Gender", "State*Legal",
             "Gender*Legal"))
forecast(prison.gts, method="bu", fmethod="arima")

can you provide an example of how this can be done using scikit-hts package

https://scikit-hts.readthedocs.io/en/latest/hts.html#hts.HTSRegressor.__init__
You can see which approaches are available here

@carlomazzaferro Is bottom-up the only available approach? I do not see anything explicit but I found the function forecast_proportions to be a possible candidate for the Top-Down approach? But how can I use this function to perform this approach?


push: compiles in windows

libc::gethostname is not  available in Windows.
/cc @syepes

@overvenus Thanks!

LGTM

PTAL @disksing

LGTM


Fix module upd vers and add docker

solves problem #152

@gjsjohnmurray , this is for ZPM publishing only, that's why they are in /build. Those files are not duplicates (check e.g. file StaticContext.cls).
This is because @ZitRos has a pretty sophisticated Build procedure, which generates a release XML file.
Those files in /build/src are the exported cls from the release XML. Which are different with source files in /src/cls
This will just help to issue ZPM module now via OEX.
If @ZitRos or/and @daimor have better ideas how to manage the ZPM support, please suggest them.

Hello! As soon as we have IRIS available in docker (we have, right?), this could be achieved in adding a GitHub Actions pipeline to this repo with the build instructions in the container. Actually, this is the only clean way to builds; just adding *.cls files to this repo built somewhere else will likely break compatibility with older IRIS/Cache releases - you need to carefully test it.
@evshvarov why XML files aren't enough? By having Docker-enabled GitHub Actions pipeline you'd easily refactor and run that build scripts which will output XMLs for use. I can help @SergeyMi37 to implement it - let me know

I.e. this PR should be rather a PR that adds .github/actions pipeline, which can then commit these built files to this repo, if needed.

For ZPM package to be published the repository needs to have files to put
in it.
On Sat, 29 Oct 2022 at 23:24, Nikita Savchenko @.***>
wrote:

I.e. this PR should be rather a PR that adds .github/actions pipeline,
which can then commit these built files to this repo, if needed.
—
Reply to this email directly, view it on GitHub
https://github.com/intersystems-community/webterminal/pull/153#issuecomment-1296002550,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAVHEP4QUEE7VXLRXEGOVKTWFWPYTANCNFSM6AAAAAARRX7QXU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Yea, so why wouldn’t we generate these files in GitHub Actions pipeline? It
will be automated and most importantly will always produce the same version
of files for the same code.
The setup should not be complex, linux build scripts are already here.
On Sun, 30 Oct 2022 at 05:47, Evgeny Shvarov @.***>
wrote:

For ZPM package to be published the repository needs to have files to put
in it.
On Sat, 29 Oct 2022 at 23:24, Nikita Savchenko @.***>
wrote:

I.e. this PR should be rather a PR that adds .github/actions pipeline,
which can then commit these built files to this repo, if needed.
—
Reply to this email directly, view it on GitHub
<
https://github.com/intersystems-community/webterminal/pull/153#issuecomment-1296002550
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAVHEP4QUEE7VXLRXEGOVKTWFWPYTANCNFSM6AAAAAARRX7QXU
.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/intersystems-community/webterminal/pull/153#issuecomment-1296106918,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGCCSADGBBPJ7G6CNAKUIDWFX4VLANCNFSM6AAAAAARRX7QXU
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Nikita https://zitros.tk

The necessary files SHOULD be in the repository. So can GitHub Actions help
here? Not sure
On Sun, 30 Oct 2022 at 08:43, Nikita Savchenko @.***>
wrote:

Yea, so why wouldn’t we generate these files in GitHub Actions pipeline? It
will be automated and most importantly will always produce the same version
of files for the same code.
The setup should not be complex, linux build scripts are already here.
On Sun, 30 Oct 2022 at 05:47, Evgeny Shvarov @.***>
wrote:

For ZPM package to be published the repository needs to have files to put
in it.
On Sat, 29 Oct 2022 at 23:24, Nikita Savchenko @.***>
wrote:

I.e. this PR should be rather a PR that adds .github/actions pipeline,
which can then commit these built files to this repo, if needed.
—
Reply to this email directly, view it on GitHub
<

https://github.com/intersystems-community/webterminal/pull/153#issuecomment-1296002550

,
or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/AAVHEP4QUEE7VXLRXEGOVKTWFWPYTANCNFSM6AAAAAARRX7QXU

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
<
https://github.com/intersystems-community/webterminal/pull/153#issuecomment-1296106918
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ABGCCSADGBBPJ7G6CNAKUIDWFX4VLANCNFSM6AAAAAARRX7QXU
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Nikita https://zitros.tk
—
Reply to this email directly, view it on GitHub
https://github.com/intersystems-community/webterminal/pull/153#issuecomment-1296170379,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAVHEP5YV6XHEEASWXFZ4ULWFYYKZANCNFSM6AAAAAARRX7QXU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Absolutely!

Bring in docker-compose action
Spin up IRIS container in the GitHub Actions pipeline
Do all necessary install/import things there, actually running the npm run build and then import to IRIS scripts (as for import, please check these commands, to import built files)
Export classes in whatever format you need from IRIS
Then add-commit GitHub action will help you to commit exported files back to this repo, if you need it for ZPM

Thus, you get fully automated build pipeline, and build files commited back to this repo by the GitHub Action.

Can you make it?
On Sun, 30 Oct 2022 at 14:25, Nikita Savchenko @.***>
wrote:

Absolutely!

Bring in docker-compose
https://github.com/KengoTODA/actions-setup-docker-compose action
Spin up IRIS container in the GitHub Actions pipeline
Do all necessary install/import things there, actually running the npm
run build and then import to IRIS scripts (as for import, please check these
commands
https://github.com/intersystems-community/webterminal/blob/master/import.bat,
to import built files)
Export classes in whatever format you need from IRIS
Then add-commit https://github.com/marketplace/actions/add-commit
GitHub action will help you to commit exported files back to this repo, if
you need it for ZPM

Thus, you get fully automated build pipeline, and build files commited
back to this repo by the GitHub Action.
—
Reply to this email directly, view it on GitHub
https://github.com/intersystems-community/webterminal/pull/153#issuecomment-1296272564,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAVHEP2DFK4WMXAQCRKVYRLWFZZMHANCNFSM6AAAAAARRX7QXU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Can you ask @SergeyMi37 or anyone to take a look? I can guide them if it's not clear how to implement it still.
Personally I have a really really limited spare time for it.

If the files end-up in the repo with GitHub action this PR could be
approved then?
On Sun, 30 Oct 2022 at 14:27, Nikita Savchenko @.***>
wrote:

Can you ask @SergeyMi37 https://github.com/SergeyMi37 or anyone to take
a look? I can guide them if it's not clear how to implement it still.
Personally I have a really really limited spare time for it.
—
Reply to this email directly, view it on GitHub
https://github.com/intersystems-community/webterminal/pull/153#issuecomment-1296272938,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAVHEPYLLCBFA567QT3XTDDWFZZUHANCNFSM6AAAAAARRX7QXU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Yes, this is the only right way to implement builds. Otherwise supporting every release will be harder and harder. I wish we had actions back then when I was active on the project :)

Later we can (relatively easily!) even add a testing suite for WebTerminal - like a basic test open browser - exec something - see if it works.
Then everyone who commits will be sure they're at least not breaking things

Projects of this scale should certainly have it. When I was at ISC, I was very closely preparing and testing each release on a few product versions. This, however, can and should be done by the GitHub Actions pipeline

If you urgently need the release I can approve this PR, however I have no clue will it work anywhere etc - along with others who contribute to the project

I agree.
And we run GitHub actions to test other projects. With ZPM it is really
easy:
Zpm “test project-name”
On Sun, 30 Oct 2022 at 14:32, Nikita Savchenko @.***>
wrote:

Projects of this scale should certainly have it. When I was at ISC, I was
very closely preparing and testing each release on a few product versions.
This, however, can and should be done by the GitHub Actions pipeline
—
Reply to this email directly, view it on GitHub
https://github.com/intersystems-community/webterminal/pull/153#issuecomment-1296273859,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAVHEP2JFB7N7SXCEJWDTOTWFZ2FVANCNFSM6AAAAAARRX7QXU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Yes, that’be great. It should give any harm as it just adds files for ZPM
enablement.
On Sun, 30 Oct 2022 at 14:33, Nikita Savchenko @.***>
wrote:

If you urgently need the release I can approve this PR, however I have no
clue will it work anywhere etc - along with others who contribute to the
project
—
Reply to this email directly, view it on GitHub
https://github.com/intersystems-community/webterminal/pull/153#issuecomment-1296274050,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAVHEPZUYAJFA4TMWBEOEMDWFZ2JPANCNFSM6AAAAAARRX7QXU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Merging as-is to publish ZPM release


Batch Size

Hello,
We noticed the input of FamNet is the whole image and batch size equals to 1.
Have you tried random crop and increase the batch size?
Best wishes! Thank you for your answers in advance!

Hey,
We did not use random crops with batchsize > 1 in the paper since the crop might not contain one or more of the exemplars. In order to use batchsize > 1, you'll need to address this case. One easy way of doing so would be to not use ROI pooling for those exemplars which go outside the crop, and computing the Resnet features for such exemplars independently.


[Bugs] Cli issue on init command

Description
Issue 1
When using CLI to generate new project on linux, getting multiple nested folders of the project name.
tsed init bandApp => band-app/band-app/band-app

node_modules are in the correct place in the root folder.
Issue 2
When moving the files to the correct place and running the project i get this error
import { $log } from "@tsed/common";
SyntaxError: Cannot use import statement outside a module

Issue 3
Issue on babel generation. Some dependencies require babel v7 or v6.

:tada: This issue has been resolved in version 2.13.0 :tada:
The release is available on:

v2.13.0
GitHub release

Your semantic-release bot :package::rocket:


[#279] Processor to create (new) PyBossa tasks for new or modified indications

This new processor makes a query for new or modified indications, extracted from FDA approvals. The result is used to create new PyBossa tasks, deduplicate and POST them to the server as new tasks.
The main issue here is with API rate limiting, so there are a few mechanisms to avoid data loss due to 429 status codes from the API. Basically, when rate limit is reached, the processor waits a bit before sending out new payloads.

@nightsh Your code was looking fine to me. However, I started looking deeper into it to create the new PyBossa processor for our deduplication project and got a bit carried away. In hindsight, I should've merged your code and then started working on it, but haven't.
I'm assigning it back to you so you can review my changes. I changed the way we check for the rate limits to use only the status code of the response we get (i.e. not doing another request to check the limits). The biggest change was extracting the PyBossa-related code to a separate class (to be reused by our other processors) and adding the logic to keep the tasks in our database in sync with the tasks in PyBossa, adding/removing/updating them as needed.

@vitorbaptista it's a very good addition that further shapes up the desired modularity for our PyBossa related processors. Following this, I would probably rewrite a bit of the indications processor, to make it use more of the logic in the tasks updater, but this will probably be a good candidate for either the hack day or a later update.
Bottom line, I'm quite happy with it. TBH, I didn't run it locally, but by going through all the code I'd say it's merge material, feel free to do it 👍


[CLI] Update returned exit code not zero

Update returned exit code not zero
Error: Update returned exit code not zero
    at ChildProcess.<anonymous> (/snapshot/sift-cli/sift-cli.js:547:23)
    at ChildProcess.emit (events.js:315:20)
    at maybeClose (internal/child_process.js:1021:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:286:5)

I use ubuntu 20.04 and am facing the above mentioned issue.

Closing this as 20.04 is not yet supported.

Closing this as 20.04 is not yet supported.


java method that checks whether a string is blank

Thanks sir!
Deployed here:
https://onelinerhub.com/java/check_if_string_is_blank


Load Testing

We need a benchmark for our backend services through the REST API.
What we know

Etsy is happy with ~200ms queries on their website
RTT to the connectordb.com is roughly 53ms
Pulling the AskReddit homepage json takes roughly .8 seconds
3G latency is roughly 300ms
Various other latencies

What we want:

10000 users on our service
Each user has 5 devices and ten streams each submitting a point every 30 seconds (16000/sec)
Every REST query goes through the system within 200ms (plus RTT)
20% of our userbase is doing queries at any one time
The length of a data query is 1 month of data (we may want to set a hard limit on this to avoid DOS attacks)

Not currently a priority.


Add ESP32 c3/s2/s3 variants

Why did you add a submodule if Improv serial ?

I didnt even see that and no idea how it got there


Update Omezarr v4

OpenOMEZARRFromS3Command

@constantinpape
This method was renamed in mobie-io, I'll make a PR on mobie-viewer-fiji.

@constantinpape
This method was renamed in mobie-io, I'll make a PR on mobie-viewer-fiji.

Great, let me know when it's there. And please make the PR onto develop.

I checked it out now @KateMoreva and loading an image with a single channel works correctly now and also uses the correct scale information.
However, there is still an issue when loading an image with multiple channels: the channels seem to get different scales in that case. See the example for tczyx below:

The nucleus and cell boundary channel should look like this: (same data loaded in napari)

Could you look into fixing this? Let me know if you need more information to reproduce the issue.

@constantinpape, could you please test this again?

@tischi this is good to go from my side. Would be great to merge this soon and then get out a new MoBIE release so that we can start advertising ome.zarr v0.4.


Fmd18 revert

@AndrewGibson27: Here's the support page roll back. Amanda saw the test link and I made a few text changes accordingly.

I'll take a look at this tomorrow during my on-call shift.

@emilyyount LGTM. I'll be in Monday @ ~9:30a to help roll this stuff back.


docs(groupBy): fix a typo in the example of the instance operator gro…

hi, @benlesh. I'm working in progress on Chinese localization of the RxJS 5 official document.
During this process I find something wrong such as a typo, a problematic example code.
I will report these issues consistently for making RxJS better.
By the way, I'm your huge fan.

Description:
In the second example of groupBy operator, there is a typo(anme) in the example code as belowing:
.groupBy(p => p.id, p => p.anme)
Related issue (if exists):

Coverage remained the same at 97.735% when pulling b2459dcbcb365fc69e29b7e4319699bb73f6c919 on SangKa:fix-groupBy-docs-example into da96b6af6472947b8428b5a341d5c6acef0960df on ReactiveX:master.

LGTM

Thank you @SangKa


0.30.16 version is error

Instead rename magic-string.cjs.js to end in .cjs, change the requiring code to use import(), or remove "type": "module" from<EMAIL_ADDRESS>

Fixed in 0.30.17. Sorry for the inconvenince.


feat: gracefully handle partial failure for multiple accounts

Closes https://github.com/gitify-app/gitify/issues/423
simulated multiple account setup with a partial failure

updated to set the account header row background as red if an account has encountered errors
Example


Fix minor bugs 01

Main Fixes:

A loading gif is added while loading the map markers
The loading gif containers are optimised for mobile viewing.
Menu buttons are optimised so that they can be clicked on from a smaller screen.
Fixed the inivisble map issue on country summary page in small screens.

File Changes:

public/javascripts/countrypagemap.js - the gif loader for the map  marker is added to the getJSON method.
public/stylesheets/aip-styles.css - Removed the hidden style option for maps on small screens.
public/stylesheets/main.css - Optimised the font-size and loading gif container for mobile view.
views/countries/country.html.erb - Added the map marker loader gif container.
views/layouts/landing.erb - Optimised loading gif container for mobile view.
views/layouts/layout.erb - Optimised loading gif container for mobile view.
views/layouts/layout_forms.erb - Optimised loading gif container for mobile view.
views/partials/_countries-tabs.html.erb - Optimised menu buttons for mobile view.
views/partials/_location-tabs.html.erb - Optimised menu buttons for mobile view.
views/partials/_project_list.html.erb - Optimised loading gif container for mobile view.
views/partials/_projects-header.html.erb - Optimised menu buttons for mobile view.
views/partials/_regions-tabs.html.erb - Optimised menu buttons for mobile view.
views/search/search.html.erb - Optimised loading gif container for mobile view.

Note: The font-size optimisation works with modern browsers.

Rumy, this looks good.
Would you make a few small changes before we merge this please?

Don't display the search field in the menu bar.  In aip-styles.css, keep display:none on #global-header #search

Do you need a fallback for older browsers when using vw units? See http://stackoverflow.com/questions/14431411/pure-css-to-make-font-size-responsive-based-on-dynamic-amount-of-characters. Although coverage looks fairly comprehensive. http://caniuse.com/#feat=viewport-units

Remove inline styles in your modal html - move into a css class instead.

Hi John,
Let me answer in sequential order:

Updated.
Well you have already seen that most of the browsers are already updated with this feature support. Still if there's a need for a fall back then probably we should use some js manipulation or media query. But, is it necessary?
Moved the inline codes to main.css file.

Thanks Rumy. I think 2 is as simple as adding a font-size:14px (whatever the default size needs to be) on a line above the font-size: 0.3vw. I think that sets a value older browsers can handle.

Yep. I left it commented out last time for one of the loading container but now I added it for the rest also.

Hi Rumy, I think you've added too many font-size: vw elements. It is making things like the core navigation, headers and other elements much too small on small screens. Can you remove all vw sizing except where you need it (is that just on the map overlay?)


app icon when installing from deb is the electron icon

same icon as in the screenshot in #18

I see... this issue will be the case for the RPM as well.

This is actually something I thought it will work fine with it's default options, but I guess I need to add the icon key to the config for both DEB and RPM makers.

Fixed in commit: f1717ad4


Add ability to initialize InferenceSession with a model that is already loaded.

Support use cases where the Model instance is already loaded.
e.g. https://github.com/Microsoft/onnxruntime/pull/168 constant_folding.cc dynamically creates a model but is forced to serialize it to use the current API.
Add generic loading method to InferenceSession::Impl and convert existing methods to use that so load logic isn't cut-and-pasted in 4 places.

Is this function only for testing purposes?

@snnn it can be used for some graph transformers, I think.

There's a link in the description to the PR for constant folding that will use this.

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


Reimplement all in Rust

Described in #92.

This changes are far from releasing so I make it draft.

We have the syntax of statements now.
After this language obtain a way to generate some functions, I will start to review this PR.

Now, we can create some functions with kaprino.
But its syntax is far from useful because there are only one-line functions but not multiple-line functions.

Ops. I haven't noticed that the checks was failed.
And also I already figure out why.


Redeploying Subconscious wiped sphere profile

After a week out of date (and past provision expiry) I built this morning and redeployed; my sphere had nickname @j2 and a one-line description (95% sure) previously, but since starting with new build, my photo, description are empty and nickname "unknown".
From the logs, this looks suspicious:
2023-05-18 14:09:04.918015-0700 Subconscious[55521:2754491] [app] [action] succeedFetchNicknameFromProfile(unknown)
2023-05-18 14:09:04.918552-0700 Subconscious[55521:2754491] [app] Nickname saved: unknown

TL;DR: data migration issue, shouldn't affect anyone else going forward.

I think I worked out what this is. It's obscure.
Before profiles, we used to store the nickname in AppDefaults (the Apple solution for persistent data like settings). After adding profiles, we had both ways of setting a nickname side by side. My recent PR #639  removed the AppDefaults side of things and relies solely on the profile.
Importantly, that PR ensures the nickname from onboarding is actually persisted in the profile.
For the situation @jsantell encountered above, my hypothesis is that the app had a nickname in AppDefaults but not the user profile and then when deploying the latest version of the app we lost whatever the old value was.

Closing this for now since I believe a one-off, if it re-appear we'll re-open.


Partial updates for lodash to 3.10.0

Updates for lodash according to docs and usage:

_.take no longer accepts predicates
_.takeWhile added to accept predicates
_.filter can have predicate optional
_.union is chainable
_.sortByAll can accept multiple predicates

Hi,
Can you add _.defaultsDeep aswell?
Should not be more than (and some tests of curse):
//_.defaultsDeep
interface LoDashStatic {
    /**
    * This method is like _.defaults except that it recursively assigns default properties.
    * @param object The destination object.
    * @param sources The source objects.
    * @return The destination object.
    **/
    defaultsDeep<T, TResult>(
        object: T,
        ...sources: any[]): TResult;
}

interface LoDashObjectWrapper<T> {
    /**
    * @see _.defaultsDeep
    **/
    defaultsDeep<T, TResult>(...sources: any[]): LoDashObjectWrapper<TResult>
}

I can create a pull request to your branch if you want.

Never mind, it exist already a pull request for default deep: #5072

lodash/lodash.d.ts
to author(@bczengel). could you review this PR?
:+1: or :-1:?
check list

[ ] pass the Travic-CI test?

:+1:

@bczengel thanks!
@mdekrey thanks mate!

@vvakame Glad to be of service!


Extended parameter attributes for handlers & generic [Entity] attribute

The HTTP support has this:
/// <summary>
///     Base class for attributes that configure how an HTTP endpoint on a parameter is handled by applying
///     middleware or error handling rules
/// </summary>
[AttributeUsage(AttributeTargets.Parameter)]
public abstract class HttpChainParameterAttribute : Attribute
{
    /// <summary>
    ///     Called by Wolverine during bootstrapping to modify the code generation
    /// for an HTTP endpoint with the decorated parameter
    /// </summary>
    /// <param name="chain"></param>
    /// <param name="parameter"></param>
    /// <param name="container"></param>
    public abstract Variable Modify(HttpChain chain, ParameterInfo parameter,
        IServiceContainer container);
}

Which is used for the [Document] and [Aggregate] attributes in the Marten integration. Good stuff IMO. But there's nothing for that in the message handlers, and nothing like that for the EF Core or RavenDb support.
What if we had something more generic Wolverine core for message handlers that could also be helpful to HttpChains as well? And extend the IPersistenceFrameProvider abstraction to help generate code for a more generic [Entity] attribute?
There's another wrinkle. The existing [Document] attribute for Marten HTTP integration has the concept of being Required, and will return a 404 if not found. Maybe the HandlerChain / HttpChain model has some concept of "early exit" so that the same basic code can be used in both places, but let the chain alter based on whether or not it's HTTP or a message chain.

Okay, this might end up being pretty large, but let's go for it! This might end up being sold as the "Wolverine is awesome at VSA with persistence" story.
ChainParameterAttribute

[ ] Like HttpChainParameterAttribute, but generalized for IChain too
[ ] HandlerChain picks it up on handler methods
[ ] HandlerChain picks it up on Before methods
[ ] HttpChain picks it up on handler methods
[ ] HttpChain picks it up on Before methods

Data Loading Attributes
Need to be able to "find" the value for a requested key to make any of this work. At the same time, I'd like to see Wolverine support:

A member on the input type, be that a message type or a request type for HTTP
Default to "[TypeName]Id" or gulp, a strong typed value of type "[TypeName]Id" if nothing else is specified. Fall back to "Id"
Specify a route value
Specify from the query string (ick, but people love doing that)
Specify a header value -- can be HTTP request or Envelope here

Also need to specify if it's required or not

[ ] New attribute base class or interface for finding a value from the message or request.
[ ] IChain.FindKeyVariable(IVariableQuery) : Variable?
[ ] IChain.AssertValueIsNotNullInCode(Type variableType)
[ ] IChain.AssertValueIsNotNullInCode(Variable variable)
[ ] Implement methods above on HandlerChain, throw not supported on QueryString or RouteValue
[ ] Implement methods above on HttpChain
[ ] Test Required & not required when null
[ ] [Edit] attribute loads and applies transaction for HttpChain
[ ] [Edit] attribute loads and applies transaction for HandlerChain
[ ] [Read] attribute with HandlerChain
[ ] [Read] attribute with HttpChain
[ ] [Read] on Before() with HandlerChain
[ ] [Read] on Before() with HttpChain

IPersistenceFrameProvider

[ ] Ensure Marten transactional middleware is idempotent from a codegen perspective
[ ] Ensure EF Core transactional middleware is idempotent from a codegen perspective
[ ] Ensure RavenDb transactional middleware is idempotent from a codegen perspective
[ ] Ability to generate code for the Persist option down below where it could be nothing, update, delete

ISideEffectAware
public interface ISideEffectAware : ISideEffect
{
    static abstract void ConfigureChain(IChain chain);
}

[ ] New ISideEffectAware interface
[ ] Modify SideEffectPolicy to be aware of ISideEffectAware and branch
[ ] Store<T>, also test nulls
[ ] Update<T>, also test nulls
[ ] Insert<T>, also test nulls
[ ] Delete<T>, also test nulls
[ ] Persist<T>, has options to delete, insert, update, store, nothing
[ ] Test Store with EF Core
[ ] Test Insert with EF Core
[ ] Test Update with EF Core
[ ] Test Delete with EF Core
[ ] Test Persist w/ EF Core
[ ] Test Store with Marten
[ ] Test Insert with Marten
[ ] Test Update with Marten
[ ] Test Delete with Marten
[ ] Test Persist w/ Marten
[ ] Test Store with RavenDb
[ ] Test Insert with RavenDb
[ ] Test Update with RavenDb
[ ] Test Delete with RavenDb
[ ] Test Persist w/ RavenDb


aioblescan can't find HCI device

Hello,
since i've updated my Ubuntu from 21.10 to 22.04, aiovlescan is not functional anymore.
The error i'm getting is
#aioblescan -m 50:FB:19:4E:77:7C Traceback (most recent call last):   File "/usr/local/bin/aioblescan", line 33, in <module>     sys.exit(load_entry_point('aioblescan==0.2.13', 'console_scripts', 'aioblescan')())   File "/usr/local/lib/python3.10/dist-packages/aioblescan-0.2.13-py3.10.egg/aioblescan/__main__.py", line 176, in main     mysocket = aiobs.create_bt_socket(opts.device)   File "/usr/local/lib/python3.10/dist-packages/aioblescan-0.2.13-py3.10.egg/aioblescan/aioblescan.py", line 1760, in create_bt_socket     raise exceptions[0] OSError: [Errno 19] error while attempting to bind on interface 0: No such device
The HCI Device is functional:
#hcitool dev Devices: 	hci1	E8:48:B8:C8:20:00
and for example hcitool lescan is working properly:
#sudo hcitool lescan LE Scan ... 18:93:7F:F2:AD:42 (unknown) 18:93:7F:F2:AD:42 Somfy Outdoor 0076B10826E4 00:7C:2D:C3:E3:9F (unknown) 61:0C:47:C5:90:58 (unknown) 54:D2:72:79:44:0C (unknown) 61:0C:47:C5:90:58 nut 50:FB:19:4E:77:7C MIBFS 61:0C:47:C5:90:58 (unknown) 50:FB:19:4E:77:7C (unknown) 50:FB:19:4E:77:7C MIBFS F0:50:63:D4:E4:11 (unknown) F0:50:63:D4:E4:11 Mi Smart Band 5 32:24:A9:6C:2A:72 (unknown) 32:24:A9:6C:2A:72 (unknown)
#dmesg | grep Blue  [    8.412679] Bluetooth: Core ver 2.22 [    8.412714] Bluetooth: HCI device and connection manager initialized [    8.412720] Bluetooth: HCI socket layer initialized [    8.412724] Bluetooth: L2CAP socket layer initialized [    8.412730] Bluetooth: SCO socket layer initialized [    8.530612] Bluetooth: hci0: RTL: examining hci_ver=0a hci_rev=000b lmp_ver=0a lmp_subver=8761 [    8.531562] Bluetooth: hci0: RTL: rom_version status=0 version=1 [    8.531567] Bluetooth: hci0: RTL: loading rtl_bt/rtl8761bu_fw.bin [    8.533610] Bluetooth: hci0: RTL: loading rtl_bt/rtl8761bu_config.bin [    8.533804] Bluetooth: hci0: RTL: cfg_sz 6, total sz 27814 [    8.675581] Bluetooth: hci0: RTL: fw version 0x09a98a6b [   11.074229] Bluetooth: BNEP (Ethernet Emulation) ver 1.3 [   11.074235] Bluetooth: BNEP filters: protocol multicast [   11.074242] Bluetooth: BNEP socket layer initialized [   12.795945] Bluetooth: RFCOMM TTY layer initialized [   12.795955] Bluetooth: RFCOMM socket layer initialized [   12.795964] Bluetooth: RFCOMM ver 1.11 [50015.919166] Bluetooth: hci1: RTL: examining hci_ver=0a hci_rev=000b lmp_ver=0a lmp_subver=8761 [50015.920153] Bluetooth: hci1: RTL: rom_version status=0 version=1 [50015.920161] Bluetooth: hci1: RTL: loading rtl_bt/rtl8761bu_fw.bin [50015.920309] Bluetooth: hci1: RTL: loading rtl_bt/rtl8761bu_config.bin [50015.920382] Bluetooth: hci1: RTL: cfg_sz 6, total sz 27814 [50016.059153] Bluetooth: hci1: RTL: fw version 0x09a98a6b
How can i solve the problem?

You really should read the error message.
It shows  "interface 0: No such device"

And when you show hcitool works  it shows
hcitool dev Devices: hci1 E8:48:B8:C8:20:00
0 == hci0 vs hci1
If you had used
aioblescan -h
You would have seen
-D DEVICE, --device DEVICE
Select the hciX device to use (default 0, i.e. hci0).
Try
-D 1

you are absolutely right.
unfortunately  if i strat aioblescan -D 1, i don't get any messages.

It works for me on Kubuntu 22.04.  Make sure you run as root to avoid permission issues
sudo aioblescan -D 1 ....

oh, got it now, it works with root.


Makefile: Use tr in a POSIX-conform way

I stumbled upon this issue when trying to build on Alpine Linux, where busybox provides tools like 'tr'.
Busybox' tr doesn't interpret the :digit: character class, and instead literally removes the characters :digit from the input, resulting in the OS string "lnux"
0-9 should be a compatible and more standard-conform way.

LGTM