'ascii' codec can't decode byte 0x9c in position 1: ordinal not in range(128) when snipe

Expected Behavior
run snipe
Actual Behavior
stop when run move_to_map_pokemon.py and get error:
Traceback (most recent call last):
  File "pokecli.py", line 503, in <module>
    main()
  File "pokecli.py", line 89, in main
    bot.tick()
  File "/Users/mrwbsn/Workspaces/pogobot/pokemongo_bot/__init__.py", line 434, in tick
    if worker.work() == WorkerResult.RUNNING:
  File "/Users/mrwbsn/Workspaces/pogobot/pokemongo_bot/cell_workers/move_to_map_pokemon.py", line 255, in work
    return self.snipe(pokemon)
  File "/Users/mrwbsn/Workspaces/pogobot/pokemongo_bot/cell_workers/move_to_map_pokemon.py", line 214, in snipe
    self._teleport_back()
  File "/Users/mrwbsn/Workspaces/pogobot/pokemongo_bot/cell_workers/move_to_map_pokemon.py", line 339, in _teleport_back
    data={'last_lat': last_position[0], 'last_lon': last_position[1]}
  File "/Users/mrwbsn/Workspaces/pogobot/pokemongo_bot/base_task.py", line 27, in emit_event
    data=data
  File "/Users/mrwbsn/Workspaces/pogobot/pokemongo_bot/event_manager.py", line 61, in emit
    formatted_msg = formatted.format(**data)
KeyError: 'last_long'
2016-08-08 12:11:08,403 [sentry.errors] [ERROR] Sentry responded with an error: 'ascii' codec can't decode byte 0x9c in position 1: ordinal not in range(128) (url: https://app.getsentry.com/api/90254/store/)
Traceback (most recent call last):
  File "/Users/mrwbsn/Workspaces/pogobot/lib/python2.7/site-packages/raven/transport/threaded.py", line 174, in send_sync
    super(ThreadedHTTPTransport, self).send(data, headers)
  File "/Users/mrwbsn/Workspaces/pogobot/lib/python2.7/site-packages/raven/transport/http.py", line 47, in send
    ca_certs=self.ca_certs,
  File "/Users/mrwbsn/Workspaces/pogobot/lib/python2.7/site-packages/raven/utils/http.py", line 66, in urlopen
    return opener.open(url, data, timeout)
  File "/Users/mrwbsn/Workspaces/pogobot/lib/python2.7/site-packages/future/backports/urllib/request.py", line 494, in open
    response = self._open(req, data)
  File "/Users/mrwbsn/Workspaces/pogobot/lib/python2.7/site-packages/future/backports/urllib/request.py", line 512, in _open
    '_open', req)
  File "/Users/mrwbsn/Workspaces/pogobot/lib/python2.7/site-packages/future/backports/urllib/request.py", line 466, in _call_chain
    result = func(*args)
  File "/Users/mrwbsn/Workspaces/pogobot/lib/python2.7/site-packages/raven/utils/http.py", line 46, in https_open
    return self.do_open(ValidHTTPSConnection, req)
  File "/Users/mrwbsn/Workspaces/pogobot/lib/python2.7/site-packages/future/backports/urllib/request.py", line 1284, in do_open
    h.request(req.get_method(), req.selector, req.data, headers)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 1053, in request
    self._send_request(method, url, body, headers)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 1093, in _send_request
    self.endheaders(body)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 1049, in endheaders
    self._send_output(message_body)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 891, in _send_output
    msg += message_body
UnicodeDecodeError: 'ascii' codec can't decode byte 0x9c in position 1: ordinal not in range(128)
2016-08-08 12:11:08,404 [sentry.errors.uncaught] [ERROR] [u"KeyError: 'last_long'", u'  File "pokecli.py", line 503, in <module>', u'  File "pokecli.py", line 89, in main', u'  File "pokemongo_bot/__init__.py", line 434, in tick', u'  File "pokemongo_bot/cell_workers/move_to_map_pokemon.py", line 255, in work', u'  File "pokemongo_bot/cell_workers/move_to_map_pokemon.py", line 214, in snipe', u'  File "pokemongo_bot/cell_workers/move_to_map_pokemon.py", line 339, in _teleport_back', u'  File "pokemongo_bot/base_task.py", line 27, in emit_event', u'  File "pokemongo_bot/event_manager.py", line 61, in emit']

Steps to Reproduce
run with MoveToMapPokemon Tasks
Other Information
OS: OSX
Git Commit: 563f898f61233ff86d38d83b68d8399c1f63051b
Python Version: Python 2.7.10

theres a problem with the Snipe" type in the configs . i hope someone can fix

I had this error too.
reload(sys) sys.setdefaultencoding('utf-8')
When I put this code under import sys in file 'pokecli.py', the error fixed.
OS:windows 7
python: 2.7

@Samtoto I have add those code below import sys but still no luck

There's a typo in this file: "/Users/mrwbsn/Workspaces/pogobot/pokemongo_bot/cell_workers/move_to_map_pokemon.py". On line 338, in _teleport_back, the 'last_long' keyword argument should be 'last_lon'.
formatted=('Teleporting back to previous location ({last_lat}, '
-                       '{last_long})'),
+                       '{last_lon})'),

@shoelsch My move_to_map_pokemon.py file don't have 338 lines? Are you sure it's this file you've made the changes to?

@mike9011 I'm using latest commit on the dev branch: https://github.com/PokemonGoF/PokemonGo-Bot/commit/bdcf2519d63555ac32246439fd17672896ee64e4#diff-13dfb7470ca1360a74ad4eabf7bb11a2R338

I already submitted a PR (#3003) just wait it get merged.

@raulgbcr nice! now it works. thank you everyone! can't wait to see this fix is merged.


can't work well

when input a tag ,after input enter, not autocomplete.

I agree the auto-complete is not intuitive where the closing tag is not added when the opening tag is manually closed.
<SomeTag> --> tab / enter --> no closing tag
However, the extension appears to expect the opening tag to not be competed:
<SomeTag --> tab --> opening tag completed and closing tag added.

I understand what you mean, but this is a snippet extension and I can not include this feature.
I will attempt to develop this extension as a complete html auto complete extension when I have the chance.

thanks everyone. I already know how to use the and before the cause of the failure, for example, input a tag 'html' after, input enter can complete  ,but before I input the   after input enter. sorry ,my English is very poor.

Hello,
Sorry I did not understand what you mean.
Can you provide a example as to what you mean.

I mean, I had to use it is wrong, and now know how to properly use.


first work

creating the pull request, still a work in progress
solves #22

The PR was not yet ready ! Sorry for the confusion. I wrote "still a work in progress"

@GTimothee Just a heads up - I've done refactoring. This function now resides in the feature_engineering submodule. Please do a Git Pull to update to the latest master.


arm: cortex_m: prep_c: Fix compile warning with relocate_vector_table

Its possible that memcpy is marked to check that a NULL value is passed
to it.  The instance of memcpy in relocate_vector_table its valid that
we're are copying to 0/NULL.  So supress the possible warning generated
by gcc -Wnonnull.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

We see build error's w/sanity check and sdk-0.10.0 (since newer newlib enables memcpy NULL check) on nucleo_f030r8.  This fixes the issue.

Codecov Report

Merging #13601 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #13601   +/-   ##
=======================================
  Coverage   52.45%   52.45%
=======================================
  Files         322      322
  Lines       46592    46592
  Branches    10768    10768
=======================================
  Hits        24442    24442
  Misses      17247    17247
  Partials     4903     4903

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 80c9a6a...7b802eb. Read the comment docs.


Requested node configuration is not available

I am trying to submit a new job to the slurm scheduler. There are seven idle nodes (from jobs that just finished up).  I am requesting 10 nodes for the new job.  I get this error message:
sbatch: error: Batch job submission failed: Requested node configuration is not available
If I wait until all 7 idle nodes are removed from the scheduler and then submit the 10 node job, the scheduler will accept it and then slowly start launching the nodes to meet the request.
Here's my sbatch script:
#!/bin/bash
#SBATCH --job-name=ebyycncre
#SBATCH --partition=compute
#SBATCH --account=centos
#SBATCH --threads-per-core=1
#SBATCH --sockets-per-node=2
#SBATCH --cores-per-socket=18
#SBATCH --nodes=10
#SBATCH --ntasks=360
#SBATCH --ntasks-per-node=36
#SBATCH --mail-type=ALL
#SBATCH<EMAIL_ADDRESS>#SBATCH --output=./s%j
#SBATCH --no-requeue
mpirun -np 360 fireFoam -parallel
And the output from sinfo:
PARTITION AVAIL  TIMELIMIT  NODES  STATE NODELIST
compute*     up   infinite      7   idle ip-10-15-20-[11,34,104,154,195,214-215]

This has been fixed since ParallelCluster 2.3.1.


node split: proposal building

Motivation
Adds proposal building to the node split poc.
Description
Adds support for proposal building to the node split.
The proposal is partially built on the node, then handed over to the smesher to generate the missing fields and sign+publish it.
The setup can be brought up with the provided ./start.sh file that makes it easier to start the setup.
Test Plan

[x] manual testing with docker-compose setup

TODO

[ ] Explain motivation or link existing issue(s)
[ ] Test changes and document test plan
[ ] Update documentation as needed
[ ] Update changelog as needed

bors try

bors try

bors try

bors try

bors try

bors try

bors try

bors try

bors try

bors cancel

bors try

bors try

bors try

bors try-

bors try

bors try


Addition of screen transitions.

Right now, There is no screen transition animation when an intent is called. We can add one of the following transit between screens:-

Fade-In , Fade-Out
Slide-Left, Slide-Right
No new external decencies will be required for this.
If this is considered enhancement, I would like to work on it.
Thanks

@opticod ^

We need to first see, which animation looks better in which case.
So can you list the animations you would like to add with which fragment/activity transitions?

@octacode are you working on this enhancement?

@sm86 yup!

@opticod
I have tried the slidein-left and slide out right transitions and for what I have seen is that they aren't really smooth. So I am thinking of using fade-in and fade out cause they are not just smooth but they fit with the very flow of the app.

@opticod
@croozeus
As I have noticed in apps like Whatsapp, there's a common screen transition for all the intent changes.
I too want to do the same to the Toolkit. I have tried the default android transitions provided and I find fade_in and fade_out really consistent both in terms of performance and visuality. So I am making the PR for the same.
Any Suggestions?

Thanks @octacode, our team will review this and merge it shortly.

Closed via #398


RefreshToken Expiration causing Undesired User Logout

Issue / Steps to reproduce the problem
Currently we have our IdentityServer client set to use defaults for
AbsoluteRefreshTokenLifetime & SlidingRefreshTokenLifetime. This is causing a lot of our users to re-enter their credentials which we want to increase.
We are going to increase these times to better support our desired functionality.
However, for PersistedGrants that have already been created, is it possible to increase their Expiration by modifying that column in the PersistedGrants table? Or would we also need to modify the data column to truly extend the expiration of a given PersistedGrant?
Relevant parts of the log file
<no relevant logs>

You'd have to change the data in the json itself.


Navigation Pane Hide/Show Keyboard Shortcut

What's the Problem?
No response
Solution/Idea
Would be useful to have the ability to quickly hide and show the sidebar/navigation pane without using the mouse using a keyboard shortcut.
Alternatives
Just a simple keyboard shortcut such as Ctrl+Alt+S like in macOS Finder!
Priorities
No response
Files Version
No response
Windows Version
No response
Comments
No response

This issue should be resolved with #8157


Support xdist plugin to run test in parallel

Currently pytest-testrail plugin creates Test-Run for each xdist workers and then publishes test cases like Untested
We could modify pytest_collection_modifyitems() to postpone call of self.create_test_run()
and run it in pytest_sessionfinish() if self.results

Hi, I'm having the same issue.
pytest-testrail plugin creates Test-Run for each xdist workers.
I see the corresponding pull request is closed but this issue is still open.
Could you please tell me when this will be implemented, or if you have a workaround I could use?
Thank you!

Hi @astroVale pls see https://github.com/allankp/pytest-testrail/pull/90#issuecomment-469741084

Thanks for your reply!
I was trying to understand that comment before writing here.
From your comment:
Step 1. My base conftest.py is modified in order not to add the testcase id to each single test function, but to take it from a list:
`with open("all_tests.json") as json_file:
data = json.load(json_file)
tests = [[test["testcase_id"], test["testcase_name"]] for test in data]
def pytest_collection_modifyitems(session, config, items):
for item in items:
for testid, testname in tests:
if item.name == testname:
print(testid)
item.add_marker(pytestrail.case(testid))`
So the suggested code is not working.
Step 2. Writing that class to a new file, from where is that class called in the end?
Step 3, 4 and 5. Isn't the pytest_testrail plugin supposed to run those steps automatically already?

@YakovLittle, I see the fix you did in #90 was not merged to master branch, that's probably why for me pytest-testrail plugin still creates a test run for each xdist worker.

Hi, any updates on this? I'm getting the same issue.

@YakovLittle could you help update this issue?

@PeterNgTr xdist-plugin is using separate processes (workers) and also merge test reports by rsync locally that means we couldn't use pytest-testrail plugin for import test report inside each process.
I suggest to run the import into TestRail by separate step in your CI.

@YakovLittle thanks for your reply. Could you maybe expose more details step how to do this? I'm sorry, I'm just getting to use this.

@PeterNgTr pls see #90 (comment)

https://github.com/pytest-dev/pytest-html/blob/master/pytest_html/plugin.py
pytest_html can make it work with xdist. Could we do that? :)

