синхронизация времени с приемником GPS вывод последовательных данных NMEA

Hi!
Synchronization of time with the GPS receiver conclusion of consecutive data of NMEA.
Prompt what GPS receiver to use and where to connect on the scheme of hours?

Hi, I don't really understand your question.
However, there is experimental gps support on the gps branch.
To test, I hooked up NMEA GPS module like this one
https://www.banggood.com/GPS-Module-for-Auto-Car-DVR-Navigator-Tracking-Device-Recording-Car-Dash-Camera-p-1049257.html?p=WX0407753399201409DA
which outputs serial nmea at either 9600 bps into P3_6:rxd (look at uart.h)
Also, take a look at gps.h, and see that it is currently looking for $GPRMC strings, however if you have GLONASS, you may need to modify this to $GLRMC (I'm in USA, I have only ever seen $GP strings.)

It is clear.
This experimental program can approach by these o'clock?
https://ru.aliexpress.com/item/DIY-Kit/32903754608.html
If is not present, then what needs to be changed?
And I cannot find hours on the STC15W408AS microcontroller yet. Specify the reference if you know.

It may be difficult to tell from the pictures. Unfortunately even in that listing:

schematic shows both STCW404AS and STCW408AS as mcu ;)
pictures show STC15F204EA

STCW408AS is best choice as it has 8KB flash, and hardware UART peripheral.
STCW404AS might work, if you can either get code size down to <4KB or use tricks like EEPROM space (9KB) for storing large constants and look-up tables.
Probably will not work with STC15F204EA (no hardware UART peripheral) and only 4KB of code space, and 1KB EEPROM.

And at you with what microcontroller synchronization of time with the GPS receiver is used?
An exit from the GPS TX or PPS receiver?

And at you with what microcontroller synchronization of time with the GPS receiver is used?
An exit from the GPS TX or PPS receiver?

I don't exactly understand what you are asking.
Yes, I am parsing the NMEA output strings from GPS TX, particularly RMC sentence.
https://www.gpsinformation.org/dale/nmea.htm#RMC
(I did look at ZDA however, some gps receivers did not seem to provide this. RMC seems to have what I need).

Through online the translator it is difficult to communicate. If I correctly understand, the speech in your article goes about synchronization of the electronic clock on the STC15W408AS or STC15F204EA processor with the GPS receiver. I wanted to learn from you whether it is possible to use your pilot project for the electronic clock on processor 15W404AS (it is used at me in the electronic clock) and where to connect the GPS receiver in the scheme of hours. I plan to use this receiver
https://ru.aliexpress.com/item/1811853522.html?productId=1811853522&productSubject=2pcs-lot-GY-NEO6MV2-new-GPS-module-with-Flight-Control-Flight-Control-EEPROM-MWC-APM2-5&af=506321&aff_platform=aaf&cpt=1556717407284&afref=https%253A%252F%252Fpafutos.com%252F&dp=5a44cc0233724e66173c22d8dd025695&onelink_item_from=1811853522&onelink_thrd=0.0&onelink_page_from=ITEM_DETAIL&cv=47843&onelink_item_to=1811853522&pvid=440653f3-527f-42aa-aff5-fc0c7ca2cfaa&mall_affr=pr3&onelink_duration=0.704075&sk=VnYZvQVf&aff_trace_key=459ae4f3dc8a49f19aff30a2cda40390-1556717407284-00579-VnYZvQVf&onelink_status=noneresult&scm=1007.22893.125764.0&terminal_id=cc16b801dfa144a0805022164fcb5f08&onelink_page_to=ITEM_DETAIL
I do not develop the program for microcontrollers. Did not find articles of similar to this yours in the Internet. Therefore wanted to obtain exact information on the questions interesting me.

It should be possible with STC15W404AS - this is basically same mcu as STC15W408AS. Difference is 404 has 4k code space + 9k eeprom, 408 has 8k code + 5k eeprom.
Also both have hardware uart peripheral. STC15F204EA does not have hardware uart, so that along with less overall code space, would be difficult (but not impossible) to support GPS decoding.
On either STC15W404AS or STC15W408AS, both support hardware remapping of the UART1 to P3_6:rxd, P3_7:txd.
So to support any serial/uart gps module you need to know a few things to adjust firmware to support it.

need to know output format. Should be NMEA.
need to know serial baud rate. Probably either 9600 or 4800 bps.
need to know what kind of NMEA "sentences" it outputs, e.g. $GPRMC... or $GLRMC... or others

Привет!
Снова я к Вам, но с конкретной просьбой. Вы можете изменить прошивку для STC15W404AS, чтобы функциональность была такая:

55 секунд отображается текущее время в формате 24 часа
5 секунд отображается температура только в градусах Цельсия
ночью, когда освещение падает, яркость часов автоматически уменьшается (как в стандартной прошивке с датчиком освещения, идущим в комплекте)
простой будильник (согласно стандартной прошивке, но без почасового сигнала)
GPS (модуль GY-NEO6MV2):

стандартные данные NMEA
скорость 9600 bps
выводит GNRMC и GNGGA
Всё остальное, согласно стандартной прошивке, не нужно.


Implement TGI model config from path

Implement TGI model config from path:
TGIModelConfig.from_path(model_config_path)

Follow-up to:

#434

Related to:

#439

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Documentation for C58862 Error Code appears missing

using ...
namespace somenamespace {
[IBindable(BindableSupport.Yes,Serializable()]
public record class ClassName() : IDisposable , IXmlSerializable
{
public_ ClassName()
{
//throws a C58862 Error even with class properties initialized.
}
...
}}

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

ID: bbb051ee-f020-66ba-c601-4a08ad32ef61
Version Independent ID: 58a19505-ac46-c5c8-12cc-fe87b6aa18a7
Content: Sorry, we don't have specifics on this C# error
Content Source: docs/csharp/misc/sorry-we-don-t-have-specifics-on-this-csharp-error.md
Product: dotnet-csharp
Technology: csharp-diagnostics
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for raising this @mbarnet
This is an area where we've fallen behind in the docs.
Here's a smaller example:
public record ClassName(int num)
{
   //Compiler generates C58862 Error even with class properties initialized.
    public ClassName()
    {
       this.num = 0;
    }
}

This is because the compiler's static analysis requires that the generated primary constructor is called from all user-written constructors in that class.

Fixed in #35324


Evaluate support for other file formats aside from JSON

Currently, the responses can only be provided as JSON key-value pairs. To extend this also to other formats like YAML or a custom format could be a good addition. This should be evaluated and how this could look like.

Currently, there is no need for other formats.


Download the HTML for the page when screenshotting

IMO it's best to think of it as a part of the screenshot (enhanced screenshot). We also replace relative URLs for CSS / JS with absolute URLs, adding the server url in front of them. This isn't perfect, as CORS will prevent the page from fully loading, but it's close enough, people will still have the screenshot if they want exactly what the page looked like.
Fixes #763, probably some other related ones as well.
In this PR, I have:

[x] Added tests for any new features or bug fixes (if relevant) (no existing tests for ensuring JPGs are saved, not sure how to write them)
[ ] Added my changes to the CHANGELOG.md at the top, under the "Unreleased" section (have avoided doing so until we're ready to merge, to avoid merge conflicts with other open PRs)
[x] Ensured issues that this PR closes will be automatically closed

Any manual testing I have done to ensure my PR is working
Have manually run the tests, and have made sure that:

all screenshots have a partnering HTML page
said HTML page mostly loads in Firefox. See issues with CORS mentioned above

Confirmed that the HTML files are uploaded correctly!

From synchronous discussion: we aren't sure how much performance gain we get from the Promise.all stuff, and it's a bit easier to read/ maintain without it, so going to remove that.

The HTML also comes out a bit weird (I looked specifically at the upload page where there were doubled fields and such)

Yeah, I mentioned that in passing earlier, but most of that is because the javascript on the page can't run. Loading a webpage from a local file that then tries to call a docassemble server will fail for several cascading reasons:

sometimes the JS on the page calls back to relative links on the server, so loading the file off of the server breaks those links
The server won't know what interview is occuring when you load the page from a different browser, so most of the JS calls will still fail
Finally, sometimes the JS will make POST calls, which need to have CORS (Cross Origin Resource Sharing) rules enabled. Most browsers won't let you set those rules properly when opening files locally in your browser, for security reasons.

But I agree that it's good enough. Just saying that "it'll be different, still use the screenshots, and only look at the page if something's missing in the screenshot, or you need to examine the HTML code directly (opening in the browser to use inspect tools is helpful here)" should be enough. I'm gonna document a lot of other stuff tomorrow, I'll try to do this one as well.


Can you mix and match readonly / mutable MQT types?

Hello,
I have not tried out this library yet but I have faced similar issues this library tries to solve, so I am really interested in giving this a try.
When going over the documentation there is section about mixing MST and MQT:
https://github.com/gadget-inc/mobx-quick-tree#mixing-mobx-state-tree-and-mobx-quick-tree
It is clear to me you need to fully switch to importing MQT as you cannot combine MST and MQT.
However, what i am missing is some notes about whether you can use MQT readonly types with mutable MQT types.
For example, can the following be done with this library:

Have readonly MQT type of Song, and a collection Songs.
Have a mutable Playlist type with an array of PlaylistItems.
Have both Songs and Playlist in the same RootStore?
Will this maintain the same performance upgrade that MQT offers on the Song objects (especially their initialization).
Have type.references between PlaylistItem and Song(s).

If this is supposed to work. Would RootStore be a readonly, or mutable store, considering it holds both readonly and mutable MQT types ({ songs: Songs, playlist: Playlist })?

Hi @jrmyio -- I don't think we've ever tried mixing and matching! At Gadget our tree is either entirely observable or entirely readonly. I suspect it won't work out of the box, but I think you could work around it by setting the parts of your tree that you want to be readonly to use a types.frozen and constructs the readonly instances for those bits before passing the snapshot to your root. We could add a types.readonly that does the instantiation under the hood as well I guess -- that seems generally useful!


Deploy Contracts to Sepolia Tesnet

Problem
Sepolia Testnet env does not have Connext contracts
Impact
De-risks product development, and integrations
Proposed Solution
Use our deploy and init scripts
Acceptance Criteria
From an end user perspective, the below criteria must be met to consider this done:

[x] Add Sepolia chain ID to configs
[x] Deploy hub and spoke contracts, using testnet deployer, for all chains supported by graph studio or decentralized network
[x] Init contracts

Fixed by https://github.com/connext/monorepo/pull/5702

Slow and Op mode tests were completed by @wanglonghong


Fix CodeQL Regex warning

Description
Fixes the CodeQL warning for regex by replacing the URL check from regex form to the actual exact URL match.
We face this because the URL we are visiting is https://owasp.org?trickIndexOf=https://github.com/bkimminich/juice-shop and the regex check of codeQL throws a failure because we aren't validating the entire URL ie the end.
Affirmation

[x] My code follows the CONTRIBUTING.md guidelines

I don't think this needs fixing really, it'd be enough to ignore on CodeQL level.

I meant just actually ignoring as with the other 180+ warnings currently reported by CodeQL... 😇
Latest scan
1 hour ago
Pull request
#1911
Workflow
CodeQL Scan
Lines scanned
56.8k / 56.8k
Duration
3m 30s
Result
189 alerts

I intend to go through these one day and actually ignore-list the intentional vulns (and false positives) and fix all others, but never found the time so far.

But then doesn't it show the Red failing cross in the CI if we just ignore them? :/

Closing this since you said we will be disabling them through CodeQL


Store: Add Dynamic Module Registration support

What problem does this feature solve?
Hi.
Is there any plan to support Dynamic Module Registration of Vuex store ? I didn't see it on the document of NuxtJs.

This feature request is available on Nuxt.js community (#c6809)

Hi,
You can already do so in nuxt for example, in a SFC :
import yourAwesomeModule from '~/store/modules/yourAwesomeModule';
[...some code...]
this.$store.registerModule('yourAwesomeModule', yourAwesomeModule, { preserveState: true });

Thanks for your reply.
But I think it will be better if nuxt can automatcally create an association between page and store module.
For example:

pages
-- awsomepage.vue
store
-- _awsomepage.js

It will be better if nuxt can register the module just before awsomepage.vue created and unregister it after awsomepage.vue destroyed automatcally.

Oww, ok, I do unsterstand what you mean. I don't know if I want that magic to happen though, because some component won't have a store module requirement and some will. This question is for the core team :)

We're considering Vuex integration improvements post-2.0. Closing this for now!


On hit effects

I am not too sure about this.
Somehow I ended up having to look in 2 places:
One of them you already did, just needed to be shown. It's the Effect.Item_Set_Melee_Effect and Effect.Item_Set_Ranged_Effect in BGEntity.
The other needs to go through the items and parse its effects, abilities, and the effects of the abilities. Which is weird, because I would think the previous step takes care of that.
In any case I wouldn't merge. Some stuff that doesn't make sense or is incomplete is showing up.
But maybe you can take a look and tell me what you think?

I'll have a look in the evening

We can't leave it like this:

There should be some filtering / proper human readable interpretations

I've also checked Balor:

it seems like the save type is incorrect - (it says its spell when it should be death)

Yes I agree. I think i'll leave the branch as is, that was as far as my knowledge would get me, in case someone figures it out

Yes I agree. I think i'll leave the branch as is, that was as far as my knowledge would get me, in case someone figures it out

But you did so much already - I'll see what I can get off of it

Guess whaaat? :)

Yes I agree. I think i'll leave the branch as is, that was as far as my knowledge would get me, in case someone figures it out

But you did so much already - I'll see what I can get off of it

Yes, but I was seriously struggling by the end of it :joy: very uncertain about what I was doing.
Thank you for taking a look. I'll fire it up in a bit, but it looks awesome!

Guess whaaat? :)

