Capacitor 5 update

This PR updates this plugin so it can be used with Capacitor 5. All steps from: https://capacitorjs.com/docs/updating/plugins/5-0 were followed + some updates to support BuildConfig from gradle 8+
This update does not modify minSdkVersion so it should work for clients who are switching to capacitor v5. Any suggestions are welcomed, maybe switching to a greater version like 7 (because this targets Java 17 and gradle 8+)
@pazlavi

Waiting for this PR to merge so I can integrate in my project using Cap 5

Same here, waiting for this PR to get merged to support cap5

+1

For people not wanting to wait: you can do npm i https://github.com/nowjobs/appsflyer-capacitor-plugin/tree/main to get capacitor 5 support inside your projects.
Once Appsflyer releases the Capacitor 5 version, then this fork will be removed.

@MattijsE I'm curious, why did you added the dist folder?

@MateiNenciu When using your branch I couldn't get it to compile without an extra build step, since it couldn't find the definitions. So by adding the dist folder this problem is solved.

@MattijsE adding dist folder can be avoided.
In package.json add "prepare": "npm run build". I use prepare in my plugins when I want to install directly from github without publishing to npm etc.

@MattijsE Is there any target date to merge this PR?

The target date is when the maintainer decides to merge my PR or do the changes itself

@pazlavi can you provide us update on this PR please

Updating the plugin and releasing a new version is on our roadmap for Q3. I don't have an ETA at this moment.

So basically you are saying that we should wait for Q3 to merge a simple platform update?

We'll not release any new version without running a proper QA cycle and documenting the breaking changes. At this moment, our resources focus on other tasks.

Ok, anyone who want to use Capacitor v5 please feel free to use my fork till this understaffed corporation will handle this. Capacitor v5 was announced from april, it's not like it was released 2 days ago.
I will strongly advise my data dept to look for an alternative MMP

I effectively used the fork from @MateiNenciu in my current project and everything is working well. @suyundukov @gelinger777 @parthnayak7448 could you please find someone to approve and merge this PR once and for all 😄 ?
Thank you in advance !

6.12.1  release and support now Capacitor 5


next-sass: CSS is not included when running build for all modules.

Expected Behavior

running yarn build with @zeit/next-sass and using css-modules should result in ALL css being compiled and available in the build site.
Current Behavior

That's not the case.  Only CSS for a single page / component is included, everything else is unstyled.
Steps to Reproduce (for bugs)

clone this repo: https://github.com/Kyle-Mendes/next-sass-modules
run yarn build
run yarn start
visit: localhost:3000/

The button may or may not be pink

visit localhost:3000/test

The headline may or may not be large and green.

Likely, the button is unstyled and the headline is styled.
My main project has hundreds of components, and right now, only one style is applied for the entire site.  If components in the example was larger, there would still be only one styled component.

Note that this issue exists when using the configuration in #168  as well.

the same issue when I use next-less without cssModule.

Did you upgrade to the latest version?

upgrade next-less to the latest version solved my problem, thank you very much!


arrow function binding test 2

Seems like the test should be assert.strictEqual(fn.call(anotherObj), bound);, the same as test one, since you are proving that a new context can't be provided.

Do you mean this test here?
https://github.com/tddbin/katas/blob/master/katas/es6/language/arrow-functions/binding.js#L16
or another one?
All katas are stored in this repo https://github.com/tddbin/katas

That's it :)

is that broken?

I guess there is not any other way to make it fail.
If you change it toassert.strictEqual(fn.call(anotherObj), bound); then it always passes,
var anotherObj = {};     assert.strictEqual(fn.call(anotherObj), anotherObj); just seemed really strange way to compare, I think I just needed to sleep on it :P

I think I know what you mean
it('can NOT bind a different context', function() {
    var bound = new LexicallyBound();
    var fn = bound.getFunction();
    var anotherObj = {};
    assert.strictEqual(fn.call(anotherObj), anotherObj);
});

would better be
it('can NOT bind a different context', function() {
    var bound = new LexicallyBound();
    var fn = bound.getFunction();

    var anotherObj = {};
    var functionResult = fn.call(anotherObj);

    assert.strictEqual(functionResult, anotherObj);
});

more explicit of what is compared, is that what could be improved?

That looks a little better.  I honestly think I just needed some sleep.  Thanks for taking a look at it though.


Top-level Anchor tags break out of the SVG namespace

Describe the bug
On this example the red square doesn't render
https://playground.solidjs.com/anonymous/81577cd9-1162-4cf9-b312-9b8197bae11b
The underlying issue is that xmlns is lost in at least two situations:

on component change https://playground.solidjs.com/anonymous/133e8b12-defc-4a89-9c42-9cc401b3b602
when reactivity takes over and for example a Show toggles

I believe the only way to fix this is to create a context with the value of the xmlns and use it to create any children. This is not about SVG, this will happen with any XML element/future element that needs the xmlns to work correctly.
Your Example Website or App
https://playground.solidjs.com/anonymous/81577cd9-1162-4cf9-b312-9b8197bae11b
Steps to Reproduce the Bug or Issue

Expected behavior

Screenshots or Videos

Platform

Additional context
No response

In your specific case it is because  is both an SVG tag and HTML tag and we treat it as HTML tag. It's hard to know out of context which one it should be. If I remove the  or use any other SVG element it works. Trying to figure this out at runtime seems like a significant overhead given we clone templates, which we don't introspect at the time of cloning (for good reason). I don't know enough about how these namespaces work, but I imagine if we always treated  as XML that would be problematic in others.
In general though what we do is decide based on the topmost element if it is an SVG element. If it is we clone it inside and SVG and then extract it from there. That preserves it. We assume that any SVG partial would need to be top-level as if there was a HTML element present it would break it. And there would need to be a nested <svg> element. The best I can think of doing here is changing our heuristic to look at more than the topmost tag and if it finds any SVG element not inside an <svg> it gets SVG treatment. Or rather, if the topmost element is an  then we should check its children.
I have no idea how to support XML in the general sense and that is probably beyond the scope of the project at this point.

For context in this scenario React creates the  element under the SVG namespace, as @titoBouzout expected.
It's a tricky situation, because how does one fix it in general? Let's say I have a component that renders something inside an SVG, this components creates an  element, which namespace should be used for it? I think we can't know until it is attached somewhere, because if it's attached right inside an <svg> then we should use the SVG namespace, but maybe it gets rendered inside a <foreignObject> inside an SVG, or some other combination of what's possible. Basically until the node is attached to the DOM it looks like we can't know for sure what namespace to use.
Maybe there's some kind of assumption that can be made to 90% fix this with little downsides.

Basically, whenever you find a xmlns attribute you create any children with that namespace, doesn't matter the tag names nor anything else. The foreingObject, you can think of it as a tag that resets the xmlns, the browser does it alone, nothing special about it.
Following that simple rule, it will make it work for most cases, if not all, without trying to be xml compliant.
If you can keep the xmlns attribute in a context (at compile time), then you can switch the template function based on that context.
For example, in the following snippet, if _el$ has xmlns attribute then children getter is wrapped in a function that switches _$template to use that namespace.
const xmlns = '..' // compile time xmlns value of `_el$`
 _$insert(_el$, _$createComponent(Show, {
      get when() {
        return read();
      },
      get children() {
        return useXMLNSTemplate(xmlns, () => _tmpl$())();
      }
    }), _el$4);

and _$template could do something in the lines of
function _$template(xmlns, content) {
  const template = document.createElementNS(xmlns, "template");
  template.innerHTML = content;
  const element = template.cloneNode(true);

  // should return childNodes instead, this is for demo purpooses
  return element.firstChild;
}

const el = _$template(
  "http://www.w3.org/2000/svg",
  `
        <rect fill="red" x="25" y="25" height="50" width="50" />
      `,
);

This should work with partials, playground https://playground.solidjs.com/anonymous/44630452-7e01-4097-90ce-84a080538c92
The issue @fabiospampinato mentions, about not knowing where you are going to be, that's right and expected, you can mess up like that with regular dom apis too. (ie creating a regular link to append it in a svg tag, it wont work). For example, there's no way to know where this is going to end:
const Link = () => example.org

That's why the framework should just handle the xmlns attribute, because the user is aware that they are creating an element out of nowhere, so they will do the following, and will work as expected when the framework respects the xmlns attribute:
const Link = () =>
   example.org

Hmm.. understanding the proposed solution it would need to be a bit more complicated than that because it can escape the analyzable scope. In our case we only look in the scope of a single JSX block, so like escaping into expressions and going back into JSX resets it. And I believe that is the case for children of components, like <Show> and definitely if people use the callback form. So there are always going to be exceptions.
And the namespace outside of analyzed scope would need to be restored in the future. Like the SVG could continue into a component with a Show in it. At which point the component does not contain an element that has a xmlns attribute. So if we set it like a global context, then we could read it in the component at runtime. But then we'd need to be able to restore it when the <Show> changes to continue propagation from that point. Basically with this sort of approach compilation alone would be insufficient.
This is a significant overhead I think. Mostly in that right now we create the template once and this would push us to allow the templates to be created either way.  We'd also need a way of collecting and restoring xlmns at the base of each JSX template even if it didn't contain xmlns itself.
Although as pointed out (and with what I proposed) we currently have no way to know a lone  is in the right context otherwise as we can't tell that from compilation. It means currently stuff like @solidjs/router doesn't work properly when people use  in SVG. But a local compiler-driven solution also wouldn't handle that either. We'd need a full runtime context solution I described above to handle this completely properly.
This is always going to be tricky without a solution that impacts how every element is created.
<svg>

    <Show when={show()} fallback={<rect />}>Some Text</Show>

</svg>

So far as I know the overlapping elements are ,


NPOI.POIFS.FileSystem.NotOLE2FileException

Good day can help me with the following error that is generated when uploading an excel file and trying to read it. in asp.net core 2.0
Where stream is of type IFormFile file     ( file.OpenReadStream())
1. ---line where the error occurs--------
  var workbook = new HSSFWorkbook (stream);
2. ---line where the error occurs--------
var workbook = new XSSFWorkbook(stream);
-------------Version NPOI--------------

---------Error------------
NPOI.POIFS.FileSystem.NotOLE2FileException: 'Invalid header signature; read 0xE011BDBFEFBDBFEF, expected 0xE11AB1A1E011CFD0 - Your file appears not to be a valid OLE2 document'

@paulPlaza  I can't reproduce you error .  Can provide more detailed information, such as your environment, code, documents


