In-memory caching for endpoints

@hepplerj @qtrinh2
The data API's performance is acceptable. However, that is in part because it isn't under high load, and in part because we do some crude caching of expensive queries. For example, the North America endpoint loads the data in a closure when the endpoint handler is created at startup. So the data is essentially saved in memory, and the handler will never go back to the database for new data until the application is restarted. This of course does not work if the endpoint actually needs some kind of parameter to know what to query, e.g., a date. We also set a pretty long-lived client-side cache, telling browsers not to ask for any endpoint for one week.
Probably the biggest gain we could get in performance is to cache responses from the database. Even if the queries are small (and some are definitely not) the majority of the time is spent going out to the database. And there is a lot of reason to think that many of our queries are the same. E.g., a RelEc map will load the same data; APB will always query the featured verses.
We can probably cache in-memory, because we do not run multiple instances of the API, and because Baird has a lot of memory and isn't doing much else.
The simplest way to do this is to add middleware that caches the responses. That's what this draft pull request does. See the diff for details.
This passes all the tests. I've also added a test endpoint /cache which will show you the startup time and the time the handler was last called. The startup time will never change until application restart. The handler time will remain the same if the response is cached, until the TTL expires (currently an hour) or the memory limit for the cache is exceeded (unlikely in manual testing). You can also visit /cache?nocache to get an uncached version. That query parameter will work on any of the endpoints as well.
If we are going to do this, we will want to make sure we have set the right cache invalidation algorithm, the right TTL, and adjusted the client cache header accordingly. We will also want to test it before the release of APB on December 13, which is my motivation for adding this now. We will also want to check whether our crude, "closure caching" should be changed for any of the existing endpoints.
What are your thoughts?

Thanks for looking it over, @hepplerj. Let me think through the "closure caching" question and add some benchmarks before merging.

@hepplerj Did you want that commit on a different branch?

@qtrinh2 Thanks, Tony. That's very helpful.
Your points about benchmarking and considering a more granular approach are very useful, so I've tried to do that. I added some benchmarks to test various kinds of endpoints with and without the caching. You can run this with make bench.
I am for the moment running the Go app on my laptop; I think for the results to be most useful I'd want to run this on Baird so the database connection isn't going over the network. But here are the preliminary results.
goos: darwin
goarch: arm64
pkg: github.com/chnm/apiary/cmd/apiary
BenchmarkCachedCounties-10           	     390	   3314876 ns/op
BenchmarkNotCachedCounties-10        	       4	 282302802 ns/op
BenchmarkCachedNorthAmerica-10       	    2224	    543992 ns/op
BenchmarkNotCachedNorthAmerica-10    	    2035	    530551 ns/op
BenchmarkCachedVerseTrend-10         	   40942	     27651 ns/op
BenchmarkNotCachedVerseTrend-10      	  283446	      4259 ns/op
PASS
ok  	github.com/chnm/apiary/cmd/apiary	391.410s

For an endpoint serving large GeoJSON (Counties) there is a very significant speed up.
The NorthAmerica endpoint serves a smaller GeoJSON object, and it is in a sense already cached in memory since the data from the database is loaded by a closure at application start only one time. Still, there is a minor speed up.
Surprisingly to me, an endpoint serving not very big quantitative data VerseTrend is significant slowed down by using the caching.

I need to do some more work on this. But I think the strategy here should be modifying things as Tony suggested and caching only specific endpoints where the benchmark demonstrates that it is warranted.


Bug: Library is not properly supported in nuxt app

The updated version 0.0.2 does work pretty fine in the Playground. But does not support in nuxt app. It does show a UI, but nothing happens when clicking "+" button. Also the below error appears:
[Vue warn]: Failed to resolve component: FormKit If this is a native custom element, make sure to exclude it from component resolution via compilerOptions.isCustomElement.

Closed by 157b171dc65acfb90f5e975045142401858fdcf1


[Bug]: Additional Pokémon showing up in Team Viewer After Battle with Teammate

Describe the bug
In Pokémon Emerald, after the fight where you team up with Steven against Team Magma, the Pokémon he picks for his team confuse the team viewer, and end up sticking around even after trying to reset it with my own Pokémon from the PC. I assume this issue would probably occur with any dual battle with a teammate. tried refreshing the tracker, but that didn't seem to fix it.
Steps to Reproduce

Make it to Mossdeep city
fight team magma
make it to boss battle
team up with Steven
selected only 1 pokemon for myself (not sure if that mattered or not but felt it was worth noting)
after the battle, team viewer should be broken

What tracker version are you using?
v7.5.3
What operating system are you using?
Windows 10
What emulator(s) are you seeing the problem on?
Bizhawk 2.9
Relevant log output
No response
Additional Information

Whoops, I seem to have replied to the wrong conversation around the Team Viewer. I believe this one has yet to be resolved, and is not currently fixed. Sorry about that.


Editor UI Themes

I've been working on my own editor UI theme, and I've run into an issue where some styles seem to be core styles whereas other styles are surprisingly defined only in the theme.
For example, the .redsquiggly class, which is applied to error underlines, only seems to appear in theme CSS. So if I apply my own custom theme to VS and forget to define .redsquiggly, all of the sudden, error marks are invisible. On the other hand, red error marks show up in the scrollbar regardless of whether my custom theme defines anything about them.
I think there are a few possible solutions here:

a well-defined list of selectors that custom themes need to implement styles for
Or, the core editor has styles for all features, custom themes only override these styles

If I'm misunderstanding something here, any recommendations on the best way to make a custom UI theme?

I would suggest that you set the theme to vs my-custom and then you'll get all the vs selectors kick in + you can overwrite via my-custom.

Thanks, I'll give that a shot and see how it works out. Small sidenote, the documentation seems to be a touch out of date then:
/**
* Theme to be used for rendering. Consists of two parts, the UI theme and the syntax theme,
* separated by a space.
* The current available UI themes are: 'vs' (default), 'vs-dark', 'hc-black'
* The syntax themes are contributed. The default is 'default-theme'
*/
theme?: string;

I don't believe the UI theme vs. syntax theme distinction actually exists, and I don't think 'default-theme' actually exists either?

Closing this as I think it's answered

thanks @JKillian :+1:


getInvite not returning valid Invite Object

I am putting the invite URL in correctly however the returned Invite Object incorrect. I have also tried using just the Invite Code without the URL portion and get the same results. I have also experimented with infinite and timed invites and both return the same basic Object.
Invite {
  maxAge: undefined,
  code: '0qwPf0WZWbvq6Qmg',
  channel: { type: 'text', id: '152871118972256256', name: 'general' },
  server:
   { splash_hash: null,
     id: '152871118972256256',
     name: 'Tankerkiller125' },
  revoked: undefined,
  createdAt: NaN,
  temporary: undefined,
  uses: undefined,
  maxUses: undefined,
  xkcd: null }

Is it a possibility that I'm not using it correctly? Or maybe joinServer() is not correct is some way?

Quick question -- Is the account you're experiencing the bug on an OAuth accounts?
OAuth accounts are unable to accept invitations.

I have tried it with standard login as well.
On Mon, May 23, 2016, 15:22 Programmix<EMAIL_ADDRESS>wrote:

Quick question -- Is the account you're experiencing the bug on an OAuth
accounts?
OAuth accounts are unable to accept invitations.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/hydrabolt/discord.js/issues/370#issuecomment-221066705

No, but is your account an OAuth account? Doesn't matter how you're logging in.

Seems valid for a basic invite object. Does your bot have Manage Channel permissions in that channel?


Compatibility with others WYSIWYG editors.

Is there an easy way to add compatibility with the editors? I have already editors like jHtmlArea and TinyMCE , i just need the Icons Picker and the autocomplete functions. Do you know how can i achieve this?

Not yet,
there was no such goals.

Thak you for the quick response and congratulations for this awesome work. I really apreciate it!

Thanks)

doesn't "standalone mode" solve this problem?


CocoaPods 1.6.0 or above are not supported for resource-containing static frameworks

I got an error when running with CocoaPods 1.7.0.beta.2:
TypeError - no implicit conversion of Array into String
/usr/local/lib/ruby/gems/2.6.0/gems/cocoapods-binary-0.4.2/lib/cocoapods-binary/Prebuild.rb:131:in `basename'
/usr/local/lib/ruby/gems/2.6.0/gems/cocoapods-binary-0.4.2/lib/cocoapods-binary/Prebuild.rb:131:in `block (2 levels) in prebuild_frameworks!'
/usr/local/lib/ruby/gems/2.6.0/gems/cocoapods-binary-0.4.2/lib/cocoapods-binary/Prebuild.rb:129:in `each'
/usr/local/lib/ruby/gems/2.6.0/gems/cocoapods-binary-0.4.2/lib/cocoapods-binary/Prebuild.rb:129:in `map'
/usr/local/lib/ruby/gems/2.6.0/gems/cocoapods-binary-0.4.2/lib/cocoapods-binary/Prebuild.rb:129:in `block in prebuild_frameworks!'
/usr/local/lib/ruby/gems/2.6.0/gems/cocoapods-binary-0.4.2/lib/cocoapods-binary/Prebuild.rb:118:in `each'
/usr/local/lib/ruby/gems/2.6.0/gems/cocoapods-binary-0.4.2/lib/cocoapods-binary/Prebuild.rb:118:in `prebuild_frameworks!'
/usr/local/lib/ruby/gems/2.6.0/gems/cocoapods-binary-0.4.2/lib/cocoapods-binary/Prebuild.rb:206:in `block in <class:Installer>'

Looks like this is due to the change for Pod::PodTarget#resource_paths

doc: https://www.rubydoc.info/gems/cocoapods/Pod%2FPodTarget:resource_paths
A PR which introduced the change: https://github.com/CocoaPods/CocoaPods/pull/7820

The method now returns [Hash{String=>Array<String>}] over [Array<String>]
Included since 1.6.0.beta.1.

Affected code: https://github.com/leavez/cocoapods-binary/blob/9f40c5df4149598b03b44c01d33b04e78ff38772/lib/cocoapods-binary/Prebuild.rb#L120-L130

Could you take a look? Thanks in advance!

@leavez Thanks for the quick fix! 💖