:heart_eyes:

Just pushed another fix - this one should be rock-solid

Another one:

Another one:
Thanks for the reports!
Can you check what this dagger is exactly?
The only way it can be spell -1 when both its spell names are -1, I believe.

Also, if you could please pay attention  to the saving throw types of weapon onhit effects. I was checking it against Corsomyr, Silver sword (planetar's) and Balor's skull - Got the death save and no save right (hopefully), but the other types still require checking

Ah yes, I should have mentioned the excellent dagger - that's from the SCS component that transforms many +1 weapons into "fine" ones. So it's looking good there.
I'll keep an eye on mismatched info regarding saving throws types/penalties :+1:

Ah yes, I should have mentioned the excellent dagger - that's from the SCS component that transforms many +1 weapons into "fine" ones. So it's looking good there.
I'll keep an eye on mismatched info regarding saving throws types/penalties 👍

Pretty sure excellent daggers don't have any "cast spell" effects on them, so its very suspicious.
I was thinking if I should omit the "use type" abilities from the items (like aoe dispell on Corsomyr) as technically they aren't "on-hit" effects

About the excellent/fine weapons: agreed. I thought it might have been some shenanigan introduced by SCS but I can't seem to find any reference to it in NearInfinity.
The AoE dispell and similar are good candidates to be omitted if problematic, I think - it's pretty obvious that a green ball of magic is coming toward you, so the feedback is there ("Dispel effects" in the log).
When it comes to on-hit effects, for me the greatest offender was Berena Elkan. She's normally a wizard slayer, so she applies spell failure. But in Ascension she gets Spellstrike (!) + Breach on every hit.

About the excellent/fine weapons: agreed. I thought it might have been some shenanigan introduced by SCS but I can't seem to find any reference to it in NearInfinity.
The AoE dispell and similar are good candidates to be omitted if problematic, I think - it's pretty obvious that a green ball of magic is coming toward you, so the feedback is there ("Dispel effects" in the log).
When it comes to on-hit effects, for me the greatest offender was Berena Elkan. She's normally a wizard slayer, so she applies spell failure. But in Ascension she gets Spellstrike (!) + Breach on every hit.

Oh Im excited to see how radar would react to her

I checked yesterday, and I'm like 80.33% sure it displayed all 3 effects (spell failure, spellstrike, breach), but I can't remember the exact description it showed. I'll get a screenshot later.

Here are some findings:
Hakeashars look awesome:

And so does the Mace of Disruption, both the save type and the penalty are correct.

Berena is almost right:

I believe Mindflayers might be missing the int drain, but it's odd - I'm almost sure I saw it at some point!

In a Frost Salamader I noticed again the "cast spell at creature: -1" bit. In this case I'm almost sure after digging in NI a bit that it's the aura, the SPL it references has an effect for cold damage.

I also went looking for the sword Gram. I am not quite sure what is going on, I took a pic of both the inventory description and the listed effects.

Oh and dragons show the wing buffet ability :D

Could you please check it with the latest release?
And lets create a new issue for that

Can't wait to try!
I'll port the stuff from above in this issue to the new issue if necessary, and add any other findings.

I've just pushed an update for the on-hit protection removals, could you please check it?
I don't have ascention installed so can't really test - if its fine I'll update the release

yap, will do!


⚠ Bofrost.at has degraded performance

In 118b67b, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2623 ms

Resolved: Bofrost.at performance has improved in 358ffa1 after 7 minutes.


test script fails. No node_modules\jest-cli

npm run test results in
Error: Cannot find module './node_modules/jest-cli'

Cannot reproduce. Closing,


Support not etc/hosts config files,such as sysctl.conf and limits.conf

I need also write configuration parameters  to some Linux config files such as sysctl.conf and limits.conf.
Can we add a flag such as --nohosts to disable verification?

I wrote a bash script following the idea of hostctl.Thanks!


SfCarousel controls (prev/next) break SfCarouselItem containing link

Related issue
Closes #539
Scope of work
It fixes .sf-carousel__wrapper to be properly positioned between SfCarousel's controls (prev/next buttons).
Screenshots of visual changes

Checklist

[x] I followed composition rules for my component
[x] I tested the component in most common device sizes (can be tested in Storybook from viewport addon in top menu)

@psmyrek can you check it with SfOverlay component just to be sure it's z-index is not breaking anything??

@filrak It depends on the order of components in template but if SfOverlay is used above SfCarousel then it is broken :(

How do you propose to fix it? Maybe changing z-index in SfOverlay to big enough value that is not overridden by any other component's z-index (currently it would be 11)?

thanks, @psmyrek #585 fix this bug :)


Undefined function error in advancements function reward is missing Quick Fix

Using the /function command, there is a quick fix "Create function in same datapack".
That is missing in function rewards from advancements though.

Duplicate of #672


Feature/expose path

Please merge #16 before this one

This exposes the root, so it can be accessed by developers. After object creation.


fix(widget-markdown): don't strip new lines from text nodes

Fixes https://github.com/netlify/netlify-cms/issues/3755
Tagging @erquhart for feedback

Going to merge this as https://github.com/netlify/netlify-cms/pull/3813/files#diff-5b6ff5f0f58982f4be6c3e2a375300adL296 seems to only replace the first line break anyway so it's not working as intended.

@erezrokah Nice work. Thank you.


Font stylistic set support

Issue Type: Feature Request
In the new version 2.0 of Fira Code, there are some stylistic options that can be set (e.g. old-style figures).  In Sublime Text there's a simple json key you can add to its settings file to enable the ones you want.
In VSCode, you have to use an extension which modifies the "custom css".  This seems hacky and needs to be enabled after every update.
I'd rather be able to do this with a settings key like Sublime Text.
VS Code version: Code - Insiders 1.39.0-insider (0f1bd7721b793ed8b7805ba621061ce5a1f5c263, 2019-09-09T05:26:48.181Z)
OS version: Darwin x64 18.7.0

Implemented via https://github.com/microsoft/vscode/commit/ab0eb6de2d5fb4563385d98f302f78caa1767efd (https://github.com/microsoft/vscode/issues/82153) and it already shipped in Insiders.
e.g.
"editor.fontLigatures": "'ss01', 'ss02', 'ss03', 'ss04', 'ss05', 'ss06', 'zero', 'onum'"

or
    "editor.fontFamily": "Fira Code"
    "editor.fontLigatures": true,
    "[cpp]": {
        "editor.fontLigatures": "'ss06', 'ss19'",
    },
    "[javascript]": {
        "editor.fontLigatures": "'ss06'",
    },

Hey @alexandrudima, awesome work on this – it makes my code font project so much nicer to use in VS Code, and is yet another reason I'm really happy to be using VS Code as my go-to editor. :) Thank you!


101 role fails due to invalid Finch hash

2019-12-11 22:45:25,043 p=1011 u=root |  fatal: [localhost]: FAILED! => {"changed": false, "msg": "The checksum for /opt/FinchPython120.zip did not match dce5cea6562cb4b2c1f2489131227d211d559198; it was 177345bd2029f8055c53f3f078e90c72ca80c260."}

FinchPython120.zip appears to still be the current version, but the file contents are dated 10/24/19

So it might be hard to tell how long this has been an issue. #330 is when we started actually validating the checksum of the file we downloaded. If you run tina you'll see you get the file with the hash 177345bd2029f8055c53f3f078e90c72ca80c260 and you just download a new file every time because on the next run the hash doesn't match the expected value. Now that we actually validate the hash on download as well (the correct behavior), we get a hard failure.
It is alarming to me that at some point, with no announcement, a file with python scripts, dlls, and all other sorts of things had its hash change. I have tried to reach out to Bird Brain Technologies and we'll see what their response is. I don't seem to have an earlier version of this file that I could use to compare either.
I don't believe there is a section of CS 101 in the spring so I don't feel a particular urgency about this. We could always ship with either 101 or the Finch role disabled.

My small concern here is that 101'ers from last semester may try to add 159 to their image, which would re-run the Finch playbook and fail. Whether we update the hash or disable it, I'd recommend we consider a backport.

I have confirmed with BirdBrain Technologies support that 177345bd2029f8055c53f3f078e90c72ca80c260 is the expected SHA1 hash. They have said they'll pass the suggested to increment version numbers or release a changelog to their engineering team for future releases.


inconsistent derivative of sqrt(relu) under forward vs. reverse mode

from jax import jacfwd, jacrev, nn, numpy as jnp
def f(x): return jnp.sqrt(nn.relu(x))
jacfwd(f)(-10.)    # -> DeviceArray(nan, dtype=float32)
jacrev(f)(-10.)    # -> DeviceArray(0., dtype=float32)

Noticed thanks to @craffel, who expected nan from a related use of grad.

If I remove the custom derivative of the relu function that was introduced in #2342, this problem doesn't seem to persist anymore. Although that might reduce performance, so not sure if this there is a better solution to this.

The same inconsistency can be observed in nn.relu(jnp.sqrt(x)):
from jax import jacfwd, jacrev, nn, numpy as jnp
def f(x): return nn.relu(jnp.sqrt(x))
jacfwd(f)(-10.)    # -> DeviceArray(0., dtype=float32)
jacrev(f)(-10.)    # -> DeviceArray(nan, dtype=float32)

which would have an easy fix, however in that case I am not sure whether the expected value of 0. would actually be more correct since jacrev(nn.relu)(float('nan')) == DeviceArray(0., dtype=float32)?


Create Toyota Giraffe Switch Settings Flowchart.pdf

Flowchart to help users understand how to configure Toyota Giraffe switch settings.

This looks awesome! One thought though (and I can do this myself too if you would like to) I would like to see this as a PGN and replace the text in the README with this image. Look how we've done it on the main giraffe readme and that should help you out. I can do this myself too if you would like.


Set http_proxy, https_proxy, and no_proxy env vars in rendered PowerShell profile

This is fixed in v1.1.0


Reworking Output

The current output is hard to use/modify, we need to rework it.

Output is now done through Sink under VIPRA/Base/output/sink.hpp
Need to add to documentation


[MRG] Tidy up the ask and tell notebook

Tidy up that left over line in the ask and tell notebook

Codecov Report

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

@@           Coverage Diff           @@
##           master     #414   +/-   ##
=======================================
  Coverage   85.76%   85.76%
=======================================
  Files          21       21
  Lines        1440     1440
=======================================
  Hits         1235     1235
  Misses        205      205

Continue to review full report at Codecov.

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


Trauma kits/Gauze consumes 2 of the stack instead of 1

???

This is because we don't have limb targeting yet


App freezes when you attempt to open the same pre-loaded view twice (on Android)

A pre-loaded webview can only appear in the layer stack once. Attempting to open the same webview more than once freezes the app instead of just calling the error callback.
Repo app: https://github.com/dantabel/supersonic-push-freeze.git
NB this is a follow on from conversation on #913 regarding trying to detect if a pre-loaded webview is already in the layer stack or not.

Just to clarify; is this happening on both iOS and Android or just either one? Crosswalk or Platform WebView if Android?

I've just tested in iOS simulator and it doesn't freeze, so it's just Android by the looks of it.
I've only tested on Crosswalk on Android and it freezes.

Just tested on Platform and it crashes there too.

Thanks for testing, we'll put this on the backlog!


Make 'nlohmann-json' cacheable

Testing:

https://ci.appveyor.com/project/ingenue/hunter/build/1.0.1291
https://travis-ci.org/ingenue/hunter/builds/188504502

Released:

https://github.com/ruslo/hunter/releases/tag/v0.17.12


May I have permission to open a repo, please?

Hello, may I have GitHub permissions to open / add to a new repo, please? It's for some educational videos and writing I'm working on, about CUTLASS.

Hi @ericauld sure! What would you like to call the repo I'll give you admin access over that one

Thanks, how about "Cutlass Tutorial"?


⚠ Bofrost.at has degraded performance

In 96cac0f, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1164 ms

Resolved: Bofrost.at performance has improved in 5bc32a6 after 6 minutes.


Q: Media Element on ready FallBack and Peaks Values

Hi,
I'm trying to get an array of peak value for generating peaks file.
The trick is to use wavesurfer.backend.getPeaks(960) as you describe in this issue. https://github.com/katspaugh/wavesurfer.js/issues/240
I use it with the .on('ready') event so that I can get the peak values after the waveform is drawn.
With WebAudio backend, it works all fine. An array of peak data is returned. (I use the console.log to debug).
With MediaElement backend, it return an empty array.
It seems that .on('ready) event is called before peaks are drawn when using MediaElement, wich is not the case with WebAudio.
Do you confirm ?
Is this a normal behavior ?
Thanks again for your support !

Yes, the two backends are inconsistent in this regard. It's already reported in #736. Please watch for the progress of that issue.


Application structure

Landed in https://github.com/metarhia/metarhia.com/commit/4b23cee69268a275951f134d080a38cca117fb26 and https://github.com/metarhia/metarhia.com/commit/cd64894fe2fd6a846e98f9095a6e1488894e4756


fix the return error of value's type

This function may return either an integer or a float as described in the phpdoc.

those functions return type is string now

Oh, do they! That looks like a mistake.

The aggregate function is marked as only floats and integers, so I guess we need a proper fix there.

Ok, I've provided a fix: https://github.com/laravel/framework/pull/14781.

This has been reverted. These functions support max dates which are strings. Our doc is just wrong.


Question: Display Week Number in calendar

Is it possible to display the week number in this calendar?

+1

Sure,where would you like it displayed? Do you have an example design?

@bryankeller I'd like it to be displayed as the first column on the left.
I'm working with locales / markets such as Sweden, Germany, Netherlands etc, and they tend to use the week number to help with their planning.
I've attached a design to show you the placement of the week number column.

Thank you


”Glodeni” is listed in the drop-down menu

Description:
"Glodeni" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Glodeni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Glodeni, MUREȘ".


feat(adapter-d1): set maxBindValues: 100

This PR contributes to https://github.com/prisma/team-orm/issues/1070.
Please merge after https://github.com/prisma/prisma-engines/pull/4878 is released.
This should fix the issues mentioned in https://github.com/prisma/team-orm/issues/1070#issuecomment-2079497069 as well (TODO: check after this is merged and well-integrated).

Do we have some test(s) for this now? (For D1 and other adapters)

Not that I could find, except the prisma-engines one, query-engine/connector-test-kit-rs/query-engine-tests/tests/new/regressions/prisma_7434.rs.

I remember that I actually some tests in this PR, I guess that's enough? Could you check @jkomyno ?


Not serializing Int32

Is there any particular reason why it fails to serialize a field of type Int32?

Int32 is not supported by default. You need to create a Transform for this type. See this transform that pulled from the Test classes for int64Value
(map["int64Value"], TransformOf<Int64, NSNumber>(fromJSON: { $0?.longLongValue }, toJSON: { $0.map { NSNumber(longLong: $0) } }))


Porting components of system.web* to ensure wingtiptoys work as is in .netcore(linux/windows)

Porting wingtiptoys to .net core(linux/windows) using systemweb-adapters-ui /systemweb-adapters. As a part of the process, porting components which are needed to systemweb-adapters-ui. List of components will be added as a port.

closing as tracking as one big task


Export SVG for Visualizations

Partially closes #202
There's a download button in the analysis panel with a tooltip that currently only supports SVGs. In the future, the button should have a popover with the option to select between formats.

I'll take a look, thank you!
On Wed, Nov 13, 2024 at 2:16 PM Evan Patterson @.***>
wrote:

@.**** requested changes on this pull request.
Thanks Hamidah for preparing a clean branch! I left some comments below.
Don't worry about fixing the merge conflicts. I can do that later.
In packages/frontend/package.json
https://github.com/ToposInstitute/CatColab/pull/258#discussion_r1841248277
:

@@ -45,6 +46,7 @@
"prosemirror-state": "^1.4.3",
"prosemirror-view": "^1.33.8",
"solid-js": "^1.9.2",

   "svg2png-wasm": "^1.4.1",

This dependency is not needed for this PR.
In packages/frontend/src/document/analysis_document_editor.tsx
https://github.com/ToposInstitute/CatColab/pull/258#discussion_r1841249288
:

@@ -31,6 +31,9 @@ import type { AnalysisDocument, ModelDocument } from "./types";

import PanelRight from "lucide-solid/icons/panel-right";
import PanelRightClose from "lucide-solid/icons/panel-right-close";
+import { handleExportSVG } from "../visualization";
+import { Download } from "lucide-solid";
+import { Tooltip } from @.***/core/tooltip";
The Tooltip component is not being used here (and the linter should
report this, except that the merge conflict is preventing the CI from
running).
On packages/frontend/src/visualization/export_visualization.tsx
https://github.com/ToposInstitute/CatColab/pull/258#discussion_r1841249904
:
This file contains no JSX, so it can be called simply
export_visualization.ts.
In packages/frontend/src/visualization/graphviz_svg.tsx
https://github.com/ToposInstitute/CatColab/pull/258#discussion_r1841256557
:

         <GraphSVG graph={props.graph && parseGraphvizJSON(props.graph)} />

     </div>
 );

}
+
+// Create a ref for the visualization container
+let visualizationRef: HTMLDivElement | undefined;
+
+// export handling
+export const handleExportSVG = () => {

if (visualizationRef) {

   exportVisualizationSVG(visualizationRef);

}
+};

The use of a global variable for the ref and a global export callback
should be a warning. In fact, this code will not work as expected if an
analysis contains multiple visualizations (which is possible!).
The easiest way to fix this is to first move the export button from the
top-level toolbar to the cell itself (perhaps in the GraphSVG component),
then forward the ref
https://docs.solidjs.com/concepts/refs#forwarding-refs to the svg
element up the hierarchy.
—
Reply to this email directly, view it on GitHub
https://github.com/ToposInstitute/CatColab/pull/258#pullrequestreview-2434573856,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR6IQEHNGYH3JS6IILWSKUT2APFSPAVCNFSM6AAAAABRWKXTDWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDIMZUGU3TGOBVGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Hamidah Oderinwale
@.***
hamidah.me


Revert UPM pack on non-Unity machines

Overview
Follow-up to #9276
versionmetadata.yml depends on the project being opened in Unity to generate meta files for the files it adds. Therefore, we must run UPM pack after that point (which happens when we pack the .unitypackages). Unfortunately, this means UPM packing needs to be moved back to the Unity machines and turned back off for PR validation.

/azp run

/azp run


how link mssql2000

CAN NOT LINK MSSQL 2000.
err
Login error: unexpected EOF

MSSQL 2000 NOT SUPPORTED.

There is a very old project in my company, I tried to connect MSSql  2000 for several days and success by gofreetds !
Solution below：
import (
	// others ...
	"github.com/jinzhu/gorm"
	_ "github.com/minus5/gofreetds"
)

func conn() {
      // code ...
     db, dbErr = gorm.Open("mssql", fmt.Sprintf("user=%s;pwd=%s;database=%s;host=%s", user, password, dbName, host))

	if dbErr != nil {
		log.Println(dbErr)
	}
}

func query() {
  // code ...
     var user models.User
     row := db.Raw("SELECT SEQ,ID,PWD FROM dbo.Member WHERE ID = ? and PWD = ?", 123, "abc").Row()

     rowErr := row.Scan(&user.ID, &user.PWD)

}

thank you..
I TRY


run build-on-hosted-runner workflow only for PRs that have github actions related code changes

Currently this workflow is being run for every PR

fixed in #340


[Ember 2.0] Can't use intl-get for placeholder in input component

// placehoder evaluates to [object object]
{{input value=email placeholder=(intl-get 'auth.signUp') class='form-control'}}

I also tried this:
{{input value=email placeholder={{t 'auth.signUp'}} class='form-control'}}

but ember-cli can't compile it.
Using it in normal (not a component) input works as expected:
<input type='text' value='' placeholder={{t 'auth.signIn'}}>

@ryrych The intl-get helper returns the translation object, not the final string.
You can use the t helper to get a translated string, as you used on the non-component input, so the following should work:
{{input value=email placeholder=(t 'auth.signUp') class='form-control'}}

Thanks @rlivsey :)