Electron win32 ia32 app is not working but x64 is

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] I have searched the issue tracker for an issue that matches the one I want to file, without success.

Hi,
I am using Windows 7 64bits and have used electron packager to build my project to both win32 ia32 and win32 x64
The x64 version is working perfectly on windows 64 bit machines but the ia32 version is not working on neither 32 bit or 64 bit machines. I click the 32bit executable and absolutely NOTHING happens.
This is very frusturating because I haven't found a single person who's had this issue before.
Please help!
Thank you.
Console output when you run electron-packager with the environment variable DEBUG=electron-packager. Please include the stack trace if one exists.
D:\GP>electron-packager . --arch=ia32
  electron-packager Electron Packager 9.0.0 +0ms
  electron-packager Node v6.11.2 +15ms
  electron-packager Host Operating system: win32 (x64) +0ms
  electron-packager Packager Options: {"":["."],"all":false,"derefSymlinks":true
,"download":{"strictSSL":true},"overwrite":false,"prune":true,"quiet":false,"arc
h":"ia32","DEBUG":"electron-packager","dir":".","protocols":[]} +0ms
  electron-packager Target Platforms: win32 +0ms
  electron-packager Target Architectures: ia32 +0ms
  electron-packager Inferring application name from name in D:\GP\package.json +
0ms
  electron-packager Inferring appVersion from version in D:\GP\package.json +0ms

  electron-packager Inferring target Electron version from electron in D:\GP\pac
kage.json +16ms
  electron-packager Application name: gp +32ms
  electron-packager Target Electron version: 1.6.11 +0ms
  electron-packager Ignored path regular expressions: [ '/node_modules/electron(
$|/)',
  '/node_modules/electron-prebuilt(-compile)?($|/)',
  '/node_modules/electron-packager($|/)',
  '/\\.git($|/)',
  '/node_modules/\\.bin($|/)',
  '\\.o(bj)?$' ] +0ms
  electron-packager Downloading Electron with options {"strictSSL":true,"platfor
m":"win32","arch":"ia32","version":"1.6.11"} +0ms
Skipping win32 ia32 (output dir already exists, use --overwrite to force)

D:\GP>electron-packager . --arch=ia32 --overwrite --DEBUG=electron-packager
  electron-packager Electron Packager 9.0.0 +0ms
  electron-packager Node v6.11.2 +0ms
  electron-packager Host Operating system: win32 (x64) +0ms
  electron-packager Packager Options: {"":["."],"all":false,"derefSymlinks":true
,"download":{"strictSSL":true},"overwrite":true,"prune":true,"quiet":false,"arch
":"ia32","DEBUG":"electron-packager","dir":".","protocols":[]} +0ms
  electron-packager Target Platforms: win32 +0ms
  electron-packager Target Architectures: ia32 +0ms
  electron-packager Inferring application name from name in D:\GP\package.json +
0ms
  electron-packager Inferring appVersion from version in D:\GP\package.json +16m
s
  electron-packager Inferring target Electron version from electron in D:\GP\pac
kage.json +0ms
  electron-packager Application name: gp +31ms
  electron-packager Target Electron version: 1.6.11 +0ms
  electron-packager Ignored path regular expressions: [ '/node_modules/electron(
$|/)',
  '/node_modules/electron-prebuilt(-compile)?($|/)',
  '/node_modules/electron-packager($|/)',
  '/\\.git($|/)',
  '/node_modules/\\.bin($|/)',
  '\\.o(bj)?$' ] +0ms
  electron-packager Downloading Electron with options {"strictSSL":true,"platfor
m":"win32","arch":"ia32","version":"1.6.11"} +0ms
Packaging app for platform win32 ia32 using electron v1.6.11
  electron-packager Creating C:\Users\Taha\AppData\Local\Temp\electron-packager\
win32-ia32-template +2s
  electron-packager Extracting C:\Users\Taha\AppData\Local\electron\Cache\electr
on-v1.6.11-win32-ia32.zip to C:\Users\Taha\AppData\Local\Temp\electron-packager\
win32-ia32-template +16ms
  electron-packager Initializing app in C:\Users\Taha\AppData\Local\Temp\electro
n-packager\win32-ia32\gp-win32-ia32 from C:\Users\Taha\AppData\Local\Temp\electr
on-packager\win32-ia32-template template +3s
  electron-packager Ignored paths based on the out param: [ 'D:\\GP\\gp-darwin-x
64',
  'D:\\GP\\gp-linux-ia32',
  'D:\\GP\\gp-linux-x64',
  'D:\\GP\\gp-linux-armv7l',
  'D:\\GP\\gp-linux-arm64',
  'D:\\GP\\gp-mas-x64',
  'D:\\GP\\gp-win32-ia32',
  'D:\\GP\\gp-win32-x64' ] +3s
  electron-packager Pruning modules via: npm prune --production +0ms
  electron-packager Renaming electron.exe to gp.exe in C:\Users\Taha\AppData\Loc
al\Temp\electron-packager\win32-ia32\gp-win32-ia32 +31s
  electron-packager Running rcedit with the options {"version-string":{"FileDesc
ription":"gp","InternalName":"gp","OriginalFilename":"gp.exe","ProductName":"gp"
},"file-version":"1.0.0","product-version":"1.0.0"} +0ms
  electron-packager Moving C:\Users\Taha\AppData\Local\Temp\electron-packager\wi
n32-ia32\gp-win32-ia32 to D:\GP\gp-win32-ia32 +203ms
Wrote new app to D:\GP\gp-win32-ia32```

Have you tried console.log-style debugging (i.e., figure out where in your app code it's not printing anything)?
If that doesn't work, could you perhaps try a different version of Electron?

I dont know how i can use console.log to find out what's wrong in this case. the app is finished and tested and works perfectly in the x64 build and doesnt even launch in the ia32 build.
I have the latest version of electron but ill try older ones since im desperate for a solution.

Putting a console.log(...) statement at the very top of your main script would tell you whether the main script is even running.

Hello!
So, what I ended up doing is installing node 32bit and Electron 32bit and rebuilding everything and it worked.
Problem solved, I guess.


Data API should have offline mode when working locally

Our Data API, even the local one, saves files to R2. This makes our life easy in some ways (e.g. fallbacks to production API are handled by the worker and you don't need to download the entire catalog), but it's not a great dev experience if you have poor internet connection
We could add it either with:

Enable Grapher to save files locally
Have a simple HTTP server serving those files as “local” Data API

or by rewriting endpoint /admin/api/data/variables/data/819356.json in grapher Admin and adding support for local files.
We should also have make command for fetching the entire catalog from R2 locally.

Keep it open. I'll do it one day, I promise...


In small device "Add 6th Signing Device" is not visible

Resolved,
https://github.com/bithyve/bitcoin-keeper/assets/61345820/3b9017cb-c6d4-4ef9-8241-1fe9f9c797de

Verified this issue on  Keeper dev app v 1.1.7(262)


Implement Dankdrad-3

IPA becomes the default proof system.
Not covered by this PR:

endianness when serializing data, this PR is using big endian when the spec requires little endian
deduplication of commitments/fis/yis in multiproof

Codecov Report

Merging #113 (e4ac545) into master (96e9b15) will decrease coverage by 1.74%.
The diff coverage is 97.36%.

@@            Coverage Diff             @@
##           master     #113      +/-   ##
==========================================
- Coverage   86.80%   85.05%   -1.75%
==========================================
  Files           6        8       +2
  Lines         773      542     -231
==========================================
- Hits          671      461     -210
+ Misses         78       62      -16
+ Partials       24       19       -5

Impacted Files
Coverage Δ

config.go
66.66% <ø> (-28.79%)
:arrow_down:

empty.go
77.77% <66.66%> (+44.44%)
:arrow_up:

hashednode.go
52.00% <85.71%> (ø)

tree.go
85.82% <97.61%> (+2.57%)
:arrow_up:

config_ipa.go
100.00% <100.00%> (ø)

encoding.go
84.31% <100.00%> (ø)

ipa.go
100.00% <100.00%> (ø)

proof_ipa.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Decoration RTL

Decoration.createBoxPainter should take a TextDirection.
Decoration.padding should return an EdgeInsetsGeometry.

Depends on Border RTL (https://github.com/flutter/flutter/issues/11566).

Depends on BorderRadius RTL (https://github.com/flutter/flutter/issues/11568).

Depends on DecorationImage RTL (https://github.com/flutter/flutter/issues/11999).
One thing we probably don't need to convert is BoxShadow. RTL apps still get shadows from the same light source.

Depends on LinearGradient and RadialGradient RTL (https://github.com/flutter/flutter/issues/12000).

Depends on paintImage RTL (https://github.com/flutter/flutter/issues/12001).

This and all the subissues are done now.


Added inflight_requests metrics (version 5 and later).

What does this PR do?
Fixes #785
Motivation
User request
Review checklist

[ ] PR has a meaningful title or PR has the no-changelog label attached
[ ] Feature or bugfix has tests
[ ] Git history is clean
[ ] If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo

Additional Notes
Rearranged tests a bit, going to rearrange more at a later point

Codecov Report

Merging #2360 into master will increase coverage by 4.74%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2360      +/-   ##
==========================================
+ Coverage      80%   84.75%   +4.74%
==========================================
  Files         210        4     -206
  Lines       14447      492   -13955
  Branches     1561      101    -1460
==========================================
- Hits        11559      417   -11142
+ Misses       2488       47    -2441
+ Partials      400       28     -372


fix missing custom message in multi line string diff

What's the motivation for this PR?

Multiline string diff did not allow custom error message. Support was added.

There are no changes visible anymore. I guess this has been implemented already.


[Suggestion] use yarn instead of npm for performance/reliability

Just a suggestion -- it's ultimately still npm in the background:
https://www.sitepoint.com/yarn-vs-npm/

Yep, this duplicate issue should be closed.


DOC: Incorrect documentation for nan_policy='propagate in multiple stats functions

Raising as a follow up to #9815. The documentation for nan_policy='propagate' is either incorrect or ambiguous in many scipy.stats functions and should be corrected. The current documentation reads:
https://github.com/scipy/scipy/blob/1a2be4e025701f4b767521463f25f58ac2f4f5e4/scipy/stats/stats.py#L2205-L2208
According to the docstrings,  propagate should return nan if any nan values occur. However, the usual behaviour (as demonstrated by sem) is to perform the calculations, including nans, and see what happens. This will usually result in a nan value.
>>> import numpy as np
>>> from scipy.stats import sem
>>> a = np.arange(16, dtype=np.float64).reshape((4, 4))
>>> sem(a) #sem with no nan values
array([2.5819889, 2.5819889, 2.5819889, 2.5819889])
>>> a[1, 1] = np.nan
>>> sem(a, nan_policy='propagate')
array([2.5819889,       nan, 2.5819889, 2.5819889])

The documentation should be updated to accurately reflect the de-facto behaviour.
@chrisb83

I'm closing this issue as a final consensus of how to use propagate was never reached.


Improvements on phone finder action

This pull request builds upon #40 and addresses:

ForegroundService not starting automatically (at least on my phone)
Notification instead of AlertDialog because otherwise the phone keeps ringing in loop and cannot be stopped (when app's activity is not active but it's just listening through the foreground service).
PhoneFinder as a standalone utils class / object, like Flashlight or CameraCapture.

Link #42


Only change one region of the sequence?

Hi,
Thank you again so much for your awesome software! I was wondering if there's a way to only modify a region of the seed sequence when running gradient ascent (i.e. of a 60 nt sequence, only update the first 30 nt and leave the rest unchanged)? Is there an option for this or a way to incorporate into the loss function?
Thanks for your help!

Hi,
I'm glad you're finding the package useful!
Just to clarify, this is how I interpret what you want to do: Your predictor model takes 60nt sequences as input. You would like to only optimize the first 30nt of the sequence and leave the remaining 30nt fixed at some constant onehot-coded sequence content.
If the above is incorrect, could you help me understand a little better what it is you're after?
If the above is correct, then yes, that functionality is implemented in the package :)! Overly long explanation follows:
The build_generator-function of the seqprop.generator module takes two keyword arguments you should use to achieve this:
sequence_templates - This parameter expects a list of "sequence templates", one for as many sequences you are optimizing (the n_sequences parameter), of course you can set all sequence templates to be identical if you want to optimize many instances with different random inits.
Each sequence template is a string of, in your case, 60nt. The letters A/C/G/T specifies that you wish to lock that specific base position at a constant, non-changeable nucleotide. The letter N specifies that this base position should be optimized by SeqProp, and will either be initialized randomly or by the init_sequences-parameter (see below). The letter X specifies that the base position should be uniformly initialized (0.25 on all nucleotides) but not optimizable.
E.g. "NNNNNNNNNNNNNNNNNNNNNNNNNNNNNNACGTTACGTTACGTTACGTTXXXXXXXXXX" means the first 30nt are optimized by SeqProp, the next 20nt are locked at sequence ACGTTACGTTACGTTACGTT and the final 10nt are uniform 0.25-bases.
See the following notebooks for examples on how to use this parameter:

https://nbviewer.jupyter.org/github/johli/seqprop/blob/master/examples/apa/seqprop_aparent_isoform_optimization.ipynb
https://nbviewer.jupyter.org/github/johli/seqprop/blob/master/examples/apa/seqprop_aparent_cleavage_optimization_init_sequences.ipynb

init_sequences -  In the second example notebook, we are using both the sequence_templates and init_sequences parameters together. The init_sequences parameter specifies what to initialize the optimizable bases  (The 'N'-letters) to in your sequence template. If init_sequences is not set, the 'N'-letters are randomly and uniformly initialized between -0.2 and +0.2 in logprob space. The letter 'X' in init_sequence tells SeqProp that this base position should be randomly uniformly initialized (or whatever constant sequence is in the corresponding position on the sequence template).
E.g. "AAAAAAAAAAAAAAAAAAAAXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX" means the first 20nt should be initialized to A:s, and the next 40nt should be initialized to whatever is in the sequence template (i.e., constant sequence if that's in the sequence template or randomly if the corresponding bases are 'N'-letters).
Let me know if it feels unclear or if you have follow up questions, I'm happy to help :)!
Best,
Johannes

Hi Johannes,
Thank you for your detailed response! That is incredibly helpful.
Building off of my last question, do you have any functionality built-into SeqProp where you only change the first 30 nucleotides of a 60 nt sequence and part of the remaining un-optimized 30 nt are changed dynamically to be the reverse complement of part of the first 30 nts?
i.e. in the above example you gave: NNNNNNNNNNNNNNNNNNNNNNNNNNNNNNACGTTACGTTACGTTACGTTXXXXXXXXXX - the Xs are the reverse complement of nucleotides 10 thru 20 in the N region of the first 30 nt?
To rephrase my question: is there a way to only optimize part of the sequence, but dynamically change part of the sequence or template in response to changes in the first part? Or would this be something best put in a custom loss function?
Thanks again for your help and for this fantastic resource!

Hi Katie,
The SeqProp package should now have support for what you're after! I added a generalized parameter where you can supply any custom transform function between the generator and predictor to transform the PWM (or Sampled Onehot), for example to reverse complement parts of it.
Specifically, I updated the build_generator-function with a new keyword parameter, pwm_transform_func, which expects one lambda function (same style as how you define the loss function). This function should take the PWM as input and return the transformed PWM, which in turn will be passed to the predictor.
Please see the notebook example I added at: https://nbviewer.jupyter.org/github/johli/seqprop/blob/master/examples/basic/seqprop_basic_sequence_transform.ipynb
Here I use a very simple dummy "Predictor model" that simply scores the number of "GTT" motifs in the optimized sequence, and the loss function simply tries to maximize that score. I.e., instead of having the complex APA model I went with a simpler predictor to keep the example concise.
The get_revcomp_transform-function is where I define the sequence transform that takes the first 30nt of the optimized PWM, reverse complements it, and appends it as the next 30nt of the sequence (hopefully this is similar to how you wanted to use it?). The get_revcomp_transform_opt-function is an optimized version of this function, if you want it.
Let me know if the added functionality is enough for your needs, or if there is more you want added! Also let me know if you experience any bugs with it, I haven't tested it extensively yet.
Best,
Johannes

Awesome!! Thank you so much! That is perfect.
Thank you again!
Best,
Katie

Perfect, glad it helped :)!
Let me know if you have any other suggested improvements, I’d be happy to implement them.
Best,
Johannes


Default setting for KeyGenerator ColumnName and algorithm

Question
In my application, I am only doing database level sharding. There is no table level sharding. So, only setting the DefaultDatabaseShardingStrategyConfig works fine. But, if I want to use snowflake for key generation for my tables, I need to add TableRuleConfiguration for all my tables to specify the algorithm and the column name.
Is there anyway to set a default key generation config as well?

@terrymanu is there any current setting for this?

You can see the official documents carefully. There are detailed descriptions
e.g:defaultKeyGenerator、keyGenerator https://shardingsphere.apache.org/document/current/cn/manual/sharding-jdbc/configuration/config-yaml/

@geomonlin Thanks for your reply. I found the setting in the doc but actually I am looking for a way in io.shardingsphere v3.1.0 . My bad, I should have mentioned this earlier.

@geomonlin Thanks for your reply. I found the setting in the doc but actually I am looking for a way in io.shardingsphere v3.1.0 . My bad, I should have mentioned this earlier.

This is the official website.
https://shardingsphere.apache.org/


Should we freeze dependencies?

If we do not have concrete versions of dependencies in requirements, things like error in #54 might be happening from time to time. On the other hand, if we freeze the dependencies, we should check for updates from time to time, which means more work. I'm leaning towards automatic updates and  letting the build fail from time to time. We test things fairly regularly now, so we should be able to catch and repair breaking changes. What do you think?

I am for updated dependencies. But that does not mean some of them shouldn't be freezed - like we did in the case of the newest nltk 3.2.1 which crashed when computing BLEU, when we replaced it with 3.2.0.. We should design performance tests for this (train on a very small data and evaluate and keep the scores from these tests). Maybe it's also meaningful to measure running time of the test.

Keep dependencies updated. And never use NLTK again :-)

OK.
To keep meaningful test scores, we would need to fix random seeds (I don't think it works at the moment). I don't think there is a reasonable way to reliably measure meaningful running time.

It does not have to be reliable - if the timing is suspicious, you can re-run the test and if the timing is still bad, you can investigate.


Catalog - built in Apps shows unknow schema error

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

run the latest rancher:master-head in HA mode
go to Apps -> Charts
try to deploy any built-in apps from the list

Result:
None of them can be deployed. The following error shows:

on the latest master-head, HA installation, trying to deploy apps from the built-in list, the following error will show in the last step.
resource name may not be empty

tried to install the monitoring app on a freshly-installed mater-head HA setup via the dashboard, and here is the POST request looks like
curl 'https://my-server/v1/catalog.cattle.io.repos/default/dev-charts?action=install' \
  -H 'authority: my-server' \
  -H 'pragma: no-cache' \
  -H 'cache-control: no-cache' \
  -H 'accept: application/json' \
  -H 'dnt: 1' \
  -H 'x-api-csrf: 3ba61cbfef' \
  -H 'user-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/84.0.4147.105 Safari/537.36' \
  -H 'content-type: application/json;charset=UTF-8' \
  -H 'origin: https://my-server' \
  -H 'sec-fetch-site: same-origin' \
  -H 'sec-fetch-mode: cors' \
  -H 'sec-fetch-dest: empty' \
  -H 'referer: https://my-server/dashboard/c/local/apps/catalog.cattle.io.release/create?repo-type=namespace&repo=dev-charts&chart=rancher-monitoring&version=8.16.100&step=3' \
  -H 'accept-language: en-US,en;q=0.9,zh-CN;q=0.8,zh;q=0.7' \
  -H 'cookie: CSRF=3ba61cbfef; R_SESS=token-my-token; R_USERNAME=admin; R_PCS=light; R_LOCALE=en-us' \
  --data-binary '{"type":"chartInstallAction","charts":[{"chartName":"rancher-monitoring","version":"8.16.100","releaseName":"rancher-monitoring","values":{}}]}' \
  --compressed \
  --insecure

and the response
{"type":"chartActionOutput","links":{},"operationName":"helm-operation-qvj2g","operationNamespace":"default"}

The issue is closed for now since the bug is not reproduced on the latest master head master-3398-head


Language files not working properly?

Hi! Days and Months are not translating properly. Any idea why?
Thanks for the effort by the way!

Can you create a demo via plunker or some other source to duplicate?

I've found the solution meanwhile! Apparently with Angular you need to add
a localization script file (by Angular)!


[firebase_dynamic_links] Working Only on 2nd Open

Describe the bug
If install an app with a firebase deeplink, the deeplink won't work on the original install. But it will work on subsequent opens of the app.
To Reproduce
Steps to reproduce the behavior:

Set Up DeepLinking
Install The App And Notice That It Doesn't Work
See That One 2nd time clicking the link, deeplinking successfully works.

Expected behavior
I would expect the deeplink to work on the initial install.
Additional context
There could be something on the install where it's not passing through.

Hi @CedYF
could you please provide your updated  flutter doctor -v?
Thank you

#255 fixed it for me. Currently running that fork, it will hopefully be merged soon.

If this issue is happening in iOS then you may find some good information in this article
https://medium.com/flutter-community/handling-firebase-dynamic-links-in-flutter-7c1de6a4e2e

Hi again @CedYF
did the proposed solutions worked for you?
Thank you both @HofmannZ  and @ch-muhammad-adil

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of 'flutter doctor -v'  please.
All system setups can be slightly different so its always better to open new issues and reference related issues.


[Feature Request]: custome embeddiings

Describe the problem
I get the embeddings just from a URL from cloud ,
such as
import requests
import json

url = "http://<IP_ADDRESS>:8021/query2emb"

payload = json.dumps({
   "inputs": "我擦"
})
headers = {
   'Content-Type': 'application/json'
}

response = requests.request("POST", url, headers=headers, data=payload)

print(response.text)

then how i set the Embedding function ?
Describe the proposed solution
want to support the URL requests
Alternatives considered
No response
Importance
nice to have
Additional Information
No response

i got the bug refer to the https://docs.trychroma.com/guides/embeddings
from chromadb import Documents, EmbeddingFunction, Embeddings

class MyEmbeddingFunction(EmbeddingFunction):
    def __call__(self, input: Documents) -> Embeddings:
        # embed the documents somehow
        return embeddings

could u pls help me ?

@ucas010 the simple way to do this is to use the
collection = client.get_or_create_collection(name='example')
collection.add(embeddings=[...], ...)
collection.query(query_embeddings=[..], ...)


[turborepo] 1.7.0 with yarn berry doesn't work

What version of Turborepo are you using?
1.7.0
What package manager are you using / does the bug impact?
Yarn v2/v3 (node_modules linker only)
What operating system are you using?
Mac
Describe the Bug
I've upgraded turbo version 1.6.3 -> 1.7.0.
I've got error like this
thread 'main' panicked at 'Failed to execute turbo.: Os { code: 2, kind: NotFound, message: "No such file or directory" }', crates/turborepo/src/main.rs:23:10
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Expected Behavior
It should be work well like 1.6.3
To Reproduce
.yarnrc.yml
nodeLinker: pnp

yarn set version berry
yarn add<EMAIL_ADDRESS>yarn dev

Reproduction Repo
No response

Reproduced, thanks for the issue - we're taking a look.

@kyun you can unblock by using nodeLinker: node-modules, but we're identified the issue and are looking into solutions for installing with pnp on our end.

Hey, just to add to this: I'm seeing the same thing when using<EMAIL_ADDRESS>and<EMAIL_ADDRESS>I've tried with node-linker=hoisted but I still get the same error.

@alexghr could you provide a reproduction? I'm not seeing any issues with<EMAIL_ADDRESS>locally so want to make sure I'm testing correctly.

Hmm, my weird environment might also be at fault. My machine runs NixOS so ever since Turbo switched to a pre-compiled binary I can't install it directly from npm (see #2556 and #2293). I'm using TURBO_BINARY_PATH to tell the JS wrapper where the binary is and I install the binary through a Nix flake (so that I can patch it for my system).
I've pushed a branch to this test repo here https://github.com/alexghr/turbo.nix-test/tree/turbo-1.7.0+pnpm. The files are copy-pasted from the create-turbo-app template, to which I've added flake.nix so that I could install turbo.
Commands like turbo --version and turbo --help do work with this setup.

Awesome, thanks @alexghr I'll take a look.

I ran into this issue as well, as a temporary workaround. specify the TURBO_BINARY_PATH using yarn bin can fix the issue
i.e. yarn to TURBO_BINARY_PATH=$(yarn bin turbo) turbo in package.json
and in case you are using dotenv-cli
dotenv -v TURBO_BINARY_PATH=$(yarn bin turbo)

it looks like the go-turbo TURBO_BINARY_PATH is missing in the "yarn bin turbo" folder
as a temporary workaround, I set the TURBO_BINARY_PATH manually in package.json
"build": "TURBO_BINARY_PATH=$(echo \"console.log(require('child_process').execSync('find .yarn/unplugged/turbo-'+process.platform+'-'+process. Arch+'* -name turbo').toString())\" | node) turbo run build"

I use yarn PnP in a monorepo and I have the same issue.
Using a globally installed version of turbo work fine for now:
npm install turbo -g

NB: make sure to remove any locally installed version of turbo

I think you made a typo error, it should be + I needed to remove the x in x64 so I added .replace('x', '').
TURBO_BINARY_PATH=$(echo \"console.log(require('child_process').execSync('find .yarn/unplugged/turbo-'+process.platform+'-'+process.arch.replace('x', '')+'-* -name turbo').toString())\" | node) turbo run build
Note the process.arch instead of process. Arch)

One note, for 1.7.0 there are two binaries now that need to be colocated. turbo and go-turbo

Ah that was the problem! Well, in the meantime, @joedevivo opened alexghr/turbo.nix#1 to add the new binary and everything is working perfectly with pnpm and turbo 1.7.0 :)

Does Turborepo support PnP mode officially?

@kyun it does not. Officially we support node_modules linker only. However it looks like we made things worse in 1.7.0 so we have that on our radar.

This should be fixed in<EMAIL_ADDRESS>

This should be fixed in<EMAIL_ADDRESS>
I know it isn't official support yet but do you recommend using PnP mode with turbo?

@tknickman  Turborepo supports yarn berry with pnp mode..? I heard that it doesn't support , but it works on my repo. so i want to know official response for this!


Enabled GPT pull request reviews

related to Issue #215

Requires two github secrets to be created before merging if acceptable.
AZURE_PR_REVIEW_OPENAI_API
AZURE_PR_REVIEW_OPENAI_API_KEY


Release/1.1.0 nets v1.0

SUMMARY

Rebased to 1.1.0
Added detokenisation
Added query optimisation
Added s3 export and email functionality
Implemented docker build and compose for custom vis charts

ADDITIONAL INFORMATION

[ ] Has associated issue:
[x] Changes UI
[x] Requires DB Migration.
[x] Confirm DB Migration upgrade and downgrade tested.
[x] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #20 (67e33d9) into master (8de94cb) will decrease coverage by 0.14%.
The diff coverage is 51.55%.

@@            Coverage Diff             @@
##           master      #20      +/-   ##
==========================================
- Coverage   76.95%   76.81%   -0.15%
==========================================
  Files         874      875       +1
  Lines       45178    45420     +242
  Branches     5371     5434      +63
==========================================
+ Hits        34767    34888     +121
- Misses      10288    10411     +123
+ Partials      123      121       -2

Flag
Coverage Δ

cypress
58.51% <ø> (-0.01%)
:arrow_down:

javascript
62.33% <ø> (ø)

python
80.08% <51.55%> (-0.30%)
:arrow_down:

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

Impacted Files
Coverage Δ

...-frontend/src/visualizations/presets/MainPreset.js
100.00% <ø> (ø)

superset/app.py
81.20% <0.00%> (-0.29%)
:arrow_down:

superset/common/tokenise_query_obj.py
37.39% <37.39%> (ø)

superset/tasks/schedules.py
73.16% <41.66%> (-3.20%)
:arrow_down:

superset/common/query_object.py
85.54% <54.54%> (-4.74%)
:arrow_down:

superset/views/schedules.py
58.62% <57.14%> (-0.71%)
:arrow_down:

superset/viz.py
56.15% <87.50%> (+0.14%)
:arrow_up:

superset/models/schedules.py
87.50% <89.47%> (+0.70%)
:arrow_up:

superset/common/query_context.py
82.56% <92.30%> (+0.40%)
:arrow_up:

superset/charts/schemas.py
100.00% <100.00%> (ø)

... and 5 more

Continue to review full report at Codecov.

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


need to format code blocks

right now it's just pre/code but no color

I might have the change the style approach on the  blocks, because there is a different in classes between a  block in a , and a plain  block. I'm not sure if our MDX processor will be able to apply different classes depending on context. Otherwise this all looks good :+1:

Yeah whatever you recommend!

Actually, ignore the note about $green-dark text. Turns out I can just set the code color in the Bootstrap SASS theme file. :)
@nring


can't be built on Go 1.20

Describe the bug
a build of corazawaf/coraza-caddy fails due to old requirement in go.mod line 69 with error:
/go/pkg/mod/github.com/lucas-clemente/quic-go@v0.29.2/internal/qtls/go120.go:5:13: cannot use "The version of quic-go you're using can't be built on Go 1.20 yet. For more details, please see https://github.com/lucas-clemente/quic-go/wiki/quic-go-and-Go-versions." (untyped string constant "The version of quic-go you're using can't be built on Go 1.20 yet. F...) as int value in variable declaration

To Reproduce
Clone https://github.com/corazawaf/coraza-caddy and build:
git clone --depth 1 https://github.com/corazawaf/coraza-caddy
...
go build caddy/main.go

ideas
Two ideas to solve the issue

Either update quic-go (latest release seems to be far ahead.
Alternativly switch to quic-go/quic-go like they did at syncthing project

is this a caddy dependency or a coraza dependency?
On Tue, 18 Jul 2023, 09:19 m-si, @.***> wrote:

Describe the bug
a build of corazawaf/coraza-caddy fails due to old requirement in go.mod
line 69 with error:
@.***/internal/qtls/go120.go:5:13: cannot use "The version of quic-go you're using can't be built on Go 1.20 yet. For more details, please see https://github.com/lucas-clemente/quic-go/wiki/quic-go-and-Go-versions." (untyped string constant "The version of quic-go you're using can't be built on Go 1.20 yet. F...) as int value in variable declaration
To Reproduce
Clone https://github.com/corazawaf/coraza-caddy and build:
git clone --depth 1 https://github.com/corazawaf/coraza-caddy
...
go build caddy/main.go
ideas
Two ideas to solve the issue

Either update quic-go (latest release
https://github.com/quic-go/quic-go/releases seems to be far ahead.
Alternativly switch to quic-go/quic-go like they did at syncthing
project https://github.com/syncthing/syncthing/issues/8768

—
Reply to this email directly, view it on GitHub
https://github.com/corazawaf/coraza-caddy/issues/90, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXOYARVKGH2UDIYNBVGLM3XQY2J7ANCNFSM6AAAAAA2N6ONGE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

First of all , thank you for your fast response. As I am new to this, it's a bit difficult to find out for me. I can't find it in their go.mod of v2.6.4 and v2.7 so I guess it seems to be a corazza requierment, isn't it?

It is a Caddy dependency https://github.com/caddyserver/caddy/blob/master/go.mod#L20, we might need to upgrade this repo to latest caddy and go 1.19 once a new release. Any idea on when go 1.20 is going to be supported @mholt?

I have tested caddy main and it works fine with go 1.20

@jptosso could you add 1.19 and 1.20 to the CI and drop 1.18?

Uhmm https://github.com/corazawaf/coraza-caddy/pull/91
On Tue, 18 Jul 2023, 11:31 Juan Pablo Tosso, @.***>
wrote:

I have tested caddy main and it works fine with go 1.20
—
Reply to this email directly, view it on GitHub
https://github.com/corazawaf/coraza-caddy/issues/90#issuecomment-1639873891,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXOYAWJY5TB2YMQWATQJBTXQZJWZANCNFSM6AAAAAA2N6ONGE
.
You are receiving this because you commented.Message ID:
@.***>

@jcchavezs today I have no time, but I can try later this week. But to me, it looks even more promising to go with 1.19 or 1.20 as we need to do that anyway to keep up.

@jcchavezs To clarify, the latest Go is (almost?) always supported. We just have to support one prior Go release to please the Debian packaging gods. :man_shrugging:

Since https://github.com/corazawaf/coraza-caddy/pull/96, we are building coraza-caddy in the CI with Go 1.20 and caddy is updated to v2.7.3, I think we can close this one


Feature Request: Shutdown/Cleanup commands

Just as we currently can execute processes on the startup of the launcher, I would love it if we could define processes to execute as the launcher is shut down.
My use case is to utilize both a shutdown and a startup script to keep a remote copy of my VisitedStarCache synced between the different systems I would like to play Elite with since FDev won't keep that on their servers.
Thanks for your time and consideration.

I'd also find this useful.  I currently use Playnite, a "game launcher", which offers the same feature to do this – but I'm stuck with powershell scripts running in-process, or doing nasty things, to make this work.    Be very nice to have something in the launcher instead.

This would be useful to turn some stuff off at the end of a play session, like, for example, shutting down SteamVR and disabling joysticks devices so they don't interfere with Windows' sleep mode. It can also be used to sync game data, etc.
I tried a workaround to have one of the processes to monitor for a quit message and do some wrapping-up, but it appears that min-ed is quite aggressive when shutting everything down.

Added in cebed90

Thanks for this! I can't wait to cut out all the steps I currently have in order to run the game!


Fixed screen error

Description

Black theme Exploration title color is not white
When scrollbars appear on the dashboard filter layer, overlap the bottom text and lines
In engine monitoring, key and value overlap.
Button status is hard to see on dashboard status screen.
There is no bottom margin in Administration> Workspaces.
The content is displayed outside of the Administration> Users> Members pop-up.
The pop-up menu in the Filter panel is too wide to navigate.

Related Issue :
How Has This Been Tested?

Check if the explosion title is visible in the black theme.
If scrollbars appear on the dashboard filter layer, make sure the bottom text is visible without overlapping.
Under Engine Monitoring, verify that the content is displayed without overlapping.
Check if the button text is visible on the dashboard status error screen.
Verify that there is an Administration> Workspaces bottom margin.
Verify that content is displayed within the Administration> Users> Members pop-up.
Make sure the pop-up menu in the Filter panel is narrow.

Need additional checks?
Types of changes

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

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.

Additional Context

run build
deploy to 5

@eltriny

Blank Dashboard Background Image Error

run build
deploy to 5


$ end of text anchor not working

Perhaps I was misunderstanding my attempted regexes but trying to use $, as an indicator of the end of text input, didn't seem to work.
I've moved on from the issue, but I seem to recall I was using something like:
\d+$
To catch digits at the end of text input.

Overall, I really like the online tool!

\d+$ works perfectly with This is test 123

I seem to recall it involves having several lines of text in the input textbox.  One line does indeed work fine, but with several lines, none are cached.  Clicking the single-line or multi-line options do not seem to make a difference either.
Thanks for checking up on this.

Hmm, it's a server-side issue. Returned markup shows this. Should be tested further. For the time being, you could try regex101.com, they recently added .Net support as well.

No problem.  I was able to work through my regex issues, so this is not a blocking issue for me.


Auto-create tenant on first request if it doesn't already exist

       expected {"external_tenant" => "external_tenant_uuid", "id" => "146", "name" => "default"} to include {"id" => 146}
       Diff:
       @@ -1,4 +1,4 @@
        "external_tenant" => "external_tenant_uuid",
       -"id" => 146,
       +"id" => "146",

@gtanzillo that looks like a valid failure, should have been `"146" originally not sure why it wasn't


Hide deprecated commands from help section

[ ] All tests have passed. If this feature is not already covered by the tests, new tests have been added.
[ ] The pull request is targeting the dev branch.
[ ] The code has been validated to compile successfully by running go vet ./....
[ ] The code has been formatted properly using go fmt ./....

Security content moved to: https://github.com/jfrog/jfrog-cli-security/tree/main/cli/docs

currently won't work due to https://github.com/urfave/cli/issues/1860
@yahavi


Duplicating an item doesn't generate new ID

Hi there, thanks for creating these widgets, I think NetlifyCMS is lacking behind with lots of things for sure.
I am using the ID Widget to generate unique IDs for collection items. Now the issue I am running into is that once I duplicate one of the items it copies the ID field too. Since it is supposed to be the unique identifier obviously this causes issues.
2 options i see:

Is there a way to add a regenerate directly once someone duplicates an item?
Is there a way to add a Regenerate Button next to the ID field to do this manually?

I'd be happy to help with a PR, but I since I never wrote a Widget I am not well aware of the possibilities.

Hi @henrikwirth, thanks for opening this issue!
It looks like there's a 'prepublish' event we can hook into to generate a uuid for a field. I think this could potentially solve this issue:
https://www.netlifycms.org/docs/beta-features/#registering-to-cms-events
This repo is quite outdated so I'm hesitate to dive back in, if you want to take a crack at setting something up (a new widget, or an update to this one) I'm happy to assist with any issues you might run into!

I think I might have found a way. There is a newRecord: true on the data that is passed to the prePublish hook if the item is newly created/duplicated. If we check for that, we can trigger the generation of the ID right there instead of initially pre-populated.

Currently the generate function is coded within the React component that drives the input: https://github.com/d4rekanguok/netlify-cms-widgets/blob/67ac5d4d7076b1c3ac7d1df8bd9336252b056e60/packages/widget-id/src/control.tsx#L26-L39
I think it'd be good to extract it out into a util function & export it


fix: fixed core bug with getPreservedText (closes #328)

I also fixed the tests, not sure why src="" was being changed to src.

@niftylettuce this seems to be failing on node@latest with that src thing. Without having looked into that test much, if you want to include some value inside of that src so it isn't rewritten in what I presume to be "HTML 5" style, that is probably fine.
I am a bit unclear on why decided it was better to add a parameter to those couple functions rather than have ignoredPseudos and preservedPseudos both attached to options, which feels like it would be more consistent with the existing way that code is setup - meaning that all the options are on options instead of function params.
Thank you for noticing the issue and taking the time to correct it

@jrit looks like there's a bug with web-resource-inliner, you can't have "#" as the value for src attributes, it attempts to read a file from it

I can fix that

@jrit 	here's the fix:
util.getFileReplacement = function( src, settings, callback )
{
    if( !src || util.srcIsCid( src ) || src.indexOf('#') === 0)

@jrit 	I've fixed tests here, once you patch and release new version of web-resource-inliner, just version bump it here and then this PR should merge just fine.  Ping me once released to NPM so I can stop using a scoped package 👍 👏
Thank you Jarrett!!!

These changes have been merged in outside the PR and are now in master

oops, haha, too tired. can you fix the conflicts and then I'll merge?

@jrit Can you version bump and release to NPM?

@niftylettuce done 😄


Error with table data

@fangq
An indexing error occurs when parsing table data using savejson.m
For instance, if one calls the following:
>> j = savejson('test_data', open('census1994.mat'), 'test.json');
you get the following error:

Error using savejson>matlabobject2json (line 444)
You cannot subscript a table using only one subscript. Table subscripting requires both row and variable subscripts.
Error in savejson>obj2json (line 175)
txt=matlabobject2json(name,item,level,varargin{:});
Error in savejson>struct2json (line 285)
txt{end+1}=obj2json(names{e},item(i,j).(names{e}),...
Error in savejson>obj2json (line 171)
txt=struct2json(name,item,level,varargin{:});
Error in savejson (line 140)
json=obj2json(rootname,obj,rootlevel,opt);

I hacked a solution, but I'm not sure if this leads to more issues with other files. See attached.
Cheers,
Paul
paul_savejson.txt

@paul
Your solution works in R2015b, but not in later versions. Moreover, table properties (like RowNames) are not preserved.
Greetings,
Michel

sorry @pjkoprowski and @hillsm for the long delay in processing this issue. I patched savejson with Paul's patch, and I also added support for RowNames, but in the current code, either the VariableNames or RowNames allows to have space or other non-variable-name strings.
In the future, need to move valid_field() from loadjson to a separate unit and use it to convert Table name fields.


feat: add elasticsearch search backend

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 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1961

Relevant Lines:
1961

💛  - Coveralls


Added some HTML

I made some containers in the HTML, one for the Events and one for the Weather.
I also linked the CSS and JS files.

Let me know when it's been approved.


USHIFT-2090: MicroShift Cluster ID generation and display

Closing this PR after an agreement for an alternative implementation.
/close


Link to differential is broken

The current links on Jenkins to differentials is domain.comd1, not domain.com/D1. Using v1.2.

Depends on how you have your phab url configured. We should be smarter and
insert a slash if necessary.
On Saturday, June 6, 2015, Taylor Mansfield<EMAIL_ADDRESS>wrote:

The current links on Jenkins to differentials is domain.comd1, not
domain.com/D1. Using v1.2.
—
Reply to this email directly or view it on GitHub
https://github.com/uber/phabricator-jenkins-plugin/issues/27.

But it's not just the slash, the D has to be capitalized too.

Interesting. This works properly at Uber. I'll dig in tomorrow. Thanks for
the report.
On Saturday, June 6, 2015, Taylor Mansfield<EMAIL_ADDRESS>wrote:

But it's not just the slash, the D has to be capitalized too.
—
Reply to this email directly or view it on GitHub
https://github.com/uber/phabricator-jenkins-plugin/issues/27#issuecomment-109682582
.

Just wondering -- where are you seeing this? The only code path I see that puts a "D" anywhere is here:
https://github.com/uber/phabricator-jenkins-plugin/blob/9fd0f873645815e89553ec88d3f3ac9ad254d45c/src/main/java/com/uber/jenkins/phabricator/conduit/Differential.java#L58-L60

@lavahot can you provide me with a screenshot/log of where exactly you're seeing this lowercase "d" so I can debug?

The two places I've noticed this is on the build page and in the build history widget.

You can't see my mouse cursor in either of these (thanks Ubuntu), but they are hovering over the D1 links. You can see Chrome show the link addresses in the bottom left corner.

ok, so the capitalization is correct, but the links themselves are
malformed? mind reviewing the linked PR
On Sun, Jun 7, 2015 at 2:44 PM, Taylor Mansfield<EMAIL_ADDRESS>wrote:

The two places I've noticed this is on the build page and in the build
history widget.
[image: screenshot from 2015-06-07 14 37 52]
https://cloud.githubusercontent.com/assets/962444/8026225/2f495d4a-0d23-11e5-8d03-bce0b0a22f0d.png
[image: screenshot from 2015-06-07 14 37 38]
https://cloud.githubusercontent.com/assets/962444/8026226/2f4b125c-0d23-11e5-82d3-2a00705e710b.png
You can't see my mouse cursor in either of these (thanks Ubuntu), but they
are hovering over the D1 links. You can see Chrome show the link addresses
in the bottom left corner.
—
Reply to this email directly or view it on GitHub
https://github.com/uber/phabricator-jenkins-plugin/issues/27#issuecomment-109802121
.

#29

OH, yes, the text is correct, but the url is bad. Sorry, I thought that was clear.

Fixed by #29


Unity terrain is not voxelized

I'm on Unity 2019.2.19, I have marked the terrain as geo but its not voxelized during prebake

Thanks for reporting this bug. For our bookkeeping, I'll note that we're tracking this internally with ADO bug # 25298969.

We're not planning on adding support for terrains any time soon. Let us know if this is a major blocker for you.


Enhancement: Show Date/Time in text

Date/Time should show in the GRLevel3 tooltip (placefile Text). As GRLevel3 is the primary interface this allows the user to see when the APRS data was last updated for that user, instead of having to look at the SpotterLevel3 interface. Date/Time should be displayed in a friendly, easy to read format. The Timestamp used by the SpotterLevel3 interface is NOT easy to read at a glance during high stress activities such as Skywarn.
It would also be nice to annotate the tooltip better.
MOVING: coords have changed since last x update
STATIONARY: coords have remained unchanged since last x update
STATUS: Beacon comment/status from APRS
ex.
Lee (K9MP-9)
2022-06-03 21:20:22 UTC
MOVING 3 ° @ 47 mph
STATUS: Skywarn

or
Lee (K9MP-9)
2022-06-03 21:20:22 UTC
STATIONARY

instead of:
K9MP-9
3 ° @ 47 mph

I might have the time formatting done at least, just haven't released. There's a lot of things that could be nice to have in the tooltips, ideally it could be configurable eventually. As you can see I've got a lot on my wishlist, but I'm just trying to get to a minimum usable tool at the moment. The main obstacle is just getting it to save all the settings and data between sessions, which should be relatively straightforward, but as usual, isn't. I definitely welcome suggestions though, send as many as you like, or comment on existing ones. I haven't had time to really think through all of them in detail yet and some other input could help with that.
As you implied, the SpotterLevel3 interface is mainly for initial configuration and then also for troubleshooting, and ideally will be forgotten about most of the time. But it sort of gives a preview of what I'm trying to do with the datafile, as I need to see it in the SL3 interface to verify the data is being pulled in correctly before I try to send it out again.


Add reasonable excitation curves

Add files and names to mapping in excitation_curves.py.

maybe add generalized excitation curves already.

A question: what should it return when asked for a conversion outside the table range, or should it raise an exception?

Added. User values from 3D magnet model field analyses when available, estimated otherwise.


ROS 2 Migration Plan

Rationale
The rover software should eventually be migrated to ROS 2 to take advantage of new feature and use actively developed libraries.
Required Actions

[ ] Research ROS 2 and how it differs from ROS 1
[ ] Read the documentation for Migrating from ROS 1 to ROS 2
[ ] Develop a migration plan for switching rover software to ROS 2

 Look into upgrading to ROS Noetic

 ROS 2 Migration Plan


unexpected crash in vs 2017 community

editing:
[Window Title]
devenv.exe
[Main Instruction]
An unexpected error occurred
[Content]
Please press Ctrl+C to copy the contents of this dialog and report this error to our issue tracker.
[^] Hide details  [Close]
[Expanded Information]
Build: 15.7.18131.1

System.InvalidOperationException: Collection was modified after the enumerator was instantiated.
   at System.ThrowHelper.ThrowInvalidOperationException(ExceptionResource resource)
   at System.Collections.Generic.Queue`1.Enumerator.MoveNext()
   at System.Linq.Enumerable.LastOrDefault[TSource](IEnumerable`1 source)
   at Microsoft.PythonTools.Intellisense.LocationTracker.CanTranslateFrom(Int32 version)
   at Microsoft.PythonTools.Intellisense.VsProjectAnalyzer.<GetNavigationsAsync>d__174.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.VisualStudioTools.UIThread.<>c__DisplayClass18_0`1.<<InvokeTask>b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.PythonTools.Navigation.DropDownBarClient.<RefreshNavigationsFromAnalysisEntry>d__36.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.PythonTools.Navigation.DropDownBarClient.<Microsoft-PythonTools-Editor-IPythonTextBufferInfoEventSink-PythonTextBufferEventAsync>d__35.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.PythonTools.Infrastructure.VSTaskExtensions.<HandleAllExceptions>d__5.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.VisualStudio.Telemetry.WindowsErrorReporting.WatsonReport.GetClrWatsonExceptionInfo(Exception exceptionObject)

Thanks! This is a duplicate of #4176 and the fix will be available in the next 15.7 update.


Included first line of email body as subject

Fixes #10539
This PR includes the subject line as the first line of message.
Link to the issue: https://github.com/zulip/zulip/issues/10539
Please review my Pull Request.

Closing this as abandoned.


Minor networking fixes (cleanups in struct net_tcp, samples, etc)

Re: "samples: net: Use consistent IP addresses in echo_client (qemu_x86)" - did you check that https://www.zephyrproject.org/doc/subsystems/networking/qemu_setup.html#network-connection-between-two-qemu-vms still works?

Re: "samples: net: Use consistent IP addresses in echo_client (qemu_x86)" - did you check that https://www.zephyrproject.org/doc/subsystems/networking/qemu_setup.html#network-connection-between-two-qemu-vms still works?

@pfalcon is correct here. By swapping the addresses in prj_qemu_x86.conf file, then connectivity between linux<->qemu will work, but it will prevent the qemu<->qemu connection described in documentation working.
The other two patches look ok so could you re-send PR without the address swap patch.

The IP addresses makes sense that way, yeah.  Rebased and pushed without that particular patch.


建表sql在mariadb中无法使用

https://github.com/mask-pp/eggroll/blob/0a8dc94894d9d87cd4ac4feb78f6868c0822fcf3/jvm/core/main/resources/create-eggroll-meta-tables.sql#L83，主键用这么大的字段，这种用法不合适吧？使用了DATATIME 字段后在mariadb中无法使用，是不是拿了sqlite建表语句没做任何修改？

和h2建表语句相同，希望能把这方面问题修复一下

Translate:
https://github.com/mask-pp/eggroll/blob/0a8dc94894d9d87cd4ac4feb78f6868c0822fcf3/jvm/core/main/resources/create-eggroll-meta-tables.sql#L83
Isn't it inappropriate to use such a long field for primary key? DATETIME field cannot be used in mariadb. Did you copy the create table statements in sqlite and did not make any change?
(The create table statements) are the same with h2. Hope you can fix these issues.

@mask-pp Sorry for the inconvenience. But we've tested the statements in mysql 5.7+ and it is working well online.
But we do notice that the create table statements are not working in some databases (e.g. mariadb and tdsql). There are 2 possible issues:

These databases do not support long primary keys (typical length is 767 bytes).
These databases require an explicit primary key for every table.

We have made changes and they will be released in v2.2.1 (eta mid Dec 2020).
For the time being, if you are not using a long session id, please feel free to shorten the key size to 760.
Sorry again for the inconveniences.


Fix TextureAtlas not properly padding for edge smoothness

Was resulting in edge bleeding at lower mipmap levels in https://github.com/ppy/osu-framework/pull/2781.

Comparisons:

description
old
new

text (128x)

text (1024x)

whitepixel


⚠ Turnos 24x7 has degraded performance

In 72dfc5c, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 11841 ms

Resolved: Turnos 24x7 performance has improved in 0d68b08.


WIP: remove GlyphSerdeRepr

The test in fontir::ir::tests::glyph_yml reproduces the error (probably coming from underlying libyaml) found in https://github.com/googlefonts/fontmake-rs/pull/81/files#r1067967739:
expected SCALAR, SEQUENCE-START, MAPPING-START, or ALIAS
Surprisingly, if the ir::Glyph contains more than one sources, then the error disappears... :/
Try uncommenting the lines that add the additional GlyphInstance to the Glyph::sources and see how serde_yaml completes without error. I have no idea what's going on.

Try uncommenting the lines

these lines, to be precise:
diff --git a/fontir/src/ir.rs b/fontir/src/ir.rs
index 6547ac0..e41515e 100644
--- a/fontir/src/ir.rs
+++ b/fontir/src/ir.rs
@@ -207,9 +207,9 @@ mod tests {
             .try_add_source(&loc1, GlyphInstance::default())
             .unwrap();

-        // // Uncomment lines below to see libyaml error disappear...
-        // let loc2 = NormalizedLocation::on_axis("Weight", NormalizedCoord::new(1.0));
-        // glyph.try_add_source(&loc2, GlyphInstance::default()).unwrap();
+        // Uncomment lines below to see libyaml error disappear...
+        let loc2 = NormalizedLocation::on_axis("Weight", NormalizedCoord::new(1.0));
+        glyph.try_add_source(&loc2, GlyphInstance::default()).unwrap();

         println!("{:#?}", glyph);

and then do cargo test -p fontir --lib to see how this time serde_yaml serializes successfully.

I managed to reproduce using only serde_yaml, filed issue upstream https://github.com/dtolnay/serde-yaml/issues/348

Nice! My experience with serde so far is it only "just works" for simple data structures so turning anything that's in any way complex into a simple linear structure increases your odds considerably. I also find the errors when it fails to be remarkably unhelpful. All of which may just be noob user errors ofc.

@anthrotype are you still planning to land this?

well, not until upstream bug gets fixed. we can close I guess


Nobody needs the unittests in the installed package

Moved unittests from the package directory to the top level directory

Oops, we have projects that rely on the testsuite being present, because they build their testsuite on top of this one.


give user option to specify a per-scene theme prompt after init image generation

see also #71

nb: at present, modifying the prompt actually does nothing here.


feat: add swoole 5.1.4 release

update swoole with release 5.1.4

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


[Improvement][SET-PROPERTY]Support for set query_timeout property

Proposed changes
Issue Number: close #xxx
Problem summary
[Improvement][SET-PROPERTY]Support for set query_timeout property
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 40.1 seconds
load time: 584 seconds
storage size:<PHONE_NUMBER>8 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221019225024_clickbench_pr_31450.html

@morningman PTAL


Port two new internal methods of ClassTag from upstream.

This is a port of the changes to ClassTag in https://github.com/scala/scala/commit/31d6481b8a1322bd1ecfa6d68c49f6c03ab42f19
Although that commit has not reached the 2.13.x branch, we also apply the changes to the 2.13.x overrides to be future-proof.

This fixes the nightly build on 2.12.x. Locally tested with
> set resolvers in Global += "scala-integration" at "https://scala-ci.typesafe.com/artifactory/scala-integration/"
> ++2.12.13-bin-490e3e3
> testSuite2_12/test

The naive port caused a code size increase, because our ClassTags are stack-allocated most of the time. I changed the lazy vals into defs for Scala.js, and added a comment explaining why.


Info about unit returned in Guild#getMaxFileSize

Pull Request Etiquette

[x] I have checked the PRs for upcoming features/bug fixes.
[x] I have read the contributing guidelines.

Changes

[ ]  Internal code
[ ]  Library interface (affecting end-user code)
[x]  Documentation
[ ]  Other: _____

Closes Issue: NaN
Description
This pull request adds explicit information about the unit Guild#getMaxFileSize returns in.

Doc changes should targets docs/development


Metrics v0

Draft PR to track metrics using prom/grafana.

[x] Set up metrics binding
[x] Track lower level tx success/failure for each step
[x] Track high level transaction attempt and success
[x] Track fees generated and total volume
[ ] Track gas paid
[ ] Track auction attempt/success
[ ] Track onchain liquidity
[ ] Set up docker compose file to spin up prometheus and dashboard

Quick note: For gas tracking, I wasn't able to find an easy way to track any burned gas as a result of failed txs - can add this in based on feedback or just leave it out for now in the interest of getting basics shipped.


Error on install

install.packages("sassy")
Installing package into ‘/home/dbosak01/R/x86_64-pc-linux-gnu-library/3.6’
(as ‘lib’ is unspecified)
also installing the dependencies ‘common’, ‘reporter’, ‘procs’

trying URL 'https://cloud.r-project.org/src/contrib/common_1.0.8.tar.gz'
Content type 'application/x-gzip' length 428968 bytes (418 KB)
downloaded 418 KB
trying URL 'https://cloud.r-project.org/src/contrib/reporter_1.4.1.tar.gz'
Content type 'application/x-gzip' length 3393348 bytes (3.2 MB)
downloaded 3.2 MB
trying URL 'https://cloud.r-project.org/src/contrib/procs_1.0.3.tar.gz'
Content type 'application/x-gzip' length 1038340 bytes (1014 KB)
downloaded 1014 KB
trying URL 'https://cloud.r-project.org/src/contrib/sassy_1.2.0.tar.gz'
Content type 'application/x-gzip' length 3975248 bytes (3.8 MB)
downloaded 3.8 MB

installing source package ‘common’ ...
** package ‘common’ successfully unpacked and MD5 sums checked
** using staged installation
** R
** inst
** byte-compile and prepare package for lazy loading
** help
*** installing help indices
** building package indices
** installing vignettes
** testing if installed package can be loaded from temporary location
** testing if installed package can be loaded from final location
** testing if installed package keeps a record of temporary installation path
DONE (common)
installing source package ‘reporter’ ...
** package ‘reporter’ successfully unpacked and MD5 sums checked
** using staged installation
** R
** inst
** byte-compile and prepare package for lazy loading
Warning: replacing previous import ‘common::labels<-’ by ‘fmtr::labels<-’ when loading ‘reporter’
** help
*** installing help indices
** building package indices
** installing vignettes
** testing if installed package can be loaded from temporary location
Warning: replacing previous import ‘common::labels<-’ by ‘fmtr::labels<-’ when loading ‘reporter’
** testing if installed package can be loaded from final location
Warning: replacing previous import ‘common::labels<-’ by ‘fmtr::labels<-’ when loading ‘reporter’
** testing if installed package keeps a record of temporary installation path
DONE (reporter)
installing source package ‘procs’ ...
** package ‘procs’ successfully unpacked and MD5 sums checked
** using staged installation
** R
** inst
** byte-compile and prepare package for lazy loading
Warning: replacing previous import ‘common::labels<-’ by ‘fmtr::labels<-’ when loading ‘procs’
Warning: replacing previous import ‘common::labels<-’ by ‘fmtr::labels<-’ when loading ‘reporter’
Error in object[[i]][["order"]] : subscript out of bounds
Error: unable to load R code in package ‘procs’
Execution halted
ERROR: lazy loading failed for package ‘procs’
removing ‘/home/dbosak01/R/x86_64-pc-linux-gnu-library/3.6/procs’
Warning in install.packages :
installation of package ‘procs’ had non-zero exit status
ERROR: dependency ‘procs’ is not available for package ‘sassy’
removing ‘/home/dbosak01/R/x86_64-pc-linux-gnu-library/3.6/sassy’
Warning in install.packages :
installation of package ‘sassy’ had non-zero exit status

The downloaded source packages are in
‘/tmp/RtmpKb1hfJ/downloaded_packages’

On Linux/Unix, the SASSY metapackage seems to be loading an old version of fmtr.  The old version is conflicting with the common package, and causing the procs package to fail.  Temporary work-around is to install each package individually, instead of installing everything from SASSY.  You can run the following code:
remove.packages(c("fmtr", "common", "logr", "libr", "reporter", "procs", "sassy"))
install.packages(c("fmtr", "common", "logr", "libr", "reporter", "procs", "sassy"))
sassy::run_oq(location=tempdir())
The run_oq() function will run a script that verifies that everything is installed and working correctly.

It seems like some environments refuse to install the newer version of fmtr.  I don't know why.  But that is messing up all the other packages, in particular procs.  So if procs can't install, then the new version of sassy cannot install.  I was just talking to someone with R 4.0.5 on Linux, and they can't get the new version of sassy to install, even with the above code.  No matter what we did, we couldn't get fmtr >= 1.5.9. It was always installing fmtr 1.5.0.  Yet I have R 3.6.0 on Linux and the above code works for me.  Not sure what is happening.

Also I just upgraded fmtr to 1.6.0, because there was an insignificant update on Github that I thought might be interfering with the normal CRAN install.

This turned out to be a problem with this particular users environment.  They are on an IT controlled environment, and only IT can install packages.  They had an old version of fmtr, and it was messing up everything else.  So I think this is pretty specific to the client environment.  Will keep an eye on this issue, but closing for now.


Ergo 0.7

Coverage decreased (-0.5%) to 99.482% when pulling dbe0d9ef1f8a11545366239fe657e7817181c953 on jeromesimeon:ergo-0.7 into eceb58f7504b543f2707c64e2b42320f174a8dd1 on accordproject:ergo-0.7.


Research on PDF import

Research on how we can import a PDF file into a website and convert it to text

Method 1: Convert PDF to HTML
One way to convert a PDF file to text and integrate it into a website is by converting the PDF to HTML format. HTML is a widely used file format for web pages and can be easily displayed in web browsers. Here's a general outline of the process:

Convert the PDF to HTML: Use a PDF to HTML converter tool or service to convert the PDF file into an HTML file. This conversion process will extract the text and structure of the PDF and convert it into HTML format.
Import the HTML file into the website: Once you have the HTML file, you can import it into your website builder or content management system (CMS). This will allow you to create customized web pages that look like the original PDF, including links, headings, and other elements.
Display the HTML content: Use HTML and CSS to display the converted HTML content on your website. You can use CSS to style the text and layout according to your website's design.

Adobe Acrobat and other online services like DocHub offer PDF to HTML conversion capabilities
Method 2: Extract Text from PDF
Another approach is to extract the text directly from the PDF file and display it on your website. Here's a general outline of the process:

Extract text from the PDF: Use a PDF to text converter tool or service to extract the text from the PDF file. These tools use Optical Character Recognition (OCR) technology to recognize and extract the text from the PDF.
Convert the extracted text to HTML: Once you have the extracted text, you can convert it to HTML format. You can use programming languages like JavaScript or server-side languages like PHP to process the text and generate HTML markup.
Display the HTML content: Use HTML and CSS to display the converted HTML content on your website. You can style the text and layout according to your website's design.

Services like Smallpdf offer PDF to text conversion capabilities. Additionally, you can find various libraries and frameworks that provide PDF parsing and text extraction functionalities, such as Apache PDFBox, PDF.js, or PyPDF2, depending on the programming language you are using.


Feature/uri support

This change is will raise another unexpected issue, I close this, because FileDownloader2- okdownload has already cover such case.


Fix #26: Link in confirmation mail was not clear as manage link

It was not obvious that the link in the confirmation mail is also the manage link for the party. Closes #26

ziet er goed uit. Mss nog ff btn_validate renamen naar btn_goto_manage


Added some suggestions and recommendations

Hello maintainers,
I am new to open source, so please merge my pull request.
Please add ‘hacktoberfest’ & ‘hacktoberfest-accepted’ label to my PR
Reference: https://hacktoberfest.digitalocean.com/hacktoberfest-update

Hi @5hre9a , it would be great if you merge my suggestions and recommendations :)


feat(data-grid): focus on cell remove header rows from tabfocus queue

Closes #994

I'm fine with the code changes, but what about a11y? Headers are not read by ATs anymore it seems (quickly tested with VoiceOver), that seems like a regression.

VoiceOver

@olaf-k I think it is the same as the spec here: https://www.w3.org/WAI/ARIA/apg/patterns/grid/examples/data-grids/
Do you see the same effect?

@olaf-k thanks to you, the issue was is going to be discussed in the next W3C meeting: https://github.com/w3c/aria-practices/issues/2626#issuecomment-1445612824
Closing this now as they mention they are going for focusable headers as the preferred way to do it.
@yinonov - any final words?

Closing this now as they mentioned they are going for focusable headers as the preferred way to do it.

your aria markup looks fine so I did a bit more testing and realized that the column headers are properly read when you navigate from cell to cell using the AT controls but not when using the arrow keys.
so, focusable headers or not, I'm fine as long as there is a way to know where you are in the table.


Add filter to return only executed transactions on all transactions endpoint

What is needed?
A filter to return only executed transactions on all transactions endpoint

@jpalvarezl @rmeissner This have been deployed already on dev/staging, could you confirm it's working for you as you expected?


New PR to test SignCLA at 2017-10-26 15:21:08.599354793 -0700 PDT m=+4.508477457

Body of PR

Thanks for your pull request.  It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify.  Thanks.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address.  Check your existing CLA data and verify that your email is set on your git commits.
If your company signed a CLA, they designated a Point of Contact who decides which employees are authorized to participate. You may need to contact the Point of Contact for your company and ask to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the project maintainer to go/cla#troubleshoot.
In order to pass this check, please resolve this problem and have the pull request author add another comment and the bot will run again.


setNotificationClickListener Triggering Automatically on Subsequent Notifications Without User Interaction

I am experiencing an issue with the Pushy.setNotificationClickListener method in my Flutter application. The method is supposed to handle routing when a user clicks on a notification. However, the function is being triggered automatically when a new notification is received, without the user actually clicking on the notification.
Steps to Reproduce:

Initial Setup:

Add Pushy.listen, Pushy.register, and Pushy.setNotificationListener in the main app’s initState.
Add Pushy.setNotificationClickListener in HomeDashboard’s initState.

First Notification:

Send the first notification for the details page when the app is killed or quit.
Click the notification from the list or banner. The app opens and navigates to the details page as expected.

Second Notification:

Send the second notification for the profile page while the app is still open from the first notification.
Observe that the app receives the second notification and the setNotificationClickListener function is triggered automatically without clicking the notification.

Expected Behavior:
The setNotificationClickListener should only trigger when the user clicks on the notification, not when the notification is received while the app is open.
Actual Behavior:
The setNotificationClickListener is being triggered automatically upon receiving a new notification, causing unintended navigation based on the first notification's data.
`@pragma('vm:entry-point')
Future backgroundNotificationListener(Map<String, dynamic> data) async {
Pushy.listen();
Pushy.setNotificationIcon('ic_launcher');
debugPrint("BackgroundNotificationListener");
debugPrint('Received notification: $data');
String notificationTitle = MasterConfig.app_name;
String notificationText = data['message'] ?? '';
if (Platform.isAndroid) {
debugPrint("Testing: From Notify");
Pushy.notify(notificationTitle, notificationText, data);
}
Pushy.clearBadge();
}
static Future initPlatformState() async {
Pushy.listen();
Pushy.setNotificationIcon('ic_launcher');
try {
String deviceToken = await Pushy.register();
if (deviceToken != "Loading...") {
await MasterSharedPreferences.instance.setDeviceToken(deviceToken);
}
} catch (error) {
debugPrint('Error: ${error.toString()}');
}
Pushy.toggleInAppBanner(false);
Pushy.setNotificationListener(backgroundNotificationListener);
debugPrint("Testing: End Of init notii Plactform State");
}
Pushy.setNotificationClickListener((Map<String, dynamic> data) {
final String barcode = data['barcode'] ?? '';
final String type = data['type'] ?? '';
if (barcode.isNotEmpty) {
context.pushNamed(
RoutePaths.parcelDetails,
extra: ParcelDetailsIntentHolder(
filterBy: "barcode",
parcelId: barcode,
),
);
} else if (type.isNotEmpty) {
context.pushNamed(
RoutePaths.profileDetails,
);
}
});**Additional Context:** The issue occurs when the app is open after navigating from the first notification. Subsequent notifications trigger thesetNotificationClickListener` without user interaction. I suspect this behavior might be a bug in the Pushy Flutter plugin.
Environment:

Flutter version: 3.19.6
Pushy Flutter version:^2.0.18

Conclusion:
This issue is affecting the user experience as it leads to unintended navigation within the app. I would appreciate any guidance or fixes for this issue.

This detailed report should help the Pushy Flutter team understand the problem and provide a solution or workaround.

I think, in your case, the reason is that you call Pushy.listen() inside the background notification listener, which is wrong. As per the documentation, it should be called only once at the initialization phase.
The reason for that is that under the hood, it checks whether the click listener is subscribed and calls it:
https://github.com/pushy/pushy-flutter/blob/7432d15e66858a8b453bb268c4c1ab8a9ef8272a/lib/pushy_flutter.dart#L42-L59
However, I have faced a similar false behavior. In my case, the root cause was that after a user logged out, the click listener was still subscribed, and for a new user logged in, push notifications were triggered for an unknown reason. The fix for me was to explicitly unsubscribe for the listeners on sign out:
Pushy.setNotificationClickListener((_) {});
Pushy.setNotificationListener((_) {});

Hi @aungmyopaing890,
Thanks for reaching out. We'd be glad to assist with your inquiry.
@hmG3 is correct. We were able to reproduce the issue after adding Pushy.listen() inside the backgroundNotificationListener.
Please remove the invocation of Pushy.listen() from your backgroundNotificationListener. Please only call Pushy.listen() once, from within initPlatformState(), and the problem with the click listener being incorrectly invoked will be resolved.
Please let me know if there's anything else I can help with.

Hi @aungmyopaing890,
Thanks for reaching out. We'd be glad to assist with your inquiry.
@hmG3 is correct. We were able to reproduce the issue after adding Pushy.listen() inside the backgroundNotificationListener.
Please remove the invocation of Pushy.listen() from your backgroundNotificationListener. Please only call Pushy.listen() once, from within initPlatformState(), and the problem with the click listener being incorrectly invoked will be resolved.
Please let us know if there's anything else we can help with.


Add note for gpt-35-turbo-16k quota

Add note for gpt-35-turbo-16k quota

@mrbullwinkle
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Fix 'unused parameter' warnings

When compiling with -Wall -Wextra, I get about 20 unused parameter warnings.
I'm not sure if these were voluntarily left there or not, but I silenced them.

I'm assuming that a lot (all?) of them are in callbacks? At some point I just turned that warning off since the "fix" for them is to make empty statements using the unused parameter to trick the compiler into thinking you are using something you aren't. The other case where they tended to come up was when a public API started to ignore a parameter, but it couldn't be removed without breaking binary compatibility. In both cases the warning was a false positive, and fixing them just obfuscates the code and removes the possibility that they could be useful in the future.

In both cases the warning was a false positive, and fixing them just obfuscates the code and removes the possibility that they could be useful in the future.

With afterthought I came to the same conclusion ^^


Undefined variable gets printed as void

Assume that we have an undefined variable x
bsh % System.out.println(x);
// Error: EvalError: Undefined argument: x  : at Line:
1 : in file: <unknown file> : ( x )
But if we print out this variable with another string:
bsh % System.out.println(x+" test string");
void test string
An undefined variable should not be printed as void.
Reported by: borgacan
Original Ticket: "beanshell/bugs/255":https://sourceforge.net/p/beanshell/bugs/255

... This actually may be a useful way to get around the need to void/null check everywhere! I vote for actually leaving it as-is (or making this an optional strict feature).

Hmmmm very interesting...
bsh % void + 7;
// Error: Evaluation Error: illegal use of undefined variable

bsh % void + '';
--> $6 = "void" :String

My first thoughts are that it must have something to do with toString().


Fixed typo in docs

Basically I just wanted to try submitting a PR.

PR was fine and merged without any problems.


Problem downloading from youtube "unable to extract html5 player"

[generic] watch?v=88mdiPJ_XEg: Requesting header
[redirect] Following redirect to https://www.youtube.com/watch?v=88mdiPJ_XEg&gl=CA
[youtube] 88mdiPJ_XEg: Downloading webpage
[youtube] 88mdiPJ_XEg: Downloading video info webpage
[youtube] 88mdiPJ_XEg: Extracting video information
WARNING: unable to extract html5 player; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
[youtube] {43} signature length 46.43, html5 player None
ERROR: Signature extraction failed: Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 700, in _decrypt_signature
video_id, player_url, s
File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 592, in _extract_signature_function
raise ExtractorError('Cannot identify player %r' % player_url)
ExtractorError: Cannot identify player u'https://s.ytimg.com/yts/jsbin/player-en_US-vflgFv_Kx/base.js'; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
(caused by ExtractorError(u"Cannot identify player u'https://s.ytimg.com/yts/jsbin/player-en_US-vflgFv_Kx/base.js'; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.",)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 700, in _decrypt_signature
video_id, player_url, s
File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 592, in _extract_signature_function
raise ExtractorError('Cannot identify player %r' % player_url)
ExtractorError: Cannot identify player u'https://s.ytimg.com/yts/jsbin/player-en_US-vflgFv_Kx/base.js'; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 656, in extract_info
ie_result = ie.extract(url)
File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 279, in extract
return self._real_extract(url)
File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 1200, in _real_extract
encrypted_sig, video_id, player_url, age_gate)
File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 710, in _decrypt_signature
'Signature extraction failed: ' + tb, cause=e)
ExtractorError: Signature extraction failed: Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 700, in _decrypt_signature
video_id, player_url, s
File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 592, in _extract_signature_function
raise ExtractorError('Cannot identify player %r' % player_url)
ExtractorError: Cannot identify player u'https://s.ytimg.com/yts/jsbin/player-en_US-vflgFv_Kx/base.js'; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
(caused by ExtractorError(u"Cannot identify player u'https://s.ytimg.com/yts/jsbin/player-en_US-vflgFv_Kx/base.js'; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.",)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

resolved with update


Prepare 0.15.1 Release

I have left out 3 PRs that I'll add the changelogs for on master in another PR.

[Buttons] Deprecate linkButton property #4197
[EnhancedButton] Fix href style #4457 (regression introduced in 1)
[Badge] Fixed incorrect color usage (primary/accent were swapped) #4449

Thanks dude. Will check out asap.

Sterling work @alitaheri 👍.