[Impeller] Run perf tests with Skia and again with Impeller

Example of how to run devicelab tests with Skia and again with Impeller without creating new builders.

I didn't write tests: https://github.com/flutter/flutter/blob/master/dev/devicelab/test/perf_tests_test.dart
This likely means impeller_ will be prepended in data points forever. If we ever remove impeller_ (like when Skia backend is removed) then it will get smooshed together with the old Skia data.
This doesn't handle only wanting to run impeller but not Skia.  That should be easy to add.

On my machine complex_layout_scroll_perf_ios__timeline_summary test time went from ~21s to ~41s.
Benchmark data running complex_layout_scroll_perf_ios__timeline_summary:
"average_frame_build_time_millis": 0.5411564245810051,
"90th_percentile_frame_build_time_millis": 0.514,
"99th_percentile_frame_build_time_millis": 5.54,
"worst_frame_build_time_millis": 6.051,
"missed_frame_build_budget_count": 0,
"average_frame_rasterizer_time_millis": 2.172000000000001,
"90th_percentile_frame_rasterizer_time_millis": 1.798,
"99th_percentile_frame_rasterizer_time_millis": 17.822,
"worst_frame_rasterizer_time_millis": 35.345,
"missed_frame_rasterizer_budget_count": 6,
"frame_count": 179,
"frame_rasterizer_count": 179,
"new_gen_gc_count": 12,
"old_gen_gc_count": 2,
"frame_build_times": [
(omitted)
],
"frame_rasterizer_times": [
(omitted)
],
"frame_begin_times": [
(omitted)
],
"frame_rasterizer_begin_times": [
(omitted)
],
"average_vsync_transitions_missed": 1.125,
"90th_percentile_vsync_transitions_missed": 2.0,
"99th_percentile_vsync_transitions_missed": 2.0,
"average_vsync_frame_lag": 84.87222222222222,
"90th_percentile_vsync_frame_lag": 117.0,
"99th_percentile_vsync_frame_lag": 166.0,
"average_layer_cache_count": 0.0,
"90th_percentile_layer_cache_count": 0.0,
"99th_percentile_layer_cache_count": 0.0,
"worst_layer_cache_count": 0.0,
"average_layer_cache_memory": 0.0,
"90th_percentile_layer_cache_memory": 0.0,
"99th_percentile_layer_cache_memory": 0.0,
"worst_layer_cache_memory": 0.0,
"average_picture_cache_count": 5.189944134078212,
"90th_percentile_picture_cache_count": 6.0,
"99th_percentile_picture_cache_count": 6.0,
"worst_picture_cache_count": 6.0,
"average_picture_cache_memory": 17.358125122905047,
"90th_percentile_picture_cache_memory": 22.740189,
"99th_percentile_picture_cache_memory": 22.740189,
"worst_picture_cache_memory": 22.740189,
"total_ui_gc_time": 33.342,
"30hz_frame_percentage": 0.0,
"60hz_frame_percentage": 100.0,
"80hz_frame_percentage": 0.0,
"90hz_frame_percentage": 0.0,
"120hz_frame_percentage": 0.0,
"illegal_refresh_rate_frame_count": 0,
"average_cpu_usage": 11.187499999999996,
"90th_percentile_cpu_usage": 16.5,
"99th_percentile_cpu_usage": 23.8,
"average_gpu_usage": 1.05,
"90th_percentile_gpu_usage": 5.0,
"99th_percentile_gpu_usage": 7.0,
"average_memory_usage": 65.03671875,
"90th_percentile_memory_usage": 66.609375,
"99th_percentile_memory_usage": 67.4375,
"impeller_average_frame_build_time_millis": 0.5462666666666666,
"impeller_90th_percentile_frame_build_time_millis": 0.516,
"impeller_99th_percentile_frame_build_time_millis": 4.114,
"impeller_worst_frame_build_time_millis": 5.637,
"impeller_missed_frame_build_budget_count": 0,
"impeller_average_frame_rasterizer_time_millis": 4.301200000000001,
"impeller_90th_percentile_frame_rasterizer_time_millis": 4.979,
"impeller_99th_percentile_frame_rasterizer_time_millis": 12.203,
"impeller_worst_frame_rasterizer_time_millis": 13.646,
"impeller_missed_frame_rasterizer_budget_count": 0,
"impeller_frame_count": 180,
"impeller_frame_rasterizer_count": 180,
"impeller_new_gen_gc_count": 12,
"impeller_old_gen_gc_count": 2,
"impeller_frame_build_times": [
(omitted)
],
"impeller_frame_rasterizer_times": [
(omitted)
],
"impeller_frame_begin_times": [
(omitted)
],
"impeller_frame_rasterizer_begin_times": [
(omitted)
],
"impeller_average_vsync_transitions_missed": 0.0,
"impeller_90th_percentile_vsync_transitions_missed": 0.0,
"impeller_99th_percentile_vsync_transitions_missed": 0.0,
"impeller_average_vsync_frame_lag": 85.86666666666666,
"impeller_90th_percentile_vsync_frame_lag": 113.0,
"impeller_99th_percentile_vsync_frame_lag": 183.0,
"impeller_average_layer_cache_count": 0.0,
"impeller_90th_percentile_layer_cache_count": 0.0,
"impeller_99th_percentile_layer_cache_count": 0.0,
"impeller_worst_layer_cache_count": 0.0,
"impeller_average_layer_cache_memory": 0.0,
"impeller_90th_percentile_layer_cache_memory": 0.0,
"impeller_99th_percentile_layer_cache_memory": 0.0,
"impeller_worst_layer_cache_memory": 0.0,
"impeller_average_picture_cache_count": 0.0,
"impeller_90th_percentile_picture_cache_count": 0.0,
"impeller_99th_percentile_picture_cache_count": 0.0,
"impeller_worst_picture_cache_count": 0.0,
"impeller_average_picture_cache_memory": 0.0,
"impeller_90th_percentile_picture_cache_memory": 0.0,
"impeller_99th_percentile_picture_cache_memory": 0.0,
"impeller_worst_picture_cache_memory": 0.0,
"impeller_total_ui_gc_time": 33.03,
"impeller_30hz_frame_percentage": 0.0,
"impeller_60hz_frame_percentage": 100.0,
"impeller_80hz_frame_percentage": 0.0,
"impeller_90hz_frame_percentage": 0.0,
"impeller_120hz_frame_percentage": 0.0,
"impeller_illegal_refresh_rate_frame_count": 0,
"impeller_average_cpu_usage": 16.56750005,
"impeller_90th_percentile_cpu_usage": 26.0,
"impeller_99th_percentile_cpu_usage": 38.000001,
"impeller_average_gpu_usage": 14.925,
"impeller_90th_percentile_gpu_usage": 32.0,
"impeller_99th_percentile_gpu_usage": 38.0,
"impeller_average_memory_usage": 69.80625,
"impeller_90th_percentile_memory_usage": 71.578125,
"impeller_99th_percentile_memory_usage": 72.40625

Full output:
complex_layout_scroll_perf_ios__timeline_summary.txt
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

In the weekly sync, we decided to punt on this for a couple of months. Filed https://github.com/flutter/flutter/issues/116692 to track. Closing and putting it in the icebox.

Re-opening to see if this actually impact devicelab capacity.  I expect it to decrease queue time, as there will be fewer builders, but increase run time as the tests need to run twice (though the app should not need to build or install twice).
The outstanding question is if everyone is okay with the impeller_ key prefixes.

Requesting review just for the pattern of passing the skia and impeller keys in the same upload.  I still want to rename that one variable and write tests, so this should not yet be merged, but I won't waste my time if this is a no-go.

I intend to get back to this, made it a draft instead of closing it.

Talked to Zach, the flag is going to get flipped, the existing Skia tests will all become Impeller, the Impeller-only benchmarks will be removed, and a few Skia-only benchmarks will be added.
Closing.


Still supported?

If the project is no longer supported I'd suggest putting the notice somewhere in the repo main page :)

hi @caraya , this repo is still supported in case of, but maybe you should take a look to gulp-axe-webdriver.

I saw the pointer and I'm using that. Thanks


writeQuery does not emit new event for request stream after modifying TypePolicy in cache

Hi!
I've spent a few hours trying to investigate it and I don't know what's the problem.
I have such a function for clearing some list in the cache. It works fine. Stream emits new data with the cleared list of items.

But after adding such TypePolicy to the cache then it does not work.

I mean that the cache is modified properly, but the stream does not emit new data in order to refresh page content.
In both cases event is printed in the first screenshot at line 137.
Why is that happening? 🤔

Good question!
I don't know yet, but I see something wrong:If the RootQueryType is your root query type, you should add this to your type policy.
This might be it.
 I had that before, but I don't know why I removed it...
After indicating that RootQueryType is my queryType it works fine ❤


save config json outside of model checkpoint file

Right now, the config exists in the pth file for each model checkpoint, but it would be nice to have it saved explicitly as a json file in the output folder during training, for easier access.

fixed with PR #18


Bogue2018 with one location produces no equal area plot

This magic file has one Location and sites but nothing lower. No plot is produced but expected one equal area plot of directions
magic_contribution_16592.txt
Maybe their is a problem with the data but it passes validation and has directions, so I think an equal area plot of the sites should be produced.
While this similar file with two locations produces two plots.
magic_contribution_16587.txt
The first one without plots is on the MagIC homepage as a recent contribution. It displays a "No plots" image, so it would be nice get this fixed quickly.

eqarea_magic is being called appropriately, but it only plots records that contain "This study" in the citations column.  "This study" is not included in any of the rows in this sites table:
10.1029/2018GC007748:10.1029/JB089iB12p10341

Should this be fixed in the contribution, or relaxed in the contribution_builder module?

this is a hold-over from the old data model.  we can ignore this.

On Feb 18, 2019, at 11:51 AM, Lori J<EMAIL_ADDRESS>wrote:
eqarea_magic is being called appropriately, but it only plots records that contain "This study" in the citations column. "This study" is not included in any of the rows in this sites table:
10.1029/2018GC007748:10.1029/JB089iB12p10341
Should this be fixed in the contribution, or relaxed in the contribution_builder module?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/PmagPy/PmagPy/issues/519#issuecomment-464856784, or mute the thread https://github.com/notifications/unsubscribe-auth/AAKrDeTb52Aqk24KoGXtLnvFx-ioSWj0ks5vOwRXgaJpZM4a-arc.

Yea. Go ahead and plot them. We now allow using the doi of the contribution in the citation table instead of "This study"

@njarboe and @ltauxe -- just confirming, I will remove this filter entirely, so all rows will be plotted regardless of citation.  If that is incorrect, let me know.

Works. Great. Plot now on website.


Fix invalid buffer updates in SDFGI

