UTF32toUTF8 can create automata that produce/accept invalid unicode

Description
When converting a unicode (UTF32) automaton down to a UTF8 representation, UTF32toUTF8 can create an automaton that produces/accepts invalid UTF8. This happens when a transition in the input automaton contains a code-point range that spans different UTF8 byte counts.
As an example/repro, create an automaton that accepts the two single-character terms: U+65535 and U+65536 (sample code for this in the repro test case below). Before compiling, the UTF32 representation is:

This range correctly covers 65535 - 65536 (0xffff - 0x10000). Note that the min value (65535) is the largest value that can be represented in three UTF8 bytes (it's representation is 11101111 10111111 10111111 in UTF8), and the max value (65536) is the smallest valid UTF8 representation that uses four bytes (11110000 10010000 10000000 10000000).
After converting with UTF32toUTF8#convert, the resulting automaton looks like this:

This automaton incorrectly accepts/produces everything in the range 0xf0 0x80 0x80 0x80 - 0xf0 0x8f 0xbf 0xbf. These are four byte UTF8 sequences encoding all code points from 0x00 - 0xffff (0 - 65535). So this automaton now incorrectly accepts/produces everything between 0 - 65534 (65535 is valid of course). Additionally, these are all invalid UTF8 byte sequences, known as "overlong encodings" (at least according to Wikipedia; I didn't cross-reference an actual spec).
The bug lies in the UTF32toUTF8#end logic.
Here is a full test case that demonstrates the bug:
  public void testFoo() throws Exception {
    Automaton.Builder b = new Automaton.Builder();
    // start state:
    int s1 = b.createState();

    // single end accept state:
    int s2 = b.createState();
    b.setAccept(s2, true);

    // add two single-code-point terms: U+65535 and U+65536
    b.addTransition(s1, s2, 65535);
    b.addTransition(s1, s2, 65536);

    Automaton a = b.finish();

    CompiledAutomaton c = new CompiledAutomaton(a);
    FiniteStringsIterator it = new FiniteStringsIterator(c.automaton);
    int termCount = 0;
    for (IntsRef r = it.next(); r != null; r = it.next()) {
      termCount++;
    }
    assertEquals(2, termCount);
  }

The assertion fails as:
expected:<2> but was:<65538>

Version and environment details
Reproduced on 9.7 but appears to have existing for quite some time (relevant code dates back at least to 2014).

Hmm curiously the 1 -> 2 byte test case passes:
  public void testUTF8OneToTwoBytes() throws Exception {
    Automaton.Builder b = new Automaton.Builder();
    // start state:
    int s1 = b.createState();

    // single end accept state:
    int s2 = b.createState();
    b.setAccept(s2, true);

    // add two single-code-point terms: U+65535 and U+65536
    b.addTransition(s1, s2, 127);
    b.addTransition(s1, s2, 128);

    Automaton a = b.finish();

    CompiledAutomaton c = new CompiledAutomaton(a);
    FiniteStringsIterator it = new FiniteStringsIterator(c.automaton);
    int termCount = 0;
    for (IntsRef r = it.next(); r != null; r = it.next()) {
      termCount++;
    }
    assertEquals(2, termCount);
  }

And ... 2 -> 3 bytes DOES fail, fun :)
  public void testUTF8TwoToThreeBytes() throws Exception {
    Automaton.Builder b = new Automaton.Builder();
    // start state:
    int s1 = b.createState();

    // single end accept state:
    int s2 = b.createState();
    b.setAccept(s2, true);

    // add two single-code-point terms
    b.addTransition(s1, s2, 0x7FF);
    b.addTransition(s1, s2, 0x800);

    Automaton a = b.finish();

    CompiledAutomaton c = new CompiledAutomaton(a);
    FiniteStringsIterator it = new FiniteStringsIterator(c.automaton);
    int termCount = 0;
    for (IntsRef r = it.next(); r != null; r = it.next()) {
      termCount++;
    }
    assertEquals(2, termCount);
  }

   >     java.lang.AssertionError: expected:<2> but was:<2050>
   >         at __randomizedtesting.SeedInfo.seed([E20E213BD5F7763C:1E8F5BB354E27C83]:0)
   >         at org.junit.Assert.fail(Assert.java:89)
   >         at org.junit.Assert.failNotEquals(Assert.java:835)
   >         at org.junit.Assert.assertEquals(Assert.java:647)
   >         at org.junit.Assert.assertEquals(Assert.java:633)
   >         at org.apache.lucene.util.automaton.TestUTF32ToUTF8.testUTF8TwoToThreeBytes(TestUTF32ToUTF8.java:335)

I have discovered the bug.
if (endUTF8.numBits(upto) == 5) {
        // special case -- avoid created unused edges (endUTF8
        // doesn't accept certain byte sequences) -- there
        // are other cases we could optimize too:
        startCode = 194;
      } else {
        startCode = endUTF8.byteAt(upto) & (~MASKS[endUTF8.numBits(upto) - 1]);
 }

In the provided Java code snippet, there is an assumption that the codepoint will start at 0x80 when the number of bits is 6. However, this assumption is incorrect. In reality, when the length of the codepoint is 3 and the first byte is 0xE0, it will start from 0xA0. Similarly, when the length of the codepoint is 4 and the first byte is 0xF0, it will start from 0x90. You can verify this information on the following website: https://www.utf8-chartable.de/unicode-utf8-table.pl
Here is the PR to fix that #12472

Whoa, great find!  T hanks @tang-hi -- I'll try to review the PR soon.

Fixed by @tang-hi in GH#12472. Thanks!!!


CI does not test with schema.json

All tests in test_schema_completer are skipped.

Fixed with
9d10c058dfa16824caba8125a7ad9b2d7bfe7ca8
ca939151337f7208ba6617c7273e171fdfb84bcc


Correct content-type for excel download

Every time you download a generated excel with this library in Firefox, a popup will open with this option: "open with 7-zip manager" . This is because the download file has application/zip as mime-type.
I suggest change the line 32 with the following code:
compressedFile.saveFile = function (fileName, callback) {
return zip.generateAsync({ type: "blob" , mimeType: "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet" })
.then(function (content) {
try {

Thanks for pointing it out. Done.


Fix code to update mutation consequence

Before the mutation consequence is updated there is a check to make sure the entry is not already in the db.
That check was only using:

genomic feature
allelic requirement
mutation consequence

This PR adds the disease_id to the check as the disease is part of what defines an unique ID.
This PR also fixes the alert messages (they were not displayed).

Merged into release/112 and master


Some enhancement proposals for CMake based usecases

Currently we only support making dbg(...) no-op with DBG_MACRO_DISABLE. When using with CMake it is more intuitive to make it no-op when CMake is invoked in Release mode.
Following the instructions from README, the dbg statements looks like this

[..m/mines/src/grid.cpp:103 (reset)] _state = 0 (Grid::State)

The source file path could be cleaned up.

Currently we only support making dbg(...) no-op with DBG_MACRO_DISABLE. When using with CMake it is more intuitive to make it no-op when CMake is invoked in Release mode.

Having the macro active in release mode is a deliberate design choice. See https://doc.rust-lang.org/std/macro.dbg.html for reasoning.

The source file path could be cleaned up.

If you have any ideas on how, please let us know. I wanted to shorten it to make the main content readable.
I'm closing this for now, but let me know if there is anything you think we should do.


Menu doesn't include 'list' in class key

The documentation says 'list' should be a valid css class key and Menu.js appears to use it, but I get a compiler error when trying to use it since it's missing in Menu.d.ts. I can fix it locally but don't want it to break when updating my version of material ui.

@ChrisBlueStone Thank you for reporting the issue. We miss the style rule in the definitions. Do you want to submit a pull request? :) I think that the following diff should do the job.
diff --git a/packages/material-ui/src/Menu/Menu.d.ts b/packages/material-ui/src/Menu/Menu.d.ts
index df8247a80..8198fd9a9 100644
--- a/packages/material-ui/src/Menu/Menu.d.ts
+++ b/packages/material-ui/src/Menu/Menu.d.ts
@@ -16,7 +16,7 @@ export interface MenuProps
   variant?: 'menu' | 'selectedMenu';
 }

-export type MenuClassKey = 'paper';
+export type MenuClassKey = 'paper' | 'list';

 declare const Menu: React.ComponentType<MenuProps>;

I would like to take this up!


make-options-doc: fix string context issues

Motivation for this change
When using documentation.nixos.includeAllModules = true; with external
modules, the string context might contain dependencies to derivations
and so toFile refuses to evaluate;
error: in 'toFile': the file 'options.xml' cannot refer to derivation outputs, at
[...]/nixpkgs/nixos/lib/make-options-doc/default.nix:89:16

This is not an issue when using writeText (instead of manually
stripping the context).
Possibly addresses: https://github.com/NixOS/nixpkgs/issues/73743
For my particular configuration utilizing home-manager this fix is insufficient, because some of the formatting for some home-manager options fail validation;
manual-combined.xml:19469: element link: validity error : IDREF attribute linkend references an unknown ID "opt-xsession.windowManager.i3.config.assigns"
[...]

But maybe this is one step forward anyways.

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @domenkozar

iirc this will fail regardless, there should be no /nix/store paths in the manual (there's a check for that).

@domenkozar: I don't know those parts too well yet, but saw the check. The check emits a better error message in comparison to the toFile error. I thought it would make sense to get this out of the way in the context of documentation.nixos.includeAllModules to maybe reduce confusion.
Maybe it would also make sense to include some additional pointers of how to go about it - or some type of flag to ignore if it's just for local use in the manpage.
Otoh, in general it seems to me like It greatly reduces usefulness if one would have to go and fix upstream modules from repos like home-manager, just to have the options being shown in the manpage.
What do you think?

I agree about the error.

This change causes a giant options.xml.drv file to be generated (#83863). Aside from that, this fix seems wrong since options.xml shouldn't depend on derivations to begin with! After all, it can things to be built that wouldn't otherwise be built. (This is why we have literalExample, BTW.)

+1 and add a comment.

Reverted in 513cec9b81a7c87682eb42af651798298f3f0311 and cab6b019b17881f42f618dcc9fce9341f1a6df81, backported to 20.03 in 9412ae37b08883d6a9d80ffafa8c6cde0a168726. Verified that jobsets now evaluate.

@edolstra This was an issue for me in https://github.com/NixOS/nixpkgs/pull/86163/commits/cc94d5e8f4999068a272d37b1683725183989057.


wallet params was returning a string for kdfParams value

this is invalid json as kdfParams is an object. for languages
that actually check types this causes a problem in the json
parser.

Coverage increased (+0.02%) to 94.03% when pulling 1de51caedb650a2aeab2d683f0d3396b417c80f9 on json-parse-kdfparams into 097de232333ec8233584931e44c7e1d270119aad on master.

This change will probably break stellar-wallet-js-sdk and stellar-client. If we want to merge it we should update those repos accordingly.

we're gonna hold off on this change for now
On Tue, Mar 24, 2015 at 5:09 AM, Bartek Nowotarski<EMAIL_ADDRESS>
wrote:

This change will probably break stellar-wallet-js-sdk and stellar-client.
If we want to merge it we should update those repos accordingly.
—
Reply to this email directly or view it on GitHub
https://github.com/stellar/stellar-wallet/pull/75#issuecomment-85471133.


Cache rust deps/targets in GHA

Uses the rust-cache action to cache cargo dependencies and target files for before building for python tests.
Motivation: Building the rust bindings takes b/w 6-20minutes depending on the OS and any speedups here should significantly improve CI turnaround time

Codecov Report

Merging #1033 (7cddb76) into main (600996c) will increase coverage by 0.39%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #1033      +/-   ##
==========================================
+ Coverage   80.82%   81.21%   +0.39%
==========================================
  Files          77       77
  Lines        4338     4338
  Branches      779      779
==========================================
+ Hits         3506     3523      +17
+ Misses        658      639      -19
- Partials      174      176       +2

Impacted Files
Coverage Δ

dask_sql/physical/rex/core/literal.py
60.95% <0.00%> (+2.85%)
:arrow_up:

dask_sql/_version.py
35.31% <0.00%> (+3.95%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thanks for looking into this @ayushdg 😄 interested in your thoughts on also implementing Conda environment caching using the setup described here:
https://github.com/conda-incubator/setup-miniconda#caching-environments
Using that we were able to speed things up pretty significantly in Distributed's CI (xref dask/distributed#6855), but the main concern is that we wouldn't always be pulling the latest Python packages for each run unless we manually skip caching.

Yeah, I'm open to adding a conda cache step as well. The main concern is us getting old cache hits when a one of our unpinned dependencies cut a release right? If we refresh the cache daily like mentioned in the example above, we'd potentially only catch breakages a day late which is okay in most cases.

On second thought let's break off the Conda caching step into a followup pr. Ideally I'd like to see if I can create a composite action that does the whole conda caching rather than reusing those steps across jobs/workflows

I've opened #1036 to track the conda caching step. @charlesbluca This pr should be good to review


docs(contributing): add manim.typing guide

Motivation
It's important to document the intention behind each TypeAlias present in the manim.typing module, as well as explain when to use each. For example, InternalPoint3D vs Vector3D have very different connotations, but the same signature.
From the guide:

In order to provide the best user experience, it's important that typehints are chosen correctly. With the large variety of types provided by Manim, choosing which one to use can be difficult. This guide aims to aid you in the process of choosing the right type for the scenario.

Link to Changed Documentation
https://manimce--3669.org.readthedocs.build/en/3669/contributing/docs/types.html
Progress

[X] Coordinates
[x] Colors
[ ] Béziers
[x] Functions
[x] Images

CC @Viicos

If anyone wants to write a quick description on the different representations of Béziers in Manim, feel free to push a commit adding it.


Deserializing nested types

Hello. I faced with an issue that nested types are not deserialized into objects. So I have Foo type, then I have Nested and AnotherNested types. Foo has properties of types Nested and AnotherNested. Also it has a property which is represented by Dictionary<string, string>. When I'm deserializing my object all properties of primitive types are ok, but nested objects and dictionary are null while they have actual data. Tried with IncludeInherited option but have no success. Can you please provide an info is there a bug for that and if not how to achieve this. Thanks

Have played with JilDirective.IsUnion property - now it works. Sorry and thanks :)


Add make install command when building from source

When building from source, there is some complexity around installing crystal in /usr/bin since it also requires .build/crystal to be copied to /usr/lib/crystal/bin/. Adding a make install would be very helpful, though there's some complexity around overwriting the existing crystal install.
Thanks!

I don't think we should recommend end users to make install anything, but it could be a good thing to do to standardize packaging.
Some inspiration on what the standard locations could be: https://github.com/jhass/PKGBUILDs/blob/b62760f9a47d31a117f942362c8e6b885a91acf7/crystal-git/PKGBUILD#L64-L87 (also used in main Archlinux package).

Just curious, whats the concern with users doing make install. For example, I've was hitting a bug that was fixed after 1.0, so building from source is my only option currently. I would imagine that wouldn't be that uncommon. Not the first way to install crystal, but helpful when you need it imho.

Because software installed outside a package manager is hard to cleanly update and remove from a system again :)
I keep alias dcrystal="$HOME/projects/crystal/bin/crystal" around to work with a locally build version.

I think installing could actually work really well, if you want to use that.
But yeah, standardizing this for packaging might be the real value.
The package building scripts could make use of make install:

https://github.com/crystal-lang/distribution-scripts/blob/6bdf20d18ea7a5b14dd31a118d828a9280a4ff75/linux/Dockerfile#L106-L131
https://github.com/crystal-lang/distribution-scripts/blob/6bdf20d18ea7a5b14dd31a118d828a9280a4ff75/omnibus/config/software/crystal.rb#L66-L74

The only significant precondition is that we need to include the wrapper script in this repository.

The Archlinux package shows for many years now that proper for distribution packaging needs no wrapper script. I would not like to see for our make install to include any workarounds just to make another project (cross-distribution packaging) easier.

Ok, than the precondition would be to remove the wrapper script.
But doesn't that require to bake the CRYSTAL_PATH config into the binary, essentially mixing build and install?
Or, alternatively, place the logic for relative path resolution in the compiler binary. I suppose if we have a install structure defined by make install, this shouldn't be much of a problem.
And package maintainers could obviously still use the wrapper script to override the default behaviour.

A package always builds and installs in a coordinated process. Thus baking packaging specific defaults into the binary is perfectly normal. This what we introduced all the CRYSTAL_CONFIG_ settings for in the first place. The most traditional solution for these kinds of things are ./configure flags which you find in thousands of projects. So think of our CRYSTAL_CONFIG_ variables as our ./configure. Projects in the suckless family actually go as far as making changing any configuration require a recompile. I encourage you again to have a look at the Archlinux package script, it's just a simple shell script that shows this setup.
Now that we got a 1.0 we should actually try to find distribution package maintainers for official distribution packages. They will actually expect behavior like this and be irritated by requiring configuration through a wrapper script.
Such an approach of a wrapper script and what not is still fine for things like our repository where we want to find a packaging style which works across as many distributions as possible with as least effort as possible. But it's the responsibility of the project creating those packages to make that work, it has no place in the main project and the main project should make no decisions or set any defaults just to make that job easier.

Yeah well, I see a problem with baking in a fixed path like CRYSTAL_CONFIG_PATH="/usr/lib/crystal" because it forces a specific install location. With normal package installs, this might be fine because it would likely be the canonical location anyways.
But you can't just download a tarball and run that to use a different Crystal build, because it would pick up the stdlib at the canonical location /usr/lib/crystal, not the one that was shipped with the specific compiler version in the tarball.
A generic solution to that could be to resolve the stdlib path relative to Program.executable_path with a default value of ../share/crystal/src for example. So the only fixed thing would be the path relation between the compiler binary and the associated stdlib sources. And I think that should be fine, even as a cross-platform solution (although the relative paths may differ based on the operating system).
There's still enough flexibility through configuring specific paths at compile or run time if necessary.

A pre-compiled tarball is essentially a variant on a cross-distribution package. Shipping a wrapper script with such a package to make it work seems entirely reasonable. I don't understand why we need to change the compiler's behavior to accomodate that special usecase.

I assumed if we want to get rid of the wrapper script, we could get rid of it entirely. That makes more sense to me, than keeping it around only for some distribution channels.
It would only require a tiny compiler change to make this possible. This seems very reasonable to me.
The pre-compiled tarball is not just a single use case, it also serves as a basis for other channels, such as our deb and rpm packages. It would be great if we can get rid of the wrapper script for those, too (without having to re-build binaries).

Is it a tiny change though? Code-wise maybe, but semantic wise? You wouldn't want to apply this logic to CRYSTAL_PATH obviously but only to CRYSTAL_CONFIG_PATH. But now the semantic of their value is substantially different and thus harder to understand. It'll also make it impossible to include additional paths relative to the working directory (like we do for lib) via CRYSTAL_CONFIG_PATH.

I was thinking of using a special notation to designate a path relative to the executable. Or we could introduce an entirely new config variable for this. Perhaps it could even be a fixed default in the compiler source, where we currently have {{env("CRYSTAL_CONFIG_PATH") || ""}} the empty string could be replaced by relative path resultion at runtime.


Replace T-key rotation in the editor with ActionListeners

With regards to #96, the current behaviour was defined in the tick() function of the EditorApplication which made it ignore the input and focus checks.
This PR moves the behaviour to MenuItems and assigns new commands, Ctrl + Left and Ctrl + Right for rotation, CW and CCW respectively (as the original T key binding is reserved for texture rotation actions).

Although the changed key binding is not optimal, the alternative to this would be hacking into the EditorInput and InputProcessor logic and skip the MenuAccelerator usage. That's also viable, of course, but would result in hardcoded overridden behaviour.
Let me know what you think on the remapping. If undesirable to remap, we can bypass in EditorInput.


[bug] save button in post is not disabled on loading

'Save' button in post creating or editing is not disabled while loading.
It is causing a lot of fetch request while loading too.
To reproduce the issue

Go to create or edit post
Open network tab on dev tools to notice fetch requests.
Click 'Save' button continuously

Resolved in - PR-270


Fix checkbox CSS in node selection UI

When selecting nodes as part of run options/schedules, there is a CSS issue with the checkboxes coming from the blueprint library:

This is also an issue in the create schedule modal as well, so a general problem with blueprints checkbox CSS.


✨ feat: Ability to specify and go to a redirect url after successful logout

Based on Jeroen's answer in the linked ticket, I am now forwarding to the /logout url on Spring Auth Server instead. Once logged out successfully, is there any OAuth2 magic that will use the post_logout_redirect_uri angular-oauth2-oidc passes in as a request param?
Off topic: As I don't want to overwrite the /logout POST method, I am trying to find a hook in Spring Web MVC that lets the caller specify a redirect url. I'm guessing a filter could listen to /login?logout but wondering if there is a more standard way of doing redirects in OAuth.
🍺

I think this needs an update. It is not a bug, it's a feature.
Based on Jeroen's answer in the linked ticket, I am now forwarding to the /logout url on Spring Auth Server instead. Once logged out successfully, is there any OAuth2 magic that will use the post_logout_redirect_uri angular-oauth2-oidc passes in as a request param?
Off topic: As I don't want to overwrite the /logout POST method, I am trying to find a hook in Spring Web MVC that lets the caller specify a redirect url. I'm guessing a filter could listen to /login?logout but wondering if there is a more standard way of doing redirects in OAuth.

@bjornharvold Can you please update the main description comment and/or last comment as both are different. The main description refers to revoking a token and the last comment is logout. It's not clear what you are asking.

✅

@bjornharvold Support for post_logout_redirect_uri will be added in gh-266 and gh-58.
For now, you can customize logout using Spring Security's LogoutHandler and/or LogoutSuccessHandler. See Handling Logouts.
I'll close this as a duplicate.


ORA-17008: Closed connection

Description
Dear Team,
Can you please help me getting connected to database that runs on Oracle 12c
I am facing the below error from the DBeaver Community Edition
ORA-17008: Closed connection
Thanks
DBeaver Version
Community Edition Version <IP_ADDRESS>409011551
Operating System
Windows 10
Database and driver
Oracle database connectivity
Steps to reproduce
I tried to connect to database that runs on oracle by providing all details like port, hostname, username, pass
And also tried entering by making the TNS entry as well, but still ended up with below error
ORA-17008: Closed connection
Additional context
Kindly help to fix this connectivity issue

We kindly ask you to reproduce the issue once again. After you reproduce the issue, please send us the DBeaver log files.
The log files are generated automatically and called .log and .dbeaver-debug.log exactly. They are located in the %APPDATA%\DBeaverData\workspace6\.metadata directory. Here is our wiki page on this matter: dbeaver.com/docs/wiki/Log-files/#log-files

Seems this issue is stale. If the issue is still actual please let me know and provide requested log files.


⚠ SceneTime has degraded performance

In a35a4db, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 3052 ms

Resolved: SceneTime performance has improved in 18b0b7c after 8 minutes.


Mipmap pow2 compatibility

Fixes #5407
This is compatibility PR, that way webgl1 and webgl2 produce the same result. If someone want non-pow2 texture to have mipmaps, they'll have to specify it.
baseTexture.mipmap = PIXI.MIPMAP_MODES.ON;

The values are compatible with booleans in v4. false is OFF=0, true is POW2=1.
Demo: https://www.pixiplayground.com/#/edit/jk06IGwLU1Q9yT41pv7_j

Mipmap off, mipmap on.

Updated example. WTH is going on here?

Now, I need @GoodBoyDigital decision, whether we keep old behaviour as default, that's the same on webgl1 and webgl2, or we actually go for webgl2 is different even on small examples route.


Accounts listing page metadata displays "all" instead of actual number of total accounts

Blocked by https://github.com/Granola-Team/mina-indexer/issues/1085


Odd % sign beside timestamp

@Tuanhao Do you want to try to fix this?

Maybe try reversing the order of the color for yellow to {0, 30, 43}. Do we also need to add a Color that resets the input to black background and white foreground>

Oh nice fix. Yeah we might need to do the reset of the colours than

@Tuanhao What is the %s for in \x1b[0m\x1b[30m\x1b[43m?

The "%s" is like a sign so that the machine know the following text will apply the color code that is after "%s". In the color module, everything after "%s" is the code to clear all color.


Album art off-center when used with Spicetify genre extension

When an additional line is added under the title and artist, it causes the album art to get offset a little too far up. I did a little quick testing, and it looks like it fixes itself with this change:
.rVxzkDirgkuRPv5V1HYF, .main-coverSlotCollapsed-container {
    top: auto
}

Thanks for the great theme, and I hope you have a chill day!

When an additional line is added under the title and artist, it causes the album art to get offset a little too far up. I did a little quick testing, and it looks like it fixes itself with this change:

.rVxzkDirgkuRPv5V1HYF, .main-coverSlotCollapsed-container {

    top: auto

}

Thanks for the great theme, and I hope you have a chill day!

You are always welcome to open a pr

@cpulley would you open a pr or may i include your fix in css by myself?

Done! Sorry, hadn't checked github for a few days, haha.

It's all right
Thanks


UI: [Bug] Column widths for Filtered and Unfiltered Lists are not the same

Limitations of having a scrollbar


Bump Go version range in CI to 1.15-1.17

What type of PR is this?
(REQUIRED)

[ ] bug
[x] cleanup
[ ] documentation
[ ] feature

What this PR does / why we need it:
(REQUIRED)

Only support latest 3 versions, and more urgently, stop the binary size from failing CI on Go 1.14 (which we presumably no longer care about).
Which issue(s) this PR fixes:
(REQUIRED)

Special notes for your reviewer:
(fill-in or delete this section)

Testing
(fill-in or delete this section)

Release Notes
(REQUIRED)

NONE

@urfave/cli quick version bump if anyone is around to approve.

@saschagrunert @asahasrabuddhe if either of you have a minute to look this over, this fixes the broken pipeline in the main branch.


[BEAM-6923] limit number of concurrent artifact write to 8

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

R: @robertwb @pabloem @lgajowy

Note that gRPC does support pushback and signalling to the client via the HTTP/2 protocol it is built on top of, should we be using that?

Note that gRPC does support pushback and signalling to the client via the HTTP/2 protocol it is built on top of, should we be using that?

If that works, it'd be good to use, but would this apply to multiple concurrent connections, which I think is the issue here?

Yes since you can keep track of the number of calls and on a per call basis
provide pushback.
On Tue, Sep 24, 2019 at 12:43 PM Robert Bradshaw<EMAIL_ADDRESS>wrote:

Note that gRPC does support pushback and signalling to the client via the
HTTP/2 protocol it is built on top of, should we be using that?
If that works, it'd be good to use, but would this apply to multiple
concurrent connections, which I think is the issue here?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/9647?email_source=notifications&email_token=ACM4V3ERFQ2J4S4BI6ELMTLQLJUWRA5CNFSM4IZ2K6FKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7PSLUY#issuecomment-534717907,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACM4V3BMEGBM7P7EP6DAMELQLJUWRANCNFSM4IZ2K6FA
.

I searched a bit but could't find good documentation and usage.
Their is also an ongoing issue open on grps for limiting concurrent connection https://github.com/grpc/grpc-java/issues/1886
Do you have any pointers which we can use?

Fair enough : )
On Tue, Sep 24, 2019 at 3:29 PM Ankur<EMAIL_ADDRESS>wrote:

@angoenka commented on this pull request.
In
runners/java-fn-execution/src/main/java/org/apache/beam/runners/fnexecution/artifact/BeamFileSystemArtifactStagingService.java
https://github.com/apache/beam/pull/9647#discussion_r327860963:

@@ -77,6 +78,13 @@
public static final String MANIFEST = "MANIFEST";
public static final String ARTIFACTS = "artifacts";

private final Semaphore permittedConcurrentWrite;

public BeamFileSystemArtifactStagingService() {
super();
permittedConcurrentWrite = new Semaphore(8);

The name of the variable permittedConcurrentWrite seems to be descriptive
enough hence i avoided creating a new constant.
Let me know if you think it will be better to create a constant.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/9647?email_source=notifications&email_token=AAJ5Z3AODUI22INBO4ZVBTLQLKICPA5CNFSM4IZ2K6FKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCFZOLIY#discussion_r327860963,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJ5Z3B5HC2565SEVC4GY3TQLKICPANCNFSM4IZ2K6FA
.

It may be much simpler to set the GCS upload buffer size to 1MiB to solve BEAM-6923 via setting GcsUploadBufferSize. I think historically we have used 1MiB in the past while the default is 64MiB

This makes sense. Do you think we should apply this configuration for all the gcs write.
I am a bit concern as it can impact performance of data path of gcs sink. We can enhance the filesystems configuration to allow passing filesystem specific options.

It may be much simpler to set the GCS upload buffer size to 1MiB to solve BEAM-6923 via setting GcsUploadBufferSize. I think historically we have used 1MiB in the past while the default is 64MiB

This makes sense. Do you think we should apply this configuration for all the gcs write.
I am a bit concern as it can impact performance of data path of gcs sink. We can enhance the filesystems configuration to allow passing filesystem specific options.

In portable pipeline execution we don't expect to run user transforms for reading/writing from GCS and this would only be for Runner interactions with GCS so I believe we could make it 1MiB for the GcsFileSystem by default.

It may be much simpler to set the GCS upload buffer size to 1MiB to solve BEAM-6923 via setting GcsUploadBufferSize. I think historically we have used 1MiB in the past while the default is 64MiB

This makes sense. Do you think we should apply this configuration for all the gcs write.
I am a bit concern as it can impact performance of data path of gcs sink. We can enhance the filesystems configuration to allow passing filesystem specific options.

In portable pipeline execution we don't expect to run user transforms for reading/writing from GCS and this would only be for Runner interactions with GCS so I believe we could make it 1MiB for the GcsFileSystem by default.

Makes sense.
Made the change to limit the buffer usage only for flink and spark job server

Run Java PreCommit


Matching improvements

@bigodines

Could you add an example or note to the docs with the new features?

@bigodines see latest commit :)


version tag mismatch github: v2.5.0 python: 2.5.1

if you build from source from github tag v2.5.0 it produces a module file that's 2.5.1

nevermind I may have accidentally pulled HEAD instead of v2.5.0

nevermind I pulled branch r2.5 rather than tag v2.5.1


feat(semantic2): slot based mangler

Some function calls are not mangled because they are declared after the calls. Probably need to visit all function declarations first.


For packaged apps using BI (PushTrigger) handle foregroundHandled=false scenario

Description: For packaged apps using BI (PushTrigger enabled), if channel Object is present and foregroundHandled is set to false handle the scenario to do protocol activation.

The Handled property has been removed from PushNotificationReceivedEventArgs so this issue doesn't apply anymore.


Useful link

Dear Tal, you might want to add the following link that contains many advices: https://news.ycombinator.com/item?id=12118554

That's an interesting thread! Thanks for sharing, I added it to the list 👍


Changing colors of markers in edit mode

Hi guys!
I need to set color of these markers(orange ones) in edit mode for polygon dynamically. Did not found the right method in documentation.
I can set the fill color for polygon, lines, but no luck with markers. Is it possible?
Would really appreciate any help.

This markers are only styled by css, but I think you have found this already because of the change to orange.
https://github.com/geoman-io/leaflet-geoman/blob/7cdb0edadc800edf3498fe6f78d140bccaa0aaa9/src/css/layers.css#L1-L12
You can add a css class to the map and if the class is applied to the map change the css of the vertex markers:
https://jsfiddle.net/falkedesign/ojg3v0dt/
var changed = false;
function changeColor(){
  if(!changed){
    L.DomUtil.addClass(map.getContainer(),'custom-vertex');
  }else{
    L.DomUtil.removeClass(map.getContainer(),'custom-vertex');
  }
  changed = !changed;
}

CSS:
 #map.custom-vertex .marker-icon {
   background-color: #f00;
 }

This markers are only styled by css, but I think you have found this already because of the change to orange.
https://github.com/geoman-io/leaflet-geoman/blob/7cdb0edadc800edf3498fe6f78d140bccaa0aaa9/src/css/layers.css#L1-L12
You can add a css class to the map and if the class is applied to the map change the css of the vertex markers:
https://jsfiddle.net/falkedesign/ojg3v0dt/
var changed = false;
function changeColor(){
  if(!changed){
    L.DomUtil.addClass(map.getContainer(),'custom-vertex');
  }else{
    L.DomUtil.removeClass(map.getContainer(),'custom-vertex');
  }
  changed = !changed;
}

CSS:
 #map.custom-vertex .marker-icon {
   background-color: #f00;
 }


feat: :sparkles: adding apache flink operators crds

This commit adds the following CRDs for the
"flink.apache.org" group:
flinkdeployments.flink.apache.org
flinksessionjobs.flink.apache.org
The schemas were extracted with:
(Summary)

Passing YAML validation: 1/1

Passing Kubernetes (1.24.0) schema validation: 1/1

Passing policy check: 1/1

+-----------------------------------+----+
| Enabled rules in policy "Starter" | 34 |
| Configs tested against policy     | 1  |
| Total rules evaluated             | 34 |
| Total rules skipped               | 0  |
| Total rules failed                | 0  |
| Total rules passed                | 34 |
| See all rules in policy           |    |
+-----------------------------------+----+
[INFO] You're running Datree in offline mode%

@howard-oc I see you only added a link to the CR docs.
can you please also add a reference link to the github repo that contains the CRD itself?

@eyarz  the github repo is https://github.com/apache/flink-kubernetes-operator/blob/main/helm/flink-kubernetes-operator/crds/
Thanks


spirv-opt module builder injects extra OpLine instructions that don't exist on the input

This is a consequence of #3951
This breaks some line number attribution in Tint, and breaks one of its tests.
Here's the sample SPIR-V assembly:
OpCapability Shader
OpMemoryModel Logical Simple
OpEntryPoint GLCompute %2 "main"
OpExecutionMode %2 LocalSize 1 1 1
%15 = OpString "myfile"
%void = OpTypeVoid
%4 = OpTypeFunction %void
OpLine %15 42 53
%uint = OpTypeInt 32 0
%60 = OpConstantNull %uint
OpNoLine
%2 = OpFunction %void None %4
%1 = OpLabel
OpReturn
OpFunctionEnd

The test checks to see that %1 is instruction number 13 (start counting from 1).
That worked until #3951 was merged, and now it is reported as instruction number 14.
The Tint cod uses module->ForEachInst to walk the module.  To help debug the issue I added a print to see the opcode for each instruction that is visited.  I get this output:
[ RUN      ] SpvParserTest.Impl_Source_WithOpLine_WithOpNoLine
 inst 17
 inst 14
 inst 15
 inst 16
 inst 7
 inst 19
 inst 33
 inst 8
 inst 21
 inst 8          ; <<<<< This was an OpLine injected by the change in #3951
 inst 46
 inst 317
 inst 54
 inst 248
 inst 253
 inst 56

Basically, the module no longer accurately represents the input module.

I can address this for my purposes by adding an "IsSynthetic" flag to the instruction.  Then I can see which OpLines have been injected and I can skip them.


Add method to set a cell value in FitData

Make it possible to edit a single cell in FitData.
This is needed to solve EddLabs/eddington-gui#9

solved by #29


Ensure CLI can load without git on the PATH

In practice most customers will have git on their PATH, but it's
probably preferable not to fail loading the entire CLI just because git
is not available (especially considering we're able to handle that
gracefully).
cc @fancyremarker @blakepettersson

:shipit:


Cleanup SSR

Title of change
why the change and what has changed
Checklist

[ ] Unit tests written
[ ] Manually tested
[ ] Prettier & ESLint were run
[ ] New dependencies are included in package-lock.json

Screenshot
insert screenshot if needed

Deployed to: http://staging.quran.com:32774

Deployed to: http://staging.quran.com:32777

Deployed to: http://staging.quran.com:32779

Deployed to: http://staging.quran.com:32780


Keep event keyboard on overlay blur

Hello, I'm trying to create an overlay to track a user's actions.
I'm using the following code in the overlay html code to track mouse movements:
document.addEventListener('mousemove', (e) => {           console.log(e);         });
For the keyboard, I use the following code:
document.addEventListener('keydown', (e) => {           console.log(e);         });
When the overlay is not in focus, none of the codes work.
By modifying the code a little, in particular by adding the { forward: true } option in the code when calling the setIgnoreMouseEvents function, I get the mouse movement.
However, for keyboard actions, if the overlay is not focused, I get no result.
Can anyone think of a way to retrieve keyboard inputs even if the overlay isn't focused?

Obviously only the event loop of focused window receives inputs from OS. So you need a hacky way to hook into global events, look into uiohook-napi package

Yes, clearly, I was wondering if it was possible to get the inputs directly from the C files and send them back to electron.
I've started looking at the uiohook-napi package and I'm going to continue on this path.
Thanks for the reply

The uiohook-napi work perfectly, thx for the work and have a good day !


nvm.sh: added nvm_is_terminal()

See https://github.com/creationix/nvm/issues/2005#issuecomment-466690060

You've got good tests! I didn't know I had to add nvm_is_terminal to the unload section. Fixed. I just added it at the end. Hope that's OK.

I'll try to add unit tests tomorrow.
I ran a few simple tests nvm_is_terminal on my machine (macOS 10.13.6, bash 3.2.57), and it seems to work. For example, I added these lines at the start of nvm_ls():
if nvm_is_terminal; then
  echo term
else
  echo no term
fi

When I call the function with / without a pipe, the results look OK:
> nvm_ls
term
N/A
> nvm_ls | cat
no term
N/A

(I ran . ./nvm.sh without proper configuration, so the N/A is OK.)
Could you post more details about what's not working on your machine?

I put the function in my shell, and piped something to it.

Testing the function in the terminal can be a bit delicate...
First, I'm not sure what you mean by "piped something to it", but it might be the wrong thing. The 1 in [ -t 1 ] stands for file descriptor 1, i.e. stdout. So the function nvm_is_terminal only tests whether stdout is a terminal or not and ignores stdin and stderr. That makes sense for our use case: As far as I can tell, no nvm command reads any data from stdin, they all just write stuff to stdout (and error messages to stderr, but they don't matter here). If you pipe something into the function, e.g. nvm_is_terminal </dev/null, you're replacing stdin, which doesn't affect the result of the function.
Shell syntax can also be tricky. For example, this command
$ nvm_is_terminal ; echo $?
0

has the expected result 0, because I'm calling nvm_is_terminal in a terminal. (As usual, shell truth is the opposite of C truth: 0 means success or true, all other numbers mean failure or false.)
If I want to check the result of nvm_is_terminal in a pipe, I might do this:
$ nvm_is_terminal ; echo $? | cat
0

Oops... Still 0. Is nvm_is_terminal broken? No, my syntax is wrong. The pipe | only affects echo, not the part before the ;. This syntax is correct:
$ (nvm_is_terminal ; echo $?) | cat
1

Now both commands are attached to the pipe, and we get the expected result 1 (not a terminal).
I'm pretty sure nvm_is_terminal is working correctly on my machine, but I may be missing something. Could you post the exact shell commands you're using?

nvm_is_terminal() { if [ -t 1 ]; then echo y; else echo n; fi } ;
echo hi | nvm_is_terminal // i expect n, i get y
nvm_is_terminal // i get y as expected

echo hi | nvm_is_terminal // i expect n, i get y

That's the stdin case I mentioned in my previous comment. y is the expected result, because the output of nvm_is_terminal still goes to a terminal in this case.
Try this instead:
$ nvm_is_terminal() { if [ -t 1 ]; then echo y; else echo n; fi }
$ nvm_is_terminal | cat
n

nvm_is_terminal only checks whether the output (stdout) goes to a terminal. It doesn't care about the input (stdin). And that's the way it should be: We need to check whether we should modify the output of nvm ls etc. We don't care about any input.

Thanks, that clarifies it. Perhaps a better name then would be nvm_outputs_to_terminal or something? Also, does this only check stdout - what about stderr? If stdout is redirected to a file, or to dev/null, is that "a terminal"?

It only checks stdout. (It would be easy to also check stderr, but I don't think that would make much sense. When people redirected stderr, I think they expect normal output not to be affected.)
If stdout is redirected to a file or /dev/null, that would not be a terminal. The specification for the [ -t ] flag says: "True if ... file_descriptor is open and is associated with a terminal. False if ... file_descriptor is not open, or if it is open but is not associated with a terminal."
I like short names, so I chose nvm_is_terminal, but maybe that's too short. :-) Maybe nvm_stdout_is_terminal would be best? Clear, precise, not too long...?

nvm_stdout_is_terminal seems clear enough that if i forget all about this PR and find it in a year, I'll still know what it means :-) let's go for it.

OK, I renamed the function to nvm_stdout_is_terminal (also in the unload section) and renamed this PR. I'll look at the unit tests next.

The nvm_stdout_is_terminal test passed in all shells. Nice. :-) Is there anything else I can do for this PR?

Added another little fix. (A failure in one of the tests didn't properly signal a failure, it just printed an error message.)

I found a way for the test with the pipe to transfer the return value of nvm_stdout_is_terminal back to the test engine, but it's a bit cumbersome and brittle. I temporarily checked in a deliberately broken version of the test to see if it fails on travis-ci. I simply changed this line
(! nvm_stdout_is_terminal || echo "boo!") | read && die 'nvm_stdout_is_terminal should be false when stdout goes to a pipe'

to this (without the negation at the start):
(nvm_stdout_is_terminal || echo "boo!") | read && die 'nvm_stdout_is_terminal should be false when stdout goes to a pipe'

The results are here: It signals an error in bash and zsh (good), but signals success in dash and sh (bad).
I don't have dash, so I can't test it. I do have sh, and when I run the deliberately broken test locally in sh, it signals an error. I don't know what's going on with dash and sh on travis-ci, but I guess bash and zsh are much more relevant anyway, so I'd say the tests are good enough...

sh is usually bash on many machines; dash is something you'd have to install separately.
I'll take a look; it'd be ideal to make the test fail in dash as well, when expected. Can we modify the test so it's both testing true and false cases?

I just installed dash (via Homebrew) and ran the test by executing ./nvm_stdout_is_terminal in ~/.nvm/test/fast/Unit tests. Same result as for all other shells: When I break one of the tests by toggling the ! at the start, the test fails and the return value is 1 (I checked it by echo $?). I checked all 13 tests in both dash and sh. So it looks like travis-ci does something strange with dash and sh in this case.
The test already checks true and false cases as much as possible, but we can't test the "real" true case on travis-ci, because the tests there are not attached to a terminal. I added a few tests that redirect stdout to /dev/tty, and nvm_stdout_is_terminal reports true in that case.
Anyway, I wouldn't bother with the tests on travis-ci anymore. I mean, the function nvm_stdout_is_terminal() is extremely simple (just 40 characters long, 22 of which are the name). I'm sure it's correct. I ran the tests in all four shells on my machine, and they work as expected. Also, the test file has 1805 characters, so that's an extremely good test code tp production code ratio. ;-)


Postcss transform

Would be nice if there was a babel transform to run the template literal through a configurable postcss pipeline.

csjs has it – maybe you can fork theirs.
But since this library is using postcss internally already there might be a better way, for example here is how we do this in suitcss-preprocessor

Which plugins would you like to see? We're looking towards linting mostly, which others are interesting to you?

We're using a modified version of PostCSS internally but we may change that up in future. We're not trying to build a flexible CSS processor we're just providing a minimum of functionality (i.e. nesting). If there are particular transforms people want to use then let us know.
The good news is that a lot of the use cases for postcss transforms can be replaced with pure-JS approaches. I've just updated the docs with an example: https://github.com/styled-components/styled-components/blob/master/docs/tips-and-tricks.md

The idea is not that we need a specific list of transforms for use with this library, more so that leveraging the previous work and extensive ecosystem of postcss would a great benefit to this project.
I recent, concrete example was that we wanting to use the object-fit polyfill for certain browsers. There is already a postcss transform designed exactly for our use case. Other things might be responsive text, like we've got in rucksack. It's not really about exactly which ones I want to use, but opening up the tool to leverage an ecosystem that already exists.
One of the things I like about the babel-transform option is that it remove the need for styled-components to have any kind of preprocessing included. You could just set a few sane default postcss plugins and allow the option for addition/overwriting of postcss plugins. csjs has the right idea here in my opinion. They've made a csjs specific babel plugin for postcss transformations at build time. (That is another good reason to use a babel transform, the pure js solutions to postcss (jss has many of these types of things) tend to be designed to run at runtime on the string literal or object. A babel plugin would reduce runtime cost as well as runtime dependency weight.
I can carve out some time to look into this if it's something the style-components team would consider integrating.
Thanks.

Yeah I get where you're head's at, but I'm reluctant to move in that direction at this moment. The way we use tagged template literals and interpolations makes it difficult to integrate any pure-CSS tool like PostCSS. Which is why we've had to break compatibility already (just minor stuff so far) to get it running where we want.
The other thing is that the two use-cases you mention (object fit polyfill & responsive text) seem far better suited to a component-driven solution (e.g. react-object-fit-cover). Styled Components presupposes both a component system & a runtime JS environment which I feel like can cover most of the things people have needed PostCSS transforms for in the past.
But depending on what #59 ends up with, it might be possible. I'm going to close this in favour of that one, but I'll keep the idea in the back of my head as I work on it.


Container scrolling too far.

Issue Description
The scroll view seems to be going too far when I point at 7pm to scroll to.
Code I'm using with CalendarKit
let timeline = TimelineView()
timeline.date = startDate
timeline.frame.size.height = timeline.fullHeight

let container = TimelineContainer(timeline)
container.contentSize = timeline.frame.size
container.translatesAutoresizingMaskIntoConstraints = false
container.addSubview(timeline)

view.addSubview(container)
NSLayoutConstraint.activate([
    container.topAnchor.constraint(equalTo: view.topAnchor),
    container.bottomAnchor.constraint(equalTo: view.bottomAnchor),
    container.leadingAnchor.constraint(equalTo: view.leadingAnchor),
    container.trailingAnchor.constraint(equalTo: view.trailingAnchor)
])

timeline.eventDescriptors = ...

container.scrollTo(hour24: Float(startDate.hour))

Notice the green background view that's showing through.

Currently, CalendarKit does not take "overscroll" possibility into account. You have to implement it either manually, or apply a patch to TimelineContainer

Awesome!  @richardtop please remember to update the POD to include these fixes.

@grosch I'll update pod soon, but if you want to receive all the newest changes, you can add link to this repository in the podfile.

@richardtop ETA on when you'll fix the POD?


ff4j-spring-boot-starter would not start due to circular reference

With latest version of ff4j-spring-boot-starter there is an exception when you boot
org.springframework.beans.factory.BeanCurrentlyInCreationException: Error creating bean with name 'getFF4JServlet': Requested bean is currently in creation: Is there an unresolvable circular reference?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/34906527-ff4j-spring-boot-starter-would-not-start-due-to-circular-reference?utm_campaign=plugin&utm_content=tracker%2F1038413&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1038413&utm_medium=issues&utm_source=github).

As a workaround use the following configuration
import org.ff4j.FF4j;
import org.ff4j.web.embedded.ConsoleServlet;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.boot.autoconfigure.EnableAutoConfiguration;
import org.springframework.boot.context.embedded.ServletRegistrationBean;
import org.springframework.boot.context.web.SpringBootServletInitializer;
import org.springframework.context.annotation.Bean;
import org.springframework.context.annotation.Configuration;

@Configuration
@EnableAutoConfiguration(exclude = {org.ff4j.spring.boot.autoconfigure.FF4JWebConfiguration.class})
public class FF4JWebConfig extends SpringBootServletInitializer {
  @Autowired
  private FF4j ff4j;

  @Bean
  public ServletRegistrationBean servletRegistrationBean() {
    return new ServletRegistrationBean(getFF4JServlet(), "/ff4j-console");
  }

  @Bean
  public ConsoleServlet getFF4JServlet() {
    ConsoleServlet ff4jServlet = new ConsoleServlet();
    ff4jServlet.setFf4j(ff4j);
    return ff4jServlet;
  }
}

Thanks for work around. Now when accessing /ff4j-console, page displays only text, no graphics (looks like issue with css or script dependency ???) not sure. Can you please provide quick help ?

You're right. There is a new (and very cool) UI in next version. (see #118) I think i broke the old one
You can test it by defining the following Servlet (and use the version in HEAD.)
	<servlet>
		<servlet-name>ff4j-console</servlet-name>
		<servlet-class>org.ff4j.web.FF4jDispatcherServlet</servlet-class>
		<init-param>
			<param-name>ff4jProvider</param-name>
			<param-value>${your-implementation-of-FF4JProvider-className}</param-value>
		</init-param>
		<load-on-startup>1</load-on-startup>
	</servlet>

If you want the old override the version of dependency ff4j-web with the version 1.5 (and not 1.5.1 with a bug). I will fixe the old one with 1.5.2 may be this week, sorry for the convenience


Add wiki sections

Not an issue but feature request...
would be very nice to add option to select "did you know" , "on this day" snippets from wikipedia instead of totally random articles!
thanks in advance

added 'did you know' category as an option https://github.com/mihairinzis/MMM-wiki/commit/075a5e9f8dce019d3c1e174e8ca6e0dec61b6f54


view web page without an iFrame?

  I'm trying to browse in an iFrame and security measures like Content Security Policy are killing me.
I imagine retrieving a page into an html string and dumping it into a DIV would be a non-functional mess.

Very helpful @frankhale !


Fix pit download_data script TypeError (#978)

Fix pit download_data script TypeError (#978)
Description

See #978.
Motivation and Context

How Has This Been Tested?
<! ---  Put an x in all the boxes that apply: --->

[ ] Pass the test by running: pytest qlib/tests/test_all_pipeline.py under upper directory of qlib.
[ ] If you are adding a new feature, test on your own test scripts.

Screenshots of Test Results (if appropriate):

Pipeline test:
Your own tests:

Types of changes

[x] Fix bugs
[ ] Add new feature
[ ] Update documentation

Thanks so much!

Thanks so much!


restore html_format method

👋,
I was looking through this package and noticed some unreachable code that looks like it was missed when python 2.5 support was dropped here.
Now I'm not totally sure this code is even needed still. It looks like the EscapeFormatter will check for and call it. If it is still needed then perhaps a test is in order.

Hmm not sure why travis keeps failing. coverage combine keeps exiting with an error.

@davidism can you provide a review of this when you have a chance? Thanks!


Refactored/SfSlidingSection

Related issue

Scope of work

Imported hammerjs and better mobile behaviour form SfSlidingSection
Screenshots of visual changes

Checklist

[x] I followed composition rules for my component
[x] I tested the component in most common device sizes (can be tested in Storybook from viewport addon in top menu)

How about Storybook? Maybe we should just add a story with full-green image and some text with heading below?

Also make sure it work in SSR

About Storybook

Also we might want to have a SCSS modifier to distribute content in 2 columns on desktop (instead of below each other )
@filrak but on desktop we have 2 columns?

Also make sure it work in SSR

unsupported, because hammerjs unsupported ssr 👎

@przspa can we add mobile padding (with spacer to the slot by default) to make it look a bit better by default/ If you think it's a bad idea you can add just to this story

@przspa can we add mobile padding (with spacer to the slot by default) to make it look a bit better by default/ If you think it's a bad idea you can add just to this story

@filrak Done

Thanks!


error on trigger sync for python based linux function app in eastus

Question, Bug, or Feature?
Type: Bug
Enter Task Name: AzureFunctionV1
list here (V# not needed):
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
Environment

Server - Azure Pipelines or TFS on-premises? Azure Pipelines

If using Azure Pipelines, provide the account name, team project name, build definition name/build number:
smartnexus/SNX IoT Platform V1/platops-apim-splunk-release
/Release-14

Agent - Hosted or Private:

If using Hosted agent, provide agent queue name:
Hosted Ubuntu 1604

Issue Description
Our Deploy Azure Function App task fails. We are deploying a python based linux function app in east us. But we can then go manually start function app via azure portal and it works fine.
error:
Error: Failed to sync triggers for function app 'prodiat-apim-splunk-func'. Error: Unauthorized - Encountered an error (Forbidden) from extensions API. (CODE: 400)
Task logs
[Enable debug logging and please provide the zip file containing all the logs for a speedy resolution]
tasklog_6.zip

@SteveDevOps , Is the issue intermittent or consistent?

It is consistent. I've had to turn on allow error on the deploy task so that final function start step will occur. screenshot

I am seeing same errors on new linux funcs / python runtime in new subscription, new resource groups

@SteveDevOps , thanks for the info.
We have directed the issue to the App Service team. Will upate you once there is an response.

If I switch to an app service task, but specifying a linux function type for deployment.. I get a little more info around the error -- however I have tried to mitigate by hardcoding in both app settings mentioned. Additionally my vanilla function is setup with terraform (without code/package), then I'm trying to use azure devops to deploy. Here is error when using an app service task (specifying linux function):
2019-07-15T15:20:31.4024214Z Updating App Service Application settings. Data: {"FUNCTIONS_WORKER_RUNTIME":"undefined-WEBSITES_ENABLE_APP_SERVICE_STORAGE","":"true"}
2019-07-15T15:20:32.8471046Z ##[error]Error: Failed to update App service 'myfunctionname-func' application settings. Error: BadRequest - Parameter name cannot be empty. (CODE: 400)
2019-07-15T15:20:33.2937520Z ##[warning]Error: Failed to update deployment history. Error: Site Unavailable (CODE: 503)

And to clarify, I have tried specifying:
-WEBSITES_ENABLE_APP_SERVICE_STORAGE true
As well as including it as part of setup via terraform ahead of time.
Also found this reference to problem with linux function apps: https://github.com/Azure/azure-functions-ux/issues/3195

figured out how to run with debug, here is output:
2019-07-15T15:44:36.7681626Z ##[debug]old value : python. new value: undefined-WEBSITES_ENABLE_APP_SERVICE_STORAGE
2019-07-15T15:44:36.7681889Z ##[debug]old value : undefined. new value: true
2019-07-15T15:44:36.7686027Z ##[debug][PUT]https://management.azure.com/subscriptions/0a0cfdee-8f61-4057-ad7b-dc024cced801/resourceGroups/platops_westus/providers/Microsoft.Web/sites/iat2-apimlogger-func/config/appsettings?api-version=2016-08-01
2019-07-15T15:44:37.2265225Z ##[debug]Deployment Failed with Error: Error: Failed to update App service 'iat2-apimlogger-func' application settings. Error: BadRequest - Parameter name cannot be empty. (CODE: 400)
2019-07-15T15:44:37.2265578Z ##[debug]task result: Failed

Any updates? still seeing this: Error: Unauthorized - Encountered an error (Forbidden) from extensions API. (CODE: 400)
different function, different region: west us. Trigger info (an eventhub connection string) is correct and referenced as app setting. Function runs fine on manual start/restart after deploy.

Please share the logs to RM_customer_Queries [at] Microsoft [dot] com

@SteveDevOps Are you still facing the issue?

Well, we changed our development efforts and switched to nodejs. We don't
have any issues with nodejs and its much more performant.
On Thu, Oct 10, 2019 at 2:38 AM Sumiran Aggarwal<EMAIL_ADDRESS>wrote:

@SteveDevOps https://github.com/SteveDevOps Are you still facing the
issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/azure-pipelines-tasks/issues/10699?email_source=notifications&email_token=ACLZDECP7IMTZT75GJ535BTQN3EV7A5CNFSM4HZNBG32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEA27OWY#issuecomment-540407643,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACLZDEH57ZACO2NOLKEYZVTQN3EV7ANCNFSM4HZNBG3Q
.

Great!!
Thereby, closing this issue. Feel free to reopen if you face any issues.

Please re-open. We also have this exact same issue.


10.0 ms sync conflicts imp fix 4

network_is_on calling in on_notification_do seems to be excessive:
https://github.com/KolushovAlexandr/pos-addons/blob/75cea6b30dd0e13458c7a9923d840ec129c3250d/pos_longpolling/static/src/js/pos_longpolling.js#L146-L153

Possible renamings:
set_poll_status -> rerender_poll_status
status -> is_online
poll_response_status -> poll_is_restoring

Update changelog please

@yelizariev this PR into 10.0-ms-sync-conflicts, so I updated the last version changes


Getting error in the latest version of nestjs (v10)

I get the following errors:
npm ERR! code ERESOLVE
npm ERR! ERESOLVE could not resolve
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@nestjs/common
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   peer<EMAIL_ADDRESS>|| ^9.0.0 || ^10.0.0" from<EMAIL_ADDRESS>npm ERR!   node_modules/@nestjs/config
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   4 more (@nestjs/core, @nestjs/microservices, ...)
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@ntegral/nestjs-sentry
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Conflicting peer dependency<EMAIL_ADDRESS>npm ERR! node_modules/@nestjs/common
npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@ntegral/nestjs-sentry
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR!
npm ERR! For a full report see:
npm ERR! /Users/vahid/.npm/_logs/2023-08-15T17_10_24_411Z-eresolve-report.txt

npm ERR!      /Users/vahid/.npm/_logs/2023-08-15T17_10_24_411Z-debug-0.log

I am having the same issue in the nestjs latest version

@vahidvdn did you find any solution for this?

@JoshiKeval Not yet, there is another library as well, I may use that or implement on my own

@vahidvdn Could you maybe share the link to the new library?
@ntegral You can bump the version of NestJS without code changes.

@rafaelklaessen Check out this one: https://github.com/mentos1386/nest-raven

@vahidvdn thank you!!


How to handle (synt) parameters that use values >127

The Introduction
Currently i'm writing a interface for a Mopho. But i'm running into some problems.
The DSI Mopho makes by default use of NRPN midi messages. Standard CC messages are possible BUT in some cases it don't handle the full parameter bandwidth. There are a limited set of parameters that handle values >127 (up to 254 == -127 to 127 ).
Currently i'm unable to load (No Error) or save (kind of expected error based on #334) Sysex files .
How to replicate
As an example I add two Sysex files. Random Stepper is the first program of the original sound files. RandomStepper_edit is a modified version where i changed the filter to 0x80 / 128. This is completely valid value according toe the Mopho manual (max value is 0xa4/163).

RandomStepper.syx
RandomStepper_edit.syx

the used code is:
import os
from pathlib import Path
import mido

filein = "RandomStepper.syx"
# filein = "RandomStepper_edit.syx"
fileout = "test.syx"
path = os.fspath(Path(__file__).resolve().parents[3] / "Downloads/Mopho/Mopho1_2/{}")  # optional

# test open Sysex files
sysex_file = mido.read_syx_file(path.format(filein))

#test modify sysex file
single_message = sysex_file[0]
message_list = list(message_data)
message_list[28] = 163
new_message = mido.Message('sysex', data=message_list)

# test save Sysex files
mido.write_syx_file(path.format(fileout), [new_message])

When i open the file RandomStepper.syx it breaks on the line new_message = mido.Message('sysex', data=message_list).
When i open the file RandomStepper_edit.syx it breaks on the line message_list = list(message_data) because my loaded file is an empty list.
The question
Based on this what are the best workarounds / methods to handle files that use the full byte as range of values?
For me another (
RandomStepper_edit.syx.txt
RandomStepper.syx.txt
better suited) library is also an option
References

The Mopho Manual (With his midi specification) can be foundhere
If you want you can download the full set of sounds here.
The Mopho editor can be download here
RandomStepper_edit.syx.txt (remove the .txt)
RandomStepper.syx.txt (remove the .txt)

MIDI, per specification, uses the most significant bit to detect whether the current byte is Status (MS bit set) or Data (MS bit not set).
Therefore SysEx messages can only contain Data bytes which are, by design, 7 bits long only allowing 128 values.
To transmit/receive it's internal 8 bit representation (256 values), your synth uses an 8 bit to 7 bits encoding/decoding scheme to pipe these down the MIDI data stream that is described page 44 of the manual you kindly provided.
Since this is part of the System Exclusive extension to MIDI, it is your duty to encode/decode these values as needed.
You can't just push illegal values in the MIDI stream expecting it not to break.

Oh and by the way #334 is bogus for the reasons explained above.

My question was about reading, writing from / to file. The question was not about sending the SysEx data to a device.
The midi specification don't say anything about file storing as far as i read it...
This is what the Manual say about it (For manual see link original message):

Packed Data Format
Data is packed in 8 byte “packets”, with the MS bit stripped from 7 parameter
bytes, and packed into an eighth byte, which is sent at the start of the 8 byte
packet.

The same rule applies to storing SysEx data which is analogous to the wire protocol.
Your quote from the manual emphases exactly that.
Instead of naively directly modifying a value to anything larger than 7 bits (0x7F) you need to do a bit of encoding first to convert your 8 bit value to 7 bits and insert your most significant bit (MS) into another byte at the correct position.
In summary:

setting it to 128 (0x80) manually as you did will result in a corrupted file that cannot be interpreted
setting it to 163 (0xA3) programmatically will raise a ValueError as expected by design

What you should do is:

Write code that packs values that your Synth support into it's expected SysEx format as outlined above

It would look something like this:
"""
Warning: this is untested code written to help you understand.
What should really be done is write a specific parser for that specific SysEx format
and encapsulate that logic into a nice function.
"""

import mido

syx = mido.read_syx_file('RandomStepper.syx')

print(syx[0].data[28])
# 87 (0x57 0b00111001)

# Set to 163 (0xA3 0b10100011)
value = 163
syx[0].data[28] = value & 0x7F  # Set the 7 bits
if value & 0x80:  # MS is set
    last_byte_in_packet = __  # FIXME: find the boundaries of the 8 byte packet and select the last byte
    bit_position = __ # FIXME: find which byte of the 8 byte packet we just modified (A value between 0 and 7)
    syx[0].data[last_byte_in_packet] = syx[0].data[last_byte_in_packet] | (1 << bit_position)  # Set the most significant bit in the 8th byte at the correct position assuming

Hope this clears things up for you.
I may sound blunt at times but remember that I'm here to help ;)


Firebase/mohamad/Firebasefunctions

Integrated Firebase Auth and Firestore

Had the same question tbh but yeah we'll just add those to the User Model as statics makes... more sense

Gotcha, I'll merge it in and we can revise later if need be!


Keep the previous certificate in the bundle as well as the new one to prevent periods of service outage

During our tests we discovered that every time the certificates rotated we got a lot of TLS bad certificate errors in our cluster.
2022/06/02 13:46:25 http: TLS handshake error from <IP_ADDRESS>:60186: remote error: tls: bad certificate
When we looked into it we discovered that the refresher sets the secret and the caBundle in the webhooks at the same time.
However, that does not account for the time it takes for a pod to get the secret updated and the operator to refresh the certificate.
During that period of time the control-plane is using the new caBundle and the operator the old one, failing all requests and provoking a service outage since none of the resources created at that time will be serviced by kubemod.
The proposed solution keeps the previous certificate in the bundle along with the new one, so that the operator can work well before and after swapping its own certificate.
This means there are always 2 certificates in the caBundle and each certificate will be valid for 2 cron periods, if this is a concern, halving the cron period should work.

Hi @arkaitzj,
Thank you for your pull request!
I tested it on my end and I have several comments:

When deploying kubemod with your version of kubemod-crt on a clean cluster (no kubemod previously installed), the logs of kubemod-crt yield the cryptic message No bundle found. I'd rather see a log line saying Extracting previous CA certificate... followed by either No previous CA certificate found, or [N] previous CA certificates found

I tested kubemod + your version of kubemod-crt with a CA cert refresh cron job running every minute.
I found out that kubemod is still occasionally unhappy, issuing remote error: tls: bad certificate right around the time of the cert rotation. My theory is that there is still timing issue involved.
The current order of renewal is:

cert-generate.sh
cert-create-secret.sh
cert-patch-webhooks.sh

I think the issue might be resolved if we switch to:

cert-generate.sh
cert-patch-webhooks.sh
cert-create-secret.sh

Hi @vassilvk,
I added the extended logging.
As for the order if I am not wrong, I don't think it matters too much because the pods can have delays of up to several minutes refreshing the configmap values, so if the refresh period is inferior to, say, 5 minutes, you will see tls errors due to the operator still using a cert from 2 periods ago.
This could be mitigated by increasing the number of certs to keep, which is just a flag into the tool I added.

Hi @arkaitzj, thanks, this makes sense.
Reordering without adding ugly artificial delays would likely not make a difference.
Let me test again with a 5 min cron job - if I see no bad cert throughout the weekend, we should be good to go with this PR.

Closes #15


feat(android_alarm_manager_plus) : we can now send extra data to alarm manager and receive it in our callback

Description
Now we can send extra data to alarm manager as Map and also we can receive them in our function callback. For example we can do this :
AndroidAlarmManager.oneShot( Duration(minutes: interval), 25, alarmManagerCallback
      , allowWhileIdle: true, exact: true, rescheduleOnReboot: true,
      params: <String,dynamic>{
    "title":alarmModel.title,
    "message":alarmModel.subTitle,
      });

and in our alarmManagerCallback we can receive the extra params :
@pragma('vm:entry-point')
void  alarmManagerCallback(int id,Map<String,dynamic> params) async{
  print("alarm manager callback title is :${params["title"]}");
}

Related Issues
Issue 990
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process.

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See [Contributor Guide]).
[x] All existing and new tests are passing.
[x] I updated the version in pubspec.yaml and CHANGELOG.md.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the [Flutter Style Guide].
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

Hey! Due to changes in our contributor guidelines, all PRs should now:

Have a title that follows the Conventional Commits format. e.g. feat(package_name)!: feature description
DO NOT modify the CHANGELOG.md or the version in the pubspec.yaml. This is going to be an automated process now.

We would ask you kindly to update the PR following these changes.
Thanks!


WebSocketController doesnt inject websocket events

I am new to the dart as well as angel websocket, I want to make a websocket demo for handling events, I followed your "from scrath tutorial " but didnt succeed.
Please help me to find out the issue in following code. The websocket event is not triggered.
Server file..
server.txt
lib/angel_hello file..
angel_hello.txt
test/websocket_test file..
websocket_test.txt
pubspec.yaml file...
pubspec.txt
Please give me a simple demo working code for angel websocket if possible

Apologies, hadn't seen this before. The Angel from Scratch tutorial is now quite outdated, as it is from only about two months after when Angel 1.0.0 was released. I will need to release an updated version sooner or later.
Many things have changed.
Can you put project on Github so that I can see what might be wrong?
In the meantime, I'll close the issue itself, as it's not a bug report.

Hi dear friend,
I have uploaded the project on github, please check it in your meantime. If
anything is wrong then please forgive me I am new to this dart and trying
to make the apps in flutter.
Below is the link to clone the repository...
git@github.com:sanketwarule/FeedServer.git
On Tue, Aug 7, 2018 at 1:10 AM, Tobe Osakwe<EMAIL_ADDRESS>wrote:

Apologies, hadn't seen this before. The Angel from Scratch tutorial is now
quite outdated, as it is from only about two months after when Angel 1.0.0
was released. I will need to release an updated version sooner or later.
Many things have changed.
Can you put project on Github so that I can see what might be wrong?
In the meantime, I'll close the issue itself, as it's not a bug report.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/angel-dart/websocket/issues/51#issuecomment-410828815,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AlrwdHUQMFZsQ3hgaYw1Ol7YkU8KrVI5ks5uOJukgaJpZM4VmPmB
.

--
Thanks,
Sanket Warule
+917709777170

One thing - change your app.configure calls to await app.configure. I'm trying to figure out why your setup wouldn't work - it all looks good to me.

Also, a note: Flutter doesn't support dart:mirrors, but Angel makes use of it. You can't run Angel within Flutter; you'll need the standalone Dart VM. That might be your issue?

As said I have changed to await app.configure....
but then also it gives me the same output..
below is my output
00:00 +0: boats
{"id":null,"eventName":"test","data":null,"params":null}
{"id":null,"eventName":"get_boat","data":null,"params":null}
On Tue, Aug 7, 2018 at 4:08 PM, Tobe Osakwe<EMAIL_ADDRESS>wrote:

Also, a note: Flutter doesn't support dart:mirrors, but Angel makes use
of it. You can't run Angel within Flutter; you'll need the standalone Dart
VM. That might be your issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/angel-dart/websocket/issues/51#issuecomment-411013588,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AlrwdG8X1WPbsCYLynTY0UMXQOaUcKXZks5uOW5BgaJpZM4VmPmB
.

--
Thanks,
Sanket Warule
+917709777170

Previously I have used socket.io but it is not supported for flutter, so I
am migrating towards angel framework which can be used in client side as
well as server side.
So in my flutter application will angel framework will work or not?
My motto of this project is to create the realtime server e.g. Stocks
Market using websockets.
A random numbers (stock price) will be generated and will be broadcasted
over the websockets to client (client is the mobile application built using
flutter).
I followed your tutorials, it is very worth-full and explained very nicely.
Thank you for your valuable time.
On Tue, Aug 7, 2018 at 4:58 PM, sanket warule<EMAIL_ADDRESS>wrote:

As said I have changed to await app.configure....
but then also it gives me the same output..
below is my output
00:00 +0: boats
{"id":null,"eventName":"test","data":null,"params":null}
{"id":null,"eventName":"get_boat","data":null,"params":null}
On Tue, Aug 7, 2018 at 4:08 PM, Tobe Osakwe<EMAIL_ADDRESS>wrote:

Also, a note: Flutter doesn't support dart:mirrors, but Angel makes use
of it. You can't run Angel within Flutter; you'll need the standalone Dart
VM. That might be your issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/angel-dart/websocket/issues/51#issuecomment-411013588,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AlrwdG8X1WPbsCYLynTY0UMXQOaUcKXZks5uOW5BgaJpZM4VmPmB
.

--
Thanks,
Sanket Warule
+917709777170

--
Thanks,
Sanket Warule
+917709777170

Well, the application will work, but here are notes on the correct structure:

The backend shouldn't be run on the user's device, instead on a server of your own, using the standalone Dart VM, which supports dart:mirrors.
The client side, which can be Flutter, CLI, Angular, etc., should use the client-side support: https://github.com/angel-dart/websocket/blob/master/lib/flutter.dart

In the meantime, I will have to re-run the tests in Travis to make sure everything is working correctly. I've just downloaded Dart 2.0.0, so I can finally reliably predict Travis outputs without having to install so many versions of Dart.

Thank you so much dear friend,
Just help me out in this feedserver application.
The output is not coming as expected even writting the await before the
app.configure...
Output:
00:00 +0: boats
{"id":null,"eventName":"test","data":null,"params":null}
{"id":null,"eventName":"get_boat","data":null,"params":null}
expected output:
it should print the
{'model': 'cx9'}
or
'test'.
On Tue, Aug 7, 2018 at 6:00 PM, Tobe Osakwe<EMAIL_ADDRESS>wrote:

Well, the application will work, but here are notes on the correct
structure:

The backend shouldn't be run on the user's device, instead on a
server of your own, using the standalone Dart VM, which supports
dart:mirrors.
The client side, which can be Flutter, CLI, Angular, etc., should
use the client-side support: https://github.com/angel-dart/
websocket/blob/master/lib/flutter.dart
https://github.com/angel-dart/websocket/blob/master/lib/flutter.dart

In the meantime, I will have to re-run the tests in Travis to make sure
everything is working correctly. I've just downloaded Dart 2.0.0, so I
can finally reliably predict Travis outputs without having to install so
many versions of Dart.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/angel-dart/websocket/issues/51#issuecomment-411039194,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AlrwdJkyYS4ofLlM8b-CCJHSr0-jI45vks5uOYhPgaJpZM4VmPmB
.

--
Thanks,
Sanket Warule
+917709777170

Going to clone your project now and get to the bottom of this. Sorry for any inconvenience!

Hi Tobe Sir,
Sending this mail just to remind you that you have not given me the
solution yet, please try to help me in your free time as soon as possible.
I am waiting for your response.
On Wed, Aug 8, 2018 at 10:50 AM, sanket warule<EMAIL_ADDRESS>wrote:

Thank you dear
On Tue, Aug 7, 2018 at 6:30 PM, Tobe Osakwe<EMAIL_ADDRESS>wrote:

Going to clone your project now and get to the bottom of this. Sorry for
any inconvenience!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/angel-dart/websocket/issues/51#issuecomment-411047430,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AlrwdBUufrxlUcRXdLYE38-pWkC39kt8ks5uOY9vgaJpZM4VmPmB
.

--
Thanks,
Sanket Warule
+917709777170

--
Thanks,
Sanket Warule
+917709777170


Does satellizer consider tablets to be mobile?

Right now, I'm specifying $authProvider.platform = 'browser' for tablets - should it actually be 'mobile'?
I'm using bowser for easy platform detection

That flag was introduced in https://github.com/sahat/satellizer/pull/294 and as far as I can tell it's only relevant if you are building an ionic or phonegap app. For mobile browsers like Chrome and Safari you should leave it as browser.

Ah. Thanks for the quick reply!
So even for mobile phones, it should be browser?

No problem.


Add support for waitForChildren performance API

it's not in the specs yet https://develop.sentry.dev/sdk/performance/
it's implemented by sentry-java and sentry-cocoa though, ideally, this would be added there after getting approval via a DACI

https://github.com/getsentry/sentry-dart/pull/643/files


RegexNotFoundError: Unable to extract uploader id

Checklist

[x] I'm asking a question
[ ] I've looked through the README and FAQ for similar questions
[ ] I've searched the bugtracker for similar questions including closed ones

Question

WRITE QUESTION HERE
How to deal with: Unable to extract uploader id; 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.
After 1st try, --verbose gave same message

Y'all, go /youtube_dl/extractor/common.py.
Line 983, change: fatal=True to fatal=False

#30839

Thanks!
Op zo 19 feb. 2023 om 12:15 schreef JohnTRL @.***>:

Y'all, go /youtube_dl/extractor/common.py.
Line 983, change: fatal=True to fatal=False
—
Reply to this email directly, view it on GitHub
https://github.com/ytdl-org/youtube-dl/issues/31598#issuecomment-1435960238,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A57Y3ZDBPULSNBO6R2Q6TVLWYH6EBANCNFSM6AAAAAAVA42GG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Met vriendelijke groet,
Hans van Zonneveld


delete bottle deprecate key

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

delete bottle "prefix" key because of the comment.

Thanks @hyuraku!


Grafana Dashboard

Issue Ref: [#166]
Description:
PR adds a sample Grafana dashbaord that be imported to visualize Prometheus metrics exposed by Kubeless.
TODOs:

[x] Ready to review
[ ] Automated Tests
[x] Docs

This PR is about adding a grafana dashboard. You can open an issue if there does not exists any tests for testing metrics.
Appreciate any comments on fit-and-finish or how the metrics are preseneted, if any other metric is relevent from kubeless perspective.

Yes, agree, better to have a separate PR to do the tests. What do you think about adding a how-to guide that explain how to use the dashboard? For people without experience with Grafana (like me).

I think this can be merged. We can create a separate issue to improve the docs.

@andresmgot importing dashboard json is really very basic perhaps does not require instructions. However i will add few line documentation on monitoring setup (installing prometheus, grafana and necessary scrape config)


genesis allocation seems to be lost after init

After the rebase, the genesis allocation seems to be missing from the state. This is probably due to the fact that the correct snapshot isn't found.

It goes deeper than that, the entire post-state is lost - except for the block reward.


Floats are truncated to 2 decimal points

The gridRednerer forces floats to 2 decimal points, and I couldn't find a way to configure this.
Is there an settings I am missing?
If not, would you accept a patch adding formatting/visualization options to a table, similar to the way a chart visualization is configures?

You're not missing anything, it's indeed impossible. And not only I'll
accept such a patch, it will be much appreciated.
I was thinking of a popup (modal) with configuration per column setting its
type, and relevant options for this type. You can see an example of opening
a modal here:
https://github.com/EverythingMe/redash/blob/master/rd_ui/app/scripts/controllers/query_view.js#L219
On Fri, Aug 21, 2015 at 6:20 AM, Shay Elkin<EMAIL_ADDRESS>wrote:

The gridRednerer forces floats to 2 decimal points
https://github.com/EverythingMe/redash/blob/master/rd_ui/app/scripts/visualizations/table.js#L71,
and I couldn't find a way to configure this.
Is there an settings I am missing?
If not, would you accept a patch adding formatting/visualization options
to a table, similar to the way a chart visualization is configures?
—
Reply to this email directly or view it on GitHub
https://github.com/EverythingMe/redash/issues/536.

Arik, that could be a really good way to add any customization, on format
and even links on the columns.
Regards
Matias
Matias Carazzo
Tech Lead
DevSpark http://www.devspark.com/
*Skype *matias.carazzo
On Fri, Aug 21, 2015 at 12:59 PM, Arik Fraimovich<EMAIL_ADDRESS>wrote:

You're not missing anything, it's indeed impossible. And not only I'll
accept such a patch, it will be much appreciated.
I was thinking of a popup (modal) with configuration per column setting its
type, and relevant options for this type. You can see an example of opening
a modal here:
https://github.com/EverythingMe/redash/blob/master/rd_ui/app/scripts/controllers/query_view.js#L219
On Fri, Aug 21, 2015 at 6:20 AM, Shay Elkin<EMAIL_ADDRESS>wrote:

The gridRednerer forces floats to 2 decimal points
<
https://github.com/EverythingMe/redash/blob/master/rd_ui/app/scripts/visualizations/table.js#L71
,
and I couldn't find a way to configure this.
Is there an settings I am missing?
If not, would you accept a patch adding formatting/visualization options
to a table, similar to the way a chart visualization is configures?
—
Reply to this email directly or view it on GitHub
https://github.com/EverythingMe/redash/issues/536.

—
Reply to this email directly or view it on GitHub
https://github.com/EverythingMe/redash/issues/536#issuecomment-133473154
.


[Bug] no valid_geo as a precondition in QuadKey class

from pyquadkey2.quadkey import QuadKey
QuadKey.from_geo((90,180),3) #?
Out[3]: 111

QuadKey.from_geo((91,180),3) #???
Out[4]: 111

QuadKey.from_geo((999,180),3)
Out[5]: 111

QuadKey.from_geo((999,999),3) #??????
Out[6]: 111

from pyquadkey2 import quadkey
quadkey.from_geo((90,180),3) # this might be expected, but the given exception without any explaination is way too wild

Traceback (most recent call last):
  File "/Users/ruic/Projects/venv_3.10/lib/python3.10/site-packages/IPython/core/interactiveshell.py", line 3550, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-8-f73f676a55f9>", line 1, in <module>
    quadkey.from_geo((90,180),3)
  File "/Users/ruic/Projects/venv_3.10/lib/python3.10/site-packages/pyquadkey2/quadkey/util.py", line 9, in wrapper
    assert precondition(*args, **kwargs)
AssertionError

I have tested it on what the tile, it allows (90,180) as a input.
I personally think the geo limit is okay. The geo does exists, it just does not belong to any tiles anymore.(So maybe a specific tile unknown?) Or just forbid any one to use that as a input. Anyway it should be consistent. Any maybe we can have a user-friendly warning or exception as a reminder instead of Assert.
And one more thing, there is no way to tell whether your geo is (lat,lon) or (lon,lat), unless you read the code, which is also not user-friendly, might consider fix it while you fix the above.

To Do:

[ ] Move validity checks into class methods (and get rid of decorator)
[ ] Improve error messages


Create rule node-param-name-untrimmed

Example: https://github.com/n8n-io/n8n/pull/4259

Closed by https://github.com/ivov/eslint-plugin-n8n-nodes-base/pull/134


mgmt, upgrade authorization

manual change:
https://github.com/Azure/azure-sdk-for-java/pull/32266/commits/3c85c26d1ae98552d769d14e5281eb2445231534
https://github.com/Azure/azure-sdk-for-java/pull/32266/commits/e05220a1fa587da6c2f879249944c2e130658183 due to change on API spec of "encoded = true" in list of RoleAssignmentClient
Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


"Use guzzlehttp/guzzle instead"

Hi
I downloaded the repo and copied it to xampp/htdocs.
And executed "composer install".
It returns "Package guzzle/guzzle is abandoned, you should avoid using it. Use guzzlehttp/guzzle instead.".
Any idea, @fillup @vanhoavn @simoheinonen ?

See https://github.com/fillup/walmart-partner-api-sdk-php/issues/7#issuecomment-244391968
After https://github.com/guzzle/guzzle-services/pull/109 has been merged it would be possible to get rid of that message

Please let me know what to do on my end. Can you explain more details about the solution?

OK, it works, thanks.


Why Dubbo and RPC?

[x] I have searched the issues of this repository and believe that this is not a duplicate.
[x] I have checked the FAQ of this repository and believe that this is not a duplicate.

I searched in FAQ and docs and I didn't find an answer.
Why and when use Dubbo?
Given that REST APIs are increasingly common, why use RPC instead of web services?
What's the advantages to use RPC instead REST web services?
What's the point that I'm don't catching?
I talk with people that have the same doubt. I think that this point could be answered in FAQ.
Thanks.

You can have a look the official website.
https://dubbo.apache.org


Navigation Options: Resources

Description
Updates the top-level navigation options. For now, just appending to the long list as it's the most mobile-view compatible option. Will then open a new issue for the re-style which will likely result in something like this.

Related Issue(s)
Closes #319
Checklist

[x] I have read the contribution guidelines
[x] I have considered the performance impact of these changes
[-] Suitable unit/system level tests have been added and they pass
[-] Documentation has been updated

Accidentally created this from a branch on my own website repo, switching this PR to merge into a new branch on /flowforge, and then will do a new PR to merge to main