Allow column resize and auto-resize on browser resize

A community request was mentioned on our mailing list https://groups.google.com/d/topic/openrefine/H3FAf5DSHPk/discussion
This example demonstrates the basic need of having resize-able columns and also auto-resizing the columns if the user changes the size of their browser window.
http://6pac.github.io/SlickGrid/examples/example15-auto-resize.html

there is one for AugularJS. just for the records:
https://tympanix.github.io/angular-table-resize/

@thadguidry I would like to work on the following

@kushthedude This is a baby step issue, to the larger goal of having a better grid technology that supports many features we would like such as draggable re-orderable columns, resizable columns, etc.  But sure, if you want to isolate on only the "resize columns" feature that's fine.  But how you do it and which technology choosen will impact the larger goal later (or you'll have to refactor/redo).

@kushthedude This is a baby step issue, to the larger goal of having a better grid technology that supports many features we would like such as draggable re-orderable columns, resizable columns, etc. But sure, if you want to isolate on only the "resize columns" feature that's fine. But how you do it and which technology choosen will impact the larger goal later (or you'll have to refactor/redo).

@thadguidry Hmm, I was first thinking to introduce table-layout property to our container and then go for column resize property for each column. But I just looked into code and saw the table is bound differently, All the column headers are coming from the DataTableHeader Container and the table data is coming from TableDataContainer which renders the data on row basis by labelling them with odd/even id.

Yes, it's a big mess! Resizing is done in JavaScript explicitly, which is massively inefficient since it triggers many DOM reflows. Also the size of the headers often get out of sync with the size of the grid.

I was thinking to reimplement the tables and bind the content to a single table container if this can be done efficiently. Then the goal of having a better grid technology will be much more easier as we can directly apply our styling to the container rather worrying about how it would affect other columns and their headers.

Myself and @wetneb are not exactly front-end wizards.  So we are depending on folks like you, maybe your peers or others in your circle that do have that experience to set the right course forward.  I think this is going to take a bit of experimentation to see what works best.  Have you looked at how agGrid with Vue works? https://www.ag-grid.com/vuejs-grid/  or others?  This might serve useful to your experiments.

@thadguidry I will do the experiment to try bind the table to a single container for a staft and see how it goes. Coming to the Vue.js point, since the data-table is one of the biggest component of OpenRefine introducing vue.js directly into it may do more harm than good. I am looking to migrate the dialogues to vue.js for a start and if it looks good, we can look for migration of other components to Vue.js in summer, also once the data-table is converted into any progressive framework, introducing infinityLoader will be much more easier in them.

@thadguidry Rather than going for a complete rewrite for the tables part, I
tweaked the UI a bit and got the complete data-table under a single table
container.
Since a complete rewrite in vue
Js will need opinion from more users.
Now I am trying to add resizable feature to the current container and will
keep you posted with the progress.
On Fri, 13 Mar, 2020, 07:26 Thad Guidry<EMAIL_ADDRESS>wrote:

Myself and @wetneb https://github.com/wetneb are not exactly front-end
wizards. So we are depending on folks like you, maybe your peers or others
in your circle that do have that experience to set the right course
forward. I think this is going to take a bit of experimentation to see what
works best. Have you looked at how agGrid with Vue works?
https://www.ag-grid.com/vuejs-grid/ or others? This might serve useful to
your experiments.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OpenRefine/OpenRefine/issues/1312#issuecomment-598510635,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKQMTLWUZADSP56G2GPIXK3RHGHEPANCNFSM4ECZYX5Q
.

@kushthedude great! looking forward to seeing your progress!

Since the following was to be solved with the migration to a new data-grid while looking at chances of Vue.js. I think the GSoC candidates may resolve it during the work at infinityScroll.

I can take this up once infinite scrolling is implemented. However, for the gsoc project, there hasn't been any plans to migrate the table to a new data-grid library, as of now.

Can anyone report any progress on this much needed request?

@wetneb is it a duplicate of #4806 ?

Resizing we have, auto-resize not really, but I don't know if the auto-resize part of that issue is described clearly enough for it to be actionable, so maybe it's fine to close it as such indeed?

The original request doesn't mention browser resize. That was added. Here's what was asked on the mailing list:

Is there any way to adjust the width of columns in OpenRefine?  Sometimes, depending on the length of the content of a column, the width can be so long that it pushes columns off to the right that I would like to be able to see.

I know I can hide columns, but what I am hoping for is to keep a column visible while limiting its width somehow, perhaps by specifying a max-width property on a per-column basis (if not be able to resize columns more dynamically, as one might expect to in a spreadsheet application).

Given that, I think #4806 was a duplicate of this.


Feature: Makefile EXCLUDE_COMPONENTS

Add project build variable, readthedocs.io documentation for EXCLUDE_COMPONENTS
Target audience: CI/team build environments, VM/ARM/legacy users, component developers

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

I'm not sure how the commit made after the pull request was included in this.  That doesn't happen on other git systems.  I'll see if I can revert changes and commit to a feature branch.

Hi @ndotb ,
Thanks for this change. Happy to merge the EXCLUDE_COMPONENTS feature once the branch is cleaned up.

I'm not sure how the commit made after the pull request was included in this. That doesn't happen on other git systems. I'll see if I can revert changes and commit to a feature branch.

