Documentation needs improvement

I'm following the documentation to install easymail. After all the steps are completed, I notice that the default sending address is<EMAIL_ADDRESS>instead of<EMAIL_ADDRESS>

Putting the system through its tests, a sample email to mail-tester.com reports a score of 0/10. It says that the DKIM keys are invalid.

The DNS records that need to be applied warrant a special page in themselves. There's too much ambiguity between what the system generates vs what the user expects to see...
For ex: Does mail._domainkey need to be applied to root domain or mail subdomain?
The input that the code accepts for $HOSTNAME is shown to be mail.example.com which makes this mail._domainkey.mail in most cases (atleast in my case with linode).
We need atleast a few examples that will show the setup in the right direction. I tried to set it up for one of my domains and backed out because there was too much trouble following instructions.
I would like to give this a more patient shot with a spare domain I have lying around if thats okay by you guys.

We will release a new version very soon which and we will have to change the documentation on the website because there will be many changes. We will take into account your notes.
About the DKIM. With the new version, we will provide a new solution for the DKIM because its configuration will be moved to the ManagementAPI.
Your note:

For ex: Does mail._domainkey need to be applied to root domain or mail subdomain?
The input that the code accepts for $HOSTNAME is shown to be mail.example.com which makes this mail._domainkey.mail in most cases (atleast in my case with linode).
We need atleast a few examples that will show the setup in the right direction. I tried to set it up for one of my domains and backed out because there was too much trouble following instructions.
I would like to give this a more patient shot with a spare domain I have lying around if thats okay by you guys.

will be added to the new document. Thanks!

All of this problems and suggestion are already applied, closing this one, no more action needed. Please re-open in case still need improvement


Does this plugin work with cordova-plugin-googlemaps

I am using ngmap in my app which is being compiled by Cordova.
How would this interact with https://github.com/mapsplugin/cordova-plugin-googlemaps ?

Sorry, I have no idea about that plugin. That means no.


(PUP-1153) force group removal

The groupdel command from https://github.com/shadow-maint/shadow refuses
to remove groups which are set as the primary GID for a user.  This
effects operating systems such as Debian which uses the shadow codebase.
This is problematic in a Puppet managed world, where it is difficult
to ensure that any dependent users are removed prior to removing the
group.
I added a force option to the groupdel command to force the removal even
when a user has the group set as their GID, so use that flag if it is
available.

Can one of the admins verify this patch?

CLA signed by all contributors.

any chance someone could take a look at this?

Forcing group deletion feels like the wrong thing to do as has been discussed on the ticket. I added some comments in https://tickets.puppetlabs.com/browse/PUP-1153?focusedCommentId=659565

closing based off of our discussion in PUP-1153


How to install it under Apache2

/var/www/screeenly# node -v
v11.15.0
/var/www/screeenly# npm -v
6.7.0
I want it reachable under DOMAIN.TLD/screeenly
I followed the guide and I came to the point:
npm install

><EMAIL_ADDRESS>install /var/www/screeenly/node_modules/puppeteer
> node install.js

ERROR: Failed to download Chromium r686378! Set "PUPPETEER_SKIP_CHROMIUM_DOWNLOAD" env variable to skip download.
{ Error: EACCES: permission denied, mkdir '/var/www/screeenly/node_modules/puppeteer/.local-chromium'
  -- ASYNC --
    at BrowserFetcher.<anonymous> (/var/www/screeenly/node_modules/puppeteer/lib/helper.js:111:15)
    at Object.<anonymous> (/var/www/screeenly/node_modules/puppeteer/install.js:64:16)
    at Module._compile (internal/modules/cjs/loader.js:816:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:827:10)
    at Module.load (internal/modules/cjs/loader.js:685:32)
    at Function.Module._load (internal/modules/cjs/loader.js:620:12)
    at Function.Module.runMain (internal/modules/cjs/loader.js:877:12)
    at internal/main/run_main_module.js:21:11
  errno: -13,
  code: 'EACCES',
  syscall: 'mkdir',
  path: '/var/www/screeenly/node_modules/puppeteer/.local-chromium' }
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node install.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2019-09-14T21_08_16_294Z-debug.log

/etc/apache2/sites-avaible/screeenly.conf looks like:
Alias /screeenly "/var/www/screeenly/public"

DocumentRoot /var/www/screeenly/public/
<Directory /var/www/screeenly/public/>
  Options +FollowSymlinks
  AllowOverride All

 <IfModule mod_dav.c>
  Dav off
 </IfModule>

 SetEnv HOME /var/www/screeenly/public
 SetEnv HTTP_HOME /var/www/screeenly/public

</Directory>
        ErrorLog ${APACHE_LOG_DIR}/screeenly_error.log
        LogLevel warn
        CustomLog ${APACHE_LOG_DIR}/screeenly_access.log combined

Looks like this:

The hyperlinks are going to Domain.tld/[try]
[for example] and not to Domain.tld/screeenly/try
Thx for help

Hm, I have no experience with installing a Laravel app in a sub-folder.
Could you post the value for APP_URL in your .env-file?
I think this should be domain.tld/screeenly.
The npm install problem seems to be permission based.
permission denied, mkdir '/var/www/screeenly/node_modules/puppeteer/.local-chromium'

Can you manually create the folder?

Here a litte overview what I´ve tested:
CLICK ME

npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2019-09-16T20_55_55_114Z-debug.log
root@nextcloud:/var/www/screeenly# service apache2 restart
root@nextcloud:/var/www/screeenly# mkdir /var/www/screeenly/node_modules/puppeteer/.local-chromium
mkdir: cannot create directory '/var/www/screeenly/node_modules/puppeteer/.local-chromium': No such file or directory
root@nextcloud:/var/www/screeenly# mkdir /var/www/screeenly/node_modules/puppeteer/.local-chromium
mkdir: cannot create directory '/var/www/screeenly/node_modules/puppeteer/.local-chromium': No such file or directory
root@nextcloud:/var/www/screeenly# mkdir /var/www/screeenly/node_modules/
Display all 702 possibilities? (y or n)
root@nextcloud:/var/www/screeenly# mkdir /var/www/screeenly/node_modules/pu
public-encrypt/          pump/                    pumpify/                 punycode/                purgecss/                purgecss-webpack-plugin/
root@nextcloud:/var/www/screeenly# mkdir /var/www/screeenly/node_modules/pu
public-encrypt/          pump/                    pumpify/                 punycode/                purgecss/                purgecss-webpack-plugin/
root@nextcloud:/var/www/screeenly# mkdir /var/www/screeenly/node_modules/puppeteer
root@nextcloud:/var/www/screeenly# mkdir /var/www/screeenly/node_modules/puppeteer/.local-chromium
root@nextcloud:/var/www/screeenly# npm install

<EMAIL_ADDRESS>install /var/www/screeenly/node_modules/puppeteer
 node install.js

ERROR: Failed to download Chromium r686378! Set "PUPPETEER_SKIP_CHROMIUM_DOWNLOAD" env variable to skip download.
{ Error: EACCES: permission denied, mkdir '/var/www/screeenly/node_modules/puppeteer/.local-chromium'
  -- ASYNC --
    at BrowserFetcher.<anonymous> (/var/www/screeenly/node_modules/puppeteer/lib/helper.js:111:15)
    at Object.<anonymous> (/var/www/screeenly/node_modules/puppeteer/install.js:64:16)
    at Module._compile (internal/modules/cjs/loader.js:816:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:827:10)
    at Module.load (internal/modules/cjs/loader.js:685:32)
    at Function.Module._load (internal/modules/cjs/loader.js:620:12)
    at Function.Module.runMain (internal/modules/cjs/loader.js:877:12)
    at internal/main/run_main_module.js:21:11
  errno: -13,
  code: 'EACCES',
  syscall: 'mkdir',
  path: '/var/www/screeenly/node_modules/puppeteer/.local-chromium' }
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node install.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2019-09-16T20_57_41_049Z-debug.log
root@nextcloud:/var/www/screeenly#

/etc/apache2/sites-avaible/006-screenly.conf

<VirtualHost *:80>
        # The ServerName directive sets the request scheme, hostname and port that
        # the server uses to identify itself. This is used when creating
        # redirection URLs. In the context of virtual hosts, the ServerName
        # specifies what hostname must appear in the request's Host: header to
        # match this virtual host. For the default virtual host (this file) this
        # value is not decisive as it is used as a last resort host regardless.
        # However, you must set it for any further virtual host explicitly.
        ServerName screen.DOMAIN.tld

        ServerAdmin<EMAIL_ADDRESS>        DocumentRoot /var/www/screeenly/public

        # Available loglevels: trace8, ..., trace1, debug, info, notice, warn,
        # error, crit, alert, emerg.
        # It is also possible to configure the loglevel for particular
        # modules, e.g.
        #LogLevel info ssl:warn

        ErrorLog ${APACHE_LOG_DIR}/error.log
        CustomLog ${APACHE_LOG_DIR}/access.log combined

        # For most configuration files from conf-available/, which are
        # enabled or disabled at a global level, it is possible to
        # include a line for only one particular virtual host. For example the
        # following line enables the CGI configuration for this host only
        # after it has been globally disabled with "a2disconf".
        #Include conf-available/serve-cgi-bin.conf

RewriteEngine on
RewriteCond %{SERVER_NAME} =screen.Domain.tld
RewriteRule ^ https://%{SERVER_NAME}%{REQUEST_URI} [END,NE,R=permanent]
</VirtualHost>

# vim: syntax=apache ts=4 sw=4 sts=4 sr noet

I also get the permission error. How you see even after creating the directories.
After chown www-data:www-data /var/www/screeenly/ it finally looks like it.

But now I get the problem, that only the design is back, but the 4 links on the page gives 404 back.
https://screen.DOMAIN.tld/try

Thanks for your logs.
The error during npm install is puppeteer related; so it's probably better to search for solutions on their repo.
A quick search gave me the following possible solutions:

https://github.com/GoogleChrome/puppeteer/issues/4574
https://github.com/GoogleChrome/puppeteer/issues/375#issuecomment-363466257
https://github.com/GoogleChrome/puppeteer/issues/367#issuecomment-499315095

Running the following command seems to fix the problem.
sudo npm install puppeteer --unsafe-perm=true

Concerning links pointing not to the sub-directory: All links in the navigation are hardcoded to use / at the beginning. See the try-link here.
You would have to manually update the _navigation.blade.php-file on your server, or fork the project.

I just forgot to add this to vhost configuration:
<Directory /var/www/screeenly/>
        Options +FollowSymlinks
        AllowOverride All
</Directory>

Thanks for your help with npm install puppeteer.
Then I guessed how the github callback link is defined.
For all others:
GITHUB_REDIRECT_URL=https://DOMAIN.tld/oauth/github/handle

