feat(#709): support temporary uuid in quarantine filter

Current dependencies on/for this PR:

main

PR #710   👈

This stack of pull requests is managed by Graphite.

Merge activity

Jan 17, 9:09 AM: @MartinBernstorff merged this pull request with Graphite.


fix cors issue

This fixes a cors error below:
Access to fetch at 'http://<IP_ADDRESS>:9527/socket.io/?EIO=3&transport=polling&t=MnHng1h' from origin 'https://imjoy.io' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

@MartinHjelmare I don't know why the test failed, it also fails if I checkout the master branch, it seems something else changed. I checked our requirements we have every lib version fixed. Could you please take a look?

python-engineio changed version from 3.8.2 in last successful build to 3.9.0 in this build. It's not pinned in python-socketio. So we need to pin it ourselves if we want total stability in that dependency too.

Perhaps also check why that KeyError happens.

python-engineio changed version from 3.8.2 in last successful build to 3.9.0 in this build. It's not pinned in python-socketio. So we need to pin it ourselves if we want total stability in that dependency too.

So you mean you can pass the test if you pin the version to 3.8.2?

I don't know. I have just read the Travis logs and checked python-engineio recent version updates.

I just tried to fix the engineio version 3.8.2 and my local test still fails, no idea.

https://github.com/oeway/ImJoy-Engine/pull/78#issuecomment-517655380

It works by pin python-engineio==3.8.2 now, strange that the master branch still has issue @MartinHjelmare
The keyError is from python-engineio version, they changed something which cause this error.

@MartinHjelmare Thanks for your help!


[PDI-19474] Changes help links to always open in default browser

:white_check_mark: Build finished in 19m
Build command:
mvn clean verify -B -e -Daudit -amd -pl ui

:exclamation: No tests found!
:information_source:  This is an automatic message


Vnish miners don't return configs and pool info

Vnish miners when trying to get information via miner.api.pools or miner.get_config returns None.

Good to know.  Working on fixing configs, as well as better Vnish support, so hopefully I can fix this soon.

@Awote What is your Vnish version?

@s0kil  '1.2.0-rc4'

@Awote I've tested with multiple machines we have running on 1.2.0-rc4, can't reproduce the issue you are having.
Make sure your pyasic version is up to date:
pip3 install pyasic --upgrade

i used version 0.45.0 before your last message @crc32 . Thenn i updated and tested.
import pyasic

async def t(ip):
    miner = await pyasic.get_miner(ip)
    config = await miner.get_config()
    pool = await miner.api.pools()
    pass
asyncio.run(t("<IP_ADDRESS>"))

async def t1(ip):
    net = pyasic.MinerNetwork(ip)
    pyasic_miners = await net.scan_network_for_miners()
    pass

asyncio.run(t1("<IP_ADDRESS>-<IP_ADDRESS>"))```

first ffunnc return correct but second return empty list in Terminal i have issues:
[WARNING][11/28/23 10:14:52](root) - .: Unhandled ping exception: encoding with 'idna' codec failed (UnicodeError: label empty or too long)
c:\Users\NeShdw\Desktop\1\venv\Lib\site-packages\pyasic\network\__init__.py:204 in ping_and_get_miner
[WARNING][11/28/23 10:14:52](root) - .: Unhandled ping exception: encoding with 'idna' codec failed (UnicodeError: label empty or too long)
c:\Users\NeShdw\Desktop\1\venv\Lib\site-packages\pyasic\network\__init__.py:204 in ping_and_get_miner
[WARNING][11/28/23 10:14:52](root) - .: Unhandled ping exception: encoding with 'idna' codec failed (UnicodeError: label empty or too long)
c:\Users\NeShdw\Desktop\1\venv\Lib\site-packages\pyasic\network\__init__.py:204 in ping_and_get_miner
[WARNING][11/28/23 10:14:52](root) - .: Unhandled ping exception: encoding with 'idna' codec failed (UnicodeError: label empty or too long)
c:\Users\NeShdw\Desktop\1\venv\Lib\site-packages\pyasic\network\__init__.py:204 in ping_and_get_miner
[WARNING][11/28/23 10:14:52](root) - .: Unhandled ping exception: encoding with 'idna' codec failed (UnicodeError: label empty or too long)
c:\Users\NeShdw\Desktop\1\venv\Lib\site-packages\pyasic\network\__init__.py:204 in ping_and_get_miner
[WARNING][11/28/23 10:14:52](root) - .: Unhandled ping exception: encoding with 'idna' codec failed (UnicodeError: label empty or too long)

Maybe i should use only
get_miner()

on every ip rather than
scan_network_for_miners

?

Try 0.41.0 with await MinerNetwork.from_address("<IP_ADDRESS>-255").scan()

if i use this await MinerNetwork.from_address("<IP_ADDRESS>-255").scan() i ll get not at all miners (80% of all miners).
if i use scan_network_for_miner  in pyasic==0.45.0 , i ll get all miners

And some times i get error "POOLS" which maybe mean that pyasic can't get miner's pools

Some thing interesting. on first iteration pyasic find 30 miners (for example), on second iteration it find 50 miners which include some of first iteration miners and new miners. I really don't know why does it happen

This should be much more consistent as of 0.42.2, but if youre still having issues on that version you can set the settings timeouts with pyasic.settings.update(k, v).  Possible settings options are shown here - https://pyasic.readthedocs.io/en/latest/settings/settings/


Chat svc bug - chat disappears when refreshed

Might b a concern for persistence D:

@Teeesa7 fixed it!

#82 fixes


package Mac releases as DMG release

Would be nice to have a proper DMG release with the whole "drag to install" thingy.

related to #168, would make sense to work on both together as part of upgrading the build process (see #162)

https://www.christianengvall.se/dmg-installer-electron-app/
https://github.com/mongodb-js/electron-installer-dmg


Azure - MSSQL - Unable to switch DB

Dear All,
I am trying to use DBeaver to connect to the SQL Server Database on Azure.
DBeaver automatically downloaded the necessary drivers and the connection was successful, but the connection only connects to the master database. I am unable to switch to any of the other Databases. It either hangs or gives me a "Graphic is disposed" error message. I am also not able to use the fully qualified query option i.e. db.schema.tablename option. Getting an error message for that as well.
I already found a couple of similar issues in Github, all of them in closed state and none of them with any solutions. Only solution provided was to create a separate connection for each of the databases. I feel that would mean creating way too many connections.
Please note currently I am able to query using Azure Data Studio without any issues.
Kindly help.

Perhaps you need to upgrade to the most recent dbeaver version (problem with "Graphic disposed" was fixed in 7.0.3).
Can you specify target database name in connection settings (not master)?
Azure SQL Server doesn't support active database change as well as cross-database queries. It has nothing to do with dbeaver.

Hi @serge-rider , thank you very much for your response.
You are correct, I do not get the Graphic Disposed error anymore when I try to switch databases...but the databases do not change. But as you said it has to do with Azure DB not supporting and not particularly an issue with DBeaver. I wonder how Azure Data Studio and SSMS do it though..
Anyways this works for me for now as nothing beats the other features that DBeaver provides. Appreciate your work and the time taken to respond to my question. All the best (y)

Hello.
An update on this.
Best regards
Describe the problem you're observing:
I get a "Graphic is disposed error" when I try to alter connection settings.
Steps to reproduce, if exist:

Being on the Database Navigator, I right-click at the desired connection and I select "Connect".
I right-click again and I select "Edit Connection".
The -Connection "xxx" configuration- window appears with "Connection settings" selected.
I select "General" (beneath "Connection settings").
I select "Connection settings" again.
A window titled "Page switch" appears with the following Error:

Error switching active page
Reason:
Graphic is disposed

Whatever option I click on next "Connection settings", "General", Initialization" etc. the same error occurs and the right hand display does not change its contents.

Please note that in I don't experience the same problem with the application from Windows Store (CE 7.0.0).
System information:

OS Name: Microsoft Windows 10 Enterprise
OS Version: 10.0.17763 N/A Build 17763
DBeaver CE 7.0.3 (portable x64)
Additional extensions: N/A

Connection specification:

Adaptive Server Enterprise/12.5.4
com.sybase.jdbc4.jdbc.SybDriver
Do you use tunnels or proxies (SSH, SOCKS, etc)?:

Hello uslss. No, it's not. I performed the same actions using ce-7.1.3.zip and the problem does not appear anymore.
Regards

Hello.
I posted a replay/comment on this on GitHub. The problem is fixed. I am not yet familiar with how things work regarding replies on GitHub so, I am replying to this e-mail also, in case you didn’t receive a notification for my reply/comment due to misuse.
Regards
From: uslss<EMAIL_ADDRESS>Sent: Πέμπτη, 23 Ιουλίου 2020 12:44 πμ
To: dbeaver/dbeaver<EMAIL_ADDRESS>Cc: andreasnikiforosgeorgiadis<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [dbeaver/dbeaver] Azure - MSSQL - Unable to switch DB (#8477)
is it still actual?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/dbeaver/dbeaver/issues/8477#issuecomment-662712945 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AOQBLVQ2OYQQ3TQUDVL4OXDR45MSJANCNFSM4MLFKSLQ .  https://github.com/notifications/beacon/AOQBLVTNCKQWTLUPYUA5JKDR45MSJA5CNFSM4MLFKSL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOE6ADE4I.gif


CS:IP entry point calculation seems to be incorrect

According to: https://en.wikibooks.org/wiki/X86_Assembly/16,_32,_and_64_Bits

... to create a 20 bit address, it would be done by taking the 16-bit value of a segment register and put it on the address bus,
but shifted 4 times to the left (thus effectively multiplying the register by 16), and then by adding the offset from another
register untouched to the value on the bus, thus creating a full a 20-bit address.

If CS = 258C and IP = 001216, then CS:IP will point to a 20 bit address equivalent to "CS × 16 + IP" which will be
258C × 1016 + 001216 = 258C0 + 001216 = 258D2 (Remember: 16 decimal = 1016).

However the implementation is: https://github.com/python-intelhex/intelhex/blob/master/intelhex/scripts/hexinfo.py
        if keys == ['CS','IP']:
            entry = ih.start_addr['CS'] * 65536 + ih.start_addr['IP']

Is it not an implementation mistake?

Yes, this is error.

fixed, thanks.


Popup remove event triggered twice before actually opening the popup

Hello, thank you for the great work over this package.
The bug is, the very first time I click on a popup, the handlePopupClose function is fired two times before actually opening the popup. It is then properly fired when closing the popup.
The problem stops after the first popup.
import MarkerClusterGroup from 'react-leaflet-cluster'

<MarkerClusterGroup>
  <Marker>
    <Popup eventHandlers={{ remove: handlePopupClose }}>
      // ...
    </Popup>
  </Marker>
</MarkerClusterGroup>

leaflet 1.9.3
leaflet.markercluster 1.5.3
react 18.2.0
react-leaflet 4.2.1
react-leaflet-cluster 2.1.0
The bug disappear when I delete <MarkerClusterGroup>
Do you know any workaround to this problem ?
Any idea how you can patch this ?
Again thanks for the great work with this package

@ndevplanque Could you create a sandbox demo for me?


[i2c,dv] Add write-check csr_excl for FMTFIFO/TXFIFO registers

Fixes #22853

CHANGE AUTHORIZED: hw/ip/i2c/data/i2c.hjson

CHANGE AUTHORIZED: hw/ip/i2c/data/i2c.hjson

Failing CW310 rom test is unrelated to this DV-only change, hence merging.


non float: left on col-*-12 cause overlap of elements when the number of columns in a row add up to > 12

Example:
``
.col-xs-6 .col-md-4
.col-xs-6 .col-md-4
.col-xs-6 .col-md-4
``
see: http://stackoverflow.com/questions/18428450/bootstrap-3-layout-with-panel-display-not-correct-on-small-device/18433782

@kmukku : Yes and that causes it to overlap.
To fix it, all i did was to add .col-sm-12, .col-md-12, .col-lg-12, .col-xs-12{ float: left; }
Not sure but it seem to globally fix all my issues


Filtering for Things

Implement a support of filtering for Things (items in /things/ REST API endpoint).
Parameter-based filtering is proposed. To filter all things by their placement, the following request will be able to be used:
curl --request GET \
  --url 'http://dpl_host:dpl_port/things/?placement=R1' \
  --header 'Authorization: your_auth_token_here'

In support of parameter-based filtering approach, see the following descriptions and mentions:

in GitHub documentation
in "Web API Design" e-book
on Hackernoon: article
and also here and here

But parameter-based approach have its own issues. For example:

collisions are possible if objects in filtered collection have fields called like page, fields, sort, direction and at the same time those parameters are used for service tasks like pagination, field selection, result sorting and so on;
it's unclear what to do with requests which have a parameter that doesn't correspond neither to a set of service parameters or to field names of objects;
some approach is needed to address two mentioned issues;
one of those approaches is a usage of whitelist of allowed filters (like in django), but whitelist checking will affect an overall request processing speed.

From this perspective, it's much easier to parse exactly one filter statement specified in single filter parameter (like in http://docs.oasis-open.org/odata/odata/v4.0/errata03/os/complete/part1-protocol/odata-v4.0-errata03-os-part1-protocol-complete.html#The$filter_System) than to parse all parameters and to build some filtering expressions based on them.

Base support implemented in https://github.com/s-kostyuk/adpl/pull/20. If you need any additional filtering by other fields, updated_since filtering, pagination and complex search - please, let me know by endorsing of this comment and/or by creating of a comment in this issue.


Use only ascii chars in log statements

See https://discuss.elastic.co/t/invalid-json-somewhere-in-pipeline/174820


Markdown roundtrip

Markdown posted on Matrix gets posted on Gitter fine, but markdown in Gitter shows up as text in Matrix. I must have missed something in the message format.

Apparently the sender is expected to render markdown into HTML and add the undocumented keys "format": "org.matrix.custom.html", "formatted_body": "html_here" to the message.

markdown2 seems like a good choice.
markdown2.Markdown(
    extras=['fenced-code-blocks', 'footnotes', 'link-patterns', 'wiki-tables'],
    link_patterns=link_patterns).convert(...)

Implemented by ad665119.

Looks like markdown2 has some serious bug and does correctly format GitHub Flavored Markdown: trentm/python-markdown2#226

New option: Markdown with pymdown-extensions.
import markdown
markdown.markdown(...', ['pymdownx.github'])

Implemented by 644039f3.


Support for MALMEN

MALMEN: Massive Language Model Editing Network https://arxiv.org/abs/2311.04661 is the newest SOTA on Meta Learning method. In their experiments, it has the excellent performance with massive editing even better than MEMIT in certain aspects. I think it is a combination of MEND and MEMIT. May you add support for this method? Thank you! https://github.com/ChenmienTan/malmen

We will support MALMEN soon, thank you for your suggestions.

Hi, the support for MALMEN is performing internal testing of the code and aiming for release within a month.

I apologize for the delay. Currently, Easyedit has implemented partial support for the Malmen method, but due to framework compatibility issues, there are some performance compromises in certain functions. We will continue to improve support in the first half of the year. Thank you for your patience.

We sincerely apologize for the inconvenience. Due to certain compatibility issues between our framework and the original MALMEN code, we can only provide partial support at this time. We’ve found that performance is relatively stable when the batch size is set to 1, but when the batch size is greater than 1, there is a noticeable performance drop. Despite multiple attempts, we believe the issue may be related to our data loading process. However, given the complexity of modifying the framework and our resource limitations, we’ve decided not to pursue further optimization in this area for the time being.
We greatly appreciate your attention to this issue and your valuable feedback. If you have any better solutions or ideas, we warmly welcome you to submit a PR to our GitHub repository. We would be delighted to see contributions from the community to help improve the quality of the project.


Tags on Saved Queries don't work after upgrading to Superset 3.0.0

Adding tags to saved queries does not seem to be possible from the frontend UI. When creating Charts or Dashboards you have the option of adding Tags!
This is very useful for organising elements into logic groups.
Under the Saved Queries tab you can clearly see a column & search bar dedicated to Tags on Saved Queries, however there is no way to add these queries from the Frontend.

In a discussion on Slack a suggestion was to use Bulk Select which would then expose the Add Tags option, however no such option becomes available.

When adding the tag manually directly in the database it does render to the frontend

Expected results
Something similar to Charts but the ability at add Tags to Saved Queries.

Environment
(please complete the following information):

browser type and version: Chrome Version 116.0.5845.187 (Official Build) (arm64)
superset version: Version: 3.0.0rc2
python version: python 3.9
FEATURE_FLAGS = {
'ALERT_REPORTS': True,
'ENABLE_TEMPLATE_PROCESSING': True,
'ALLOW_FULL_CSV_EXPORT': True,
'DASHBOARD_RBAC': True,
'TAGGING_SYSTEM': True,
'HORIZONTAL_FILTER_BAR': True,
}

Any suggestions?

I think this feature was more fully fleshed out with the 3.1 release, where 3.0 was still fairly "beta" in nature. @yousoph do you know the current status? If it's all good in 3.1, I think we can close the 3.0 complaint as resolved since we won't be backporting features (just fixes).

Hello! We are running image tag 4.0.2-dev and still can not see this in the UI


Slapper Issue on your Latest update?

[05:01:26] [Server thread/CRITICAL]: TypeError: "Argument 1 passed to PiggyAuth\Main::isAuthenticated() must be an instance of pocketmine\Player, instance of slapper\entities\SlapperHuman given, called in phar://C:/Users/Administrator/Desktop/RP/plugins/PiggyAuth.phar/src/PiggyAuth/EventListener.php on line 195" (EXCEPTION) in "/PiggyAuth.phar/src/PiggyAuth/Main" at line 148
[05:01:26] [Server thread/CRITICAL]: "Could not pass event 'pocketmine\event\player\PlayerExperienceChangeEvent' to 'PiggyAuth v2.0.1': Argument 1 passed to PiggyAuth\Main::isAuthenticated() must be an instance of pocketmine\Player, instance of slapper\entities\SlapperHuman given, called in phar://C:/Users/Administrator/Desktop/RP/plugins/PiggyAuth.phar/src/PiggyAuth/EventListener.php on line 195 on PiggyAuth\EventListener

this only occurs on server startup, btw. Ummm /reload doesn't trigger the error.

[07:50:18] [Server thread/CRITICAL]: "Could not pass event 'pocketmine\event\player\PlayerExperienceChangeEvent' to 'PiggyAuth v2.0.1': Argument 1 passed to PiggyAuth\Main::isAuthenticated() must be an instance of pocketmine\Player, instance of slapper\entities\SlapperHuman given, called in phar://C:/Users/Administrator/Desktop/RP/plugins/PiggyAuth.phar/src/PiggyAuth/EventListener.php on line 195 on PiggyAuth\EventListener
[07:50:18] [Server thread/CRITICAL]: TypeError: "Argument 1 passed to PiggyAuth\Main::isAuthenticated() must be an instance of pocketmine\Player, instance of slapper\entities\SlapperHuman given, called in phar://C:/Users/Administrator/Desktop/RP/plugins/PiggyAuth.phar/src/PiggyAuth/EventListener.php on line 195" (EXCEPTION) in "/PiggyAuth.phar/src/PiggyAuth/Main" at line 148

Fixed.


k8s: check and possibly optimise the launch of pending pods

There was a situation in a cluster running many concurrent workflows, which generated many jobs, that many jobs were pending, because the cluster did not have enough memory resources to run them all.
For example, here is one snapshot in time:
$ kgp | grep reana-run-j | grep -c Running
110

$ kgp | grep reana-run-j | grep -c Pending
71

This means that only 60% of jobs could be running, the remaining 40% were pending. (Some for many hours.)
Some nodes were really busy, for example:
$ kubectl top nodes -l reana.io/system=runtimejobs
NAME                CPU(cores)   CPU%   MEMORY(bytes)   MEMORY%
mycluster-node-12   4175m        52%    10308Mi         73%

$ kgp | grep node-12
reana-run-job-bda3d5d0-23da-494d-9973-5992707fed3f-hg8js          1/1     Running     0          11h     <IP_ADDRESS>    mycluster-node-12   <none>           <none>
reana-run-job-cd68d874-5bf5-44ba-a9bf-d4425ed5f466-prslv          1/1     Running     0          4h      <IP_ADDRESS>    mycluster-node-12   <none>           <none>
reana-run-job-ea877fa3-e346-4e3b-92d0-d7dabf2ce66b-sm695          1/1     Running     0          15h     <IP_ADDRESS>    mycluster-node-12   <none>           <none>
reana-run-job-fdac7a57-7e53-434b-a7e9-0231803bbcfa-8gnd2          1/1     Running     0          11h     <IP_ADDRESS>    mycluster-node-12   <none>           <none>

However, other nodes were less so, for example:
$ kubectl top nodes -l reana.io/system=runtimejobs
NAME                CPU(cores)   CPU%   MEMORY(bytes)   MEMORY%
mycluster-node-33   3156m        39%    8596Mi          61%
mycluster-node-35   2073m        25%    5263Mi          37%

$ kgp | grep node-33
reana-run-job-4f95d722-7a41-456c-9437-4c7842f416aa-fh428          1/1     Running       0          21m     <IP_ADDRESS>     mycluster-node-33   <none>           <none>
reana-run-job-751198a7-3596-473a-bb29-7ab9e97456ad-2jscw          1/1     Running       0          63m     <IP_ADDRESS>     mycluster-node-33   <none>           <none>
reana-run-job-9ab284e3-2e9e-47f2-be86-37664109fcb4-p2z6p          1/1     Running       0          63m     <IP_ADDRESS>    mycluster-node-33   <none>           <none>

$ kgp | grep node-35
reana-run-job-9659c2d7-403a-4392-8b7b-1d1525be3bee-lh9bq          1/1     Running     0          5m13s   <IP_ADDRESS>    mycluster-node-35   <none>           <none>
reana-run-job-9c71d885-3e6c-423d-a4ad-d3da2e9df5d8-rb69x          1/1     Running     0          3m42s   <IP_ADDRESS>    mycluster-node-35   <none>           <none>

It seems that our pending pods aren't consumed as rapidly as they in theory could (e.g. the node-33 and node-35 above had free capacity).
Here is one such Pending pod described:
$ kubectl describe pod reana-run-job-ea984ce5-6f04-4b92-893e-6d271d2a5454-22gnp | tail -7
                 node.kubernetes.io/unreachable:NoExecute op=Exists for 300s
Events:
  Type     Reason            Age    From               Message
  ----     ------            ----   ----               -------
  Warning  FailedScheduling  81m    default-scheduler  0/62 nodes are available: 1 node(s) had taint {node-role.kubernetes.io/master: }, that the pod didn't tolerate, 14 node(s) didn't match Pod's node affinity/selector, 39 Insufficient memory, 8 node(s) were unschedulable.
  Warning  FailedScheduling  11m    default-scheduler  0/62 nodes are available: 1 node(s) had taint {node-role.kubernetes.io/master: }, that the pod didn't tolerate, 1 node(s) had taint {node.kubernetes.io/disk-pressure: }, that the pod didn't tolerate, 14 node(s) didn't match Pod's node affinity/selector, 38 Insufficient memory, 8 node(s) were unschedulable.
  Warning  FailedScheduling  5m44s  default-scheduler  0/62 nodes are available: 1 node(s) had taint {node-role.kubernetes.io/master: }, that the pod didn't tolerate, 14 node(s) didn't match Pod's node affinity/selector, 39 Insufficient memory, 8 node(s) were unschedulable.

Let's verify our Kubernetes cluster settings related to the behaviour of Pending pods and let's see whether we could make the memory-checks and the scheduling of these pending pods faster.

More information about situations of this kind.
Here is a node which appears memory-busy:
$ kubectl top nodes | grep 'node-3 '
mycluster-node-3     4521m        56%    10261Mi         73%

and it is running four user jobs:
$ kgp | grep 'node-3 '
reana-run-job-232b1162-c04e-46d6-9018-843ee84a072c-g5mzz          2/2     Running     0          14h     <IP_ADDRESS>    mycluster-node-3    <none>           <none>
reana-run-job-6a86b3ae-c0a1-4b14-ae53-e8b8fb15d97a-5thvb          2/2     Running     0          4h6m    <IP_ADDRESS>    mycluster-node-3    <none>           <none>
reana-run-job-a3137e4c-48c7-4d3a-a073-335b18761231-bjpns          2/2     Running     0          4m5s    <IP_ADDRESS>    mycluster-node-3    <none>           <none>
reana-run-job-ee000c25-32f1-4375-900c-e4ae8a9fb30c-wrmxt          2/2     Running     0          11m     <IP_ADDRESS>    mycluster-node-3    <none>           <none>

Each job requires 3Gi memory, i.e. about 12Gi total, but the jobs actually use much less, about 5Gi only:
$ kgp | grep 'node-3 ' | awk '{print $1}' | xargs -n1 kubectl top pod --no-headers
reana-run-job-232b1162-c04e-46d6-9018-843ee84a072c-g5mzz   1012m   1300Mi
reana-run-job-6a86b3ae-c0a1-4b14-ae53-e8b8fb15d97a-5thvb   965m   1239Mi
reana-run-job-a3137e4c-48c7-4d3a-a073-335b18761231-bjpns   941m   1167Mi
reana-run-job-ee000c25-32f1-4375-900c-e4ae8a9fb30c-wrmxt   956m   1114Mi

This is because the jobs are by default requiring 3 Gi if the user does not specify any other value.
If, instead of silently adding 3 Gi memory requirement to each job, we let the jobs consume as much memory as they wish, and have a parallel "memory watcher" daemonset on the nodes that would monitor and kill any user job pods if these start to consume a lot of memory, we would be able to pack twice as many jobs on nodes as we do now, in this very example.
(And, if a user does require some 4 Gi, we would simply respect that. This would only change the default behaviour. when a user does not ask for any specific memory limit)

Sorry to jump in!
Average memory and (max)PeakRSS metric monitoring might help to define min request.memory and max limits.memory. Because, in the node-3 example, let's assume this scenario (I don't know this can be the worst-case scenario or common case):

Minimum default memory defined like 2Gi and one job is started to run with 2Gi memory usage
Node's total memory usage becomes (10+2)12Gi and its max resource is 14Gi
One of the previous jobs "momentarily" requires to use 2+Gi memory additional to its ~1Gi memory usage
So, it will be killed/restarted because there is not enough memory


Make sure all tests pull from current block (or a few back) vs some really old one (pre london)

Updated all tests to pull from more recent block


update parent pom to 4.43.1

4.43.1 contains htmi-unit 2.63 with a fix for flaky PCT test
org.jenkinsci.plugins.workflow.libs.FolderLibrariesTest.checkDefaultVersionRestricted

[ ] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[ ] Ensure that the pull request title represents the desired changelog entry
[ ] Please describe what you did
[ ] Link to relevant issues in GitHub or Jira
[ ] Link to relevant pull requests, esp. upstream and downstream changes
[ ] Ensure you have provided tests - that demonstrates feature works or fixes the issue

java.lang.IllegalArgumentException: ref can not be empty
	at jenkins.plugins.git.traits.DiscoverOtherRefsTrait.<init>(DiscoverOtherRefsTrait.java:52)

weird


packaging: bump to current node.js

Node 14 is in maintenance LTS releases, the active releases are on the Node 16 release path.  Node seems to indicate that Node 16 should be preferred.  This updates the target to use node 16 rather than node 14.  This behavioural chance also now matches the behaviour of the VSCode extension when it resided in SourceKit-LSP.

I think I added that flag because VS Code generated the extension to target 14 by default. If we bump this, we should probably also bump the version in package.json and verify everything still works.


quantize.cpp:58 input->type == kTfLiteFloat32 || input->type == kTfLiteInt16 || inp was not true

@tensorflow/micro
System information

Host OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04
TensorFlow installed from (source or binary): binary
Tensorflow version (commit SHA if source): 2.5.0
Target platform (e.g. Arm Mbed OS, Arduino Nano 33 etc.): Arduino Nano 33 BLE Sense

Describe the problem
I have converted our CNN to TFLite using full integer quantization (in and out: uint8)
AllocateTensors fails with quantize.cpp:58 input->type == kTfLiteFloat32 || input->type == kTfLiteInt16 || inp was not true

@AdamMiltonBarker ,
This issue is more suitable for TensorFlow Micro repo. Please post it on tflite/micro repo from here. Thanks!

Did do thanks.


🌱Owners list synced with main

What this PR does / why we need it:
This PR is syncing owner list with main branch.

/lgtm

Merging it since it is just a syncing of OWNERS files.
/retitle 📖 Synchronize OWNERS file with main branch
/test-v1a5-ubuntu-integration
/approve
/lgtm


Missing Node.js stream.Readable.from

Missing/Incorrect APIs
Node.js 12.3.0 introduced stream.Readable.from method. Flow's Node.js definitions do not recognize this method.
Relevant documentation
https://nodejs.org/api/stream.html#stream_stream_readable_from_iterable_options

Actually this is also part of node v10, I don't know the latest doc talks about node v12.3. My guess is that it appeared both in v10 and v12 at roughly the same time.


[#55190] Fix Tab Nav vertical spacing in Page Header

https://community.openproject.org/work_packages/55190
What are you trying to accomplish?
Fix Tab Nav vertical spacing in Page Header when deployed
Screenshots
Before:

After:

Risk Assessment

[x] Low risk the change is small, highly observable, and easily rolled back.
[ ] Medium risk changes that are isolated, reduced in scope or could impact few users. The change will not impact library availability.
[ ] High risk changes are those that could impact customers and SLOs, low or no test coverage, low observability, or slow to rollback.

Merge checklist

[ ] Added/updated tests
[ ] Added/updated documentation
[ ] Added/updated previews (Lookbook)
[x] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge

Hi @HDinger, I pushed a new update and wrote some info about my implementation choices

The reason I originally removed margin bottom from .tabnav was the 2x1rem margin-bottom in the end; From the PageHeader and from the TabNav. Now I keep the original margin bottom in .tabnav and removed it from the .PageHeader when TabNav is rendered.
I also renamed the modifier --noBorder to --withTabNav to give it more context as it does more than just removing the bottom border now.
I could not use gap as the vertical spacing of the TabNav is larger than the other spacings. To solve that I used the .PageHeader--withTabNav for the .PageHeader-description; There is only a margin-bottom on the description when TabNav renders.

.PageHeader--withTabNav .PageHeader-description {
  margin-bottom: var(--space-xlarge);
}


chore(nodebuilder/header): Update trusting period to two weeks

Updates trusting period to two weeks instead of one week for celestia-node
Closes #2610

Closing in favour of bumping go-header


feat: BoundedSMul for matrix norms

This provides the fact that the norm of scaling a matrix of quaternions by a quaternion is bounded by the product of the norms.

[x] depends on: #9796

Can you add this for the operator norm (from EuclideanSpace) too? It's in Analysis.NormedSpace.Star.Matrix

Can you add this for the operator norm (from EuclideanSpace) too? It's in Analysis.NormedSpace.Star.Matrix

There's not really any point, because that uses [IsROrC 𝕜] and so there is no case where BoundedSMul is true and NormedSpace isn't.

Ok, I guess it would be statable, but we're missing the corresponding instances on the operator norm on continuous linear maps:
theorem instL2OpNBoundedSmul [SeminormedRing R] [Module R 𝕜] [SMulCommClass 𝕜 R 𝕜]
    [BoundedSMul R 𝕜] : BoundedSMul R (Matrix m n 𝕜) :=
  .of_norm_smul_le fun r x => by
    rw [l2_op_norm_def]
    have := norm_smul_le r ((toEuclideanLin (𝕜 := 𝕜) (m := m) (n := n)).trans toContinuousLinearMap x) -- fails
    sorry

Thanks. I'll add the bounded smul myself for continuous linear maps if I remember.
bors merge

bors merge
(the failure was a caching issue)


ll40ls nsh reading issue

Hey, I'm using a Lidar Lite v3 but when I try to test this(ll40ls test i2c), nsh displays:
ll40ls: inmediate read failed: try again.
I tried a lot of times and it doesn't work.
Can you help me? Please.
THANKS.
#UPDATE 1: When I try using "ll40ls info i2c" there is a distance parameter which displays the measured distance (I tried in two different positions). But, this "distance" doesn't change if I change the position of the lidar (to measure other distance), when I try with "ll40ls info i2c" the display for "distance" is the same.
HELP! and thanks again.

http://discuss.px4.io/t/poshold-with-optical-flow-works-weird/2432/9
This has a few tips. It sounds like a setup / wiring issue.


[Feature]: Add React Native Course

Related Issue
Fixes #3754
Description
[Feature]: Add React Native Course
Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Security enhancement
[ ] Other (specify): _______________

Screenshots / Videos (if applicable)
Screencast from 2024-07-22 23-45-46.webm
Checklist

[x] I have performed a self-review of my code.
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have followed the code style guidelines of this project.
[x] I have checked for any existing open issues that my pull request may address.
[ ] I have ensured that my changes do not break any existing functionality.
[ ] Each contributor is allowed to create a maximum of 4 issues per day. This helps us manage and address issues efficiently.
[ ] I have read the resources for guidance listed below.
[ ] I have followed security best practices in my code changes.

Resources for Guidance
Please read the following resources before submitting your contribution:

[x] Code Harbor Hub Community Features
[ ] Docusaurus Documentation
[ ] React.js Documentation
[ ] Markdown Guide
[x] MDX Documentation
[ ] Mermaid Documentation

Resources for DSA Features on Our Site:

[x] Tabs: Switch Languages
[ ] Line Numbering
[ ] Math Equations

Contribution Guidelines
Thank you for considering contributing to our project! To ensure smooth collaboration and effective contribution management, please adhere to the following guidelines:
Issue Creation

Limit on Issues:

Each contributor is allowed to create a maximum of 4 issues per day. This helps us manage and address issues efficiently.

Contribution Levels

Basic Contributions:

This project is primarily focused on documentation. Most of the setup has been completed, so contributors will generally need to work on basic code tasks, such as writing tests.
For these tasks, issues will be assigned the level1 label.

Acknowledging Hard Work:

If a contributor puts in significant effort on a task, the issue will be upgraded to level2. This is our way of recognizing and appreciating extra effort.

Feature Additions and Component Work:

Contributors working on new features or components using JSX/TSX will be assigned a level based on the complexity and quality of their work.
The more complex and valuable the contribution, the higher the level assigned.

Level Definitions

Level1:

Tasks are straightforward, such as fixing minor bugs, writing tests, or making simple documentation updates.

Level2:

Tasks require more effort, such as addressing complex bugs, improving existing features, or making substantial documentation improvements.

Level3:

Tasks are highly complex and involve significant new feature development, major refactoring, or extensive contributions to the project’s core components.

We look forward to your contributions and appreciate your effort in helping us improve the project!

Hello @Ajay-Dhangar  please merge this..

Hello @Ajay-Dhangar please merge this,
I have a one doubt, The Lighthouse Report can more time to take load the details. If there any other option to optimize the time.

Hello @Ajay-Dhangar please merge this...


Enter Improve the text at the top of the home page.

Received the latest home page text from the product team

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: heromcp sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

test


PoC for enhanced Flow support in the monorepo

Link: https://github.com/fusionjs/fusionjs/pull/472:

Closing this as it is not intended to land.  Proof-of-concept may be referenced or used in a future pull request.


Added support for base paths defined by server definitions.

Base paths defined by server definitions are now included when performing a path lookup.

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (fafe15e) 100.00% compared to head (423f0ec) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main        #6   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           18        18
  Lines          930       946   +16
=========================================
+ Hits           930       946   +16

Flag
Coverage Δ

unittests
100.00% <100.00%> (ø)

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

Impacted Files
Coverage Δ

paths/paths.go
100.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


这都周五早上9点了，咋还没更新

持续集成行不行

先手动触发更新了。
根据文档https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#schedule，

Note: The schedule event can be delayed during periods of high loads of GitHub Actions workflow runs. High load times include the start of every hour. To decrease the chance of delay, schedule your workflow to run at a different time of the hour.

GitHub Action具体的调度执行时间，受到系统负载影响，不保证完全准时，另外建议不要放到整点。
我大致看了下，阮老师更新文章的时间大概是北京时间早晨八点钟，晚点我修改下调度策略。
感谢提醒！


Train on LIDC and test on LUNA

Hello, as I read your paper, I understand that you trained your model on a pool of dataset including LIDC, then you test on LUNA16. I have two questions for this:
1/ The figure 2 in the paper means you use all 888 images of LUNA as the test set? If not how did you have this result?
2/ LUNA16 is a subset of LIDC, even though you explained on the paper but I still doubt that two datasets are mostly overlapped, could you explain a bit more on this?
Thanks for your help.

Hi @vankhoa21991 ,
thanks for your question. In our setting, there is a difference between a test set and the test pool since the development of nnDetection contains a "meta optimization" process. During this process, we developed a set of rules which deduce a set of rules to adapt to unseen data sets / data sets with different Data Fingerprint.
Data sets that are part of the validation pool have a separated test set, i.e. we keep a certain amount of the data completely separated from the development process of nnDetection and only evaluated this subset for our final reported results. This type of evaluation is commonly found and measures the generalization of the trained model to unseen data.
The test pool contains data sets that were not part of the development process of nnDetection but have a different Data Fingerprint. These data sets are meant to measure the generalization of our developed rules to new data sets.
As we explain in our paper, while the LUNA16 data is a subset of LIDC it has different scan selection criteria and different annotations. These result in different Data Fingerprints and also different training/inference configurations. Please note that the models trained on LIDC are not evaluated on LUNA16, all models which were evaluated on LUNA16 were only trained on LUA16 data (see exact evaluation protocol below).
LUNA16 protocol: According to the official guidelines, LUNA16 should be trained in a 10 Fold cross-validation fashion, i.e. there are 9 subsets to train and 1 subset to predict. The subset to predict changes with every fold, so after training 10 times, the whole data set was predicted at some point.
Best,
Michael


"Zero bytes" file after transferring.

The file transfer goes fine from my phone (P7 Pro) to my computer (M1 Mac) but the file that remains is practically nothing, "zero bytes". Any fix or is it something that I'm doing wrong? Thanks.

I'm having the same issue and I don't get how to fix it. Tried in more ways, but nothing


CI is failing on main since Nov/Dec 2022

CI has been failing for quite some time, and I have no experience with purescript.
@aimorris @emiel would either of you have a moment to see if there's an easy fix here?
There are quite a few open PRs that we probably should pull in, if we can get CI passing.
/cc @ErikSchierboom for visibility

Hi all.
I did a little debugging and found the root cause. We are still using PureScript 0.14.x (0.15.x is the latest). There was an issue with 0.14.7 having an incorrect dependency on libtinfo (see https://github.com/purescript/purescript/issues/4253).
If you invoke purs --version on CI you'll see:
purs: error while loading shared libraries: libtinfo.so.5: cannot open shared object file: No such file or directory

To resolve we can bump PureScript to 0.14.9 which appears to work. We usually bump the PureScript in tandem with purescript-runner. Another option is upgrade to the latest (0.15.x) but that requires much more work because the exercises need to be adapted as well.
I can create a couple of PRs for this if needed. Let me know.

See:

https://github.com/exercism/purescript-test-runner/pull/44
https://github.com/exercism/purescript/pull/263

That was quick! Thank you so much.
Yeah, I think bumping purescript to 0.14.9 is the way to go here. If someone wants to take on the bigger task of bumping up to the next minor version, that's great, but it doesn't need to happen right this second.
I've reopened the PR that bumps the patch version, and will merge it as soon as CI reports back (it reruns CI when the PR is reopened, it looks like).


Not working Vuex Export

When i push button Export in Vuex section vue-devtools print "Copied to clipboard!" but clipboard is empty.
vue-devtools version 3.1.2
vue version 2.2.3
vuex version 2.2.1

Hey, I tested and it's working fine with the last version of the devtools. Can you provide a repro using this template?
Thanks!

It does not depend on the code

Well, in that case I can't do anything for you... In case you want to look further, this is the piece of code that does the copy: https://github.com/vuejs/vue-devtools/blob/master/src/devtools/views/vuex/VuexStateInspector.vue#L111. It should work on your env


Japanese Gen 1 and Promos

The YAML contains all the meta information of the cards.
The other TXT document is a helpful clarifying aid to disambiguate which Promo is which. It tells what Promotion it is from, for example CoroCoro Comic April 1998 issue insert.  Also included in it are the bulbapedia links to the cards/attacks/artwork themselves

Changes were merged into the main repo.


ENH: NAV-52 - First throw at defining interfaces for a public transport interface.

Proposal for a general interface for the Public Transport Service Interface

Please see #21


Installation using pip 19.3.1 using git+ fails

./bin/pip install 'git+https://github.com/dephell/dephell works, but the same doesnt work here 'git+https://github.com/dephell/dephell_archive' because it doesnt have a setup.py/.cfg/etc.  I appreciate that you likely dont want to create these files for all repos, but maybe there is a workaround, or possibly a new dephell tool?.
./bin/pip install 'git+https://github.com/dephell/dephell_archive'
Collecting git+https://github.com/dephell/dephell_archive
  Cloning https://github.com/dephell/dephell_archive to /tmp/pip-req-build-ozd9kiqg
  Running command git clone -q https://github.com/dephell/dephell_archive /tmp/pip-req-build-ozd9kiqg
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
    Preparing wheel metadata ... done
Building wheels for collected packages: UNKNOWN
  Building wheel for UNKNOWN (PEP 517) ... done
  Created wheel for UNKNOWN: filename=UNKNOWN-0.0.0-cp37-none-any.whl size=1788 sha256=2fbed720ce0681e2e09b4da90a395e31ee224dd3ad086344d4fd1b83b12fa6f0
  Stored in directory: /tmp/pip-ephem-wheel-cache-mfoqfw4o/wheels/1d/e9/34/a547e91f29439fb316449bcc47f1738c9264b2706e352fd0d9
Successfully built UNKNOWN
Installing collected packages: UNKNOWN
Successfully installed UNKNOWN-0.0.0

lib/python3.7/site-packages/UNKNOWN-0.0.0.dist-info/top_level.txt is empty, so nothing was installed.
Likely there is an upstream issue in pip project about it, worth finding and tracking.

I will have an update for this in a few days.
(related to https://github.com/dephell/dephell_specifier/issues/13, https://github.com/dephell/dephell_pythons/pull/8 and https://github.com/dephell/dephell_shells/pull/3)

I'm going to make PEP-518 interface for DeoHell, it will solve such cases

@orsinium , any progress with the PEP-518 interface?  Is it an existing issue/PR for that?
I guess this could be solved by adding adding a poetry PEP-517 handler into pyproject.toml for each @dephell project, like
[build-system]
requires = ["poetry>=0.12"]
build-backend = "poetry.masonry.api"

fwiw, I have built https://github.com/jayvdb/pypidb to make it possible to generically build tools that bridge the gap between PyPI name and SCM.


WIP: Arcade ibc training

The failure makes sense. We need to condition doing this to building with Full Framework msbuild.


Debug SMS service

Actual:

Sending SMS more than once, doesn't seem to be communicating that it sent

Expected:

One SMS pls

Fixed - was a deployment issue


Sub name / search / sort bar cut off when opening post

Top bar is cut off when opening post. Scrolling down so it disappears then back up again fixes it. iPad Pro 9.7 12 DB6

Fixed!


WebTorrent not support for Buffer on browser

WebTorrent version: 0.85.1
Node.js version: v4.2.2
Browser name/version (if using WebTorrent in the browser): Chrome/49.0.2623.87

using webpack as a bundler.
Because it do not support require('webtorrent') with webpack(see https://github.com/feross/webtorrent/issues/347 ),i use require('webtorrent/webtorrent.min') in the browser.
But it return undefined when webtorrent.get(new Buffer(...)).
It seen  that webtorrent.min cannot parse a Buffer Object.

The problem is that create-torrent currently expects buffers and streams to have a name. While its not common you can just attach the name property to the buffer object. Perhaps in an upcoming version this will be obsolete.
Aside of using the static webtorrent.min built you can checkout the configurations which have to be set to enable the webtorrent package to work in webpack.

@Autarc The create-torrent issue seems unrelated. @jimliang is discussing the client.get function, which is used to look up an existing torrent.
@jimliang Please share a complete code example. It sounds like you're using the wrong function. client.add allows you to download a new torrent, not client.get.

html:
<script src="webtorrent.min.js"></script>
<script src="bundle.js"></script>

bundle.js is created by webpack.Source:
const client = new WebTorrent();
client.add(new Buffer(...),(torrent)=>{});

Throwing Invalid torrent identifier(…) on the browser.
Now I use  browserify instead of webpack.This issue can close. :3


Having trouble integrating MediaPlayer into an NX Monorepo React project

Discussed in https://github.com/vidstack/player/discussions/1075

Originally posted by balazs-karsai January  8, 2024
I'm trying to replace Plyr with vidstack player on an NX Monorepo in a React app.
When I import and add MediaPlayer component, I get the following Typescript error:
Type instantiation is excessively deep and possibly infinite
I can dismiss the error with @ts-ignore, but I would like to solve it instead of just ignoring.
Also, while the app and video playing works, my Jest tests fail with the following error and it might be related to the issue above:
 ● Test suite failed to run     Jest encountered an unexpected token     Jest failed to parse a file. This happens e.g. when your code or its dependencies use non-standard JavaScript syntax, or when Jest is not configured to support such syntax.     Out of the box Jest supports Babel, which will be used to transform your files into valid JS based on your Babel configuration.     By default "node_modules" folder is ignored by transformers.     Here's what you can do:      • If you are trying to use ECMAScript Modules, see https://jestjs.io/docs/ecmascript-modules for how to enable it.      • If you are trying to use TypeScript, see https://jestjs.io/docs/getting-started#using-typescript      • To have some of your "node_modules" files transformed, you can specify a custom "transformIgnorePatterns" in your config.      • If you need a custom transformation specify a "transform" option in your config.      • If you simply want to mock your non-JS modules (e.g. binary assets) you can stub them out with the "moduleNameMapper" config option.     You'll find more details and examples of these config options in the docs:     https://jestjs.io/docs/configuration     For information about custom transformations, see:     https://jestjs.io/docs/code-transformation     Details:     \node_modules\@vidstack\react\dist\prod\vidstack.js:3     import { D as DOMEvent, f as isString, l as listenEvent, A as createReactComponent, u as useStateContext, B as useReactContext, a as useSignal, C as composeRefs, b as useSignalRecord, e as effect, E as useReactScope, s as signal, x as scoped } from './chunks/vidstack-8AXyyPGc.js';     ^^^^^^     SyntaxError: Cannot use import statement outside a module
I tried to figure out the cause of the issue here but I couldn't find any solution. It seems like some Jest config issue, but I haven't changed anything, only added @vidstack-react as a dependency.
We are using
Node 18.19.0
Typescript 5.3.3
Jest 29.4.1
React 18.2.0

Thanks for reporting. No need to cross-post in the future, either open an issue or a discussion but not both. In this case it seems to be an issue. If you can provide a repro so we can test the issue that would be great.

Closing until a repro is provided.

videoplayer_bug.zip
Here I attached a sample package that reproduces the issue.
Please unpack the archive, than issue:
yarn install
yarn test-all
This should produce the above error. Also, in the Root.tsx file, there is a @ts-ignore annnotation to suppress another error: Type instantiation is excessively deep and possibly infinite. These two errors maybe related.
I used yarn 1.22.19 if that matters.
Please let me know if you need anything else.

Closing until a repro is provided.

Can I reopen the issue or create a new one?

Unfortunately no luck reproducing this issue. I created a fresh NX monorepro project and found no issues. Try updating your package lock and version of TypeScript.

I still encounter the above error, even when updating to @vidstack/react 1.10.9
Have you run the tests for the example I provided? Because the code compiles, but tests fail with import errors in Vidstack.

I also tried to add
transformIgnorePatterns: ['/node_modules/(?!@vidstack)'],
into jest config, because it seems that some js files in Vidstack are not transpiled, but this config option only raised further errors.

I think the main problem here is that Vidstack is using ESM modules that work fine with Vitest, but not with Jest. Do you know any way that we can use Vidstack without having to switch from Jest to Vitest?

Has anyone found a way to get it to pass tests with Jest?  Anyone have a mock they can share?  I'd love to use Vitest, but that isn't an option at the moment as we were all in on Jest long before we looked at the Vidstack Player as a solution for video.


http://www.diamondproducers.com multiple 200 pages marked as 404

http://www.diamondproducers.com multiple 200 pages marked as 404 .
Is it a bug or an unusual way of blocking crawlers ?

Does the same occur with the v0.8.0 branch?

This is not a bug.
This website doesn't seems to accept the request method 'HEAD', which is the default value as specified in: https://github.com/stevenvachon/broken-link-checker#optionsrequestmethod
Solution:
let options = {
    /* [...] */
    requestMethod: 'GET',
};

Upon Forking this repo and mergin this PR can say that this does fix the issue. Great work! #tested

Can I control this from the command line ?

@merav2110 Not yet, unfortunately. It shouldn't be too hard to implement yourself if you need it; fork my fix branch and see #115 for inspiration.


Added images for docs and changed images path

Added images for documentation according to this comment.

Thanks!


Auto-approve infrastructure pull-requests policy

@RussKie the issue has been resolved, close and reopen this PR the validation will pass

the issue has been resolved, close and reopen this PR the validation will pass

@DragosDanielBoia 👍


[WebView] WebView not loading source / Staying in loading state

Hello,
seems like there Is an issue with the WebView not loading its source.
I have followed the basic example on the DOCS and tried to load google.com and plain html. Either one didnt work.
RN V0.19.
Staying in loading state.
Sample code (nested component):

'use strict';
import React, {
  AppRegistry,
  Component,
  StyleSheet,
  Text,
  View,
  Image,
  TextInput,
  Button,
  TouchableHighlight,
  TouchableOpacity,
  StatusBarIOS,
  WebView
} from 'react-native';

StatusBarIOS.setStyle(1);

import Dimensions from 'Dimensions';
var width = Dimensions.get('window').width;
var height = Dimensions.get('window').height;

var CStyles = require('./CStyle');

class TestComp extends Component {
  constructor(){
    super();

    this.state = {
      url: 'https://www.google.at/'
    }
  }
  render() {
    return (
      <View style={{flex: 1}}>
          <WebView
            ref="webview"
            style={{width: width}}
            automaticallyAdjustContentInsets={false}
            source={{uri: this.state.url}}
            javaScriptEnabled={true}
            domStorageEnabled={true}
            decelerationRate="normal"
            startInLoadingState={true}
          />
      </View>
    );
  }
}
module.exports = TestComp;

+1

the problem is when I use the old prop named url it shows a warning says url has been deprecated, use source prop in version of 0.20, but use source it says source is invalid prop

Source has to be source={{ uri: YOUR_URL }}, whereas URL was just url={YOUR_URL}

Thanks @tomauty

@tomauty I am using {{uri: this.state.url }} and yet still it doesn't work

Any errors in Safari's developer console pointing to the simulator?

Unfortunately I get no errors in chrome live debugging and no errors in the app itself. I have an open SO question regarding that issue - we can continue over there so that his doesn't get spammed.

push

I had this issue when I put webview under view. An alternative solution is to move the webview out of the view(ex: as a root view), the webview came out. I thought this issue might be related to the iteration of view, because if you check the debug navigator in Xcode, there was abnormal numbers of RCTViews by a single request.

I was trying to reproduce this issue.  where is the content for CStyle?
var CStyles = require('./CStyle');

Just some Basic Styling. I use it at an implemented style sheet

The webview is nested in a regular view with a flex .9 height and a header element above it with .1 height. Nothing more to the CStyles :)

i'm having the same issue :(

Also having this issue. WebView loads when it's the only component being returned, but not when it's nested inside a View component.

Also not showing when nested inside of a View, works when it's alone.

+1
This is my tree:
<View style={styles.self}>

                <View style={styles.header.self}>
                    <Text>xxx</Text>
                </View>

                <View style={styles.content.self}>
                    {this.state.view}
                </View>

                <View style={styles.header.self}>
                    <Text>xxx</Text>
                </View>

            </View>

Here a RNPlay! https://rnplay.org/apps/xMm3OQ

Workarround:
{
                    !this.state.showBrowser &&
                    <View>
                        <Text>xxx</Text>
                    </View>
                }

                {
                    this.state.showBrowser &&
                    <WebView
                        source={{uri: this.state.url}}
                    </WebView>
                }

In my case an alignItems: 'center' in the parent container causing the problem.

I render a top WebView (used for authorization and has redirects) in my component : startInLoadingState=true causes the WebView to finish in a loading state.
This never happened after I've changed initial loading property to false.

Closing this issue because it has been inactive for a while. If you think it should still be opened let us know why.

I think this issue should not be closed. It can only work when at top level, but not work under a  tag.

shall we use https://github.com/jsdf/react-native-htmlview instead? working better?

That seems like a reasonable workaround.

well, I tryed htmlview, it's not fitting my needs: I would like to have a map (osm, leaflet) and other html pages inside my RN app, not a 'link launcher with an external browser' and I'm facing other problems:

WebView only renders on ios if https although there is a workaround which implies to declared the http website known urls
on android, WebView only renders if it's not a child of a View element
I don't want to switch to angular/cordova/ionic as I'm coding a Mastodon social network client, which is written in React
I would like to have a working WebView to share my code to be cross-platform...

It doesn't work in React Native 0.44 with startInLoadingState set to true.  I am trying to load a file from my web service into a webview.
<WebView
source={{
uri,
headers: {"Authorization": Bearer ${token}},
token: ${token},
}}
startInLoadingState={true}
scalesPageToFit={false}
/>

@fengerzh was right, Webview only work on top level

It doesn't work in React Native 0.45 either. Flex:1 seems to not affect the WebView at all unless I set a specific width or height.

Having a similar issue with the WebView on a physical Android device, OnePlus 3T.
startInLoadingState renders the loading state, but the content never loads … unless I switch the WiFi on or off, then the WebView content loads instantly.

Same problem here.
Webview only works on top level.

@matthewvincent when it's nested inside a View component,you setting flex : 1 and webview's  flex : 1,it does work

I've tried every single workaround mentioned in here, and combinations of them. I continue getting a blank screen. It doesn't work on the top level either.

Even I tried everything mentioned in the thread but nothing worked for me. At the end, I switched over to https://github.com/react-native-china/react-native-webkit-webview that  uses WKWebView instead of UIWebView as recommended by apple for iOS 8 and above, and everything started working normally.

This is still an issue, but only on iOS. Android works perfectly fine. The issue is a hidden UIWebView never exits the loading state.
React Native: 0.52.2
iOS: 11.3
Can we get this reopened, @hramos?
Logging webView.loading
I added a simple logging statement and it logs:
2018-06-07 22:32:34.481172-0500 HarvestPoint[52403:2799087] LOADING:: TRUE, TRUE

That's the only log it gets when startInLoadingState={true} on JS. If I turn that off, you'll see a list of TRUE, TRUE then a FALSE, FALSE which then renders the UI.
  NSLog(@"LOADING:: %@, %@", webView.loading  ? @"TRUE" : @"FALSE", webView.isLoading  ? @"TRUE" : @"FALSE");
  if (_injectedJavaScript != nil) {
    NSString *jsEvaluationValue = [webView stringByEvaluatingJavaScriptFromString:_injectedJavaScript];

    NSMutableDictionary<NSString *, id> *event = [self baseEvent];
    event[@"jsEvaluationValue"] = jsEvaluationValue;

    _onLoadingFinish(event);
  }
  // we only need the final 'finishLoad' call so only fire the event when we're actually done loading.
  else if (_onLoadingFinish && !webView.loading && ![webView.request.URL.absoluteString isEqualToString:@"about:blank"]) {
    _onLoadingFinish([self baseEvent]);
  }

The Fix
In WebView.ios.js, editing the hidden style fixes the loading:
  hidden: {
    height: 1,
    flex: 0, // disable 'flex:1' when hiding a View
  },

Originally the height is 0.
My Component
// @flow

import React, { Component } from 'react';
import PropTypes from 'prop-types';
import { WebView } from 'react-native';
import { Spinner } from 'native-base';

import * as actions from '../data/actions';
import { HPLoading } from '../components';

export class Iframe extends Component {
  static propTypes = {
    url: PropTypes.string,
  };

  render() {
    return (
      <WebView
        allowsInlineMediaPlayback={true}
        domStorageEnabled={true}
        renderLoading={this._renderLoading}
        source={{uri: this.props.url}}
        startInLoadingState={true}
      />
    );
  }

  _renderLoading = () => {
    return (
      <HPLoading />
    );
  }
}

@johncblandii - looks like the issue is in flexbox itself. My workaround is based on your finding, but without modifying the source code of WebView. Just wrapped webview into a custom component and return this in render() method:
<View style={this.props.style}>
  <WebView
    style={ this.state.loading ? { flex: 0, height: 1 } : { flex: 1 } }
    source={this.props.source}
    onLoadEnd={ () => this.setState({ loading: false }) } />
  { this.state.loading &&
    <ActivityIndicator animating size="large" style={{ flex: 0, width: '100%', height: '100%' }} />
  }
</View>

I did test adjusting the style and many other options to no avail. The issue to me is RN shouldn't require people to implement workarounds for it to work properly.
That's why #19617 exists.

All I wanted to say is that they need to fix an underlying issue, which seems to be in RN flexbox implementation. I don't see any reason why it should work differently when setting height to 0. Setting height to 1 in WebView stylesheet fixes the symptoms, not the problem...


[NEW] Adds an inclusion/exclusion list to specify the repositories that should be combed for leaderboard statistics.

With this PR, repositories can now be included/excluded to be considered in the leaderboard.

Closes #44

@djcruz93 Can you please review it?

@djcruz93 Can you please review it?

Thanks for your thorough review @djcruz93 . Made the changes!

Thanks for your thorough review @djcruz93 . Made the changes!

will do this evening (my time) 👍

will do this evening (my time) 👍

@Sing-Li here's a screen record for your reference
https://user-images.githubusercontent.com/28918901/104484545-6a115e80-55ef-11eb-888e-d6540a9bff1a.mp4

@Sing-Li here's a screen record for your reference
https://user-images.githubusercontent.com/28918901/104484545-6a115e80-55ef-11eb-888e-d6540a9bff1a.mp4

Hey @RonLek ! as pre-push hook PR merged, there're  some conficts there, can you please resolve it?

Hey @RonLek ! as pre-push hook PR merged, there're  some conficts there, can you please resolve it?

Please resolve conflicts.  Then it is good for merge and deployment.  Thanks.

Please resolve conflicts.  Then it is good for merge and deployment.  Thanks.

Done @Sing-Li @lolimay . Thanks.

Done @Sing-Li @lolimay . Thanks.


Prerequisites

[X] I have searched all issues/PRs to ensure it has not already been reported or fixed.

Criteria

[X] Reasonably well-known and widely used (e.g. if it's a GitHub project, it should have at least 100 stars and/or 50 forks)
[X] English interface (or at least English documentation)
[X] Latest stable version
[X] Full version (i.e. not a trial version)
[X] Fairly standard install (e.g. uses a version-specific download URL, no elaborate pre/post install scripts)

Name
nosleep
Description
A tool that prevents Windows from automatically going into ScreenSaver / Sleep / Lock modes
Homepage
https://github.com/CHerSun/NoSleep
Download Link(s)
https://github.com/CHerSun/NoSleep/releases
Some Indication of Popularity/Repute
88 github stars. A minimalistic tool with no need to install.

@sitiom Should this be added even though it has below 100+ stars?

Let's wait until it reaches 100 stars. In the meantime, I recommend using PowerToys Awake, as mentioned in the readme.

Cool

Let's wait until it reaches 100 stars. In the meantime, I recommend using PowerToys Awake, as mentioned in the readme.

102 stars now. I'd appreciate if it could be added.
PS: PowerToys is something like 2 GB in size, so if 1 doesn't really wants it - that's too much for just 1 minor tool.

PS: PowerToys is something like 2 GB in size

How would it reach 2GB?

@sitiom Scoop's Powertoys installation:

3.5 GB, 2 GB compressed.

@HUMORCE  Wow, that's nice, thanks. I've updated and it's indeed ~0.5 GB now, 0.3 GB compressed:

Much appreciated powertoys change tbh, I like the tool.


Feat - Form Builder Improvements

Reference to #480

As @wychoong mentioned, this should really be it's own isolated PR as it's very difficult to see what's going on. It does feel it should be able to stand alone without knowledge of data table or brand changes...

Sure no problem I will base this on the channels form to show how this is working to refactor the existing forms.

@alecritson @wychoong
I have rebased this branch to only show the form builder commits and just added an example refactor for Channels.
Note currently supports Text and Toggle I will add the other fields + create and fix existing tests once approved.

Another interesting one would be to refactor Taxes as this has multiple columns and blocks.
Let me know if you would like me to proceed with this example?

@alecritson just merged with the latest develop branch and fixed the issues with the channel test.
You can see how simple these form components are and much more developer friendly zero dependencies outside GC for this one.
Hope you will consider this to be merged over to develop.
Really saves a lot of time creating and modifying existing forms. Similar to the way you have created GetCandyTable TableRegistry and TableExtension I would like to continue with this to offer the same functionality which will give developers complete control when adding / modifying fields in a form schema.

Thanks for taking the time on this @adam-code-labx
We've been chatting over this kind of thing internally and we've realised we really want to limit what third party packages we bring into the eco system, especially when they will be providing core functionality like form building. I think if we was going to do something, we'd want it do it ourselves to have more control and not get into issues when updating to new versions.
Unfortunately, I'm going to close this PR as I don't think this is the direction we'll be taking, but thank you for the effort.

Thanks for taking the time on this @adam-code-labx
We've been chatting over this kind of thing internally and we've realised we really want to limit what third party packages we bring into the eco system, especially when they will be providing core functionality like form building. I think if we was going to do something, we'd want it do it ourselves to have more control and not get into issues when updating to new versions.
Unfortunately, I'm going to close this PR as I don't think this is the direction we'll be taking, but thank you for the effort.

Well that does not make sense this has nothing to do with Filament forms and no third part dependencies so what’s the problem here @alecritson 🤷♂

@alecritson I think you've assumed Filament Forms here? It looks like Adam has introduced something specifically for GetCandy, perhaps taking influence from Filament?
This kind of goes in line with our plans for other bits, which I won't go into here as they're not public yet.

Yeah there has obviously been some confusion here. Will re-open

@alecritson we'll need to book some time to review this properly at some point soon.

We still need to review this properly, but priorities have prevented it. Due to this and file conflicts, I have converted to Draft for the moment.

Closing this, as it's not longer relevant with our plans to move to Filament v3.

No problem @glennjacobs makes perfect sense and great move with Filament v3
I am also adjusting my plans around the v3 TALL Stack 💪


[Fix] Search for web map navigation bugs

Description
This PR fixes a regression where opening the Search for web map sample would just show "No Sample Selected" on iPhone (iOS 17). This was caused by the navigation stack added in  #477. Those changes were reverted, and the stack was moved to the NavigationSplitView detail. This solution should work for samples that use a navigation link in the future.
Linked Issue(s)

swift/issues/5798

How To Test
Test the Search for web map and Augment reality to show hidden infrastructure samples:

On iPhone, ensure the samples launch as expected.
On iPad, ensure the samples have a navigation back button after clicking the navigation link in landscape mode.

Screenshots

Before
After

Removing Mark since he is out for the next couple days.


proposal: cmd/go: private repos on hard-coded hosting providers require special configuration

What version of Go are you using (go version)?
$ go get -u golang.org/x/vgo
$ vgo version
go version go1.10 darwin/amd64 vgo:2018-02-20.1

Does this issue reproduce with the latest release?
yes (coming from https://github.com/golang/go/issues/25590)
What operating system and processor architecture are you using (go env)?
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/brunetto.ziosi/Library/Caches/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/brunetto.ziosi/Code"
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/m3/145jfhdd5qsf66_41wt9p02mhc4hp5/T/go-build518939426=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
I ran
vgo get github.com/MY_ORGANIZATION/MY_REPO

where:

I ran the command in a folder with a working golang app and a proper go.mod file -> vgo is working here and I can vgo build the app
the repo is a private repo with SSO enabled I can access (git pull/push, ...)

What did you expect to see?
The package code in MY_REPO or the go module downloaded
What did you see instead?
vgo get github.com/MY_ORGANIZATION/MY_REPO: git ls-remote -q https://github.com/MY_ORGANIZATION/MY_REPO in /Users/brunetto.ziosi/Code/src/mod/cache/vcs/2cb00fa3547080bb301974fe87cf912af55d50f7a6d8a9955e11fed7a20ac6d3: exit status 128:
	remote: Repository not found.
	fatal: repository 'https://github.com/MY_ORGANIZATION/MY_REPO/' not found

What happens if you attempt to go get the same repository?
Are you using the credential helper?

What vgo commit did you build from?

I don't know, I just updated it before running it... how can I know the commit?

What happens if you attempt to go get the same repository with the ordinary go tool?

The simple go get does some stuff in the background and says nothing, if I run go get -v -u I see the repo itself downloaded and some dependencies downloaded, few others blocked by our firewall (-insecure seems to allow the download for everything).

Are you using the credential helper?

I don't think so, I exchanged the keys with github so my remote is like
$ git remote -v
origin	git@github.com:MY_ORGANIZATION/MY_REPO.git (fetch)
origin	git@github.com:MY_ORGANIZATION/MY_REPO.git (push)

The error text shows the command being run. It is one of the few git commands that doesn't actually care about having a git repo to run, so you can just try it in any directory you want:
git ls-remote -q https://github.com/MY_ORGANIZATION/MY_REPO

Does it work?

Possible duplicate of #26145 but the error message from git is different (no mention of terminal prompts disabled).

If you can make the https form work then vgo (and old go get) will be happy. The two possible ways to do that are:

Add to $HOME/.gitconfig:
[url<EMAIL_ADDRESS>insteadOf = https://github.com/MYORGANIZATION/

Add to $HOME/.netrc:
machine github.com login YOU password APIKEY

where APIKEY is an API key obtained from the GitHub API page with access to private repos.

Retitling this to "cmd/go: github private repos require special configuration".
Maybe for Go 1.12 we should think about some way to get this right by default. I'm not sure exactly how.

The first option, adding
[url<EMAIL_ADDRESS>    insteadOf = https://github.com/MYORGANIZATION/

to $HOME/.gitconfig work like a charm!!
Thanks!

Hi Guys,
I don't know if this is the correct issue to comment on so please forgive me if it's not correct, I ended up here because of an Issue I am having with Gitlab. We have a project structure where we use subgroups for project organisation by client/project/repo. So our package import paths are 3 levels deep rather than the standard 2 you see on Github. This is fine except for an issue with go get. Gitlab has a strict security policy where requests for the go-import meta data is incorrect for private sub groups by design for unauthenticated requests. So for example if I wanted to go get my.gitlab.com/foo/bar/fizz the meta data request would be https://my.gitlab.com/foo/bar/fizz?go-get=1 and the meta data returned would be:
<meta name="go-import" content="my.gitlab.com/foo/bar git https://my.gitlab.com/foo/bar.git" />

Gitlab recently did implement support for a HTTP header to override this behaviour over here: https://gitlab.com/gitlab-org/gitlab-ce/issues/42817 so we can send a Private-Token header to get the correct meta. However I cannot find a way where I can get go get to send that header to resolve the source path.
We could update our import paths to include a .git suffix but that feels a little broken?
I guess what we need is a way for go get to find crednetial information for these sorts of projects, perhaps another file that lives alongside go.mod and go.sub like go.creds or something, which could contain credential information (you wouldn't want to check that into a repo tho). Or perhaps a way to override go-import meta data locally? :man_shrugging:

To add to the list of options in this space (https://github.com/golang/go/issues/26134#issuecomment-403885803), we should also add use of a keychain/keyring as a third option (which is independent of remote VCS). Instructions vary quite widely between platforms, unsurprisingly:

Mac: https://help.github.com/articles/updating-credentials-from-the-osx-keychain/
Linux: https://wiki.archlinux.org/index.php/GNOME/Keyring (or equivalent)
Windows: ???

I've switched back from using the ssh solution on Linux to using Gnome keyring (even though I don't use Gnome) to great effect... not least because it can be used to unify all credentials.
@FiloSottile are there are any security implications that I'm unaware of with respect to the use of keychain/keyrings for this sort of thing?

With Go tip (f2131f6e0c) I can use private repos without modules (with the url "ssh://git insteadOf config above), but not with modules:
dev:barlogaamp $ GO111MODULE=on go install -x -v .
# /home/bradfitz/pkg/mod/cache/vcs/61e3c76780847e514802ec6af8f940f641c6017f711444f05c59cb17ac46d456 for git2 https://github.com/bradfitz/private
go: finding github.com/bradfitz/private v0.0.0-20180808000000-9069f54cff2af5e9fc291df126155504d71d1fd9
cd /home/bradfitz/pkg/mod/cache/vcs/61e3c76780847e514802ec6af8f940f641c6017f711444f05c59cb17ac46d456; git -c log.showsignature=false log -n1 '--format=format:%H %ct %D' 9069f54cff2af5e9fc291df126155504d71d1fd9
0.002s # cd /home/bradfitz/pkg/mod/cache/vcs/61e3c76780847e514802ec6af8f940f641c6017f711444f05c59cb17ac46d456; git -c log.showsignature=false log -n1 '--format=format:%H %ct %D' 9069f54cff2af5e9fc291df126155504d71d1fd9
cd /home/bradfitz/pkg/mod/cache/vcs/61e3c76780847e514802ec6af8f940f641c6017f711444f05c59cb17ac46d456; git tag -l
0.002s # cd /home/bradfitz/pkg/mod/cache/vcs/61e3c76780847e514802ec6af8f940f641c6017f711444f05c59cb17ac46d456; git tag -l
cd /home/bradfitz/pkg/mod/cache/vcs/61e3c76780847e514802ec6af8f940f641c6017f711444f05c59cb17ac46d456; git ls-remote -q https://github.com/bradfitz/private
0.177s # cd /home/bradfitz/pkg/mod/cache/vcs/61e3c76780847e514802ec6af8f940f641c6017f711444f05c59cb17ac46d456; git ls-remote -q https://github.com/bradfitz/private
cd /home/bradfitz/pkg/mod/cache/vcs/61e3c76780847e514802ec6af8f940f641c6017f711444f05c59cb17ac46d456; git fetch -f --depth=1 https://github.com/bradfitz/private 9069f54cff2af5e9fc291df126155504d71d1fd9:refs/dummy
0.165s # cd /home/bradfitz/pkg/mod/cache/vcs/61e3c76780847e514802ec6af8f940f641c6017f711444f05c59cb17ac46d456; git fetch -f --depth=1 https://github.com/bradfitz/private 9069f54cff2af5e9fc291df126155504d71d1fd9:refs/dummy
go<EMAIL_ADDRESS>git fetch -f --depth=1 https://github.com/bradfitz/private 9069f54cff2af5e9fc291df126155504d71d1fd9:refs/dummy in /home/bradfitz/pkg/mod/cache/vcs/61e3c76780847e514802ec6af8f940f641c6017f711444f05c59cb17ac46d456: exit status 1
go: error loading module requirements

That final git fetch fails, but that hash exists in the private repo:
$ git cat-file -t 9069f54cff2af5e9fc291df126155504d71d1fd9
commit

And it's on the remote (it's remote/origin/master^), but it doesn't have a remote tag or other ref, if that's the problem.
dev:src $ git --version
git version 2.11.0

@bradfitz I think you're running up against https://github.com/golang/go/issues/26894

Note that this issue only applies to the hard-coded hosting sites.
If we have some sort of general HTTPS auth mechanism (#26232), HTTPS servers that use <meta> tags should provide ssh URLs as appropriate to begin with.

Issue #31376 was duplicated here, but is not using one of the hard-coded hosting sites. The matching HTTPS server provides no  tags, and documented methods to make 'go get' skip VCS detection step do not seem to be working.

@stub42, my apologies. We do have a special case for launchpad.net; I had missed the fact that git.launchpad.net should not match that special case.

Thank you for this issue, I was debugging the access to my private github repos for a week..
$HOME/.gitconfig:

[url<EMAIL_ADDRESS>insteadOf = https://github.com/MYORGANIZATION/

@rsc I have a huntch that capturing private repo information in the go.mod file might help. This information and auth details can get used directly or passed to Athens.
Maybe instead of insteadOf

Add to $HOME/.gitconfig:
[url<EMAIL_ADDRESS>insteadOf = https://github.com/MYORGANIZATION/

we could  write something like
module my/package

replace github.com/MYORGANIZATION =><EMAIL_ADDRESS>v1.1.1

require github.com/MYORGANIZATION/someprivatepackage v1.1.1

or
module my/package

require github.com/MYORGANIZATION/someprivatepackage v1.1.1 at git@github.com:MYORGANIZATION/someprivatepackage

I'd also like to see a solution to this where the private repo information can be stored in the repository. This is currently holding me back from using Go modules for projects that depend on private repositories because it's not seamless enough (requires modifying $HOME/.gitconfig locally).

@marksamman any ideas on how authentication should be done?
Is putting a public key in the go.mod file a security issue?
eg
module my/package

require github.com/MYORGANIZATION/someprivatepackage v1.1.1 at git@github.com:MYORGANIZATION/someprivatepackage using ssh-rsa AAAAB3NzaC1yc2EAAAADAQA...2JN millergarym@development_machine_and_ci

@marksamman any ideas on how authentication should be done? ...

@millergarym if you mean a private key, yes.

I am not suggesting including authentication credentials in the repository, just a way of pulling over SSH instead of HTTPS and leaving the rest for Git to handle.

@marksamman any ideas on how authentication should be done? ...

@millergarym if you mean a private key, yes.

No, only public keys

EDIT: I've learned that my problem can be solved with the go-import meta tag, so this is no longer a blocker for me to use Go modules with private repositories.

Can meta tags point to private repos? If so please provide an example.

Meta tags require serving public document which references the private branch or repo, which is data leakage and not acceptable at some sites. The Launchpad web site returns a 404 for private branches, unless you are both logged in and have been granted read access. A tool can't tell if the repo exists unless it has a valid oauth token or speaks SSH.
My current workaround is to embed my private branches as git subtrees, as 'replace' in go.mod already allows me to reference a relative local path:
replace api/fooclient v0.0.0 => ../api/fooclient
require api/fooclient v0.0.0

I think allowing 'replace' to specify full URLs would be a workable solution, or a new command, ideally without the v0.0.0 version pin. Say, theoretically:
rewrite api/fooclient => git+ssh://git.launchpad.net/~stub/foo/+git/foo-client
require api/fooclient

(Interestingly, you should be able to use the existing replace to force use of a fork without changing imports, perhaps even a fork of parts of the standard library)

The first option, adding
[url<EMAIL_ADDRESS>    insteadOf = https://github.com/MYORGANIZATION/

to $HOME/.gitconfig work like a charm!!
Thanks!

DO NOT MISSING THE LAST '/' CHAR!

Note that this issue only applies to the hard-coded hosting sites.

I have reason to believe that this statement may have been mistaken. I plan to investigate further.
This issue somehow ended up in the Proposals queue in Feb. 2021, but I'm not sure why — I think it may be just a bug.

I've started to suspect that this may just be a problem in the way that we probe for which protocol to use.
It appears that both GitHub and GitLab support SSH by having everyone use the username git, and then treating the public key presented by the user as the “effective user ID”. So probably when we probe for SSH support, in addition to trying ssh with the user's actual username, we should also try ssh with the hard-coded username git.

I'm a little surprised this is still an issue, I've encountered this at 3 different companies over the last 6 years, and we're always doing the .gitconfig InsteadOf dance.
I got here because if you have the following setup this does NOT work:
~/.gitconfig
[includeIf "gitdir:/company/"]
        path = /company/.gitconfig-company

/company/.gitconfig-company
[url "git@github.com:"]
        insteadOf = https://github.com

[url "git@github.com:company"]
        insteadOf = https://github.com:company

I guess because the git commands are being run with the gitdir being inside the GOENV rather than in the project's repo?

This needs some more thought and investigation

This is a 6yr old ticket 🤣

If you need conditional git includes, using hasconfig:remote is better:
[includeIf "hasconfig:remote.*.url:https://github.com/org/*"]
  path = org.gitconfig

That said, should this be retitled to focus on ssh probing?


suggestion: migrate lang change issue template to  golang/proposal

Proposal Details
Background
I think its best to restrict this repo to Go 1.x.x proposals.
Summary

[ ] Migrate https://github.com/golang/go/blob/master/.github/ISSUE_TEMPLATE/11-language-change.yml to  https://golang/proposal

[ ] Update language-change.yml to redirect to  golang/proposal/issues ...

The go project uses a single issue tracker for all its repos, there's no point in spreading it out elsewhere.


Fix Custom Website Recipe

The Custom Website recipe ID had "Franz" in it.
This way we can have ferdium-custom-website on the Data folder instead of franz-custom-website

Before merging this PR, please look for the discussion on https://github.com/ferdium/ferdium-app/pull/468

Reverted these changes


Question: getting all tokens by address

I would like to get all the tokens owned by a user (address). I see that he main contract imports IERC721Enumerable.sol but does not implement all of its functions (specifically tokenOfOwnerByIndex). Is there an easy way to implement this, based on the data the token maintains already?
One of the other things I thought of doing is emitting a Minted(address, quantity, startIndex, endIndex) event at the end of my mint function, and I assume I can later query for those events, but I'd much rather see if I can get these programmatically.
Thanks!

See #115.

@Vectorized Thanks! Perfect answer there, useful code and use cases.
The reason I added the Minted event is we're not actively listening to the contract, and it'd be far less events to collect than listening to Transfer.
Also, we'll further need to filter to filter Transfer events to those with a from address 0x0, since the event is also used to transferring between accounts, and burning.


Implement rest functions in ICP MongoDB chart

The MongoDB Operator has implemented the basic function of the ICP MongoDB chart.
There are some functions in the ICP MongoDb chart we haven't added into the operator.

[ ] Dashboard
[ ] Helm hooks

@kgcarr I list some functions, we haven't implemented in the operator. Please take a look, if they should be added.


Merge shared files

Merge of more identical or similar files.
The only notable difference is in SignatureVerificationCache.cs
netcore: _cache.Set(cacheLookupKey, result, DateTimeOffset.UtcNow.AddDays(10));
netfx: _cache.Set(cacheLookupKey, result, DateTimeOffset.UtcNow.AddHours(8));
I chose the 10 days but I think this needs consideration before merging.

I don't see any troubles using 10 days for both .net framework and .net core. I have re-run the failing pipelines and the previously failing tests look like random ones.


Support for 405 HTTP Status Code, Method Not Allowed.

Solves issue #60 and #37.
First checking requested method and it's registered routes if the route not found for the requested method, then instead of returning 404 instantly, it checks all other methods for the same request path.

Great !!! Thanks for the pull request :+1:

@squiidz looking forward to do more contributions.
Thanks for accepting it. 😊


SCILAB_EXECUTABLE Environment Variable Plot Option

Hello,
i do not have ADMIN Rights but setting the Environment Variabl with setx in the Windows Command line helped.
setx SCILAB_EXECUTABLE C:\Tools\scilab-2023.1.0\bin\Scilex.exe
setx PATH "%SCILAB_EXECUTABLE%;%PATH%"

Unfortunately it JUPITER Notebook is not ploting with error message:
Scilab 'plot2d' Funktion in -nogui oder -nwni Modus deaktiviert.
Which Scilab Exe do I have to set for SCILAB_EXECUTABLE?
How can I set Options like -nwin?
Thanks for any support.

Please use WScilex.exe or WScilex-cli.exe to have the plot features.

Thank you, for the quick reply.
Unfortunately it JUPITER Notebook is not ploting.
Error message:
Scilab 'plot2d' Funktion in -nogui oder -nwni Modus deaktiviert.
Using WScilex.exe instead is opening Scilab Application each time a cell in Jupyter Notebook is executed
setx SCILAB_EXECUTABLE C:\Tools\scilab-2023.1.0\bin\WScilex.exe
Which Scilab Exe do I have to set for SCILAB_EXECUTABLE? How can I set Options like -nwin?

WScilex-cli.exe should work for you then.

Thank you for your kind support. If i change the SCILAB_EXECUTABLE C:\Tools\scilab-2023.1.0\bin\WScilex-cli.exe
and RUN the Cells in jupxter notebook, nothing happens
KERNEL status: BUSY
Following you find my PowerShell Printout
Windows PowerShell
Copyright (C) Microsoft Corporation. Alle Rechte vorbehalten.

Lernen Sie das neue plattformübergreifende PowerShell kennen – https://aka.ms/pscore6

PS C:\Users\gschmidt> Get-ChildItem -Path Env:SCILAB_EXECUTABLE

Name                           Value
----                           -----
SCILAB_EXECUTABLE              C:\Tools\scilab-2023.1.0\bin\WScilex-cli.exe

PS C:\Users\gschmidt> python -m scilab_kernel.check
0.00s - Debugger warning: It seems that frozen modules are being used, which may
0.00s - make the debugger miss breakpoints. Please pass -Xfrozen_modules=off
0.00s - to python to disable frozen modules.
0.00s - Note: Debugging will proceed. Set PYDEVD_DISABLE_FILE_VALIDATION=1 to disable this validation.
Scilab kernel v0.9.10
Metakernel v0.29.5
Python v3.11.4 (tags/v3.11.4:d2340ef, Jun  7 2023, 05:45:37) [MSC v.1934 64 bit (AMD64)]
Python path: C:\Users\gschmidt\AppData\Local\Programs\Python\Python311\python.exe

Connecting to Scilab...
Scilab connection established
Scilab version "2<IP_ADDRESS>4825887"
scilab-2023.1.0

PS C:\Users\gschmidt> jupyter notebook
[I 2023-08-02 09:40:54.049 ServerApp] Package notebook took 0.0000s to import
[I 2023-08-02 09:40:54.134 ServerApp] Package jupyter_lsp took 0.0887s to import
[W 2023-08-02 09:40:54.134 ServerApp] A `_jupyter_server_extension_points` function was not found in jupyter_lsp. Instead, a `_jupyter_server_extension_paths` function was found and will be used for now. This function name will be deprecated in future releases of Jupyter Server.
[I 2023-08-02 09:40:54.166 ServerApp] Package jupyter_server_terminals took 0.0331s to import
[I 2023-08-02 09:40:54.173 ServerApp] Package jupyterlab took 0.0000s to import
[I 2023-08-02 09:40:54.284 ServerApp] Package notebook_shim took 0.0000s to import
[W 2023-08-02 09:40:54.284 ServerApp] A `_jupyter_server_extension_points` function was not found in notebook_shim. Instead, a `_jupyter_server_extension_paths` function was found and will be used for now. This function name will be deprecated in future releases of Jupyter Server.
[I 2023-08-02 09:40:54.294 ServerApp] jupyter_lsp | extension was successfully linked.
[I 2023-08-02 09:40:54.295 ServerApp] jupyter_server_terminals | extension was successfully linked.
[I 2023-08-02 09:40:54.303 ServerApp] jupyterlab | extension was successfully linked.
[I 2023-08-02 09:40:54.305 ServerApp] notebook | extension was successfully linked.
[I 2023-08-02 09:40:54.716 ServerApp] notebook_shim | extension was successfully linked.
[I 2023-08-02 09:40:54.784 ServerApp] notebook_shim | extension was successfully loaded.
[I 2023-08-02 09:40:54.785 ServerApp] jupyter_lsp | extension was successfully loaded.
[I 2023-08-02 09:40:54.785 ServerApp] jupyter_server_terminals | extension was successfully loaded.
[I 2023-08-02 09:40:54.785 LabApp] JupyterLab extension loaded from C:\Users\gschmidt\AppData\Local\Programs\Python\Python311\Lib\site-packages\jupyterlab
[I 2023-08-02 09:40:54.785 LabApp] JupyterLab application directory is C:\Users\gschmidt\AppData\Local\Programs\Python\Python311\share\jupyter\lab
[I 2023-08-02 09:40:54.785 LabApp] Extension Manager is 'pypi'.
[I 2023-08-02 09:40:54.785 ServerApp] jupyterlab | extension was successfully loaded.
[I 2023-08-02 09:40:54.795 ServerApp] notebook | extension was successfully loaded.
[I 2023-08-02 09:40:54.795 ServerApp] Serving notebooks from local directory: C:\Users\gschmidt
[I 2023-08-02 09:40:54.795 ServerApp] Jupyter Server 2.7.0 is running at:
[I 2023-08-02 09:40:54.795 ServerApp] http://localhost:8888/tree?token=fbfbca805ab46c134052d86e8cbd6540733db434310bf158
[I 2023-08-02 09:40:54.799 ServerApp]     http://<IP_ADDRESS>:8888/tree?token=fbfbca805ab46c134052d86e8cbd6540733db434310bf158
[I 2023-08-02 09:40:54.799 ServerApp] Use Control-C to stop this server and shut down all kernels (twice to skip confirmation).
[C 2023-08-02 09:40:54.860 ServerApp]

    To access the server, open this file in a browser:
        file:///C:/Users/gschmidt/AppData/Roaming/jupyter/runtime/jpserver-27380-open.html
    Or copy and paste one of these URLs:
        http://localhost:8888/tree?token=fbfbca805ab46c134052d86e8cbd6540733db434310bf158
        http://<IP_ADDRESS>:8888/tree?token=fbfbca805ab46c134052d86e8cbd6540733db434310bf158
[I 2023-08-02 09:40:54.949 ServerApp] Skipped non-installed server(s): bash-language-server, dockerfile-language-server-nodejs, javascript-typescript-langserver, jedi-language-server, julia-language-server, pyright, python-language-server, python-lsp-server, r-languageserver, sql-language-server, texlab, typescript-language-server, unified-language-server, vscode-css-languageserver-bin, vscode-html-languageserver-bin, vscode-json-languageserver-bin, yaml-language-server
0.00s - Debugger warning: It seems that frozen modules are being used, which may
0.00s - make the debugger miss breakpoints. Please pass -Xfrozen_modules=off
0.00s - to python to disable frozen modules.
0.00s - Note: Debugging will proceed. Set PYDEVD_DISABLE_FILE_VALIDATION=1 to disable this validation.
[W 2023-08-02 09:41:19.285 ServerApp] Notebook Untitled.ipynb is not trusted
[W 2023-08-02 09:41:20.884 ServerApp] Notebook Untitled.ipynb is not trusted
[I 2023-08-02 09:41:21.134 ServerApp] Kernel started: 87e97bb2-1512-4ca5-94c9-d6af0008a502
0.00s - Debugger warning: It seems that frozen modules are being used, which may
0.00s - make the debugger miss breakpoints. Please pass -Xfrozen_modules=off
0.00s - to python to disable frozen modules.
0.00s - Note: Debugging will proceed. Set PYDEVD_DISABLE_FILE_VALIDATION=1 to disable this validation.
[I 2023-08-02 09:41:23.155 ServerApp] Connecting to kernel 87e97bb2-1512-4ca5-94c9-d6af0008a502.
[I 2023-08-02 09:43:20.971 ServerApp] Saving file at /Untitled.ipynb
[I 2023-08-02 09:45:21.019 ServerApp] Saving file at /Untitled.ipynb

I believe it is not woking, because the are limitation in my user rights on the PC. In case i figure out, what is the issue, i let you know.
It worked perfectly on my home pc.
python-installer.exe
pip install scilab_kernel
pip install jupyter
set SCILAB_EXECUTABLE WScilex-cli.exe
set PATH=%PATH%;%PROGRAMFILES%\scilab-2023.1.0\bin\
set PATH=%PATH%;SCILAB_EXECUTABLE


feat(gctl): add explicit update & delete logic to generic controller

This commit adds logic to update or delete attachments that were not created by the generic controller. In other words, a controller developer can update or delete any attachment via a generic controller even if this attachment was not created by this controller.
This feature was already available in GenericController via UpdateAny and DeleteAny flags. However, these flags were too generic and enabled update or delete of any kind of attachment. With this new change a developer can selectively update or delete specific attachment instances.
This commit also removes the old way to implement an integration test. This introduces a new approach to test any feature that is more readable and concentrates on the logic at hand instead of dealing with k8s based CRUD operations. Any integration test can be implemented as a series of steps. Each of this step can be specified either as a create or update or delete or assert of any kubernetes resource.
In addition generic, composite as well as decorator tests are clubbed into a single package. Integration tests are still divided into two categories (i.e. crd or config mode) based on the way metac binary can be started.
There will be more enhancements to integration tests going forward which in turn will align metac to be production grade all the times in-spite of new features or changes to existing ones.
Note to Reviewer
PR is divided into 3 commits:

feat(gctl): add explicit update & delete logic to generic controller
feat(integration-test): add integration test framework
feat(gctl): add integration tests for explicit updates & deletes

Signed-off-by: AmitKumarDas<EMAIL_ADDRESS>

Hi @AmitKumarDas , still unfortunately my go skill are on very basic level, therefore I cannot do proper code review. However still I am able to ask some questions ;)
So, if attachment is managed by other controller, and  by metac, could it cause some infinite update loop ? Maybe this is something which should be put to documentation ?

thanks @grzesuav I will add doc about GenericController in this PR.
Infinite update loop can happen if multiple controllers try to update same field(s) with different values. As per best practices a custom resource can be updated by more than one controllers if  these controllers don't interfere updating same fields.

@grzesuav While looking more at documentation aspects. I felt it will be best to have them in a separate PR. I will send a new PR with just the doc updates.

:tada: This PR is included in version 0.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fixed messed up table

Looking more closely, I think the actual content of this table needs review. The "index of" example seems to have gotten mixed up with the ".." notation for selecting descendants. Maybe this pull request should be rejected in favor of a more complete correction?

Closing as whole doc already updated https://docs.mulesoft.com/mule-runtime/4.1/dataweave-cookbook-extract-data