refactor: update dependencies to be compatible with vaadin 22

Close #4

This should be a new version of the component in order to be able to create a server-side component compatible with Vaadin 22.
I've updated the component version to 1.0.0.


Implement routing adjacency status capture for OSPF and BGP

Is your feature request related to a problem? Please describe.
When upgrading PAN-OS on firewalls using the pan-os-upgrade utility, it is important to ensure that the routing adjacencies and neighbor relationships are maintained and re-established correctly after the upgrade. Currently, the utility does not have a built-in mechanism to capture and compare the routing adjacency status before and after the upgrade, which may lead to situations where routing issues go unnoticed or are difficult to troubleshoot.
Describe the solution you'd like
Enhance the pan-os-upgrade utility to capture the routing adjacency status for OSPF and BGP before and after the upgrade process. The utility should:

Before initiating the upgrade, determine if OSPF or BGP is currently running on the firewall.
If OSPF is running, execute the appropriate CLI command to capture the status of OSPF neighbors.
If BGP is running, execute the appropriate CLI command to capture the status of BGP neighbors.
Store the captured outputs in a structured format along with metadata such as the protocol, timestamp, and device information.
Proceed with the normal upgrade process.
After the upgrade is completed and the firewall is back online, re-check if OSPF or BGP is running on the upgraded firewall.
If OSPF is running, execute the same OSPF neighbor status command and capture the output.
If BGP is running, execute the same BGP summary command and capture the output.
Store the post-upgrade outputs in a similar structured format as the pre-upgrade outputs.
Compare the pre-upgrade and post-upgrade outputs to identify any changes in the routing adjacency status, such as neighbors that are no longer present or have changed states.
Generate a report or display the comparison results to the user, highlighting any discrepancies or potential issues.

Describe alternatives you've considered
An alternative approach could be to rely on external monitoring systems or network management tools to track the routing adjacency status. However, this would require additional integrations and may not provide a seamless experience within the pan-os-upgrade utility itself.
Additional context
Here are a few additional points to consider:

The utility should handle scenarios where OSPF or BGP is not running on the firewall gracefully and skip the adjacency status capture for those protocols.
Consider providing options to customize the CLI commands used to capture the neighbor status, as different network environments may have specific requirements or configurations.
Handle cases where the neighbor status commands may fail or return unexpected output, and provide appropriate error handling and logging.
Provide clear documentation and examples on how to interpret the comparison results and troubleshoot common routing adjacency issues.
Consider integrating with external tools or APIs to enable automatic notification or ticketing systems when routing adjacency issues are detected.
Update the project's documentation to include information about this new feature, explaining how it helps monitor and maintain routing adjacencies during the upgrade process.

By implementing this feature, the pan-os-upgrade utility will provide an additional layer of assurance by capturing and comparing the routing adjacency status for OSPF and BGP before and after the upgrade. This will help identify potential routing issues early, ensure a smooth transition, and minimize disruptions to network connectivity.

will revisit when time permits, closing for now


added functionality to swap login method display text for value defined in config.js

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@b5c908f). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #125   +/-   ##
=========================================
  Coverage          ?   79.83%
=========================================
  Files             ?      117
  Lines             ?     7742
  Branches          ?     1167
=========================================
  Hits              ?     6181
  Misses            ?     1561
  Partials          ?        0

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 b5c908f...eb7ec23. Read the comment docs.


Bug fix: not displaying up arrow when reaching the last menu item

If number of menu items was below maxRows, the up icon wasn't displayed when it was supposed to.

The up-arrow indicates that there's more above the item displaying in the first line of the LCD. So, when menuItemsCount < maxRows, the first item is always in the first line (no items above it), therefore there's no need to show the up-arrow.

Hmm.. I thought the purpose of the icons is to show when it still possible to go up/down regardless of how many rows.
If it supposed to work as you said, then there is a bug with the down arrow that is showing when there are less menu items than maxRows.
Are you sure that was the purpose of the arrows?
if (top == 1) {
            //
            // Print the down arrow only
            //
            lcd->setCursor(maxCols - 1, maxRows - 1);
            lcd->write(byte(1));
}

I don't unterstand the arrow behaviour. For me, they should indicate if there are menu-items above or below the items currently displayed. That way the user knows, if there's more that is currently not visible.
Created a fork and adjusted the behaviour for me. If you'd like to have a look: https://github.com/gswdn/LcdMenu


Error when building composite map with zero significantly enriched attributes

If you build an enrichment landscape where no attributes are significantly enriched, you will get an error when you attempt to build a composite map.
@abarysh Should we disable steps 3/4 in this case?

Yeah, but ideally, the user would get a message about why steps 3 and 4 are not available. So how about leaving the button active but then popping out a message (and not proceeding to steps 3/4) if no attributes are enriched?


Use CxxString instead of CxxStringWrapper

This is a PR because we aren't 100% sure this isn't a memory leak, so @karikera please merge this if there is no leak.

it makes a memory leak.
but it seems it should not make a memory leak.
So I fixed it and merged it


Documentation changes for supporting RSV in bulk result upload

Why should we do this?

Provide additional support and instruction to SimpleReport users who want to use the CSV Results Bulk Uploader to capture and submit RSV results to California.

Action Requested

Update guidance documentation for bulk upload users who will be sending RSV results to CA.[Google Doc]
Update the banner copy appearing on the uploader page, reading "What is the bulk results uploader?"
Add an RSV page to the static site under Test for other diseases

Acceptance Criteria

Guidance for both the documentation and the banner will be updated when support for RSV data capture in the CSV Bulk Results Uploader is live on production.

Additional Context (Recommendations from Laura Kerry & Laura King)

add “and RSV” to the existing flu comms/upload guide once that’s live -  RSV is for CA submitters only
RSV support is for bulk upload only. No guidance would be needed for single-entry
all fields will be the same on bulk upload for RSV as they are for flu
add specific guidance for where to find RSV-specific values - link to RSV LOINCs instead of flu
consider adding an RSV page under Test for other diseases in the static site — which will basically just say "for CA senders for now"
related to GitHub issue Update guidance for flu pilot #5515

References

SimpleReport flu reporting - Marketing Plan [Google Doc]
Banner currently on uploader page, reading "What is the bulk results uploader?" *Note: Link goes to a PDF of the bulk upload guide - flu pilot version hosted on the static site.
Private Zenhub Image

@lauraykerr We will need your assistance in adding the updated RSV text copy for the banner.

@jayna-SkylightDigital @mehansen Updated banner copy for bulk upload page:

New: Report flu and RSV results to California
Organizations sending results to the California Department of Public Health (CDPH) can now use the bulk results upload feature to report positive flu and RSV tests. Report COVID-19, flu A and B, and RSV results all from a single spreadsheet. See more information and guidance about flu and RSV reporting.

All organizations can continue to report COVID-19 results to their public health department using the bulk uploader. Flu and RSV reporting will be available in other jurisdictions soon.

The guidance link will stay the same and I'll upload the updated version of the PDF to the static site.


[sozo] invalid manifest writes values in some case

Describe the bug
If set! is used in a loop, manifest writes field do not includes used models.
If set! is used in a function called by contract, manifest writes field do not includes used models.
get! looks ok
To Reproduce
loop https://github.com/notV4l/dojo-starter/blob/bad_writes_gen/src/actions.cairo#L43
function https://github.com/notV4l/dojo-starter/blob/9d938d9be97c0b9266bf4c9dd57affe59b41e660/src/actions_2.cairo#L28
     "reads": [
        "Moves",
        "Position"
      ],
      "writes": [],
      "computed": []

This needs to be investigated again with the new procedural macro design from scarb. Will consider it to be blocked right now as I'm not sure if the new manifest system is computing the write. @lambda-0x any insight on that?

in the new manifest system i didn't change any logic related to how this values are calculated

i don't think we will be able to do this automatically with the new macro system unfortunately

i don't think we will be able to do this automatically with the new macro system unfortunately

Same here. Let’s move on for now and consider this to be a manual input from the user into the overlay if you agree guys.
I will add the overlay concept in the todo list of the sozo documentation issue. 👍


fizmat

faex.zip

@9ag0 Please provide us with:

the school official website URL
the school street address, including city and country

http://physmath.am/

Azatutyan Backstreet, Yerevan

@9ag0 Thank you for the clarification. Pull request merged.


Improve book and fix serde dependency

The correct feature to enable deriving for Serde is derive, not serde_derive source.
I went through and fixed mistakes in the Book. I also ran a spell checker so it should all be correct.

Great! thank you very much!  😁


how to restrict the jenkins  not to trigger the job for pull request (build trigger is -webhook)

In Jenkins I have a job which is configured to trigger on web-hook with GitHub, In GitHub I have 2 branches one is master and other is working branch. when I committed on master Jenkins trigger the job which is perfectly all right. But when I tried to pull request to working branch from master for the committed changes again a new build getting triggering which is completely annoying.
so is there any logic to stop the triggering the job for only pull-request

deselect this Build every pull request automatically without asking (Dangerous!). under advanced section.

@forpix not sure what you really mean. You created two PRs, both were built. Jenkins does not care about "things are already merged".
Two options:

only trigger builds for PRs manually (you have to set the triggers accordingly)
only automatically trigger builds for a set of valid target branches

Will close this ticket in 7 to 14 days if we do not get a response :)

A source branch whitelist is needed. There are cases only a specific feature is needed to be executed upon a pull request event.

only trigger builds for PRs manually (you have to set the triggers accordingly)

Can you please point to how to do that?

deselect this Build every pull request automatically without asking (Dangerous!). under advanced section.

But where we can get this option exactly.Actually I want to try this for my multibranch pipeline.I can not see this option for multibranch pipeline

deselect this Build every pull request automatically without asking (Dangerous!). under advanced section.

Where we can get this option exactly. I need to try this for my multibranch pipeline.I can not get this option.pls help me with this

Where I can get this option for multibranch pipeline.How can I disregard the pull request from triggering the multibranch pipeline


feat: configurable i18n translation feature / default language