Fix invalid buffer updates in SDFGI that caused feezes.
Also supersedes #40849

Thanks!


Discord broke settings menu on canary.

Discord updated their SettingsView Component and broke bd's injection. I've done some researching and found out that the fix is pretty easy. All you have to do is explained below.
In line ui/settings.js L#64 you need to replace it with WebpackModules.getByDisplayName("SettingsView")
The last part is that they renamed UserSettings.prototype.generateSections => UserSettings.prototype.getPredicateSections. So you need to update two lines ui/settings.js L#68 and ui/settings.js L#100 and replace generateSections with getPredicateSections.


PointDistributionModel IO method

PointDistributionModel IO method implementation for the new domain concept.

Added the corrections and added support for reading a model with older version number (0.8).

Great. We are getting there.
There is still the separate PointDistributionIO object. My suggestion would be to get rid of it and move the method into the StatisticalModelIO. There the user can find all the statistical model methods, including those for reading intensitymodels (which are not pdms). Or would you prefer to separate each type of model?

I've now added a few tests to the StatismoDomainIO and some general PDM io tests.
Currently all PDMs are written with the datasetType set to POLYGON_MESH , would it make sense to  write the mesh connectivity into the model itself?

Yes, the type should be written into the model. This is specified in the statismo format. The corresponding constant that should be read/written can easily be specified in the typeclass.


Install rose 2016.09.0 and cylc 6.11.1

resolution_fixed   |    by<EMAIL_ADDRESS>

Issue migrated from trac:284 at 2024-01-31 18:25:33 +1100

<EMAIL_ADDRESS>set owner to mrd599

<EMAIL_ADDRESS>changed status from new to accepted

<EMAIL_ADDRESS>changed status from accepted to closed

<EMAIL_ADDRESS>set resolution to fixed


remove toString() method on object

Purpose
This PR fixes #191
Changes
Call the .identifier method on the identifier object to obtain the actual identifier as a string.

This is an offer with the correct offer identifier:
myoffer-3.pdf

This is an offer with the correct offer identifier:
myoffer-3.pdf

 Fix issue #191

 Fix issue #191

 Fix issue #191

 Fix issue #191

fixed in the data model lib instead.

fixed in the data model lib instead.


If run make tre-destroy gives Operation returned an invalid status 'Bad Request'

╔══════════════════════════════════════╗
║          Azure TRE Makefile          ║
╚══════════════════════════════════════╝

»»» ✅ Checking pre-reqs...

Checking for Azure CLI...

Loading local environment variables...
»»» 🔨 Azure details from logged on user
»»»   • Subscription:
»»»   • Tenant:

Deleting locks...
Operation returned an invalid status 'Bad Request'
make: *** [Makefile:130: tre-destroy] Error 1

If manually remove the locks from cosmos and firewall the command works.

This has now been resolved and was due to the az cli output format not being JSON. It is now explicitly set on these commands.


doc: link to cla-bot in action_required message

Closes https://github.com/adobe/cla-bot/issues/47
Adds a link to the cla-bot issues page in the action required message when a CLA check fails.

Closing this in favour of #75, so that we can get full test suite results. Let's continue the convo in there.


Authorization

Added support for authrole in wamp_session as it is described in crossbar.io/authorization
Added support for user_role and authorize functions in auth_provider. It is possible now to implement authorization in auth_provider as it is done in crossbar.io/authorization
Added support for authorize in wamp_session and wamp_router which throws a wamp_error exception with WAMP_ERROR_NOT_AUTHORIZED in case the uri, action is not authorized for that particular session
Added examples for custom authentication and authorization
Fixed wrong use of WAMP_ERROR_AUTHORIZATION_FAILED when it should be WAMP_ERROR_AUTHENTICATION_FAILED

Thanks for these features, and fixing the error-reply bugs.


Checking "I am an author" defaults to blank instead of "AASHE"

This is happening for Monika and Julian when logged in as AASHE staff. Also happening when Monika logs in under<EMAIL_ADDRESS>(For this one, the organization should either be blank, or show up as Frostburg State University).

