Case list icon resize

So adding this property will scale the image to take up the entire space available to the image (the red in the second picture) without messing with the aspect ratio. I think this is more often to be what we want, especially if HQ will continue adding the "13%" default space to icons since this will end up being tiny on small screens.
Long term, I think better HQ case tile support is the correct-er solution here
@orangejenny

:+1:

Hey @wpride -- I'm doing a bunch of work right now to make the way we inflate images standardized across CommCare, and predictable for users in terms of what size their images will be, so it seems at odds with that goal to be adding custom resizing rules in specific places. (It also looks to me personally like the after image here has the icons too big, but that's something more specific we can debate). Either way I'm gonna throw a don't pull tag on this for now until we talk tonight and can sync about what the right strategy for this is.

this can be pulled now that @amstone326 confirmed its unrelated to her refactor

@orangejenny @wpride: I don't agree with the strategy here. We don't generally distort peoples images by forcing them to scale to fill a space entirely, and I don't think we should.

@ctsims @amsagoff this feels like something we should discuss in roadmap... I can see what you're saying (people might want a smaller image than max) but the current situation feels pretty arbitrary, especially given how pixel density can futz with things. In general it feels like people want icons to take up some consistent portion of screen size across devices and layouts - I'd prefer a system where the icon scales to max and they can adjust the width allotment personally

Can discuss in roadmap. Aliza's recent PR fixes the ambiguity and makes the
image pop-in entirely deterministic. If people want things to take up the
full space that way it is easy for them to do so, but especially with flags
and stuff people often want some amount of padding/whitespace and it looks
really disproportionate for it to take up the full column height when
nothing else in the row does.
On Thu, Nov 5, 2015 at 6:42 PM, wpride<EMAIL_ADDRESS>wrote:

@ctsims https://github.com/ctsims @amsagoff
https://github.com/amsagoff this feels like something we should discuss
in roadmap... I can see what you're saying (people might want a smaller
image than max) but the current situation feels pretty arbitrary,
especially given how pixel density can futz with things. In general it
feels like people want icons to take up some consistent portion of screen
size across devices and layouts - I'd prefer a system where the icon scales
to max and they can adjust the width allotment personally
—
Reply to this email directly or view it on GitHub
https://github.com/dimagi/commcare-odk/pull/708#issuecomment-154233613.

@wpride, @ctsims Agreed we should discuss in the same room. I'm a little unclear on what the original behavior of case list image sizing was (before this PR), I know there was something about HQ automatically resizing to 13% of the screen? Seems like that might be relevant / something we want to change. But agreed with Clayton that this isn't the behavior we want. In general the direction we want to be moving with images is to do less artificial resizing, and base everything on the original image size, assuming that the users intended it to be the size they made it.


Fix java home path inside container

With the original setting, the VSCode Java extension complains that the java.home is not correctly set. As a consequence, no build, run or debug is possible out of the box.

Interesting - it looks like this changed in the base image.  Thanks for the PR!!


Octal to Binary conversion and vice versa

Enter calculator name -
Octal to binary and vice versa
Describe calculator functionality -
converts a string/number from octal to binary and vice versa
Under which category you wish to place it -
[] Computer Science

Opened this issue, pls assign this issue to me for Hacktoberfest '23

Hey @7twik,
Assigned

@7twik,
You may contribute your code under the folder path -
calc_for_everything\calculators\src\computer_science\OctToBinAndBinToOct

Kindly take pull as the codebase is updated.


Speed up workflow checks

Our CI checks for the workflow are a bit slow, mostly because every time we run them, we are reinstalling all the dependencies for scpcaTools. If we were to use the image from https://github.com/AlexsLemonade/alsf-scpca/tree/main/images/scpca-r, we should be able to speed these checks quite substantially.

I don’t know how slow this is so it is hard for me to comment on cost-benefit, but I wanted to ask — do you think think there’s some merit to testing these install steps? Particularly if the Docker image is built somewhat infrequently

Slow is 20 minutes sometimes. The install steps are for dependencies that are not our problem, in my mind. (The docker image should not include scpcaData; that would still get installed on each run.) there is some chance that we would catch dependency version conflicts/changes with the current setup, but I think that is pretty small.
We could also do both! Use docker for PR checks and the full install for pushes to main.

CI is now failing because of package install it looks like, while no errors are seen locally, so as Josh pointed out it might be worth bumping up priority on this. Including some of the links to errors thrown during install of dependencies during a CI run that failed for reference:
https://github.com/AlexsLemonade/scpcaTools/pull/29/checks?check_run_id=3255336874#step:4:263
https://github.com/AlexsLemonade/scpcaTools/pull/29/checks?check_run_id=3255336874#step:4:7770

Closing this, as the new version of the R CMD check action seems to do more caching, which improves processing times.


Web console: add a button to get out of restricted mode, make capability detection more robust

Making the capability detection better and letting the user override it if needed.

i guess the button is to avoid a page refresh if it fails to detect capabilities correctly?

Or if the capabilities detection code fails in general for some reason and you want to override it.


RoadEventSourceBuilder.WithFeature() Doesn't Add Feature

As built, this function returns a whole new RoadEventSourceBuilder rather than adding provided feature builder to this. End result is no features end up in the feed. Meanwhile, _features is never added to.

Thanks! We will take a look and see what we missed. This is why it's not at 1.0 yet :)

@rhuebner76 can you take a look please. Thanks!

Above PR appears to address in local testing. Thanks!

Thanks, @rhuebner76! See discussion on #3. I think these are all rooted in the same place.

Confirmed fix with 0.0.6 release. Thanks!


Inline in if condition

Input:
float digit(int n)
{
  int d = 10;
  bool A=n!=1;
  bool B=n!=5;
  if(A)
      d=1;

  if(B)
      d=2;

  return d;
}

A and B are currently not inlined.

Already fixed by https://github.com/laurentlb/shader-minifier/commit/0e4070167b4e8160825ad1f226ea520e03092d56


Why is spark not mentioned in docker compose file but its there on the Architecture diagram

Hey i was trying to do a follow up on your project but i have just realized that you mentioned that you are using apache spark but its not mentioned anywhere on the docker composefile , am i missing something here or it actually not part of the project

It indeed uses Spark, but I haven't set up a dedicated Spark cluster for this. The reason is that Spark's task in this system is very light and doesn't need scalability. The system is already resource-intensive (with Kafka, Airflow, and Ray) so I don't want to bloat it further if not necessary.
You can find the Spark code in /services/airflow/dags. It's used within a DAG and executed by Airflow workers.

Okay, I understand now. I'm trying to figure out why I'm encountering the following problem. I'm having a challenge with Docker here. When I try to set everything up, it works, but the issue is that sometimes it's very slow to open certain ports, while others don't open and just keep loading, especially pgAdmin. Sometimes I can log in, but I can't see the data in the database. So, I'm thinking maybe something is wrong with the DAGs in Airflow as they keep failing. Then for the  machine I'm using  4 cores, 16GB RAM, and 32GB on GitHub Codespaces. What could be the problem? Is it the machine is slow, or am I missing something here?

I recommend you look at the logs from all services when you docker-compose up. If something went wrong, you should see the warnings or errors there. With only this information, It's hard to say what went wrong in your case. If you suspect the spec of your machine, you can try docker-compose up without Airflow first and see if other things work fine. Then you can start Airflow after that and try running a DAG.
For pgAdmin, you have to connect to the database server manually after you start the system. Otherwise, there won't be anything to show there. You can surely find a quick tutorial on this from Google or YouTube.

Great to hear that! I'm closing the issue now but feel free to reopen or create a new one if you face any further issues.


Supporting Email or Mobile as the Preferred Communication Channel

Issue
Currently IS supports different types of communication channels such as Email, SMS etc. for second-factor authentication, but not for account confirmation and notification paths. Account confirmation and notification paths only work with Email as the communication channel.
Requirement
Supporting email or mobile as the preferred communication channel for the user based on user preference. The requirement needs to be implemented on account verification, account recovery and notification flows.
Communication channel on the above cases should be selected based on the following scenarios.

SMS : If the user have provided only the mobile number.
Email : If the user have provided only the email address.
Either SMS or Email : If the user have provided both mobile and email. In this case communication channel should be the preferred channel selected by the specific user.

User scenarios associated with Multiple Communication Channels

User self-registration and account confirmation
User account recovery (Username and Password recovery)
Notify on Failed Login Attempts
Account Suspension

Identity / IT admin scenarios associated with Multiple Communication Channels

Onboard users with email based account verification and ask password
Forced Password Reset with OTP
Lock a user account with notifications
Disable a user account with notifications

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Ban

if u ban player and try to spawn him back u et ISE

That's about the least helpful issue you can make- please link the console output of the internal server error.

Can you please provide more information?

Probably a crash that was caused when doing /s s name when a player is banned or the server is whitelisted. Fixed in https://github.com/falkirks/Specter/commit/43ee5232601820f683204344c0ab6e7e59f73e78


TecPlot binary files with FE cell-centered block data don't load properly

Describe the bug
TecPlot binary files with FE cell-centered block data don't load properly.
To Reproduce
Steps to reproduce the behavior. For example:

Open a file containing TecPlot binary FE cell-centered block data (example data attached)
In Plots: Add -> Mesh -> Surface
Draw
See error ("The compute engine running on localhost has exited abnormally")

Expected behavior
Plot should be drawn.
Attachments
test_triangles_block_cell.plt.zip
Desktop

OS and version: Mac
VisIt Version: 3.1.2
Server info (if applicable):

Additional context
The method which calculates offsets for data may be incorrect. It assumes all data is on points. I believe numItems needs to be modified per variable, depending on if centering indicates point or cell-centered data. I haven't yet built VisIt from source so I cannot test a fix.
https://github.com/visit-dav/visit/blob/542958590f8c728eec82d85df2b74ebc1e557120/src/databases/Tecplot/TecplotFile.C#L2228-L2252

Fixed in #5375

Fixed in #5375


TypeError: unhashable type: 'dict' - Dashboard

