Letsencrypt / Trafik issues.

The outstanding issue with letscenrypt/trafik creates a problem with cyphernode that is not easily solvable without a decent level of knowledge of DNS and how these systems work and lead to two questions.

Can we fix it?

Can it be fixed, if so is it an easy fix, a medium one or a hard fix?  This, of course, is a subjective question with a lot of factors such as what other work does the cyphernodes devs have to do (let's assume a lot), Does it require major under the hood changes (is the juice worth the squeeze)

Should we fix it?

This is the inverse of the other question and is a perfectly valid one to ask.  As this is not a showstopper in all but a few use cases for cyphernode and the fixes as well documented for this issues, regardless if it is a complicated fix or not.
An Assumption:
For the remainder of this issue let us assume we should fix it for no other reason it would help my particular use of cyphernode no end, however, if we chose not to fix it that is fine as well (based on the answer to question 2)  as long as we justify that decision.
The issues created by this issue
There are several issues created by these issues that I will describe now, please note there may be more issues that I am unaware that (hopefully) will be fixed by the fix.
I will summarise the issues below with their proposed fixes (this can change as I am most likely wrong) and then go into each of these issues in more detail.
Use cyphernode over a domain
Easy fix by setting up the domain in letsencrypt that we can provide documentation for
Use cyphernode over TOR to access dashboard
I think, same as above?
Accessing cyphernode via code to extend it functionality
Upgrade Trafik on cyphernode (most likely other fixes exist)
A detailed breakdown of the issues:
Use cyphernode over a domain
I have set up a cyphernode testnet at the following URL
https://<IP_ADDRESS>/welcome

and this shows the SSL issue.
Use cyphernode over TOR to access dashboard
I have set up a Raspberry PI with a test cyphernode to test TOR and we get the same issues as above when accessing the dashboard.   You can view it at the following URL:
https://t66jfcrnerds2uydne3ggaw2fdgny2l25acscc5zywhclgvudn7xndad.onion:443/welcome
Accessing cyphernode via code to extend it functionality
This is a complicated issue so let me preface this before I get into to.  I am not talking about if you build apps inside the docker images and are accessible on the same box as these will work as they are inside the firewall (for want of a better term) I am speaking about when you try to access it outside of this, for example, an e-commerce cart that wants to retrieve a BTC address to give to a customer for payment.
accessing the RPC over standard SSL
Using bitcoin-core, If we look at the example below we will get an SSL error
const Client = require("bitcoin-core");
client = new Client({
  host: "host",
  port: 18332,
  wallet: "wallet",
  username: "username",
  password: "password",

});
client.getNewAddress().then(address => {});

accessing the RPC over standard TOR
Accessing via TOR
If you want to access it via TOR if you use the following code
process.env.CYPHERNODE_API_KEY="xxxxxxx"
//is this required when connecting over TOR?
//process.env.CYPHERNODE_HTTP_TRANSPORT_PROXY="socks://<IP_ADDRESS>:9050"
process.env.CYPHER_GATEWAY_URL="t66jfcrnerds2uydne3ggaw2fdgny2l25acscc5zywhclgvudn7xndad.onion"
const {btcClient} = require('cyphernode-js-sdk');
const client = btcClient();
client.getBalance().then((result) => {
	console.log(result);
}).catch((err) => {
	console.log(err)
})

The following error is presented

Now, I could be doing something wrong with my TOR code here (as I am new to TOR in general) in which case this one could be ab easy fix, and in that case, yay.
A final note both of these code issues can be fixed in the following manner, but neither is optimal in my opinion

Set NODE_TLS_REJECT_UNAUTHORIZED to 0 which basically ignores SSL, which is fine for dev purposes but madness in a production environment.

The code is running on the same instance as cyperhnode, ideally, we would like the abstract the cyphernode instance away from any public-facing boxes.

Hey there @cryptoskillz !  Thanks for the writeup.  A lot of stuff to address!  :)  I will take this opportunity to summarize how the Cyphernode network works or is intended to work.  I know you already know most of what I'm going to write, but I think it's important to have that written somewhere for others.  Maybe it will eventually be part of a doc file.
Cyphernode client applications and Cyphernode networks
An application that is a Cyphernode client can be run wherever you want.  The most secure and simple way is to run it "inside" the Cyphernode ecosystem.  If you look at the Cyphernode architectural schema, you can see the two Cyphernode networks:

cyphernodenet (left side, "cypherzone") where the core pieces of Cyphernode run
cyphernodeappsnet (right side, "warpzone") where the Cyphernode clients can run