1/9/16 update: Now, the author field shows up as blank instead of AT Still. Could be because AASHE no longer shows up in the dropdown (see latest comment under #

Hey, @rerb - any idea how we can get AASHE into the list of orgs in the ISS even if SF forces it to have the exclude_from_website flag? Could we simply add an exception for that account_num?

@jamstooks - I wonder where else exclude_from_website is used?
If we hard-code AASHE to be exclude_from_website = False in the ISS sync, any other system that uses exclude_from_website (Drupal?) will misbehave.
The other option is to flip the bit closer to the Hub.  Maybe add an option to the sync_iss_organizations management command?
What would be a good name for that option?

@monika-aashe - Where did the "I am author" checkbox go?  I can't see it.

It should be on the submission forms - here's an example: https://aashe-hub-dev.herokuapp.com/submit-resource/publication/
And screenshot (blue button):

Ahh, I thought it was supposed to be on every type of submission -- when I
didn't see it on the first couple I looked at I got confused.
Thanks!
On Mon, Feb 1, 2016 at 3:08 PM, monika-aashe<EMAIL_ADDRESS>wrote:

It should be on the submission forms - here's an example:
https://aashe-hub-dev.herokuapp.com/submit-resource/publication/
And screenshot (blue button):
[image: screen shot 2016-02-01 at 3 06 47 pm]
https://cloud.githubusercontent.com/assets/15214797/12729588/903b2852-c8f5-11e5-94d8-dbd217c59569.png
—
Reply to this email directly or view it on GitHub
https://github.com/AASHE/hub/issues/141#issuecomment-178165873.

--
Robert Erb
Web Application Developer
Association for the Advancement of Sustainability in Higher Education
www.aashe.org |<EMAIL_ADDRESS>|<PHONE_NUMBER> ext 35
--
Join us in Baltimore, Maryland from Oct. 9 - 12 for the AASHE 2016
Conference & Expo, themed Beyond the Campus. Call for proposals and
reviewers now open https://ww4.aievolution.com/ash1601!
Connect with us:   Website http://www.aashe.org/ | Facebook
https://www.facebook.com/aasheorg | Twitter
https://twitter.com/AASHENews | LinkedIn
http://www.linkedin.com/company/aashe

Email Confidentiality Notice
http://www.aashe.org/email_confidentiality_notice

Oh right, it's only  on some of the submission forms!

@monika-aashe would you check this out and see if it's working (pre-filling "AASHE") for you now?  It's not working for me, but I think my Drupal data might be messed up.  Hoping it works for others now.

It's still blank for me.

FYI, since AASHE can now be selected from the dropdown, we have a workaround. While it would be nice for AASHE to automatically autopopulate when staff are submitting a resource, we can simply add the AASHE in later. I'm fine with delaying this fix until a later milestone.

Is it acceptable if the organization appears as "AASHE (PA)"?

I believe this works in production. Closing.

Yep, I think it was resolved when we switched to Membersuite.


No Longer Compatible with Latest GitHub Actions Runners Because of Node 12

When I attempt to use this GH Action I'm getting the following errors:

Node.js 12 actions are deprecated. Please update the following actions to use Node.js 16<EMAIL_ADDRESS>For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/.

Maybe this is not actually the issue because I'm also getting the error

Not Found

from this action.

I'm seeing this in some of my output from the step run:

Run<EMAIL_ADDRESS>with:
previousReleaseTagNameOrSha: v0.1.0
nextReleaseTagName: v0.1.1
env:
OLS_VERSION: 1.7.16
current: v0.1.0
next: v0.1.1
nextStrict: 0.1.1
Error: Not Found

Looks like this was related to me reading the guide wrong and incorrectly assuming what nextReleaseTagName should have been set to. I set it to main and things worked.


Add ability to scale resizable mode HUD

I'm not sure about the possibility of doing this, but a scaleable HUD would be a world of difference and open up playing on Resizeable Mode on pretty much most modern PC's. As it is, I really have trouble reading chat without leaning and clicking on menus would be far easier.
Cheers.

Related to #715, though probably not a duplicate, since being able to scale things might be nice in general, even without having an obscenely high DPI

There is also #615, which is related, and is stretched fixed mode. It was merged earlier.

I've tried Stretched fixed mode in other clients and the quality is very bad obviously, but the bigger thing is being used to resizeable. At this point I'd rather have small HUD in resizeable than stretch fixed mode. Might just be me and stretch fixed mode would fix it for many.

N.B. this is closed by #6119 which does not really scale the UI but is scaling the whole game, which effectively will scale the UI.

N.B. this is closed by #6119 which does not really scale the UI but is scaling the whole game, which effectively will scale the UI.

I guess if you're fine with making everything blurry? I can't find a way to scale the resizable HUD while keeping the game clear :/

gpu plugin will render the scene at full res which makes the only thing stretched the UI at that point

Huh. I did not realize that. Thanks!


fix poetry run access logs

this pr fixes this issue: https://github.com/lnbits/lnbits-legend/issues/785

removes click and uses env variables instead

Codecov Report

Merging #832 (92a7391) into main (1374aab) will decrease coverage by 0.45%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #832      +/-   ##
==========================================
- Coverage   41.83%   41.37%   -0.46%
==========================================
  Files         216      216
  Lines       11379    11379
==========================================
- Hits         4760     4708      -52
- Misses       6619     6671      +52

Impacted Files
Coverage Δ

lnbits/server.py
0.00% <0.00%> (ø)

lnbits/wallets/lndrest.py
18.51% <0.00%> (-20.38%)
:arrow_down:

lnbits/wallets/cln.py
22.11% <0.00%> (-12.51%)
:arrow_down:

lnbits/db.py
80.80% <0.00%> (-9.61%)
:arrow_down:

lnbits/wallets/macaroon/macaroon.py
27.53% <0.00%> (-5.80%)
:arrow_down:

lnbits/extensions/satspay/views.py
65.38% <0.00%> (-2.62%)
:arrow_down:

lnbits/commands.py
72.85% <0.00%> (-1.43%)
:arrow_down:

lnbits/core/views/api.py
41.04% <0.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

This PR removes click. Why? You did not update the poetry.lock and pyproject.toml, why?

click in this context with poetry run lnbits would always use it's default values (<IP_ADDRESS>:5000), which for me is unexpected and inconsistent with our specified ENV variables.
to make things worse the server.py would start with the click default values, but our logger would log our ENV. so it basically showed the wrong PORT and HOST.

Fixed in #836.


Assume consistent

I guess we are saving stuff to save time

If you're talking about n_bytes_position and initial_frame_index then yes :)


Review and normalize how we handle deep links and scanning

Description
Deeplink handling has some pieces that are not included in the scanning handling, and scanning handling has some pieces that are not included in deeplink handling. To normalize and cleanup, and then test with all the different methods is a decent amount of work. Estimating as a 5.
Added to this ticket: Show a 'show details' as part of the scanner error message 'Invalid QR code' that shows the root error message and add support for _oob query param handling. Check if changes fix vc authn deeplinking issues
Related issues:

https://github.com/bcgov/bc-wallet-mobile/issues/1956

Closing. No actionable items to test. Related ticket fixed.


Error: Type mismatch in argument 'parameter' at (1); passed INTEGER(8) to INTEGER(4)

Got this error trying to build fortran-curl:
src/curl.f90:748:13:

  748 |         rc = curl_easy_setopt_c_long(curl, option, parameter)
      |             1
Error: Type mismatch in argument 'parameter' at (1); passed INTEGER(8) to INTEGER(4)
compilation terminated due to -fmax-errors=1.
make: *** [libfortran-curl.a] Error 1

macOS 10.6.8, ppc32, gfortran 12.2.0

Uhh, interesting platform. Unfortunately, I can’t test on 32-bit, but perhaps this works:
        rc = curl_easy_setopt_c_long(curl, option, int(parameter, kind=c_long))

@interkosmos I will try it, thank you.
I got a similar error now here: https://github.com/interkosmos/fortran-zlib/issues/3
Passing -fallow-argument-mismatch had no effect.

Now it complains about another instance:
make: Entering directory `/opt/local/var/macports/build/_opt_PPCRosettaPorts_fortran_fortran-curl/fortran-curl/work/fortran-curl-0.3.0'
/opt/local/var/macports/build/_opt_PPCRosettaPorts_fortran_fortran-curl/fortran-curl/work/compwrap/cc/opt/local/bin/gcc-mp-12 -Os -arch ppc  -Wall -c src/curlv.c
/opt/local/var/macports/build/_opt_PPCRosettaPorts_fortran_fortran-curl/fortran-curl/work/compwrap/fc/opt/local/bin/gfortran-mp-12 -Os -m32  -Wall -Wno-unused-dummy-argument -std=f2008 -fmax-errors=1 -fcheck=all -c src/curl.f90
src/curl.f90:592:8:

  592 |         function c_strlen(str) bind(c, name='strlen')
      |        1
Warning: Symbol 'c_strlen' at (1) is marked PRIVATE but has been given the binding label 'strlen'
src/curl.f90:530:8:

  530 |         function curl_easy_setopt_c_funptr(curl, option, parameter) bind(c, name='curl_easy_setopt')
      |        1
Warning: Symbol 'curl_easy_setopt_c_funptr' at (1) is marked PRIVATE but has been given the binding label 'curl_easy_setopt'
src/curl.f90:510:8:

  510 |         function curl_easy_setopt_c_long(curl, option, parameter) bind(c, name='curl_easy_setopt')
      |        1
Warning: Symbol 'curl_easy_setopt_c_long' at (1) is marked PRIVATE but has been given the binding label 'curl_easy_setopt'
src/curl.f90:520:8:

  520 |         function curl_easy_setopt_c_ptr(curl, option, parameter) bind(c, name='curl_easy_setopt')
      |        1
Warning: Symbol 'curl_easy_setopt_c_ptr' at (1) is marked PRIVATE but has been given the binding label 'curl_easy_setopt'
src/curl.f90:549:8:

  549 |         function curl_easy_strerror_(code) bind(c, name='curl_easy_strerror')
      |        1
Warning: Symbol 'curl_easy_strerror_' at (1) is marked PRIVATE but has been given the binding label 'curl_easy_strerror'
src/curl.f90:738:13:

  738 |         rc = curl_easy_setopt_c_long(curl, option, int(parameter, kind=i8))
      |             1
Error: Type mismatch in argument 'parameter' at (1); passed INTEGER(8) to INTEGER(4)
make: *** [libfortran-curl.a] Error 1

Sorry, should have added that you have to change line 738 as well:
        rc = curl_easy_setopt_c_long(curl, option, int(parameter, kind=c_long))

Thank you, now it worked!

Thank you for reporting.


Update readme

Update readme

Codecov Report

Merging #13 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #13   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          12     12
  Lines         157    157
=====================================
  Hits          157    157

Continue to review full report at Codecov.

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


[Question] Are requests all been hangs up when Service Worker loading the additional caches?

My config looks like this:
new OfflinePlugin({
  caches: {
    main: [':rest:'],
    additional: ['page-a.chunk.js', 'an-huge-image.png']
  }
})

In main.js:
import { install, applyUpdate } from 'offline-plugin/runtime'

install({
  onUpdateReady () {
    applyUpdate()
  },
  onUpdated () {
    window.location.reload()
  }
})

When I updated my site, it takes a long time during Service Worker's activate event because of there is an huge image (almost 9MB). Before activate event end, every request (such as page-a.chunk.js or just reload the page) has been hang up. After activate event end, the page reload itself and everything looks good.
Is this an expected behavior, so I shouldn't put huge assets in additional caches?

I tested with an huge image (almost 9MB) and capture the Network screenshots:

Are you on http2? If not, then for sure you have a limit with a number of requests at a time.

https://medium.com/@paularmstrong/twitter-lite-and-high-performance-react-progressive-web-apps-at-scale-d28a00e780a3#c8cf

Okay, what I think you should do is just call OfflinePluginRuntime.install() in window's load event.


refactor: clean up :ids-transform and :ids-spi

What this PR changes/adds
Refactors modules :ids-transform and :ids-spi.

Introduces package structure to :ids-transform
Refactores package structure in :ids-spi
Unifies DefaultValues and IdsProtocol to IdsConstants in :ids-spi, removes IdsProtocol from :ids-transform
Deletes IdsIdParser and introduces methods toUri, from(URI) and from(String) to IdsId
Removes transformer classes for IdsId to URI and vice versa
Renames transformer classes to "toIds" and "fromIds" for a better class name ordering
Refactors all transformer classes
Merge ContractOfferFromIdsContractOfferTransformer and ContractOfferFromIdsContractRequestTransformer to ContractOfferFromIdsContractOfferOrRequestTransformer
Cleans up some transformer tests
Adds mapping of edc-sepcific policy properties to and from IDS contract objects

Why it does that
Cleaning up ids modules
Further notes
core clean up will follow, e.g. for better readability
Linked Issue(s)
Closes #1473
Relates #1351
Relates #1761
Checklist

[x] added appropriate tests?
[x] performed checkstyle check locally?
[x] added/updated copyright headers?
[x] documented public classes/methods?
[x] added/updated relevant documentation?
[x] assigned appropriate label? (exclude from changelog with label no-changelog)
[x] formatted title correctly? (take a look at the CONTRIBUTING and styleguide for details)

Codecov Report

Merging #1750 (b5cbfac) into main (3ef48d5) will decrease coverage by 0.05%.
The diff coverage is 71.04%.

@@            Coverage Diff             @@
##             main    #1750      +/-   ##
==========================================
- Coverage   67.40%   67.34%   -0.06%
==========================================
  Files         781      780       -1
  Lines       16761    16622     -139
  Branches     1057     1042      -15
==========================================
- Hits        11297    11194     -103
+ Misses       5008     4976      -32
+ Partials      456      452       -4

Impacted Files
Coverage Δ

...ispatcher/IdsMultipartRemoteMessageDispatcher.java
50.00% <0.00%> (ø)

...nder/MultipartCatalogDescriptionRequestSender.java
6.00% <0.00%> (ø)

...tcher/sender/MultipartContractAgreementSender.java
28.57% <0.00%> (-50.01%)
:arrow_down:

...pi/multipart/handler/ContractAgreementHandler.java
70.58% <ø> (ø)

...aceconnector/ids/core/IdsCoreServiceExtension.java
0.00% <0.00%> (ø)

...nnector/ids/core/service/ConnectorServiceImpl.java
100.00% <ø> (ø)

...tor/ids/core/service/ConnectorServiceSettings.java
1.07% <0.00%> (ø)

...taspaceconnector/ids/spi/domain/DefaultValues.java
0.00% <0.00%> (ø)

...econnector/ids/spi/domain/connector/Connector.java
0.00% <ø> (ø)

...ctor/ids/spi/domain/connector/SecurityProfile.java
92.85% <ø> (ø)

... and 68 more

Continue to review full report at Codecov.

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

@paullatzelsperger @alexandrudanciu Added the mapping of whatever Policy properties to/from the ids ContractOffer and ContractAgreement. See PropertyUtilTest for a legalText example (you may call it differently).


Support text-media block

This ticket is logged to provide the abitlity to the authors to specify a  .text-media block which looks like this

Reference page https://www.zeiss.com/semiconductor-manufacturing-technology/news-and-events/smt-press-releases/2023/zeiss-expands-at-the-rossdorf-research-and-development-site.html

This can now be achieved through 'autoblocking' by having a italic paragraph of text directly under the image.
@sdmcraft while the code for this is merged now and applied to the above page, I wonder are there any other pages in sharepoint that need to updated so that they are using this functionality?


Fix typo

missing word in sentence

Docs Build status updates of commit 868c909:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/overview/excel.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


some data ingestion mapped data in uint8 to int8 incorrectly

mrc definition of mode 0 was not clear until 2014 definition, and some software still used uint8 a few years after that.  We have quite some of those in jensen dataset and possibly some in others.
There is no mode defined for uint8, so we will have to shift the values.
A bad one:  https://cryoetdataportal.czscience.com/runs/11824?prev=%2Fdatasets%2F10274%3Fprev%3D%252Fbrowse-data%252Fdatasets%253Fdataset_id%253D10274
If you view its tomogram in neuroglancer, you will notice that the fiducials (the strong signal radiate a cross pattern in the top-right panel have white center.  These are supposed to be at the darkest in the image in such rendering.
Screenshots

A good one: https://cryoetdataportal.czscience.com/runs/11804?prev=%2Fdatasets%2F10273%3Fprev%3D%252Fbrowse-data%252Fdatasets%253Fdataset_id%253D10273

I compiled a list of original jensen dataset that failed my simple test of min, max values in its mrc header in its first 5 runs.  From spot checking, those passed the validation seem to be good beyond the first 5 runs, so they are likely consistent.
See https://github.com/czimaginginstitute/czii-data-portal-processing/commit/5449a64d525786001da85aa43ae5ae42bc0ce5a9
Additional context
ingestion issue

@jgadling looks like this has been in staging for a while now, has this been completed?

This was addressed by https://github.com/chanzuckerberg/cryoet-data-portal-backend/pull/77


Add back roll forward to rzls

Fixes https://devdiv.visualstudio.com/DevDiv/_workitems/edit/2088486
Seems this was inadvertently removed in https://github.com/dotnet/razor/pull/10453
@ryzngard I'm pretty out of my depth here about what is required, but a local build of rzls with this change does fix the problem. Are there any other project properties that might be important, that I should bring back?

Fixes devdiv.visualstudio.com/DevDiv/_workitems/edit/2088486
Seems this was inadvertently removed in #10453
@ryzngard I'm pretty out of my depth here about what is required, but a local build of rzls with this change does fix the problem. Are there any other project properties that might be important, that I should bring back?

oof sorry about that. Thanks for putting in a way more reasonable place than I did.  The only other thing I can think of is for source build but I haven't heard back yet: https://github.com/dotnet/sdk/pull/41536#issuecomment-2179406555


Test varying io schedules for the same fuzz ops

Storage is supposed to be deterministic, so if we take the same set of fuzz ops but set a different seed for test/storage, we should get the same storage byte-for-byte at the end.

This won't work until https://github.com/tigerbeetledb/tigerbeetle/issues/269 is done (the third bullet, which references https://github.com/tigerbeetledb/tigerbeetle/pull/247).


Pry-debugger won't install on MRI 2.1.0-preview2

On MRI 2.1.0-preview2, doing 'gem install pry-debugger' gives:
ERROR:  Error installing pry-debugger:
	    ERROR: Failed to build gem native extension.
    /home/mydir/.rbenv/versions/2.1.0-preview2/bin/ruby extconf.rb
Ruby version 2.1.0 is not supported.

OK - so it says right there that 2.1.0 is not supported.  But it should be!  :-)

:+1:

:+1:

:+1:


Issue 14321 - Unnecessary destructor call with and AA's

https://issues.dlang.org/show_bug.cgi?id=14321

Auto-merge toggled on


Listings files are being added.

Listings files are being added.

listings branch files are being added to the main branch correct and safely.


XenonJobs.getStreams remove Streams from request/response message names

The request and response are already streamed adding Stream(s) to message name is duplication.
Rename:
rpc getStreams(stream JobInputStream) returns (stream JobOutputStreams) {}

to
rpc getStreams(stream JobInput) returns (stream JobOutput) {}

Replaced by
rpc submitInteractiveJob(stream SubmitInteractiveJobRequest) returns (stream SubmitInteractiveJobResponse) {}


Cirrus: Fix minor python deprecation warning

Signed-off-by: Chris Evich<EMAIL_ADDRESS>

Fixes this in the 'ci' task of the gating job:
/usr/local/bin/entrypoint.sh -C ${CIRRUS_WORKING_DIR}/${SCRIPT_BASE}/packer test
make: Entering directory '/usr/src/libpod/contrib/cirrus/packer'
-c:1: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.
-c:1: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.
packer_1.4.2_linux_amd64.zip: OK
Archive:  packer_1.4.2_linux_amd64.zip
  inflating: packer
./packer inspect libpod_base_images.json > /dev/null
./packer inspect libpod_images.json > /dev/null
All good
make: Leaving directory '/usr/src/libpod/contrib/cirrus/packer'

LGTM

/lgtm
/approve
/hold
hold cancel when tests pass


chore(deps): upgrade compiler dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-compiler-dependencies-k8s-29-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4433

Questions ?
Please refer to the Backport tool documentation


Break-up problematic background shorthand.

Helps to address #18 for angular users.

Good afternoon,
This problem occurs in particular with LESS strictMath mode
I think it can help to put parenthesis.
Reference :
http://lesscss.org/usage/#less-options-math
https://github.com/gruntjs/grunt-contrib-less/issues/195
https://gillianshelpdesk.tumblr.com/post/127773807047/calc-issues-in-grunt-contrib-less

@cthrax Sorry this took so long to get to, thanks again!
@mageho Yes, the issue is that Less currently defaults to using --math=always and I've been compiling it with --math=parens. To make the experience of using this port more plug-and-play, I’ve decided to just use the default always option and work around the cases where we want to output literal slashes (and not divide).


homebrew?

would you consider making a homebrew formula? that might make installation simpler on macOS. a homebrew formula for a python app will be contained within its own virtualenv

I am planning on making a pipx-bootstrap package on PyPI to avoid the curl method (https://github.com/cs01/pipx/issues/44).
I think the hardest part of the installation is modifying the PATH and opening a new terminal. What do you think would be made easier by moving to brew?

Thank you!
I have no objection to someone adding it to brew, but I won’t be able to take that on myself.  Also note that standard installation is now more straightforward with just “pip install pipx” so if you add it to brew there is no need for the bootstrap command.

The advantage of a Homebrew installer would be that pipx could be installed in /usr/local/bin/pipx with the other homebrew-installed binaries.
Currently, if you install pipx is installed using Homebrew-installed python3:
python3 -m pip install --user pipx

The pipx binary gets installed to $HOME/Library/Python/3.7/bin/pipx, which isn't in $PATH by default.
I'm currently working around this by installing pipx with pipx...
python3 -m pip install --user pipx
python3 -m pipx install pipx

And ensuring that $HOME/.local/bin is at the front of $PATH:
export PATH="$HOME/.local/bin:$PATH"

It's fairly straightforward to create a Homebrew formula from a Python package. Here's the formula for doitlive: https://github.com/Homebrew/homebrew-core/blob/master/Formula/doitlive.rb .
I then use this script to update the Homebrew formula after I make a PyPI release: https://github.com/sloria/doitlive/blob/dev/release_homebrew.sh
@cs01 Would you be up for adding this to your release workflow?

PR's welcome to build it into the publish target of the makefile. That way I can keep running the same command, and homebrew will be updated.

hi @sloria do you want to submit a PR for adding this to homebrew or should I take a stab at it?

@cs01 The biggest reason as far as I can tell is that the second command in the README: pipx ensurepath will fail if $HOME/.local/bin isn't already in the PATH.
It's a circular dependency

Could you explain more about what the issue with pipx ensurepath is? Maybe the homebrew formula could have the equivalent of pipx ensurepath built into it?
Btw, not sure if this is relevant, but pipx no longer installs itself to $HOME/.local/bin, it just installs itself to site-packages.

Not sure if this is related but the instructions in the pipx README.md say to use pip install --user pipx.
If you are using HomeBrew built Python the --user argument is disabled due to a bug in distuils.
https://docs.brew.sh/Homebrew-and-Python

@iain2k thanks for sharing! pipx technically does not require a --user install. I wrote that because I thought it would get around common issues on mac and avoid polluting the global environment.
For anyone else reading, I would like to set expectations on this appropriately: I welcome having pipx on homebrew but want to completely rely on someone owning the pipx/homebrew experience.
I want my involvement to be make publish (as it currently is). Any homebrew issues that come up would be owned by someone else. Is anyone interested in volunteering for this?

re:

Btw, not sure if this is relevant, but pipx no longer installs itself to $HOME/.local/bin, it just installs itself to site-packages.

Right, so the README states to run pip install --user pipx which means it will get installed in $HOME/.local/bin. If the user doesn't already have this in their PATH, the next command, pipx ensurepath would fail

re: automatically running ensurepath
I believe homebrew has a strict policy against modifying a user's shell profile automatically so pipx ensurepath cannot be done there. The general practice is to list it under the "Caveats" which is printed at install time. This is similar to how pip install --user pipx works right now where a warning is printed if the $HOME/.local/bin is not in the PATH.
The experience would be identical in terms of the printed warning except for the fact that the pipx tool is guaranteed to be on the PATH via homebrew. This way, the pipx ensurepath command or pipx --help etc. will always work.
If the user missess the caveat to add $HOME/.local/bin the first time, running any pipx command can notify them to do so since pipx is guaranteed to be in the PATH.

re: maintainance
yep, if @sloria doesn't want it, I'll volunteer to maintain. I use the tool a fair amount and I've been evangalizing it at work. would love to add support for homebrew since we install the majority of our CLI tools in this way.

@AlJohri  All yours--I won't have time to work on this.

which means it will get installed in $HOME/.local/bin

Is this true? On my computer it installs to ~/Library/Python/VERSION/bin
What does
python -c "import site; print(site.USER_BASE + '/bin')"

output for you?

re: automatically running ensurepath
I will defer to you to work through those details and dogfood them on your coworkers 😄

I'll volunteer to maintain. I use the tool a fair amount and I've been evangalizing it at work. would love to add support for homebrew since we install the majority of our CLI tools in this way.

Nice, thanks so much!

@cs01 you got it! 😄

Is this true?

My default python installed via asdf (similar to pyenv) has path of ~/.local/bin. Homebrew python has path of: ~Library/Python/3.7/bin (this folder does not exist).
asdf python
$ asdf list
nodejs
  11.10.0
  11.5.0
python
  2.7.15
  3.7.2
ruby
  2.5.3

$ which python
/Users/al/.asdf/shims/python

$ python --version
Python 3.7.2

$ python -c "import site; print(site.USER_BASE + '/bin')"
/Users/al/.local/bin

homebrew python
$ /usr/local/bin/python3 -c "import site; print(site.USER_BASE + '/bin')"
/Users/al/Library/Python/3.7/bin

$ stat /Users/al/Library/Python/3.7/bin
stat: /Users/al/Library/Python/3.7/bin: stat: No such file or directory

ran into an issue where pipx ensurepath can see the PATH is set properly but pipx install <package> doesn't when installed via my test homebrew formula.
will work through debugging but just throwing this here in case its immediately obvious to someone

I think I've identified the issue. Homebrew requires creating an isolated virtualenv for any python app. For some reason, creating a venv as pipx does, when it is already inside a virtualenv has odd behavior.
The two virtualenvs (the homebrew isolated virtualenv for pipx) and the venv that pipx creates for a single package become linked.
Looking for a way around this...

Got it working. Solution was to not use hombrew's virtualenv creation mechanism. Here's the pull request:  https://github.com/Homebrew/homebrew-core/pull/39020

Merged! https://github.com/Homebrew/homebrew-core/commit/e3e53ec6baac499fc7679199cfbe140e9e6559cd
@cs01 next step will be updating the readme to add brew install pipx for macOS and updating your make publish command. I'll use @sloria's release_homebrew script as a base.
PRs to come!

Nice, thanks!
I just tried running brew install pipx but got Error: No available formula with the name "pipx".

@cs01 try running brew update and then brew install pipx? maybe you have export HOMEBREW_NO_AUTO_UPDATE=1 set so it didn't automatically update

lol wait nevermind, there was a small issue! https://github.com/Homebrew/homebrew-core/pull/39483

@cs01 merged! https://github.com/Homebrew/homebrew-core/commit/dfbe626a2899a5067cef753a147a09d887a48061

On macOS, as stated by some earlier, I had to
export PATH=$HOME/Library/Python/3.7/bin:$PATH

To make the command pipx work.


Repeated Group: Answers within group are not present in the QuestionnaireResponse, if child question depends on enableWhen condition of another child question

Describe the bug
Suppose a child question depends on enableWhen condition of another child question and both the questions are part of repeated groups, answers are not present in the QuestionnaireResponse for the question which depends on enableWhen condition.
Component
SDC library
To Reproduce
Steps to reproduce the behavior:
Use test-questionnaire.txt as questionnaire json.
https://user-images.githubusercontent.com/82660329/209086244-0d64468c-d58a-46aa-9fc8-674cd584a930.mp4
@jingtang10

fyi @kevinmost

http://community.fhir.org/t/questionnaire-repeating-groups-what-is-the-correct-format/2276/3

The implementation currently adds answers to item.answer for repeated groups, and then later createRepeatedGroupResponse takes care of putting answerItemComponents to the repeated groups item.item field.
While writing the UT I have set the initial answer of the child question of the repeated group. But the code that populates questionnaireResponse with initial answers adds the answer of the repeated groups in item.item , so this results in createRepeatedGroupResonse returning an empty list, since it cannot find any answers in the item.answer field.
Suggested change : Initial answer for repeated groups should be set in the item.answer field so that it is consistent with the rest of the repeated group flow.
cc: @jingtang10 @kevinmost

This issue has been resolved. There is a new issue related with this #1895.


Trying to use "Clear clipboard history?" modal in the main Cliphist window causes user's logout

Login to the system.
Call the Cliphist wiwndow.
Click [Clear].
Move the cursor to the modal window appeared.
See the result.

ER: User can click [Click] or [Close] buttons on the modal.
AR: User is logged out.
ENV: MSI modern 14 Intel core i5, 32G ram running nwg-live-2024.08.21-x86_64.iso both live and installed on ext4.
https://github.com/user-attachments/assets/76d5058c-9ac5-4a9e-a32f-e03535eeb194

Fixed in fe30e02a7a97665ebf87ae78f781e9bfb3706a42.

👍 Many thanks!


⚠ Processors has degraded performance

In a63b1e9, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 78 ms

Resolved: Processors performance has improved in a516dfb.


Add some documentation for bridging rooms

There was no information on how to do this, and while it is common knowledge around those who actively use the bridge, it would be better if it was properly documented. Open for suggestions on how to improve on this.

Thanks!


Remove unused files

Description of changes:

Remove unused files

Checklist

[ ] Tests have been added and are passing
[ ] Documentation has been updated

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

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


Racer tests

Calculations for tests are in spreadsheet here. I put link in Telegram for editable version where you can play with the numbers in pink at the upper left and see how the plot (picture of racer+rays, enemies, between walls) changes. There are a few tabs at the bottom.
Notes by commit:

"add to tests in rays.cairo"

In test_raystrait_new, I had to reduce precision down to 1e-04 in some cases (theta, cos_theta, sin_theta). I'm not sure why theta needed that, but cos_theta and sin_theta might make sense due to the fast trig functions. Had to lower precision to 1e-03 or 1e-02 for p.x, p.y, q.x, and q.y, I think because of propagation of error from sin_theta and cos_theta. Is 1e-02 ok for functionality?
I got the error below, which I worked around by commenting out part of test_raytrait_intersects and then all uncommented tests pass. Then I uncommented these lines and commented out other part of test_raytrait_intersects and all these uncommented tests pass. So all the tests pass, just not all at once now. Still not sure how to solve this one. The error:Failed setting up runner.

Caused by:
    #16187->#16188: Got 'Offset overflow' error while moving [210].

"add method vertices_scaled"

This was to fix a u128_mul overflow error when running test_closest_position from racer.cairo. This line passes scaled values but enemy PositionTrait vertices accepted only unscaled Position values, then used them to make array of scaled Fixed type. The source of the error was here because self.x and self.y were already scaled.
The fix is in enemy.cairo, a new method vertices_scaled which accepts scaled Position values (not sure if best name). Now other files call only vertices_scaled.
I left method vertices as-is in case ever needed for unscaled values inside enemy.cairo. (If so, we may want it to return a 2d vector with unscaled members, instead of Vec2.) Before this change, we used vertices method only for scaled values (called only from racer.cairo closest_position and collision_check). Method vertices is not currently called in enemy.cairo using unscaled Position values.

"add test_closest_position"

The test results here are probably close enough for good functionality but not super precise, so i used Option::Some(184467440737095000) // 1e-02 in assert_precise to get the tests to pass.

"fix fn move"

Prior to this fix, two tests, test_move_without_respawn and test_move_through_execute, were failing. This fix causes the tests to pass. I am pretty sure that line was the issue. However there are parts of move and test_move_through_execute I'm still trying to understand fully, so I am not 100% confident that this was the issue.
Tangential issue: Wierdly, one of the racer tests that previously passed was now running out of gas, so I made the available gas 10x and it now seems fine.


Updated style condition to handle VFs

Description
This pull request addresses the problems described at issue #xxxx
To Do

[x] update CHANGELOG.md
[x] wait for all checks to pass
[x] request a review

@yanone, as this PR deals with inferring information from the filename, please make sure to also read the ongoing conversation about this with @justvanrossum at https://github.com/googlefonts/fontbakery/issues/4005


Migrate the Zulip LDAP integration to be web-configurable

The LDAP integration is theoretically configuration for how to login to a particular realm, not an entire server, and so conceptually should be configurable by the realm administration via the web.
Even if we can't do that, we should at least make it possible for different realms to use different LDAP servers.
This may be a fairly complicated project since it probably involves replacing the django-auth-ldap module entirely.

https://github.com/django-auth-ldap/django-auth-ldap/issues/27#issuecomment-404510455 is a path forward for completing this.

The feature we need here was merged upstream in https://django-auth-ldap.readthedocs.io/en/latest/changes.html.  So this is now something we could start working on.

I would only like to use different query bases per realm. So, do I need to wait for that feature?

I just took a look at that: https://django-auth-ldap.readthedocs.io/en/latest/multiconfig.html
But (if my experiments are correct) this has to be done on initialization of the Auth Backend.
But as far as I understand it the auth backend is for all realms.
So I do not see how I could get this to work


BaseDateTransformer and standardised date data handling

Created this in response to this issue re inconsistent date data check:
https://github.com/lvgig/tubular/issues/142
Solution - create BaseDateTransformer which all datetime transfomers inherit from (excluding toDatetime). This has the methods that are listed below.  In the cases where casting is done, a temporary copy of the data is made which is used to generate the correct transformed columns, which are then inserted back into the original data. NB I don't think adding this functionality will break or change any historic models or datasets - anything that gets casted after this PR just wouldn't have worked beforehand.

check columns are date or datetime: this checks whether the data is datetime.datetime or datetime.date for the columns in self.columns, and raises a type error if not.
cast columns to datetime: Some transformers can work with datetime data, others need the date data to be temporarily cast to datetime. This calls the above function AND casts and datetime.date columns to datetime.
match_column_dtypes: some comparative transformers cannot handle one column as datetime.date and one as datetime.datetime. In the case where they are different, both are cast to datetime.date to avoid artifically creating (e.g.) seconds precision that we don't have for one column. This also calls the basic data check function first.

Testing:
For each transformer I have tried to test:

Are the correct errors raised if the data is not a date.
Are the correct warnings raised in the case of casting.
Is the output correct for both datetime.datetime and datetime.date.

The testing data set up is quite messy for the date transformers, there definitely needs to be more fixtures, but I figured cleaning it was outside the scope of this PR.
@Decima2014 @davidhopkinson26

Given how this is a potentially impactful change to all the date transformers I wonder if it is worth rerunning the example notebooks? Although looking at them, a few of them are out of date so unsure if this is something we keep supporting


Update CompileTimeErrorCode.IMPLEMENTS_DISALLOWED_CLASS for enhanced_enums

The error code CompileTimeErrorCode.IMPLEMENTS_DISALLOWED_CLASS is misleading when the class being implemented is Enum and the 'enhanced_enums' feature is enabled. For example, this code:
class C implements Enum {}

produces a diagnostic that reads "Classes and mixins can't implement 'Enum'.". It would be more accurate to say that "Concrete classes and mixins can't implement 'Enum'." because under 'enhanced_enums' abstract classes are allowed to implement Enum. Of course, the feature doesn't impact other disallowed classes, so the message for them should be unchanged.

@scheglov @srawlins

https://dart-review.googlesource.com/c/sdk/+/233701


Ability to search by artist

When searching only by the artist -> pick up user geolocation and return to SeatGeek to look for events near the user.


Development runtime error

Reopen in container with vscode
Run FastAPI >

發生例外狀況: ImportError
cannot import name 'MutableMapping' from 'collections' (/usr/local/lib/python3.10/collections/__init__.py)
  File "/workspace/api.py", line 10, in <module>
    from flask import Flask, Response, render_template


REQ-MCT-019 - Command Scheduling

The MCT shall be able to schedule commands and automated command sequences for future execution.
See #37 for POC demo

Related: #68, #113

node-scheduler is a time-based scheduling library which lets us schedule jobs to run on specific overpass dates
https://www.npmjs.com/package/node-schedule


Add add/get_alternate_id to po contract's state module

Add the add_alternate_id & get_alternate_id method to the PurchaseOrderState object, within the po smart contract's state module. This will allow for alternate IDs to be added and referenced if passed in to the po smart contract.

After discussion with @shannynalayna, we don't think this is necessary as Alternate IDs are part of the Purchase Order state representation already. Any lookup using alternate IDs is done at the store level.


How handle different environments (dev/prod)?

Hi, I'm totally new at Flux and I have a question please. I'm reading about Flux and it looks great. However, it's still not clear to me how Flux handles different environments (like development and production).
 According to what I read, that update occurs automatically when a new version of the image is detected on its Docker repo.
However, on the same Docker repo you will have both "production-quality" images (built from "master" branch) and "development-quality" images (built from development/topic branches). Supposing that I have a production Kubernetes cluster and a development Kubernetes cluster (to run and test development versions), how can I instruct to Flux to update the "right" images on each case? I mean, to prevent that the Flux service running on the production cluster blindly update the latest version of the Docker image (which most of times will be a development image).
Or maybe the automated update feature is not suitable for this kind of use case?
Thanks in advance for your feedback!
Cheers,
Leo

Hi leo,  we using different config repo for dev & prod, with two flux installation in different cluster to do the sync.

@jasonyihk Great, thanks for your info


feat(cloud_firestore_odm_generator): Support all serializable types

Description
Support querying enum fields
Related Issues
#11364
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

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

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

[ ] Yes, this is a breaking change.
[x] No, this is not a breaking change.

We should merge #11361 in first

:wave:
This needs testing.

@rrousselGit Let me know what you think of the new way of doing this

This is looking great! Some small test changes and we should be good to go :)

So I actually tried generating this in my project and there's an issue with inherited classes.
class A {}

@firestoreSerializable
class B extends A {}

@firestoreSerializable
class C extends A {}

@Collection<A>('path')
final aRef = ACollectionReference();

This generates code that tries to use _$APerFieldToJson which obviously doesn't exist. I want to just fix this in my code by creating my own _$APerFieldToJson, but I was wondering if you had any better ideas.

Maybe creating my own perFieldToJson isn't a great idea since all the methods are static 🙃

Unless we could change the way the PerFieldToJson code is generated so that you can call do something like:
abstract class A {
  final TypeEnum type;

  Function perFieldToJson(String field) => switch (type) {
        TypeEnum.a => _$APerFieldToJsonMap[field],
        TypeEnum.b => _$BPerFieldToJsonMap[field],
      };
}

This generates code that tries to use _$APerFieldToJson which obviously doesn't exist.

We can check for the presence of JsonSerializable/Freezed. That should be enough
But honestly I'm fine requiring that all Collection models are using JsonSerializable.

Alright. What do you want me to do with the ManualJson tests then? Just remove them?

Also now that I think about it... do we even need to check for type support anymore? If json_serializable successfully generates for a class then we should have a PerFieldToJson call for every serialized field in the class right? This would allow us to automatically support any type json_serializable inherently supports (right now I'm thinking the Duration type specifically, but I'm sure there are others). This would also give us support for anything using a JsonConverter.

Yes that's the goal :)
I didn't want to manually handle types on a case-by-case basis, hence why I made that PR to json_serializable

Do you plan on fixing the CI?

Yeah I'll fix it. Do you want me to turn this PR into the one where we support all types or do that after? It's a simple change.

Let's support all types :)

CI is failing but I want to talk about the changes required before fixing it

LGTM once CI is fixed

@rrousselGit how is this looking?

Sorry I missed your notification. Looking good!

What's the plan for releasing this?

First, I would like to thanks @Rexios80 and @rrousselGit for the work. While non-blocking, having a bunch .whereFieldPath(FieldPath.fromString('fieldName'), isEqualTo: MyEnum.myValue.name) partly defeated the type safe promise from ODM IMHO. It was my main pain point regarding ODM, and you solved it. Thank you you two.
But I think you missed something, since models won't generate until you set create_per_field_to_json: true with version 1.0.0-dev.70. If the breaking change is necessary, changelog and documentation should be updated. If it's the case, I started a PR: https://github.com/firebase/flutterfire/pull/11728/


Move more stuff to the buffer_validation module

Move some image-related helper routines from CV to buffer_validation:

ResolveRemainingLevelsLayers
VerifyClearImageLayout
RecordClearImageLayout
PreCallValidateCmdClearColorImage
PreCallRecordCmdClearImage
PreCallValidateCmdClearDepthStencilImage

Move CmdClearDepthStencilImage validation from image.cpp to buffer_validation

Added suggested changes and pushed to Master.


"Clear All" button on Reports->Logs should have verification

We've had some users confused about this button, thinking it's used to clear the log filter options.  A simple verification that says "Are you sure you want to clear the entire Concrete5 log?" would prevent inadvertent log clearing.

I agree, adding a confirmation popup to this button would prevent users from deleting logs accidentally.  Given that there is no way to recover these deleted logs,  some failsafe seems necessary.

:+1:

👍

This is complete. Can it be closed?

👍


Support for VertexAI models

Support for VertexAI models.
Added support for LLM, ChatModel and Embeddings provided by VertexAI (Google Cloud).

@lkuligin happy to review whenever you think it's ready

@dev2049 please, take a look (I can't change reviewers myself unfortunately).


There are some issues with type hints

Visual Studio Code: 1.66.2
Volar version: v0.34.6
Earlier versions did not have this problem, eg 0.33.10.

I can't reproduce this problem in https://github.com/johnsoncodehk/volar-starter, could you provide minimal reproduction?

I found the reason, when the value of the strictNullChecks option in typescript.config is false, there will be this prompt.


extras: add 'RutField' for Django models, DRF and MM

A separate field class for:

Django model.
DRF (Django REST Framework) serializer.
MM (marshmallow) schema.

Some tests have not been implemented.

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@fd7b377). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             develop     #5   +/-   ##
========================================
  Coverage           ?   100%
========================================
  Files              ?      2
  Lines              ?     67
  Branches           ?      8
========================================
  Hits               ?     67
  Misses             ?      0
  Partials           ?      0

Continue to review full report at Codecov.

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


botan2: 2.17.2 -> 2.17.3

Motivation for this change
Fixes CVE-2021-24115.
(cherry picked from commit 9a18802edfb9db4d826ec967d7301c6c7460dad9)
Things done

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

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

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

Result of nixpkgs-review pr 115316 1

2 packages marked as broken and skipped:

neopg
tlspool

4 packages built:

biboumi
botan2
kea
softhsm

@LeSuisse You might change the name of the PR to "[20.09] botan2: 2.17.2 -> 2.17.3" since you're trying to merge to release-20.09

Done, sorry about that :/


"The authentication flow has completed. You may close this window."

Hi Michimoto
Thank you for your post, That is exactly what i want.
but i got some error here.
i create my credentials file(json) and save it in my project.
when i run here:
flow = InstalledAppFlow.from_client_secrets_file(
'credentials.json', SCOPES)
creds = flow.run_local_server(port=0)
My chrome start and i am login with no problem.
but finnally my browser show me this tip and struck that.
"The authentication flow has completed. You may close this window."
do you know why,
i have got some similar error when i am trying to run other blogger project.
Thank you.

Or am i made the wrong credentials?
which type should i select?
web ios andriod desktop or chrome app ....

Or am i made the wrong credentials?
which type should i select?
web ios andriod desktop or chrome app ....


WorksharingSession.HostIP node returning mac addresses rather than IP addresses

Describe the bug
WorksharingSession.HostIP node returning mac addresses rather than IP addresses
Steps to Reproduce
See attached image

Expected behavior
IP address returned rather than mac address
My setup

Operating System: Win 10
Dynamo version: 1.3.3

Housekeeping
Please make sure these boxes are checked before submitting your issue:

[X] I am running the latest available version of Journalysis

Comments
Fantastic package, really useful...great work Andy!

Yes, I should rename that to HostAddress since it seems to return IP or MAC addresses based on the environment. Same goes for WorksharingSession.ServerIP btw - I've seen that one output MAC addresses, too.


Add smooth scroll component

https://github.com/moxienyc/skaled.com-web-app/blob/develop/app/components/smooth-scroll/controller.js
(works with the duScroll lib)

Same as https://github.com/moxie-leean/ng-patterns/issues/20#issuecomment-210677848


Aspect Ratio seems not working correnctly with iframe. Black bars in Vimeo video.

See: https://codepen.io/nnico/pen/OJJRzyL
I am trying to use it for a Vimeo Video with a very wide Aspect Ratio and it creates black bars.
I saw in the source that you at some point take the dimensions of the element specified and that you set the css to positon: absolute; but I could not figure out how the aspect ratio it kept. No position: relative ?
I would rather like to just pass a string of HTML to be used for wrappers I have prepared with the padding-top element as sibling where the + position: absolute iframe is.

Thank I am looking forward to this.
Just so you know, in case flexbox comes into play at some point. I have seen Bootstrap moving from putting a padding-bottom: xx% on the wrapper to using a ::before pseudo element and putting a padding-top: xx% on that. https://github.com/twbs/bootstrap/blob/master/scss/helpers/_embed.scss
There was the change https://github.com/twbs/bootstrap/pull/19739/files
AMP does it this way too. They not use a pseudo but a div where the put padding-top: xx% on as the first element of in the wrapper. Its probably a better technique to use nowadays.
I would also create a additional wrapper for the responsiveness. Like inside the #bp_sv I have seen things get messy with max-width and this responsiveness hacks b4.

Added in 2.3.0
https://codepen.io/henrygd/pen/vYYyZZW
https://github.com/henrygd/bigpicture#change-dimensions-of-embed--youtube--vimeo

@henrygd Thanks so much.


Get rid of the MetaTestNetwork

This PR will conflict with https://github.com/bpfkorea/agora/pull/176 and will need a small rebase if that one is merged first (or vice-versa).

Green.

Wanted to merge #176 first. Needs a rebase now.

Thanks. Rebased.


chore(deps): update dependency firebase to v10.24.0

This PR contains the following updates:

Package
Update
Change

Firebase
minor
10.23.1 -> 10.24.0

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
Because you closed this PR without merging, Renovate will ignore this update (10.24.0). You will get a PR once a newer version is released. To ignore this dependency forever, add it to the ignoreDeps array of your Renovate config.
If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.


UDF creation

Hi,
Thank you for your work on this project - it is very promising!
I am trying to add a temporary UDF to a unit test but I cannot get it to work. Using ADD JAR statements just gives a usage message in the output:
Usage: add [FILE|JAR|ARCHIVE]  []*
I notice the instructions say you should just exclude these statements but if I do that, after adding the jar to my project classpath, I get an error:
2015-07-02 15:49:40,481 INFO  [main] ql.Driver (Driver.java:execute(1243)) - Starting command: create TEMPORARY function myfunction as "com.xxx.yyy.algorithms.udf.zzz"
...
2015-07-02 15:49:40,488 ERROR [main] exec.Task (SessionState.java:printError(565)) - FAILED: Class com.xxx.yyy.algorithms.udf.zzz not found
Can you give an example of a test that creates a UDF from a jar as a unit test?
Cheers,
Lee

Hi,
Also run into this issue. I suspect a parsing error while parsing "ADD JAR...": at some point it tries interpret "ADD" as one of the "FILE", "JAR" or "ARCHIVE" expressions.
Try this syntax:
CREATE FUNCTION foo_bar_udf AS 'foo.bar.UdfClass' USING JAR 'hdfs:///path.to/JarConatiningUdfClass.jar'
Peter

Hi Peter,
Thank you for the suggestion. I tried that syntax but it appears to convert the hdfs path to local filesystem and fails. Here is the test code I am using:
// copy jar file from local filesystem to hdfs
FileSystem fs = testSuite.getFS();
fs.copyFromLocalFile(new Path("C:\path\to\JarConatiningUdfClass.jar"), new Path("/path/to/JarConatiningUdfClass.jar"));
// run test
List results = testSuite.executeScript("src/test/resources/scripts/test.sql");
test.sql contains:
CREATE FUNCTION foo_bar_udf AS 'foo.bar.UdfClass' USING JAR 'hdfs:///path.to/JarConatiningUdfClass.jar'
select foo_bar_udf("test string");
The exception is:
java.lang.IllegalArgumentException: Pathname /C:/Users/leew/AppData/Local/Temp/48b359f7-5071-4793-8bab-7d101f85a42d_resources/JarConatiningUdfClass.jar from hdfs://<IP_ADDRESS>:56108/C:/Users/leew/AppData/Local/Temp/48b359f7-5071-4793-8bab-7d101f85a42d_resources/JarConatiningUdfClass.jar is not a valid DFS filename.
The exception occurs when running the SQL line where the UDF is actually used, not the CREATE FUNCTION line. There are no errors during copying of the jar from local FS to hdfs.

@nem2k2  @csaszarp  sorry for being a little late to the party, I didn't get a notification, this was not actually a use case that I needed to deal with in my test cases, but if you can give me a little bit more detail about what exactly you are doing, then maybe I can come up with a fix.
Its great that you guys are finding some value in it.  It really makes me smile :smiley:

Hi @bobfreitas, I am trying to add a jar file that contains a UDF I need to run as part of a unit test but I get a usage message in the output:
Usage: add [FILE|JAR|ARCHIVE] []*
Adding the jar to my classpath and then attempting to call the UDF (without adding it with a "add jar" statement) results in an error because the UDF class could not be found.
I need to be able to create and call a UDF function as part of a unit test.

I am thinking you probably have something going on with your classpath,
because it should be working.
I needed to exclude the ADD JAR directive because of where I was accessing
the Hive API.
Sorry if its not what you need, you can always fork and it make some
customizations.
Best of luck
On Mon, Aug 10, 2015 at 4:55 PM, nem2k2<EMAIL_ADDRESS>wrote:

Hi @bobfreitas https://github.com/bobfreitas, I am trying to add a jar
file that contains a UDF I need to run as part of a unit test but I get a
usage message in the output:
Usage: add [FILE|JAR|ARCHIVE] []*
Adding the jar to my classpath and then attempting to call the UDF
(without adding it with a "add jar" statement) results in an error because
the UDF class could not be found.
I need to be able to create and call a UDF function as part of a unit test.
—
Reply to this email directly or view it on GitHub
https://github.com/bobfreitas/hiveunit-mr2/issues/1#issuecomment-129649620
.

Hi,
I'm having the same issue, so I followed @csaszarp 's suggestion and it worked.
But this only worked for java UDF, now I need to test python UDF, what can I do? Since python cannot use CREATE FUNCTION USING JAR, how can I load the python script without using add FILE?
Thanks in advance for help!
Li

@yishuaili sorry but Python was never a goal of mine.  All of my development and testing have been targeted for Java.  If you need this, then please feel free to fork the project and make whatever changes you need.


Question: How can TestDataFactory be setup for custom SObjects?

Hi @benahm - Well done on this library.  It seems easy to use and will definitely be useful in my org.  Can you provide an example of how to use the framework for custom objects?  It isn't clear how to define default values for custom objects.

Hello @chazwatkins
It is similar to using it with the Standard objects, you just need to adapt the field/relationship name with the __c & __r
Let's suppose you have a FirstObject__c that has a lookup to a SecondObject__c
You can create the first & second object using the TestDataFactory
TestDataFactory.createSObject('FirstObject__c', new Map<String, Object>{
       Description__c => 'value for description field on FirstObject__c',
       LookupToSecondObject__r.Description__c =>  'value for description field on FirstObject__c'
})

This creates a record for the FirstObject with a related record of the SecondObject
Hope this help
Ahmed

Thanks for the help, Ahmed.


Nested classes generation configuration

This PR is a part of base#508.
It enables the generation of strongly-typed columns and messages for all Model Compiler users by default.

@armiol, @dmdashenkov PTAL. This is what the configuration looks like after the changes.


Assignment 1 [Ji Li]

Introduction

My name is Ji Li. I am from the Faculty of Applied Science. I am from China and now have been studying here for one and half years. I love cooking and cycling.
Describe your research in 2-3 sentences to someone that is not from your field (please avoid abbreviations)

My ongoing research focuses on wastewater treatment processes, a crucial aspect of environmental protection. The key strengths in my work include the design and innovation of treatment technologies aimed at energy saving and recovery. Additionally, my career ethos prioritizes delivering added benefits to stakeholders and making substantial contributions to public environmental welfare.
My research entails the following aspects:

Research Aspect
Answer

Use/collect personal data (health data, interviews, surveys)
No

Use/collect experimental data (lab experiments, measurements with instruments)
Yes

Collaborate with industry
Yes

Write/develop software as the main output of the project
No

Use code (as in programming) for data analysis
No

Work with large data (images, simulation models)
No

Other:
N/A

Reflections on the importance of RDM videos

Many unexpected events can lead to the loss or destruction of our experimental data and documents.
Losing data means having to recreate or even restart experiments, which wastes a lot of time and energy.
After the long-term operation of my reactors, I often accumulate a significant amount of data. If I don't manage it well, I become confused, make mistakes, and sometimes even mess it up.
I once had a terrible experience when my laptop crashed while I was working on my paper and analyzing data.
What would you like to learn during this course?

Some tools that could help me to manage my data.
Some experience on how to handle and share a large amount of data.
Keep in mind that the improtance of data management.
Checklist assignments

[x] Assignment 1: creating a GitHub issue (the overview issue) (before 15 February, 10:00)
[x] Respond to the GitHub issue #101
[x] Assignment 2: Data Flow Map 1 (share a link in this issue before 22 February 13:00). Link: https://surfdrive.surf.nl/files/index.php/apps/richdocuments/public?fileId={file_id}&shareToken=XSgnw5xV7Cx77ol
[x] Provide feedback to at least one Assignment 2 from another participant
[x] Respond to the GitHub discussion #103
[x] Respond to the GitHub discussion #102
[x] Assignment 3: Data Flow Map 2 (share a link in this issue before 5 March 13:00). Link:https://surfdrive.surf.nl/files/index.php/s/RNxAj5TRDjFYG9U
[ ] Provide feedback to at least one Assignment 3 from another participant
[x] Assignment 4: Data Management Plan (before 8 March, 13:00)
[ ] Respond to the GitHub discussion #104 if you have any questions (optional)
[ ] Assignment 5: Data Flow Map 3: submit your slide (before 11 March, 17:00)

Hey Ji! I was reading your data flow map, and I was wondering a few things:

Why do you use a chat app for data storage? These are typically not designed for that, so maybe that is not very safe.
Why did you buy a drive of your own for data storage if the TU actually provides multiple options? If I read up about the company Baidu, I see there have been multiple reports of Baidu apps secretly gathering data, such as sensitive user data. It's a bit like why we shouldn't use google drive either, but with an extra layer: Being based in China, it is subject to state censorship by the Chinese government, meaning that the Chinese government has access to TU Delft research data. I would doublecheck whether this is allowed.

https://www.zdnet.com/article/baidus-android-apps-caught-collecting-sensitive-user-details/
https://www.reuters.com/article/idUSKCN0VX0EW/

Thanks @18810704625 for sharing your assignment 2! It looks very clear!
I have the same comments/suggestions as @SiemEerden - thank you for the feedback and questions!
In addition: are you using a paper or electronic lab note book? You could also consider this information as a data flow/set so that you can reflect upon how to store/manage this information for Assignment 3.

Thanks @JiLi24 for sharing the assignment!
Some suggestions:

To shorten your file names you can also take out the spaces. I don't think you're working with code/analysis scripts - otherwise that becomes even more important! Although you do mention GitHub later in the assignment - so it is not entirely clear to me if you're working with software or not!
What documentation ensures that you're always able to find these samples you'll need to store for the long term? Even with labels it can be a pain to find things back in the lab in my experience!
Regarding metadata: what are these parameters/conditions/steps?
While word/excel files are now interoperable, they're not necessarily open as you'll need a microsoft license to make use of all of the functionalities. Open alternatives are .txt or .csv files.
Uploading data in supplementary materials is not following the FAIR principles since supplementary materials do not have their own DOI associated with it so links might break in the long term, and if you publish behind a paywall the supplementary materials may become even more inaccessible. This is why also supplementary materials can be shared on a data repository such as 4TU.ResearchData! If needed, 4TU.ResearchData can also be used for restricted access datasets, where you/supervisors would need to approve access requests before people can reuse the data. This is only for confidential data as most data can be made publicly accessible!
While GitHub indeed allows for the assignment of a license, it is not a long term preservation platform and it does not assign DOIs to the code - Zenodo has a nice integration with GitHub that shares a snapshot of the repository on Zenodo and assigns this DOI that you then can link back to your GitHub Repo. You can also share your code via 4TU.ResearchData/GitHub, but the process is a bit more manual right now.
CC-BY is a license used for data but generally not for code: please check out the TU Delft approved software licenses: MIT, BSD, Apache, CC0, GPL, AGPL, LGPL, EUPL.

Thanks for your very nice feedback@EstherPlomp.
here are the answers and discussion to your comments/questions:
•	To shorten your file names you can also take out the spaces. I don't think you're working with code/analysis scripts - otherwise, that becomes even more important! Although you do mention GitHub later in the assignment - so it is not entirely clear to me if you're working with software or not!
A1: Good idea. I also think the name is a little long sometimes. However, after long-term storage, the short name will confuse me. The special explanation documentation or the file captions are needed to list all the detailed information (data types, collection data, purpose, reactor condition, samples lists, etc) about these documents.
Yeah, I don’t need to deal with codes or scripts. All my data are from the experiments and lab measurements.
•	What documentation ensures that you're always able to find these samples you'll need to store for the long term? Even with labels it can be a pain to find things back in the lab in my experience!
A2: I totally agree with you. That is also a challenge for me to quickly find and clarify my samples after a long storage time. Instead of managing data, the abilities of how to label, store and manage lab materials/stuff are also important and meaningful. Our technicians have already trained us though, we still need to exchange and study the cases from other experienced people.
•	Regarding metadata: what are these parameters/conditions/steps?
A3: Sorry for making you confused. I just regard our basic and standard parameters and procedures in the measurements and analysis as the “special metadata”, as these even should not be changed for reproducing the measurements. These data are always from the standard book or manual.
•	While word/excel files are now interoperable, they're not necessarily open as you'll need a microsoft license to make use of all of the functionalities. Open alternatives are .txt or .csv files.
A4: Thank you for your advice. I got the difference between them. .txt or .csv files will be my preference for these types of data documents.
•	Uploading data in supplementary materials is not following the FAIR principles since supplementary materials do not have their own DOI associated with it so links might break in the long term, and if you publish behind a paywall the supplementary materials may become even more inaccessible. This is why also supplementary materials can be shared on a data repository such as 4TU.ResearchData! If needed, 4TU.ResearchData can also be used for restricted access datasets, where you/supervisors would need to approve access requests before people can reuse the data. This is only for confidential data as most data can be made publicly accessible!
A5: very good discussion for the data sharing. Most of our important data will be described in the publication text. Besides, there are some explanations and supplementary data to give more detailed procedures and parameters for our main results. As you mentioned, it has obvious weaknesses in the long-term storage and also the paywall. 4TU.ResearchData would be the best choice for me to keep these datasets, which help us manage the data and approve the request easily.
•	While GitHub indeed allows for the assignment of a license, it is not a long term preservation platform and it does not assign DOIs to the code - Zenodo has a nice integration with GitHub that shares a snapshot of the repository on Zenodo and assigns this DOI that you then can link back to your GitHub Repo. You can also share your code via 4TU.ResearchData/GitHub, but the process is a bit more manual right now.
A6: Thanks for your recommendations. I will try to use 4TU.ResearchData and also the integration of Zenodo with GitHub for my sharing instead of only GitHub.
•	CC-BY is a license used for data but generally not for code: please check out the TU Delft approved software licenses: MIT, BSD, Apache, CC0, GPL, AGPL, LGPL, EUPL.
A7: Thanks. I will check it later. I am not familiar with these different types of licenses.

Glad to hear the feedback is helpful! And thanks for clarifying and answering my questions!
If you're not producing analysis scripts/software that you will share, you can disregard the licenses for software (the last comment)!

Hi Ji, thanks for sharing the assignment.
As Esther said, .txt or .csv files are open file formats and I would suggest keeping a .pdf version of the protocols as well. The other files are easy to edit, hence changes that happen by mistake could lead to inaccuracies in the protocol. It would be difficult to spot minor errors also. Therefore, having a copy of the protocols in pdf can be helpful.


Remove Copyright Assignment from the CLA

The CLA for contributing to this project requires copyright assignment, which is unnecessary. Please consider removing the copyright assignment from your CLA and instead replacing it with only copyright license.
Examples of decent CLAs that only contain copyright license:

GitHub's CLA – https://cla.github.com/agreement
Google's CLA – https://cla.developers.google.com/about/google-individual

Disclaimer: I'm not a lawyer and this isn't legal advice. I'm a software engineer who would like to submit a contribution without signing an agreement that contains copyright assignment.

Thank you for bringing this up.
I'll make sure it's seen by our lawyers & relevant decision makers (I'm also not a lawyer :) )

Closing due to inactivity.


Fix concurrency issue in HiveTableOperations when Table is reused

The same Table instance can be shared by multiple jobs meaning that TableOperations can be reused. Right now, doCommit in HiveTableOperations does not check whether base is still valid. It checks whether the current metadata location is valid. As a result, if another thread/job that uses the same HiveTableOperations updates the current metadata location, this might lead to data corruption.

@Parth-Brahmbhatt could you take a look?

@rdblue, this PR is ready for another review round

+1 when tests pass.

@rdblue, I am going to merge this one if there are no objections.

Go for it!


Show species in breeds tab

When going to Reference data tab, it doesn't automatically show the list of "Breeds/Species/Sexes", the user needs to use the Search button.

Also, when using the Search button, the Name & code column isn't displayed correctly at 1280px x 694px. I need to zoom out to see it properly.

It's working as expected.
In the future, we could set it to hide the columns at 1280px (including) and below.


PR: Add simplified chinese auto translation

Description of Changes

Issue(s) Resolved

Related to https://github.com/spyder-ide/spyder/issues/10672
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct:

Hi @kingmo888, @nnnewb and @gepcel, I added automatic (google) translation so you could start reviewing the work when you have some time :-)
Thanks!

Closing in favor of PR #10845.


Changed methods names

Changed a better methods names

Coverage remained the same at 100.0% when pulling 024c203278126888b871f1ff18f67b1edb7512e0 on ricardogobbosouza:naming into d77d90cc4b583b4d0cf0da60cccce4f9c2b4c7b6 on cknow:master.


Webpack includes jstree/dist/jstree when I include jstree/src/jstree

Bug report

What is the current behavior?
I'm including jstree like this:
import 'jstree/src/jstree.js'
import 'jstree/src/jstree.dnd.js'

I have checked jstree.js and jstree.dnd.js and none of them have any imports.
If the current behavior is a bug, please provide the steps to reproduce.
I have set up a test repository here (project generated with vue create test-jstree : https://github.com/Herz3h/test-jstree-webpack
Just run:
$ yarn install
$ yarn serve

What is the expected behavior?
It should only includes the files in jstree/src not the jstree/dist ones....

Other relevant information:
webpack version:  4.43.0
Node.js version: 10.21.0
Operating System: MacOS Mojave
Additional tools: -

Not related to webpack, look here https://github.com/vakata/jstree/blob/master/src/jstree.dnd.js#L13, jstree.dnd.js require jstree package and wepback resolve it from dist, you can use alias https://webpack.js.org/configuration/resolve/#resolvealias to avoid it

Ideally you should not use files from src

I somehow skipped that top part... Okay I'll close this. thank you.


Doubly Linked List in C++

Description
I have added proper documentation for doubly linked list
Domain
C/CPP
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign


fix posthog

#4449  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @jnsdls and the rest of your teammates on  Graphite


Not opening in WebView but in browser

install.log
install starting  - Tue Sep 24 21:14:39 +03 2024
installing on aarch64 processor system
the os is PRETTY_NAME="Debian GNU/Linux 12 (bookworm)" NAME="Debian GNU/Linux" VERSION_ID="12" VERSION="12 (bookworm)" VERSION_CODENAME=bookworm ID=debian HOME_URL="https://www.debian.org/" SUPPORT_URL="https://www.debian.org/support" BUG_REPORT_URL="https://bugs.debian.org/"
total        used        free      shared  buff/cache   available
Mem:            7810         584        5493          45        1900        7225
Swap:            199           0         199
Updating packages ...
Hit:1 http://deb.debian.org/debian bookworm InRelease Hit:2 http://archive.raspberrypi.com/debian bookworm InRelease Hit:3 http://deb.debian.org/debian-security bookworm-security InRelease Hit:4 http://deb.debian.org/debian bookworm-updates InRelease Reading package lists...
apt-get update  completed ok
Reading package lists...
Building dependency tree...
Reading state information...
pv is already the newest version (1.6.20-1).
0 upgraded, 0 newly installed, 0 to remove and 5 not upgraded.
apt-get upgrade  started
apt-get upgrade result =rc=0 Reading package lists...
Building dependency tree...
Reading state information...
Calculating upgrade...
The following packages have been kept back:
chromium-browser chromium-browser-l10n chromium-codecs-ffmpeg-extra
raspi-utils rpi-chromium-mods
0 upgraded, 0 newly installed, 0 to remove and 5 not upgraded.
Installing helper tools ...
Reading package lists...
Building dependency tree...
Reading state information...
curl is already the newest version (7.88.1-10+deb12u7).
wget is already the newest version (1.21.3-1+b1).
git is already the newest version (1:2.39.5-0+deb12u1).
build-essential is already the newest version (12.9).
unzip is already the newest version (6.0-28).
0 upgraded, 0 newly installed, 0 to remove and 5 not upgraded.
installing on bookworm
Cloning MagicMirror ...
Cloning MagicMirror Done!
Installing dependencies ...

<EMAIL_ADDRESS>postinstall > npm run install-vendor && npm run install-fonts && echo "MagicMirrorÂ² installation finished successfully! > " ><EMAIL_ADDRESS>install-vendor > echo "Installing vendor files ... > " && cd vendor && npm install --loglevel=error --no-audit --no-fund --no-update-notifier Installing vendor files ... added 10 packages in 4s ><EMAIL_ADDRESS>install-fonts > echo "Installing fonts ... > " && cd fonts && npm install --loglevel=error --no-audit --no-fund --no-update-notifier Installing fonts ... added 2 packages in 2s MagicMirrorÂ² installation finished successfully! ><EMAIL_ADDRESS>prepare > [ -f node_modules/.bin/husky ] && husky || echo no husky installed. no husky installed. added 433 packages in 5m 102 packages are looking for funding run npm fund for details
Dependencies installation Done!
package.json update for aarch64 or Electron missing, completed ok
setting up initial config.js
Check plymouth installation ...
Splashscreen: Checking themes directory.
Splashscreen: Create theme directory if not exists.
Splashscreen: Changed theme to MagicMirror successfully.
gsettings screen saver already disabled
disable screensaver via lightdm.conf
disable screensaver via lxsession
Found: wayfire.ini
wayfire screen saver already disabled
We're ready! Run DISPLAY=:0 npm start from the ~/MagicMirror directory to start your MagicMirror.
install completed - Tue Sep 24 21:26:26 +03 2024

package.json update for aarch64 or Electron missing, completed o

so, elevtron didn't install..
that means I setup for using chrome (or chromium)  over the server mode
BUT I dont check if chrome is installed..
did you setup to use pm2 to autostart?
if so, from a terminal window do
pm2 stop all
then do
which chrome
which chromium
if they don't return anything then they are not installed
you can also use firefox, midori or push  (ee the doc on my script at the end descrives what to do
we can confirm
cd ~/MagicMirror
npm start

and look at the messages

package.json update for aarch64 or Electron missing, completed o

so, electron didn't install..
that means I setup for using chrome (or chromium)  over the server mode
BUT I dont check if chrome is installed..
did you setup to use pm2 to autostart?
if so, from a terminal window do
pm2 stop all
then do
which chrome
which chromium
if they don't return anything then they are not installed
you can also use firefox, midori or push  (ee the doc on my script at the end descrives what to do
we can confirm
cd ~/MagicMirror
npm start

and look at the messages

No, I did not do autorun, but when I run an instruction npm start
A white screen appears for a few seconds and then disappears The screen freezes

did you check from chrome? or chromium?

Yes it works correctly
sam detweiler @.***> schrieb am Mi., 25. Sept. 2024,
00:23:

did you check from chrome? or chromium?
—
Reply to this email directly, view it on GitHub
https://github.com/MagicMirrorOrg/MagicMirror/issues/3557#issuecomment-2372410754,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZ2AJENU53KTTUQQIV55V5TZYHJ3XAVCNFSM6AAAAABOZAPWK6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNZSGQYTANZVGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

can you show the output of npm start


Indenting with spaces doesn't work

I think this has to do with saving using tabs by default and empty lines being added between all events.

Actually this doesn't only mess up indenting but the events themselves. They somehow don't get recognized correctly so shortcode events just turn into texts.


Newly created Card Bug Fixed

The newly created cards appear to be compressed in width, resulting in a less-than-ideal appearance.  Please Add hactoberfest2023 label  or hactoberfest-accepted .

it's fixed


Bitmap encoding

Additional encoding algorithm for distributed training, with static memory requirements. Suitable for dense updates.

Closing PR temporary