Cool, thanks!


Azure Service Principal for Azure CLI Rewrite

@daphnemamsft , This PR looks like a WIP as I believe you are working to break this Tutorial down into multiple files, refresh and add possible content to each section (like output code blocks)?  I won't review until you are ready.


[2.4] Only restart kubelet once, do not check managed agent

Fix https://github.com/rancher/rancher/issues/26583

https://github.com/rancher/rancher/issues/23254


Eq 1 questionnaire manager

What
Initial implementation of the questionnaire manager, the schema loader, renderer and the response store. Also moving towards flask class based method views.
How to test

Checkout this branch
Browse to http://localhost:5000/questionnaire/1

Note: this renders the json in survey/mci.json
Who can review
Anyone apart from @warrenbailey

LGTM :+1:


Hide all but latest comment

Fixes #36.

Waiting for #33 as this conflicts with that PR. Needs to be integrated with it (replace Unit Test Result string`.


Publish on rubygems?

I was search for this gem https://rubygems.org/search?utf8=✓&query=omniauth-openid-connect Do you plan to publish?

This repository is a forked version.
Unfortunately, I can't submit it to rubygems.org.
Instead, I will provide a gem package when the upcoming version is available.

@hhorikawa Why do you not publish the gem under a new name?


fix patent agreement link

