Passwordless authentication does not work.

I have been trying to use the passwordless authentication, but the library doesn't support passwordless authentication.

@oguduvictor
You will need to change the template for your hosted login page. In the Dashboard go to the Hosted Pages menu, and for the Login page select the option Lock (Passwordless). This will change the hosted login page to use Passwordless instead.


feat(ui): Add ability to build UI without cast

With this, if you build Shaka Player with "-@cast", the UI will contain a dummy cast proxy that does not require the full cast system.

Actually, thinking about it, it might be best for the dummy cast proxy class to be in a new folder. Something like conditional/dummy_cast_proxy.js.
That would simplify the process of integrating this into the google codebase, since we won't need to start excluding the class from a bunch of build targets.
I'll give it a shot...


ENH: add get_gdal_data_path to query GDAL data directory

Fixes #159
I made the small function I tested when commenting about this on the geopandas repo into a PR.

Since it's only a tiny helper function, I suppose it can be included in 0.4.2?


Half up rounding

Towards infinity rounding for fixed_point datatypes with only values midway between two representable values move in the given direction. Other values move to the nearest representable value. The tag is named as "towards_infinity_rounding_tag".

@hbe72 are you able to add johnmcfarlane/cnl as a remote and to push to that?
git remote add upstream git@github.com:johnmcfarlane/cnl
git fetch upstream
git push -u upstream half_up_rounding

Happy to push this for you but for next time, if you're able to push directly to a feature branch in johnmcfarlane/cnl then CI should work for you.

I made a couple of comments in the code. LMK if you don't see them.
I think I may have misunderstood the intent of this mode. I'm expecting something which behaves the same as ceil for +ve values and floor for -ve values.
Also, is this not the opposite of what you're aiming for? Don't you want the same behavior as right-shift? If so, I would think you want to round toward -ve infinity.

I see where my confusion arose. Previous discussion, #405, refers to a fixed-point behavior which is optimised for use of shift operator. That's what I assumed you were talking about. I thought perhaps you were describing this as 'add 0.5 then round to nearest' but you're actually looking for 'round to nearest or to +ve infinity where there is no nearest'.
So sorry but towards_infinity_rounding_tag isn't right after all.
I previously claimed that your desired rounding mode was round_to_infinity in P0105R1 but now I think it's actually tie_to_pos_inf. Perhaps you can confirm that and consider this name. Alternatively, 'round half up' or 'half round up' might be better.
I'm frankly indifferent to all of them. You need to know what tie and all are before Lawrence's enumerators make sense. Alternatively, I don't see a citation for terminology involving 'half' and it doesn't leap out as obvious. In the end, I think this is quite a niche behavior: look at all of P0105R1's options. It's a lot of obscure customization!
Please do the honors and pick one. :)
Longer term, I think whatever you chose is likely to become an alias for a parameterised tag of the form:
namespace cnl {
using half_up_rounding_tag = nearest_rounding_tag<positive_inf>;
}

One more correction: nearest_rounding isn't as well defined as you think. It's more like round_to_nearest_but_dont_care_how. In that sense, it's a bit more like the "I'm not picky" option from P0106R1!

I think now we are on the same page ;). I'm exactly looking for 'round to nearest or to +ve infinity where there is no nearest'.  I just did not have the correct (exact) vocabulary to describe it.
I had previously missed the end of the P0105R1, the enumerations, and now the rest of the paper makes more sense. tie_to_pos_inf is the correct choice. Is it then tie_to_pos_inf_rounding_tag?

Yes, that's fine.

Duplicate. Closed.


Why are so many class XMLUtil members inline?

I see that even StringEqual() is declared inline. Is that necessary? What would happen if it was inside the .cpp?

There was some intent for performance, but it went too far. And clutters the headers. I'd support moving the larger ones (StringEqual) inline, but leave the small ones (IsUTF8Continuation.)


Refactor wasmtime-wasi crate

Changes:

uses Rust edition 2018
returns wasm32 errno directly rather than relying on
wasi_common::{host, memory} modules
wraps extraction of memory and WASI context in a macro

@sunfishcode before we go ahead with this PR, here's my thinking (please lemme know what you reckon): I'm currently in the process of providing some basic docs for our wasi-common crate cranestation/wasi-common#77, I remembered that in order to facilitate better error message logging in wasmtime-wasi, I've had to make wasi-common::{host, memory} public. But with the changes included in this PR, this is no longer the case. Now my question is, do we want to leave the said modules public in wasi-common or not? If yes, then I'd suggest we figure out how to handle constants repetition between wasi_common::wasm32 and wasi_common::host as I'd hate to copy-paste all the docs between the two ;-) I'd love to hear your thoughts on this!


Adding HuggingFace project files to aims master

Sudhanshu, we have to go with git clone rather than gdown as gdown from terminal won't work unless we install it using a package manager like brew or yum or apt-get.
Therefore, please update the README with:

Step 1: Cloning AI-Makerspace repository using:
git clone https://github.com/DigitalProductschool/AI-Makerspace.git

Remove Step 2
Make Step 3 as Step 2 with following command:

Step 3: Changing working directory to HuggingFace
cd AI-Makerspace/HuggingFace/

Don't forget to update the each step count i.e. when Step3 will become 2, all the step counts will need to be updated as well.

Added Note: Another way of using gdown is via python but again that would require people to first download the python file and run it locally to get the Google Drive data which will increase the complexity.

Sudhanshu, we have to go with git clone rather than gdown as gdown from terminal won't work unless we install it using a package manager like brew or yum or apt-get.
Therefore, please update the README with:

Step 1: Cloning AI-Makerspace repository using:
git clone https://github.com/DigitalProductschool/AI-Makerspace.git

Remove Step 2
Make Step 3 as Step 2 with following command:

Step 3: Changing working directory to HuggingFace
cd AI-Makerspace/HuggingFace/

Don't forget to update the each step count i.e. when Step3 will become 2, all the step counts will need to be updated as well.

Okay I'll update it


[BUG]: Machine Operator "Custom Name" Field Not Saving Unless Facings Change

Bug Description
When setting up a Machine Operator, the "Custom Name" field for the machine doesn't save unless you modify one of the facings afterwards.
But modifying one the the facings also seems to revert it to the previously saved name, as long as that previously-saved name is longer than the current name, which means that it's also not possible to shorten a machine's custom name.
Logs and Crash Reports
None
Screenshots and/or Videos
No response
Bug Reproduction

Have a storage system and summon a machine operator.
Use the Book of Calling, in "Set Managed Machine" mode, on the machine you want to manage.
Change the Custom Name, but not any of the facings.
Close out of the GUI, either with Escape or with the X button.
Finish setting up the Machine Operator.
Look at the Managed Machine in the storage system. Its Custom Name is not present.

To get the Custom Name to update, add in a step between 3 and 4:
3.5. Change one of the facings.
Expected Behavior
The Custom Name should save, at least, when the "X" button is pressed, without having to change a facing.
Minecraft Version
1.18.2
Mod Version
1.67.2
Modpack
Other

hmm yes it should definitely save.
Weirdly enough I did that just yesterday while testing and it did work. I will investigate


draft zeebe api tutorial

THE FINAL TUTORIAL.
This one was a little trickier with the request body, but let me know where I messed up.
Related to, and closes out https://github.com/camunda/developer-experience/issues/142.
Any future API tutorial work should be created with a new issue.
@pepopowitz I'll resolve merge conflicts once Modeler is approved and merged in.
Will draft docs upon approval.

Wahoo @pepopowitz! I'll draft up the docs on Monday.


Inconsistent datatype in args

may be link with #5
seem that if there is more than one element in an array ints are not parsed
(if there is only one int it is parsed)
here a strace of a program with a select
 > ./strace a.out
select(255, [0], [1 2], [0 1 2], {tv_sec=0, tv_usec=962357}) = 2 (out [1 2], left {tv_sec=0, tv_usec=962344})

which will produce
{
   "syscall": "select",
   "args": [
      255,
      [
         0
      ],
      [
         "1",
         "2"
      ],
      [
         "0",
         "1",
         "2"
      ],
      {
         "tv_sec": 0,
         "tv_usec": 962357
      }
   ],
   "result": "2 (out [1 2], left {tv_sec=0, tv_usec=962344})",
   "timing": null,
   "pid": null,
   "type": "SYSCALL"
}

if we look at the args array, the second parameter is an array of integer but the third parameter is an array of strings.
Thanks for this module it's really useful.

i might have find a solution but it's not really a nice one ...
in the select when you have an array, elements are not separated by comma.
so in the grammar, i change that:
array_elements
  = (
    head:data_structure
    tail:(',' _ value:data_structure { return value; })*
    { return [head].concat(tail); }
  )?

by (replace ',' by ' ') :
array_elements
  = (
    head:data_structure
    tail:(' ' _ value:data_structure { return value; })*
    { return [head].concat(tail); }
  )?

and it seem to work, all the ints are parsed now.
but i think there might have a lot of side effect with this way.


Set maps common version

Set at the latest released version of Maps-Common

API change check
API changes are not detected in this pull request.


Time Traveler's rewind animation is not working

And again, Time Traveler. In the beatmap folder you can find time travel dir with the rewind animation. It doesn't play for some reason, and I don't think it's render lag because the cursor moves smoothly.

This may be due to #29, I'll investigate

The bug fixed itself after switching to osu-parsers

The rewind animation is offset in time due to #29


fix: #1191 Confirmation popup has too many white space

added question sign

other styling is on common repo.

Hi Jota, it looks good, just wondering if this delete confirmation popup is a little big?

and shall we remove this <br />? I forgot to remove it in the past, thanks!!

I still feel the popup width is too big... let's add the change in FAM, set the .p-dialog width to around 650px (need convert to rem), I feel that looks better

VS

How you feel about this?

fyi, I published the theme package v1.8.6, let's install the latest version, thanks!!

I still feel the popup width is too big... let's add the change in FAM, set the .p-dialog width to around 650px (need convert to rem), I feel that looks better  VS
How you feel about this?

did you test with the changes in max-width from the common repo? 650px makes it slightly bigger

I still feel the popup width is too big... let's add the change in FAM, set the .p-dialog width to around 650px (need convert to rem), I feel that looks better  VS
How you feel about this?

did you test with the changes in max-width from the common repo? 650px makes it bigger in smaller resolutions

