.NET 4.7 System.Windows.Forms.DataGridView.OnMouseWheel causes induced GC

Changes in System.Windows.Forms.Screen in .NET 4.7 have caused the DataGridView to trigger an induced GC on every mouse wheel event. Specifically, this stack trace ends in a call to GC.Collect:
System.Internal.HandleCollector+HandleType.Add(IntPtr)
System.Windows.Forms.Internal.DeviceContext.FromCompatibleDC(IntPtr)
System.Windows.Forms.Internal.WindowsGraphics.CreateMeasurementWindowsGraphics(IntPtr)
System.Windows.Forms.Screen..ctor(IntPtr, IntPtr)
System.Windows.Forms.Screen.FromHandleInternal(IntPtr)
System.Windows.Forms.DataGridView.FlushDisplayedChanged
System.Windows.Forms.DataGridView.ScrollRowsByCount(Int32, ScrollEventType)
System.Windows.Forms.DataGridView.OnMouseWheel(MouseEventArgs)
System.Windows.Forms.Control.WmMouseWheel(Message&)
System.Windows.Forms.Control.WndProc(Message&)
System.Windows.Forms.NativeWindow.Callback(IntPtr, Int32, IntPtr, IntPtr)

This makes scrolling a DataGridView with the mouse wheel effectively unusable -- the UI thread is blocked for 1-3 seconds every time you move the wheel.

Before using the mouse wheel, this is the ".NET CLR Memory" display from Process Explorer:

After scrolling the DataGridView with the mouse wheel several times:

Probably worth mentioning that this isn't the only path to triggering an induced GC in 4.7; it's just the best example I've found. Showing a ContextMenuStrip can also trigger an induced GC, delaying the context menu from appearing on the screen for more than a second. Haven't tracked down the exact call sequence for this one, but I suspect it's essentially the same -- constructing a Screen object to measure for placement.

Came across this problem as well through another situation - merging of the toolbar for MDI child forms.
Attached is a DotTrace stack tree with information. Note that all significant percentages were a result of this same call to System.Internal.HandleCollector+HandleType.Add(IntPtr) so the total result is that 96% of the time to switch activation of MDI forms is in that call (the CG.Collect and Thread.Sleep inside it, specifically).
This is causing severe performance problems in our application when run in .Net 4.7.
DotTrace stack tree.txt

Thank you for reporting this issue. We are fixing this bug in the next release of .Net Framework (4.7.1) as well as working on a "hot fix" for .Net 4.7.

Thanks! That is very good to know. It will make our users very happy as most of them are avoiding the Windows 10 Creator's Update as well as the normal .Net 4.7 updates because of this issue.

@Tanya-Solyanik Thanks! We've seen this from various code paths leading to the Screen.ctor, I assume they will all be fixed (i.e. not just the DataGridView described here)? Any idea when the hot fix will be available?

@apwr Screen class constructor is fixed, thus all paths that invoke it should be taken care of. We are working on a hot fix, but don't have release dates yet.

For those who also suffer from this problem (for us, creation a complex modal form went up from a few seconds to 30 seconds), here is a dirty workaround: before creating the form, suspend the HandleCollector:
Type typeHandleCollector = Type.GetType("System.Internal.HandleCollector,System.Windows.Forms, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089", true);
MethodInfo methodInfoSuspendHandleCollector = typeHandleCollector.GetMethod("SuspendCollect", BindingFlags.Static | BindingFlags.NonPublic);
methodInfoSuspendHandleCollector.Invoke(null, null);

Don't forget to call "HandleCollector.ResumeCollect" after creating the form.
MethodInfo methodInfoResumeHandleCollector = typeHandleCollector.GetMethod("ResumeCollect", BindingFlags.Static | BindingFlags.NonPublic);
methodInfoResumeHandleCollector .Invoke(null, null);

Hope this has no side effects...

Official patches for Windows 10 creators Update that take care of the bug in Screen constructor are available here:
http://www.catalog.update.microsoft.com/Search.aspx?q=KB4032188

@Tanya-Solyanik I confirm it fixes the performance issues on Windows 10.
Is the fix for 4.7 on Windows 7 already out as well ?
Thanks !

@gluck - thank you for verification. Update for Windows 7 and all other OSs will be available in the end of August.

@Tanya-Solyanik Thanks for keeping us in-the-loop. This is very nice to know.

Pre-release version of .Net 4.7.1 that contains this fix and is compatible with Windows 7 is available here:
https://github.com/Microsoft/dotnet-framework-early-access/

Hello !
I think I've the same issue using the 4.6.2 July 2017 Preview update (KB4024848).  I've a lot of performance issue and profiling call stack is pointing the same problem.
Do you know if this will be fixed as well before the next official 4.6.2 ?
Regards,
Florian

@reeflog
Hi Florian, you are probably facing a different issue.
Please verify what version of .Net framework you are running on, to get the version look up the reg key as described here:
https://docs.microsoft.com/en-us/dotnet/framework/migration-guide/how-to-determine-which-versions-are-installed
This particular bug had been introduced in 4.7 and is not present in the lower versions.
Thank you, Tanya

@Tanya-Solyanik I can confirm the August Cumulative Update for Windows 1703 fixes this issue.
However, are you absolutely sure the bug hasn't been backported to 4.6.2? We had blocked .NET 4.7 and the Creators Update because of this bug, but I have users now reporting severe performance problems on .NET 4.6.2. The symptoms look exactly the same: induced GC while scrolling, opening context menus, etc. The problems started after last week's Windows patches, and the affected machines have Windows 1607 and .NET 394802.

Looking further (and for some unknow reason), the July Preview update for 4.6.2 seems to actually install 4.7. At least, the assembly version show 4.7 after the update.
I don't know in which way to consider it but either it was a package issue which will be most probably fixed. Or the next 4.6.2 is 4.7... and it's fixed as well.
Thank you.

Version of the .Net Framework is stored in:
HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\NET Framework Setup\NDP
registry key, and is not modified by the updates.
Servicing updates install newer versions of the updated dlls only. Your file version after the update should match file version listed in the KB description article:
https://support.microsoft.com/en-us/help/4033989
In this case KB4033989 fixes the GDI+ leak that was indeed backported to 4.6.2 in the July update and is now fixed in the August update.
Thank you, Tanya

I can confirm that I also have the same issue on WIndows 10 1607 version.
@Tanya-Solyanik do you know if there will be a KB that include the fix for Windows 1607 versions?
Thank you.

@heeko, do you have the latest updates? KB4034661 should install System.Windows.Forms.dll v 4.7.2106 on your machine.  What version do you have?
https://support.microsoft.com/en-us/help/4034661
http://www.catalog.update.microsoft.com/Search.aspx?q=KB4034661

@Tanya-Solyanik I currently have KB4035631, which seems to be 'younger' than the KB4034661 that you mentioned (base on catalog.update modification date). I will try to get this KB.
Thanks a lot for your help.

You can get the fix here: https://blogs.msdn.microsoft.com/dotnet/2017/09/12/net-framework-september-2017-security-and-quality-rollup/


Improve unit test

Include test-api on npm test, and delay test, to get time to retrieve token

:clap:


SideNav with affix disappear from screen while scrolling in safari

These seams to happen in safari and in @souperneon browser

HI! I solve this by using affix-top instead of affix
https://github.com/tangrams/blocks/commit/cc221323e6d36d118cd1586c4074fd5b026bb084
I find affix behavior still a little mysterious : )


Handle root context

This solves: https://github.com/greenkeeperio/rc/issues/6
The issue is trying to split a path without a '/' and thus returning undefined, falling back to the default value 'root' solves this.

Thanks :)


Branch testing4.1

Codecov Report
Base: 38.82% // Head: 43.38% // Increases project coverage by +4.56% :tada:

Coverage data is based on head (fc908f2) compared to base (2f08b53).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #137      +/-   ##
============================================
+ Coverage     38.82%   43.38%   +4.56%
- Complexity      493      544      +51
============================================
  Files           134      134
  Lines          2872     2872
  Branches        337      337
============================================
+ Hits           1115     1246     +131
+ Misses         1657     1517     -140
- Partials        100      109       +9

Impacted Files
Coverage Δ

...c/main/java/seedu/address/model/person/Person.java
81.35% <0.00%> (+1.69%)
:arrow_up:

...main/java/seedu/address/model/pet/DateOfBirth.java
46.66% <0.00%> (+6.66%)
:arrow_up:

...ain/java/seedu/address/model/pet/ColorPattern.java
85.71% <0.00%> (+7.14%)
:arrow_up:

...ain/java/seedu/address/model/person/Deliverer.java
57.69% <0.00%> (+7.69%)
:arrow_up:

...in/java/seedu/address/model/order/OrderStatus.java
66.66% <0.00%> (+8.33%)
:arrow_up:

src/main/java/seedu/address/model/person/Name.java
90.00% <0.00%> (+10.00%)
:arrow_up:

src/main/java/seedu/address/model/pet/Species.java
40.00% <0.00%> (+10.00%)
:arrow_up:

...rc/main/java/seedu/address/model/person/Email.java
90.00% <0.00%> (+10.00%)
:arrow_up:

...rc/main/java/seedu/address/model/person/Phone.java
90.00% <0.00%> (+10.00%)
:arrow_up:

.../main/java/seedu/address/model/person/Address.java
90.00% <0.00%> (+10.00%)
:arrow_up:

... and 10 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

LGTM!


Commit "Rewrite local links for RST" emits "WARNING: file not found"

Latest release 0.3.2 breaks the world for us as the last commit https://github.com/spatialaudio/nbsphinx/commit/7e791655205beebccce65b920d4cf798c8f6d614 breaks a link to a plot directive script (here empirical_cdf-1.py):
Warning, treated as error
/tmp/openturns/python/src/sphinx_build/theory/data_analysis/empirical_cdf.rst.rst:file not found: 'theory/data_analysis/empirical_cdf-1.py'

Here is the rst file, it contains a plot directive and as well a link to a notebook:
https://github.com/openturns/openturns/blob/master/python/doc/theory/data_analysis/empirical_cdf.rst
Here is the rendered version, before the 0.3.2 update:
http://openturns.github.io/openturns/master/theory/data_analysis/empirical_cdf.html

So you are saying everything works perfectly but there is a warning?

No, the renderend version was built with 0.3.1, before the 0.3.2 update. We treat sphinx warnings as errors. If I believe the message emited, the pyplot script may be missing in the html.

And what happens if you allow warnings?

Thanks for confirming that the warning is the only problem, now we can try to solve this.
Ideas:

just remove the warning
create a new config option to disable this specific warning
modify all warnings to be able to use supress_warnings, already discussed in #130

Any more ideas?
Any comments?

I'd prefer to actually fix the warning in nbsphinx. Do you have a rough idea of what could cause it in that commit I pointed out ?