@littledan can you take a look if i got the correct patent agreement here?

Nope, people who join Ecma should not sign this form.

We can just delete the patent link from here. Soon, in how-we-work, we will have a more detailed explanation of how to join


[BUG] Global RU/UA localization issue

Describe the bug
Due to the peculiarities of the programme, it is not possible to declare localization for some plural words correctly.
While in English it is easy, you just have to add "s" at the end if the word is plural, in Russian and Ukrainian it is difficult.
To Reproduce
Steps to reproduce the behavior:

Open Launcher
Go to downloads tab
Find the game with two or more download options

Expected behavior
Let us take the example of "download_options_other". ("{{count}} download options")
If a number ends in 1 but not 11, the form "вариант" is used:
21 вариант, 31 вариант, 101 вариант.
If a number ends in 2, 3, or 4 but not in 12, 13, or 14, the "варианта" form is used:
2 варианта, 3 варианта, 3 варианта, 4 варианта, 22 варианта, 23 варианта, 24 варианта.
For all other numbers, including all numbers ending in 5-9, 0, and numbers ending in 11-14, the "вариантов" form is used:
5 вариантов, 6 вариантов, 7 вариантов, 8 вариантов, 9 вариантов, 10 вариантов, 11 вариантов, 12 вариантов, 13 вариантов, 14 вариантов.
Screenshots

Additional context

OS: Windows 10
Hydra Version: 1.1.0
Additional information and context of your problem:

+1
There has to be third locale option for this in UA/RU/BE

I added this issue to the roadmap. The main collaborators are not so free right now, so feel free to work on it if you can.
Documentation about the library used for i18n on the project: https://www.i18next.com/translation-function/plurals

https://www.i18next.com/translation-function/plurals#interval-plurals could work fine, but it is a lot of intervals. interesting to know there are these intrincacies in RU/UK tho, remembered me about how frech counts from 60 to 99 heheh
https://www.lawlessfrench.com/vocabulary/numbers-and-counting-2/


Move xUnit tests to new folder

Motivation
I have a PR where there are many new xUnit tests.
It would also be useful to create new xUnit tests for public APIs.
The number of xUnit tests will increase and their ordering is required.
PR Summary

Move C# xUnit tests in new folder. This allows to put new xUnit tests in directory structure like Pester tests (in accordance with directory structure where cs files are).
Use an xUnit TestCaseOrderer attribute to sequentially process tests.
Update README.md

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[ ] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[ ] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[x] Not Applicable
OR
[ ] User-facing Documentation needed

[ ] Issue filed - Issue link:

Testing - New and feature

[x] Not Applicable or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

[ ] Add [feature] if the change is significant or affects feature tests

Fixed race condition when access powershell.config.json
https://ci.appveyor.com/project/PowerShell/powershell/builds/20262667/job/ta0qkiiemeyxkhyb#L644
Starting test execution, please wait...
645
dotnet : [xUnit.net 00:00:03.35]     PSTests.Parallel.FileSystemProviderTests.TestClearProperty [FAIL]
646
At C:\projects\powershell\build.psm1:1431 char:13
647
+             dotnet test --configuration $Options.configuration --no-r ...
648
+             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
649
    + CategoryInfo          : NotSpecified: ([xUnit.net 00:0...Property [FAIL]:String) [], RemoteException
650
    + FullyQualifiedErrorId : NativeCommandError
651

652
Failed   PSTests.Parallel.FileSystemProviderTests.TestClearProperty
653
Error Message:
654
 System.IO.IOException : The process cannot access the file 'C:\projects\powershell\test\xUnit\bin\Release\netcoreapp2.1\powershell.config.json' because it is being used by another process.
655
Stack Trace:
656
   at System.IO.FileStream.ValidateFileHandle(SafeFileHandle fileHandle)
657
   at System.IO.FileStream.CreateFileOpenHandle(FileMode mode, FileShare share, FileOptions options)
658
   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options)
659
   at System.IO.StreamReader..ctor(String path, Encoding encoding, Boolean detectEncodingFromByteOrderMarks, Int32 bufferSize)