Is your feature request related to a problem? Please describe.
Currently, i18n only automatically detects the local language and applies the corresponding translated content. However, I hope to allow users to decide whether to enable this feature during the deployment phase.
The reason for such a demand is because the translation quality of various languages is currently not stable enough (and I am assisting in improving this at https://github.com/open-webui/open-webui/pull/1247 ). Additionally, considering that some non-native English speakers may still prefer using the English interface, I hope to have an option available during Docker deployment for users to decide whether to enable the translation into the local language.
Describe the solution you'd like
I hope to allow users to decide whether to enable this feature(i18n translation) during the deployment phase by setting config.json mentioned in https://github.com/open-webui/open-webui/issues/1238#issuecomment-2010994094, https://github.com/open-webui/open-webui/pull/1151#issuecomment-2008159124.
Describe alternatives you've considered
Alternatively, allowing users to specify the desired target language for translation after disabling the automatic detection of the local language.
Additional context
Nothing.

@changchiyou I've made some crude adjustments in asedmammad/feat/i18n-navigator-config branch that I believe address the issue you reported. Please be aware that the code is not yet ready for production use and has low quality. Could you please confirm whether these changes address your request?

@changchiyou I've made some crude adjustments in asedmammad/feat/i18n-navigator-config branch that I believe address the issue you reported. Could you please confirm whether these changes address your request?

@asedmammad Looks like it indeed meets my needs. Bravo! 👍

Thank you for the confirmation, @changchiyou ! Once I've cleaned up the code, I'll create a pull request to merge the changes.


NodeMCU isn't receiving any data

Esp8266 ip,port etc. is configured right
visualizer on pc works no problem.
But Esp8266 decided not to work ¯_(ツ)_/¯
any help will be appreciated

Assuming the visualizer part on the pc is configured right (it needs to know where to send the data), you could try using WireShark to see what data is going to and from the esp8266’s ip and or your pc’s ip. I’d do that to confirm if the esp is responsive, and to make sure the pc is sending data.

Did you already do a Strandtest to check if the wiring for the LEDs is correct?
And then i would try to adjust the arduino code and then check the serial output if the connection is working fine.
Would check those 2 things first.

As @biejay has requested, please run strandtest and let us know how it goes.
Can you give me some more info like what you mena by not receving any data?
Is it displaying fps in the serial monitor or does it stay at FPS: 0?

sorry for not posting back but problem solved itself but thanks everyone for helping anyways


MS Test in VS code

I have created a MS Test project in VS Code and using selenium C#. In the project I got with launch.json and tasks.json files. I want to have some configuration values like connectionstring and applicationurl in the project. Where I have to create those values and how to read them in the project.

Thanks @vinaysimharaju. Have you tried submitting this as a product feedback or asking on StackOverflow?    If you do submit an issue, please post the link here so others can find it as well.


Make a package to use pkgimages based precompilation

~/src/related_post_gen$ ./run.sh julia
Running Julia
Julia:

Benchmark 1: julia --project related.jl
Processing time (w/o IO): 48 milliseconds
Processing time (w/o IO): 48 milliseconds
Processing time (w/o IO): 48 milliseconds
Processing time (w/o IO): 50 milliseconds
Processing time (w/o IO): 48 milliseconds
Processing time (w/o IO): 48 milliseconds
Processing time (w/o IO): 49 milliseconds
Processing time (w/o IO): 48 milliseconds
Processing time (w/o IO): 49 milliseconds
Processing time (w/o IO): 48 milliseconds
Processing time (w/o IO): 48 milliseconds
Processing time (w/o IO): 49 milliseconds
Processing time (w/o IO): 48 milliseconds
  Time (mean ± σ):     651.2 ms ±  13.6 ms    [User: 547.7 ms, System: 288.9 ms]
  Range (min … max):   634.2 ms … 669.5 ms    10 runs

Checking output
Success: related_posts_julia.json is valid!

~/src/related_post_gen$ ./run.sh julia_con
Running Julia Concurrent
Julia Concurrent:

Benchmark 1: julia --project --threads auto related.jl
Processing time (w/o IO): 22 milliseconds
Processing time (w/o IO): 22 milliseconds
Processing time (w/o IO): 23 milliseconds
Processing time (w/o IO): 22 milliseconds
Processing time (w/o IO): 23 milliseconds
Processing time (w/o IO): 22 milliseconds
Processing time (w/o IO): 22 milliseconds
Processing time (w/o IO): 23 milliseconds
Processing time (w/o IO): 22 milliseconds
Processing time (w/o IO): 22 milliseconds
Processing time (w/o IO): 15 milliseconds
Processing time (w/o IO): 22 milliseconds
Processing time (w/o IO): 22 milliseconds
  Time (mean ± σ):     631.5 ms ±  10.1 ms    [User: 617.6 ms, System: 292.4 ms]
  Range (min … max):   618.6 ms … 645.7 ms    10 runs

this shouldn't matter because we have manual warm up, it's weird you're seeing a difference
https://github.com/mkitti/related_post_gen/blob/c1495369e79520dc0afd06decbef92c912009b7a/julia/related.jl#L12

This is actually meant to decrease the final time at the bottom of output. This is technically not part of the benchmark, but it is noticeable.
One issue is that on my particular home potato system this makes the concurrent time increase. However, it is much faster overall for iterations. On my workstation, it is very fast.
Based on how the conversation is going over there, I think we might be more convincing if we a actually cached some code.


specify which params are required/optional

Sorry but this change only adds to the confusion. Operators (like UNIQUE or LIKE) apply to either a field or a field, value tuple. The text "This parameter is required" leaves one wondering: which parameter?
Perhaps a better layout for these constraints would be something like:
### GROUP_BY operator
**Value**: (optional) integer, for example `"3"`

... some explanation follows ...

### LIKE operator
**Value**: (required) a valid regular expression, for example `".*-foo"`

... some explanation follows ...

That makes lots of sense, @jdef . Can you point me to where in the code I can find what form the values need to take (e.g., integer)?

@jdef That makes a lot of sense -- thanks for chiming in. Can you point me to where I can find the required value type (e.g., integer) where that's not clear fro the context of the doc?

Thank you for your comments, @aquamatthias  -- I have 2 small outstanding questions, then I think the doc should be complete.

@aquamatthias For the UNIQUE operator, what would be a good example value?
And can you clarify what you mean in your comment by "this value is ignored for hostname"?

@sascala
If you specify hostname UNIQUE, then any value specified will be ignored.

Thank you! If this looks good to you, please merge.

@aquamatthias Is there a reason this was closed? Are these clarifications no longer needed?

@sascala according to the log, you closed this PR. I reopen it.

@aquamatthias I am my own worst enemy, it seems. If this looks good, please merge.

@jdef or @aquamatthias -- is @mlunoe 's comment a bug to warn a user about or a feature to be documented?


Feat: Context menu extension

With the current state of the package it is not possible to set context menu extension directly. This pr allows to setting it directly in the msix configuration yaml.
Created a guide for how to create context menu dlls and the structure of context menu configuration yaml here.

Warning: But this pr changes the toast extension with seperating its com server extension and toast extension itself. So still need testing to check toast notification works correctly.

[x] Test on Windows 11
[x] Test on Windows 10
[x] Test toast notification
[x] Try creating dll only with following the guide

Hello @berkekbgz
Excellent work and effort 🥇
Unfortunately I don't know the C++ language.
So as far as I'm concerned, it's enough for me to check that there are no regression bugs and confirm this PR.
Is that okay with you?

Hey, yes that is ok for me.
If there are any bugs in code or guide, please let me know. I will do my best to fix them.


Godot should support import dependencies

When everything we need to support can be described in single file and goes auto - there is no problem. The problem arises when several objects are needed to be processed by import script - for example we build special texture out of a set of pngs. Alternatively - having large number of animations imported as several .escn files which are used to build single Godot scene's AnimationPlayer but are manageable only as separate files; Creating octree paging maps at import time so that all objects are loaded only when player visits certain world coordinates. All that stuff depends on running other importers first and then running second or third pass.
Current workaround is to tamper .import directory files when dependency change is detected by run of appropriate .import script, but this requires losing/gaining focus of Godot window several times (depending on dependency steps) which is ugly and hard to consider feasible solution.
I think better solution would be somehow to specify dependencies between import scripts, which would allow to re-import dependant assets when dependency changes.
Another alternative would be multi-import scripts which will handle dependencies inside. Any ideas?

Current workaround is to tamper .import directory files when dependency change is detected by run of appropriate .import script, but this requires losing/gaining focus of Godot window several times (depending on dependency steps) which is ugly and hard to consider feasible solution.

Maybe we could have an opt-in editor setting to reimport assets even if the window is not focused. This could also speed up the 3D workflow if you don't care much about CPU usage (e.g. on a desktop).

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Stateful ASC1 Overview - Typo

Typo
At https://github.com/algorand/docs/blame/staging/docs/features/asc1/stateful/index.md#L385
Delete last ")" from [opcodes](../../../reference/teal/opcodes.md))

Thank you! 0d313575d1f61024444146eff7d8faaaec1ef640


Setup for JPA and cucumber

Codecov Report

Merging #7 into develop will increase coverage by 48.53%.
The diff coverage is 66.66%.

@@              Coverage Diff               @@
##             develop       #7       +/-   ##
==============================================
+ Coverage      20.43%   68.96%   +48.53%
- Complexity         9       13        +4
==============================================
  Files              3        2        -1
  Lines             93       29       -64
  Branches           2        0        -2
==============================================
+ Hits              19       20        +1
+ Misses            74        9       -65

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/lukuvinkki/domain/Tip.java
76.19% <ø> (+39.04%)
11 <0> (+4)
:arrow_up:

...main/java/lukuvinkki/controller/TipController.java
50% <66.66%> (ø)
2 <2> (?)

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 c5bbf3d...91e2e66. Read the comment docs.


.env.local file not working

API_KEY configured in .env.local file not being passed properly

This was fixed. Next.js needs 'NEXT_PUBLIC_SOMETHING=value' way of writing things


IconButton should have an InkWell by default

From @tonygentilcore on June 20, 2015 4:52
@abarth suggested I file this.
/cc @HansMuller
Copied from original issue: domokit/mojo#256

From @abarth on June 20, 2015 6:2
The ink well for the icon button is slightly different than for the other sorts of buttons, but we can start by just using the normal ink well.


Update specs2-core, specs2-scalacheck to 4.12.0

Updates

org.specs2:specs2-core
org.specs2:specs2-scalacheck

from 4.11.0 to 4.12.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.specs2" } ]

labels: test-library-update, semver-minor