I know exactly what causes it:
https://github.com/spatialaudio/nbsphinx/blob/89cbb765a1daa5aefd07f7f2f32c04da23aefc3a/src/nbsphinx.py#L1080-L1083
nbsphinx was already doing this for a long time for Jupyter notebooks, but with the commit you mentioned, it started to apply the same thing to all Sphinx source files, including RST files.
Now the problem is that the "plot" directive is generating "invalid" RST code in the sense that it uses link locations that don't exist in the source directory (but they will at some point exist in the output directory).

Why do we have to touch links other than ipynbs ?
The plot directive does not yield warnings by itself if it links to the not preexisting python script, how is this case handled ?

Why do we have to touch links other than ipynbs ?

We don't have to, that's a free decision.
I thought it would be nice if links in RST files work the same as links in notebook files.
As an added benefit, those links work also when viewing the RST file on Github and other rendered RST views.
See http://nbsphinx.readthedocs.io/en/0.3.2/a-normal-rst-file.html#links-to-notebooks-and-other-sphinx-source-files

The plot directive does not yield warnings by itself if it links to the not preexisting python script, how is this case handled ?

I don't exactly understand the question, but I'm trying to answer what I think you are asking:
Normally, the RST parser doesn't check if links point to anything meaningful, they are just blindly generated.
I added a warning message in nbsphinx if a link points to a non-existing local file, because if you manually write this link in a Jupyter notebook, this is most likely a mistake, e.g. a typo.
As it turns out, when extending this to RST files, there are some cases where people would either manually or via a Sphinx extension create local links that don't point to any file in the source directory, but to a file in the output directory, which is created long after parsing the source file.

This is clearer now.
Would it be possible to add an option to decide whether links should be checked against non-existing local files then ?

Would it be possible to add an option to decide whether links should be checked against non-existing local files then ?

Sure. But what should it do for existing local files?
If a local link points to a non-existent file, nbsphinx doesn't do anything anyway. It's just showing the warning we're talking about.

I found #130 is related.

It indeed is. That's why I mentioned it in https://github.com/spatialaudio/nbsphinx/issues/174#issuecomment-377488389 and https://github.com/spatialaudio/nbsphinx/issues/174#issuecomment-379046962. In fact, this issue is a duplicate of #130.

When the link points to a non-existent file, it could avoid emitting warnings to sphinx,
but I dont see where in the nbsphinx code this warning is explicitely raised.

The warning is raised in the code snippet I showed above: https://github.com/spatialaudio/nbsphinx/issues/174#issuecomment-379536943.
You could also search for this combination of words: "file not found". It should be the only match in nbsphinx.py.

@jschueller Thanks for #177!
I've just created an alternative PR: #178.
What do you think about it?
I think my PR #178 would solve your immediate problem.
But if we want to completely solve #130, we could merge your PR #177, too.

@jschueller Thanks for #177!
I've just created an alternative PR: #178.
What do you think about it?
I think my PR #178 would solve your immediate problem.
But if we want to completely solve #130, we could merge your PR #177, too.

#178 is totally ok for me, imho no need for #177


[FEATURE] Android TV Support

Will there be any support for Android TV at any time?

I don't have any device in this form factor so it's pretty hard to push out support, plus I don't have any prior experience with other it either. I'll keep this open since I'd love to work on it at some point but for now it's not a priority.

As far as I can tell, Viscerion 4.0 works on Android TV, but for the fact that I can't find a way to add a VPN without a QR Code, which can't be done without a camera.

As far as I can tell, Viscerion 4.0 works on Android TV. The only issue I've found is an interface one, as one could expect. The + button only shows the option to create from scratch.

It does install and run fine-ish (going to settings crashes the app). As for importing the config it could be done through "Create from file or archive" which seems absent or rather not visible on TV form factor (the only option I can see is "Create from Scratch").

As far as I can tell, Viscerion 4.0 works on Android TV. The only issue I've found is an interface one, as one could expect. The + button only shows the option to create from scratch.

It does install and run fine-ish (going to settings crashes the app). As for importing the config it could be done through "Create from file or archive" which seems absent or rather not visible on TV form factor (the only option I can see is "Create from Scratch").

The settings crash I believe is the same as in #128, I'm still investigating it. About the import bottom sheet, I have recently acquired a Xiaomi smart TV which seems to run Android apps semi decently so I'm going to try and work towards this.

@molgar @kaambiz I think I've solved the bottom sheet problem, please try this out and see if it resolves the issue for you. You should be able to upgrade from the Play Store build just fine. Thanks a lot for all the feedback so far, it's been immensely helpful :)

I can't install the attached apk. Application not installed is what I get when I try to install it on top of the 4.0 release.

I can't install the attached apk. Application not installed is what I get when I try to install it on top of the 4.0 release.

Ah my bad, Play Store builds use App Bundles which are resigned by Google Play on their servers. Here's a debug build that can be installed alongside: viscerion_4.0.0-93-gf466c29ad203-debug.apk

Same thing with that new build. Can't install

Ugh what the hell... Can you try installing it on to your phone with ADB and send me the error?

Sorry, I on iPhone. I do have an android phone lying around though. What's ADB?

Sorry, I'm on iPhone. I do have an android phone lying around though. What's ADB?

Oh well, I'm just gonna try to reproduce the issue locally then I guess. ADB is explained here.

Here you go:
`./adb install ~/Downloads/viscerion_4.0.0-95-gaabca423b0a8-debug.apk

daemon not running; starting now at tcp:5037
daemon started successfully
Performing Streamed Install
adb: failed to install /Users/aleemtectonic/Downloads/viscerion_4.0.0-95-gaabca423b0a8-debug.apk: Failure [INSTALL_FAILED_TEST_ONLY: installPackageLI]`

Here you go:
./adb install ~/Downloads/viscerion_4.0.0-95-gaabca423b0a8-debug.apk
* daemon not running; starting now at tcp:5037
* daemon started successfully
Performing Streamed Install
adb: failed to install /Users/aleemtectonic/Downloads/viscerion_4.0.0-95-gaabca423b0a8-debug.apk: Failure [INSTALL_FAILED_TEST_ONLY: installPackageLI]

Hmm, that's gonna be a problem. Try this one please.

./adb install ~/Downloads/viscerion_4.0.0-103-g036e6a7db374-debug.apk
Performing Streamed Install
Success

winner. Will sideload onto my Android TV if I get chance over the next few days.

./adb install ~/Downloads/viscerion_4.0.0-103-g036e6a7db374-debug.apk
Performing Streamed Install
Success

winner. Will sideload onto my Android TV if I get chance over the next few days.

So I can't rollout with Q support. That's a bummer...

winner

I was also able to install it on my Android TV set.

winner

I was also able to install it on my Android TV set.

Does the Create button correctly show the bottom sheet now?

Yes, it does. I can either create from scratch, import from a file, or create from a QR-code.

Yes, it does. I can either create from scratch, import from a file, or create from a QR-code.


Metrics gradually disappearing over time

After running for 10 minutes (and thus doing 2 requests to cloudwatch), the exporter slowly starts reporting NaN as the values of the metrics.

As you can see, at 01:36 there are still some values left, but 5 minutes later even they are missing.
This is what the metrics page of the exporter looks like after 01:47:

I found this comment https://github.com/ivx/yet-another-cloudwatch-exporter/issues/378#issuecomment-843487412
But after setting period to 330 and restarting the service, my aws_ec2_cpucredit_usage_average metrics simply disappeared (i.e. I was left with only aws_ec2_info metrics).
Here's my config:
---
discovery:
  jobs:
  - regions:
      - eu-central-1
    type: ec2
    enableMetricData: true
    metrics:
      - name: CPUCreditUsage
        statistics:
        - Average
        period: 300
        length: 600

And my systemd service file:
[Service]
ExecStart=/usr/local/bin/yace --config.file=/etc/cloudwatch_exporter/example.yml
Environment="AWS_ACCESS_KEY_ID=<secret>"
Environment="AWS_SECRET_ACCESS_KEY=<secret>"

[Install]
WantedBy=multi-user.target

I doubt that this is AWS rate-limiting because the metrics show up again for 15 minutes after restarting the exporter.
I would appreciate any insight.

@aobertyshev Do you see any warnings or error messages in YACE logs?

No, here's the log output (journalctl -u cloudwatch_exporter) from restarting it just now:
-- Logs begin at Mon 2021-06-14 10:47:15 UTC, end at Wed 2021-06-16 06:49:34 UTC. --
Jun 16 06:49:34 aws-frt-prometheus systemd[1]: Stopping cloudwatch_exporter.service...
Jun 16 06:49:34 aws-frt-prometheus systemd[1]: Stopped cloudwatch_exporter.service.
Jun 16 06:49:34 aws-frt-prometheus systemd[1]: Started cloudwatch_exporter.service.
Jun 16 06:49:34 aws-frt-prometheus yace[10670]: {"level":"info","msg":"Parse config..","time":"2021-06-16T06:49:34Z"}
Jun 16 06:49:34 aws-frt-prometheus yace[10670]: {"level":"info","msg":"Startup completed","time":"2021-06-16T06:49:34Z"}

The metrics appear:

Here's the /metrics page:

2 NaN here are from the VMs that are not running.
5 min later:
Logs:
-- Logs begin at Mon 2021-06-14 10:47:15 UTC, end at Wed 2021-06-16 06:55:09 UTC. --
Jun 16 06:49:34 aws-frt-prometheus systemd[1]: Stopping cloudwatch_exporter.service...
Jun 16 06:49:34 aws-frt-prometheus systemd[1]: Stopped cloudwatch_exporter.service.
Jun 16 06:49:34 aws-frt-prometheus systemd[1]: Started cloudwatch_exporter.service.
Jun 16 06:49:34 aws-frt-prometheus yace[10670]: {"level":"info","msg":"Parse config..","time":"2021-06-16T06:49:34Z"}
Jun 16 06:49:34 aws-frt-prometheus yace[10670]: {"level":"info","msg":"Startup completed","time":"2021-06-16T06:49:34Z"}

/metrics page:

You can see some NaN appearing.
5 more mins later (thus 10 minutes after restarting):
Log stayed the same.
/metrics page:

Grafana:

Some additional info:

I'm running this version https://github.com/ivx/yet-another-cloudwatch-exporter/releases/tag/v0.27.0-alpha
System info (uname -a): Linux <hostname> 4.9.0-14-amd64 #1 SMP Debian 4.9.246-2 (2020-12-17) x86_64 GNU/Linux (Debian 9)
If I restart the exporter, I receive the metrics again
IAM permissions of the account that is being used by the exporter:

Even I get the same thing when I add new resources. Until I restart the exporter, the YACE won't get the data.

Did you found any solution for this?
Only way that i was able to see stable results was with setting length: 300 period: 600
but then cloudwatch-exporter logged warning
length(300) is smaller than period(600). This can cause that the data requested is not ready and generate data gaps