I have a query that is executed against athena. The result is a table with 3 columns -  Account, DayOfYear and Cost. I am trying to send this as a stacked bar chart to the dashboard. The dashboard has only this chart and it throws 500 internal server error.
Exception on /superset/dashboard/7/ [GET] Traceback (most recent call last):   File "/root/venv/lib64/python3.6/site-packages/flask/app.py", line 2447, in wsgi_app     response = self.full_dispatch_request()   File "/root/venv/lib64/python3.6/site-packages/flask/app.py", line 1952, in full_dispatch_request     rv = self.handle_user_exception(e)   File "/root/venv/lib64/python3.6/site-packages/flask/app.py", line 1821, in handle_user_exception     reraise(exc_type, exc_value, tb)   File "/root/venv/lib64/python3.6/site-packages/flask/_compat.py", line 39, in reraise     raise value   File "/root/venv/lib64/python3.6/site-packages/flask/app.py", line 1950, in full_dispatch_request     rv = self.dispatch_request()   File "/root/venv/lib64/python3.6/site-packages/flask/app.py", line 1936, in dispatch_request     return self.view_functions[rule.endpoint](**req.view_args)   File "/root/venv/lib64/python3.6/site-packages/flask_appbuilder/security/decorators.py", line 109, in wraps     return f(self, *args, **kwargs)   File "/root/venv/lib64/python3.6/site-packages/superset/views/core.py", line 1616, in dashboard     for datasource, slices in datasources.items()   File "/root/venv/lib64/python3.6/site-packages/superset/views/core.py", line 1616, in <dictcomp>     for datasource, slices in datasources.items()   File "/root/venv/lib64/python3.6/site-packages/superset/connectors/base/models.py", line 285, in data_for_slices     metric_names.add(utils.get_metric_name(metric)) TypeError: unhashable type: 'dict' ERROR:superset.app:Exception on /superset/dashboard/7/ [GET] Traceback (most recent call last):   File "/root/venv/lib64/python3.6/site-packages/flask/app.py", line 2447, in wsgi_app     response = self.full_dispatch_request()   File "/root/venv/lib64/python3.6/site-packages/flask/app.py", line 1952, in full_dispatch_request     rv = self.handle_user_exception(e)   File "/root/venv/lib64/python3.6/site-packages/flask/app.py", line 1821, in handle_user_exception     reraise(exc_type, exc_value, tb)   File "/root/venv/lib64/python3.6/site-packages/flask/_compat.py", line 39, in reraise     raise value   File "/root/venv/lib64/python3.6/site-packages/flask/app.py", line 1950, in full_dispatch_request     rv = self.dispatch_request()   File "/root/venv/lib64/python3.6/site-packages/flask/app.py", line 1936, in dispatch_request     return self.view_functions[rule.endpoint](**req.view_args)   File "/root/venv/lib64/python3.6/site-packages/flask_appbuilder/security/decorators.py", line 109, in wraps     return f(self, *args, **kwargs)   File "/root/venv/lib64/python3.6/site-packages/superset/views/core.py", line 1616, in dashboard     for datasource, slices in datasources.items()   File "/root/venv/lib64/python3.6/site-packages/superset/views/core.py", line 1616, in <dictcomp>     for datasource, slices in datasources.items()   File "/root/venv/lib64/python3.6/site-packages/superset/connectors/base/models.py", line 285, in data_for_slices     metric_names.add(utils.get_metric_name(metric)) TypeError: unhashable type: 'dict' INFO:werkzeug:<IP_ADDRESS> - - [05/Sep/2020 08:36:06] "GET /superset/dashboard/7/ HTTP/1.1" 500 -
Environment
CentOS Linux release 7.8.2003 (Core) - Linux 3.10.0-1127.el7.x86_64

superset version: Superset 0.37.0
python version: Python 3.6.9
node.js version: -bash: node: command not found
npm version: -bash: npm: command not found

thanks for reporting @pkshr , I'll look into this.

@villebro  Thank you. FYI, I tried adding metric in the table as mentioned in https://github.com/apache/incubator-superset/issues/8041 . The issue still persists.

def get_metric_name(metric: Metric) -> str:
    fallback = metric
    if isinstance(metric, dict):
        fallback = metric.get('label', metric['optionName'])
    return metric["label"] if is_adhoc_metric(metric) else fallback  # type: ignore

Or you could do a similar fix in the is_adhoc_metric function ...


Collaborate with Existing MetPy Colormaps

See where we can bring in MetPy related colormaps (see https://github.com/Unidata/MetPy/issues/3112), and encourage their gallery to use this package.

Agreed!


Align claims for Type and Status with JWT claims

I think we should use short JWT-style names for the type and status list claims in order to show that these claims are at the core of the JWT (just like exp, iss, etc.). I therefore propose that we use scm (for "scheme", since the short form of "type" is already taken) and stu (for status list URI).

I don't think that this document is the right place for these things, as things should and are being described in the corresponding specification: https://github.com/vcstuff/draft-looker-jwt-cwt-status-list
If there a discussions on the claim names, this should happen over there

Hi.
Depends on the data model design/extension capabilities. There are 2 options:
a) This specification defines a claim (e.g., status/stu/...) as object with mandatory property type. Any other specification can then extend the claim, like the example in the previous comment
b) This specification does not define the status claim, other specifications define the additional claim etc.
With a) there's a better control over the data model and the approach is more interoperable, since the claim name is always the same, and the validation rules depend on the "type"/sub-type. In our projects, where we support several status methods (since there's no on-fits-all solution), this approach is more appropriate.
With b) every "status" specification needs to define their own data model (essentially), and there's always a risk that compatibility is broken; e.g., by setting required claims as optional, selecting different claim names, etc.
simplest definition would be:
status:
description: discovery of information about the current status of the credential, such as whether it is suspended or revoked.
type: object
required:
- type
properties:
type:
description: Claim that expresses the status type (and with it the vocabulary and the validation rules) The type MUST be understood within the context of the Trust Framework under which the credential has been issued. Type also defines other optional/required claims.
type: string

@danielfett @tlodderstedt @paulbastian @awoie  the design decision, whatever it is, should be applied consistently to all elements that can be extended according to this specification.

stu

I agree that status should be defined in https://github.com/vcstuff/draft-looker-jwt-cwt-status-list.

I'd not recommend using scm (or scheme) since this could be confused with JSON schemas (which might be a feature we could add in the future).

stu

I agree that status should be defined in https://github.com/vcstuff/draft-looker-jwt-cwt-status-list.

This means option b) above?

the short form of "type" is already taken

FWIW it's not actually taken. "typ" is a registered JOSE header https://www.iana.org/assignments/jose/jose.xhtml#web-signature-encryption-header-parameters but not a registered JWT claim https://www.iana.org/assignments/jwt/jwt.xhtml#claims
"type" is already pretty short and JWT-style though.  No need to change that one.

I started the very similar discussion here: https://github.com/vcstuff/draft-looker-oauth-jwt-cwt-status-list/issues/21
@alenhorvat please take a look at the first draft of jwt status list

@paulbastian thank you for pointing out the issue and the status list documentation.
Indeed, the two are related. Here, the overall design decision should be addressed a) or b). Depending on that the JWT status list it will be clear how to bind it to this data model; via type or via claim name.
Does this make sense?

I would be fine with an external definition of the status list claim as its definition and contents may depend on the mechanim used.
I still think that a three-letter JWT-style claim instead of type would be good (but typ would be confusing).

I agree that typ is confusing for the credential type. What about dty which is short for Document Type?

Copying from a comment in related PR:
I worry about collisions and confusion with typ (and with cty) - those of us that work with this stuff will likely understand the difference, but there is probably a better descriptor we could use than type for this reason.
Maybe specifying "schema" or "schemaName" or "sn" (for nice short form) would be more useful here
Pretty strongly opposed to calling the property 'type' but think that the concept needs to be a property for sure

Also good with 'dty' as suggested above by @awoie

Just FYI
type is used in the JWT Embedde Tokens

Just FYI
type is used in the JWT Embedded Tokens

Yeah - in that case I believe it is inside of token

We could also name it vct -> Verifiable Credential Type.

@awoie
I'd prefer Digital Credential Type, then dct but I admit that I dream a world where the parameter names are self explanatory.
dog -> it's a dog
token -> it's a token!
even if we have the iat -> issued_at, that's good for me, but for the world I would give to my sons I would like issued_at.
Because once you have read it, you don't need to read the documentation, because the good semantic is self-documentationary. But coming back to the world of the conventions, I'd say
the loved ISO 18013-5 defines docType, and doctype is not so bad.
from Digital Credential Type (or Verifiable) I'd remove the first word, so Credential Type since the flow is digital ...
credential_type or dctype or vctype or whatever but  the word type would be good to be there

I plan to do a PR for this, moving to dct, but will wait for PRs #173 and #172  to be merged first.

we call the credentials verifiable credentials not digital credentials, so the claim should be 'vct' in my opinion.


Doorbell ring notification support for tvOS 14

Hi @lukasroegner,
as always, thank you so much for maintaining this great plugin!
Now that the doorbell ring notifications work on iOS, I just tried if the notifications also work on tvOS 14.
In the "AirPlay and HomeKit" Settings on my Apple TV, the door shows up and notifications can be enabled. But in my case, no notifications show up when I ring the bell.
Maybe it it only supported for video doorbells?
Any thoughts about this?

Hello again,
is there any possibility to open the door directly from the doorbell notification when it comes in? I remember something you implemented for the nello plugin. Is this a function that is also only available for video doorbells?
Maybe an optional fake video doorbell with a static image would be great for this purpose.
Have a good Sunday evening!

Hi @felixmueller,
I haven't had time to take a look at it. I think it only works on the Apple TV if paired with a camera.
Maybe you can try if it works when you add a camera to the same room as the lock?

Hi,
unfortumately, I do not own a camera. Maybe there is someone else who can try it out?
Since iOS 14.1 for HomePod arrived yesterday, I was very curious about the new Doorbell Chime feature.
Following this post, It should be possible to enable it for the doorbell accessory and the HomePod:
https://www.homekitauthority.com/ios-14-beta-3-enables-homekit-doorbell-chime-for-homepod/
But it seems impossible for this plugin, maybe because there is no separate doorbell accessory present.
I think it could be also possible by providing a fake video doorbell accessory as mentioned above.
@lukasroegner What do you think?

Hi,
unfortumately, I do not own a camera. Maybe there is someone else who can try it out?
Since iOS 14.1 for HomePod arrived yesterday, I was very curious about the new Doorbell Chime feature.
Following this post, It should be possible to enable it for the doorbell accessory and the HomePod:
https://www.homekitauthority.com/ios-14-beta-3-enables-homekit-doorbell-chime-for-homepod/
But it seems impossible for this plugin, maybe because there is no separate doorbell accessory present.
I think it could be also possible by providing a fake video doorbell accessory as mentioned above.
@lukasroegner What do you think?

Wow, I have a quick update about this topic:
Someone rang the bell today while I was sitting next to my HomePod in my office listening to music.
And what should I say? The music faded out and the HomePod played a doorbell chime.
Awesome - and this without configuring anything 👌

I have a camera in the same room as the Opener but still no notifications on the Apple TV. I think you need a doorbell accessory for this to work.

I'd like to see that as well.


Add tests to SortCommand and HelpWindow

Codecov Report