I found some changes in copying resources in cocoapods 1.7 for static framework. The resource coping still not work.
https://github.com/CocoaPods/CocoaPods/issues/8573#issuecomment-476970611

I found there's a change in copying resource files for static framework in cocoapods 1.7.
When building a pod with 'static_framework = true' and with cocoapods 1.6, it will generate a framework containing the resource files in framework bundle. But when use cocoapods 1.7, no resources will copied to the framework bundle.
This plugin cannot handle the change right not. So It doesn't work in cocoapods 1.7

Hey @leavez ,
Thanks for your amazing work!
Is there any estimation for a fix for cocoapods 1.7 static frameworks with bundle?

do we have a solution for this yet?

not support resources now?


feat: reduce varlet cli dependencies size

Checklist

[x] Fix linting errors
[x] Check all command works
[x] Logger part using pure color.

Change information

Now. Like babel or others popular libraries are using picocolors repalce chalk. Becasue chalk after v5 is pure esm. It's too radical. but v4 support cjs and esm and the size too large. So picocolors is a simplify terminal color. I think it can work well with varlet-cli. And the nanospinner is the faster than ora and mininal.
#865

Codecov Report

Merging #866 (0c2d6ef) into dev (09292b0) will increase coverage by 0.60%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##              dev     #866      +/-   ##
==========================================
+ Coverage   94.58%   95.19%   +0.60%
==========================================
  Files         268      268
  Lines        5931     5931
  Branches     1084     1084
==========================================
+ Hits         5610     5646      +36
+ Misses        321      285      -36

Impacted Files
Coverage Δ

packages/varlet-ui/src/button/Button.vue
95.34% <0.00%> (+13.95%)
:arrow_up:

packages/varlet-ui/src/button-group/provide.ts
100.00% <0.00%> (+66.66%)
:arrow_up:

...ackages/varlet-ui/src/button-group/ButtonGroup.vue
94.11% <0.00%> (+94.11%)
:arrow_up:

packages/varlet-ui/src/button-group/index.ts
100.00% <0.00%> (+100.00%)
:arrow_up:

packages/varlet-ui/src/button-group/props.ts
100.00% <0.00%> (+100.00%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[Sign in/Sign up] System allows user to sign in with old password after user has changed password to new one

Environment: MacOS 11.4, Google Chrome Version 94.0.4606.71 (Official Build) (arm64)
Reproducible: always
Build found: 7.11.2021
Preconditions

Login to GreenCity as a User https://ita-social-projects.github.io/GreenCityClient/#/

Steps to reproduce

Click on logged in user in the right top corner of the screen
Select 'UBS-user' from drop down list
Click on "Дані користувача" tab and scroll to the end of a page
Click on "Змінити пароль" button
Fill in pop up window with correct data (old password, new password and confirm password)
Click on "Змінити пароль" button
Log out from personal cabinet (select "Вийти" from drop down list)
Click on "Увійти" button on menu bar
Fill in form with email address and NEW password and click on "Увійти" button
Erase password field and populated this field with OLD password

Actual result

System does't allow user to sing in with new password and warning message "Неправильна пошта або пароль" appears

System allow user to sing in with old password

Expected result

System should allow user to sing in with NEW password
System shouldn't allow user to sing in with OLD password

User story and test case links
E.g.: "User story #227

Done - verified


Close resources left by go-git after request is finished

Rebase of #8915 onto master
In #7947 I figured out that go-git leaves handles to the pack files open. These are eventually garbage collected, but this isn't very deterministic.
What I did:

Close any open repository handles after each request via a common middleware so it is on one place instead of littered throughout the source code. (There might still be a possible race condition here if many concurrent page views are happening on the repository and a transfer ownership or rename is attempted)
Close the repository before ownership transfer or rename which are the operations that touch the directory.
Log if closing the repository fails, apparently go-git can return an error.
Tested on Windows based on the testcase in #7947 as I'm a Windows user.

I believe this should be quite easy to merge and addresses all issues when it comes to resources left open by requests.

Codecov Report

Merging #8916 into master will increase coverage by <.01%.
The diff coverage is 48%.

@@            Coverage Diff             @@
##           master    #8916      +/-   ##
==========================================
+ Coverage    41.3%   41.31%   +<.01%
==========================================
  Files         545      545
  Lines       70368    70393      +25
==========================================
+ Hits        29068    29080      +12
- Misses      37601    37611      +10
- Partials     3699     3702       +3

Impacted Files
Coverage Δ

routers/repo/setting.go
13.13% <0%> (-0.2%)
:arrow_down:

modules/git/repo.go
56.79% <100%> (+0.42%)
:arrow_up:

routers/routes/routes.go
82.61% <100%> (+0.04%)
:arrow_up:

modules/context/context.go
50.22% <72.72%> (+1.17%)
:arrow_up:

models/unit.go
62.16% <0%> (-5.41%)
:arrow_down:

modules/migrations/migrate.go
21.22% <0%> (-1.68%)
:arrow_down:

modules/migrations/gitea.go
8.05% <0%> (-0.64%)
:arrow_down:

models/repo.go
49.07% <0%> (-0.06%)
:arrow_down:

models/error.go
33.86% <0%> (+1.18%)
:arrow_up:

modules/task/migrate.go
28.94% <0%> (+3.94%)
:arrow_up:

Continue to review full report at Codecov.

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


Display street View options for Expo apps

Feature request
First Thanks to the developers for this great package.
It would be really helpful if this package could support streetView
A prop or a method to open street view in the MapView.
There is a package called react-native-streetview but it doesn't work with expo
Possible implementation
I implemented the street view in react app using the window.google object. maybe something like this or the similar props would work
new window.google.maps.StreetViewPanorama(
          document.getElementById('street-view') as HTMLElement,
          {
            position: { lat: latitude, lng: longitude },
            pov: { heading: HEADING, pitch: PITCH },
            zoom: 1,
            fullscreenControl: false,
          },
        )

This would be a great addition

Any Support?

how can i use this in RN?


Refactor allo-isolate to make it cleaner

Hi thanks for the lib! I am trying to refactor it to make it cleaner. Things that I have done can be seen in the messages of each commit.
Possibly related: @jerel @shekohex

@shekohex Seems that "2 workflows awaiting approval". May I get the approval?

Try cargo +nightly fmt.

@shekohex done


Add Payload::readEntireBody?

Currently, to read an entire body you must write:
$body = '';
while($payload->isReadable()) {
  $body .= $payload->read();
}

A body larger than 8kb or so is truncated when simply doing a ->read()
Further, it would be nice to specify the number of bytes (in the case of Content-Length header being sent) when reading, and an exception should be thrown if the full size and exactly that size isn't met (so a 4xx response can be sent).

On another note, it would be nice if all these classes weren't final so modifications to behavior (like this) can be made through inheritance without having to open an issue.

What does this have to do with Amp? There is no Payload class here.

Payload only exists to make buffered consumption even easier, so there's of course already a method for that:
https://github.com/amphp/byte-stream/blob/daa00f2efdbd71565bf64ffefa89e37542addf93/src/Payload.php#L89
If you're referring to http-client: The content length is automatically enforced, so there's no need for changes. Generally, we prefer composition over inheritance, so all classes are final by default.


[DOCS] Coordinate information is deleted from the results after the Inference operation

Thank you for helping to improve solaris!
Documentation request summary

The coordinate information is deleted from the results after the Inference operation.
Config file metric field.

Task detail and notes

Details
When I examined the results after performing an inference, I noticed that the coordinate information was lost. Do you have any suggestions on this topic? Why is it disappearing? I didn't do anything specifically to erase this coordinate information. I did the training and inference with the automatic machine learning command you specified in your documentation.
To explain myself better. For example, if I say to divide 512x512 pieces, pixel information is coming instead of coordinate information. This prevents me from drawing the polygons on the original tif. If there is any information in this documentation, please do not hesitate. If you don't, I'd appreciate it if you could help.

Details
You have placed the metric option in the configuration file for the training and valid fields. It should be in list format as far as I have examined in the documentation. But you have not provided information about the spelling format of metrics. For example, ['f1', 'acc'] and so on. Can you give me an example of how I should edit the metric field?

Hi @williamobrein,
The problem is that neural networks have no idea how to handle coordinate reference systems - predictions have to be made in pixel coordinates and then converted. Have a look at sol.vector.mask.mask_to_poly_geojson() in the tutorials and the API docs.

Hi, I solved the error when you debug your code. You have arranged the mask_to_poly_geojson () part according to the projection of the Spacenet data. Because the projection was different, the value of the min_area should have changed. Thanks!
You didn't say anything about metric? What exactly is the format? I didn't realize.

Metrics are still under active development. At present, they can only be provided for Keras models using the standard Keras naming scheme; it's something we're planning to add for Torch when we get to it (#175)

Thanks!


dm/doc-github

The  CI is broken because mkdocs depends on pyyaml, which has a CVE, so pipenv check fails.
The problem is that version 3.12 is vulnerable, and version 4.0 is not out yet.
So let's merge this and try to solve the CI issue in an other branch.


LaTeX / MathJax on gitpitch.com

Greetings,
I have noticed that when using gitpitch.com and trying to subscript in latex it seems I run into trouble when trying to use more than one multi-character subscript.  I am using: mathjax : TeX-AMS_SVG.  Both of these work on mathjax.org and otherwise in latex.
Works:
$$ A_{n+1} = A_n + A_0 $$
Does not work:
$$ A_{n+1} = A_{n+0} + A_0 $$
Thanks for your help

Hi there. Can you provide a link to a presentation that reproduces the problem you have reported?

Sure:
http://gitpitch.com/dhidas/GitPitchTest

On Mar 19, 2018, at 12:04 PM, David Russell<EMAIL_ADDRESS>wrote:
Hi there. Can you provide a link to a presentation that reproduces the problem you have reported?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

And you mentioned there is a working example of this somewhere on mathjax.org. Again, can you please share a link. Thanks.

$$ A_{n+1} = A_n + A_0 $$
$$ A_{n+1} = A_{n+0} + A_0 $$
You can cut and paste that into: https://www.mathjax.org/#demo
Both work on mathjax.org (and are valid latex)

On Mar 19, 2018, at 1:24 PM, David Russell<EMAIL_ADDRESS>wrote:
And you mentioned there is a working example of this somewhere on mathjax.org. Again, can you please share a link. Thanks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I am getting this same problem. Indeed @Takahiro-Itazuri solution works, so $$ A_{n+1} = A_{n+0} + A_0 $$ becomes $$ A_{n+1} = A_{n+0} + A_0 $$. This does seem less than obvious though.

The underlying cause of this problem is a parsing issue when using the Reveal mathjax plugin within markdown. Which is the case when working with GitPitch. The original issues on this are here and here.
I have added a note to the Mathjax page on the GitPitch wiki so in future users will be alerted to this issue rather than having to spend time to discover it.


converting data from EAD / PULFA into figgy

We need to revisit how the data is being pulled from EAD and used in figgy/DPUL.  For example: figgy shows language and a date, but DPUL only shows the title.

@joycebcat I'm moving this issue over to Figgy — can you create issues over there going forward?


[BUG] v0.8.4 does not work on databricks without changes

Minimal Code To Reproduce
captures_table = spark.sql(f"""SELECT id, keyword_context, model_prediction
                            FROM bronze.captures
                            WHERE model_prediction = -1 AND
                           policy IN ({policies})""")

def predict(df:pd.DataFrame, model:Pipeline) -> pd.DataFrame:
  df.loc[:, 'keyword_context'] = df.keyword_context.apply(lambda x: x.replace("\n", " "))
  prediction = model.predict(df.keyword_context)
  df['model_prediction'] = prediction
  return df

result = transform(
  captures_table,
  predict,
  schema="*+model_prediction:int8",
  params=dict(model=model),
  engine=spark
)

Describe the bug
v0.8.4 stopped working on databricks spark.
FugueInterfacelessError: (‘DataFrame[id: bigint, keyword_context: string, model_prediction: smallint] is not a valid creator’, AttributeError(‘DataFrame[id: bigint, keyword_context: string, model_prediction: smallint] is not a function’))
Thanks to Han Wang, this was identified as a missing prerequisite: cloudpickle. Further, I used to be able to simply add 'fugue' as a pypi library in databricks clusters, but now that also gives an error, so I need to set it as 'fugue[spark]'. With those two things done, fugue once again works for Databricks.
One other point is that cloudpickle is included by default in 'ML' versions of databricks clusters, but not standard clusters (it is not necessary to take the GPU ML version, though).
Expected behavior
The function is called and the model prediction added to the dataframe as a new column.
Environment (please complete the following information):

Backend: spark 3.4
Backend version: databricks 13.0 on Azure
Python version: 3.10
OS: linux

This has been resolved by https://github.com/fugue-project/fugue/pull/474
Please try fugue==0.8.5.dev1


[RFC] Allow to make arbitrary cluster test suite with yaml files

Currently cluster test has two related concepts

Experiment - you can see experiment as what is called #[test] in unit test. Experiment contains set of actions and have true/false output. It's a bit more complex then just #[test] - it can emit performance metrics, exposes set of validators affected, but overall it is very close to a test function. Examples of experiments are  - RebootRandomValidator, PerformanceBenchmarkNodesDown, RecoveryTime, etc

Test suite - test suite right now is basically Vec<Experiment>. Currently we encode test suites as just some rust code in suite.rs

Proposal is to allow to encode test suites in toml files:
git\
  cluster_test\
     suites\
         ci_suite.yml
         perf_suite.yml
         land_hook_suite.yml

And then specific suite can look like:
land_hook_suite.yml
- RebootRandomValidators
    - repeat: 5
    - params:
       - num_instances: 3
- PerformanceBenchmarkNodesDown
    - params:
       - percent_down: 10

We can then parse this toml file into Vec.
We have some plumbing already - each experiment have parameters that implement ExperimentParam, and we already deserialize them with StructOpt, we will need to add yaml deserialization for them.
This will be helpful for few reasons:

We already have two test suites, and going to have one more for land hook
Research can leverage those and write more yaml files

@ankushagarwal @pdhamdhere

High level question about what we now call "experiments".
My intuition is that each experiment can be represented via a set of "actions" (some nodes are down, some are rebooted, some keys are rotated, etc.), as well as a set of "measurements" (measuring tps, measuring latencies, verifying safety properties).
I wonder whether the representation of the tests hardcoded as PerformanceBenchmarkNodesDown or RebootRandomValidators is going to limit our ability to easily combine test "actions" and "measurements". I might be wrong but it seems that today experiments must be run separately.
Would it make sense to construct "experiments" from the lego pieces of the predefined "actions" and "measurements"? Something like:
my_funny_experiment.toml
- Actions
  - SomeNodesDown
     - percentage: 10
  - RandomReboots
     - percentage: 5
  - MultyRegionLatency
     - num_regions: 3
     - latencies: ...
- Measurements
  - SafetyProperties
  - TransactionThroughput

I understand that it might not follow the current direction of the project, just wanted to hear your thoughts about supporting different combinations :).

As you noticed yes, this document only cover test suites(e.g. sets of experiments to be run in different parts of CI pipeline), but leaves aside what experiment is. But let's discuss it.
So there are two ways to approach encode an experiment.

Declarative approach. This is what you propose - we have a config file with sections like 'actions', 'measurements' and 'topology' and let cluster-test to figure out what to do.
*Programmable approach. Instead of having config file of fixed format, we provide convenient API for a person to write such experiment in a rust code. We need to make this API concise enough so that in simple experiments such as my_funny_experiment in your code it looks nice and short, smth like that:

impl Experiment for MyFunnyExperiment {
   pub fn run() {
     apply(NodesDown{percentage:10});
     apply(MultyRegionLatency{num_regions:10});
     measure(xxx);
     measure(yyy);
   }
}

I am now in favor of second approach, because it provides higher degree of flexibility. You might want to do A, measure X, then do B measure Y, you might want to do some kind of loop in there, or maybe something else.
I am open to discussion indeed

Great, thanks for the explanation!
I think the dichotomy of declarative vs. programmable experiments makes sense.
Both would work (declarative seems more intuitive for me, but it's probably a matter of taste).
In case of programmable experiments, what would be the structure of the .toml file? Is it the name of experiment + its params? (note that in case of MyFunnyExperiment we have several parameters for actions and measurements that comprise the experiment, need to be careful with naming there).

Yes, so those suite files as I see them are basically to encode which things need to be done in CI pipeline.
So that we would have on_commit.yml, pre_nighyl.yml, pre_release.yml, etc

Agree with what @andll said.
The declarative approach for writing experiments would be too restrictive and we would be adding another translation layer which is not necessary.


fix(examples): Refactor examples for reference architectures

Description
PR delivers changes for example combined_design:

introduce optional in order to define whole type of variable vpcs (it requires to defined Terraform 1.3 as minimum in versions.tf)
remove vpc_subnet, add separate keys for vpc and subnet
simplify usage of module subnet_sets (without changing internal logic module)
simplify creating VPC routes

Motivation and Context
There were few motivations for this PR:

issue #37 (in which centralized_design was mentioned, but the same style of code is in every example, including combined_design)
internal discussion on planning meeting

As in previous year there were prepared examples for references architectures, there were done with below assumptions:

internal modules should not to be rebuild during preparation of examples
most of settings can be changed via modification of example.tfvars (e.g. new VPC, new subnet, additional route via TGW, IGW, NAT Gateway or GWLB endpoint etc.)
minimal version of Terraform for modules and examples remains the same (Terraform 1.0, so there is no possibility to use optionals in examples)
concepts used in modules (e.g. sets for subnets, NAT gateway, GWLB endpoints etc.) are not going to be changed

About the need of refactor of AWS modules we were talking multiple times, but we wanted to finish at first refactor of Azure modules - it's was finished already on April in https://github.com/PaloAltoNetworks/terraform-azurerm-swfw-modules/releases/tag/v3.0.0.
In order to define best user friendly example.tfvars (according to the Amazon Method - Working Backwards), refactor of internal modules, embedding some logic of existing modules in other modules (e.g. maybe subnet inside vpc module?) and rethink of set concept needs to be done (it's not in this PR) - this PR introduces fixes for the most painful fragments of the example combined_design. Refactor of AWS modules will take time, this fixes can be introduced quicker.
If you still see another improvements in example combined_design (without rebuilding internal modules):

please put it as a comment and propose changes
or prepare your own PR with proposition of completely new approach for example combined_design

While doing review, please suggest if the same changes should be applied for other examples (before refactor of AWS modules will be started) ?
How Has This Been Tested?
Code was tested locally and via ChatOps.
Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist

[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes if appropriate.
[x] All new and existing tests passed.

/idempotence paths="examples/combined_design examples/combined_design_autoscale examples/isolated_design examples/isolated_design_autoscale"

/idempotence paths="examples/centralized_design examples/centralized_design_autoscale examples/panorama_standalone examples/vmseries_standalone"

/idempotence paths="examples/vmseries_standalone"


Dates moved by one day when set international and also formatting error

Version of flutter_calendar_carousel
1.3.18
Expected behavior
Issue 1
Week view and month view should show similar dates and days.
Issue 2,
when the locale is set, then the text formatting should also be appropriate
Actual behavior
Issue 1.
The date between the two calendars should be the same. However, as you can see in the below attachments the dates (in week view) shows "1" as Tuesday and '2' as Wednesday, whereas the full month view shows them correctly where 1 is Monday and 2 is Tuesday
When showing week view the dates are moved by one day.

Issue 2.
On Week view Monday is shown in red (weekend format), whereas in the month view it is shown properly. I can raise a separate issue for this.
flutter doctor result (run flutter doctor in terminal)
✓] Flutter (Channel dev, v1.5.8, on Mac OS X 10.13.6 17G65, locale en-DK)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
[✓] Android Studio (version 3.3)
[✓] VS Code (version 1.35.1)
[✓] Connected device (1 available)
Steps to reproduce the behavior
I have just done git clone of the repo and using the example app, passing the locale as string 'da'.
I have just enabled expanded arrow button to hide the other calendar which by default shows up in the example you have
last but not the least thanks for the good library it is great :)
EDIT: found that even without locale it is having the same issue with dates. On week view it is one day added for July, but in the month view, it is perfect.

Thank you for the great issue. I might have to debug on the first issue as soon as possible.
Looks very promising.

I have this problem too, the date and the day is not match, the difference is a day

Thanks for such a great library.
Getting this issue with the example code month view too. Any idea by when this will be fixed?

@godon019 Would you prefer to help people with this issue?

@godon019 Would you prefer to help people with this issue?

I would like to work on it !


Can't Deploy

Describe the bug
I've tried deploying many times and tried fixing the problem but no luck.
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-txmhhb
Steps to reproduce
I just hit the Deploy button and it fails over and over again.
Expected behavior
Deployment
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 130
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 82196592,
  "usedJSHeapSize": 76018828,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = dferrara
Chat ID = 2609ebb1d73c

Additional context
Error:
Failed building the project. Make sure the build command is correct and try again.

i have exactly the same problem cost me alot of tokens


Handle Wikipedia table faulty boolean design; and HTML headers with row/colspan > 1 (also XLS)

TO thombashi, Tsuyoshi Hombashi
SUBJ Handle Wikipedia table faulty boolean design, and headers with row/colspan > 1
I have noticed a couple design flaws in how Wikipedia generates tables.
Maybe you would like to incorporate some workarounds in sqlitebiter:
A) Some (all?) tables use a GREEN CHECK / RED X  indicator for Boolean columns. A broken design, they are using
only TD attributes to display the image, and the actual TD cell content is blank, empty.  This results in NULLs
coming out of conversion programs (incl sqlitebiter).
This situation can be searched/fixed by looking for pattern <td data-sort-value="Yes" ...  (or "No"), and injecting 1/0.
B) Some tables have header cells with rowspan/colspan > 1, for making a fancy "grouped" header.
This breaks the column assignment for later data values.
For example, if the first header row has 5 cells including one with colspan=2, and the 2nd header row and data rows have 6 cells,
then converters (incl SqliteBiter) will see only the 5 cells, and some data will go missing or end up in wrong-named columns.
It's a little tricky to detect/automate, since the rowspan/colspan settings are across multiple html lines.
Example broken Wikipedia tables at https://en.wikipedia.org/wiki/Comparison_of_text_editors
NOTE OF APPRECIATION FOR Tsuyoshi Hombashi
Very nice work on SqliteBiter.  And great name, clever with good rhyme.
Just what we need to grab/manage those tables floating around everywhere.
I had come up with some multi-stage dataflows through other apps, but too many steps, kept searching
for a better solution, which you have made.
Thanks for producing such a useful comprehensive CLI converter tool.
Best regards,
John
-- jbthiel