Glad everything works now. 👍🚀

I thought it is working now. But when I try to reach the ".tld/api/v1" I got a 404. Any ideas?
When I do composer install and npm install, all works fine.
<VirtualHost *:80>
       # The ServerName directive sets the request scheme, hostname and port that
       # the server uses to identify itself. This is used when creating
       # redirection URLs. In the context of virtual hosts, the ServerName
       # specifies what hostname must appear in the request's Host: header to
       # match this virtual host. For the default virtual host (this file) this
       # value is not decisive as it is used as a last resort host regardless.
       # However, you must set it for any further virtual host explicitly.
       ServerName screen.DOMAIN.TLD

       ServerAdmin<EMAIL_ADDRESS>       DocumentRoot /var/www/screeenly/public

       # Available loglevels: trace8, ..., trace1, debug, info, notice, warn,
       # error, crit, alert, emerg.
       # It is also possible to configure the loglevel for particular
       # modules, e.g.
       #LogLevel info ssl:warn

       ErrorLog ${APACHE_LOG_DIR}/error.log
       CustomLog ${APACHE_LOG_DIR}/access.log combined

       # For most configuration files from conf-available/, which are
       # enabled or disabled at a global level, it is possible to
       # include a line for only one particular virtual host. For example the
       # following line enables the CGI configuration for this host only
       # after it has been globally disabled with "a2disconf".
       #Include conf-available/serve-cgi-bin.conf

<Directory /var/www/screeenly/>
       Options +FollowSymlinks
       AllowOverride All
</Directory>

RewriteEngine on
RewriteCond %{SERVER_NAME} =screen.DOMAIN.TLD
RewriteRule ^ https://%{SERVER_NAME}%{REQUEST_URI} [END,NE,R=permanent]
</VirtualHost>

# vim: syntax=apache ts=4 sw=4 sts=4 sr noet

Is it, because I´m using subdomain?

The API endponit is /api/v1/fullsize not just /api/v1.  (Docs)
Could you test your installation with the correct endpoint?
What exactly do you mean with "When I do composer install and npm install, all works fine."?
Are you trying to run the application without running these commands? Do you delete vendor and node_modules regularly?!

Is it, because I´m using subdomain?

This shouldn't have any impact.


fix SCRIPT_NAME to not disclose location on disk

This closes an information disclosure hole in Wagi, and also makes it consistent with the letter of the CGI 1.1 RFC, though not with the intent of the RFC.
Signed-off-by: Matt Butcher<EMAIL_ADDRESS>

DO NOT MERGE YET! I just re-read the RFC and this is incorrect.


B 20164 int

20164
Summary
This task updates the ppm estimator to use the pickupAddress.postalcode and destinationAddress.postalCode. This is done in support of the effort to move ppm's to using a full address. I have attached screenshots of previous PPMs and the attached information as examples that the changes did not change the results from the ppm estimator.
Verification Steps for the Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Have the Agility acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

[ ] Has the branch been pulled in and checked out?
[ ] Have the BL acceptance criteria been met for this change?
[ ] Was the CircleCI build successful?
[ ] Has the code been reviewed from a standards and best practices point of view?

Setup to Run the Code

Instructions for starting storybook
Instructions for starting the MilMove application
Instructions for running tests

How to test

Access the
Login as a

Frontend

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, Edge).
[ ] There are no new console errors in the browser devtools.
[ ] There are no new console errors in the test output.
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.
[ ] This change meets the standards for Section 508 compliance.

Backend

[ ] Code follows the guidelines for Logging.
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Database
Any new migrations/schema changes:

[ ] Follows our guidelines for Zero-Downtime Deploys.
[ ] Have been communicated to #g-database.
[ ] Secure migrations have been tested following the instructions in our docs.

Screenshots

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

  Generated by :no_entry_sign: dangerJS against 68f77afad3f36848b19d2a530260085469c3613f

Estimated Incentive amount didn't change. Looks good.


[FontIcon] does not display on Android Browser

Problem description
FontIcon does not display on Android Browser
Steps to reproduce
go to the following page on a Samsung Browser (tried on S6). Not Chrome but Android browser shipping with Samsung
http://www.material-ui.com/#/components/font-icon
Public icons do not display
Versions

Material-UI: 16
React:
Browser: Samsung Galaxy S6 Browser

Can you please provide a screenshot?

Icons are being displayed correctly from the Google Material Design Icon site. If I stop using ligatures in my code it works. I am using Android Browser 4 so it should work though. Not exactly sure where the problem is with Material UI

You could try using react-icons which uses SVG images instead of relying on an icon font...
See also #5256

@tracker1 Or just material-ui/svg-icons and save an additional dependancy.

We have been porting the component on the v1-beta branch. We reimplemented it from the ground-up. While we haven't tested it, I think that the issue is most likely fixed on that branch. Hence, I'm closing it.
Still, we will accept PR fixes until v1-beta takes over the master branch.


Do not fetch app from current folder on heroku destroy

Require the app to be passed as a param/arg instead.
The motivation is to avoid something like:
$ heroku destroy app1
! WARNING: Potentially Destructive Action
! This command will affect the app: very-important-production-app

@bgentry Thanks for the input.  Arguably the value of a stopgap such as this is related to how long it is likely to be before the command is fixed, I'm working on that and related things over the course of the next couple weeks.  So how urgent do you think this needs fixing?
@pedro How urgent do you think this is as well?  If it isn't too urgent I'd prefer to get a proper fix in.

I wrote this patch in response to a support ticket, so we're still seeing users getting bitten by this.
But despite that, I'd say this is not particularly urgent: it's just the way the gem has been working for a long time. And even if we apply this patch today, it will take a considerable amount of time until everyone updates - so I think waiting a few more weeks for a proper solution sounds reasonable.
If anything this issue also points out that confirmation messages are not working the way they were intended. Out of curiosity, do you guys have plans to get rid or replace them?

(But yeah closing this request then, will wait for proper option parsing).

Let's fix this case with a stopgap measure. It's too easy to make a really bad mistake right now.


Control available tasks

Can you extend the documentation with a pointer to how to control the available tasks on a custom build agent? We would like to only allow for a set of 40 custom tasks we provide to the particular Azure DevOps org.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: dcea6ccb-570f-da20-f9ba-97894cc91cbf
Version Independent ID: 0444933a-f1c6-cd0c-8386-e0da3bd0fd9c
Content: Other security considerations - Azure Pipelines
Content Source: docs/pipelines/security/misc.md
Product: devops
Technology: devops-cicd-security
GitHub Login: @juliakm
Microsoft Alias: jukullam

@cveld Thanks for your suggestion. While we look into the article, consider these resources:

Azure DevOps Services
Azure DevOps Support Bot
Azure DevOps on Stack Overflow


Add topic "kokkos"

Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/main/CONTRIBUTING.md.
[x] I am not the sole author or employee of a company who created the topic or collection I'm changing.

Which change are you proposing?

[ ] Suggesting edits to an existing topic or collection
[x] Curating a new topic or collection
[ ] Something that does not neatly fit into the binary options above

Curating a new topic or collection

[x] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[x] My folder contains a *.png image (if applicable) and index.md
[x] All required fields in my index.md conform to the Style Guide and API docs: <https://github.com/github/explore/tree/main/docsa

Kokkos is widely used in HPC projects and at the moment of writing there are already 58 public repositories matching this topic.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Available fields with descriptions: https://github.com/github/explore/blob/main/docs/API.md.


Added SSR Resolution Selection

Full, Half, and Quarter setting in WorldEnvironment under Ss Reflections.
Replaces 0.5 with a custom multiplier.
Higher resolution requires more steps, else the range of the reflections is shorter.
Slight performance changes per resolution. Quarter res + lowering max depth may be useful for lower end devices.

Note there are minor changes in appearance based on resolution. For ex, blur/roughness is more apparent on higher resolutions.
Closes #19737

Could you squash the commits together?

Should be fine now I think.

This doesn't seem to do anything, just changing a multiplier should not add more resolution. Probably need to use the upper or lower mipmaps too.

Hmm that's what I figured too but saw visual and performance changes when making adding this. I'm assuming you don't have time to pull and check every PR! But here are my findings:
Visual changes between settings:
Note that we can decrease "max depth" (which improves performance) with lower resolutions (which also improves performance) since the lower resolution results in "longer" reflections. i.e. moving from depth 512 full res to depth 256 half res yields the same "amount" of reflections, but blurrier.

I also ran a very brief "benchmark". Just played the loop in the GIF below 3 times on a bunch of settings while recording with MSI Afterburner. Results done with R5 2400G and GTX 1080.
https://giphy.com/gifs/Xep0EOWXwTAGRQyutQ/html5

Does Godot render everything at full res natively in SSR? Perhaps this is just the "Output Resolution"? So to avoid being misleading we could rename it from "Resolution" to "Output Resolution"

From Juan's tip I looked into it more and it appears Godot uses half resolution mipmaps for the reflection, and if I understand correctly this commit is the output and is similar to Unity's SSR Quality setting which is "The size of the buffer used for resolve".
I haven't been able to get the hang of using full resolution mipmaps, but like I said renaming the setting is an option since it appears to have tangible benefits. I'll update if I work out "input"/"render" resolution (or whatever you might name it from using upper/lower mipmaps).

I think something went wrong with your branch?

Yes, I have merged master to fix the merge conflict incorrectly, give me a few minutes to rebase.

Apologies for the confusion, it should be fixed now...

This is unrelated to this PR, but I noticed the "edge" between the material and the reflection becomes more visible as the resolution is lowered. Is there a way to make it less noticeable (e.g. by using a bias)?

Do you mean this effect?

This happens with the current implementation when setting the depth to a high value and fade out to a low value, as well. I think the change in buffer size just allows it to glitch earlier (or later). Similar effect here.

Or are you instead referring to the fade out being more sudden? (Also somewhat visible above)
If so, I suppose you just have to raise the value of "Fade Out" to counter it. Although, I suppose it is possible to have the engine compensate for it automatically for consistency.

@puchik I meant the blue halo that's visible here:

Ah, I see. Yes, I would think that would be a separate fix/PR since it appears on all resolutions (just at different severity) and a bias is a good idea. I like that approach. I suppose it would raise the reflection in a very small amount and would be user-defined.
Increasing roughness also helps but will blur the entire reflection (not to mention issues with roughness values and reflection edges in SSR at the time of writing, so this would only help once those issues are resolved).
Fade out compensation also makes sense. Up to core devs to decide if that's the way to go.