Merging #51 (0e68d52) into master (4d30bf5) will increase coverage by 0.78%.
The diff coverage is 60.00%.

@@             Coverage Diff              @@
##             master      #51      +/-   ##
============================================
+ Coverage     58.30%   59.08%   +0.78%
- Complexity      419      427       +8
============================================
  Files            82       82
  Lines          1619     1623       +4
  Branches        151      151
============================================
+ Hits            944      959      +15
+ Misses          637      622      -15
- Partials         38       42       +4

Impacted Files
Coverage Δ

src/main/java/seedu/address/ui/MainWindow.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/HelpWindow.java
2.98% <40.00%> (+2.98%)
:arrow_up:

...java/seedu/address/logic/commands/SortCommand.java
72.72% <100.00%> (+72.72%)
:arrow_up:

...rc/main/java/seedu/address/model/ModelManager.java
89.65% <100.00%> (+3.44%)
:arrow_up:

...c/main/java/seedu/address/model/person/Person.java
97.82% <0.00%> (+2.17%)
:arrow_up:

Continue to review full report at Codecov.

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


Add support for compose navigation

It seems the current milestone release doesn't support compose navigation, which is very useful for any non-trivial application - https://developer.android.com/jetpack/compose/navigation

https://github.com/arkivanov/Decompose

Or maybe just contribute to Decompose and advocate its usage?

@olonho thanks for the reply. Isn't compose navigation (https://developer.android.com/jetpack/compose/navigation) the official one supported by Google and used by most of the compose developers? IMHO, using a third-party library will spit the ecosystem more than using the official one. Are you folks have any plans to support  https://developer.android.com/jetpack/compose/navigation in the future?

My five cents here.
Decompose is a bit more than just navigation. It also provides Lifecycle, state preservation, ability to retain instances over configuration changes, etc. Here is an article about it. Decompose components are like Fragments (or BLoCs).
My own opinion - we should use Compose for dumb simple UI, and navigation and business logic should be kept outside of UI. This has one very big benefit: we can plug any UI. Decompose already supports iOS and JS. We already can write shared business logic and navigation. And easily plug different UI: Compose for JVM and Android, SwiftUI for iOS, React for JS, etc. Checkout this sample.

It would be great if the team could make a final decision on this, as from using this framework, navigation is a key part of the development experience. For example when a developer asks how should I show a modal or new screen to accomplish a task, they need a definitive answer with supporting documentation.
Decompose or Compose Navigation, someone needs to make a final decision and advocate usage through the documentation.

@arkivanov is it possible to split decompose to use only the navigation part?
I think that if there is an oficial way to navigate, it should be isolated instead of getting an end to end framework.

I think adding Jetpack Compose Navigation is not a bad idea at all. Someone would prefer Decompose, someone will prefer JCN(like me). It is better to give people choice, what tools hey want to use :D

@JavierSegoviaCordoba You can build your navigation with Decompose so it will be like Jetpack Compose navigation. You can create your Navigator like in this gist: https://gist.github.com/arkivanov/c19fbc0dac4b5c296cc4a888426fa17e
Usage example:

@Composable
fun Root() {
    Navigator(
        initialConfiguration = { Configuration.Screen1 },
        configurationClass = Configuration::class
    ) { configuration ->
        when (configuration) {
            is Configuration.Screen1 -> Screen1(onNext = { push(Configuration.Screen2) })
            is Configuration.Screen2 -> Screen2(onBack = ::pop)
        }
    }
}

sealed class Configuration : Parcelable {
    object Screen1 : Configuration()
    object Screen2 : Configuration()
}

@Composable
fun Screen1(onNext: () -> Unit) {
    Button(onClick = onNext) {
        Text("Next")
    }
}

@Composable
fun Screen2(onBack: () -> Unit) {
    Button(onClick = onBack) {
        Text("Back")
    }
}

Perhaps I could add the Navigator to the Decompose library :-)

@arkivanov yeah, and should be great it was an independent artifact so everyone could pick the Decompose pieces they want.
Do you think it has sense to have more independent pieces or only navigation?

@JavierSegoviaCordoba I have doubts it is possible with Decompose, since the Decompose routing functionality does not depend on Compose. It is multiplatform and supports other targets as well. So it is possible to build Compose routing based on Decompose, but not vise versa.

There are Decompose extension modules for Android and Jetpack Compose. So I will try to put it there in such a way so it won't leak Decompose. Failed an issue: https://github.com/arkivanov/Decompose/issues/15

Our general attitude is not enforce people to use certain third-party library, shall it be Decompose or smth else. Neither we are, at the moment, able or willing to support all possible libraries ourselves. So if someone will implement Jetpack Compose Navigation - would be amazing to use it with Compose for Desktop.  But practically speaking, Decompose is flexible and widely ported solution supporting Compose for Desktop now, so using it looks like a decent option.

My personal advice is to avoid using Compose for anything but UI. But if you really want, please read the following article: A comprehensive hundred-line navigation for Jetpack/Desktop Compose. It explains how to easily create a Composable Navigator.

@olonho If that's the general attitude, cool, however make sure that's written down on the front page of the docs under a navigation section. That way when developers go to look at how to achieve navigation, they may find something helpful like the article from @arkivanov that allows them to at least achieve something. Otherwise they will have to trawl the internet and find this github issue explaining everything which isn't ideal.

I've just published a Jetbrains Compose library that brings similar functionality of Jetpack ViewModel, LiveData, and Navigation to Jetbrains Compose,  the source code is here: https://github.com/Tlaster/PreCompose, hope it can help.

Btw, there is a tutorial available: https://github.com/JetBrains/compose-jb/tree/master/tutorials/Navigation

@arkivanov  It looks a little complicated, consider separating the navigation separately?
decompose-navigation
decompose-state ...

@lwj1994 The main decompose module contains only navigation stuff, and depends only on things required for navigation. Please note that it is a multiplatform module supporting many targets, and there is no dependency on any UI framework.

For those who think that Decompose is too complex, check out this one. It provides all necessary functionality for navigation in a minimalistic design.

androidx navigation's primary purpose is to handle back stacks, but desktop apps do not typically have back stacks. I don't think it should be part of Compose Desktop. But, if Compose for iOS is ever a thing, it would make sense there.

You forgot that Kotlin-Multplatform also includes the web browser.

@brianguertin I have ported my android app to desktop and have back buttons in my app, I do need a backstack for this

Any UI element that can be touched/clicked to navigate someplace else requires navigation to decouple the nagivation-destination from it's actual implementation.
Any composable function that uses a ViewModel requires navigation for the job of cleaning up ViewModels based on the back navigation stack.
Any drill-down, multi-step wizard or settings area that requires UP  (not just BACK) navigation requires navigation.

Or maybe just contribute to Decompose and advocate its usage? :-)

Sorry, but ur library is so hard for understanding, I spend 4 hours to understand how to implement ur navigation, but I was tired and decided to use different library

Is there a youtrack issue for this, is this planned since the label has been recently changed to commonization?

@FunkyMuse
Currently Compose uses this GitHub as a public issue tracker. So, you can follow the status in this thread.
KT-62479 that you mentioned was about just docs note and not about implementation or so.

In future, Compose Multiplatform intends to provide an out-of-the-box navigation library.

https://www.jetbrains.com/help/kotlin-multiplatform-dev/compose-navigation-routing.html

I have published a library named solivagant:

Link Github: https://github.com/hoc081098/solivagant.
Link docs: https://hoc081098.github.io/solivagant/docs/0.x

Pragmatic, type safety navigation for Compose Multiplatform. Based on Freeletics Khonshu Navigation. ♥ ViewModel, SavedStateHandle, Lifecycle, Multi-Backstacks, Transitions, Back-press handling, and more...

Hope it can help anyone just need a simple solution for navigation in Compose Multiplatform 😁

Fixed in JetBrains/compose-multiplatform-core#1219 and available in the latest dev build.
Usage example: https://github.com/MatkovIvan/nav_cupcake
Official examples/docs will be added a bit later

Wow, awesome 😎
The Android team just released safe arguments based on Kotlinx serialization.
I bet that's coming later or?

Yep, It's in my todo list after stabilizing the current version a bit

Is there a plan for handling deep links, or will KMP applications have to handle that outside of the library?

Hi @MatkovIvan :wave:  will this navigation is gonna support browser router?

Is there a plan for handling deep links

Yes, but not in the first release

will this navigation is gonna support browser router?

It's in our plans too, but again, not for the initial release

Nothing specific for now - I didn't dig deep into it yet

Good luck 😅
Some weird edge cases because of the History API's ability to go backwards and forwards, as well as move to an arbitrary item in the history stack.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Bump version to 0.5.1

Since the deps have been updated recently, could you please push a new tag 0.5.1.  Thanks!

Tagged!


[BUG] Milvus metadata not being retreived

Describe the bug
I save a textsegment with metadata, but when i retreive it again, the metadata is gone.
To Reproduce
TextSegment ts = TextSegment.from("I like triangles", Metadata.from("test", "testvalue"));
Embedding embedding = embeddingModel.embed(ts).content();
embeddingStore.add(embedding, ts);
Embedding embed = embeddingModel.embed(new TextSegment(query, Metadata.from("test", "testvalue"))).content();
List<EmbeddingMatch<TextSegment>> relevant = embeddingStore.findRelevant(embed, maxResults);
//relevant list contains 1 entry, but metadata is empty.

Expected behavior
I expect the metadata to be the same as when i've stored it.
Please complete the following information:

LangChain4j version: 0.27.1
Java version: e.g. 17
Spring Boot version (if applicable): 3.1.0

Note: I just tried replacing milvus with chroma, and it works as intended on Chroma.

Hi @AbdullahGheith thanks for reporting!
This behavior is documented:
/**
 * Represents an Milvus index as an embedding store.
 * Does not support storing {@link dev.langchain4j.data.document.Metadata} yet.
 */