no I didn't try it on a smaller screen, shall we just make it a little bit smaller?


Fix empty id management and message type

I've split the test into a sse_test subpackage: do you have any objection about it?

@jadr2ddude I'm done with this PR.
I would really appreciate if you could review it :-)


Switched docs theme to pytorch-theme

Deployement to check the rendering:

https://vfdev-5.github.io/functorch/

@vfdev-5 sorry, looking at the rendering reminded me of the actual reason why I didn't use the pytorch theme. The top bar links to various things that are unrelated to functorch!
For examples, tutorials links to PyTorch tutorials, not functorch tutorials...

Ok, I can customize that quickly. Do you want to change the topleft logo by something "functorch" ?

Does the theme look OK if we delete the entire top bar? (and the logo?)
Also, there are things missing, like the + in the below image:

I won't remove the header just replace titles and links. Like in pytorch.org/ignite
As for toctree and list expansion I can check that.
On the other hand I do not insist on switch, if you prefer rtd theme, let's just refresh it

I won't remove the header just replace titles and links. Like in pytorch.org/ignite

That SGTM, I didn't realize it was easy to replace those.

As for toctree and list expansion I can check that. On the other hand I do not insist on switch, if you prefer rtd theme, let's just refresh it

I think it'll be good to be consistent with the PyTorch theme, especially because eventually we do want to make sure our docs render nicely under the PyTorch theme. So let's try to make the pytorch theme work for us if it is not too much work.

Ok, I can customize that quickly. Do you want to change the topleft logo by something "functorch" ?

That would be nice, if we had a graphic designer lol.

@zou3519 I updated the docs:

added vjp entry
added functorch text logo


pull request for e2e test at 12/26/2016 22:45:13

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Dummy Bank and Manual Transactions

Hi is it possible to add feature for manual transactions (also recurring) and Dummy Banks(Accounts) or Budgets .
I would like to use the app but my Bank does not support Saltedge and Plaid, and no CSV export unfortunately.
Thanks

I wrote this project specifically to have a tracker that didn't have any manual input or budgeting whatsoever, just a purely automatic expense tracker - apologies that it doesn't align more with your interests, I'll reopen this issue if the project ever moves in that direction.


HydroDyn user documentation update for large platform rotation

Feature or improvement description

This PR updates the HydroDyn user documentation to reflect the recent input file changes for the modeling of large platform yaw motion.
It also updates error messages, variable descriptions, and comments in HydroDyn to match the finalized implementation of large platform rotation.

Related issue, if one exists
This is the documentation update for PR #2203
Impacted areas of the software
Docs, HydroDyn
Test results, if applicable
The changes have no impact on the results.

@andrew-platt Looks like we have the wrong tag here.

Whoops.  I was looking at something in AeroDyn when I set the tag...


Get @brief-response test working again

This pull request updates the functional tests so that the @brief-response scenario works again.
It does not:

fix the brief creation test
provide a general way to fill out fields on a page that has hidden fields
fix all of the general radio choose() or checkbox check() functions

I've been working on the last one (there are a few functions in common_steps.rb around clicking random things that we're not using, it looks like), but then removed all that code for a follow-up pull request.

👍
Nice work Paul. Thanks on the very easy to read commit messages too. It might be good to mention in the PR or one of the commits why we are doing this (because the new frontend toolkit has a bunch of hidden fields).
Is your plan to afterwards fix all of the bullet points you mentioned above or just the last one?

Plan is to fix them all, @idavidmcdonald.


docs(zh-cn)： Reviewed No.24  - The Trainer API

#390

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Problems with constructors and setting with unique construct

in the example below, there is an issue with the second constructor
generated for A. the constructor just assigns aName to name instead of calling
setName(aName) as is done in the first constructor. because of that, there is
no check for duplicate names and the hashmap asByName is not properly updated.

class A {
   1 -- 1 B;
   unique name;
 }

 class B {
   anotherName;
 }

Similarly, if a
state machine is defined for A, then the second constructor also does not set
the initial state.
The above was communicated by email from Gunter Mussbacher

there is another issue with the setName(aName) method. assume that an
instance has "myName" as the value for the name attribute. if setName(aName)
is called and the same name, i.e., "myName", is passed into it, setName(aName)
fails, because it detects that "myName" already exists in the hashmap, but
does not realize that it belongs to the same instance that is being changed.
this issue occurs for any unique attribute, i.e., it also occurs for a simple
model with one class and one unique attribute.

For point 1, this is a feature interaction between

the 1-1 association capability that constructs both objects to be linked woth the 1-1 association at the same time
the unique capability that prevents duplicate objects from being created. See constructor_AttributeAssignUnique.ump which is invoked from constructor_DeclareDefault.ump:177

When both features are present, aspects of both constructors need to be present.


Incorrect Visual Studio version information for 17.4.5

The recent update for the release of VS 17.4.5 on February 14, 2023 was entered incorrectly as 17.4.4. There are now two entries for 17.4.4. The most recent version number should be 17.4.5.

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

ID: 7ea27919-8e31-6592-9b9e-9a0cabb0a7ff
Version Independent ID: 093eeeb2-0306-4af9-2bfc-6d4eefbebf9b
Content: Visual Studio build numbers and release dates
Content Source: docs/install/visual-studio-build-numbers-and-release-dates.md
Product: visual-studio-windows
Technology: vs-installation
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@craigktreasure Thanks for letting us know and for the PR, too, which will get merged and then published today.


List of CSS Themes for RemarkJS

I'm starting to work on a CSS theme framework. My idea at the moment is to separate the layout and styling of the text and other items.
Slide layouts are a fairly typical feature of presentation themes in any presentation tool such as PPT or Keynote, for example:

Title and full-width content
Image and caption
Title, left column for text content, right column for an image

And so on. My thought is to create those layouts in one set of CSS as class names you can apply to a slide, and then use another set of CSS classes to change things like color, font, bullet stylings, and so on. Any thoughts on this topic are welcome.
Initially I'm simply going to list other Remark slide styles I've found. Please comment with more:

my own
https://github.com/sfc-arch/remark-theme
https://github.com/yihui/xaringan/blob/master/inst/rmarkdown/templates/xaringan/resources/default.css
Presla

xaringan comes with more than the default CSS: https://github.com/yihui/xaringan/tree/master/inst/rmarkdown/templates/xaringan/resources

I've been working on the theme I had in mind, for a while; and I decided to just leave it as a part of my Hugo static site blog theme. I've published that blog theme's demo site at https://story.xaprb.com/ and you can see the built-in Remark theme's capabilities at https://story.xaprb.com/slides/
I'm going to close this issue since it doesn't seem like it is going to be "fixed." However there's no harm in adding comments with more themes, so I encourage anyone who wants to, to add more information to this issue even after it's closed!

@xaprb: Could apron be used with xaringan or is there more than CSS/JS to it?

Apron is pure CSS and could be used alone. I don't know xaringan. I built Apron so that I don't need to use any special markup, but the compromise I made in order to do that, is that it assumes certain conventions about the markup. For example, in some layouts it assumes the first content will be an image, and in others it assumes the first content will be an H1.

@xaprb: Would you mind sharing a direct link to the CSS and if I get it to work in xaringan to submit it as a theme there?

Sure: it is https://story.xaprb.com/css/apron.css or https://story.xaprb.com/css/apron.less
On the source github repo, it's https://github.com/xaprb/story/blob/master/static/css/apron.css or https://github.com/xaprb/story/blob/master/static/css/apron.less

I made a template for remarkjs that contains some themes, layouts, and components. I hope it works for you.
demo | source


[Identity] Allow configurable process timeouts