660
   at System.Management.Automation.Configuration.PowerShellConfig.ReadValueFromFile[T](ConfigScope scope, String key, T defaultValue, Func`4 readImpl) in C:\projects\powershell\src\System.Management.Automation\engine\PSConfiguration.cs:line 358
661
   at System.Management.Automation.Utils.GetPolicySettingFromConfigFile[T](ConfigScope[] preferenceOrder) in C:\projects\powershell\src\System.Management.Automation\engine\Utils.cs:line 534
662
   at System.Management.Automation.Utils.GetPolicySetting[T](ConfigScope[] preferenceOrder) in C:\projects\powershell\src\System.Management.Automation\engine\Utils.cs:line 520
663
   at System.Management.Automation.Host.PSHostUserInterface.GetSystemTranscriptOption(TranscriptionOption currentTranscript) in C:\projects\powershell\src\System.Management.Automation\engine\hostifaces\MshHostUserInterface.cs:line 910
664
   at System.Management.Automation.Host.PSHostUserInterface.CheckSystemTranscript() in C:\projects\powershell\src\System.Management.Automation\engine\hostifaces\MshHostUserInterface.cs:line 430
665
   at System.Management.Automation.Internal.Host.InternalHostUserInterface..ctor(PSHostUserInterface externalUI, InternalHost parentHost) in C:\projects\powershell\src\System.Management.Automation\engine\hostifaces\InternalHostUserInterface.cs:line 23
666
   at System.Management.Automation.Internal.Host.InternalHost..ctor(PSHost externalHost, ExecutionContext executionContext) in C:\projects\powershell\src\System.Management.Automation\engine\hostifaces\InternalHost.cs:line 51
667
   at System.Management.Automation.ExecutionContext.InitializeCommon(AutomationEngine engine, PSHost hostInterface) in C:\projects\powershell\src\System.Management.Automation\engine\ExecutionContext.cs:line 1453
668
   at PSTests.Parallel.FileSystemProviderTests.GetExecutionContext() in C:\projects\powershell\test\xUnit\csharp\test_FileSystemProvider.cs:line 45
669
   at PSTests.Parallel.FileSystemProviderTests.GetProvider() in C:\projects\powershell\test\xUnit\csharp\test_FileSystemProvider.cs:line 49
670
   at PSTests.Parallel.FileSystemProviderTests.TestClearProperty() in C:\projects\powershell\test\xUnit\csharp\test_FileSystemProvider.cs:line 151
671
Results File: C:\projects\powershell\ParallelXUnitTestResults.xml
672
673
Total tests: 35. Passed: 34. Failed: 1. Skipped: 0.
674
Test Run Failed.
675
676

I'll move some fixes in separate PRs.

The PR contains commits from #8249 to fix race conditions. After merging #8249 I will rebase.
Please review the PR.

Close and open new PR #8356


Better pull error for fully-qualified images

When pulling a fully-qualified image that fails, we should not be talking about
registries/search registries in the the error message as it is not applicable.  If
a image that is fq'd and fails to pull, the error should be simplified.
$ sudo podman pull this-does-not-exist.example.com/foo
Trying to pull this-does-not-exist.example.com/foo...Failed
error pulling image "this-does-not-exist.example.com/foo": unable to pull this-does-not-exist.example.com/foo: unable to pull image, or you do not have pull access
$

Resolves: #1212
Signed-off-by: baude<EMAIL_ADDRESS>

LGTM, though we really need to look into getting the error strings from c/image out to the user - we are going to need them for debugging at some point

Correct, I think we got to allow @mtrmac to work through a common solution for Buildah and Podman, and I guess eventually CRI-O.

@TomSweeneyRedHat We aren't actually even printing c/image errors right now... So this is basically independent of that work. Once the c/image errors work happens, we can print the error for each pull on a given name.

However, I am going to reopen #1212 if this closes it, because we need the c/image error work to really resolve this

(ACK on the general idea, sure.  Ultimately c/image errors, or some better version of them, should be reported, but this is a clear localized improvement.)

Alright, second try
@rh-atomic-bot r+


VS doesn't kill refact-lsp after close

@reymondzzzz is this after visual studio is closed or just one of the files or both?

@reymondzzzz is this after visual studio is closed or just one of the files or both?
After visual studio is closed.

It exits correctly if VS is shut down correctly.
It looks like refact-lsp stays open if it panics inside some of its rust internals.


LearningRateScheduler: add verbose flag.

Add verbose flag, similar to ReduceLROnPlateau callback.
Don't increase lr. Lower current lr almost certainty means that another callback (like ReduceLROnPlateau in cifar10_resnet.py example) reduced it.

Yes, some papers advocate increasing lr from time to time, but it's an exotic advice, doesn't work AFAIK, and if needed we can add another parameter force or something.

I found a much better solution for not increasing lr or overriding other ways to set lr, see https://github.com/keras-team/keras/pull/8865

I was studying the Keras source, and looking at the current version of the LearningRateScheduler in master. I might be misunderstanding this commit, but I think this introduces a regression.
"K.set_value(self.model.optimizer.lr, lr)" line seems to me removed entirely as part of this PR, so "lr" is calculated, but not used?
https://github.com/keras-team/keras/pull/8829/files#diff-ca6be3eb60c04bc5fcd73355b7f99d0dL576

@fchollet, yes it's the real thing. Please merge https://github.com/keras-team/keras/pull/8888 ASAP. Accidentally removed in the third commit together with logic for not increasing lr.
@sumitgouthaman, thank you very much for your PR.


use Sass color variables

consistent colors (pixels widths, etc later) saves time and drift for ongoing development. This PR replaces all hard-coded hex values with $sass-variables. Eventually we should ideally do #1169 as well for our Vega visualizations.

https://mattferderer.com/use-sass-variables-in-typescript-and-javascript

closes #1169

Should adjust to only export and type the select colors that are needed in the HTML/vega.
Should also confirm if we can reduce some of these colors... I believe some were typos and some others were "close enough" versions of the same colors...

Sorry for letting this sit for so long!

Great work!


Adds Support of maxNumRowsPerTask in RealtimeToOfflineSegmentsTasksGe…

Fixes: https://github.com/apache/pinot/issues/12857
(Issue description: Currently RealtimeToOfflineSegmentsTask that is used to move real time segments to offline segments, does not have the ability to tune maxNumRowsPerTask. This is the parameter that determines the input to a task. Without this configuration, we end up creating one minion task, which takes in all the input (i.e all segments that meet the criteria to be converted to offline segments) which prevents us from using other minions.
There's no parallelism for this task.)
Solutions Considered:

Divide a task into multiple subtasks based on max num of rows per subtask. Once subtasks are generated, during the execution of subtasks, each subtasks atomically updates zookeeper state after it's execution is finished. The last subtask pending will updates the watermark after checking zookeeper if any subtasks is pending.
Divide a task into multiple subtasks based on max num of rows per subtask. Once subtasks are generated, during the execution of subtasks, update the segments metadata, marking them as processed for RTO task. In next iteration of generating RTO subtasks, filter out segments already processed and pick newer eligible segments and update the watermark as the startTime of the earliest segment selected.

Going ahead with solution 1, As solution 2 involves updating segment metadata for all input/selected segments.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 55.38%. Comparing base (59551e4) to head (4ab1610).
Report is 1411 commits behind head on master.

:exclamation:  There is a different number of reports uploaded between BASE (59551e4) and HEAD (4ab1610). Click for more details.
HEAD has 53 uploads less than BASE

Flag
BASE (59551e4)
HEAD (4ab1610)

integration
7
0

integration2
3
0

temurin
12
1

java-21
7
1

skip-bytebuffers-true
3
0

skip-bytebuffers-false
7
1

unittests
5
1

unittests1
2
1

java-11
5
0

unittests2
3
0

integration1
2
0

custom-integration1
2
0

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #14578      +/-   ##
============================================
- Coverage     61.75%   55.38%   -6.37%
- Complexity      207      703     +496
============================================
  Files          2436     2112     -324
  Lines        133233   111344   -21889
  Branches      20636    17677    -2959
============================================
- Hits          82274    61665   -20609
+ Misses        44911    44768     -143
+ Partials       6048     4911    -1137

Flag
Coverage Δ

custom-integration1
?

integration
?

integration1
?

integration2
?

java-11
?

java-21
55.38% <ø> (-6.25%)
:arrow_down:

skip-bytebuffers-false
55.38% <ø> (-6.37%)
:arrow_down:

skip-bytebuffers-true
?

temurin
55.38% <ø> (-6.37%)
:arrow_down:

unittests
55.38% <ø> (-6.37%)
:arrow_down:

unittests1
55.38% <ø> (+8.49%)
:arrow_up:

unittests2
?

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

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

Will update tests once approach seems good to go.


Rule idea: Require interpolation when string variables are used as custom identiefiers

Some CSS features like animations allow the user to specify a custom identifier for animation name.
So if you have animation-name as a string inside a Sass variable, you need to use interpolation to get rid of the quotes. There could be a rule that requires interpolation to be used in those cases.
So this SCSS:
$var: "my-anim";

.class {
  animation-name: $var;
}

@keyframes $var {
}

...gets compiled to:
.class {
  animation-name: "my-anim";
}

@keyframes "my-anim" {}

Which does not work. You need to use interpolation to get rid of the quotes:
$var: "my-anim";

.class {
  animation-name: #{$var};
}

@keyframes #{$var} {
}

...and that gets compiled to:
.class {
  animation-name: my-anim;
}

@keyframes my-anim {}

More info:

animation-name - MDN: https://developer.mozilla.org/en-US/docs/Web/CSS/animation-name
custom CSS identifier - MDN: https://developer.mozilla.org/en-US/docs/Web/CSS/custom-ident

I'm working on this feature, but as it is a bit complicated, I won't open a PR yet.

Pull request: #9

Great project! Can't wait to start using it. One thing: could you please set Assignees for issues if someone takes them up? For those who are thinking to conquer their shyness and try to contribute :)

Yeah, sure thing.


Key error: config.yaml

When I run:
model = lp.Detectron2LayoutModel(r"/content/drive/MyDrive/models/id_card_detection/config.yaml",
                                 r"/content/drive/MyDrive/models/id_card_detection/model_final.pth",
                                 extra_config=["MODEL.ROI_HEADS.SCORE_THRESH_TEST", 0.8],
                                 label_map={1: "id_card"})

everything works fine, but when I run:
model = lp.EfficientDetLayoutModel(config_path = "/content/drive/MyDrive/models/id_card_detection/config.yaml",
                                 model_path = "/content/drive/MyDrive/models/id_card_detection/model_final.pth",
                                 extra_config={"num_classes": 1})

I'm getting the error:
KeyError                                  Traceback (most recent call last)
<ipython-input-20-d4717462bf15> in <module>()
      1 model = lp.EfficientDetLayoutModel(config_path = "/content/drive/MyDrive/models/id_card_detection/config.yaml",
      2                                  model_path = "/content/drive/MyDrive/models/id_card_detection/model_final.pth",
----> 3                                  extra_config={"num_classes": 1})

3 frames
/usr/local/lib/python3.7/dist-packages/layoutparser/models/effdet/layoutmodel.py in __init__(self, config_path, model_path, label_map, extra_config, enforce_cpu, device)
    136         extra_config = extra_config if extra_config is not None else {}
    137
--> 138         self._initialize_model(config_path, model_path, label_map, extra_config)
    139
    140         self.output_confidence_threshold = extra_config.get(

/usr/local/lib/python3.7/dist-packages/layoutparser/models/effdet/layoutmodel.py in _initialize_model(self, config_path, model_path, label_map, extra_config)
    192                 bench_task="predict",
    193                 pretrained=True,
--> 194                 checkpoint_path=model_path,
    195             )
    196

/usr/local/lib/python3.7/dist-packages/effdet/factory.py in create_model(model_name, bench_task, num_classes, pretrained, checkpoint_path, checkpoint_ema, **kwargs)
      9         checkpoint_path='', checkpoint_ema=False, **kwargs):
     10
---> 11     config = get_efficientdet_config(model_name)
     12     return create_model_from_config(
     13         config, bench_task=bench_task, num_classes=num_classes, pretrained=pretrained,

/usr/local/lib/python3.7/dist-packages/effdet/config/model_config.py in get_efficientdet_config(model_name)
    679     """Get the default config for EfficientDet based on model name."""
    680     h = default_detection_model_configs()
--> 681     h.update(efficientdet_model_param_dict[model_name])
    682     h.num_levels = h.max_level - h.min_level + 1
    683     h = deepcopy(h)  # may be unnecessary, ensure no references to param dict values

KeyError: '/content/drive/MyDrive/models/id_card_detection/config.yaml

Hi @mzhadigerov ! The reason for your issue is that lp.Detectron2LayoutModel and lp.EfficientDetLayoutModel use different configuration "systems". So you might want to directly use some models here https://layout-parser.github.io/platform/ by copying the corresponding weights. For example, if you'd want to try with the MFD model using efficientdet, you can just hit the copy button (indicated by the red arrows in the screenshot below) and it will give you the code for instantiating the corresponding model lp.AutoModel("lp://efficientdet/MFD/tf_efficientdet_d0"). I would say this is more like a documentation issue rather than a bug -- and I'll update the modeling documentations very soon!

But I don't want to use a pre-trained model. I used https://github.com/Layout-Parser/layout-model-training to train a model on my own dataset. As you can see in the question, I'm referring to /content/drive/MyDrive/models/id_card_detection/config.yaml  and /content/drive/MyDrive/models/id_card_detection/model_final.pth.

Yes for now the layout-model-training only works for Detectron2 based models (and thus you can only use the yaml for lp.Detectron2LayoutModel. And we'll provide the code for training EfficientDet models soon.


Odd behaviour with packages included in both [packages] and [dev-packages]?

I originally made this issue as I had Django pinned in [packages] but also as an extra [dev-packages]:
[packages]
django = "~=1.11.10"
[dev-packages]
hypothesis = {extras = ["django"]}

And the resulting Pipfile.lock:
{
    "_meta": {
        "…": "…"
    },
    "default": {
        "…": "…",
        "django": {
            "hashes": [
                "sha256:ac4c797a328a5ac8777ad61bcd00da279773455cc78b4058de2a9842a0eb6ee8",
                "sha256:22383567385a9c406d8a5ce080a2694c82c6b733e157922197e8b393bb3aacd9"
            ],
            "version": "==1.11.10"
        }
    },
    "develop": {
        "…": "…",
        "django": {
            "hashes": [
                "sha256:7c8ff92285406fb349e765e9ade685eec7271d6f5c3f918e495a74768b765c99",
                "sha256:dc3b61d054f1bced64628c62025d480f655303aea9f408e5996c339a543b45f0"
            ],
            "version": "==2.0.2"
        }
}

The package is pinned twice with different versions and pipenv install merrily installs one of them based on whether or not --dev is used.
While looking at it a bit more I tested adding Django (this time not as an extra) with "~=2.0.2" in [dev-packages], it had the same kind of result.
This causing an unsatisfiable constraint would more sense to me at the moment. I think of [dev-packages] as being for development/testing related tools which are added on top of the environment specified in [packages], but that doesn't seem to be how things are handled at the moment.
At least for my case it was as simple as dropping the extra, but it took more poking around than I'd have liked. I think I would be happier if there was at a message at some point to highlight to that things may not be as the user expects.
Should [dev-packages] have the same constraints as [packages]? I am wondering if this is a side effect of something that is unreasonable to change, or if it is just odd behaviour that can be improved.

Arg! After looking into things more to write the above I found this is the same as issue #1220, I was only looking for extras related issues before.


Add debugging aids

Please review and merge PR #39 before reviewing this one!
Update
This PR primarily introduces debugging aids in the CapacityExpansion and Technology classes. I also added the ability to turn a list of technology objects into a pandas dataframe.

@nsryan2 @yardasol @LukeSeifert @abachma2 would one of you mind taking a look at this?

Thanks for the review @yardasol! I think I have addressed your comments.

@yardasol I think you can go ahead and merge this since others are reviewing #41

It all looks good to me!


Fix addOrSubtract function call

I need to make this change to get this module to compile with webpack, and actually scan a qr code in the browser.

Thanks !


Config-UI: Delete single connection endpoint

Description
We need a way to delete connections for Plugins.
Describe the solution you'd like

[ ] We need a button in the "Connections" tables for each plugin, on connection items. (Frontend)
[ ] We need a backend route to delete the connection information. (Backend)

Has the Feature been Requested Before?
No.
Additional context
PLEASE NOTE: Only Jira currently supports multi-sources in the DB. So the Jira plugin can delete source from the database. GitLab and Jenkins will need to simply assign their credentials in the .env file to empty.

Make sure pop up a Confirmation dialog


Feat add beta auto publish

CI add beta auto publish

Codecov Report

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

@@           Coverage Diff           @@
##           master      #92   +/-   ##
=======================================
  Coverage   78.79%   78.79%
=======================================
  Files          25       25
  Lines         962      962
  Branches      186      186
=======================================
  Hits          758      758
  Misses        204      204

Continue to review full report at Codecov.

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

需要同时更新一下 https://github.com/ice-lab/iceworks/blob/master/.github/CONTRIBUTING.md#pull-request-guidelines

reviewed


Include separate cyclepaths and trails

Splitting from a-b-street/abstreet#139. The goals here are just:

connectivity -- can bikes successfully route over things like the Burke Gilman as expected
geometry + rendering -- does it look reasonable
bike-only -- just model as one or two bike lanes, ignore foot access for the moment. (Once this bug is solved, a-b-street/abstreet#139 can add a special lane type that has mixed access)

Moved to the osm2streets repo, where I'm renewing efforts on this. No promises, but State of the Map next week is the much-needed deadline...

The default in osm2streets is now using separate cycletracks and footpaths as they are in OSM. Still have ongoing efforts to improve the geometry and optionally zip/snap things together


Form Validation not working if inputs are "children" in parent components default slot

Versions:

VueJs: 2.4.1
Vee-Validate: 2.0.0-rc.7

Description:
Attempting to validate whole form "on submit" with the body of my form being passed as the "default slot" within another component. this.$validator.validateAll() appears to be returning different results depending on whether or not the wrapper component is "native" or not.
Steps To Reproduce:
Use this codepen:
https://codepen.io/anon/pen/YxzOjZ
Click submit, it will console.log() "success" even though the field is required. But if you change <form-wrapper> to just <form>, it will display the error message and will console.log() "failed" as intended.

Seems to work fine with the latest changes:
https://codepen.io/anon/pen/jLPwMP
I've removed functional since it blocks the rendering changes of the errors.collect result which is expected, from the Vue docs on functional components:

It doesn’t manage any state, watch any state passed to it, and it has no lifecycle methods. Really, it’s just a function with some props.


start of checkstyle inclusion...

an initial pr to get the work going on checkstyle
this fails the build....should we turn that off? merge it in and fix stuff in a different pr?

Change Travis.yml.to not run check for now. Have it run just test.


[18.03] firefox: provide clang configure flags on i686

Motivation for this change
With the upgrade to firefox 62 (and potentially before) clang flags are now required on i686.
This might just work as it is. I haven't had the time to run/check for rebuilds. I intend to do that in the evening / during the day.
It seems that this would also be required on stable and the 18.09 release branch.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
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)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-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)
[x] Fits CONTRIBUTING.md.

Compiles and runs on my machine.


Feature/backend settings password admin app sprint#91

Added forgot password flow pending mail server .

@askinss
resolved

Thanks for quick resolution.


Added MarkDown formatting to examples/cifar10_resnet.py

Summary
This PR adds MarkDown formatting to examples/cifar10_resnet.py.
Result:

Related Issues
https://github.com/keras-team/keras/issues/12219
PR Overview

[ ] This PR requires new unit tests [y/n] (make sure tests are included)
[ ] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[x] This PR is backwards compatible [y/n]
[ ] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

Thanks for the PR! Could you put the table in the markdown header? Thanks!

Sure thing! Give me a sec.

This was the cleanest solution I came up with.
Results:

Thanks!


Landing page: Update opinionated template images to not use hipster ipsum

1x and 2x versions. These probably need to be renamed.
ready for dev, @ericlindley-g to either close or move to the next sprint for dev. im open to either.

Bumped into fixit sprint — assuming the images are sized the same as the old ones, is this replacement something you could do, @spacedino ?

i forgot about these two.

PRs:

swap out images so the new ones have lorem ipsum #365
replace 2x coming soon image #364


Do not add empty user displaynames in rooms

According to the logic in User.get_display_name, this is how it should have been. Also fixes an exception thrown in Room.display_name (when sorting).
Signed-off-by: Valentin Deniaud<EMAIL_ADDRESS>

LGTM. Thanks!


Inconsistency in tensorflow version requirements in /doc

development/development_guide.rst lists tensorflow 1.3 as the requirement, whereas introduction/circuits.rst lists tensorflow 2.0. The former should be changed to 2.0

Hey @ilan-tz! Good catch, you are right: the development guide is outdated. Would you mind making a pull request fixing this bug?

Closed by #468


Version attribute for aiven_kafka_schema isn't incremented on apply

What happened?
We see that the version attribute for aiven_kafka_schema isn't getting incremented on apply, even if a new version is created. We observe this using  an output.
For example (all apply's below change the schema in some way):
Apply 1 -> version = 1
Apply 2 -> version = 1
Apply 3 -> version = 2
...

So it appears that the provider observes the previous version bump only on a subsequent plan/apply.
What did you expect to happen?
Version should have been incremented right after apply.
What else do we need to know?
Terraform version 1.5.5, Aiven terraform provider version 4.14.0

Hey, @jpalomaki! 👋
Thank you for raising this issue. We'll check things out and will let you know shortly.

Hey, @jpalomaki! 👋
We've introduced a fix. It should be available with the next release.


Level:none is broken

Including Level: none in a spec's metadata is broken with the following error:
FATAL ERROR: Not all required metadata was provided:
    Missing a 'Level' entry.

https://tabatkins.github.io/bikeshed/#metadata-level says this should work.

Sigh, whoops, that's definitely my bad. Let me fiddle this back into working.


Pin image not work on Android

Hi,
My pin image works on Ios but not works on Android, this is my code :
  var pin = new TKCustomMapPin
                            {
                                Position = new TK.CustomMap.Position(elements.Latitude, elements.Longitude),
                                ID = elements.SerialNumber,
                                Title = elements.FirstNameUser,
                                Subtitle = elements.LastNameUser,
                                ShowCallout = true,
                                IsDraggable = false,
                                Anchor = new Point(0.5d, 1d)
                            };

                            pin.Image = ImageSource.FromResource("Mu.Resources.Drawables.Map.iconmap.png");

                            Pins.Add(pin);

Any idea to resolve that? I'm with 2.0.0-pre3.

Why have this been closed. The problem still occurs. In fact I used the same library version from the sample with the latest TK version.


[4.0] [Physics 3D] Rigidbody3D/MeshInstance3D act jittered on plane CollisionShape3D

Godot version
v4.0.beta3.official [01ae26d31]
System information
Windows 11
Issue description
Trying to create simple physics with plank object, I notice the physics make the objects sometimes very jittery.
What I tried:
I tried almost every settings in RigidBody3D, and Physics in Project Settings,
What I imagine:
That my objects with physics are more “stable”.
https://user-images.githubusercontent.com/4235617/198302173-f47b2120-6971-4f03-8e4f-5352111af161.mp4
Steps to reproduce
Create a ground:

Add StaticBody3D.
Add as child to the body a MeshInstance3D, create a PlaneMesh3D for mesh property.
Add as child to the body a CollisionShape3D, create a WorldBoundaryShape3D for the shape property.

Create object:

Add a RigiBody3D
add as child a MeshInstance3D, create a cube with the size (0.2, 0.03, 1)

add as child a CollisionShape3D, fit the size with the MeshInstance3D

Minimal reproduction project
Please find the attached Godot project from the video above:
PhysicsTest.zip
Controls:
Mouse Click: to add object
Space: remove all objects

Any updates on this?


Rename sidecar-injector subchart to sidecarinjector

This is part of a series of PRs to improve the Helm charts support for the sidecar injector & Istio 0.5.0 in general.
It appears when using a hyphen in a chart name, Helm does not properly pass values to the subchart from the top level values.yaml file. I ran into this when adding a new caBundle field to the Helm chart, to allow customising the webhooks ca bundle without the need for the pre-made scripts for templating manifests.
This PR renames the chart from sidecar-injector to simply injector, which resolves issues passing subchart values from the top level chart.
/cc @luisyonaldo

I can confirm the bug. Maybe sidecarinjector is a better name. I'm ok with injector but we are trying to make chart more in sync with plain manifest.
Thanks @munnerz

cc @linsun @sdake

Let me know either way and I'll make a change! 😄
I've managed to stand up an Istio 0.5.0 control plane with mTLS using the helm chart now after a few adjustments. I'm currently running into issues with the bookinfo application, although I'm having that same issue with a vanilla manifests deployment (using istio-auth.yaml) so starting to wonder if something more is broken here 😄

/ok-to-test

@munnerz: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-smoke.sh
6293d67794c4ea96475d9201845208ae6dacb3a3
link
/test e2e-smoke

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-smoke

@munnerz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-smoke.sh
6293d67794c4ea96475d9201845208ae6dacb3a3
link
/test e2e-smoke

prow/istio-pilot-e2e.sh
6293d67794c4ea96475d9201845208ae6dacb3a3
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thank you for raising this issue @munnerz I would prefer sidecarInjector to be somewhat consistent with other places when we call sidecar-injector.  for example, in our docs and istioctl, we call it sidecar injector or sidecar-injector.

@munnerz can you resolve the conflict in requirements.yaml?  A change to move ingress to its own subchart modified this file.
Cheers
-steve

@sdake done

@munnerz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-smoke.sh
6293d67794c4ea96475d9201845208ae6dacb3a3
link
/test e2e-smoke

prow/istio-pilot-e2e.sh
6293d67794c4ea96475d9201845208ae6dacb3a3
link
/test istio-pilot-e2e

prow/istio-unit-tests.sh
d8befe2f6e5b916fbfa017180b90d9d79ae738d9
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@sdake resolved (another) conflict

@munnerz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-smoke.sh
6293d67794c4ea96475d9201845208ae6dacb3a3
link
/test e2e-smoke

prow/istio-pilot-e2e.sh
6293d67794c4ea96475d9201845208ae6dacb3a3
link
/test istio-pilot-e2e

prow/istio-unit-tests.sh
bbe53846184bf681cd62a0330c663e01af93a7d1
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@munnerz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
bbe53846184bf681cd62a0330c663e01af93a7d1
link
/test istio-unit-tests

prow/istio-pilot-e2e.sh
bbe53846184bf681cd62a0330c663e01af93a7d1
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@munnerz: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
bbe53846184bf681cd62a0330c663e01af93a7d1
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@linsun did you want @mrunnerz to change this PR to %s/injector/sidecarinjector?  Unfortunately it appears a dash won't work correctly with Helm.

PR rebased on latest master.
@linsun I'm happy to change this if needs be. What would be preferable? As @sdake says, we can't use a hyphen so we'll likely need to name the chart sidecarinjector if so 😄
Let me know and I'll get it updated! This currently blocks using the injector at all, as the caBundle cannot be configured.

@munnerz: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
bbe53846184bf681cd62a0330c663e01af93a7d1
link
/test istio-pilot-e2e

prow/istio-unit-tests.sh
5f3fdcfe1ec6906cdc54d20efa02df639577cf6c
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@munnerz @sdake yes, would be nice to have it changed to be consistent with other docs and cmds, if you don't mind.  Thank you!  (understood about the hyphen issue, thanks for spotting that!)

@linsun I've updated to call the chart sidecarinjector. PTAL 😄

Thank you @munnerz!   Do you know how to trigger the retest on circleCI?  we need to get that test passing.

@linsun I've typically seen the bot spit out a list of tests that have failed after the jobs finish up - which permits a /retest or test on a specific job.

Although it looks like Circle doesn't retest when a PR is re-opened 😢

@munnerz: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
ffe8ad15d1e91bb9ec049b60481a219d2984a020
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sdake can you try going here: https://circleci.com/gh/istio/istio/30513?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link and hitting "Restart"? I am told I need write permission to the repo in order to restart tests 🤔

Never mind, I just rebased on master in order to re-trigger everything.. 🤞

@munnerz: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
9e1e9d29fc672f75cc995ac75fe4ddf63ac33bfc
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

It failed again... I don't think I have write permission to trigger the circle CI re-build for you.  can you follow the step here to see if you can run the job since you own the job?
https://github.com/istio/istio/wiki/Working-with-CircleCI

can you rebase and resolve conflicts ?
I pinged #3037 for the test failure

also the circleci shadow error is fixed on master sometime today so you do need to rebase if only for that (#3471 )

Btw, you can retest circle CI by commiting and empty commit. I allways do git commit --allow-empty -m retest, that should force a retest

/test istio-pilot-e2e

Btw, you can retest circle CI by commiting and empty commit. I allways do git commit --allow-empty -m retest, that should force a retest

a) that creates noise compared to clicking "retest"
b) that doesn't help with master fixes, for which pull/merge/rebase is what circleci forces you to do

@munnerz PR needs rebase

@sdake ^

@sdake @ldemailly rebased on master. :crossed_fingers: it passes now

@ajy can you "/ l g t m" if you're ok with this ?

I just opened #3536 for something else, but fixed the underlying problem here by changing the top level value name in values.yaml.tmpl The hyphen itself isn't a problem, but that the sidecar injector values were defined as sidecarInjector rather than sidecar-injector to match the chart name.

ping @munnerz @ayj

I think we can close this PR with the merging of https://github.com/istio/istio/pull/3536.


zkUNO - UNO on chain

Proposal to build a zkUNO card game powered by Plonky3.

The application under review [ Moved to next step for judging the application ]


Keycloak

initial wrapper around http that will enable SSO

This just adds the wrapper, it doesn't change anything specific in the UI, right?

Right, it doesn't do anything in the UI atm. When you where already logged in it will add the token to the request to the backend. We could create a login button or a check in the UI in future as well.


Wrong schema generation for composed types

When configuring inheritance through allOf property in the @Schema annotation, the generated OAS 3 specification is wrong. It is indeed shown corectly in swagger-ui, however when using the specification to generate client code, the generated code is not correct.
With this java classes
class Parent {
	private String sharedProperty;
}

@Schema(allOf = Parent.class)
class Child extends Parent {
	private String property;
}

the following OAS schemas are generated:
...
{
  "components": {
    "schemas": {
      "Parent": {
        "type": "object",
        "properties": {
          "sharedProperty": {
            "type": "string"
          }
        }
      },
      "Child": {
        "type": "object",
        "properties": {
          "property": {
            "type": "string"
          }
        },
        "allOf": [
          {
            "$ref": "#/components/schemas/Parent"
          }
        ]
      }
    }
  }
}

According to the OAS correct would be:
...
{
  "components": {
    "schemas": {
      "Parent": {
        "type": "object",
        "properties": {
          "sharedProperty": {
            "type": "string"
          }
        }
      },
      "Child": {
        "allOf": [
          {
            "$ref": "#/components/schemas/Parent"
          },
          {
            "type": "object",
            "properties": {
              "property": {
                "type": "string"
              }
            }
          }
        ]
      }
    }
  }
}

FYI, in your example you named it baseProperty and in the OAS you switched to sharedProperty

See #3049 which points out that if this we're fixed to generate the correct OAS, that OAS would not be parsed correctly and would end up duplicating the "property" property.

I wonder if the order of items in the allOf matters. You show it with the $ref last. Can you try it with the $ref first?

Yes exactly, refs have to go after the actual object

Sorry I was wrong. Actually ref should be first, otherwise the Child class does not inherit from Parent at all.
See the example https://github.com/m-kay/oasgenerateallofexample, generate the code with mvn clean install and compare the generated code in the packages reffirst and reflast.

thanks for reporting this, also duplicates #3197 and #3063; fixed in #3262 (subtypes must be added to parent to be correctly resolved, see #2863 and related test


What is the license for this repo?

https://github.com/yiiext/nested-set-behavior/commit/85cb0d2dd6129ab56d98014170af09eeed0f2694


Can not get the text of an "input" element with a type other than "text"

@Va1tra Good catch! Could you please submit a pull request with the fix?

https://github.com/WileyLabs/teasy/pull/203

fixed in version 2.0.13


Inconsistent alignment of checkboxes/radioboxes and their labels in various multi-line constellations

codepen: https://codepen.io/readonly/pen/PjzQgv
When a <a> or <strong> (or most other markup) is added to a checkbox/radiobox label that spans multiple lines, the horizontal alignment of the label's text is off. Depending on the browser/os to a different degree.
Firefox is the most consistent on all OSes. In Chrome (tested on Linux and OSX) the first line of a text-only label is too far to the left, while those with more markup look fine. Same happens on IE 11 on Win7.
In addition, the vertical position of the checkbox or radio itself is affected by those combinations. In FF (both Linux + OSX) the radio/checkbox next to a text-only label is ~1px too far too the top, while next to a label with markup it is ~1px too far down. On Chrome, it is fine for text-only labels, but is ~1px too far to the top for labels with markup. The IE11 behavior seems to follow the Chrome behavior.
The issue is more obvious if a radio group or checkbox group features different kinds of labes (see codepen).
bootstrap: v4-alpha6

this is apparently due to the these lines
.form-check-input:only-child {     position: static; }
does anybody know with what purpose this was added?

 Yeah, that's a misuse of :only-child on my part. I think the only straightforward option is to wait for position utilities to arrive and replace that broken CSS and it's docs example with:
<div class="form-check">
  <label class="form-check-label">
    <input type="checkbox" class="form-check-input position-static">
  </label>
</div>


[bugfix] Filter out empty strings before writing shell

Use filter to filter out empty strings before finalizing shell.

User raw string to suppress invalid escape sequence warning.

Fixes #407

Codecov Report

Merging #410 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #410      +/-   ##
==========================================
+ Coverage   91.22%   91.22%   +<.01%
==========================================
  Files          70       70
  Lines        8582     8583       +1
==========================================
+ Hits         7829     7830       +1
  Misses        753      753

Impacted Files
Coverage Δ

reframe/core/shell.py
94.82% <100%> (+0.09%)
:arrow_up:

Continue to review full report at Codecov.

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

@jenkins-cscs retry all

@jenkins-cscs retry dom


Update language for job results

This change updates the text document language used when opening downloaded job results to json. This enables syntax highlighting and formatting.

Confirmed after local build that the downloaded results show up as json now:


Shrinkwraps dependencies so we can secure our own fork of nodeunit

Fixes https://github.com/tessel/t2-cli/issues/557#issuecomment-232981469.
See the above issue for commentary on the source of a bug that's causing intermittent test failures.
Basically, there is a bug in one of our well-maintained dependency's unmaintained dependency (nodeunit). I have opened a PR on nodeunit but it's unlikely to be merged and published to npm. That means our dependency will be unlikely to pull the new version in to its project.
npm shrinkwrap gets around that because it lets us specify where we are pulling those dependencies from. Instead of pulling nodeunit from the original repo, this shrinkwrap.json pulls it in from our own fork which already has a fix from the bug.
I'm honestly not sure if this is the most maintainable way to fix the problem but it does seem to work for the short term

GAHHH maybe this didn't fix the issue? https://ci.appveyor.com/project/rwaldron/t2-cli/build/1095/job/j657nph6t9b5sjht

Naw, this isn't necessary. The appropriate fix is to explicitly require our dependencies where they are depended on ;)
Since I agreed with you here, and I introduced the bug, I will do the work to update the files.


Good first issue

Hi guys, I see your project in OSPP. I'm wondering are there any good issues for kick start?

IDK how deep you want to dive into this project
But firstly you can start with the llama model and run the demo

download llama from hugging face
convert param: https://github.com/openppl-public/ppl.pmx/tree/master/model_zoo/llama/huggingface#convert-model-params
run an example:https://github.com/openppl-public/ppl.pmx/tree/master/model_zoo/llama/huggingface#testing-model

@Alcanderian thanks, I will have a try :)

@Alcanderian What we need to do in this issue?


Getting Bridge Password?

I am new to docker, and I am having a very hard time figuring out how to get the bridge password that is created when hydroxide first logs in. I tried docker log, but that didn't show that output.
Is there a specific command I can run when doing docker-compose to re-create the container that will have hydroxide show me the output? Or is there a better method?

Hi @DiscordiaChaos :wave:
Once the container is running, you should find the bridge password at /opt/hydroxide-container/data-hydroxide/info.json
The file should contain the following data:
{
	'user'<EMAIL_ADDRESS>	'hash': 'yourhyDrOxIdEhaShHerE' // <---- for example sake, it will look different of course
}

On your local system, you can draw this information from that folder to make a SMTP call on the same port as your hydroxide container.
Are you familiar with python? If so, the send_mail.py file provides an example of how to access that file and send a test message.
Does that help answer your question?

Thank you, that does help.
I did try the send_mail.py just now but I saw this error (emails replaced below as to not expose them)
root@www:~/hydroxide-docker-master# python send_mail.py
Traceback (most recent call last):
  File "send_mail.py", line 115, in <module>
    example()
  File "send_mail.py", line 104, in example
    send('SENDER','RECEIVER','Test message','This is a test message to verify that this script is working accordingly.')
  File "send_mail.py", line 72, in send
    mailserver.login(account, password)
  File "/usr/lib/python2.7/smtplib.py", line 623, in login
    raise SMTPAuthenticationError(code, resp)
smtplib.SMTPAuthenticationError: (454, '4.7.0 Invalid username or password')

I will try to completely redo the container fresh and see if it helps. The login and password should be correct, but I'll make 100% sure of it when redoing it.

Gotcha! Yes, try cloning the repository again. Carefully enter your password into the docker config, and let me know if you see the same error.

Alright, so I downloaded it fresh and everything,  but it's still saying this on the send_mail.py:
 File "/usr/lib/python2.7/smtplib.py", line 623, in login
    raise SMTPAuthenticationError(code, resp)
smtplib.SMTPAuthenticationError: (454, '4.7.0 Invalid username or password')

I checked in that account's authentication logs and it shows the login was successful from the server's IP address. Should I open a different ticket to debug that?

Oh it was User Error! I ran docker run -it -p 1025:1025 -v /opt/hydroxide-container/data-hydroxide:/data hydroxide and the docker-compose command after... no wonder I was having issues. Thank you for your help. :)

Thanks for reporting back @DiscordiaChaos, that's great news that it worked out!


display annotations on sequences and alignments

Original report by GavinH (Bitbucket: 557058:e40c23e1-e273-4527-a2f8-5de5876e870d, ).

Add a get_figure method to Annotatables for displaying feature tracks.

Closed on bitbucket


docs: Explain Loaded<MyEntity, never> Type.

Previously,const myEntity = await em.find(MyEntity, {id: 123}) would return typeof myEntity -----> MyEntity | null.
With newer Versions, a type of Loaded<MyEntity, never> is returned. Why isn't there any null being returned anymore for NOT FOUND  and will a negation check like if (!myEntity) {} always use the if-branch when the entity for id=123 was not found (previously it would be like if (myEntity !== null || myEntity !== undefined)? Thanks for a clarification.

Thanks for the poke, I know this is missing. I would suggest you take a look at the joist orm docs, as this concept is mostly implemented based on that:
https://joist-orm.io/docs/goals/type-safe-relations#the-magic-escape-hatch
Some differences from joist:

the type safe escape hatch is called $ or get() (as a method, not a property)
this is opt in, it matters only for Reference wrapper and Collection properties

Thanks!
Another related thing is:
Let's say MyEntity has another relation (N:1) to OtherEntity.
If myEntity.otherEntityis not loaded, I could previously use await wrap(myEntity.otherEntity).init() to initialize it.
However, I am seeing now that const other = await wrap(myEntity.otherEntity).init() would always assign the type never to the const other.
So coming from your answer above, I should now use em.load() or em.populate() and const other = await myEntity.get.otherEntity() to load single entity relations.
My goal is the following:
I cannot guarantee (also or reusability and conditional function branches) that a relation is loaded. Therefore, I want to check whether other is not undefined aka loaded or, if not, load it safely and then check again and handle the exception or go to a different if-branch.
Is this the right approach? A hands on example would be cool, as this lazy loading etc. is pretty amazing.

Another thing related to the Loaded<MyEntity, never> Type topic posted first is making nullable attributes known to typescript.
export class OrganizationInvitation extends BaseEntity<
  | 'other'
  | 'otherOptional'> { ... }

@Property({ type: OtherEntity, nullable: true })
other?: OtherEntity;

@Property({ type: String, nullable: true })
otherOptional?: string;

The BaseEntity looks like this and would take in the optional params by the implementing class and define them as OptionalProps.

// id is there automatically but put here just not to cause confusion
export type OPTIONAL_BASE_ENTITY_ATTRIBUTES_TYPE =
  | 'id'
  | 'createdAt'
  | 'updatedAt'
  | 'deletedAt';

export abstract class BaseEntity<O = OPTIONAL_BASE_ENTITY_ATTRIBUTES_TYPE> {
  // O is set to be OPTIONAL_BASE_ENTITY_ATTRIBUTES_TYPE by default.
  // When O is set however, we make sure that O & OPTIONAL_BASE_ENTITY_ATTRIBUTES_TYPE are optional.
  [OptionalProps]?: OPTIONAL_BASE_ENTITY_ATTRIBUTES_TYPE | O;

  @PrimaryKey({
    type: 'uuid',
    nullable: false,
    defaultRaw: 'uuid_generate_v4()',
  })
  public id: string = v4();

...

}

Hope it helps someone.

Optional properties are a different story, its unrelated to the Loaded type - Loaded type directly intersects the T, in other words its always assignable to T.
This is already documented here: https://mikro-orm.io/docs/property-validation

However, I am seeing now that const other = await wrap(myEntity.otherEntity).init() would always assign the type never to the const other.

You can use em.populate(myEntity, ['otherEntity']) instead, it will return the same entity reference, with adjusted type based on the populate hint.

I cannot guarantee (also or reusability and conditional function branches) that a relation is loaded.

If you want guarantees, you need to use the async API (either init/load on collection/reference or em.populate in general).

Therefore, I want to check whether other is not undefined aka loaded or, if not, load it safely and then check again and handle the exception or go to a different if-branch.

Its never undefined - not loaded relation will always be represented by entity reference (entity with just PK).
https://mikro-orm.io/docs/entity-references

Cool, thanks! The optional property thing solved the assignment of potentially non-defined attributes, that is why I mentioned it. Perhaps you can add the example with the BaseEntity to your docs because you cannot have OptionalProps defined in BaseEntity and then again in the inheriting Entity without overwriting it.

Yeah agreed its a common question, got it on stack overflow very early after 5.0 release :] just checked, and it was less than 24 hours :D
https://stackoverflow.com/questions/71013801/optionalprops-in-mikro-orm
I am thinking about a separate type safety section in the docs with more examples about both the populate type hints and optional properties.

I am seeing the never issue again.
await this.getEm().populate(myEntity, ['other']);
if (!myEntity.other) {
  throw new Error('myEntity.other is not defined');
} else if (
  myEntity.other &&
  !myEntity.other?.myJsonAttribute
)

results in the typescript error Property 'myJsonAttribute' does not exist on type 'never'.ts(123) for the line  !myEntity.other?.myJsonAttribute. How can I solve this? myJsonAttribute marked as optional with BaseEntity.
Thanks!

It wont change the type of your entity, it will return it. You would have to have a new variable:
const loadedEntity = await this.getEm().populate(myEntity, ['other']); // loadedEntity === myEntity, its just a different type
if (!loadedEntity.other) {
  throw new Error('myEntity.other is not defined');
} else if (
  loadedEntity.other &&
  !loadedEntity.other?.myJsonAttribute
)  {
throw new Error('myEntity.other.myJsonAttribute is not defined');
}

const loadedEntity = await this.getEm().populate(myEntity, ['other']);

has the type const contact: Loaded<MyEntity, "other">[] (NOTE: it is an Array).
However, it is supposed to be of type Loaded<MyEntity, "other"> (not an Array). Any hints how to solve this? Using the firs element of the Array?

Just destruct, this is correct behaviour in v5, it always returns array as it was giving hard times to TS inference otherwise:
const [loadedEntity] = await this.getEm().populate(myEntity, ['other']);

Perfect, I believe this will help a lot!

To whom it interests,
if myEntity: MyEntity is already set as a parameter for the function or whatever, you can simply overwrite it with
[myEntity] = await this.getEm().populate(myEntity, ['other']);, which is nice to not having to deal with so many variables that are essentially the same thing.

And of course [myEntity] = await this.getEm().populate(myEntity, ['other.connectedEntity.connectedUsers']); works as well to load the the entire chain.


Add tree test to figure out miscalculation of tree size

What this PR does / why we need it?
Add test-case to figure out miscalculation of indextree size during concurrent tree editing
Any background context you want to provide?
What are the relevant tickets?

Fixes #
Checklist

[ ] Added relevant tests or not required
[ ] Didn't break anything

Summary by CodeRabbit

Tests

Added a new test case to verify correct index tree size calculation during concurrent editing.

@JOOHOJANG, @raararaara Thanks for your contribution.
I continued this PR with https://github.com/yorkie-team/yorkie-js-sdk/pull/844.