Fix title suffix

This PR is a bit hard to review due to its size and because it mixes several different areas with different owners. It would be easier to review these kind of changes per folder.
This also has some merge conflicts right now.

@Thraka do you want to move the suffix into the titles directly? Maybe we do this in a separate PR.
This keeps getting merge conflicts because it's a large PR that involves many areas/reviewers.

@mairaw We'll keep discussing it as a team, I don't know that we totally decided what to do. But let's get @tdykstra items out of this PR then merge it and then move forward.
@Youssef1313 we'll get this merged soon 😄

I'm closing this, for the following reasons:
As @Thraka discussed above, we are having an internal discussion on whether title suffix should be on the article, or in folder based metadata. While that plays out, this PR gets more merge conflicts. Rather than updating this PR repeatedly, let's wait until a decision to make the work.
Next, this is a large PR and a number of the changes are not modifying the title suffix, but changing titles and H1s. That makes it much harder to review. When we revisit this, this should be separated into several PRs, by area to keep the scope of the changes simpler to review.
We appreciate all you do to help us improve @Youssef1313 We don't like abandoning the work that went into this. When it looks like a set of changes may get too large, or may need discussion, you could open an issue or a WIP PR to discuss your plans before investing too much time on it.


🐞 CurrentPage property not respected IPropertyPaneConfiguration

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

Expected or Desired Behavior
When setting the currentPage property on the IPropertyPaneConfiguration, SPFx should set the current property pane page to the specified value.
Observed Behavior
Doesn't matter what number you use, including trying 0-index base, the first page is always shown.
Steps to Reproduce
git clone https://github.com/andrewconnell/spfx-bugrepo-proppanepages
yarn install
gulp serve

add the SPFx Bug Repro web part to the workbench & read the description... tells you how to reproduce
this cswp has 2 pages in the property pane... currently the currentPage property is set to 1, but changing it to 0 or 2 has zero effect

Thanks Andrew for a top-notch repo.  :-)  We're investigating this issue.

This feature was originally introduced to support certain first party web parts, but they ended up taking a different approach, so we removed it.  The API property was left in the interface for our GA release due to an oversight.  Sorry about that!  Because the feature wasn't really designed for third parties, it would need some work beyond simply implementing the interface property as-is.  If that's an important scenario for your work, I would encourage you to open a feature request on User Voice so that others can vote for it.  Thanks!

Thanks for the followup @pgonzal


I would love more documentation

Hi
Amongst all the project I saw wrapping IndexedDB, yours is the most tested but one of the less documented. It's like using db.js requires a non-trivial knowledge of IndexedDB, plus a knowledge of how to use db.js itself.
An example, as I'm just starting to use it : when using db.open, the server and version are required, but the schema. I had the read all the open-db.js spec file to learn how to create a database. Not very cool for a library that tries to simplify the usage of complex API.
But worry not, this comment is not a rant, rather a proposal of help. I'd like to write some public facing docs so that people effectively don't need to read code/specs to use it. Just need approval and some guidance about how you see it.
Sidenote : your project seem to be compatible with the IndexedDBShim, which is pretty cool, and one of the reason I choose to use your project.

