Adding binary for four phonons thermal conductivity

This commit a new binary for the thermal conductivity, adding four phonon and including single mode, collective and off-diagonal contributions.
The code is less demanding in terms of memory, significantly faster than the previous one and implement a Monte-Carlo double grid sampling to reduce drastically the cost of the scattering computation.
Changes compared with the previous binary (in no particular order)
Added

Four phonon scattering (both in the single mode and collective contribution)
Added the possibility to compute kappa in the classical limit, by changing phonons occupations and heat capacity to their classical values
Use of linear algebra to compute the scattering matrix elements (as for lineshape) for faster computation
Use of irreducible triplet and quartet to reduce the number of scattering element to compute
Imposes more symmetries (for the thermal conductivity tensor, and also the scattering matrix elements). There is no more tiny but spurious xy kappa from the iterative or off-diagonal contributions
Includes a Monte-Carlo integration for the scattering, where it is computed on a subset of the full grid.
Added a lo_fftgrid_help in libolle. This contains the thing related to the Monte-Carlo grid and the functions to get a third or fourth q-point conserving quasi-momentum for a fft grid. I put it here to make it possible to use these tools for other binaries.
In the log, I added a little thing to recap the parameters used in the calculation. I'm also using the lo_timer types to provide a detailed summary of the timing of the different part of the code

Removed

Not compatible with tetrahedron integration. This is due to the Monte-Carlo double grid. It might be possible to implement it, but that's probably a lot of work
Is restricted to one temperature at a time. This comes from the memory reduction scheme, which was necessary for the fourth-order contribution to the scattering matrix. (Shortly, the matrix is directly computed with the linewidth instead of computing the processes and adding temperature prefactors when solving for kappa)
Does not give access to + and - (or ++, +- and --) scattering rates (this is due to the previous point)
The cumulative kappa has not been reimplemented (yet).
There is no skip of processes if the energy difference is larger than 4 times the sigma parameter, all processes are included. This is because when we get to the moment to compute the scattering, all the heavy computation necessary to get the scattering matrix element are already done, so skipping processes would not incur substantial speed

Modified

