Adding non-breakable spaces lua filter

This adresses #114 .
Its first pull request I have ever made, so hopefully I did not messed anything up. I know it is a whole bunch of files, which should be avoided, but I think that in this case it should be put-and-packaged as one ...
Per #114 it still has not implemented the change in tables and presence-check, since I am having trouble understanding that syntax (idiom) and using it. Hopefully it doesnt mind too much.
Regards, Tomas

OK, it seems that somehow I did not upload the makefile?
I have added makefile to my fork, which looks like added the makefile also here ... Well, I have to learn about git and GitHub :D

On GitHub a Pull Request is a request to merge one branch into another, in this case your fork's master branch into this repository's. You can update PR's by adding, removing, or rebasing commits in a branch. Typically (and to get more GitHub tooling) you'd open a PR from a branch on your fork other than master. In that case maintainers of the target project can also make changes to the PR before merging. As it is only you can, but anything you do in your master branch will continue to change this PR until it is merged or closed.

Oh, I wasnt sure if I should create new branch in my fork or not. Next time I do better.
Thank you for clarification.
Do I understand you correctly, that now nobody (except me) can make edit to this PR? I thought, that the "Allow edits by maintainers" checker is to control this. I have it checked, which I hoped is for making it possible for project maintainers to correct any errors in this PR (well, if they prefer that over telling me what should I fix).

I thought, that the "Allow edits by maintainers" checker is to control this. I have it checked

Really? Last I checked (less than a week ago) that checkbox was disabled for all PR's from master branches. If you see the box and have it checked then GitHub has updated something recently, and that's a nice fix that I'm glad to hear about.

I have used this manual:
Pull request from a fork
Step no. 7 is the one which we are talking about, I hope. Except I did the exact opposite: Left it checked to allow maintainers to do with this PR anything they deem neccessary ...
Btw. It seems that bibliography test are misbehaving; maybe because of newest update?

Interesting. Today, I was looking at some legacy production code with the exact same purpose; thinking by myself that I better replace it by a Lua filter. The current code is a monster of a nested sed that lives in my makefile.
The sed command creates secondary sed substitution commands from a list of so called short begin words.
These secondary sed commands will replace the ordinary space after a short begin word at the beginning of a sentence with a non-breaking space.
The list of short begin words is language dependent. Here are the English and Dutch lists.
Finally, we do not want such substitutions to occur in <PRE>-formatted code blocks.
That is where the /```/,/```/! is for.

@stroobandt Thank you for posting that!
Sadly, I am not enough versed in makefiles or sed to make much of it, but from the lists I see that you went in simillar direction. I am actually not sure how to implement different behavior according to language (well, I would have to read somehow the metadata variable lang).
If you are concerned about code-block (I was too) I have tested that (and it is even in test file of the PR) - codeblocks and inline code is recognized well and no space substitution happens there.

Thanks for the updates! I'll take a closer look soon, I just need to fix the tests first. This may take a while, as I'm going to spend less time in front of the screen for the next couple of days.
The filter name "nonbreakingspace" is a bit generic and somewhat cryptic. Can we find something more specific? How about using the same name as the TeX package, "vlna"? Would that be too confusing? Maybe "west-slavic-nbsp"? I'm open for ideas.

Well, "pandoc-vlna" in itself could be good, but I would think that it would introduce different "crypticity;" that is, that LaTeX packages will be known only to LaTeX users; and most probably to the subset that doesnt already used theyre own homemade solutions (I would guess that those with homemade solutions will be actually plenty).
I thought that naming the filter by what it does as much as possible will prevent its name from being misleading. Very often I am taking inspiration from lua filter pagebreak - which does exactly what it is named after: inserts pagebreak.
But in the spirit of your previous comment, also "insert-nbsp" (or no-abbreviated form "insert-nonbreakable-space") could work too. I would like to aviod specifying any nationalities or geografic locations to the filter, because it might be offputting to someone (or even any way worse). The code state is mostly caused by me being lua newbie and natively Czech, which leads me to make the default setting suitable for Czech language requirements. I hope that the filter as it is is customizable enough for anybody to make changes according to his native language requirements.
If anybody would know how to make this filter conformant to more than one language, I would be happy to make more improvements. In this regard, my only idea is to have multiple (well, depending on how many languages a lot) tables for each language and then (somehow) read metadata lang variable. But that is highly beyond my linguistic and pandoc-hacking skills ...
@tarleb Let me know which identification is more suitable:

pandoc-vlna
insert-nbsp
insert-nonbreakable-space

At the end of the day, as famous classic says:

"A rose by any other name would smell as sweet"

... as long as its well described in README.md (hopefully it is).
Regards, Tomas

I'd like to ask a favor. I compiled a new contributing guide over the last two weeks and would like to get some feedback on it.  I'd appreciate if you could take a look at the document and tell me about all passages that are unclear or that you feel should be improved for other reasons.
I am not commenting on the code yet, as I'm also trying to evaluate if the information I added to the guide is sufficient to prevent some common issues which often come up during code review. Hope that's ok with you?
As for the name: each options seems fine. Since Czech is the only supported language, we could also be more explicit about the filter's purpose and include czeck in the name.  I'll leave the choice to you.

Considering the new guide, everything mentioned seems pretty completely clear and OK to me. I can accomodate for the ammount of characters per line. Editorconfig seems like a great suggestion, although I am not familiar with that utility.
I take another look at the diagram-generator filter and try to accomodate for setting this fiter according to lang metadata value. I can try to make another set of rules for english (and make that actually default).
That could even make it valid to name the filter pandoc-vlna (well, in my opinion).
I let you know when all changes are finished (could be pretty fast).

@tarleb I have finished changes according to the newest contributing guide. All should be good to go!
Only thing I realized that in case of Spans or Divs it doesnt check, if those have set theyre lang (attribute?) differently. It is something I can work on when there will be more languages supported (for two languages it might not make much sense).
The only thing is that I am having trouble changing the directory name itself from nonbreakablespace to pandocVlna (now I dare to name it that way) ... :)

Added another update with all improvements suggested in review. Also updated filter file header according to new contribution guide.

Check are failing, but this is issue of other filters and seems to be connected to change to having ATX headings by default.

Many thanks for this filter, @Delanii! I wonder if it is possible to add the % character to the list. In Czech, we write non-breakable space between the number and % sign (when percentages are used as an adjective, there is no space). I've tried to change the filter, but failed miserably. Maybe % needs to be escaped as \%, but that fails too (as well as \\% or \\\% does). Help would be greatly appreciated!

Just a side note: in lua regexp, % is the escape char, so you have to type %% so
that it escapes itself.
I wonder if this filter could be made more general by creating some metadata
fields taking a list a characters to be associated with a non-breakable space.
Something like "non-breakable-space-before", "non-breakable-space-after",
"thin-space-before", "thin-space-after". Of course it would be complementary
to the built-in configuration files per language that you mentioned. What are
your thoughts about that?
Le Thursday 06 May 2021 à 03:31:12AM, Jan Netík a écrit :

Many thanks for this filter, @Delanii! I wonder if it is possible to add the %
character to the list. In Czech, we write non-breakable space between the
number and % sign (when percentages are used as an adjective, there is no
space). I've tried to change the filter, but failed miserably. Maybe % needs to
be escaped as %, but that fails too (as well as \% or \% does). Help would
be greatly appreciated!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.*

Many thanks for this filter, @Delanii! I wonder if it is possible to add the % character to the list. In Czech, we write non-breakable space between the number and % sign (when percentages are used as an adjective, there is no space). I've tried to change the filter, but failed miserably. Maybe % needs to be escaped as \%, but that fails too (as well as \\% or \\\% does). Help would be greatly appreciated!

Hello,
well, this filter doesnt cover it all. I admit I have settled up to dealing with things that were the easiest -- just list few prepositions and deal with split numbers. The percent character could be thought about as a unit, which has actually the opposite requirement -- has to be coupled with preceeding text element (number, as in "10 %"), not with the following one (word, as in "10 apples").
As I wrote in the beginning, I was and still am a lua newbie, so by cutting off units, chapters (ex: "chapter 9") and such, I have made creating this filter a lot easier.
I am not very sure how I would deal with units and such in this filter. In LaTeX this is being dealt with by means of famous siunitx package, so I could take a peek at its code and try to inspire myself, but I think that that would require actually adding extra markup that would be recognized by filter, something like @unit{5}{%} or @unit{10}{m/h}, but I am not sure if everybody would wont to write so much markup just to have nonbreakable spaces between number and unit.

Just a side note: in lua regexp, % is the escape char, so you have to type %% so that it escapes itself. I wonder if this filter could be made more general by creating some metadata fields taking a list a characters to be associated with a non-breakable space. Something like "non-breakable-space-before", "non-breakable-space-after", "thin-space-before", "thin-space-after". Of course it would be complementary to the built-in configuration files per language that you mentioned. What are your thoughts about that? Le Thursday 06 May 2021 à 03:31:12AM, Jan Netík a écrit :

Hi!
Well, that is a great idea, I was actually thinking about something like that in the beginning, but then I thought/hoped that this filter could grow by feedback to incorporate more languages with their respective rules. I wanted the filter to be as simple to use as possible so there is no configuration. But the code is open as well, so you can modify it to your respective needs (by just modifying the table with prepositions) to tweak the behaviour as you need.
I also have no idea how to implement configuration of a lua filter. And how would that be stated (configuration file, default file, stating at command line??)
Do you have an idea how to do that?

I am not an expert either, but here is my guess.
If you wish to support multiple languages, I think that the filter itself should
contain only general-purpose functions, not language-related data, exactly what
you do in l. 117-140. So we could imagine an Inlines function that only calls
functions like: replace-space-by-nbsp and replace-space-by-thin (if you expect
from the user that he or she types a normal space where the nbsp should be, which of
course is debatable). Every function would then read a table containing the
characters that need a nbsp.
For the configuration, there could be two non-exclusive ways: the one that I
mentioned, and another based on configuration files per language. The easiest
way from the programmer's point of view is to write configuration files as
Lua tables, so that they can be directly imported in the lua filter with dofile.
It could look like this (simplified for French):
local nbsp_before = {
  ':',
  '»'
}

local nbsp_after = {
  '«'
}