If your client application runs outside of the Cyphernode docker network, you will have to expose/publish the gatekeeper port outside the docker network (default port 2009, default to not being exposed, it's one of the questions asked during setup).  The gatekeeper is the door to your bitcoins.
Accessing the Cyphernode API, aka the gatekeeper (cyphernodenet)
The Cyphernode API (gatekeeper) should not be published to the Internet: it is not meant to be exposed to potential attacks and be a public API.  The layering approach we took in our architecture makes it easier to concentrate on specific tasks on each layer.  For example, a reverse proxy with access rules should be used in front of the gatekeeper.  Maybe also a monitoring system, a DDoS detection system, some kind of fail2ban app, etc.  Cyphernode is meant to be a personal/family/departmental/company (ie intranet) gateway to Bitcoin functionalities.
If you need to access the Cyphernode API from outside the docker network and/or local network, you should take good care of having a VPN, iptables rules, or any secure networking infrastructure around it so you are the only one accessing it.
Since the gatekeeper is not meant to be exposed to the Internet, obviously it is not possible "out of the box" to use Let's Encrypt for the gatekeeper certificates.  The gatekeeper certificates are generated during setup and they are self-signed.  Of course, you could make the gatekeeper listen to ports 80 and 443 and have forward rules so that Let's Encrypt can take care of its certificates, but then there will be one door between the Internet jungle and your bitcoins.  We don't recommend this.
You should take the generated TLS certificates (found in one of the downloadable AES-256 encrypted 7z files on the Cyphernode Welcome app page) and add them to your client applications so you don't have to set NODE_TLS_REJECT_UNAUTHORIZED to false or anything similar that weakens security.
Accessing the CypherApps Zone (cyphernodeappsnet)
This zone is where the applications that are Cyphernode clients run.  They most likely are web applications, but it's not required.  Those applications have "local" access to the gatekeeper API.  They are "authorized" applications, they own API IDs and keys for the API.
Traefik
We chose traefik (https://docs.traefik.io/) to expose the CypherApps to the Internet.  Traefik can route the requests coming from the Internet to the right web application running in the cypherapps network.  It takes care of the creation and renewal of the Let's Encrypt certificates.  It supports multiple domain names and complex and flexible routing rules.  It is open source, well known and well tested.
The cypherapps you want accessible from the Internet (like the Cyphernode Welcome app) will tell traefik how to route the requests to them with special docker labeling in the app's docker-compose file.  It's very cool to be able to access your web apps with your own domain name and valid TLS certificates.
Traefik is there to access the cypherapps, not the gatekeeper API directly.  Once again, you could bypass that by manually configuring the gatekeeper to be routed by Traefik.  But it's just a security risk to expose your API.
Having layers of indirection with specific rules is always a good idea to protect your sensitive stuff.  The cypherapps are one of those layers.  They should have their own authentication, authorization, security implementations and application logging systems.  The user is your cypherapp client and your cypherapp is the gatekeeper API client.  Two separate things with different auth.
Tor
Another way to access your web applications is through Tor.  Using Tor is a convenient way to avoid port forwarding configuration on your router.  It is also fully encrypted by default.  In Cyphernode, we generate v3 onion addresses that make them also one more layer of secret, as long as you don't publish them.
That is why Cyphernode is generating different onion addresses for the Bitcoin node, the Lightning Network node and Traefik.  You can publish your LN onion address without publishing your Traefik web entrypoint address.
Since Tor traffic is encrypted by default, you will notice that most onion sites are HTTP only, not HTTPS.  For example, if you ask Let's Encrypt to generate TLS certificates for an onion address, it will fail.
Tor and Traefik communication
To take advantage of the Traefik automatic routing functionality, we decided to make Tor use Traefik to avoid the need to duplicate the routing rules of Traefik in Tor.  This means Tor will proxy the requests to Traefik when the Traefik Onion address is used.
Here is where it gets problematic: if we want Tor to serve an HTTP page (instead of HTTPS), we need to activate HTTP on Traefik.  If we activate HTTP on Traefik, it will be possible to access it without encryption and potentially leak your passwords.  We don't want that.
We tried to write Traefik rules so that it would accept HTTP requests ONLY if they are relayed by the Tor node.  Unfortunately, we failed at it.  We don't know yet how to do it.  We think upgrading Traefik to v2 would allow us to do so though.  That's one of our tasks on the huge todo list and it's difficult to estimate when it will be done.
Your Tor URL to the Welcome app would be http://t66jfcrnerds2uydne3ggaw2fdgny2l25acscc5zywhclgvudn7xndad.onion/welcome and you wouldn't have any invalid certs error messages in your browser.
One workaround would be to stop publishing your Traefik 80/443 ports outside docker and allow it to serve HTTP requests.  Then, the only way to access your apps would be through Tor.  You couldn't use Let's Encrypt with a clearnet domain name.
Let's Encrypt
I already opened an issue (https://github.com/SatoshiPortal/cyphernode/issues/127) where you also made a comment :) but here I will add more details about how to activate the Let's Encrypt functionality in Traefik.

You need a domain name.  Let's use cryptoskillz.com here.
You need to add the public IP address used to access Traefik to your DNS records.
You need to open and forward your router's 80 and 443 ports to your Cyphernode/Traefik server, if necessary.

If you built the Cyphernode images from sources:

Change cyphernodeconf_docker/templates/traefik/traefik.toml with the changes below
./build.sh
From dist/ run ./setup.sh and start Cyphernode

If you are using prebuilt images:

You will unfortunately have to reapply the changes AFTER each ./setup.sh execution.  You have to edit the file traefik.toml in the directory where you traefik files are stored.  The default is cyphernode/traefik/traefik.toml from where setup.sh resides, but it can be changed during setup.

In both cases, here are the changes to do to the file:
Change yourdomain.com to cryptoskillz.com in the last [[acme.domains]] section and also change the email field for the e-mail you want Let's Encrypt to use to contact you.
Next time you start Cyphernode (./start.sh), Traefik will read that file and do its stuff to create/renew/take care of the Let's Encrypt certs.  You should be able to access the Welcome app using https://cyphernode.cryptoskillz.com/welcome.

What a wall of text, sorry about that!  Let me know if it helps... or not.  :)   Thank you for the issue!


Update how-to-configure-api-proxy-module.md

Add section to expose port when a port other than 8000 is used

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @ancaantochi,
I copied your commit to the private repo, added my changes, and merged. https://github.com/MicrosoftDocs/azure-docs-pr/pull/244172. I'll close this PR since it's not needed.
Thanks,
Pat
#please-close


ci: Add "build" label type to accepted PR titles

Overview
This PR adds the "build" label type to the accepted "Conventional Types" labels that are accepted.
This is specifically for build-related things (e.g., Makefiles). The existing labels did not really address certain types of possible changes, in my opinion.

I'm good w/ this :)


Cursor is not showing when creating a new file in VsCode

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:
VSCode Version: 1.75.1
https://user-images.githubusercontent.com/61721914/219273641-221b87ef-64ff-4131-815c-eb1e85eb6163.mp4
OS Version:  MacOs Ventura

Select One file
Try to create a new file
Error will occur when the box is bordered with white color
As a result there is no cursor and I can't type the file name

Can you click inside the editor to type first to make sure VS Code has focus? Does that then fix this?

Can you click inside the editor to type first to make sure VS Code has focus? Does that then fix this?

No, when I click the editor to type it will cancel me to make a new file

I mean click in the editor first to ensure it has focus then click the new file button and try typing

It can't

Hey @lramos15, this issue might need further attention.
@vincenthartanto1999, you can help us out by closing this issue if the problem no longer exists, or adding more information.

You cannot type in the open code document first and then make a new file?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!

I've had this issue for over a year. Obviously it's fine if you are already focused or left-click the window. But if we right-click and click New File... the window isn't focused and we can't type, and more annoyingly when we focus the window the new file is cancelled because it has no name. Can the window be focused at the same time this function is called, or does the OS not allow this?


Explicit black text colour for line readability in reports

Hi,
Just a quick fix for a common omission: the HTML report generated during compilation styles the line background colour, but does not do so for the text on top of it.  I guess most people have their browser set to black text on white background so don't notice, but mine is white on black—the report ends up somewhat difficult to read with white text on a white/yellow background!
Hopefully this PR is all that's needed? Let me know if there are any problems.

We have exact colours for everything else, so this makes sense. Thanks.

Thanks! (for the accept, and for cython :-))


Upgrade config and profile for Drupal 10

There's some outdated configuration

fixed https://github.com/skilld-labs/druxxy/commit/5f5a1a1e3e9887228c6f8fdf67789a7fea6f2b51
one more 02d80be

  [RuntimeException]
  Adding non-existent permissions to a role is not allowed. The incorrect per
  missions are "access in-place editing", "administer footer menu links", "ad
  minister main menu links", "assign contributor role", "create landing_page
  content", "create local_video media", "delete any landing_page content", "d
  elete any local_video media", "delete landing_page revisions", "delete own
  landing_page content", "delete own local_video media", "edit any landing_pa
  ge content", "edit any local_video media", "edit own landing_page content",
   "edit own local_video media", "revert landing_page revisions", "translate
  landing_page node", "translate local_video media", "translate paragraph", "
  translate shortcut".

I think we should create 10.x branch with https://github.com/skilld-labs/druxxy/pull/29

And migrations with locale import command are broken https://github.com/skilld-labs/skilld-docker-container/pull/445

Upgrade SDC packages also has nuance https://github.com/skilld-labs/skilld-docker-container/pull/444

I went ahead a published https://github.com/skilld-labs/druxxy/releases/tag/1.0.0


E   ValueError: low is out of bounds for int32

mtrand.pyx:973: ValueError
_____________ TestFeatherReader.test_delete_partial_file_on_error _____________

self = <pyarrow.tests.test_feather.TestFeatherReader testMethod=test_delete_partial_file_on_error>

    def test_delete_partial_file_on_error(self):
        # strings will fail
        df = pd.DataFrame(
            {
                'numbers': range(5),
                'strings': [b'foo', None, u'bar', 'qux', np.nan]},
            columns=['numbers', 'strings'])

        path = random_path()
        try:
            write_feather(df, path)
        except:
            pass

>       assert not os.path.exists(path)
E       AssertionError: assert not True
E        +  where True = <function exists at 0x0000020693214400>('feather_90d59fbf6104454bbb9f777ca849b132')
E        +    where <function exists at 0x0000020693214400> = <module 'ntpath' from 'C:\\Users\\wesm\\Miniconda\\envs\\arrow-test\\lib\\ntpath.py'>.exists
E        +      where <module 'ntpath' from 'C:\\Users\\wesm\\Miniconda\\envs\\arrow-test\\lib\\ntpath.py'> = os.path

pyarrow\tests\test_feather.py:267: AssertionError
========= 4 failed, 163 passed, 37 skipped, 2 xfailed in 2.30 seconds =========

It seems that the 128-bit string-to-integer function used in decimal parsing maybe doesn't work on windows
void StringToInteger(
    const std::string& whole, const std::string& fractional, int8_t sign, int128_t* out) {
  DCHECK(sign == -1 || sign == 1);
  DCHECK_NE(out, nullptr);
  DCHECK(!whole.empty() || !fractional.empty());
  *out = int128_t(whole + fractional) * sign;
}

I'm looking into the two other test failures

here's print statements from inside StringToInteger
pyarrow/tests/test_convert_builtin.py::TestConvertSequence::test_decimal_large_integer whole:<PHONE_NUMBER>37548693 fractional: 42983 out: -39402950693754869342983
whole:<PHONE_NUMBER>2932 fractional: 01033 out:<PHONE_NUMBER>293201033
FAILED

digging down into the conversion back to pydecimal

Here's telemetry from inside decimal::ToString
(Pdb) data[0]
Decimal('-394029506937548693.42983')
(Pdb) arr[0]
0000000000000000000000000
0000000000000000000000005
0000000000000000000000015
0000000000000000000000015
0000000000000000000009015
0000000000000000000.39015
0000000000000000005.39015
0000000000000000005.39015
0000000000000000905.39015
0000000000000003905.39015
0000000000000003905.39015
0000000000000703905.39015
0000000000000703905.39015
0000000000040703905.39015
0000000000440703905.39015
0000000009440703905.39015
0000000049440703905.39015
0000000649440703905.39015
0000001649440703905.39015
0000071649440703905.39015
0000571649440703905.39015
0008571649440703905.39015
0038571649440703905.39015
-038571649440703905.39015
Decimal('-38571649440703905.39015')

Alright, all tests are passing now except for the Decimal128 issue

Looks like there might be more issues than just the decimal thing
(C:\arrow-conda-env) C:\projects\arrow\python>py.test pyarrow -v   || exit /B
============================= test session starts =============================
platform win32 -- Python 3.5.3, pytest-3.0.7, py-1.4.33, pluggy-0.4.0 -- C:\arrow-conda-env\python.exe
cachedir: .cache
rootdir: C:\projects\arrow\python, inifile:
collecting ... collected 206 items
pyarrow/tests/test_array.py::test_total_bytes_allocated PASSED
pyarrow/tests/test_array.py::test_repr_on_pre_init_array PASSED
pyarrow/tests/test_array.py::test_getitem_NA PASSED
pyarrow/tests/test_array.py::test_list_format PASSED
pyarrow/tests/test_array.py::test_string_format PASSED
pyarrow/tests/test_array.py::test_long_array_format PASSED
pyarrow/tests/test_array.py::test_to_pandas_zero_copy
(C:\arrow-conda-env) C:\projects\arrow\python>set lastexitcode=-1073741819
(C:\arrow-conda-env) C:\projects\arrow\python>set  1>C:\Users\appveyor\AppData\Local\Temp\1\tmp6968.tmp
(C:\arrow-conda-env) C:\projects\arrow\python>echo C:\projects\arrow\python  1>C:\Users\appveyor\AppData\Local\Temp\1\tmp6969.tmp
(C:\arrow-conda-env) C:\projects\arrow\python>exit /b -1073741819
Command exited with code -1073741819

Will run test suite under valgrind on Linux to see if there's a bad memory access that we've been ignoring

Valgrind output on master branch. Seems like some of the typical spurious Python valgrind warnings, don't see anything overtly worrisome:
https://gist.github.com/wesm/fbbd9ccf19267a6bbd6872a6e5dc31dc

So it seems there's a segfault the first time using the NumPy C API is attempted. Not sure why it works locally and fails in Appveyor.

I'm completely stumped about this Appveyor issue with the NumPy C API. Cannot reproduce locally on Windows 10. If we can fix the Decimal unit tests (locally), then I suggest we merge this without running the unit tests in Appveyor until we can figure out a way to debug

@wesm , wrong shared library .lib file looks like might be really the reason of NumPy C API issue, but I have tried to run your changes locally and rebased against latest commits, in both cases got python test session errors, like following:
________________ ERROR collecting pyarrow/tests/test_array.py _________________
ImportError while importing test module 'C:\Work\TwoSigma\wesm-arrow-src\python\pyarrow\tests\test_array.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
C:\Users\Max\Miniconda3\lib\site-packages\pandas\__init__.py:25: in <module>
    from pandas import hashtable, tslib, lib
E   ImportError: DLL load failed: The specified procedure could not be found.

During handling of the above exception, another exception occurred:
pyarrow\tests\test_array.py:22: in <module>
    import pandas as pd
C:\Users\Max\Miniconda3\lib\site-packages\pandas\__init__.py:31: in <module>
    "the C extensions first.".format(module))