Also, the scattering processes are computed a bit differently. The old binary uses a Boltzmann equation formulation while here all was derived from a many-body correlation functions approach. Theoretically, both way to compute the scattering are strictly equivalent. However this relies on a perfect conservation of energy, so for underconverged grid, the results may be different. The formulation is also different for the off-diagonal term (due to the new derivation using displacement operators instead of creation/anihilation). In the end, the new derivation using spectral function things, it is more in line with what is happening in lineshape in grid mode. With this, I actually have single mode results agreeing with RTA results from lineshape (not the case before !)
Changed the way the things related to thermal conductivity are written in type_phonons_dispersion. This is because the old version only printed it if p_minus was allocated and, as stated above, we don't have access to it now.
Changed a bit the way the adaptive smearing for Gaussians is computed, with a theoretical justification of the new way (it's in the paper). The new version seems to converge faster
Slightly different way to pretransform the force constants for the linear algebra scheme. Only difference is that the eigenvectors are divided by the square root of the frequencies from the start, thus avoiding some loops to renormalize the force constants afterwards.
The scattering matrix is in matrix form. So the iterative solution for the collective kappa is done slightly differently, but uses BLAS to apply the scattering matrix on the F vectors.

The documentation and test are included for this binary. Also, a note to warn about this new binary has been included in the documentation for the thermal_conductivity binary.
Only thing missing is the link to the paper going with this implementation, that can be added as soon as the preprint is out.

I changed the binary to kubo_transport, as discussed, to avoid confusion with the fourth order and the thermal_conductivity binary.

I had to increase the symmetry tolerance here from lo_sqtol to lo_tol to make it work for my system. Should we make this the default? Are there any possible unwanted side effects? Pinging @ollehellman

Increasing the symmetry tolerance should be fine.

Remaining for me:

[ ] general code review
[ ] test the test
[ ] document high-kappa simulations, ongoing investigation in comparison to ShengBTE/FourPhonon

When the paper is out: Document for which materials thermal_conductivity was tested and benchmarked, and for which we are (yet) uncertain. Reminder --> https://github.com/tdep-developers/tdep/issues/102

One more comment: Now that we have seed, we should test the full 4ph binary. I suggest for tests/thermal_conductivity/Makefile:
testfiles:
        extract_forceconstants -rc2 0 -rc3 3 -rc4 0 --polar
        ln -sf outfile.forceconstant infile.forceconstant
        ln -sf outfile.forceconstant_thirdorder infile.forceconstant_thirdorder
        ln -sf outfile.forceconstant_fourthorder infile.forceconstant_fourthorder
        thermal_conductivity -qg 5 5 5 --fourthorder -qg4ph 3 3 3 --seed 1

which takes 3s on my computer. Alternatively it takes 0.5s with a 2x2x2 4ph grid

Thanks @OrbitalC ! Great development!


[CT-656] [Feature] Allow for merge exclude columns

Is this your first time opening an issue?

[X] I have read the expectations for open source contributors

Describe the Feature
Right now, for incremental models using the merge strategy, we allow users to specify a specific list of merge_update_columns to include in the merge statement.
For wide, denormalized models, it's likely that users may want/need to exclude only a couple of columns, and update all the rest. Right now, this would require the user to manually specify a potentially long list of all columns that need to be updated, rather than a shorter,  more straightforward list of columns not to update.
Once incremental materializations are simplified in the effort to create more streamlined materialization code, this seems like a simple quality of life enhancement!
Describe alternatives you've considered
Create a custom version of get_merge_sql to allow for an additional config merge_exclude_columns to achieve this behavior.
Who will this benefit?
This will be beneficial for users who have wide incremental models, and may need tight controls over the merge behavior on those tables.
Are you interested in contributing this feature?
heck yeah
Anything else?
This was mentioned in the original issue for merge_update_columns

@dave-connors-3 Okay, I buy it :) thanks for opening to help us keep track of the request, since we've gotten it before!
The incremental materialization code is getting away from us, and I hesitate to add more complexity there in its current state. The good news is that we're soon to meaningfully refactor it, starting with incremental "strategies" in core and our adapter plugins. We're going to be freezing net-new code additions in the meantime. That should clear the path for a straightforward addition like this one, or like support for arbitrary incremental_predicates (#3293/#4546). At that point, this can be either a line of code that we write, or a good first issue. I can't guarantee that we ourselves will manage to do it before v1.2, but I'm adding it to the milestone anyway, if just to signal that we're on a linear path in its direction.
As discussed in #5215, I don't believe we should support or encourage any merge statement logic more complex than:

match criteria: unique_key, "lookback window" predicates to speed up performance
do update these columns with new data
don't update these columns with new data


METRON-1641: Enable Pcap jobs to be submitted asynchronously

Contributor Comments
https://issues.apache.org/jira/browse/METRON-1641
This enables Pcap Jobs to be run asynchronously. The PcapJob class itself is now a Statusable implementation that allows clients to poll for current JobStatus. This implementation exposes the new functionality on the job class but keeps the existing PcapCli functionality intact and unchanged. The tests for this will be in a comment below, taken from https://github.com/apache/metron/pull/256.
This validation should check that the current pcap functionality does not break. Follow on PR's will leverage the new asynchronous capabilities.
Pull Request Checklist
Thank you for submitting a contribution to Apache Metron.
Please refer to our Development Guidelines for the complete guide to follow for contributions.
Please refer also to our Build Verification Guidelines for complete smoke testing guides.
In order to streamline the review of the contribution we ask you follow these guidelines and ask you to double check the following:
For all changes:

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

For code changes:

[x] Have you included steps to reproduce the behavior or problem that is being changed or addressed?

[x] Have you included steps or a guide to how the change may be verified and tested manually?

[ ] Have you ensured that the full suite of tests and checks have been executed in the root metron folder via:
mvn -q clean integration-test install && dev-utilities/build-utils/verify_licenses.sh

[x] Have you written or updated unit tests and or integration tests to verify your changes?

[ ] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered by building and verifying the site-book? If not then run the following commands and the verify changes via site-book/target/site/index.html:
cd site-book
mvn site

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.
It is also recommended that travis-ci is set up for your personal repository such that your branches are built there before submitting a pull request.

Looks good!  One thing I'm trying to wrap my head around is how we get status if we only have a job id or unique identifier for a job?  JobStatus doesn't have an id so I'm assuming resultPath is the unique identifier here.
As far as I can tell an instance of org.apache.hadoop.mapreduce.Job is kept in memory and is responsible for reporting status.  I can think of a couple scenarios where this might be problematic.
One is if I ran a query from the CLI but then wanted to get status from REST.  How would that work?  That's probably not a likely use case so maybe not an issue there.  What happens if I submit a query through REST and REST is restarted while jobs are running?  Do we lose job status information?

I think for a first cut, it's ok to have the restrictions that:

the REST API controls only the jobs it creates.  Otherwise, we would need more refactoring in the CLI to drop the output in the same HDFS directory rather than it be user specifiable and output locally.  Ultimately, while they use the same mechanism, the UX is different between the two approaches (e.g. the CLI entirely cleans up after itself and outputs to the local directory whereas the REST approach stores the results in HDFS until manual cleanup).
If a job is running while the REST API dies, we should consider that job to be runaway and needs to be killed by the admin or left to complete without the result being published.  One thing that we might consider doing is enabling the job naming to have a prefix of METRON_REST_PCAP so, upon REST start, it can kill existing jobs.  I think for THIS PR, we should just have REST pcap jobs have that prefix and leave it to a follow-on PR to do the actual killing.

One thing I didn't see.  Can we make sure we pass along the yarn queue to the job?

One more comment about restartability, I think we could potentially support this with this architecture in the future.  You can recover the Job object from MR via the JobClient
Configuration conf = new Configuration();
JobClient jobClient = new JobClient(new JobConf(conf)); // deprecation WARN
JobID jobID = JobID.forName("job_201107011451_0001");   // deprecation WARN
RunningJob runningJob = jobClient.getJob(jobID);

We could look for jobs which are completed but not in the HDFS structure and recover them on REST start.  I would suggest doing that as a follow-on though.

Perfect.  That addresses my concern.  Doing this in a follow on is fine since it's not necessary when using the CLI.

I spun this up in full dev, ran a fixed query, and got this error:
Exception in thread "main" java.lang.IllegalArgumentException: Wrong FS: file://./, expected: hdfs://node1:8020
	at org.apache.hadoop.fs.FileSystem.checkPath(FileSystem.java:666)
	at org.apache.hadoop.hdfs.DistributedFileSystem.getPathName(DistributedFileSystem.java:214)
	at org.apache.hadoop.hdfs.DistributedFileSystem$25.doCall(DistributedFileSystem.java:1181)
	at org.apache.hadoop.hdfs.DistributedFileSystem$25.doCall(DistributedFileSystem.java:1177)
	at org.apache.hadoop.fs.FileSystemLinkResolver.resolve(FileSystemLinkResolver.java:81)
	at org.apache.hadoop.hdfs.DistributedFileSystem.mkdirsInternal(DistributedFileSystem.java:1195)
	at org.apache.hadoop.hdfs.DistributedFileSystem.mkdirs(DistributedFileSystem.java:1169)
	at org.apache.hadoop.fs.FileSystem.mkdirs(FileSystem.java:1925)
	at org.apache.metron.common.utils.HDFSUtils.write(HDFSUtils.java:71)
	at org.apache.metron.pcap.writer.ResultsWriter.write(ResultsWriter.java:38)
	at org.apache.metron.pcap.mr.PcapJob.writeResults(PcapJob.java:270)
	at org.apache.metron.pcap.query.PcapCli.run(PcapCli.java:155)
	at org.apache.metron.pcap.query.PcapCli.main(PcapCli.java:52)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.apache.hadoop.util.RunJar.run(RunJar.java:233)
	at org.apache.hadoop.util.RunJar.main(RunJar.java:148)

Looks like the MR jobs succeeded but partitioning the files to the local FS did not work.

Good catch on the FS @merrimanr - also finding that via manual testing. I believe I have a workaround that degrades nicely to the configuration default and also allows you to pass in the scheme in the path.
FileSystem fs = FileSystem.newInstance(outPath.toUri(), config);

Also @merrimanr agreed about the points you made regarding restart-ability, etc. in the long run. In the short term, @cestella has done a rather good job of summarizing my thoughts on a v1 pass at this feature set.
I will:

add Job ID in the return status (was going to add that when I do the job manager follow-on to this PR, but I'll just do it here)
add the ability to pass a job name, ie job.setJobName(name) - I'll handle the actual job naming in the pcapservice and pass that as a parameter. I think that's a natural place for that logic.
Add ability to pass the queue name

How's that sound to you both?

Heads up, hadoop config class is where you set queue name iirc. We already pass that in as an arg. This would simply need to be provided via the calling job manager class. config.setProperty("mapreduce.job.queuename", "somequeue");

That sounds good to me.

I tested this and it's working for me in full dev.  I think it's good enough to go into the feature branch.  +1

Can we merge this?  Any other items you would like addressed @cestella?

+1, lgtm


the image generated is is not clear.

the image generated is is not clear when using custom fonts(new handwritten fonts). even though the preview is looking great.

Is it only happening in pdf? can you check the output of image?

no the image generated in the webpage  is also like this

I tried it with my handwriting file and it seems to work well for me

Can you mention the device, browser, and operating system you tested it on?

using latest edge.windows

This is how it looks in my device, it looks similar to the input (I tested it in chrome and firefox in ubuntu).
The font is a little unclear but still, the output in windows is not supposed to look blurred. I'll try to test it in windows by tomorrow and see if the problem is in windows.
About the font-spacing, there is a font spacing issue in html2canvas, the library that we use internally so for the workaround there is word-spacing and letter-spacing customization option in the tool.

the text looks more clear in the preview side and less clear in the image side. basically teachers need it a bit more clearer. thats why i complained. hope u understand

yes yes it's a fair issue, I'll check it out in windows and see if we can do something

This is from windows chrome, It looks fine in the image. I will check in edge as well

try checking the pdf try to zoom it. some fonts are ok

this is what im getting with my font. even in the pdf

i tried with brave browser  . and didnt got it so different

was bettr with another custom font . maybe u should make the page a bit more clearer

Oh yes it looks off in PDFs even in Ubuntu.
How about acting as if it is a feature :c? I mean if you turn images to a PDF with some online software, they get blurred as well so isn't that making it look more legit?
But yes in some cases, like in case of your font, that blurriness will make the text unreadable.
I'll think and I guess I'll have to check the jsPDF docs (the library we use to convert images to PDF)
Let me know your thoughts as well

more like bugs = features 😁 yeah ! or maybe u should give a guide to the font creation which will better match with your project

I'm trying to solve this along with the new design. I am not sure if it will work out but let's see.

can you please add me as the contributor on the site. I contributed here#71

Hi @Jobin-S  It is supposed to show up automatically after some time. It fetches contributors from GitHub Contributors.

Oh and I resolved this issue sometime back. Now I don't remember which commit 🤦♂ but it is apparently fixed now

great @saurabhdaware


Feature request: add support for testing the package manager

Considering that package manager information is collected during provision, it would be great to be able to test it in order to tweak test execution, e.g.:
  - because: dnf5 does not support --noautoremove
    environment+:
      DNF_HAS_NOAUTOREMOVE: "yes"
    when: package_manager == dnf

Thank you for the idea. Moved to tmt as the definition of exact names and values belongs there.  (the fmf itself is just the format / parser and is agnostic to the meanings)

Hmm, a chicken and egg problem: when adjust is evaluated, by fmf library, the package manager is very much unknown as guests would be provisioned based on the already resolved fmf tree.
I think there were a few similar requirements calling for some kind of "runtime adjust", to apply somewhere in post-provisioning when tmt acquires additional info about the test environment by inspecting guests.


Conflicts with eslint-plugin-import

Environment

Operating System: Darwin
Node Version:     v20.17.0
Nuxt Version:     3.13.2
CLI Version:      3.13.2
Nitro Version:    2.9.7
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      compatibilityDate, devtools, future, experimental, ssr, typescript, routeRules, hooks, runtimeConfig, extends, modules, sourcemap, ui, postcss, tailwindcss, vite, nitro, components, css, imports, content, ogImage, i18n
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Package
None
Reproduction
import importPlugin from 'eslint-plugin-import'
import withNuxt from './.nuxt/eslint.config.mjs'

export default withNuxt({
  importPlugin.flatConfigs.recommended,
})

Describe the bug
I'm trying to use eslint-plugin-import (https://github.com/import-js/eslint-plugin-import) and I get the error:
Error: ESLintFlatConfigUtils: Different instances of plugin "import" found in multiple configs:
nuxt/import/rules, import/recommended.

This is because import plugin is already used here (totally undocumented btw) https://github.com/nuxt/eslint/blob/main/packages/eslint-config/src/flat/configs/import.ts
I'd rather use the eslint-plugin-import plugin because:

In nuxt/eslint 'import/order' is gated behind enabling 'stylistic' which I don't want to use
https://github.com/antfu/eslint-plugin-import-x is now archived and not maintained

Can I suggest we switch to eslint-plugin-import which is maintained?
Additional context
No response
Logs
No response

Same issue, I don't want to disable standalone and config a lot.🧐

You are linking to the wrong repo: https://github.com/un-ts/eslint-plugin-import-x which is a modern and more active version to eslint-plugin-import
You can always disable import/order explicitly if you don't want it.
I'd say eslint-plugin-import-x is designed to replace eslint-plugin-import, which I don't see the point to use the old one as the modern one is already included by Nuxt ESLint. Of course, you can do some hacks to the config array to completely remove that - or I would suggest using the standalone mode if you really don't agree with Nuxt's default (which is totally fine). I would consider this issue out-of-scope. Thanks.

I can see https://github.com/un-ts/eslint-plugin-import-x is now longer marked as archived, which makes sense if it's the replacement. Thanks

Thanks, that makes sense, I was able to enable the import/order without stylist by adding it to my own config:
'import/order': 'error'


Tracing

Request span level (Kamon Tracing)
Recipe Instance level (X-Bakery-Intent: Recipe-Instance-Id:)


Fix getOutputReport, getInputReport and getFeatureReport: not being able to create more than one report + temporal coupling

This fix is related to Issue #805 in the NimBLEHIDDevice class.
What fixes:

The user can call getXXXXXReport() in any order.
The user gets a different characteristic for each different reportID.
The user gets the same characteristic for the same reportID.

Fix description:
A new private method is introduced: locateReportCharacteristicById(): given a reportID, it returns the characteristic if already created, nullptr otherwise.
getXXXXXReport() tries to locate the characteristic. If found, it is returned "as is". If not found, it is created and returned.
Drawbacks:
There is a computational overhead. The user should store/cache the value returned by getXXXXXReport() instead of repeating the calls. Perhaps we should warn about this in the documentation.

https://github.com/h2zero/NimBLE-Arduino/pull/805#issuecomment-2548600648

Thank you @afpineda !


Add options for file audit configuration

fixes #733
File audit logging can be configured with:
audit:
  file:
    path: /var/log/kube_audit/audit.json
    max_size: 100
    max_age: 30
    max_backups: 20

This also moves the webhook audit under audit.webhook which is a breaking change.

why not enable this by default?

Audit generates quite a bit of data. E.g. in my cold-running test cluster it generates ~80M per hour. So I thought one should not really have it unless you really want the have audit

can user configure both file & webhook audit

Yes.

Audit generates quite a bit of data. E.g. in my cold-running test cluster it generates ~80M per hour. So I thought one should not really have it unless you really want the have audit

Does it really matter if we have sane defaults for size & max_age? Writing audit log to a file should not be too resource intensive?

Does it really matter if we have sane defaults for size & max_age? Writing audit log to a file should not be too resource intensive?

We can make it on by default. How about with something like:
          'path' => '/var/log/kubernetes/audit.json',
          'max_size' => 100, # Max 100M files
          'max_age' => 30, # Max 30 days old audits
          'max_backups' => 20 # Max 20 rolled files, each 100M --> 2G used at max

@jakolehm there's now defaulting in place PTAL


CLI added login, logout, and whoami

Auth prompting was taken out of the api package and moved into CLI
commands. A user now needs to authenticate manually with one of these
commands, as opposed to the old way, in which the user would simply be
prompted upon receiving a 401
API functions were added for contacting POST /v2/auth/login, GET
/v2/auth/logout, and GET /v2/auth/id.
Parsing functions for getting v1 and v2 API error responses were
added. v1 endpoints on v2 APIs will currently be unable to get error
messages, but this will be okay once the v1 endpoints go away.
Error types were added for 401s and 403s, namely ErrUnauthorized and
ErrForbidden. These can be type checked to determine if an api error was
due to authentication problems.
The core API exposes test endpoints if the debug flag in the daemon
config is set.
The v2 api now gives 401s if the user is not authenticated (as opposed
to 403)
The api.Backend type now accepts an APIVersion int, for being told
which APIVersion to use. This value can be stored in the in-memory
version of the config for use in the rest of the CLI, but is never written out
to disk.
testdev now sets the debug flag for the shield daemon config.
The login endpoint now gives back information about the user which was
authenticated in the same format as /auth/id
api_version is no longer exposed by the status endpoint (because you
need to be authenticated to reach it), and is instead exposed by
/v1/ping

@jhunt resume the reviewing

@jhunt waves hands it was rebased all along

Oh Wow, I Don't Know What Pull Request I Was Looking At ™


After hours of running the history updates/graphing stops working

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
Home Assistant
0.33.0.dev0
Python release (python3 --version):
Python 3.4.2
Component/platform:
History
Description of problem:
After running HASS for hours history updates and graphing stop working. I cannot run HASS for a day without this happening, which makes it pretty useless.
It simply keeps logging the old value, while the current value still gets updated.
I can still control things, so HASS runs.
Expected:
Problem-relevant configuration.yaml entries and steps to reproduce:
Enabled stuff::
media_player
device_tracker.nmap
notify pushover
speedtest
systemmonitor
cpuspeed
fritzbox_callmonitor
coinmarketcap
toon (custom component)
google_travel_time
wunderground
camera
device_tracker.owntracks
mqtt
knx

Running in virtualenv on
Linux home-logger 3.16.0-4-amd64 #1 SMP Debian 3.16.36-1+deb8u2 (2016-10-19) x86_64 GNU/Linux
cat /etc/debian_version
8.6
user  27968  0.6  5.2<PHONE_NUMBER>96 ?       Sl   Nov14   6:10 /opt/home-assistant/virtualenv/bin/python3 /opt/home-assistant/virtualenv/bin/hass -v --config /opt/home-assistant/config --pid-file /opt/home-assistant/run/hass.pid --daemon
git rev-list --count HEAD
7274

Traceback (if applicable):

Additional info:
The log shows nothing related to this as far as I can see.
Restarting HASS fixes it until it fails again within a day.
Were to look for clues, anyone else seen this?

There was an issue with the device tracker that could break the recorder. It has been fixed in #4393 and has been released as 0.32.4 (and also part of 0.33.0.dev0).


Add procedure to assign custom script to node meta

Fixes #100074
The problem lies beneath the function node->set_meta for custom type script not being called anywhere. This PR add a new call in an appropriate location.

Before
After


Built-in role management for SWA with FrondDoor

Should built-in roles management (invited via portal) work when Static Web App is configured to work with FrontDoor and custom domain? In this case, the custom domain is setup on FrontDoor side and points to SWA. But SWA doesn't know about it. When you want to invite a new user in Role management section of SWA, there is no option to generate the invitation link for that custom domain. If you generate the link for default SWA URL and manually change the link to the custom domain, you get 400 Bad Request. Am I doing something wrong or I am just not supposed to use built-in SWA role management when using FrontDoor?

Hi @superhp,
You've raised a good question. If you are using your own AFD, currently our role invitation logic does not know that a custom domain is configured against your site and as a result will not allow you to generate invitation links based on that domain. To get past this you have a few options:

You can add your custom domain to the SWA as well. This may not be used by your AFD (as AFD will be connecting to the default hostname) but will allow you to create the links you need
If you don't need to manage the AFD yourself, then switching to enterprise-grade edge will solve this problem for you
If neither of the above options work, you can assign roles programmatically as shown here: https://docs.microsoft.com/en-us/azure/static-web-apps/assign-roles-microsoft-graph

Thank you for response, @miwebst! I didn't think about assigning the domain to both AFD and SWA option but it seems it could be the easiest way. I will try it.


🐛: Uploading files using web form causes error

Contact Details
No response
Location
runBiosimulations
What happened?
Using the webform to upload files, an error is presented when trying to create the formdata for submission.
Almost certainly due to 0ae13a8b7b755988c016d8985d775b45a82c5844 having a different type for the uploaded files
What made it happen?
Submit a file for simulations
Version
Development (Deployed on a .dev domain)
What browsers are you seeing the problem on?
No response
Relevant log output
TypeError: Failed to execute 'append' on 'FormData': parameter 2 is not of type 'Blob'.
    at p.submitJobForFile (dispatch.service.ts:155)
    at i.onFormSubmit (dispatch.component.ts:760)
    at dispatch.component.html:5
    at Wl (core.mjs:15291)
    at Object.c [as next] (core.mjs:15329)
    at Object.next (Subscriber.js:110)
    at Z._next (Subscriber.js:60)
    at Z.next (Subscriber.js:31)
    at Subject.js:31
    at l (errorContext.js:19)
handleError @ error-handler.ts:36
n4 @ core.mjs:10976
Wl @ core.mjs:15294
c @ core.mjs:15329
(anonymous) @ Subscriber.js:110
_next @ Subscriber.js:60
next @ Subscriber.js:31
(anonymous) @ Subject.js:31
l @ errorContext.js:19
next @ Subject.js:26
emit @ core.mjs:22776
onSubmit @ forms.mjs:5352
(anonymous) @ forms.mjs:5434
Wl @ core.mjs:15291
c @ core.mjs:15329
(anonymous) @ platform-browser.mjs:546
invokeTask @ zone.js:406
Se.onInvokeTask @ task-tracking.js:65
invokeTask @ zone.js:405
onInvokeTask @ core.mjs:25864
invokeTask @ zone.js:405
runTask @ zone.js:178
invokeTask @ zone.js:487
j @ zone.js:1600
x @ zone.js:1626

Code of Conduct

[X] I agree to follow this project's Code of Conduct

This was a part of the changes in #3668. I noticed it when submitting projects from the physiome repo.  I have pushed a fix in #3721. Ready to merge with a review.

:tada: This issue has been resolved in version 7.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Remove stub pages which are not used anywhere

Both the things that would have gone on these pages are now part of the updates page.

👍


Duplicating sites does not duplicate their content

I have a multilingual website with a site for English, French and recently Spanish. Checking the mirror site box copies the layouts and pages but without their content.
Is there a way to get it to copy the content too? An option for this in future would be really helpful for those copying large sites who could then just change the content they needed rather than all of it.
Is there any way to do what I'm looking for?

Mirroring is gone as of 2.0. Pages can be translated directly.


Allow other operation labels in Poutine

This PR simplifies and generalizes the dispatch mechanism in poutine.Messenger to allow handlers for other types of operations (beyond "sample" and "param").  In particular:

It defines a wrapper function poutine.runtime.effectful and uses it to implement pyro.param.  effectful takes a type label (e.g. "sample") and a callable.
poutine.runtime.apply_stack and poutine.Messenger use the type label passed to effectful, which is stored in msg["type"] to dispatch to the appropriate operation: getattr(self, "_pyro_{}".format(msg["type"]))(msg)

I was using this elsewhere to experiment with lazy evaluation and dependency tracking.

Good to merge pending review.  There should be no performance impact.


Rest position control

Hello, I have issues with the rest position as my model is quite round and the arms clip into the body when resting. There don't seem to have an option for movement constraints or arm colision in VRM.
Here's the example :

I would like to be able to control the angle of the arms at rest. Thanks in advance.

@porkyoot
I merged #887 to support rest position control feature in next version (v3.0.2), so I close the issue.
If you feel the feature is not enough, please re-open the issue or create a new one.


[dart fix] support fixes for deprecated lints in options

Expected behavior: dart fix fixes deprecated lints in analysis_options.yaml files where possible, by:

removing them, or possibly
replacing them with another preferred lint (see discussion below)

Note:  currently we have a simple diagnostic code DEPRECATED_LINT_HINT but we may want to support more to support the different kinds of fixes would be appropriate.

Will we get warnings as well?

I have a small concern with this proposal.
If the lint has been deprecated because it no longer makes sense (such as the removal of the API being checked), then it makes sense to me to remove it when the language feature can no longer be opted out of.
But if the lint has been deprecated in favor of another lint (such as a more general lint or even a renamed version of the old lint), then it doesn't make sense to remove it; it should instead be updated (or the user should be required to decide whether they want the behavior of the new lint without the tool opting them into to a more general lint).
Perhaps the right answer is that we need something more nuanced that 'deprecated' for some kinds of lint changes, and that this fix is fine for the more restricted meaning of 'deprecated'.

Great points!

[I]f the lint has been deprecated in favor of another lint (such as a more general lint or even a renamed version of the old lint)

I was thinking renames would be different than deprecations but another way to think of them is as a special kind of deprecation.  In that case,

Perhaps the right answer is that we need something more nuanced that 'deprecated' for some kinds of lint changes, and that this fix is fine for the more restricted meaning of 'deprecated'.

💯
The more I think about it, the more I like the idea of a nuanced deprecation since that would afford some flexibility down the road.

This would be a stretch but not impossible depending on how we prioritize the next few weeks.
/fyi @jacob314 @bwilkerson @natebosch

We should be careful that dart fix can still be run once and then be stable.
If the first dart fix migrates from one lint to a new one, it would also need to fix any existing cases of the new lint.
IMO this shouldn't be a very high priority since it's mainly saving a small bit of toil within the packages whose authors (by using lints in the first place) have already expressed they are OK with a small amount of toil.

I'm not realistically going to be able to tackle this given what we've got left for the analyzer/linter.

@pq if this is not required, I am going to move this to the Dart 3 stable milestone to be triaged again later.  Thank you for the update.

SGTM!

@pq - do we need anything for stable here or should we bump to Dart 3 cleanup?

Dart 3 cleanup SGTM.


chore: add info on Enterprise Server

Hey @AriPerkkio! Per a request from @EricSimons wanted to get something in about Enterprise Server into the docs here. Feel free to touch base with @Nemikolh if you have any questions on this.

 Submitted with StackBlitz.

@Nemikolh i won't be here to follow up on that!


Configurable file types as in ack

It would be nice with a configuration file where one can configure own file types, similar to what ack has.

I would actually like this as well. For instance, I need to tell the difference between "jsx" and "js" but right now you have to use --js which finds both js and jsx files.

I actually came here looking to find documentation on this, hoping it was a feature already. I guess not, as of yet :)

Can configurate now?
I want to find "aml" file,but there is not this file type,can i configurate it?

This seems to be a duplicate of #392 no?

Yep - indeed a duplicate of #392 but this one seems more popular, so I'll add my +1 here. :smiley:

but how to add new file type fo ag ?
i can not the above answer,can you help me ?

@hanyimin that's not currently supported hence this issue and #392, you can't add custom file types to ag in the way you do with ack. So you either use the -G, --include and --exclude flags explicitly (maybe with aliases e.g. alias agjs=ag --include='*.js' or you use something else or you add support for it and create a PR.

@xmo-odoo  thanks, I think I can use the alias method ,thanks !

+1! This is a must feature.

+1  Please add.

This would be really helpful!

+1, but --include is also nice!

`--include ...

bedge: +1

+1 I'm also running into a use case for this. Need to filter by handlerbars (.hbs) files.

Also need handlebars, since Ember relies heavily on it.

I would also like to have this feature, we're using in-house file extensions in our project.
Level 0: same feature as Ack
Level 1: handle extensions including a . (e.g. .generated.cs)
Level 2: handle glob/pcre patterns in the extensions (e.g. .log\..*)

+1

+1

Even worse; how is there not a built-in type for .txt files?
$ ag --version
ag version 2.1.0
$ ag --list-file-types | grep 'txt'
      .mas  .mhtml  .mpl  .mtxt

Argh, I was really hoping this feature would've been added by now. +1

I am shocked that this feature is missing!!

I switched to https://github.com/BurntSushi/ripgrep for this reason:
$ cat ~/.ripgreprc
--type-add
tpl:*.tpl
$ grep ripgrep ~/.zshrc
export RIPGREP_CONFIG_PATH=$HOME/.ripgreprc
$ rg searchMe -ttpl # finds the string 'searchMe' in files with the extension 'tpl'
$ rg searchMe -Ttpl # finds the string 'searchMe' in files with an extension not equal to 'tpl'


added jsonwebtoken dependency, updated server.js and auth.js file

will have to npm i to add dependency to your local

Done!
On Fri, Sep 22, 2023 at 11:45 AM Marianna @.***> wrote:

Assigned #1 https://github.com/ulianaim/mindfulness_app/pull/1 to
@iis4u2nv https://github.com/iis4u2nv.
—
Reply to this email directly, view it on GitHub
https://github.com/ulianaim/mindfulness_app/pull/1#event-10448880418,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6XU5EX6TLBRCMQ6RBC2DODX3WXCLANCNFSM6AAAAAA5DJQDQY
.
You are receiving this because you were assigned.Message ID:
@.***>


Apache log shows local IP instead of visitor IP

With pagespeed enabled I get the local ip in apache logs, I think it is not for all requests, is it normal ?
<IP_ADDRESS> - - [25/Jan/2020:04:49:18 +0000] "GET /thumbs/lanci-thesis-acidentata-17996-1.jpg?1382006911 HTTP/1.1" 200 2739 "https://www.site.tld/lancia/thesis" "Serf/1.3.9 (mod_pagespeed/<IP_ADDRESS>-0)"
<IP_ADDRESS> - - [25/Jan/2020:04:49:18 +0000] "GET /thumbs/lancia-thesis-vand-sau-schimb-25300-1.jpg?1393576311 HTTP/1.1" 200 38743 "https://www.example.ro/lancia/thesis" "Serf/1.3.9 (mod_pagespeed/<IP_ADDRESS>-0)"
<IP_ADDRESS> - - [25/Jan/2020:04:49:18 +0000] "GET /thumbs/lancia-thesis-2-4i-gpl-6996-1.jpg?1375351567 HTTP/1.1" 200 16164 "https://www.site.tld/lancia/thesis" "Serf/1.3.9 (mod_pagespeed/<IP_ADDRESS>-0)"
66.249.68.XX - - [25/Jan/2020:04:49:18 +0000] "GET /small/casa-doua-camere-si-teren-de-5000-metri-runcu-jud-15747-1.jpg HTTP/1.1" 200 68314 "-" "Googlebot-Image/1.0"
<IP_ADDRESS> - - [25/Jan/2020:04:49:13 +0000] "GET /scripts/ref/js.php HTTP/1.1" 200 - "https://www.site.tld/settings?previous_url=%2Fsettings%3Fprevious_url%3D%252Fsettings%253Fprevious_url%253D%25252Fsettings%25253Fprevious_url%252525253D%25252525252Fad%25252525252Fvand-geanta-casual" "Serf/1.3.9 (mod_pagespeed/<IP_ADDRESS>-0)"

This is expected behavior, closing this


Add multi-version pkgdown documentation

This will create multiversion docs for admiral

Note that the real effect will only take place after this has been promoted from devel to main, as the GHA workflow rules only runs pkgdown documentation on main.
So you won't see the effect after merging this PR.

So I know this kind of violates our lockdown of main, but I'm interested in testing this out before we do a release?  @thomas-neitmann  can we release that restriction to test this out on main?  If so, my inner @rossfarrugia is telling me not to this until Monday!

Hi @cicdguy and @dgrassellyb.  Are we able to try this out now?  Was hoping to test it out before official release on March 6th.

@cicdguy @dgrassellyb Hey all.  I was confused about the slack message.  Is this one ready to go or is the admiraldev issue still being investigated.  I actually saw that 0.2 issue in a fork of a admiral repo I did recently.

Many thanks @dgrassellyb and @cicdguy for putting this together.  I think a lot of Pharma companies will like this versioning of the website as some drug studies might be locked into a particular version of admiral for several years.  While the pkg documentation is within the package it isn't always so pleasant to view and I Find the website to be very helpful and clean to look at.


Pull in upstream changes from cisagov/skeleton-ansible-role

🗣 Description
This pull request pulls in upstream changes from cisagov/skeleton-ansible-role.  It also modernizes this module by separating the RedHat and Debian tasks into separate files.  Finally, we modify the code to use update: dist to update Debian systems.
💭 Motivation and Context
Oh the joys of modern life!
🧪 Testing
All pre-commit hooks and molecule tests pass.
🚥 Types of Changes

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

✅ Checklist

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

  LGTM insists that this is a JS project, but it is not.


WFLY-4243: adding bindings not component scoped to module (instead of cl...

...ass) binding configurations

Linux with security manager Build 730 is now running using a merge of 49c70aa0d7305e696f9828dbf20b59eed50b533e

Linux Build 5743 is now running using a merge of 49c70aa0d7305e696f9828dbf20b59eed50b533e

Windows Build 854 is now running using a merge of 49c70aa0d7305e696f9828dbf20b59eed50b533e

Linux with security manager Build 730 outcome was SUCCESS using a merge of 49c70aa0d7305e696f9828dbf20b59eed50b533e
Summary: Tests passed: 788, ignored: 369 Build time: 0:05:26

Windows Build 854 outcome was SUCCESS using a merge of 49c70aa0d7305e696f9828dbf20b59eed50b533e
Summary: Tests passed: 2906, ignored: 364 Build time: 0:49:24

Linux Build 5743 outcome was FAILURE using a merge of 49c70aa0d7305e696f9828dbf20b59eed50b533e
Summary: Tests failed: 1 (1 new), passed: 2905, ignored: 364 Build time: 0:52:40
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ejb.stateful.passivation.PassivationTestCase.testPassivationMaxSize: javax.ejb.EJBException: java.lang.IllegalStateException: WFLYCLEJBINF0008: Stateful session bean {[49, -52, 62, 92, 29, -120, 72, -77, -127, -120, -64, 61, 102, 116, 101, -77]} refers to an invalid bean group cfbc0b5f-9772-4e8a-8ef8-81b6c929b3b2
	at org.wildfly.clustering.ejb.infinispan.bean.InfinispanBeanFactory.createBean(InfinispanBeanFactory.java:75)
	at org.wildfly.clustering.ejb.infinispan.InfinispanBeanManager.findBean(InfinispanBeanManager.java:266)

Shouldn't a similar change be done for resources defined in the XML descriptors?
I confirm that your PR fixes the issues in the TCK resourcedefs/annotations tests but the resourcedefs/descriptor (which are the same tests except the JMS resources are defined in the XML descriptors) still fail.

actually, scratch my comment: the resourcedefs/descriptor also passes (the failures I noticed where due to a pb in my environment. once I fixed it, theses tests also passed).
This looks good to merge, thanks!

retest this please

Linux Build 5762 is now running using a merge of 49c70aa0d7305e696f9828dbf20b59eed50b533e

Windows Build 873 is now running using a merge of 49c70aa0d7305e696f9828dbf20b59eed50b533e

Linux with security manager Build 749 is now running using a merge of 49c70aa0d7305e696f9828dbf20b59eed50b533e

Linux with security manager Build 749 outcome was SUCCESS using a merge of 49c70aa0d7305e696f9828dbf20b59eed50b533e
Summary: Tests passed: 795, ignored: 371 Build time: 0:05:21

Windows Build 873 outcome was SUCCESS using a merge of 49c70aa0d7305e696f9828dbf20b59eed50b533e
Summary: Tests passed: 2913, ignored: 366 Build time: 0:50:39

Linux Build 5762 outcome was SUCCESS using a merge of 49c70aa0d7305e696f9828dbf20b59eed50b533e
Summary: Tests passed: 2912, ignored: 366, muted: 1 Build time: 0:53:04

This PR breaks quite a bit of stuff. The bindings belong to the class, not the module, and this is important because classes can be used by components from multiple modules (resulting in a single class level definition creating different java:module bindings in different modules).
What is the underlying reason for this change? Not all bindings make sense to be present in the app client, it is not a full server.

@stuartwdouglas This PR was about fixing a regression in JMS TCK dealing with JMS resource definitions that are defined in app client archives.
The regression started with https://github.com/wildfly/wildfly/commit/3a3c044e344b542dbdce5f4e6f7e5978e6944714 .
Before that, I added the bindings to the module too for war and appclient deployments.
When @emmartins generalized the code in the ee subsystem, this special case disappeared.
This PR was generalizing this special case to any bindings. If that's not acceptable, I need a hook to be able to handle the JMS resource definition binding appropriately.
Is that acceptable?

This PR is only for resource definition annotations (EE.5.18) not on java:comp, these are specific pure metadata and must be bound in every module. Having these on "standard" class bind configs results on not being processed for some components (e.g. app client) as it is mandated by the spec

I did not notice any issue with the change, can you please indicate what it breaks so I have a look? If this is fact breaks something then we need to change how class bindings are processed, cause @jmesnil  hack was not correct yet the tick tests will continue to fail if we continue to rely on this single process of these special binding annotations

just to clarify why the previous hack was not correct, it was being applied just to JMS resource definitions on war and app client, and the "process" must be the same for all resource definition annotations, and the bindings were being copied to both class and module binding configs


Fails when caching hidden folders

Hi,
Thanks for all the patches you did since the upstream.
I noted an issue, It's not critical but the job cacher seems to ignore folders starting with ., such as .git.

This seems to be because of the default excludes (see org.apache.tools.ant.DirectoryScanner#DEFAULTEXCLUDES. To change this, you need to pass the useDefaultExcludes parameter while calling the super constructor in jenkins.plugins.itemstorage.local.LocalObjectPath.IsModifiedGlob#IsModifiedGlob.

can you open a PR to fix that?


Talk is not live.

Hi! Yesterday I installed talk example, and added pusher keys. The problem is - my laravel talk is not live. Can you tell me why? I am using this on localhost.

I am having the same issue, any help please ?

me too, any help

Hello,
I have tried this talk example. I can check in multiple browser, I have created account in Pusher and I have updated the credentials in the .env file.  Even I have run the php artisan queue:listen and redis-cli monitor. I can see the data receiving in queue and redis also. But I didn't get the data in pusher account debug console and also not happening live update in other end. Is there any configuration missing my side or I need to do any extra step on this for live update. Could you please help me on this. I'm struggling on this last 3 days.


DELETE EDGE breaks transaction in remote connections.

Using version 2.0.13 the DELETE EDGE command seems to break transactions in remote connections. Please have a look at the following example:
Schema
CREATE CLASS Person EXTENDS V
CREATE PROPERTY Person.name STRING
ALTER PROPERTY Person.name MANDATORY TRUE
ALTER PROPERTY Person.name NOTNULL TRUE
CREATE INDEX PersonNameUnique ON Person(name) UNIQUE

CREATE CLASS IsFriend EXTENDS E
CREATE PROPERTY IsFriend.out LINK Person
ALTER PROPERTY IsFriend.out MANDATORY TRUE
CREATE PROPERTY IsFriend.in LINK Person
ALTER PROPERTY IsFriend.in MANDATORY TRUE
CREATE INDEX IsFriendInOutUnique ON IsFriend(in, out) UNIQUE

Code
OrientGraphFactory factory = new OrientGraphFactory("remote:localhost/testdb", "admin", "admin");
OrientGraph orientGraph = factory.getTx();

StringBuilder sb = new StringBuilder();

sb.append("BEGIN;");

// Create a couple of vertex and an edge between them.
sb.append("LET p1 = CREATE VERTEX Person SET name = :name1;");
sb.append("LET p2 = CREATE VERTEX Person SET name = :name2;");
sb.append("CREATE EDGE IsFriend FROM $p1 TO $p2;");

// Uncomment the following to trigger the reported behaviour.
//sb.append("DELETE EDGE FROM $p1 TO $p2;");

// This will cause 'ORecordDuplicatedException', but that's intended.
sb.append("LET p3 = CREATE VERTEX Person SET name = :name2;");

sb.append("COMMIT;");

OCommandScript script = new OCommandScript("sql", sb.toString());
Map<String, String> p = ImmutableMap.of("name1", "Peter", "name2", "John");

try {
  orientGraph.command(script).execute(p);
} catch (OException ex) {
  LOGGER.error(ex.getMessage(), ex);
} finally {
  orientGraph.shutdown();
}

When I leave the DELETE EDGE... line commented, a SELECT FROM Person query shows no records (as expected). However, truncating the classes and repeating the test with the line uncommented will end up leaving the two vertexes p1 and p2 in the database (altough the transaction was presumably rolled back). Logs say this
2015-08-13 14:55:31:194 CONFIG Remote client connected from: OClientConnection [id=5, source=/<IP_ADDRESS>:56508, since=1439474131194] [OClientConnectionManager]{db=testdb} Error during transaction commit, transaction will be rolled back (tx-
id=4)
com.orientechnologies.orient.core.storage.ORecordDuplicatedException: Cannot index record #12:2: found duplicated key 'John' in index 'PersonNameUnique' previously assigned to the record #12:1 RID=#12:1

What can be causing this behaviour?

I forgot to mention that this example is totally contrived, as I could be using Tinkerpops for this particular use case. In the real-life use case I'm using OCommandScript to execute a batch, and that's how I noticed the problem.

This is the expected behaviour, look at: http://orientdb.com/docs/last/Transactions.html#breaking-of-acid-properties-when-using-remote-protocol-and-commands-sql-gremlin-js-etc. In v3.0 we will fix this by supporting transaction flushing to the server.


Fix Permissions lint tests

Fixes #1005

You downgraded lint on purpose right? I will fix the conflict

Yes, I downgraded it on purpose, this is what Compose does as well. Thanks!


Test 23.2 during staging

Note: URL points to a personal stage branch currently

Checklist

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

@conda-forge-admin, please rerender

It seems to be a race condition. I am not sure, what happens there, probably there is something strange about how OBJECT libraries are transfered by CMake into ninja rules...

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin please rerender

@conda-forge-admin please rerender

@conda-forge-admin please rerender

@conda-forge-admin rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender


Continue button

Steps to reproduce

Getting into the manga
Add to library
There is no add button because there is already a manga with the same name

Expected behavior
I think there should be a button to add the manga even though it has the same name as another manga.
Actual behavior
Well, I remember ever using the button to add them even though they had the same name
Crash logs
No response
Tachiyomi version
1.9.3
Android version
Android 13
Device
Samsung A71
Other details
No response
Acknowledgements

[X] This is a TachiyomiSY specific issue that does not happen in Tachiyomi Preview.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 1.9.3.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

This is a main Tacvhi issue, if its still happening in the next build, open a issue in the main Tachi repo


Share Posts as story

I have tried to share a post as story with the code for uploading stories, but turns out it gives error because no file is given.

My code is this:
await ig.publish.story({
    media:{
      media_id :media_id_of_certain_post,
    }
  });

Is something I am missing?

You need to use the sticker-builder to add the post to your story.
Use the attachment-sticker: StickerBuilder.attachment({ mediaId: '...', mediaOwnerId: '...' }). I hope this still works.
Note: this won't render the post.

You need to use the sticker-builder to add the post to your story.
Use the attachment-sticker: StickerBuilder.attachment({ mediaId: '...', mediaOwnerId: '...' }). I hope this still works.
Note: this won't render the post.

Unfortunately is no longer working, is there a chance of having this feature again in the future?

is there a chance of having this feature again in the future?

Maybe

Hi @Nerixyz , what do you think is the best approach for us to help bringing this feature back?

Hi @Nerixyz , what do you think is the best approach for us to help bringing this feature back?

Someone would use burp/mitmproxy (or whatever) and look at the request and update the sticker accordingly.

Someone would use burp/mitmproxy (or whatever) and look at the request and update the sticker accordingly.
I'm guessing you need to do that using an Android device? Or maybe an emulator would work?

Someone would use burp/mitmproxy (or whatever) and look at the request and update the sticker accordingly.

I'm guessing you need to do that using an Android device? Or maybe an emulator would work?

Good afternoon! Did you manage to track the requests? How did you do that ?

Someone would use burp/mitmproxy (or whatever) and look at the request and update the sticker accordingly.

I'm guessing you need to do that using an Android device? Or maybe an emulator would work?

Good afternoon! Did you manage to track the requests? How did you do that ?

Hello! I've installed mitmproxy on my computer and an Android emulator through Genymotion. I've found a modified Instagram APK that disabled SSL pinning, so I installed that on the Android VM and was able to configure mitmproxy and use it in Insta.

const res = await  ig.publish.photo({ file: finalBuffer }); const sticker = new StickerBuilder().add(StickerBuilder.attachmentFromMedia(res.media, { width: number, height: number }).center()).build(); const story = await ig.publish.story({       file: img,       stickerConfig: sticker});
that one works

@xfrcc is this working for you? You said it doesn't render the post. Then you said you need to prepare the image with node-canvas. I am confused. Does it work with the above method or not?

@xfrcc is this working for you? You said it doesn't render the post. Then you said you need to prepare the image with node-canvas. I am confused. Does it work with the above method or not?

It does work
I meant that the sticker of the shared post will not be visible, what the sticker configuration does is mark the coords where if the user is tapping triggers what it is in this case the view post  in the app.

And the resolution is to maintain the aspect ratio used in the stories (9:16)

Basically in the app, when you tap on a post image in story anywhere, it will open the post. So you need the sticker to mark the location while upload the image separately in the story. @xfrcc

Basically in the app, when you tap on a post image in story anywhere, it will open the post. So you need the sticker to mark the location while upload the image separately in the story. @xfrcc

Yes, the image must be passed in the file object, and strictly speaking, when stories from posts are uploaded through the app, they become a single image
You must implement the download of the post image and edit it by putting a background so that it looks as if it were from the official app

The example I send makes it share the publication that was uploaded by the same module, but I don't think it is difficult to obtain the media of a specific post

On the app, it picks the background gradient kind of color from the post image.

For gradient background this should do the work
 const canvas = createCanvas(1080, 1920);
    img = new Image;
    img.src = im;
    const context = canvas.getContext('2d');
    const x = (canvas.width  - img.width) * 0.5;
    const y = (canvas.height - img.height) * 0.5;
    const grd = context.createLinearGradient(x, 0, x, canvas.height);
    grd.addColorStop(0, randomHexColor());
    grd.addColorStop(1, randomHexColor());
    context.fillStyle = grd;
    context.fillRect(0, 0, canvas.width, canvas.height);
    context.drawImage(img, x, y);
    const final = canvas.toBuffer('image/jpeg');

Hi @Nerixyz , what do you think is the best approach for us to help bringing this feature back?

Someone would use burp/mitmproxy (or whatever) and look at the request and update the sticker accordingly.

I followed your advice and after that started learning about this, now trying to build a private api for other app, but how do you dealed with the API Key?
As I see in request when sending a request for login the first thing that goes in the headers is the Api key, any way of bypass this?

As I see in request when sending a request for login the first thing that goes in the headers is the Api key, any way of bypass this?

You need to send the key as well.

As I see in request when sending a request for login the first thing that goes in the headers is the Api key, any way of bypass this?

You need to send the key as well.

Yes, but it is possible to generate new ones?
Does this repo use the same api key for everyone?

Yes, but it is possible to generate new ones?

Not every API is the same so I can't answer.

Does this repo use the same api key for everyone?

The private api doesn't really have the concept of API keys.


Russian Translation of Getting Started

[ ] Learn how Free Code Camp Works
[ ] Create a Github Account and Join our Chat Rooms
[ ] Configure your Code Portfolio
[ ] Join a Campsite in Your City
[ ] Learn What to Do If You Get Stuck

@jniro when do you plan to start the translation?

I am planning on starting today.
On Tuesday, January 19, 2016, Arsen Melikyan<EMAIL_ADDRESS>wrote:

@jniro https://github.com/jniro when do you plan to start the
translation?
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/6232#issuecomment-172843851
.

@bugron @jniro any progress on this? We are almost ready to launch the Spanish translation this week. Russian will be possible as soon as the translation work is done :)

@QuincyLarson actually, I don't know. No one has contacted me about the Russian translation.

@bugron I can take part in making Russian translation in general. Can do ru-en en-ru translations pretty fast and of high quality, but I'm pretty busy lately. I could, for example, do translation of at least one item (exercise) per day. Will be able to provide more certainty not earlier than I finish voting app, which will hopefully happen soon. Are there any recommendations except https://github.com/vtamara/fcc_trad/blob/master/Recomendaciones.EN.md?

@rfprod don't know if we have anything else than Vladimir's article. If you're going to work on this take a look at this repo https://github.com/alnero/FCC-Russian-Translation, this may decrease the translation time.

@bugron this information is not public, but can be shared privately in case of substantial need.

@rfprod good point, sorry.

I am in the process of finishing the getting started Russian translation.
Can someone explain how merging works with translation?
On Tuesday, February 2, 2016, Arsen Melikyan<EMAIL_ADDRESS>wrote:

@rfprod https://github.com/rfprod good point, sorry.
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/6232#issuecomment-178576381
.

@jniro do you mean this?

@jniro when you open a PR I'll QA and merge.

Can someone tell me how the merging with the translation works. I dont suppose to pull before pushing to my branch, second, I do not think I cant push it to my branch before merging what I have on my branch? is that right?

@jniro any updates?

I am having trouble with pushing my local work to hit hub. Can someone give
me some info on that. I learned that I can't pull since I am working on
different language. I did that last time and lost all the translation
On Tuesday, February 16, 2016, Arsen Melikyan<EMAIL_ADDRESS>wrote:

@jniro https://github.com/jniro any updates?
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/6232#issuecomment-184577124
.

@jniro try this article https://github.com/FreeCodeCamp/FreeCodeCamp/wiki/How-To-Create-A-Pull-Request-for-Free-Code-Camp

Hey guys!
I can help you with Russian and Romanian translation.

@IonatanMocan that would be awesome!

yep. contact me on gitter @ionatanmocan

@IonatanMocan done.

Do you guys still need help with translating to Russian? If yes conact me please @ernarkazakh

Can someone let us know where we had. I wanted to start working on it last
night but didn't want to do double work.
On Wednesday, March 9, 2016, Yernar Akimzhanov<EMAIL_ADDRESS>wrote:

Do you guys still need help with translating to Russian? If yes conact me
please @ernarkazakh https://github.com/ernarkazakh
—
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/6232#issuecomment-194513289
.

@jniro Can you please give a clear answer what is the status of translation, you are holding issue more than 2 weeks allowed, I would like to help with translation and can take your part.
P.S. Everybody willing to help please join gitter room and please check my glossary for translation so everybody would use single terms. If there is glossary already, please give link to it.
P.P.S If I want to translate other blocks I can open issue by myself or there is some algorithm to do it?

@Chichavl you may use React projects translation as a reference for templates for phrases for your glossary

I could also help with the translation. If needed, I can just seek out typos and grammar mistakes, but if you want, I can help with most of the work.

@Nikitaw99 this issue lasts for ages. If you want to do proofreading for Russian translation, you may do it for recently merged Russian Translation of Front End Development Certification: HTML5 and CSS. Not sure if someone except the interpreter did QA. At least, I was not requested to make any changes before the pull request was merged. A fresh perspective may reveal some flaws.

Hey all, any updates on this?
We have moved our translation collaboration to the https://github.com/FreeCodeCamp/TranslationExpansion repo. This issue can continue here, owing to the discussion length it has, but please log any new issues on the above repo.
Thanks, and Happy Translating!

Mrugesh Mohapatra<EMAIL_ADDRESS>於 2016年8月10日 星期三寫道：

Hey all, any updates on this?
We have moved our translation collaboration to the https://github.com/
FreeCodeCamp/TranslationExpansion repo. This issue can continue here,
owing to the discussion length it has, but please log any new issues on the
above repo.
Thanks, and Happy Translating!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/6232#issuecomment-238844858,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARhp2d61sclX9Vs9RU0FiszZu83zIoiVks5qebzqgaJpZM4HGYaJ
.

--
從 Gmail 行動版傳送


When invoke-customizations, why run_create_pipeline is always true

In order to apply changes in, say, global customizations, we have to run aft-invoke-customizations state machine.
As a first step, it get's all AFT provisioned accounts from aft-request-metadata table.
Then it starts to iterate over it with first action - create a customization pipeline for each of the accounts. But we already have all of them.
Creation of a pipeline defined by condition run_create_pipeline.
Apparently, it's always true.
In aft_common layer, I found this line:
invoke_event["account_provisioning"]["run_create_pipeline"] = "true"

why it's always true? It has to be false (since we're reinvoking existing pipelines to apply customizations), or at least customizable

It's actually not a bug it seems, it's more of a feature request. Could you please update the labels?

Hey @vaspoz, thanks for the request! I'll go ahead and create a backlog with the team to discuss optionally executing the customization pipeline creation process.

Hi @vaspoz,
This action is always set to true to ensure that an account's customization pipeline is in a healthy state (not deleted out of band, or an older version of AFT that uses the legacy roles, etc.). If the pipeline already exists, this stage is a quick no-op.


Missing StorageEvent equivalent functionality in v12

Suggestion path from customer:

The equivalent StorageEvent functionality is added to the SDK v12.
The bug in SDK v8 for Content-Disposition header is fixed and released.
There is a disconnect about the capabilities of SDK v12 and it is possible to do what is needed with StorageEvents.

Low priority as this has not been previewed and does not take in track 2.

For context, StorageEvent was an object passed into callback methods at various points of v8's ExecutionEngine. It had the connection object, request result, and operation context. I am fairly confident that the customer's ask, and any workflow's related to this type, can be supported via a policy. The customer just has to create a policy to process the outgoing request/incoming response as they see fit and add it to their pipeline.

In light of this customer ask, perhaps it would be a good idea to include some more thorough samples on creating a pipeline policy and including it in a client.

@rickle-msft let talk about this tomorrow.

spoken offline, as Rick say's we will add a sample that shows how to do this in track2.


Feature/apm 332220 change validation in installations scripts to accept only services with featuresets

looks ok to me, but please squash commits before merging it :)


Add Tab with Show Details

Provide show details and link back to kgwl.net

Instead of tab, probably going to just have a header on the show.


error TS5055: Cannot write file 'three.ts' because it would overwrite input file.

I set the allowJs option to true and set the outDir option,but it doesn't work.
my tsconfig.json:
{   "compileOnSave": false,   "compilerOptions": {     "outDir": "./dist/out-tsc",     "sourceMap": true,     "declaration": false,     "allowJs": true,     "moduleResolution": "node",     "emitDecoratorMetadata": true,     "experimentalDecorators": true,     "target": "es5",     "typeRoots": [       "node_modules/@types"     ],     "lib": [       "es2017",       "dom"     ],   } }

Which TypeScript Compiler? What editor, what tools? Was the the file opened by another editor? Which one(s)?
--need-more-info

I use Webstorm to write my program, and my js file has been opened by nodepad ++


Breadcrumbs in the modeline

Extracted from https://github.com/kakoune-lsp/kakoune-lsp/pull/715

Sorry I had to revert to the old version to be able to post my comments that's why I pushed thrice.
I think you pushed a stale version, see git range-diff 80afe65...46c30a6 --creation-factor=100

@krobelus I also did a force push before I've read your messages. Thanks for the changes! Helps me to learn Rust.

@krobelus Maybe it'd be easier to collaborate if I move branches from https://github.com/kakoune-lsp/kakoune-lsp/pull/711 and https://github.com/kakoune-lsp/kakoune-lsp/pull/715 to this repo? Although, not sure if the review history won't be lost.
Would nice perhaps to have them around as sort-of official kakoune-lsp WIP features.

@krobelus Hm, maybe, not sure too. Is it because you want to save the space?
There are a number of possibilities:

Skip N top most symbols
Trim long/multiline symbols (impl SomeTrait...)
Silence one-line symbols
...?

Is it because you want to save the space?

No, it's because it might be distracting that the modeline updates when I use j to step through struct fields.
Normally only line/column numbers change and often it's the same amount of digits so I'd not notice unless I'm actively looking.
Will test some more


Improve configuration API and support for initializers

Initializers like the following can now be used to configure repositories:
Rails.application.configure do |app|
  app.config.rom.repos[:yaml] = {uri: 'yaml:///config'}
end

Do you think we could yield rom's config instead of the whole app? Or maybe that wouldn't be flexible enough?

What would that look like? Rails.application.config.rom.configure { |config| ... }?

I thought about something like:
ROM::Rails.configure do |config|
  config.repositories[:cosmos] = { uri: "nasa://outerspace/galaxy" }
end

Close enough?
ROM::Rails::Railtie.configure do |config|
  config.repositories[:yaml] = {uri: 'yaml:///config'}
end

Can you tell why we can't do ROM::Rails.configure? Or you just think it's better to put it there on Railtie class?

The latter. Railtie is a weird kind of singleton already and I don't want it to creep into other parts. Maybe we should actually move things out of Railtie and make it really lightweight. This would lessen the confusion around config having two meanings (The global Rails.application.config and Rom's subset Rails.application.config.rom).

Yes this sounds very reasonable. I'm really unexperienced when it comes to building railties. I put things together to make them work somehow which was a success in itself for me. It definitely needs some reorganization and more love.


added readme to integrate with vue

Added a section in the docs to document how this can be integrated with vue.

What about this
https://github.com/ankurk91/vue-cleave-component/

Merged, cheers


Add large model selection in the "import audio file" option, please

Can you please add the large model selection in the import audio file option?
There are many errors when transcribing in high quality. It's not enough for an accurate transcription.
I could see that the first versions of Buzz had the large model as an option, but they didn't allow to import audios.

I removed it earlier because the larger models had high RAM requirements and seemed to crash on some PCs. I'll add it back in the next release.

Thank you so much 💯


Fixed #26739 -- Made reverse RemoveField handle non-nullable columns

Ticket: https://code.djangoproject.com/ticket/26739
Thanks @Gagaro for the report and @apollo13 for the initial patch
and pair programming.

An additional test for the questioner part would be great!

Yeah, but since the questioner has not tests whatsoever, we opted out of that. If you write the remaining ones, I'll happily add one :þ

@MarkusH you could work on that during the sprints scnr -- pretty sure we opened that at a sprint :D

Closing due to inactivity.


Optimal Penalizer Coef

Hi everyone, somebody knows if there is a way to find the optimal penalizer coef ?? something like a gridsearch ???
I know that there is a way plotting the gamma distribution, but i have the doubt if there is a numerical combination. I have tried with differerent numbers of penalizer coef and studying the population mean, p, q, v and the negative log likehood and still dont find an answer.
Here is an example.

In my context, the  best CLV (in terms of magnitude) is when i use a penalizer coef of 0.09, but i want to program a way when we use this method to find an optimal.
Thanks!!!

Hey
Just one question how you finalized that 0.09 is the best pernalizer value for CLV..?


Time Tracking Plugin - Enable Notifications for Current Farming Contract

Currently the plugin allows you to enable notifications for a specific patch via the bell toggle. Can a checkbox option be added to the plugin settings named "Farming contract notification" that will notify whenever the currently planted contract finishes?

I would love this.


Javascript dependencies needs an update - tiptap 2 update ahead

Dear @xndbogdan
We are working on updating tiptap to version 2: https://github.com/statamic/cms/pull/6043
As this might break your addon, we are willing to help updating it.
Could you update the Javascript dependencies?
If you got time and are willing to update javascript dependencies, we will help updating this addon, so it works with tiptap 2.

Sure.
I think I got some time this weekend to patch it up 😅

https://github.com/xndbogdan/statamic-bard-text-color/tree/feature/update-tiptap
This is WIP but should work with tiptap2

Released V3.0 of this package
It's now compatible with statamic 3.4


502 Server Error

Hi @alexander0042 ,
I noticed I wasn't getting any weather data today and checked logs. It appears the API call is throwing a 502 Server Error.
Logger: custom_components.pirateweather.sensor
Source: custom_components/pirateweather/sensor.py:841
Integration: pirateweather ([documentation](https://github.com/alexander0042/pirate-weather-ha), [issues](https://github.com/alexander0042/pirate-weather-ha/issues))
First occurred: 8:13:23 AM (1 occurrences)
Last logged: 8:13:23 AM

Unable to connect to PirateWeather: 502 Server Error: Bad Gateway for url: https://api.pirateweather.net/forecast/**API-KEY-MASKED**/40.0003497,-83.1582209?units=us

Any troubleshooting needed from me on my end?
Thanks,
Mike

Great. Thank you for your quick response!

Ok, and it's live! Seems to be working now, but let me know if you have any other issues ☀

I can confirm that everything looks to be working again!


Build API form generator service

Blocked on #254
Once the work in #254 is complete & we have the database structures needed to support dynamic form generation from configs, we can build the API endpoint/service that will build these forms dynamically on request.
As a developer
I want an API route/service for form generation
So that when it is called from an emission form it returns a generated schema
As a developer
When I call the API route with the general stationary combustion activity/source types
It returns the correctly generated forms for those inputs
AC:

[x] The reporting app in django has an api router
[x] A route & service exists for dynamic form generation (/api/reporting/generate-json-schema?)
[x] The service pulls a base schema based on the activity-sourceType pair from the base_schema table
[x] The service dynamically adds the fields from the reporting_fields table related to the gasType/methodology selections
[x] The service returns a json schema made up of the base schema & dynamic fields
[ ] ~The api route is protected so only logged in users can hit it~ let's do this in #253 to make it easier to start development on that card without protections in place
[ ] pytests for the service

Pulling this out of 'in progress' for now in favour of #270 as our understanding of what is required to build the forms has changed. Once #270 is in, some of the work already done here will provide the basis for building the form generator service in a way that is modified to handle the new database models. @pbastia @patriciarussellCAS


Fix interpolate function

string/interpolate function is buggy since if we pass 0 as token value it is parsed as nullish value and the token key is returned.
Instead of converting the token value as a boolean ( !!value), we should check if it is not null or undefined.
That could be an idea to add a new lodash-like isNil function to evaluate this condition.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Compile error:  "Cannot read property 'type' of undefined"

Not sure if this is a Node Sass error or a Create React App webpack build error.
When running CRA build script with Node Sass as a dependency, I was getting a vague error
Failed to compile.

Cannot read property 'type' of undefined
CompileError: Begins at CSS selector undefined

In local development, everything was working fine, no compile errors.
After a fun debugging session, I found that I had a translate3d transform value that was missing a 0
The following causes compile error:
transform: translate3d(0, 0);

The following fixed the compile error
transform: translate3d(0, 0, 0);

NPM version (npm -v): 6.13.4
Node version (node -v): v12.16.1
Node Platform (node -p process.platform): win32
Node architecture (node -p process.arch): x64
node-sass version (node -p "require('node-sass').info"):  4.13.1
npm node-sass versions (npm ls node-sass): 4.13.1

Transferring to libsass since it's a syntax issue


Reduce on the size of the table

How do i reduce the number of elements returned and rendered on the table because incase of multiple values, others are hidden like in the image

@16isaac89 you can skip being displayed in index view by using ii  option
ex.  body text textarea ii
more info read document: http://labs.infyom.com/laravelgenerator/docs/5.8/getting-started

@16isaac89 or you can even remove fields from DataTable class that is created. There will be a function getColumns. Keep only those columns that you want want to display.


Add courses as part of AddCommand

Closes #13

I've tested it locally.
Users are now able to add courses with the add command.
LGTM!
I like how lowercase course IDs were automatically displayed in uppercase. I also like how the relevant prompts are displayed when an invalid input, for example, CS2103,T, is entered.

Perhaps we should look into course validation in future? Currently, the courses entered has to be an alphanumeric string, and any alphanumeric string is considered a valid input.
For example, ARANDOMMODULE is also accepted as a valid course input.

As stated as a comment in the code, we should also look into how we could modify the edit command so that courses could be edited as well. The test cases have to be updated to reflect the changes here and in other parts of the code as well.

Thanks @chewjh1234 for the comprehensive review! For now, can merge to branch-addcourse in this repo instead of the master branch so we can work on the tests in the meantime.


Improving a sentence in getting_started.rst

Summary
Related https://github.com/Qiskit/qiskit-finance/pull/247
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 90.892%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3423

Relevant Lines:
3766

💛  - Coveralls


Hugo Ananke align help

Hi I am using the Hugo Ananke theme and I would like to change the font size and alignment on specific pages. I have so far been unable to do this - does anyone have any advice? Thank you in advance.

Have you tried adding a custom css file?

Thank you - I will try this now!

I'm having the same issue. I would like to change the width of the rendered text (on desktop). Right now it's centered and uses only ca. 33% of the width. Yes, I have tried adding a custom CSS File. The file is picked up correctly, but I simply don't know what exactly to put in there. My Initial test was simple:
p {
    text-align: left;
  }

But nothing happens. So my question is: What (exactly) do I have to put in my custom.css file to a) let the text (article) use more of the screen, and  b) Left Align the Body Text
Any help will be much appreciated!
Yonz


Added support for manual dispatching of events.

Google analytics has the ability for you to turn off the dispatchInterval by setting it to 0. This in turn requires the user to call [[GAI sharedInstance] dispatch]; to send the events to google. In our implementation we only send the events out when there is no other network activity happening. Adding this function to the proxy object allows us to control when the events are sent.

Thanks for this! I'll take a look and try to get it merged into the next release soon.

It's been a while, but I'm working to push out a 3.2.0 release which will fix up some issues with Android and include this pull request among others. The event will most likely be called manualDispatch when it is merged into the final release.
Cheers!


⚠ Panel - Hydeos has degraded performance

In e9ada67, Panel - Hydeos (https://pterodactyl.hydeos.fr) experienced degraded performance:

HTTP code: 200
Response time: 1881 ms

Resolved: Panel - Hydeos performance has improved in ea1707d after 11 minutes.


raftstore-v2: fix incorrect GC peer requests to source peer after merge

What is changed and how it works?

Issue Number: Close #15623
Depends on https://github.com/pingcap/kvproto/pull/1190
What's Changed:

After merge, target region sends GC peer requests to removed source peers,
however the region_id in requests is set to target region id incorrectly.
As results, source region removed peers may be left forever.

This commit fixes above issue by putting source removed_records to
merged_records, so that region id can be set correctly.

Related changes

PR to update pingcap/docs/pingcap/docs-cn:
Need to cherry-pick to the release branch

Check List
Tests

Integration test

Release note
None

/merge

/merge

/merge

/merge


Logo is not displayed correctly from site peviitor.ro

URL: https://peviitor.ro/
browser: Microsoft Edge
device: Computer
OS: windows

Steps to reproduce

Open the company page: https://dualit.ro/careers/scala-developer/
In the ribbon you will saw logo
Open peviitor URL
Put the name "dualit" in the search field
Press the button "Caută"
You will saw that the logo are not the same

Expected results

Logo to appear on the "peviitor.ro" site just like on the "dualit" site.

Actual results

Logo not appear on the "peviitor.ro" site just like on the "dualit" site.

Fixed, great catch!


This doesn't look right

https://github.com/vitaly-t/iter-ops/blob/f752a64c3fc3990850b5caa5374e3796b805c3e3/src/helpers.ts#L144-L162
So the code is saying:
If the value to get an iterable of is a promise. Wait for the process to fulfill. Return the promise's result wrapped in an iterator result; marked as not the last element of the iterator. The next value of the iterator is undefined and is the last element of the iterator.
This probably wasn't what was intended.
If I remove that code; the only test that fails is the one specifically testing this. So none of the actual operations are using this behavior and thus I believe this is probably left over code that isn't actually used anymore.

Nope, everything there is exactly by design, including the test.
You can test it locally, give it a promise, and see how it works ;)

In that case, why isn't the iterator just like this:
if (isPromiseLike(i)) {
     return {
         [$A](): AsyncIterator<T> {
             return {
                 next(): Promise<IteratorResult<T>> {
                     return i.then((value: T) => ({value, done: true}));
                 },
             };
         },
     };
 }

In that case, why isn't the iterator just like this:

Because that way the iterator has no termination logic at all, it's iternal :smile:

Ahh, right. Just me misremembering how iterators work

That's why you should see a lot of finish - bound logic in the library, to explicitly separate a sequence of values from its termination value.

I believe there is a small bug though.
With the following line, if i is only PromiseLike then the return value is only guaranteed to be PromiseLike, not a Promise. But an AsyncIterator's next function must return an actual Promise (at least according to the TypeScript typings).
return i.then((value: T) => ({value, done: false}));

I wouldn't consider it a bug, it's more like the thing we are not supposed to care about. We do only Promise-Like check for performance reasons. But documentation states everywhere that we work with promises only.
That promise-like check should be sufficient though.

hmmm, in that case; it might be best to change isPromiseLike back to isPromise and just add a note to say that it's not using instanceof for performance reasons.

add a note to say that it's not using instanceof for performance reasons

Not just performance reasons, but also because Promise is often overridden by custom implementations, like bluebird.
I think at this point best is to leave it as is, it's not that important. After all, it is not part of the public API, it's just an internal implementation aspect.

Promise is often overridden by custom implementations, like bluebird

I was going with the line of thought of custom implementations of promises not being of type Promise; but instead only being of type PromiseLike. Thus if PromiseLike isn't being supported then neither would custom implementations be.


Move documentation to Github

Move component documentation from confluence to github wiki.

JavaDoc link is broken: https://github.com/linksmart/service-catalog/wiki/SDK-in-Java

Moved completely: https://github.com/linksmart/service-catalog/wiki


Update README.md

Delete logo

Current coverage is 100% (diff: 100%)

Merging #3 into master will not change coverage

@@           master    #3   diff @@
===================================
  Files           1     1
  Lines          74    74
  Methods        12    12
  Messages        0     0
  Branches       11    11
===================================
  Hits           74    74
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 8adb78e...9cbdc09


Add support for use in #[no_std] crates.

A lot of derivative has paths to ::std::* hardcoded, even though much of this would be fully functional in a #[no_std] environment.

Support was added in v.1.0.2.


merge documents in gateway, basePath  is wrong. 3.0.0-SNAPSHOT

Sorry, I don't speak English, I used google translation.
I tried to merge documents in gateway, the basePath of webmvc is correct, but it is wrong in webflux. I think the cause of the error is in Swagger2ControllerWebFlux.

Webflux doesn't have a concept of context-path. Not sure how to fix it

Added extensibility mechanisms to override this behavior, specifically the ones for swagger 2.0 and open api 3.0


Hope that repeating tasks will work for subtasks

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.
[X] I am only requesting a single feature. Multiple changes should be split into individual requests, with links between them.
[X] I believe my requested feature will be generally applicable to other users, not just me: it is not uniquely tied to my personal workflow.

🔖 Feature description
When the recurring task is completed, the date behind the subtask also changes.
✔ Solution
I am managing some periodic review tasks
❓ Alternatives
No response
📝 Additional Context
No response

This request is not clear to me.
Please give example markdown lines that show the before and after of what you want.

This request is not clear to me. This request is not clear to me.
Please give example markdown lines that show the before and after of what you want.

like this

But the current situation is like this

Thanks but I do not have time to spend typing up markdown.
Please paste in the source. Thank you.

Thanks but I do not have time to spend typing up markdown.
Please paste in the source. Thank you.

The kanban plug-in used

No, that is another screenshot, not text.
Please read this section of the Tasks documentation on what we need, and why: Give us text to copy.
Thank you.

No, that is another screenshot, not text.不，那是另一个屏幕截图，而不是文本。
Please read this section of the Tasks documentation on what we need, and why: Give us text to copy.请阅读任务文档的这一部分，了解我们需要什么以及为什么：给我们要复制的文本。
Thank you. 谢谢。

Thank you for using the Tasks plugin.
感谢您使用任务插件。
And I appreciate your time in creating this request.
感谢您花时间提出此请求。
I have used an Optical Character Recognition program to try and extract the text from the images, to get the Markdown text:
我使用光学字符识别程序尝试从图像中提取文本，以获取 Markdown 文本：
Wanted behaviour:
想要的行为
- dasdasdasdas ✅ 2024-07-19
    - dasdasd 📅 2024-07-19 🔁 every day
- dasdasdasdas 📅 2024-07-20
    - dasdasd 📅 2024-07-20 🔁 every day

Current behaviour:
目前的行为
- dasdasdasdas ✅ 2024-07-19
    - dasdasd 📅 2024-07-19 🔁 every day
- dasdasdasdas 📅 2024-07-20
    - dasdasd 📅 2024-07-19 🔁 every day

The difference between the two is in the due date of the new set of tasks:
两者之间的区别在于新任务集的截止日期

I have just realised that I forgot to put the checkboxes in to the text above.
This is why I ask users to supply the text - it wastes a lot of time trying to recreated it by hand.
我刚刚意识到我忘记将复选框放入上面的图像和文本中。
这就是为什么我要求用户提供文本 - 尝试手动重新创建它会浪费大量时间。

This is a helpful reminder that when we release support for subtasks, we will need to work out logical behaviour for when there are recurring (repeating) tasks.
这是一个有用的提醒，当我们发布对子任务的支持时，我们需要制定出出现重复（重复）任务时的逻辑行为。
We will have to consider a wider variety of examples than given here.
我们必须考虑比这里给出的例子更广泛的例子。

Parent task not recurring, and one recurring subtask

Parent task not recurring, and multiple recurring subtasks

Parent task not recurring, and subtasks sub-tasks are recurring and some are not

父任务不重复，并且有一个重复子任务

父任务不重复，并且多个重复子任务

父任务不重复，子任务重复，有些不重复

Parent task recurring, and one non-recurring subtask

Parent task recurring, and one recurring subtask

Parent task recurring, and multiple recurring subtasks

Parent task recurring, and subtasks sub-tasks are recurring and some are not

父任务重复，以及一项非重复子任务

父任务重复，以及一个重复子任务

父任务重复，以及多个重复子任务

父任务重复，子任务重复，有些子任务不重复

This is a helpful reminder that when we release support for subtasks, we will need to work out logical behaviour for when there are recurring (repeating) tasks.

Okay, thank you for your answer, your answer is really good


Added a save button to the Cookie Preferences page

Trello card
https://trello.com/c/oyXVIACa
Context
Currently the user does not realise their changes are being saved as they update their cookie preferences
Changes proposed in this pull request

Added a save button
Only save when save button is clicked
Show a message to the user notifying them that their preferences are being saved
Hide the message if the user subsequently updates their preferences

Guidance to review
Probably worth downloading the PR locally and having a play with it

Yeh - good call - I'll talk to Ben in the morning.
I think it will need to be a separate link, this is all client side so there's no 'flash' - hence navigating away removes the ability to give feedback.


Secure Password Storage

Currently, the program stores plaintext passwords in the database. Sometimes, this dangerous practice can be hacked if a hacker intercepts your MongoDB database. Consider using password hashing instead. This is a modern practice for securing passwords rather than them being stored as plaintext. Consider using passlib and other python modules used for encryption. Passlib has a lot of hashing algorithms, but I suggest sha_256 or sha_512 as they are more secure than other hashing algorithms.

Yeah, this had been on my radar for a while, I just haven't gotten around to it. Expect an update in the next week

Sure. If you need any help with any password hashing, I can help you out with how it works. For now, I will close the issue.

okay, I have pushed a commit that adds password hashing and a ton of other good stuff.


How to cite?

Hello,
I am writing a paper, in which I used your package. How do I cite this in my paper?
Thanks,
Elious

Hi @EliousMondal,
I'm glad to hear that you found the package useful!
Unless your paper specifically focuses on FFT in a way that the use of this particular package is crucial, I don't believe you need to cite it. However, if you do decide to cite it, please be aware that this package is essentially a wrapper around PocketFFT, which performs the actual transformations.
I think making your code public and thus implying its dependencies would be ideal. Some journals also provide guidelines on how to cite software, so it's worth checking those. If you find it necessary to cite this package, you can use the following citation:
@misc{rocket_fft,
author = {Styfen Schär},
title = {Rocket-FFT},
year = {2024},
version = {0.2.5},
note = {Available from https://github.com/styfenschaer/rocket-fft/releases/tag/v0.2.5},
}
In any case, I appreciate that you’re taking citations seriously!
Best regards,
Styfen

Thanks a lot. My code is written in python using numba and also FFT is the most important ingredient of my work. So your wrapper is an essential necessity for my current version. Thanks for sharing. I will cite this and PocketFFT also. The code and paper should be available within next few months.


chore: add documentation generator

closes: #6

There are some files that can be ignored for adding to Git, the .yarn folder for example. That's not something you want in a Git repository. I see the .gitignore already has a .yarn mention in it, so something is going wrong here

@ca-d can you look at this after the release?


你好，请问参考文献可以分享一下吗？谢谢

[1]Ni, F., Hao, J., Lu, J., Tong, X., Yuan, M.J., Duan, J., Ma, Y., & He, K. (2021). A Multi-Graph Attributed Reinforcement Learning based Optimization Algorithm for Large-scale Hybrid Flow Shop Scheduling Problem. Proceedings of the 27th ACM SIGKDD Conference on Knowledge Discovery & Data Mining.
[2]Ruiz, R., & Rodríguez, J.A. (2010). The hybrid flow shop scheduling problem. Eur. J. Oper. Res., 205, 1-18.
Ahmet Bolat, Ibrahim Al-Harkan, and Bandar Al-Harbi. 2005. Flow-shop scheduling for three serial stations with the last two duplicate. Computers & Operations Research , Vol. 32, 3 (2005), 647--667.
[3]Fernandez-Viagas, V., & Framiñan, J.M. (2020). Design of a testbed for hybrid flow shop scheduling with identical machines. Comput. Ind. Eng., 141, 106288.
[4]Dabiri, M., Yazdani, M., Naderi, B., & Haleh, H. (2021). Modeling and solution methods for hybrid flow shop scheduling problem with job rejection. Operational Research, 22, 2721-2765.
[5]Jun, S., & Park, J. (2015). A hybrid genetic algorithm for the hybrid flow shop scheduling problem with nighttime work and simultaneous work constraints: A case study from the transformer industry. Expert Syst. Appl., 42, 6196-6204.
[6]Ruiz, R., & Maroto, C. (2006). A genetic algorithm for hybrid flowshops with sequence dependent setup times and machine eligibility. Eur. J. Oper. Res., 169, 781-800.
[7]Naderi, B., Zandieh, M., & Roshanaei, V. (2009). Scheduling hybrid flowshops with sequence dependent setup times to minimize makespan and maximum tardiness. The International Journal of Advanced Manufacturing Technology, 41, 1186-1198.
[8]Paternina-Arboleda, C.D., Montoya-Torres, J.R., Acero-Dominguez, M.J., & Herrera-Hernandez, M.C. (2004). Scheduling jobs on a k-stage flexible flow-shop. Annals of Operations Research, 164, 29-40.
[9]Naderi, B., Ruiz, R., & Zandieh, M. (2010). Algorithms for a realistic variant of flowshop scheduling. Comput. Oper. Res., 37, 236-246.
多阶段混合排产问题研究报告—高级运筹学第6组.pdf

谢谢，请问如果想引用你们的工作，是直接引用这个链接吗？

谢谢，请问如果想引用你们的工作，是直接引用这个链接吗？

是的

好的谢谢


state sync modifications

this PR is an experiment in testing state sync from engine.

Closing since this PR is out of date.


Update author list

From Jim:

Reading this to review it, I noticed that neither your name nor mine is in the list of contributors at the top. Maybe we should update that. And maybe include maskrays real name too.

Fixed via https://github.com/riscv-non-isa/riscv-elf-psabi-doc/pull/220