Superseded by #293.


Will the training data be made available?

Hi, when attempting to train Physformer out of the box it seems to fail because it is missing a path to input_data. The default is set to "/scratch/project_2003204/" which is not found in the repo. I was wondering if this folder would be made available?
Thank you.

So, the training data is available?


Groupby on dask bags does not group correctly

What happened:
Groupby on dask bags does not group correctly.
What you expected to happen:
When using group by, group correctly by the output key.
Minimal Complete Verifiable Example:
from dask.distributed import Client
client = Client(n_workers=4)

import dask.bag as db

b = db.from_sequence(['Alice', 'Bob', 'Ana', 'Dan', 'Borja', 'Frank'])
b.groupby(lambda x: x[0]).compute()  # names grouped first letter

Produces:
[('A', ['Alice']),
 ('B', ['Bob', 'Borja']), # <-- Correct, grouped by first letter
 ('F', ['Frank']),
 ('A', ['Ana']),          # <-- Should be grouped with 'Alice'
 ('D', ['Dan'])]

Anything else we need to know?:
Environment:

Dask version: 2.20.0
Python version: 3.8.8
Operating System: Ubuntu 20.04
Install method (conda, pip, source): conda (see https://github.com/dask/dask-tutorial)

This is a known issue with using bag on with the distributed scheduler. Here is the ticket where the fix is being discussed https://github.com/dask/dask/issues/6723


Support packages without "lib" prefix in pkg-config

This fixes #300.
I met the same problem with opencv.
Packages without "lib" prefix seems common between larger projects.
This patch tries "lib"~lib and lib, then add existing one to pkgconfig_libs list.

thx


Very nice, How to handle Array please @DefaultStorage("accounts") var accounts: [Account] = []

Hi,
Thank you for this wrapper its very well written, I am just wondering how can we implement something like this:-
@DefaultStorage("accounts") var accounts: [Account] = []
I came across the following article
https://swiftsenpai.com/swift/create-the-perfect-userdefaults-wrapper-using-property-wrapper/
The simple wrapper in the above article allows us to do something like this:-
    @UserDefaultStorage(key: "accounts", default: [])
    private static var accounts: [Account]

Is there a way to have the best of both worlds between this wrapper in the article and DefaultStorage?

Couple things:

My backports are designed to be drop-in replacements for Apple's APIs, therefore if its not supported by the original, it won't be supported in mine either.
That being said, this should be doable because AppStorage supports Data, therefore if you look at the init methods for the property wrapper, it should be pretty obvious how you could go about adding Codable support, which under-the-hood would just translate to/from Data
Lastly, I've now migrated all of my backports to gitHub.com/shaps80/SwiftUIBackports. I recommend you move to that since it includes a better Backport implementation, as well as a LOT more backports as well.

@Backport.AppStorage("") private var value: Bool = false // for e.g.


Question: Future of Unbox given Codable

What are the plans for the future of Unbox now that Swift 4 has Codable?

Hi @bhstrifork 👋
That's a great question! 👍 I'm planning to keep working on both Unbox & Wrap for the foreseeable future, continuing to improve their APIs and make adjustments for new versions of Swift. Eventually they will probably start using Codable under the hood, and at some point they might simply become extensions on top of the Codable API, as it evolves and probably becomes more powerful.
But for now, I think Unbox/Wrap and Codable can definitely co-exist. So the developers of the over 3000 apps that rely on Unbox for their JSON needs don't have to worry, because it's not going away anytime soon 😉


The 9.0.4 release in pypi isn't tagged in git

The pyee release in pypi isn't tagged in github

just pushed the tag. apologies!
but I have to know: ...people use the tags? I mostly do them out of old habits, rather than any specific ask from users.

It's very useful for audit, to know what is in the associated release elsewhere, such as on pypi, especially for things like CVEs and response management.


min:1 fails to validate empty string in non-strict mode

min:1 rule fails to validate empty strings.
The problem occurs only in standard mode. In strict mode validation works correctly.

This is why all the file are in strict mode and you should always use strict mode.
This is more a JavaScript language thing, I don't know if @thetutlage can/wanna do something about it.

@RomainLanz shouldn't this validator be always in strict mode? IMO this should be enough.

BTW,

you should always use strict mode

I've done this in my Adonis project however, this feature is not documented.

shouldn't this validator be always in strict mode? IMO this should be enough.

If you browse the file you can see that they all begin with 'use strict'. We cannot do more.

I've done this in my Adonis project however, this feature is not documented.

Because there's no need to document this and it's out of the scope of Adonis. Strict mode is a Javascript "feature". Your application may work completely fine without being in strict mode. You just need to learn what change will occur if you are in one mode or another.
Here's more information about JS Strict mode https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Strict_mode

@RomainLanz I'm talking about this little thingy:
https://github.com/poppinss/indicative/blob/9bf99faafb5a553f26489a8d152ac0c2be201407/src/Validations/index.js#L30-L32
It's possible to set Indicative in strict mode which validates all non-undefined values.

Damnit, I was thinking you were speaking about JS Mode and not Indicative's one 😂
Forget what I said.

Shouldn’t that be validated via required rule?

Shouldn’t that be validated via required rule?

Not always.
Given this rule:
{
    name: 'min:1'
}

I expect that if name is defined in data object it should be validated with min:1.
required assumes that name has to defined every time.

Empty string is not existy value, it works same as Javascript if
You must sanitize data to convert empty strings to null

Empty string is not existy value, it works same as Javascript if

Yea, I know. That's the problem.
If I pass data like {name: ''}, { name: 'min:1'} rule should fail. It should not treat empty string as not existy value.

There are so many other people who want this to behave as undefined for so many other reasons.
Sanitizing data is the only way here

There are so many other people who want this to behave as undefined for so many other reasons.

I'm talking here only about min validation. Should it also apply to it? I understand that for other validations this makes sense.

Sanitizing data is the only way here

I'd say that changing mode to strict is the only way :)

I've come to the same thing and agree with @radmen . For now I need to figure out how to set strict mode in adonjis.js.

@erzzo you can add it to hooks in start/hooks.js file:
const Modes = require('indicative/src/Modes')
const { hooks } = require('@adonisjs/ignitor')

hooks.after.providersBooted(() => {
    Modes.set('strict')
})

@radmen @erzzo I have been cleaning Indicative for a while and made it compatible to work in browsers.

It has some breaking changes.
You can install it using npm i indicative@next
Docs are here https://epic-albattani-83fe42.netlify.com/docs/installation

Feel free to share your feedback.
Also instead of modes new version uses configure method to define settings. More here https://epic-albattani-83fe42.netlify.com/docs/api/configure


After running npx @backstage/create-app unable to exec yarn build or yarn dev

(Thank you for your time on this project! 🚀)
Expected Behavior
A fresh new installation from npx @backstage/create-app should allow yarn build (or yarn dev) to run successfully.
Actual Behavior
After running npx @backstage/create-app, yarn build fails with the following error:
Failed to compile.

Error: Failed to compile.
Module not found: Error: Can't resolve 'debounce' in 'C:\projects\..\backstage\node_modules\@mate
rial-table\core\dist'

Steps to Reproduce
With the following installed:

Node v16.14.2
NPM 8.5.0
Yarn 1.22.4

Run npx @backstage/create-app
Run yarn build (or yarn dev)

Context
I had problems with npx @backstage/create-app not getting past the Yarn install step and carefully ensured I had the above dependencies installed. Create-app is now working, but I still get the above issue with debounce not being available for @mate rial-table.
 Something's odd with that C:\projects\..\backstage. What's up with that?

@freben That's me hiding a bit of the path 😉

Strange we have not heard any other issues, could you try running yarn cache clean and install again to see if this was just something temporary. Otherwise kindly requesting other windows users to give this a try 🙏


[DO NOT MERGE] Testing release automation for template-second - 13

not-specified

Both packages added with correct commit message to PR successfully


App crashes on the Household Members screen

Bug Description
App crashes after user selects  'Individual' on Patient
Steps to reproduce the behavior:

Login
Click on Reports on side menu
Click on Household Members
Select Individual

See screencast attached
Smartphone (please complete the following information):

Device: [e.g. Samsung Galaxy A2 emulator]
Android version: 8 & 10

N/B - Issue is also happening on Household Members ANC
Uploading household members.mp4…

@charlene-570 highlight the app that you are using to test:
Example: eCBIS v.0.1.3.


Add optional SMART_SELECTS_CHECK_MODEL_PERMISSION setting.

If enabled, a standard django permission check is performed on the chained model in
the API endpoint. This prevents information leaks to unauthorized users.
The extra check is disabled by default to not break backward compatibility.

Hi @cveilleux, thanks for taking the time to prepare this PR. Adding permission checks would be really good.
In the Django Admin, you don't need permission on the Foreign Key model, but on the model you are editing. Django uses the base manager to get the related objects. I think we should do the same here.
Please let me know if this makes sense.

You are right. You need edit permission on the model containing the foreign key, not view permission on the referenced model.
So the permission check would have to check if you have edit permission on any model containing a foreign key referencing the auto-complete.
So if we have a model name, is there a way in django to find all the ChainedForeignKey fields referencing this model?

Hi @cveilleux, in the filterchain function:
def filterchain(request, app, model, field, foreign_key_app_name, foreign_key_model_name,
                foreign_key_field_name, value, manager=None):

app, model: the ones you are requesting information for
field, foreign_key_app_name, foreign_key_field_name: the source model and field

So when editing a Location like in http://<IP_ADDRESS>:8000/admin/test_app/location/1/change/, values are:

app: test_app
model: Area
field: country
foreign_key_app_name: test_app
foreign_key_field_name: Location

So your code is almost right, as you check for permission test_app.view_Location.
Please adjust your code to use lower() in the permission (though it works anyway), and also test for change permission, like:
    perm = '{0}.change_{1}'.format(foreign_key_app_name, foreign_key_model_name).lower()

I tested with those changes and everything seems to work ok.
Some considerations:

Default for SMART_SELECTS_CHECK_MODEL_PERMISSION should be True, this is an important change, so I prefer security over backwards compatibility, people would only need to adjust this setting.
We need tests for 200 and 403 responses checking this new functionality
The javascript code should be handling the response and informing the user, otherwise you would end up with a bad selection and no warnings.

Let me know if you will take on those too.


bug: z-= suggestion window won't auto close after choice has been inserted

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
0.9.1
Operating system/version
Arch Linux
Describe the bug
z-= suggestion window won't auto close after choice has been inserted.

Steps To Reproduce

z-= on top of any word
Select the choice and press

Expected Behavior
z-= suggestion window should auto close after choice has been inserted.
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  "folke/LazyVim",
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

Note:
Not reproable using issue template's repro.lua (above) as it works normally.

However it is reproable using https://github.com/LazyVim/starter/commit/a13d5c90769ce6177d1e27b46efd967ed52c1d68

fixed upstream

Thank you! May I know which plugin did this?


Update README.md

remove grammatical mistakes

I am a GSSOC participant. Can I work on this issue?

Ok
On Tue, 4 May 2021, 19:38 KajalSinghBAGHEL, @.***>
wrote:

I am a GSSOC participant. Can I work on this issue?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/vikasdo/Book-Recommendation-Analysis/issues/110#issuecomment-831971064,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKT3M7BMPTKPCJWEEWLMI73TL75WLANCNFSM44CZCE4Q
.


Failed to restore plugin (engine.platform or engine.scriptSrc is not defined)

After upgrading to v2.2.0, I started getting this error during cordova prepare android:
Failed to restore plugin "phonegap-plugin-push" from config.xml. You might need to try adding it again. Error: code: engine.platform or engine.scriptSrc is not defined in custom engine "cordova-browser" from plugin "phonegap-plugin-push" for android warn

It appears to build just fine but it exits with code 1 so it breaks CI builds. I could just force exit code 0 but I don't want to do that. And I am not even using the cordova-browser platform.
Cordova CLI version and cordova platform version
cordova --version                                    8.0.0
cordova platform version android                     6.3.0

Plugin version
cordova plugin version | grep phonegap-plugin-push   2.2.0

@nprail unless you specifically need one of the new features in 2.2.0 I'd stick with 2.1.3 for now.

Mea culpa everyone. Apparently my workspace had a development version of cordova linked that supports limiting the version of cordova-browser. I have released a 2.2.1 to fix the issue. Please let me know if it is not resolved.


fixes issue #102, display correct num of projects

me again :) This should fix issue #102 following @k3KAW8Pnf7mkmdSMPHz27 's very helpful instructions!