Github will update the PR whenever you push to the matching branch in your fork ('master' in this case). If you create a feature branch, you can edit the PR here on the github site to change the branch it points to.
BTW, the eclipse-includes change looks interesting and clever (I'm impressed you could generate valid XML from a makefile!). It doesn't 100% replicate correct header parsing by Eclipse, because each source file has its own list of header directories - including private headers, etc. We have a new build system in the works which should be significantly more IDE friendly and solves this problem in a much nicer way. Planning for this to be released as a preview soon (the existing build system won't be going away in a hurry, either.)

Ok did a hard reset to 765824e1fd429cb16a5646ef7756a250acd566ef, good to go!
Good to hear on the IDE thing.  I'm a slacker on doing it myself, but I would just tell everyone to use the Linux on Windows subsystem and point them to the Microsoft docs on how to install it.

Looks good, have pushed this branch into our internal review & merge queue.


Accept ed25519 and other common SSH Keys

https://github.com/ComputerScienceHouse/proxstar/blob/2b9d32720b466d4b145e733497a00e13a36c7af4/proxstar/static/js/script.js#L226
Hard coded for RSA

Resolved by #200


Document the setup and use of local secrets via JSON-file

Update the README with information on how to add local secrets.json file, and fill it with a secret.

Fixed documentation in different issue.


README uses cryptoIonHasherProvider

The README uses cryptoIonHasherProvider but it was renamed to cryptoHasherProvider in #26.

Fixed by #41.


Missing moderator folder

I have downloaded the zip file and i don't find any folder by the name ' moderator'. As mentioned in the step 'cd moderator'

Good catch. I updated the readme to use the default name when you clone the repo.
For downloading it with zip, cd in the extracted folder. I believe it's meeting-moderator-sample-master


SAK-49512 Lessons: Improve styling when the file cannot be previewed

Thanks for the review Jesús.


HyperlinkSpan does not work ...

I built the code in the current release and it didn't work ...
The label works but not the HyperlinkSpan

This is on MacOS/iOS only I believe


Android app cannot set status to never clear

When trying to set a custom status in the Android mobile app to never clear, it seems initially that it worked. However, the button then says "Failed to update status. Try again." The status falls back to what was there before, and when trying to set a new status to never clear, it doesn't have a status to fall back on, so it's nothing.
I will happily provide screenshots if needed, and I have a sneaking suspicion that the code is not properly checking for the "Don't clear" option because this happens after restarting the app and my phone, and I forgot about it for about a month and came back and tried again and it still didn't work.
My app version is v1.48.1 and the server version is V6.3.0 (the Mattermost community server).
Thank you to any and all who help! I would try to figure it out on my own but my JavaScript skills are abismal.

We have a ticket for this here https://mattermost.atlassian.net/browse/MM-40206, and we are planning to release a fix for it soon.

Ah thanks! I didn't see that site when I searched for the issue. I'll wait patiently for the fix!

This has now been fixed in v1.48.2 dot release.

Thanks! It's working perfectly.


Small barcode can not scan correctly

Hi, thanks for share good library, we used ZXing library in our product.
Our customer meet a problem.
The customer carries an authorization card, which contains a QR code. So, the QR code must be small.
Authentication is required to perform certain tasks, and customers use QR code scanning for this authentication.
Below is example barcode.

In our test.
For size 29px * 29px, it's NG.
For size 58px * 58px, it's OK.
Expect: Small size barcode also can be scan success.
I would appreciate if such issue can be solve. Thanks in advance.

hello dear,
how do you make zxing library scan the small barcode? is it an application or web design?

@AthraaFaraj
Hi, we are use zxing library in iOS application. We used xamarin.


Allow Manual Setting of Complications

Manual Setting
Most complications in watches can be manually set. This includes moonphase, calendar complications, and day/night indicators for example. The ability to manually set these complications within TickTock should be included.
Specs

The work will reside in the appropriate complication's module

Complications to include:

[x] Moonphase

[ ] Calendar

[ ] Day
[ ] Date
[ ] Month
[x] Year
[x] Week

[x] Day Night Indicator

Functionality
By default, TickTock will add an event listener for the right arrow key to toggle between dials to manually set the time. Leveraging this, the complications that support manual setting should be included in the array that the right arrow traverses. This designation will likely be done within the primary Watch Class to store the complications.
Each complication should move relative to its own degree intervals. For example, the day complication should rotate 51.43deg (306 / 7) with each step. Following the boolean flags used within the dials to denote manually set time, these additional complications should work similarly.
The main Watch class contains this.activeDial and this.dialInstances into which dial references are stored. By incrementing this.activeDial that determines on which dial to run rotateHands(). These values will likely need to be updated to this.activeComplication and this.manualComplications. Each complication (except Dials) will need to have its rotateHands() method updated to support forward and backward setting as well as boolean manualtime flags.

Is this module (crown, manual time) even worth keeping?

Still debating the value of this feature and will keep a running list below of pros and cons. Ultimately, a decision will be made for V3 to either fully implement this feature across complications or remove it entirely in lieu of simplicity.
Pros

Greater accuracy to real mechanical watches
Provides a level of interaction

Cons

Without having it for every component, the value is a bit lost
Creates very complicated and difficult to maintain logic in the DIals component
Not sure how much this feature is even utilised

Removed the crown functionality removing the need for this issue.


bump(jq-lsp): update to v0.0.0-20231016080857-b4707e7776a4

Hello. It looks like this LSP version has never been updated since originally being added. Since then there have been some bug fixes in the LSP.

Thanks! It should've been picked up by Renovate. After reviewing the logs it seems like it ends up using the semver versioning strategy which fails because it's not semver compliant. I'd have assumed that Renovate would use Go's versioning strategy. Let's see if this gets picked up by Renovate via https://github.com/mason-org/mason-registry/pull/3648 (has to be merged first), if not we'll unfortunately have to manually manage the version for the time being.

Never mind, it seems like the old version has been pulled and v0.0.0-20231016080857-b4707e7776a4 is the only one. I'd like to believe this is an issue with how they version it, it'd be nice if it was actual versions.

Hey @williamboman, I’m not quite sure why CI is failing here. Anything I need to do to get it passing?

Hey @williamboman, I’m not quite sure why CI is failing here. Anything I need to do to get it passing?

It's failing because the specified version is not available in the Golang pkg registry API. I'm not entirely familiar with how they allow versioning of packages but I assume in this case the versions are sourced upstream from the git repo.

Hey, now there is https://github.com/wader/jq-lsp/releases/tag/v0.1.0

Sweet! That version went through automatically without any issues, https://github.com/mason-org/mason-registry/pull/3700, so we can close this PR


Slightly faster map and filter implementations

This is done on top of #1047, so that need to be merged first.
Details: https://github.com/quantified-uncertainty/squiggle/pull/1047#issuecomment-1241805927

Note: there's a slight change of semantics here. Previously maps were applied in reverse, from the last element to the first. Not it's in order.

Codecov Report

Merging #1115 (c59404e) into develop (45323b0) will decrease coverage by 0.08%.
The diff coverage is 79.09%.

@@             Coverage Diff             @@
##           develop    #1115      +/-   ##
===========================================
- Coverage    59.31%   59.23%   -0.09%
===========================================
  Files           83       83
  Lines         4528     4523       -5
===========================================
- Hits          2686     2679       -7
- Misses        1842     1844       +2

Impacted Files
Coverage Δ

...ducer_Expression/Reducer_ExpressionWithContext.res
81.25% <0.00%> (-18.75%)
:arrow_down:

...educer/Reducer_Expression/Reducer_Expression_T.res
64.00% <0.00%> (-8.73%)
:arrow_down:

...ript/Reducer/Reducer_Type/Reducer_Type_Compile.res
80.00% <50.00%> (+5.00%)
:arrow_up:

...e-lang/src/rescript/Reducer/Reducer_ErrorValue.res
64.70% <55.55%> (+0.70%)
:arrow_up:

...r/Reducer_Expression/Reducer_Expression_Lambda.res
72.41% <60.00%> (-10.35%)
:arrow_down:

...rc/rescript/FunctionRegistry/Library/FR_Danger.res
70.24% <66.66%> (+1.14%)
:arrow_up:

...ucer/Reducer_Dispatch/Reducer_Dispatch_BuiltIn.res
60.43% <75.00%> (+1.09%)
:arrow_up:

...ript/ReducerProject/ReducerProject_ProjectItem.res
83.13% <75.00%> (+1.20%)
:arrow_up:

.../Reducer/Reducer_Expression/Reducer_Expression.res
81.48% <81.81%> (+0.23%)
:arrow_up:

...educer_Dispatch/Reducer_Dispatch_BuiltInMacros.res
96.72% <94.44%> (-0.25%)
:arrow_down:

... and 9 more

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

@umuro can you review this? You're much close to this code.

I left some comments. Most of what I looked at generally seems fine, especially with FR. I'd like Umur to take a closer look at the non-FR stuff though.
I'm unsure about the try strategy, but will leave that to you two.

Most of what I looked at generally seems fine, especially with FR.

Uh, only FR changes are supposed to be here. The rest comes from exceptional-reducer, which this is based upon.
I'm not sure why GitHub still shows the full diff, if exceptional-reducer got merged to develop.

Uh, only FR changes are supposed to be here.

Ah, I fixed it by changing the base to something else, and back. Github's diffs get stale when new code gets into the destination branch.


Add enum autocomplete.

This extension doesn't seem to autocomplete enum struct writing, would be a nice feature.

It appears to be working now.


1949: Add dist.rb Wordmarks

Add dist.rb files with "Wordmark" constants in an attempt to make
Chef Server distributable under a configurable name. This is a starting
point which replaces most of the instances of chef-server and
chef-sevrer-ctl within the Ruby code of the Chef Server repo, but
is not a complete fix for #1949
Signed-off-by: Josh Gitlin<EMAIL_ADDRESS>The original PR can be found at #1976
Creating this branch to run tests.
Build:
https://buildkite.com/chef/chef-chef-server-master-omnibus-adhoc/builds/1258#1a7de4cf-926a-4b6d-a412-0924d1da6eeb
Integration tests:

Trying this out myself in a local build i get
================================================================================
Recipe Compile Error in /var/opt/opscode/local-mode-cache/cookbooks/private-chef/recipes/default.rb
================================================================================

NameError
---------
uninitialized constant Chef::Dist::Server
Did you mean?  Chef::ServerAPI

Cookbook Trace:
---------------
  /var/opt/opscode/local-mode-cache/cookbooks/private-chef/recipes/default.rb:25:in `from_file'

Relevant File Content:
----------------------
/var/opt/opscode/local-mode-cache/cookbooks/private-chef/recipes/default.rb:

 18:
 19:  require 'uuidtools'
 20:  require 'openssl'
 21:
 22:  # Because these symlinks get removed during the postrm
 23:  # of the chef-server and private-chef packages, we should
 24:  # ensure that they're always here.
 25>> %W(private-#{ChefConfig::Dist::SHORT}-ctl #{Chef::Dist::Server::SHORT}-ctl).each do |bin|
 26:    link "/usr/bin/#{bin}" do
 27:      to "/opt/opscode/bin/#{bin}"
 28:    end
 29:  end
 30:
 31:  # Ensure that all our Omnibus-ed binaries are the ones that get used;
 32:  # much better than having to specify this on each resource!
 33:  ENV['PATH'] = "/opt/opscode/bin:/opt/opscode/embedded/bin:#{ENV['PATH']}"
 34:

System Info:
------------
chef_version=15.8.23
platform=amazon
platform_version=2
ruby=ruby 2.6.5p114 (2019-10-01 revision 67812) [x86_64-linux]
program_name=/opt/opscode/embedded/bin/chef-client
executable=/opt/opscode/embedded/bin/chef-client

Running handlers:
Running handlers complete
Chef Infra Client failed. 0 resources updated in 03 seconds
[2020-07-17T14:35:42+00:00] FATAL: Stacktrace dumped to /var/opt/opscode/local-mode-cache/chef-stacktrace.out
[2020-07-17T14:35:42+00:00] FATAL: Please provide the contents of the stacktrace.out file if you file a bug report
[2020-07-17T14:35:42+00:00] FATAL: NameError: uninitialized constant Chef::Dist::Server
Did you mean?  Chef::ServerAPI

If you fix the above and get to chef-server-ctl reconfigure  this is the next error.
      Mixlib::ShellOut::ShellCommandFailed
      ------------------------------------
      Expected process to exit with [0], but received '1'
      ---- Begin output of /opt/opscode/embedded/bin/sv restart /opt/opscode/service/oc_id ----
      STDOUT: fail: /opt/opscode/service/oc_id: unable to change to service directory: file does not exist
      STDERR:
      ---- End output of /opt/opscode/embedded/bin/sv restart /opt/opscode/service/oc_id ----
      Ran /opt/opscode/embedded/bin/sv restart /opt/opscode/service/oc_id returned 1

This symlink does not exist at all.

@PrajaktaPurohit we decided it makes sense to close this and create smaller PRs for each major area we need to fix for dist.

Closing this based on the comments above.


add notification to admin dash if no stats generated yet

Description

Fixes #344 and removes error being thrown if no stats have been calculated and generated yet by the dailyStatsJob

Contributor Checklist

Make sure to do the following steps if they are applicable to your PR:

[ ] Update e2e tests: If you changed the /template/app, then make sure to do any neccessary updates to /template/e2e-tests also.
[x] Update demo app: If you changed the /template/app, then make sure to do any neccessary updates to /opensaas-sh/app_diff also. Check /opensaas-sh/README.md for details.
[ ] Update docs: If needed, update the /opensaas-sh/blog/src/content/docs.

@infomiho no I just left it becuase its only for opensaas.sh and for that we have a lot stats data in the db

@vincanger Makes sense, but it still feels little dirty not to hide it in case we don't have any data. I know we have it, but you know, to make this as nice as possible of an implementation :)

ok I fixed it and pushed to main


Create order only after the payment are paid

What I'm trying to achieve
In traditional B2C, the ShopBasket are only clean and the order is created, when the payment is confirmed
Describe a proposed solution
A possible solution is only create the order when the payment is confirmed, is possible to pass only the checkout id to the gateway payment?
The create the order when the payment is confirmed?

Hi,
The proposed solution is already implemented in API. Here you have PR - https://github.com/mirumee/saleor/pull/4154
We are focusing on the API approach and we don't have enough resources to rebuild the whole checkout process in our old Django approach (which is quite really complicated). I recommend you to switch to API.

@korycins, by API you mean Graphql + React Fronted?
Regards

@michelveloso yes

@korycins thank you for the great job!


Allow easy changing of ALLSKY_IMAGES location

Many people have an SSD attached to their Pi and they want to use it to store images.
A proposed solution:
Add a "Change location of saved images" menu item.  It will:

Prompt the user for AN EXISTING location (i.e., the location of the SSD).
Prompt the user if they want to move all existing images to the new location.
Create uservariables.sh if not already there.  Either way, set ALLSKY_IMAGES to the new location in that file.
Stop Allsky.
Update /etc/lighttpd/lighttpd.conf and ~/allsky/html/includes/allskyDefines.inc with the new location.
Optionally move existing images to new location.
Start Allsky

Future upgrades of Allsky need to make sure uservariables.sh is sourced in prior to updating any files.

Probably want an "allsky_images" setting.

@Temppu76, when changing the location of ALLSKY_IMAGES after installing Allsky there are a few other things to change, which are summarized in the first post:

Stop Allsky: sudo systemctl stop Allsky
Undo the change you made to variables.sh and instead put that line in a new file called ~/allsky/config/uservariables.sh.  When you upgrade Allsky a new variables.sh file is created which wipes out your changes.  Your prior uservariables.sh file is reused however.
You need to use sudo to edit /etc/lighttpd/lighttpd.conf and replace the current location of the images (often /home/pi/allsky/images) with the new location.  Please create a directory called images somewhere on your SSD and put the images there.
Run sudo systemctl stop lighttpd ; sudo systemctl start lighttpd to have the system reread the file.
Edit ~/allsky/html/includes/allskyDefines.inc and do the same thing, but do NOT use sudo.
Restart Allsky: sudo systemctl start allsky
Make sure things work - images are saved, can be viewed, end of night processing works (timelapse, etc.), modules work, etc.

Note that I have never done this and don't know if anyone has upgraded Allsky with this change, so I am counting on your feedback.

@EricClaeys Thanks for the guidance, I did the changes and images are saved to the SSD location, but the images view in the WebUI still does not show the saved images.
Any other ideas what could help here?
I'll do re-installation to SSD, but I can try some tricks before that to find out how this could be done :)

In the Images page of the WebUI, try forcing a refresh of your browser by pressing Shift-F5.

If that doesn't work, double check that ~/allsky/html/includes/allskyDefines.inc points to the new location - if it does, on the web page, right-click and select "View page" or equivalent.  That will bring up the html for the page.  Please paste it in your response.

Hi,
All seems to be as you instructed. I refreshed the page with shift+f5 and also cleared the browser cache.
Attached the page source.
Images - WebUI.txt

Good to hear.  I will add that to the instructions.


CESM2 tasmax model

Currently CESM2 model output is only available for SSPs in GCS and AWS, not historical. Given how widely used this model is, we likely need to get it migrated from ESGF to GCS/AWS.

Per this discussion, CESM2 historical tasmax/tasmin output is not available. @brews @delgadom @kemccusker


database_migrations

🍰 Pullrequest

Replace skeema and have a full migration service with up and down capabilities for complex migration tasks.
Issues

fixes #315
fixes #94
fixes #704

Todo

[x] Basic Setup & Library choices
[x] Complete setup to have working migration functionality
[x] Linting
[x] Import all Database definitions from skeema
[x] Import all Database Values from skeema
[x] Docker integration
[x] Backend integration (restart till correct database version is present)
[x] Workflow Integration

This PR affects the database. Take care, make a backup and read carefully. docker-compose up --build will render some test inaccessible.
This PR requires a docker rebuild docker-compose up --build, since skeema & mariadb are affected and those are not responsive to changes in the codebase
How 2 Test
To test this you can simply run docker docker-compose up --build on an empty database. This will create a database gradido_community with a few tables including migrations which will include fileName as column and one row including 0001-init_db as fileName.
If you already have a database consider to make a backup and check the gradido_community database for a migrations table which includes a field db_version (to bring up the database without starting the migrations run docker-compose up --build mariadb). Once you run docker-compose up --build this should change to a new migrations table with the fileName column and one row including 0001-init_db as fileName. This operation should not affect the data in your database except for the migrations table
For further testing you can run docker-compose run -T database yarn reset which will run a complete downiteration of the database and a complete up iteration. Be aware that this will delete the whole content of your database.

Cannot wait to have seeding!

We only have reset with this. Factories can live in this subproject tho. @Mogge


update(10.1): migration to remove solution style tsconfig is not run

🐞 Bug report
Command (mark with an x)

[ ] new
[ ] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[x] update
[ ] lint
[ ] xi18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?
No
Description
Running ng update @angular/cli to update to v10.1.0 from v10.0.8 does not run the migration to remove solution style tsconfigs.
🔬 Minimal Reproduction

ng update @angular/cli

Your global Angular CLI version (10.1.0) is greater than your local
version (10.0.8). The local Angular CLI version is used.

To disable this warning use "ng config -g cli.warnings.versionMismatch false".
The installed local Angular CLI version is older than the latest stable version.
Installing a temporary version to perform the update.
Installing packages for tooling via npm.
Installed packages for tooling via npm.
Using package manager: 'npm'
Collecting installed dependencies...
Found 54 dependencies.
Fetching dependency metadata from registry...
    Updating package.json with dependency @angular-devkit/build-angular @ "0.1001.0" (was "0.1000.8")...
    Updating package.json with dependency @angular/cli @ "10.1.0" (was "10.0.8")...
UPDATE package.json (3764 bytes)
✔ Packages installed successfully.
splaktar:xlts.dev splaktar$ npm i
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.

removed 4 packages and audited 2828 packages in 12.241s

126 packages are looking for funding
  run `npm fund` for details

found 4 high severity vulnerabilities
  run `npm audit fix` to fix them, or `npm audit` for details

Notice that there are no migrations run
Check that tsconfig.base.json still exists

🔥 Exception or Error
None
🌍 Your Environment
Before

Angular CLI: 10.0.8
Node: 12.16.1
OS: darwin x64

Angular: 10.1.0
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router
Ivy Workspace: Yes

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.1000.8
@angular-devkit/build-angular     0.1000.8
@angular-devkit/build-optimizer   0.1000.8
@angular-devkit/build-webpack     0.1000.8
@angular-devkit/core              9.1.12
@angular-devkit/schematics        9.1.12
@angular/cdk                      10.1.3
@angular/cli                      10.0.8
@angular/fire                     6.0.0
@angular/material                 10.1.3
@ngtools/webpack                  10.0.8
@schematics/angular               9.1.12
@schematics/update                0.1000.8
rxjs                              6.6.2
typescript                        3.9.7
webpack                           4.43.0

After

Angular CLI: 10.1.0
Node: 12.16.1
OS: darwin x64

Angular: 10.1.0
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router
Ivy Workspace: Yes

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.1001.0 (cli-only)
@angular-devkit/build-angular     0.1001.0
@angular-devkit/build-optimizer   0.1001.0
@angular-devkit/build-webpack     0.1001.0
@angular-devkit/core              9.1.12
@angular-devkit/schematics        9.1.12
@angular/cdk                      10.1.3
@angular/fire                     6.0.0
@angular/material                 10.1.3
@ngtools/webpack                  10.1.0
@schematics/angular               9.1.12
@schematics/update                0.1001.0
rxjs                              6.6.2
typescript                        4.0.2
webpack                           4.44.1

Anything else relevant?
I'm not sure why it's showing 9.1.12 for @angular-devkit/core, @angular-devkit/schematics, and @schematics/angular.
Here's the output of npm ls @angular-devkit/core:
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ └──<EMAIL_ADDRESS>deduped
│ ├─┬<EMAIL_ADDRESS>│ │ ├─┬<EMAIL_ADDRESS>│ │ │ └──<EMAIL_ADDRESS>deduped
│ │ └──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ └──<EMAIL_ADDRESS>deduped
│ ├──<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ └──<EMAIL_ADDRESS>deduped
│ ├─┬<EMAIL_ADDRESS>│ │ └──<EMAIL_ADDRESS>deduped
│ └─┬<EMAIL_ADDRESS>│   ├──<EMAIL_ADDRESS>│   └─┬<EMAIL_ADDRESS>│     └──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>deduped
└─┬<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>deduped
  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>deduped
  └─┬<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ └──<EMAIL_ADDRESS>deduped
    ├─┬<EMAIL_ADDRESS>    │ └──<EMAIL_ADDRESS>deduped
    └─┬<EMAIL_ADDRESS>      └──<EMAIL_ADDRESS>deduped

Here's the output of npm ls @schematics/angular:
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    └──<EMAIL_ADDRESS>

I thought that perhaps Scully was causing issues since it had the older versions, but removing it actually resulted in an error (rather than things working properly):
ng update @angular/cli
Your global Angular CLI version (10.1.0) is greater than your local
version (10.0.8). The local Angular CLI version is used.

To disable this warning use "ng config -g cli.warnings.versionMismatch false".
The "@angular-devkit/schematics" package cannot be resolved from the workspace root directory. This may be due to an unsupported node modules structure.
Please remove both the "node_modules" directory and the package lock file; and then reinstall.
If this does not correct the problem, please temporarily install the "@angular-devkit/schematics" package within the workspace. It can be removed once the update is complete.
splaktar:xlts.dev splaktar$ npm ls @angular-devkit/schematics
<EMAIL_ADDRESS>/Users/splaktar/Git/devintent/xlts/xlts.dev
├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ └──<EMAIL_ADDRESS>deduped
│ └─┬<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>

Workaround
With my Scully libs put back in and everything restored to the original state:

rm -rf node_modules/
rm package-lock.json
npm i
ng update @angular/cli

Your global Angular CLI version (10.1.0) is greater than your local
version (10.0.8). The local Angular CLI version is used.

To disable this warning use "ng config -g cli.warnings.versionMismatch false".
The installed local Angular CLI version is older than the latest stable version.
Installing a temporary version to perform the update.
Installing packages for tooling via npm.
Installed packages for tooling via npm.
Repository is not clean. Update changes will be mixed with pre-existing changes.
Using package manager: 'npm'
Collecting installed dependencies...
Found 54 dependencies.
Fetching dependency metadata from registry...
    Updating package.json with dependency @angular-devkit/build-angular @ "0.1001.0" (was "0.1000.8")...
    Updating package.json with dependency @angular/cli @ "10.1.0" (was "10.0.8")...
UPDATE package.json (3757 bytes)
✔ Packages installed successfully.
** Executing migrations of package '@angular/cli' **

❯ Removing "Solution Style" TypeScript configuration file support.
DELETE tsconfig.base.json
UPDATE tsconfig.json (795 bytes)
UPDATE tsconfig.spec.json (385 bytes)
UPDATE projects/angularjs.xlts.dev/tsconfig.app.json (190 bytes)
UPDATE projects/xlts.dev/tsconfig.app.json (190 bytes)
  Migration completed.

Questions
Can the situation with The "@angular-devkit/schematics" package cannot be resolved from the workspace root directory. be improved?

First it didn't find the latest version and used a version from a library that didn't match the version used by the CLI. There was no warning or error here. It seems like there should have been when doing ng update to 10.1 and the update using schematic libs from 9.1.
Second, it didn't find the schematics package when it was clearly visible to npm ls. Why? Is there a bug in this package resolution code?

I am encountering a similar if not the same problem.
I have tsconfig.base.json(but no tsconfig.json) in the root of my project and get a failure of
[error] Error: Could not read 'tsconfig.json'.
    at new JSONFile (<path_to_repo>/node_modules/@schematics/angular/utility/json-file.js:22:19)

every time I try to run
ng update @angular/cli @angular/core

@cwhaught that seems like a separate issue. Can you please open a new issue for it?

Apologies @Splaktar, I thought I saw the same error in your comments at first glance.

Hi @Splaktar,
Find below the answers to your questions, feel free to ask more 😀.

Can the situation with The "@angular-devkit/schematics" package cannot be resolved from the workspace root directory. be improved?

This should be handled with the new version of schematics, aka schematics runtime, which is still currently in it's early design phase.

First it didn't find the latest version and used a version from a library that didn't match the version used by the CLI. There was no warning or error here. It seems like there should have been when doing ng update to 10.1 and the update using schematic libs from 9.1.

Maybe we can issue a warning when the installed @angular/cli version and @schematics/angular mismatch.

Second, it didn't find the schematics package when it was clearly visible to npm ls. Why? Is there a bug in this package resolution code?

While npm ls does list @angular-devkit/schematics, none of these is directly under the root level node_modules directory. This resolution is intended, this is because certain packages with schematics will not have @angular-devkit/schematics as a direct dependency. One such package is @angular/core, in which case having the above node_module structure will cause failures when running ng update @angular/core. Te schematic runtime will handle such situation, however this concept is still in it's early design phase.
Side note: I'd suggest that a lock file maintain is done periodically. Overtime when packages are added and updated, the presence of a lock file will hinder package hoisting and deduping.
There are two quick ways to do such maintainance:

If your project is on GitHub, enable Renovate bot lock file maintenance feature..
Run rm -rf node_modules package-lock.json && npm i

I ran the following command and it worked out:
ng update @angular/cli --from=10.0.0 --to=10.1.3 --migrateOnly=true
As I forgot to use ng update after updating my packages with npm-check-updates to latest CLI version (v10.1.3).

I am going to close this issue as this is a known limitation which is caused by the decision of not have explicit dependencies in a library and unfortunately, at this point there is not much we can do without a total re-design of how ng update works.

Maybe we can issue a warning when the installed @angular/cli version and @schematics/angular mismatch.

This would be helpful to at least give people an idea of where to start investigating. It doesn't look like this was pursued in @angular/cli?
Would you suggest that all schematic library authors out there start doing checks like this? Is that even something that a library would be able to evaluate at runtime? or would this have to be done in the CLI?

Would you suggest that all schematic library authors out there start doing checks like this? Is that even something that a library would be able to evaluate at runtime? or would this have to be done in the CLI?

Typically 3rd party users using schematics will list their dependencies explicitly and in many cases users if they are using @schematics/angular, @angular-devkit/schematics etc...
Side note: typically 3rd party libraries shouldn't dependent on @schematics/angular as most of the APIs in that package are private and not meant to be used by downstream users. 99 out of 100 cases these using

Right, should the warning be for when the default "resolved" @angular-devkit/schematics and @angular-devkit/core versions do not match the version of @angular-devkit/build-angular or @angular/cli (roughly)?

I ran the following command and it worked out:
ng update @angular/cli --from=10.0.0 --to=10.1.3 --migrateOnly=true
As I forgot to use ng update after updating my packages with npm-check-updates to latest CLI version (v10.1.3).

Thanks fr this, it worked great for me with Angular 11.
If anybody like me missed the migration back to non-solution-style tsconfig during v10, then updated to 11, then tried to migrate to eslint and was confrontend with an error message telling you off for still having around a tsconfig.base.json, then this solution is also for you.  Just to be explicit, I ran the command above as is, even though I was at v11 of everything already.
Oh, and that was after manually installing @angular-devkit/schematics, but I didn't test without it, so I don't know if that is part of the solution as well.


Improve VirtualAwg interface

Continuation of #510.

Refactor measuresegment functionality for m4i
Remove legacy fpga acquisition device

@CumulonimbusCalvus

@peendebak
I will squash this PR afer @qSaevar has has done #510.

Tested on the XLD and V1.


Suggestions needed for integrating 3D render

I wrote a 3D renderer based on the same version of wgpu, and I need to render it to an iced window.
The 3D renderer now renders to a winit window:
pub struct Renderer {
    pub size: winit::dpi::PhysicalSize<u32>,
    device: wgpu::Device,
    queue: wgpu::Queue,
    swap_chain: wgpu::SwapChain,
    render_pipeline: wgpu::RenderPipeline, // textured pipeline for mesh
    uniforms: Uniforms,
    scene: Scene,
    depth_texture: DepthTexture,
}

impl Renderer {
    pub fn new(
        device: wgpu::Device,
        queue: wgpu::Queue,
        size: winit::dpi::PhysicalSize<u32>,
        swap_chain: wgpu::SwapChain,
    ) -> Result<Self> {
    /* render stuff */
}

The current render target is from wgpu swap_chain:
// in renderer, it outputs to winit window
let render_target = &self.swap_chain.get_current_frame()?.output.view;

If I want to make it renders to iced, I have 2 solutions:

reuse iced low level device + queue + swap_chain, so I can directly output to iced render target
create my own gpu device, render to an image in memory, copy the buffer to iced widget, maybe need a hack from winit side

Am I doing it right? By searching iced graphics & wgpu docs, I didn't see things I need. What is intended to integrate with iced frontend?

How did you end up doing this?


get/set currentTime for buffer source

@mattdesl what stops you from merging this PR? currentTime is a handy property.

I'll have to test this a bit, last I looked it wasn't this simple when you consider play/pause/etc and all the variables, but maybe it will just work. :smile: Thanks for the PR!


BUILD FAILED

Followed the instructions, ran Ant, and the build failed:
 [exec] Process finished normally.
 [exec]     errors: 348
 [exec]     warnings: 163
 [exec]     build time: 156.611 seconds

BUILD FAILED
C:\sde\web\dojo\enterprise_build_sample_js\enterprise-build-sample-js\build.xml:82: Failed running dojo build: js: "<eval'ed string>#1(Funct
ion)#1(eval)#185(Function)", line 11: Couldn't open file "dojo.js".
js: "<eval'ed string>#1(Function)#1(eval)#185(Function)", line 12: Couldn't open file "tests/runner.js".

Hi Joel,
There could be a couple things going on here:

Did you unpack the dojo source code into a seperate folder on your computer?
Is that folder accessible without administrator permissions? (If it isn't, try running the build as admin)

Thanks gbochenek,

Yes. Unpacked the dojo source packages.
No, I have not tried that.  I will let you know what happens.

Running ant with administrator permissions produced the same result.  I got it to run successfully by commenting out the "buildFailed" tag in build.xml.  The resulting dist code worked in FireFox.  Here is a sample of the build errors:
error(354) Root bundle indicates localized bundles that don't exist.
Root: esri/nls/symbol_en-gb; missing bundles: 'dojo/cldr/nls/gregorian','dojo/cldr/nls/number','esri/nls/jsapi'.
Root: esri/nls/symbol_es-es; missing bundles: 'dojo/cldr/nls/gregorian','dojo/cldr/nls/number','esri/nls/jsapi'.
Root: esri/nls/renderer_it-it; missing bundles: 'dojo/cldr/nls/gregorian','dojo/cldr/nls/number','esri/nls/jsapi'.
Root: esri/nls/symbol_ROOT; missing bundles: 'dojo/cldr/nls/gregorian','dojo/cldr/nls/number','esri/nls/jsapi'.
Root: esri/nls/renderer_nb; missing bundles: 'dojo/cldr/nls/gregorian','dojo/cldr/nls/number','esri/nls/jsapi'.
error(311) Missing dependency.
module: dojo/js-dojo-1.10.4/tests/_base/loader/requirejs/layers/layer1; dependency: beta
module: dojo/js-dojo-1.10.4/tests/_base/loader/requirejs/layers/layer1; dependency: gamma
module: dojo/js-dojo-1.10.4/tests/_base/loader/requirejs/layers/layer1; dependency: gamma
module: dojo/js-dojo-1.10.4/tests/_base/loader/requirejs/layers/layer1; dependency: epsilon
module: dojo/js-dojo-1.10.4/tests/_base/loader/requirejs/i18n/commonA; dependency: i18n!nls/colors
module: dojo/js-dojo-1.10.4/tests/_base/loader/requirejs/i18n/commonB; dependency: i18n!nls/colors
module: dojo/js-dojo-1.10.4/tests/DeferredList; dependency: doh/main
module: dojo/js-dojo-1.10.4/robotx; dependency: doh/main
module: dojo/js-dojo-1.10.4/tests/debounce; dependency: doh
module: dojo/js-dojo-1.10.4/tests/_base/loader/requirejs/text/local; dependency: text!./resources/local.html
module: dojo/js-dojo-1.10.4/tests/store/JsonRest; dependency: doh/main
module: dojo/js-dojo-1.10.4/tests/_base/i18nExhaustive; dependency: doh/main
module: dojo/js-dojo-1.10.4/tests/date; dependency: doh
module: dojo/js-dojo-1.10.4/tests/_base/xhr; dependency: doh/main
error(307) Failed to evaluate module tagged as pure AMD (fell back to processing with regular expressions).
module: esri/renderer; error: SyntaxError: illegal character
module: esri/symbol; error: SyntaxError: illegal character
module: dojo/js-dojo-1.10.4/tests/_base/loader/requirejs/requirejs-setup; error: ReferenceError: "window" is not defined.
module: dojo/js-dojo-1.10.4/tests/_base/loader/xdomain/local1; error: ReferenceError: "xdomainExecSequence" is not defined.
error(356) The optimizer threw an exception; the module probably contains syntax errors.
module identifier: esri/renderer; exception: InternalError: illegal character
module identifier: esri/symbol; exception: InternalError: illegal character


verify s6 rpm package works as expected

create a s6 service.
mkdir -p /tmp/here

create the run script in /tmp/here. more /tmp/here/run
#!/bin/sh

echo 'run script send log' | systemd-cat -p info

make run executable.
chmod +x /tmp/here/run

create symbol link in scan dir.
cd /var/lib/s6/service
ln -s /tmp/here here

mock the package install, which will trigger the following command.
s6-svscanctl -an /var/lib/s6/service

check the log with the following command.
journalctl -f

you should see "run script send log".


Portal Live stream issue

Live streaming the video will destabilize the objects on the hololens (they start following the camera..).
This only happened after the culling commit, before that it was working as expected.

Hello,
I am also facing this issue as well. It is kind of annoying when trying to show my app to a group of people and would be nice if we have a fix for it.
Regards,
Omar.

Sorry for the delay. I'll take a look at this issue in the next couple of days.

Hi there, are there any updates on this issue?
I've had the same problem when I'm trying to view my app on live preview.

I pushed a  fix to the develop/fix-mrc branch. It works for me now; please give it a try.

Hello Cristi,
I just tried it out and it works great.
Thank you.

The fix is in master. Please let me know if you still experience issues with mixed reality capture.


[tool_crash] ProcessException: The directory name is invalid. Command: D:\flutter\flutter\bin\cache\dart-sdk\bin\pub.bat, OS error code: 267

Command
flutter doctor

Steps to Reproduce

...
...
...

Logs
ProcessException: The directory name is invalid.
Command: D:\flutter\flutter\bin\cache\dart-sdk\bin\pub.bat, OS error code: 267
#0      _ProcessImpl._start (dart:io-patch/process_patch.dart:390:33)
#1      Process.start (dart:io-patch/process_patch.dart:36:20)
#2      LocalProcessManager.start (package:process/src/interface/local_process_manager.dart:39:20)
#3      ErrorHandlingProcessManager.start.<anonymous closure> (package:flutter_tools/src/base/error_handling_io.dart:638:33)
#4      _run (package:flutter_tools/src/base/error_handling_io.dart:532:20)
#5      ErrorHandlingProcessManager.start (package:flutter_tools/src/base/error_handling_io.dart:638:12)
#6      _DefaultProcessUtils.start (package:flutter_tools/src/base/process.dart:472:28)
#7      _DefaultProcessUtils.stream (package:flutter_tools/src/base/process.dart:491:35)
#8      _DefaultPub.batch (package:flutter_tools/src/dart/pub.dart:282:34)
<asynchronous suspension>
#9      _DefaultPub.get (package:flutter_tools/src/dart/pub.dart:222:7)
<asynchronous suspension>
#10     PubDependencies.update (package:flutter_tools/src/cache.dart:774:5)
<asynchronous suspension>
#11     Cache.updateAll (package:flutter_tools/src/cache.dart:562:9)
<asynchronous suspension>
#12     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1109:7)
<asynchronous suspension>
#13     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1009:27)
<asynchronous suspension>
#14     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#15     AppContext.run (package:flutter_tools/src/base/context.dart:149:12)
<asynchronous suspension>
#16     CommandRunner.runCommand (package:args/command_runner.dart:197:13)

[✓] Flutter (Channel stable, 2.0.1, on Microsoft Windows [Version 10.0.17134.1], locale en-US)
    • Flutter version 2.0.1 at D:\flutter\flutter
    • Framework revision c5a4b4029c (4 months ago), 2021-03-04 09:47:48 -0800
    • Engine revision 40441def69
    • Dart version 2.12.0

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\dell\AppData\Local\Android\sdk
    • Platform android-30, build-tools 30.0.3
    ✗ No Java Development Kit (JDK) found; You must have the environment variable JAVA_HOME set and the java binary in your PATH. You can download the JDK from
      https://www.oracle.com/technetwork/java/javase/downloads/.

[✗] Chrome - develop for the web (Cannot find Chrome executable at .\Google\Chrome\Application\chrome.exe)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).

[✓] VS Code (version 1.57.1)
    • VS Code at C:\Users\dell\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (1 available)
    • Edge (web) • edge • web-javascript • Microsoft Edge 91.0.864.59

! Doctor found issues in 3 categories.

Flutter Application Metadata
No pubspec in working directory.

Closing this as duplicate of https://github.com/flutter/flutter/issues/80418


Update storage-import-export-data-to-files.md

The following should be stated for clarity: "Attach the external disk to the file share and run WAImportExport.exe file."

@alkohli
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@alkohli - Can you review the proposed changes?

@alkohli
Can you review this old PR and determine whether it needs to be closed or merged?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge. If the changes no longer apply, add an explanation and a #please-close comment to close the PR.
@MicrosoftDocs/public-repo-pr-review-team

I sent an email to the content owner today.

@akashdubey-ms - Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
fyi @alkohli
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

I sent a Teams message to the content owner today.

@stevenmatthew
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

I sent an email to the content owner today.

#sign-off

I've reviewed but am unable to sign off despite being an owner.
From: Courtney Wales @.>
Sent: Friday, September 13, 2024 8:53 AM
To: MicrosoftDocs/azure-docs @.>
Cc: Stephen Haas @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Update storage-import-export-data-to-files.md (PR #122444)
I sent an email to the content owner today.

Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/pull/122444#issuecomment-2349271558, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AF3FMF7NHW6VLSCBJEMN56LZWMC6PAVCNFSM6AAAAABHT2LGQGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNBZGI3TCNJVHA.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@stevenmatthew - PRMerger uses the author metadata value to assign the PR and to validate the #sign-off . If you own the content, get @alkohli to agree and update those values in this files. thanks.


 That's unexpected. I will investigate...

I had an uncommitted example.go file in the directory root from something I was doing a year ago, it had package main at the top which confused go.


Fix compatibliity with numba 0.54

Contributor Checklist:

[x] I have added or updated my entry in the creators.json file
[x] I have added a changelog entry for my contribution
[N/A] I have added/updated documentation for all user-facing changes
[x] I have added/updated test cases
[N/A] I have included the rebuilt production build of the client (only if changes were made to the GUI)

Reviewer Checklist:

[ ] /azp run libertem.libertem-data passed

/azp run libertem.libertem-data

Merging! :-)


Missing "Earns" for app users: Node SDK should fail gracefully for "unknown" return code from Agora, instead of returning "success"

The submit transaction result only accounts for "error" or "success", and mis-handles "unknown" as "success"
https://github.com/kinecosystem/kin-node/blob/4295b4fbde73714c7bd63771ca0ac066b0300f7a/src/client/internal.ts#L49
this causes the issue where if Agora or blockchain has a problem (e.g. runs out of SOL, the blockchain has some delay), then Agora will return "unknown" instead of "error" or "success".  Here's the problem:

ios/android SDKs handle "unknown", and gracefully retry
node SDK does NOT handle it properly, and returns "success"

this leads to the problem where:

when Agora/Solana blockchain has a hiccup, it returns txn IDs to node SDK clients as "successful" txns, however those txns do not exist on the blockchain
clients don't know any better, and thus cannot retry properly ==> missing Earns for app users

see discussion in this Discord thread.

Transferring information here posted by @richardmands
https://discord.com/channels/808859554997469244/923258669846655036/928755367654084638


Add a triangle shape

This is a PR to add a triangle shape to the basic shapes library. In this case, a Triangle shape is defined by three points (three tuples). I've made some tests and it looks like it's working fine, there are still some exceptions to write, maybe not letting the user pass two or more collinear points.
Please feel free to suggest any modification either on the small line of docs or on the code itself.
This PR also adds black to Pipfile.

Coverage decreased (-1.4%) to 77.874% when pulling 7ac463332d8f2d2130577a9ddc6d42e2afb8e664 on mvinoba:add-triangle-shape into 84f3a74cf9cb29c6d24b990dc9a474562114392b on alcarney:master.

Awesome! :smiley:
I'm just sorry for all the teething problems you are having to deal with. I wouldn't worry about the issues with flake8 just yet. I'm going to suggest some changes and I hope that they will make the issue go away. It is a shame that black and flake8 don't seem to agree, I thought one of the points of using black that it means your code would pass a flake8 check automatically....
So here are a few requests:

Can you split up your def draw(self): method up into a few smaller methods on the class? Taking a look at your code I suggest adding def get_q(self), def get_t(self, x, y) and def get_s(self, x, y) methods. I'm hoping by doing that the issue with flake8 will go away.
Could you add a comment somewhere explaining what technique is being used to determine which points are inside/outside the triangle and perhaps a link to some of the resources you used to write this?
Finally could you add a test to tests/shape/test_basic.py? You should be able to use the tests that are already there as an example and here is some additional documentation. But feel free to message me if there is something that's not explained clearly.

I can't tell you how excited I am to have someone else take the time to work on this with me and I really appreciate the effort you are putting in to deal with all quirks - we'll get there eventually :smile:

@alcarney hey! so I got some time to finish the code I was writing:

I've splited the code in smaller methods and documented each of them in separate docstrings
I've added a test to tests/shape/test_basic.py and they are all passing just fine
Black and Flake8 seems to finally agree with each other now. I just had to add a # noqa E501 to a long line with an URL.

There are some conflicts now regarding the new changes to basic.py that should be resolved, I'll take a look at the new __repr__ methods and try to implement it to the Triangle shape in the next commit before we can hopefully merge this :)
Thank you for showing your appreciation, I'm enjoying a lot working on this package!

I went ahead and resolved the conflict, turns out I happened to add the __repr__ method  for the Square class on the same line as the one you started the Triangle class.
Congratulations, your Triangle class is now officially a part of stylo! :tada: :smile:


Update zen cpp channel to v3.20

replacing this pr with https://github.com/IBM/ibm-common-service-operator/pull/810 for clarity


🌱 Remove release-1.2 upgrade test description from README

jjbs have no mention of release-1.2 branch. This is a leftover trigger

/test-centos-e2e-integration-main
/test-ubuntu-integration-main

lgtm


Make sure all end game messages are prominently displayed when game is ended

Probably the messages for special outcomes are not shown now (Camping, Restart) and perhaps the other messages are not all displayed in a consistent way, with the same code. With the messages now shown in F1 scenario screen, more will be written, so it makes sense to show them all also at game end.
Of course, if a message is empty, the game should gracefully skip it (or display a default message; probably it does already? for all outcomes?).
If it turns out the code is complete and uniform and nothing needs to be done, just flesh out the comment for 'mendMsg :: [(Outcome, Text)]`, saying that each message is displayed at game end and that if empty then a default one is.

Also, perhaps some or all of the screens shown during game over sequence should be available for viewing as part of the stats display as cycled menus. Care needs to be taken not to confuse the player that the game ends again, though.

One more thing: at least some longer game over messages are displayed in the most boring font possible. They should at least be bumped to bright white or yellow or, in this one case, we can go overboard and colour even every word differently.


Improve label values

Changes proposed

[x] Based on https://github.com/Hyperfoil/Horreum/pull/1899
[x] Remove LIMIT when not provided, i.e., get all
[x] Remove DISTINCT from main query
[x] Add index on dataset.testId and partial indexes on label.filtering and label.metrics

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.

Postgres query plan running /api/test/339/labelValues without any query parameter and benchmark results with 20rps.
Before
 Limit  (cost=15670.89..15671.05 rows=64 width=56)
   ->  Sort  (cost=15670.73..15672.33 rows=637 width=56)
         Sort Key: dataset.runid DESC
         ->  Unique  (cost=15631.51..15641.07 rows=637 width=56)
               ->  Sort  (cost=15631.51..15633.10 rows=637 width=56)
                     Sort Key: (COALESCE(jsonb_object_agg(label.name, lv.value) FILTER (WHERE ((label.name IS NOT NULL) AND (label.name <> $1))), '{}'::jsonb)), dataset.runid, dataset.id, dataset.start, dataset.stop
                     ->  GroupAggregate  (cost=15123.68..15601.84 rows=637 width=56)
                           Group Key: dataset.id
                           ->  Gather Merge  (cost=15123.68..15565.43 rows=3793 width=102)
                                 Workers Planned: 2
                                 ->  Sort  (cost=14123.65..14127.60 rows=1580 width=102)
                                       Sort Key: dataset.id
                                       ->  Hash Left Join  (cost=294.29..14039.71 rows=1580 width=102)
                                             Hash Cond: (lv.label_id = label.id)
                                             Filter: ((label.id IS NULL) OR ($3 AND label.filtering) OR ($4 AND label.metrics))
                                             ->  Nested Loop Left Join  (cost=0.42..13738.01 rows=2981 width=55)
                                                   ->  Parallel Seq Scan on dataset  (cost=0.00..9179.24 rows=265 width=24)
                                                         Filter: (testid = $2)
                                                   ->  Index Scan using by_dataset on label_values lv  (cost=0.42..16.80 rows=40 width=35)
                                                         Index Cond: (dataset_id = dataset.id)
                                             ->  Hash  (cost=261.16..261.16 rows=2616 width=57)
                                                   ->  Seq Scan on label  (cost=0.00..261.16 rows=2616 width=57)

After
 Sort  (cost=13088.86..13090.42 rows=623 width=56)
   Sort Key: dataset.runid DESC
   ->  HashAggregate  (cost=13052.16..13059.95 rows=623 width=56)
         Group Key: dataset.id
         ->  Hash Left Join  (cost=296.31..13023.71 rows=3793 width=102)
               Hash Cond: (lv.label_id = label.id)
               Filter: ((label.id IS NULL) OR ($3 AND label.filtering) OR ($4 AND label.metrics))
               ->  Nested Loop Left Join  (cost=0.72..12709.30 rows=7155 width=55)
                     ->  Index Scan using dataset_testid_key on dataset  (cost=0.29..1856.27 rows=623 width=24)
                           Index Cond: (testid = $2)
                     ->  Index Scan using by_dataset on label_values lv  (cost=0.42..17.02 rows=40 width=35)
                           Index Cond: (dataset_id = dataset.id)
               ->  Hash  (cost=261.93..261.93 rows=2693 width=57)
                     ->  Seq Scan on label  (cost=0.00..261.93 rows=2693 width=57)

Hi @willr3 @johnaohara, in order to properly compare the composable PoC with the current Horreum I tried to investigate why we have poor performance in the current setup.
At the moment this is what I was able to find, not huge improvement though!
I think that further improvements can be done if we re-design the query, e.g., using multiple queries rather than one single one or de-normalizing the data to avoid some joins or .. (?). Then, the same logic/reasoning could be then applied to the PoC as well. wdyt?


[PLUGIN REQUEST] flutter-tools.nvim

Field
Description

Plugin
flutter-tools.nvim

Homepage
https://github.com/nvim-flutter/flutter-tools.nvim

Nixpkgs
https://github.com/NixOS/nixpkgs/blob/nixos-unstable/pkgs/applications/editors/vim/plugins/generated.nix#L4072

Extra Information
Flutter-tools is an extensive plugin with many flutter-specific features that enhance flutter development in neovim.
Features

Telescope support
In-editor flutter run
Debugging
Widget guides
Nvim-dap support
Statusline decoration with app version
Closing tags
Hot reloading

Opened https://github.com/NixOS/nixpkgs/pull/352744

Hi, are there any updates on flutter-tools support?


unresolved import

having the following problem when compiling this library:
   Compiling rustyline-derive v0.3.1
error[E0432]: unresolved import `proc_macro`
 --> /home/$USER/.cargo/registry/src/github.com-1ecc6299db9ec823/rustyline-derive-0.3.1/src/lib.rs:1:5
  |
1 | use proc_macro::TokenStream;
  |     ^^^^^^^^^^ help: a similar path exists: `syn::proc_macro`
error: aborting due to previous error
For more information about this error, try `rustc --explain E0432`.
error: could not compile `rustyline-derive`.
To learn more, run the command again with --verbose.

Which version of rust are you using? (rustc -V)

https://doc.rust-lang.org/proc_macro/index.html

1.15.0

rustyline-derive requires the 2018 edition of rust, which was introduced in rustc 1.31.

@bjorn3 Sorry, you are right.

I'm encountering this after cloning and running examples. Normal cargo build works fine.
$ rustc -V
rustc 1.41.0
$ cargo -V
cargo 1.41.0

https://github.com/rust-lang/cargo/pull/7700
=> cargo 1.42


Fix build errors for multi-module build

Multi-module dependency resolution fails for
certain plugins that require dependency resolution
in early phases of the maven lifecycle.  This
behaviour is described in the associated JIRA
ticket: https://issues.apache.org/jira/browse/MNG-3283
This pull-request aims to address instability
due to dependency resolution by adopting the
suggestion in the linked ticket above.

@leerho I've opened this PR with the intent to address some of the issues you raised in the dev mailing group - I'd be grateful if you could review this.

Two comments:

The readme says that only JDK 8,9 and 11 are required to compile.  First, using mvn clean compile  I could not get it to compile, even using the --toolchains tools/toolchains.xml option unless the installed JDKs on my system matched both the list of JDKs in the GitHub actions matrix [] command as well as the specified JDKs in the toolchains.xml.  So I don't know if I was doing something else wrong.  If this is true this means that the developer user may need to modify these two lists based on what JDKs are available in the developer's system; and a note to this effect should also be in the README.
When loading this into Eclipse, it immediately showed an error in
datasketches-memory-java8-tests/src/test/java/org/apache/datasketches/memory/test/VirtualMachineMemoryTest.java
This file has the package declaration of
package org.apache.datasketches.memory.test.internal; while it is in the org.apache.datasketches.memory.test package.
I am puzzled how you were able to get this to compile without errors.

Nonetheless, Eclipse had numerous other problems too, and I am hoping your PR will address those.  I was not able to get Eclipse to work.

I reviewed your PR, approved it and made some comments in the PR.
Lee.
On Sun, Jul 11, 2021 at 4:37 PM David Cromberge @.***>
wrote:

@davecromberge https://github.com/davecromberge requested your review
on: #132 https://github.com/apache/datasketches-memory/pull/132 Fix
build errors for multi-module build.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/apache/datasketches-memory/pull/132#event-5004628017,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADCXRQSFQG27KSGLUOFKIPTTXITKDANCNFSM5AFYERNA
.

@leerho thank you for your review comments.

The readme says that only JDK 8,9 and 11 are required to compile. First, using mvn clean compile I could not get it to compile, even using the --toolchains tools/toolchains.xml option unless the installed JDKs on my system matched both the list of JDKs in the GitHub actions matrix [] command as well as the specified JDKs in the toolchains.xml. So I don't know if I was doing something else wrong. If this is true this means that the developer user may need to modify these two lists based on what JDKs are available in the developer's system; and a note to this effect should also be in the README.

I have added additional documentation in line with your suggestion to clarify how to setup a local environment for use with toolchains.

When loading this into Eclipse, it immediately showed an error in
datasketches-memory-java8-tests/src/test/java/org/apache/datasketches/memory/test/VirtualMachineMemoryTest.java
This file has the package declaration of
package org.apache.datasketches.memory.test.internal; while it is in the org.apache.datasketches.memory.test package.
I am puzzled how you were able to get this to compile without errors.

This was unfortunately introduced whilst I migrated the changes for my PR from one branch to another, and would have been checked in the former branch but missed during the migration.  Interesting, this does not result in a compiler error in maven or IntelliJ, for example using:
   $ mvn process-classes compiler:testCompile

The Eclipse compiler correctly detected the issue and this has now been corrected.

Nonetheless, Eclipse had numerous other problems too, and I am hoping your PR will address those. I was not able to get Eclipse to work.

I have updated the development documentation for Eclipse to help address any outstanding issues.  Also, Eclipse is strict in compiling the module definition for the datasketches-java9-tests, as it detects that there is a mixture of classpath and module path usage due to the MultiRelease JAR.  The solution I have adopted is to remove the troublesome datasketches-java9-tests module to ensure Eclipse builds correctly.
Finally, whilst I have tested locally, I would appreciate if you could try out this branch to give me an opportunity to further refine the documentation and/or code before merging.

See comments in dev@

(previous comment not finished) : ...
/target/


Support for full api

thanks for the PR!
I'd prefer not to introduce any external dependencies (six and requests).  could you revert those parts in your sauceclient.py?

I felt these common external libs are preferable to wacky "if PY2" conditionals. But fair enough. I'll revert those back tonight. :)
Drew
On Mon, Nov 7, 2016 at 8:21 PM -0500, "Corey Goldberg"<EMAIL_ADDRESS>wrote:
thanks for the PR!
I'd prefer not to introduce any external dependencies (six and requests).  could you revert those parts in your sauceclient.py?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Code has been updated with no external dependencies. I didn't use any PY2 conditionals, just catching import errors for the two modules that changed location in Py3.

thanks.
I just merged it as-is.  If you are up for it, the documentation in README.rst needs to be updated to reflect these changes.

Cool. As long as you're good with the code, I'll go update to docs in the next day or two.
Also, I had updated the version to 0.3dev... Wasn't sure what you wanted to go with once this gets packaged up.


Can't run tests with extra driver

driverFactory 49
switch (Configuration.getDriverType())
-> reuse desired capabilities values to determine type

ChromeCapabilities->getCapability (32 line)
analyze howto split caps init from config and desired list only
AbstractCapabilities.initCapabilities
for (Map.Entry<String, String> entry : capabilitiesMap.entrySet()) {

ExtendedElementLocator (111 line)
WebDriver driver = DriverPool.getDriver();
why default driver here?

fixes should be available in <IP_ADDRESS> core build. pay attention that capabilities should be generated using general method outside of carina!


How to connect async submit_sm and its submit_sm_resp

I am using cloudhopper to send async message submissions. Since I have to log every messages sent through the system, I make a data store entry for every message before it is submitted. Then I do an async submit with the row_id from data store as an optional parameter. Now my session handler captures smpp_async_resp, reads the row_id from optional parameter and updates the row with the message id.
So basically i am passing row_id to the smpp server so that when the server responds, the response pdu will have the row id in it. Is there a better way to do this? There has to be some other way to connect the async submit with the submit response. What am I missing? Please help

Well, while you're method works -- each SMPP pdu is assigned a unique id from a sequence.  The response back on a session contains the same numeric identifier.  The id is only valid per-session though.  So to match up responses to requests, I'd suggest you use the numeric id of the packet.  sendRequestPDU will assign the value for you unless it's been pre-assigned (which i would not recommend doing in your own code).
Basically, you send the PDU, then store the id assigned to it in your db (and the update to your db needs to be done quickly in case a response comes in fast).  When the response comes in, you look it up via that id.  And you need to do this per-session.
What many people do is simply keep the sequence id <-> your id in-memory.
You're pretty lucky your upstream SMSC allows you to include an optional parameter.  Most don't.  If you only plan on supporting them then the way you're doing it may be entirely ok.  Just know that most SMSCs don't support it and you'll end up doing what i suggested at a later time.

ok what if my function using two smpp ( 1. Vodafone, 2. ABC)
now the sms send thorugh  vodafone will have its own set of sequence no (1,2,3,4,5.. so on) and ABC will have its own (1,2,3,4,5... so on)
i m stuck here


Description
Closes #575

MS2 Criteria
(to be filled out by PR reviewer)

[ ] the model validates with the SAMM SDS SDK in the version specified in the Readme.md of this repository by the time of the MS2 check  (e.g., 'java -jar samm-cli.jar aspect <path-to-aspect-model> validate ). The  SAMM CLI is available here and in GitHub
[ ] use Camel-Case (e.g., "MyModelElement" or "TimeDifferenceGmtId", when in doubt follow https://google.github.io/styleguide/javaguide.html#s5.3-camel-case)
[ ] the identifiers for all model elements start with a capital letter except for properties
[ ] the identifier for properties starts with a small letter
[ ] all model elements at least contain the fields "preferred name" and "description" in English language. The description must be comprehensible. It is not required to write full sentences but style should be consistent over the whole model
[ ] Property and the referenced Characteristic should not have the same name
[ ] the versioning in the URN follows semantic versioning, where minor version bumps are backwards compatible and major version bumps are not backwards compatible.
[ ] use abbreviations only when necessary and if these are sufficiently common
[ ] avoid redundant prefixes in property names (consider adding properties to an enclosing Entity or even adapt the namespace of the model elements, e.g., instead of having two properties DismantlerId and DismantlerName use an Entity Dismantler with the properties name and id or use a URN like io.catenax.dismantler:0.0.1)
[ ] fields preferredName and description are not the same
[ ] preferredName should be human readable and follow normal orthography (e.g., no camel case but normal word separation)
[ ] name of aspect is singular except if it only has one property which is a Collection, List or Set. In theses cases, the aspect name is plural.
[ ] units are referenced from the SAMM unit catalog whenever possible
[ ] use constraints to make known constraints from the use case explicit in the aspect model
[ ] when relying on external standards, they are referenced through a "see" element
[ ] all properties with an simple type have an example value
[ ] metadata.json exists with status "release"
[ ] generated json schema validates against example json payload
[ ] file RELEASE_NOTES.md exists and contains entries for proposed model changes
[ ] all contributors to this model are mentioned in copyright header of model file

MS3 Criteria
(to be filled out by semantic modeling team before merge to main-branch)

[ ] All required reviewers have approved this PR (see reviewers section)
[ ] The new aspect (version) will be implemented by at least one data provider
[ ] The new aspect (version) will be consumed by at least one data consumer
[ ] There exists valid test data
[ ] In case of a new (incompatible) major version to an existing version, a migration strategy has been developed
[ ] The model has at least version '1.0.0'
[ ] If a previous model exists, model deprecation has been checked for previous model
[ ] The release date in the Release Note is set to the date of the MS3 approval

@Modeling Team
There are still 3 minor requirements pending, which means that the model must be probably updated again. But it would be helpful if someone from the could do a first check. Also the release notes need to be revised and more details about what has been adapted need to be added.

Eclipse ECA issue should be fixed as i already have re-executed my ECA (Eclipse Contributor Agreement) accordingly.


merge package template from Kronos-Tools/npm-package-template

Updated standard to latest version

Codecov Report

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

@@           Coverage Diff           @@
##           master     #183   +/-   ##
=======================================
  Coverage   88.53%   88.53%
=======================================
  Files           1        1
  Lines         218      218
=======================================
  Hits          193      193
  Misses         25       25

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 2455c02...a2afd2d. Read the comment docs.


[LIBCLOUD-681] Allow for multiple record changes in one call

Issue: https://issues.apache.org/jira/browse/LIBCLOUD-681

Thanks @smstone!
This looks good to me, but could you please rebase and squash into a single commit?

@erjohnso I rebased and squashed, PTAL. Thanks.


Remove unneeded dockerfiles

Pending praekelt/mama-ng-dockerconfig#44, we don't need some of the images built here.

:+1: if you've checked that the containers still work after these changes. :-)

:+1:


Issue on page /notebooks/04/04-exam-room-scheduling.html

Your issue content here.
I applied this fix and the software works
if 'google.colab' in sys.modules:
    %pip install pyomo >/dev/null 2>/dev/null
    %pip install highspy >/dev/null 2>/dev/null
    !apt-get install -y -qq coinor-cbc > coinor-cbc.log
    import sys
    solver = 'cbc'
    import pyomo.environ as pyo
    SOLVER = pyo.SolverFactory(solver, executable='/usr/bin/cbc')

assert SOLVER.available(), f"Solver {solver} is not available."

Thanks for spotting this error. The solver that should have been used should have been "appsi_highs" and not "cbc". The latest commit fixes it.


Preview HTML renderer doesn't honor "displayAlign" setting

The Preview HTML renderer doesn't seem to honor the "displayAlign" setting (and probably the "displayIndent" setting neither). That's a problem if you set it to anything but "center" because then the rendered LaTeX will jump from the center to the left/right when being typeset with the high-quality renderer.

That's expected as it has not been implemented; the PreviewHTML output is not feature complete.
To work around this, set the text-align for the MJXp-display class.

Duplicate of #1713.

MathJax v3 no longer has a PreviewHTML renderer.


Execute preTasks before calculating externalDependencies

Makes sure all compiled files and folders exist already.
When a Play Framework project has no target folder yet (because the project wasn't compiled yet) and run sbt eclipse on such a not-yet-compiled and not-yet-eclipsified Play project the target/scala-2.11/classes_managed folder won't get added to the .classpath file (despite it gets created on the filesystem eventually) - which results in a lot of errors when importing the project in eclipse.
That's because right now we add managedClassDirectory folders to the .classpath file only if the folder does exists on the filesystem - but because the compilation from preTasks hasn't been done yet the folder isn't there yet.
Most users won't recognize this bug because they probably run sbt eclipse after they already compiled their project at least once - having therefore created the target/scala-2.11/classes_managed folder on the filesystem already.

@benmccann If you could release new version 5.0.2 after merging this that would be great. Thank you!

@benmccann if you could release a new version that whould be great!

5.1.0 should be released now!
Would you be able to return the favor for me? :-)  I'm looking to get my two pending PRs merged at https://github.com/typesafehub/js-engine/pulls

Thanks Ben!
Unfortunately I don't have commit rights for the js-enginerepo...

You probably want to ping James, Rich, Greg or Christopher Hunt

Ok, thanks!


Add task to notify users 2 weeks before membership expiration.

Coverage decreased (-0.2%) to 92.857% when pulling<PHONE_NUMBER>aea6d8647eb90a51f03372dbfbe382 on johngian:renewal-notifications into 35ea5cc556f9d4cf80cbe9d0ca91d323afa70e63 on mozilla:master.

Coverage increased (+0.05%) to 92.96% when pulling 487eeda4ddd74c16078ea62d6e7f3789b9cebf33 on johngian:renewal-notifications into e8ffe642324d16d9fb177d4c9b78710435e9b19b on mozilla:master.

Coverage increased (+0.04%) to 92.948% when pulling cc501786ac50e686d719975c0896a0e3514b97cb on johngian:renewal-notifications into e8ffe642324d16d9fb177d4c9b78710435e9b19b on mozilla:master.

Coverage increased (+0.04%) to 92.945% when pulling 71f01e550a12b3268924b6cfe6f1a6bde09731ec on johngian:renewal-notifications into e8ffe642324d16d9fb177d4c9b78710435e9b19b on mozilla:master.

Coverage increased (+0.04%) to 92.953% when pulling 71f01e550a12b3268924b6cfe6f1a6bde09731ec on johngian:renewal-notifications into e8ffe642324d16d9fb177d4c9b78710435e9b19b on mozilla:master.


Adding buttons to accept on a component if the cookie category is not (yet) accepted

(This is more a question of best practice on how to do something, and twitter is just an example.)
Lets say I want to add a twitter flow on a page and let the visitor accept/not accept content and cookies from twitter.
I add

It gets displayed if the user has accepted (or when enabling) the category "twitter". Fine.
Problem
If the user has not accepted the category I want to show some text (preferably reuse some text from the cc settings?) and buttons (accept this, accept all from twitter) on the location of where the twitter flow would have been.
What is the best/some good way to achieve this?
My current attempt
NOTE: This uses jquery and is a piece of code that is run after cc has been loaded and "this" is a container around the html-code above.
		var self = this;
		this.$el = $(element);

		if (!cc.allowedCategory('twitter')) {
			var newDiv = $('
' +
				'
A twitter feed will be displayed here if you allow that. Twitter will set cookies.
' +
				'<button class="allowall">Always allow twitter feeds</button>
');
			newDiv.find('.allowall').on('click', function (e) {
				e.stopPropagation();
				e.preventDefault();
				cc.accept('twitter'); // This will trigger the js to be activated
				newDiv.remove(); // Hide button and disclaimer
			});

			this.$el.append(newDiv);
              }

PS I am currently implementing a solution for us based on this library. That you for making it!

A second question, based on my comment to reuse texts.
cc.getConfig('current_lang');
lets me read a config value.
But if I have the category and want to read the name and description of a category in the currently active language? (something also defined in config). Is that possible, and if so, how?
(sorry for adding multiple things in in the same issue, just tell me and I can make another one)

Hi @melker,
I think this task should be entrusted to iframemanager, since you're trying to load an iframe. Unfortunately, the plugin is still in its very-first release and doesn't support yet “custom iframes” generated by such as:

This aspect will improve in v1.1 (work-in-progress).
.getConfig() returns only configuration options which are stored in the internal _config variable, and you won't be able to retrieve the languages object or any of the callbacks. This can be easily changed though!

As of v2.8, .getConfig() will now search on the input config. too:
https://github.com/orestbida/cookieconsent/blob/7bb43cf12bfcb3598fd9451744c55af2c6b6db58/src/cookieconsent.js#L1793-L1795

Thanks for replies @orestbida
About the iframe - ok, I might add some issue on iframemanager, (but currently we are not using that as we want some other initial behavior, I might add a feature request for that too.)
Reading config, works good!

Just a side note:
Perfect: I also realized that I may add a few extra strings elements in the config to use in the blocks on the pages (for my approval buttons there)


sort_primitive result is capped to the min of limit or values.len

Closes #235
What changes are included in this PR?
The result slice in sort_primitive is the size of all the values being sorted even when a limit is provided. Change the size to be the maximum possible result (thus avoiding the panic in #235)
There are other slice size errors as well (such as when the number of nulls exceeds the limit) that are included.
Are there any user-facing changes?
No
Notes
I've provided a reproducing test, but skipped other variants of limits /w nulls. Those should be easy so let me know if they're of interest. Additionally it is possible that result is intentionally over-large in which case I can try alternative fixes.

Codecov Report

Merging #236 (7bc3523) into master (d008f31) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #236   +/-   ##
=======================================
  Coverage   82.51%   82.51%
=======================================
  Files         162      162
  Lines       43655    43664    +9
=======================================
+ Hits        36022    36030    +8
- Misses       7633     7634    +1

Impacted Files
Coverage Δ

arrow/src/compute/kernels/sort.rs
94.37% <100.00%> (+0.04%)
:arrow_up:

parquet/src/encodings/encoding.rs
94.86% <0.00%> (-0.20%)
:arrow_down:

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 d008f31...7bc3523. Read the comment docs.

BTW we are working on the Dev PR / Process failing check in #242

Thanks again @medwards !


Expand to multiple instances?

Is there a way to run this on multiple instances working on the same data set? like EMR?

Yes, if you set up an MPI cluster, it should work.  The framework detects the lack of P2P connectivity between the GPUs in play and automagically switches to MPI system memory collectives.
However, at 10 Gb/s between g2.8xlarge instances plus ~2-3 GB/s upload/download because virtualization, you will need a rather large layer to get efficient scaling.  That said, if you just want to run large models, it should just work.  It has, of course, never been tried.
What will be interesting down the road is running this on RDMA-enabled servers that allow inter-system P2P copies.  That will need a few more lines of code, but it also ought to work.

Cool, thanks! I finally can run Skynet on AWS.

As @scottlegrand mentioned this is build on MPI from day one and you should be able to run across instances for Training.  We are working on model parallel predictions across machines
If you are using Multiple Instance please ensure that all your nodes are in the same placement group.
I would try Multiple EC2 instances on the same Placement group connected by Elastic File System so that  you can share the data across the instances
Sharing the data on HDFS with EMR will be tricky I guess

@Ins1ghtLabs, did the responses help you? If so, I'll close this issue.

perhaps we should add this comments to the FAQ before closing this issue so other users can find it

Added to Q&A


FancyZones change between editor and real world use

Environment
Windows build number: 1909 (18363.900)
PowerToys version:  v0.19.1
PowerToy module for which you are reporting the bug (if applicable): Fancy Zones

Steps to reproduce
I have a notebook with a second display attached. It's a 4K panel.
I have set my FancyZones to have 3 columns.
Expected behavior
Snapping should work and look like set in the editor, even on sedondary displays.
Actual behavior
When trying to snap to the columns of the FZ suddenly all columns begin in the middle of the display (= at half the height), but they extend below the physical limit of it, so the total height is correct, just offsetted down.

I have a similar behaviour with my task bar on the left side, Windows 1909, PowerToys up to date.
The fance zone is moved to the left with the size of the taskbar, once I close the editor

the behaviour that I described in my previous comment is fixed in v0.19.2

This is fixed on my primary display with v0.19.2 but not on my secondary display.
Taskbar down left-hand side of both displays, secondary display is running at 4k, primary at 1080p - zones on the secondary display now appear to be offset to the right only by the width of the taskbar on the primary display (i.e. half the width they need to be offset by for the zones to be aligned with where the taskbar is).
Zones appear correct in editor but when dragging windows with shift held down, can see the zones are off to the left of where they should be.

@craiglowson
can you provide screenshots?
Thanks.

@enricogior Think it is working following full reboot - had it doing the above on both displays while on v0.19.1 so updated to v0.19.2 per above and it was only failing on second display following the Explorer restart as part of PowerToys update (and setting/resetting layouts). Full reboot and can now no longer replicate.

I have to admit I can't replicate it now either... 🤔
So if no on else has this problem I deduce this issue could be closed.

@runxel @craiglowson
thank you for the feedback. I'm closing the issue, but please feel free to reopen it if the problem comes back.
Thank you.


Gael/feature bypass permissions generate

Closed due to history rewrite to make sure my personal GH account is author of commits
See https://github.com/gaelmotte/gmotte-sfdx-plugin/pull/42


[feature-request] Improve syntax definitions to allow more highlighting options

The alsyntax.tmlanguage file contains only 17 patterns, with only 1 (procedure/trigger) real syntax pattern.
With few ways to change the code highlighting, this makes writing AL code in Visual Studio Code seems like writing code in a text editor.
Is it possible to extend the alsyntax file, or perhaps a repository to contribute?

Great idea! Go ahead and post this to our Ideas forum at https://aka.ms/BusinessCentralideas, or vote up the idea if its already there. We're constantly monitoring top Ideas and will consider them for a future release.


Add setting to filter entities by tags

There should be ways to only show or hide entities with certain tags

I don't see a necessity for this anymore, maybe it comes back. I will close it for now.