yep, sure - we're using pytest-html too

@YakovLittle thanks!
can anyone help providing me an example code base how to make the testrail publishment with xdist? I'm in the middle of nowhere. Not sure what to do. Many thanks

@PeterNgTr I've pushed source code example into https://github.com/allankp/pytest-testrail/pull/90#issuecomment-469741084

@YakovLittle thanks, I tried to read it but sorry I'm noobie. Could you help format it to make it easier to read.
Step 2. Writing that class to a new file, from where is that class called in the end?
Step 3, 4 and 5. Isn't the pytest_testrail plugin supposed to run those steps automatically already?

@allankilpatrick would you mind considering fixing this in the core?

@astroVale I would manage to publish the test results to testrail when running tests with xdist without creating multiple test runs. Thanks @YakovLittle! I followed your path to parse the test results from junitxml report. I'm happy to help if you need me!

Appears this issue has been resolved in another comment. https://github.com/allankp/pytest-testrail/pull/90#issuecomment-469741084


I cannot get GHC and HLint to agree..

Versions:
$ ghc --version
The Glorious Glasgow Haskell Compilation System, version 8.6.3
$ hlint --version
HLint v2.1.13, (C) Neil Mitchell 2006-2019

For this module:
{-# LANGUAGE ImplicitParams       #-}
{-# LANGUAGE TypeSynonymInstances #-}
{-# LANGUAGE FlexibleInstances    #-}

module Test where

data X a = X
type XI = X Int

class Foo a where
  foo :: (?p :: Int) => a -> Int

instance Foo XI where foo = undefined

I don't seem to be able to make both GHC and HLint happy at the same time:
$ hlint Test.hs
Test.hs:2:1: Warning: Unused LANGUAGE pragma
Found:
  {-# LANGUAGE TypeSynonymInstances #-}
Perhaps you should remove it.
Note: Extension TypeSynonymInstances is implied by FlexibleInstances

Test.hs:3:1: Warning: Unused LANGUAGE pragma
Found:
  {-# LANGUAGE FlexibleInstances #-}
Perhaps you should remove it.
Note: Extension FlexibleInstances is implied by ImplicitParams

2 hints

Of the 3 LANGUAGE extensions in the module, I tried all 8 possible combinations. (Having none to having all of them) and there isn't a single instance where both GHC accept the program and HLint not complain about it.
In fact, it seems that GHC requires all three pragmas to be present to compile it; and HLint isn't happy about that.
Is there some other trick I'm missing to make both happy? Or is this an hlint oversight?

Just for the sake of future documentation, I think https://gitlab.haskell.org/ghc/ghc/blob/e8004e5d1e993363a89b0107380604c5bc02be6b/compiler/main/DynFlags.hs#L4668 is a better source for implied flags.

Ah, thanks for that link @moll. I've added docs to the code. Unfortunately the GHC notion of extension doesn't quite match that of Haskell extensions, but it's certainly the ground truth.
Sadly that function isn't exposed in GHC, or else we could use it directly. @shayne-fletcher-da - do you think it's worth patching GHC to expose it?

@ndmitchell Yes. There's a couple of things we can do:

Get it patched upstream so we can use it from ghc-lib-parser;
We can copy the implementation to ghc-lib-parser-ex and use from there it in hlint today;
In the fullness of time, when it's generally available in ghc-lib-parser, the ghc-lib-parser-ex reimplementation can be removed and redirect.

@shayne-fletcher that sounds a great plan

MR https://gitlab.haskell.org/ghc/ghc/merge_requests/2654 sent upstream;
Issue for ghc-lib-parser-ex : https://github.com/shayne-fletcher/ghc-lib-parser-ex/issues/19.

Update : MR approved; milestone set for 8.12.1.


Reading: Add STAR reading to comprehension, with box and line chart (v2)

This is replacing https://github.com/studentinsights/studentinsights/pull/2778
Who is this PR for?
K5 reading folks
What problem does this PR fix?
STAR wasn't included in the reader profile.
What does this PR do?
Adds in a tab, and a view for STAR reading, refactoring a few pieces along the way.  Highlighting uses the same heuristic folks decided on in the class lists (<30th percentile), and the code abstracts that out.
Data is provided by updating reader_profile.rb to re-use some of the StudentProfileChartData class for querying and shape of the data.  In the UI, parts <BoxChart /> is factored out into pure structural UI elements so it can be re-used here.  And the highlighting heuristic is factored out of the class list feature.  No materials are added in this PR.
This PR builds on what was in https://github.com/studentinsights/studentinsights/pull/2778, but also revises a bit of UI code around the BenchmarkBoxChart, BenchmarkCohortChart and the underlying UI elements and styling.  This was driven by some small style changes (eg, decoloring text on cohort charts) that revealed coupling that we don't want anymore now that these aren't UI and design elements aren't just about benchmark reading points.
Screenshot (if adding a client-side feature)

Checklists
Which features or pages does this PR touch?

[x] Reader profile

Does this PR use tests to help verify we can deploy these changes quickly and confidently?

[x] Included specs for changes
[x] Improved specs for existing code in need of better test coverage
[x] Manual testing made more sense here

hrm, blocked on Travis issues at the moment

selfie


no review form button should be displayed

It will display a friendly message when there is no review form available.


Bug: No sound when the computer wakes from sleep (Catalina)

Disclaimer:
Please go through existing issues to avoid creating duplicates. If you found a similar issue please post a comment there. If you are uncertain your issue is related to others then create the issue. Obvious ignorance of this rule might get you blocked from posting new issues. Please tick the box below to show that you have followed the guideline.

[x] I have checked for a similar issue and sure it hasn't been reported before.

Describe the bug
No sound is heard when the computer wakes from sleep. Sometimes the eqMac crashes.
Steps to Reproduce
Wake the computer from sleep and play any sound or music, e.g. from youtube. There is nothing to hear. Need to switch the audio output to another and then return to the original. After is sound back if eqMac will not crash.
Additional information
Tested on MacPro 2013, 64GB RAM, internal audio output, DualHead display via displayport, Catalina 10.15.5, eqMac version 0.2.0

This issue should be resolved in v0.3.1.
Please update the app and the driver when launching and let me know if this issue is still relevant. Thank you


[dev-tool] - samples publish command no longer honors --force flag

Just something I noticed in passing. See output below:
core-tracing tracing-samples % npx dev-tool samples publish samples-dev --force
[publish] Creating camera-ready samples for<EMAIL_ADDRESS>[publish] Created camera-ready samples.
core-tracing tracing-samples % gst
On branch tracing-samples
Your branch is up to date with 'origin/tracing-samples'.

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git restore <file>..." to discard changes in working directory)
        modified:   samples-dev/basicTracing.ts
        modified:   samples/v1/javascript/basicTracing.js
        modified:   samples/v1/typescript/src/basicTracing.ts

no changes added to commit (use "git add" and/or "git commit -a")
core-tracing tracing-samples % npx dev-tool samples publish samples-dev --force
[publish] Creating camera-ready samples for<EMAIL_ADDRESS>[publish] the directory /home/mleger/workspace/azure-sdk-for-js/sdk/core/core-tracing/samples/v1 exists and is dirty (according to `git`); commit or stash your changes first
[dev-tool] Errors occured. See the output above.
core-tracing tracing-samples %

It's a little impactful because it makes it harder to iterate on a sample in a dev-loop

Yeah, I will add this flag back. Originally the force argument was only about allowing the command  to overwrite any files at all, but now it needs to override the git test.

👋 @witemple-msft I'm cleaning up old issues, mind moving this to the appropriate milestone when you get a chance?


(Question) winit support

Does this program use winit? because I get a wayland-backend crate error in a system that don't support Wayland yet.
winit or some GUI toolkit would fix it.

It shouldn't have any Wayland specific code (being a console based tui app and not a gui). It's possible that there's a system specific dependency in the graph.
Please provide more details on the build failure and a copy of the output of cargo tree. Thanks

@joshka Yes, sorry for my mistake, it uses Ratatui for the TUI interface.
This is the dependency using Wayland:
── copypasta v0.10.1
│   ├── smithay-clipboard v0.7.0
│   │   ├── libc v0.2.153
│   │   ├── smithay-client-toolkit v0.18.1


What is the purpose of case .viabilityChanged

I have received this from the websocket, but I have no idea what is this for? Why does this happen?

+1 for this question. the only evidence I could found now is that when you are connected to socket, and you turn of the internet connection, it'll trigger viabilityChanged event

Found a little comment in Transport.swift, I think this could explain the usage of this enum

Ref: https://github.com/daltoniam/Starscream/blob/5fbeab4c123753eb5207d11631e1fc8930927b4b/Sources/Transport/Transport.swift#L33

It is forwarded from this update handler in Apple's "Network" framework:
https://developer.apple.com/documentation/network/nwconnection/2998578-viabilityupdatehandler
From the docs:
    /// Set a block to be called when the connection's viability changes, which may be called
    /// multiple times until the connection is cancelled.
    ///
    /// Connections that are not currently viable do not have a route, and packets will not be
    /// sent or received. There is a possibility that the connection will become viable
    /// again when network connectivity changes.
    final public var viabilityUpdateHandler: ((Bool) -> Void)?

false indicates that it can't send/receive packets, true indicates that it can now send/receive packets. You can do whatever you want with that information! You could attempt to reconnect, show an offline warning to the user, etc.


Use data to measure how well we are meeting our user needs

What
Review and refine the performance framework (map this to the OKRs, objective and key results) to make it more actionable and trackable.
Measuring email sign up journey. (1 specific journey that will probably be in the performance framework)
Use GA / GTM as relevant to the performance framework.
Why
Our existing performance framework is too complicated and covers everything. Now that we have implemented GA, we can  focus on specific user journeys to measure first.
Who needs to know about this
PA, PM, CD, UR
Done when

[ ] Thing to do

Should we document how we measure the email signup journey?


[Breaking Bug] Crash when change country after entering the phone number

The application crashes when I enter a phone number, then try to change the country (by clicking on the dropdown country picker).
The code:

The error:

I get away with that problem by just giving a single country
countries: ['MK']

I have downgraded to 0.5.2 version and error is gone.


Updates to release notes

Quite a few updates, some of them were mainly styling ones (please tell me to either revert them or update if they do not match the style guide!).
Clarified test steps and improved release scripts so that they are easier to read.
I have also removed a note about tags and CircleCi pushing a release since that's already done for us by a update-oss-version.
Another update is that we now mention steps to check that are printed by test-manual-e2e.sh as I have noticed they were recently updated, so better to just clarify they are printed out.

That is very nice, thanks for doing this

Thanks @bestander
@mkonicek I've also added a note about making a release note, esp. to double check commit messages and reword them when appropriate.

Looks great, thanks a lot @grabbou!
@facebook-github-bot shipit


What does packagesToInclude do?

Just wondering. Is it necessary? should it specify the package of the library for example: package com.demo.lib;
packagesToInclude = ['com.demo.lib']

If you have a dependency that has transitive dependencies, by default, they will not be added to the final AAR. If you want to add all transitive dependencies you can do it with includeAllInnerDependencies propertie, but if you only want to add one in particular you can specify its package in that array.
I don't know if I explain myself clearly?

cool. So would you specify the full package name eg com.demo.lib ?

Yes.


How to solve the problem that all avatars and icons in the control panel are not displayed

How to solve the problem that all avatars and icons in the control panel are not displayed

@Lotusrootsoup see docs, Intranet deployment option: https://casdoor.org/docs/deployment/deploy-intranet and CDN deployment option: https://casdoor.org/docs/deployment/deploy-cdn


Update ssis-azure-deploy-run-monitor-tutorial.md

Modified some of the Powershell code to actually work better.

@dbaduck : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@swinarko - Hi, would you review the proposed update to your article?
Publishing in this repository is suspended until Tuesday, September 22; merging will resume at 7 AM Pacific Tuesday. With your #sign-off, we can merge the PR at that time.
Thanks.

#sign-off

@swinarko Thanks for the review. This PR is ready to merge on 9/22.


Can Versioneer be used as a dependency instead of added to the package?

Hi @warner!
Thank you so much for developing Versioneer. It is one of the most useful tools I know and I use it whenever I can. There is one thing I believe that could be improved: not adding its code to the package. The problem I see with that is relying on the packages to keep it up-to-date, making them update those files as described in Updating Versioneer.
From my understanding, versioneer.py has to be installed like that because its code has to exist before calling setup() and there is really no way to list it as a dependency. Is that the case? Because it seems that incremental has found a way:

Add this to your setup.py's setup() call, removing any other versioning arguments:

   setup(
       use_incremental=True,
       setup_requires=['incremental'],
       install_requires=['incremental'], # along with any other install dependencies
       ...
   }

Then run python -m incremental.update <projectname> --create (you will need click installed from PyPI).
It will create a file in your package named _version.py and look like this:
   from incremental import Version

   __version__ = Version("widgetbox", 17, 1, 0)
   __all__ = ["__version__"]

Then, so users of your project can find your version, in your root package's __init__.py add:
   from ._version import __version__

What about _version.py? If that specific code must really exist in that file, can't it be populated by importing from versioneer? (assuming that it is now installed and used just like a regular package)
Thanks!

Yeah, I think we'd need setup_requires= to avoid the need for adding source files to the project tree.
I'm hesitant to use setup_requires= because it bypasses pip's controls on where/when downloads happen, or at least it used to. When I first started Versioneer, I was looking at peep as a way to force pip to only download/install/execute code that matches a hash included in a requirements.txt file, and at that time, if any package used setup_requires=, those dependencies would get installed without the hash check, violating the security goals. To maintain the ability of downstream projects to stick to this hashed-sources-only mode, I've avoided using setup_requires=.
But I have to admit that I haven't used that mode much myself, so maybe it isn't really that useful. And I haven't looked at pip's current hash-checking behavior: now that peep is merged into pip, maybe it's capable of controlling setup_requires= too? And I'd certainly agree that Versioneer's functionality would be more modular and easier to install if it were a plugin like how Incremental manages it.

On Mon, Mar 06, 2017 at 11:24:52PM -0800, Brian Warner wrote:

Yeah, I think we'd need setup_requires= to avoid the need for adding source files to the project tree.
I'm hesitant to use setup_requires= because it bypasses pip's controls on where/when downloads happen, or at least it used to. When I first started Versioneer, I was looking at peep as a way to force pip to only download/install/execute code that matches a hash included in a requirements.txt file, and at that time, if any package used setup_requires=, those dependencies would get installed without the hash check, violating the security goals. To maintain the ability of downstream projects to stick to this hashed-sources-only mode, I've avoided using setup_requires=.
And I haven't looked at pip's current hash-checking behavior: now that peep is merged into pip, maybe it's capable of controlling setup_requires= too? And I'd certainly agree that Versioneer's functionality would be more modular and easier to install if it were a plugin like how Incremental manages it.

peep's readme points to the Hash-Checking Mode which has the warning:

Beware of the setup_requires keyword arg in setup.py. The (rare)
packages that use it will cause those dependencies to be downloaded by
setuptools directly, skipping pip's hash-checking.

So looks like that has not been solved yet :/

But I have to admit that I haven't used that mode much myself, so maybe it isn't really that useful.

I agree and I think the reason is that it's the developer's responsibility to keep track of the hashes of all of the package's dependencies if they want to keep them up-to-date.
If there was a feature which you could specify signing key fingerprints, it would certainly more popular. Unfortunately it will increase the complexity as it has to handle signatures and not just hashes. I would certainly use it if there was such thing.
From the New and Changed setup() Keywords:

setup_requires
A string or list of strings specifying what other distributions need
to be present in order for the setup script to run. setuptools will
attempt to obtain these (even going so far as to download them using
EasyInstall) before processing the rest of the setup script or
commands. This argument is needed if you are using distutils
extensions as part of your build process; for example, extensions
that process setup() arguments and turn them into EGG-INFO metadata
files.

(Note: projects listed in setup_requires will NOT be
automatically installed on the system where the setup script is
being run. They are simply downloaded to the ./.eggs directory
if theyâ€™re not locally available already. If you want them to be
installed, as well as being available when the setup script is
run, you should add them to install_requires and
setup_requires.)

I just realized that the packages are not actually installed and that's why incremental uses both setup_requires and install_requires. With that in mind and also that after the build _version.py has the version in it, maybe versioneer.py would not have to be used at all if there was some kind of hack using with open('package/_version.py') and exec()`, but I'm not sure if that would work but mostly if it's too wrong to do so. (Even of we made sure that the Versioneer that generated that file is legit, it might not be acceptable)
I believe this approach would solve the problem for users installing built packages, but not from source and I do not know how to solve that.
I do not know how hard it is for pip to kind of spawn a second instance in those cases because if this would work:
pip install versioneer && pip install <package>

why isn't that be possible?
It looks like that solving the problems described above we need changes to pip and that might not be feasible.
Anyway, I also don't see people using the hash-checking mode (myself included :/ ) and maybe for them using setup_requires is acceptable but I'm sure there would be the ones who prefer to stay the way it is as they can verify the Versioneer they are running is really from you.
Thanks!


Pixel2 XL, video is capturing more than the time specified, (i.e trying to record a max 1 min duration video but recording 1min 1 sec)

Current Behavior

Expected Behavior

Your Environment

software
version

react-native-video-processing

react-native

node

Facing the same issue here and also for Android getting preview for second is very slow


Lenovo Legion 5 (15ACH6H) Config added

Description of changes
New config for Lenovo Legion 5 (15ACH6H)
Things done

[x] Tested the changes in your own NixOS Configuration
[x] Tested the changes end-to-end by using your fork of nixos-hardware and
importing it via <nixos-hardware> or Flake input

Tested working on my laptop :)

@mergify rebase

@piousdeer for review

To be clear, I use the 16ach6h aka Legion 5 Pro. However, afaik, the only difference between our models is the display, and this config is, appropriately, a copy of 16ach6h minus the display stuff. So, the config looks good to me

@mergify merge

@mergify queue

@mergify rebase

@mergify queue

Please link the profile in the table in /README.md and in flake.nix
Also, I think you should remove or update the setup info left over from 16ach6h from your profile's README.md

Sorry about this, I've been extremely busy with work and didn't see this until now. Someone has amended it for everyone luckily :)
https://github.com/NixOS/nixos-hardware/issues/953


Rename m0_conf_pool::pl_order to pl_pver_policy.

Created by: rajanikantchirmade
Related mero charges are @ c/16306.

Closing as an obsolete


Add a new error handling that preserves neutral strings

ErrorHandling was enhanced with a new ReturnErrorInfoPreserveNeutral case
that makes the translator return neutral language strings without error
info.
This closes issue #11
Signed-off-by: Axel Gembe<EMAIL_ADDRESS>

Nice with a PR with tests!
Are you happy with it as it is now or should I wait with the merge?

Yea, considering the ammount of tests you have I didn't want to be the one to submit a PR without tests. The tests actually helped me find some bugs. You can merge, but more work needs to be done for StaticExtension, not sure how to do it yet


Multiselect box does not become enabled when ng-disabled expression changes to true

The following multiselect box should be disabled until the ng-disabled expression evaluates to true, and then should become enabled. The value of ng-disabled changes as expected, but the multiselect box never becomes enabled again if it started out disabled, or disabled if it started out enabled.
    <ui-select multiple ng-model="filterCtrl.contractsSelected" theme="bootstrap" ng-disabled="{{!filterCtrl.companiesSelected.length}}" search-disabled="searchDisabled">
        <ui-select-match placeholder="Select contracts...">{{$item.title}}</ui-select-match>
          <ui-select-choices group-by="filterCtrl.groupByCompany" repeat="contract in filterCtrl.contractList | filterContractsByCompany:filterCtrl.companiesSelected | filter:$select.search | orderBy:'title'">

        </ui-select-choices>
    </ui-select>

@chelzwa I had the same issue, but found that there is no need of {{}} in the ng-disabled directive, because it accepts expression. Look here: https://docs.angularjs.org/api/ng/directive/ngDisabled
I think that this issue can be closed as it is not ui-select connected.
Cheers,
Marta


settings: Reorganize options and add new section in `Organization Per…

as a consequence of too many options in Other permissions section the Save button get far up for the bottom most settings.
This commit will reorganize these settings and add a new section so that Save button is easily reachable.
Fixes: #14274
Testing is done by manually changing the settings and check whether the settings are getting saved.

LGTM :)

@SiddharthVarshney how did you test whether settings are getting saved?  I just confirmed that changing "Waiting period before new members turn into full members " to "3 days" and then reloading the page loses the setting, which is a sure sign that they are not actually being saved properly.  Please use that test procedure on each and every setting.  I think you need to change at least get_complete_data_for_subsection and maybe more functions that reference the section names for this to be correct.

OK, after doing that test myself, there's just one bug, easily corrected by moving a block of code within get_complete_data_for_subsection, so I took care of moving that and will merge this with that fix.
@SiddharthVarshney the other feedback I have for you on this PR is that your commit message, while it had decent information, wasn't written very clearly.  See 328961a4fc2ba8aa7ea6ab963569f96086a91eb5 for my cleaned up version of the commit.
Thanks for fixing this @SiddharthVarshney!

I think a good follow-up project after this would be to refactor get_complete_data_for_subsection in a way that ensures we won't have the same problem next time.  That function has a bunch of blocks like this:
        } else if (subsection === 'other_permissions') {
            const add_emoji_permission = $("#id_realm_add_emoji_by_admins_only").val();

            if (add_emoji_permission === "by_admins_only") {
                data.add_emoji_by_admins_only = true;
            } else if (add_emoji_permission === "by_anyone") {
                data.add_emoji_by_admins_only = false;
            }

And each block could instead be structured in a way that doesn't duplicate the information as to which section each widget is in inside settings_org.js. I'm not sure what the best way to do it is, but if we passed the subsection_elem into that function, we could make the conditionals (without else blocks) be something like if (subsection_elem.find(""#id_realm_add_emoji_by_admins_only"")) { ... }
@SiddharthVarshney do you want to work on this?

yeah, @timabbott I would love to refactor that part.
will try to submit  PR later this weak


examples/text_editor.py exception on windows (prompt-toolkit 2.0)

command:
E:\Github\python-prompt-toolkit\examples\full-screen>python text-editor.py

result:
Traceback (most recent call last):
  File "text-editor.py", line 300, in <module>
    scroll_offset=1)),
  File "build\bdist.win32\egg\prompt_toolkit\layout\menus.py", line 196, in __init__
  File "build\bdist.win32\egg\prompt_toolkit\layout\containers.py", line 1203, in __init__
AssertionError

Hi @skywind3000,
I'm not able to reproduce this. Can you make sure that you have the latest prompt_toolkit (the 2.0  branch)? If this still raises an exception,  then it would be great if you could print the values of the assertion which fails.
Thanks!

Closing this. Let's open it again, if either somebody can reproduce it (if it's still an issue), or if we know the actual exception.


error while loading shared libraries: libssl.so.1.1

When downloading the linux binary on arch linux.
$ ./sc
./sc: error while loading shared libraries: libssl.so.1.1: cannot open shared object file: No such file or directory

$ uname -a
Linux hq 6.11.5-arch1-1 #1 SMP PREEMPT_DYNAMIC Tue, 22 Oct 2024 18:31:38 +0000 x86_64 GNU/Linux

I ran into the same issue on Ubuntu 24.04.1 LTS (which comes with libssl v3) and "fixed" it with the following:
$ curl -sLO http://nz2.archive.ubuntu.com/ubuntu/pool/main/o/openssl/libssl1.1_1.1.1f-1ubuntu2_amd64.deb
$ sudo dpkg -i libssl1.1_1.1.1f-1ubuntu2_amd64.deb

I found the answers on stackoverflow in a couple of different places. It looks like http://nz2.archive.ubuntu.com/ is semi-official, even though downloading openssl over plain http seems like a bad thing to be doing.
I don't know if there is a similar package repo for Arch by the way.

Thanks. I bumped into this issue with several other rust / go projects.
This is normally solved without having the users install other libraries.

I guess the only thing I can do on my end is to vendor openssl. I tried it and it doubles the binary size so I'm not sure it's the way to go (especially since it to matters to some users).
Open to suggestions from someone more knowledgeable than me!

This is the last place I reported a similar issue and the maintainers fixed it
https://github.com/guilhermeprokisch/see/issues/5
Also Rust.
This is the diff between the bugged version and the fixed version
https://github.com/guilhermeprokisch/see/compare/v0.2.8..v0.2.9#diff-2e9d962a08321605940b5a657135052fbcef87b5e360662bb527c96d9a615542R30
Perhaps this is the line responsible for the fix (from Cargo.toml)?
reqwest = { version = "0.12.7", features = ["blocking", "native-tls-vendored"] }

Yes that's vendoring!
Doing it this bumps the binary size to 9.3MB for everyone; up from 4.4MB and I'm not sure that's a change I want to make :thinking:


Support ActionGroup action as an action that groups and executes an array of actions

The problem we are trying to solve is that there is a lot of code that is there just because it is not possible to call most of the actions in parallel. For example - you cannot call invokeAPI in parallel in yaml, cannot call executeCode first to set a value and then call invokeAPI/showDialog etc.
ActionGroup will solve these issues. Since it's an action itself, it will work wherever actions work.
I preferred ActionGroup over allowing an array of actions directly on an event as this allows multiple ActionGroups to be executed on conditions and provides a nice encapsulation as well

covered in #1229


Disabling referer headers

network.http.sendReferheader
 0 means never send, breaks many sites
 1 means send only for clicked links, works fine for most sites
 2 (default) means send always.

Recommendation can be to set to 0 or 1. More details here.
network.http.sendSecureXSiteReferrer
true (default) Send referer header when navigating from one https site to another.
false  Don't send referer header when navigating from one https site to another
Recommendation can be false. More details here

This list aims to block core Firefox features which *actively* leak data to third-party services.
We are not breaking the browsing experience, so you won't find things like spoofing referrers and canvas properties here.


fix txt records returned as string instead of UInt8List

fixes #33

Isn't this PR identical to #37 ?

I backed one out as it was missing a part
On Fri, May 27, 2022, 8:10 PM Nimrod Dayan @.***> wrote:

Isn't this PR identical to #37
https://github.com/Nimrodda/flutter_nsd/pull/37 ?
—
Reply to this email directly, view it on GitHub
https://github.com/Nimrodda/flutter_nsd/pull/35#issuecomment-1140152324,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGPGORJ6AJVD2W623DKR4DVMGFDTANCNFSM5XCITRIA
.
You are receiving this because you authored the thread.Message ID:
@.***>


HBASE-26712 Balancer encounters NPE in rare case

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 22s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2.4 Compile Tests _

+1 :green_heart:
mvninstall
2m 57s
branch-2.4 passed

+1 :green_heart:
compile
2m 12s
branch-2.4 passed

+1 :green_heart:
checkstyle
0m 42s
branch-2.4 passed

+1 :green_heart:
spotbugs
1m 26s
branch-2.4 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 17s
the patch passed

+1 :green_heart:
compile
2m  1s
the patch passed

+1 :green_heart:
javac
2m  1s
the patch passed

-0 :warning:
checkstyle
0m 40s
hbase-server: The patch generated 2 new + 90 unchanged - 0 fixed = 92 total (was 90)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
11m  0s
Patch does not cause any errors with Hadoop 2.10.0 or 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
1m 20s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 10s
The patch does not generate ASF License warnings.

30m 43s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4117/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4117

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile

uname
Linux 108db2b9805a 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.4 / 0020da472d

Default Java
AdoptOpenJDK-1.8.0_282-b08

checkstyle
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4117/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt

Max. process+thread count
60 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4117/1/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 52s
Docker mode activated.

-0 :warning:
yetus
0m  4s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2.4 Compile Tests _

+1 :green_heart:
mvninstall
2m 32s
branch-2.4 passed

+1 :green_heart:
compile
0m 31s
branch-2.4 passed

+1 :green_heart:
shadedjars
3m 34s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 23s
branch-2.4 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m  5s
the patch passed

+1 :green_heart:
compile
0m 30s
the patch passed

+1 :green_heart:
javac
0m 30s
the patch passed

+1 :green_heart:
shadedjars
3m 32s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 20s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
319m 19s
hbase-server in the patch passed.

335m 48s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4117/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4117

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 84b6abae619b 5.4.0-1025-aws #25~18.04.1-Ubuntu SMP Fri Sep 11 12:03:04 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.4 / 0020da472d

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4117/1/testReport/

Max. process+thread count
2299 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4117/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  2s
Docker mode activated.

-0 :warning:
yetus
0m  4s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2.4 Compile Tests _

+1 :green_heart:
mvninstall
2m 50s
branch-2.4 passed

+1 :green_heart:
compile
0m 41s
branch-2.4 passed

+1 :green_heart:
shadedjars
3m 55s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 26s
branch-2.4 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 34s
the patch passed

+1 :green_heart:
compile
0m 43s
the patch passed

+1 :green_heart:
javac
0m 43s
the patch passed

+1 :green_heart:
shadedjars
3m 51s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 25s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
319m 30s
hbase-server in the patch passed.

337m  7s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4117/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4117

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux dd48b30eee3b 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.4 / 0020da472d

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4117/1/testReport/

Max. process+thread count
1914 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4117/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 22s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2.4 Compile Tests _

+1 :green_heart:
mvninstall
2m 25s
branch-2.4 passed

+1 :green_heart:
compile
1m 49s
branch-2.4 passed

+1 :green_heart:
checkstyle
0m 40s
branch-2.4 passed

+1 :green_heart:
spotbugs
1m 18s
branch-2.4 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m  7s
the patch passed

+1 :green_heart:
compile
1m 50s
the patch passed

+1 :green_heart:
javac
1m 50s
the patch passed

+1 :green_heart:
checkstyle
0m 39s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
9m 58s
Patch does not cause any errors with Hadoop 2.10.0 or 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
1m 20s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 10s
The patch does not generate ASF License warnings.

27m 15s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4117/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4117

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile

uname
Linux 2b9bf5df265f 5.4.0-96-generic #109-Ubuntu SMP Wed Jan 12 16:49:16 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.4 / 0020da472d

Default Java
AdoptOpenJDK-1.8.0_282-b08

Max. process+thread count
60 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4117/2/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  3s
Docker mode activated.

-0 :warning:
yetus
0m  4s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2.4 Compile Tests _

+1 :green_heart:
mvninstall
2m 26s
branch-2.4 passed

+1 :green_heart:
compile
0m 41s
branch-2.4 passed

+1 :green_heart:
shadedjars
3m 50s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 25s
branch-2.4 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 30s
the patch passed

+1 :green_heart:
compile
0m 40s
the patch passed

+1 :green_heart:
javac
0m 40s
the patch passed

+1 :green_heart:
shadedjars
4m  5s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 34s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
321m 35s
hbase-server in the patch passed.

339m 14s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4117/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4117

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 90e40cbfdcfc 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.4 / 0020da472d

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4117/2/testReport/

Max. process+thread count
1986 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4117/2/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


[Android] - Messages - Refresh private conversation

env: Prod
platform: Android, chrome browser
steps:

LogIn
Go to your inbox
Open particular conversation
Scroll up to see earlier conversation/post

actual results: user sees empty inbox screen, no Load more info, then ends up in the inbox but not on the conversation

@galeza it seems you triggered a page refresh by dragging down too much?

@ddveloper I think you're right. It's very easy to trigger page refresh on Android.
However can we improve page refresh for Messaging inbox?

@galeza @rabbi08 , please have a check with this review build https://feature1960-inbox-scroll-up.fightpandemics.xyz


no_std-targeted build fails, getrandom dependency uses std

Hello everyone,
Looking at #22, I'd expect to be able to build the library for no_std target, but this currently failson getrandom like this:
$> cargo build --target thumbv7em-none-eabihf
   Compiling getrandom v0.2.3
   Compiling typenum v1.13.0
error[E0463]: can't find crate for `std`
 --> /home/user/.cargo/registry/src/github.com-1ecc6299db9ec823/getrandom-0.2.3/src/error_impls.rs:9:1
  |
9 | extern crate std;
  | ^^^^^^^^^^^^^^^^^ can't find crate
  |
  = note: the `thumbv7em-none-eabihf` target may not support the standard library

Is this expected to build at all, or things (e.g. your approach to supporting no_std) have changed since #22?

Indeed, that was the reason, I apparently scanned the Cargo.toml too quickly and setting the features like that does compile fine. Thanks!


add solo.io to the list of users/vendors

Signed-off-by: Lin Sun<EMAIL_ADDRESS>Please ensure your pull request adheres to the following guidelines:

[ ] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[ ] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] All commits are signed off. See the section Developer’s Certificate of Origin
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] Thanks for contributing!

Fixes: #issue-number
<!-- Enter the release note text here if needed or remove this section! -->

Thanks, and super excited to have solo using Cilium, looking forward to seeing your feedback/contributions  :dancing_men: :dancing_women: :100:

yay, thank you the cilium team @sayboras @xmulligan!


DBZ-5089: Add partition to outbox event router

DBZ-5089
In some use cases, it would be useful to be able to specify on which partition an outbox event should be produced. This is currently not possible as null is hardcoded into the record constructor - let's add it to the config and plumb it down.

Hi @connorszczepaniak-wk there appear to be some test failures related to this change.  Can you take a look?

@connorszczepaniak-wk The code LGTM. I've triggered another test run. Could you please also prepare a docs change? Also I think you can squash the commits into a singla one. Thanks a lot.

@connorszczepaniak-wk I've just re-read documentation and have a second thought about the implementation. How about using https://debezium.io/documentation/reference/1.9/transformations/outbox-event-router.html#emitting-messages-with-additional-fields and introduce partition placement? It would be better aligned with the other configs.

@jpechane I've put the partition into additional field placement. Much simpler now; if it looks good to you, can we kick off a CI run?

@connorszczepaniak-wk Good job, the code is done! I rebased your PR on the latest main and added a commit that uses AtomicReference to make it consistent with other instances of the similar code in the codebase. Beware! I force pushed into your branch.
Could you please just update docs in https://github.com/debezium/debezium/blob/main/documentation/modules/ROOT/pages/transformations/outbox-event-router.adoc and https://github.com/debezium/debezium/blob/main/documentation/modules/ROOT/pages/transformations/mongodb-outbox-event-router.adoc
After that the PR can be merged.

@jpechane I've added the documentation change as well. Quick question about Debezium releases -- will this change get back-ported to older releases, or will it only be available in 2.0?

@connorszczepaniak-wk Applied, thanks a lot. Givent the scope of the change and longer 2.0 dev cycle I am going to backport this to 1.9 too


Get rid of finalizer hack

This PR changes the way memory is handled. It copies each instruction from the array we get from
Capstone into a separate memory region that is managed by FFI.
The instruction array is then freed. Care is taken not to leak inst.detail, which is freed together with
the corresponding instruction (using plain free, which is what Capstone does in cs_free).

Nice, thanks! I'll have to fix parens in your method definitions though ;)
If you're using this it also needs a few things added to get up to date with capstone/next, if you're interested in working on those...


Docker user is root

In the spark-operator dockerfile, the user is intended to be switched, but spark_uid is not defined. See:
https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/blob/244033925a235f6f1c9f0c597e378960794857e5/spark-docker/Dockerfile#L38
The end result is that the user is still root:
❯ docker run --rm -it --entrypoint="" gcr.io/spark-operator/spark-operator:v1beta2-1.3.0-3.1.1 /bin/bash
root@ff2aa907910b:/opt/spark/work-dir# id
uid=0(root) gid=0(root) groups=0(root)

Is there any way to change the users in the Spark image？

Yes, we can run the sparkpp using non root as you described @AlexNavara .
The question was about the sparkoperator container.
It runs as root and cannot run as non-root so easily due to the manual procedure used in the image to generate certificate on the fly.

The problem is this hack that needs root permissions :
https://github.com/GoogleCloudPlatform/spark-on-k8s-operator/blob/5f2efd4ff97e7c0bfdb726a066118d3401576730/hack/gencerts.sh


spirv-val incorrectly lints a bitcast from int to bool

https://gist.github.com/MaikKlein/da94caa05105c13d1cb7782e946805a0#file-shader-spv-L85
         %52 = OpIAddCarry %_struct_51 %uint_1 %uint_1
         %53 = OpCompositeExtract %uint %52 0
         %54 = OpCompositeExtract %uint %52 1
         %55 = OpBitcast %bool %54

Result Type must be an OpTypePointer, or a scalar or vector of numerical-type.

And a scalar is defined as

Scalar: A single instance of a numerical type or Boolean type. Scalars will also be called components when being discussed either by themselves or in the context of the contents of a vector.

But it lints with

error: 80: Expected Result Type to be a pointer or int or float vector or scalar type: Bitcast

Version:

SPIRV-Tools v2018.4-dev unknown hash, 2018-06-24

Result Type must be an OpTypePointer, or a scalar or vector of numerical-type.

This should be read as two possibilities:

OpTypePointer
Numerical type (integer or floating point) that is either a scalar or a vector

I see, thanks for correcting me. I guess I should have read it more carefully.


Use forever.js

Coverage remained the same at 86.17% when pulling 90f32f0e3753e9e72783f79bf441887c6286d4f5 on foreverjs into f21a67c8bf649fa4bf33fbb1f870e423f81e79f9 on master.

Coverage remained the same at 86.17% when pulling 90f32f0e3753e9e72783f79bf441887c6286d4f5 on foreverjs into f21a67c8bf649fa4bf33fbb1f870e423f81e79f9 on master.


[FLINK-9610] [flink-connector-kafka-base] Add Kafka Partitioner that uses the hash of the provided key.

What is the purpose of the change
Add the simple feature of being able to route records into Kafka using a key based partitioning.
Brief change log

Added the FlinkKeyHashPartitioner class with some tests.

Verifying this change
This change added tests and can be verified as follows:

Use this instead of the FlinkFixedPartitioner while instantiating the FlinkKafkaProducer. Also add an KeyedSerializationSchema implementation that returns the right key that is to be used.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): yes
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector:  no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? JavaDocs

@tzulitai
I did some more digging and it is in fact there is a good reason for this patch.
Have a look at https://github.com/apache/flink/blob/master/flink-connectors/flink-connector-kafka-0.11/src/main/java/org/apache/flink/streaming/connectors/kafka/FlinkKafkaProducer011.java
There you'll find constructors with essentially the following variations in parameters:

Either a SerializationSchema or a KeyedSerializationSchema
Either a FlinkKafkaPartitioner or no partitioner which means it actually uses FlinkFixedPartitioner as the default.

Looking at all of these constructors

If you do not specify a partitioner then all constructors use the FlinkFixedPartitioner.
If you do specify a partitioner then it will use that partitioner.

Even the constructor that uses a KeyedSerializationSchema will NOT use that key for the partitioning (which we saw in production which caused problems).
Essentially the current FlinkKafkaProducer API makes it very hard to 'not' specify a partitioner and use the hash(key) partitioning which is nativly present in Kafka.
So give the current API we came to the conclusion an extra partitioner is needed.
Because the Flink API works this way I never looked deeper into base code to see how it really moves below the surface.
Given what I understand now I see two viable ways forward:

We change the behavior of the API so that if a KeyedSerializationSchema is used that in that case the hash of the key will be used by Kafka to partition the data over. This is however an impact full change in the way the API behaves. I.e. breaking the API.
We simply add the partitioner I created.

Yes, I noticed that too when digging into the Kafka client code. Depending on what we actually implement (see dev list) this should be the same as what Kafka does.

Is there still interest in getting this PR in? It's a tad old by now. 😅

I had a look at this (old) ticket/pull request today.
The main goal at the time was to enable having control over defining a key so that the records with the same are all going into the same Kafka partition.
What I found is that as part of https://issues.apache.org/jira/browse/FLINK-11693 that you (@aljoscha) created the KafkaSerializationSchema which produces a ProducerRecord which has a key and a value (both byte[]).
It looks to me like this feature is the better implementation of the goal of this ticket.
@aljoscha Can you confirm that this change already does what this ticket intends to do also?

I think you can implement the use case with KafkaSerializationSchema, yes, because it gives you all the power of the ProducerRecord "API". It's a bit different from the old API, where partitioner and serialization were separate, though. But if that's not a problem for you then I would say we can close FLINK-9610.

That is fine.
The problem when I started this was that it was not possible (or atleast very hard).
So let's close this.


[Float8] Add static constructor that will be used in Observer workflow

Stacked PRs:

->#869

[Float8] Add static constructor that will be used in Observer workflow

There is a interesting issue here:
E               torch._dynamo.exc.TorchRuntimeError: Failed running call_function <built-in function linear>(*(FakeTensor(..., device='cuda:0', size=(32, 128, 256)), LinearActivationQuantizedTensor(AffineQuantizedTensor(data=FakeTensor(..., device='cuda:0', size=(64, 256))..., shape=torch.Size([64, 256]), block_size=torch.Size([64, 256]), device=cuda:0, dtype=torch.float32, requires_grad=False, layout_tensor=Float8AQTLayout(
E               float8_data=FakeTensor(..., device='cuda:0', size=(64, 256), dtype=torch.float8_e4m3fn),
E               scale=FakeTensor(..., device='cuda:0', size=()),
E               transposed=False, layout_type=Float8LayoutType(mm_config=Float8MMConfig(emulate=False, use_fast_accum=True, pad_inner_dim=False)))), functools.partial(<function _input_quant_func_dyanmic_fp8 at 0x7f5c17584160>, activation_granularity=PerTensor(), activation_dtype=torch.float8_e4m3fn, scale=tensor(0.0105, device='cuda:0'))), None), **{}):
E               Please convert all Tensors to FakeTensors first or instantiate FakeTensorMode with 'allow_non_fake_inputs'. Found in aten.view.default(tensor(0.0105, device='cuda:0', size=()), [1, 1, 1])
E
E               from user code:
E                  File "/home/drisspg/meta/ao/test/dtypes/test_affine_quantized_float.py", line 46, in forward
E                   x = self.linear1(x)
E                 File "/home/drisspg/meta/pytorch/torch/nn/modules/linear.py", line 125, in forward
E                   return F.linear(input, self.weight, self.bias)
E

I have a stash that patched this but broke other quant. The TLDR is that the input_quant_func captures a torch.Tensor. The
LinearActivationQuantizedTensor stores this callable and it is specified in as a tensor attributed not a one of the tensors via unflatten. And this inner tensor doesn't get fakified. TBH I am not really sure whats the best way to handle this. cc @bdhirsh @jerryzh168

The
LinearActivationQuantizedTensor stores this callable

I mean...can we just not store the callable for the main paths?  It seems super hard to debug if the logic that changes numerics is stored in a callable instead of being clearly spelled out.  cc @jerryzh168


Cross compiling Qt 5.8 for arm fails due to OpenSSL

I know that big changes within the OpenSSL API have been made between 1.1.0 and 1.0.2 (https://wiki.openssl.org/index.php/OpenSSL_1.1.0_Changes). However, I can not resolve the following problem: I have to compile Qt with WebEngine-support, OpenSSL is required.
1.)
When compiling Qt 5.8 with OpenSSL 1.1.0 I get the following error:
error: invalid use of incomplete type ‘X509 {aka struct x509_st}’
(see: http://stackoverflow.com/questions/35488166/error-compiling-qt5-with-openssl-linked)
2.)
Compiling Qt 5.8 with OpenSSL 1.0.2 (or less) results in the following error:
Qt 5.8 can not find async.h. This file is indeed missing in the earlier OpenSSL versions.
So how am I supposed to resolve this? Both versions do not work. What are my options?
// Edit:
I just checked OpenSSL 1.1.0 with Qt 5.9. Still getting errors such as

ssl/qsslkey_openssl.cpp:182:49: error: invalid use of incomplete type ‘DSA {aka struct dsa_st}’
         case QSsl::Dsa: return q_BN_num_bits(dsa->p);
                                                 ^
In file included from /home/swg/opensslcross/opensslArm/include/openssl/crypto.h:35:0,

The last updates on this page suggest that OpenSSL 1.1.0 support won't be added until Qt 5.10:
https://bugreports.qt.io/browse/QTBUG-52905
I don't know about the async.h issue - but it sounds like a question for the Qt team rather than OpenSSL.


Kernel stability fixes

This PR contains a few small fixes to the kernel that have accumulated in the various branches I've been working on:

Fixes a deadlock in the priority donation code, where a spinlock could be held across a scheduling point.
Workaround a source of stalls in the TLB shootdown code.
Increase kernel stack size (needed for random + crypto)

A quick fix might be increasing the memory of the VM?

Ah, hmm, I need to look into this.

Okay, the kernel was leaking thread references on exit due to a subtle part of the scheduler code, causing all these (now larger) kernel stacks to build up. On top of that, the cleanup process for threads kinda assumes we're not creating a ton of kernel threads dynamically, so I made the wait function for the run_closure_in_new_thread handle call into the thread cleanup routine when waiting to keep the number of pending threads post-exit low.


Following the movements on the map as action?

It is possible to follow the movements from the user on the map? changing the location, zoom and perhaps even insert a central point where the user is focused on the map and get it as an action?
Thanks for the worldmap node, is very useful.

Under the menu - auto pan map will follow the latest point updates
For control try looking at the readme.... http://flows.nodered.org/node/node-red-contrib-web-worldmap.
The latest version you can set the start point in the edit config.


Bridging Native assets (ETH or Matic) with Etherscan

Review and format https://docs.google.com/document/d/1Bl45GZBUDXiICGNMHJh6xw0FbsOOafjxr-_lB9Ck9xE/edit#heading=h.n52p6c4itq8

Adjusted formatting and made small writing edits.


Ref/intro apps

First impressions

add the case history image to the index page

Apps

more description on how to get up and running with Binder, Azure

General cleanup

update copyright year to 2018
bibtex cleanup
add live links that are slow to resolve to the linkcheck ignore

before merging

[ ] coda patch
[ ] release notes

Thanks @sgkang!


fix: job status is now tolerant of Slurm Job HSDS upload retries

What new features does this PR implement?
A previous change introduced retry logic for HSDS uploads for HPC simulation jobs running on Slurm.  This introduced the possibility that one or more Save-results-to-HSDS Slurm steps can fail but eventually complete successfully.  In these cases, the entire job should be considered successful.
For example, the following Slurm job did complete successfully (see 7601352|Simulation-run-6598a8dce0dd9fac8e8c3b43|COMPLETED), but some intermediate steps failed and were retried (see 7601352.10, 7601352.11, 7601352.12 and 7601352.13).
The current logic fails the entire job if any Slurm steps fail, this is no longer needed and has been removed.
-bash-4.2$ sacct --jobs 7601352 --format jobid,jobname,state --noheader --parsable2 --delimiter "|"
7601352|Simulation-run-6598a8dce0dd9fac8e8c3b43|COMPLETED
7601352.batch|batch|COMPLETED
7601352.extern|extern|COMPLETED
7601352.0|Download-project|COMPLETED
7601352.1|Execute-project|COMPLETED
7601352.2|Save-raw-log-to-S3-1|COMPLETED
7601352.3|Save-structured-log-to-S3|COMPLETED
7601352.4|Zip-outputs|COMPLETED
7601352.5|Save-numerical-outputs-to-S3|COMPLETED
7601352.6|Save-other-outputs-to-S3|COMPLETED
7601352.7|Unzip-COMBINE-archive|COMPLETED
7601352.8|Save-COMBINE-archive-contents-to-S3|COMPLETED
7601352.9|Save-raw-log-to-S3-2|COMPLETED
**7601352.10|Save-results-to-HSDS|FAILED**
**7601352.11|Save-results-to-HSDS|FAILED**
**7601352.12|Save-results-to-HSDS|FAILED**
**7601352.13|Save-results-to-HSDS|COMPLETED**
7601352.14|Save-raw-log-to-S3-3|COMPLETED

How have you tested this PR?
This is a simple change, but has not been tested.  It will be tested on the staging server.

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


Represent some form of dataflow in the graph

This is a rollup/discussion issue for experimental dataflow support, starting with C++.

This would be super useful!
Right now dataflow analysis requires language specific tooling.  Moreover since Kythe understands cross language references the cross language dataflow opportunities are great.  Constant propagation would be an awesome first step.


fix linke turbidity time interpolation

As indicated in in clearsky.lookup_linke_turbidity on L212-224

This is approximate and could be made more accurate.

>>> import numpy as np
>>> np.linspace(-15, 380, num=14)
array([ -15.        ,   15.38461538,   45.76923077,   76.15384615,
<PHONE_NUMBER>4,<PHONE_NUMBER>2,<PHONE_NUMBER>1,<PHONE_NUMBER>9,
<PHONE_NUMBER>8,<PHONE_NUMBER>6,<PHONE_NUMBER>5,<PHONE_NUMBER>3,
<PHONE_NUMBER>2,  380.        ])

Another method would be to use the calendar month days:
>>> import calendar
>>> import numpy as np
>>> mdays = calendar.mdays[1:]
>>> np.concatenate([[-calendar.mdays[1] / 2.0],
                    np.cumsum(mdays) - np.array(mdays) / 2.,
                    [365 + calendar.mdays[-1] / 2.0]])
array([ -15.5,   15.5,   45. ,   74.5,  105. ,  135.5,  166. ,  196.5,
        227.5,  258. ,  288.5,  319. ,  349.5,  380.5])

This still doesn't account for leap years, but we can use calendar.isleap(<year>):
import calendar
import numpy as np

def calendar_month_middles(year):
    mdays = np.array(calendar.mdays[1:])  # copy
    if calendar.isleap(year):
        mdays[1] = mdays[1] + 1
    return np.concatenate([[-calendar.mdays[1] / 2.0],
                    np.cumsum(mdays) - np.array(mdays) / 2.,
                    [365 + calendar.mdays[-1] / 2.0]])

now you can see the difference in leap year
>>> calendar_month_middles(2016)  # a leap year
array([ -15.5,   15.5,   45.5,   75.5,  106. ,  136.5,  167. ,  197.5,
        228.5,  259. ,  289.5,  320. ,  350.5,  380.5])
>>> calendar_month_middles(2015)  # NOT a leap year
array([ -15.5,   15.5,   45. ,   74.5,  105. ,  135.5,  166. ,  196.5,
        227.5,  258. ,  288.5,  319. ,  349.5,  380.5])

Note: it's important to copy mdays because it's a borrowed reference, so if you change it, the change is permanent.
>>> calendar.mdays
[0, 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31]
>>> calendar.mdays[3] = 100
>>> calendar.mdays
[0, 31, 28, 100, 30, 31, 30, 31, 31, 30, 31, 30, 31]
calendar_month_middles(2015)  # yikes!
array([ -15.5,   15.5,   45. ,  109. ,  174. ,  204.5,  235. ,  265.5,
        296.5,  327. ,  357.5,  388. ,  418.5,  380.5])

I'm guessing that the biggest practical impact of this will be that people see an example of the calendar module, but sounds good to me!

Do you want me to add this to pull request #264 ?
Also I forgot to change 365 to 366 in calendar_month_middles()

Sure. It might make sense to do it all at once if it's going to change the
unit test expected values.
On Wed, Nov 16, 2016 at 8:50 AM Mark Mikofski<EMAIL_ADDRESS>wrote:

Do you want me to add this to pull request #264
https://github.com/pvlib/pvlib-python/pull/264 ?
Also I forgot to change 365 to 366 in calendar_month_middles()
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pvlib/pvlib-python/issues/265#issuecomment-260981488,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AELiRyE-vtSDIc7XqppD0TMh1p_NAwetks5q-yYtgaJpZM4KzLt3
.


To Discuss: Consider collecting more than PUTs

A user may reasonably want to detect whether a select or call has been triggered as well.
Outside opinions on this would be helpful. 😃

I've decided that it makes to include call Effects in the results array, since one would want to make sure that a call were executed with the correct args (as well as stubbing it on the envMapping). The question is how best to incorporate it while breaking the API as little as possible. There are a few ideas I will write here.

I'm definitely a fan of adding this functionality to this library. I'm interested to hear if you've made any progress on this so far!

@hmillison I'd be happy to hear if you have thoughts on how best to accomplish this. Namely, some may prefer to capture only PUTs, and some may prefer to capture only CALLs, and others may want to capture both - of even other events that occur.
One idea is that the lib should export a function that should accept a configuration object or array in which the user specifies what effects they want to capture.
const makeTestEngine = require('redux-saga-test-engine')
const testEngine = makeTestEngine(['PUT', 'CALL'])
// Then normal usage...

I like the pattern of having a makeSagaTestEngine function that would allow the consumers of this library to pass in a configuration option. Would it make sense to make that an object so we can leave open the opportunity to add additional configuration in the future?
Additionally, would it make sense to export both the default sagaTestEngine and makeSagaTestEngine  helper as an object to give users both options?

OK, here is a rough draft of the new API.
const {
  // The currently exposed func would be the default script for ES6-style imports. eg:
  // import sagaTestEngine, { makeSagaTestEngine } from 'redux-saga-test-engine'
  default: sagaTestEngine,
  // Can pass a config object, eg:
  // const testEngine = makeSagaTestEngine({ effects: ['PUT', 'CALL']})
  makeSagaTestEngine,
  // Some "pre-filled" testEngine funcs.
  collectPuts,
  collectCalls,
  collectCallsAndPuts,
} = require('redux-saga-test-engine')

That looks very sensible to me!
I like the idea of providing "pre-filled" options for users that don't want to worry about configuring this themselves 💯

Let me know if I can lend a hand implementing this @timbuckley

@hmillison If you wanted to take a crack at this, I would appreciate it. A little crunched for time and I know you've been waiting on this. :-)

Handled by https://github.com/DNAinfo/redux-saga-test-engine/pull/16


Update README.md

just correcting the spelling

Its just a small correction.
Make something good contributions so it will be valuable to merge
Thanks.

Ok. Doing more RnD. Closing this PR.


Controller oriented movement

+1. Disabling head directional movement is all that is needed. Makes platforming difficult.

@QwertyUieo That's a slightly different suggestion. Do you think aiming with the left hand would also make platforming difficult? Because then this means three different choices:

Head oriented movement;
Controller oriented movement;
Movement following only the analog stick input, nothing else.

I'm not sure I want to have so many options...

As a game of exploration, and not of "survival" (which reminds me of the locomotion system of The Solus Project. I liked being free Look/head to see the environment, while moving corps independently.
But i don't have finish all the game yet, perhaps some platforme passage a very tricky
By the way, you can see my progression in VR, if you want
(In french but subtitle in YT)
https://youtu.be/cJovV42uDa0

@yohaskan nice videos :) Would appreciate if you could link to the mod in the description (either link to this repo or to www.outerwildsmods.com)

In the Youtube description?
it's already there, but we only see the first three lines

@yohaskan Ah I see it, sorry about that.

I have changed some text to see a little more

@Raicuparta I was suggesting your 3rd option. "Movement following only the analog stick input, nothing else." As @yohaskan suggested the game's focus is more on exploration. I feel that being able to view the environment while moving would be a nice feature to have.

@QwertyUieo I see... The problem is that someone else suggested option 2. Option 3 is definitely easier to implement. I just don't like having too many options, especially options that conflict with each other. I'll have to think about it.

Going with option 2 for now. I think it works pretty well and lets you easily look around while moving. Option will be available in next release (0.45)


Addition of main bower option

Addition of main option allows tools and plugins such as wiredep to automatically inject the library. See:
https://github.com/taptapship/wiredep

thank you for your contribution.


[Task]: Workflows to get sorted based on recently used objects per user in a workspace

SubTasks
The app within a workspace gets sorted based on the user action, i.e the last edited app is seen as the first card. This ordering is not present for the workflows.
Ref PR for getting workspaces and apps in recently used order https://github.com/appsmithorg/appsmith/pull/29004

Hey team! Please add your planning poker estimate with Zenhub @ayushpahwa @nsarupr @Parthvi12


PURLs for LOGGERHEAD

I'm getting a 500. I think the BBOP redirect is circular. @cmungall

This ontology has no products:
---
layout: ontology_detail
id: loggerhead
title: Loggerhead nesting
contact:
  email: Peter Midford
  label<EMAIL_ADDRESS>homepage: http://www.mesquiteproject.org/ontology/Loggerhead/index.html
is_obsolete: true
---

A demonstration of ontology construction as a general technique for coding ethograms and other descriptions of behavior into machine understandable forms. An ontology for Loggerhead sea turtle (<i>Caretta caretta</i>) nesting behavior, based on the published ethogram of Hailman and Elowson.


content variant and single file addressing requirement error message are hard to understand

it is really hard to debug this error message, almost impossible for big releases like DBpedia:
All used sub-properties of dataid:contentVariant MUST be used by all dataid:Parts exactly ONCE.
useful would be enhancing it with the following insights

[ ] the conent variant that causes the violation
[ ] the files that are missing a value for it
[ ] the files that can not be uniquely adressed grouped into the clusters that are not distinguishable

Added a new validator BEFORE the SHACL step. Since the SHACL validation for this issue was a little wonky anyway!
Report will now look like this:
Invalid content variant setup. Two or more files are not distinguishable by either dataid:formatExtension, dataid:compression or any custom content variant.
[
  {
    "downloadURLs": [
      "https://holycrab13.github.io/webid.ttl",
      "https://raw.githubusercontent.com/dbpedia/webid/master/webid-dbpedia.ttl"
    ],
    "http://dataid.dbpedia.org/ns/core#formatExtension": "ttl",
    "http://dataid.dbpedia.org/ns/core#compression": "none",
    "http://dataid.dbpedia.org/ns/cv#bleb": "asdf"
  }
]


actions: Update both stable & unstable flake.lock

When triggering manually we can update:

both branches
Only the main branch (unstable)
Only the chosen stable branch

Is there a reason we need an input? workflow_dispatch already has a ref selection built in.
For the rest API ref sits alongside inputs, for the gh CLI you can use --ref, for the web UI there should be a drop-down box.

Is there a reason we need a branch input? workflow_dispatch already has a ref selection built in.

I believe this is for the workflow itself, not the branch it targets I believe. The main reason for the branch trigger is to allow to manually update only a single branch, instead of both stable & unstable branches

Also, is there a reason not to automatically run the CI for the stable branch(s) too, even if less frequently?

I believe this workflow will run the update every saturday at noon for both the stable & unstable branch, though I have yet to test it

believe this is for the workflow itself, not the branch it targets I believe

Correct, though the checkout action defaults to that workflow ref.
If you want to be able to matrix multiple branches that's not useful, though.


Why isn't this implementation in the standart library?

By the README this lib rocks, and since it is so much faster than the std why isn't it there?
Just out of curiosity.

An attempt was made to make std use this implementation. But it was ultimately abandoned/put on hold for a number of reasons. There were issues with certain platforms that did not handle time/sleeping very well (such as Wasm and SGX) and a large review of parking_lot yielded a number of issues that had to be addressed before it would be approved into std, and these things have not been fixed.
If you want to read a lot more you can dig deep into this PR: https://github.com/rust-lang/rust/pull/56410

Note that the Std implementation has been overhauled in Rust v1.62.


Reconsidering Gateway Architecture

A first pass implementation of the design proposed in #714
We have a working gateway based on CLN plugin and assumptions of serving a single federation. This implementation of the proposal referenced looks to open ground for further development and inclusion of new features like:

support for multiple federations
support for multiple lightning implementations
servicing: support swapping out of ln rpc on a live gateway

Working plan

[x]  Define standard gateway messaging pattern

[x] Extract message senders from existing impl of oneshot sending
[ ] Implement a gateway message router (WIP: DEFFERED)

[x]  Implement LnGateway with API to register_ln_rpc and register_federation
[x] Extract GatewayActor API from existing LnGateway struct
[x] Refactor cln rpc instantiation code

[x] Generalize ln rpc creation under an LnRpcFactory trait
[x] Use GatewayLnRpcConfig structure for staged instantiation of an ln rpc. By 'staged' I mean : create a factory, then call it later from within the LnGateay with some owned messaging channel

[x] Apply LnGateway API in ln_gateway.rs binary.

[x] uses cln rpc factory in gateway register_ln_rpc
[x] uses creates a federation client outside the gateway, and registers it with register_federation

[x] Migrate integration tests

[x] Fake fixtures: Define and apply LnRpcFactory,GatewayLnRpcConfig when building GatewayTest
[x] Real fixtures: Define and apply LnRpcFactory,GatewayLnRpcConfig when building GatewayTest

Note to the reviewers:
I'm thinking of this as a one-step-jump in how we build the gateway. It's a large PR, on which we could then incrementally build gateway features.
closes #714, closes #481

Codecov Report
Base: 59.35% // Head: 59.45% // Increases project coverage by +0.10% :tada:

Coverage data is based on head (3ac1818) compared to base (a2f26a8).
Patch coverage: 24.48% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #709      +/-   ##
==========================================
+ Coverage   59.35%   59.45%   +0.10%
==========================================
  Files         100      101       +1
  Lines       16227    16319      +92
==========================================
+ Hits         9632     9703      +71
- Misses       6595     6616      +21

Impacted Files
Coverage Δ

ln-gateway/src/bin/ln_gateway.rs
1.85% <0.00%> (+1.31%)
:arrow_up:

ln-gateway/src/cln.rs
0.00% <0.00%> (ø)

ln-gateway/src/ln.rs
0.00% <ø> (ø)

ln-gateway/src/webserver.rs
44.00% <50.00%> (+9.62%)
:arrow_up:

ln-gateway/src/lib.rs
58.97% <65.97%> (+8.00%)
:arrow_up:

ln-gateway/src/messaging.rs
66.66% <76.19%> (ø)

modules/fedimint-mint/src/lib.rs
90.90% <0.00%> (-0.38%)
:arrow_down:

fedimint-server/src/net/peers.rs
92.10% <0.00%> (-0.27%)
:arrow_down:

core/api/src/server.rs
39.75% <0.00%> (-0.21%)
:arrow_down:

core/api/src/encode.rs
13.50% <0.00%> (+0.27%)
:arrow_up:

... and 2 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Split project up into multiple packages

Not ready to merge, but would be good to get initial feedback
TODO

[ ] Pick core project name
[x] Sort out package JSONs (dependencies, de-duplication etc.)
[ ] Fix tests
[ ] Sort out Typescript compilation
[ ] Ensure it works

Aweessssommme

I'm also adding you as a collaborator

I've put all the tests in the root folder. Not sure if this is best or not—would be interested to hear what you think. If it is, we need to decide what to do with render.test.tsx. But aside from that, the other tests work.


Deploy it online?

Hi, Really thanks for sharing this project. I just want to know that is there anyway to deploy this on server? Like whenever a user hits a url, it gets the webcam opened of its laptop?
Waiting for your response.
Very much Thanks,
Zirsha

Hi @ZirshaRiaz
yeah, you do what you are saying, but for input streaming,  you should use external ipCamera, like your own camera (with IP address). to test you could use external open Camera.

Hello,
Thanks, I resolved it with doing the same. Appreciate your response.

you are welcome.


windows 10 logo animation

https://github.com/pranjay-poddar/Dev-Geeks/assets/120590200/f0898d5a-88f0-41d9-8df4-0c204caf625b

@MohitGupta121 @TusharKesarwani Kindly assign this issue.

@MohitGupta121 @TusharKesarwani raised the pr please review it


Overlord v2

Refactored with Dan Kotowski and Daniel Schepers' comments.

Bomb class is cleaner
Javascript is more organized
BombState class used for clarity
Less static UI and more error message handling with JSON

@ssapra looks great! Check out the comments I've left and let me know if you have any questions.
@paulghaddad I recommend to close & approve.

@ssapra Excellent work on this, and I second Dan that you are good to move on!


Electron  db.find() at the Main Process returns Empty array

https://github.com/chentsulin/electron-react-boilerplate
this is the boilerplate that i use and when you load nedb in app\main.developement.js ,  it will insert docs just fine but when you perform a read on it and pass it to the IpcMain (example) it will return an empty array [ ] no matter what.
i'm using the path join described in the readme but still no luck, any idea about what goes wrong?

The methods on the store are not supposed to return data.
You give them a callback that will take data and do what you want or need with them.
I use find like this to load my collection into a rxjs BehaviorSubject:
this._db.find<Stereotype>({},
(e,docs) => {
    if(!e) {
        this._zone.run(() =>
            this._sub.next(docs)
        )
    }
});

In short : read the examples in the wiki

i use this perfectly fine on the app "browser" , and yes i pass the data to a callback , still no luck , tested on windows 8 and windows 7

Do you mean that docs is empty or that there is an error?

no error no docs , just an empty array , all the time.

Are you sure there are data in your db? Is there an error when you insert docs in it?

the insert method works just fine i can insert docs and can see the database file size growing i can find my docs inside it , the problem is in the read operation , even if u write
 db.find({ }, function(err , docs){ console.log(docs) }) it will return empty all the time

I have setup my own angular+wepback project, so I'm not sure what main.development.js exactly is or does but you're now talking about the electron Main Process.
This begs some new questions in my mind:

Do you write and read on the same db object?
Do you write and read in the same electron process?

check this link https://github.com/chentsulin/electron-react-boilerplate , this is the boilerplate that i use , and inside of the folder app , there's a file called main.developement.js which is the main process of this electron app , if i use nedb there , the read will not work and give only empty array without error , but insert method does work.
If you require and use nedb in the view , react or angular or whatever , both read and insert works just fine.
The thing that i've noticed with nedb is that the library is splitted in two , one version for the browser , and one version for other environments(node , electron...) and i suspect that the nedb library that i'm using is returning the browser version in both render processes ( main , renderer )... but i'm not sure about it.

Yeah if you get the browser version in the main process it can be where the issue resides. Especially if everything works in the renderer process.
I don't actually know what module is returned if you just require nedb.

Fixed , it wasn't the library version i was overrinding  some variables at the ENV process including the variable that i used for path join... my bad u.u
Thanks again for your valuable help.

Haha we all do these x)

how could I send response inside dB.find({}). if I console log m data then it is perfectly fine. but I couldn't send the response...in the client-side, it only shows undefined


Create XRDs using Kubebuilder

Adds boilerplate so that the XRDs can be generated using Go and Kubebuilder.
This removes the need to manually craft the OpenAPI scheme for the XRDs, reducing human mistakes.
Closes: #15
Checklist

[x] PR contains a single logical change (to build a better changelog).
[x] Update the documentation.
[x] Categorize the PR by setting a good title and adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog.
[x] Link this PR to related issues or PRs.

@ZuGao
You need to be in the root dir of the repo, not in /apis or any other.

I tried to delete the whole apis folder but it still does not generate the APIs.

That's definitely the wrong option, as you delete the source code and makefile :smile:

You need to be in the root dir of the repo, not in /apis or any other.

yes that worked

That's definitely the wrong option, as you delete the source code and makefile

yeah just read the piece of code from the makefile :)


Sandglass Star Pattern in Java

Would like to add a sandglass pattern in Java. Can this issue be assigned to me as a part of Hacktoberfest.
The number of rows will be defined by the user.
The pattern will look something like this -

Sure, You are welcomed ☺

Hi @AdarshAddee, could you please clarify why have you closed this issue? I haven't yet made a PR for this issue.
Can you please re-open it and also label it as hacktoberfest?
Thanks.


DatabaseMigration example fails because database does not exist.

Problem
When using a MigrationService as shown on the DatabaseMigration  example, I got an error saying that the database was not created.
This seems to be caused by the transaction used alongside the call to MigrateAsync, and can be solved by removing it. However, we do want to run the migrations inside a transaction, but I am not sure how can we create the databa (if it doesn't exist), the run the migrations.
private static async Task InitializeDatabaseAsync(AppDbContext dbContext, CancellationToken cancellationToken)
    {
        using var activity = s_activitySource.StartActivity("Migrating database", ActivityKind.Client);
        await dbContext.Database.MigrateAsync(cancellationToken);
        //var strategy = dbContext.Database.CreateExecutionStrategy();
        //await strategy.ExecuteAsync(async () =>
        //{
        //    await using var transaction = await dbContext.Database.BeginTransactionAsync(cancellationToken);
        //    await dbContext.Database.MigrateAsync(cancellationToken);
        //    await transaction.CommitAsync(cancellationToken);
        //});
    }

Versions

aspire 8 preview 4
Npgsql 8 preview 4

Hold on. This is already fixed? It looks like I was using the aspire/playground sample instead of the one in this repo. The playground example should probably be removed, I'm leaving this open so the confusion is noticed 🙏

Closing as this is resolved it seems.


Create nn_lib git submodule

Can we create a common git submodule to serve all purposes?

Seems to be going pretty well to work inside this framework so no need to change! My suggestion would be to close this issue and just work within the HLS4ML repo.  Agreed?

Yeah I think we can table this for now.  We can revisit it later if the need arises.