My temporary solution is to use Cloudwatch plugin for Grafana https://grafana.com/grafana/plugins/cloudwatch/
It ships by default and you can add the Cloudwatch data source directly.

The period variable isn't really relevant here.  The problem is that length is greater than the scraping interval.
The scraping interval defaults to 300.  The code is supposed to make sure that it's equal or greater to the maximum job length, but I believe that doesn't happen due to a bug (see https://github.com/ivx/yet-another-cloudwatch-exporter/pull/406)
To fix this, you should either set the value of length to 300, or increase the scraping interval (scraping-interval) to be the same as your max length setting.


Unrecognized source names in OSS-Fuzz coverage mode

See https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=32258 for an example of the failure.
Originally reported in https://github.com/google/oss-fuzz/pull/5326.

Fixed in https://github.com/google/oss-fuzz/pull/5548


FTP export fails, but no error in log

Hello,
I an FTP-exporting a site, which fails in the FTP export step at around 50%, but there are no errors in the logfile that I can see. Attached is the log. There are files on the destination site, but I'm not sure if its everything. Having different loglevels might be interesting, i.e. for categories such as crawling, exporting, etc. so that the log can be selectively detailed, else it will be huge for large sites.
2019-02-23 06:41:57
PHP VERSION 5.6.39
OS VERSION Linux wp-hosting-main 4.9.124-paas-2270098 WP2Static/wp2static#1 SMP PREEMPT Tue Aug 28 15:11:59 CEST 2018 x86_64
WP VERSION 5.1
WP URL https://SOURCESITE.com
WP SITEURL https://SOURCESITE.com
WP HOME https://SOURCESITE.com
WP ADDRESS https://SOURCESITE.com
PLUGIN VERSION 6.6.5
VIA WP-CLI?
 SFTP option coming in V7, workaround for now is to use ZIP or folder deploy options and transfer using CLI or GUI ftp client.


Adding Vitest to Angular app results in   NullInjectorError: No provider for TestComponentRenderer!

Please provide the environment you discovered this bug in.
I can't reproduce it - I've run the same commands on codesandbox.io and it works fine there. However, I tried creating a new Angular app and running the same app on a different machine and both resulted in issues.
On two machines I own, running ng new, adding Vitest as documented and fixing the issues mentioned in the description result in the error in the title.
Which area/package is the issue in?
Don't know / other
Description
I can't reproduce it - I've run the same commands on codesandbox.io and it works fine there. However, I tried creating a new Angular app and running the same app on a different machine and both resulted in issues.
Two issues I encountered running the commands: ng g @analogjs/platform:setup-vitest --project [your-project-name] adds vitest 1.31.1 to package.json which can't be installed because this version doesn't exist. I assumed it was meant to be 1.3.1, after changing and running npm i I get an error about not being able to use ESBuild modules - I might remember wrong, but the solution was to change extension of vite.config from ts to mts. Those errors also happened on codesandbox so they probably don't cause my issue.
It's probably more of a question than a bug since I know the code works on codesandbox, but I have no idea whatsoever what causes it to fail on my machines.
Please provide the exception or error you saw
C:\repos\app\asd [feature-frontend-skeleton ≡ +1 ~2 -0 !]> npm run test --verbose
npm verb cli C:\Program Files\nodejs\node.exe C:\Users\***\AppData\Roaming\npm\node_modules\npm\bin\npm-cli.js
npm info using<EMAIL_ADDRESS>npm info using<EMAIL_ADDRESS>npm verb title npm run test
npm verb argv "run" "test" "--loglevel" "verbose"
npm verb logfile logs-max:10 dir:C:\Users\***\AppData\Local\npm-cache\_logs\2024-03-18T19_11_57_493Z-
npm verb logfile C:\Users\***\AppData\Local\npm-cache\_logs\2024-03-18T19_11_57_493Z-debug-0.log

><EMAIL_ADDRESS>test
> ng test

 DEV  v1.4.0 C:/repos/app/asd

 ❯ src/app/app.component.spec.ts (1)
   ❯ AppComponent (1)
     × should create the app
       \ [ afterEach ]

⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯ Failed Tests 1 ⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯

 FAIL  src/app/app.component.spec.ts > AppComponent > should create the app
NullInjectorError: R3InjectorError[TestComponentRenderer -> TestComponentRenderer]:
  NullInjectorError: No provider for TestComponentRenderer!
 ❯ NullInjector.get ../../app/asd/node_modules/@angular/core/fesm2022/core.mjs:1654:27
 ❯ R3Injector.get ../../app/asd/node_modules/@angular/core/fesm2022/core.mjs:3093:33
 ❯ R3Injector.get ../../app/asd/node_modules/@angular/core/fesm2022/core.mjs:3093:33
 ❯ TestBedImpl.inject ../../packages/core/testing/src/test_bed.ts:548:58
 ❯ TestBedImpl.createComponent ../../packages/core/testing/src/test_bed.ts:618:40
 ❯ Function.createComponent ../../packages/core/testing/src/test_bed.ts:366:33
 ❯ src/app/app.component.spec.ts:11:33
      9|   });
     10|
     11|   it('should create the app', () => {
       |                                 ^
     12|     const fixture = TestBed.createComponent(AppComponent);
     13|     const app = fixture.componentInstance;
 ❯ _ZoneDelegate.invoke node_modules/zone.js/fesm2015/zone.js:368:26
 ❯ ProxyZoneSpec.onInvoke node_modules/zone.js/fesm2015/zone-testing.js:273:39
 ❯ _ZoneDelegate.invoke node_modules/zone.js/fesm2015/zone.js:367:52

⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯
Serialized Error: { ngTempTokenPath: null, ngTokenPath: [ 'TestComponentRenderer', 'TestComponentRenderer' ] }
⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯[1/2]⎯

 FAIL  src/app/app.component.spec.ts > AppComponent > should create the app
NullInjectorError: R3InjectorError[TestComponentRenderer -> TestComponentRenderer]:
  NullInjectorError: No provider for TestComponentRenderer!
 ❯ NullInjector.get ../../app/asd/node_modules/@angular/core/fesm2022/core.mjs:1654:27
 ❯ R3Injector.get ../../app/asd/node_modules/@angular/core/fesm2022/core.mjs:3093:33
 ❯ R3Injector.get ../../app/asd/node_modules/@angular/core/fesm2022/core.mjs:3093:33
 ❯ TestBedImpl.inject ../../packages/core/testing/src/test_bed.ts:548:58
 ❯ TestBedImpl.tearDownTestingModule ../../packages/core/testing/src/test_bed.ts:771:31
 ❯ TestBedImpl.resetTestingModule ../../packages/core/testing/src/test_bed.ts:479:16
 ❯ ../../packages/core/testing/src/test_hooks.ts:34:15
 ❯ _ZoneDelegate.invoke node_modules/zone.js/fesm2015/zone.js:368:26
 ❯ ProxyZoneSpec.onInvoke node_modules/zone.js/fesm2015/zone-testing.js:273:39
 ❯ _ZoneDelegate.invoke node_modules/zone.js/fesm2015/zone.js:367:52

⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯
Serialized Error: { ngTempTokenPath: null, ngTokenPath: [ 'TestComponentRenderer', 'TestComponentRenderer' ] }
⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯[2/2]⎯

 Test Files  1 failed (1)
      Tests  1 failed (1)
   Start at  20:11:59
   Duration  1.48s (transform 215ms, setup 409ms, collect 184ms, tests 90ms, environment 466ms, prepare 638ms)

 FAIL  Tests failed. Watching for file changes...
       press h to show help, press q to quit

Other information
No response
I would be willing to submit a PR to fix this issue

[X] Yes
[ ] No

Not sure what's going on here because there definitely is a 1.3.1 version of Vitest
https://www.npmjs.com/package/vitest?activeTab=versions

Ahh, I see the issue. We add an incorrect version to the package.json
https://github.com/analogjs/analog/blob/beta/packages/nx-plugin/src/generators/setup-vitest/versions/ng_17_X/versions.ts#L8 should be ^1.3.1

If you can share the test repo that would help. I'm able to run the tests after adjusting the version and changing vite.config.ts to vite.config.mts
npx @angular/cli@latest new angular-analog-vitest
cd angular-analog-vitest
npm i @analogjs/platform --save-dev
ng g @analogjs/platform:setup-vitest --project angular-analog-vitest
(Update package.json to ^1.3.1 after npm install error)
npm i
Rename vite.config.ts to vite.config.mts
npm run test

I see, that is weird. We fixed the issues with running the schematic, but that wouldn't resolve this.


setup renovate

Renovate has been added.
It will auto merge if it is a minor changes. makes life easier :)


Skeleton Animation - cannot find module moti/skeleton

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
The import from the docs for the skeleton loading is not working on my expo project
All other import works, and other animations are working with no issues
Also there is no existing npm package called moti/skeleton - I also tried motify/skeleton which  seems to be the old version, the npm install but it's not working either in import
import { MotiView, AnimatePresence, View } from "moti";
import { Skeleton } from 'moti/skeleton';

export default function Skeleton() {

  const colorMode = 'light';

  return (
      <MotiView
        transition={{
          type: 'timing',
        }}
        style={[styles.container, styles.padded]}
        animate={{ backgroundColor: '#ffffff' }}
      >
        <Skeleton colorMode={colorMode} width={250} />
      </MotiView>

  )
};

Expected Behavior

Steps To Reproduce
No response
Versions
"expo": "~47.0.14",
 "react-native-reanimated": "^2.14.4",
 "moti": "^0.8.0",
 "react-native": "0.70.8",
 "react-native-web": "~0.18.12",

Screenshots

Reproduction

I've found a working solution by adding this config to my webpack, not sure if this is the right way to go
const config = await createExpoWebpackConfigAsync(
    {
      ...env,
      babel: {
        dangerouslyAddModulePathsToTranspile: ["moti"],
      },
    },
    argv
  );

   config.module.rules.push({
    test: /\.mjs$/,
    include: /node_modules/,
    type: "javascript/auto",
  });

Are you on the latest moti?

Maybe add moti/skeleton to dangerouslyAddModulePathsToTranspile?

I've found a working solution by adding this config to my webpack, not sure if this is the right way to go
const config = await createExpoWebpackConfigAsync(
    {
      ...env,
      babel: {
        dangerouslyAddModulePathsToTranspile: ["moti"],
      },
    },
    argv
  );

   config.module.rules.push({
    test: /\.mjs$/,
    include: /node_modules/,
    type: "javascript/auto",
  });

This looks like what the docs recommend.


Support for NVIDIA Jetson Xavier NX

Make sure you have read the documentation, and have put forth a reasonable effort to find an existing answer.
Expected behaviour
I followed the installation document as I thought this would be also working on Jetson Xavier NX. I am using Xavier instead of Nano right now. Is this why I can't execute the following command?
picovoice_demo_mic \
--keyword_path resources/porcupine/resources/keyword_files/jetson/picovoice_jetson.ppn \
--context_path resources/rhino/resources/contexts/jetson/smart_lighting_jetson.rhn

If it is due to my hardware, I would like to know how I could get this resolved!
Thank you!
Actual behaviour
ss@ss-desktop:~/picovoice/picovoice$ picovoice_demo_mic \
> --keyword_path resources/porcupine/resources/keyword_files/jetson/picovoice_jetson.ppn \
> --context_path resources/rhino/resources/contexts/jetson/smart_lighting_jetson.rhn
WARNING: Please be advised that this device (CPU part = 0x004) is not officially supported by Picovoice. Falling back to the armv6-based library. This is not tested nor optimal.
Traceback (most recent call last):
  File "/usr/local/bin/picovoice_demo_mic", line 5, in <module>
    from picovoicedemo.picovoice_demo_mic import main
  File "/usr/local/lib/python3.6/dist-packages/picovoicedemo/picovoice_demo_mic.py", line 21, in <module>
    from picovoice import Picovoice
  File "/usr/local/lib/python3.6/dist-packages/picovoice/__init__.py", line 12, in <module>
    from .picovoice import Picovoice
  File "/usr/local/lib/python3.6/dist-packages/picovoice/picovoice.py", line 14, in <module>
    import pvporcupine
  File "/usr/local/lib/python3.6/dist-packages/pvporcupine/__init__.py", line 15, in <module>
    LIBRARY_PATH = pv_library_path('')
  File "/usr/local/lib/python3.6/dist-packages/pvporcupine/util.py", line 94, in pv_library_path
    raise NotImplementedError('Unsupported platform.')
NotImplementedError: Unsupported platform.

Steps to reproduce the behaviour
Followed the documentation procedure without errors until the last line.
(Include enough details so that the issue can be reproduced independently.)

I'm sorry that I have labeled as a bug instead of feature request 😿

Yes, we only officially support the Jetson Nano. We can keep this issue open to track if there is interest from others and use that to determine if adding such support is a business priority.

We didn't get much traction here. closing for now


Update interested-technical-writers.md

I'm participating in GSOD, please, receive my pull request and merge it to through the Technical Writer documents.

Hi! Would you please make a new pull request adding your information in alphabetical order by surname/family name? If you only use one name, alphabetize under that name. If you have questions about where your name would go in order, please reply to this comment. (Please alphabetize by more than the first letter: for example, Cat would go after Call but before Cave.)
The names are LISTED firstname, surname, but they are alphabetized by surname. So if you have:
Anwar Able
Bill Lion
Caesar Sun
Someone named Amy Pleasant would go between Bill Lion and Caesar Sun.

But sir, I provided my full name "Ali Hamza". What is wrong with that?
On Mon, May 17, 2021, 9:58 PM Erin McKean @.***> wrote:

Closed #279 https://github.com/google/season-of-docs/pull/279.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/season-of-docs/pull/279#event-4755829363, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ARAYAS2XBCC42NWVLDTOZZTTOFDMTANCNFSM447BLRUA
.


Move disable page to gui

Description
Currently there's this static page inline in kuma: https://github.com/kumahq/kuma/blob/master/pkg/api-server/gui_handler.go#L10-L29
Would be nice to have it in here and have it styled as well

We'll still be able to disable the GUI though no?

Adding a field: 	Disabled    bool   json:"disabled"`` in the index.html payload.


Document not stored when property of Document uses underscore

I'm updating an older project which seemed to work fine so far with this document:
/** @MongoDB\Document(collection="histories") */
class History
{
    /** @MongoDB\Id */
    private $id;

    /** @MongoDB\Field(type="string") */
    private $notes;

    /** @MongoDB\Field(type="string") */
    private $date;

    /** @MongoDB\Field(type="string") */
    private $type;

    /** @MongoDB\Field(name="contact_id", type="int") */
    private $contact_id;

    /** @MongoDB\Field(type="string") */
    private $contact_name;

Using underscore here seems to be legit. As it worked before and other people successfully used it e.g. here:

http://stackoverflow.com/questions/32274827/symfony-2-doctrine-odm-returning-errors-on-existing-fields

But suddenly the Document stopped persisting the $contact_id class property. While $contact_name indeed was added!
I renamed the properties using camelCase:
    /** @MongoDB\Field(name="contact_id", type="int") */
    #private $contact_id;
    private $contactId;

    /** @MongoDB\Field(type="string") */
    private $contact_name;

Now BOTH fields are persisted correctly.
Looks like this issue is only related to the int type. I cleared the metatdata cache several times.
I'm gonna use camelCase from now. But maybe this is important to you. Maybe it is related to an update to PHP7 some months ago.
I'm using dev-master.
Possibly related:

https://github.com/doctrine/mongodb-odm/issues/1459
https://github.com/doctrine/mongodb-odm/issues/842

Could you please provide a failing test case? This looks like one of "those" bugs ;)

It looks like this is NOT a bug. I recognized some problems with several naming strategies.
Instead of using the doctrine:mongodb:cache:clear-metadata command I completely removed the app/cache/dev folder. Only then everything started to work fine.
Looks like it is a problem with the command then?

@webdevilopers most probably the Hydrators needed to be regenreted as well and that happens when you clear the cache :)