Ok, I swear I'm not trying to push broken code here. I see the deploy preview failed to compile; this fix works perfectly on my local machine. Let me know how I can help debug (or not, I'm afraid I'm a bit out of my depth here).

hi, sorry, haven't had time to review yet. I'll have time in the afternoon, just quickly I'd guess you don't see errors locally because,

You are behind the main branch? You might have to merge it into your branch
I have missconfigured something 😛  Which development environment are you using?

To fix the errors you'll probably have to pass the update state as its own parameter so that typescripts type inference works a bit better

P.S. 1. The ESLint/Prettier configs aren't really finalized yet. Sorry about that. There are some more details about it in #83. The short version is that there are some 90-150 small problems that should be addressed before they work as expected.
P.S. 2. If you are running into issues with the ESLint/Prettier you can ask in the Code for America Slack as well. People are usually very helpful in #engineering/#enginnering-help channels. There is also, of course the #brigade-project-index channel and I am most likely reachable as @Jonathan Asketorp (and otherwise@Jonatan Asketorp).

Ok, second time's the charm, I think.
I double checked in the React discord and my idea that you can't pass usestate setters directly as props is something I invented out of thin air(!) so I have updated the code to be more direct. Also changed the variable names to be more descriptive (rowCounter vs counter) for future readability. Thanks much!


fix: fee on instant mining

Fixes the max fee validation issue by applying the current fee configuration in the instant mining. Resolves #873.

duplicated with #878


StringSlice flag and default values