We discussed this with @reduz, this PR will likely have to wait some more as merging it now would introduce conflicts with the refactor of the VisualServer and rendering API in the vulkan branch, which would make rebasing the vulkan branch on master harder.
So we should wait for 3.2 to be released, and then the vulkan branch will be merged in master. This PR can then be rebased on the new architecture.

@puchik Is this still desired? If so, it needs to be rebased on the latest master branch, and reworked to support Vulkan.
If not, abandoned pull requests will be closed in the future as announced here.

I'm not sure if this is still relevant with the new SSR implementation in the Vulkan branch. I think it has quality settings available in the Project Settings already.

I was actually just thinking about this last week (since there was a progress update). I just checked and there's only a quality setting for roughness in the Vulkan branch. There's also an unused use_half_res in the SSR shader, though, so I suppose there's already a plan on how to do it.
Since there's still a decent amount of work to do for rendering in 4.0 I think I'll close this. If reduz already has a plan on how to do it, it's probably better to leave him finish his implementation (not to mention I wouldn't be looking at this for a bit).
I would honestly scrap what's already in here anyway because of how different it is now so there's no point rebasing it. This PR is getting kind of messy, too.


video test promblem

Dear Arthur151, when test video on your method, last frame always gets mistake, hope you reply.
this is issue:
pygame 2.0.0 (SDL 2.0.12, python 3.7.9)
Hello from the pygame community. https://www.pygame.org/contribute.html
/home/neu307/miniconda3/envs/env_centerhmr/lib/python3.7/site-packages/quaternion/numba_wrapper.py:21: UserWarning:
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
Could not import from numba, which means that some
parts of this code may run MUCH more slowly.  You
may wish to install numba.
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
warnings.warn(warning_text)
INFO - 2021-11-16 15:56:39,929 - acceleratesupport - No OpenGL_accelerate module loaded: No module named 'OpenGL_accelerate'
INFO - 2021-11-16 15:56:40,399 - base - {'tab': 'hrnet_cm64_test', 'configs_yml': 'configs/video.yml', 'demo_image_folder': 'None', 'local_rank': 0, 'model_version': 1, 'multi_person': True, 'collision_aware_centermap': False, 'collision_factor': 0.2, 'kp3d_format': 'smpl24', 'eval': False, 'max_person': 64, 'input_size': 512, 'Rot_type': '6D', 'rot_dim': 6, 'centermap_conf_thresh': 0.25, 'centermap_size': 64, 'deconv_num': 0, 'model_precision': 'fp32', 'backbone': 'hrnet', 'gmodel_path': '../trained_models/ROMP_hrnet32.pkl', 'print_freq': 50, 'fine_tune': True, 'gpu': '0', 'batch_size': 64, 'val_batch_size': 1, 'nw': 4, 'calc_PVE_error': False, 'dataset_rootdir': '/home/neu307/wh/dataset/', 'high_resolution': True, 'save_best_folder': '/home/neu307/wh/checkpoints/', 'log_path': '/home/neu307/wh/log/', 'total_param_count': 85, 'smpl_mean_param_path': '/home/neu307/wh/ROMP-master/models/satistic_data/neutral_smpl_mean_params.h5', 'smpl_model': '/home/neu307/wh/ROMP-master/models/statistic_data/neutral_smpl_with_cocoplus_reg.txt', 'smplx_model': True, 'cam_dim': 3, 'beta_dim': 10, 'smpl_joint_num': 22, 'smpl_model_path': '/home/neu307/wh/ROMP-master/models', 'smpl_uvmap': '/home/neu307/wh/ROMP-master/models/smpl/uv_table.npy', 'smpl_female_texture': '/home/neu307/wh/ROMP-master/models/smpl/SMPL_sampleTex_f.jpg', 'smpl_male_texture': '/home/neu307/wh/ROMP-master/models/smpl/SMPL_sampleTex_m.jpg', 'smpl_J_reg_h37m_path': '/home/neu307/wh/ROMP-master/models/smpl/J_regressor_h36m.npy', 'smpl_J_reg_extra_path': '/home/neu307/wh/ROMP-master/models/smpl/J_regressor_extra.npy', 'kernel_sizes': [5], 'GPUS': 0, 'use_coordmaps': True, 'video_or_frame': True, 'input_video_path': '../demo/test_videos/C1/C1_single_2.mp4', 'webcam_mesh_color': 'LightCyan', 'save_mesh': False, 'save_centermap': False, 'save_dict_results': True, 'webcam': False, 'fps_save': 30, 'multiprocess': False}
INFO - 2021-11-16 15:56:40,399 - base - ------------------------------------------------------------------
INFO - 2021-11-16 15:56:40,399 - base - start building model.
Using ROMP v1
INFO - 2021-11-16 15:56:43,931 - train_utils - using fine_tune model: ../trained_models/ROMP_hrnet32.pkl
INFO - 2021-11-16 15:56:44,085 - base - finished build model.
Initialization finished!

I guess you are running code on a computer with multiple GPUs.
Please run code with
"CUDA_VISIBLE_DEVICES=0 "
For instance, when you run it in processing video, please run it via
CUDA_VISIBLE_DEVICES=0 python -u -m romp.predict.video --configs_yml='configs/video.yml'

Sorry about that. I guess you are running code on a computer with multiple GPUs. Please run code with "CUDA_VISIBLE_DEVICES=0 " For instance, when you run it in processing video, please run it via CUDA_VISIBLE_DEVICES=0 python -u -m romp.predict.video --configs_yml='configs/video.yml'

i run it on pycharm, have setted the cuda=0

Then that would be pretty weird.
I am chasing for a deadline now, for now, you use try except + repeat the last few frame to make it pass.
After the deadline, I will look into it.
Is this ok?

Then that would be pretty weird. I am chasing for a deadline now. For now, you may use try except + repeat the last few frame to make it pass. After the deadline, I will look into it. Is this ok?

thank you, i will try

I add try-except, and it works. Thank u

My appologize to the unpleasant back.
I will get back to this later! Please remind me of this if you are in a hurry.

@jiandandian2 I am back to this issue.
I have just updated a new version of ROMP.
I just test it with processing 161/162/163/164 frames. It works fine.
Could you please try it now?


Remove specs without open changes for git-sync

@charlierudolph @allewun @ricmatsui
Implements parts of #506

:ship: after comment is addressed

Yes, that's a leftover from the rename-branch PR that must somehow slipt in here. Since that PR also got the green light, I'm going to submit it as part of this one.

I'm sorry for the files from the wrong branches. I think its because I have been using Dropbox to sync my local Git Town setup between computers (a pre-Git Town setup that I used to write Git Town initially). Apparently with such large changes the synchronization takes a while, and some files show up delayed in the workspace.
I am no longer using Dropbox for code any longer, so this shouldn't be an issue any longer. Good that you caught this issue @charlierudolph! Thanks to Git Town getting really awesome, I will now sync-all the full workspace with GitHub before switching machines!


Install error

Hello,
I can't install this package, i have the following error :
> devtools::install_github("hrbrmstr/uaparserjs")
Downloading GitHub repo hrbrmstr/uaparserjs@master
from URL https://api.github.com/repos/hrbrmstr/uaparserjs/zipball/master
Installing uaparserjs
"C:/Softs/R-33~1.3/bin/x64/R" --no-site-file --no-environ --no-save --no-restore --quiet CMD INSTALL  \
  "C:/Users/Victor/AppData/Local/Temp/RtmpMxl2XI/devtools2b043354fb0/hrbrmstr-uaparserjs-ecdc86c" --library="C:/Softs/R-3.3.3/library" --install-tests

* installing *source* package 'uaparserjs' ...
** R
** inst
** tests
** preparing package for lazy loading
** help
*** installing help indices
** building package indices
** testing if installed package can be loaded
*** arch - i386
Error : .onAttach failed in attachNamespace() for 'uaparserjs', details:
  call: context_eval(join(src), private$context)
  error: RangeError: Maximum call stack size exceeded
Erreur : le chargement a échoué
Exécution arrêtée
*** arch - x64
ERROR: loading failed for 'i386'
* removing 'C:/Softs/R-3.3.3/library/uaparserjs'
Error: Command failed (1)

I have V8 version 1.4, and here's my sessionInfo() :
R version 3.3.3 (2017-03-06)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows >= 8 x64 (build 9200)

locale:
[1] LC_COLLATE=French_France.1252  LC_CTYPE=French_France.1252    LC_MONETARY=French_France.1252 LC_NUMERIC=C                   LC_TIME=French_France.1252

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

loaded via a namespace (and not attached):
[1] httr_1.2.1      R6_2.2.0        tools_3.3.3     withr_1.0.2     curl_2.5        memoise_1.0.0   git2r_0.18.0    digest_0.6.12   devtools_1.12.0

Thanks !

From the README.md:

This was briefly on CRAN but the i386 Windows build of it has issues due to the limitations of the v8 engine (not the V8 package) in that environment.

Sounds like your Windows system is similar to CRAN's.

I encountered the above issue too. To install the package, I ran the following first:
options(devtools.install.args = "--no-multiarch")
Reference: https://stackoverflow.com/questions/41186353/install-github-with-no-multiarch-argument


fix(template): update template.pug to allow popups in the html version

Checklist
Same issue with the HTML preview as the text preview. Fixes https://github.com/forwardemail/test-preview-emails-cross-browsers-ios-simulator-nodejs-javascript/issues/36

[x] I have ensured my pull request is not behind the main or master branch of the original repository.
[x] I have rebased all commits where necessary so that reviewing this pull request can be done without having to merge it first.
[x] I have written a commit message that passes commitlint linting.
[x] I have ensured that my code changes pass linting tests.
[x] I have ensured that my code changes pass unit tests.
[x] I have described my pull request and the reasons for code changes along with context if necessary.

cc @titanism, this is ready!


Support clipboard

Support copy image to clipboard
#19

@matsuyoshi30 I supported clipboard :)

@skanehira Thank you for your great contribution and nice clipboard package!


Create Your SAP Fiori Launchpad Project and its Module

Tutorial URL: https://developers.sap.com/tutorials/cp-workflow-2-create-module-cf.html
Please specify the step you are referring to: Step 4.
The link I generate in the deploy process does not work (even if I add /cp.portal as instructed in the tutorial). It says - Internal server error. In step 3 I could easily build and deploy, but the link aint working. Can you help?

@AstridHerbst will you answer on this issue in the upcoming days? I cannot move forward on my tutorials, if I have the server error. Do you have an advice how I can solve this as soon as possible?

Just an update - it seems that I had replaced one too many uaa_workflowtiles (resource) entries with a my-workflow-instance (resource) resource entry, so the "Internal Server Error"was because there were no uua resources left in the Tutorial 2, Step 2. Issue is solved


[Python] Chained projections