You are right! I added the doctrine:mongodb:generate:hydrators command and the issue is solved!
Thanks!


DataManagementApi: add ContractAgreementService and ContractAgreementTransformer

For the implementation of the contract negotiation endpoint we need an ContractAgreementService (and an ContractAgreementServiceImpl) with roughly the following signature:
public interface ContractAgreementService {
  ContractAgreement findbyId(String contractAgreementId);
  Collection<ContractAgreement> query(QuerySpec query);
}

In addition, there needs to be an ContractAgreementTransformer, that converts a ContractAgreement into a ContractAgreementDto
Important aspects:

service must provide transactional consistency
service must perform defensive checks before performing an operation:
invoking the transformer is done by the controller, not the service.
transformer does not perform any validation, it simply converts

depends on #1008


Allow single property binding

Allow source or some other new attribute to have ng-repeat like syntax for binding to model.
e.g:
item.id for item in items
where the ng-model bound model will become an array of id properties.

Can this issue be closed?

Closing due to inactivity.


Store user input data in database

This pull request implements a database to store the user input data.
Currently we are using the Dataset class to represent the user input data. It reads the actual input file and performs cleaning and validation of the data on the go. This is innefficient because you need to read the complete file every time you want to get the data of a single record. Also, the logic for cleaning and validating the data is quite messy and it is difficult to customize the data format.
This pull request adds two main classes: Record and DataStore. The DataStore class gives a Python API to write data to an SQLite database and read data from it in the form of records, or pandas DataFrames. The Record class represent a single input record. It defines the fields that are allowed, how they should be validated and how they should be converted to Pandas data. The idea is that it should be possible to customize the Record, for example add different fields, and DataStore should still work with this custom record. I tried to put a lot of comments in the source code of the two classes to explain why I made the things the way they are.
Most of the tests are passing, but not all, and this PR is not a finished product. In particular, I didn't touch the readers in any way, and it still uses the Dataset class to convert input to Record objects. Also, I didn't look at exporting data at all, so I assume some things will go wrong there. I did try to remove all use of Dataset other than for import and export, so if you still see that somewhere else, that's a mistake.
Let me know what you think, and feel free to merge/change/build further with this PR!
Here is also a random list of TODO's, that I'll drop here in case someone wants to work more on this:

[ ] Streamline the data readers
[x] Make the data readers use Record.__alternative_column_names__ for the allowed columns
[x] Make the data readers use the order of Record.__alternative_column_names__ to the determine which column to read if there are multiple possible input columns that correspond with a record column (this would solve #1807)
[ ] Move Record.dataset_row and dataset_id to the Base record class? Then any custom implementation would be forced to have them.
[ ] Determine which classes should be put in the __init__.py of asreview.data
[ ] I'm using no connection pool at the moment. That was giving errors when cleaning up the database file during test. We probably want to use the connection pool, but properly close everything on exit?
[ ] Remove the ASReviewData object, or turn it into something that cleans the output from readers
[ ] Create a database constraint that either title or abstract should be present?
[ ] Are the record fields all we need, for example, do we need tags, prior?

Great work!!! And these changes will open-up many new opportunities!

Create a database constraint that either title or abstract should be present?

What will you do if both are missing in the dataset? Yes, the model will neither be able to process this record, nor a human can provide a label based on the title+abstract, but such records sometimes are part of the dataset. In a traditional pipeline, you would search for more information online and select a label based on that information. In our case, this record can not be processed. Will we test for it when users import their dataset and present a message/error that the data contains an _x _ amount of records without title+abstract? Or, will we simple remove such records from the data and continue with only those records that do have either a title and/or an abstract?

Are the record fields all we need, for example, do we need tags, prior?

For post-processing, I think we do need the tags and used_as_prior in the exported data.

See this interesting issue: https://github.com/asreview/asreview-datatools/issues/48. PR #1820 standardizes the input variables, so this should not be an issue. It can be a nice test case to see if everything works as expected.

Locally all my tests pass. I think this is ready to merge. We could do more refactoring and renaming and maybe simplifying the logic in places, but I think that's better to do in new pull requests.

I fixed the docs with a workaround. It's a well-known issue with sqlalchemy and sphinx.

Massive impressive :)


Is there a possible logic problem problem with Promises not defined?

Is there a possible logic problem problem with Promises not defined?

Nothing found.


Solution almost already in the code cell in Message Validation... and something else

Hello Brook,
I found most of the solution already coded in the code cell.
Also, the git commit at the end mentions a math exercise.

Hmm, I think the second problem relates to a known issue with the git script. #463


How to install

Hi
Sorry i couldnt find any other method of trying to get a hold of you.  How would one install this onto a server.
Any quick ref guides

First of all make sure your server meets the requirements:
PHP >= 7.1.3
OpenSSL PHP Extension
PDO PHP Extension
Mbstring PHP Extension
Tokenizer PHP Extension
XML PHP Extension
Ctype PHP Extension
JSON PHP Extension
BCMath PHP Extension
After that you can follow any laravel installation on a server.
https://www.youtube.com/watch?v=6g8G3YQtQt4


Add RPC postgres functions for deleting student quiz submissions

I need to do this often by netid (or email, or name) and meeting slug. It is an annoying manual procedure and prone to error. Need an RPC.

Students often need to have their quizzes "reset".

I think this is done w/ the adding of quiz grade exceptions now. closing


Feature/business metrics

Adding business metrics to orders

feat: Adding business metrics to orders module to show metrics for total number of orders, orders for each watch type and total order price


Exclamation marks and asterisks issue

Issue summary
Some suffixes in public_suffix_list.dat are prefixed with exclamation marks or asterisks and it's an issue.
Some examples:

!city.kawasaki.jp
*.awdev.ca

System informations

Information
Description

Pdp version
6.0.0

PHP version
8.0.2

OS Platform
Ubuntu

Standalone code, or other way to reproduce the problem
echo Rules::fromPath('/public_suffix_list.dat')->resolve('www.city.kawasaki.jp')->registrableDomain()->value();
echo Rules::fromPath('/public_suffix_list.dat')->resolve('www.awdev.ca')->registrableDomain()->value();

Expected result
www.city.kawasaki.jp
www.awdev.ca
Actual result
null
null

Well, actually I don't share your opinion about an expected behavior. But let it be.

Well, actually I don't share your opinion about an expected behavior. But let it be.

Well it is not my opinion, it is the specification and there is a public test suite to confirm that behaviour.

@nyamsprod I've tried to execute my code once again and it seems I was wrong. By bad, sorry. Closing the issue.


capture a piece of JSON/XML as a literal

An important case in GeoSPARQL is to capture fragments of the input XML (GML) or JSON (GeoJSON) as literals with custom datatypes: geo:gmlLiteral, geo:geoJSONLiteral.
This happens when converting eg GML, CityGML; and JSON-FG, CityJSON to RDF.
I think this is easy with RML, but I just want to be sure. Is it possible to add test cases for it?
I faced that problem recently, so I wrote some issues for different tools:

https://github.com/w3c/json-ld-syntax/issues/425
https://github.com/SPARQL-Anything/sparql.anything/issues/452
https://github.com/semantalytics/xsparql/issues/38

Hi @VladimirAlexiev !
I had a look at this issue and I also think it will just work because you can specify the data type with rml:datatype in the mapping. Then you use a rml:reference to capture the part of the file you want as Literal.
However, I'm a bit puzzled how it looks exactly if we want to add a test case for it. Do you have an example of input and expected RDF output? The referenced issues give only a faulty output as an example. I tried to reconstruct from those what you want, but please confirm or give another example:
INPUT
<plu:geometry>
        <gml:Polygon srsName="http://www.opengis.net/def/crs/EPSG/0/25833" srsDimension="2">
          <gml:exterior>
            <gml:LinearRing>
              <gml:posList>385919.31860777 5824690.697477636 ...</gml:posList>
            </gml:LinearRing>
          </gml:exterior>
        </gml:Polygon>
      </plu:geometry>

OUTPUT
<ex:somethign> <ex:polygon> <gml:Polygon srsName=\"http://www.opengis.net/def/crs/EPSG/0/25833\" srsDimension=\"2\">\n                   <gml:exterior>\n                        <gml:LinearRing>\n                            <gml:posList>385919.31860777 5824690.697477636</gml:posList>\n                        </gml:LinearRing>\n                    </gml:exterior>\n                </gml:Polygon>\n            " ^^geo:gmlLiteral;

Moreover, this is RML-Core thing as we only deal in RML-IO with access to Sources and Targets, I moved this issue to the right repository.


[New templating] Templates for multiple files at once

It would be useful, in particular for #100, to be able to declare templates for multiple files at once, with a syntax such as:
type MyTemplate = Template<"main.html, widgets.html, buttons.html">

Done in be5749dea253fc0932ce11eb306e7f18471f9292.


docs(getting_started): Add subsection for type-checking instructions

Issue: https://github.com/denoland/manual/issues/348
This PR updates the getting_started section to incorporate changes from v1.23 regarding type-checking by default.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Warning message when running a container

When running a container a warning message is displayed:

bash: warning: setlocale: LC_ALL: cannot change locale (en_US.UTF-8)

As a side-effect, switching to Ubuntu fixed the issue.


Broken Home Link

The home link on the top of the page links to http://feedback.ld.intricati.com/href= (404) instead of http://feedback.ld.intricati.com

Thanks, it's now fixed.


VD.ah2: check WinExist before WinGetMinMax

On Windows 11, I'm getting some errors on WinGetMinMax that window doesn't exist, I think because we still get the hwnd from the old desktop which doesn't exist on the new desktop, maybe some form of race condition.
Adding the check for WinExist avoids the error, and it works as expected with the window under activated.
Thanks for the script/library!

I was using v1 to v2 converter,
https://github.com/FuPeiJiang/ahk_to_v2.ah2/commit/31dc951759c4439e039f1311f2931c2aef0795d8, now fixed
it was incorrect to translate DetectHiddenWindows, off to DetectHiddenWindows "off" because in v2 that's equivalent to DetectHiddenWindows true, which is the opposite of what you want


US 05.02.01

As an owner, I want help to specify the description for an item by automatically identifying the product barcode from an image taken by the camera and using associated product information such as description.
Requirements/Prerequisites:

US 05.01.01, must be able to scan and process the product's barcode
US 01.01.01, must be able to add the item
Must fill out the add item view with all providable information from the product
User should also be able to go back and retake the picture or select a new image, if necessary
Inherently difficult task that is hard to break up into multiple discrete tasks with real meaning for an end user

US 05.01.01, must be able to scan and process the product's barcode ✅
US 01.01.01, must be able to add the item ✅
M̶u̶s̶t̶ ̶f̶i̶l̶l̶ ̶o̶u̶t̶ ̶t̶h̶e̶ ̶a̶d̶d̶ ̶i̶t̶e̶m̶ ̶v̶i̶e̶w̶ ̶w̶i̶t̶h̶ ̶a̶l̶l̶ ̶p̶r̶o̶v̶i̶d̶a̶b̶l̶e̶ ̶i̶n̶f̶o̶r̶m̶a̶t̶i̶o̶n̶ ̶f̶r̶o̶m̶ ̶t̶h̶e̶ ̶p̶r̶o̶d̶u̶c̶t̶ ==> Broken down into smaller tasks:

Perform type check on the barcode, notify users if not the correct type ✅
Check if the product is already in the database ❌
Get product information by web scraping with the product barcode ❌

User should also be able to go back and retake the picture or select a new image, if necessary ✅

C̶h̶e̶c̶k̶ i̶f̶ t̶h̶e̶ p̶r̶o̶d̶u̶c̶t̶ i̶s̶ a̶l̶r̶e̶a̶d̶y̶ i̶n̶ t̶h̶e̶ d̶a̶t̶a̶b̶a̶s̶e̶ ==> Every type of item has an unique barcode, not single items
Get product information by web scraping with the product barcode ✅
User should also be able to go back and retake the picture or select a new image, if necessary ❌
Unit test❌

User should also be able to go back and retake the picture or select a new image, if necessary ✅
Unit test❌

U̶n̶i̶t̶ t̶e̶s̶t̶ => The same as test in 05.01.01 and 05.01.02. Also updated the test file


Parallel tests are broken

I get random test failures, which I think are due to separate project's trying to setup the database at the same time.
For instance :

project A starts
project A : seeds the authors table
project B starts
project B: seeds the authors table --> duplicate key violation
project A cleanup
etc.

Quick-fix solution: build and/or test with one worker.

PGHOST=localhost PGDATABASE=caqti_study PGPORT=5433 dune build @all @runtest -j1

Or:
PGHOST=localhost PGDATABASE=caqti_study PGPORT=5433 dune build
PGHOST=localhost PGDATABASE=caqti_study PGPORT=5433 dune runtest -j1

It'd be nice to find a better solution. Maybe something like:

setup the database once (create and seed data for all the apps)
run each test into a transaction, then they wouldn't influence each other

Dune has a (locks VIRTUAL_PATH) attribute which can be used to avoid conflicting concurrent runs (cf the Caqti testsuite).  But I'll think some more about it and comment, since I'm having similar issues with test suites depending on DBs (and not just in Caqti).
Another motivation I have for rethinking this is that if someone (like myself) already has set up access to a personal database, then we want to avoid accidentally creating tables in its public namespace when the PG* variables are unset.

Thanks for the tip!
To avoid accidentally creating or deleting tables, I can require mandatory prefixed env vars, something like:

CAQTI_STUDY_PGHOST
CAQTI_STUDY_PGDATABASE
CAQTI_STUDY_PORT
etc.

Would that work for you?
Env vars are pretty flexible IMO, and make it so we never have to worry about committing sensitive data by accident.
We can simulate having something close to a config file for free using a tool such as envdir, from the daemontools package.
$ echo 1 >/tmp/env/CAQTI_HELLO
$ echo 2 >/tmp/env/CAQTI_WORLD
$ envdir /tmp/env/ env | grep CAQTI
CAQTI_WORLD=2
CAQTI_HELLO=1

# Or if that wasn't clear
$ envdir /tmp/env/ bash -c 'echo $CAQTI_HELLO'
1

So one could run the test suite like such:
envdir ~/env/caqti-study/ dune runtest