I think we can close this issue now since the README has been enhanced to cover the API. There are already issues for Docco generation (issue #21 ) and for porting your blog (issue #19 ).

Indeed, README looks complete with regard to usage and API. No interest in db.js now, but closing anyway to keep the issues clean. Thanks for the reminder.


Can't save to directory on a different drive

Not sure whether this is an issue or a fault in my understanding of snaps, but I cannot save to a different drive (I have a big spinning data drive expressly for storing large data while I'm working on it). This was my default behavior using the ppa, now I can't seem to browse to /media/data/ in the save browser. I tried making a symlink to it in my regular home directory but permissions are denied. Is this not supported in snaps, or is there something I can change to get my regular workflow back?

Bind mounts seem to work to an extent, not certain if that's a particularly user friendly solution.

There's a permission for removable-media that needs to be granted explicitly. Either use the "Snap Store" snap to do it with a graphical utility, or run snap connect makemkv:removable-media in a terminal.


Run copy-i18n-tags

Merged later PR https://github.com/codecombat/codecombat/pull/6204 - closing this.


Release 0.10.0

Changes
Added

Public userSettings property with current theme preference
New event USER_SETTINGS_CHANGE to detect an update of userSettings on the change

Main branch dependencies

Neodora Analysis ⚠
 ⚠ Unknown tribe name

Message: The property owner in repository.json must be a valid tribe name present in mission-tracking
Repository file: repository.json

Owner status

'GRAVITY' is an unknown tribe name

🔍 How to fix it

 ⚠ Missing cache in Github Workflow

Message: Github workflow should be using caching when step uses actions/setup-node

Workflow file
Job name
Step index
Actual cache value
Expected cache value

on-commit.yml
lint
2
Not found
npm

🔍 How to fix it

 ⚠ Incorrect runner in Github Workflow

Message: Github workflow must run on a self-hosted runner

Workflow file
Job name
Actual runs-on value
Expected runs-on value

on-commit.yml
lint
ubuntu-latest
Any of [eks-runner, eks-runner-php, eks-runner-nano, eks-runner-debug, eks-runner-compute, eks-runner-cd, eks-runner-ci, eks-runner-neodora]

🔍 How to fix it

 ⚠ Use of actively maintained version of Node.js

Message: A service using Node.js should use an actively maintained version before it reaches EOL.

Current Node.js version
Message

16
The current version of Node.js has already reached its EOL (2023-09-11)

🔍 How to fix it

Commit cb18e3a  Job 346718


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v2.6.31 to before v4.4.292",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=985cafccbf9b7f862aa1c5ee566801e18b5161fb",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=e0e6a63fd97ad95fe05dfd77268a1952551e11a7"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "985cafccbf9b7f862aa1c5ee566801e18b5161fb",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "e0e6a63fd97ad95fe05dfd77268a1952551e11a7",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "comedi: vmk80xx: fix bulk-buffer overflow\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.4.292 by commit e0e6a63fd97ad95fe05dfd77268a1952551e11a7, it was introduced in version v2.6.31 by commit 985cafccbf9b7f862aa1c5ee566801e18b5161fb. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1001931


Make install fails with ERROR:  Error installing html-proofer:

System: Fedora 33
It points to a nokogiri error and mentions
Building Nokogiri with a packaged version of libxml2-2.9.10
with the following patches applied:
	- 0001-Revert-Do-not-URI-escape-in-server-side-includes.patch
	- 0002-Remove-script-macro-support.patch
	- 0003-Update-entities-to-remove-handling-of-ssi.patch
	- 0004-libxml2.la-is-in-top_builddir.patch
	- 0005-Fix-infinite-loop-in-xmlStringLenDecodeEntities.patch

What fixed it for me was:
# completely remove current environment
> rm -rf ~/miniconda3/envs/galaxy_training_material/

# attempt a new install where it should fail with the
# titled erorr
> make install

# activate the damaged environment and
# attempt the suggested nokogiri fix
> conda activate galaxy_training_material
> gem install nokogiri -- --use-system-libraries

# reattempt a make install, where the error no
# longer persists
> make install

I think this will be solved by https://github.com/galaxyproject/training-material/pull/2605#event-4853038023


Improve passage support

This commit adds parsing of unencrypted passage identities and automatically mounts an existing passage store under the mount point passage/ if it's detected during setup (not later).
Fixes #2059
RELEASE_NOTES=[ENHANCEMENT] Improve passage support
Signed-off-by: Dominik Schulz<EMAIL_ADDRESS>

Re: the last todo, the last time I talked to Filippo, he wasn't sure to want to expose the encrypted identity code sadly, so we might need to re-implement it ourselves.


Docs: Fixed map spec example

Replaced "items" with proper key "values"
Just a small fix, so apologies for not following the whole process.
Make sure you have checked all steps below.
Jira

[ ] My PR addresses the following Avro Jira issues and references them in the PR title. For example, "AVRO-1234: My Avro PR"

https://issues.apache.org/jira/browse/AVRO-XXX
In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference Jira issues in their subject lines. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain Javadoc that explain what it does

See #922

Duplicate of #922


.error.raise on 403 Forbidden responses causes method_missing blowup

When attempting to find a bucket, a 403 response's error member, when raised, explodes due to a missing message member.
/usr/local/share/gems/gems/aws-s3-0.6.3/lib/aws/s3/error.rb:63:in `method_missing': undefined local variable or method `message' for #<AWS::S3::Error:0x0000000248e840> (NameError)
        from /usr/local/share/gems/gems/aws-s3-0.6.3/lib/aws/s3/error.rb:38:in `raise'
        from /usr/local/share/gems/gems/aws-s3-0.6.3/lib/aws/s3/base.rb:72:in `request'
        from /usr/local/share/gems/gems/aws-s3-0.6.3/lib/aws/s3/base.rb:88:in `get'
        from /usr/local/share/gems/gems/aws-s3-0.6.3/lib/aws/s3/bucket.rb:102:in `find'
        from write_file.rb:101:in `<main>'

The 403 error itself was made more difficult to debug by this, as we had to step down a ways into aws-s3 to find it at all.

My Ceph RadosGW service does not return 'message' element in error response, so I've replaced it with 'code' element and created pull request:
https://github.com/marcel/aws-s3/pull/116


AMBARI-26122:Security tool reports vulnerability CVE-2016-2183.

What changes were proposed in this pull request?
Resolved this vulnerability by disabling the 3DES and DES cipher.
How was this patch tested?
Use nsfocus to do security scan on port 8441.

@tyoushinya thanks for reporting and working on this.. So, you are adding "DES" to exclusion list right.? Hope this is also tested.

If there are no other review comments, it will be merged in 24 hours.


Fix TLF string in 'bad folder' KBFS error notification

See keybase/client#2564.

Looks good to me, thanks!


⚠ Samsung has degraded performance

In 6705b51, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2913 ms

Resolved: Samsung performance has improved in 163fbba.


forceUpdate in console.clear() not working under react framework

I'm borrowing some code from this project to help with building my research prototype. While I was studying the code I noticed that in the components/Console.js there is a clear() function that uses forceUpdate() after manually set the console state to {}. While this works perfectly with preact, it doesn't seem to function in with the actual react. Seems like the forceUpdate() function ignores the reset thus the console doesn't clean up itself after calling this function.
I did some tweaks to make it more react-like. Basically I added a lines field to the Console's state so that I could use this.setState({ lines: {} }) to clean up the lines. That way it will both work with react and preact. I know this doesn't change much but just thought it would be nice to merge it back. Will start a pull request for this.

ah actually just noticed that I'm not authorized to push to this repo. Lol anyway let me know if how I can merge this fix back.

Merged - thank you 👍


Fix for callbacks invocation

Tweaked callbacks overwriting and preventing to trying of calling strings

Please, forward PR into dev-branch.


RivaTuner incompatibility with MSIRGB

RivaTuner, which comes bundled with MSI Afterburner, seems to stop MSIRGB from working if the application detection level is 'high'.
Need to know why and whether there is a way to work around it.

Closing this as I have no info and it's probably not important anyway.


Feature request: Disable IntelliSense prompts while pressing the dot key inside a comment.

While writing code documentation, pressing dot at the end of a sentence, an auto-complete box appears. Sometimes, it doesn't get immediately noticed and the user presses the Enter key, adding the first identifier at the list (usually abs) and requiring a Ctrl-Z.

yes, is annoying.
closing as dupe of https://github.com/ionide/ionide-vscode-fsharp/issues/74


USelect change the type of a option's value

Environment

Operating System: Darwin
Node Version:     v20.14.0
Nuxt Version:     3.12.2
CLI Version:      3.12.0
Nitro Version:    2.9.6
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools, css, app, runtimeConfig, sourcemap, components, modules, eslint, i18n, postcss, ui, vite
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Version
v2.17.0
Reproduction
https://stackblitz.com/edit/nuxt-ui?file=app.vue
Description
USelect component will change the type of a option's value to string.

Additional context
No response
Logs
No response

Addendum: I found that the .number modifier solves this problem. But I don't think the component should need to rely on modifier handling. And it doesn't make mixed-type options work either.


[Not quite a bug] animations i cant find anymore

Describe the bug
About month ago i used to play gmod on home PC but travel now, i installed all my subs on notebook and powershell installed beatrun from this fork. i checked "1" option from animation changer (reanimation project) but it seems like old, before on home PC i had new animation set eg when coiling (jump+ctrl) animation was crossarmed jump, and not straight forward "Kong" with two legs higher than middle of screen, and had hand animation when come close to a wall just like in Mirrors Edge, cant find what animation set was that, MAY be it was not reanimation project but i cant find any info bout what was it then.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
Any ideas bout that animation set?
Screenshots/Videos
If applicable, add screenshots or videos to help explain your problem.
Additional information:

Game Version: x64
Game Type: singleplayer

Additional context

MTB changed them because some playermodels look bad with crossarmed jump.
Hand animations on the wall was disabled because of my bad implementation, i maybe will remake them


Move beaver restart to it's own play

The way we currently do the beaver restarts means that it takes an
inordinate amount of time to restart beaver across all hosts.
This commit moves the beaver restart tasks to their own play so that
they can be targeted at the correct hosts in the way that ansible is
designed to be used.
An example of the difference this makes to the timings, from an AIO:
OLD WAY:
PLAY RECAP ********************************************************************
rpc_pre_upgrade | Restart beaver if installed ------------------------- 445.65s
rpc_pre_upgrade | Check if beaver is installed ------------------------- 45.07s

real       8m13.294s
user       0m10.392s
sys        0m1.471s

NEW WAY:
PLAY RECAP ********************************************************************
Restart beaver if installed -------------------------------------------- 41.22s
Check if beaver is installed -------------------------------------------- 1.42s

real       0m51.494s
user       0m4.022s
sys        0m1.427s

Connected #1419

:mag_right: Preview

:eyes: Preview

@git-harry @evrardjp tag added

recheck_all

recheck_all

recheck_ceph

recheck_swift

recheck_ceph

recheck_swift

recheck_swift

recheck_swift


Egeria security updates

we need to upgrade following libs due to security vulnerabilities

org.codehaus.jackson:jackson-mapper-asl:1.9.13 -> moved as jackson-databind
jackson-databind upgrade to 2.11 or higher
org.apache.tomcat.embed:tomcat-embed-core to 9.0.40 due to CVE-2020-9484 ,  CVE-2020-13935
gremlin-shaded to 3.4.8 due to CVE-2020-24750, CVE-2020-24616, CVE-2020-14195, CVE-2020-14062, CVE-2020-14061, CVE-2020-14060,
io.netty:netty-all to 4.1.44.Final due to CVE-2019-20444
com.thinkaurelius.thrift:thrift-server to 0.3.9  due to CVE-2019-0205 ( looks like very outdated lib)
org.apache.thrift:libthrift to 0.13.0 due to CVE-2019-0205 ( looks like very outdated lib)
org.yaml_snakeyaml due to   CVE-2017-18640

@cmgrote / @mandy-chessell  can we please prioritize this issue ?

@Shrinivas-Kane Thanks Shrinivas. It looks like moving Cassandra up to 3.11.9 would resolve the thrift related dependencies.
@lcpopa Is there are a reason that we are using Cassandra 2.2.13? It looks quite old now (2018). Would it be possible to move up to at least 3.11.9?

Actually I think my assertion above about 3.11.9 is incorrect:
Our current Janusgraph (latest, 0.5.2) is tested with Cassandra 3.11.0, but that still uses
thrift-server 0.3.7
libthrift 0.9.2
Even Cassandra 3.11.9 appears to still use the same versions.
So we would need to 'manually' pull in later versions of the dependencies, but we should assess whether that would be feasible. @planetf1 @lcpopa Thoughts?

This is also mentioned in #2671

We generally update the prereqs once a month.

jackson-mapper is not used directly, though we do specify a transitive dependency version in the top level pom.xml. This is still pulled in by janus's cassandra support & hadoop-common. We do then define dependencies on these so the jar does get pulled into egeria. We have a plan to move the hadoop support to a new repo (and look at updating versions there). Graham has commented on janus - and more generally we need to dynamically load connectors to reduce what dependencies are brought in for everyone.
Jackson-databind is already at 2.11.3
tomcat-embed-core is 1 step back -- we usually update all our dependencies in each build cycle. We did back-off our spring related changes this month (which may have included tomcat) due to a compatability issue with our azuul usage in the UI
gremin-shaded is already at 3.4.8
netty-all is already much later at 4.1.53
see graham's comments on thrift
snakeyaml is already much later at 1.27

I'll try to add specific references on next steps...

@planetf1 Nigel, I agree that Egeria should not depend on Cassanda and Janus

@planetf1 Nigel, I agree that Egeria should not depend on Cassanda and Janus

I think this needs to be moved to an issue to track, rather than a PR. The simple version changes are already handled by our monthly cycle (just done January's) or are held back due to other issues (the spring stack, due to netflix azuul we hope to update later in the month).
That issue needs to assert the need to have cassandra etc as optional dependencies - at least when packaged, and we need to improve our docs in this area. The foundation work to ensure libraries are dynamically loadable is already underway and addressed in issues, but this seems a good validation case.

I think this needs to be moved to an issue to track, rather than a PR. The simple version changes are already handled by our monthly cycle (just done January's) or are held back due to other issues (the spring stack, due to netflix azuul we hope to update later in the month).
That issue needs to assert the need to have cassandra etc as optional dependencies - at least when packaged, and we need to improve our docs in this area. The foundation work to ensure libraries are dynamically loadable is already underway and addressed in issues, but this seems a good validation case.

#2671 is already open to address the janusgraph/cassandra cleanup so I believe we can close this as the other issues are addressed


Seems to be broken with Sketch 39 release.

Command+J doesn't seem to be pulling up the relabel button dialog. I reckon it has something to do with Sketch's introduction of resizing functions (Pin to corner, Stretch, etc).

Hi there, I've tried it in Sketch 39 and am having no issues whatsoever.  Could you describe in detail what's happening for you?  Have you tried disabling other plugins and/or reinstalling Relabel Button plugin?

It stopped working for me with the Sketch update, but after uninstall/reinstall it works again.

Ok, good to hear.  You're the 2nd report of needing to reinstall... if I hear of more then I'll report back to the Sketch team.


Methods needed to assess fit of Hawkes Process

I think it would be extremely helpful to have methods to assess the fit of a Hawkes Process, namely:

Residuals. To assess fit, we would like to check that the residual process is homogenous and has exponentially distributed inter-event times (F. Lorenzen: Analysis of Order Clustering Using High Frequency Data: A Point Process Approach pdf).

Conditional intensity function given the history/timestamped data (+1 for plotting functionality with respect to the corresponding timestamped data).

Hi, indeed it looks pretty interesting, we simply did not have enough time for that yet. I can help you if you want to add one of these features, otherwise, I might implement them but maybe not in a very near future.

Hi, I have implemented a workaround to asses the quality of the fits. Not perfect, definitely slow but should work. See: https://github.com/majkee15/TICK-goodness-of-fit

Thanks @majkee15, I just used it on my 1d data. It seems to work fine.

@majkee15 / @mvicrob
Would either of you be able to provide me with a small script which tests this so I can see about adding it

In fact, the notebook provided by @majkee15 is probably what you are looking for https://github.com/majkee15/TICK-goodness-of-fit/blob/master/time_transform_with_tick.ipynb.
I slightly modified this notebook for my need:

For a 1d Hawkes process (I'm interested in an intertrade time series)
With Statsmodel QQ plot instead of Scipy Probplot (because of https://stackoverflow.com/questions/48108582/how-to-interpret-scipy-stats-probplot-results)
With a loop on potential decays, as in the example of https://stats.stackexchange.com/questions/366381/fit-hawkes-process-to-1d-data-using-python-package-tick

Below if a very draft idea of the result (tested with Python 3.6):
import matplotlib.pyplot as plt
import numpy as np
from tick.hawkes import SimuHawkesExpKernels, HawkesExpKern
from tick.plot import plot_point_process
from scipy import integrate
from scipy import stats
from tick.plot import plot_basis_kernels, plot_hawkes_kernels
import statsmodels.api as sm

plt.style.use('ggplot')

# Part 1: simulate and fit
###############################################################################################################################

end_time = 30000
decay = 0.3
decay_candidates = [0.05, 0.1, 0.2, 0.3, 0.4, 0.5, 0.75, 1.0, 1.5, 2.0]
baseline = [0.12]
adjacency = [[0.1]]

hawkes_exp_kernels = SimuHawkesExpKernels(adjacency=adjacency, decays=decay, baseline=baseline, end_time=end_time, verbose=False, seed=1039)
hawkes_exp_kernels.track_intensity(0.1)
hawkes_exp_kernels.simulate()

best_score = -1e100
best_hawkes_learner = None
for i, decay in enumerate(decay_candidates):
    hawkes_learner = HawkesExpKern(decay, penalty='l1', C=10, gofit='likelihood', verbose=True, tol=1e-11, solver='svrg', step=1e-5, max_iter=10000)
    hawkes_learner.fit(hawkes_exp_kernels.timestamps)

    hawkes_score = hawkes_learner.score()
    if hawkes_score > best_score:
        print(f'Obtained {hawkes_score} with {decay}')
        best_hawkes_learner = hawkes_learner
        best_score = hawkes_score
        best_decay = decay

print('Best decay is:' + str(best_decay))
t_min = 100
t_max = 200
fig, ax = plt.subplots(1, 1, figsize=(10, 6))
best_hawkes_learner.plot_estimated_intensity(hawkes_exp_kernels.timestamps, t_min=t_min, t_max=t_max, ax=ax)
plot_point_process(hawkes_exp_kernels, plot_intensity=True, t_min=t_min, t_max=t_max, ax=ax)

# Enhance plot
# Set labels to plots
ax.lines[0].set_label('Estimated')
ax.lines[1].set_label('Original')

# Change original intensity style
ax.lines[1].set_linestyle('--')
ax.set_alpha(0.8)

# avoid duplication of scatter plots of events
ax.collections[1].set_alpha(0)

ax.legend()
plt.title("1d Hawkes process with exponential kernel and it's likelihood best fit")

fig.tight_layout()
plt.savefig('Estimated_vs_Original.pdf', bbox_inches='tight', dpi=500)

# Part 2: residuals process computation and their QQ plot
###############################################################################################################################

arrivals = hawkes_exp_kernels.timestamps
step = 0.01
intensities = best_hawkes_learner.estimated_intensity(arrivals,step)[0]
intensities_times = best_hawkes_learner.estimated_intensity(arrivals,step)[1]

def resid(intensities_times, intensities, timestamps, dim, method):
    arrivals = timestamps[dim]
    thetas = np.zeros(len(arrivals)-1)
    ints = intensities[dim]
    for i in range(1, len(arrivals)):
        mask = (intensities_times <= arrivals[i]) & (intensities_times >= arrivals[i-1])
        xs = intensities_times[mask]
        ys = ints[mask]
        try:
            thetas[i - 1] = method(ys,xs)
        except:
            thetas[i - 1] = np.nan
    return thetas

def goodness_of_fit_par(learner, arrivals, step, method):
    dimension = learner.n_nodes
    intensities = learner.estimated_intensity(arrivals, step)[0]
    intensities_times = learner.estimated_intensity(arrivals, step)[1]
    residuals = [resid(intensities_times, intensities, arrivals, dim, method) for dim in range(dimension)]
    return residuals

def plot_resid(res):
    fig, ax = plt.subplots(nrows=1, ncols=1)
    fig.subplots_adjust(hspace=0.5)
    fig.suptitle('Goodness-of-fit for parametric Hawkes process')

    # QQ-plot
    probplot = sm.ProbPlot(res, stats.expon, fit=True)
    fig, ax = plt.subplots()
    fig.suptitle('QQ Plot', fontsize=20)
    probplot.qqplot(line='45')
    plt.savefig('QQPlot.pdf', bbox_inches='tight', dpi=500)

residuals = goodness_of_fit_par(best_hawkes_learner, arrivals, step, integrate.simps)
plot_resid(residuals[0])


New rule that gives replacement suggestions if title isn't short enough

@realmarv please rebase PR

And this needs to be rebased again sorry :(

And this needs to be rebased again sorry :(

No worries. I understand


How to track billing and usage in Livekit Managed Cloud using api

Ideal usecase
Monitor use case of Particular AI Agent and track there billing ?
Want to track agent level billing.
Feel free to point towards correct resource.

If an agent consumes Cloud resources, they'll be charged the same way. There's no special case pricing for Agents. Please contact<EMAIL_ADDRESS>with additional pricing questions.


Allow PrimaryDataset default to be None for MCFromGEN

Fixes #7410
@ticoann please review. Tested in my VM.
The next step (to be eventually done) would be to not save - in the workload_cache doc - parameters with None value.

Tests are Ok on my vm. @ticoann please review and merge.


Warning about TypeScript 3.5 occurred when execute yarn dev

Version
v2.8.1
Reproduction link
https://github.com/haryu703/ts_express
Steps to reproduce
git clone git@github.com:haryu703/ts_express.git
cd ts_express
yarn install
yarn dev

What is expected ?
No warnings occur.
What is actually happening?
Following warning has occurred.
=============

WARNING: You are currently running a version of TypeScript which is not officially supported by typescript-estree.

You may find that it works just fine, or you may not.

SUPPORTED TYPESCRIPT VERSIONS: >=3.2.1 <3.5.0

YOUR TYPESCRIPT VERSION: 3.5.1

Please only submit bug reports when using the officially supported version.

=============

Additional comments?
This warning has occurred when developing Nuxt.js+express application.
Related issues:
https://github.com/typescript-eslint/typescript-eslint/issues/577
https://github.com/facebook/create-react-app/issues/7160

This bug report is available on Nuxt community (#c9324)

@haryu703 I noticed that lately too, It's a typescript-eslint issue which is not a dependency of Nuxt TypeScript, it's something you manually add in your project, so there's unfortunately nothing we can do (unless revert typescript version but doesn't sound so good), you just need to wait typescript-eslint next release which should be really soon (issue fixed in https://github.com/typescript-eslint/typescript-eslint/pull/597)
As it's not a Nuxt direct issue, i'm closing this.

@kevinmarrec Thank you for answering. It helped me to know the cause of this.


Teach the server and client how to create new topics and schemas

This PR includes the following:

Logic in the protocol for creating new topics (and their associated schemas)
A new "create topic" command in the REPL
Plumbs logic through database internals for validating new schemas against existing topic hierarchies.

closes #108

This also introduces a new list command, "list schemas", which returns a list of topics that have a non-string schema set.

This also introduces a new list command, "list schemas", which returns a list of topics that have a non-string schema set.

Should we add a new column to the query output for type? Or for list topics with schema?

Should we add a new column to the query output for type? Or for list topics with schema?

I think so. Either a column, or the client should indicate non-string data, I.e "int32(40)" or "binary()"

I think so. Either a column, or the client should indicate non-string data, I.e "int32(40)" or "binary()"

I decided to go with putting the schema directly in the data for non-strings, since we'll have to elide it in some cases anyway (e.g. for binary)

What this looks like now:
> query all
+-------------------------------------+---------------+------------+
|                TIME                 |     TOPIC     |    DATA    |
+-------------------------------------+---------------+------------+
| 2022-12-31T17:37:20.480499375-08:00 | /             | some data! |
| 2022-12-31T23:22:25.647325-08:00    | /sensors/temp | int32(32)  |
+-------------------------------------+---------------+------------+


Fix deprecated hidden-inset title bar style

As of https://github.com/electron/electron/pull/11975 the hidden-inset
titleBarStyle has been removed so the custom-inset title bar option
renders a normal title bar which is not desired.

This looks like it might be the cause of https://github.com/atom/atom/issues/17377?

@Arcanemagus Yea that's correct


gdal2isce_xml.py for ampdispersion and ps_pixels results

It would be convenient to include the xml conversion into the ampdispersion and ps_pixels script.

We welcome contributions.


doc: Update SLES installation documentation

Remove the warning as a CI of SLES in available
https://github.com/kata-containers/ci/pull/142.
Fixes #425
Signed-off-by: Gabriela Cervantes<EMAIL_ADDRESS>

/test


Only run Travis-CI tests for PRs and when master branch changes

Fixes: #76
This PR should only run the Travis-CI tests once, for the PR not for the branch.
Should speed up the time it takes to approve and merge small PRs
Release note:
NONE

I'm not adverse to this, but given the two jobs run in parallel I feel like there's little to gain by doing this. The only reason I can think why we would keep testing both the PR and the branch, is to make it easy to see that the reason your PR is failing, is due to the the merge with master. This is pretty useful and could save someone spending ages trying to work out why something passes locally, but fails in CI (and as such, requires rebasing and fixing up against master).

/lgtm
/accept

Hey @munnerz thanks for the review and for merging this.
I must say that I didn't realise that subtle difference between branch and PR builds.
I agree that this won't make the builds faster, but hopefully with fewer tests to run, we'll reduce the likelihood of spurious test failures due to minikube flakiness / network problems etc.

👍  let's leave it like it is for now. If we start getting issues that would be easier to debug with the additional tests I think we should add them back in.


I can't use FGO_JP_REGULAR.lua on Non-rooted Phone

You have to use Daemon. Firstly, you are on Android 7 which doesn't support Accessibility mode. Follow the instructions on installing Daemon on your device.


TiDB Grafana gRPC request source duration is inaccurate

Bug Report

What version of TiKV are you using?

v8.0.0 ~
What operating system and CPU are you using?

Steps to reproduce

Deploy a new cluster and check the gRPC request source duration metrics.
What did you expect?

What did happened?

severity/minor


added whitespace function

It was documented, but not included.

 Thanks!!


Require the number of passed tests to match up.

This forces the number of passed tests to stay constant by default. For changes that actually aim to fix failing test cases, the expected number needs to be updated accordingly. The goal is to detect any unintended logic changes, no matter if the result is positive or negative.

Green now!

@WalterBright do you have any objections? Would be good to get this merged, so that #15 and #16 can be rebased on top of this to have an official test result.


Add Array.take method

Takes N elements from the start or end of the array
let array = [1, 2, 3, 4, 5];
assert Array.take(array, 2) == [1, 2];
assert Array.take(array, -2) == [4, 5];
assert Array.take(array, 10) == [1, 2, 3, 4, 5];
assert Array.take(array, -99) == [1, 2, 3, 4, 5];

Any feedback would be appreciated)

Anything?

The more general signature seems okay to me because it's still possible to pass a Nat without any additional conversions (example). The +/- convention is reasonably intuitive for developers familiar with Python or JS.

Also, do we want a version for mutable arrays too?

+1

Looking at the module signature, I see that most (but not all) operations only operate on immutable arrays. Instead of adding a mutable variant of just this operation ( takeVar or takeVarArray?),  perhaps we should consider later adding a new module VarArray with  mutable versions of all relevant operations (VarArray.take, VarArray.filter etc).
So I'd say let's just merge this for now.


GH actions experiment

Core - Full Integration Build 9518 outcome was UNKNOWN using a merge of fb8f49cb36b641a2f409d6bc832663d3fb8c65f1
Summary: Canceled Build time: 00:00:15

Core - Full Integration Build 9512 outcome was UNKNOWN using a merge of fb8f49cb36b641a2f409d6bc832663d3fb8c65f1
Summary: Canceled (Exit code 143 (Step: Build & test core (Maven)) (new)) Build time: 00:00:15


[move-stdlib/diem-framework] Update error categories in Errors.move

An alternative approach to #8498
This updates the error categories in Errors.move to be framework agnostic*. This adds a remapping of the new framework-agnostic names in the Errors module to old names to preserves backwards compatibility since these names occur in the output responses to json-rpc responses.
This is another option for the Move/Diem separation, while still maintaining backwards compatibility and a single source for error categories.
A remapped error will look something like this (note the comment in the last sentence of the category description is added automatically for all remapped categories):
Category:
  Name: REQUIRES_CAPABILITY
  Description:  The signer of a transaction does not have a required capability. Name remapped from FRAMEWORK_ONLY_REQUIRES_CAPABILITY to REQUIRES_CAPABILITY.
Reason:
  Name: EACCOUNT
  Description:  The `DiemAccount` resource is not in the required state

We needed to keep requires_capability for backwards compatibility reasons for now, but this might be able to be removed in upcoming changes to the framework.

@vgao1996 Not sure if you still want to split the error module or not. If you are, the approach in this PR is probably the most principled of the options. If not I'll close out this PR.


Enhancement: Improve the dropdowns throughout the plugin.

Is your feature request related to a problem? Please describe.
Currently the dropdown for searching tokens will always show max 6 results. If you have a large number of tokens, this can make it hard to find the token you're looking for.
Describe the solution you'd like
Make the dropdown larger, so more tokens are shown in one view.
Additional context
Add any other context or screenshots about the feature request here.

Blocked by #1265

Closing this as #1203 should solve this already.


How easy is for Alamofire users to integrate Vinyl

I am unsure if it's quite straightforward, to do this or not, as it stands. I need to actually use Alamofire.

I don't think it's possible, since you deal with the Alamofire's singleton the whole time.  Plus the inner NSURLSession it's constant. https://github.com/Alamofire/Alamofire/blob/d409ef6e89bcb98092943492acea81670a155180/Source/Manager.swift#L85 :-1:

via @dmcrodrigues it's possible with:
    public init?(
        session: NSURLSession,
        delegate: SessionDelegate,
        serverTrustPolicyManager: ServerTrustPolicyManager? = nil)
    {
        self.delegate = delegate
        self.session = session

        guard delegate === session.delegate else { return nil }

        commonInit(serverTrustPolicyManager: serverTrustPolicyManager)
    }

Can you elaborate on this a bit? I'm not able to get it working.

Yeah, I don't think that'll work because a Turntables session is nil by default, and the SessionManager's init checks if the session's delegate and the SessionManager's delegate are the  same.

Ok. I'm just trying to get a definitive answer if I can use Vinyl for testing with Moya (Alamofire) and there seems to be some conflicting information.

@damianesteban I will come up with an answer soon. I need to investigate a bit more. I haven't used Vinyl in a while. 😖 Is that cool?

Yes of course 👍 . Thanks for looking into the issue.


Fix workflowlevel2sort object permission

Purpose
A user should be able to create and edit workflowlevel2sort objects.
Related issue: TolaDataV2/#574

Coverage increased (+0.1%) to 53.561% when pulling 46f4ce487fb4f89f1faa23a0b65c7d498b802b15 on jefmoura:fix-workflowlevel2sort-permission into f825f5989d5bcf91f3e29c8149eac3c42f46ef14 on toladata:dev-v2.


Replace my.cdn in code sample with example.com

The code samples in Usage with CDN is not clear enough. One might copy paste it and expect to work. Which would not, since the links are placeholders and needs to be replaced with actual entries from CDN.
This PR aims to make it little more obvious that, they are example links.
I used example.com because,

It is a reserved domain for such cases, more info in this wiki page.
It would provide a nice feedback to the user while reading the code it self.
Visiting the  urls won't result in an error, and give a page stating that it is used for example purpose.

As a bonus, am wondering if this also future proofs it, in case somebody do register my.cdn and use the link for malicious purpose.

The other idea I had when I was doing this last night is to use Handlebars syntax {{cdn}} or something like that so it's very clearly a placeholder/variable.

So something like<EMAIL_ADDRESS>Sounds good. It's short and unambiguous.

@RunDevelopment I have done the changes requested, please review :)

Thank you for contributing @d34dman!


brainstorm 2016 issue/advocacy campaign ideas

based on previous doc with Trilogy

think about connections to copyright week, fair use week, open ed week, oa week, possibly more weeks


Added Namespace property to UIRoot

Hello,
I had an idea to make CodeBehind layers easier. If a "Namespace" property is added to UIRoot, then you can specify the namespace to generate the code in - E.g.
<ek:UIRoot
     ...bla bla bla...
     Namespace="MyApp.MonoGame.UI.Core"

Then, the generator generates a MyApp.MonoGame.UI.Core.MyUIPage class rather than an EmptyKeys.UserInterface.Generated.MyUIPage class, which makes the CodeBehind match up a bit better.

Hmm nice :) I can see that could be useful in some scenarios. Tho you would have to add using to Generated namespace in case of merged dictionaries.
What about bit more generic solution, another optional command line option perhaps?

You're right, that probably would work better. I originally tried to hook into the x:Class (auto-generated by VS) but couldn't find a way to access it, so this is almost as good.
How does this look? (I also did a slight modification to print all possible RenderMode values rather than hard coding the "SunBurn/MonoGame" value)

looks good, I will include that in next major version

Well, I was trying to make a separate pull request for this, but here's as good as anywhere.
EmptyKeys does not have support for Label controls. To counteract this, I modified the generator to emit TextBlocks from Labels. Right now, this is a simple conversion and isn't perfect, but it's better to support Label controls in some way than not at all.

To be honest there is no reason to support Label. Labels were old way from WinForms and pretty much died (no label control in new XAML like Win Store etc.). If you need it, because you are porting some old XAML then it's OK, but I won't support Label in lib.

Reverted. :)

feature added, tho I had to merge it locally. Closing :)


Kubeadm on Autoscaling Groups.

My bad. I meant to use my fork as upstream. Anyway, I leave this here to see if you are interested.

+1 for the PR.

Hi @christopinka, thanks for the PR, we will look into this and consider it for the next release.


Update rules_go and gazelle

closes #130

In .travis.yaml you will also have to change the version of Bazel we are building with to something modern. WDYT about using 0.16.1?

@achew22 I've updated the PR regarding your comments. Also because I'm running NixOS as my linux distribution, the e2e package does not work for me because it downloads a bazel binary, but binaries do not simply run on NixOS, can you please try running it on your end to make sure it worked?
$ bazel test //ibazel/...
INFO: Build options have changed, discarding analysis cache.
INFO: Analysed 11 targets (37 packages loaded).
INFO: Found 8 targets and 3 test targets...
INFO: Elapsed time: 1.505s, Critical Path: 0.10s
INFO: 0 processes.
INFO: Build completed successfully, 1 total action
//ibazel:go_default_test                                        (cached) PASSED in 0.2s
//ibazel/command:go_default_test                                (cached) PASSED in 3.3s
//ibazel/output_runner:go_default_test                          (cached) PASSED in 0.0s

Executed 0 out of 3 tests: 3 tests pass.
INFO: Build completed successfully, 1 total action
$ bazel build //ibazel/...
INFO: Analysed 11 targets (0 packages loaded).
INFO: Found 11 targets...
INFO: Elapsed time: 0.263s, Critical Path: 0.01s
INFO: 0 processes.
INFO: Build completed successfully, 1 total action
$ bazel run //:gazelle
INFO: Analysed target //:gazelle (23 packages loaded).
INFO: Found 1 target...
Target //:gazelle up-to-date:
  bazel-bin/gazelle.bash
INFO: Elapsed time: 0.338s, Critical Path: 0.01s
INFO: 0 processes.
INFO: Build completed successfully, 1 total action
INFO: Build completed successfully, 1 total action
$ bazel test //...
ERROR: /code/opensource/stories/bazel-watcher_update_rules_go/src/github.com/bazelbuild/bazel-watcher/e2e/live_reload/BUILD:5:1: no such package '@build_bazel_bazel_0_14_1//': Traceback (most recent call last):
        File "/home/kalbasit/.cache/bazel/_bazel_kalbasit/0b6768f9b6f944595468314c7a019f08/external/com_github_bazelbuild_bazel_integration_testing/tools/repositories.bzl", line 48
                _extract_bazel(rctx)
        File "/home/kalbasit/.cache/bazel/_bazel_kalbasit/0b6768f9b6f944595468314c7a019f08/external/com_github_bazelbuild_bazel_integration_testing/tools/repositories.bzl", line 39, in _extract_bazel
                fail(("`bazel version` returned non z...)))
`bazel version` returned non zero return code (256): java.io.IOException: Cannot run program "/home/kalbasit/.cache/bazel/_bazel_kalbasit/0b6768f9b6f944595468314c7a019f08/external/build_bazel_bazel_0_14_1/bin/bazel-real" (in directory "/home/kalbasit/.cache/bazel/_bazel_kalbasit/0b6768f9b6f944595468314c7a019f08/external/build_bazel_bazel_0_14_1"): error=2, No such file or directory and referenced by '//e2e/live_reload:go_default_test/bazel0.14.1'
ERROR: Analysis of target '//e2e/live_reload:go_default_test/bazel0.14.1' failed; build aborted: no such package '@build_bazel_bazel_0_14_1//': Traceback (most recent call last):
        File "/home/kalbasit/.cache/bazel/_bazel_kalbasit/0b6768f9b6f944595468314c7a019f08/external/com_github_bazelbuild_bazel_integration_testing/tools/repositories.bzl", line 48
                _extract_bazel(rctx)
        File "/home/kalbasit/.cache/bazel/_bazel_kalbasit/0b6768f9b6f944595468314c7a019f08/external/com_github_bazelbuild_bazel_integration_testing/tools/repositories.bzl", line 39, in _extract_bazel
                fail(("`bazel version` returned non z...)))
`bazel version` returned non zero return code (256): java.io.IOException: Cannot run program "/home/kalbasit/.cache/bazel/_bazel_kalbasit/0b6768f9b6f944595468314c7a019f08/external/build_bazel_bazel_0_14_1/bin/bazel-real" (in directory "/home/kalbasit/.cache/bazel/_bazel_kalbasit/0b6768f9b6f944595468314c7a019f08/external/build_bazel_bazel_0_14_1"): error=2, No such file or directory
INFO: Elapsed time: 2.182s
INFO: 0 processes.
 I'm getting the dreaded This branch cannot be rebased due to conflicts which is preventing the merge button. Do you have any idea what is causing that?

I don't see that on my end, try reloading the page sometimes Github retains bad state. The PR is up to date with the current master.

Huh, I'm getting this

@achew22 ok let me squash the commits then, it might help.

Good idea. Thanks for doing that

@achew22 I squashed the commits and added you as a Co-Authored-By which seems to have pissed off the googlebot, I hope you can still merge, can you please try again?

As the admin for the project I am happy to override the CLA bot given that it was me trying to be helpful that broke it. Let's make sure the tests pass then I'll merge you. Thanks so much for your contribution!

Of course, thank you for your quick feedback!

I'm a quick reviewer when I'm not off the grid :wink:


CREATE ROUTE TO GET ALL COMPANY GOALS.

◾Create a route to get ALL company goals from the API.
◾Send the response in Json format to the client.
◾This route can be accessed by Guests and Admin.

I I am working on this

Assign this to me

live link: https://goals.zuri.chat/api/v1/goals
A little UI we built to test out our endpoint
Link to test: http://test-my-api-endpoint.herokuapp.com/61349ad779f1c23b00c15559


the LOST last frame in VQ MotionDataset.

In the getitem method of the MotionDataset, I found that the LAST frame of each motion segment cannot be retrieved.
For example, in the first motion, self.cumsum[1] = 129. When item=129, there is idx = 128 since idx=item-self.cumsum[motion_id=0]-1, resulting in the motion segment being [128:128+64] = [128:192], while in fact, the length of this motion should be 129+64=193 according to this line. This issue occurs with every subsequent motion segment. Should I submit a PR to fix this problem?

This issue can be fixed as follows:
self.lengths.append(motion.shape[0] - opt.window_size)
self.lengths.append(motion.shape[0] - opt.window_size + 1)  # Fixed

Cool! Thanks for your careful fix. To keep the code consistent, I might not be able to update this.


Update taxonomy fix

Q
A

Bug fix?
yes

New feature?
yes

New sample?
no

Related issues?
n/a

What's in this Pull Request?
This PR started with me looking into fixin the issue of new child terms not being added to existing terms on incremental deploys (you would have to delete any previously provisioned term in order to get new child terms added)
In the process i re-wrote the provisioning code for TermGroups as the old one was hard to figure out and wrap your head around.
The new code does add more importance to the term id's than before. say you have a term previously provisioned with a template. at some point a term store admin has moved the term somewhere else. In your template you have added some new child items to said term, on the next deployment those terms should be added to the existing term, no matter where its located now (as long as the term id is intact).
Have a look @VesaJuvonen @erwinvanhunen @PaoloPia and others and let me know if this is a viable upgrade to the existing TermGroup provisioning code.

Hi Ole,
Looks like a useful update and we'll have a look at it. Given the holidays this will most likely not be before the next master merge.

Hi Ole Kristian,
Your PR looks really good and refactoring the ObjectTermGroups handler really makes sense. However, due to some code-conflicts and to the fact that this handler is really critical and fundamental in the provisioning engine, we decided to test it deeply and most likely we will merge it in the October release. Thanks for your cooperation and efforts.

Current code base throws an exception if you try to re-run a template containing a set of terms that already exists. This is a meaningless ALM pattern, not being able to rerun a template on top of a site. At least it should be possible to parameterize that overwrite/ignore existing is allowed. Seems like this refactoring with checking/ensuring existing terms is more in the right path. Maybe some improvements to the upcoming release?

Hi Ole Kristian,
I'm sorry but due to the conflicts in code it is becoming harder than expected to evaluate and properly test this PR. We are still working on it, and we will not be able to merge it in the next master merge (which will happen today). But we will keep on working on this PR and try to merge it next month. Thanks for your patience. It is a matter of resources.
Thanks,
Paolo

Paolo,
I appreciate the effort you guys are putting into this, but I understand
this is a moving target and that a lot has happened since i submitted the
PR. I don't want you to spend more time trying to merge this than i did
writing the code. If you are interested, I can probably make another go at
this and create a new PR in the near future, or we can just shelve this
idea if its not something that's too interesting.
Either way is totally fine by me :)
Thoughts?
On Sun, Nov 6, 2016 at 5:16 PM, Paolo Pialorsi<EMAIL_ADDRESS>wrote:

Hi Ole Kristian,
I'm sorry but due to the conflicts in code it is becoming harder than
expected to evaluate and properly test this PR. We are still working on it,
and we will not be able to merge it in the next master merge (which will
happen today). But we will keep on working on this PR and try to merge it
next month. Thanks for your patience. It is a matter of resources.
Thanks,
Paolo
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OfficeDev/PnP-Sites-Core/pull/652#issuecomment-258691312,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKYo_BBnomj8fng09DvM7bt_VPGxclGks5q7f16gaJpZM4JBSKD
.

--
Ole Kristian Mørch-Storstein

Hi Ole,
sorry for the small delay (team has been traveling). We would highly appreciate another PR only for the changed pieces, so that we can get it processed more efficiently. We'll make sure that it's processed fast, when it arrives. Thanks for your submission by advance. I'll close this now, due the big conflict. thx.


Fancy Zones Resetting after reboot

Microsoft PowerToys version
0.71.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
FancyZones
Steps to reproduce
After restarting PC Fancy zones resets and I have to choose my layouts again.
✔ Expected Behavior
That I set my zones once and each time I restart the PC that my zones stay as I left them. As it was doing before the last update
❌ Actual Behavior
After each PC restart, I am having to reselect the fancy zones from the launcher.
Other Software
After each PC restart, I am having to reselect the fancy zones from the launcher

Experiencing the exact same issue on 0.71.0. I tried deleting %localappdata%\Microsoft\PowerToys and setting everything up from scratch, but the issue persists.

Same issue. First time user of PowerToys. Installed v 0.71.0 Friday, set up FancyZones layout. Restarted PC this morning and all the layouts are gone.

it's the same for me, the fancyzones are reset each time powertoys is restarted

I have the same issue about daily, see this comment for more info. The layouts are still there, but my two external monitors have their layouts swapped, so I have to reselect them.
Updated PowerToysReport: PowerToysReport_2023-07-12-07-36-17.zip
@SeraphimaZykova found something in 0.70.0 and made improvements in 0.71.0, but I don't experience any improvements, unfortunately.

I have the same issue.
Disabling en enabling FancyZones (in the PowerToys settings) restores my FancyZones layout.

When PowerToys autostarts at boot, I get the default zones. If I manually disable and re-enable FancyZones or restart PowerToys, the correct zones are applied.

If I'm understanding you correctly, a temporary workaround is disabling PowerToys "Run at startup". After PC restarts, launch PowerToys and prior zones are applied?

/bugreport

PowerToysReport_2023-07-13-11-22-18.zip

+1 to this bug. Happens every time PT starts up.  I have to manually enable my custom layout.

PowerToysReport_2023-07-14-00-02-27.zip

/dup https://github.com/microsoft/PowerToys/issues/16370

/dup #16370

This is not a duplicate of this old issue, as it still worked for me in version 0.68.x. Only after the update to 0.72.0 I have a similar problem.
Fancy Zones does not work after reboot. When key combination is used to activate, the default template becomes active and only after deactivating / activating Fancy Zones does it work as expected.

This is not a duplicate of this old issue, as it still worked for me in version 0.68.x. Only after the update to 0.72.0 I have a similar problem.

@GitHub-Mike would you like to share a /bugreport so we can check if it's the same issue?

This is not a duplicate of this old issue, as it still worked for me in version 0.68.x. Only after the update to 0.72.0 I have a similar problem.

@GitHub-Mike would you like to share a /bugreport so we can check if it's the same issue? Thank you!

Unfortunately no. The problem occurred after the update from 0.68.x to 0.72.x. After the reboot Fancy Zones was no longer active and my Custom Zone Layout could not be activated either. I then deactivated and reactivated Fancy Zones and since then it works without problems.
Maybe it was an update problem?


Convert deprecated InterpolatedPropertyDefinition to Expr.

Codecov Report

Merging #963 into master will increase coverage by 0.04%.
The diff coverage is 60%.

@@            Coverage Diff             @@
##           master     #963      +/-   ##
==========================================
+ Coverage   47.88%   47.93%   +0.04%
==========================================
  Files         228      239      +11
  Lines       21657    21882     +225
  Branches     4944     4979      +35
==========================================
+ Hits        10371    10489     +118
- Misses      11286    11393     +107

Impacted Files
Coverage Δ

@here/harp-datasource-protocol/lib/Expr.ts
89.84% <100%> (+0.35%)
:arrow_up:

...atasource-protocol/lib/InterpolatedPropertyDefs.ts
56.66% <48%> (-43.34%)
:arrow_down:

.../harp-datasource-protocol/lib/StyleSetEvaluator.ts
84.03% <87.5%> (-0.05%)
:arrow_down:

@here/harp-materials/lib/Utils.ts
20% <0%> (-80%)
:arrow_down:

@here/harp-materials/lib/IconMaterial.ts
44.44% <0%> (-55.56%)
:arrow_down:

@here/harp-map-controls/lib/Utils.ts
46.15% <0%> (-23.85%)
:arrow_down:

@here/harp-omv-datasource/lib/OmvTiler.ts
50% <0%> (-16.67%)
:arrow_down:

@here/harp-utils/lib/Logger/ConsoleChannel.ts
38.46% <0%> (-11.54%)
:arrow_down:

@here/harp-utils/lib/ObjectUtils.ts
83.33% <0%> (-10.22%)
:arrow_down:

@here/harp-utils/lib/UrlResolver.ts
91.66% <0%> (-8.34%)
:arrow_down:

... and 29 more

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 0c48613...8281810. Read the comment docs.


IronMQ's package just had a backwards-incompatible change

See here: https://github.com/iron-io/iron_mq_php/releases
The only real change seems to be that they're now using namespaces and PSR-4 autoloading. The naive fix for this would probably just be referencing the new IronMQ\IronMQ and taking out the old IronMQ. I started putting together a PR to do this, but then realized that it may not be ideal, especially for people who don't want to upgrade their Iron packages.
So I'm opening this issue so that we can decide on a common approach if it's decided that one is needed. In any case, we'll need to change the section of the docs that defines the Iron dependency.

We'll be updating in L5.1.

We only support ~1.5 in L4.2 and L5.0. See https://github.com/laravel/framework/blob/5.0/composer.json#L99.

Cheers, thanks @GrahamCampbell


Tauri example postgres

A part of the tauri example. This is a WIP, more changes are needed.
To load a dump of data into the db, download this  https://www.dropbox.com/s/oqzy4igb89fi953/data.zip?dl=1 and extract it to ./examples/tauri-postgres/db/data/. then run:
ISSUES_TO_LOAD=2000 pnpm run db:load-data

👋 we've been working the last month on a rebuild of the Electric server over at a temporary repo https://github.com/electric-sql/electric-next/
You can read more about why we made the decision at https://next.electric-sql.com/about
We're really excited about all the new possibilities the new server brings and we hope you'll check it out soon and give us your feedback.
We're now moving the temporary repo back here. As part of that migration we're closing all the old issues and PRs. We really appreciate you taking the time to investigate and create this improvement!


Roll engine ff1bcdc00941..a6b8e70ad53a (3 commits)

https://github.com/flutter/engine/compare/ff1bcdc00941...a6b8e70ad53a
git log ff1bcdc00941188ba27123a4857b443e3c9392bb..a6b8e70ad53a730c72420757e5e419ccf8321a31 --no-merges --oneline
a6b8e70ad Remove unused variable (flutter/engine#8490)
377dc6400 Roll src/third_party/dart 86b5304ccd..8e0a669e74 (6 commits)
7ee3a31c5 Roll src/third_party/dart 907c514c89..86b5304ccd (27 commits)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (jsimmons@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/6588980135985152

Commit queue failed; closing this roll.


[Question] Lower audio quality for mobile phone

Hi there, I am using wavesurfer just to display the waveform and fill it out while the song plays. I am actually setting it mute when it start playing because the song is playing in another audio component, not the wavesurfer component and I have not found a good way to integrate them, so I actually mute the wavesurfer song. The issue with this is that it loads 2 files at the same time, and I don't need wavesurfer to actually play the song in good quality, it can be the lowest quality possible because it is muted anyawys. Is there a way to do that?

can you describe the problem with some minimal code, what you have tried etc

no feedback, closing.


handling of transparancy

I've just been playing with your library and it kept telling me that images were very different when they looked the same.  It seems that it was due to the color present in transparent areas.
For example; if I create two images as:
from PIL import Image, ImageDraw

# greyscale image, completely white and also completely transparent
im1 = Image.new('LA', (100, 100), (0xff, 0))

im2 = im1.copy()
# replace most of it with "transparent black"
ImageDraw.Draw(im2).rectangle((10, 10, 90, 90), (0, 0))

and then run dhash on them:
from dhash import dhash_row_col
print(dhash_row_col(im1), dhash_row_col(im2))

I get very different results, even though they'll be rendered the same as far as most viewers are concerned.  They're obviously "different" images and you can use an image editor to pull the channels apart and show this.  But given that this library is about perceptual differences I think it should probably do something about this.
I'm working around this by filling transparent areas with white, as suggested in:

https://github.com/python-pillow/Pillow/issues/2609#issuecomment-313922483

def fill_transparent(image, fill_color='white'):
  if image.mode in ('RGBA', 'LA'):
    cleaned = Image.new(image.mode[:-1], image.size, fill_color)
    cleaned.paste(image, image.split()[-1])
    return cleaned
  return image

print(*[dhash_row_col(fill_transparent(im)) for im in [im1, im2]])

Not sure if this is deliberate, or if/how it should be exposed to users of the library — hence opening an issue rather than pull request.

This is an interesting issue, thanks. Yeah, I think putting that conversion logic inside get_grays would make sense. I probably won't get to this anytime soon, but if you'd like to submit a PR for it, that'd be good. It be good if it also worked for wand/ImageMagick.

OK, I'll see what I can do.  I've forked a copy that I've done the PIL stuff in.
I've not used Wand before, so might take a while to figure the API out
Will submit a pull request when I've got something sensible together…


Set up automatic deployment to PyPI

See: https://docs.travis-ci.com/user/deployment
Note that this will try to publish for all tags.  This is probably fine if we only use tags for releases.  If more control is needed, see the "conditional deployment" section.
CC @seibert (and see your TODO).  I guess I could do this too, but I'll let you store the token ;)

looks good.  going to merge and commit encrypted token directly to master


What did you expect to see?
go.work.sum file unchanged, because the -mod=readonly flag was given.

@matloob

Related Issues

cmd/go: go get -u yields a different result compared to go get on a clean go.mod #38362 (closed)
x: keep versions syncronized #37840 (closed)
Is that correct that "go get -u" inside a module directory will break dependent package's go.mod requirement? #33465 (closed)
cmd/go: livelock when computing module graph in a workspace with GOPROXY=off #53874 (closed)
cmd/go: 'go mod tidy' retains sums for modules in the requirement graph but not the build list #39424 (closed)
x/vgo: possible non-determinism in updating go.sum #26310 (closed)
cmd/go: line added to go.sum with no change in requirements #36667 (closed)
cmd/go: git shallow fetches broken at CL 556358 #66147 (closed)
cmd/go: duplicate lines for module in go.sum #28456 (closed)
go mod download modifying go.sum after go mod tidy #44512 (closed)

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

We did this intentionally, though I agree it's a bit awkward and we might potentially need to add a feature to help users with checked in go.mod files.
Since -mod=readonly is the default mode for build commands, the normal workflow in a single module, outside of a workspace, is to run go get/go mod tidy to make required changes to go.mod files and also update go.sum files before doing builds that require those changes.
If the -mod=readonly behavior also made go.work.sum read only, another command would have to be run after go get and go mod tidy change go.mod files to ensure that go.work.sum contains all the relevant sums. (We wouldn't want go get and go mod tidy to make the modifications themselves because those commands operate on a single module level and a single module can participate in multiple workspaces) This command would add significant friction to users of workspaces for not much benefit.
The most important goal of the mod=readonly mode (as I understand it) is that doing operations like builds and lists should not (surprisingly) change the set of requirements of a module in turn changing the actual code that's built. The operation that changes the build is put into into a separate command so that it is done explicitly. Since go.sum is also checked in, and only needs to be changed when module dependencies are changed, it makes sense to have the rule apply to both go.mod and go.sum.
When we started working on workspaces, the intention was that go.work (and go.work.sum) files would only be checked in in very rare circumstances. We still encourage users to consider carefully if they should check in go.work files because it does cause a lot of headaches, but it seems now like we need to address some of those headaches.
I do think that the current behavior is the best behavior for most users, but perhaps we should add a command (maybe called go work tidy that will ensure that the go.work.sum file is complete for the all pattern). Then CI could check that that command doesn't make any changes, and in the use case where you clone oscar and do a build there would be no modifications to go.work.sum because it would already be updated?

I'll close the issue for now. Please reopen it if you want to continue the discussion.


feat: port Geometry.Euclidean.Basic

I already have the entire file fixed; I'm going to push.

@Parcly-Taxel thanks, but I'd suggest you focus on porting files that are not already being worked on


Ball_out goal kick automated test.

This PR is aimed to automatically test the AutoRef's behavior when the ball leaves the field from the opponent's goal line. It relates to issue #17.

Merged after a few clarifications.


New congress score indexing

Accounts for new scorecard format where name column is split into first name, last name
Sets invalid total percentages to N/A

@kokonakajima added one more commit to get the 'Members no longer in Congress'. Also updated the private data to make it easier to merge.

@dphiffer Omg I was trying to figure out why the new commits weren't picking up John McCain's votes... turns out the two scoresheets for session 115 label that bottom section a little differently, one is Members no longer in office and the other is Members no longer in Congress - resigned/left office before end of session. Is that something we should just update in the source spreadsheet, and redownload? Or account for those other labels in the script?
also wow this emoji! 📇

@kokonakajima oh no! yeah I think relabeling it is the way to go. In fact, I just went and did that. Will post a message about it on Slack.

(also updated the elections-api-private data and submodule)

ah @dphiffer (hopefully!!!) one last thing, I think those legislators at the bottom need ACLU IDs, because I'm still not getting John McCain's score sessions in my local endpoint

ohhhh yes, thank you!


Refactor fix-workflow-db to use singledispatch

Refactor the _translate_chars to use single dispatch with the functools.singledispatch decorator on Python 3 or the singledispatch package on Python 2.

Just a heads up - as per my research into performance related issues with large documents - this piece of code (translate chars) is quite performance critical and adds substantial overhead on large documents (executions) with many keys so we need to make sure we are not introducing any additional overhead :)

This is neat trick.

I have some rough performance profile results.
Methodology
Housekeeping
I only ran the code under Python 3:
$ python --version
Python 3.8.1

First I made sure that the profile code was testing out every statement and branch of mongoescape.py:
PYTHONPATH=.:st2common coverage run --branch --include=st2common/st2common/util/mongoescape.py --concurrency=eventlet profile_mongoescape.py && coverage annotate && coverage html

Then I ran the code with cProfile:
PYTHONPATH=.:st2common python -m cProfile -s cumtime profile_mongoescape.py

I profiled both branches with 10000 iterations and 1000000 iterations to try to amortize any overhead, and only looked at the cumulative time spent in unescape_chars and escape_chars.
I also profiled the code after @m4dcoder's rebase to use the previous explicit for loop instead of the list comprehension.
Results
unescape_chars - cumulative time
(lower values are better)

for loop
10000x
1000000x

List comp
10000x
1000000x

Explicit switches
1.163
121.389

1.326
133.674

Single dispatch
1.587
161.334

1.779
170.857

------
--------

---------
------

Relative
36.457%
32.907%

34.163%
27.816%

escape_chars - cumulative time
(lower values are better)

for loop
10000x
1000000x

List comp
10000x
1000000x

Explicit switches
0.887
91.946

1.000
100.790

Single dispatch
1.229
124.071

1.361
130.67

------
--------

---------
------

Relative
38.557%
34.939%

36.1%
29.646%

Conclusion
Unfortunately, as cool as it is, using functools.singledispatch under Python 3 substantially (approximately 33%) slower than simply explicitly coding the field type switches.
Because this code needs to be fast more than it needs to be simple to understand, I'll be closing this PR.
Code
profile_mongoescape.py
from st2common.util.mongoescape import escape_chars, unescape_chars

for i in range(1000000):
    escape_chars(4)

    escape_chars("four")

    escape_chars("dot.notation")

    escape_chars("dollar$sign")

    escape_chars({
        "untouched": "untouched value",
        "dot.notation": "dot.notation",
        "dollar$sign": "dollar$sign value",
    })

    escape_chars({
        "ne$ted_dict": {
            "untouched": "untouched value",
            "dot.notation": "dot.notation",
            "dollar$sign": "dollar$sign value",
        },
        "nested_dict": {
            "untouched": "untouched value",
            "dot.notation": "dot.notation",
            "dollar$sign": "dollar$sign value",
        },
    })

    escape_chars([{
        "untouched": "untouched value",
        "dot.notation": "dot.notation",
        "dollar$sign": "dollar$sign value",
    }, {
        "untouched2": "untouched2 value",
        "dot.notation2": "dot.notation2 value",
        "dollar$sign2": "dollar$sign2 value",
    }])

    unescape_chars(4)

    unescape_chars("four")

    unescape_chars(u"dot\uFF0Enotation")

    unescape_chars(u"dollar\uFF04sign")

    unescape_chars({
        "untouched": "untouched value",
        u"dot\uFF0Enotation": u"dot\uFF0Enotation value",
        u"dollar\uFF04sign": u"dollar\uFF04sign value",
        "literal dot.notation": "literal dot.notation value",
        "liternal dollar$sign": "literal dollar$sign value",
    })

    unescape_chars({
        "ne$ted_dict": {
            "untouched": "untouched value",
            u"dot\uFF0Enotation": u"dot\uFF0Enotation value",
            u"dollar\uFF04sign": u"dollar\uFF04sign value",
            "literal dot.notation": "literal dot.notation value",
            "liternal dollar$sign": "literal dollar$sign value",
        },
        "nested_dict": {
            "untouched": "untouched value",
            u"dot\uFF0Enotation": u"dot\uFF0Enotation value",
            u"dollar\uFF04sign": u"dollar\uFF04sign value",
            "literal dot.notation": "literal dot.notation value",
            "liternal dollar$sign": "literal dollar$sign value",
        },
    })

    unescape_chars([{
        "untouched": "untouched value",
        u"dot\uFF0Enotation": u"dot\uFF0Enotation value",
        u"dollar\uFF04sign": u"dollar\uFF04sign value",
        "literal dot.notation": "literal dot.notation value",
        "literal dollar$sign": "literal dollar$sign value",
    }, {
        "untouched2": "untouched2 value",
        u"dot.notation2": "dot.notation2 value",
        u"dollar$sign2": "dollar$sign2 value",
    }])

Thanks @blag for digging deeper and @Kami for raising the performance concerns.


Using a dictionary instead of an object?

The following easily fails:
const tlru = require('tiny-lru');
const cache = tlru();
cache.get('toString');

and this is a common issue when objects, instead of dictionaries, are used in conjunction of the in operator.
In NodeJS, they've benchmarked that the best way to go is:
function Dict() {}
Dict.prototype = Object.create(null);

Once you have that, instead of this.items = {}; you'd go this.items = new Dict; and you grant no name clashing through the prototype could possibly ever happen.
Thoughts?

Not sure I'm following, but the proposed changes fixes a bug and for the rest would work the same.
This issue is well known and documented, in NodeJS query string parser the __proto__ attack was causing disasters and here you'd have the same.
It's really not an option to use literals like dictionaries.
Use a Nap instead, or go real dictionary

So this is not based on performance, but instead the prototype chain? I'm not interested in any such change; Maps are too slow for a cache.

you have at least 12 breaking keywords for this cache
"toSource"
"toString"
"toLocaleString"
"valueOf"
"hasOwnProperty"
"isPrototypeOf"
"propertyIsEnumerable"
"__defineGetter__"
"__defineSetter__"
"__lookupGetter__"
"__lookupSetter__"
"__proto__"
"constructor"

plus others potentially introduced by libraries extending, for whatever reason, the Object prototype.
There is a bug, and it looks like you are not willing to fix it.
I guess a fork is the next step.
Regards

Guess how many issues have been filed on those words. The scenario you're considering doesn't block you from someone already overriding Object.create(); this is a straw man.

The scenario you're considering doesn't block you from someone already overriding Object.create();

if executed once and the module is early required it's safe.
I don't understand why anyone would like a bug super easy to solve in their code.
NodeJS core team fixed their objects used as cache like you are doing here, because there is a precedent on the issue created.

Zero issues filed; if you don't sanitize keys in your code you're doing it wrong.

This is also core design of the language, while Object.create() was created for a single selfish reason; if you don't trust the prototype chain don't run javascript.

if you don't sanitize keys in your code you're doing it wrong.

I am using a cache, which should act like one. There is a bug in your code, feel free to ignore it.

Maps are too slow for a cache

This is also quite a bold statement, as Maps these days can be even be faster.

while Object.create() was created for a single selfish reason; if you don't trust the prototype chain don't run javascript.

Nobody said you don't trust the Object prototype, the bug is about the existing prototype already.
Also, the moment __proto__ passes through, goodbye your cache.
Once again, it's your project, I'll just try to be sure I won't include it anywhere, since this conversation makes no sense for such popular module.
Best Regards

The "bug" is the language. By using those strings as keys you'd block access to the prototype methods of Object; no class methods utilize those keys, so the scenario you're describing is outside of the bounds of what this library is concerned with.
If the dictionary mode solves the deopt, I'm super interested in it. If you don't know what this is, you're not really looking at how it works.

By using those strings as keys you'd block access to the prototype methods of Object

No, the module fails, as already described in the first post.
const tlru = require('tiny-lru');
const cache = tlru();
cache.get('toString');

do you understand there is an error there that shouldn't, since any other key wouldn't fail?

If that's a real world problem, open a PR. Its impossible in my work so I don't care to do it. I'm not going to block you from landing it.

I would've landed a PR after a normal conversation, as I've already told you what I would've done differently, and asked your opinion about it.
But after all this, I'm now convinced me I shouldn't care about this module and actually stay away from it.
What a sad interaction.
Best Regards.

This is a normal conversation, from my perspective. You asked a question, and I answered truthfully, and provided my reason.
The entire outcome is by your doing; other people simply open PRs and describe why it's useful, not present a language design as a bug.

Just to be clear:

using an object literal as trash bin has never been a good idea in general, that is why Map was introduced in the language
assuming you can be careless and use the object literal, you should not use the in operator, because it reveals prototype chain details
even using items.hasOwnProperty would be a bad choice, 'cause hasOwnProperty could be used as cache, so that having hasOwnProperty.call(items, key) would be at least safer
because of all these gotchas described in previous points, they introduced Object.create(null) to exactly address all these common hidden caveats with objects literals
as result, you are using the wrong primitive to obtain unsafe, easy to break,

I've been working and contributing in Open Source for 20 years, and just landing PRs is not usually welcome, because devs might have reasons to pick one approach instead of another.

Best Regards.

^ could've opened the PR in that time frame. You're hung up on me not caring as much about a potential issue as you do. Your bias is only your's.

The truly telling thing for me is you'd rather argue & fork, then make the 1 liner.

@avoidwork I read the conversation and I'm a bit shocked. @WebReflection found a neat bug and even provided a solution. What's missing? The bug is obviously a name clashing scenario because of the "in" operator.
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/in#Inherited_properties

Shocked? I was asked for an opinion, and I gave one. Why does this problem even exist in your code? What's the scenario that this actually manifests? I can randomly delete attributes or methods and break the cache the same.

Shocked? I was asked for an opinion, and I gave one. Why does this problem even exist in your code?

It doesn't exist but that's not the point. This issue was opened to prevent it.

What's the scenario that this actually manifests? I can randomly delete attributes or methods and break the cache the same.

@avoidwork don't start hypothetical scenarios when this issue already happened in our community. We are talking about some lines which will improve the overall reliability of this module.

You're free to make the change. You guys are bringing a hypothetical situation to me. I purposefully chose the operator knowing the issue that it could expose. I don't make love that protect devs from themselves, but you're able to make those changes if you want

I'm also gonna lock this; anyone can open a pr if they want to land a change.


Top10 Test Performance chart showing TestData

It would be nice if we display only Test case name any way we are showing Test data in Test metrics.

Agree, Will add new column to display test data (parameters). But I'm worried when same test case is executed multiple time using data provider then as a user I dont know which test case have more failures or  failed due to same name of test


Change configuration from userbeans -> user-beans

https://github.com/jabrena/spring-boot-user-beans/pull/109


Compiles Pillow-SIMD with libwebp support, closes #193

Resolves #193. We compile Pillow-SIMD from source; if libwebp is not present we do not compile support for webp. This changeset fixes this mistake.
Pillow-SIMD compilation feature check: https://github.com/python-pillow/Pillow/blob/9b927155e693e0a337114422af46677991a010b0/setup.py#L659-L668
cc @oneOfThePeople

If you wait a bit, Travis will build new latest docker images which should then fix the webp issue.


HELP approach to minimum data consumption and low cost

Hi @mobizt
One more question:
Once Firebase charges for data download, what is the best approach to use downloading minimum data as possible? (so it can become expensive over time with thousand of devices).
Wich one uses more data download,  Stream or Stream Callback methods?
Which one is better?
best regards,
Danilo

No difference in network data consumption between loop reading the stream and callback.
You need to design your database to minimize the network data usage.  The node that being monitor its changes should be the top level child of the stream path (parent).
Use the short node name, integer instead of string or boolean to reduce the payload data,.
Don't get or set data too frequently, every 1 minute is enough.
Free 1 GB quota limit per month is enough for few devices or tenth of devices depends on your database design and the frequency of acquire/retrieve data.


How to set a column to URL?

PHP version: 7.1.3
Laravel version: 5.6
Package version: 3.0
I'm in the process of migrating from 2.3 to 3.0. I have everything switched over however I have not figured out how to set a column to be a URL. I've read this documentation: https://laravel-excel.maatwebsite.nl/3.0/exports/column-formatting.html and have set up the columnFormats() method as specified but can't figure out how to set the column.
Thanks

I don't think it has do do anything with column formatting.
You'll have to set the hyperlink on the cell:
$sheet->getCell('E20')->getHyperlink()->setUrl('http://www.google.com');

In my scenario my Excel downloads were dynamic the number of rows and columns varied however they all consistently had a link in the last column, I wanted each link to be clickable in excel.  For future reference, this is what I did:
Create a Macro (I created this in the exportable class file):
Sheet::macro('setURL', function (Sheet $sheet, string $cell, string $url) {
	$sheet->getCell($cell)->getHyperlink()->setUrl($url);
});

Add the register events function to the exportable class and register a AfterSheet event
public function registerEvents(): array
{
   //Rows in data array
   $datacount = $this->data->count();
   //How many headers gives us column count
   $headersCount = count($this->headers);
   //Get the name of the last column based on the number of headers, as last column is always a link
   $column = $this->getExcelColumnName($headersCount);

   return [
      AfterSheet::class => function(AfterSheet $event) use ($column, $datacount, $headersCount) {

         //Do iteration for each row, get the column + row and set the column to a link using the cell value
         for ($i=2; $i <= $datacount; $i++) {
            //Get cell value as it's value is always a link
            $cellValue = $event->sheet->getCellByColumnAndRow($headersCount, $i)->getValue();

            //Call the new macro
            $event->sheet->setURL(
	       $column . $i,
	       $cellValue
	    );
         }
      },
   ];
}

@SFPink This is the way I use it. You can refer to it.


Enhance Kinesis consumer

Do not use a separate thread to fetch Kinesis records (this can fix the potential race condition)
Follow the same convention with Kafka:

Return the non-empty message batch immediately
Wait up to timeout if there is no record fetched, then return an empty message batch

Change back the default timeout to 5s
Change the default max records per fetch to 10,000 (Kinesis default)
Remove some unused dependencies

Codecov Report
Attention: Patch coverage is 0% with 41 lines in your changes are missing coverage. Please review.

Project coverage is 0.00%. Comparing base (59551e4) to head (8d1a28f).
Report is 219 commits behind head on master.

Files
Patch %
Lines

...e/pinot/plugin/stream/kinesis/KinesisConsumer.java
0.00%
39 Missing :warning:

...java/org/apache/pinot/spi/stream/StreamConfig.java
0.00%
2 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #12806       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436     2387       -49
  Lines        133233   131080     -2153
  Branches      20636    20269      -367
=============================================
- Hits          82274        0    -82274
- Misses        44911   131080    +86169
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <0.00%> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <0.00%> (ø)

java-11
?

java-21
0.00% <0.00%> (-61.63%)
:arrow_down:

skip-bytebuffers-false
?

skip-bytebuffers-true
0.00% <0.00%> (-27.73%)
:arrow_down:

temurin
0.00% <0.00%> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

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

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


add definitions of EBML Document, Header, and Stream

needs review

This PR starts to tackle some of the things about EBML that are presumed but unwritten. Part of this is inspired by a conversation between myself and @robUx4 at VDD regarding defining EBML Document and EBML Stream so that requirements or constraints could be associated with these concepts individually when appropriate. Please review.

scrapping this PR, will resubmit soon


GCC warning: enumeral and non-enumeral type in conditional expression [-Wextra]

The code is very suspicious:
drivers/gles3/rasterizer_scene_gles3.cpp: In member function 'void RasterizerSceneGLES3::initialize()':
./core/typedefs.h:124:40: warning: enumeral and non-enumeral type in conditional expression [-Wextra]
 #define MIN(m_a, m_b) (((m_a) < (m_b)) ? (m_a) : (m_b))
                        ~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~
drivers/gles3/rasterizer_scene_gles3.cpp:5042:31: note: in expansion of macro 'MIN'
   state.max_ubo_reflections = MIN(RenderList::MAX_REFLECTIONS, max_ubo_size / sizeof(ReflectionProbeDataUBO));
                               ^~~

That code seems correct to me. Maybe RenderList::MAX_REFLECTIONS needs to be cast to int to silence the warning?
The RenderList struct uses an enum as a shortcut to define various int constants. Could maybe be changed to define all as individual constants without using enum if that's not standard.

That code seems correct to me. Maybe RenderList::MAX_REFLECTIONS needs to be cast to int to silence the warning?

Question is whether max_ubo_size / sizeof(ReflectionProbeDataUBO) expression will always match an enum value defined in the anonymous enum? If so, then casting would be fine.

The RenderList struct uses an enum as a shortcut to define various int constants. Could maybe be changed to define all as individual constants without using enum if that's not standard.

Alternative approach would be to define state.max_ubo_reflections as an integer type.
That's

@akien-mga What do you think?

Question is whether max_ubo_size / sizeof(ReflectionProbeDataUBO) expression will always match an enum value defined in the anonymous enum? If so, then casting would be fine.

I mean, if we do MIN((int)RenderList::MAX_REFLECTIONS, max_ubo_size / sizeof(ReflectionProbeDataUBO)); (which is the intended logic), it should work correctly I suppose?

Alternative approach would be to define state.max_ubo_reflections as an integer type.

I think that makes sense. All of the RenderList enum should probably be moved to constants, it seems to only be used as sugar for defining constants within a common scope.
Maybe @reduz can advise on what he'd do to make this cleaner?

Question is whether max_ubo_size / sizeof(ReflectionProbeDataUBO) expression will always match an enum value defined in the anonymous enum? If so, then casting would be fine.

I mean, if we do MIN((int)RenderList::MAX_REFLECTIONS, max_ubo_size / sizeof(ReflectionProbeDataUBO)); (which is the intended logic), it should work correctly I suppose?

Problem is that the enum has "holes", so e.g. value of 999 is not a valid enum value.

Alternative approach would be to define state.max_ubo_reflections as an integer type.

I think that makes sense. All of the RenderList enum should probably be moved to constants, it seems to only be used as sugar for defining constants within a common scope.

Then I would do the casting into integer type.

Maybe @reduz can advise on what he'd do to make this cleaner?


micro:bit 2021 release blog

Import, edit, size images and otherwise prepare the blog post for "micro:bit 2021 Release".
Closes #8273

Adding @abchatra who will make the call on Monday whether we release...

We decided to wait atleast a week before shipping new CODAL & bluethooth bootloader. Will update next ewek.

@eanders-MS @riknoll

@anzhou5 - I read your email from Friday, briefly, but my agency botched my account renewal for FY22 and my laptop logged me out just moments later. I can't reply quite yet over email so maybe post an update here on what you want to do for the tutorial section image. Thanks.

@ganicke - thanks so much for being patient with me. Looks like we can go ahead and add back in the blurb about tutorials being released as they will be ready to go in time. I've attached the release image file on this comment as well!


Goal descriptions are written into game.tw-pddl files

Currently, the goal descriptions are written into game.tw-pddl files. This leads to race conditions during parallel runs.

Sorry for reviving the dead here but is this still an issue @MarcCote ? I want to use ALFWorld as an example for tutorials for NLP/ML/RL. As a part of the tutorial I want to illustrate parallelization via multi-processing. It seems like I might be encountering this race condition @MohitShridhar mentioned here. In particular, the PDDL files get corrupted and then subsequent processes cannot deserialize JSON successfully from the game files. Deleting all data files and redownloading them solves the problem.

Working on a two-step process where users will first call generate_tw_pddl.py to generate the game files, then launch their training experiments on the generated tw-pddl which will remain static.

@vzhong if you want to give the PR (#53) a try to make sure it works with your workflow, that would be appreciated. This is the command you want to use to re-generate the games so they use the human annotations.
python scripts/generate_tw_pddl.py --data_path $ALFWORLD_DATA/json_2.1.1 --save_path custom_dataset/ --goal_desc_human_anns_prob 1

Then you need to change the appropriate settings in the yaml
dataset:
  data_path: 'custom_dataset/train'
  eval_id_data_path: 'custom_dataset/valid_seen'
  eval_ood_data_path: 'custom_dataset/valid_unseen'

Yes I think this works - thank you! Can we please make this a command similar to alfworld-download? this way the user doesn't need to clone the repo and can instead just pip install it? Other than that look great to me!

Done. The command is now
 alfworld-generate --data_path $ALFWORLD_DATA/json_2.1.1 --save_path custom_dataset/ --goal_desc_human_anns_prob 1


[#3186] Manage Access - Vue Instance

This PR follows up on #3186 by implementing a vue instance containing all frontend logic related to Manage Access.
With this work we start seeing the benefits of Vue.js. The code has been simplified significantly and we should see a more reactive UI. All functionality inside the Manage Access modal is now performed via AJAX and no page refresh is required.
Notice that changing resource owners won't yet reflect on the Owners listing on the top left of the page. This will be implemented later on when rewriting logic for this "Highlights" component.
However, changing sharing status does now reflect on other items on the page because this was easy to setup temporarily until the "Highlights" component is implemented.
This work can be tested at https://riverdev.hydroshare.org

Pull Request Checklist:

[x] Positive Test Case Written by Dev

[ ] Automated Testing

[ ] Sufficient User and Developer Documentation

[ ] Passing Jenkins Build

[ ] Peer Code review and approval

Positive Test Case

Create a resource
Test functionality inside Manage Access
Verify that no action refreshes the page and the UI state is handled correctly.

@cuahsimarko, @pkdash - This is now ready for review. Note that I changed some responses in the backend to handle AJAX requests.

@sblack-usu, @pkdash, @cuahsimarko I have now fixed the import issue following @sblack-usu's instructions. Please add your reviews when you get a chance so we can move this work forward. There are still a lot of things to do that depend on this one.

All I was able to find was the close button bug, but not sure if that's present in develop. You should merge develop, now it has the release in it. The Close button on manage access bug could have been pre-existing/not in scope for this PR. I recommend a stakeholder/someone more experienced with the UI have to do a more detailed functional review of the UI. Some light comments in the source, left as well.

retest this, please

@cuahsimarko - The bug with the close button has been fixed. I believe I have addressed @sblack-usu's suggestions.