public class MilvusEmbeddingStore implements EmbeddingStore<TextSegment> {

Metadata persistence and filtering will be supported in https://github.com/langchain4j/langchain4j/pull/610 which is going to merged today/tomorrow.

@AbdullahGheith I have no opinion on that, but can recommend starting with InMemoryEmbeddingStore if you have a PoC and don't want to invest time in setting up a dedicated embedding store. InMemoryEmbeddingStore can be persisted to String/file if you need that.

Fixed in 0.28.0


Add Portuguese language

🚀 Feature Proposal
It would be amazing to integrate translations of the app. Personally, I would invest myself in translating HospitalRun in portuguese.
Motivation
This app is quite incredible, as it could reduce a lot the big costs of hospital management software in low sized structures which are in hard times (in France in particular), and all around the world we speak different languages.
Example
It would be integrated in the frontend part of the app, as the server is only for devs.

Hi @helenapaixao, thanks for your submission.
I move this issue to frontend repo and connect with other issues already related to the language.
You can join our Slack workspace to speak more directly with other users interested in HR translation.


Searchbar w/ Virtual List (undefined error)

I've created a searchbar to be used as the center part of a Navbar with what should be a drop-down virtual list on search the way I understand things. I've followed the F7 + Vue docs example of using a Virtual List with searchbar and I'm getting the error that something is undefined. Upon further inspection, this appears to be the Virtual List ('#search-list') as the last item in the stack reads TypeError: Cannot read property 'offsetHeight' of undefined at g.r.setListSize (framework7.min.js:18)
So, this makes sense, given that the example shows using a Vue template with a component script, but there are multiple root elements in that example, So I combined the component to contain the whole navbar,and that led me to the above error. I guess I'm just at a loss at this point to even describe what I think could be wrong because I've been at this all day, So I'll post my template and component code:
<template id="f7navbar">
  <f7-navbar>
    <f7-nav-left>
      <f7-link><f7-icon material="list"></f7-icon></f7-link>
    </f7-nav-left>
    <f7-nav-center sliding>
      <f7-searchbar
        cancel-link="Cancel"
        data-search-list="#search-list"
        placeholder="Find By Destination..."
        :init="true"
        :clear-button="true"
        >
      </f7-searchbar>

      <!-- This block will become visible when there is nothing found -->
      <f7-list class="searchbar-not-found">
        <f7-list-item title="Nothing found"></f7-list-item>
      </f7-list>

      <!-- Search through this list -->
      <f7-list
        id="search-list"
        class="searchbar-found"
        virtual
        :virtual-items="items"
        :virtual-height="48"
        :virtual-search-all="searchAll"
        :virtual-init="true"
        >
        <!-- Templte 7 Virtual List Item Template -->
        <t7-template>
          <f7-list-item :title="'{{title}}'" :subtitle="'{{subtitle}}' "></f7-list-item>
        </t7-template>
      </f7-list>
    </f7-nav-center>
    <f7-nav-right>
      <f7-link><f7-icon material="map"></f7-icon></f7-link>
    </f7-nav-right>

  </f7-navbar>

</template>