So running this code with the latest master, seem to add any value you give on the commandline to the default set as defined in the flag. Is this the intended behavior? (This is also new behavior - didn't git bisect)
package main

import (
	"fmt"

	"github.com/spf13/pflag"
)

func main() {
	fFlag := pflag.StringSlice("a", []string{"1", "3"}, "a bogus option")
	pflag.Parse()
	fmt.Printf("%v\n", fFlag)
}

Running:
% go run flagtest.go --a 2
&[1 3 2]

ok. I'll try to look at it today. I think I know how to fix it.  I added the ability to do
-a 2 -a 3 -a 4 -a 5 and you will get []string{2,3,4,5}
But I wasn't getting rid of the default on the first -a

see #41 where I believe this is fixed.

I'm going to mark this as closed, since your test case works (as well as the new go tests I wrote).  Let me know if it doesn't do what you want/need!

Looks legit. Thanks for the quick turnaround.


Accounts Maillist Fix

Describe the bug
We receive an error django.db.utils.OperationalError: no such table: accounts_maillist due to maillist trying to access a table that doesn't yet exist upon first build.
A fix to this issue may be attempted by utilizing the try-except clauses.
To Reproduce
Steps to reproduce the behaviour:

Clone the Django Project.
python manage.py makemigrations.
See error

Expected behavior
The Django Table for maillist should be created without any error popping up.
Additional context
The current workaround to this error is to,

Comment out the files SheetMe.py and accounts/admin.py.
python manage.py makemigrations
Uncomment and proceed on with life.

While making a new database, OperationalError: No Such Table …
This error occurs when the table is not available in your database and some program scripts wants to access the table in the database to proceed further.
Fix:

Comment out all the scripts that use this specific table. (Eg: Commenting out SheetMe.py, admin.py [from all the django applications])
Delete the database file (would recommend to do so as this error only occurs when you create a new database)
Run python manage.py migrate : this will create the database and converts the changes to sqlite commands.
Run python manage.py makemigrations: this will update any latest changes and updates the database with the sqlite commands.
Uncomment all the scripts which you had previously commented out.

There is no permanent fix for this as this is only required at developing stage of the project. This error won't come up again unless we tamper with the Database manually by entering the Django commands to the console.

There is no permanent fix for this as this is only required at developing stage of the project. This error won't come up again unless we tamper with the Database manually by entering the Django commands to the console.

The problem is while deploying it on the server. Cannot push and migrate changes for each deployment.

Since we've converted to PostgreSQL, we can close this issue as there won't be problems with migration anymore.
For future references:
If you plan to refresh the database follow steps below:

Install heroku CLI.
heroku run python manage.py migrate
python manage.py migrate

As an alternative, you can also login to heroku and use their console and enter python manage.py migrate.
Once done, you are good to go.


⚠ SceneTime has degraded performance

In 4dbd171, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 6201 ms

Resolved: SceneTime performance has improved in e9556b7 after 5 minutes.


Use MemAvailable when it's available on Linux 3.14+ kernel

Use MemAvailable when it's available on Linux 3.14+ kernel, otherwise calculate it as free memory plus the caches and buffers.
(Fixes https://github.com/elastic/beats/issues/4202)

We need a CHANGELOG entry too.

@andrewkroh Ready for the final review.


Can't upload files on meteor production.

I'm based on :

Meteor : 2.7.1
<EMAIL_ADDRESS>Hi all, I tried to use upload files on my app. it works well on development but I have a problem after I build(deploy) to production.
here is my code:

// Server
const Files = new FilesCollection({
  storagePath: () => {
    return '/data/file_uploads'
  },
  debug: true,
  collectionName: 'app_files',
})

// Client code
const it = fileList.value[0]
          FilesCollection.insert({
            file: it,
            fileName: it.name,
            chunkSize: 'dynamic',
            onUploaded(err, res) {
              if (err) {
                console.log('err', err)
                // reject(err)
              } else {
                console.log('uploadId', res)
                // resolve(res)
              }
            },
          })

here are my logs :

on Development

on Production

Could you help me? I appreciate you.

Hello @roeurbnavy,
I don't see an error or exception on in your issue. Please, update your post following our ISSUE_TEMPLATE

From the quick look I assume /data/file_uploads doesn't exists or not writable on your server

@dr-dimitru Sorry my fault, Now it works fine. Thank you for your reply.


check what Aalto's K8S has to offer

in terms of images on Linux (Ubuntu), Windows and MacOS latest (can we find the images, what are the costs etc.)

Some comments even though I don't really know what this is about:

As far as I know, Aalto has no K8S cluster, but CS IT does. Aalto IT services outside CS IT do not really have any K8S experts.
Aalto has a contract for Microsoft Azure and ITS has used it for some container needs.
CSC sells a K8S service to universities, but we don't use it.
The K8S cluster at CS IT can run Linux containers and A+ currently sends submission grading jobs there. The grading containers are based on Debian and they are hosted in hub.docker.com.
I don't think that CS IT could run any Mac or Windows containers in K8S, but I don't really know.

Hi! Small comment from CSC's side. Our Rahti service (https://rahti.csc.fi/) is free to use for research and education (https://research.csc.fi/free-of-charge-use-cases), for others there is a cost.
As for Windows and Mac mages, I don't know of a container runtime that supports them, as containers are generally tied quite tightly to the linux kernel.


New option: AddResourceHeader

Allows setting headers on PageSpeed resources for CORS support.
Fixes https://github.com/pagespeed/ngx_pagespeed/issues/1005

@jmarantz I addressed your comments, PTAL?

Sorry I put in a couple of notes but I am actually taking the day off today
and if I'm not careful I'll wind up working most of it:)
I'll finish the re-review tomorrow.  Cool?
On Wed, Dec 9, 2015 at 6:12 AM, Otto van der Schaaf <
<EMAIL_ADDRESS>wrote:

@jmarantz https://github.com/jmarantz I addressed your comments, PTAL?
—
Reply to this email directly or view it on GitHub
https://github.com/pagespeed/mod_pagespeed/pull/1200#issuecomment-163191140
.

@jmarantz Sounds good. Meantime, I'll address your new comments. Have a nice day off!

@jmarantz updated & force pushed.

@jmarantz any more comments?  LGTY?

sorry all; fell off my radar.  I'm looking now.
On Wed, Dec 16, 2015 at 8:25 PM, Jeff Kaufman<EMAIL_ADDRESS>wrote:

@jmarantz https://github.com/jmarantz any more comments? LGTY?
—
Reply to this email directly or view it on GitHub
https://github.com/pagespeed/mod_pagespeed/pull/1200#issuecomment-165308308
.

@jmarantz All comments addressed, updated pull.
The field separators are now pulled from a 256-element bool array.

@jmarantz comments addressed again

LGTM
RE check_from and counts -- I know we have counts elsewhere basically that function captures $1, shifts, and echos the captured text into<EMAIL_ADDRESS>So we can put whatever function we want e.g.
check_from "$OUT" grep_check_count pattern count
if we define
function grep_check_count() {
count=$(grep "$1")
if [ $count -ne "$2" ]; then
...
something like that.  My bash is not quite fluent to do that off the top of my head :)

See also automatic/system_test_helpers.sh where check_from is defined.  More helpers can be added there.


Modular RAG - Query Analysis

Query Analysis

Introduce Query Analysis Module
Define QueryTransformer API and TranslationQueryTransformer implementation
Define QueryExpander API and MultiQueryExpander implementation
Support QueryTransformer in RetrievalAugmentationAdvisor (support for QueryExpander will be in the next PR together with the needed DocumentFuser API).

Improvements

Refine Retrieval and Augmentation Modules for increased robustness
Expand test coverage for both modules
Define clone() method for ChatClient.Builder

Tests

Introduce “spring-ai-integration-tests” for full-fledged integration tests
Add integration tests for RAG modules
Add integration tests for RAG advisor

Relates to #gh-1603

Just some general thoughts, not necessary to resolve for this PR.

The Query record. This is essentially the user text.  What is this was instead the UserMessage so that it can take into account multi-modality (media)  A QueryTransformer could for example retrieve images related to the text.
The QueryAugmentor - what if we had as the signature QueryAugmentor extends BiFunction<Query, RetrievalContext, Query> where RetrievalContext is

      private final List<Document> documents;
      private final Map<String, Object> metadata;
   ...
``` this way additional information that can be used by an implementing class can be taken into account.  The document is the core part, but there could be additional information to influence how the query gets augmented.  Perhaps that would be passed in the constrcutor to the QueryAugmentor impl.  Just looking to move away from types that don't have extensibility in the interface.

I get test failures for tests  org.springframework.ai.integration.tests due to the api-key not being set.  Prob works in your environment but in my I don't have any spring.ai.openai.*properties set.

merged in 263fe2fba7b6e940ac802e996db474d8909fe5d2


bugfix: when using proto3 optional field

Hi.
bug exists when using proto3 optional field
hello.proto
syntax = "proto3";
package hello;

import "google/api/annotations.proto";

option java_multiple_files = true;
option java_package = "hello.example";

service HelloService {
  rpc Hello (HelloRequest) returns (HelloResponse);
}

message HelloRequest {
  string message = 1;
  optional string optional_message = 2;
}

message HelloResponse {
  string message = 1;
}

service class
@Service
class HelloServiceGrpcService : HelloServiceGrpcKt.HelloServiceCoroutineImplBase( ){
    override suspend fun hello(request: HelloRequest): HelloResponse {
        return super.hello(request)
    }
}

error message
Caused by: java.lang.IllegalStateException: Could not find oneof case method.

Cause I guess
optional field haven't getFieldNameCase method, but does have getField method
Can you review this pr ?

Hi @sangyongchoi - https://github.com/curioswitch/protobuf-jackson/pull/4/files should have added support for optional. Could you check why it may be different from your case and add a unit test if you find it? Thanks!

@chokoswitch how to gradle build ?
I occur error No matching variant of org.curioswitch.curiostack:gradle-helpers:0.1.0 was found. The consumer was configured to find a runtime of a library compatible with Java 11, packaged as a jar, and its dependencies declared externally, as well as attribute 'org.gradle.plugin.api-version' with value '7.4.1' but:
If you tell me way how to build, I find my case and write test case.

@sangyongchoi Sorry this is probably missing from the README, Java 11 is required for building (or perhaps Java 17, we only verify that on CI). I suppose you are trying with Java 8, can you try with newer Java?

@chokoswitch

I use currently Java 11 or Java 17.
Need to set up your account information or maven info?

@sangyongchoi Sorry for the confusion and lack of docs on it, indeed right now Java 17 is needed to build, I'd like to fix that to work with Java 11 too at some point. I could verify getting your same error with Java 11 but not with Java 17. Can you try Java 17?
Verified Java 17 works with
docker run -it —rm ubuntu
apt update
apt install git openjdk-17-jdk-headless
git clone https://github.com/curioswitch/protobuf-jackson
cd protobuf-jackson
java -version
./gradlew check

@chokoswitch oh.. sorry.
I just found out  supporting proto3 optional since version 2.0.
This is not bug.
sorry give you confusion.


cvc4 modeling error in tutorial

This example comes from the ivory-tutorial exercises. Attempting to model check potion usage.
[ivory|

struct Character
  { hp     :: Stored Uint16
  ; max_hp :: Stored Uint16
  ; mp     :: Stored Uint16
  ; max_mp :: Stored Uint16
  ; potions:: Stored Uint8
  }

void use_potion(*struct Character c) {
  let ps = *c.potions;

  if (ps > 0) {
    -- decrement the number of available potions
    store c.potions as (ps - 1);

    -- increase mp up to the maximum
    let mp_val = *c.mp;
    let mp_max = *c.max_mp;
    if ((mp_val + 25) < mp_max) {
      store c.mp as (mp_val + 25);
    } else {
      store c.mp as mp_max;
    }
  } else {}
}
|]

potions_available ref =
  checkStored (ref ~> potions) $ \ potions_val ->
    1 <=? potions_val

check_potion_use :: Def ('[Ref s ('Struct "Character")] ':-> ())
check_potion_use =
  proc "check_potion_use" $ \ ref ->
  requires (potions_available ref) $
  body $ do
    old_potions <- deref (ref ~> potions)
    call_ use_potion ref
    new_potions <- deref (ref ~> potions)
    assert (old_potions >? new_potions)

We're getting this error message with the 1.5 developer snapshot and the 1.4 release of cvc4:
Unsafe ["QUERY deref1 > mc_1deref2 ;\t % [ No location available ];"]

We also turned on Inlining for this example because use_potion needs to be inlined.
@elliottt, @leepike thinks this may be related to the fresh name generation with inlining.

After some more debugging (using asserts in the concrete syntax, and no function call to inline) we still see the issue, but commenting out the inner most if-statement the model checker can find a proof.

It's been a while since I looked at the model checker, but this line
https://github.com/GaloisInc/ivory/blob/eceb031a3b6a71bd7302ac6805a65037f80b1a22/ivory-model-check/src/Ivory/ModelCheck/Monad.hs#L351
looks suspicious to me.
It should probably be sets_ (\s -> s { symCond = b' &&. b }) to track nested if-statements, which is exactly the scenario here.

Thanks, you're probably right but I just tried it and I'm still getting unsafe. We'll dig deeper tomorrow.
Thanks!

Ah, @leepike and I had talked about the nested if stuff, but we must have never made the change that @gridaphobe mentioned above.

Actually, that's probably a separate bug. Digging in a bit more I noticed a few things

These two assertions correspond to the two branches of the inner if-statement, but the 2nd refers to the variable set by the first. That's totally bogus!

ASSERT ((deref2 + 25) < deref4) => (mc_2var0 = (mc_1var0 WITH .mp := deref2 + 25)) ;	 % [ No location available ];
ASSERT (NOT ((deref2 + 25) < deref4)) => (mc_3var0 = (mc_2var0 WITH .mp := deref4)) ;	 % [ No location available ];

If you rewrite the 2nd line to
ASSERT (NOT ((deref2 + 25) < deref4)) => (mc_3var0 = (mc_1var0 WITH .mp := deref4)) ;	 % [ No location available ];

the query is valid.

My copy of cvc 1.4 choked on mc_1var0 WITH .mp := deref2 + 25, it seemed to want extra parentheses around deref2 + 25.

So this is a bug in the translation to cvc4. We need to restore the latest references when we return from inBranch.

Closing this, as we're rewriting the model checker (@eddywestbook).


Index definition in CREATE TABLE doesn't support partial indexes

MySQL allows to define partial indexes on VARCHAR columns.
For example, see the index on meta_key below, with the length 191, even though the entire column length is actually defined as 255.
CREATE TABLE `wp_commentmeta` (
    `meta_id` BIGINT(20) UNSIGNED NOT NULL AUTO_INCREMENT,
    `comment_id` BIGINT(20) UNSIGNED NOT NULL DEFAULT '0',
    `meta_key` VARCHAR(255) COLLATE UTF8MB4_UNICODE_CI DEFAULT NULL,
    `meta_value` LONGTEXT COLLATE UTF8MB4_UNICODE_CI,
    PRIMARY KEY (`meta_id`),
    KEY `comment_id` (`comment_id`),
    KEY `meta_key` (`meta_key` (191))
)  ENGINE=MYISAM AUTO_INCREMENT=119 DEFAULT CHARSET=UTF8MB4 COLLATE = UTF8MB4_UNICODE_CI;

Trying to parse this statement will result in:
net.sf.jsqlparser.parser.ParseException: Encountered " "(" "( "" at ...
Was expecting one of:
    "," ...
    ")" ...

Will be glad to hear your recommendations on how to resolve this.
I looked into the Index class under /schema, and it seems that the columns of the index are just saved as a list of strings, so therefore there is no built-in way to define extra properties (such as length) on the index column.
Once we'll reach a common grounds I'll look into implementing the fix and adding a pull request.
Thanks!

JSqlParser 4.0 parses this.

JSqlParser 4.0 parses this.


 Not sure how I could missed that, I could have swore I tested it several times.
This changes it to subprocess.Popen

I probably should've noticed too, but don't have Windows easily handy to test on :/
This used to use https://docs.python.org/3/library/os.html#os.startfile, which (perhaps significantly) returns as soon as the associated application is launched. It doesn't look like there's a way to pass arguments to that however, so moving to subprocess probably makes sense. Does the default timeout value need to increase, or is 5 seconds still working fine?

Does the default timeout value need to increase, or is 5 seconds still working fine?

Ironically, the default 5 seconds isn't long enough, but the socket still connects with 3 seconds to spare! Turns out that sock.connect_ex(('localhost', 10429)) takes 2 seconds to detect a connection failure, which was not accounted for in the loop.
In reality, it takes about 6 seconds for the socket to open and accept connections on Windows, and that's on a computer with a pretty fast SSD.
Without any changes, the code will probably allow up to 8 seconds for a connection, which is pretty tight. I've updated this PR to set the default timeout to 15 seconds, and I've changed the way it's enforced to use wall clock time.
I've also added a return value to the function, it will return True when the socket was able to connect within the timeout.


Docker Image Can Not Connect To InfluxDB

The Docker image is unable to connect to the InfluxDB database per the error below. My Grafana instance can connect to the same database in question just fine.
2020/12/31 06:31:48 [ERROR] influxdb.Write(points): Post "http://localhost:8086/write?consistency=&db=unifi&precision=ns&rp=": dial tcp <IP_ADDRESS>:8086: connect: connection refused

Switch localhost with the IP of your influxdb.

Ah, internal IP is needed instead of localhost or <IP_ADDRESS> even when Docker is running on the same machine as InfluxDB. Thanks.

I put all my containers on a custom bridge network. Then you can use DNS and refer to them by container name.


feat: integrate gray logger to improve scicat error logging

Description
@user-office-software/duo-logger package is installed and gray logger is configured.
Motivation
Enabling grayLogger allows SIMS team to get error messages in the first place.
Tests included/Docs Updated?

[x] Included for each change/fix?
[ ] Passing? (Merge will not be approved unless this is checked)
[ ] Docs updated?
[ ] New packages used/requires npm install?
[ ] Toggle added for new features?

Question: does it fallback to use the default logger with out a loggers.json present, or is minimal loggers.json required?

@bpedersen2  minimal loggers.json is required to enable default logger. and the configuration can be found in loggers.json.example


Issue7

Issue #7 : Ready for Review 💯

I noticed this PR is showing issue #6 work in addition to 7.
Since 6 was already merged I am assuming here that you need to either rebase or merge main into your branch.
This will make it easier for me or any reviewer to see what are issue 7 specific changes.
I will take a detailed look as soon as that is completed. If you have any questions let me know.


Configure location of config & cache dirs

Filecoin Station needs to configure the directory where Saturn L2 Node is going to store the configuration and the cached data. This will be typically two different places.
For example, we can use environment variables like SATURN_CACHEDIR and SATURN_CONFIGDIR.
See https://github.com/ipfs/ipfs-desktop/issues/1656 for a detailed discussion on why is this important on Windows and what are the standard locations for different types of data.
The issue tracking the work in Filecoin Station: https://github.com/filecoin-project/filecoin-station/issues/54

Let's put this on hold for now. We are discussing how to configure Station Modules and leaning toward keeping config-related stuff inside the Station. In that case, this discussion becomes irrelevant, and we can keep using the ROOT_DIR env var for the cache location.

big advantage of storing module configuration in the station is that if one uninstalls and reinstalls a module, that module's configuration persists
the downside is that each module is now responsible for dealing with configuration that may be out of date. eg imagine module v1 doesn't have some cfg key foo:, user uninstalls v1, installs v2, and v2 expects foo: but that cfg key doesn't exist


load recently viewed module in my account

Related issues

closes #3722
QA
Same steps as described in https://github.com/DivanteLtd/vue-storefront/issues/3722
Which environment this relates to
Check your case. In case of any doubts please read about Release Cycle

[ ] Test version (https://test.storefrontcloud.io) - this is a new feature or improvement for Vue Storefront. I've created branch from develop branch and want to merge it back to develop
[x] RC version (https://next.storefrontcloud.io) - this is a stabilisation fix for Release Candidate of Vue Storefront. I've created branch from release branch and want to merge it back to release
[ ] Stable version (https://demo.storefrontcloud.io) - this is an important fix for current stable version. I've created branch from hotfix or master branch and want to merge it back to hotfix

Upgrade Notes and Changelog

[x] No upgrade steps required (100% backward compatibility and no breaking changes)
[x] I've updated Changelog on how to port existing VS sites with this new feature

IMPORTANT NOTICE - Remember to update CHANGELOG.md with description of your change
Contribution and currently important rules acceptance

[x] I read and followed contribution rules

Both issues resolved, gj :)


Multi-sample text input with global bar visualization tested

Added "shap.models" to packages within setup.py so the folder is part of the build process
Modified _explainer.py and _explanation.py to allow multi-sample text input

This looks good! I am going to merge it and then play around with it a bunch and let you know if I find any corner cases

This looks good! I am going to merge it and then play around with it a bunch and let you know if I find any corner cases

Thank you!


apparently fix completions not appearing immediately (fixes #17)

this huge fix appears to make completions show up immediately again. as far as suspicion goes, it appears to be a conflict caused by deoplete invoking the plugin as soon as anything is typed (min_patter_length==0) and the gather_candidates refusing to find anything until there's at least two characters
I might be mistaken about the intent of this, in particular since the problem seems to only have started recently or only affects very few people, but it fixed it on my system.

@tom95 This does neither fix nor solve the problem. Actually this breaks the handling of i.e. structs, vim wont really resolve them anymore afterwards. And what I also noticed is that, since some of the "latest" updates of deoplete it seems like this whole module operators far less good than before. Before it resolved functions from a different header just fine. Now it does not anymore, only when I type it out myself as a function and press backspace after that it will resolv the clang completion correctly. I assume something in deoplete just lately changed which altered the behavior hugely.

When I created this patch it solved all problems I encountered. Trying it right now with the newest deoplete, I only receive a lot of errors, both with and without my patch, so I'd guess some API change in deoplete is causing trouble.

probably, would also be my guess, before I've updated today, everything worked even without your patch though. So it is probably entirely due to the API. Unfortunately i don't know which version I used before :/

Reprodocued.
It should be removed.
I don't know the implementation though.
You can configure it manually.
call deoplete#custom#set('clang2', 'min_pattern_length', 2)

@tweekmonster Ping.

@Shougo I will test it in SpaceVim，and will fix it.

@Shougo I was trying to take a look, but I can't seem to get logging to work.

Also, the reason for having the min length set to 0 was because it relied on the auto_complete_delay to get the "min length" from human behavior instead of an arbitrary fixed minimum.  That way typing whatever-> and pausing would actually give you results instead of needing to type two more characters.  Don't know about you, but I often don't know what I'm looking for and rely on completions to peek into struct membership.
It looks like the default delay was lowered from 150 to 50.  Try raising it to 150.  In my experience, 250ms is where it starts to annoyingly noticeable.
50ms is pretty low considering that in most fast apps, there's a ~20ms round trip from hardware key press to software key processing.  In terminal nvim, the delay could be greater since there's a Terminal emulator that processes the keystrokes before forwarding it to nvim, even more so if tmux and/or ssh is involved.  nvim GUIs could have noticeable delays as well since nvim is controlled via RPC.
In any case, these completions could benefit from maintaining a stale cache to get completions out faster (sort of like deoplete-jedi's stale cache).

@Shougo I was trying to take a look, but I can't seem to get logging to work.

It is deoplete's bug.  It is already fixed in master.

because it relied on the auto_complete_delay to get the "min length" from human behavior instead of an arbitrary fixed minimum.

Why does not use input_pattern feature?

Why does not use input_pattern feature?

It was deferred to get_complete_position() because pattern has to be slightly different for ObjC and C++.  I can't remember the exact reason I had, but it was easier to handle that way.


Fix progress bar issue in screen where output clobbered previous output

When running model.fit with verbose=1 inside screen, the progress bar currently prints out lines that scroll upward on the terminal. For instance, on the following test script:
from keras.utils.generic_utils import Progbar

pb = Progbar(target=10, verbose=1)
for i in range(1, 11):
  pb.update(i, [('test', i)])

The output is:
10/10 [==============================] - 0s - test: 5.0000
me@test:~/tmp/kerastest$ ========>...] - ETA: 0s - test: 5.0000
 8/10 [=======================>......] - ETA: 0s - test: 4.0000
 7/10 [====================>.........] - ETA: 0s - test: 4.0000
 6/10 [=================>............] - ETA: 0s - test: 3.0000
 5/10 [==============>...............] - ETA: 0s - test: 3.0000
 4/10 [===========>..................] - ETA: 0s - test: 2.0000
 3/10 [========>.....................] - ETA: 0s - test: 2.0000
 2/10 [=====>........................] - ETA: 0s - test: 1.0000
 1/10 [==>...........................] - ETA: 0s - test: 1.0000

This clobbering of previous screen output also prevents previous epochs' results from being visible after the end of the call to fit.

I've never seen this issue... but in any case, this seems like a reasonable change.


Fix namespace resolution for nested includes across different namespaces

Description
During recursive processing of <include> tags, the parser would always use the initial namespace context, causing failures to find the correct SQL fragments in multi-level nested includes across different namespaces.
Since English is not my native language, to express it more clearly, I will directly describe the issue I found through code.
The following code will throw an exception because the namespace relative to the refid is not correctly resolved.
<mapper namespace="com.example.MapperA">
  <select id="selectSpecial" resultType="int">
    SELECT aaa FROM table1 WHERE
    <include refid="com.example.MapperB.specialCondition"/>
  </select>
</mapper>

<mapper namespace="com.example.MapperB">
  <sql id="commonCondition">
    field1 = 1 AND field2 = 2
  </sql>
  <sql id="specialCondition">
    <include refid="commonCondition"/> AND field3 = 3
  </select>
</mapper>

sqlSession.selectList("com.example.MapperA.selectSpecial");

The Fix

Update the applyIncludes method:

Introduce a currentNamespace parameter to keep track of the namespace context during recursion.

When processing an <include>tag, check if the refidcontains a namespace (indicated by a dot .).

If it does, extract the namespace from the refid and update currentNamespace.
If not, use the currentNamespace to fully qualify the refid.

After finding the included SQL fragment, recursively call applyIncludes with the updated currentNamespace.

Ensure correct namespace resolution:

The currentNamespace is updated appropriately at each recursion level, ensuring that nested includes are resolved using the correct namespace context.

I think this change is more consistent with the default behavior where cross-namespace includes require the refid to include the namespace, and same-namespace includes can omit it.

Hello @rain7fine11 ,
It is not a bug.
Please see my explanation on the same request #2590 .

Thank you for reviewing my pull request and providing your feedback. I understand that this issue is not considered a bug and respect your decision. I appreciate the time and effort you put into maintaining MyBatis. Thanks again!

We appreciate your contribution, @rain7fine11 !
I added a test case 66a96f5 to let contributors know the current namespace resolution is by design (I thought I had done it already 😣).


NPM / Node version tric uses on it's container is old-ish

I can run tric npm version and get this back:
{ npm: '5.6.0',
  ares: '1.10.1-DEV',
  cldr: '31.0.1',
  http_parser: '2.7.0',
  icu: '59.1',
  modules: '57',
  nghttp2: '1.25.0',
  node: '8.9.4',
  openssl: '1.0.2n',
  tz: '2017b',
  unicode: '9.0',
  uv: '1.15.0',
  v8: '6.1.534.50',
  zlib: '1.2.11' }

NPM 5.6 and Node 8.9.4 is a little older than I was hoping.
It would be great to be able to use a newer version like NPM 6.13+ or Node 12.16+

https://github.com/moderntribe/tric/pull/5

My work around for this is for JS tests to just use actions/setup-node@v1:
name: 'JS Tests'
on:
  pull_request:
    paths:
      - 'src/resources/js/**'
jobs:
  test:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2
      - name: Use Node.js ${{ matrix.node-version }}
        uses: actions/setup-node@v1
        with:
          node-version: '12.x'
      - name: Install dependencies
        run: npm install
      - name: Run JS test suite
        run: npm run my-test-suite

This issue has been fixed by the addition of the npm_lts service and command in version 0.5.7!


Make statusbar plugin give event when statusbar height changes on ios

hello  every one i meet a issue with ios hotspot
this picture is my app
https://docs.google.com/file/d/0B6tIg9_b_ZnLbExWTFBYRHRIS2M/edit?pli=1
when i open iphone hotspot share wifi , the tabs will too low
https://docs.google.com/file/d/0B6tIg9_b_ZnLcC0wTEYyQTFrXzA/edit?pli=1

I'm not sure if this is related, but I'm seeing the behaviour whereby the app initially is pushed down, and then bounces back up again - so the hotspot or in-call additional status bar heigh is over the top of the Ionic header. I'm of the view that the header should drop down (most apps do this) and appear below the increased iOS standard status bar height.
Have I done something wrong in the setup as my app looks ok when the status bar doesn't have extra hotspot or in-call height. I get the correct colour and lighten text effect as expected - just odd behaviour when hotspot or in-call height is added to the native status bar ????


CI: Skip stable doc deployment on non-final releases

To prevent failures when CI is triggered by tags.
The doc deploy action does not yet support pre- or post-releases

I cannot find an easy way to match a pattern for the various not-supported tags. We'll have to wait for the action to do the right thing, or define a separate step (maybe powershell) to skip the action step


Slash commands do not register / update on Discord

Describe the bug
I have used Interactions.registerSlashCommand() and a sample slash command to register it, but the slash commands on my server do not update / refresh. The additions have worked once or twice, but very unpredictably. Restarting Discord and the bot did not help at all.
To Reproduce
Use Interactions.registerSlashCommand() attempting to add different commands.
Expected behavior
The slash commands I have removed should have disappeared, and the ones I have added should have appeared.
Desktop (please complete the following information):

OS: Linux
Dart version: 2.15.0-51.0.dev
Nyxx version: 2.0.0-rc.15

https://github.com/nyxx-discord/nyxx/blob/dev/nyxx_interactions/example/basic.dart#L38-L41
You need to call syncOnReady in order to sync commands to discord api

https://github.com/nyxx-discord/nyxx/blob/dev/nyxx_interactions/example/basic.dart#L38-L41
You need to call syncOnReady in order to sync commands to discord api

Yes, I do call Interactions.syncOnReady after trying to add slash commands.

if commands are global then yes, because discord caches them for one hour. Post command line output and sample code if problem persists, because Im not able to provide further help without those.

Thank you for the information.


Add highlight color and show in mentions to automod messages

For now, the added/removed term messages don't show up in /mentions, but I'll add a commit doing that if we decide to do it.
Previous messages change color when the user changes settings

Scrollbar color

Scrollbar color also changes when settings are changed

Message showing up in mentions

Message doesn't show up in mentions if setting is disabled

I was hitting an assert when allowing/denying a message, but I tested in master and it also happens there. It seems to be caused by the request created in Helix::manageAutoModMessages not being concurrent, but I didn't look too much into it. This was the call stack when the assert triggered

Fixes: #5206

You didn't cause the assertion issue: isInStreamerMode is called in a PubSub handler: https://github.com/Chatterino/chatterino2/blob/ea19c5c989dbea36b114bd524d50457adcb27206/src/Application.cpp#L710
The PubSub handler is running in it's own thread - not the GUI thread. isInStreamerMode isn't annotated, but has to run on a single thread as it accesses cache. This thread must be the GUI thread as it emits streamerModeChanged.

Oh, nice. I think I mixed some things up. I put a breakpoint in isGuiThread for when the result was false, but this function isn't called just inside assertInGuiThread, and it ends up begin called by some functions in NetworkData. Ended up posting the correct call stack

I think the "header", i.e. message that contains the Allow & Deny buttons, should be colored based on the AutomodHighlight

Is it ok to set MessageFlag::AutoModOffendingMessage for that message or should I make a new flag?

Is it ok to set MessageFlag::AutoModOffendingMessage for that message or should I make a new flag?

The issue with reusing the flag is that you'd create two highlights. You can avoid creating a new flag by checking for MessageFlag::Timeout, which is only set for offending messages, and the "header" in combination with AutoMod.

The issue with reusing the flag is that you'd create two highlights. You can avoid creating a new flag by checking for MessageFlag::Timeout, which is only set for offending messages, and the "header" in combination with AutoMod.

Something like this?
if (ctx.preferences.enableAutomodHighlight &&
            (this->message_->flags.has(MessageFlag::AutoModOffendingMessage) ||
             this->message_->flags.has(MessageFlag::Timeout))

checking for MessageFlag::Timeout, which is only set for offending messages

Is it a good idea to rely on this? I feel like it could break something in the future

Is it a good idea to rely on this? I feel like it could break something in the future (I guess it's unlikely because we would the same combination of flags)

Yea, it's probably better to have a new flag.

Added coloring for the message header

in mentions

Is it intended that AutoMod's "Hey! Your message is being checked" is not customizable with your changes?

Is it intended that AutoMod's "Hey! Your message is being checked" is not customizable with your changes?

Good question. I interpreted the issue as "highlight messages captured by automod", not "highlight that automod caught your message", but I'm not sure about the original intent.

They look really similar right now on master and are from the same source. I am of the opinion that they should be thrown into the same bag as mod-side automod messages. I don't think the issue author even considered their existence. Interestingly enough currently these messages don't show up in /automod.

I interpreted this as a moderator/broadcaster tool, so that's what I think is important


Garage: Update queries to use new lookups schemas

Closes RIG-52

Ugh github doesn't automatically sign commits when you "merge and rebase" via the UI, will need a re-approval in a minute


Option for auto closing comments

This is an implementation of feature requested in #1966.
The option itself was added in a complete inspiration of autoClosingBraces and autoClosingQuotes and it's in a separate commit to ease review of actual feature implementation.
Not entirely happy with the logic implementation, but there is already quite convoluted interaction between checks in _isAutoClosingOpenCharType and _typeInterceptorElectricChar.
Wend with a way which minimizes amount of changes between those two.
The biggest challenge fitting this new functionality into existing _isAutoClosing* functions is that they are currently designed to work with a single character pairs. While it's possible to make them aware of multi-character ones, it is still seems changes to _typeInterceptorElectricChar() are needed. And this is where some entanglement begins.
Really looking forward having feedback from a core developers to find solution which fits design in a best possible way.

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

Continued in https://github.com/microsoft/vscode/pull/192335


Generate Celebrity's Face using DCGAN

Is your feature request related to a problem? Please describe.
I'd Like to add Generating Face Project.
I would be using This Data set to generate faces of celebrities using DCGAN.( will be following the total pipeline)
Describe the solution you'd like
In this project, I'll define and train a DCGAN on a dataset of faces. the goal is to get a generator network to generate new images of faces that look as realistic as possible!
1.I'll use, CelebFaces Attributes Dataset (CelebA) to train the adversarial networks.
2.The project will be broken down into a series of tasks from loading in data to defining and training adversarial networks.
3.At the end of the notebook, We'll be able to visualize the results of the trained Generator to see how it performs.
4.I'll add Model pkl File and Images as well.
I'd like to work on this issue as part of GSSOC-21.
Thanks

please add your implementation in awesome face operations


AddSwaggerGen and route group names not working

Hi @dj-nitehawk!
I trying to configure Swagger to use the routes groups names properly. But I believe I doing something wrong or there is some kind of error.
This is how my route is shown:

No matter what why try, I cannot change this group name (in this case I just want to make first letter uppercase, as "Login").
I tryied to use the WithTags method, but the route appears duplicated:

I´m not sure but I think there is some kind of bug on this.
And in my research for a solution, I realized the AddSwaggerGen method is not working. I putted a breakpoint on it and never stopped there. Is there something about this method not work with FastEndpoints and minimal APIs?
Regards!

here's how you do it:
global using FastEndpoints;
using FastEndpoints.Swagger; //add this

var builder = WebApplication.CreateBuilder();
builder.Services.AddFastEndpoints();
builder.Services.AddSwaggerDoc(tagIndex: 0); //add this

var app = builder.Build();
app.UseAuthorization();
app.UseFastEndpoints();
app.UseOpenApi(); //add this
app.UseSwaggerUi3(s => s.ConfigureDefaults()); //add this
app.Run();

public class LoginEndpoint : Endpoint<EmptyRequest, EmptyResponse>
{
    public override void Configure()
    {
        Get("/login");
        AllowAnonymous();
        Options(x => x.WithTags("Login"));
    }

    public override Task HandleAsync(EmptyRequest req, CancellationToken ct)
    {
        return SendOkAsync();
    }
}

gotta set the tagIndex: 0 to disable the auto tagging based on url segment and specify the tag you want via WithTags(). also don't use AddSwaggerGen(). the correct one is AddSwaggerDoc()

Worked like a charm:

Tks for super quick answer!


Broken link of cool front-end template in Amazing-Js-Projects

Issue Type:

[x] Bug Report
[x] Feature Request

Describe the bug
-while clicking on the the navbar cool front-end template we can't redirect it to the another website.
Screenshots

assign me under hactoberfest

@arpit456jain  I have made the changes have a look😄


Image cache token

Summary
This change ensures that the proper authentication token is sent when retrieving images uploaded by users within a chat window. While this doesn't seem to affect production builds, images will not download and YellowBox errors are shown when attempting to retrieve images in debug mode.
Ticket Link
https://jira.uberinternal.com/browse/UCHAT-3761
Checklist
Device Information
This PR was tested on: Nexus 6 (Android), iPhone X (iOS)
Screenshots

via @dseawel

@dseawel where are we with this one?

Seems to be no longer valid. Closing.


Requirements for sp110e not found

After installing this using HACS and adding the necessary content to the configuration.yaml it shows the following error when checking the configuration:
Platform error light.sp110e - Requirements for sp110e not found: ['sp110e==1.4.0'].
How can I avoid this error?

I have the same error...
Platform error light.sp110e - Requirements for sp110e not found: ['sp110e==1.4.0'].

Fixed in v1.0.1. Tested in HASS 2022.8.6, HACS 1.26.2. Thanks

Mine is working now. Thank you!


Use configs.NamespaceType as key for State.NamespacePathes

I think this is more convenient because for working with namespaces we
using configs.NamespaceType.

LGTM


Make dropdown stay always open and visible and have content

For debugging purposes. While styling the dropdown, it's extremely helpful to have the dropdown stay open then the input looses focus, so one can easily fiddle in firebug (or what have you) to finetune styling.
Currently the drodown closes when doing anything in firebug, so it's almost impossible to see what's applied and fiddle with it, especially on suggestion items.

With 0.10.5 I worked around the issue with this code:
var tt = $.data([DOM node], "ttTypeahead");
tt.input.off("blurred");
tt._onBlurred = function () {
    this.isActivated = false;
};
tt.input.onSync("blurred", tt._onBlurred, tt);

[DOM node] is the DOM element on which the typeahead was created (e.g. $([DOM node]).typeahead(...)).
Unfortunately, any significant change to the core of the typeahead code will probably break this workaround. I tend stay away from such workarounds but the default behavior was significantly impacting the productivity of my users.

I was able to get this working with a little css:
.tt-menu { display: block !important; }
Not ideal, but it does the trick!

With 0.11.1, if I want to display the menu when it is hidden, I just add the "open" class to the menu. If you stick with the defaults, that would be:
$el.parent().find('.tt-menu').addClass('tt-open');
where $el is the typeahead element.


[CMake] Add missing unified build dep on llvm-readelf

@swift-ci please smoke test and merge

Ugh, sorry about that!  Thanks for fixing it


Fix#48281 - Indentation not respected when executing various refactorings (TypeScript/JavaScript)

Fixes #48281

I really think something is wrong here: https://github.com/microsoft/TypeScript/blob/bf0eef435321527583837f41d62b54e94cda71a4/src/services/formatting/smartIndenter.ts#L73
Deleting the addition fixes most of your test without having to use the indentation options in the codefix itself. There is just no reason why the logic you added should need to be in a specific codefix. TextChanges or the formatter is doing something wrong. Many other codefixes and refactors might incidentally print an object literal, so I don’t think the codefix is the right place to make changes (other than deleting indentation: 0).

@andrewbranch, I think you are right. It is better to fix in textChanges/smartIndenter. I pushed changes for consider curly brace as block when formatting, what do you think? 🤔 (I tried to run tests locally, it passed)


Create library with PackageCompiler.jl

I played around with compiling libtrixi directly from the Julia sources (without having to use our C library layer). This is very much WIP and lots is missing, but I didn't want to lose the effort, so here we go...

Enthused that you already got that started!
This is something hidden deeply in my notes.

The decrease in coverage is imho a false negative. Other than that, I think this PR is ready for review and merge.


Fix left-arrow message-edit issues.

Automated message from Dropbox CLA bot
@showell, it looks like you've already signed the Dropbox CLA. Thanks!

Merged, thanks @showell!  Deploying to chat.zulip.org now.


Using SwiftAudioPlayer with pre-existing AVAudioEngine graphs

Hi,
Thanks for sharing this impressive project!
In many cases, you can expect clients to have a pre-existing AVAudioEngine already set up to play and process sounds from different sources (e.g. from an AVAudioSourceNode wrapping a synth, or from AVPlayerNode playing back local files).
So what approach would you recommend if we want to integrate this into a pre-existing audio engine? Maybe wrapping the SwiftAudioPlayer in an AUAudioUnit subclass?

Hmm I am exposing both the underlying AVAudioEngine and AVAudioPlayerNode along with all the AVAudioUnit modifiers. Is that sufficient to hook up to your engine? I haven't exploring connecting one AVAudioEngine to another so I'm not exactly sure.


transformLinkUri not passing through uri parameter

markdownEx = [example.com](https://example.com)
... <Markdown source=markdownEx  transformLinkUri={ ({uri}) => { ... } />
Maybe I'm not doing this correctly, but I'm not able to access the uri parameter so I can manipulate it. I can return a different value (ex. ` return "github.com" ), but I want to return a value conditional to the original.