A related issue is in XLS sheets converted via Gnumeric, from HTML tables with header rowspan/colspan = 2.
If all the headers are merged (per all rowspan=2) it's ok, but if there is a mix of merged and unmerged cells, then sqlitebiter fails on XLS with:
$ sqlitebiter file -f excel ../first-sheet-merged-headers.xls
[WARNING] convertible table not found in ../first-sheet-merged-headers.xls
[INFO] converted results: source=0

Example is the first sheet fromthe HTML tables at
https://en.wikipedia.org/wiki/Comparison_of_text_editors
converted to XLS via Gnumeric.


fix: use default socket path for TCP connections

Changes

:bug: Fix issues with TCP TLS connection.

resolves: #668

/hold

The Func E2E Test / E2E Test (1.16.x, ubuntu-latest) failure is probably flake.

The Func E2E Test / E2E Test (1.16.x, ubuntu-latest) failure is probably flake.

I re-ran the jobs - we'll see how it goes.

@lance  this needs rebase on the fix for node builders

/unhold


Pipe waits for the first command to be finished

Since there are 281 open issues, I'm not sure if this is reported before or not
Try a command that takes too long (possibly forever) and produces lots of output, and we want to give the output to another command using pipe, for example:
find / -iname '*.exe' | less

PowerShell waits for the first command to be finished, and then runs the second command and give it the whole output. That is not what Unix pipes have been about for decades! We expect the second command to be executed right after we press enter (try sleep 10 | echo test in Bash), and send any output from first command immediately to the second command.
Tested with PowerShell 6.0.0 on Debian GNU/Linux

This is a core PowerShell design feature and is not likely to be changed as would be a major breaking change due to the way that the pipeline works and is designed.
an example to understand how the pipeline is used in Powershell would be like
 Get-ChildItem C:\scripts\ -Recurse | Where-Object {$_.Name -like "*.ps*1"} | Group-object -Property Extension | Select-Object Count,Name

To break this down we are getting all the items in the C:\Scripts\ folder and recursive folders and then with the collection of objects that would output we pass this to Where-Object using the Pipeline to then only return files that have a ps*1 extension and then with the collection of objects that would output this is then passed again using the pipeline to Group-Object where we get an output based on the grouping of the items based on the file extension and the we pipe this to Select-Object and tell it that we want only the Count and Name returned from that collection
The Output we would get from would get something similar to the below
Count Name
----- ----
   76 .ps1
    1 .psm1

Happy to answer any other questions on the Pipeline if you have them?

Hi!
@kilasuit, could be off base, but IIRC commands with a proper process block implementation will stream their output, allowing something akin to @ilius' example.
I'm guessing there might be some odd behavior if you mix and match output from *nix tools and PowerShell tools, and this would be expected in cases where you have a blocking command (e.g. you need all items to sort them).
Cheers!

@RamblingCookieMonster is correct. Powershell will stream objects down the pipe when it can. A good common example would be to tail a log file and pipe to a search string.
Get-Content -Path $path -Tail | Select-String 'Error'
But there are other commands that will block the pipe and wait for all the objects before continuing. Sort-Object is an obvious example.