Describe the enhancement requested
Hi Arrow Team,
I'd like to be able to chain projections - by which I mean something like this:
With a dataset that has schema food: string, cost: int64:
dataset = dataset.project({ "food": ds.field("food"), "costPlusOne": ds.field("cost") + 1 })
  .project({ "food": ds.field("food"), "costPlusTwo": ds.field("costPlusOne") + 1 })

Note the chaining of the cost properties.  I don't believe this is possible atm from Python(?)
Thanks
Component(s)
Python

A more representative example would look like this:
dataset = dataset.project({ "food": ds.field("food"), "costPlusOne": ds.field("cost") + 1 })
  .project({
      "food": ds.field("food"),
       "costPlusTwo": ds.field("costPlusOne") + 1,
       "costPlusThree": ds.field("costPlusOne") + 2,
       "costPlusFour": ds.field("costPlusOne") + 3
})

It seems convenient to be able to reference the intermediate column (costPlusOne) to build up the expressions.  OTOH we could handle the expressions outside of the dataset and do a single projection (i.e. columns in .scanner and .to_table:
dataset.to_table(columns={
       "food": ds.field("food"),
       "costPlusTwo": ds.field("cost") + 1 + 1,
       "costPlusThree": ds.field("cost") + 1 + 2,
       "costPlusFour": ds.field("cost") + 1 + 3
})

I don't know details of the execution / query planning side, but would have anticipated the repeated work is easier to optimise out in the former.

There is very little (by design) query planning so you are correct that the former would be more efficient.  I think this is a reasonably addition to pyarrow.  The execution internals (Acero) should be able to handle this currently (this would just be two project nodes).


Add related projects

I propose to add a section with related projects in README.md.
I'm developing an open source reverse-engineered unofficial Dante implementation - Inferno. Currently it emulates Dante receiver and can be used for recording audio.
The earliest public attempt of reverse engineering Dante is wycliffe. I haven't tried running it, but looks like it implements requesting subscriptions from Dante devices. It probably won't be visible in Dante Controller.
dante-aes67-relay.js - this small Node.js program relays a Dante multicast stream to AES67
I couldn't find any other open source projects related to Dante, but many Dante devices are AES67-compatible, so this list of AES67 audio resources could be linked to as well.

Hi @teodly
Not sure, if Chris is ever going to answer, I think he only merges pull requests.
I started to compile an awesome-inspired list of active AoIP projects here,
maybe that's a better place to add a link to your software.


ENH: allow to check for specific WKT flavor in the is_wkt() function

There is a is_wkt function that allows you to check if a given string is a WKT string:
https://github.com/pyproj4/pyproj/blob/2d3cc7e0b83f59df2c0f5d3c2a8446ce0f224700/pyproj/_crs.pyx#L32-L48
The proj method it is using for that (proj_context_guess_wkt_dialect) should also allow to check for an exact flavor. So there could be an optional version keyword, to allow something like is_wkt(..., version="WKT2_2019")?

Yes, that is possible to check for a specific version: https://proj.org/development/reference/datatypes.html#c.PJ_GUESSED_WKT_DIALECT
Another alternative could be to add a new function that returns the WKT version.

Thoughts on adding a new method called something like guess_wkt_version that returns the WktVersion or None if it is not a WKT?


Manage required fields through restrictions

Can use a method check_validity(self: T) InvalidErrors!void { ... } to ensure the message adheres to restrictions and options before encoding and after decoding.
Can be a first step for required fields on proto2

We're not gonna bother with proto, job has been do to handle required fields similarly to other implementations.


Swiper is not working after change image in slide

When I change image data inside swiper-slide div, Swiper breaks and not working properly. Unable to swipe with buttons after that. Is there any way to re init the swiper after changing anything?
If yes then how I can re init the swiper in component type, Not in Directive type.

I don't know internals of the Swiper, but usually you need to call update from the Swiper after something has changed.

How can we call an update to swiper??

@ViewChild(SwiperDirective) directiveRef: SwiperDirective;
ngAfterViewInit() {
if (!!this.directiveRef) {
this.directiveRef.upadte();
}

I want to update the image in swiper. How it will work with ngOnInit()? And this will work only first time. I want to update swiper on every update of image in swiper.

Seriously you have closed without any solution. Great developers. Hatsoff

This is not issue related to this library so yes I have closed it after no activity for a while. Its fully normal process to close issues that do not have any activity. If you are looking for usage support there are proper forums for that, GitHub issues is reserved for bugs only.

Yes this is not the repository for Swiper so yes I do not want to fix Swiper issues :) I am not the maintainer of Swiper. This is just a Angular wrapper for Swiper that handles initializing of the Swiper nothing else.
For Swiper issues you need to report them to the Swiper repository although do not expect any action from there since most of the Swiper issues reported here are due to the dynamic nature of Angular and Swiper has never been designed to work with Angular type of libraries so many issues you need to find a workaround.


"Surani" is listed in the drop-down menu

The commune is  listed in the drop-down menu alongside its corresponding county, according to the law
Preconditions : the website (https://peviitor-ro.github.io/ui.orase/) is up and runing. After every step delete the text you typed from the search bar.

Step 1
Write in the search bar "Surani"
Expected results
The location appears in the drop-down menu as "Comuna Surani, PRAHOVA"


Implement std::error::Error for QueryParserError

This adds impls for std::fmt::Display and std::error::Error so that
QueryParser::parse_query can be used with the failure crate more easily

@pwoolcoc Is it something you need?

no, I suppose not. It saves me a small snippet of code application-side, but that's no big deal, and I can remove it easily when 0.8 is available

@pwoolcoc  If it makes your life a little bit easier feel free to reopen that.

That's ok, I'd rather help get this moved to failure


Revert "Ignore failing test in Gradleception build"

This reverts commit 66947bca23c1f0390addbee9e2a08f86c4907335.

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test and merge

I've triggered a build for you.

Pre-tested commit build failed.

I've triggered a build for you.


Implement modifyAckDeadline methods, add javadoc and tests

System tests against emulator will come as soon as also pull methods are implemented.
/cc @jgeewax @lesv

Coverage increased (+0.03%) to 84.292% when pulling 81158100ea0d10416fa6c739198be85ed18d18ab on mziccard:pubsub-modify-ack-deadline into fe4c90cba1be85c6885676eb1fa0214318448845 on GoogleCloudPlatform:pubsub-alpha.

LGTM :+1:


get_tvl_in_pool crashes with "Provided tick is out of bounds" error

I am trying to use get_tvl_in_pool to get the best option between 0.3% and 1% pools on UniV3
When I use it for WETH/USDC pair 0.3% pair it gets reverted with "Provided tick is out of bounds: (-887220, 887220)" error.
My code:
from web3 import Web3
from uniswap import Uniswap
from config.config import get_provider

address = None                    # or None if you're not going to make transactions
private_key = None                # or None if you're not going to make transactions
version = 3                       # specify which version of Uniswap to use
provider = get_provider('ethereum')['rpc']    # can also be set through the environment variable `PROVIDER`

uniswap = Uniswap(address=address, private_key=private_key, provider=provider, version=version)

token_address_usdc = '0xA0b86991c6218b36c1d19D4a2e9Eb0cE3606eB48'
# token_address_ape = '0x4d224452801ACEd8B2F0aebE155379bb5D594381'

weth_address = uniswap.get_weth_address()

token_address = Web3.toChecksumAddress(token_address_usdc)

pool_03 = uniswap.get_pool_instance(weth_address, token_address, 3000)
tvl = uniswap.get_tvl_in_pool(pool_03)
print(tvl)

Error
Traceback (most recent call last):
  File "/home/iurii/Desktop/blockchain/retrodrop/arbitrum/test.py", line 20, in <module>
    tvl = uniswap.get_tvl_in_pool(pool_03)
  File "/home/iurii/Desktop/blockchain/retrodrop/arbitrum/venv/lib/python3.8/site-packages/uniswap/uniswap.py", line 1342, in get_tvl_in_pool
    _min_tick = self.find_tick_from_bitmap(
  File "/home/iurii/Desktop/blockchain/retrodrop/arbitrum/venv/lib/python3.8/site-packages/uniswap/uniswap.py", line 1298, in find_tick_from_bitmap
    _min_tick = self.get_min_tick_from_wordpos(
  File "/home/iurii/Desktop/blockchain/retrodrop/arbitrum/venv/lib/python3.8/site-packages/uniswap/uniswap.py", line 1255, in get_min_tick_from_wordpos
    min_tick_in_word = nearest_tick(_tick, fee)
  File "/home/iurii/Desktop/blockchain/retrodrop/arbitrum/venv/lib/python3.8/site-packages/uniswap/util.py", line 111, in nearest_tick
    assert (
AssertionError: Provided tick is out of bounds: (-887220, 887220)

Process finished with exit code 1

However, for the WETH/APE pool it works just fine.
So this is probably some corner case that makes script fail. Not really sure how to fix it

This assertion error is being triggered because, when finding the nearest viable tick to the one derived from the bitmap, an "out of bounds" tick is being returned.
May be some weird edge case - my solution to calculating tvl on-chain at the time was a bit hacky and had to do with reverse engineering how the tick-bitmap was built for each pool.
Using the subgraph  for now is probably your best proxy for TVL data.

Just stumbled into the same issue when trying to use mint_liquidity.
AssertionError: Provided tick is out of bounds: (-887200, 887200)


chore(ci) fix release build and artifacts upload

#508 has fixes for the spurious [warn] issues about the socket deletion.

@casimiro branch is looking good, nice to see all the release jobs in green! :green_heart: I took the liberty to push an extra tmp commit to make it list the downloaded assets, just so we can see that the addition from d74a11cab32f8cc08b18dc4f4aa228158b0e85c0 combined all assets as expected. Once we have that last bit of confirmation, I think this is good to go!

Merged in #510.


[Feature Request] properties-maven-plugin

Checklist

[X] I searched exising issues and no one else requests similar feature.
[X] I think that 25%+ users are positive towards this feature.

Describe the feature
https://stackoverflow.com/questions/50861122/system-property-is-always-null-in-maven-project
https://mvnrepository.com/artifact/org.codehaus.mojo/properties-maven-plugin/1.2.1
Additional context
No response

Hello @unknowIfGuestInDream. This issue is marked as deprecated and closed.


Flow: Use the correct expression to calculate the option padding.

The old one could lead to unaligned memory access.
Signed-off-by: Guilherme Iscaro<EMAIL_ADDRESS>

+1

merged


Linux and macOS: put logs to user's AppData folder [INSTALLER]

Download:
Environment: QT=5.12, ARCH=32, CONFIGURATION=release, QTIFW=C:\Qt\Tools\QtInstallerFramework\3.1
trik-studio-master-i686-installer.exe


refactor: tidy up compiler

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[ ] Code style update
[x] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[ ] It's submitted to the main branch for v2.x (or to a previous version branch)
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[ ] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
[ ] New/updated tests are included

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

In general, I'm not a big fan of small refactorings like this when there are no obvious readability issues with the original code, which would make the history of the real logic changes harder to track.

I think this will make the logical distinction more obvious. If it is not satisfied, it will return directly, and it will reduce the situation where the nesting level of many conditions is deep and the code readability is poor.

I'm not saying that early return is not a useful pattern, and I use it myself. Rather, for code that has been around for a long time and without obvious readability issues, the benefits may not outweigh the costs. The costs include, but are not limited to, the cost of review by maintainers, the additional complexity of tracking changes to the code.

Response to previous efforts:
https://github.com/vuejs/vue/pull/5085#issuecomment-284050268
https://github.com/vuejs/vue/pull/12559#issuecomment-1158471380
git blame readability before v.s. after this PR:
https://github.com/vuejs/vue/blame/9dd006b481b4299462e044741bac0861c0b1775c/src/compiler/codegen/index.ts#L513-L545
https://github.com/vuejs/vue/blame/bb38547b3d492b658a93fd8fe39e9e57d5c372f1/src/compiler/codegen/index.ts#L513-L546


[Segment] Basic attached segments

Hi,
Is there any way to use basic attached segments? The margin between 2 basic segments, if detached, is too high.
Here's a fiddle that demonstrates the differences.

Are you suggesting that basic segment should not have margin? They are currently designed to have the same positioning as segment but without styling.

Yes, I'm suggesting that bordering basic attached segments don't have a double margin (just like normal attached segments). What do you think?

basic attached and attached do have the same margin
http://jsfiddle.net/enhr600j/

Sorry for misunderstanding here. I assumed that basic segments don't have borders, since example on the site shows none. Then my question would be: is there any way to make basic attached segment borderless? (maybe change the title of this post as well?)
Consider the following fiddle. If I use detached basic segments, then the gap between the button and the list is way too wide (debatable on a narrow screen). The wider you make your screen the more this gap seems to grow.


gRPC 4.2.0

Although this is a minor upgrade in a patch release. We do it because gRPC 4.2.0 is built with 4.2.0 and contains the gRPC transitive dependency upgrade which fixes the bug with the itty upgrade

Is "itty" a typo? 🤔


Added tests for model and logic

#131 #150

Pull Request Test Coverage Report for Build 514

0 of 0   changed or added relevant lines in 0 files are covered.
68 unchanged lines in 10 files lost coverage.
Overall coverage increased (+5.9%) to 60.427%

Files with Coverage Reduction
New Missed Lines
%

file:/home/travis/build/AY1920S1-CS2103T-T10-4/main/src/main/java/seedu/address/model/Model.java
1
0.0%

file:/home/travis/build/AY1920S1-CS2103T-T10-4/main/src/main/java/seedu/address/model/person/PersonList.java
1
98.75%

file:/home/travis/build/AY1920S1-CS2103T-T10-4/main/src/main/java/seedu/address/model/group/Group.java
2
82.61%

file:/home/travis/build/AY1920S1-CS2103T-T10-4/main/src/main/java/seedu/address/model/person/Person.java
3
86.6%

file:/home/travis/build/AY1920S1-CS2103T-T10-4/main/src/main/java/seedu/address/logic/commands/AddNusModCommand.java
4
88.64%

file:/home/travis/build/AY1920S1-CS2103T-T10-4/main/src/main/java/seedu/address/logic/commands/AddNusModsCommand.java
5
92.31%

file:/home/travis/build/AY1920S1-CS2103T-T10-4/main/src/main/java/seedu/address/model/person/PersonDescriptor.java
5
79.37%

file:/home/travis/build/AY1920S1-CS2103T-T10-4/main/src/main/java/seedu/address/logic/parser/ParserUtil.java
10
90.98%

file:/home/travis/build/AY1920S1-CS2103T-T10-4/main/src/main/java/seedu/address/model/ModelManager.java
17
86.13%

file:/home/travis/build/AY1920S1-CS2103T-T10-4/main/src/main/java/seedu/address/model/util/SamplePersonList.java
20
0.0%

Totals

Change from base Build 507:
5.9%

Covered Lines:
4242

Relevant Lines:
7020

💛  - Coveralls


Tooltips are currently green and do not fit the theme colors

Currently tooltips such as those on the buttons in the top right are green which is not a current theme color.

As per theme. @Madeline-UX will need to advise.

Changed to primary color


Update _TZE204_vqaajp1a.yaml

add capability to smart-value

Hi Wisly,
Could you please approve my PR to support boiler (smart-value) capability

These commands mentioned doesn't exist.
Attributes "range" and "pitch" also doesn't exist.
Attribute "unit" doesn't work that way.
Added datapoints might exist, but they are not mapped to valid/existing/implemented commands.
Valid commands are implemented here: https://github.com/w35l3y/EdgeDrivers/blob/beta/personal-tuya-devices/src/commands.lua#L144-L951
When I say "doesn't exist", I meant they currently are not implemented in the commands.
Also, new devices referencing incompatible profiles won't display new capabilities in the app.
Check whether there are other valid/existing/implemented commands that best fit your needs, for example, the command value.
But again, updating a device with new capabilities and referencing the same profile won't take any visual effect.
So I can't merge it. Sorry.


convert_hf_mpt_to_ft is hanging while downloading shards

Hi Team,
I am converting mosaicml/mpt-30b HF model to TF using convert_hf_mpt_to_ft.py. The progress is always hanging while downloading shards. Once all the shards of HF model are downloaded, the progress just hang there for 10-15 mins. I am not sure whether it is a hang or should I wait for some more time?
Steps performed:

Launch the Tensorrt-LLM container
make -C docker release_run LOCAL_USER=1

python convert_hf_mpt_to_ft.py -i mosaicml/mpt-30b -o ./ft_ckpts/mpt-7b/fp16/ --tensor_parallelism 4 -t float16

I have tried this on launchpad as well as on EC2 g5.12xlarge machine.
Also, as I can see for llama model we can directly pass HF model name to build.py however for mpt models I believe conversion from HF to FT model is mandatory. Is our understanding correct?
Thanks!

Update:
It was not a "hang". It made progress after around 25-30 mins.

Update:
It was not a "hang". It made progress after around 25-30 mins.

Thanks @tapansstardog for the update. I’m closing as Not-a-Bug. Reopen it if there is a real issue, please.

@jdemouth-nvidia Can you confirm the second part:

Also, as I can see for llama model we can directly pass HF model name to build.py however for mpt models I believe conversion from HF to FT model is mandatory. Is our understanding correct?

@jdemouth-nvidia Can you confirm the second part:

Also, as I can see for llama model we can directly pass HF model name to build.py however for mpt models I believe conversion from HF to FT model is mandatory. Is our understanding correct?

The conversion is mandatory now. This process can be merged into build.py, but we split them because it is more convenient to do calibration on the weight for lower bit inference.


bugfix/current_formula_fields

Are you a current Fivetran customer?

Fivetran created PR
What change(s) does this PR introduce?

This change fixes a bug that was introduced within the 0.6.2 release of this package. The change adjusted the conditional within the sfdc_formula_views macro to reference the properly named current_formula_fields variable opposed to the incorrect old_formula_fields variable that is not longer referenced.
Did you update the CHANGELOG?

[X] Yes

Does this PR introduce a breaking change?

[ ] Yes (please provide breaking change details below.)
[X] No  (please provide an explanation as to how the change is non-breaking below.)

This is a bugfix that will not result in a breaking change.
Did you update the dbt_project.yml files with the version upgrade (please leverage standard semantic versioning)? (In both your main project and integration_tests)

[X] Yes

Is this PR in response to a previously created Bug or Feature Request

[ ] Yes, Issue/Feature [link bug/feature number here]
[X] No

How did you test the PR changes?

[X] CircleCi
[X] Local (please provide additional testing details below)

I also tested this on our local data set and was able to reproduce the issue where the macro did not bring any formulas through when the inclusion version of the macro was not used. After applying this update I was able to see that be resolved.
Select which warehouse(s) were used to test the PR

[X] BigQuery
[X] Redshift
[X] Snowflake
[X] Postgres
[ ] Databricks
[ ] Other (provide details below)

Provide an emoji that best describes your current mood

👔
Feedback
We are so excited you decided to contribute to the Fivetran community dbt package! We continue to work to improve the packages and would greatly appreciate your feedback on our existing dbt packages or what you'd like to see next.

@fivetran-sheringuyen you can reproduce the original error by running the 0.6.2 version of the package on a model that uses the macro without the inclusion method, like below:
{{ salesforce_formula_utils.sfdc_formula_view('test_order_c') }}

You will then see no formulas are brought through in the compiled output. Switching to this version of the package will then bring those formulas through. Let me know if you have any questions.

Good catch Joe, LGTM!


Docs need resolution specific styles

Description:
The APIM 3.0.0, 3.1.0, 3.2.0 needs to have resolution specific styles so that when the docs have the yellow banner at the top, the docs render properly in lower resolutions.

This issue is not present when the banner is not present.

@chanaka3d , please work on this task as discussed offline today.
Thanks,
Mariangela


[Question] About parallel requests

I'm curious to know if frankenphp can effectively manage multiple incoming requests simultaneously. Does it spawn processes similar to fpm?
<?php

echo "here\n";

sleep(2);

if five users request this script simultaneously, does it take 2 seconds or 10 seconds?

For each request, there'll be a 2-second wait... the real answer is: "it depends."
For example, if you have 2 cores in cgi-mode: the first two requests will take 2 seconds, with the remaining 3 requests will wait for the first two requests to finish in a queue. So these first two requests will wait two seconds, the next two, four seconds, and the final one six seconds. You can override this with the num_threads setting in your Caddyfile:
frankenphp {
		num_threads 48
	}

In worker mode, you can override the number of workers, for example, for 48 workers:
frankenphp {
		worker testdata/index.php 48
	}

This means the first 48 requests would only have to wait 2 seconds. Therefore all 5 requests would be handled simultaneously.
Does that help?

Thanks
I hope it will have similar behavior to fpm, which spawns processes and does not depend on my cores but rather on memory.

Surely there's some limit to the number of workers though, no?

Yes, hardware/OS limits. If your hardware can handle 100k threads, you can configure 100k threads.

You'd have to set the workers relative to the available computing resources as well as how compute-intensive the application is - finding some equilibrium that maximizes throughout. Any tips for that?

It largely depends on whether you're memory and/or CPU-constrained (on the hardware level), your application and its memory/CPU usage.
It takes some art to get it to the best like tuning fpm. However, if you already have fpm configured, it will likely be about the same (if not less).

I hope it will have similar behavior to fpm, which spawns processes and does not depend on my cores but rather on memory.

Yes, it spawns processes like fpm (fpm doesn't care about memory though, or how many cores you have). However, the default number of maximum processes is very different. IIRC, FPM defaults are some incredibly low numbers, like 10 workers, but Frankenphp's default is something like 2x the number of cores on your machine. So, it may require tuning, just like any other server software, to find the right balance for your application.

I can't test it out because it doesn't have support for Linux/arm64.

@dunglas, are we just missing a build here, or does it not support arm64? I think we are just missing a build in the build matrix.


Record user and datetime for all votes

Currently when a suggestion is approved or rejected, the only thing recorded is the new state.  It would be very valuable to be able to track votes by user when doing analysis of how reliable voters are.  Date and time allows correlation with the actual edit made as well as time analysis of votes.

We are already storing these data in the database in order to produce some statistics displayed at https://tools.wmflabs.org/wikidata-primary-sources/status.html and https://tools.wmflabs.org/wikidata-primary-sources/dashboard.html
We should maybe provide an API for them.

Indeed. The data is all there - I was just waiting for good requirements on what kind of data people would be interested in to surface. Any suggestions?

Oh, cool.  The typical RESTful/JSONy thing to do is marshall up whatever data you have that you're not filtering for security or other reasons.
Specifically, I think that statements where state!=unapproved should be returned in the API with the user and datetime the statement was processed.

Ok, let's see if I can easily add this tomorrow. I was originally only thinking of what kind of statistics you'd like to generate, but adding this information to the JSON serialization makes sense.
There's no real privacy issue here, we only store user name, statement id, timestamp and new state in the activity log.

When you retrieving statements that have been updated, you now have a list of activities (usually only one, but can be more) wrt this statement. Try it with:
curl -i "https://tools.wmflabs.org/wikidata-primary-sources/statements/any?state=wrong"
To get 10 random wrong statements. ;-)


TO API: urlkeys endpoint returning a HASH

I was trying to automate url key extractions from TO, although looks like this API doesn't return the content I was expecting. This might not be a documented API either.
https://to.kabletown.net/api/1.2/deliveryservices/xmlId/test_ds/urlkeys.json
{"response":"HTTP::Response=HASH(0x7ecc440)"}

https://issues.apache.org/jira/browse/TC-323


disappear today mark

Today mark is at wrong date: 3 days ago.
Then view next month and return to this month, there is not a today mark.
This machine is in sleep a few times in day.
OS: Windows 10 64bit
note: NetBeans debugger cannot attach via process id.

After the PC awake from sleep at morning, the today mark is on yesterday.

ScheduledExecutor uses System.nanoTime() to calculate a schedule delay time.
In Windows OS, nanoTime() is a wrapper of QueryPerformanceCounter Win32 API.
The counter values from QueryPerformanceCounter are discontinuity when OS is suspend and resume.
To fix this problem,

Detect OS Resume from suspend, and re-schedule.
Detect UI event and re-schedule. (work around)

JDK Bug report "Scheduled execution ignores PC suspend mode"
https://bugs.openjdk.java.net/browse/JDK-8146527
Discussing about ScheduledExecutorSerivce's schedule delays as long as OS suspend time.


VS2022 run EffekseerEditor.csproj error

after build, I use VS2022 run the Effeckseer.sln  int the build folder, however, when run the EffekseerEditor.csproj, i face a error:

libpng error: Not a PNG file
Fatal error. System.AccessViolationException: Attempted to read or write protected memory. This is often an indication that other memory is corrupt.
Repeat 2 times:
at Effekseer.swig.EffekseerNativePINVOKE.GUIManager_AddFontFromFileTTF(System.Runtime.InteropServices.HandleRef, System.String, System.String, System.String, Single)
at Effekseer.swig.GUIManager.AddFontFromFileTTF(System.String, System.String, System.String, Single)
at Effekseer.GUI.Manager.Update()
at Effekseer.Application.Run()
at Effekseer.Program.Exec(Boolean, System.String, System.String, System.String, System.String, Single, Boolean)
at Effekseer.Program.Main(System.String[])
i dont know why and how to solve, i will apprecaite who can help me, thank you

Did you clone with github lfs?

no, i clone the zip, and i download other files  by myself, i put them into  TestData, ResourceData, Downloads, Dev/Cpp/3rdParty folders. Is there any problem with this？

i solve the problem, because the png ttf lib files are downloaded failed，they are 0KB, thank you.


add re-rxjs

Hi @dai-shi
Thanks a ton for this project!
I've been working on these react-rxjs bindings. This thing started as an experiment for my side-projects and we ended up using it in my company. After some iterations I think that now they are mature enough to be published on their own library, document them properly, etc. I still have a long way to go, though.
I've recently made some API changes to fully support React concurrent's mode. After I made the changes I thought that I would give a shot to these tests and I'm pretty happy with the results :slightly_smiling_face:. Could you please add re-rxjs into the main list? I'm currently working on the docs and I would like to be able to include a link to these results.
Thanks

Hi, thanks for your support.
Looking at the code at https://github.com/re-rxjs/re-rxjs/blob/master/src/useObservable.ts,
I don't understand why it passes all the tests.
Unless we use useMutableSource or use only one React state, that wouldn't happen. Either, 1) I misunderstand your code, 2) your spec is wrong, or 3) the tool is wrong.
If you have any explanation how to support CM, I'm happy to hear it.

Looking at the code at https://github.com/re-rxjs/re-rxjs/blob/master/src/useObservable.ts,
I don't understand why it passes all the tests.
Unless we use useMutableSource or use only one React state, that wouldn't happen.

I don't agree with this statement. useMutableSource does make sure that a generic mutable source behaves correctly with react, but why can't I leverage the way that RxJS behaves to accomplish the same thing? That's in a nutshell what this library is about and the reason why I think it's so cool :slightly_smiling_face:

Either, 1) I misunderstand your code, 2) your spec is wrong, or 3) the tool is wrong.

I think that you misunderstood my code. I still have to cleanup the code and document a lot of things, and I'm planing on writing an advanced blog-post explaining the magic behind it.

If you have any explanation how to support CM, I'm happy to hear it.
Sorry, but I didn't understand this bit... What do you mean by "how to support CM"?

That's in a nutshell what this library is about and the reason why I think it's so cool 🙂

Yeah, but my point is my tool does not 100% behave how I expect. So, this is a good chance to improve the tool itself.
I did try to reproduce what I tested by hand with jest spec, but it's tricky and sometimes not stable.

Both useSubscription and useMutableSource are custom hooks built on top of the existing primitive hooks

Yes for useSubscription and no for uMS, AFAIU.
Most of my libs are based on useSubscription technique now. (not directly using it)
If you find another way, I'm sooooooo much interested in it.
Look forward to hearing more from you.

"proper branching with transition"

I'm not sure if you could understand what I'd like to test from the jest spec.
Let me try to explain it.
We have "normal button", "transition button", and "count".
A: At first, count=0
B: if we click transition button twice, count will be one,
C: click the button twice again within the transition, count will become two after one.
D: but while it's in transition, we click normal button twice, this will not wait the transition, so it will increment the counter, before C is effective, this is the whole point of the test.
In the result we will see 0->1->2 while pending, and finally C is done and see 3 and pending finished.

@josepot Just run manual tests with your branch. This is truly amazing.
I still don't understand how this is happening underneath.
https://github.com/re-rxjs/re-rxjs/blob/704245ade5c64de9d0155c57934a6cd95faa11ea/src/useObservable.ts#L59-L62
Does this the trick? Really look forward to your explanation.
Can you explain why this technique can't be applied to react-redux? or can it?
Modified your code to be inline with others. Here's the diff:
diff --git a/src/re-rxjs/index.js b/src/re-rxjs/index.js
index c0c3372..970e9fb 100644
--- a/src/re-rxjs/index.js
+++ b/src/re-rxjs/index.js
@@ -1,12 +1,15 @@
 import React, { unstable_useTransition as useTransition } from 'react';
-import { Subject, merge, asapScheduler } from 'rxjs';
+import { Subject, asapScheduler } from 'rxjs';
 import {
   map, scan, startWith, observeOn,
 } from 'rxjs/operators';
 import { connectObservable } from 're-rxjs';
+
 import {
   syncBlock,
   useRegisterIncrementDispatcher,
+  initialState,
+  reducer,
   ids,
   useCheckTearing,
 } from '../common';
@@ -14,16 +17,12 @@ import {
 const normalClicks$ = new Subject();
 const normalIncrement = () => normalClicks$.next();

-const externalClicks$ = new Subject();
-// This is only needed to prevent the tearing with auto increment (check #9)
-const externalClicksAsap$ = externalClicks$.pipe(observeOn(asapScheduler));
-const externalIncrement = () => externalClicks$.next();
-
 const [useCounter] = connectObservable(
-  merge(normalClicks$, externalClicksAsap$).pipe(
-    scan((x) => x + 0.5, 0),
-    map(Math.floor),
-    startWith(0),
+  normalClicks$.pipe(
+    observeOn(asapScheduler),
+    startWith(initialState),
+    scan((x) => reducer(x, { type: 'increment' })),
+    map((x) => x.count),
   ),
 );

@@ -36,7 +35,7 @@ const Counter = React.memo(() => {
 const Main = () => {
   const count = useCounter();
   useCheckTearing();
-  useRegisterIncrementDispatcher(externalIncrement);
+  useRegisterIncrementDispatcher(normalIncrement);
   const [localCount, localIncrement] = React.useReducer((c) => c + 1, 0);
   const [startTransition, isPending] = useTransition();
   const transitionIncrement = () => {
@@ -50,7 +49,7 @@ const Main = () => {
       <button
         type="button"
         id="normalIncrement"
-        onClick={isPending ? transitionIncrement : normalIncrement}
+        onClick={normalIncrement}
       >
         Increment shared count normally (two clicks to increment one)
       </button>

Could you review it?
It may be unsatisfying that check 7 doesn't pass. But, the purpose of the check is to detect state branching and state branching is technically impossible with external state. (I didn't realize isPending ? :  bypasses it. I wonder if there's a way to detect it.)
I still don't understand the asapScheduler and how your lib solves the tearing, however, I'm happy to merge with the diff above even without me understanding your lib at the moment.
BTW, is there a better way to dispatch { type: 'increment' } in the event source, instead of  directly feeding it into scan? Please note that I will be modifying reducer in src/common in the future, and I'd like to avoid depending on the implementation detail. (Would be good to fix mobx one.)

Hi @dai-shi !
First of, thanks a lot for your interest and for helping me at adding Re-RxJS into the list.
I promise you that I'm dying to explain in detail all what's happening behind the scenes. However, right now I'm smashed with work and I still have to:

Add a ton of tests to Re-RxJS plus improving the present ones
Finish the first set of documentation for the library
Add a good set of examples and use-cases
Finish writing a presentation blog-post for the library

I'm very sorry, but as I'm sure that you will understand I'm beyond busy with all those chores right now, but I promise that as soon as I have a chance I will explain everything in detail.
All that being said, I wanted to challenge one of your comments... Perhaps I misunderstood what you are trying to say, but I think that this statement is not accurate:

It may be unsatisfying that check 7 doesn't pass. But, the purpose of the check is to detect state branching and state branching is technically impossible with external state. (I didn't realize isPending ? : bypasses it. I wonder if there's a way to detect it.)

As you can see in this example Re-RxJS does handle branching perfectly well during transitions.
I will try to take some time in the next few days to explain why IMO the test "proper branching with transition" has some room for improvement... I could be wrong, but nevertheless I think that it's a conversation worth having.
Again, I'm sorry that I can't yet get back to you... Although, if you want to contribute to Re-RxJS, I would be super happy to get some help :sweat_smile:

I've also tried manual testing to check out how does re-rxjs manage to avoid tearing (test num. 2), and adding some logs to see the sequence I think it's due to re-rxjs using react's state to get and set the value.
In the case of react-state, the sequence of actions I see in the logs are:

State gets incremented to 1
It starts rendering each of the Count, all of these components receive value = 1
State gets incremented to 2
The remaining Count components continue rendering, receiving value = 1
All Count components re-render, receiving value = 2

In libraries with tearing issues what happens is that in step 4, the remaining Count components receive value = 2 - which then it's the inconsistent state rendering known as tearing. That happens because that library is getting that value from an external source, and react can't know whether that value is from the current render loop or the next one.
What I think that re-rxjs does to work around this is that every component holds a state value through useReducer, and when there's an update in the stream, all the components subscribed to that stream will update their state (and so it will trigger a re-render for that component).
This way, if the stream emits a value while react is delaying the rendering of some components, those components will use a react primitive to dispatch that the state has changed, and react will make that change effective in the next render loop.

@josepot Suppose you implement a transition with rxjs, it has to be in sync with React transition in your library code (I'm not sure even if it's possible) to get benefit from CM. If it's really implemented so, you don't need isPending ? : . You can assume it as a definition here of "proper branching".

@voliva

This way, if the stream emits a value while react is delaying the rendering of some components, those components will use a react primitive to dispatch that the state has changed, and react will make that change effective in the next render loop.

Yes, this is important. What I'm not sure is it could still tear after the dispatch. (But, it won't)
So, if this is a real thing, why can't we apply the same technique to react-redux?
(I'm pretty sure I'm missing something.)

Oh, I think I get it. It somehow synchronizes all renders.
Then my expectation (by the design definition of this tool) is check 3 should fail.

    re-rxjs [
      189, 183, 183, 186,
      193, 200, 171, 175,
      195, 193
    ]

(But, it's fast enough. Really amazing.

This is my current understanding, but I'm by no means an expert in React's implementation that much, so here's a foreword of caution.

What I'm not sure is it could still tear after the dispatch

Here's a section which is relevant to the internal implementation details of useReducer
https://overreacted.io/a-complete-guide-to-useeffect/#why-usereducer-is-the-cheat-mode-of-hooks
The most relevant bit is:

The answer is that when you dispatch, React just remembers the action — but it will call your reducer during the next render

In the React source, this is seen in
https://github.com/facebook/react/blob/30b47103d4354d9187dc0f1fb804855a5208ca9f/packages/react-reconciler/src/ReactFiberHooks.new.js#L788
where markWorkInProgressReceivedUpdate is only called inside of the reducer-related functions.
Why it doesn't tear
Tearing happens when uncommitted values are derived from during render.
https://github.com/facebook/react/issues/14099#issuecomment-440794435

In concurrent mode, last render doesn't necessarily mean "latest committed state". So a low-priority render with new props or state would overwrite a reference used by current event handler.

A reducer's deterministic nature of processing everything in order of dispatch, and thus leading to the same result if it were replayed in the same order, is what enables their evaluation to be deferred, rather than eagerly evaluated during render passes.
Writing ref.current = value means: assign this value right now, because I could be using it in the next line. On the other hand, dispatch(action) means: the resulting value will be updated at some point in the future through this payload, i.e. the laziness is a given for how reducers work; the reducing process through the action queue is what allows it to be batchable, pausable and resumable.
This leads to the assumption: the reducer's function and value only is propagated in the commit phase of the component, which would be why that closure reducer from Dan Abramov's post can safely capture the parent's props.
Why the example works
Because the subscription's pushes are being enqueued as actions, thus only propagated during the commit phase. Through useReducer, React will resolve their update at the proper time, in which, by that point, each component will be able to access their own latest copy of the immutable (I think?) state pushed from the Observable.

Those assumptions mostly rely on the "Reducer value only propagates during the commit phase" idea. If that's correct, then it'd also mean only useLayoutEffect is the only safe place to derive state from props and mutable references, because that's when React will have settled down with all the render passes which might lead to inconsistencies, such as tearing.

So, if this is a real thing, why can't we apply the same technique to react-redux?

I think a similar implementation can be applied to react-redux, but it'd mean having those immutable store copies living in a reducer for each component, rather than a storage outside of React. I think that'd also make the connect APIs not work anymore, as props are subject to tearing problems.

@resolritter Thanks for the note!
Oh, I forgot about the cheat mode, and I didn't realize when I saw useObservable code at a glance.
I think/thought I'm fairly familiar with the cheat mode, as I actually did it in react-redux,
one for stale props issue and another for concurrent mode.
https://github.com/reduxjs/react-redux/pull/1505
https://github.com/reduxjs/react-redux/pull/1509
Now, observing the behavior of re-rxjs, I'm not sure if I fully understand the cheat mode.
There might be a better implementation than what I did before. The implementation of re-rxjs looks so clean, and I'm sure I can learn something from it.
Anyway, my biggest concern about my misunderstanding is gone, even though I don't yet fully understand the implementation, I will merge this PR with some modifications I made here and in master.
We can continue the discussion in this PR even if it's closed, or one can open a new issue.

(wow, check 7 passes with my new check. Really impressive. I need to learn more about this.

(btw, I still wonder how check 9 passes here. It does pass for sure and the tool seems OK. My gut feeling is there's some limitation somewhere. Hope to learn more eventually.


Can add Volume with invalid name and can't delete them

Version - v1.0.1
Steps to Reproduce:

Add glusterfs and convoy glusterfs catalogs
Go to Add Volume and create a volume with a space in it
Create a container that uses that volume

Results:  Error (500 Server Error: Internal Server Error ("create abc def: Invalid volume abc def. Must be only contains 0-9, a-z, dash(-), underscore(_) and dot(.)")).  Since the name is bad, when I try to delete, it gets stuck in purging.
Expected: If there are limitation on volume name, we should catch that during the Add.

Add volume on the storage driver page

Version - v1.2.0-pre4-rc12
Verified fixed


FR: Possibility to Sync the File System's Modification Date with the Yaml Date

Is Your Feature Request Related to a Problem? Please Describe.
Regarding YAML and file system modification dates that become unsynced with each other due to moving/downloading the Vault to a new location.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Running "Lint all files" / "Lint folder" will update all files because file sys mod date is later than Yaml mod date, but the file's content hasn't actually been modified.
Describe the Solution You'd Like
Is it in theory possible to programatically set the file system's mod date from within an Obsidian plugin?
In that case it would be nice to have a function that one runs only once the file sys mod dates gets out of sync.
The function would go through all files in the vault and make sure that the file mod date is set to the current value in whatever Yaml key represents the modification date.
Describe Alternatives You've Considered
Is there any stand-alone app? But it would have to run on all platforms.
Additional Context
Having run the "sync mod dates" function would allow "lint all files" to just update the  Yaml mod dates on the files that were actually modified.

This relates to #386 . There is nothing that can be done by the Linter in regards to this. However you may be able to use Custom Commands to handle this.

But to be clear,the Linter cannot do this because Obsidian does not allow for this in their API.

There probably is a way to do so, but those functions are not added to an API or anything. It would take some work and testing on your part to get it working.

Would it be possible to have a "Lint all Files Modified Today" feature in Linter?

It would be possible so long as the date modified is consisered the source of truth. It would check if the date modidied was today and if so, then lint those files.

Does linting on file change (#799 ) and autosave (#392 ) not address this issue?
I am assuming the latter handles your scenario.

Looks like I linked to the wrong issue. I was referring to #183 for autosave. It would lint the file after a change is made to it (thus no need for running a lint on the current file manually).
As for linting a file when the active file changes, that makes sense. I was pretty sure it would not meet your needs, but thought I would mention it as well.

I was referring to #183 for autosave. It would lint the file after a change is made to it.

How do you detect changes made to a file?

I listen to the current active file's editor content (source mode displayed content). If the rditor says something changed, then I assume something changed.

I listen to the current active file's editor content (source mode displayed content). If the rditor says something changed, then I assume something changed.

Ok.
Autosave as in #183 doesn't work for me since I don't want the file to suddenly change.
What would work is if you manage a list (as the Recent Files plugin manages a list of recent files in a json) where you add each file as soon as a change occurs. This would be a list of files pending linting. Then make a command available that lints all files in the list and then clears it.
Then there's no need to iterate through all files as suggested above.
But this list of files pending linting might cause problems when multiple clones of the same vault exists, where files haven't been synced.

I can say that I would not add that data to a json file if I were to add it. So there would be no issue with syncing.
It sounds like you are looking for a feature that currently has not been requested, so I can put it in the backlog and see what interest current users have in this feature. I am open to PRs in the meantime, but I at this time I don't see myself working on this anytime soon.

Sounds good. I am glad that you found a solution.


[Compute] Support ppg and spg in VMSS orchestration mode

Description

Resolve https://github.com/Azure/azure-cli/issues/14364
Support proximity placement group and single placement group in VMSS orchestration mode
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

Compute

Compute


feat(examples): Added graphql-hooks example

Added example project with graphql-hooks and GraphCMS. In this project, Next.js and Typescript were also used.
Close #173

@notrab Can you approve my pull request?


Create metrics endpoint for prom-scraper consumption

Currently CCNG uses statsd-injector to gather Statsd style metrics and forward them to loggregator.
We should build a /metrics endpoint to supply those same metrics, so we can use prom-scraper to forward the metrics instead.
Doing this will allow us to remove the EventMachine/Thin code we have:
https://github.com/cloudfoundry/cloud_controller_ng/blob/4a23a419862ef9b7272f411a4435e33fc7305f30/lib/cloud_controller/metrics/periodic_updater.rb#L18-L20
and consequently address https://github.com/cloudfoundry/cloud_controller_ng/issues/2254 and move CCNG from Thin to Puma.

Currently we emit metrics named with periods, e.g. cc.job_queue_length.total.  However the Prometheus data model prohibits periods and most other non-alphanumeric characters in metric names, so to migrate to prom-scraper would mean having to rename almost all of our metrics.  So  cc.job_queue_length.total would become cc_job_queue_length_total or similar.
@stephanme @philippthun do you have any thoughts on how we would handle an upgrade here?
A few options:

Do a hard cut where we switch from period-named metrics to underscore metrics in one release, causing a breaking change where monitoring tools would have to be updated
Introduce the new non-period metric names in a release to be emitted alongside the period metric names, giving operators a deprecation window to update their monitoring tools
Make a configurable option that, when enabled, stops emitting the period metric names, and emits the new metric names

Would appreciate your input here.

Option 1 would work for us; we would need to adapt our monitoring tools beforehand to support both formats and then change them a second time to clean this up (i.e. support only the new format).
But I'm not sure if this works for others as well, thus I would prefer option 2. This also has the advantage that we only have to adapt our monitoring tools once without any disruption.
There might be even an option 4, that combines 2 and 3; i.e. a config parameter that specifies what to emit: either the old, or the new format, or both.
But given the fact that we are not talking about hundreds of metrics, that might be over-engineered; so simply having both formats for one release seems to be the best/easiest solution.

Sounds good.  Having one release that contains both I think makes the most sense.  We will move forward with that idea.


Add configurable timestamp precision to InfluxDB output

fix #17

Hi @karimra, I think the better approach would to set the precision option for the InfluxDB client using: https://github.com/influxdata/influxdb-client-go/blob/94dd2a00d884087ebdd5ec65692aeeaea1acca25/api/write/options.go#L93
In this way, influxdb client will use configured precision in the write calls to the DB:
http://<ip-address>:8086/api/v2/write?bucket=gnmic&org=&precision=ms

Otherwise, I think it will still use ns, just adjusting the timestamps to nanosec by filling zeros.

Changed. Thanks, I wasn't aware fo that option.


Slack chat?

I forgot to mention it during the ASWG meeting in Barcelona, but do you think it would be a good idea to set up a chat for rapid communication between users? Sometimes I want to have a quick chat with somebody but the only way is through email or opening an issue. I propose slack:
https://slack.com/
because it has a nice interface, it is successfully used in several of the collaborations I work for and if you do not have the app opened, you will get an e-mail with the message you got.

I never used slack, but I have a similar feeling that sometimes a quick chat might be just the thing we want. If currently unclear stuff is being asked in (private) emails, then I believe newbies like me could learn a lot by just following this chat.

Yes absolutely!
And I have been wondering the same about ASWG group.

I think we are in desperate need for something like this

if @kosack agrees, I can set it up and send the invitations around.

Maybe the invitation can go to the ASWG mailing list? Then anyone who is interested can join.

Sounds good to me - I have Slack installed, but have so far never had a use for it!

Please go for it... Hopefully, CTAO will provide something for everyone at some point.

Hi,

On 25 April 2018 at 11:27, Karl Kosack<EMAIL_ADDRESS>wrote:

Sounds good to me - I have Slack installed, but have so far never had a
use for it!

fine for me too, the only question is whether we should make this move on
our own (i.e. ctapipe/DPPS & ASWG)  - Karl, wouldn't this be redundant with
Confluence, in case it is adopted in CTA at large? What is the status of
that?
Best regards,
Abelardo
--
Abelardo Moralejo Olaizola
Institut de Física d'Altes Energies
Tel : +34 931641662
Fax: +34 935811938
Avís - Aviso - Legal Notice - (LOPD) - http://legal.ifae.es
--
Avís -
Aviso - Legal Notice - (LOPD) - http://legal.ifae.es
http://legal.ifae.es/

Slack is free but in order to save the chat, (browse them, if I am not wrong) one needs to pay a per person per year fee. I am not sure CTA would like to do that, but I would definitely be happy. It's useful for ordered chat

On a free account a maximum history of 10,000 messages is kept, which is browsable and searchable. Slack could be used for minor direct messaging and conversations, but anything important that needs a permanent record of should still be placed on github.

If CTAO doesn't want to pay for a full slack service, it could use an open source alternative like mattermost and have it's own servers. The use of a free slack site shall certainly be limited to casual chat and for nothing critical. It could still be worth to have a general site with different sub channels... As so many people seem to be interested

ok, so do we wait until you discuss it with CTAO @moralejo, @GernotMaier, @kosack?

As far as I know, CTAO is currently looking into purchasing the full Atlasssian suite, which includes something like this (Stride).  But I have no idea if there is a timescale for that, or even if it will be the accepted solution.  So I would say for now, having a Slack channel would be useful, and we can always migrate  to something official when we have it (the same goes for GitHub, TravisCI, etc)

As far as I know, CTAO is currently looking into purchasing the full Atlasssian suite, which includes something like this (Stride).

Mmh. Any chance to go for GitLab / GitLab Mattermost?

ok, so I created the ctapipe team in slack.
The access to slack is by invitation only, so I only sent the invitation to some people I have the e-mail of. Please drop me an e-mail<EMAIL_ADDRESS>if you want to receive the invitation. Any team member can send these invitations so you can contact other members in the near future and feel free to invite anybody interested on ctapipe.
If @kosack or anybody else wants to make more publicity of the slack chat (sending an e-mail to the ASWG list, etc...), please go ahead.

I guess we can close this since it's set up! We can perhaps update the developer docs to show this as well.


Revise color palette

Over time, new colors have been added and changed from time to time. Let's bring everything together and make our color palette consistent for light & dark mode.

If you have any questions or uncertainties, please contact Selina or Manu.

Please also keep accessibility in mind!
For example, right now the contrast on e.g. buttons is too low in both light and dark mode:

This site has helpful information and tools regarding color contrast and a11y: https://dequeuniversity.com/rules/axe/4.7/color-contrast

The colors were proved and for our use – white medium/bold letters and icons on buttons (Ui components) they will be fine.


drawTextInRect/adjustsFontSizeToFitWidth doesn't always use original string for re-calculating size

If drawTextInRect adjusts the font size based on the adjustsFontSizeToFitWidth property, it saves off an originalAttributedString and restores the value after it's finished drawing the label. When it restores the value it explicitly avoids resetting the framesetter - I don't know enough to know why it needs to keep around the framesetter, but my guess is it's for performance or maybe for correctly calculating further calls to things like instrinsicContentSize? The problem occurs on subsequent calls to drawTextInRect: sizeThatFits passes attributedText (the original value) into CTFramesetterSuggestFrameSizeForAttributedStringWithConstraints, but the sizing is done based on the reduced-size string used by the current framesetter.
This showed up as problem for me because I had an animation that momentarily rendered a smaller version of my label before rendering the full size version. The font got scaled down for the initial small version, but never got scaled back to its original size for the full size version.
The workaround I'm using now is just to explicitly reset the framesetter every time drawTextInRect wants to calculate sizing information:
// Adjust the font size to fit width, if necessarry
if (self.adjustsFontSizeToFitWidth && self.numberOfLines > 0) {
    [self resetFramesetter]; // Framesetter could still be working with a resized version of the text, need to reset so we start from the original font size.
    // Use infinite width to find the max width, which will be compared to availableWidth if needed.
    CGSize maxSize = (self.numberOfLines > 1) ? CGSizeMake(TTTFLOAT_MAX, TTTFLOAT_MAX) : CGSizeZero;

    CGFloat textWidth = [self sizeThatFits:maxSize].width;

resetFramesetter is:
- (void)resetFramesetter
{
    [self setNeedsFramesetter];
    [self setNeedsDisplay];

#if __IPHONE_OS_VERSION_MAX_ALLOWED >= 60000
    if ([self respondsToSelector:@selector(invalidateIntrinsicContentSize)]) {
        [self invalidateIntrinsicContentSize];
    }
#endif
}

Thanks for the report, @ChrisLoer. Your suggested fix is now merged in.


Add NetBSD install

Hi,
Saw this on r/rust yesterday and found it very useful. So, I've packaged it for NetBSD and just merged it into pkgsrc main branch.
Please consider adding the information to the projects README.md.
For your reference, here's the commit message, http://mail-index.netbsd.org/pkgsrc-changes/2023/05/09/msg274528.html
Thanks!

Thanks for this. I appreciate you taking the time. I'll merge this once 3.0.1 is released.

Sure, no rush.
I've added it to my RSS feed reader, so I'll get notified when you release 3.0.1

Thanks! I won't be able to update the package before tomorrow but, will do it as soon as I can.


Microsoft Ignite Exam Registration link gives 404

At https://www.microsoft.com/en-us/ignite, going to Attendee Tools > Register for an exam leads to https://docs.microsoft.com/en-us/learn/certifications/msignite, which gives a 404 error:

None of the 'search results' seem to lead to the correct site.

Thank you for letting us know @BradAF. I'm trying to see if I can find someone who can fix that.
Tagging @sadadow here.

Wondering about this too, hoping registrations aren't closed. Is there an alternate way to register in the meantime?

@mairaw The link now goes to the generic exam page without any info on how to register for an exam at Ignite. Don't want to be a pest but I see this ticket still isn't assigned. Any update?

They added a temporary fix so it doesn't 404 but they are working on the appropriate solution. I don't know if the folks working on this have GitHub, but I'll ask them to comment here about this process. Thanks!

Thanks, in that case I'll just keep checking!

Hi @mairaw - looks like they have removed the 'Register for an exam' link completely.

Have you heard back anymore from them? Will it be possible to register for exams at Ignite?

@BradAF Found this in the FAQ, second question:
https://www.microsoft.com/en-us/ignite/faq
Seems like they are trying to keep this hard to find. I was able to go through the registration process on Pearson and when it came to checkout, a 100% discount was applied. Unfortunately, even thought the FAQ makes it sound like you can take as many exams you like as long as they're part of the covered selection, it appears you can only get one free one. I signed up for AZ-400 for free no problem, but trying to add AZ-300 or any of the other exams that should be covered, the discount is not applied.
Hope this helps. Pretty disappointing that this information was so squirreled away and misleading. The amount of time I've wasted trying to figure out what to do plus the limit of one exam has made this arguably even worth the discount.

Thanks, @ngschroeder - I was at least able to get one exam registered!

Given the changes to the navigation, we're closing this issue as it's no longer applicable.