I think your code should be:
<Markdown source=markdownEx transformLinkUri={ (uri) => { ... } />

The parameter to the transformLinkUri function is not an object, but the href itself if I'm not wrong.

This works, thank you.


fetch external kubebuilder binaries

When running tests for this PR: https://github.com/kubernetes-sigs/cluster-api-bootstrap-provider-kubeadm/pull/7
I get this error: "error": "fork/exec /usr/local/kubebuilder/bin/etcd: no such file or directory"
We need to fetch kubebuilder binaries like done for cluster-api: https://github.com/kubernetes-sigs/cluster-api/blob/master/scripts/ci-test.sh#L26

i'd say scope the fetch to verify-gotest.

/close


Label and associated input element

An HTML label element has a for attribute. To associate the label with an input element the value of the for has to be the id of the "labelable" element (e.g. input/select/textarea). If this is the case you can click on the label and the associated input element is focused (i.e. input cursor is set into the field).
RadzenLabel has a property Component which suggests that this is a reference to a RadzenTextBox etc. But instead it is a Magic String, that only works when you set the RadzenTextBox's id to the same value. You have to keep this in mind when using labels with input elements. I guess there is no better solution (except of using nameof()). Referencing in Blazor doesn't work before things are rendered, so a Label can't access the properties of the TextBox that is rendered later.
But at least Radzen Blazor Studio should be aware of this. Currently it creates pages where Label's Component and TextBox's Name have the same value, but TextBox's id is not set and gets a random unique value.
And it would generally be better when string values that are actually references to a property of something are rendered with nameof(something.Property). You can then see what the value really means, especially in cases where the same value is used for a lot of different things. Renaming is also easier done by the IDE without manually searching the value in the whole code base.

I am afraid I couldn't understand what the problem is.

One problem is that Radzen Blazor Studio creates pages with RadzenLabels

You can log an issue in the RBS repo.

Component is a misleading name. It doesn't refer to a component, instead you have to enter an id that you also have to give a Component that renders it to an input/select/etc. element.

It probably is but it is too late to change it. This would break all existing applications created with Radzen and Radzen Blazor Studio.

And the third: less magic strings, when nameof() can provide these strings for concrete identifiers.

Feel free to submit a PR addressing that.

@akorchev Hi Atanas, we've run into this problem as well, which looks/feels like a bug.
RadzenLabels use the "Component" attribute to emit the "for" attribute which corresponds to the related input's "name" attribute.
According to the HTML5 specification for Labels If the "for" attribute is specified, the attribute's value must be the ID of a labelable form-associated element in the same Document as the label element.
As a result the relationship between the labels and the inputs is broken which has implications with Web Content Accessibility Guidelines, more specifically with label click event focusing the related input.

Have you tried setting the id of the component as well? This works as expected:
<RadzenLabel Text="Label" Component="textbox" />
<RadzenTextBox id="textbox"  />

@akorchev Thank you for looking at this. Yes, this is the way to make it work, but unless I am missing something it feels buggy with respect to Component... for example, the following is a custom component:
<RadzenLabel
    class="w-100"
    Component="EconomicActivityDropDown" // <---- this refers to the "id" attribute
    Text="@L["EconomicActivity:Singular"]" />

<RadzenDropDown
    @ref="FilterDropDown"
<EMAIL_ADDRESS>    AllowClear="true"
    AllowFiltering="true"
    AllowVirtualization="true"
    class="w-100"
    Count="State.TotalCount"
    Data="State.Items"
    LoadData="LoadDataAsync"
    id="EconomicActivityDropDown"  // <----- These lines feel duplicated, and unless they are equal
    Name="EconomicActivityDropDown" // <---- potentially break the implementation
    TextProperty="Name"
    TValue="int?"
    ValueProperty="Id" />

@if (Required)
{
    <RadzenRequiredValidator
        class="w-100"
        Component="EconomicActivityDropDown" // <---- this refers to the "Name" attribute
        Text="@L["FieldRequired"]" />
}

Not sure why you think its buggy. Name serves a different purpose than id. Feel free to explore the source and see how validators work with regards to Name.

It feels buggy because in Radzen docs the Component attribute is used for both "id" and "name" matching (which in this case need to be equal - something not necessarily true in vanilla HTML).
Label
https://www.radzen.com/documentation/blazor/label/

Component
string
null
The ID of a labelable form-related element.

Validator
https://blazor.radzen.com/docs/api/Radzen.Blazor.ValidatorBase.html?q=Validator
Component: Specifies the component which this validator should validate. Must be set to the Name of an existing component.

Still not sure what buggy is. IMHO a bug is something which doesn't work. In this case both work as advertised. Anyway we don't plan to change this behavior as it would break a lot of customer applications. Feel free to use for in RadzenLabel if Component bothers you too much:
<RadzenLabel for="textbox" Text="Textbox" />
<RadzenTextBox id="textbox"/>

Perhaps "Buggy" is not the right word for it, because it works., "Unexpected" then?
We had to look for it in the documentation and it was not self-evident that both "Id" and "Name" attributes must be specified and equal in order to make Labels and Validators work with the same "Component" attribute connotation.
My intent was to help elaborate on the possible confusion, nothing else.
Thank you for suggesting the direct usage of the "for" attribute. It just feels more straightforward.