@MaxRis can you create a clean conda environment and follow the build instructions in
https://github.com/apache/arrow/blob/master/ci/msvc-build.bat
The pandas import error looks unrelated

@MaxRis indeed the static lib issue was the cause of the problems I was having. Now we just have test failures remaining

Is line needed to import numpy? https://github.com/apache/arrow/pull/575/files#diff-6397ece82ce8eb339620f294eecdb045L42

Yes, that line is required to import numpy

I disable the py.test call in ci/msvc-build.bat. Let's merge this when the build is green, then when we fix the decimal tests we can turn back on the unit tests

+1


Add a new file-explorer

Description of the new feature / enhancement
A file-explorer more like win 10 with a ribbon layout.
It would be nice if it was updated developed further than win 10 like adding a OneDrive tab with features like sharing selected folders/file, open in web/desktop app.
Would also like to see the old dropdown list for the view stats came back.
Add on option to choose tab layout or not (like win 11 file-explorer)
Scenario when this would be used?
Insteed of win 11 bad file-explorer.
In a ribbon layout every function is easily find. Like: open (with dropdown for open with other program)/edit, Copy/past/cut/copy file path/past file path, Select all/Invert selection/select none, group after: Type/date/…, Sort by.

Supporting information
No response

Windows 11 23H2 will have some nice updates for Explorer. Also, there are many other apps like Explorer already.

/feedbackhub


Buggy kerning for the letters "fil"

Example:

In this gif, I first try to backspace the i, then retype it and try to backspace both the i and the f.  I move the cursor to where it looks like it's behind the f and it still deletes the f.  It takes one more arrow-key to the left for my backspace to delete the new-line behind the f.

VSCode Version: 1.14.2
OS Version: 10.11.5

Steps to Reproduce:

type fil and try to delete characters from it.

Reproduces without extensions: Yes

Also demonstrable with just fl.

This is an upstream issue:
Here are the methods we invoke on Chromium - https://jsfiddle.net/x0r2ega6/
offset 0: 8
offset 1: 8
offset 2: 18

It would be expected that the client rect for character at offset 0 would be different than the client rect for character at offset 1.
Please feel free to report upstream at https://bugs.chromium.org/p/chromium/issues/list and point to the issue here.

This is an upstream issue (possibly within Chromium), and there are no known workarounds that we could implement on our side. This means there is no possible code change for us to do in VS Code sources to improve things.
To keep the number of issues in our inbox at a manageable level, we’re closing issues that have been on the backlog for a long time but haven’t gained traction or that are not actionable. Thank you for your time and understanding.


Make SchedulingInfo builder method naming consistent

Context
Current naming on SchedulingInfo Builders is a mixed of (SingleWorkerStage vs add.*Stage).
Expected behavior
Use consistent naming convention 'add.*Stage'.

Hi can I pick up this task?

I've raised a PR please let me know if it looks fine


Live Server disappeared and can't get it to show

I cannot get Live Server to function at all and not sure what's wrong

[x] Other:

My Issue
I used Live Server before for a long time, but when I now use it after a few months, it doesn't show in anywhere, no status bar icon, not in right click. I tried uninstall and install a few times.  After installed, is it supposed to show up in Microsoft VS Code/resources/app/extensions (it isn't)? Can't open http://localhost:5500/
Environment
Browser:

- [x] Chrome (desktop) version 97

For Tooling issues:

- Live Server: 5.7.4
- Platform:  Windows 10 Home x64
- Visual Studio Code: 1.63.2

Others
Below are some screenshots

showing installed and enabled
html file opened but no Go Live icon
right click on file doesn't show Live Server
settings/config
resources/app/extensions

Hi, I tried installing an older version (5.6) and it is working. It seems to a problem with v5.7

This may be related to issue #2334  ??? Does the button appear when you open an HTML file?

@MattBrooks95  Hi, nope it's not related to #2334
@deanstreetlab Hi, something is wrong. Btw thanks for the debugging info. It supposes to show here %USERPROFILE%\.vscode\extensions.  Can you please install v5.7 and check if it's there or not. And also, which project you're working on? It seems it's a GitHub repo. Can able to share that so that I can try it in my local?


update HGDP pop labels and groups for v4

I made it to all 54 pops in 7 groups with the latest names. One small question: from L8-L44, the group names are sometimes in quote and sometimes not, does it matter?

@jkgoodrich Could you have a look at this soon?

@mattsolo1 It's all good to go from our end, could you merge it? I don't know if it would affect your current setup of the browser if I merge it.

Yes we can merge when we need to.. thank you Qin!


Deserialized DB is Readonly(EdgeMeta)

Hi Ben. In my tests I use data from OSM. To speed up loading I tried to use serialized version of RouterDb.
But the problem is that I should modify data. Unfortunately when I try to modify RouterDb it throws exception that EdgeMeta is in ReadOnly state. So the question is how to deseralize RouterDb keeping possibility to change data?

No, that's not possible at the moment. What you could do is create a new AttributeIndex and copy things over.
I'm not sure anymore why the attribute index becomes readonly, probably just because the usecase of adding attributes after the routerdb was serialized never came up... I would need to investigate to make sure but perhaps it's possible to change it after deserialization but I'm not sure.
You can always look at the code here:
https://github.com/itinero/routing/blob/develop/src/Itinero/Attributes/AttributesIndex.cs

Thx Ben. I have one more question: what is branch? I saw property Bransches in route but I can't understand what is it for?

To be able to generate turn-by-turn instructions a route also needs 'branches'. These are the streets that are not used at intersections for example. An example, blue is the route, red the branches:
http://geojson.io/#id=gist:anonymous/e41979ef18d6d2643ebed96d941eca5f&map=17/51.26860/4.79938

Did you find another solution?

No. Unfortunately I was unable to create writable copy of RouterDb. I got stuck on AttributesIndex, It occured that it's impossible to clone right because all Reminiscence collections are readonly. And IndexMode is not saved during serialization-deserialization.

What's your usecase maybe I can help. Perhaps there is another way...

Well I just wanted my test data from OSM to load a little faster. But now I realized that my code is taking much more time than OSM loading. So I postponed OSM loading speedup as it's not critical.
Regardinbg usecase. I have to add some additional data which is not contained in OSM. And I test it in conjunction with OSM data.

There is an option to add a 'filter' to the source stream, you can change the tags of the OSM-ways when they are loading.

Closing issue, AttributesIndex will remain readonly for now, the idea is to add all the needed data at startup.
This could be a future usecase of this: https://github.com/itinero/routing/issues/103


[sil-opt] Fix covering all-case of trapping instruction of mayTrap in sil-opt

Hi, I found that the function which checking trapping SIL instruction may have an issue, so I want to fix by this PR.
Abstract
Making SILInstruction:: mayTrap  covering all cases of trapping instruction.
Issue
Having no issue-report
Description
It seems that SILInstruction::mayTrap() is checking that the instruction may trap or not.
And, according to the sil.rst, there are 4 instruction may trap (runtime-failure).

cond_fail
unconditional_checked_cast
unconditional_checked_cast_addr
unconditional_checked_cast_value

ref: https://github.com/apple/swift/blob/master/docs/SIL.rst#cond_fail and https://github.com/apple/swift/blob/master/docs/SIL.rst#checked-conversions
However, unconditional_checked_cast_value is missed on mayTrap  case in current implementation so this commit added it to cover all-case.

I found that commit message has issue so let me fix it

Moved to https://github.com/apple/swift/pull/33851


Replace gslcblas with openblas

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Tests pass now

ping @conda-forge/gsl

Thanks.

Thanks for the PR!


Remote to remote string

There should be ways to convert Remote[A] to Remote[String]. This is not only useful for logging remote values for example, but could also be important once we start doing real things with activities.
I think we should have:

a generic conversion to string (like .toString)
more specific ones for some types like numbers and dates with formatting options

Please assign to me.


Option to include parameters that aren't in the docstring

Is your feature request related to a problem? Please describe.
Where I work, we try to avoid redundant docstrings, i.e. parameter names are as self-explanatory as possible, and they are type-annotated. However, those parameters then don't appear in mkdocstrings' rendered table/list/spacy of parameters.
Describe the solution you'd like
An option to show all parameters of a function, even if some parameters aren't explicitly mentioned in the docstring. Likewise for the return type, if it is specified in the code.

I'll close this as out-of-scope. Docstring standards are already quite hard to support correctly, plus the code isn't designed to fetch parameters from signatures when rendering docstring sections. There would be several edge-cases, like, should we render a section with all params if none of them are documented?, etc.. Finally we already have lots of options and I would like to avoid adding too many :slightly_smiling_face:


Improve UI of navbar items

With the google translate item visible, the navbar items seem bit crowded. This can be solved by font size reduction or updating navbar design.
Please assign this to me under GSSoC'23.

hey its already good so no improvement required

Actually i am planning to remove google translate so it will look more cleaner

Yeah, that seems better.


Sling Logger configurations should align to common env-specific use-cases

Expected Behaviour
It would be nice if the custom Sling Logger generated by the Archetype would set up to align to common log-use patterns for the supported environments (dev, stage, prod)

/apps/..../config -> (Default) For LocalDev/Dev, set logger to DEBUG level to the error.log
/apps/../config.author|publish.stage -> For Stage, set logger to WARN level to the error.log
/apps/.../config.author|publish.prod -> For Prod, set logger to ERROR level (or leave undefined) to the error.log