AzureCliCredential and AzurePowerShellCredential now allow users to pass in a custom timeout. This addresses scenarios where these proceses can take longer than the current default timeout values. DefaultAzureCredential now also has an optional keyword argument to allow users to pass in timeout values to the underlying developer credentials.
Closes: #27509
Note: The .NET identity sdk has this implemented already (https://github.com/Azure/azure-sdk-for-net/pull/28792) and variable names were based off of names there.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-identity

@christothes Hey, how's this look naming wise? Consistent naming across individual credentials, with the descriptive name for DAC.

@christothes Hey, how's this look naming wise? Consistent naming across individual credentials, with the descriptive name for DAC.

Looks good to me


Delete line shortcut will focus on the previous line instead of on the next line after deletion

In VS Code and IntelliJ, after deleting a line, caret will be put on the following line. But with this plugin, caret will move one line up.
Could this behavior be fixed to mimic other text editors?

I'm not seeing the behaviour where the cursor moves one line up. Could you please provide an example where it's doing this?

It looks like I had an old version of the plugin. Updating from 1.2.1 to 1.4.0 fixed it.

I also face this issue
When putting a cursor here

Note:
updated: [2023-04-02T22:48:14](https://hyp.is/xy3AdtFtEe2MiXtkpn06YA/www.thegioididong.com/game-app/top-7-phan-mem-web-kiem-tra-dao-van-online-mien-phi-chinh-xac-1312510)
highlight:: DoIT là phần mềm kiểm tra đạo văn trực tuyến và là sản phẩm của Trường Đại học Công nghệ thuộc Đại học Quốc gia Hà Nội
annotation:: [a](http://doit.uet.vnu.edu.vn/)

That one I can reproduce, will look into it. Appears to happen only when the line is long enough to be wrapped and your cursor is on the non-wrapped portion.

This should be fixed in v1.13.1


XDP fee on numeric asset creation

XDP Fees :

0.50 XDP fee for the initial issuance of a named assets (unchanged)
0.25 XDP fee for the initial issuance of a sub-asset (unchanged)
0.10 XDP fee for the initial issuance of a numeric asset.

looks good in production. Am seeing 0.1 XDP charged for numerics and no fee on additional issuances.
[2023-12-22 19:55:14][INFO] Issuance: DKS27K7w1trFpSYFU2kLaWPjDKfHF7guC1 created 0 of indivisible asset A9332489510908800000 (70e07e6e0c323ba5bb15de0fb099bea05dec7985acab731e8cb6ecb9e1ee5dfb) [valid]
[2023-12-22 19:55:16][INFO] Block: 5017002 (4.29s, hashes: L:e8d29 / TX:2ae68 / M:3316b)
...
[2023-12-22 19:57:56][INFO] Issuance: DStCmPF7iHzyrWCqVnF9bJeuQwrcdcFcHs created 0.0 of divisible asset DOGECOINNEWS (bc47adce70a1e1b3cc5684234cc9148ebdb925b71bbc85139bcbb6da0e9d7277) [valid]
[2023-12-22 19:57:57][INFO] Block: 5017006 (4.31s, hashes: L:ba777 / TX:39789 / M:f9ed8)
```


feat: Add RSpec/ContextMethod rule only as warning

What is the goal?
Improve clarity, readability, consistency, and avoid ambiguity in our RSpec tests by adding the RSpec/ContextMethod  rule.
🤓 Using describe and context methods explicitly communicates the code's intention more clearly, making it easier for other developers to understand the structure of the tests.

Currently, we have 186 offenses to this rule.
Is this a restricting or expanding change?
RESTRICTING change
References

Related pull-requests: https://github.com/sequra/simba/pull/8245

How is it being implemented?
I just added the rule to default.yml
Opportunistic refactorings
No
Caveats
I'll fix the 186 offenses in PRs like this one https://github.com/sequra/simba/pull/8245 after this rule is approved 🤞🏽 💕

I find that warnings are often ignored. If a rule does not throw a failure in the CI pipeline, we'll have a lot of cases where developers don't notice this.

I find that warnings are often ignored. If a rule does not throw a failure in the CI pipeline, we'll have a lot of cases where developers don't notice this.

@valdis Agreed. We have a card in our backlog to review this flow 👍

@valdis Thank you for pointing this out. @gorka-sq, Do you think we should keep it as info or do we make the CI pipeline fail?
Also, I'd be fixing most of the offenses I don't think it should be a problem to remove the info, I just added it to follow our guidelines 😊

@DaniPB some of our projects run rubocop in full-mode, that's why we decided to take this informational mid-step. For now we are keeping it, but we will review this process soon(-ish) to remove the bureaucracy it introduces.


Major refactoring of the GeoTables.jl package

This PR moves the Data interface and MeshData definitions and tests from Meshes.jl to GeoTables.jl and renames them to AbstractGeotable and GeoTable.
This PR also deprecates the functions: load, save and gadm, which were moved to the GeoIO.jl package, as well as removes their codes and tests.

I'm a bit surprised this was not given a major version increment. This is a massively breaking change. I now had to search through all PRs to find a mention of "load" to discover why following, .e.g., https://discourse.julialang.org/t/accessing-x-y-values-in-a-geopackage/103196/15 was failing at the very first hurdle. Poor discipline w.r.t. semantic versioning is a real pain to anyone depending on your package, and you should really be more careful.

Definitely an issue of the past. The ecosystem changed so much with so many
breaking changes already that the post from Discourse should be updated
instead.
The load lives in GeoIO.jl for a long time...
Em ter., 27 de fev. de 2024 19:26, KeithWM @.***>
escreveu:

I'm a bit surprised this was not given a major version increment. This is
a massively breaking change. I now had to search through all PRs to find a
mention of "load" to discover why following, .e.g.,
https://discourse.julialang.org/t/accessing-x-y-values-in-a-geopackage/103196/15
was failing at the very first hurdle. Poor discipline w.r.t. semantic
versioning is a real pain to anyone depending on your package, and you
should really be more careful.
—
Reply to this email directly, view it on GitHub
https://github.com/JuliaEarth/GeoTables.jl/pull/40#issuecomment-1967759983,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZQW3JGWMXZ3TVMEXKJTS3YVZMR7AVCNFSM6AAAAAA4BOTSQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRXG42TSOJYGM
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Indeed, can't change the past, but I was a bit annoyed by this. If the API is constantly changing, it should really still be in 0.x, where a step to 0.x+1 implies it is breaking. Let's hope it's stable now.

I'd like to ask you for a bit more care in the future. This a PR was breaking. Code from the README stopped working. (GeoTables.gadm). Why didn't you make this version 2.0?
Even if you made this 2.0 it would have been nice to provide guidance on how to migrate code in the README.

We added this migration information a long time ago in the README as far as
I remember. After months of the release we decided to remove from the
README.
If we knew gadm or GeoTables.jl was heavily used, we would be more careful.
The package has very few stars so we assumed that few people were actually
using it.
Em qua., 6 de mar. de 2024 07:14, Fabian Greimel @.***>
escreveu:

I'd like to ask you for a bit more care in the future. This a PR was
breaking. Code from the README stopped working. (GeoTables.gadm). Why
didn't you make this version 2.0?
Even if you made this 2.0 it would have been nice to provide guidance on
how to migrate code in the README.
—
Reply to this email directly, view it on GitHub
https://github.com/JuliaEarth/GeoTables.jl/pull/40#issuecomment-1980524090,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZQW3OMYW45CE2FNXCPLWLYW3UAJAVCNFSM6AAAAAA4BOTSQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBQGUZDIMBZGA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

As far as I can see the README was last changed in this PR with no migration information.
Could you point me to the package where GeoTables.gadm was moved?

GeoArtifacts.jl is the new home.
Em qua., 6 de mar. de 2024 09:18, Fabian Greimel @.***>
escreveu:

As far as I can see the README was last changed in this PR with no
migration information.
Could you point me to the package where GeoTables.gadm was moved?
—
Reply to this email directly, view it on GitHub
https://github.com/JuliaEarth/GeoTables.jl/pull/40#issuecomment-1980746071,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZQW3JEO5HLOB3OHCLK6B3YW4CQBAVCNFSM6AAAAAA4BOTSQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBQG42DMMBXGE
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Help solving SVG based captcha

The images you provided are PNGs. If the CAPTCHA is truly an SVG (which is essentially a text file) then you might be able to simply parse out the shapes that way without any image operations at all. I've even seen some CAPTCHAs label the shapes with the name of the letter or not combine shapes like the lines with the letters. SVG is probably the worst possible image format for making a secure CAPTCHA, but without examples of the SVG itself I can't really help.

@skotz The images are from this source: HERE
They are all in PNG, I managed to train some things, but I was not successful in positioning the parts for division.
I managed to remove the top lines, and separate the top part with a black and white color and the colored bottom parts (the colored part I couldn't remove);
OBS: Working only with PNG, according to the original captcha.


Add package wayfire.

Motivation for this change
Wayfire is a wayland compositor. The wayfire package itself is available as package in nixpkgs-overlays repo but it doesn't have the packages required to give us the gui desktop. So this pull request was made. Also if possible we can have modules for wayfire like sway or gnome3.
Things done

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

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

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

I am using the wlroots in the submodule because wayfire isn't compatible with wlroots in nixpkgs. So we can create a specific version of wlroots for wayfire or use the submodule.

I am now using wlroots from nixpkgs for building wayfire.


Buffer length from a BIO

Currently, i have a filled BIO pointer 'p'
i would like to know if there is any BIO function that would return the buffer length (say 'buf_len') from this BIO pointer which can later be used in the BIO_read() function.
Ex: BIO_read(p, data , buf_len);

It's a little unclear what exactly you are asking, since your question assumes that all BIOs have a buffer. This is not the case. However, some certainly do.
Probably what you want is the BIO_pending macro. The documentation says this about it:

BIO_pending(), BIO_ctrl_pending(), BIO_wpending() and BIO_ctrl_wpending()
return the number of pending characters in the BIOs read and write buffers.
Not all BIOs support these calls. BIO_ctrl_pending() and BIO_ctrl_wpending()
return a size_t type and are functions, BIO_pending() and BIO_wpending() are
macros which call BIO_ctrl().

https://www.openssl.org/docs/man1.1.1/man3/BIO_pending.html

OK
Thank you


[VS Code] Joining workspace: Unable to connect to the host. Verify both you and the host are online.

Error:
Joining workspace: Unable to connect to the host. Verify both you and the host are online.
Steps to Reproduce:

Version Data

extensionName
VSLS

extensionVersion
0.3.954

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.29.0

platformName
MacOS

platformVersion
17.7.0

@zoelalala  Thanks for reporting the issue. Did restarting the collaboration session help to resolve the issue?
Was this a one off issue, or does it reproduce consistently? Consolidating this with https://github.com/MicrosoftDocs/live-share/issues/327 Please add your comments there.


Teams Bot Node.js - send a proactive message to a particular person that is not part of conversation but has app installed?

Hello
how can i send a proactive message to a particular person that is not part of conversation but has app installed. all i have is that person's email address. can teams bot application send him/her a proactive message by just knowing email address?

@sanasz91mdev -Sending a proactive message is different from sending a regular message. There's no active turnContext to use for a reply. You must create the conversation before sending the message. For example, a new one-to-one chat or a new conversation thread in a channel. You can't create a new group chat or a new channel in a team with proactive messaging.
Reference Document-https://learn.microsoft.com/en-us/microsoftteams/platform/bots/how-to/conversations/send-proactive-messages?tabs=dotnet

@sayali-MSFT alright, i have a case where i need to send a proactive notification to a team's member that a ticket is assigned to him/her. I might assign ticket in a conversation with bot , but the person to whome ticket is assigned should be notified
how may i achieve this? ... again, i just have email address of Team's member to whom i assigned ticket.

@sanasz91mdev - Currently, Teams doesn't support sending proactive messages using email or User Principal Name (UPN).
Reference document- https://learn.microsoft.com/en-us/microsoftteams/platform/bots/how-to/conversations/send-proactive-messages?tabs=dotnet#send-the-message
Could you please help by suggesting this feature on - Microsoft Teams · Community

ht assign ticket in a conversation with bot , but the person to whome ticket is a

@sanasz91mdev - Currently, Teams doesn't support sending proactive messages using email or User Principal Name (UPN).
Reference document- https://learn.microsoft.com/en-us/microsoftteams/platform/bots/how-to/conversations/send-proactive-messages?tabs=dotnet#send-the-message
Could you please help by suggesting this feature on - Microsoft Teams · Community

So i cannot send proactive notification in my case? ... i can only send the notification IFF the person to whom i want to send proactive notification is part of conversation, right?

@sanasz91mdev -Yes. Correct.
You can create a new conversation with a user or a conversation thread in a channel and you must have the correct ID.

We are closing this issue for now, Please feel free to reopen it, if required.


rancher2.0 how to enabled logging

rancher2.0 how to enabled fluentd logging?

@deniseschannon  can you help me?

Pinging people directly within a day is not really nice. In your Cluster you can enable logging via Tools -> Logging and pick an option, hit Save to enable. In your Project, you can enable logging via Resources -> Logging.


form encoded cookie login

Hi, noob question. on the ofetch powered auth, how should loginWith("cookie", ...args) be used some those args get multipart/form-data encoded?
if I understand correctly ...args becomes endpoint so on nuxt.config.ts I have
strategies: {
  cookie: {
    cookie: {
      server: true,
      name: "session-cookie-name",
    },
    endpoints: {
      login: {
        url: `${apiPrefix}/login`,
        method: "post",
        headers: {
          "Content-Type": "multipart/form-data",
        },
      },
    ...
  },
}

and on login.vue
const app = useNuxtApp();

// form
interface Form {
  username: string;
  password: string;
  returnTo?: string;
}

const form = reactive<Form>({
  username: "",
  password: "",
});

const submitForm = ($event: Event) => {
  app.$auth.loginWith("cookie", { body: form });
  ...
}

but the form gets encoded as json instead.
and when using { data: form }  instead of body it doesn't send any payload with the POST.
what am I doing wrong?
Thanks

use:
app.$auth.loginWith('cookie', {
    body: new URLSearchParams(form).toString(),
    headers: {
        'content-type': 'application/x-www-form-urlencoded',
    },
})

Thank you


work around for windows path separator for status_file

part of #340. Suggested by https://github.com/rust-lang/cargo/pull/5820#issuecomment-409021860

👍 Thanks!


feat: snowflake query generator

Pull Request Template
Use this template to quickly create a well written pull request. Delete all quotes before creating the pull request.

Description

Add a TL;DR when description is extra long (helps content team)
Please include a summary of the changes and which issue has been fixed. Please also include relevant motivation
and context. List any dependencies that are required for this change
Links to Notion, Figma or any other documents that might be relevant to the PR

PR fixes following issue(s)
Fixes #24781

if no issue exists, please create an issue and ask the maintainers about this first

Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Chore (housekeeping or task changes that don't impact user perception)
This change requires a documentation update

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[ ] Manual
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/build-deploy-preview env=release

The base branch for this PR is feat/one-click-mssql-mysql because this was branched off of that branch. A PR is out for that branch once its merged we can merge this snowflake PR.

/build-deploy-preview

/ok-to-test

@sbalaji1192 no existing cypress for snowflake at all. I think its because there is no free version of snowflake . Tagging @Aishwarya-U-R for more clarity. Thanks!

@sbalaji1192 no existing cypress for snowflake at all. I think its because there is no free version of snowflake . Tagging @Aishwarya-U-R for more clarity. Thanks!

Yes, no snowflake since we dont have a free account/test account for the same.


reWriteBatchedInserts with executeBatch doesn't return the correct count

Driver version
v<IP_ADDRESS> . Guess happens in all versions
Redshift version

PostgreSQL 8.0.2 on i686-pc-linux-gnu, compiled by GCC gcc (GCC) 3.4.2 20041017 (Red Hat 3.4.2-6.fc3), Redshift 1.0.35480
Client Operating System

Mac
JAVA/JVM version

Java 11
Table schema

Any Schema
Problem description
When I try to set the reWriteBatchedInserts to true and invoke executeBatch() on the Prepared statement it does't return the updated count. All the array elements have a value of -2. This works fine if I don't set the reWriteBatchedInserts.
JDBC trace logs

N/A
Reproduction code
    Connection connection = DriverManager.getConnection(
            String.format(
                    "jdbc:redshift://%s:%s/%s?reWriteBatchedInserts=true",
                    "test",
                    "5439",
                    "test"),
            "test",
            "test");

    connection.setAutoCommit(false);
    Statement statement = connection.createStatement();
    statement.execute(" CREATE TABLE IF NOT EXISTS \"Repro_Batch\" ( \"c0\" VARCHAR(MAX) );");
    connection.commit();

    PreparedStatement insert = connection.prepareStatement("INSERT INTO \"Repro_Batch\" ( \"c0\" ) VALUES (?)");

    for (int i=0; i<100 ; i++) {
        insert.setString(1, "prefix-"+i);
        insert.addBatch();
    }
    int[] count = insert.executeBatch();
    connection.commit();
    int totalUpdates = IntStream.of(count).sum();
    assert totalUpdates==100 ;

-2 represents Statement.SUCCESS_NO_INFO. We don't have individual batch count right now. we will see if we get it from server and how to implement it.

@iggarish Thanks a lot for your reply. Another behavior I noticed was that if the batch has even number of records all of the response has Statement.SUCCESS_NO_INFO whereas if it is odd everything except the last has one -2 and the last one is 1. Is this expected and is this a documented behavior? Does reWriteBatchedInsertsSize have any impact on this?

executeBatch() API has this expectation:
Returns:
an array of update counts containing one element for each command in the batch. The elements of the array are ordered according to the order in which commands were added to the batch.
So you added 100 batch. Now because of reWrite, this actually becomes 3 batches (64, 32 ,4). So now driver can't return array of 3 elements. It returns array of 100 elements and saying that SUCCESS_NO_INFO for each of that element.

As such you can't do: IntStream.of(count).sum(). Because each element in return array can have one of the following value:

A number greater than or equal to zero
SUCCESS_NO_INFO
EXECUTE_FAILED

That was just a sample I had but when I had a batch size of 101, 100 of those had -2 as the return and the lats one had a return value of 1. This was happening only when the batch size is 1. Is this expected?

Yes when batch size is one then there is no specials logic and count should return for  that single execution.

Yes when batch size is one then there is no specials logic and count should return for that single execution.

If the batch size is 101 - It should get divided into (64,32,5 ), right? If yes, I would expected to see SUCCESS_NO_INFO for all the 101 records but I see SUCCESS_NO_INFO for 100 and 1 for the the 101th record which makes me believe that the it is broken into (64, 32, 4, 1). So, each batch size here is a power of 2. Is that how it works?

Yes each batch is power of 2.

But main issue is reWrite reduce the original number of batches provided by application. So we can't return actual count in original number of batches, because driver has count for each reduce batches.

We won't get count as needed in batch re-write. closing the issue.


[SPARK-13493][SQL] Enable case sensitiveness in json schema inference.

What changes were proposed in this pull request?
Pass "caseSensitiveAnalysis" to InferSchema. Change field name to lowercase when "caseSensitiveAnalysis" is false.
How was this patch tested?
Add new test case in DataFrameSuite.

Can one of the admins verify this patch?

Jenkins, test this please

What would be the case like in CSV? I don't think anybody want to conbine "user" and "User" column when "caseSensitiveAnalysis" is false.
`scala> val jsonDF = sqlContext.read.option("header", "true").csv("file.csv")
jsonDF: org.apache.spark.sql.DataFrame = [user: string,  User: string ... 1 more field]
scala> jsonDF.show
+-------+-------+-----+
|   user|   User| hits|
+-------+-------+-----+
|     om|  scala|  120|
| daniel|  spark|   80|
|3754978|  spark|    1|
+-------+-------+-----+`

@rxin Currently, do we have any policy to handle case-sensitivity inside spark? e.g., postgresql always holds names into lower case (https://www.postgresql.org/docs/9.5/static/sql-syntax-lexical.html) and h2 does into upper one. It seems there are many case-sensitivity issue in not only json, but also other other datasources.
unquoted names are always folded to lower case.

Can one of the admins verify this patch?

@maropu Yes. We need to consider case sensitivity for the built-in data sources at least.

@zuowang @maropu Are you still working on this? We should resolve this issue.

I'm not, so it's okay to do so.

I'm not, could you work to resolve it?

Let's close this if it is not in a progress.

We are closing it due to inactivity. please do reopen if you want to push it forward. Thanks!


Feedback for Samsung page

Disable adaptative battery too?  Text says disable all toggles but in image , adaptative battery is on

Hello Tiego, do you have some insight that this is really needed? In fact maybe adaptive battery is the ASOP adaptive battery implementation which we do not have reports it should be causing issues... many thanks for the details..

Yes, I believe that Adaptive battery can stay enabled.


is the project being maintained?

There was a commit a day ago by the creator on a feature branch.
https://github.com/appetizermonster/hain/branches

I see a bit of activity on that branch but not much more anywhere else (from months). Not trying to be obnoxious just asking

Hi, to be honest, it's not actively maintained. but i'm very struggle to commit something which makes hain better.
For now, Complex architecture of hain makes me hard to add some features. so I'm trying to refactor this first.

@appetizermonster I love this project (been wanting to build one myself for a few weeks until I found this) so thank you for the response! Not sure if/how can immediately contribute but would love to in the coming weeks.

Thanks!

@appetizermonster Should we wait until the refactor is finished? I may want to slowly start involving myself with things like simple plugins.

you can start making plugins immediately. there will be no changes for plugin interface.
but there will be only internal structure changes. so if you want to submit new feature PR, it may not be merged immediately.

so you'll accept it after the refactor is complete?

@rickmed Yes i will

I found this linked off of the electron homepage while I was working on a simple plugin system (think like this with no features and niceties, just the plugin context object)
I would be highly interested in working on this with you guys, let me know what kinds of work you would like done.
I'm:

mostly competent with
node
electron
powershell
highly competent with
.net/windows programming (recently acquired/used through work)
html/css
graphic design
typescript (if you guys want a rewrite lol)
client javascript (webpack/react/bower[although i just use npm now])

Im highly interested in contributing as well, fluent in JS and some electron.  What can we help with?
Is the refactoring complete?
Ive written my own plugin or hain already but dislike the preference UI thats available, not sure if there are any plans to change/improve that or not.

It seems there's a fair amount of people keen to contribute but the owner's lack of time available to commit is holding the project back.  I wonder if he'd be interested in allowing the community to drive it forward, maybe by endorsing a project manager who could facilitate the creation of a roadmap and/or design goals for the project, and to clean up the open issues and PRs.

Sorry for the late.
@cpriest Yes! the refactoring has completed after v0.5.0
@staff0rd
As you said, Currently I'm super busy. so i couldn't handle this project at the right time.
and Yes, I'd like to make this project open for the community :-)
however I have no idea how to make it open.
I'm very beginner for the open source project culture.
I would appreciate it if anyone could tell me any idea for the change.

@appetizermonster Maybe it would be a first step to create a GitHub organization and move to repo(s) to it. Then it's very easy to choose some project maintainers who can also merge PRs or make decisions.
A project where this model works very good is The Lounge. There all PRs have to be reviewed by 2 maintainers and then can be merged.
GitHub organizations are free for open source projects.

@toXel Thanks for letting me know a nice project model. I need to take a look at "The Lounge" project and Github Organization :smiley:

FTR, I'm working on the project as a contributor.  Most anyone (at the moment) can fork the project, build something and submit a pull request.  I'm looking through the pull requests and working on some of the tickets as well.

Some great news in this regard, @appetizermonster has made hain into an organization and it's being actively developed.  I'm working on a few tickets here and there and actively looking for additional help.  Going to keep this ticket open, if anyone wants to get involved or keep updated we now have a mailing list you can join to ask questions of those involved.
You can join by sending an email with subscribe in the subject line to<EMAIL_ADDRESS>

@cpriest I'd like to suggest closing this issue


Guaranteed Amounts vs Approximation

Our buying/minting form includes a disclaimer about the resulting OUSD being an approximate amount. Arguably, our selling/redeeming form should also include this.

This is an approximation due to possible movements in the exchange rate(s) for the stablecoin(s) involved in the transaction between the time that the form is rendered and the time that the transaction is constructed in the contract. If https://github.com/oplabs/origin-dollar/issues/90 were to account for depth of liquidity, then there could also be slippage. With the introduction of https://github.com/oplabs/origin-dollar/issues/110, there would be actual slippage.
My hypothesis is that, as we move toward a more mainstream userbase, the approximation of prices and lack of clarity might be a deterrent that causes people to abandon the transaction. Less sophisticated users might actually prefer a worse price if they don't have to think about price fluctuations. Uniswap addresses the slippage problem by promising a minimum output below and estimated output. We should consider doing something similar.
Alternatively, I'm wondering if it's possible for us to give the user the specific amount that they ask for, account for slippage in the rate, and then collect any bonus/excess resulting OUSD (or unconverted stablecoin) as revenue for either the pool itself or for governance token. This would create a small revenue opportunity and also give users a cleaner transaction that is easier to mentally process. I suppose this could be considered "OUSD as a market maker".
@matthewliu points out that this might involve significant amounts of money for users transacting with large amounts or people for whom a few dollars is very meaningful. Users who understand what's happening might be frustrated that they're losing money on their transactions. It would probably also depend on the APY that they're likely to enjoy and the gas costs associated with their transactions.

With the advent of https://github.com/OriginProtocol/origin-dollar/pull/390 and https://github.com/OriginProtocol/origin-dollar/pull/410, our contracts now have the ability to abandon mints and redeems if the output is less than expected. We'll implement a frontend config feature for the user to set a threshold for oracle price movement much like slippage parameters on AMMs. My alternate solution proposed above might make sense with more mainstream users who are less familiar with AMMs, but those aren't likely our target users with this DApp. We probably want them to just buy OUSD on centralized exchanges or with fiat onramps when the time comes.

With the advent of https://github.com/OriginProtocol/origin-dollar/pull/390 and https://github.com/OriginProtocol/origin-dollar/pull/410, our contracts now have the ability to abandon mints and redeems if the output is less than expected. We'll implement a frontend config feature for the user to set a threshold for oracle price movement much like slippage parameters on AMMs. My alternate solution proposed above might make sense with more mainstream users who are less familiar with AMMs, but those aren't likely our target users with this DApp. We probably want them to just buy OUSD on centralized exchanges or with fiat onramps when the time comes.

According to @sparrowDom, the DApp currently passes an amount that will cause the transaction to fail if the output would be 3% below $1 rather than 3% below the expected output. We'll design a way for the user to configure the threshold to a specific percentage and then update the calculation to be based on output instead of peg when we implement the user-facing improvements.

According to @sparrowDom, the DApp currently passes an amount that will cause the transaction to fail if the output would be 3% below $1 rather than 3% below the expected output. We'll design a way for the user to configure the threshold to a specific percentage and then update the calculation to be based on output instead of peg when we implement the user-facing improvements.

Here are the designs for allowing the user to see and change the threshold:

mint
redeem

Here are the designs for allowing the user to see and change the threshold:

mint
redeem

solved via: https://github.com/OriginProtocol/origin-dollar/pull/544


Question on SPIFFS_ERR_BAD_DESCRIPTOR(-10010) error

Hello spiffs team,
I am a beginner to the file system. I am working on Giga device(GD25Q20B) NOR Flash(2M-bit)(256KB).
The flash has

Block size: 64kB
Sector size : 4KB
Page size : 256 bytes

I integrated spiffs with SPI low level driver but facing an issue with reading and writing into the file. Low level driver is working fine. While creating the file, the reading and writing is not happening.
Error log
page index byte len:         256
object lookup pages:         4
page pages per block:        256
page header length:          9
object header index entries: 52
object index entries:        61
available file descriptors:  2
free blocks:                 0
mount res: 0
errno -10010
errno -10010
-->  <--

Please let me know how to resolve this problem.
@baboundu81 @igrr @maichaell
Thanks in advance for your help.
Example code: https://gist.github.com/saiprasadsamudrala/7f8a794786110565acd87222c069e735.

@pellepl


Map instantiation fails when map is created from shared dataset

Working on #9516 I faced a trouble related to the query and the map instantiation. Follow theses steps to reproduce it:

Share a dataset with write permission. Don't qualified the sql query (aka, include the username as part of the table name in the query).
Using the user that it's shared with, create a map from this dataset.
💥  Dashboard has some errors: Error: Map instantiation failed.

This is happening because the queries present in the vizJSON are not qualified with the owner's username.

This is hard to fix in the frontend (in the right way) because the map is initially instantiated in the deep-insight (and cartodb.js) side, from the data coming from the vizJSON. I tried to patch the vizJSON to fix the queries, and it solves the issue, but the solution smells. I feel the queries should come qualified in the vizJSON, in the same manner as table_name properties.

Of course, qualifying the query manually in the sql editor fixes the issue.

I forgot to ping @javitonino and @gfiorav, I think we will need your point of view with this.

Yes, I think this is a better fix in backend. We are doing a lot of query rewriting already anyway. In fact, I think this should be already be handled in some way, but let me check.

After looking at this, we are not doing query rewriting on map creation at all.

There are two behaviours right now:

When adding a new layer, the query is not copied
When creating a new map from a dataset, the query is copied (which could be wrong)

In fact, in the second case, everything is copied, queries, styles, infowindows, basemaps, etc. Basically, the canonical visualization is duplicated. This made a lot of sense in the editor, where canonical visualizations where editable, but makes less sense in the new builder.
I think we should try to make the two cases above the same, so adding a layer behaves the same way as creating a map from a dataset. So either we copy (and rewrite) the query in both cases or we don't do it at all.
Right now, I'm inclined to think that we should never copy anything from the datasets when creating a map (for the builder, keeping current behaviour for editor). This way, it always behaves as adding a layer and gets the default query. This is also consistent with adding tables in analysis nodes.
The other option would be to copy and rewrite the query always, but it has some complications with current implementation, most notable that the layer creation is done mostly by frontend (backend is just acting as a persistence layer here, and has no say on the process).

+1 to merge both cases into "add layer" one.

S/B: 9695778

Did we decide what to do with this one after all? @javitonino @juanignaciosl

First of all, in order to keep issues linked, this is somehow related to #9478. Not the problem but the implementation.
Answering your question, I think that there hasn't been an explicit agreement. Nevertheless, comments at both issues are aligned: let's do this (1) in backend, (2) during map creation (3) with the "add layer" logic.
Disclaimer: as you know I still don't love (1), but there's no clear winner and @xavijam and @javisantana have a broader vision on this
@javitonino , @gfiorav , agree?

Agreed. I dislike having add layer creating the layer options in frontend and map creation doing it in backend, but I think the best solution for doing thing at creation time is in the backend.
Note that for this issue we will have to check if the builder is enabled or not to toggle the behaviour, as we want to keep the old behaviour (copy map with TableBlender) for the editor.

Final decision after some offline discussion. On map creation we will reset the styles from the dataset, but keep the query that's been applied (in order to avoid surprises when going from the dataset preview to create a new map). So basically, we are just keeping the query, rewritten.
This is being done in #9749 (queries) and #9726 (styles).

:+1:


Add lint rule to detect '@artsy/palette' usage.

Description
This change adds a lint rule to detect when components are imported
from @artsy/palette. The rule also has the ability to automatically fix
any import violations.

Merging with the rule disabled pending further conversations when everyone is available to prevent this PR from lingering.


Endless 'The file has changed since it was last read' question

The editor is great for so many reasons.
I have been running into a issue lately. When I edit a logfile each time the logfile is updated the editor aks if the file should be reloaded. If I answer no or yes it doesn't matter the next time the file changes the editor will again ask if it should reload the file.
This becomes a endless loop that makes it impossible to view logfile's especially when the logfile changes frequently

Might be because of a plugin. I noticed that a little bit ago as well when working on some things, but it seemed to have stopped when I updated some things. Check for plugin updates.

The question is endlessly returns in case the file you are editing changes endlessly (like a logfile).
I have checked the code, I think the issue is caused by the call: v.Buf.CheckModTime() in view.go.
I see a view options to resolve the issue:

Remember the answer to the question and apply this each time the file changes,
Allow disabling the behaviour via the settings file
Some other method..

I’m seeing this now with the Truffle framework watcher running. I’ve got auto save enabled if that makes a difference.

Pressing esc when prompted to reload will now cancel the reload and all future reloads.


No module named 'importlib.metadata'

When trying to run your code example in colab I get:
---------------------------------------------------------------------------

ModuleNotFoundError                       Traceback (most recent call last)

[<ipython-input-9-eb27f3a312b1>](https://localhost:8080/#) in <module>()
----> 1 from aleph_alpha_client import ImagePrompt, AlephAlphaClient
      2
      3 client = AlephAlphaClient(
      4     host="https://api.aleph-alpha.com/",
      5     token="<your token>"

1 frames

[/usr/local/lib/python3.7/dist-packages/aleph_alpha_client/aleph_alpha_client.py](https://localhost:8080/#) in <module>()
      1 from typing import List, Optional, Dict, Union
----> 2 from importlib.metadata import version
      3
      4 import requests
      5 import logging

ModuleNotFoundError: No module named 'importlib.metadata'

Trying to pip install importlib or importlib-metadata do not help.

Hello @To3No7 ,
thanks for giving the API a spin. Not sure what exactly is going on here, but at least I do not think that we need the importlib dependency at all. We currently just use it to parse the API version of the service.
Cheers, Markus

It seems that there is a change in version 3.10 of this library "Changed in version 3.10: importlib.metadata is no longer provisional." https://docs.python.org/3/library/importlib.metadata.html
It seems that functionality is taken over by importlib-metadata https://github.com/python/importlib_metadata , but I do not know how to program your usage of this to cope with various versions of python and these libraries.
Will you update aleph_alpha_client.py so that it no longer depend on this library?

This is my intention. If my current understanding is correct, we only actually use the version type of this library to parse a version string of the http API in order to inform users that they may want to update the client in case the API has changed in a non backwards compatible manner.
Basically I think the use of this package can be replaced with a split.

Hi @To3No7,
we removed our dependency on importlib.metadata. A colleague just tested the latest release 1.4.2 in google colab and it worked. Please let us know, if you run into fuirther problems.


Test for endless loops

The endless loop we ran into in the iOS code would have been detected if we had a test where we connect to a peer, send data back and forth, disconnect and then try to connect again to the exact same peer. So we need to add that to the node.js level tests to prevent regressions.

This should get into story 0.0

Need a bit more detail please.

This was a test that we had agreed on adding way back when due to an early iOS bug.
The test was:
Step 1 - Connect to peer X
Step 2 - Transfer some data
Step 3 - Disconnect from peer X
Step 4 - Re-connect to peer X (and no other peer)
Step 5 - Send some data
We had a test that just did steps 1-3 and so we thought everything was fine. But then it turned out that if we had instead run steps 1-5 we would have found that bug. Since this pattern should be very common amongst our users this seemed like a good thing to test for.

So.. test I added in cle_fixes  to ThaliNativeLayer basically covers this scenario. I think we're good.


The ExtendedOverrides element is incorrect.

I am following this article, and it tells the user to:
Link the mapping file to the manifest
Immediately below (not inside) the  element in the manifest, add an ExtendedOverrides element. Set the Url attribute to the full URL of a JSON file in your project that you will create in a later step.
XML
Copy
...

The compiler in VS 2022 throws an error indicating that this element is not allowed in the xml schema. Also, the schema url is invalid. There is no file there.

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

ID: 4fee4083-e0cd-e483-a0d7-c3bd85042c52
Version Independent ID: 4973703e-1687-4f52-4718-8767aacf566c
Content: Custom keyboard shortcuts in Office Add-ins - Office Add-ins
Content Source: docs/design/keyboard-shortcuts.md
Product: non-product-specific
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

This is a known issue. Visual Studio has not kept the manifest definition files that it uses to validate the manifest up-to-date. You have to manually update them. Please follow the instructions in the following section and let us know if that fixes it.
Manifese schama validation errors in Visual Studio projects


Getting a format error, but only in Chrome.

I have a webapp that uses the Asciinema Player, and is working perfectly in FireFox and Safari, but not Chrome. It's the Uncaught: only asciicast v1 and v2 formats can be opened error at Line 1026 in asciinema.js.
It's the same app, loading the same cast. The only thing I can think of is that I'm creating the asciinema-player element programatically, as I need to set the speed, and setting it on the element after the element has loaded does not seem to work.
Ideally, there would be a writable speed property, but for now, does anyone have any ideas what would cause this error to be Chrome specific?

I am having the exact same issue.

Having the same issue. I'm using the example .cast file from the documentation.
<asciinema-player font-size="small" src="https://asciinema.org/a/28307.cast"></asciinema-player>

this demo ,you will get cors error absolutely


indexing a document with custom id

I need to index the documents with custom id. unfortunately I can't do it. is it possible in plastic?

check https://github.com/sleimanx2/plastic#partial-updating-a-document


Add datovka.app v4.10.0

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[x] Named the cask according to the token reference.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.
[x] Checked there are no open pull requests for the same cask.
[x] Checked the cask was not already refused in closed issues.
[x] Checked the cask is submitted to the correct repo.

P.S. Datovka is a free and opensource client for the Czech eGov data boxes.

Should this be in https://github.com/caskroom/homebrew-eid, then?

@adidalal not sure - it has nothing to do with electronic identity card software


Hr graph

Add an HR strip graph to the top of the page like the cadence graph on the bottom.  The zone colours match those from garmin, and can be set via the app settings options. There's some error checking to verify the string is ok, but you can't catch errors from string.toNumber()...which can be a problem.

Great feature!

Great feature!


EitherN heads-up

I'm working on my own crate https://github.com/cramertj/EitherN-rs to allow for Either over more than two types (e.g Either4 with variants One, Two, Three, and Four). I wanted to check with @bluss and the other contributors. As my library's functionality will share many implementation details with this one, I wanted to make sure no one here feels like I'm stealing their hard work. Let me know if you have any problems or would prefer I do anything differently. Thanks!

Ok. thanks for the head's up and don't worry about it.

Cool. Thanks-- I appreciate it.


fix: Capabilities for property setters to be manages instead of uses

:tada: This PR is included in version 1.7.0-next.6 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.7.1-next.1 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.7.2-next.1 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


155 image magic

Not ready for merging
These things need consideration:

layout of the pull and build buttons on the Images page
Sometimes kim can't find its build daemon - this is most likely a minikube issue

I'm assuming a bunch of the changes are from upstream dashboard? Otherwise forking this will get pretty confusing soon. They should go into a clearly-labeled separate commit, too, so that we can skip it in review.

Yes, I've put them in https://github.com/rancher-sandbox/rd/pull/173 -- so when that gets merged and I rebase this branch the PR should drop them.

I think the underlying kim error (failed to solve: rpc error: code = Unknown...) happens because a mounted volume got GC'ed. Has nothing to do with this PR

For the image pulling output (up to you if this should go here, in #180, or as an additional PR):

The textbox should be readonly.
The font should be fixed width, so that the bars drawn are readable.

I noticed setupmac.js was gone -- thanks for pointing out the kim download fell through the cracks.

textbox fixes done here as well.


[fix] polar-cartesian

Making a fix for pytest tests/util/test_coordinate_systems.py, which was failing because one of the angles wasn't yet implemented for as_polar.
@alexrockhill and I were trying to figure out which coordinate convention should be used here though. Proper https://en.wikipedia.org/wiki/Spherical_coordinate_system does not seem to match the common convention in ultrasound, as shown by the difference between:

and:
https://github.com/magnusdk/vbeam/blob/0da5675/vbeam/wavefront/plane.py#L15-L21
But I couldn't find a good, specific reference for the conversion between spherical coordinates (polar/azimuth) and ultrasound angles (azimuth/elevation).
Do you have a good reference & an idea of which should be used in vbeam?

Hi, thanks for looking into this. The plane wave code was implemented a long time ago, and I think I copied it from USTB. The truth is that I don't have a good reference of what should be used in vbeam, things have just sorta worked so far 😬 I will ask my colleagues on Monday about their opinions as well, and then we can hopefully merge your work early next week. I agree that getting this stuff right is important 😄

Thanks @magnusdk !
By the way, this preprocessing seems to work with vbeam for us to get the right transmit-times empirically, but not sure if the entire convention should be changed
    # from vbeam.util.coordinate_systems.as_polar
    # typically azimuth=atan(y, x) so y -> x, x -> z, z -> y
    # elevation, slight modification from:
    # https://en.wikipedia.org/wiki/Spherical_coordinate_system#Cartesian_coordinates
    # "If θ measures elevation... arccos... becomes an arcsin"
    r = np.sqrt(z**2 + x**2)
    azimuth_vbeam = np.arctan2(x, z)
    # Pending discussion on https://github.com/magnusdk/vbeam/pull/44
    elevation_vbeam = np.arcsin(y, r)

Hi, I've learned some things after discussing with colleagues. The convention used in the plane wave delay model is a version of the spherical coordinate system but with a different ordering of the rotations. Ordering matters when rotating, and in this convention, rotation around y is applied first. I.e.: we rotate the xz-plane first, then in elevation after.
The reason for the convention used in the plane wave code is historical, in that 3D coordinates were added after 2D coordinates (only after probes became 2D). E.g.: consider the 2D coordinates case:

To make this 3D, it made sense to "build upon" this rotation from the 2D case:

Where $a$ is azimuthal rotation in radians and $e$ is elevation rotation in radians. Azimuthal rotation is applied first.
I'd love to use the more standard spherical coordinate system if it is more widely recognized, but — from what I can tell — the convention described above is more used in the ultrasound context. F.ex. that's what USTB uses. So I think we should stick with the one used in USTB and for the plane wave wavefront function.
I still have some final exams that take up a lot of my time, but I will definitely be able to look into how we can merge your fixes while still keeping the current convention soon. I also have to double-check that this convention is used consistently and everywhere in vbeam.

Makes sense, thanks for clarifying @magnusdk ! I'll come back to this in a week or so to try to match that convention here. Good luck with exams!


Switch to B version of A/B test of topic and core mainstream formats

Remove the A/B test and always apply the boosts for topics (aka specialist sectors) and core mainstream formats.
https://trello.com/c/8xLTxEUQ/452-deploy-format-weighting-b-version
Marked as "do not merge" until alphagov/cdn-configs#26 has been deployed.

cdn-configs has been deployed, so I'm merging this now.


[TOPI][ARM] Improve injective schedule

The generic injective schedule does not have vectorization and is therefore slow on ARM CPU. With vectorization, it can run 2-3x faster. For example, for a upsample_relu layer with 48 x 48 x 48 (C, H, W), the vectorized code runs at 0.003 ms/iter compared to 0.008 ms/iter on raspberry pi.

@ajtulloch please review :)

Yeah. this is very useful likelayout_transform op.
@hlu1 How about (io, ii) = s[x].split(list(s[x].op.axis)[-1], 8) if s[x].op.axis[-1] <8 and how about poor arm cpu like A9 (maybe 4 is better)? I have done it like this:
  if len(s[x].op.axis) >= 5: # it is very useful when we have NCHWxC.
        fused = s[x].fuse(s[x].op.axis[0], s[x].op.axis[1], s[x].op.axis[2])
        #inner_out, inner_in = s[x].split(s[x].op.axis[3], factor=7)
        s[x].parallel(fused)
    elif len(s[x].op.axis) >= 3:
        fused = s[x].fuse(s[x].op.axis[0], s[x].op.axis[1])
        s[x].parallel(fused)
    else:
        s[x].parallel(s[x].op.axis[0])
    s[x].vectorize(list(s[x].op.axis)[-1])

We also need to consider the case of int8/unit8. For example, when you add two int8 numbers together to produce 1 int16 number, the simd width is 128/16 = 8. I think in general 8 should be a good compromise.

Ok. Could you add >=5 like we do it in x86? https://github.com/dmlc/tvm/blob/master/topi/python/topi/x86/injective.py#L26. This could help us in NCHWxC layout transform.

That should have been covered by:
if len(s[x].op.axis) >= 3:
        fused = s[x].fuse(s[x].op.axis[0], s[x].op.axis[1], s[x].op.axis[2])
        s[x].parallel(fused)

I used len(s[x].op.axis) >= 3 because it's needed for special cases like [1, 1, 224, 224].

Oops. I haven't see it clearly.

@hlu1 Could you help to see this discussion on discuss forum? https://discuss.tvm.ai/t/relay-build-target-rasp3b-something-wrong/2195 This issue should be related with this changeset.

@FrozenGene, thanks for letting me know. Fixed in https://github.com/dmlc/tvm/pull/3061


How to convert to onnx

Hello, I noticed that the pre-trained onnx file is provided, but I am not very successful in converting my own model, how do I convert the dino model to onnx format?

Has anyone ever had the same problem as me?

is there a specific error that you encountered?


[Server] Can't connect to bridge

Seeing the following traceback when running wemo server in my network:
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/gevent/greenlet.py", line 327, in run
    result = self._run(*self.args, **self.kwargs)
  File "/usr/local/lib/python2.7/dist-packages/socketio/virtsocket.py", line 403, in _receiver_loop
    retval = pkt_ns.process_packet(pkt)
  File "/usr/local/lib/python2.7/dist-packages/socketio/namespace.py", line 155, in process_packet
    return self.process_event(packet)
  File "/usr/local/lib/python2.7/dist-packages/socketio/namespace.py", line 225, in process_event
    return self.call_method_with_acl(method_name, packet, *args)
  File "/usr/local/lib/python2.7/dist-packages/socketio/namespace.py", line 240, in call_method_with_acl
    return self.call_method(method_name, packet, *args)
  File "/usr/local/lib/python2.7/dist-packages/socketio/namespace.py", line 282, in call_method
    return method(*args)
  File "/usr/local/lib/python2.7/dist-packages/ouimeaux/server/__init__.py", line 130, in on_statechange
    ENV.get(data['name']).set_state(data['state'])
AttributeError: 'Bridge' object has no attribute 'set_state'
<Greenlet at 0x7f30dbf990f0: <bound method Socket._receiver_loop of <socketio.virtsocket.Socket object at 0x7f30e001ac50>>> failed with AttributeError

Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/gevent/greenlet.py", line 327, in run
    result = self._run(*self.args, **self.kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1836, in __call__
    return self.wsgi_app(environ, start_response)
  File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1820, in wsgi_app
    response = self.make_response(self.handle_exception(e))
  File "/usr/local/lib/python2.7/dist-packages/flask_restful/__init__.py", line 270, in error_router
    return original_handler(e)
  File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1403, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1817, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1478, in full_dispatch_request
    response = self.make_response(rv)
  File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1566, in make_response
    raise ValueError('View function did not return a response')
ValueError: View function did not return a response
<Greenlet at 0x7f30dbf99050: <Flask 'ouimeaux.server'>({'HTTP_COOKIE': 'wanted_view=thumb; manage_view=th, <function <lambda> at 0x7f30dbfa3500>)> failed with ValueError

I can use wemo light ... on/off directly without an issue.

Forgot to mention: I have two LED lights attached to the bridge.

same problem here. I can use the api, but the web shows no devices. Same traceback as rafaelmagu

same issue here I have 4 lights connected to a bridge, can use them using terminal but web crashes when I click on the bridge and have to refresh the page and the switch works then with no issues unless I click on the bridge
error I am getting is
Traceback (most recent call last):
File "/Library/Python/2.7/site-packages/gevent-1.1b5-py2.7-macosx-10.10-intel.egg/gevent/greenlet.py", line 523, in run
result = self.run(*self.args, **self.kwargs)
File "/Library/Python/2.7/site-packages/Flask-0.10.1-py2.7.egg/flask/app.py", line 1836, in call
return self.wsgi_app(environ, start_response)
File "/Library/Python/2.7/site-packages/Flask-0.10.1-py2.7.egg/flask/app.py", line 1820, in wsgi_app
response = self.make_response(self.handle_exception(e))
File "/Library/Python/2.7/site-packages/Flask_RESTful-0.3.4-py2.7.egg/flask_restful/init.py", line 270, in error_router
return original_handler(e)
File "/Library/Python/2.7/site-packages/Flask-0.10.1-py2.7.egg/flask/app.py", line 1403, in handle_exception
reraise(exc_type, exc_value, tb)
File "/Library/Python/2.7/site-packages/Flask-0.10.1-py2.7.egg/flask/app.py", line 1817, in wsgi_app
response = self.full_dispatch_request()
File "/Library/Python/2.7/site-packages/Flask-0.10.1-py2.7.egg/flask/app.py", line 1478, in full_dispatch_request
response = self.make_response(rv)
File "/Library/Python/2.7/site-packages/Flask-0.10.1-py2.7.egg/flask/app.py", line 1566, in make_response
raise ValueError('View function did not return a response')
ValueError: View function did not return a response
<Greenlet at 0x10c5e6f50: <Flask 'ouimeaux.server'>({'HTTP_COOKIE': 'advanced_toggle_checked=; wanted, <function  at 0x10c7bb1b8>)> failed with ValueError

Abandoned this project in favour of Home Assistant and Homebridge.


Sbujam managedocfiles and fireroles patch

According to the docs, fireroles may include multiple rules, one per line separated by '\n'.
For example: "firerole: deny until '2011-01-01'
allow any"
(note the ^M or \n before 'allow any')
When revising bibdocs with such restrictions in Document File Manager, (even when one does not change the 'Restrictions' dropdown) the firerole is 'cropped' in the first \n character, changing the actual rule!
With this patch, newlines are 'escaped' before written to the bibdocactions.log, so that the firerole is properly parsed and reapplied when needed.

notes:

pylint produces several warnings, but this is because in general the code does not fully abide by the coding standards.
The patch is based on the maint-1.2 branch (but could/should be applied to other legacy branches)
This is my first PR in the invenio repo, so please accept my apologies if you find the code/style/documentation not up to your standards. I will try to do my best to fix it, should you find any issues with it.
I have not filed an issue in advance. If needed, I could do that for historical reasons

@ludmilamarian Done suggestions 1 and 2 [the easy ones :) ]
Creating a test is a bit more tricky (I could try, but I don't feel confident enough meddling in to that area)
Regarding your last point, I think it applies 'as-is' (with an offset) to maint-1.1. For maint-1.0 the patch must be applied to another file (modules/websubmit/lib/websubmit_managedocfiles.py) instead.
What do you think is the best approach to deal with it?


Support for Cassandra Map, List and Set datatypes

If there's no other complex requirements to make this work with upstream, can we add support for Cassandra Map, List and Set datatypes?
I understand that this might affect base.js as well.

@charyorde The datatypes of db-migrate should not be extended with such types. This will need a change at the datatypes, but in the best case you would extend them from your driver instead. This is not yet possible, but I will open a work item to make this possible.


Access Token Save in DB

Hi,
I am using  identityserver4. My question is very simple that how can i save my access token in my DB because I have a scenario in which i want to save my access token in DB. Can anyone please guide me how can i do this?
Thanks in advance.
Regards,
Alex

This seems to be a general question about IdentityServer - not a bug report or an issue.
Please use StackOverflow for that. This has the advantage that questions and answers can be easily found by search engines, and that there are more people answering questions than just us.
For IdentityServer3
https://stackoverflow.com/questions/tagged/?tagnames=identityserver3&sort=newest
For IdentityServer4
https://stackoverflow.com/questions/tagged/?tagnames=identityserver4&sort=newest
For commercial support
https://identityserver.io/


Mount Mansfield Union High School

@wrightw First stage of verification passed, official domain is mmu.mmuusd.org as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://mmu.mmuusd.org/students/student_services/about_student_services

@wrightw Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Bug - Shipping method sometimes calculated as $0

Turns out this is due to poor async in the Gatsby storefront. It will process a payment before it gets a response back with a shipping method. Since shipping method is making a request for a rate from 3rd part api, if u move fast u can beat the race and get free shipping :) lol
So, not an issue with plugin. This is a frontend implementation issue...


1:1 Aspect ratio for Android

Proposal
I noticed that the aspect ratio 1:1 is only featured for iOS and not android, any reason why it is not available for android and if there are plans to enable it there?
Describe your new feature
1:1 Aspect ratio on android

Any updates on this?


Stacked graphs in /graph have increasingly incorrect max values

Bug Report
Generated a stacked graph that was many small aggregates over a long window (a week).  As I increased the window size from 1 hour to 1 week, the max Y value of the graph was increasingly inaccurate, and the stacked data went from filling 90% of the min -> max Y value to filling only 5-10% of the vertical area of the graph.  There were no outlier values that I am aware of above that limit.  If I disable stacking on the long window, the series occupy most of the range of the vertical area.
I expected as I increased the window that stacked would continue to take up most of the Y range.

It looks as if it is estimating the max Y incorrectly the larger the range.
Environment

Prometheus version:

2.3.2
71af5e29e815795e9dd14742ee7725682fa14b7b
20180712-14:02:52
go1.10.3

It'd be good to see if this is still happening with the new UI.

Hello from the bug scrub,
Nobody has complained about this in the last four years, so we assume this is not an issue anymore with the new UI. If you have evidence that it is still an issue, please follow up here so that we can re-open.


Emit error when the wrong keys are retrieved

We don't currently detect if the wrong keys are retrieved.  In this scenario, a license has been requested and retrieved, but the license does not contain the keys that the browser needs to decode the content.
We could detect this in many cases and emit an error.  During integration and debugging, this would help point out issues with the license server/proxy.

I believe this would involve two events, waitingforkey and canplay, to enter and exit the "waiting on keys" state, respectively.  Needs investigation to confirm.

Do you want to detect the wrong key based on ID or the actual key? In the latter case, you may just get a decode error.

We're already able to detect decode errors, but in cases where we have the wrong key IDs for the content, we currently just wait forever.  This happens sometimes during early app development due to mistakes in the player configuration or the license server.
If we can detect this, we can save people debugging time and effort.


Feature/run avs with offline demo

Changes to run AVS along with sales demo if sales demo is installed on the Pi as well. Some documentation present in https://xmosjira.atlassian.net/wiki/spaces/~701212200/pages/2190147591/Changes+made+to+the+xmos+avs-device-sdk+repo+to+run+sales+demo+alongside+AVS.
This PR needs to be merged along with https://github.com/xmos/vocalfusion_3510_sales_demo/pull/30

This changes have been merged in xmos_v1.20.1:
https://github.com/xmos/avs-device-sdk/pull/48


SQL parameter substitution functionality differs substantially from string substitution functionality

In both the user guide (http://mybatis.github.io/mybatis-3/sqlmap-xml.html#Parameters) and the GitHub FAQ (https://github.com/mybatis/mybatis-3/wiki/FAQ), the #{} syntax and the ${} syntax are treated as being identical, with the exception that ${} performs a direct string substitution and #{} inserts the value as a parameter to a prepared statement.
However, in practice, the #{} syntax limits the expressions that can be used in unexpected ways.
First, the #{} syntax cannot retrieve a value from a map when any syntax other than map.key is used. This is the deal-breaker that first brought this issue to my attention. Given:
try(SqlSession session = ...) {
    HashMap<String, String> map = new HashMap<>();
    map.put("name", "Bob Smith");
    session.getMapper(TestMapper.class).test(map);
}

/* TestMapper.java */
int test(@Param("map") HashMap<String, String> map);

<!-- TestMapper.xml -->
<select id="test" resultType="int">
    SELECT ID FROM PERSON WHERE NAME = #{map.name}
</select>

This will work; however, none of the below will work:
<!-- TestMapper.xml -->
<select id="test" resultType="int">
    SELECT ID FROM PERSON WHERE NAME = #{map['name']}
</select>

<!-- TestMapper.xml -->
<select id="test" resultType="int">
    <bind name="bound" value="'name'"/>
    SELECT ID FROM PERSON WHERE NAME = #{map[bound]}
</select>

<!-- TestMapper.xml -->
<select id="test" resultType="int">
    <!-- #{map.get(bound) with bound defined as above will also fail. -->
    SELECT ID FROM PERSON WHERE NAME = #{map.get('name')}
</select>

Null is used to set the parameter in each case, despite each expression evaluating correctly if used with ${} syntax instead.
Additionally, functions may not be used in parameter bindings. Given an object mapWrapper with a method name() passed as a parameter:
<select id="test" resultType="int">
    SELECT ID FROM PERSON WHERE NAME = #{mapWrapper.name()}
</select>

This results in an error. (Using ${} syntax, obviously, does not.)
Finally, attempting to use  to overcome these limitations results in unexpected behavior. Consider the following:
<select id="test" resultType="int">
    <bind name="actualParameter" value="map['name']">
    SELECT ID FROM PERSON WHERE NAME = #{actualParameter}
</select>

This works as expected, which is great. However, let's say we have a list of columns (name, title, favorite_cookie_variety), and a map of columns to values (name => "Bob", title => "Senior Cookie Tester", favorite_cookie_variety => "Mint Chocolate"). We want to include these columns and values in our WHERE clause, subject to some arbitrary criteria which we don't want to work out beforehand.
<select id="test" resultType="int">
    SELECT ID FROM PERSON
    <where>
        <foreach collection="columns" item="column">
            <if test="column.someArbitraryCriteria">
                <bind name="actualParameter" value="#{map[column.name]}"/>
                AND ${column.name} = #{actualParameter}
            </if>
        </foreach>
    </where>
</select>

If you're familiar with the order in which MyBatis performs operations, you already see what's going to happen here. We re-bind "actualParameter" to a different value every iteration through the loop, but because parameter setting occurs after the script parsing has been completed, each parameter in the SQL query will be bound to the same value: the last value assigned to actualParameter.
While these examples may seem silly and contrived, I believe that the difference between ${} syntax and #{} syntax is unexpected and confusing. The available documentation does not describe or even hint at these limitations. Clarifying the documentation might be helpful, but I believe the usefulness of MyBatis will be enhanced by enabling users to leverage all the power available to them in ${} expressions in #{} expressions as well.
I would propose that the script driver be modified to parse #{} expressions as OGNL expressions the same as ${} expressions are. The result value could be stored in a new temporary variable, and that temporary variable be used in the parameter-setting stage.

Does it exist a workaround for last issue? How can I set actual value in following statement?
<foreach item="c" collection="filter.getFilter()" separator=" AND " open="(" close=")">
    <bind name="column" value="_parameter.mappingWhere(c.colCode)"/>
    <bind name="operator" value="_parameter.conditionOperator(c.condition)"/>
    <bind name="value" value="_parameter.conditionValue(c.condition, c.value)"/>
        ${column} ${operator} #{value}
</foreach>

I've tried static methods, not static methods,..., but nothing works :-(

Help! I have the exactly the same problem! How to use bind in foreach. Thanks.

See #575


bug: Indexer not syncing sales

Provide environment information
mac os chrome
mac os Brave
Describe the bug
This gnome is still showing as open to buy on the main store page but has been purchased multiple days ago.

Asset can be found here:
https://market.realms.world/token/0x0158160018d590d93528995b340260e65aedd76d28a686e9daa5c4e8fad0c5dd/1539
To reproduce
Simply go to: https://market.realms.world/collection/0x0158160018d590d93528995b340260e65aedd76d28a686e9daa5c4e8fad0c5dd
Additional information
No response

I’ll like to take this up


SITES-2874 Accessibility - Visual list is not marked up as list.

Ticket: SITES-2874
Added role list and listitem to the required elements

after applying the change, axe devTools can't identify the problem that appears in the failing build

Please fix the curly braces. For the complexity warning, if you cannot simplify the complexity, and it's code that wasn't introduced by you, it might generate a code smell report but should be fine to merge.

@petcud check warnings not related to the actual change, should we resolve this reports for every PR created as long as it were not generated by this PR?

Hi @petcud the role "list" is only to be use for the wrapper of the list and the role "listitem" is to be used for the each element from inside the wrapper as suggested in the ticket and in the accessibility guides https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/listitem_role

@Pareesh is there anything else we should do regarding this ticket? If not could you merge it?


Some mad crazy suggestions

I decided to post it cuz this is the most singleplayer-friendly currency mod ever, at least for now, so it might be even more amazing to make it take one step further.
When you have Hero of the Village on you (or ever had this effect on you since having to fighting back raids again and again would be kinda lame), you can ask for a magical brochure from the librarian in any villages for you to order goods from home (the available goods don't have to be the same as what the whole village is selling, it should be simply data-driven, with a default set of vanilla-ish goods, and customizable for modpack creators).
A few seconds after you placed your order, a nitwit with one llama or two, carrying all the things you bought will magically spawn nearby and walk toward you. After the inventory was emptied, they'll despawn seconds later.
There's a cooldown between two purchases.
Killing any one of them causes the termination of service, you'll have to fight against another raid.
It can be an addon mod for Numismatic Overhaul if it sounds too off the goal. Should prolly go viral among RPG-themed modpack creators.
The more I describe it the more I think it's a bit too wild. So feel free to just close this.

I do kinda like this idea, but I do feel like it would definitely be better as its own mod. I would be interested to support it if it ever goes anywhere.
For now I will close this as not planned. Thanks for the suggestion though


build: remove unnecessary cve from .cveignore

While running the cra ad-hoc report job against the java core, the resulting report stated:

So, this PR removes that entry from .cra/.cveignore.

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


Run functional tests for REST API during a build

Resolved #420

@EvgenyShashkin , please read my comments and fix them. Also please add a message to CHANGELOG.md file that we added running tests. If a test is fail what will we have (just post an example of messages from logs)?

@EvgenyShashkin , OK. I add WIP prefix for the request. I need more information to accept it. Not all questions in the PR were resolved. Let me know if you need more information. Don't hesitate to write your questions into itlab git.


refactor: move cli command logic

remove ifelse in main
rename api folder to cli
prepare decoupling of cli from getdeck lib

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

60 of 85 (70.59%)  changed or added relevant lines in 10 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.5%) to 66.122%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

getdeck/cli/console.py
10
11
90.91%

getdeck/cli/hosts.py
2
3
66.67%

getdeck/cli/remove.py
4
5
80.0%

getdeck/fetch/fetch.py
2
3
66.67%

getdeck/cli/version.py
3
5
60.0%

getdeck/cli/list.py
2
5
40.0%

getdeck/main.py
19
25
76.0%

getdeck/cli/telemetry.py
6
16
37.5%

Files with Coverage Reduction
New Missed Lines
%

getdeck/main.py
1
86.96%

Totals

Change from base Build<PHONE_NUMBER>:
0.5%

Covered Lines:
1376

Relevant Lines:
2081

💛  - Coveralls


defending against clock set in past problem

I have just experienced vsthrottle blocking a lot of IP which doesn't reach rate limit.
This was due to fact that the clock was set 2 days before ...

Could you add below defending code in calc_tokens in src/vmod_vsthrottle.c ?
if (b->tokens < 0)
	b->tokens = 0;

Thanks!
I've pushed 99e2a16be860718ea6a698145afb9ec3339f59fc that should take care of it.

Not sure to post in the rigth place...
i tried this fix. It was ok on preprod but on production, with about 800 request/s i've got one assert each 15-30 seconds:
<<
varnishd[23893]: Child (23272) Panic message:#012Assert error in calc_tokens(), vmod_vsthrottle.c line 92:#012 Condition(delta >= 0) not true.# ...
<<
Reading the source code, i think that b-lastused can be update since you get now.
Moving down
<<
double now = get_ts_mono();
<<
juste after
<<
AZ(pthread_mutex_lock(&v->mtx));
<<
in vmod_is_denied, fixed the issue for me in production.
Regards.

Thanks for the feedback - this is certainly a race condition.
I've commited your suggested fix in 8441901.