I think adding a prefix to the variables is good, since it allows people to set the variables globally if they wish, without affecting other applications.  However, we would also need to drop the postgres:// fall-back, since that may point to the wrong database.  But we could consider making the fall-back postgres://localhost:5433/local_db for convenience.  It's already the default in the makefile, and I think it's safe enough, since the port is non-standard and the database name is unique enough (we could also call it caqtistudy or caqti_study).
(I didn't know about envdir, but an alternative is to put the environment variables in a script which executes the rest  of the command-line at the bottom.)

However, we would also need to drop the postgres:// fall-back, since that may point to the wrong database

Sounds good to me. I'll make this a failing condition.

an alternative is to put the environment variables in a script which executes the rest of the command-line at the bottom

Yes that's another way to do it. I'll write a little wrapper script to make things easy to manage.


Guidelines: As an end user I can clearly see who is the owner of this app

Introduction
There is a need both from app owners and end users that our apps can be styled/customized in some way. The main need is to communicate who the responsible app owner is.
Design proposal
We change the background color in our apps to a more neutral color.
Our blue color can be a little too "Altinn" and might take away from who is owning the application. We should also remove the "organization" icon in the header.
We should make a new component that communicate to the user who is owning the app
This component should present the user with the app owners name, logo and color. It could also include contact information and general information about the app. The main goal of this component is to clearly communicate who is behind the app and give them direction if they need help.
We do not recommend changing the CSS for individual apps that use our Altinn design.
If we open up our CSS for bigger changes this would hurt the consistency between apps. The consistent design between app owners is something we should strive for. The need here is to communicate with the user who is owning the app. We think it is better to create one solution that can be used and improved by everyone, instead of many different solutions trough CSS changes. One bonus is that it will make it easier for us to control accessibility across apps on our platform.
Advantages if we can maintain a consistent user experience:

When users encounter an app/service, they bring their own expectations from previous experiences with them. When these expectations are met, confidence is built, and they will know how to interact with the components. When users don’t need to learn new interactions, their focus will be entirely on the apps content and messaging.

NN/g
Acceptance criterea

[x] Background color changed
[x] navigation buttons fixed
[ ] Fix header

[x] change color
[ ] service owner
[x] remove corp icon

[ ] Footer with contact info

In scope

[x] Decide on solution
[x] Design component
[x] Mobile design
[x] specify new background color

#138

Footer is done

I'm closing this as all tasks are done within the scope of this epic. However a more extended version with theming support will be created to allow for even more standard customization and alignment with a service owners design system.


New version: spacelift-io.spacectl 1.7.0

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:
{"packageData":{"DefaultLocale":{"PackageLocale":"en-us","PackageName":"spacectl","Publisher":"spacelift-io"},"installerHash":"52f4e19b-019b-46ff-b00a-cb975ba73f4d"},"submissionData":{"submissionIdentifier":"68e66a91-bb0a-4eb0-9252-d20b550382cf"},"supportedMetadataVersion":"1.2","version":"1.0"}

Installation failed with exit code -1978335187
2024-11-12 13:17:36.710 [CLI ] Manifest fields: Name [spacectl], Version [1.7.0]
2024-11-12 13:17:36.761 [CLI ] Generated temp download path: C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\spacelift-io.spacectl.1.7.0\656336fd5239fd2b9eb76908e0b126dee259600a68b86918e8dbec384d76474b
2024-11-12 13:17:36.762 [CORE] Downloading to path: C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\spacelift-io.spacectl.1.7.0\656336fd5239fd2b9eb76908e0b126dee259600a68b86918e8dbec384d76474b
2024-11-12 13:17:36.763 [CORE] DeliveryOptimization downloading from url: https://github.com/spacelift-io/spacectl/releases/download/v1.7.0/spacectl_1.7.0_windows_amd64.zip
2024-11-12 13:17:38.937 [CORE] Started applying motw to C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\spacelift-io.spacectl.1.7.0\656336fd5239fd2b9eb76908e0b126dee259600a68b86918e8dbec384d76474b with zone: 3
2024-11-12 13:17:38.981 [CORE] Started applying motw using IAttachmentExecute to C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\spacelift-io.spacectl.1.7.0\656336fd5239fd2b9eb76908e0b126dee259600a68b86918e8dbec384d76474b
2024-11-12 13:17:40.966 [FAIL] Installer failed security check. Url: https://github.com/spacelift-io/spacectl/releases/download/v1.7.0/spacectl_1.7.0_windows_amd64.zip Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
Automated error analysis suggests 80004005 may mean Resource missing
(Automated response - build 897.)

@wingetbot run

Automatic Validation ended with:
{"packageData":{"DefaultLocale":{"PackageLocale":"en-us","PackageName":"spacectl","Publisher":"spacelift-io"},"installerHash":"a6a85b93-0e4d-4df7-9e09-8422504ed25b"},"submissionData":{"submissionIdentifier":"8f5b33ca-2fba-4a9d-8304-0780ac347f53"},"supportedMetadataVersion":"1.2","version":"1.0"}

Installation failed with exit code -1978335187
2024-11-13 11:04:54.002 [CLI ] Manifest fields: Name [spacectl], Version [1.7.0]
2024-11-13 11:04:54.090 [CLI ] Generated temp download path: C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\spacelift-io.spacectl.1.7.0\656336fd5239fd2b9eb76908e0b126dee259600a68b86918e8dbec384d76474b
2024-11-13 11:04:54.093 [CORE] Downloading to path: C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\spacelift-io.spacectl.1.7.0\656336fd5239fd2b9eb76908e0b126dee259600a68b86918e8dbec384d76474b
2024-11-13 11:04:54.094 [CORE] DeliveryOptimization downloading from url: https://github.com/spacelift-io/spacectl/releases/download/v1.7.0/spacectl_1.7.0_windows_amd64.zip
2024-11-13 11:04:55.180 [CORE] Started applying motw to C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\spacelift-io.spacectl.1.7.0\656336fd5239fd2b9eb76908e0b126dee259600a68b86918e8dbec384d76474b with zone: 3
2024-11-13 11:04:55.269 [CORE] Started applying motw using IAttachmentExecute to C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\spacelift-io.spacectl.1.7.0\656336fd5239fd2b9eb76908e0b126dee259600a68b86918e8dbec384d76474b
2024-11-13 11:04:57.527 [FAIL] Installer failed security check. Url: https://github.com/spacelift-io/spacectl/releases/download/v1.7.0/spacectl_1.7.0_windows_amd64.zip Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
Automated error analysis suggests 80004005 may mean Resource missing
(Automated response - build 897.)

@wingetbot run

Automatic Validation ended with:
{"packageData":{"DefaultLocale":{"PackageLocale":"en-us","PackageName":"spacectl","Publisher":"spacelift-io"},"installerHash":"bd4821d8-dc0f-4e6b-8bb9-e71472db41cc"},"submissionData":{"submissionIdentifier":"9ba4e6a9-74fe-4408-8a3c-11bc08fa513c"},"supportedMetadataVersion":"1.2","version":"1.0"}

Installation failed with exit code -1978335187
2024-11-14 10:18:13.081 [CLI ] Manifest fields: Name [spacectl], Version [1.7.0]
2024-11-14 10:18:13.158 [CLI ] Generated temp download path: C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\spacelift-io.spacectl.1.7.0\656336fd5239fd2b9eb76908e0b126dee259600a68b86918e8dbec384d76474b
2024-11-14 10:18:13.159 [CORE] Downloading to path: C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\spacelift-io.spacectl.1.7.0\656336fd5239fd2b9eb76908e0b126dee259600a68b86918e8dbec384d76474b
2024-11-14 10:18:13.160 [CORE] DeliveryOptimization downloading from url: https://github.com/spacelift-io/spacectl/releases/download/v1.7.0/spacectl_1.7.0_windows_amd64.zip
2024-11-14 10:18:14.894 [CORE] Started applying motw to C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\spacelift-io.spacectl.1.7.0\656336fd5239fd2b9eb76908e0b126dee259600a68b86918e8dbec384d76474b with zone: 3
2024-11-14 10:18:14.992 [CORE] Started applying motw using IAttachmentExecute to C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\spacelift-io.spacectl.1.7.0\656336fd5239fd2b9eb76908e0b126dee259600a68b86918e8dbec384d76474b
2024-11-14 10:18:18.775 [FAIL] Installer failed security check. Url: https://github.com/spacelift-io/spacectl/releases/download/v1.7.0/spacectl_1.7.0_windows_amd64.zip Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
Automated error analysis suggests 80004005 may mean Resource missing
(Automated response - build 897.)

@wingetbot run


Argument parser should validate all arguments

Currently the argument parser only validates a single argument at a time. This is obviously not optimal. It should validate all arguments and provide errors for all of them at the same time. As such, it should provide a list of errors, one per failed field. Perhaps with an option to enable/disable multiple errors such as fail_fast=True or validate_all=True.

Is this still worth fixing due to #335?

Taken care of by #431


Recognize combining accents in sanitize_filename()

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.06.27. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2016.06.27

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
$ youtube-dl -v <your command line>
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2016.06.27
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue required an account credentials please provide them or explain how one can obtain them.

Hi,
There are extra underscores in the filename in restrict mode when there should not be :
$ youtube-dl --version
2016.06.27
$ youtube-dl -f 18 --get-filename --restrict-filename hYDQZleciXE
Joseph_Prince_-_La_Gra_ce_est_Supe_rieure_a_La_Loi_-_FR__18__hYDQZleciXE.mp4

The command line youtube-dl -f 18 --get-filename --restrict-filename hYDQZleciXE should return:
Joseph_Prince_-_La_Grace_est_Superieure_a_La_Loi_-_FR__18__hYDQZleciXE.mp4
Can you have a look ?

The letter â is in fact a plain alphabet letter a with "combining circumflex accent". Such combinations are not supported yet.
Reference: http://www.utf8-chartable.de/unicode-utf8-table.pl?start=768&number=128

Hi, why does the "é" appears like "_" instead of "e" ?

é is not turned into a underscore. Instead, é is a combination of e and accent symbol, and the accent symbol is turned into _, so the result is e_.
That's indeed a problem and seems it's hard to fix. Apparently an algorithm to identify accent symbols is necessary.


Check if all of our artifacts contain a LICENSE file and add one where necessary

Task Description
We need to check, if all of our artifacts contain a META-INF/LICENSE file and add it, if wherever this may be missing.
You can find which modules don't have such a file quite easily:

Build the artifacts:

mvn clean install -DskipTests

Check which artifacts don't have the META-INF/LICENSE (ignore tests artifacts, but it should be in the default jar, as well as in the sources one):

#!/bin/bash

cd $1

jarFiles=`find . -name \*.jar`

for jarFile in $jarFiles
do
    echo "Scanning ${jarFile}..."
    matches=`jar -tvf ${jarFile} | grep $2`

    if [ -n "$matches" ]; then
	echo " --> Matches found <-- "
	jar -tvf ${jarFile} | grep $2
    fi
done

hpiFiles=`find . -name \*.hpi`

for hpiFile in $hpiFiles
do
    echo "Scanning ${hpiFile}..."
    matches=`jar -tvf ${hpiFile} | grep $2`

    if [ -n "$matches" ]; then
        echo " --> Matches found <-- "
        jar -tvf ${hpiFile} | grep $2
    fi
done

Of course, you can do the check by hand, if you like.
Tasks
The following tasks will need to be carried out:

[ ] Scan all the artifacts of the strongbox projects and check if they have a src/main/resources/META-INF/LICENSE file (this is only for modules that have sources).
[ ] Copy strongbox/strongbox/blob/master/strongbox-commons/src/main/resources/META-INF/LICENSE to src/main/resources/META-INF/LICENSE under each of module where this is missing.
[ ] Do the above steps for the strongbox-db project.

Help

Our chat
Points of contact:

@carlspring
@sbespalov
@steve-todorov

may I work on this issue?

Hi @BhuwanUpadhyay ,
Sure! Feel free to dig in! :)


Logo for 'sub-configuration' viewers.

How do I display a different logo in the title section of a 'sub-configuration'?
titles: {
header: 'Aloha Threat Zone Widget Preview',
subHeader: 'This is an example of using Web App Builder widgets with CMV',
pageTitle: 'Aloha Threat Zone Widget Preview'
},

I tried this with no luck.
titles: {
header: 'CMV WAB Widgets Test',
subHeader: 'This is an example of using Web App Builder widgets with CMV',
pageTitle: 'CMV WAB Widgets Test',
logo: true,
path: './cmv/images/Knight_Final_40x40.png'
},

This is a good possible addition for cmv-app. Closing the issue here since this is unrelated to cmv-wab-widgets.


Admin faction power isn't used for max power when using auto-claim