Actual Behaviour
The Sling Logger configuration provided by the Archetype routes log messages for the bundles root package to a custom log file (project-xxxx.log) at the INFO level.
Platform and Version
The proposal to route all the messages to error.log is to allow the log messages to be easily obtained via Cloud Manager which only exposes withe error.log as the aemerror log.
If Cloud Manager supports exposing custom log files in the future, then this could be revisited, however it seems prudent to align w/ the current state/near-future-state.
Feedback
I'd like to hear feedback on the default Logger level configurations and granularity to ensure they jive w/ others general needs.
EDIT: Per Rubens Slack feedback for AMS support; make default log level as DEBUG for local quickstart dev, and specific stage and prod runmodes explicitly.

fixed in 90735aa900ab62ab7c5d61a988d2d90cf8d742f1
and 6cff98d6286b579d0eb323494c59d038c274284f


Investigate OpenMP flags correctness

Upon further testing, openMP wasn't actually getting enabled when compiling the project. Even though the arguments in OpenMP_C_FLAGS are correct (-Xclang -fopenmp), CMake tries to encapsulate both arguments in a single set of quotes which makes it invalid. Using the alternative syntax for passing normal clang arguments to clang-cl works though (/clang:-fopenmp). This seems to function correctly in my testing:
	if (MSVC)
            if (CMAKE_C_COMPILER_ID STREQUAL "Clang")
                 set(SAIL_OPENMP_FLAGS "/clang:-fopenmp" CACHE INTERNAL "")
            else()
                set(SAIL_OPENMP_FLAGS "/openmp:llvm" CACHE INTERNAL "")
            endif()
        else()
            set(SAIL_OPENMP_FLAGS ${OpenMP_C_FLAGS} CACHE INTERNAL "")
        endif()

Originally posted by @Error-mdl in https://github.com/HappySeaFox/sail/issues/203#issuecomment-1856526290

CMakeCache.txt
Seems to be an error in Visual Studio's cmake support. Building from directly from CMake works, and there's no quotes around the arguments in the CMake cache. The error only happens when compiling the generated solution file in VS. Building from VS throws this error:
clang-cl : warning : unknown argument ignored in clang-cl: '-Xclang -fopenmp' [-Wunknown-argument]
VS seems to incorrectly see -Xclang -fopenmp as a single argument and puts both in a single set of quotes which causes the issue (shows up in the Additional Options as %(AdditionalOptions) "-Xclang -fopenmp"). Removing the quotes fixes the error.

I see. What's your CMake version? Maybe it's a bug in the CMake VS generator.


Create a doc for themes

Add a doc for themes
https://user-images.githubusercontent.com/46993493/126548429-bdc21252-33a6-44f3-af66-5ed7763870c4.mov
Motivation
We consider our theme as the decision we make besides all our options, aka tokens.
Changes

create doc to theme
remove theme on components documentation

@all-contributors please add @mmartinsoliveira for doc


[BUG] Export org to latex and opening them doesn't work

Describe the bug
I know this probably shouldn't be asked here but
~app-launcher did not find many of my applications on the NixOs system.~ . Only zathura was not found
What annoys me is when I want to export org to latex and open it. They don't work on my system, Compile to latex is quite slow and doesn't open the .pdf file after compile.
Checkup list

[x] You've tested disabling your config.el using MINEMACS_IGNORE_CONFIG_EL=1 emacs?

To Reproduce
Steps to reproduce the behavior:

Open simple org file
export to latex and open with C-c C-e l o
It takes about 15-20 seconds to export a file with only 10 lines
No applications opened with the pdf file