I can't see how this breaks compatibility
If a command (like ``Sort-ObjectorSelect-Object`) needs the whole input to process, it can wait for the whole input itself, that's up to the second command (receiver), that does not prevent the first command (sender) or the Shell to send it's output.
https://en.wikipedia.org/wiki/Pipeline_(Unix)

Closing as this is a duplicate, see #559.  It is fixable, but a bit tricky.


Build error when 'Jest' option chosen during setup

This error was generated after attempting to run my initial 'npm run build' after creating a library.
MC02L20JSFFT4:ui-common-testing data$ npm run build

><EMAIL_ADDRESS>build /Users/data/data-repositories/testing
> gulp build

module.js:471
    throw err;
    ^

Error: Cannot find module 'node-sass-tilde-importer'
    at Function.Module._resolveFilename (module.js:469:15)
    at Function.Module._load (module.js:417:25)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/cbentson/trinet-repositories/ui-common-library-testing-version/tools/gulp/inline-resources.js:9:23)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: `gulp build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/cbentson/.npm/_logs/2017-06-29T16_56_45_928Z-debug.log

Steps to reproduce:

'yo angular2-library' to generate library boiler-plate code
leave name, email, name of project, and repo all blank during setup
choose jest for testing
run 'npm run build' from root of project as soon as library finishes building

The initial build works just fine when I choose the 'karma-jasmine' option for testing

Must have missed the dependency during rebase.
Sorry about this its in pr #144

PR has been merged and released as v11.0.2 👍


Add connection template support

Adds the ability to reference user info in connection configuration using a mustache/handlebars like syntax.
For any text (string) connection field, user info may be referenced using {{user.fieldName}} syntax.
This also adds additional user fields, name and data. Name will be used eventually for a the preferred human friendly user name. data is to be used to store whatever end users would like to store, and is not something SQLPad will be caring about.
Example: Let's say each SQLPad user has a specific database username they are to use when connecting to a database. You may specify this value in the SQLPad user.data field as user.data.dbuser (how that is to happen TBD), then reference that value when defining a connection, setting username to {{user.data.dbuser}}. When queries are run against a database using that connection, {{user.data.dbuser}} gets swapped out with the value associated with the user running the query.

This implements #532, but does not add any UI associated with this feature.


[RTS Viewer] Viewer is not able to show the streaming when streaming is started from Dashboard with multi-source ON and no Source ID

Viewer: Viewer is not able to show the streaming when streaming is started from Dashboard with multi-source ON and no Source ID

@vishalkharge we need some steps to reproduce this

Apparently, this bug has been also fixed previously.
cc @vishalkharge


Resize the disk to 35GB from 31 GB

As of now we are resizing the disk to 31GB but metion 35GB in our docs, better to increase since from 4.2 side 11 GB space already filled by container images.

31GB is the maximum VM disk size, but initially the qcow2 file does not occupy that much space as qcow2 is thinly allocated. However, we have additional disk space requirements, we need space for the binary (2GB), the extracted bundle (2GB), the unpacked bundle (11GB), and then the VM disk itself (11GB). This gives us 26GB, so documenting 35GB leaves us with some flexibility for the VM image to grow.

Disk resize is now part of our crc codebase so  whatever we have currently is good enough.


Move CV information to Hugo's data dir for livereload to work on CV info

By default Hugo will watch for changes when running hugo server. The configuration (e.g. config.toml) is not watched and as a result changing CV information in the config file will not get livereloaded. This commit fixes that.
As data seems to not be read from theme directories, this does however introduce the need for people testing this theme to copy the exampleSite/data/cv.toml to hugoroot/data for test-running the theme. This instruction has been added to README.md.

LiveReload not working was actually caused by params.languages in config.toml being a reserved variable by Hugo. This PR is obsolete.


🆕 Software Suggestion | Passit

Basic Information
Name: Passit
Category: Password Manager
URL: https://passit.io
GIT: https://gitlab.com/passit
Description
Passit is an open-source, cloud-based password manager. It was built by people who love digital privacy and good design, and weren't satisfied with existing options. In every decision we make, we strive to strike a good balance between features, ease-of-use, security, and privacy. Passit uses the modern and battle-tested libsodium cryptography package to encrypt data in such a way that even we can't access.

A version for android is available but not yet published in f-droid
Security informations
https://gitlab.com/passit/passit/-/wikis/security
http://passit.io/security-model.html
Active since 2016

Why I am making the suggestion
This is a very simple password manager that uses libsodium (already audited - 2017) for the encryption of keys and password hash. With extensions for Firefox and Chromium. Totally opensource, it can be self-hosted but also provides a free account with an option to share passwords with others.
My connection with the software
Only one user who loved the simplicity of this project xD

[x] I will keep the issue up-to-date if something I have said changes or I remember a connection with the software.

Libsodium is audited but I can't find any audit on Passit. I think this should be required before listing a password manager. Project looks promising.

I think so, too. But there are already some listed password managers that haven't been audited yet.

Libsodium is audited but I can't find any audit on Passit. I think this should be required before listing a password manager. Project looks promising.

KeePass has undergone an audit, but not the popular KeePassXC, or mobile clients as far as I'm aware. While audits are important, it's also worth noting that it costs time and effort to do so, and if we had to audit every application before considering it recommended we'd get no where.
With that said, this issue should be closed. It doesn't have anything than can top BitWarden, nor KeePass clients.

Although Passit doesn't have exceptionally innovative features and indeed is in competition with BitWarden, it doesn't preclude that it is an alternative that is still feature-rich from what I can see. It also has a few different features, such as the ability to import/export passwords as CSV files, which can be interesting for certain use cases.
I think it should be added as a Worth Mentioning, but not as a recommendation (since it's not audited and younger).


ARROW-3829: [Python] add arrow_array protocol to support third-party array classes in conversion to Arrow

https://issues.apache.org/jira/browse/ARROW-3829 & https://issues.apache.org/jira/browse/ARROW-5271. And as illustration for the mailing list discussion (will post there in a bit).
TODO: compatibility for older pandas versions (where ExtensionArray/IntegerArray is not yet available)

Question: if the Pandas series or dataframe column is backed by an Arrow array, does this allow pa.array() to extract the array without copying?

Yes, that would be the idea (also pa.array(numpy_array) does not create a copy of the data if possible). In that case the __arrow_array__ should return that backing Arrow array (which is what fletcher could do).

Updated the PR and added some docs for it.

Looks good @jorisvandenbossche! LGTM.
I'm looking to add support for pd.SparseArray, so setting __arrow_array__ to something like:
def __arrow_array__(self, type=None):
    return pa.SparseTensorCOO.from_numpy(data=self.sp_values, coords=self.sp_index.indices.reshape(1,-1), shape=(self.shape))

However the returned type would then not be pa.array. Do you think it would sense to support this case with __arrow_array__ interface?

@rok at the moment, the protocol is specifically meant to convert to a pyarrow.Array (it is also only used internally in the pa.array(..) code).
So I would personally keep it on that, for now. The question is also what would be the exact purpose for extending it? (but I am not very familiar with the tensor part of the library / the message spec).

@jorisvandenbossche agreed with keeping this as is.
My motivation is enabling conversion of sparse columns (pa.SparseArray) contained in and arbitrary pd.DataFrame to pyarrow type (probably pa.SparseTensorCOO).
I'm not sure what would the best place to do this be?

My motivation is enabling conversion of sparse columns (pd.SparseArray) contained in an arbitrary pd.DataFrame to pyarrow type (probably pa.SparseTensorCOO).

The main problem is that currently sparse data does not fit in the Arrow tabular format, so in general DataFrames with such sparse columns, can't be converted to Arrow tables. As you mention, the target could be pd.SparseTensorCOO, but currently the main target of this protocol is the recordbatch message format, and not the (sparse) tensor message format.
For the Tensors, I don't think there is currently a "catch all" constructor like you have for pa.array? That might be a start if you want to enable conversion of general objects to (sparse) tensors. But I would propose to open a new issue for that if you want to continue the discussion.

Thanks @jorisvandenbossche, I'll look into that direction and here's an issue I opened for it.


Question: Blazor NavigationManager.LocationChanged Async

In general what would be the best method (if any) to call awaitable methods from NavigationManager.LocationChanged, since NavigationManager.LocationChanged is not async?
Is wrapping the awaitable Task call into Task.Run/Task.WaitAll acceptable? For example something like:
var task = Task<bool>.Run(async () =>
{
   //Call await method here
    return false;
});

Task.WaitAll(task);
var retVal = task.Result;

You can declare your handler method as async Task. DotNet won't await the result, but your method will be able to await other calls.
    class Program
    {
        static void Main(string[] args)
        {
            Console.WriteLine("Hello World!");

            var p = new Parent();
            p.SomeEvent += (s, e) => Console.WriteLine("Observer 1");
            p.SomeEvent += AsyncHandler;
            p.SomeEvent += (s, e) => Console.WriteLine("Observer 2");

            p.TriggerSomeEvent();
            Console.ReadLine();
        }

        public static async void AsyncHandler(object sender, EventArgs e)
        {
            await Task.Delay(1000);
            Console.WriteLine("Async observer");
        }
    }

    public class Parent
    {
        public event EventHandler<EventArgs> SomeEvent;

        public void TriggerSomeEvent()
        {
            SomeEvent?.Invoke(this, EventArgs.Empty);
        }
    }

Output

Hello world!

@chassq thanks for contacting us.
There are patterns around this. The best option you have is to encapsulate everything with in a task returning method that takes in a cancellation token as a parameter. Then register a handle in your LocationChange method and do something like this
if(_currentTask != null && !_currentTask.IsCompleted){
   _currentCancellation.Cancel();
}
_ currentCancellationToken = new ...
_currentTask = InvokeMyAsyncHandler(_currentCancellationToken)

In general, do not use async void, and instead capture the existing task within your component and use the cancellation token to stop the work if for example another navigation gets triggered in between.
I hope this helps.

Thank you very much!


Selector not working despite matching formats being available

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

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

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2023.03.04 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
Despite the format being available, the selector string is not working, I've tried many combinations and nothing will work properly.
$ yt-dlp -f 'video[ext=mp4]' https://ok.ru/video/2409725102846 --list-formats
[Odnoklassniki] Extracting URL: https://ok.ru/video/2409725102846
[Odnoklassniki]<PHONE_NUMBER>846: Downloading desktop webpage
[Odnoklassniki]<PHONE_NUMBER>846: Downloading m3u8 information
[info] Available formats for<PHONE_NUMBER>846:
ID       EXT RESOLUTION FPS │   FILESIZE   TBR PROTO │ VCODEC        VBR ACODEC    ABR ASR MORE INFO
───────────────────────────────────────────────────────────────────────────────────────────────────────────────
mobile   mp4 unknown        │                  https │ unknown           unknown
hls-189  mp4 256x144     24 │ ~175.34MiB  189k m3u8  │ unknown      189k unknown    0k
mpd-0    mp4 256x144     24 │ ~ 97.03MiB  105k https │ avc1.64000C  105k mp4a.40.2  0k 44k DASH video, mp4_dash
lowest   mp4 unknown        │                  https │ unknown           unknown
hls-373  mp4 426x240     24 │ ~346.25MiB  374k m3u8  │ unknown      374k unknown    0k
mpd-1    mp4 426x240     24 │ ~275.21MiB  297k https │ avc1.640015  297k mp4a.40.2  0k 44k DASH video, mp4_dash
low      mp4 unknown        │                  https │ unknown           unknown
hls-752  mp4 640x360     24 │ ~696.99MiB  753k m3u8  │ unknown      753k unknown    0k
mpd-2    mp4 640x360     24 │ ~579.34MiB  626k https │ avc1.64001E  626k mp4a.40.2  0k 44k DASH video, mp4_dash
sd       mp4 unknown        │                  https │ unknown           unknown
hls-1248 mp4 852x480     24 │ ~  1.13GiB 1248k m3u8  │ unknown     1248k unknown    0k
mpd-3    mp4 852x480     24 │ ~  1.01GiB 1115k https │ avc1.64001E 1115k mp4a.40.2  0k 44k DASH video, mp4_dash
hd       mp4 unknown        │                  https │ unknown           unknown
hls-2628 mp4 1280x720    24 │ ~  2.38GiB 2628k m3u8  │ unknown     2628k unknown    0k
mpd-4    mp4 1280x720    24 │ ~  2.24GiB 2478k https │ avc1.64001F 2478k mp4a.40.2  0k 44k DASH video, mp4_dash
full     mp4 unknown        │                  https │ unknown           unknown
hls-4461 mp4 1920x1080   24 │ ~  4.04GiB 4462k m3u8  │ unknown     4462k unknown    0k
mpd-5    mp4 1920x1080   24 │ ~  3.88GiB 4288k https │ avc1.640028 4288k mp4a.40.2  0k 44k DASH video, mp4_dash

Provide verbose output that clearly demonstrates the problem

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

Complete Verbose Output
$ yt-dlp -vU -f 'video[ext=mp4]' https://ok.ru/video/2409725102846
[debug] Command-line config: ['-vU', '-f', 'video[ext=mp4]', 'https://ok.ru/video/2409725102846']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[392389b7d] (zip)
[debug] Python 3.9.0 (CPython x86_64 64bit) - macOS-10.16-x86_64-i386-64bit (OpenSSL 1.1.1t  7 Feb 2023)
[debug] exe versions: ffmpeg 5.1.2 (setts), ffprobe 5.1.2, rtmpdump 2.4
[debug] Optional libraries: certifi-2022.12.07, no_Cryptodome-None, sqlite3-2.6.0
[debug] Proxy map: {}
[debug] Loaded 1786 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>Current Build Hash: 91cad9f121c1f6f0a81b747415c46ecba0ff331ed38cc6433040b4ac7b6e15ca
yt-dlp is up to date<EMAIL_ADDRESS>[Odnoklassniki] Extracting URL: https://ok.ru/video/2409725102846
[Odnoklassniki]<PHONE_NUMBER>846: Downloading desktop webpage
[Odnoklassniki]<PHONE_NUMBER>846: Downloading m3u8 information
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
ERROR: [Odnoklassniki]<PHONE_NUMBER>846: Requested format is not available. Use --list-formats for a list of available formats
Traceback (most recent call last):
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 1518, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 1615, in __extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 1674, in process_ie_result
    ie_result = self.process_video_result(ie_result, download=download)
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 2743, in process_video_result
    raise ExtractorError(
yt_dlp.utils.ExtractorError: [Odnoklassniki]<PHONE_NUMBER>846: Requested format is not available. Use --list-formats for a list of available formats

For this it's better to use -S ext IMO
yt-dlp -S ext "https://ok.ru/video/2409725102846"
this will sort mp4 formats in front of the rest
otherwise you would want to use -f "bestvideo[ext=mp4]*+bestaudio[ext=m4a]/best[ext=mp4]"

otherwise you would want to use -f "bestvideo[ext=mp4]*+bestaudio[ext=m4a]/best[ext=mp4]"

you beat me to it. Although -f [ext=mp4] would probably also work

The problem is that none of the selectors work, I just chose ext=mp4 because it's the most obvious.
$ yt-dlp -S ext -f 'video[height<480]' "https://ok.ru/video/2409725102846"
[Odnoklassniki] Extracting URL: https://ok.ru/video/2409725102846
[Odnoklassniki]<PHONE_NUMBER>846: Downloading desktop webpage
[Odnoklassniki]<PHONE_NUMBER>846: Downloading m3u8 information
ERROR: [Odnoklassniki]<PHONE_NUMBER>846: Requested format is not available. Use --list-formats for a list of available formats

$ yt-dlp -S ext -f 'video[height>480]' "https://ok.ru/video/2409725102846"
[Odnoklassniki] Extracting URL: https://ok.ru/video/2409725102846
[Odnoklassniki]<PHONE_NUMBER>846: Downloading desktop webpage
[Odnoklassniki]<PHONE_NUMBER>846: Downloading m3u8 information
ERROR: [Odnoklassniki]<PHONE_NUMBER>846: Requested format is not available. Use --list-formats for a list of available formats

-f video is not valid syntax
https://github.com/yt-dlp/yt-dlp#format-selection

$ yt-dlp -S ext -f 'bestvideo[height>480]' "https://ok.ru/video/2409725102846"
[Odnoklassniki] Extracting URL: https://ok.ru/video/2409725102846
[Odnoklassniki]<PHONE_NUMBER>846: Downloading desktop webpage
[Odnoklassniki]<PHONE_NUMBER>846: Downloading m3u8 information
ERROR: [Odnoklassniki]<PHONE_NUMBER>846: Requested format is not available. Use --list-formats for a list of available formats

$ yt-dlp -S ext -f 'bestvideo[height<480]' "https://ok.ru/video/2409725102846"
[Odnoklassniki] Extracting URL: https://ok.ru/video/2409725102846
[Odnoklassniki]<PHONE_NUMBER>846: Downloading desktop webpage
[Odnoklassniki]<PHONE_NUMBER>846: Downloading m3u8 information
ERROR: [Odnoklassniki]<PHONE_NUMBER>846: Requested format is not available. Use --list-formats for a list of available formats

all available formats are video+audio combined, so bestvideo will not select any of them
use best[height>480] or bestvideo[height>480]*

Thanks a lot, it was an IO error after all (idiot operator), appreciate the help


No longer able to process .csv files generated by carbon black

zpub_030819.txt
This .csv file now generates the error:
[+] Found results CSV from previous run: check-results_zpub_030819.csv
[+] Appending results to file: check-results_zpub_030819.csv
[+] Processing 562 lines ...
Traceback (most recent call last):
File "munin.py", line 1528, in
processLines(lines, resultFile, args.nocsv, args.debug)
File "munin.py", line 245, in processLines
info, cooldown_time = processLine(line, debug)
TypeError: cannot unpack non-iterable NoneType object
I tried stripping the "" marks from the doc but  nothing is working.

Latest update fixed the issue.


Use the latest ibc-go and interchain-security modules - Notional & CryptoCrew

This PR updates ibc-go to v4.4.0 and interchain-security to the latest commit from the main branch.
The goal is to comply with our mutually adopted new chain launch standards (to be updated "soonTM" with consumer chain launch standards.)
Since state-breaking upgrades are much more challenging than genesis, we recommend that genesis include these changes, after conferring with Informal systems about weather or not 0a000ff can be a proper release tag, so that Neutron's go.mod file will be somewhat cleaner than it is in this PR.

#176

Should most likely be merged before launch, after speaking with Informal Systems


Add prefix format string as config parameter

Hi!
Great plugin! I had the need to customise the commit comment prefix so I added the format string as a configuration parameter. It also meets the need of gesundes in issue #3 . Please merge if you find it useful.
Best regards
almegren

Hi @almegren.
Can you give me an example of what are you trying to achieve? I ran your code and it wasn't working as expected and it was hard for me to figure out your intention from the code itself.
I can illustrate my understanding with an example:
Let's say your JIRA project is called BOT and you have a branch feature/BOT-123...

Is my understanding correct?
Thanks

Hi @almegren.
Can you give me an example of what are you trying to achieve? I ran your code and it wasn't working as expected and it was hard for me to figure out your intention from the code itself.
I can illustrate my understanding with an example:
Let's say your JIRA project is called BOT and you have a branch feature/BOT-123...

Is my understanding correct?
Thanks

Hi
Yes of course, I should have given some examples. So if you have branch called feature/BOT-3452 or BOT-3452 you might want the beginning of the commit message to either (BOT-3452), [BOT-3452], BOT-3452: , BOT-3452 : , BOT-3452 , BOT-3452 -  or something completely different. I see that you've added the possibility to choose bracket type, my suggestion is simply to make this option more general. In my team now we use my fork and has set "Commit prefix format" to %s:  which gives us commit messages starting with BOT-3452:  for the example above.

Hi
Yes of course, I should have given some examples. So if you have branch called feature/BOT-3452 or BOT-3452 you might want the beginning of the commit message to either (BOT-3452), [BOT-3452], BOT-3452: , BOT-3452 : , BOT-3452 , BOT-3452 -  or something completely different. I see that you've added the possibility to choose bracket type, my suggestion is simply to make this option more general. In my team now we use my fork and has set "Commit prefix format" to %s:  which gives us commit messages starting with BOT-3452:  for the example above.

Okay, clear now.
My main intention for this plugin was to make it super simple with maybe 2-3 config parameters and I wanted to avoid people having to write a regex or pass in some template placeholders e.g. %s. This is something that was lacking with other similar plugins.
Implementing this feature would break that.
It looks that the functionality you are looking for is already available in another plugin, maybe something that you can check out?
https://plugins.jetbrains.com/plugin/14762-git-commit-message-template
Thanks

Okay, clear now.
My main intention for this plugin was to make it super simple with maybe 2-3 config parameters and I wanted to avoid people having to write a regex or pass in some template placeholders e.g. %s. This is something that was lacking with other similar plugins.
Implementing this feature would break that.
It looks that the functionality you are looking for is already available in another plugin, maybe something that you can check out?
https://plugins.jetbrains.com/plugin/14762-git-commit-message-template
Thanks


v3: Don't use reflection to generate stuff

Fixes #2105
Here's a similar issue someone else had: https://github.com/com-lihaoyi/mill/issues/2985. I tried using the fix given there, but it seems to refer to stuff we don't have, or I guess I didn't understand it? Anyway, I gave up on that.
Then I did some reflection, after which I decided I didn't much like reflection. So I made it so that GenerateDocs and friends are main methods that can be run from build.sc. Also, I used T {} instead of T.sources, which is probably unidiomatic but I think we're already not using T.sources the way it's supposed to be used (I think we're supposed to put files in the directory Mill tells us to put them).
In the process, I merged GenerateTheseusData and GenerateKeyboard into GenerateTheseus

@lyxal Yes, I did just push a commit that makes the GenerateTheseus.scala more idiomatic, but I didn't change the actual logic at all

Oh, I think the diff may be because I used a HashMap instead of a mutable.Map in one place


Add 3 Leet Code java Array program on DSA/Array folder

i will do that .

okay do it

only problem right ??

Problem Statements and their code(solutions)

May I contribute to this?

May I contribute to this?

Yeah

May I contribute to this?

Yes you can

Assign this issue to me!!


Package for F-Droid

F-Droid is an app "store" for free/libre/open-source apps. We received a packaging request  at  https://gitlab.com/fdroid/rfp/issues/156 and would like to proceed with this. If you object publication, please let us know and we will remove this from our repository.

It's ok with me, you can publish it.

Thanks for the quick response.

coming from https://gitlab.com/fdroid/rfp/-/issues/156#note_30268270 - commenting to save the description for myself.


Archive repo?

This project is clearly abandoned in favour of renderdoc, the readme should probably be updated to reflect this fact and the repository should be archived.

Sorry, I really don't know how to word it. I'm not involved with the development of vogl or renderdoc at all, I'm just a user. As a Valve employee you probably know how to put it best.


Ubuntu plugin improvements

Update README for better documentation.
Added safe-upgrade-clean and agdu aliases.
Added Spotify control commands and it's respective documentation.

The README is better now, and the spotify function looks like it belongs to a different plugin.


if statement is crazy

Another crazyness related to if statement.
Suppose we have an optional array:
$ar = (1, 2, 3)

And, suppose that we would like to create some default array if $ar is empty (yep, we'll try to emulate null-coalescing operator from C#):
$ar1 = if ($ar) {$ar} else {(1, 2)}

Ok, looks good, but what if we would like not only "coalesce" an array, but also to convert a result to, for instance, array of strings:
$ar2 = [string[]](if ($ar) {$ar} else {(1, 2)})

Unfortunately, this code fails with an error:
if : The term 'if' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

There is two ways to solve this problem.
Use two different statements:
$tmp = if ($ar) {$ar} else {@(1, 2, 3)}
$ar2 = [string[]]($tmp)

Or by introducing temporary variable inside if statement:
$ar3 = [string[]]($tmp = if ($ar) {$ar} else {@(1, 2, 3)})

Again, I do think this is yet another bug rather than a trap!

It is probably not a bug but a trap, indeed. I think it has the same roots as Cannot-pipe-some-expressions. It's more likely that for, foreach, and other assignable statements work (do not work) in the same way. The workaround is use of $(...), not just (...).
This case probably should be mentioned, too. I'll think of this.

Hm... I don't see any pipes in my case... Maybe this is related, but this behavior is strange it terms of what PowerShell treats as an expression. For instance, why $(if ..) is behave differently from ($tmp = if ..)? Maybe there is some rationale behind this and in this case it is a trap.
On the other hand, subexpressions for $(if..) sounds like a workaround for a bug, because by definition subexpression for one expression should do nothing.

if, and others with keywords, are statements, not expressions. In v1 they could not be even assigned. In v2+ they can be assigned. This is probably a good feature. With a cost, as usual. It created an illusion that statements are expressions, as far as they can be assigned. Pipeline and redirection described in the trap are two cases of this fact. Your case is another one.
I think we should rename Cannot-pipe-some-expressions to something like "Statements-are-not-expressions" and add more examples.

() is for expressions only. But if is not expression. So it's "fine" that () does not work. It's not fine that a user may not expect this.
$() is for any code, even for 2+ statements, not just one with if.

Ok, now I got it! I thought that PowerShell is expression-based language because you can assign $foo = if {...}. In any other programming language this means that if is an expression and can be used in any other place where expression is used.
Yep, proposed rename sounds good for me. It will sounds like a general trap rather than something particular.

Renamed the case Cannot-pipe-some-expressions to Statements-are-not-expressions.  Reviewed its content and added some new facts.


Addressing CVE-2022-40897 Vulnerability in Python 3.11.9-alpine3.19 Docker Image

We have identified that our Docker image is affected by the CVE-2022-40897 vulnerability. This vulnerability allows remote attackers to execute arbitrary code or cause of DOS via a crafted image file, leading to potential system compromise or disruption.
Could you please provide an estimated timeline for fixing this issue? Additionally, any guidance on how to address this vulnerability effectively would be highly appreciated.? We would like to ensure that our system remains secure and up-to-date.
Thank you.

We explicitly already updated setuptools for this CVE in python 3.10, 3.11, and 3.12 images (https://github.com/docker-library/python/issues/781 ->https://github.com/docker-library/python/pull/783), so the scanner is incorrect.

This vulnerability allows remote attackers to execute arbitrary code or cause of DOS via a crafted image file, ...

Uhhhh, what?  If this is exploitable in your application, that would mean you're importing setuptools into your application, which seems highly unusual.  The only other alternative I can think of is that you're perhaps running untrusted images, which is already "arbitrary code" so I fail to see how this increases the surface area of attack in any appreciable way?


Duplicates

Hi!
Every time I press Save in admin I have 2 new color (#000). It doesn't depend if I change field with color, just click Save ... Why?
My model:
class Color(models.Model):
    color = RGBColorField(verbose_name=u'Color')
    ware = models.ForeignKey(Ware, verbose_name=u"Ware", related_name="colors")

My admin:
class ColorInline(admin.TabularInline):
    model = Color
    extra = 2
    max_num = 10

@admin.register(Ware)
class WareAdmin(admin.ModelAdmin):
    list_display  = ('id','title',)
    inlines = [ColorInline,]

Does your browser use the native color input or the JS fallback?

Hi Simon,
I think my problem is a bit different, but let my try to explain better. I'm writing a shift plan application. As my Department has three teams we would like to colour the rows according to the team (it will be one big table for the department and user from different teams will have a different colour of the raw... lets say team 1 - blue, team 2 - green and team 3 white). The shifts will also have their own colours (morning - green, evening - blue but we would like to leave the colour of the normal shift the colour of the Department). In my case everything is working correctly but it is not possible to leave the field empty as the default colour is always set to default #000000. Here is how my class looks:

class Shift(models.Model):
def __unicode__(self):
    return self.shift
shift = models.CharField(max_length=20)
shift_code = models.CharField(max_length=5)
shift_bg_color = RGBColorField(null=True, verbose_name="Background Color")
shift_txt_color = RGBColorField(null=True, blank=True, verbose_name="Text Color")
shift_start_time = models.DecimalField(choices=TIME_CHOICE, null=True, max_digits=5, decimal_places=2)
shift_end_time = models.DecimalField(choices=TIME_CHOICE, null=True, max_digits=5, decimal_places=2)
lunch_start_time = models.DecimalField(choices=TIME_CHOICE, null=True, blank=True, max_digits=5, decimal_places=2)
lunch_end_time = models.DecimalField(choices=TIME_CHOICE, null=True, blank=True, max_digits=5, decimal_places=2)
comments = models.CharField(max_length=20, blank=True, null=True)
department = models.ForeignKey(Department)

What I'm trying to achieve is that while creating the shift default value of #000000 will not be passed to the database... I thought that setting blank=True will fix the problem but when adding the new shift the default black colour is there.
Is there a possibility to not put the default colour and leave the field empty?
Thank you very much for your help.
Kind regards,
Tomasz

Hi Tomasz,
You have the exact same issue, you want to have a blank field and not one that defaults to #000000.
Does your browser use the native color input or the JS fallback?

For my test I use firefox 31.4.0 in CentOS VM. Sorry for asking dummy questions but how can I check it?

Don't worry, if you're using such an old version of FF you must be using the JS fallback.

Is it not possible to have it working with JS Fallback? Only Native Color Input will work?

@kraiz, it looks like the update of jquery.colorPicker made empty fields turn into #00000. Do you have an idea of what might be happening?

@hplodur, it should be working. I'm trying to figure out why it's not.

Hey...joining the conversation. I had a look at this and i think the problem is not the fallback as the screenshot above actually shows a native input which FF supports from 29+.
It seems like you can't really have an empty color input value as the spec omits any word on it and leaves this to the browsers.
<input type="color">

Chrome will give you "#000000" (requested in JS and submitted via POST). In IE11 it's an empty string (it doesn't support color input and handle the field as normal text input). I would guess FF follow Chrome on this.
The JS fallback just requests the current value and writes it to the input. Through the effect above this explicitly writes "#000000" into the input field for Chrome and FF, but "#FFFFFF" for IE as this is the colorpicker-library-default.
All in all: Seems like we can't have an empty value the easy way and may need to work around here in some clever way. But anyway i feel we should provide a browser independend default value.
Am i right on this @charettes? What do you think?

Thanks for the investigation @kraiz, greatly appreciated.
What do you think of allowing the users to force the use of a type="text" with the fallback on a widget an automatically do it when dealing with a non-required forms.ColorField. Are we even sure the fallback widget doesn't behave the same way and the native one on Chrome/FF?

Wouldn't vouch for that :) Needs to be tested. The current fallback-js-library does not support empty values either.
Native vs. fallback-enforced input in case of non-required field needs to be documented in a kind of highlighted way. I would not expect that the fancy color chooser (i love it in android chrome) disappears when i add blank=True to the model field. But this way we're able to support the basic django feature of optional fields independent of browsers implementation.
I think about that and maybe have an experiment with spectrum. It has a polyfil-mode (for required color field) and an "allowEmpty" option (for non-required color fields). It maybe saves some work on our side.

@kraiz, spectrum looks like an interesting candidate. I wouldn't mind switching to it and documenting it will be automatically used if you specify your field must be blank.

Yeah, i had a look some time ago and just want to keep you updated. I found out that spectrum is missing datalist support, which would be great for the polyfill usecase. So i added it but the PR ( https://github.com/bgrins/spectrum/pull/368 ) is hanging around unseen.
Maybe it's not to bad as we can workaround by instanciating the library (setting the preset palette as parameter) as it's done with current colorpicker but with the PR it would be way cleaner.
Anyway my time is limitted at the moment and this changes by the end of the year. When is PR is not merged until then, i'll have a look at the workaround.

I'm not sure if this is related, but running Ubuntu 14.04 both Chrome 50 and FF 46 do not render the HTML 5 color input and instead use the shim (which does not accept custom Hex input).

@cjmochrie I'm running Ubuntu 14.04 and the HTML 5 native color input is used in both cases. I just tested with Django 1.9.6 and django-colorful 1.2 on Firefox 46 and Chromium 49.0.2623.108.

@charettes  Thanks for the quick reply!
I found the issue (not colorful). It's a conflict with django-grapelli. Found this in their docs:
"GRAPPELLI_CLEAN_INPUT_TYPES Replaces HTML5 input types (search, email, url, tel, number, range, date, month, week, time, datetime, datetime-local, color) due to browser inconsistencies. Set to False in order to not replace the mentioned input types."
After setting that to False, the HTML5 input rendered as expected. This seems like an issue that might trip somebody else up as Grappelli is pretty popular. I could make a PR to the README with a note about this if you'd like?

Sure, thanks for the investigation!


possible to detect remote peers that are no longer active?

after building some tests based on the HelloWorld examples included in the Fast-RTPS distribution, I see that when I cleanly exit a process and let all the destructors run, the appropriate onPublicationMatched() callbacks are invoked for the PublisherListener whose remote peer has shut down.
however, if I simply terminate the process (ctrl-c), the publisher never gets notified that the subscriber is no longer listening.
is there a recommended approach to address this?

https://github.com/eProsima/Fast-RTPS/issues/35 might be related.


Refactor full page examples

The application main file was getting a bit big, likewise the tests for the app.
I've split out the new functionality into it's own file which makes it easier to delete and refactor more in the future.

Closing in favour of https://github.com/alphagov/govuk-frontend/pull/1198 which moves files better to preserve history


Remove deprecated properties from AbstractArchiveTask

Checklist

[x] Validate whether we should remove the API in the 7.0 release

If the removal is not possible then create a follow-up issue and link it in the overview spreadsheet

[x] Update the upgrade guide
[x] Check User Manual for dangling references and outdated documentation
[ ] Mark the story as done in the overview spreadsheet.

@bot-gradle test rFM

Sorry some internal error occurs, please contact the administrator @blindpirate

@bot-gradle test rFM

Sorry some internal error occurs, please contact the administrator @blindpirate

@JLLeitschuh @big-guy IIUC with the archivePath property gone, this is no longer an issue: https://github.com/gradle/gradle/issues/10619. Can you please confirm that?

@donat I think so.
It looks like the build failures are related to the Kotlin plugin. We should see what's the oldest version of Gradle they need to support and if we have the new methods available that far back.
Otherwise, we may need to keep these in for another cycle or mix them in at runtime.

I've talked with the JetBrains folks and the breakage is fine on their side as long as we add a reasonable error message (something like if(oldKotlinPlugingApplied()) throw new BuildException("Use new version of Kotlin").

Superseded by https://github.com/gradle/gradle/pull/15769.

Superseded by https://github.com/gradle/gradle/pull/15769.


Enhance Plotting Functionality to Differentiate Bodies by Color

📝 Description of the feature
capability to display different bodies with assorted colors when plotting a component.
💡 Steps for implementing the feature
No response
🔗 Useful links and references
No response

That is implemented in fact - but you would have to make use of the GeometryPlotter object directly. You mean doing this directly when requesting to plot a design or component right?

Okay so we need to implement the color cycler shown here https://github.com/ansys/pyansys-geometry/issues/368
Shouldn't be hard honestly. The color cycler should be an optional flag that a user can decide whether to activate or not when requesting design.plot(), component.plot(), etc.

Transferring to ansys-tools-visualization-interface since this functionality will be exposed there. I will add an example on this repo to show how to do it.


(bug) config file for basic demo ?

Observed behavior
no config in https://github.com/thomaspoignant/go-feature-flag/tree/main/examples/demo
flags.yaml.
Expected Behavior
a config.
Steps to reproduce
No response

Good catch @gedw99, let me add one to make the demo usable.

@gedw99 I have merge to main branch the fix.
It should be now possible to use the demo with the right configuration file.

thanks @thomaspoignant


Request for Access to Recipe1M+ Dataset

Greetings,
I hope this message finds you well. My name is Mahsa Keshtkarsohi, and I wanted to do a project in this field as a final project in Lewagon BootCamp.
I understand the importance of data privacy and academic integrity, and I assure you that the dataset will be used exclusively for academic purposes. Any findings or results derived from the dataset will be appropriately cited and acknowledged in my research publications.
Unfortunately, I have encountered issues downloading the dataset. Therefore, I kindly request if you could provide access to the dataset via email. My email address is<EMAIL_ADDRESS>If there are any terms or conditions associated with the use of the dataset, please let me know, and I will ensure full compliance.
Thank you very much for considering my request. I look forward to your favorable response.
Best regards,
Mahsa Keshtkarsohi
Lewagon Bootcamp, Netherlands

@mhsksh were you able to get get access to the dataset?

@mhsksh were you able to get get access to the dataset?

Hi, I hope you are doing well.
No, unfortunately I wasn't able to access to the dataset, also I checked again and I cannot get an access to it


Scroll to top when reselecting tab

Overview (Required)

Scroll to top when you tap current tab.
and tiny refactor

Screenshot

AllSessionsFragment

SearchSpeakersFragment

:eyes:

I want to comment this 😭
Sorry for late ...

@rkowase
Thank you for review!
@takahirom
Could you tell me the comment?
I will try PR again if possible...

Sorry, I currently work for the first release. Please wait 🙏

Ok, If it is a serious problem, please revert this PR 🙏

For your reference.
I think fragments were leaked. But probably I think most of the behavior will not be affected. 👍
@jmatsu fixed it
https://github.com/DroidKaigi/conference-app-2018/pull/493#pullrequestreview-93832784


Added ability to set instance name in locator

This should allow setting up instanceName in the locator for moredetails take a look at ()

@FilledStacks I will need your guidance to finish it. My app is not allowing me to use two plugins that depend one on another and they are both local it is requiring me for at least 1 plugin to be hosted, otherwise, it does not compile it, just throws an error during the build. How can I simultaneously test "Stacked" plugin changes and "stacked_generator" changes? Stacked changes are required because of the annotation, the generator uses the annotation. Can you please give me some guidance/link to some tutorial on how to test two plugins dependent one on another (generator depends on the stacked plugin)

This has been added to the generator with the newest updates to add unit tests.

Closes #584


Balance: Molten solder

Mod source
Factorio Mod Portal
Which mod are you having an issue with?

[X] pyalienlife
[X] pyalternativeenergy
[X] pycoalprocessing
[X] pyfusionenergy
[X] pyhightech
[X] pyindustry
[X] pypetroleumhandling
[X] pypostprocessing
[X] pyrawores

Operating system

=Windows 10

What kind of issue is this?

[ ] Compatibility
[ ] Locale (names, descriptions, unknown keys)
[ ] Graphical
[ ] Crash
[ ] Progression
[X] Balance
[ ] Pypostprocessing failure
[ ] Other

What is the problem?
2 recipe from Solder 2 tech:

Direct casting cost much more (2x+ for molten lead&tin variant) then using plates, don't think its correct. Even for non hot air recipe:

Steps to reproduce
No response
Additional context
No response
Log file
No response

New rates, idk but its weird. Basic recipe now better then molten from plates.

Seems like this was fixed in dev.
mk01 recipe with plates (hot air) consumes 15.8 molten tin/31.7 molten lead/min
https://i.imgur.com/YnVXN6z.png
mk02 recipe with casting (hot air) consumes 8.9 molten tin/17.8 molten lead/min
https://i.imgur.com/glSt2BT.png
Going to mark this as resolved. Thanks for the report. Will be fixed next update.


Implement basic commands via UART

Add cw, ccw, stop, brake, up and down commands

@CodiumAI-Agent /review

PR Analysis

🎯 Main theme: Implementing basic commands via UART for a BLDC motor
📌 Type of PR: Enhancement
✨ Focused PR: Yes, all changes are related to the implementation of UART commands for the motor.
🔒 Security concerns: No security concerns found

PR Feedback

General suggestions: The PR seems to be well-structured and focused on implementing UART commands for the motor. However, it lacks tests to ensure the new functionality works as expected. It would be beneficial to add unit tests for the new functions and integration tests for the overall functionality.

🤖 Code feedback:

relevant file: mcu/include/bldc/motor.hpp
suggestion: The run, brake, and stop methods in the bldc_motor class have similar code structures. Consider refactoring these methods to reduce code duplication. You could create a private method that takes the direction and enable status as parameters and use it in these methods. [important]
relevant line: void run(rotation_direction_t dir)

relevant file: mcu/src/main.cpp
suggestion: The command_parser class has a direct dependency on the bldc_motor class. This tightly couples these two classes and makes the code less flexible and harder to test. Consider using dependency injection to pass the bldc_motor instance to the command_parser class. [medium]
relevant line: bldc::bldc_motor_handle       motor_;

relevant file: mcu/src/main.cpp
suggestion: The command_parser class is doing too much work in the command_done method. It's parsing the command, executing it, and handling unknown commands. Consider breaking this method into smaller methods to improve readability and maintainability. [medium]
relevant line: void command_done(armpp::hal::uart::uart_handle& device)

How to use

Tag me in a comment '@CodiumAI-Agent' and add one of the following commands:
/review [-i]: Request a review of your Pull Request. For an incremental review, which only considers changes since the last review, include the '-i' option.
/describe: Modify the PR title and description based on the contents of the PR.
/improve: Suggest improvements to the code in the PR.
/ask <QUESTION>: Pose a question about the PR.

To edit any configuration parameter from 'configuration.toml', add --config_path=new_value
For example: /review --pr_reviewer.extra_instructions="focus on the file: ..."
To list the possible configuration parameters, use the /config command.


Fix datapackage generation from multiple taxonomies

Overview
This PR fixes datapackage generation when using multiple taxonomies to extract XBRL filings. This fixes a metadata issue in the on-going 2023 FERC integration. It does this by merging multiple XBRL taxonomies into a single datapackage before parsing filings. This datapackage is used to structure the output DB and interpret filings during extraction.

Possibly out-of-scope: Are you expecting the datapackages to be valid frictionless datapackages (a la https://github.com/catalyst-cooperative/ferc-xbrl-extractor/issues/153)? I'm seeing tons of errors when I try to validate the one I generated. Again, just let me know if this one is out of scope.

Hmm how were you checking the validation? We have an integration test, and validate the datapackage before writing to a json file.

One bigger concern: I think that this update (or one before, or many before...) breaks the sample extraction code in the README. Which means that our instruction docs are now incorrect. We should definitely fix this. If it's quick to address, I'd prefer to do it here. If not, we should open an issue for this and make it a high priority after the 2023 data is merged.

Should be fixed now! I wonder if there's an easy way to test these in CI?

Hmm how were you checking the validation? We have an integration test, and validate the datapackage before writing to a json file.

I just ran frictionless validate datapackage.json locally. I suspect this is a similar issue to the one in #153, which noted that our integration test wasn't catching this problem?
I get the following error for each table:

Please provide "dialect.sql.table" for reading


Add support of TextureAtlas frames for trimmed textures

In order to fully support the "Trim mode" of TexturePacker, the code will require some further implementation in the Slot logic.
The "Trim" option helps to win precious surface on the texture atlases by removing transparent pixels. It would be a real plus to finish its support in DragonBonesAS.
Thanks in advance!

TODO references in the code:
https://github.com/DragonBones/DragonBonesAS/blob/450d4d3013592cedd2803476e9134190b8d3e759/Starling/Starling_1.x/src/dragonBones/starling/StarlingTextureAtlasData.as#L26
https://github.com/DragonBones/DragonBonesAS/blob/450d4d3013592cedd2803476e9134190b8d3e759/Starling/Starling_2.x/src/dragonBones/starling/StarlingTextureAtlasData.as#L26

Hi,
There is no solution to fully support the starling texture atlas yet, subtexture will be dislocated regardless of whether the code is open or not, the best solution is to tirm images before the package in texturepacker, or use DragonBones Pro to manager texture atlases.


Waiting for daemons to come online ...

Hello,
I just installed miningcore and downloaded the blockchain. But when I run miningcore I receive the message: Waiting for daemons to come online ...
This is how I launched the litecoin daemon: litecoind -server -datadir= -rpcuser= -rpcpassword= -port=9333 -rpcport=9332 -rpcbind=<IP_ADDRESS>
What wrong with that? How many litecoin daemons should I run, and how?
Thank you.

@1n4ho05 This is not a bug. Please do not open a new Issue Simply b/c you don't understand how to configure the software. Issues should only be opened for bugs/problems related directly to MC code itself.
For questions, feature requests, and other, we have the Discussion tab. Please use it instead.


Optimize intro sections in schema guide

Related issues
Refs: #206
Includes: #261, #262, #266
Describe the changes made in this pull request

Copy edit some wording
Rename and edit ex-"transitive credit" section (see also #267)
Skew credit redirection section slightly towards better practice (of citing both software and paper rather than just paper)
Fix linting error in schema.json

Instructions to review the pull request

Look at the changes in schema-guide.md

If 95b61b008e3843480b9b29e6d36c999ecbae9737 goes through, this should be pulled over to cffinit as well (ping https://github.com/citation-file-format/cffinit/issues/10).

I'll approve the PR, the remaining issue is not so important that it should block further progress

I'll approve the PR, the remaining issue is not so important that it should block further progress

Thanks, I'll see to how to fix these best, then merge.