  Vue.component('f7navbar', {
    template: '#f7navbar',
    data: function(){
      var items = [];
      for(var i = 1, len=destinations.length; i<len; i++){
        items.push({
          title: destinations[i]._$data.name,
          subtitle: destinations[i]._$data.id
        })
      }
      console.log(items);
      return {items: items}
    },

    mounted: function(){
      console.log("Search Bar MOUNTED");
    },

    methods: {
        // Function to proceed search results
        searchAll: function (query) {
          console.log(query);
          var self = this;
          var found = [];
          for (var i = 0; i < self.items.length; i++) {
              if (self.items[i].title.indexOf(query) >= 0 || query.trim() === '') found.push(i);
          }
          console.log(found);
          return found;
        }
      }
  });

For the record, it does successfully log the items list in the console as expected.
Also, I tried using search-list as a param instead of data-search-list but the former didn't even register as a param in the DOM, but the latter does.
Any help is greatly appreciated. Thanks!

Just in case anyone stumbles upon this having a similar issue, I came up with a work-around:

First off, I never did get this working… I ended up scrapping templates AND components, and then just included the f7 instances of the searchbar and virtualList(s) in the root Vue instance.
I then used v-model to bind the value of query to data, which surprisingly worked despite the "search" event never being triggered on searchbar in any sort of app configuration I tried.
So then, I used a Vue watcher on query and the .f7Searchbar instance (for active state) to toggle booleans with v-show for the search-list and for the not-found list.
I then used the same watcher on query to trigger the sequence of onSearch, searchAll, and .f7VirtualList instance .filterItems() methods.
Then, for whatever reason, the "@click“ or ”v-on:click" listener did not work (trigger) on the search-list items, but vanillaJS "onclick" did, so I used that to route the event to the vm.onSelected method I created.
Lessons: as nice as the Vi rual List Vue component is, the logic I had to implement to get around this issue made it little worth the trouble and could just as easily be implemented with an plain Vue <li v-for"item in items"> call for list rendering, which would also enable better functionality for Vue list transitions like this or this, which I was not able to implement due to the templating required for Virtual List. The one advantage to Virtual List in this scenario is the caching of rendered list-Items. I hope this helps someone out and saves them some trouble haha!

Got same error, solve it by making a   the parent of the virtual list.


"Bădiuți" is listed in the drop-down menu

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

Step 1
Type "Bădiuți" in the search bar.

Expected results
The location is listed in the drop-down menu as "Bădiuți, Botoșani (Ștefănești)".

ATTENTION:
Request - "Bădiuți, Botoșani (Ștefănești)"
In dropbox, the request is displayed correctly. - "Bădiuți, Botoșani (Ștefănești)"
After you click on the text, what is displayed in the window this is different - "Bădiuți, Botoșani (ȘTEFĂNEȘTI)"


Publish to npm

It would be preferable if the package would be deployed to npm instead of installed through git.

I'm having problems to install the dependency and deploy it on GCP.
It shows some error related to corejs, but removing the agora-rtc-react made it work!
Please publish it!

Yeah I just went ahead and did it so now npm i agora-rtc-react should work

@boy51 can you share the NPM package access with me? We can officially maintain it and release updates to NPM.


esp8266 compiled, but not works.

I want to be able toggle the default led from esp8266. For the moment I want to run a loop in the esp, and to see the logs.
For the moment I builded this program
fn main() {
    esp_idf_sys::link_patches();
    // Loop to toggle the LED 20 times
    for i in 0..20 {
        println!("{} Toggle", i);
    }
}

cargo build
all works good.
but when I want to flesh, Flashing has completed, but I got something like.
espflash flash target/xtensa-esp32-espidf/debug/blink --baud=460800 --monitor
[2024-11-20T09:15:55Z INFO ] 🚀 A new version of espflash is available: v3.2.0
[2024-11-20T09:15:55Z INFO ] Detected 4 serial ports
[2024-11-20T09:15:55Z INFO ] Ports which match a known common dev board are highlighted
[2024-11-20T09:15:55Z INFO ] Please select a port
[2024-11-20T09:15:56Z INFO ] Serial port: '/dev/cu.usbserial-2140'
[2024-11-20T09:15:56Z INFO ] Connecting...
[2024-11-20T09:15:57Z INFO ] Using flash stub
Chip type:         esp8266
Crystal frequency: 26MHz
Flash size:        4MB
Features:          WiFi
MAC address:       8c:ce:4e:c1:d1:81
App size:          474,048 bytes
[00:00:49] [========================================]     463/463     0x0  [2024-11-20T09:16:47Z INFO ] Flashing has completed!
Commands:
    CTRL+R    Reset chip
    CTRL+C    Exit

**###
_��{ll��|�l�<
        �
         d�c|����s�c�
                     b��gn�do'���
                                 #x��lrdrd8�g�
                                              �

                                               c
                                                o�<l�
                                                     #��o'�d��$`�'
                                                                  d`'{���g

�lx�o�
�
      ��b
         n�<
            �;�d��c��'o�
                        $ �'od�d`g{���g
                                       d``�'
                                            ��c�ol�
                                                   ��ng�
                                                        �dp�'�
                                                              ;���x�

                                                                    cg�|

��c��'o�$�l�o'                 �dg{���g                         ���p�n�
;���
�
cg�|Č�_**
#��o'��d�nglos�ۓo
ۓ`8�g�
{���l�
co�<l�
 �
  #��o'�l`�o'l`gs�ۓo�l`8�o�
                           ���
                              b
                               '�|�d�Ǐ
                                      �c��'o�
                                             $`�'l�l or�ۓo��l{��g
                                                                 �
                                                                  �$r��'l�|��`�s��o�#�gc8�

If anyone has encountered this problem before and could help me, I would be grateful.

If anyone has encountered this problem before and could help me, I would be grateful.

esp8266 will never be supported by the esp-idf-* crates.
Initially, we implemented "some" support, in the hope/anticipation that ESP-IDf will be ported to esp8266, but this would never happen.
We need to remove any reference to esp8266 from esp-idf-sys.

The closest supported chip to esp8266 is esp32c3.
You can use any esp32* chip but it seems esp32c3 specifically is even pin-compatible with esp8266.

If anyone has encountered this problem before and could help me, I would be grateful.

esp8266 will never be supported by the esp-idf-* crates. Initially, we implemented "some" support, in the hope/anticipation that ESP-IDF will be ported to esp8266, but this would never happen.
We need to remove any reference to esp8266 from esp-idf-sys.

Please can you explain to me why?

If anyone has encountered this problem before and could help me, I would be grateful.

esp8266 will never be supported by the esp-idf-* crates. Initially, we implemented "some" support, in the hope/anticipation that ESP-IDF will be ported to esp8266, but this would never happen.
We need to remove any reference to esp8266 from esp-idf-sys.

Please can you explain to me why?

Because it does not work, will never work, and there is nobody who is willing to support esp8266. It is really time to move on to a newer chip.


phive installation fails due to a missing gpg key

$ phive --no-progress install  j13k/yaml-lint

Phive 0.15.2 - Copyright (C) 2015-2023 by Arne Blankerts, Sebastian Heuer and Contributors
Downloading https://api.github.com/rate_limit
Downloading https://github.com/j13k/yaml-lint/releases/download/1.1.5/yaml-lint.phar
Downloading https://github.com/j13k/yaml-lint/releases/download/1.1.5/yaml-lint.phar.asc
Downloading key 985E1E22802973B2
Trying to connect to keys.openpgp.org (<IP_ADDRESS>)
Downloading https://keys.openpgp.org/pks/lookup?op=get&options=mr&search=0x985E1E22802973B2
Successfully downloaded key.
[WARNING] Parsing key data failed with error code 0: No UIDs in key found
Trying to connect to keyserver.ubuntu.com (<IP_ADDRESS>)
Downloading https://keyserver.ubuntu.com/pks/lookup?op=get&options=mr&search=0x985E1E22802973B2
Trying to connect to keyserver.ubuntu.com (<IP_ADDRESS>)
Downloading https://keyserver.ubuntu.com/pks/lookup?op=get&options=mr&search=0x985E1E22802973B2
[WARNING] Failed with error code 404: Key not found on keyserver
Trying to connect to pgp.mit.edu (<IP_ADDRESS>)

re #29

Hi @jakzal, thanks for flagging this. The key should now also be on the Ubuntu and MIT key servers. I've tested with a fresh phive install and it grabs the key correctly.

I can confirm it now works. Thanks 👍
There's a warning raised, but I have no clue if that's important:
Warning:  Parsing key data failed with error code 0: No UIDs in key found

Thanks for confirming @jakzal.
Re:

There's a warning raised, but I have no clue if that's important:
Warning:  Parsing key data failed with error code 0: No UIDs in key found

This is likely because of the issue mentioned in this note from https://phar.io/howto/uploading-public-keys.html:

Important: keys.openpgp.org strips the UID from the key until ownership is confirmed. The key is useless when no UID is included so make sure you follow the instructions sent to you by openpgp.org.

Unfortunately I used my GitHub noreply email to create the key this time, so didn't receive the instruction email sent by the key server.
The other key servers don't seem to have such a requirement, so the phive install still works when it falls back to them.
I will make a point of using a 'real' email when generating the next key.


Hash plus inplace draft Should't be merged

Which issue does this PR close?

Closes #.
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

@waralexrom You can close this.


Side bar / Explorer shouldn't move scroll when navigating between files

Sorry is not a bug. Is more a suggestion.
When navigating between files the Side Bar / Explorer reflect the current file that you are on highlighting this file on the list. That is perfect but is so annoying when moves the scroll to the position of the file. If the tree is huge this behavior makes you crazy.
Sublime Text doesn't have this behavior.

You want to add this to your settings.json:
"explorer.autoReveal": false

I can't live without it :wink:
To compliment this I have this keybinding that allows me to jump to the current file in the tree:
{ "key": "ctrl+0", "command": "workbench.files.action.showActiveFileInExplorer" },

Wow! Thank you! 😃


Retain not in-scope URLs

Config option "keepNotInScopeLinks" added.
If set true, the links that are not "in-scope" will be stored in collector.not-in-scope-urls.

Thanks for your pull-request.  In addition to adding the not-in-scope URLs to the metadata, you are also adding it to the "crawl data" (HttpCrawlData).  The crawl data is meant to cache information that is needed by the crawl process (current or subsequent).  The crawler itself has no use for not-in-scope URLs so we should only store it as metadata (for submission to the committer).   Do you see a reason for having those in the crawl "store"?  If not, I would remove it from HttpCrawlData to keep the crawl "store" as compact as we can.

You are right, there is no need for this information in HttpCrawlData, so I've removed it.
Also I have corrected a method name in HttpMetadata (getDocumentRejectedUrls changed to getDocumentNotInScopeUrls).
Please let me know it there are other issues.

I made minor renames and also ensured out of scope URLs are not queued.   Thanks again for your contribution!


FSharp.DependencyManager.Paket FSI extension for #r "paket: ..."

Bringing @forki's old friend.
I've tested with dotnet/fsharp and https://github.com/fsharp/FSharp.Compiler.Service/ master.
I'm looking for ideas/guidance about how to integrate the testing: I brought the fsharpqa tests we've had in the experimental branch but not worked yet on running tests in the solution.
The first contender is getting fsi.exe that understands this, I don't know how to pull it from dotnet/fsharp CI (@cartermp?), I can either build a version that will be used in the test project until it ships, or I can rely on FSharp.Compiler.Service to use in the test infrastructure that will be defined to run the checks.
@forki, do you know if there was things in the way it works that you weren't happy with but we didn't address back then?

what's the plan here? how are we bootstrapping this thing? @cartermp suggest to use "dotnet tool install". would that be possible?
/cc @isaacabraham

@forki, I can pull an fsi.exe built from master and use that in the meantime if that sounds ok.
I haven't heard from the question of how to pull dotnet/fsharp master binary artifacts from CI.
If we want to test FCS as well, we can use the next release once it is built / published.
For the tests, I'll review those that were originally added to fsharpqa, I think there are positive and negative tests, and they may need a bit of different infrastrcture.
I'll go over recent change in master (https://github.com/dotnet/fsharp/pull/8415), and start at sketching the tests.

I hadn’t considered global tools, however that may well be a good approach.  Let me mull it over.
From: Steffen Forkmann<EMAIL_ADDRESS>Sent: Monday, February 3, 2020 1:15 AM
To: fsprojects/Paket<EMAIL_ADDRESS>Cc: Kevin Ransom<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [fsprojects/Paket] [wip] FSharp.DependencyManager.Paket FSI extension for #r "paket: ..." (#3783)
what's the plan here? how are we bootstrapping this thing? @cartermphttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fcartermp&data=02|01|Kevin.Ransom%40microsoft.com|a6519bbe99d147c6222a08d7a8897a49|72f988bf86f141af91ab2d7cd011db47|1|0|637163180722853379&sdata=jSjclVfiFKaM2DpqOjT8bfgTtZ984iSrV6ZiTtQ38Qk%3D&reserved=0 suggest to use "dotnet tool install". would that be possible?
/cc @isaacabrahamhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fisaacabraham&data=02|01|Kevin.Ransom%40microsoft.com|a6519bbe99d147c6222a08d7a8897a49|72f988bf86f141af91ab2d7cd011db47|1|0|637163180722863375&sdata=Aq8hPCbWrGkHNVMMne5oVXDay6wAassQeOnR%2BfDgass%3D&reserved=0 @KevinRansomhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FKevinRansom&data=02|01|Kevin.Ransom%40microsoft.com|a6519bbe99d147c6222a08d7a8897a49|72f988bf86f141af91ab2d7cd011db47|1|0|637163180722863375&sdata=4NcVWgnaKt7olKMcwy7W9HFsNIZFGGoKKalVDsHSgCs%3D&reserved=0
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ffsprojects%2FPaket%2Fpull%2F3783%3Femail_source%3Dnotifications%26email_token%3DABHPUFSRLBY34DBSYW7TIIDRA7N7NA5CNFSM4KNMBSMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKS7KZA%23issuecomment-581301604&data=02|01|Kevin.Ransom%40microsoft.com|a6519bbe99d147c6222a08d7a8897a49|72f988bf86f141af91ab2d7cd011db47|1|0|637163180722873374&sdata=%2FVbiPU6q2vdzEJrJ8dI3RmCzmJ8aWOX0P4zKqwvnHTM%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABHPUFXYHFYR64OJEKHWBTTRA7N7NANCNFSM4KNMBSMA&data=02|01|Kevin.Ransom%40microsoft.com|a6519bbe99d147c6222a08d7a8897a49|72f988bf86f141af91ab2d7cd011db47|1|0|637163180722873374&sdata=CYTV%2Bc4i6Ma4jHZixJ24OGJC6nrmYRJtpbAXzy5OkPc%3D&reserved=0.

Bootstrapping via a local tool should probably work.

hey @smoothdeveloper  what's the status here?

@forki last time I was working on it, there was issue with inconsistent requirement between FSI shipped in VS and the one shipped in dotnet core sdk.
I'll check what is the status wrt this issue and bring the branch up to date.

Cool. Thanks a lot!
Gauthier Segay<EMAIL_ADDRESS>schrieb am Sa., 25. Juli 2020,
09:09:

@forki https://github.com/forki last time I was working on it, there
was issue with inconsistent requirement between FSI shipped in VS and the
one shipped in dotnet core sdk.
I'll check what is the status wrt this issue and bring the branch up to
date.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/pull/3783#issuecomment-663820937,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAOANAHG43AOXOT4GPVPJDR5KAL7ANCNFSM4KNMBSMA
.

@forki this is ready for a round of review, two added tests are passing, involving dotnet fsi (running script and checking output) and FCS (type checking).

Note: I still need to revive the test set that was done in fsharp branch, it covers many advanced usecase relying on ambient paket.dependencies which are not exercised yet.

would really love to get it into the alpha

@forki, I'm almost there, there is only one of the original tests I need to have a custom unit test for due to output being non deterministic (full of paths and stack trace stuff).
I'm going to update FCS to most recent and fix that last test early this week, and the PR will be good for deep review.
There will still be some rough things in terms of the more elaborate integration (how ResolveDependenciesResult is used in the eco-system is still a bit opaque) but so far so good in FCS and FSI.
@KevinRansom could you explain in ResolveDependenciesResult the difference between Roots and Resolutions folder? I had to tweak that a bit and I'm currently passing the same "working area" folder to both, because it is needed to #I that folder, it seems Roots is used for that but I'm unsure what the other is for.

@forki, this is green, with all the original tests, plus a simple test using FCS, please let me know if there is anything we want adjusted.

I think what we need is a bit of documentation on how to use it.
Currently I can't release since the certificate is outdated. But as soon as I get a replacement I will merge and release this

@forki there is now an initial take for the documentation.
Also, it seems to work in dotnet-interactive after copying the binary:

There are rough edges, it somehow keeps adding stuff to the deps file from dotnet-interactive, all of this is still as experimental as it was when it was PR to fsharp compiler repository...

this is released in alpha channel. hurray

@forki is it possible to have the dll part of the github release artifacts? I think that's what the people will download.
Otherwise they still need to compile it themselves.
The other thing to consider is to package this either in the paket or a new nuget package.

Sure. Could you please send a pr against build.fsx - that's where push to github releases


Class '\Omnipay\PaymentSense\Gateway' not found'

Hello, hopefully some one is listening here. I've recently had a client who is insistent on using Payment sense, i've begun the process of introducing this into the app using this integration for Omnipay, but with little luck.
If i choose any other payment gateway, stripe / paypal etc, they work perfect, otherwise I get an error i cannot work out.
I similar issue has been reported here: https://groups.google.com/forum/#!topic/omnipay/Ji-YBZp7Sj0
The suggested resolution does not work. Could someone help?

Actually to test, I quickly changed over to another gateway, randomly picked... "Pagarme" and it works fine, i suspect its something to do with bad name spacing? or the Gateway.php file.
I'll continue to dig as this is something i'd like resolved to ease my integration with PaymentSense, they're documentation is horrible.

Hi Tom, this is one of those Omnipay things which isn't very obvious, but I can see why it was done that way.
When you create the instance of the driver, you need to specify the complete namespace and class name. This is because it is not one of the drivers maintained by the core Omnipay team, living under the main Omnipay namespace.
For example:
$gateway = Omnipay::create('\Coatesap\PaymentSense\Gateway');
I'll put this example on the readme page so it's there for anyone else who stumbles across this problem.

Ahh coatesap, that solved it. Thank you for your help.


Missing build artifacts

Unable to use this package in another project.  Build failure: https://github.com/OpenBeta/open-tacos/pull/204
Consumers can't find build/index.js
In the GH publish action we need to commit files in build/* to the repo

fixed in #31


Debian packaging

The current install method for RedPepper is simply a git clone via an install bash script. While this works fairly well, it would be nice to also make a .deb package that can be installed and updated like other system packages.
I'm not currently prepared to host my own Debian repo for this; I haven't researched if there are options to have it hosted. At least we should have consistently named packages in Github releases.

Switching to a uv installation which is better than OS-specific packaging.


about --url=""

Please tell me what path to take
│ Dropbox raised the following error:                                                                                │
│         ApiError('9dcd400e83224fde8d8132539a5fd655', ListFolderError('path', LookupError('not_found', None)))
I'm using the path that I take from the link from my public dropbox folder

I found it. The link was in the email, but it wasn't highlighted in color or font


No experimental type solving

From: with https://github.com/detekt/detekt/discussions/4959
Right now a lot of our rules use type-solving and it is battle tested. It works! But we still say in the documentation that it is "experimental" but, to me, it is not experimental at all. It's one of the core features of detekt.
So I would like to remove the "experimental tag" from the type-solving, what do you think?

In general I support this, but I'd seen https://github.com/detekt/detekt/issues/4465 as requiring a solution before calling this stable (at least when running detekt from Gradle).
Functional test coverage is also lacking.

What does it mean experimental in this context? Those tasks don't have pulished edges but they are stable, right? I mean, could we track anything new as an improvement?

I've listed a number of related issues in #4465 which still don't have a proper fix, though there are some workarounds in place now to quiet the console output or offer config workarounds.
Those are the kinds of open issues that make me suggest that this feature is still somewhat experimental.
The feature works in most common cases though, and we're generally aware of the outstanding issues. That's why I'm not totally opposed to merging this, since we're not enabling it by default or calling it stable just yet.

The feature works in most common cases though, and we're generally aware of the outstanding issues. That's why I'm not totally opposed to merging this, since we're not enabling it by default or calling it stable just yet.

@3flex I understand your concerns, I know that we need to pulish some edges but in general it is stable and we should promote it. It finds lots of issues and there is no good reason to don't use it.

Closing this as there is not consensous on it. We could continue the discussion on an issue and recreate another PR once we decide.


runDartdev dereferences null in edge case w/analytics

Reproduction on 2.11.0-266.0.dev under Linux but I believe this to be in head as well.
From dart-lang/dartdoc#2412:
dart2-model_test.dart: + DARTDOC_COVERAGE_DATA=/tmp/dartdoc_coverage_dataIPQYRN /home/travis/dart-sdk/bin/dart --disable-service-auth-codes --enable-vm-service:0 --pause-isolates-on-exit --enable-asserts test/end2end/model_test.dart
dart2-model_test.dart: NoSuchMethodError: The getter 'enabled' was called on null.
dart2-model_test.dart: Receiver: null
dart2-model_test.dart: Tried calling: enabled
dart2-model_test.dart: #0      Object.noSuchMethod (dart:core-patch/object_patch.dart:51:5)
dart2-model_test.dart: #1      runDartdev (package:dartdev/dartdev.dart:114:19)
dart2-model_test.dart: #2      main (file:///b/s/w/ir/cache/builder/sdk/pkg/dartdev/bin/dartdev.dart:11:9)
dart2-model_test.dart: #3      _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:297:32)
dart2-model_test.dart: #4      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168:12)

The call to https://github.com/dart-lang/sdk/blob/2.11.0-266.0.dev/pkg/dartdev/lib/dartdev.dart#L49 seems to assume that createAnalyticsInstance can never return null.  However, analytics is null under this circumstance and so we crash at https://github.com/dart-lang/sdk/blob/2.11.0-266.0.dev/pkg/dartdev/lib/dartdev.dart#L114.
I think the bug is here:  https://github.com/dart-lang/sdk/blob/2.11.0-266.0.dev/pkg/dartdev/lib/src/analytics.dart#L72
If the settings directory does not exist, but can be created, we can fall out of this else block and not assign a value to _instance.

Rerunning the command will work (since the directory gets created and we bypass the case).  But on a continuous build infrastructure where the directory is not precreated, the crash comes up like in dart-lang/dartdoc#2412.

@sigurdm @bkonyi

Possibly related to https://github.com/dart-lang/linter/pull/2319#issuecomment-720610121.
@sigurdm - I'm assuming this is related to your recent work on analytics. Can you take a look? We're seeing this issue on most external CI systems using the dev channel.

https://dart-review.googlesource.com/c/sdk/+/170088

@franklinyow @pcsosinski I would love to get this in next beta / dev release, I think it breaks dart for anyone who doesn't have a  $HOME/.dart/ folder, which is effectively all new users (or any environment that is constantly reset, like CI systems).

@jonasfj to be clear this is currently affecting dev/beta but we shouldn't roll w/o it?

Yes, this is affecting the dev release of 2.12 that came out Monday...
I would advise that include the fix in the next dev and beta releases.


Don't start tool pods for masters if masters are not detected

Should seem pretty obvious, but k8s is trying to start pods for an empty list of masters and fails.  Don't do this.

Why is the masters list empty?

See https://github.com/perftool-incubator/rickshaw/issues/70

Fixed with PR #72


[ERROR] add_library cannot create ALIAS target "fmt::fmt" because another target with the same name already exists

I tried to include this library as submodule in a project that already links the "fmt" library, thus causing a problem when it reaches the include of "fmt", as it is already included in the main project.
CMake Error at cmake-build-release/_deps/fmt-src/CMakeLists.txt:300 (add_library):
  add_library cannot create ALIAS target "fmt::fmt" because another target
  with the same name already exists.

I can see there is a property "FMT_INSTALL" which I set to "OFF" in my project, but it is never checked in the CMakeList.txt of libenvpp, and it always tries to include the "fmt" library even if it is "OFF".

Yes, PR #10 fixed it! I look forward to include the new version once released


BugFix backport 26769 to Branch 2.0

Proposed changes
bp #26769

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run performance

run buildall


Fix Upload cache

The following is an attempt to solve #616 but also considerably reduce the io induced by the split cache.
We should move away from the "transparent" caching approach to this problem and give more control to the client to decide what should be evinced from cache, what should really be cached, etc.
struct LocalSplitStore { ... }

impl LocalSplitStore {
   // Called by uploader, only if the split is not mature
   fn put_and_store(&self, split: &Path);
   // Open a split.
   // If the split is not in the store download it.
   fn open_split(&self, scratch_directory: &Path) -> anyhow::Result<Box<dyn Directory>>();
   // Remove the split. It is called from GC.
   fn delete_split(&self, split: String);
   // Remove the splits that are not within the list.
   fn retain_only(&self, split_ids: Vec<String>) -> anyhow::Result<()> {}
}

Writing into the LocalSplitStore will be done in the Uploader.
The uploader has access to the merge policy in order to decide whether it should be stored or not.

I think that makes sense. When I read your observations on #616 yesterday, I came to the same conclusion: ditch the storage interface and develop a new API that offers more control.

Branch issue/619 is a shitty implem of that. After that indexing works like clockwork on a regular instance.

(the branch is not ready at all though... It is just a hack at the moment )


CI should run automatically for non-public PR to develop branch

When I create a PR to the "develop" branch the CI unit tests are skipped and I have to set the approve-for-ci label to start them. "develop" should behave like "main" and the tests should be automatically run for non-public PRs.

@sandorkertesz our definition of a "public PR" is currently "a PR which is opened from a fork". All public PRs need to use the labels to run CI. The CI will run automatically for any PR opened from a branch within ecmwf/earthkit-data e.g. this one from James: https://github.com/ecmwf/earthkit-data/pull/105
This PR from Eddy came from a fork, which is why it was given the "contributor" label and so needs the "approved-for-ci" label for the CI to run: https://github.com/ecmwf/earthkit-data/pull/76

@dtip, it still does not work for me. I still need to add/remove the  approve-for-ci label when I create a PR into develop from a non-fork branch. Otherwise the tests are not run. See: #106.

Fixed by https://github.com/ecmwf/earthkit-data/commit/b2ce68209b8edbad022321ea92e24492e15f614b


FIXfix case for column separator default value

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

not needed anymore


[BUG] Duplicated kubernetes. namespaced metrics when a container restarts

Agent Environment
# agent version
Agent 7.47.0 - Commit: b8c31bf - Serialization version: v5.0.90 - Go version: go1.20.6

Describe what happened:
Just after an application container restart, I noticed a strange behaviour in the kubernetes.containers.restarts metric that I'm summing by env and service in a monitor: for a certain period of time it seems to report a value of 2, and than return back to the expected 1, as I have verified that just one container is restarted. In fact the metric kubernetes_state.container.restarts is reporting the correct values.
Analysing more in depth, I noticed that this metric is duplicated in relation to the tag kube_service that reports both the correct and N/A value for that period of time. I suppose that this behaviour is affecting all kubernetes. namespaced metrics related to the restarted container, as shown in the following explorer screenshot:

Clearly this is affecting also the standard dashboards that are using those metrics:

Describe what you expected:
That kubernetes. namespaced metrics are not duplicated with kube_service tag with value N/A.
Additional environment details (Operating System, Cloud provider, etc):
AWS EKS Fargate, the agent sidecar and cluster agent configurations are more or less the same as described in the EKS Fargate integration guide.

From support:

This is due to a peculiarity of how the kube_service tag specifically is handled.
While ordinarily the Agent is responsible for gathering all the Tags to associate with a given Pod/Container, the kube_service tag (in order lessen the load on the Kubernetes API Server) is the responsibility of the Cluster-Agent instead.
This will be querying the Services and creating the metadata map for Pods and their Services. An important part to note here, is that this mapping will happen once the Pod is ready.
This also means that if the Pod takes awhile to be marked as “Ready” with respect to its Readiness Probe - Kubernetes won’t associate this Pod to its Service until it is “Ready”. This will result in the kube_service tag missing a value for the first datapoints.
Now, to resolve this, there are 2 main workarounds:

If you are using the Agent version 7.45+, you can 'stop' the Agent handling this tag (related Pull Request).
This can be done on a per-Pod basis using the below annotation:
metadata:
  annotations:
    tags.datadoghq.com/disable: kube_service

Or if you wish to disable this on all Pods, using this environment variable (on the Agent, not the Cluster-Agent):
- name: DD_KUBERNETES_AD_TAGS_DISABLED
  value: '["kube_service"]'

Adjusting the Query to only return valid data.
There are also 2 options:

Filtering by tags using kube_service:* will return only data that has values for this tag.
An alternative option is using the fill(null) interpolation filter. You can read up in general on Interpolation here, or more about the Fill Method here.


request help: Gateway Api configuration

Issue description
Hi,
I try to setup a local environment with kind or minikube and use the k8s Gateway API.
I already enable the config enableGatewayAPI but it's look i need to setup also the ingress_publish_service value.
I am not sure about which service i need to use, can you help me?
ty,
Environment

your apisix-ingress-controller version (output of apisix-ingress-controller version --long): standard installation with helm
your Kubernetes cluster version (output of kubectl version): 1.24 (minikube)
if you run apisix-ingress-controller in Bare-metal environment, also show your OS version (uname -a):  Linux i-ubu 5.15.0-46-generic #49-Ubuntu SMP Thu Aug 4 10:03:25 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Typically, it is the APISIX Gateway service.

refer to https://github.com/apache/apisix-ingress-controller/blob/31b3ef84be4e96dc663de883ef5730a03bfb962b/conf/config-default.yaml#L40-L49


Backporting Issue 46 Fix

Since zendframework/zend-stdlib is still requiring zendframework/zend-hydrator with ~1.1, I would like to backport the naming strategy fix from #45.

@boesing

Since zendframework/zend-stdlib is still requiring zendframework/zend-hydrator with ~1.1…

Not for version 3 or higher:
 "require": {
    "php": "^5.5 || ^7.0"
}

https://github.com/zendframework/zend-stdlib/blob/release-3.0.0/composer.json#L15

Can you provide more detail on what exactly this PR accomplishes, please? While I know I reviewed and merged #45, it was some months ago, and I'm unclear what this particular patch accomplishes.
As noted, the justification you use in the issue summary above — dependency of zend-stdlib on zend-hydrator — is simply not true, and I suspect that it's the opposite of that which you are trying to point out here. However, I need to know what exact problem you're trying to solve before I can review this meaningfully.
Thanks!

The only thing why I would like to backport this is, that we are still using zend-stdlib v2 and cannot update to v3, yet.
Since zend-stdlib v2 requires zend-hydrator v1.1 (as mentioned above), I would love to see this fix in zend-hydrator v1 as well.
Thats all, actually. Even tho, I guess there are probably many other projects which are still running zend v2.
Hope we can get this merged. Thanks.

@boesing

…that we are still using zend-stdlib v2 and cannot update to v3, yet.

You say: "not yet". The problem if we backport this fix, then you will also update not later. 😉

I have a fix for us which applies the namingstrategy and adds both cases as strategy.
Anyway, I am still not the one who decides when to update and refactor 3 projects or not.

The point of the 1.1 release was to provide a bridge release between the zend-stdlib v2 and v3 series; it is not intended as a long term support (LTS) release.
What is preventing you from updating to zend-stdlib v3, exactly? I can only think of two things:

If you are on a PHP version prior to 5.6. Otherwise, the only removals from that library are the hydrators, GuardUtils (which only existed for PHP versions prior to 5.4), and Zend\Stdlib\DateTime, which only existed to provide a polyfill for createFromISO8601() (which is no longer necessary as of PHP 5.4).
If a library you depend upon is still pinned to zend-stdlib v2. If this is the case, and it's a zendframework or zfcampus library, let us know so we can get it updated.

There are still packages (e.g. zendframework/zend-eventmanager) which require zendframework/zend-stdlib with ^2.7.
Thats why we actually cannot upgrade to zendframework/zend-stdlib ^3.0.
I've check the other dependencies but actually only zend-eventmanager is blocking us from upgrading to at least zend-stdlib v3

@boesing
You can use zend-stdlib v3!

There are still packages (e.g. zendframework/zend-eventmanager) which require zendframework/zend-stdlib with ^2.7.

It was changed with version 3.0.1 of zend-eventmanager.

I guess that zend-eventmanager is not updated to zend-stdlib v3 since the CallbackHandler is being used.

It was changed with version 3.0.0 of zend-eventmanager.

Yes but as there are breaking changes in any v3, its not about "you can use" (yes, I know), its about "you have to refactor hundret usages of eventmanager#trigger to eventmanager#triggerEvent and thousand of listeners which implements ListenerAggregateInterface" which added a useless priority to the Method declaration.
Anyway, thanks for your statements.

@boesing
My comment was referred to zend-stdlib and v3, because the question was if there is a problematic dependency and whether we can help or must update a component.

The zend-eventmanager v2 series does pin to zend-stdlib v2, because it uses Zend\Stdlib\CallbackHandler, a class we removed in zend-stdlib v3.
CallbackHandler existed to do validations around PHP callables prior to introduction of the callable typehint in PHP. Once that typehint was introduced, the CallbackHandler was largely redundant. (The one feature it provided over PHP was to allow passing a class name of a class with an empty constructor as either a callable — if it defined __invoke().)_
In zend-eventmanager v3, as you note, we modified a few signatures within the EventManagerInterface and EventManager. These were done primarily to allow us to use the callable typehint instead of using CallbackHandler. (Additionally, we offer the triggerEvent() and triggerEventUntil() methods, which allow stronger typehinting when triggering events.) These changes could not be done in a BC way, which was why we bumped the major version.
The point I'm making is: zend-eventmanager has been updated to lose the dependence on zend-stdlib v2. However, this was done in its own major version bump, for the reasons as outlined above.
Backporting changes to the zend-hydrator 1.1 branch so that you can consume zend-stdlib v2 because another dependency requires that version is a tough sell, as it adds more maintenance overhead for us; we then have to consider any patch against the current master branch against 1.1 releases, which adds quite a lot of maintenance overhead.
Your best bet is to upgrade to zend-eventmanager v3. We provide a migration guide, and you can look even at some of our repositories to see how we managed the migration internally (see zend-view, zend-mvc, and zend-modulemanager for examples). One thing we did was to make the latest zend-eventmanager v2 minor release (2.6) contain features to make it possible to make your code forwards compatible with the v3 release. If you read through the linked migration guide, you'll find tips and tricks for doing so.

It's all fine. I've spent a day of my weekend to migrate our projects to at least eventmanager and stdlib v3.
Thanks anyway.


Exception when using MUMBAI_NETWORK

I'm submitting a...

[ ] Regression
[x] Bug report
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request => Please do not submit support request here, instead post your question on Stack Overflow.

Current behavior

When setting most of the networks constants in config options, the module connects just fine. Except when using MUMBAI_NETWORK I receive this error.
[Nest] 64120  - 08/04/2022, 4:12:18 PM   ERROR [ExceptionHandler] unsupported getDefaultProvider network (operation="getDefaultProvider", network={"name":"maticmum","chainId":80001,"ensAddress":null,"_defaultProvider":null}, code=NETWORK_ERROR, version=providers/5.6.8)

Expected behavior

I expect the module to connect to Polygon Mumbai. All other tested network constants did not throw an error.
Minimal reproduction of the problem with instructions

EthersModule.forRoot({
  network: MUMBAI_NETWORK,
}),

What is the motivation / use case for changing the behavior?

Environment

Nest version: ^8.0.0
Nest Ethers version: ^1.0.3

For Tooling issues:
- Node version: v16.14.2
- Platform: Mac
- Server: Express

Others:

I get a similar error for setting network to `http://localhost:8545` as described in options docs, when trying to connect to local hardhat node. `npm hardhat node`

`[Nest] 70323  - 08/04/2022, 5:28:08 PM   ERROR [ExceptionHandler] Invalid network http://localhost:8545.`

@vanb. I will take a look.

I believe I was able to resolve this by setting use default provider to false and setting an alchemy key. If that's required then perhaps I missed it in the docs.

Prefect. I will close this issue for now


Django / (login,registration,logout)

create an app 'accounts'
urls :login , signup , logout
use usercreationform
form fields : Full name , username or handle , password , college , DOB ,email

@dishantsethi do you want to claim this task? Can you work on this?

@AniVerma17 Yeah i can

@AniVerma17 Yeah i can

@dishantsethi alright then, I'm assigning this issue to you.

Is this issue still open?
I would like to contribute to this

@SameeranB yes the issue is open and up for grabs. Do check out the existing Django codebase on develop branch and work on the pending feature(s) required as per this issue.


[BUG] Experimental Feature of ChatGPT is not available in local docker setup

Willingness to contribute
Yes. I can contribute a fix for this bug independently.
Feathr version
1.0.0
System information

OS Platform and Distribution (e.g., Linux Ubuntu 20.0):
Python version:
Spark version, if reporting runtime issue:
Docker

Describe the problem
@blrchen
After tested the ChatGPT experimental features, I found out that the feathr.chat was not able to be run.

Tracking information
No response
Code to reproduce bug
No response
What component(s) does this bug affect?

[X] Python Client: This is the client users use to interact with most of our API. Mostly written in Python.
[ ] Computation Engine: The computation engine that execute the actual feature join and generation work. Mostly in Scala and Spark.
[ ] Feature Registry API: The frontend API layer supports SQL, Purview(Atlas) as storage. The API layer is in Python(FAST API)
[ ] Feature Registry Web UI: The Web UI for feature registry. Written in React

Tested installation with the following command. Feathr version is still 1.0.
pip install git+https://github.com/feathr-ai/feathr.git#subdirectory=feathr_project

Related PR: https://github.com/feathr-ai/feathr/pull/1132/files

@ahlag PR 1132 is not  included in version  v1.0.0. Could you please try to use latest feathr package from git repo instead of using v1.0.0 in your notebook

@blrchen
Is there a doc on how to compile or run feathr locally from the repo?

Running pip install git+https://github.com/feathr-ai/feathr.git#subdirectory=feathr_project should install latest feathr package with main branch code. This will ensure you get the code of PR 1132 which is required for the notebook.

This solved the installation problem. ChatGPT feature tested.
https://github.com/feathr-ai/feathr/blob/a1da6596928e1d8acbb74f88c4f603f2508a258a/docs/dev_guide/python_dev_guide.md#installation


Implement swagger and prometheus endpoints for the Go JobType

Swagger is already there, check out this one: https://github.com/TheRacetrack/plugin-go-job-type/blob/4171eb8eefcca04958c67e08faf9e78ea9a57df8/golang-job-type/go_wrapper/openapi.go#LL11C13-L11C13
Nevertheless, Prometheus endpoints are not there, as you stated.


Error detected while processing BufRead Auto commands for "*"

I get the error the following error message when I use :Errors:
Error detected while processing BufRead Auto commands for "*":
E492: Not an editor command: [Errno 2] No such file or directory: ''^@

This happens only if I have au BufRead * execute stuff in my Vim configuration. I intend this to match all filenames with the *. I know next to nothing about Vim syntax so I'm probably doing something wrong.
If I replace it with au BufRead *.* execute stuff, :Errors no longer triggers the above error. But this causes files that do not have . in them to not be matched. Should I be using something else instead of *?
Thanks

I can't reproduce this, and I don't really see how it could be related to syntastic.  My guess would be it's either a problem in the way you wrote stuff, or a bug in Vim.

To reproduce it, use indent-finder:
$ cd ~/.vim/bundle
$ git clone https://github.com/myint/indent-finder

And here is my temporarily workaround.
Thanks

Here is a relevant screenshot.

Oh, I figured out what the problem was. I'm doing a system() call with the au BufRead * first and then get the results. The problem is this gets passed an empty string as argument to the system call. This results in an error message from the system call [Errno 2] No such file or directory: '' to be executed in the next line, which is au BufRead * execute.
https://github.com/myint/indent-finder/blob/7f6a2c17a7d725f5e22f5818d6f7934e2d3b25ca/plugin/indent_finder.vim


Autocomplete doesn't support GetBoundingClientRect width method in IE8

http://msdn.microsoft.com/en-us/library/ms536433(VS.85).aspx

I'll close this since we're no longer adding fixes for IE8.


missing "new game" button

new game button needs to do:

delete everything that is on canvas
draw unopened squares everywhere
change current_game to a new Game instance
something else I forgot?

Edit: updated list to use class Game introduced in b8c99a07a5b852f870f9cd5402c7deee893d4077

Some things with the code added in ffda92860853a42708365b2ea8f0232a29f928bc still need to be fixed:

[ ] Make a new Game object and get rid of current_game.dict_or_list.clear()
[ ] Call new_game() when the program starts (and a new game begins)
[ ] Get rid of copy/pasted ***Lets go!*** (it's enough to have it in new_game())
[ ] Get rid of copy/pasted loop that draws unopened squares
[ ] Delete all items on the canvas from the old game because they are no longer needed

Some things with the code added in ffda92860853a42708365b2ea8f0232a29f928bc still need to be fixed:

[ ] Make a new Game object and get rid of current_game.dict_or_list.clear()
[ ] Call new_game() when the program starts (and a new game begins)
[ ] Get rid of copy/pasted ***Lets go!*** (it's enough to have it in new_game())
[ ] Get rid of copy/pasted loop that draws unopened squares
[ ] Delete all items on the canvas from the old game because they are no longer needed

ARE YOU COMPLAINING ABOUT MY CODE, HUH?????!!!!!!

Setting current_game to a new Game object is done like this:
global current_game
current_game = Game()

global is bad style, but it's unavoidable in this situation. We can later clean  up the code to use another class to avoid it, but that's beyond the scope of this issue.

Setting current_game to a new Game object is done like this:
global current_game
current_game = Game()

global is bad style, but it's unavoidable in this situation. We can later clean  up the code to use another class to avoid it, but that's beyond the scope of this issue.

Setting current_game to a new Game object is done like this:
global current_game
current_game = Game()

global is bad style, but it's unavoidable in this situation. We can later clean up the code to use another class to avoid it, but that's beyond the scope of this issue.

global current_game
current_game = Game()
This doesn't wipe anything on the canvas.. it just lets me click it again.

Setting current_game to a new Game object is done like this:
global current_game
current_game = Game()

global is bad style, but it's unavoidable in this situation. We can later clean up the code to use another class to avoid it, but that's beyond the scope of this issue.

global current_game
current_game = Game()
This doesn't wipe anything on the canvas.. it just lets me click it again.

It replaces current_game.some_list_or_dict.clear(), not any of the other code in new_game().

It replaces current_game.some_list_or_dict.clear(), not any of the other code in new_game().


Fix build of Tango Scraper

After the merge of the Tango Scraper, the main branch has some build issues. This needs to be fixed.

This is now done


Allow disabling some of the tags

I may use the words "note" and "section" in a writing, and so wouldn't want them to be considered "tags". Simply removing them from the tag list doesn't seem to do anything:
    "commentAnchors.tags.list": [
        {
            "tag": "ANCHOR",
            "iconColor": "default",
            "highlightColor": "#A8C023",
            "scope": "file"
        },
        {
            "tag": "TODO",
            "iconColor": "blue",
            "highlightColor": "#3ea8ff",
            "scope": "workspace"
        },
        {
            "tag": "FIXME",
            "iconColor": "red",
            "highlightColor": "#F44336",
            "scope": "workspace"
        },
        {
            "tag": "STUB",
            "iconColor": "purple",
            "highlightColor": "#BA68C8",
            "scope": "file"
        }
    ]

You can already disable a anchor by adding enabled and set it to false like this "enabled: false".

Got it. Thanks! And I suppose that new tags can be added as well?

Thats right! You can check the README for instructions on how to further use and configure this extension

Thanks! The "enabled" property wasn't mentioned I believe, so maybe it's worth it to update the Readme. And there's no explicit example to add a new tag even though I think I could guess that.

Thanks! The "enabled" property wasn't mentioned I believe, so maybe it's worth it to update the Readme. And there's no explicit example to add a new tag even though I think I could guess that.

My apologies, I've indeed forgot to mention it. Thanks!


How to restrictly set a MAX-SIZE-LIMIT for each download task?

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
I would like to give a limit before downloading everything, so I'm wondering if that possible to

predict the total bytes of whole download task before actually download all of the files;    or
set the limit and download is stopped once the transferred bytes exceed it

Thanks!
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

set the limit and download is stopped once the transferred bytes exceed it

https://github.com/yt-dlp/yt-dlp#video-selection
--max-filesize SIZE             Abort download if filesize is larger than
                                SIZE, e.g. 50k or 44.6M

predict the total bytes of whole download task before actually download all of the files; or

There is no fully automatic way of doing this, but you could print all of the known filesizes with this command:
yt-dlp --print "%(filesize,filesize_approx)s" "URL"

and then add them together with an external command/script

Thanks @bashonly
I tried both of them however they are not working well with twitter url, which is parsed into several m3u8 links.

--print filesize gets "NA"
--max-filesize doesn't stop the download

is there any other workaround for twitter url?

Or can I assume that there's no simple or uniform approach at this moment?

--print filesize gets "NA"

For manifest formats (e.g. m3u8/HLS and mpd/DASH), you'll need to additionally pass --compat-opt manifest-filesize-approx in order to include those formats' inaccurate filesize_approx values in the extracted info (documented here). So that would be:
--print "%(filesize,filesize_approx)s" --compat-opt manifest-filesize-approx

--max-filesize doesn't stop the download

IIRC --max-filesize doesn't work as expected with manifest formats (HLS/DASH). I am almost positive there is an open issue about this, but I can't find it right now

Thanks, it works!

... there is an open issue about this, but I can't find it right now

ytdl-org/youtube-dl#22133 covers this. Here and upstream, the filesize test is only applied for external (where the downloader supports it) and http downloads.
I guess the fragment downloader(s) should test against the filesize[_approx] in the info JSON, which would have always to be generated.


starrocks don't support substring_index which mysql did

starrocks-client：
mysql> select substring_index(substring_index('name(10)', '(', -1), ')', `1)；`
ERROR 1064 (HY000): No matching function with signature: substring_index(varchar(-1), varchar(-1), tinyint(4)).

https://dev.mysql.com/doc/refman/5.6/en/string-functions.html#function_substring-index
Returns the substring from string

@selectbook hi could you please reply here so I can assign this to you?

Could you please assign substring_index to me? thx!

@kateshaowanjou Could you please assign substring_index to me? thx!

Hi @leiyang0324, are you still working on this issue? Please let me know if you need any help. Thanks!


Feature request: fractional ply level for computer opponent

Imagine a situation where the level 1 ply is too easy to beat, but the level 2 ply is too difficult (since the level 2 engine is twice as smart). It would be great to have the ability to set a fractional ply between 1 and 2, allowing players to fine-tune the difficulty.
For instance, at a setting of 1.3, the base level would be 1, but for 30% of moves, the level would be 2.
It can be computed like that:
current_move_lvl_ply = lvl_base + int(rand(0,1.0) < lvl_fraction);

Also related to #153

Interesting idea.
I will give it some thought.
Perhaps it can be combined in some with the solution to your other issue.

Will go with solution from related issue.


Tailscale needs --accept-dns=false for PiHole/AdGuard Home

Problem/Motivation
This was discussed in #74 but prematurely closed. To use Tailscale together with AdGuard Home (or PiHole), Tailscale needs an extra config option: https://tailscale.com/kb/1114/pi-hole/

When running tailscale up, you’ll need to pass the --accept-dns=false flag. Pi-Hole uses DNS servers configured within Linux as its upstream servers, where it will send DNS queries that it cannot answer on its own. Since we’re going to make the Pi-Hole be our DNS server, we don’t want Pi-Hole trying to use itself as its own upstream.

tailscale up --accept-dns=false

Once installed, and you’ve run tailscale up --accept-dns=false on your Raspberry Pi, continue on.

Current status
Only tags can be configured.
Expected behavior and proposed changes
Additional CLI options should be available in the configuration, not just tags.

Just noticed it is actually fixed in 3656ff4484ed913d19bee2d9e72c0ab3739cc4d4, but this commit has not been released yet. A new release would be nice.


Press arrow keys got number pad key instead while using vscode by X11 forwarding

Similar to #33912

Version: 1.28.2
Commit: 7f3ce96ff4729c91352ae6def877e59c561f4850
Date: 2018-10-17T00:20:56.183Z
Electron: 2.0.9
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
Architecture: x64
OS Version: Ubuntu 16.04 ( ssh from windows10)

Steps to Reproduce:

ssh login to Ubuntu 16.04 with X11 forwarding setup
start vscode, and press arrow keys ( same as Home, End, Pgup PgDn)

By press left arrow key, you got number'6'  input repeatedly,  can be stopped by hit Ctrl-C.
By press up arrow key, you got number  '8'  input repeatedly,  can be stopped by hit Ctrl-C.
If open the keyboard shortcut editor, press arrow key you can see number pad key repeatedly pressed.

Does this issue occur when all extensions are disabled?: Yes

#24012

That's supposedly fixed, must be a new/different issue.

Maybe this is a Mobaxterm issue, I can reproduce this issue by xterm too.


Add support for statistics for 10 sensors

Adds support for statistic collection in HASS for the following sensors:

battery_level
adblue_level
fuel_level
charging_time_left
charging_power
charge_rate
electric_range
combustion_range
combined_range
outside_temperature

Using these statistics will need HomeAssistant 2021.9.0+

Thanks!


Remote ssh extension cannot switch terminal by mouse click

Does this issue occur when all extensions are disabled?: No

VS Code Version: 1.57.0 (user setup)
OS Version: Windows_NT x64 10.0.17763

when using remote-ssh to open a directory over a remote host (Remote-SSH extension v0.65.6)
the new terminal panel browser, doesn't switch on click, have to sue arrow keys in the panel browser part to switch between opened terminals
Steps to Reproduce:

remote-ssh to a host (Remote-SSH extension v0.65.6)
create multiple terminals in terminal panel
try switching by clicking on the panel browser (right pane inside terminal panel), it won't switch

I believe this is already fixed, can you see if you can reproduce in the Insiders build?