Expected behavior
For the faction auto-claim feature to also correctly use the admin max power, like the standard /factions claim add does
Observed/actual behavior
When attempting to claim a new chunk, the text "Not enough faction power to claim chunk, autoclaim toggled off" is shown, even though there is enough max power
Steps/models to reproduce
Fill the maximum possible claimed chunks, and then add more via the admin command. Then, toggle on auto-claim and attempt to claim another chunk
Version
I built the mod myself using the current version of the code in the repository
Other
I believe this could be fixed by changing the line:
https://github.com/ickerio/factions/blob/a828619e044afe4821be38fa900a30803db53087/src/main/java/io/icker/factions/core/WorldManager.java#L37
to use the same code as detailed in the commit fixing #103: https://github.com/ickerio/factions/commit/cdbebb9e4fa4500f7a688e51b278df96017d9269

To avoid this problem in other places, it may be a good idea to add a method that calculates the maxPower directly in the Faction class (e.g. it comes up for /factions info as well):
https://github.com/ickerio/factions/blob/a828619e044afe4821be38fa900a30803db53087/src/main/java/io/icker/factions/command/InfoCommand.java#L77

Duplicate of #103


amples/bevformer/tiny/pth2onnx.sh  error,

hi, I run ./amples/bevformer/tiny/pth2onnx.sh, my  terminal displayed as follows: how do I  fix it? Thanks
my env:
bev-mmdet3d               0.1
comm                      0.1.4
mmcv-full                 1.5.0
mmdeploy                  0.14.0
mmdet                     2.25.1
pytorch-quantization      2.1.2
torch                     1.10.0+cu113
torchaudio                0.10.0+rocm4.1
torchvision               0.11.0+cu113
TensorRT-<IP_ADDRESS>
cuda-11.3
(bevformer) lin@PC:~/code/BEVFormer_tensorrt$ sh samples/bevformer/tiny/pth2onnx.sh -d 0
Running on the GPU: 0
WARNING: Logging before flag parsing goes to stderr.
W0908 16:44:15.548062<PHONE_NUMBER>08640 collision.py:11] No FCL -- collision checking will not work
Loaded tensorrt plugins from /home/lin/code/BEVFormer_tensorrt/TensorRT/lib/libtensorrt_ops.so
Traceback (most recent call last):
  File "tools/pth2onnx.py", line 9, in <module>
    from det2trt.convert import pytorch2onnx
  File "/home/lin/code/BEVFormer_tensorrt/./det2trt/__init__.py", line 1, in <module>
    from .models import *
  File "/home/lin/code/BEVFormer_tensorrt/./det2trt/models/__init__.py", line 1, in <module>
    from .backbones import *
  File "/home/lin/code/BEVFormer_tensorrt/./det2trt/models/backbones/__init__.py", line 1, in <module>
    from .csp_darknet import CSPDarknetQ
  File "/home/lin/code/BEVFormer_tensorrt/./det2trt/models/backbones/csp_darknet.py", line 11, in <module>
    from ..utils import CSPLayer
  File "/home/lin/code/BEVFormer_tensorrt/./det2trt/models/utils/__init__.py", line 2, in <module>
    from .onnx_ops import rotate, rotate_p, rotate_p2
  File "/home/lin/code/BEVFormer_tensorrt/./det2trt/models/utils/onnx_ops.py", line 400, in <module>
    torch.onnx.register_custom_op_symbolic("aten::grid_sampler", grid_sampler_sym, 13)
  File "/home/lin/software/miniconda3/envs/bevformer/lib/python3.8/site-packages/torch/onnx/__init__.py", line 404, in register_custom_op_symbolic
    utils.register_custom_op_symbolic(symbolic_name, symbolic_fn, opset_version)
  File "/home/lin/software/miniconda3/envs/bevformer/lib/python3.8/site-packages/torch/onnx/utils.py", line 1255, in register_custom_op_symbolic
    ns, op_name = get_ns_op_name_from_custom_op(symbolic_name)
  File "/home/lin/software/miniconda3/envs/bevformer/lib/python3.8/site-packages/torch/onnx/utils.py", line 1245, in get_ns_op_name_from_custom_op
    raise RuntimeError("Failed to register operator {}. The domain {} is already a used domain."
RuntimeError: Failed to register operator aten::grid_sampler. The domain aten is already a used domain.
(bevformer) lin@PC:~/code/BEVFormer_tensorrt$

Maybe you should upgrade your pytorch to 1.12.1

ok, I try it，thanks

ok, I try it，thanks

I meet same error，are you solved? just upgrade pytorch to 1.12.1? Is it work?,Thanks!


[SUPPORT] FileNotFoundException would happen occasionally after cherrypick HUDI-1517

I cherry pick HUDI-1517 into internal HUDI version.
And find a FileNotFoundException during read latest snapshot of a MOR table.

The exception would happen if enable spark speculative feature, there exists concurrent writer and reader. For example:

Job1 is writing to a MOR table and not finished yet. It enables spark speculative feature.
Job2 is reading the latest snapshot from the MOR table, when it call getLatestMergedFileSlicesBeforeOrOn, it might list the log files which are written by speculative attempt task in Job1.
Job1 is finished, deletes the log files which are written by slow speculative tasks.
Job2 throws the FileNotFoundException when it read the log file which is already deleted in step3

@ad1happy2go Thanks for response. I would try it on master later. However I believe it would exist in master branch too because the related code path does not change.
I have offline discussion with @guanziyue , he would try to fix this problem recently.

Thanks @beyond1920 .
Created a JIRA for tracking  - https://issues.apache.org/jira/browse/HUDI-6484
@guanziyue You can use this JIRA for the fix if you have not created yet.

The issue [HUDI-1517|https://issues.apache.org/jira/browse/HUDI-1517] is reverted.
So the FileNotFoundException would not appear temporarily.
And would continue to solve this problem [HUDI-1517|https://issues.apache.org/jira/browse/HUDI-1517] in 1.0.0.


Add no-absolute-path rule (fixes #530)

Add no-absolute-path rule (fixes #530)

Coverage increased (+0.03%) to 97.757% when pulling b7d454ce6f03968f9af33ca9d22f5366e56cbc94 on no-absolute-path into 28dbed8ab6f6fefe9219c54ec87161168da518c4 on master.

Coverage increased (+0.03%) to 97.757% when pulling b7d454ce6f03968f9af33ca9d22f5366e56cbc94 on no-absolute-path into 28dbed8ab6f6fefe9219c54ec87161168da518c4 on master.

LGTM! :shipit:

Coverage increased (+0.03%) to 97.757% when pulling b9ec2efd91260a6767ce290a8a50bbedd7e37348 on no-absolute-path into 28dbed8ab6f6fefe9219c54ec87161168da518c4 on master.


Add teams 2/3: Add functions to manage teams

This a follow up on  https://github.com/paris-saclay-cds/ramp-board/pull/545 and adds ramp_database functions to create/join/leave and invite to teams.
Toward https://github.com/paris-saclay-cds/ramp-board/issues/505
Again it should be backward compatible, as long as those functions are not called (which will be done in a third PR).
TODO:

[ ]  add tests for the few lines not included in test coverage.

Hello @rth. I had a quick look at these diffs as I said over email. Sorry if these inputs are not relevent : I am not very familiar with the code base.
I am under the impression that the current implementation is slighly different than what you described in https://github.com/paris-saclay-cds/ramp-board/issues/505#issuecomment-853725196
The test_team::test_add_signup_leave_team()  shows the following workflow:

user creates a team
team is signed-up to an event

However the initial plan was that a team is only created in the context of a given event. Did this change ?
If the initial plan is still what you are aiming for :

I think the method tools/team.py::add_team() should be renamed to add_team_to_event() and create at the same time the Team and TeamEvent objects. It should not be used to created is_individual teams (as these are created when a user is created). It could also check that the user making this request is allready registered in this event.
as a consequencesign_up_team() should not require the optional user_name param as this would only be used
for is_individual teams.

Another note : I think you might still end up with a user belonging to several multi-user teams in a single event:

there is a mechanism in tools/teams.py::respond_team_invite() to leave the previously joined team.
however I think a user can create multiple teams in a single event (and be the admin of each of those).

this can be usefull if a teacher wants to create all the teams
however it will be difficult for this user to make a submission: which team are they submitting for ? (select_event_team_by_user_name returns the first team only).

Let me know if I can help

Thanks a lot for the review @BenjaminHabert, will try to respond to it soon.


GO-810 flagged more languages for geocoding

Objective
Correctly mark all supported languages for geocoding.
Checklist

[x] I have added relevant info to the CHANGELOG.md

@zdila Looking good, but can you bump the patch version, then we just merge and publish

@zdila Looking good, but can you bump the patch version, then we just merge and publish

done!


Network example produces different results on Linux vs Windows

I ran the SimpleReplication server/client example as outlined in this URL:
http://github.nfsdb.org/
The key difference between the example shown on the website and mine is that I declare a serverConfig or clientConfig with a setIdName("eth0"):
[server]
JournalServer server = new JournalServer(new ServerConfig() {{
setIfName("eth0");
}}, factory);
[client]
final JournalClient client = new JournalClient(new ClientConfig() {{
setIfName("eth0");
}},factory);
Expected Behavior:

As soon as I declare a JournalReader on the client, the db file must be immediately synced.
As the server publishes new data, it should be synchronized on the client side.

On Windows (as expected), as soon as I run the SimpleReplicationClientMain, I can see the sync messages as shown below:
C:\VG\dev\NFSDBTest\src>java SimpleReplicationClientMain C:\VG\dev\NFSDBTest\data
Jan 05, 2015 4:51:42 PM com.nfsdb.journal.net.mcast.AbstractOnDemandPoller
INFO: Polling on name:eth0 (Intel(R) 82579LM Gigabit Network Connection)
Jan 05, 2015 4:51:42 PM com.nfsdb.journal.net.config.ClientConfig
INFO: Connected to /<IP_ADDRESS>:7075
Client started
took: 1978, count=1000000
Jan 05, 2015 4:51:42 PM com.nfsdb.journal.net.StatsCollectingReadableByteChannel
INFO: received 39000097 bytes @ 153.059222 MB/s from: /<IP_ADDRESS>:7075 [1939 c
alls]
took: 406, count=1000000
Whereas running the same on a Linux machine, you can see that the client connected fine but no sync messages for a long time. And then about 15-min later, you start seeing the sync messages but the count remains zero. The debug messages keeps repeating for a minute and then pauses for another minute and this keeps repeating in a cycle.
DEBUG on {LINUX} SERVER:
INFO: sent 71 bytes @ Infinity MB/s to: /<IP_ADDRESS>:43112 [4 calls]
Jan 05, 2015 7:55:51 PM com.nfsdb.journal.net.StatsCollectingWritableByteChannel
DEBUG on {LINUX} CLIENT:
took:<PHONE_NUMBER>619, count=0
Jan 05, 2015 7:55:48 PM com.nfsdb.journal.net.StatsCollectingReadableByteChannel
INFO: received 57 bytes @ 0.001394 MB/s from: /<IP_ADDRESS>:7075 [3 calls]
*** Version Details ***
nfsdb-core-2.1.1-SNAPSHOT.jar
disruptor-3.3.0.jar
$ lsb_release -a
No LSB modules are available.
Distributor ID:    Ubuntu
Description:    Ubuntu 14.04.1 LTS
Release:    14.04
Codename:    trusty
$ java -version
java version "1.8.0_25"
Java(TM) SE Runtime Environment (build 1.8.0_25-b17)
Java HotSpot(TM) 64-Bit Server VM (build 25.25-b02, mixed mode)
*** Noticed the same behavior on Ubuntu running OpenJDK ***
Please advise. I can try this on a CentOS system as well in the next few days and publish my results.

I cannot think of an obvious reason why this shouldn't be working. I'm going to try to reproduce this and come back asap.

I suspect this problem is to do with both client and server running on the same machine and using external interface. I would assume that network driver has to be smart enough to realise that this is "loopback" setup, which it may or may not do. Sure way to test it out is to use separate machines or explicit loopback interface.
I suspect Ubuntu driver isn't as "smart" as windows one.
I'll try to test this theory but this behaviour could be specific to network driver you are using.

Vlad,
it would be great to figure out why this is the case and document a work around (if such is the case).
Would setting both the server and client (config) with setHostname("localhost") do the trick? In my /etc/hosts file, I have localhost defined as <IP_ADDRESS>. Please advise.
I can try this out tonight and let you know my results. Also if you're unable to reproduce this issue on your side, I am in the EST timezone so if you need me to setup a Join.me session, I can do that as well. It will have to be in the evenings in EST. Let me know.

Using localhost will most likely work fine. i should be able to try Ubuntu tonight as I just realised a quick way to do it. Might be easier than wasting your time!
If you are looking for same box replication there is a way to do it without network. I'll give you a code example if you like?

Just tried the the following for both the serverConfig and clientConfig. Same results. No sync.
**** 1st test ****
setIfName("eth0");
setHostname("localhost");

[Server]
$ ./server-run.sh
Jan 06, 2015 4:48:24 PM com.nfsdb.journal.net.config.ServerConfig
INFO: Server is now listening on localhost/<IP_ADDRESS>:7075 [eth0]
Publishing: Jan 06, 2015 4:48:24 PM com.nfsdb.journal.net.mcast.AbstractOnDemandSender
INFO: Sending on: name:eth0 (eth0)
Jan 06, 2015 4:48:26 PM com.nfsdb.journal.net.JournalServer
INFO: Connected: /<IP_ADDRESS>:43386
.......... [Done]
[Client]
$ ./client-run.sh
Jan 06, 2015 4:48:26 PM com.nfsdb.journal.net.config.ClientConfig
INFO: Connected to localhost/<IP_ADDRESS>:7075
Client started

[server-run.sh]
#!/bin/bash
java -classpath "/datastore/NFSDBTest/lib/disruptor-3.3.0.jar:/datastore/NFSDBTest/lib/nfsdb-core-2.1.1-SNAPSHOT.jar:/datastore/NFSDBTest/src/" SimpleReplicationServerMain /datastore/NFSDBTest/data
[client-run.sh]
#!/bin/bash
java -classpath "/datastore/NFSDBTest/lib/disruptor-3.3.0.jar:/datastore/NFSDBTest/lib/nfsdb-core-2.1.1-SNAPSHOT.jar:/datastore/NFSDBTest/src/" SimpleReplicationClientMain /datastore/NFSDBTest/data2

**** 2nd test ****
setHostname("localhost");
setEnableMulticast(false);
#Needed to set Multicast to false since multicast was not allowed in the "lo" interface

$ ./server-run.sh
Jan 06, 2015 5:03:11 PM com.nfsdb.journal.net.config.ServerConfig
INFO: Server is now listening on localhost/<IP_ADDRESS>:7075 [lo]
Publishing: .Jan 06, 2015 5:03:14 PM com.nfsdb.journal.net.JournalServer
INFO: Connected: /<IP_ADDRESS>:43474
......... [Done]
$ ./client-run.sh
Jan 06, 2015 5:03:14 PM com.nfsdb.journal.net.config.ClientConfig
INFO: Connected to localhost/<IP_ADDRESS>:7075
Client started

thank you very much for trying these scenarios!
I managed to reproduce the problem. You can use "eth0" as ifName on both client and server, just increase receive buffer size on Client:
        setSoRcvBuf(10*1024*1024);

Here is my output:
Jan 07, 2015 12:03:49 AM com.nfsdb.net.mcast.AbstractOnDemandPoller
INFO: Polling on name:eth0 (eth0)
Jan 07, 2015 12:03:50 AM com.nfsdb.net.config.ClientConfig
WARNING: SO_SNDBUF value is ignored
Jan 07, 2015 12:03:50 AM com.nfsdb.net.config.ClientConfig
WARNING: SO_RCVBUF value is ignored
Jan 07, 2015 12:03:50 AM com.nfsdb.net.config.ClientConfig
INFO: Connected to /<IP_ADDRESS>:7075
Client started
Jan 07, 2015 12:03:50 AM com.nfsdb.net.StatsCollectingReadableByteChannel
INFO: received 117015697 bytes @ 306.579299 MB/s from: /<IP_ADDRESS>:7075 [352 calls]
(this is newer version but you should get similar results)
If you do get warnings - ignore them, receive buffer size is evidently not ignored.
Althogh replication between two computers works better with default buffer, on my WLAN it was done at 12MB/s. After increasing receive buffer size to 10MB replication speed went to 75MB/s. So it does make a difference for both cases.
Let me know if this works for you too.

Yes, that works! Thank you very much for helping me in debugging this. I would have never guessed at setting the "receive-buffer-size" parameter.
Can you explain what made this work for my understanding?
In the code, I see that DEFAULT_SO_RCVBUF = 8192. In the code, I'm overriding the value to  10MB<PHONE_NUMBER>). Assumption is that the default value is represented as 8192MB. So how does going from 8MB to 10MB solve this problem? And also is there an optimal number to use based on my machine/nic interface?
On my Ubuntu machine here are the params for
$ cat /proc/sys/net/core/rmem_default
212992
$ cat /proc/sys/net/core/rmem_max
212992
Is there a better value I must be using to maximize the bytes sent per payload?
Secondly, will I always need to do this on Linux or will future versions of NFSdb account for this?
Please advise.
I'm including my debug output just in case others run into this.
Thanks again for your help.
[server]
$ ./server-run.sh
Jan 06, 2015 7:19:10 PM com.nfsdb.journal.net.config.ServerConfig
INFO: Server is now listening on /<IP_ADDRESS>:7075 [eth0]
Jan 06, 2015 7:19:10 PM com.nfsdb.journal.net.mcast.AbstractOnDemandSender
INFO: Sending on: name:eth0 (eth0)
Publishing: Jan 06, 2015 7:19:12 PM com.nfsdb.journal.net.JournalServer
INFO: Connected: /<IP_ADDRESS>:44373
.Jan 06, 2015 7:19:13 PM com.nfsdb.journal.net.StatsCollectingWritableByteChannel
INFO: sent 351000111 bytes @ 319.408197 MB/s to: /<IP_ADDRESS>:44373 [346 calls]
Jan 06, 2015 7:19:15 PM com.nfsdb.journal.net.StatsCollectingWritableByteChannel
INFO: sent 39000111 bytes @ 523.850763 MB/s to: /<IP_ADDRESS>:44373 [53 calls]
.Jan 06, 2015 7:19:16 PM com.nfsdb.journal.net.StatsCollectingWritableByteChannel
INFO: sent 39000111 bytes @ 39.992908 MB/s to: /<IP_ADDRESS>:44373 [52 calls]
.Jan 06, 2015 7:19:18 PM com.nfsdb.journal.net.StatsCollectingWritableByteChannel
INFO: sent 39000111 bytes @ 19.783726 MB/s to: /<IP_ADDRESS>:44373 [49 calls]
.Jan 06, 2015 7:19:20 PM com.nfsdb.journal.net.StatsCollectingWritableByteChannel
INFO: sent 39000111 bytes @ 17.898655 MB/s to: /<IP_ADDRESS>:44373 [53 calls]
.Jan 06, 2015 7:19:23 PM com.nfsdb.journal.net.StatsCollectingWritableByteChannel
INFO: sent 39000111 bytes @ 18.002616 MB/s to: /<IP_ADDRESS>:44373 [52 calls]
.Jan 06, 2015 7:19:25 PM com.nfsdb.journal.net.StatsCollectingWritableByteChannel
INFO: sent 39000111 bytes @ 17.941825 MB/s to: /<IP_ADDRESS>:44373 [49 calls]
.Jan 06, 2015 7:19:27 PM com.nfsdb.journal.net.StatsCollectingWritableByteChannel
INFO: sent 39000111 bytes @ 17.830012 MB/s to: /<IP_ADDRESS>:44373 [53 calls]
.Jan 06, 2015 7:19:29 PM com.nfsdb.journal.net.StatsCollectingWritableByteChannel
INFO: sent 39000111 bytes @ 17.643930 MB/s to: /<IP_ADDRESS>:44373 [48 calls]
.Jan 06, 2015 7:19:31 PM com.nfsdb.journal.net.StatsCollectingWritableByteChannel
INFO: sent 39000111 bytes @ 17.593853 MB/s to: /<IP_ADDRESS>:44373 [53 calls]
.
To be honest it was a bit of a hunch. Bmon was showing that despite setting ifName to "eth0" traffic was going over localhost and it was going at a very slow rate, 50b/s. I know for a fact that i'm writing large buffers to SocketChannel, something like 16MB buffer at a time. There could be two reasons for slow transfer speed, either packet loss or tiny transfer buffer. "lo" wouldn't be losing packets so I tried buffer.
1MB works just as well as 10MB. I'm on windows soRcvBuf makes no difference. I'll try to find optimal value in a few mins, need to flick back to ubuntu :)

32K seems to be the minimum. Below that things grind to a halt, above that there is no increase in performance.

Thank you once again for all your help in debugging this. I appreciate your help very much.


Enhancement to trial duration display

Can we enhance the time duration display? It's really hard to "guess" how many hours or how many days from a number with 9 digits.

Plan: change to :  h min s ; like 1h20min30s

@ultmaster can we close this issue now?

No. It hasn't been done yet. I don't know why people forget it.


wont-fix -> stale

wont-fix as a label seems a bit harsh, and probably not our intention. Let's change it to stale.

wontfix is actually the default for the stale app. If it's too harsh for us to use here, we should talk about changing the default.
I primarily chose it because it is a label that already exists on all repos and basically means "we aren't intending to fix this". It maybe caries more of a connotation like "it' ain't broke".

Hmm, alright that reasoning makes it seem more fair in my mind. It's probably not worth changing the default since stale is already pretty popular/used. 👍