local thin_before = {
  ';'
}

spaces_config = {
  nbsp_before = nbsp_before,
  nbsp_after = nbsp_after,
  thin_before = thin_before
}

Then simply add dofile "config/FR_config.lua" near the beginning of the file. Then you will be able to get the list of the characters that require a thin space before it with spaces_config.thin_before. Of course, these "characters" can be whole strings like the. That would only require to map the supported IETF language tags to the corresponding configuration files before calling dofile. Such configuration file could be easily written by users by modifying an existing one, I think.
Another thing: why do you define the nbsp character per format? Pandoc does the
conversion itself (try pandoc -t latex -f html <<< '&nbsp').
I can help if you want. As a French user who currently selects manually every special space, I am very interested in this filter! However, in that case, it may be easier to do it on a separate repo for I already have a fork of lua-filters with a pending pull request...

Hi both
I am very interested in this as well. I currently do this with a bash script as filter and use sed. Not very clean :-)
What about specifying functions depending on the language set?
For instance if meta.lang = fr then we may provide a list of functions to replace the non-breakable spaces with the right non-breakable space that should appear in French.


Update Assertions documentation

The "Assertions" documentation is out of date, providing details of the "HttpRequestBuilder.assertThat" API that no longer exists.
The documentation should discuss the "Simulation.SetUp.assertions" API instead.

Where did you find such reference? The only place I could find is the Gatling 1.5 to 2.0 migration, saying it was removed.

http://gatling.io/docs/2.0.0-RC2/general/assertions.html

Our documentation is versioned, and you're referring to a milestone (hence unstable) that was released 1 year ago. You have to use the version of the documentation that matches the version of Gatling you're using!


Bug fixes for notifySetpointsStop and cmdVelocityWorld

Add cmdVelocityWorld code in crazyswarm_server.cpp
Add notifySetpointsStop code in crazyswarm_server.cpp
Fix typo and add groupMask argument to notifySetpointsStop service call in crazyflie.py
Update crazyflie_ros submodule

This is my first PR, please let me know if I screwed up anything ;)

Looks good! I will test your changes on our system on Weds or Thurs.

@MauroPfister it looks like you are adding commits for your own experiments to this branch. They become part of the PR. Please create a new branch for that work and revert the changes here.
PRs reference a branch name, not a commit. This is confusing at first (I'm sure I made the same mistake) but it is desirable for code review cycles.

Oh I am sorry, was not aware of this. Shall I revert the commits or delete them. I know that deleting commits is usually not a good option for a public repo, but if only you pulled that branch it might be cleaner?

Just revert them - I will squash this all into one commit anyway.

Hope this is okay now. Thanks for the patience :)

Thank you! Your changes work in our lab too :)


hotswap?

hotswap for intelliJ? currently, I need to restart the game every time I make changes.

Hotswap is not really part of the template, but rather your IDE Setup! If you want a tutorial on how to set it up, check out NEUs Hot Reloading Tutorial. Keep in mind hotswapping will break some things, so you will still need to restart for some changes. I'm leaving this open as a reminder to add a hotswap tutorial into this template.

https://moddev.nea.moe/hotswap/ exists now


!feat(metadata-service): added basic snackbar handling (DSP-1598)

resolves DSP-1598

Good job!
Personally, I think there is room for improvement with the look and feel of the snack bar: I'd show it shorter, but would do the fade-in and fade-out longer. And it looks a bit chunky at the moment (maybe round edges, or some transparency, or some shadow, so that it "floats" over the page rather than being a massive block).
But again, those are details, and I like what you did.

Thanks! We can always adjust it in the future. I'lll make this component more configurable, but for now there is constant 5s display time. Transparency is kind of tricky, because in some cases it can decrease readability. All things related to styling are always personal ;)


In-house rendering

Currently, our best polytope visualization option is to export a blob of Mathematica code for the user to run. This is not great.
I looked at a couple of python packages that draw polytopes, but I didn't make any headway.

pyny3d has some really basic, matplotlib-style support for Polyhedra. My experience with drawing 3D figures in mpl is that you have to handle depth sorting yourself, and all their examples look pretty trivial, so I didn't push very hard.
fresnel looks amazing, but I couldn't manage to survive the build process.

Here is what I have been using
import matplotlib.pyplot as plt
plt.close()
fig = plt.figure()
ax = fig.add_subplot(111, projection="3d")

from weylchamber import WeylChamber
w = WeylChamber();

total_coord_list = []
for subpoly in reduced_vertices:
    subpoly_coords = [[float(x) for x in coord] for coord in subpoly]
    total_coord_list += subpoly_coords
    w.scatter(*zip(*subpoly_coords))

from scipy.spatial import ConvexHull
pts = np.array(total_coord_list)
hull = ConvexHull(pts)
for s in hull.simplices:
    s = np.append(s, s[0])  # Here we cycle back to the first coordinate
    ax.plot(pts[s, 0], pts[s, 1], pts[s, 2], "r-")

w.render(ax)

%matplotlib widget

Here is my new version on my fork


Support multiple sockets, UDP and multicast

Fixes #76

add support for UDP sockets
add support for multicast groups
update dependencies

when is this getting merged, i really need the updated dependencies xD

hopefully it will get reviewed and merged later today - we really want to have every small change smoke-tested at least
but since this crate is currently only available via a git dependency you can also temporarily just use the fork/branch - but it should get it today anyways

@jessebraham Mind testing the examples on all the supported targets with your setup? I don't expect problems but would be good to double check this


add dcm_init() startup function

to make libdicom threadsafe, resolves https://github.com/ImagingDataCommons/libdicom/issues/19

It looks like dcm_init() itself isn't thread-safe, which makes this unsafe to call from other libraries except from their shared library constructors.  (There might be two threads, where one calls library A which calls dcm_init() and the other calls library B which calls dcm_init().)
We could instead make this a shared library constructor, or call it on first use via a GOnce-style primitive.  I guess we don't want to add the dependencies for either approach?

I agree, I don't think a pthreads dependency is a good idea, which I think something like GOnce would need.
We could use __attribute__((constructor)), though I suppose we'd also need to expose dcm_init() in case the compiler or platform had issues with it.

@jcupitt we initially implemented the dictionary using a static lookup table. Given the challenges with making dcm_init() threadsafe, would it make sense to reconsider this approach?

I think the constructor tag should be simple and safe 99% of the time. The code is now:
/**
 * Start up libdicom.
 *
 * Call this from the main thread during program startup for libdicom to be
 * threadsafe.
 *
 * If you don't do this, libdidom will attempt to call it for you in a safe
 * way, but cannot guarantee this on all platforms and with all compilers, and
 * therefore cannot guarantee thread safety.
 *
 * This function can be called many times.
 */
DCM_EXTERN
DCM_CONSTRUCTOR
void dcm_init(void);

So the init function will usually be called automatically when the library is loaded. If a threaded application wants to be 100% safe, it needs to arrange to init the library as part of its own startup. This is a pretty standard and reasonable requirement and I think expected behaviour.


Logfiles: test if target is an unexisting file before download

This is checking if the target log file is not a directory and if it does not already exist. It errors with an InvalidArgument error if the argument is not appropriate.
This hopes to improve issues like https://github.com/mavlink/MAVSDK-Python/issues/323 and https://github.com/mavlink/MAVSDK-Python/issues/312 where it was not clear how to use the arguments.

Aha do I need to add -lstdc++fs somewhere? @julianoes would that be acceptable?

Not sure if we want this, but just trying to do it for the exercise (would be good to know if we can benefit from filesystem later or not, also for e.g. the mavftp plugin)

-lstdc++fs no idea? If it builds for everything we support, yes.