I just changed this in config.el
(with-eval-after-load 'ox-latex
  (setq org-latex-compiler "lualatex"))
  ;; (setq org-file-apps
  ;;       '(("\\.pdf::\\([0-9]+\\)\\'" . "zathura -P %1 %s"))))
;; (setq org-preview-latex-default-process 'dvisvgm)

Additional context
Can you try exporting the org to latex and opening it in your machine?
Does the problem lie with NixOs

Exporting Org to PDFs via LaTeX backend has always been slow for me!
The export and open is not working in my config also, I will investigate this later.
Thanks for the feedback @hmanhng !

C-c C-e l o slowness is probably due to running org-latex-export-to-pdf with async.  I tried running this command in Mx which gave it significantly faster speed.
Also I tried
(with-eval-after-load 'org
  (push '("\\.pdf\\'" . "zathura %s") org-file-apps))

eval: (org-open-file "somefile.pdf") works.

C-c C-e l o slowness is probably due to running org-latex-export-to-pdf with async. I tried running this command in Mx which gave it significantly faster speed.

Yep, it is a known tradeoff, exporting asynchronously is slower but not blocking, while exporting synchronously is faster but blocks Emacs during the export process.


Image For FileIcon

I'm submitting a ...

[ ] bug report
[ ] feature request
[X] support request

What is the current behavior?
The image appears too small.

What is the expected behavior?
I expect it to fit like any other 16x16 image.

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, etc)
I'm trying to assign an image that is 16x16x32bpp png to the FileIcon via the imageListView.DefaultImage property.

Is this the right place to assign it?
Why is my image coming out so small?  The default image is 16x16x32bpp and it looks fine.

Here's the default image screenshot.

Here's what my image looks like:

My PNG:

I got this working using an ico file of my image. Not sure why a png didn't work but ico is ok.


Change default behavior of vaffle-open-selected-split/vsplit

Objective
This PR changes the default behavior of <Plug>(vaffle-open-selected-split) and <Plug>(vaffle-open-selected-vsplit) following vim's default behavior of split and vsplit.
It solves #24 .

Merged. Thanks!


MM 2.25, node 20 HTMLElement is not defined

see this error on startup
[04.10.2023 07:59.52.457] [ERROR] ReferenceError: HTMLElement is not defined
    at Object.<anonymous> (/home/sam/MagicMirror/modules/MMM-CalendarExt3/MMM-CalendarExt3.js:1:26)

Where did you meet this error? In backend log? Or console? Or validator?

If you get that message from a Validator(or lint or any DOM Parser), that is not my fault.
HTMLElement is a standard native object of javascript in all browser, but some programmatic parser might not be able to handle it.
https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement
Anyway, I can add a fallback routine for it in the next update.
const popoverSupported = (typeof HTMLElement !== 'undefined') ? HTMLElement.prototype.hasOwnProperty('popover') : false

this is just out of Electron in  MM 2.25,  no special code..

when MMM-RemoteControl is installed
[04.10.2023 10:41.45.729] [LOG]   Connecting socket for: MMM-Remote-Control
[04.10.2023 10:41.45.729] [LOG]   Starting node helper for: MMM-Remote-Control
[04.10.2023 10:41.45.736] [LOG]   Sockets connected & modules started ...
[04.10.2023 10:41.45.834] [LOG]   Launching application.
[1435830:1004/104145.861991:ERROR:object_proxy.cc(590)] Failed to call method: org.freedesktop.portal.Settings.Read: object_path= /org/freedesktop/po>
[04.10.2023 10:41.45.905] [ERROR] ERROR! Could not validate main module js file.
[04.10.2023 10:41.45.906] [ERROR] ReferenceError: HTMLElement is not defined
    at Object.<anonymous> (/home/sam/MagicMirror/modules/MMM-CalendarExt3/MMM-CalendarExt3.js:1:26)
    at Module._compile (node:internal/modules/cjs/loader:1271:14)
    at Object..js (node:internal/modules/cjs/loader:1326:10)
    at Module.load (node:internal/modules/cjs/loader:1126:32)
    at node:internal/modules/cjs/loader:967:12
    at Function._load (node:electron/js2c/asar_bundle:2:13327)
    at Module.require (node:internal/modules/cjs/loader:1150:19)
    at require (node:internal/modules/cjs/helpers:110:18)
    at Class.loadModuleDefaultConfig (/home/sam/MagicMirror/modules/MMM-Remote-Control/node_helper.js:314:30)
    at /home/sam/MagicMirror/modules/MMM-Remote-Control/node_helper.js:268:26
    at FSReqCallback.oncomplete (node:fs:209:5)

the config..
     {
  module: "MMM-CalendarExt3",
  position: "bottom_bar",
  //disabled: true,
  title: "",
  config: {
    mode: "month",
    //instanceId: "basicCalendar",
    //locale: 'de-DE',
    maxEventLines: 5,
    firstDayOfWeek: 1,
    calendarSet: [], // ['us_holiday', 'abfall', 'mytest'],
  }
},
{
    module: 'MMM-Remote-Control',
    // uncomment the following line to show the URL of the remote control on the mirror
    // position: 'bottom_left',
    // you can hide this module afterwards from the remote control itself
    config: {
        customCommand: {},  // Optional, See "Using Custom Commands" below
        showModuleApiMenu: true, // Optional, Enable the Module Controls menu
        secureEndpoints: true, // Optional, See API/README.md
        // uncomment any of the lines below if you're gonna use it
        // customMenu: "custom_menu.json", // Optional, See "Custom Menu Items" below
        //apiKey: "12345", // Optional, See API/README.md for details
        // classes: {} // Optional, See "Custom Classes" below
    }
},

MMM-Remote-Control tries to load(require) module file in node_helper.
https://github.com/Jopyth/MMM-Remote-Control/blob/fb20bd483540ebb6eef0c8495cecd807d5dd1e0e/node_helper.js#L314C17-L314C48
Of course, it will fail because HTMLElement is not supported in NodeJS. Should I support the loading module in the NodeJS environment? (Maybe yes. Anyway, I'll add the code above in the next update)


geoP

Hello,
I wonder which script I should look at if I want to try the geoP method?
Thanks so mcuh!

Sorry for not giving a timely response, we implemented this permutation based method with a function getPermP.R in our package https://github.com/Sun-lab/asSeq
We clearly should document it better and maybe give a function a name better connected to the name we introduced in the paper.


Product download page shows only latest version

Description:
The download page does not have link to download earlier versions on APIM.
https://wso2.com/api-management/#
It will be nice to have other versions on download page. A drop-down to choose product version would be nice.
Steps to reproduce:
Content Positioning in Documentation:

Link: https://wso2.com/api-management/#

Heading (& Step):

Any other reference:

Optional Fields
Related Issues:

Suggested Labels:

Suggested Assignees:

Hi,
When you click download and scroll down, you can find a section called other resources. There you can find the previous releases section. Otherwise, use https://wso2.com/api-management/previous-releases/
Thank you!
Pubudu.


Remote-SSH broken after last update

Last update broke the Remote-SSH functionality. In this case no error is shown but multiple attempts are tried.
Build version is
Version: 1.37.0-insider
Commit: 0bd57ac9efae79f9eb1aedc3818ebbb570b9bf6b
Date: 2019-08-02T21:20:03.377Z

Output:
<EMAIL_ADDRESS>darwin x64
SSH Resolver called for "ssh-remote+localhost", attempt 1
SSH Resolver called for host: localhost
Setting up SSH remote "localhost"
Using commit id "0bd57ac9efae79f9eb1aedc3818ebbb570b9bf6b" and quality "insider" for server
Testing ssh with ssh -V
ssh exited with code: 0
Got stderr from ssh: OpenSSH_7.9p1, LibreSSL 2.7.3
Running script with connection command: "ssh"  -o ClearAllForwardings=true -o ConnectTimeout=15   "localhost" bash
Install and start server if needed
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 1
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 2
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 3
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 4
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 5
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 6
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 7
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 8
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 9
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 10
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 11
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 12
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 13
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 14
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 15
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 16
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 17
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 18
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 19
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 20
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 21
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 22
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 23
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 24
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 25
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 26
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 27
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 28
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 29
> Running remote connection script
> Installation already in progress...
> a121f07e-7581-4b42-83df-e07c4fa64c20##24##
"install" terminal command done
Received install output: a121f07e-7581-4b42-83df-e07c4fa64c20##24##
Server installation process already in progress - waiting and retrying 30

What did you do right before this? It seems like a previous install script crashed and left a lockfile behind. You can delete ~/.vscode-server-insiders/bin/0bd57ac9efae79f9eb1aedc3818ebbb570b9bf6b/vscode-remote-lock.0bd57ac9efae79f9eb1aedc3818ebbb570b9bf6b to fix it.

I just updated the app like other times. It turns out, very last update fixed the issue. Will try that suggestion next time it happens again. Thanks a lot.


[ZEPPELIN-2779] Unit test for job module (zeppelin-web)

What is this PR for?
Added few test cases for the job module under zeppelin-web/
Additionally,

removed lodash, q dependency
converted to JobModule
refactored jobmanager/*

What type of PR is it?
[Improvement]
What is the Jira issue?
ZEPPELIN-2779
How should this be tested?

cd zeppelin-web
yarn install (or npm install)
yarn run test (or npm run test)

The test should pass.
Questions:

Does the licenses files need update? - NO
Is there breaking changes for older versions? - NO
Does this needs documentation? - NO

@sravan-s Please help review this PR.

@sravan-s I updated the PR, now websocket (un)subscription is handled in this way (https://github.com/apache/zeppelin/pull/2497/commits/2e12c21197d916b864e3c29c627d929992ee89a2)
// job.component.js
  function init() {
    JobManagerService.getJobs()
    JobManagerService.subscribeSetJobs($scope, setJobsCallback)
    JobManagerService.subscribeUpdateJobs($scope, updateJobsCallback)

    $scope.$on('$destroy', function () {
      JobManagerService.disconnect()
    })

Here are logs for it.

Looks interesting, let me see :)

Thanks for the review @sravan-s
Merge if no more discussion.


Refactor analytics dashboards

Purpose
Refactor analytics dashboards. Related issue: https://github.com/wso2/analytics-apim/issues/1144
This PR contains the following changes:

Add 5 new widgets:

API usage over time (usage of selected API over selected time period)
Top 10 API usage during past 30 days(usage over time graph for top 10 most used APIs for the past 30 days)
Top 10 API performance during past 30 days(response time/performance over time graph for top 10 most used APIs for the past 30 days)
Top 10 throttled APIs during past 30 days(throttle count over time graph for top 10 most throttled APIs for the past 30 days)
Errors during past 30 days(total errors/faults during over time for the past 30 days)

Refactor styling used for 'Limit' input.
Remove the API creator from the following API name, version and creator combination used to filter analytics data.
(apiName==\'' + opt.name + '\' AND apiVersion==\'' + opt.version  + '\' AND apiCreator==\'' + opt.provider + '\')'
Since the analytics data are filtered from tenant and the api name is unique for a given tenant we can reduce the query complexity by removing the api creator condition.

Fixes https://github.com/wso2/analytics-apim/issues/1144


TK-370 add event inside calendar

Backend :

Create backend to add userEvents into weekCalendar and monthCalendar data.
Change userEventsResponse to get in the response multiple days if needed to display the userEventDayResponse in the right day.
Unit tests

Frontend :

Display userEvents in week and month calendar.
Add userEvents in the calculation of timeentries duration, to display completed tag if the 8hrs per day are reached.
Display the userEvents in day details and remove "Add entry" if the 8hrs are reached.

PR acceptée pour moi. Je n'ai pas reviewé la partie JS/Front.
Coté back sur les ajouts de "equals" j'aurai aussi testé car je pense que ce nouveau code peut avoir un impact important sur les List, les Set, et tout ce qui va stocker nos events.

The equals has been covered implicitly by some tests. For example with 'shouldCreateFromUserEvent()' it tests the equals of UserEventResponse.

When you try to access the http://localhost:3000/time_entries
I get following error in Browser

@GuillaumeDavy did you resolve all issue. If yes, update your branch please, you have some conflicts


Add native support for IIS7

See releated discussion http://biztalkdeployment.codeplex.com/Thread/View.aspx?ThreadId=82081.
Apparently detection/replacement of VDirs is not working as desired.
This work item was migrated from CodePlex
CodePlex work item ID: '5869'
Assigned to: 'tfabraham'
Vote count: '1'

[UnknownUser@6/28/2010]
Converted old VBS scripts to new MSBuild tasks using WMI.  Tested app pool config, vdir assignment and vdir create/delete on IIS 7.

[UnknownUser@6/28/2010]
Resolved with changeset 48059.

[UnknownUser@2/21/2013]

[UnknownUser@5/16/2013]


⚠ Bofrost.be has degraded performance

In 2ec92cc, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 907 ms

Resolved: Bofrost.be performance has improved in c8ae75c after 2 hours, 31 minutes.


[Question] GMS1.4 Compatible?

I was just wondering if any functions from gms2 are required for this inventory system. If not then I would suspect it would be gms 1.4 compatible, though possibly introduces bugs.

This project uses sequences to create the animations for the menu, which are GMS 2.3+ only, I believe. As far as the functions for sorting the items themselves, I don't believe there's any reason you couldn't use it in 1.4.


Compress code in repl querystring with LZString

This gives a reduction in url size of up to 60% while maintaining backwards compatibility (code will be accepted uncompressed, but the next-generated URL will use code_lz).
Some results:

URL Before
URL After
Query Before
Query After
URL Comp. %
Query Comp. %

Example 1 (Kent Dodds ES6 workshop)
1162B
776B
1011B
623B
33.2%
38.3%

Example 2
615B
360B
462B
204B
46.3%
55.8%

Addressed comments. I don't really care either way re: lz__, but that could conflict in case the start of the file contains that. This doesn't have that issue.
Also noticed flow is doing this too.

if it is backwards compat and smaller, sounds good to me!

👍


Docs: Mention the fact that this generates PEP-658 compatible metdata as well

It is cool that this produces a full index, including PEP-658 metadata! That is way more than just PEP-503, and you should probably highlight that in your docs.

Thanks! More than happy to merge a PR to update the docs if you would be inclined.


notation plugin add to abstract different platform config directories

The plugin directory varies depending upon the operating system being used.  notation config article.
This adds complexity to users attempting to configure a plugin. To simplify the process, notation plugin add should be revived to place the plugin in the appropriate directory, based on the platform.

Resolving as a dupe of #193


Open programmatically datepicker bug (found in documentation example)

Go to the example documentation: https://codesandbox.io/s/mpklq49wp
Go to the "Open programmatically" section
Click the "Open Picker" button to open the datepicker
Attempt to click anywhere on the page outside the datepicker component & calendar popup

Expected behavior
Datepicker closes on click
Actual result
Datepicker stays open
For reference, I'm using Chrome 71.x on a Mac.
This happens when you use this.$refs.datepicker.showCalendar() as well in a parent component.

Thanks @shimy1984 !
For information, if you want to open the calendar programmatically but it is in a v-show (can't be in a v-if because then you can't use the reference, you have to use nextTick to make it work:
        <datepicker
                v-show="showCalendar"
                :value="value"
                ref="datepicker">
        </datepicker>

                this.editing = true;

                let that = this;

                Vue.nextTick(function () {
                    that.$refs.datepicker.showCalendar();
                    that.$refs.datepicker.$el.querySelector('input').focus();
                });


Cross Origin Should Be Omittable

Closes https://github.com/justinmc/react-audio-player/issues/48
@dmcmorris with this change, not passing the crossOrigin prop will result in an audio tag with no crossorigin attribute at all.  Let me know if that fixes your use case.

I'm going to go ahead and merge this because it seems like a solid improvement to me either way.  Happy to revisit if not though.


Cast to a string when setting a tag

Closes #13

Thanks @eligijusvitkauskas-home24. For some reason, PRs don't trigger travis build :S. Not sure if it is because they com from a fork.


Add Header and fields in Reported Fossil Resource Model

[x] Create first Header, above “Observed by” - called “Type of Fossil” (different from Fossil Type)
[x] For first subheader, Copy from Fossil Sample RM and call it “Fossil Common Name” with subtitle “Select a name based on everyday language” – not mandatory because they may not know
[x] Add checkbox “Common Name Uncertain”
[x] Second subheader – “Description of Potential Fossil”, with freeform box same size as those in Location Description – make mandatory field

Moving this back to New and removing MVP as public submissions are not part of MVP

There appears to be an Arches Core bug that is not saving the widget types properly. Description of Potential Fossil widget currently text not rich text. Manually updated resource model JSON file, should be corrected in the next refresh.


Correct prescript for its operation in windows, making it compatible on all three platforms: linux, MacOS and Windows.

Correct prescript for its operation in windows, making it compatible on all three platforms: linux, MacOS and Windows.
I add Vulkan's logo on the console.

Oh that's awesome, nice job!


Switching dynamically between multiple LLM models on VRAM

What are you trying to do?
At the moment, Ollama needs to load LLM one by one. It should be possible to have multiple LLM resident in VRAM memory and switch dynamically between the two.
How should we solve this?

Check if the requested images can be loaded within the current hardware VRAM budget.
load multiple images in VRAM.
Augment API to switch dynamically between the two.

What is the impact of not solving this?
The present latency when loading and unloading multiple models.
Anything else?
No response

Ollama is small, using docker compose to run multiple instances may be helpful to you.
version: '3'
services:
  ollama_1:
    image: ollama/ollama
    ports:
      - "11434:11434"
    volumes:
      - ./data/.ollama:/root/.ollama
    deploy:
      resources:
        reservations:
          devices:
            - capabilities: [gpu]
    restart: unless-stopped
  ollama_2:
    image: ollama/ollama
    ports:
      - "11435:11434"
    volumes:
      - ./data/.ollama:/root/.ollama
    deploy:
      resources:
        reservations:
          devices:
            - capabilities: [gpu]
    restart: unless-stopped
  #ollama_n ...

Seconding this for the use case of multi-agent workflows

I second this as well due to wanting to have both a embedding model and a instruction tuned model loaded at the same time.
I'm seeing this useful for tasks such as dynamic/live RAG systems.
Also, I was needing this feature when I was wanting to load and compare two different quantized versions of the same model a few days ago.
I was wanted to see how much of a deviation of next token predictions were between two quantization's from the same starting prompt. And I was aiming to have the same seed, high temp's and greedy like sampling generation options for each test.
Most embedding models are small compared to regular models. So this seems reasonable. I know it would be for my setup.
Currently as ollama stands, It seems excessive and likely a waste of power, delay of time, and like causes more wear on the hardware to switch rapidly between two different models for pipelines.

This is coming soon (along with concurrency).


Add different reporting options

Add the ability for the app to report failures to different chat environments, i.e slack, discord.

After a bit of a spike, some concerns about the existing solutions. Many of the solutions don't make good use of interfaces which means that portion of the code will be impossible to stub out and unit test. So either I run a full integration test (and leak a slack api key into github) or I hold off for the time being. Seems better to provide some kind of web UI for this app to provide reporting on what the monkey is getting up to.


ORA-00911: invalid character

You can see in below statement space is missing between 'NITS' and AND, it is throwing error in FlyWay DB where as it works fine in SQLPlus and AQT, Toad.
This gives error in Update, Insert, Merge all statements if space is missing.
DELETE FROM TEST.TABLE1 where CFG_AREA_ID_1 like '%NAME%' AND SOME_ID='NITS'AND CFG_AREA_CD IN ('COND_TXT','FORM');

This can be fixed with #990?

Did any one noticed this?

Is this forum dead :)

No this forum isn't dead, it's just that sometimes people are busy :-)

Thanks Axel :)


Extend Discord Notifier

This PR extends the Discord notifier to improve the readability of alerts.
It changes 5 key elements:

A configureable icon as bot Logo (I'm currently using the GitHub Avatar. Is that alright? Is there a better way for doing this?)
A configurable name for the bot
Alert-ID in the footer
Start & end times in the footer (Even if the message is sent with a delay, the correct time will be displayed)
It uses Discords fields-feature to display labels
It sends one message per alert

It also fixes UI builds on apple silicon by forcing the linux/amd86 platform for the elm container

Before:

This looks nice! Thank you very much for your contribution -- can you please show me an example of the after with multiple labels?

Sure! Desktop:

I'm also planing to change the title to not include the labels anymore

Small update:

I'd love to get some first feedback @gotjosh

Hey. I‘d love to get some feedback on this.

I‘ll correct a few minor things this weekend. Afterwards it should be ready to be merged

The visual changes look nice! You'll need to remove the one message per alert change though as it interferes with alert grouping behavior. For example, if a user wants one message per alert then grouping can be disabled in the route, like so:
route:
    receiver: test
    group_by: ['...']

You can find it documented here.

Good catch! Thank you!

The visual changes look nice! You'll need to remove the one message per alert change though as it interferes with alert grouping behavior. For example, if a user wants one message per alert then grouping can be disabled in the route, like so:
route:
    receiver: test
    group_by: ['...']

You can find it documented here.

Could you please have a look at the latest changes? Grouping is now allowed by only sending one message per alert - containing as much 'embeds' (those alert boxes you're seeing) as allowed per Discord API limits. Is it okay like that?
I still have to clean the code up and make it a bit more readable before this is ready to be merged tho. (Argh.. and signing off the past commits. I always forget to do it.. I will add a signed-off git message template now 😄)

I noticed that when the maximum number of embeds is exceeded, just the first embed is sent? Is that expected? Looking at the code I would expect 10 embeds to be sent with the truncation message rather than just 1 embed?

I'd love to finalize this and get this into 0.27.1. @gotjosh could you please have a look at the discussions above?

I've been waiting for over two months now, and my PR still hasn't been fully reviewed, nor have my questions been answered. I know everyone is busy, but it's getting a bit frustrating. I'd love to help improve the software we all use day to day, but I'm not sure if I can keep contributing if things don't change.
For those who might not know what I'm talking about, here is my current focus: Improving Discord Integration in Alertmanager.

Implemented features aren't tested at all (see: https://github.com/prometheus/alertmanager/pull/3555)
The promised fixes aren't released (see: https://github.com/prometheus/alertmanager/pull/3728#issuecomment-1969078303 "I'm going to release v0.27.1 relatively soon [...]")
PRs aren't getting reviewed in time. (see: this one)

Please don't get me wrong; I love your work and want to keep contributing. I do honor your tight schedules. It's just getting a little... frustrating for me and, looking at the comments on some of the PRs, for many others as well.

Hi Luca! 👋
I think we are still waiting on some changes since the last time, no?

You're right. I'm gonna refactor this - using the embed title for displaying the alert summary (CommonAnnotations.summary) and adding the total alert count to the message above the embeds.

I don't see any commits since this comment, is this still something you want to change?

I noticed that when the maximum number of embeds is exceeded, just the first embed is sent? Is that expected? Looking at the code I would expect 10 embeds to be sent with the truncation message rather than just 1 embed?

I believe this isn't working as it should, did you get time to take a look?

I solved some locally without pushing because I'm waiting for the discussions above to resolve. As long as I don't know what way you want me to implement things, I don't know how to continue working on this.
See:
My last comment:

I'd love to finalize this and get this into 0.27.1. @gotjosh could you please have a look at the discussions above?

Waiting for someone else to comment on it:

Waiting on anyone to comment on it:

I solved some locally without pushing because I'm waiting for the discussions above to resolve. As long as I don't know what way you want me to implement things, I don't know how to continue working on this.

Oh! Please push those changes so we can take a look! I think those two (repeated title and missing embeds) are unrelated to the other discussions so 100% recommend we push those fixes to be reviewed 👍

Waiting for someone else to comment on it:

I'll check in with @gotjosh, let's keep making progress on the remaining issues and we can come back to this! 👍

Waiting on anyone to comment on it:

I'm not sure I see where #3821 is adding a skip_fields field? I think my opinion is unchanged on this topic – let's remove it and add it back if it turns out users need it.

Hi @LucaDev! Do you want to push the changes you made so we can take a look?

Hi @LucaDev! We are preparing an Alertmanager 0.26 release, are you still interested in this?

I apologise @LucaDev - as you can appreciate, my GitHub notifications are just unmanageable and other priorities have kept me away from Alertmanager duties.
I don't think I'll have the time herd this PR into the cut for 0.28 which is sad - but can we try to attempt for the next release?

Hey all. Apologies for letting you wait for so long. I had quite a few busy months. I’ll rebase the code and push the latest changes soon. Would love to see it in 0.28 (especially due to the long release cycles of alertmanager) but 0.29 would also be fine I guess. Better late then never after all 😄


Expanded net view border issue

Expanded nets with grouped views are not using the correct format dict for column test view rows.
They currently use N-NET & PCT-NET formats but they should use N & PCT so that the top border in rows that need background do not take the dynamic net top border color.

Resolved by #341.


Use cookieName/localStorageKey based on project name

Right now the ember_simple_auth:session key is used for every project using this addon, which can lead to problems when the same domain is shared or when running on localhost.
The getOwner() method returns an object with an application property, which itself has a name property that could be used to generate a <application-name>:session key instead, which would most likely be sufficiently unique.

You can actually configure the cookie name: https://github.com/simplabs/ember-simple-auth/blob/master/addon/session-stores/cookie.js#L65

@marcoow I know that, but it is pretty tedious having to overwrite the session-store in every single app when it can easily be handled by the lib itself. What's the reason for rejecting this proposal?

I don't really see a need for this TBH given that there will never be a conflict unless multiple apps are served from the same origin. Also customizing the property is pretty easy and straight forward.


Automatically generate results in latex format

The goal is to obtain the results from jupyter and generate them in latex format that can be pasted in a paper

Do you expect a command that can be run inside a Jupyter cell and it converts the notebook directly to a .tex file?
I think that a Pandoc integration can be done for this.

Hi @RodolfoFerro  sorry I wasn't clear.  By running the jupyter dashboard hj.get_dashboard(rm, vars(), wide_display=True) it will give us results like these.

For this issue, we are trying to create a latex widget like the one below that can visualize these results in latex table format.

So that when they are added to a paper it should look like something like this

This latex still needs more testing and extra features be super useful.
In my experience, creating latex tablesy for papers  takes a loong time and is prone to human error. The goal of this is to streamline this process and make it reliable.

it has been added see the latex tab in https://colab.research.google.com/drive/1iqmJWTfsC3Erfay_SwEoUhq_wS4l18Fd#scrollTo=7bCDi3VlFsir


[C++][Parquet] Remove unneeded private member variables from RowGroupReader ABI

The current PIMPL idiom exposes private metadata members in its header. With the metadata implementation, this can be removed.
This also helps simplify the design and remove virtual functions.
Reporter: Deepak Majeti / @majetideepak
Assignee: Deepak Majeti](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=mdeepak) / @majetideepak
Note: This issue was originally created as PARQUET-710. Please see the migration documentation for further details.

Wes McKinney / @wesm:
For what it's worth, the intent of the current structure was to permit test fixtures to be created (for testing edge cases in these classes). PIMPL was the wrong name for this, more of a dependency injection

Deepak Majeti / @majetideepak:
Do you see any issues with using a structure similar to the FileMetaData/RowGroupMetaData classes ?

Wes McKinney / @wesm:
See MockPageReader in parquet/column/test-util.h. You could create MockFileContents, MockRowGroupContents, etc.

Wes McKinney / @wesm:
Issue resolved by pull request 154
https://github.com/apache/parquet-cpp/pull/154

Wes McKinney / @wesm:
Generating test cases at the level of the file or row group is significantly more complex because many parts of the writing process are not necessarily under the control of a user of the public API. By comparison, FileMetaData is a reasonably self-contained dependency (it doesn't really benefit from any DI-decoupling because the Thrift struct is the lowest level).
The idea was to use a dependency injection sort of pattern to define the coupling between the business logic of the file reader and row group reader and the underlying machinery (the column readers, etc.) that it interacts with. If the only thing you have at your disposal to manipulate the workings of each public API (to isolate and test edge cases) is real Parquet files this makes things much less flexible.

Deepak Majeti / @majetideepak:
The injection capability is retained.
The private members are removed in line with PIMPL.
Patch available: https://github.com/apache/parquet-cpp/pull/154


Remove require-self

Should be a dev dependency not a dependency

Should be a dev dependency not a dependency

Missed that, yeah, fixed


python312Packages.twilio: 9.3.6 -> 9.3.7

Diff: https://github.com/twilio/twilio-python/compare/refs/tags/9.3.6...9.3.7
Changelog: https://github.com/twilio/twilio-python/blob/9.3.7/CHANGES.md

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 356126

x86_64-linux

:white_check_mark: 6 packages built:

authentik
home-assistant-component-tests.twilio
python311Packages.twilio
python311Packages.twilio.dist
python312Packages.twilio
python312Packages.twilio.dist


WFLY-6478 additional permissions for ServiceMBeanSupportTestCase

https://issues.jboss.org/browse/WFLY-6478

Windows Build 4608 outcome was FAILURE using a merge of ed8554b11af88e31b6cdfcff85b6a44ccf16009c
Summary: Execution timeout (new); tests passed: 3330, ignored: 195 Build time: 01:40:23


ARM Build

This looks like a great project. Any chance your can release a build for ARM machines? I'm hesitant to try compiling it myself....

Yes, I could. ARM Linux?
By the way, you might be very, very pleasantly surprised at how easy it is to cross-compile in Go. Go's tooling is excellent, and cross-compilation generally "just works". This command will procude a working ARM devd binary in the repo:

env GOOS=linux GOARCH=arm GOARM=7 go build

Yes, it really is that simple.

Yes, ARM Linux would be great. You're right, though, that is easy!

I was able to do it with the help of gox. Thanks!

I spoke to soon. Running the binary on my ARM machine, I get:
panic: could not locate box "templates"

How would I build so those files are included?

Ok, got it working by manually installing go.rice and modifying the build bash script in the devd source. If anyone needs the  arm binary before cortesi releases one, I've posted a copy here.

@judsonmitchell: I would definitely merge a pull request that added ARM support to that build script... ;)


Remove README line saying it's written in JavaScript

Remove the line saying it's written in JavaScript. It's really in ReScript, which compiles to JavaScript. I think I made a mistake when resolving a marge conflict, so this is probably my fault; hopefully this fixes it.

I feel partly this is my fault too because I modified your pull request, which is not a common practice. Thanks for fixing!


NotImplementedError: Wrong number or type of arguments for overloaded function

File "/home/x/.local/lib/python2.7/site-packages/ortools/constraint_solver/pywrapcp.py", line 3150, in __init__
    this = _pywrapcp.new_RoutingIndexManager(*args)
NotImplementedError: Wrong number or type of arguments for overloaded function 'new_RoutingIndexManager'.
  Possible C/C++ prototypes are:
    operations_research::RoutingIndexManager::RoutingIndexManager(int,int,operations_research::RoutingIndexManager::NodeIndex)
    operations_research::RoutingIndexManager::RoutingIndexManager(int,int,std::vector< operations_research::RoutingIndexManager::NodeIndex > const &,std::vector< operations_research::RoutingIndexManager::NodeIndex > const &)

I am frequently getting the following error from pywrapcp.RoutingIndexManager(). It happens when I remove the data['depot'] = 0 and/or when I experiment with different constraints. This is not the same as routing.status().
This is fairly reproducible and I would appreciate any insights/suggestions.

https://developers.google.com/optimization/reference/constraint_solver/routing_index_manager/RoutingIndexManager/ I cannot use both data['depot'] and data['starts'] and data['ends'] simultaneously.

I meet the same problem!!!

You can't, but if your depot is at 0,0 you can just input:
manager = pywrapcp.RoutingIndexManager(len(data['distance_matrix']),
                                       data['num_vehicles'], data['starts'], data['ends'])

and to make sure you start at the depot, you will need to put zeroes in data['starts'] and data['ends'] and deviate from those zeroes if you want to start or end somewhere different.

In my experience, this error can occur as the result of having numpy types as arguments (either scalar or inside an array), rather than base python types. They're supposed to be aliased, but the wrapper has difficulty parsing them, it appears. Or something.
Anyway, in my case I fixed this error by converting all my arrays' objects types to python's int.

@tom-goodacre-chowagiken This solved my problem. Thanks


feat!: Make build output streamable

This PR includes one small breaking change to the builder's ForDirectory—it now accepts an io.Writer as the second argument that will have build output written to it. It can be nil.

This looks great! Could we add a test where you pass in a bytes.NewBuffer as the writer, and then run a build to verify it does indeed write into it? Doesn't matter if the build fails.

There are now tests. Tested the command runner directly though instead of running a whole build (didn't want to run docker, etc in the unit tests).


Binding Issue with Extension prevents Function indexing

I am using the version 1.0.1 of this Nuget package in my .NET Core V2 Azure Function App to help with dependency injection and am running into an issue.
My Function app is to allow an HTTP upload of an image, and write it to blob storage and some associated meta-data to CosmosDb using the MongoDb Driver. I setup the WebJobStartup.cs class and used the '[Inject]' attribute on the interfaces I was trying to inject into the UploadFile.cs function class's Run method. See the source code.
When I run my function I get the following error:

[3/26/2019 1:34:03 PM] Error indexing method 'UploadImage'
[3/26/2019 1:34:03 PM] Microsoft.Azure.WebJobs.Host: Error indexing method 'UploadImage'. Microsoft.Azure.WebJobs.Host: Cannot bind parameter 'cosmosDbClient' to type ICosmosDbClient. Make sure the parameter Type is supported by the binding. If you're using binding extensions (e.g. Azure Storage, ServiceBus, Timers, etc.) make sure you've called the registration method for the extension(s) in your startup code (e.g. builder.AddAzureStorage(), builder.AddServiceBus(), builder.AddTimers(), etc.).

Is there more that I should be doing with WebJobStartup.cs? Another obvious issue with my program? Or an issue with this project itself?

I am closing this because I found a solution to fix the binding issue
in my .csproj I changed
    <TargetFramework>netcoreapp2.2</TargetFramework>     <AzureFunctionsVersion>V2</AzureFunctionsVersion>
To:
    <TargetFramework>netstandard2.0</TargetFramework>     <AzureFunctionsVersion>v2</AzureFunctionsVersion>


Intern ServerInventoryView inventory objects.

Allows replicants of data segments to only take up one chunklet of heap in the server view.
Related to https://github.com/druid-io/druid/issues/3237

I'm working on getting this rolled out internally

Results are promising so far. With this patch tending to lead to ~50% reduction in old-gen size as time progresses.

Per https://github.com/druid-io/druid/issues/3237#issuecomment-232425351 it seems fair to do this in order to de-dupe replicas of the same segment regardless of what happens for #3237.
For #3237 IMO this shouldn't be considered a fix, we should actually collapse the two server views into one.

@gianm agreed on not a fix. I'm hoping "Related" doesn't count as "Fixed" in github-land-auto-closing

👍

👍


refactor: converted i18n files from TOML to YAML

Closes #976
Closes #757

@craicoverflow just wondering whether you had a chance to review the changes?

Hey @alexal - not yet but will get to it soon.

We came to the conclusion that while it is easier to group keys in YAML, it actually makes it more difficult to look up the property when looking at the i18n ID.
With TOML, they are exactly the same. Eg:
https://github.com/redhat-developer/app-services-cli/blob/8b3ac77c29ca01e2c68ba59d02e14e153a9cf517/pkg/cmd/cluster/bind/bind.go#L49
https://github.com/redhat-developer/app-services-cli/blob/4930e17e17be946a01c6c5db7b9115d01edf7482/pkg/localize/locales/en/cmd/cluster_bind.en.toml#L1-L2
For that reason, TOML is a better choice. I'm going to close this PR and the issue. Thank you for your contributions 🙏🏻


Add support for nested fields for Date field in the Time oriented option.

Is your feature request related to a problem? Please describe.
In the time-oriented reindex algorithm, if the Date field that is meant to be used is part of the nested object, there will be a "The date field doesn't exists" error.
Describe the solution you'd like
The nested fields should be considered for time oriented algorithm.
Describe alternatives you've considered
None.
Additional context
None.

For a clear understanding: when you talk about a "nested" object, do you mean the array of the object, that is defined as "nested" in the index mapping like:
{
  "mappings": {
    "properties": {
      "logs": {
        "type": "nested",
        "properties": {
          "timestamp": {
            "type": "date"
          },
          "field": {
            "type": "keyword"
          }
        }
      }
    }
  }
}

with the documents like:
{
  "logs": [
    {
      "timestamp": "2023-04-08T00:00:00",
      "field": "Some text"
    },
    {
      "timestamp": "2023-04-08T00:00:00",
      "field": "Some text"
    }
  ]
}

or the inner object with mapping like:
{
  "mappings": {
    "properties": {
      "logs": {
        "type": "nested",
        "properties": {
          "timestamp": {
            "type": "date"
          },
          "field": {
            "type": "keyword"
          }
        }
      }
    }
  }
}

and the document like:
{
  "log": {
    "timestamp": "2023-04-08T00:00:00",
    "field": "Some text"
  }
}


new hidden options (so users do not use it without better knowledge) …

…and are under investigation. KUSTO_ENSURE_NO_DUPLICATED_BLOBS

use tags to ingest blobs individually and ingest only if tag does not exist, meaning blob was not already ingested KUSTO_DISABLE_FLUSH_IMMEDIATELY - dont use flush immediately if clientBatchLimit reached

Bug fix :

Exception type for retrying of commands was wrong (was hiding real error)
Exceptions in queuing were not logged

Looks good to me , will just wait for user confirmation if these options work in the private build


Fixfix error of X-Z function for nereids

Proposed changes
Issue Number: close #xxx
Problem summary
fix error of X-Z function for nereids
Checklist(Required)

Does it affect the original behavior:

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

Has unit tests been added:

[ ] Yes
[ ] No
[x] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[x] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

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

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

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 34.85 seconds
load time: 490 seconds
storage size:<PHONE_NUMBER>4 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230129104006_clickbench_pr_86394.html


Update readme's elastic configuration section

Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] Optimization
[x] Documentation Update

Description
Update readme's elastic configuration section, to explain how it is working.
Related Tickets & Documents

Related Issue #
Closes #

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please describe the System Under Test.
Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

As you focused on README.
Can you add the details, of how other teams will add their CI to this dashboard?

As you focused on README. Can you add the details, of how other teams will add their CI to this dashboard?

I added the details to the readme doc


Allow custom_jvp and custom_vjp to decorate methods

Addresses #2483

Originally posted March 2020 (https://github.com/google/jax/pull/2487)
@mattjj Sorry for the repost, but I needed to move this into a branch.  Please let me know when you have time to do a code review :)

@mattjj  Any chance you could take a look at this?  I got the change down to 8 lines of new code.  I know that object-oriented code may not be appealing to you, but it simplifies my code a lot as I mentioned in the related issue, and this code will never run except when it's used on a method, so it is transparent to other users.

closing due to lack of interest


path issue on supervised-embedding

bin/train_all.sh
bin/task1.sh: line 12: data/candidates.tsv: No such file or directory
bin/utils.sh: line 12: data/train-task-1.tsv: No such file or directory
bin/utils.sh: line 13: data/dev-task-1.tsv: No such file or directory
bin/utils.sh: line 14: data/test-task-1.tsv: No such file or directory
bin/utils.sh: line 15: data/dev-task-1-500.tsv: No such file or directory
bin/utils.sh: line 16: data/vocab-task-1.tsv: No such file or directory
cat: data/train-task-1.tsv: No such file or directory
cat: data/dev-task-1.tsv: No such file or directory
cat: data/test-task-1.tsv: No such file or directory
Traceback (most recent call last):
File "train.py", line 120, in
vocab = load_vocab(args.vocab)
File "//personalized-dialog/supervised-embedding/make_tensor.py", line 36, in load_vocab
with open(vocab_filename, 'r') as f:
IOError: [Errno 2] No such file or directory: 'data/vocab-task-1.tsv'
bin/task2.sh: line 12: data/candidates.tsv: No such file or directory
bin/utils.sh: line 12: data/train-task-2.tsv: No such file or directory
bin/utils.sh: line 13: data/dev-task-2.tsv: No such file or directory
bin/utils.sh: line 14: data/test-task-2.tsv: No such file or directory
bin/utils.sh: line 15: data/dev-task-2-500.tsv: No such file or directory
bin/utils.sh: line 16: data/vocab-task-2.tsv: No such file or directory
cat: data/train-task-2.tsv: No such file or directory
cat: data/dev-task-2.tsv: No such file or directory
cat: data/test-task-2.tsv: No such file or directory

The supervised embedding model creates .tsv files for the data before running. Make sure that these files have been created prior to training.
Maybe the paths to the data files in the task$.sh files have to be changed to work.


Get-PnPTeamsChannelFilesFolder: Fix param in examples

Type

[ ] Bug Fix
[ ] New Feature
[x] Sample

What is in this Pull Request ?
This PR replaces wrong parameter 'Identity' with 'Channel' in all examples of 'Get-PnPTeamsChannelFilesFolder'.

Well noticed @enthusol, thanks for correcting it!


Clicking Submit from preview page of Activity creation does not always work properly

In old profile, when I click on the button to submit, it does not work.
In new profile, if I click on the button more than once, it submits the same activity multiple times.

[ ] Clicking submit button should only create activity one time
[ ] Clicking submit should redirect the user to their homepage

I recreated this issue by having  not filled out all of the answers on the previous page (description of volunteer role), removed the error box, filled in the role information, then when I was forwarded to preview, when I clicked submit, I do not leave the page. I click submit again and do not leave the page. When I refresh to the main page, there is a duplicate of the activity I created. When I delete one of the activities, it deletes the duplicate as well.

@lina12345678 Please recreate this as a new issue in github with the labels design and discussion. The title can be 'Activity Categories'

The issue begins on this page:

After clicking submit multiple times, it leads to this photo after I go back to the home page. There is no automatic redirect, I have to exit the page manually.

Can no longer recreate this bug.


Bitcode Support

Hi,
I am using Obfuscator library in my project. However, I cannot create an Archive for my app because of the following error:

Are there updates/workaround to address this issue other than disabling bitcode on the target level?
Thanks,
Kenneth

https://stackoverflow.com/questions/32640149/disable-bitcode-for-project-and-cocoapods-dependencies-with-xcode7

Thanks for the link @pjebs. I've actually seen that one before. However, our client(that uses our library) is requiring bitcode support. Any plans of supporting this in the future?

Yes. Configure your cocoa pods to use bitcode. The link tells you how to turn it off. Just do the opposite.


Having an issue decoding an early transaction

I'm debugging this code:
https://github.com/SurmountSystems/gabriel/blob/8362fdcfd686e4ec461286f4ed0463da747aaf83/src/main.rs#L79-L101
Output is:
https://xyz.local/api/block-height/540
https://xyz.local/api/block/000000001958fb614e3eed2ab35305fa89a8b8db9bb5fd2575aa08fc0a06b84f
tx len: 1
https://xyz.local/api/block-height/541
https://xyz.local/api/block/00000000aa3704d9c763fac9ae41842c55a2b3a1c5b404dff2ca82c9f9ebd64b
tx len: 1
https://xyz.local/api/block-height/542
https://xyz.local/api/block/00000000bdb2a11dbc79d6f367e666448ac16aed9be8ad7c5a5712e23d0a4942
tx len: 1
https://xyz.local/api/block-height/543
https://xyz.local/api/block/0000000009b905f3db4990848b9b14fe39582147fcf6a6cb8d73bd227d75369e
tx len: 1
https://xyz.local/api/block-height/544
https://xyz.local/api/block/000000002d947997dc957cdf075dd32390f5f754d2656208d5dd82a6620179f5
tx len: 1
https://xyz.local/api/block-height/545
https://xyz.local/api/block/00000000689051c09ff2cd091cc4c22c10b965eb8db3ad5f032621cc36626175
tx len: 1
tx len: 1
https://xyz.local/api/block-height/546
https://xyz.local/api/block/000000005a4ded781e667e06ceefafb71410b511fe0d5adc3e5a27ecbec34ae6
tx len: 1
tx len: 2
tx len: 2
res len: 217
hex: 3c21444f43545950452068746d6c3e0a3c68746d6c206c616e673d22656e223e0a3c686561643e0a3c6d65746120636861727365743d227574662d38223e0a3c7469746c653e4572726f723c2f7469746c653e0a3c2f686561643e0a3c626f64793e0a3c7072653e43616e6e6f7420474554202f6170692f76312f74782f323832303463616431643766633164313939653865663466613232663138326465363235386133656161666531626265353665626463616364333036396135662f7261773c2f7072653e0a3c2f626f64793e0a3c2f68746d6c3e0a
real len: 217
test len: 193
Error: Io(Error { kind: UnexpectedEof, error: None })

Sorry if this issue is short... It should be obvious what I'm trying to accomplish, but if it isn't, feel free to ask for clarification.

 It does not decode with Bitcoin Core either.

Aha! You are correct. That's because the hex I provided is actually an encoded version of this response text:
<!DOCTYPE html>

<meta charset="utf-8">
<title>Error</title>

Cannot GET /api/v1/tx/cb6cae7d02a30fa4e6fc2d9ff83dc8acae967657e187d1ffc3caca97e28785cd/raw

I'll have to figure out why I can't get the transaction by txid...
Is there a better way to get the spend scripts used by the inputs?

I think there's some kind of bug in the mempool version I'm using that precludes raw transaction output. I had to implement my own serde structs for its own particular JSON format.
I'm going to close this now as I think this is growing out of scope. Thank you for helping me get unblocked.