I needed to link -lstdc++fs for some platforms but not all (not necessary on Windows, but it was failing with macOS/iOS and Android so I removed it there). It was needed for Ubuntu 18 at least (together with the #ifdef logic with experimental/filesystem, as described here).
I also needed to bump the DEPLOYMENT_TARGET of iOS to 13.0 to use path, which is fine because almost all the iOS devices out there are at least on iOS 13.


Updated methods under resources

test

Codecov Report

Merging #1415 into master will decrease coverage by 0.56%.
The diff coverage is 73.37%.

@@             Coverage Diff              @@
##             master    #1415      +/-   ##
============================================
- Coverage     86.52%   85.96%   -0.57%
- Complexity     3980     4054      +74
============================================
  Files           387      407      +20
  Lines         16055    16597     +542
  Branches       1786     1795       +9
============================================
+ Hits          13892    14268     +376
- Misses         1686     1835     +149
- Partials        477      494      +17

Impacted Files
Coverage Δ
Complexity Δ

...oogle/api/codegen/util/java/JavaNameFormatter.java
68% <0%> (-9.28%)
13 <2> (ø)

...google/api/codegen/viewmodel/StaticMemberView.java
0% <0%> (ø)
0 <0> (?)

...om/google/api/codegen/DiscoGapicGeneratorTool.java
0% <0%> (ø)
0 <0> (?)

...m/google/api/codegen/transformer/SurfaceNamer.java
78.91% <100%> (ø)
173 <2> (ø)
:arrow_down:

...ain/java/com/google/api/codegen/util/TypeName.java
100% <100%> (ø)
15 <1> (+1)
:arrow_up:

...om/google/api/codegen/util/java/JavaTypeTable.java
97.46% <100%> (+0.28%)
30 <2> (+3)
:arrow_up:

.../java/com/google/api/codegen/discovery/Schema.java
89.53% <100%> (+1.29%)
19 <2> (+3)
:arrow_up:

...i/codegen/transformer/SchemaTypeNameConverter.java
100% <100%> (ø)
0 <0> (?)

.../java/com/google/api/codegen/discovery/Method.java
93.18% <100%> (+0.15%)
13 <0> (ø)
:arrow_down:

...ava/com/google/api/codegen/discovery/Document.java
94.64% <100%> (ø)
12 <0> (ø)
:arrow_down:

... and 88 more

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 74f4b6b...9e444e8. Read the comment docs.


Update components.md

Fixed a logical issue on an input.

Thank you but the original code is correct. v-model expects custom component emit the whole value to its parent.

Thank you but the original code is correct. v-model expects custom component emit the whole value to its parent.

No problem, the issue is where you pass the whole event to the value prop which doesn't work correctly(object passed as a value).

The child component should do $emit('input', value) where value is string so the received $event as a string.

Aha, You are right.


Multiplication of Tendermint processes

Version: rc61 + rc60
Multiple users have reported that when they leave the app on for extended periods of time, it continuously spawns tendermint processes. This ends up taking up a significant and problematic amount of CPU resources.

When opening Pearl after the app being not running and subsequently starting the agent this is what I see

If leave Pearl running, close the lid of the laptop, and re-open it after a few mins, nothing changes. I've repeated the process 3 times.
If I pause the agent using Pearl's UI, and re-start it, a new tendermint process appears:

If I repeat the latter, new tendermint processes appear

cc @jmoreira-valory

On top of that, closing Pearl does not kill the process aea_bin
Closing subsequently killing the process aea_bin doesn't kill the tendermint processes.
Finally I can stop the tendermint processes by killing them directly.

I've also noted that when I close the app, several tendermint processes remain running.

Same for aea_bin

Different behavoiur observed: Pearl running in the background, laptop lid closed for 5-6hs, when opening laptop back lots of tendermint processes running. This time I did not pause / restart agent.

@solarw can you reference your branch with the fix here?

Potentially solved by: https://github.com/valory-xyz/olas-operate-app/pull/216

I also tested leaving the app and agent running whilst I put my Mac to sleep – closed the lid.
When I re-opened the computer, the processes remain consistent.


Brute Force - Rainbow Tables

Hey Guy,
since brute force is supported for finding file names, it would also be cool if you could use rainbow tables as an alternative.
Is this helpful in such a scenario or is it only useful for calculating passwords?
https://de.wikipedia.org/wiki/Rainbow_Table

I've debated this many times but so far I've decided not to implement it. Theoretically it could be helpful however there are complications. on the Wikipedia article you referenced it mentions that rainbow tables generally only work for 14 length passwords maximum. This is because the computational power & storage requirements increase exponentially with each new character. Passwords are usually short because a user needs to type them.
Files in WC3 maps have a folder path which is part of the hash. A common folder name is war3mapimported, which is already 15 characters before we even add the file name & extension. But some files even have multiple subfolders and not all maps use standard folder names.
There is already a file included in the program called listfile.txt, it has a similar purpose to a rainbow table, but it's not quite the same. Instead of including every possible file name, it includes every real file name that I've found when deprotecting maps. I think it's a good compromise.
Theoretically the "Live Game Scanner" should be able to find the remaining unknown files. It should make all other forms of scanning unnecessary. Anything still not found is probably not actually used and can be deleted. So, I'm not sure the rainbow tables (or even brute force) would provide enough value to be worth the effort.
I'm leaving this issue open for now but I consider it low priority so i won't be working on it any time soon. Feel free to work on this yourself if you don't want to wait for me to have time to do it.

I've decided to remove brute forcing from the UI all together, it's rarely useful and confuses most users. I've added instructions to the tool on how to manually resolve the "unknowns". I'll continue improving the "unknown" discovery if I can in the future.


Compatible with RGB images?

Hi, thanks for the repo! I was wondering if this package is compatible with a generic RGB dataset with labels for each image. Something like a CIFAR10 for example.

Ahh thanks, I will try it.
Another point, what is the function of hard_negatives and steps_per_block? I was not able to understand these two parameters.

I am going a bit by memory since I haven't worked with this for some time, but:

hard_negatives is used to select the more difficult (hard) samples which are not correctly classified by the linear classifier (it was mentioned in Hinton's paper)
steps_per_block is something I introduced which allows to sequentially freeze blocks in the network (e.g. train all network for N steps, then freeze 1st block and train the N-1 blocks, then freeze 2nd block and train the remaining and so on..). The idea I guess was to stabilize the input of the following block in order to let them converge, but I'm not sure it had any advantages over just training the whole network at the same time


Scale more than one deployment per HTTPScaledObject

First of all, a big thank you to the KEDA community: This is a great tool that has finally allowed me to scale all unused workloads down to zero and easily scale up on demand.
After implementing KEDA on several clusters, there is one feature I particularly miss: Since I use Helm, most of my projects consist of multiple deployments. When a project is inactive, I would like to scale all these deployments down and back up simultaneously, based on an HTTPScaledObject.
Right now my only option would be to consolidate all deployments into one, which is unrealistic, or I'd have to create more ScaledObjects for every interconnection that might never be inactive due to keepalive or metric traffic and so on.
However, this level of complexity could be avoided if I could just provide an array of deployments like so:
apiVersion: http.keda.sh/v1alpha1
kind: HTTPScaledObject
metadata:
  name: myScaledObject
spec:
  host: myhost.example.com
  targetPendingRequests: 1
  scaleTargetRef:
    deployments:
    - deploymentOne
    - deploymentTwo
    - deploymentThree
    service: myMainWebService
    port: 80
  replicas:
    min: 0
    max: 1

Or is this possible already and I was just unable to figure out, how?
Thanks in advance - Max

Hello!
In KEDA (in core project) we decided to not support this feature because the HPA management would be more complicated, each scaled target has to deploy its own ScaledObject.
Based on it,  the add-on would need to spawn multiple ScaledObjects per HTTPScaledObject which could introduce complex management because the relationship isn't 1 to 1.
Could you explain a bit the scenario where you want to scale multiple deployments based on the same request? I mean, the service will randomly call to all of them, so you don't have any control about the requested deployment.
I think that you could achieve the expected behavior using an HTTPScaledObject for 1 deployment and a ScaledObject for each other deployment, using kubernetes workload scaler. I mean, you could scale from 0 to 1 the deploymentA using the HTTPScaledObject, and create a ScaledObject for each other deployment replicating the amount of replicas of deploymentA.
Could this solve your requirement? (or at least help?)


Impossible to center step card when using TourMatMenuModule

Since all the steps must have an anchor point, there doesn't seem to be a way to center tour step card in the middle of the screen.

That's true.

Imho, it's rather important to have an option to skip the anchor point and just center the step card.

I'll gladly review a PR.

I could try to take a look at this issue if active item highlighting issue is solved.

I've looked into this one and it's actually more complicated than I thought. Angular Material project positions its menu using cdk overlay (see _setPosition function) which we can not access because it's a private field. Manually setting CSS styling to reposition the menu as a dirty workaround causes loss of animations and possibly other issues.
Anyway, I found a simple workaround which is ok for now: create an extra invisible "div" with fixed position and attach the first step to it:
.centered-tour-element {
    position: fixed;
    left: 50%;
    top: 50%;
    /* minus half of step element's width and height */
    transform: translate(-140px, -86px);
}

Feel free to close this one since there is a simple workaround and nobody is going to add this functionality to the library.


Handling two databases

Problem

There is no proper documentation on how to handle two databases with prisma
Suggested solution

The steps should be added to the documentation.
Alternatives

Additional context

Hey @akinyemi2161, could you share a bit more context on this question?
For example, are you talking about two separate DBs with the same schema or different schemas?


[$500] MEDIUM: Enable clicking on message text to open a thread directly from the message

Slack thread for context - https://expensify.slack.com/archives/C066HJM2CAZ/p1707432519872559
Problem:
Navigating through threads in our platform can feel counterintuitive and inefficient due to the join button's placement to enter a thread. The current design requires unnecessary movement across the screen, disrupting the user's reading flow and reducing the overall user experience.
Solution:
Introduce a feature that allows users to click directly on a message text to open or start a thread on the message. This update will reduce the need to move the cursor across the screen to start a thread.
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~015c32d4724eb60146
Upwork Job ID:<PHONE_NUMBER>592702464
Last Price Increase: 2024-03-06

@parasharrajat that's fair. Let me chat with the team more internally about this and we can revisit. I am going to close for now.


Update to 0.5

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Solves https://github.com/openmm/openmm-torch/issues/54

@peastman @jaimergp this is ready!

I don't have the permission to merge. So, I will apply to become a maintainer.

https://github.com/conda-forge/openmm-torch-feedstock/pull/14


Uploading image after blur validation

I have a FileUpload component that is connected through a redux-form Field. It calls input.onChange and input.onBlur with the selected file as a base64 string when a file is selected in an input field.
I'm using the asyncValidator redux-form option to validate the dimensions of the image, and I would like the file to be uploaded to the my server after the async validation has finished.
There doesn't seem to be any sort of afterAsyncValidation hook documented. What is the best way to accomplish this in redux-form?

Because this issue saw almost no activity for almost a year, I'm closing it.


Chat.java  displayName is null

sync history chat, I log.info("chat: {}", JsonUtils.writeValueAsString(chat))  found, "displayName": null.
Can I get displayName of a chat, because chat.name is still a phone number.

use name, not displayName

Where can I get a contact's nickname？ Because chat.name is a phone number, no nickname.


Update README.md

Adding nuget install command for copy/paste convenience for those whom prefer the command line installation

Current coverage is 93.34%

Merging #2117 into master will decrease coverage by -0.01% as of 1e0dafe

@@            master   #2117   diff @@
======================================
  Files          566     566
  Stmts        35946   35946
  Branches      2315    2315
  Methods
======================================
- Hit          33557   33554     -3
  Partial        761     761
- Missed        1628    1631     +3

Review entire Coverage Diff as of 1e0dafe

Powered by Codecov. Updated on successful CI builds.

:+1:

:+1:


Makei Build Error: Cannot decomposite filename: OPM.CLP

Hi, i was trying to follow the bob getting-started on PUB400 but i get this error with makei build
Any idea?

Tried changing OPM.CLP to OPM.CLLE and Rules.mk from OPM.CLP to OPM.PGM.CLLE
Now i get this error

maybe @edmundreinhardt  or @worksofliam  can give me a hand

Well, i almost got it to work deleting these directories and removing them from the ./Rules.mk QBNDSRC QCBLSRC QCLSRC QCMDSRC QDTASRC

Not something im proud of

OK we FINALLY have released 2.4.27 which will fix this.
Note you may have to upgrade your python
Please ReOpen if you still see a problem.


Failed to extract BoxedVariable before return result

Fix #705. Root cause of problem:
C# lock statement use temporary variable inside.
During transformation pass EliminateTemporaries, this temporary variable removed inside EliminateSingleUseTemporaries.EliminateVariable with next code:
var replacer = new VariableEliminator(
                    variable,
                    JSChangeTypeExpression.New(replaceWith, variable.GetActualType(TypeSystem), TypeSystem)
                );

Than, nobody replace JSChangeTypeExpression with any other expression and JavascriptAstEmitter simple write inner expression, ignoring type changing:
public void VisitNode (JSChangeTypeExpression cte) {
            Visit(cte.Expression);
        }

I'm not sure, that I've fixed problem in best place - just added creation of JSReadThroughReferenceExpression inside JSChangeTypeExpression.New function. Maybe it should be fixed in another way.

I think the fix is probably to never eliminate ref temporaries... but I thought I did that already. I'll have to look into this, thanks.

Oh I see, the cast expression has to be a read-through-reference, that's probably right. Good catch.

I've been thinking that JSIL's AST probably needs to explicitly represent lvalues and rvalues as categories, now that there are a few types (references, pointers, etc.)
This is a good reason to consider that more.


Fixed KeyTransform preprocess_lhs crash with lhs_only=True on Oracle.

The key_transforms variable is not declared if lhs_only is True.
Honestly, looking at it again, maybe we should remove lhs_only altogether. It's only there for micro-optimization by not creating and returning the key_transforms, but I think the performance difference is negligible. We can simplify the implementation by just always returning the key_transforms and changing this line to:
diff --git a/django/db/models/fields/json.py b/django/db/models/fields/json.py
index 340b7ee875..4139b30384 100644
--- a/django/db/models/fields/json.py
+++ b/django/db/models/fields/json.py
@@ -407,7 +407,7 @@ class KeyTransformExact(JSONExact):
         if connection.vendor == 'sqlite':
             rhs, rhs_params = super().process_rhs(compiler, connection)
             if rhs == '%s' and rhs_params == ['null']:
-                lhs, _ = self.lhs.preprocess_lhs(compiler, connection, lhs_only=True)
+                lhs, *_ = self.lhs.preprocess_lhs(compiler, connection)
                 lhs = 'JSON_TYPE(%s, %%s)' % lhs
         return lhs, lhs_params

@laymonage Thanks for this patch, however all bugfixes require an accepted ticket and a regression test. I'm not sure how this can crash as we use lhs_only only on SQLite :thinking:

Removing lhs_only as a cleanup sounds good.

@laymonage Thanks for this patch, however all bugfixes require an accepted ticket and a regression test. I'm not sure how this can crash as we use lhs_only only on SQLite

@felixxm Yeah, I know that we do not use it on other databases, so this crash won't really happen unless it is explicitly used in other contexts. However, the cause of this crash (i.e. NameError) is trivial because it's not even related to the database, just an overlooked undeclared variable reference. Even though it's not documented, the method works fine if the NameError is resolved and it's just good practice in general, so I don't see why we shouldn't fix it.
I can write a ticket and test if we can proceed with this.

Even though it's not documented, the method works fine if the NameError is resolved and it's just good practice in general, so I don't see why we shouldn't fix it.

I wouldn't say that extra checks for unreachable scenarios, added just in case, is really a good practice.

I can write a ticket and test if we can proceed with this.

I would rather remove lhs_only as a cleanup.

I wouldn't say that extra checks for unreachable scenarios, added just in case, is really a good practice.

Well, the extra check can be replaced by declaring key_transform = [] at the start of the method. I was referring to avoiding the NameError in general, not in adding an extra check :slightly_smiling_face:

I would rather remove lhs_only as a cleanup.

Yes, I also thought so in the PR's description. Let me change the patch in a moment.

buildbot, test on oracle.

buildbot, test on oracle.


Separate HTTP server and Colyseus logic

Hello!
Is there a way to specify the HTTP server after configuring the game server logic?
With Socket.IO we can use the attach method:
var io = require('socket.io')();

// Then, in bin/www
io.attach(http);

If there isn't, could anyone guide me to what should be changed? I noticed the HTTP server is used to set the onConnect callback right in the constructor of the Server. Would changing this.server to a new WebSocketServer instance and adding this.onConnect as a callback to this new instance be enough?
Thanks!

Hey @charlestati,
Out of curiosity, what's your use case? Can't you specify the http server (server option) or a websocket server (ws option) in the constructor?
As socket.io has this option, I'll try to follow it. This evening I'll release a new version having it. Cheers!

Hi @endel!
Thank you very much!
I can't specify the server in the constructor as it is defined in another file. See this Express example:
https://github.com/sequelize/express-example/blob/master/bin/www
It requires the app in /bin/www and binds it to the HTTP server, which means I could have multiple www scripts with different ports or config for example.

Hey @charlestati!
I've just implemented the attach method. It accepts the same argument as the constructor. Now the constructor argument is optional.
Hope this helps! Please don't hesitate to ask questions here or in the Gitter channel if you happen to have some. Cheers!

That's awesome! Thank you @endel!


Setting Up Ragnarok DLC Instance

I'm getting this looks like the server started but not really cause I can't join at all can you please tell me how to setup the new DLC?

Here is my Instance CFG:
arkbackupdir="/mnt/Backups/Ark/Ragnarok"
arkserverroot="/home/mc/ARK/TheIsland"
serverMap="Ragnarok"
#serverMapModId="473850"
#ark_TotalConversionMod="496735411"
ark_RCONEnabled="True"
ark_RCONPort="32332"
ark_SessionName="TafariCraft - Ragnarok"
ark_Port="7783"
ark_QueryPort="27018"
ark_ServerPassword=""
ark_ServerAdminPassword="12345"
ark_MaxPlayers="25"
ark_bRawSockets="True"
arkflag_UseBattlEye="True"
arkflag_servergamelog="True"
arkflag_log="True"
arkflag_RCONServerGameLogBuffer="600"
#ark_GameModIds="487516323,487516324,487516325"
ark_AltSaveDirectoryName="Saved-Ragnarok"
arkflag_NoTransferFromFiltering=true
arkopt_clusterid=Cluster1
#arkflag_OnlyAdminRejoinAsSpectator=true
#arkflag_DisableDeathSpectator=true
#arkopt_StructureDestructionTag=DestroySwampSnowStructures

Found the issue my Rcon port was the same for another instance just didn't catch it but got the idea from reading other troubleshooting issue on here knew I would find it. So just make sure you don't over lap ports or you will get the issue I got or something pretty close.


CI and flake.lock

This PR:

Provides 2 new CI workflows

One to keep the file flake.lock up to date, the CI will propose a PR every monday

Update flake.lock manually (nix flake update)

To test this quickly: nix develop github:drupol/typst/update-flake-lock

CC @dccsillag

I'm not a big fan of merging a PR for this every week.

Ok, closing it?

Copy pasting here a discussion I had on Discord, I feel this is relevant to this PR:
reproducibility in software is important to ensure consistent results, facilitate collaboration, and improve reliability. Nix helps in reproducibility by providing a declarative package management system that ensures the same dependencies and configurations are used across environments, thus minimizing inconsistencies and enhancing stability.
Right now, in Typst, providing a flake.nix file provides the necessary dependencies to build typst. These dependencies also have a version number and those are fixed in flake.lock.
When you think more about it, providing a binary program to the world is not only shipping the binary itself. The binary is some kind of product of all the dependencies required to build it. Therefore, it is important to make sure that the dependencies we use are fixed, stable and working.
Incorporating a flake.nix file directly within the Typst repository enables immediate building of a development version from the current sources. Outsourcing this task would necessitate continuous monitoring of the Typst repo through a CI system to update the flake.lock file, which is both energy-intensive and environmentally unfriendly. Additionally, a flake.nix file maintained by the author carries greater legitimacy compared to one contributed by a third party.
That said, having the flake.nix file in the repo, as opposed to being maintained by someone with offers several advantages:

Author's expertise: The author of the software has an in-depth understanding of the project's dependencies and configurations. Their expertise ensures that the flake.nix file is up-to-date and accurately reflects the project's requirements.
Faster updates: Including the flake.nix file within the repo allows for seamless integration with the development process (very cool to have!!), resulting in quicker updates and bug fixes. Outsourcing it to another party may introduce delays due to communication and coordination.
Greater trust: As I said before, users may trust a flake.nix file provided by the author more than one maintained by a third party, as the author is the primary authority on the software.
Easier collaboration: Developers contributing to the project can directly access and modify the flake.nix file as needed, streamlining collaboration and ensuring consistency across the potential team and contributors.
Wider adoption: Hosting the flake.nix file within the repo encourages users to adopt Nix, as it simplifies the setup process and promotes reproducible builds. This is one of the most important concepts.

While someone from the Nix community may have a deeper understanding of Nix itself, having the flake.nix file in the repo strikes a balance between Nix expertise and project-specific knowledge, leading to a more efficient and trustworthy solution.


Create LICENSE for CIS4Azure-Pester

This is the MIT License, meaning that we can allow other to use this for personal and commercial purposes. More here: https://choosealicense.com/licenses/mit/

This looks great.

Great!


Tablib released at 2020.01.13 and broke requirements

Need to fix version of tablib in setup.py

Which version of import-export?

It happened to me also, version 1.2.0

I managed to solve the issue by locking tablib to version 0.14.0

i have version 2.0

I managed to solve the issue by locking tablib to version 0.14.0

same solution

tablib dependencies have become optional in the latest release. install them all using pip install tablib[all].

tablib dependencies have become optional in the latest release. install them all using pip install tablib[all].

Actually while this indeed installs all required dependencies, the dropdown "format" list in django admin is no longer populated correctly. It seems it was dynamically fetched and this no longer works with Tablib 1.0 (latest as of today)

tablib dependencies have become optional in the latest release. install them all using pip install tablib[all].

Actually while this indeed installs all required dependencies, the dropdown "format" list in django admin is no longer populated correctly. It seems it was dynamically fetched and this no longer works with Tablib 1.0 (latest as of today)

You are right, I noticed it just now. I locked tablib to version 0.14.0 for now.

@andrewgy8 last version of django-import-export

Actually while this indeed installs all required dependencies, the dropdown "format" list in django admin is no longer populated correctly. It seems it was dynamically fetched and this no longer works with Tablib 1.0 (latest as of today)

@tamers, I wonder what is making that behaviour work, if it's not due to a dependency, it could be a change in tablib. If you figure it out, please raise an issue with https://github.com/jazzband/tablib/

I made this PR to not use major version, I believe this is not the definitive solution, but can help a little. This PR is more appropriate for definitive solution.


JOINDIN-594: Full description for pending events

Fixing event description being truncated on pending events list

Can one of the admins verify this patch?

ok to test


Treat all SQL exceptions the same in DatabaseErrorPageMiddlewareTest

@AndriySvyryd,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@Eilon should this go in preview1? Test stabilization and all that.

Have we seen this test fail recently? If so, I'm fine porting this to preview1. @ajcvickers thoughts?

Ah just saw the related bug https://github.com/aspnet/Diagnostics/issues/366#issuecomment-297567879
Definitely looks worth porting to me.

@Eilon @AndriySvyryd Looks like we should port it.

Done


Failed to configure cluster on Azure

Environment info

Version: 2.1.2-52e964c True for all versions
Deployment: **AZURE **
Customer: demo2.0

Clean system http://<IP_ADDRESS>:8080/fe/systems/clean/cluster/servers/noobaa-93874eab
Actual behavior

NTP setup is required. There is no way to configure NTP without noobaaroot.

Expected behavior

Need to think about it. Maybe automatically configure the new server with the same NTP configuration instead of asking for pre-configuration @shirimordechay?

No longer relevant due to OVA deployment support removal


Use of uberon classes for generic anatomical structures means taxon restrictions are too strict

From
https://github.com/geneontology/go-site/issues/403
[Term]
id: GO:0048856
name: anatomical structure development
namespace: biological_process
def: "The biological process whose specific outcome is the progression of an anatomical structure from an initial condition to its mature state. This process begins with the formation of the structure and ends with the mature structure, whatever form that may be including its natural destruction. An anatomical structure is any biological entity that occupies space and is distinguished from its surroundings. Anatomical structures can be macroscopic such as a carpel, or microscopic such as an acrosome." [GO_REF:0000021, GOC:mtg_15jun06]
comment: This term was added by GO_REF:0000021.
subset: goslim_chembl
subset: goslim_generic
synonym: "development of an anatomical structure" EXACT []
is_a: GO:0032502 ! developmental process
intersection_of: GO:0032502 ! developmental process
intersection_of: results_in_development_of UBERON:0000061 ! anatomical structure

[Term]
id: UBERON:0000061
name: anatomical structure
relationship: existence_starts_during_or_after UBERON:0000106 ! zygote stage

this introduces an existential dependence between ASs and zygotes, causing problems for use of terms like cell morphogenesis in yeast
We should either

[ ] weaken uberon to be inclusive of non-metazoa
[x] use CARO in the GO definition for very generic structures

Chris will work out importing the CARO term.

interesting, this is revealing previously cryptic inconsistencies in GO:

This was not revealed before; we already had an axiom classifying CL cell under CARO anatomical structure, but because we weren't using CARO in GO there was no linkage and hence this problem went unnoticed.
Good choice to go with CARO everyone!
I'll remove the disjoints in this PR
cc @dosumis

Ahh, we even annotated this disjointness axiom with a comment:
   <rdfs:comment>We examined all the child terms of 'anatomical structure maturation' and found them all to be gross anatomical structures, so we are safe in declaring this to be disjoint from cell maturation. We may rename 'anatomical structure' to 'gross anatomical structure' in future (GOC:mtg_OBO2OWL_2013)</rdfs:comment>

It seems we have reverted from this, and now consider 'anatomical structure P' in GO to be consistently about generic anatomical structures.
We could bring in a gross AS later but not clear if this worth it

I have done my bit
See the linked PR. Merging this will result in a few new classifications under 'anatomical structure maturation', since cells are now anatomical structures

I think I found another problem.  'zygote asymmetric cell division' is linked to 'animal zygote' (CL term for zygote) but has a child 'zygote asymmetric cell division in embryo sac'.  The latter is a plant term.
http://amigo.geneontology.org/amigo/term/GO:0010070#display-lineage-tab
Is there a CARO term for 'zygote'? @cmungall, how to fix this one?

Arose from this: https://github.com/obophenotype/cell-ontology/issues/454
"We will not add a new generic class now, though we can explore this in future if a use case arises (note we would need an inter-ontology subclass with PO)"
guess we have the use case now


Add an optimized version of Dockerfile.mainnet

Reduce disk usage: image size reduced by ~9x (Dockerfile.mainnet 2.82G ->  Dockerfile.mainnet.optimized 323MB).
Speed up compiling: append every "make" command with "-j4".
Make tar extraction commands more concise.
Parameterize dependency filename or version. Easier to modify for future releases.

Other notable changes:

Make rocksdb installed to /usr/local/lib
No more separate SmartBCH building directory(/smart_bch). Changed to /build same as other libs.


I can't open the 64-bit PowerShell as default in my win10

my user setting:
{
"terminal": "C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe",
"parameters": []
}
when I open the terminal ,it show the result:

while I want to use the BashOnWindows,only I can use the 64-bit PowerShell
how can i fix the problem?

This is likely Sublime Text is launching without the proper environment and the subprocess isn't receiving that. Is bash normally on your %PATH% environment variable?

what't does is mean  bash is  normally on your %PATH% environment variable?
I can run the bash in the powershell normally.

The %PATH% environment variable is how Windows normally resolves programs (e.g. if there's a folder with bash.exe in the %PATH%, then Windows will find/run bash.exe from that folder). Can you type echo "%PATH%" in a terminal launched outside of Sublime Text and one launched via Sublime Text and paste the results here?

@twolfson

Hmm, maybe double quotes isn't the proper syntax on Windows. It should look like C:\...;C:\.... Can you try again with echo '%PATH%' and echo %PATH%?

@twolfson

Ah, right -- PowerShell syntax. Hmm, those both have the same path. And you are sure bash.exe can't be found in which Window?

@twolfson I have added  the folder where the bash.exe exist.

Ah, so it sounds like this problem has been resolved then. Closing this ticket.

@twolfson the problem isn't  solved.
at first,I have add the folder in  $path .

@twolfson

Ah, sorry. It sounded like it was =/

@twolfson yes,it can work in an non Sublime Text launched PowerShell

@twolfson and it's not the / or \

@twolfson

I try to open the bash on the same folder but with diffrent PowerShell process.

@twolfson and I try to modify the sublime_terminal user setting:
{
"terminal": "C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe",
"parameters": []
}
to
{
"terminal": "powershell.exe",
"parameters": []
}
it happen again.

Can you try the following? We have a custom batch script for PowerShell:
"terminal": "PS.bat"

@twolfson

I don't understand what you are attempting to convey with that screenshot =/

@twolfson
sorry,the Image Hosting have some problem.

@czmecho There shouldn't be a \ before PS.ps1. Did you try and execute PS.ps1 from a powershell window started via the start menu?

@wbond

@wbond

Did you execute PS.bat by double-clicking in explorer? That won't work. The Terminal package sets an environmental variable to ensure the path to PS.ps1 is fully-qualified.

I　run the powershell as Administrator directly from C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe
it show nothing.
when I change to D:\Documents\soft\Sublime Text\Data\Packages\Terminal
it show the result:

I think it's because it can't load the PS.ps1

That error is because you didn't quote a file path with a space in it.

@wbond sorry.
in the powershell I post above,I run the command :set-executionpolicy -executionpolicy unrestricted
and now I can call the bash on powershell launched via Sublime Text.
thanks.


Add tests

Tests for injectables are tricky.
I was thinking that something like e2e testing done in web browsers (mostly using Chromium) could be done. A NosSmooth.TestSuite would have to be developed that would support opening NosTale, logging in, selecting server, character. Then some kind of "state initialization" would be needed - ie. get the character on given map, to given scenario...
If that kind of test suite was done, testing anything bots should be possible, but will probably be painful.


fix IPV6_TRANSPARENT define in DEBIAN

Commit Message: fix https://github.com/envoyproxy/envoy/issues/12963 , build failed in DEBIAN
Risk Level: Low
Testing: build test

@chadr123 Mind giving this a look?


On ingress updates always reconcile globalDNSs of the cluster

In case Ingress's annotation for globalDNS is changed or deleted, controller get an update event but have no information as to what was the original annotation. Thus there is no way to reconcile the endpoints for the original GlobalDNS entry.
Hence we now invoke reconcile of all GlobalDNSs for the cluster to which ingress belongs, on any ingress update.
https://github.com/rancher/rancher/issues/17831

@alena1108 @kinarashah can you please review

LGTM


Can't nevigate between inputs with tab key

Hello, I tried to create simpale form like the example , the validation and all works perfect but I can't nevigate through the inputs by using the tab key.
any suggestion why ?

hi, have you supply the tabIndex as a prop to the "Textbox" ?
Like this
<Textbox
  tabIndex="1" //Optional.[String or Number].Default: -1.
/>

<Textbox
  tabIndex="2" //Optional.[String or Number].Default: -1.
/>

<Textbox
  tabIndex="3" //Optional.[String or Number].Default: -1.
/>

Wow totally missed the tabIndex !
Thank you!


Support for Triggers and Targets

This works with the current exception of Zendesk.getTargets(). The zendesk server response is currently inconsistent with it’s other list responses, and PagedIterator currently relies on this behaviour for loop termination.

@artichikin nice work on implementing triggers and targets, do you have time to fix the merge conflicts so @stephenc can merge your PR?

I'm going to do a new pull request with our latest changes, merged up

@artichikin any luck with the new pull request?

They merged our new one into master on Feb 10th.

On Feb 20, 2015, at 1:53 PM, Johno Crawford<EMAIL_ADDRESS>wrote:
st


tauri.js CLI: Errors from the Rust CLI don't propagate up to the JS CLI

Describe the bug
It seems like errors reported by the Rust CLI do not propagate to the JS CLI. In other words, the Rust CLI exits with a non-zero exit code, yet the JS CLI does not. Since the JS CLI returns an exit code of 0, the command is considered to have passed. This poses a problem for our own smoke tests in particular -- in the event that this occurs, GitHub Actions will mark the step as "passed" even though it actually should fail.
Examples where this has occurred:
https://github.com/tauri-apps/tauri/pull/187/checks?check_run_id=357905199#step:11:112
https://github.com/tauri-apps/tauri/pull/214/checks?check_run_id=363576771#step:14:130
To Reproduce
Steps to reproduce the behavior:

Run a command from the JS CLI (e.g. yarn tauri build) which causes an error to be thrown from the Rust CLI

Expected behavior
Non-zero error codes from the Rust CLI should result in a non-zero error code from the JS CLI
Platform (please complete the following information):
This doesn't appear to be a platform-specific issue

Ah looks like this was previously reported in #202 . I'm investigating right now to see what's causing the issue

I already know what the problem is.  When an error is reported from Rust it gets obscured by the Node runtime.  If you go into the bundler and try to use println! to print a message you'll notice that it won't make it through the I/O.  Best way to get around this is with the stdout pipe and some of the functions from the common.rs module.  I'll look tomorrow, its really just a matter of adding a piece of code to invoke the error logger when an error propagates.


KNL-1320 TIKA, POI ...

https://jira.sakaiproject.org/browse/KNL-1320
Security Issue

So the POI explicit dependencies are just to avoid the beta dependencies that Tika pulls in?  I guess I would prefer leaving out the POI explicit dependencies and just using what Tika depends on.

tika - 1.
On 2 January 2015 at 20:40, Sam Ottenhoff<EMAIL_ADDRESS>wrote:

So the POI explicit dependencies are just to avoid the beta dependencies
that Tika pulls in? I guess I would prefer leaving out the POI explicit
dependencies and just using what Tika depends on.
—
Reply to this email directly or view it on GitHub
https://github.com/sakaiproject/sakai/pull/8#issuecomment-68550219.

poi [?]
Tika [?]?
On 2 January 2015 at 20:40, Sam Ottenhoff<EMAIL_ADDRESS>wrote:

So the POI explicit dependencies are just to avoid the beta dependencies
that Tika pulls in? I guess I would prefer leaving out the POI explicit
dependencies and just using what Tika depends on.
—
Reply to this email directly or view it on GitHub
https://github.com/sakaiproject/sakai/pull/8#issuecomment-68550219.


Jenkins Plugin Does Not Upload From Remote Build Slave

I am using the artifactory jenkins plugin - all items are the latest and greatest as of 4/14/2018
Declarative pipeline; building on a remote node
Everything works great and packages are created/exported
When it comes time to upload however, using the examples for how to do the upload fail.
No packages found. Is the error emitted.
Here is the basic pattern suggested from the docs. Note: I have tried NUMEROUS changes and combinations of this example (and I have it in declarative syntax). So lets not focus on that stuff - if possible.
The pattern suggested is something like (I have tried dozens of combinations etc) :
def artifactory_name = ...
def artifactory_repo = ...
node {
def server = Artifactory.server artifactory_name
def client = Artifactory.newConanClient()
def serverName = client.remote.add server: server, repo: artifactory_repo
stage("Get recipe"){
    git branch: repo_branch, url: repo_url
}

...
stage("Upload packages"){
    String command = "upload * --all -r ${serverName} --confirm"
    def b = client.run(command: command)
    server.publishBuildInfo b
}

}
I think the problem is that, the client.run (command: ...  Does NOT run on the build slave.
Rather it runs on my jenkins/artifactory server.
And so there are in fact no packages - and there shouldn't be.
The question:
How would I get the run command (upload) to execute on the build slave ?
(I actually did a good bit of searching/looking; so this could easily be my error, but it is not knee jerk)
Ubuntu 17.10
Jenkins 2.107.1
Plugin 2.15.1
Artifactory 5.10.1  (the community c++ distribution if that matters)

Thanks for the info/contact.
Right now I am proceeding from the notion that I may have messed something
up - not unusual.
If I can be sure it a problem, I'll open an issue
On Tue, Apr 17, 2018 at 4:17 AM, Luis Martinez de Bartolome Izquierdo <
<EMAIL_ADDRESS>wrote:

Closed #179 https://github.com/conan-io/conan-package-tools/issues/179.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/conan-io/conan-package-tools/issues/179#event-1578164020,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AELdufItqTKzPkO4qg5z-9nnd0SMWFPYks5tpaUpgaJpZM4TXWZk
.


Update and rename 2024-12-13-secondentry.md to 2024-12-11-secondentry.md

back in time fix....Can't future post.

lost of commits understanding how this thing works


Enable Java lints for the plugins/packages repos

A linter runs for the Dart code and Objective-C/Swift code in the flutter/plugins and flutter/packages repos.
This issue tracks enabling it for Java and Kotlin.
We should be able to just use the command provided by gradle:
See https://developer.android.com/studio/write/lint
gradlew lint
./gradlew lint

I would imagine we would initially need a config file to exclude plugins and fix the lint warnings incrementally.
cc @camsim99 @reidbaker @stuartmorgan FYI

@stuartmorgan Is this something we could do before or after https://github.com/flutter/flutter/issues/113764?

I we do it before, we just need to make sure it's added in parallel to both repos so we don't re-diverge their test set.

I would imagine we would initially need a config file to exclude plugins and fix the lint warnings incrementally.

We already have lint_baseline.xml set up as part of uploading build warning output, so hopefully we could just update that.

Would this address https://github.com/flutter/flutter/issues/91868 for Android, or are lint warnings and build warnings disjoint?

I am pro making this happen.

We already have lint_baseline.xml set up as part of uploading build warning output

... which we have because I set up a command over a year ago that runs this lint command. I forgot that's what it was doing 🤦🏻
Closing as a duplicate of https://github.com/flutter/flutter/issues/87071. I'll look into why some warnings aren't being flagged by this, like https://github.com/flutter/packages/pull/3273. Maybe it's just the exclusions, and what we need is a push to remove exclusions that were set up when we turned this on?

Also, while not relevant to it not failing CI, it appears that running the lint command twice only outputs warnings the first time, which is definitely not a behavior we want.

It looks like having warnings output doesn't cause it to return a non-zero exit code. Re-opening while I investigate further.

Investigating further: I'm not crazy thinking I had to set up baselines to get things to pass when I first did this, I just wasn't looking at the output closely enough. The output I was seeing was:
> Task :camera_android:lintDebug
Lint found no errors or warnings

2 errors/warnings were listed in the baseline file (/Users/stuartmorgan/src/packages/packages/camera/camera_android/android/lint-baseline.xml) but not found in the project; perhaps they have been fixed?

Deprecated Gradle features were used in this build, making it incompatible with Gradle 8.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/7.0.2/userguide/command_line_interface.html#sec:command_line_warnings

BUILD SUCCESSFUL in 1s
52 actionable tasks: 52 executed
/Users/stuartmorgan/src/packages/packages/camera/camera_android/android/src/main/java/io/flutter/plugins/camera/features/resolution/ResolutionFeature.java:158: warning: [deprecation] get(int,int) in CamcorderProfile has been deprecated
          return CamcorderProfile.get(cameraId, CamcorderProfile.QUALITY_HIGH);
                                 ^
/Users/stuartmorgan/src/packages/packages/camera/camera_android/android/src/main/java/io/flutter/plugins/camera/features/resolution/ResolutionFeature.java:162: warning: [deprecation] get(int,int) in CamcorderProfile has been deprecated
          return CamcorderProfile.get(cameraId, CamcorderProfile.QUALITY_2160P);
                                 ^
/Users/stuartmorgan/src/packages/packages/camera/camera_android/android/src/main/java/io/flutter/plugins/camera/features/resolution/ResolutionFeature.java:166: warning: [deprecation] get(int,int) in CamcorderProfile has been deprecated
          return CamcorderProfile.get(cameraId, CamcorderProfile.QUALITY_1080P);
                                 ^
/Users/stuartmorgan/src/packages/packages/camera/camera_android/android/src/main/java/io/flutter/plugins/camera/features/resolution/ResolutionFeature.java:170: warning: [deprecation] get(int,int) in CamcorderProfile has been deprecated
          return CamcorderProfile.get(cameraId, CamcorderProfile.QUALITY_720P);
                                 ^
/Users/stuartmorgan/src/packages/packages/camera/camera_android/android/src/main/java/io/flutter/plugins/camera/features/resolution/ResolutionFeature.java:174: warning: [deprecation] get(int,int) in CamcorderProfile has been deprecated
          return CamcorderProfile.get(cameraId, CamcorderProfile.QUALITY_480P);
                                 ^
/Users/stuartmorgan/src/packages/packages/camera/camera_android/android/src/main/java/io/flutter/plugins/camera/features/resolution/ResolutionFeature.java:178: warning: [deprecation] get(int,int) in CamcorderProfile has been deprecated
          return CamcorderProfile.get(cameraId, CamcorderProfile.QUALITY_QVGA);
                                 ^
/Users/stuartmorgan/src/packages/packages/camera/camera_android/android/src/main/java/io/flutter/plugins/camera/features/resolution/ResolutionFeature.java:182: warning: [deprecation] get(int,int) in CamcorderProfile has been deprecated
          return CamcorderProfile.get(cameraId, CamcorderProfile.QUALITY_LOW);
                                 ^
7 warnings

I missed the Lint found no errors or warnings bit. So those warnings are being dumped to the console when I run lint as part of the overall gradle run, but aren't part of the lint step itself. That may also explain why it doesn't output anything when run a second time; it's not the step actually being targetted.

Okay, it looks like what's happening here is that the Gradle lint is different from javac lints, and the deprecation warnings are coming from the latter. So this is a duplicate, and the above would be https://github.com/flutter/flutter/issues/91868


Add Bulgarian language translations.

Hello. I added the Bulgarian language translations to NeoInfo.. 2nd commit was to fix a small inconsistency in a few of the words in the About page with my translations..

I updated it but seems like we cannot update the 0 line.


remove useless old six imports

https://codesearch.debian.net/search?q=s3transfer.compat&literal=1&page=1&perpkg=1

Hi @a-detiste, these are left in place for backwards compatibility. We don't intend to remove these at this time.
For the CLI v2 source, that is authored here. That's a more reasonable place these can be removed to help minimize the distro maintainer burden.

This is an horrible mess, I'll look at it again later


Fix the first example in the readme so it can be run

The example in the read me doesn't actually run until you add a return value for propagating io results.

Thanks for the catch! Signature might change with #367


SW-4904 Update mock with HTML, render HTML directly in the deliverable Metadata component. I am using dangerouslySetInnerHTML here which is generally frowned upon because it potentially creates XSS vectors, but this HTML is not from user input and can be considered sanitized. This may change in the future if we enable deliverable management in the admin UI, at which point we may want to implement a library like dompurify or guarantee that the input is sanitized in the BE, or both.

Current dependencies on/for this PR:

#2228  👈
main

This stack of pull requests is managed by Graphite.


[Feature/Idea]: Skip generating manifest if relating PR exists

Project Name
WinGet Automation
What would you like to see changed/added?
Is there a way to let the automation skip generating manifest and just change the version in the config file if there is already a relating PR for this change, so it won't override my commits?

Adding me as a collaborator is enough. 🙂

Done :)


[packages] oneflow depends on opennebula

Description
Oneflow depends on ActionManager gem which persists only in opennebula package
To Reproduce
# apt-get install opennebula-flow
# /usr/bin/ruby /usr/lib/one/oneflow/oneflow-server.rb
Traceback (most recent call last):
	7: from /usr/lib/one/oneflow/oneflow-server.rb:60:in `<main>'
	6: from /usr/share/one/gems-dist/gems/polyglot-0.3.5/lib/polyglot.rb:65:in `require'
	5: from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
	4: from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
	3: from /usr/lib/one/oneflow/lib/LifeCycleManager.rb:19:in `<top (required)>'
	2: from /usr/share/one/gems-dist/gems/polyglot-0.3.5/lib/polyglot.rb:65:in `require'
	1: from /usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require'
/usr/lib/ruby/2.7.0/rubygems/core_ext/kernel_require.rb:92:in `require': cannot load such file -- ActionManager (LoadError)

Expected behavior
Oneflow can work separate from opennebula
Details

Affected Component: packages / oneflow
Hypervisor: KVM
Version: 5.12

Additional context
Installing opennebula package solves that problem

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

PRs to merge in master:

packages
docs


Bootstrap fixed navbar issue

Hi Hakim,
I found an issue with Bootstraps fixed navbar. Let's assume you have the following structure:

      ...

    ...

</div

When using Kontext like this, the Navbar is fixed to top but won't scroll along with the page. It just stays at the top.
I found a solution for this though, you just need to comment out the following CSS:
.kontext.capable {
	-webkit-perspective: 1000px;
	   -moz-perspective: 1000px;
	        perspective: 1000px;

	-webkit-transform-style: preserve-3d;
	   -moz-transform-style: preserve-3d;
	        transform-style: preserve-3d;
}

.kontext.capable .layer {
	-webkit-transform: translateZ( -100px );
	   -moz-transform: translateZ( -100px );
	        transform: translateZ( -100px );
}

.kontext.capable .layer.show {
	-webkit-transform: translateZ( 0px );
	   -moz-transform: translateZ( 0px );
	        transform: translateZ( 0px );
}

I have no idea why the above code makes Bootstraps navbar behave like this. Hope you can help me with this.
EDIT: I've found a better solution by just overwriting those properties like this:
.kontext.capable {
  perspective: none;
  transform-style: flat;
}

.kontext.capable .layer { transform: none; }

.kontext.capable .layer.show { transform: none; }

This solves the navbar behavior before calling k.next(). After calling it the navbar is again fixed to the top of the page (not the top of the viewport like expected)

Alright, I found the solution. If you apply position: fixed; to childs of transformed/animated elements, the fixed element will be fixed to the parent element instead to the viewport.
source: https://code.google.com/p/chromium/issues/detail?id=20574


Template file error after upgrading from lime-3.7.4/openfl-4.8.1 to lime-6.3.1/openfl-8.1.0

Hi! We're working an quite a big application using OpenFl targeting Flash / AIR. It was still running on OpenFl 4.8.1 with lime 3.7.4 but since we like to use Electron instead of Adobe AIR we figured it might be a good idea to use the latest library versions for the latest html5 support.
haxelib run openfl build flash -debug works just finde with the old setup, but after upgrading to lime-6.3.1 and openfl-8.1.0 build fails with this output:
Warning: Could not find template file: haxe
Warning: Could not find template file: flash/hxml
Called from ? line 1
Called from CommandLineTools.hx line 2069
Called from CommandLineTools.hx line 26
Called from a C function
Called from CommandLineTools.hx line 147
Called from CommandLineTools.hx line 740
Called from lime/project/PlatformTarget.hx line 107
Called from lime/tools/platforms/FlashPlatform.hx line 223
Called from /usr/share/haxe/std/neko/_std/sys/io/File.hx line 30
Uncaught exception - [file_contents,Export/flash/haxe/debug.hxml]

I know it has something todo with the template files and their locations. With the old setup the autput path for debug was e.g. Export/flash/debug/haxe/debug.hxml and for final Export/final/haxe/final.hxml. So OpenFl generated two different folders.
Now OpenFl seems to expect Export/flash/haxe/debug.hxml but it doesn't exist and won't be generated. In fact, no haxe folder will be generated. Just an empty bin and obj/assets.swf.
I did look into the haxelib directory, but there is no templates folder any more (exists in openfl-4.8.1). Is flash output not supported anymore?
I guess I make an fundamental mistake?

Well, if I upgrade using haxelib set openfl 8.1.0 and haxelib set lime 6.3.1 I get the errors from above. If I then downgrade again - using haxelib set openf 4.8.1 and set lime 3.7.4 I still get the same errors. Uninstall the newer library version doesn't help. Uninstalling Haxe doesn't help either. Just uninstalling everything, restarting and the installing everything again (to the old versions) will make it work again.
Im on the latest macOS.

It it possible that there were some files leftover from the OpenFL 8 build? Have you tried a clean build after switching back to OpenFL 4.8.1?

I can compile OpenFL and Flixel sample projects to Flash just fine with OpenFL 8.1, so this is probably specific to your project in some way. Have you tried with one of the projects from openfl-samples?

I will! But not just yet. I need to get work done now - but I also think there is a specific issue (maybe regarding my build machine) and not a general problem. At least since you tried compiling to flash and it works for you. Are you on macOS?

No, Windows.

Hmm, this seems like a tell-tale sign:
Warning: Could not find template file: haxe
Warning: Could not find template file: flash/hxml

Perhaps there is an issue here with defines you may have, conflicting with the behavior. Is it possible your project file forces "-Dair"?
I would recommend using the new AIR target instead:
lime test air

I believe there is no issue here in the tools, but if you have any further issues with your specific project, please let me know. http://community.openfl.org is a great place to discuss if it isn't working


if i deploy cluster nodes with explicit role，how to set SW_AGENT_COLLECTOR_BACKEND_SERVICES on agent side?

OS：CentOS Linux release 7.7.1908 (Core)
Skywalking version：apache-skywalking-apm-es7-8.1.0
I have a three node cluster with the following roles:
node01：SW_ CORE_ ROLE:Receiver
node02：SW_ CORE_ ROLE:Receiver
node03：SW_ CORE_ ROLE:Aggregator
In addition to the role settings of the three nodes, do I need other settings?
How should I set SW_ AGENT_ COLLECTOR_ BACKEND_ SERVICES on agent side？
Please forgive my poor English.

We would recommend you to set a proxy for node01 and node02 as LB. Or you could add both node01 and node02 at the agent config.


Issue with using 8.4.0

Hi there ! I ran into this issue

I solved it by switching to 8.3.0 and thought perhaps it would be useful to share !

Yes! The firestore dependency here must match the one used by everything else.
Note in your Podfile I believe it is possible to specify the version dynamically so they are in sync naturally without maintenance effort. I believe you will like this snippet:
https://github.com/FirebaseExtended/flutterfire/blob/1795261e09a73080703af3609dd198758b4f3032/packages/cloud_firestore/cloud_firestore/example/ios/Podfile#L37-L51

I do like that snippet ! I appreciate it, thanks !!!

I can say that the '8.3.0' reference will require manual maintenance from you going forward, and the goal of the snippet I posted was to show how to use ruby (the language of Podfiles) to programmatically feel around your apps' directory tree to find the firebase_core ruby config script, and use it to get your firebase-ios-sdk version, then use that version in place of 8.3.0 so it's dynamic.
You'll have to experiment in order to make it work exactly, it always takes me a few test/run/fix tries when I'm playing with Podfiles personally but I always get there in the end. Good luck!

I agree with the intention and approach ! Just wanted to make sure I was following your guidance ! thanks


Migrate to authentication policy v1alpha2

New changes are detected. LGTM label has been removed.

The PR is complete now. Can I have new lgtm. Thanks.

[APPROVALNOTIFIER] This PR is APPROVED
Approval requirements bypassed by manually added approval.
This pull-request has been approved by: kyessenov
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [kyessenov]
tests/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

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

Test name
Commit
Details
Rerun command

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

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

@diemtvu After this change, I'm getting this error from the new pilot e2e tests:
I'm getting this error when running the new pilot e2e tests: 2018-03-21T21:48:23.944401Z	info Command error: exit status 1
authn_policy_test.go:28: command failed: "Error from server (BadRequest): error when creating \"testdata/v1alpha1/authn-policy.yaml.tmpl\": the API version in the data (authentication.istio.io/v1alpha2) does not match the expected API version (authentication.istio.io/v1alpha1)\nError from server (BadRequest): error when creating \"testdata/v1alpha1/authn-policy.yaml.tmpl\": the API version in the data (authentication.istio.io/v1alpha2) does not match the expected API version (authentication.istio.io/v1alpha1)\n" exit status 1

I've rebased onto master .... is there anything else I need to do?

It's not allowed to have two versions for the API group. Delete your old
CRD definition for v1alpha1.
On Wed, Mar 21, 2018 at 2:53 PM Nathan Mittler<EMAIL_ADDRESS>wrote:

@diemtvu https://github.com/diemtvu After this change, I'm getting this
error from the new pilot e2e tests
https://github.com/istio/istio/pull/4296:
I'm getting this error when running the new pilot e2e tests: 2018-03-21T21:48:23.944401Z	info Command error: exit status 1
authn_policy_test.go:28: command failed: "Error from server (BadRequest): error when creating "testdata/v1alpha1/authn-policy.yaml.tmpl": the API version in the data (authentication.istio.io/v1alpha2) does not match the expected API version (authentication.istio.io/v1alpha1)\nError from server (BadRequest): error when creating "testdata/v1alpha1/authn-policy.yaml.tmpl": the API version in the data (authentication.istio.io/v1alpha2) does not match the expected API version (authentication.istio.io/v1alpha1)\n" exit status 1
I've rebased onto master .... is there anything else I need to do?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/4415#issuecomment-375108483, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJGIxjHtwaW-6dB4Pr72e6h5IuM4fXb3ks5tgsvmgaJpZM4Sy57k
.


Build support for EFS functionality

Our architecture depends heavily on EFS, which has relatively few actions to implement. I would like to propose contributing an implementation of support for EFS, submitting this issue as per: https://github.com/spulec/moto/blob/master/CONTRIBUTING.md.

Hi @pagreene, welcome to Moto! PR's are always welcome.
There's a section in the documentation here with some guidelines on how to get started. Every service is different, so some investigation might still be necessary, but it can at least get you started.
Let me know if you run into any issues -  happy to help.

I'll close this issue, as the initial EFS implementation has now been released.
Feel free to open additional issues/PR's for any other services/methods that are missing.

Sounds good, thanks!


composer install --no-dev fails

When prepping my application for deployment I want to run composer install --no-dev to remove any development only packages. Unfortunately this does not currently work with Flex, as the cache clear post-install step can not run.
You can reproduce this issue by cloning this repo and running composer install --no-dev or by running composer create-project -s beta --no-dev symfony/skeleton:4.0.x sometestfolder.
I get the error:
Executing script cache:clear [KO]
 [KO]
Script cache:clear returned with error code 255
!!  PHP Fatal error:  Uncaught Error: Class 'Symfony\Component\Dotenv\Dotenv' not found in /Users/bens92/projects/newtest/bin/console:19

This is because dotenv is currently marked as a development dependency, however in the console recipe the created console does not check for the existence of the DotEnv class before trying to use it.

There are two potential fixes to this, and I would like some guidance regarding which is preferential:

Move dotenv into the main require block (from require-dev) in this repo
Wrapping all references to dotenv in the recipes repo with a class_exists() check.

I believe that wrapping the calls in a class_exists() would be the better of the two options.

pls remove --no-dev option after retry.

For prod, env vars must be defined. If that's the case, the Dotenv part of the code is never run, and there is no issues. Am I missing something here?

@fabpot this isn't quite in production, this is in a CI build job when creating up the code artefact that will then get deployed to production.
APP_ENV=prod composer install --no-dev works as expected, but this feels a little frail as this isn't the final deployment location.
As you've confirmed that Dotenv is never ran in prod then it sounds like it certainly shouldn't be added in require, thanks.
I think a more useful error message might be useful though. What do you think to changing https://github.com/symfony/recipes/blob/56acc9cfb918c054ad56a44bafa40c953c130a3a/symfony/console/3.3/bin/console#L18-L20 from:
if (!isset($_SERVER['APP_ENV'])) {
    (new Dotenv())->load(__DIR__.'/../.env');
}

to something like:
if (!isset($_SERVER['APP_ENV'])) {
    if (!class_exists(Dotenv::class)) {
        throw new \RuntimeException('You need to define environment variables for configuration or add "symfony/dotenv" as a Composer dependency');
    }
    (new Dotenv())->load(__DIR__.'/../.env');
}

If it's a CI job, you should be able to provide all necessary env vars via its configuration. Travis-CI, CircleCI and possibly all other CI integrations allow adding environment vars for a job 🤔
Dotenv is already a dev dependency in the default Symfony 4 project skeleton, because this component shouldn't be used in prod.
The real issue here is that you do not provide env vars, not that Dotenv is not enabled in production.
Nothing prevents you to create an .env.ci file in your project and run cp .env.ci .env or source .env.ci while executing your job, even though using native env vars injection should be a better solution 😕

As noted, this is the wrong repo for this discussion, as Dotenv should indeed be a dev dependency.
Closing in favour of https://github.com/symfony/recipes/pull/247

Hello, I'm having this same error with a personal console project (https://github.com/DIOHz0r/cryptotrader) using composer install --no-dev  for distribution or production.
You mention to include the .env file but it's ignored with .gitignore for be under control version and also it's created if I run the composer install --no-dev  but it throws the 255 error


propose we remove gist:encryptedText

gist:encryptedText

I propose we remove it
Rationale:
not used in any other gist definitions
not balanced with at least 1 other subproperty (of gist:containedText)
therefore, seems out of place in gist; could add in a sub/domain ontology if needed

Makes sense to me.


Core count determination and sharing

The invoker currently hardcodes the number of containers it is willing to deploy.  Instead it should determine the available core dynamically and also share so that there may be more than one container per core.

We have decided to move this to a deployment configuration parameter and that change has been merged.


API returns a value for 29th Feb in a non-leap year.

Salam,
Try: https://www.moonsighting.com/time_json.php?year=2021&tz=Asia/Riyadh&lat=26.574848&lon=50.0105216&method=0&both=false&time=0
Response: 29th Feb is included.
Expected: 29th Feb should not be included as 2021 is not a leap year.

Apologies I was mixing dates in my code without realizing. This is not an issue.


Unable to render matrix

I used the above LaTeX statement to describe a matrix, but after compilation, I found that it was unable to render successfully while normal formulas can be rendered normally:

Could you look at #95?

Could you look at #95?

Ah, yes, sorry I didn't carefully read the README.
It works now!


Don't repeat steps if the products already exist

If there is already a transformed product for the most recent version of an ontology, we don't need to transform it again.
Comparing versions is contingent on #66

This also goes for the download - we don't want to download an ontology if we've already transformed the corresponding submission ID.


fix: AFJ token encryption

What

This refers to a security measure where sensitive information, such as authentication tokens or API keys, is encrypted before being stored in a database.

Why

Encrypting tokens adds an extra layer of security.

How

Update the codebase to include a mechanism for token encryption before storing it in the database.
Utilize cryptographic libraries or functions to perform the encryption securely.

Please resolve conflicts


Add support for specifying a proxy server in the settings dialog

The issue: Currently, Tor Browser can not be downloaded at all using the launcher if the user must use a (http or https) proxy to connect to the Internet.

The issue: Currently, Tor Browser can not be downloaded at all using the launcher if the user must use a (http or https) proxy to connect to the Internet.

I'm having this issue rn. any way to workaround this? proxychains doesn't seem to work and results in Tor showing "Are you connected to the internet?"
➜  ~ proxychains torbrowser-launcher
[proxychains] config file found: /etc/proxychains.conf
[proxychains] preloading /usr/lib/libproxychains4.so
[proxychains] DLL init: proxychains-ng 4.16
Tor Browser Launcher
By Micah Lee, licensed under MIT
version 0.3.6
https://github.com/micahflee/torbrowser-launcher
Downloading Tor Browser for the first time.
Downloading https://aus1.torproject.org/torbrowser/update_3/release/Linux_x86_64-gcc3/x/ALL
[proxychains] Strict chain  ...  <IP_ADDRESS>:10808  ...  <IP_ADDRESS>:10808 <--socket error or timeout!
Error starting download:

https://aus1.torproject.org/torbrowser/update_3/release/Linux_x86_64-gcc3/x/ALL

Are you connected to the internet?


Added support for parallel tile downloads and control of cache

This pull request is a fix for https://github.com/geopandas/contextily/issues/215
I have added support for parallel tile downloads in the bounds2raster and bounds2img functions. It gives some quite significant speed improvements, with minor changes to the code. I wasn't sure were to put max_num_parallel_tile_downloads = 32, so for now it is on line 227 in tile.py (i.e., inside the bounds2img function code). Let me know if there is anything you would like to have changed.
Thanks for an awesome package btw.! :smiley:

Just made a small change as downloads occasionally failed due a memory error when using threads for the parallel downloads. It was caused by a clash with the memory caching from joblib to cache the _fetch_tile() function, and was fixed by using processes instead of threads for the parallel download.
The processes take ~0.5s to spawn, so this added a minor delay before downloading starts, but it is still significantly faster compared to the for-loop implementation.

I wonder whether any useful default max_num_parallel_downloads will break some services' TOS?
For example, OSM asks you to restrict yourself to two download connections at a time.. I've raised TOS-violating defaults before w.r.t. caching in #202, but I'm not sure what the maintainers (@darribas, @martinfleis) think? I'm fine with letting the user be the "violator," but we should probably document that the defaults set in the package may violate some TOS for some providers.

Good point. I've just made a new commit where the default value is changed to 1. The num_parallel_tile_downloads and max_num_parallel_downloads variables were also renamed to n_connections and max_connections, respectively, and I cleaned up the code a bit.
I also added tests of raise ValueError("n_connections must be between 1 and {max_connections}") in bounds2img() to address the CodeCov errors in the checks.

Yeah, agree that hardcoding max_connections wasn't ideal. I couldn't figure out a good place to put it, but I think there should be a default limit set, so it is clear to the user to see that setting n_connections too high can put excess stress on the tile server. I've just made a new commit where I've added it as a parameter. That is one solution, but it could also just be a warning instead of an error. Or it could it be omitted entirely, and the documentation for n_connections could just state that the user should be careful with it.
In the same commit, I also added a parameter to disable the tile caching, as that makes using parallel connections in resource constrained environments much faster. I.e., it took a lot of time to spawn the processes in small serverless functions, where disabling the cache so the parallel download can be done with threads avoids this. I wasn't sure whether this should be added as a parameter though (or if it should be added at all).
So, in general I wasn't sure whether these changes were in line with the thoughts behind Contextily :sweat_smile: Let me know what you think, and then we might need another round of changes before its ready :slightly_smiling_face:

I think so, this is great!
I would make the option use_cache=True, rather than disable_cache=False, since a double negative (disable and default to False) can be confusing, but that's the only thing I'd change.

@JacobJeppesen Can you change the keyword per @ljwolf's suggestion? It makes sense.

Yeah, I agree that the double negative should be avoided. I've changed it to use_cache=True in the newest commit.
Let me know if there are any more changes needed :)

Thanks @JacobJeppesen!

Thanks to you too! :)