ts_internal_bspline_thomas_algorithm have invalid access

The application could crash when using ts_bspline_interpolate() to interpolate a curve.
The code to access to variable 'm' in ts_internal_bspline_thomas_algorithm() has a buffer overflow
/* m_0 = 1/4, m_{k+1} = 1/(4-m_k), for k = 0,...,n-2 /
const size_t len_m = n-2; / n >= 3 implies n-2 >= 1 /
float m = malloc(len_m*size_flt);
if (m == NULL)
return TS_MALLOC;
m[0] = 0.25f;
for (i = 1; i < len_m+1; i++)  //m[len_m+1] is an invalid memory address.
m[i] = 1.f/(4 - m[i-1]);

Oh, you are right. Nice one, thanks!

Fixed in: ed658b96181228ded10eb3fe004ffbfb6f07ee97


Videos creates lots of bonus thumbnails

When downloading from twitter blogs you get lots of bonus thumbnail images of videos you download. For example, every animated meme video have both a thumbnail image and a video downloaded with the same name. these videos that normally would be .GIF images.
Longer videos both with and without sound also have thumbnails but they have a different name from the video and are blurry and pixelated.

https://user-images.githubusercontent.com/74016497/155303707-c012cc45-4ac0-4dcf-ad2b-dbfd3ce76305.mp4

https://user-images.githubusercontent.com/74016497/155303846-b2e2eec3-b627-4706-a078-de3f655c7d63.mp4

Twitter converts GIF files to videos to save bandwidth among others.
The corn still looks crispy. Doesn't the blurriness and pixelation depend on the video it was taken from?
Anyways, the filename should be the same like the video or at least start with the video filename, so that they are not only grouped by date but same start of the filename too.
So what would be a good solution? No thumbnails for GIF 'images' and optional ones for video files? None at all?
Who would like to work on it?

Both pictures above is from GIF to video files. It's true that blurriness and pixelation depend on the video but more precise the first frame of the video and usually the first frames is out of focus.
Filenames is grouped on GIF-Video files but not on regular video files.
Gif:

https://user-images.githubusercontent.com/74016497/156743888-04fc32e8-6f6c-4b95-a402-8412afd678ee.mp4
The filenames of GIF-videos: E3Y2uTZXIAQbPSQ.jpg and E3Y2uTZXIAQbPSQ.mp4

Video:
https://user-images.githubusercontent.com/74016497/156744572-55c1b326-01c2-4d2c-a42f-848a905bcf72.mp4

The filenames of videos: OjABGpYZev_oPdRd.mp4 and 6iBkelCLy3BefLAz.jpg
I would vote for none at all but i guess that the best solution would be to have thumbnails as an optional setting and not default. But i can't imagine anyone that would want thumbnails of GIF-Video files. As of know i have about 20k of GIF-video thumbnails that needs to be removed.

No more thumbnails for GIFs and new option for video thumbnails.

The issue has been fixed and closed. You can still comment. Feel free to ask for reopening the issue if needed.


[FEATURE_REQUEST] Add option to forward cookies during status checks

Is your feature request related to a problem? If so, please describe.
I have most of my services behind Authelia, so every status check results in 401 unauthorized. It would be great if the status check endpoint forwarded the Cookies header when making calls to other services.
Describe the solution you'd like
A generic solution would be to add an option like statusCheckForwardHeaders, which would accept a list of headers to forward from the status check request. It would be great if that option could be defined per-service or globally.
Priority
Medium (Would be very useful)
Is this something you would be keen to implement
Maybe

No work around other than not using Authelia?

Yeah, I can add this. Do you want the option to allow for forwarding user-defined cookies, or specific cookies from the current sesh?


Installation Fails from Store in Windows 11

Fresh Windows 11 computer, installed WSlackware from the Microsoft Store and got this error message:

I've verified that WSL is enabled. Any ideas?

@ahwm Hi, Thanks for using WSLackware. You need to install WSL2 kernel. You can either install it using Windows Update or Download it from Microsoft.


Missing CDK Bootstrap Guidance

Unable to run the initial "npx cdk deploy BaseInfraStack" step due to environment not having been bootstrapped.

Performing "npx cdk bootstrap" resolved the issue and allowed me to proceed.

@nibrahv thanks for letting me know.
I have added it now - https://github.com/aws-samples/rag-with-amazon-bedrock-and-pgvector/commit/99e3c52dafaf81accc07de38cb992bbf09736cfd
Will close the issue out.

You are awesome, thank you!


Event Stream input / output mapping

The Event Stream format is great, and allows importing of additional log files for which I have written a parser (i.e., command line logs from Linux).
The Event Stream format specifies d: as a description of an event.
The input is mapped as the description for the imported event:
https://github.com/nccgroup/SteppingStones/blob/main/event_tracker/views.py#L1169
However, the related EventStream element is mapped as 'raw evidence' rather than a description:
https://github.com/nccgroup/SteppingStones/blob/main/event_tracker/views.py#L1269
Is this an intended design decision?
Would you be open to a PR which extends the Event Stream format specification with additional optional elements to further enrich imported events?

You're right - this is an oversight. Historically, SteppingStones only captured a "description", and the "evidence" (i.e. command I/O) field was added much later - after EventStream was defined. As EventStream was primarily used to capture command I/O the mapping code was quickly changed to place the data previously in the "d" field into the new "evidence" field. I agree this is confusing though.
PRs are always welcome. Can I suggest we keep "d" for "description" and add a new abbreviation for the evidence field. We'll also need to regenerate the schema & help file and tweak the https://github.com/nccgroup/SteppingStones/blob/main/pstranscript2eventstream.py script to use the new evidence variable.


13 and higher  H3-resolution

Hi!
I was testing different H3-resolutions for my area of study and when I tried with 13 and 15 h3-resolutions and it didn't print out very well, I got blank spaces in between.
My code is:
#Openning a 10m regular sample points
sample_points10m_sr=gpd.read_file(output_directory+"sample_points10m_sr.geojson").to_crs(4326)

#Getting its lat and longitude
centroides["Lng"]=centroides.geometry.x
centroides["Lat"]=centroides.geometry.y

#Genereting the h3 index with 13-h3 resolution
centroides=centroides.h3.geo_to_h3(13)

#Reseting the index
centroides=centroides.reset_index()

#Selecting columns of my interest and grouping by

centroides=centroides.loc[:,["h3_13","id","Lng","Lat"]]
centroides = centroides.drop(columns=['Lng', 'Lat']).groupby('h3_13').sum()

#Saving it as GeoJSON
centroides.to_file(output_directory+"h3_13.geojson")

For example, when I opened it in QGIS it looks like this for a 13-h3 resolution:

But When I did the same process for 12 and 10 resolution is all right, for example, here in the same area a picture of 12 resolution:

For what reason is it happening? Is it about my computer or about the h3-pandas library?
Let me know.
Thanks you so much in advance for the H3-Pandas library, I am really enjoying it.
Bryan

Hi @bwcastillo, glad you're enjoying the library!
At resolution 13, the hexagons themselves will be less then 10m in diameter. It is possible that what you're seeing is all the H3 resolution 13 hexagons that cover your points. Try plotting both your points and the generated hexagons to verify this.
Here's some sample code exemplifying this behaviour
import numpy as np
from itertools import product

# Set up a regular grid of points
lat = np.arange(0, 0.1, 0.01)
lng = np.arange(0, 0.1, 0.01)
lat, lng = np.array(list(product(lat, lng))).T
points = pd.DataFrame({'lat': lat, 'lng': lng})

# Plot points + H3 hexagons
ax = points.h3.geo_to_h3(8).h3.h3_to_geo_boundary().plot()
points.plot(x='lng', y='lat', color='red', kind='scatter', ax=ax)

You can see that the grid of hexagons is irregular even for a regular grid of points, simply because not all hexagons at this resolution contain a point.
You could achieve the effect of "filling in" the gaps, by e.g. using resolution 10 or 12 and using e.g. polyfill to find all the child hexagons at resolution 13. See e.g. this answer. However, I'm not sure the grouping operation would then provide the result you're after - presumably you're interested in some value assigned to those points and it's not clear what value hexagons that did not cover a point in the first place should have.
Hope this is clear, let me know if you have further questions.

Closing this as it was likely answered, feel free to reopen.

@DahnJ sorry because I never gave you a response about if it was solved. Actually, it was!
I introduce and recommend this library each time I can do it.
Thank you so much.


[DRAFT] ECharts v5.0 Feature Discussion

ECharts v5.0 is the next major version that is going to be released in 2020.
Here is the list of the primary features that we are going to provide or fix in this release.
Flexibility and Customization

[ ] Support mouse events on components @Ovilia
[ ] Enhancement of custom series @100pah #11869 #11870
[ ] More states in addition to current normal / emphasis

Interaction

[ ] Enhancing selection interaction and API  @pissang
[ ] Fade out other series when one is highlighted

Components

[ ] Auto layout for labels for better display @pissang
[ ] Auto layout for components to avoid overlapping @Ovilia
[ ] Enhance time axis @100pah

New Series

[ ] Area range charts

Other Enhancement

[ ] fontSize support rem

Performance

would be nice to have the possibility to add custom and mutliple html "tooltips"

https://github.com/apache/incubator-echarts/projects/6


documenting reserved words

I just reviewed #61 and ended up with some questions.
I think this might be good to document a bit on the main readme docs. What happens when I use a reserved word in my log statements? prefix disappears weirdly, but lvl just shows up and doesn't change the lvl? I could maybe make a table but I don't know well enough I don't remember exactly how each of them is used.
I'm also curious why are there reserved words in the log line stuff? what does that help with, like these aren't for the logger options these are on each log line so I'm just confused how they work? do some of them get used?

i'm guessing they show up when using json or slog in general? it makes sense to reserve these words i just don't know what happens when i use them and it surprised me, so i figure maybe a readme update will help.

Hi @dezren39, thank you for reporting this one. There are indeed some reserved words used to denote different kinds of values such as time, level, and msg. Feel free to send a PR


Add: apng, png metadata extract feature

apng - frames +
png - width, height, gamma +
Dart 3 support

@justkawal Please check this

@justkawal Could you please release this version? It says it's not compatible with Dart 3.

@chanjungkim Really, Sorry for the delay in publishing.
Version 1.0.3 is now available on pub.dev
Much appreciated for your PR.


Can only scroll backwards in idmode number for NA exclusive cards

go into ^idmode number
try ^idna 4953 (or just ^id 4953)
you can only scroll backwards (left) through the ids and not right

duplicate of #123

wait I read this too fast sorry

closed by #147


Campaign Run Titles Are Missing on Prod

Current Behavior
Titles for campaign runs got wiped when we made that field translatable #6016. Example https://www.dosomething.org/us/node/5439/edit
Desired Behavior
The original information that was stored in those fields returns (i.e. Comeback Clothes 2015 for node 5439). Previously @sergii-tkachenko has written scripts to restore similar missing information.
Why This Matters
The missing titles makes it harder to identify what the run represents especially when trying to associate it as the Current Run on the Campaign Node
Relevant Screenshots + Links

also it's plain illegal in drupal to not have titles. think of the law.

i think i found one
https://github.com/DoSomething/phoenix/pull/4943

dosomething@ip-10-100-35-14:/var/www/www.dosomething.org/current$ ds-global enable-campaign-runs
     ___           ___       ___           ___           ___           ___
    /\  \         /\__\     /\  \         /\  \         /\  \         /\__\
   /::\  \       /:/  /    /::\  \       /::\  \       /::\  \       /:/  /
  /:/\:\  \     /:/  /    /:/\:\  \     /:/\:\  \     /:/\:\  \     /:/  /
 /:/  \:\  \   /:/  /    /:/  \:\  \   /::\~\:\__\   /::\~\:\  \   /:/  /
/:/__/_\:\__\ /:/__/    /:/__/ \:\__\ /:/\:\ \:|__| /:/\:\ \:\__\ /:/__/
\:\  /\ \/__/ \:\  \    \:\  \ /:/  / \:\~\:\/:/  / \/__\:\/:/  / \:\  \
 \:\ \:\__\    \:\  \    \:\  /:/  /   \:\ \::/  /       \::/  /   \:\  \
  \:\/:/  /     \:\  \    \:\/:/  /     \:\/:/  /        /:/  /     \:\  \
   \::/  /       \:\__\    \::/  /       \::/__/        /:/  /       \:\__\
    \/__/         \/__/     \/__/         ~~            \/__/         \/__/

'all' cache was cleared.                                                                                [success]
Replacing field values for  in campaign_run node.
Done.                                                                                [ok]

Fixed

scriptzzz

:sweat_smile:


[$125] Make sure the first result in most recents is highlighted when user uses CMD+K

From the design doc.

Recent chats is the same as our existing CMD+K command, which means:

The router can be opened and focused via the same keyboard shortcut
When CMD+K is pressed and no search has been entered, the top-most result in Recent chats is highlighted. Hitting the return key closes the router and navigates a member to the corresponding report

cc @luacmartins
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021852421610290735954
Upwork Job ID:<PHONE_NUMBER>290735954
Last Price Increase: 2024-11-01

Issue OwnerCurrent Issue Owner: @Pujan92

Proposal
Please re-state the problem that we are trying to solve in this issue.
Make sure the first result in most recents is highlighted when user uses CMD+K
What is the root cause of that problem?
Improvement
What changes do you think we should make in order to solve the problem?

We should save the first chat index here
https://github.com/Expensify/App/blob/bfce2771d22cbecceee6cf3c064fe3f0627eb5d8/src/components/Search/SearchRouter/SearchRouterList.tsx#L174
    const firstFocusedIndex = sections.length;

we will pass this prop down to BaseSelectionList and in BaseSelectionList we will add this useEffect
useEffect(() => {
        if (firstFocusedIndex === undefined) {
            return;
        }
        setFocusedIndex(firstFocusedIndex);
    }, []);

What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
Make sure the first result in most recents is highlighted when user uses CMD+K
What is the root cause of that problem?
Improvement
What changes do you think we should make in order to solve the problem?

We should save the first chat index here
https://github.com/Expensify/App/blob/bfce2771d22cbecceee6cf3c064fe3f0627eb5d8/src/components/Search/SearchRouter/SearchRouterList.tsx#L173-L174
    const firstFocusedIndex = sections.reduce((resultIndex, section) => section.data.length + resultIndex, 0);

we will pass this prop down to BaseSelectionList and in BaseSelectionList we will add this useEffect
useEffect(() => {
        if (firstFocusedIndex === undefined) {
            return;
        }
        setFocusedIndex(firstFocusedIndex);
    }, []);

or
useEffect(() => {
        if (firstFocusedIndex === undefined) {
            return;
        }
        console.log(firstFocusedIndex);
        updateAndScrollToFocusedIndex(firstFocusedIndex);
    }, []);

What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
Make sure the first result in most recents is highlighted when user uses CMD+K
What is the root cause of that problem?
Improvement
What changes do you think we should make in order to solve the problem?

We should save the first chat index here
https://github.com/Expensify/App/blob/bfce2771d22cbecceee6cf3c064fe3f0627eb5d8/src/components/Search/SearchRouter/SearchRouterList.tsx#L176
    const initiallyFocusedOptionKey = styledRecentReports.at(0)?.keyForList;

we will pass it here
https://github.com/Expensify/App/blob/bfce2771d22cbecceee6cf3c064fe3f0627eb5d8/src/components/Search/SearchRouter/SearchRouterList.tsx#L240
What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
Make sure the first result in most recents is highlighted when user uses CMD+K
What is the root cause of that problem?
Improvement
What changes do you think we should make in order to solve the problem?

We should save the first chat index here
https://github.com/Expensify/App/blob/bfce2771d22cbecceee6cf3c064fe3f0627eb5d8/src/components/Search/SearchRouter/SearchRouterList.tsx#L176
    const initiallyFocusedOptionKey = styledRecentReports.at(0)?.keyForList;

we will pass it here
https://github.com/Expensify/App/blob/bfce2771d22cbecceee6cf3c064fe3f0627eb5d8/src/components/Search/SearchRouter/SearchRouterList.tsx#L240
Additionally: From the OP I can see that the list is not suppose to scrolled to the focused item on initial render but we have a code here that scroll to the focused index
https://github.com/Expensify/App/blob/bfce2771d22cbecceee6cf3c064fe3f0627eb5d8/src/components/SelectionList/BaseSelectionList.tsx#L514
we will disable this code by passing  a new param shouldNotScrollToFocusedIndex
What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
Make sure the first result in most recents is highlighted when user uses CMD+K
What is the root cause of that problem?
I don't know if the above proposal states the same issue or not, but for me; the issue is the lack of adding the  initiallyFocusedOptionKey prop to the <SelectionList/> component.
What changes do you think we should make in order to solve the problem?

Adding initiallyFocusedOptionKey={recentReports && recentReports.length > 0 ? recentReports.at(0)?.keyForList : ''} prop after the onArrowFocus prop will make sure that the first chat will be focused in recent chats if there are any.
/https://github.com/Expensify/App/blob/main/src/components/Search/SearchRouter/SearchRouterList.tsx#L243

What alternative solutions did you explore? (Optional)

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01dd32e60eb083d581

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~0139a3178534697447

Proposal
Please re-state the problem that we are trying to solve in this issue.
Make sure the first result in most recents is highlighted when user uses CMD+K
What changes do you think we should make in order to solve the problem?
The lack of an initiallyFocusedOptionKey to set the default focused item within the <SelectionList> component causes the list to open without any item pre-highlighted.
To ensure the first chat in the recent chats list is automatically highlighted when the search bar opens with CMD+K, we will:

Add a variable, initiallyFocusedOptionKey, set to the key of the first recent chat.
Pass this variable as a prop to the <SelectionList> component, enabling automatic highlighting.

https://github.com/Expensify/App/blob/bef062b4caa7f665159dc107911e708031e648c4/src/components/Search/SearchRouter/SearchRouterList.tsx#L176
const initiallyFocusedOptionKey = styledRecentReports.at(0)?.keyForList;

and we will pass initiallyFocusedOptionKey to <SelectionList>
https://github.com/Expensify/App/blob/bef062b4caa7f665159dc107911e708031e648c4/src/components/Search/SearchRouter/SearchRouterList.tsx#L240
initiallyFocusedOptionKey={initiallyFocusedOptionKey}

What alternative solutions did you explore? (Optional)

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link:  https://www.upwork.com/freelancers/~017a4e9874f15ccbd9

@Pujan92 did you get a chance to look at the proposals above?

Yes, let's please prioritize this one. It's coming up actively with our members.

Sorry for the delay, I will review the proposals today.

Usually initiallyFocusedOptionKey is a straightforward option which also consistent with other places but it won't work for all scenarios(1. Reload and the context may not be available initially 2. When a search term is cleared)  as @nyomanjyotisa mentioned. With that, I think @nyomanjyotisa's proposal  of setting the focused index explicitly makes sense.
🎀👀🎀 C+ reviewed

PR is being worked on. Hopefully up for review this week.

Payment summary:

Contributor: $125 @nyomanjyotisa via Upwork
Reviewer: $125 @Pujan92 via New Expensify

Contributor paid via NewDot. @Pujan92 do you mind filling out the BZ checklist and recommending a regression test?

I think we don't need a checklist here as this is a new change and not a bug.
Regression Test Proposal

Navigate to search with CMD+K or click on the search icon for larger screens
Verify that the top-most result in Recent chats is highlighted
Press enter and verify that the user navigated to the highlighted Recent chats

Thanks regression test created. Please submit a request via New Expensify. Closing this issue out!

$125 approved for @Pujan92


Introduce TreeNode::exists() API, avoid copying expressions

Which issue does this PR close?
Part of https://github.com/apache/arrow-datafusion/issues/8913.
Rationale for this change
This PR adds TreeNode::exists() API as a common pattern to check the presence of a matching node for a predicate in a tree.
This PR also solves the issue found by @alamb here: https://github.com/apache/arrow-datafusion/pull/9913#discussion_r1554563916. Since https://github.com/apache/arrow-datafusion/pull/9913 there is no need to call LogicalPlan::expressions() (that copies the plan's expressions) at certain places, but iterating over the expressions with LogicalPlan::apply_expressions() is enough.
What changes are included in this PR?

Adds the TreeNode::exists() API.
Replaces contains_outer_reference helper with a better solution.

Are these changes tested?
Yes, with existing UTs.
Are there any user-facing changes?
No.

@alamb, I think there are other places where LogicalPlan::expressions() can be replaced with the faster LogicalPlan::apply_expressions() but the change is non-trivial, so let's start with this smaller PR first.

I took the liberty of adding the doc comments directly to this PR. Thanks again @peter-toth

Thanks @alamb for the doc comments and the review! Actually, I wanted to ask you to do that as I'm travelling this week and have limited access to github.

Thank you @peter-toth  for all your help


keras.engine.topology _to_list

Hi, I'm a beginner in the deep learning field. I'm using python3.7 and I've already installed the new version of keras and tensorflow but I obtain the following error if I try to import the module _to_list from keras.engine.topology
ImportError: cannot import name '_to_list' from 'keras.engine.topology'

@oscar195, I believe there isn't a function _to_list in the module Keras.engine.topology. In fact, that module is deprecated as stated here.
Is it possible for you to give more information/context about what you want to do?


[Fixes #5352] When replacing Controller.Dispose with an explicit impl…

…ementation the base Dispose is an action

Looks good to me.

Sorry for not commenting as well, I discussed this in person with @kichalla - the intent here is that an IDisposable implementation will never be considered an action. We have code like this already, but it has a bug. You shouldn't need to put [NonAction] on Dispose.

🆙 📅 Folks, please take a look again.


APPS-1121 Update integration test suite

(feat) all - initial

At the moment if reviewing pls focus on Terraform and Dependency classes and their tests. This is just to address APPS-1122 ticket

Ready for review. Also implemented expected failure tests. As an e2e demo feel free to run

dxcint integration ${COMPILER_REPO_ROOT} -t "bad_status"


Auth concurrency issues

I've been using this library within the context of an Azure function application with a queue trigger.  This works by picking up 32 messages at a time to be processed concurrently.
The login method is not public, but called automatically by each method that retrieves data, therefore when the queue processor starts it hits the iRacing auth endpoint with 32 concurrent auth requests at once.  I have been contacted by iRacing to investigate the high number of auth requests they're receiving.
I have added a workaround with a SemaphoreSlim at the start of the LoginInternalAsync so only the first request will call login and the other requests will wait until the single auth request has completed and won't then call the auth endpoint multiple times.  I'm happy to raise a PR containing this change, but I'm unsure if you want to approach this a different way.
        await loginSemaphore.WaitAsync(cancellationToken).ConfigureAwait(false);
        if (IsLoggedIn)
        {
            return;
        }

I think I'd like to avoid doing this within the library itself without serious consideration. Locking is complex and can be app-specific, so I'd prefer to open up a specific method which allows you to move that lock into your own code where you can make the appropriate decisions & analysis on it to avoid deadlocks etc.
I think that with a resolution of #186 implemented this situation can be avoided altogether. I'll leave this issue open until that can be verified though.


sqlreplay: fix job running status

What problem does this PR solve?

Issue Number: close #710
Problem Summary:
Even if the last replay is finished, the next replay may fail to start because the progress is not 100%.
What is changed and how it works:
Check the field done instead of progress.
Check List
Tests

[x] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Notable changes

[ ] Has configuration change
[ ] Has HTTP API interfaces change
[ ] Has tiproxyctl change
[ ] Other user behavior changes

Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
- Fix that traffic replay may incorrectly report another job is running

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

Please upload report for BASE (main@3721208). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #713   +/-   ##
=======================================
  Coverage        ?   66.09%
=======================================
  Files           ?      122
  Lines           ?    11173
  Branches        ?        0
=======================================
  Hits            ?     7385
  Misses          ?     3255
  Partials        ?      533

Flag
Coverage Δ

unit
66.09% <100.00%> (?)

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

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


fix(ui): uncaught type error when selecting an organization after logging

There was a problem in the data in the DB for some projects that were disabled in the project table but they were enabled in the organization table. I fixed the data and now it won't show those disabled projects in the select list.


Errors Caused by the Deprecated Query API with Sqlalchemy 2.0 and Async

When using sqlalchemy 2.0 with the async adapter, I'm getting AttributeError: 'AsyncSession' object has no attribute 'query' because the not supported session.query
Do you have any plans to upgrade the code to Sqlalchemy 2.0 style so that 2.0 with async is supported?

@ChenHaolinOlym hi, can you make a PR?

@BustDot


Don't redirect on config create

Closes #1241

Coverage remained the same at 82.347% when pulling 3b77ceb298a0c29f1c10cf4ebf6e690736edfb07 on eatonphil:dont-redirect-on-config-create into 4527740c4265ad7a014a7f89fc419300a8b954da on linode:master.

thanks!


[FLINK-36575][runtime] ExecutionVertexInputInfo supports consuming subpartition groups

What is the purpose of the change
Currently, the ExecutionVertexInputInfo describes the task's input through a combination of a PartitionRange and a SubpartitionRange. However, in the case of skewed join optimization, we need to split a group of data corresponding to the specific key, which may result in a downstream task subscribing to multiple combinations of PartitionRanges and SubpartitionRanges.
Therefore, we need to modify the ExecutionVertexInputInfo to describe the input data as multiple combinations of PartitionRanges and SubpartitionRanges to meet the requirements of the aforementioned scenario and improve Flink's flexibility in describing the task's inputs.
Brief change log

Modify the description of the input in ExecutionVertexInputInfo.
Modify the connect function for edges.
Modify the InputGate creation logic of the network layer to adapt to this change.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not documented)

@flinkbot run azure

@flinkbot run azure

@JunRuiLee Thanks for review. The PR has been updated based on your comments.


Add UndoRedo.cancel_action() method

I'm working on a project where this would prove useful so I thought I'd write it and open a PR.

I downloaded and ran the macos-editor build artifact to test this, but my new method doesn't seem to exist. Not in documentation, intellisense, or when I run my test scene. I have no clue why. I believe I've followed the pattern of past changes to this class.

Are you using the correct class? And not the editor undo/redo?

extends Node2D

# Called when the node enters the scene tree for the first time.
func _ready():
	var unre = UndoRedo.new()
	unre.create_action("blah")
	unre.add_do_method(func(): print("doing something!"))
	unre.add_do_method(func(): print("undoing something!"))
	unre.cancel_action()
	print(unre.get_current_action())
	print(unre.get_current_action_name())
	print(unre.get_history_count())
	print(unre.get_version())
	print(unre.get)

The reason you don't have autocomplete is that you're not using static typing, try:
func _ready():
	var unre := UndoRedo.new()
	unre.create_action("blah")
	unre.add_do_method(func(): print("doing something!"))
	unre.add_do_method(func(): print("undoing something!"))
	unre.cancel_action()
	print(unre.get_current_action())
	print(unre.get_current_action_name())
	print(unre.get_history_count())
	print(unre.get_version())

But I'd suggest just compiling the code yourself, you should anyway to test it

Interesting, I normally do use types but I didn't even notice I'd forgotten this time. It was actually autocompleting the other method calls even without.
I do plan on getting it compiling locally, but I hoped I could get away with just using the build artifacts for my first PR.

Since you're finding problems with it I'd say to compile it locally, always important to rest the features you're adding

I got Godot building locally, and everything looks to be working just fine! :tada: I guess the build artifact from the checks doesn't pull my changes in properly for some reason 🤷
I did have a thought while testing this out, though: When you call create_action, it clears any redo actions above the current position in the stack. So even if you cancel_action, those redos will not come back. The cancelled action will still have an effect on the state of the program. I personally find this pretty acceptable, but maybe it should work differently?

You might want to rebase all three commits into one, as described here: https://docs.godotengine.org/en/stable/contributing/workflow/pr_workflow.html#the-interactive-rebase
Also, cancel_action doesn't reset the "merging" variable. This could lead to the next action getting merged when it shouldn't be.
Also, shouldn't action_level be set to 0 rather than decremented? It seems this is meant to break out of all the nested actions, not just the current one.

This needs a proposal. From my experience there is always a way to structure your actions in a way that doesn't require canceling.


Fontsize

https://github.com/aalpturkay/expense_tracking_app_flutter/blob/b0022432519d332856d778b4d7a45a106fdffb06/lib/features/expense/pages/expense_page.dart#L41
You should be use theme

I tried to use Theme for setting fontsize


Invalid Char Length on Metrics Breaks StatefulSet Creation

Warning  FailedCreate  31s (x16 over 3m15s)  statefulset-controller  create Pod sequencer-0 in StatefulSet sequencer failed error: Pod "sequencer-0" is invalid: spec.containers[0].ports[2].name: Invalid value: "sequencer-metric": must be no more than 15 characters

Edit Chart Templates to truncate the name to stay within 15 char limit
add any helper tpl if needed to fix naming

Potential path
name: {{ include "helm-chart.fullname" . | trunc 7 }}-metrics

This issue is stale because it has been open 45 days with no activity. Remove stale label or this issue
be closed in 7 days.

This issue was closed because it was stale


F#761 timestamp histogram error

Description

Allow Keep and delete from context menu by clicking the histogram when column type is timestamp
Related Issue :

#761
How Has This Been Tested?

Dataflow main grid -> click timestamp type histogram bar -> keep/ delete -> check if keep and delete works.
See screenshot

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project. it will be added soon
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document. it will be added soon
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Additional Context

run build
deploy to 1


Support Link Provider and Relative Paths

TL;DR
I am currently trying to support links within a vscode plugin, that are relative to the current working directory of the terminal, similar to how the relative paths in the terminal work today.  On Unix based systems, I'm able to access /proc/{pid}/cwd as a symlink to the terminal's shell cwd, but this is not cross platform, and seems unreliable.
I'd love to see a consistent way to expose the cwd of the terminal, or provide a way for the link provider to resolve relative paths in the context of the terminal it was clicked in.
Detailed Example
I'm currently using vscode.window.registerTerminalLinkProvider to great effect.
One of the use cases I'm currently supporting is an internalized categorization that allows for direct mapping to a specific file/class. The purpose of this is to allow for a more compact log output.
2020-12-22T14:28:58.500Z debug [@trv:base/shutdown:48] Starting { name: '@trv:model-mongo/service' }

The token @trv:base/shutdown:48 is a shorthand that would expand to a node_modules reference.
The problem arises when I need to deal with a path that is relative to the terminals's current working directory (cwd), in a similar fashion to how the relative paths work in the terminal today.
2020-12-22T14:28:58.500Z debug [src/service:48] Request received

The token src/service:48 is a shorthand that should expand to a {cwd}/src/service.ts, but I have no mechanism to determine the working directory in a cross platform (or reliable) way.

You can see how we get the cwd for relative link support here:
https://github.com/microsoft/vscode/blob/7a34c6d622724575122465bb46078af06e07ee44/src/vs/workbench/contrib/terminal/node/terminalProcess.ts#L358-L406
Unfortunately on Windows it's impossible to do this, I confirmed this in an internal email thread with folks working on the Windows Kernel.
This could potentially be an API in the future, I'd recommend you can just copy how we do it for now. Be careful not to call lsof a lot, especially on macos big sur (https://github.com/microsoft/vscode/issues/105446). This is probably what such an API would look like:
export interface Terminal {
  cwd: Thenable<string>;
}

You can see how we get the cwd for relative link support here:
https://github.com/microsoft/vscode/blob/7a34c6d622724575122465bb46078af06e07ee44/src/vs/workbench/contrib/terminal/node/terminalProcess.ts#L358-L406
Unfortunately on Windows it's impossible to do this, I confirmed this in an internal email thread with folks working on the Windows Kernel.
This could potentially be an API in the future, I'd recommend you can just copy how we do it for now. Be careful not to call lsof a lot, especially on macos big sur (https://github.com/microsoft/vscode/issues/105446). This is probably what such an API would look like:
export interface Terminal {
  cwd: Thenable<string>;
}


Upgrade robots to the latest NWS/NWC layers - 2

This PR modifies XML files for the following robots :

iCubGenova07
iCubGenova08
iCubGenova10

The mods have been done starting from the default XML file contained in the yarprobotinterface.ini file (i.e. icub_all.xml)
The new files should be tested by the users.
@randaz81 , could you please have a look at them?
cc @pattacini @traversaro

Tested all robots of this PR using DEBUG mode by adding these lines to the pc104.xml file :
    <group name="DEBUG">
        <param name="embBoardsConnected"> 0 </param>
    </group>

and setting in the general.xml file :
      <param name="skipCalibration">    true  </param>

This avoid yarprobotinterface terminating if not detects ETH boards; in this way, we test the correct parsing of the xml files.
Fixed an error on iCubGenova08
cc @pattacini


[CT-520] [Enhancement] Schema files with repeated top level keys (e.g. models) results in only the last key being parsed

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
If you have a schema file with repeated top level keys:
# models/schema.yml
version: 2
models:
  - name: my_model_a
    description: "This is a"
    columns:
      - name: user_name
        tests:
          - not_null
models:
  - name: my_model_b
    description: "This is b"
    columns:
      - name: user_name
        tests:
          - not_null

Only the last models key will apply to the project when you execute dbt tasks that depend on them (like test, docs generate). We should raise an exception for such cases.
Expected Behavior
Perhaps we should raise an error in this case instead of silently just including the last models key of the schema file in our project's manifest.
Steps To Reproduce

Add the above schema.yml file to your project along with 2 simple models:

-- models/my_model_a.sql
select 1 as user_name
-- models/my_model_b.sql
select 1 as user_name

Do a dbt run to build our models.
Do a dbt test and observe that we only tested my_model_b.
Do a dbt docs generate && dbt docs serve and observe that only my_model_b has a description (my_model_a has no description).

Relevant log output
$ dbt test
04:02:08  Running with dbt=1.0.4
04:02:09  Found 2 models, 1 test, 0 snapshots, 0 analyses, 165 macros, 0 operations, 0 seed files, 0 sources, 0 exposures, 0 metrics
04:02:09
04:02:09  Concurrency: 1 threads (target='dev')
04:02:09
04:02:09  1 of 1 START test not_null_my_model_b_user_name................................. [RUN]
04:02:09  1 of 1 PASS not_null_my_model_b_user_name....................................... [PASS in 0.07s]
04:02:09
04:02:09  Finished running 1 test in 0.24s.
04:02:09
04:02:09  Completed successfully
04:02:09
04:02:09  Done. PASS=1 WARN=0 ERROR=0 SKIP=0 TOTAL=1

Environment
- OS: macOS
- Python: 3.9.7
- dbt: 1.0.4

What database are you using dbt with?
postgres
Additional Context

Here's the relevant issue and a fix we could try which extends SafeLoader.
Add the extended class to our yaml_helper client then add in a ValueError in the following tuple:
https://github.com/dbt-labs/dbt-core/blob/37b8b65aadb811c6f56c81537da48a463857b0d9/core/dbt/clients/yaml_helper.py#L57
Like so:
    except (yaml.scanner.ScannerError, yaml.YAMLError, ValueError) as e:

Then our dbt test / dbt docs generate) should result in:
$ dbt test
04:10:34  Running with dbt=1.0.4
04:10:35  Encountered an error:
Parsing Error
  Error reading postgres: schema.yml - Runtime Error
    Duplicate 'models' key found in YAML.

(Maybe a warning instead of an error, for starters? Then we could switch it to error-level, in a future minor version)

Thanks for the guidance @emmyoop + @jtcohen6. Will take a crack at the raising a new warning instead. Possibly will need help adding the test later but will circle back on that.

At this point we've reverted the original pull request because we didn't have time to figure out a solution for anchor overrides , which were causing error messages for duplicate keys, even though anchor overrides are legal syntax and should not be considered duplicate keys.
It's still possible that we could come up with a solution, but it requires more research. Some possibilities are 1) fixing in pyyaml and doing a pull request. 2) supporting a different yaml library that does correctly handle duplicate keys (ruamel has the same issue with spurious errors for anchor overrides).  3) expanding the code that looked for duplicate keys to handle anchor overrides correctly.
One possibility is yaml.parse, which does provide information on anchors and such. A fix using this would quite possibly make more sense as a pyyaml pr though.

Hi @gshank just re-opened this issue as it's still unresolved and would be really helpful if we could potentially implement your suggestion maybe. Just sharing an additional example of how this could impact users:
If you have a sources.yml file and within it you define a source multiple times (e.g. developer doesn't realize source is defined already so adds another one of the same), the most latest defined source overrides the first ones. So if you have like:
sources:
  - name: some_source
    database: some_database
    schema: some_schema
    tables:
      - name: ac
      - name: ab
      - name: abc
  - name: some_source
    database: some_database
    schema: some_schema
    tables:
      - name: ac
      - name: ab

With the above, you basically lose table abc.
Expected Behavior:
We expect to either:
Flag Duplicate Sources: Clearly indicate duplicate source definitions and prevent overwriting.
Merge Duplicate Sources: Combine table definitions from multiple sources with the same name, database, and schema.
Impact:
This issue can lead to incorrect model definitions, unexpected errors, and wasted development time.


feat: implement set_quota_properties methods in client

Closes #295

Codecov Report

Merging #309 (b4815ef) into master (b45925a) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head b4815ef differs from pull request most recent head 8e84ae5. Consider uploading reports for the commit 8e84ae5 to get more accurate results

@@            Coverage Diff            @@
##            master      #309   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            6         6
  Lines          271       271
  Branches        31        31
=========================================
  Hits           271       271

Impacted Files
Coverage Δ

src/citric/client.py
100.00% <100.00%> (ø)

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 cba6601...8e84ae5. Read the comment docs.


Move CSP out of app code

Having the CSP in the app code is not re-usable
Just let whoever is running the app role their own in the server config

oops mixed up csp with csrf


Implement "flowPriority" (at the time of polling) feature for new Dispatching Logic

"flowPriority" is one of the execution options of a flow, which only takes effect if set by an Azkaban admin. By default all flows have a priority of 5 but if a bigger number is specified from the UI that flow will be dispatched for execution first.
With the new Dispatching Logic(#2038) setting a high "flowPriority" means that flow will be polled first by executors.
Note that it has been said "polled first" not "executed first". On the executor server there is a queue to hold executions when the maximum number of threads availables/setup to run flows is reached. This queue is NOT a priority queue.
To implement this feature for the new Dispatching Logic, a new column "flow_priority" needs to be added to table "execution_flows" to speed up access to this info (currently flowPriority is stored in "flow_data" longblob column). When new executions are inserted into "execution_flows" table, user-specified or default flow priority will be inserted as well. Later, executors will poll executions sorting by
flow_priority DESC, submit_time ASC, exec_id ASC
Adding an index on flow_priority column or a multicolumn index on (flow_priority, submit_time) for example, is not necessary because "status" column is already indexed and this column is more selective than "flow_priority" so MySQL should pick this index over the rest whenever polling executions.

Pull Request Test Coverage Report for Build 4389

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
26 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.07%) to 36.856%

Files with Coverage Reduction
New Missed Lines
%

azkaban-common/src/main/java/azkaban/jobExecutor/AbstractJob.java
2
31.03%

azkaban-common/src/main/java/azkaban/jobExecutor/AbstractProcessJob.java
3
63.1%

azkaban-exec-server/src/main/java/azkaban/execapp/event/JobCallbackRequestMaker.java
21
56.25%

Totals

Change from base Build 4387:
-0.07%

Covered Lines:
10945

Relevant Lines:
29697

💛  - Coveralls

Could you please check the query plan, for the new select with flow_priority, if not done already?

Could you please check the query plan, for the new select with flow_priority, if not done already?

Do you mean this:
mysql> EXPLAIN EXTENDED SELECT exec_id from execution_flows where status = 20 and executor_id is NULL and flow_data is NOT NULL and use_executor is NULL ORDER BY flow_priority DESC, submit_time ASC, exec_id ASC LIMIT 1 FOR UPDATE;
+----+-------------+-----------------+------+----------------------------+----------------+---------+-------+------+----------+-----------------------------+
| id | select_type | table           | type | possible_keys              | key            | key_len | ref   | rows | filtered | Extra                       |
+----+-------------+-----------------+------+----------------------------+----------------+---------+-------+------+----------+-----------------------------+
|  1 | SIMPLE      | execution_flows | ref  | executor_id,ex_flows_staus | ex_flows_staus | 2       | const |    1 |   100.00 | Using where; Using filesort |
+----+-------------+-----------------+------+----------------------------+----------------+---------+-------+------+----------+-----------------------------+
1 row in set, 1 warning (0.01 sec)

This DB has over 10 million rows in execution_flows table

Yes, thanks. How does the explain plan change if an index is added on (flow_priority, submit_time), or (flow_priority, submit_time, exec_id)? Each executor is polling with this query, so it's getting executed very frequently.
https://dev.mysql.com/doc/refman/8.0/en/order-by-optimization.html

Yes, thanks. How does the explain plan change if an index is added on (flow_priority, submit_time), or (flow_priority, submit_time, exec_id)? Each executor is polling with this query, so it's getting executed very frequently.
https://dev.mysql.com/doc/refman/8.0/en/order-by-optimization.html

Here is the EXPLAIN output after adding (flow_priority, submit_time) index:
mysql> ALTER TABLE execution_flows ADD INDEX priority_submit (flow_priority DESC, submit_time ASC);
mysql> EXPLAIN EXTENDED SELECT exec_id from execution_flows WHERE status = 20 and executor_id is NULL and flow_data is NOT NULL and use_executor is NULL ORDER BY flow_priority DESC, submit_time ASC, exec_id ASC LIMIT 1 FOR UPDATE;
+----+-------------+-----------------+------+----------------------------+----------------+---------+-------+------+----------+-----------------------------+
| id | select_type | table           | type | possible_keys              | key            | key_len | ref   | rows | filtered | Extra                       |
+----+-------------+-----------------+------+----------------------------+----------------+---------+-------+------+----------+-----------------------------+
|  1 | SIMPLE      | execution_flows | ref  | executor_id,ex_flows_staus | ex_flows_staus | 2       | const |    7 |   100.00 | Using where; Using filesort |
+----+-------------+-----------------+------+----------------------------+----------------+---------+-------+------+----------+-----------------------------+
1 row in set, 1 warning (0.00 sec)

Here is the output if we dropped index (flow_priority, submit_time) and create instead an index on (flow_priority, submit_time, exec_id)
mysql> ALTER TABLE execution_flows ADD INDEX priority_submit_execid (flow_priority DESC, submit_time ASC, exec_id ASC);
mysql> EXPLAIN EXTENDED SELECT exec_id from execution_flows WHERE status = 20 and executor_id is NULL and flow_data is NOT NULL and use_executor is NULL ORDER BY flow_priority DESC, submit_time ASC, exec_id ASC LIMIT 1 FOR UPDATE;
+----+-------------+-----------------+------+----------------------------+----------------+---------+-------+------+----------+-----------------------------+
| id | select_type | table           | type | possible_keys              | key            | key_len | ref   | rows | filtered | Extra                       |
+----+-------------+-----------------+------+----------------------------+----------------+---------+-------+------+----------+-----------------------------+
|  1 | SIMPLE      | execution_flows | ref  | executor_id,ex_flows_staus | ex_flows_staus | 2       | const |    7 |   100.00 | Using where; Using filesort |
+----+-------------+-----------------+------+----------------------------+----------------+---------+-------+------+----------+-----------------------------+
1 row in set, 1 warning (0.00 sec)

Thanks for checking the query plan for the different index variations. As you had mentioned, it is choosing to use the more selective index on status, and the additional index is not being used.


PAPI support

I wish I could use all PAPI placeholders in the commands that I use in a reward (and not just the placeholders defined in this plugin). E.g. something like "%player_name%" from the PAPI "Player" extension etc.

Screw that, I just implemented it myself. See my open pull request.


WebSearch: improve/fix "NOT" operator

Originally on 2011-02-16
When a query includes a "NOT" expression for which there is no result, the search engine returns no result, whatever the rest of the query matches. For example the following query does not return any result on the Atlantis Demo Site:
muon -foo
though "muon" matches some documents (and "foo" does not).
One could expect to still get the documents matching "muon".

Works on master (2.1.dev). @tiborsimko can you confirm that it has been fixed for 1.x?

Yes, the original problem had been fixed in Invenio 1.2 already.


bosh-cli didn't read the trailing zero '0' of stemcell version in manifest

stemcell version 1.360 is set in manifest
`---
name: testvm
stemcells:

alias: jammy
os: ubuntu-jammy
version: 1.360
`

bosh deploy command shows '1.36' instead of '1.360'
`$ bosh -d testvm deploy testvm.yml
Using environment '<IP_ADDRESS>' as client 'ops_manager'

Using deployment 'testvm'

stemcells:

alias: jammy

os: ubuntu-jammy
version: '1.36'`

And the command finally failed with complaint of non-existent stemcell version 1.36
`Task 56
Task 56 | 00:47:23 | Preparing deployment: Preparing deployment (00:00:00)
L Error: Stemcell version '1.36' for OS 'ubuntu-jammy' doesn't exist
Task 56 | 00:47:23 | Error: Stemcell version '1.36' for OS 'ubuntu-jammy' doesn't exist
Task 56 Started  Wed Mar 20 00:47:23 UTC 2024
Task 56 Finished Wed Mar 20 00:47:23 UTC 2024
Task 56 Duration 00:00:00
Task 56 error
Updating deployment:
Expected task '56' to succeed but state is 'error'
Exit code 1`

It could work if enclose stemcell version with either single quote or double quote. For example,
`---
name: testvm
stemcells:

alias: jammy
os: ubuntu-jammy
version: "1.360"
`

Yeah, this is a yaml "feature" not really a bosh-cli problem.
An unquoted number with a single dot in yaml is treated as a floating point number so the "0" is extra data and thrown away.


[BUGS] Users return upon container restart.

When running in a container like so:
`oci@server:~$ podman run -p 5000:5000 icbest/giftmanager:latest

Serving Flask app 'app'
Debug mode: on
WARNING: This is a development server. Do not use it in a production deployment. Use a production WSGI server instead.
Running on all addresses (<IP_ADDRESS>)
Running on http://<IP_ADDRESS>:5000
Running on http://<IP_ADDRESS>:5000
Press CTRL+C to quit
Restarting with stat
Debugger is active!

I will check on my side, but it seems that changes aren't saved to the json, I know Flask run primarily in Ram, maybe there's writing permission missing for the JSON?

yeah just tested, it could be that it pulls a new image which contains template storage and overwrites the current storage.
It could be that it doesn't commit changes to disk after changes are made in RAM.
Or I have a misconfiguration...
I am working to see what I can do to fix it, e.g. pass a volume through as /app for GM to write to.

ok alrighty, since restarting with this command podman run -d --replace --name gift-manager --publish 5000:5000/tcp --annotation run.oci.keep_original_groups=1  --userns keep-id --volume app:/app:Z docker.io/icbest/giftmanager:latest
and running with --volume app:/app:Z omitted I get one with custom users and one without custom users respectively

got a message!
PermissionError: [Errno 13] Permission denied: 'users.json'

after setting rw,z on the volume and manually editing the users.json to remove the trial users it is showing up correctly

podman run -d --replace --name gift-manager --publish 5000:5000/tcp --volume app:/app:rw,Z docker.io/icbest/giftmanager:latest
is a working command with persistent storage in the podman app volume
after editing users.json

Thanks, didn't used Podman, but glad you were to solved it.

well regardless if podman is a drop in replacement for docker then I wonder if in the future it will behave in the same way on docker


Template does not have access to pull ACR image

This documentation can be a tad misleading. The ACI template will not have access to pull the image from ACR. I have confirmed this locally and created one manually in the portal. When creating in the portal confirmed in the deployment template that ARM behind the scenes inserts imageRegistryCredentials with the admin username and password for ACR to pull the image. I attempted to use a User Assigned Identity w/ ACR Pull to the ACI instance still receive the same error on access.
I attempted deploying the CI w/o the imageRegistryCredentials and receive.:
"The image '<image-name>' in container group '<acr-name>' is not accessible. Please check the image and registry credential."
I have the ACR admin credentials in a key vault as part of deployment like:
    imageRegistryCredentials: [
      {
        server: acr.properties.loginServer
        username: acrUserName
        password: acrAdminPassword
      }
    ]

And it works
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 8f4a2932-5c96-4971-2c00-6f9b886b49e9
Version Independent ID: cdd9bdf4-b089-a4dd-d9d3-57224c0d5a44
Content: Quickstart - create a container instance - Bicep - Azure Container Instances
Content Source: articles/container-instances/container-instances-quickstart-bicep.md
Service: container-instances
GitHub Login: @mamccrea
Microsoft Alias: mamccrea

@JFolberth
Thanks for your feedback! We will investigate and update as appropriate.

@tomvcassidy - can you please take a look?

@JFolberth
Thank you for bringing this to my attention. It seems that the documentation is not clear enough on this point. You are correct that the ACI template will not have access to pull the image from ACR.
To deploy an ACI instance with an image from ACR, you need to provide the ACR admin username and password in the deployment template. You can do this by adding the imageRegistryCredentials property to the container group definition in the template, as you have done.
If you want to use a User Assigned Identity with ACR Pull to the ACI instance, you need to make sure that the identity has the appropriate permissions to access the ACR instance. You can grant these permissions by creating a role assignment in Azure RBAC.
I hope this helps clarify the issue. Let me know if you have any further questions.

Thanks @AjayBathini-MSFT  for the quick response!
Unfortunately I get the same error when I have assigned a User Assigned Identity to the ACI w/o passing in the imageRegistryCredentials.

@JFolberth
I've assigned this issue to the author who will investigate and update as appropriate.

@AjayBathini-MSFT  @JFolberth
Although not clearly documented, assigning the user assigned managed identity is not enough.  You must explicitly define it in the imageRegistryCredentials object:
    imageRegistryCredentials: [
      {
        server: registryLoginServer
        identity: identity.id
      }
    ]

Where registryLoginServer is the base fqdn of the container registry and identity.id is the id of the user assigned managed identity assigned to the container registry.
This gist provides a more complete example of using an existing UAMI to assign and pull an image at deploy time.
Although the documentation linked isn't unclear as it doesn't apply to this scenario - suggest an item be opened to add a quickstart template here titled "Use A Managed Identity To Pull An Image From An Azure Container Registry".

Thanks for your contribution to our documentation
We sincerely apologize for the delayed response. Unfortunately, we have been unable to review this issue in a timely manner. However, we are making overall enhancements to our content. We are closing this issue for now as there has been no activity for a while. If you feel that the problem persists, please respond to this issue with additional information.
?
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


Support RequirementType property for feature flags

With Microsoft.FeatureManagement 2.6.0-preview release, feature flags now have an additional property called RequirementType that needs to be parsed by the FeatureManagement adapter. See this PR for the new syntax: https://github.com/microsoft/FeatureManagement-Dotnet/pull/221

This is great and works very well. I do however have some issue with this new property in combination with Azure App Configuration. Is this already supported, and if so, what property do I need to add in that configuration?
I only get it to work at this moment if I add the feature also in mine appsettings.json with the property RequirementType, I can leave the rest of the configuration (filters) in Azure. It would be great if I didn't have to put anything at all in my local appsettings.json.

Hi @Joerio, thanks for the feedback! Right now, it will only work when you specify RequirementType in your config file. We are working on adding this support in Azure App Configuration too.

I have some questions while trying to implement the feature. Probably some clarification required.

Do we always set property FeatureManagement:<FeatureId>:RequirementType, even if requirement_type is not specified in raw data? If so, I think we can use its default value "Any".
What if requirement_type has an invalid value? Do we throw an exception or silently fallback to set "Any" (which is not a good idea from my gut feeling)?

Config provider is responsible for parsing feature flags stored in AppConfig service and adding them to application configuration. Then the Feature Management library consumes these feature flags through IConfiguration. This means that config provider doesn't need to add default value for RequirementType - it just needs to understand how to parse a feature flag if it contains RequirementType. Even if the value of RequirementType is invalid, config provider shouldn't worry about the semantics of RequirementType and just treat it as a free form string. The FeatureManagement library will handle these cases of missing/invalid RequirementType appropriately.

Thank you for the clarification, and it makes a lot sense to me now. Then I think I can simply regard requirement type as a string here in the provider.


Runtime error: java.io.IOException: Socket read timeout

Hi all. I had the script running just fine for a couple hours, then suddenly it started pulling up this error:

Using Nox 6.5.03 with Android 5 and AnkLua 8.6.0-pro2. I've also attached my FGO_EN_REGULAR script.
FGO_EN_REGULAR.txt
EDIT: After relogging into my computer and restarting Nox after a night of sleep, this error stopped appearing and the script runs normally again.

This error comes up occasionally. This has nothing to do with the script so we have no control over it. This error is a result of Ankulua, and thus may be fixed in future Ankulua updates. As you have already figured out, the solution is to stop the service, force close Ankulua, and start it back up. After which, this should go away. If it doesn't though, restarting your device should suffice (doesn't need to be your computer, just Nox)


autoconf: possibly undefined macros

Following the directions in the README, I ran into this set of errors:
configure.ac:3: error: possibly undefined macro: AM_INIT_AUTOMAKE
      If this token and others are legitimate, please use m4_pattern_allow.
      See the Autoconf documentation.
configure.ac:26: error: possibly undefined macro: AC_DISABLE_SHARED
configure.ac:27: error: possibly undefined macro: AC_PROG_LIBTOOL

I was able to satisfy autoconf by running libtoolize:
> libtoolize
libtoolize: putting auxiliary files in `.'.
libtoolize: linking file `./ltmain.sh'
libtoolize: putting macros in AC_CONFIG_MACRO_DIR, `m4'.
libtoolize: linking file `m4/libtool.m4'
libtoolize: linking file `m4/ltoptions.m4'
libtoolize: linking file `m4/ltsugar.m4'
libtoolize: linking file `m4/ltversion.m4'
libtoolize: linking file `m4/lt~obsolete.m4'

Would you prefer a PR with those files committed or with that step added to the directions in the README?

On a mac,
brew install libtool
and running libtool worked for me.


How to log performance metrics in the validation step/loop?

❓ How to log performance metrics in the validation step/loop?
Before asking:

search the issues.   Done
search the docs.    Done

What is your question?
I would like to log the performance metrics of each validation batch using TestTubeLogger (I am open to using alternative loggers) as well as the loss function of each training batch. I am struggling to understand the difference between adding 'log' to the output of my validation step, and including a call to the logger itself through self.logger.experiment.log({k, v}). I have seen both methods implemented in the tutorials and docs, but I do not understand the best practices nor which method to use to achieve my end goal.

Is the call to self.logger.experiment.log({k, v}) mutually exclusive with adding 'log' to the output of a training/validation step? If not, should they be used together and in what circumstance?
Reference What have I tried (3) below; why does the validation_step method not register the entries in the 'log' dictionary outputs? Are the only validation outputs that are logged those from the validation_epoch_end method?
With all of this in mind, how can I log performance metrics such as accuracy, recall, precision, from the validation loop? What are the best practices around this?

What have I tried?

I trained the network without any calls to self.logger.experiment.log({k, v}) and without adding a 'log' key to the outputs of any training/validation step or epoch end methods. This resulted in the creation of a metrics.csv file that contained only a column for the created time and epoch number.

I added in only calls to the logger experiment such that my training and validation code looked like such:

    def training_step(self, batch, batch_nb):
        x, y = batch
        x, y = x.float(), y.float().unsqueeze(1)
        y_hat = self.forward(x)
        loss = self.loss_function(y_hat, y)
        self.logger.experiment.log({'loss': loss})
        return {
            'loss': loss
        }

    def training_epoch_end(self, outputs):
        avg_val_loss = torch.stack([x['loss'] for x in outputs]).mean()
        return {
            'avg_train_loss': avg_val_loss
        }

    def validation_step(self, batch, batch_nb):
        x, y = batch
        x, y = x.float(), y.float().unsqueeze(1)
        y_hat = self.forward(x)
        loss = self.loss_function(y_hat, y)
        metrics = self.performance_metrics(F.softmax(y_hat), y)
        self.logger.experiment.log({'metrics': metrics['accuracy']})
        return {
            'val_loss': loss,
        }

    def validation_epoch_end(self, outputs):
        avg_val_loss = torch.stack([x['val_loss'] for x in outputs]).mean()
        return {
            'avg_val_loss': avg_val_loss,
        }

When running trainer.fit(model) with this arrangement the first few lines of the metrics.csv look like:

I removed all calls to the logger experiment and included only 'log' dictionary keys in the outputs such that the code looked like such:

    def training_step(self, batch, batch_nb):
        x, y = batch
        x, y = x.float(), y.float().unsqueeze(1)
        y_hat = self.forward(x)
        loss = self.loss_function(y_hat, y)
        return {
            'loss': loss,
            'log': {'train_loss': loss, 'batch_nb': batch_nb}
        }

    def training_epoch_end(self, outputs):
        avg_val_loss = torch.stack([x['loss'] for x in outputs]).mean()
        return {
            'avg_train_loss': avg_val_loss,
            'log': {'avg_train_loss': avg_val_loss}
        }

    def validation_step(self, batch, batch_nb):
        x, y = batch
        x, y = x.float(), y.float().unsqueeze(1)
        y_hat = self.forward(x)
        loss = self.loss_function(y_hat, y)
        metrics = self.performance_metrics(F.softmax(y_hat), y)
        return {
            'val_loss': loss,
            'log': {'val_loss': loss, 'metrics': metrics}
        }

    def validation_epoch_end(self, outputs):
        avg_val_loss = torch.stack([x['val_loss'] for x in outputs]).mean()
        return {
            'avg_val_loss': avg_val_loss,
            'log': {'avg_val_loss': avg_val_loss}
        }

Upon running trainer.fit(model) the first few lines of my metrics.csv file look like:

What's your environment?

OS: Ubuntu 18.04
Packaging: Conda
Python: 3.8
Pytorch Lightning: 0.7.5

Additional content
This is how I configure the logger and trainer:

logger = pl.loggers.TestTubeLogger(
    save_dir=config['logging_params']['save_dir'],
    name=config['logging_params']['name'],
    debug=False,
    create_git_tag=False,
)

trainer = pl.Trainer(default_save_path=f'{logger.save_dir}',
	         	min_nb_epochs=1,
                	logger=logger,
                	log_save_interval=5,
	                num_sanity_val_steps=5,
	                **config['trainer_params'])

I am using a binary classifier and compute my performance metrics with the follorwing method:

    def performance_metrics(self, y_hat, y):
        '''0 is typical, 1 is novel, also works with boolean arrays'''
        counts = {'TP': 0, 'FP': 0, 'FN': 0, 'TN': 0}
        for quadrant in zip(y_hat, y):

            if quadrant == (1, 1): # TP
                counts['TP'] += 1
            elif quadrant == (0, 1): # FN
                counts['FN'] += 1
            elif quadrant == (1, 0): # FP
                counts['FP'] += 1
            elif quadrant == (0, 0): # TN
                counts['TN'] += 1

        precision = counts['TP'] / (counts['TP'] + counts['FP'])
        recall    = counts['TP'] / (counts['TP'] + counts['FN'])
        accuracy  = (counts['TP'] + counts['TN']) / len(y)
        f1score   = (2 * precision * recall) / (precision + recall)

        return {
            'counts': counts,
            'precision': precision,
            'recall': recall,
            'accuracy': accuracy,
            'f1score': f1score
        }

It doesn't make sense to log accuracy, precision etc. in the validation step for every batch.
Do it in the epoch_end.
Also more generally logging in validation_step doesn't work with the loggers I know (TB for example) because they can't show these kind of plots. TensorBoard has only a single global step.


Update Release section in Makefile

This commit applies some conventions to the targets and documentation
in the release section. Mostly documenting all the tools we use to
perform releases.

/retest
@Vincent056 fixed up the content images for CI - so we should be good to rerun the e2e tests.


FotN Companion v2.0

I hope it works this time. Please merge this with FotN Companion in the roll20 API script library under System Toolbox

@yehoshua1312 Can you please remove the ._script.json binary file from this PR? Once you do, we can get this merged in. Thank you!

Done


Remove unnecessary DVR to Ref calls for template VIs

Is your feature request related to a problem? Please describe.
Our code review process aims to remove dead code or unnecessary function calls, and there's a lot of that in the SMO template methods. Most of our library code is SMO based and so when it comes time for code reviews we have to do some pretty redundant modification to these VIs.
Describe the solution you'd like
It would be great if the template VIs for all by-reference SMOs were refactored to use DVRs and more efficiently.
There is no need to operate on the Ref in-place, because it is a reference, so the In-place element structure is removed. Then, since we are operating on the DVR in an in-place element structure, there is no need to cast the DVR back to a reference, since the data has been updated in-place. This simplifies both the read and write accessors and allows for parallel reads of private data (if desired).
Additionally, the class data is never modified, so the wire can be passed straight through, simplifying wiring in the error structure.
Screenshots for details:
CreatePrivateEvents.vi
Before:

After:

SMO.BasicByRef.GetPrivateEvents.vi
Before:

After:

Describe alternatives you've considered
Manually editing all templates on my local machine is an option, but in a group of developers that means we end up with different templates, which defeats the point of templates. Of course, now we mark all of this up during our code review process, which tends to use a bit more time than we'd like.
Additional context
We have ~12 SMOs that are by ref and removing this manual work would be great. I'm not sure of any performance gains, but any time you remove code, I'm sure it gets a little bit faster, right?

@nathan-murphy , I checked into the SMO editor code to see if there was a way for you to be able to override with a plugin template, but there is currently none. I also tried to make a modification that would not break the editor in the template VIs. That also failed. (The editor is expressly trying to find the IPE+Unbundler/Bundler combination to act on the template, so it is not really scalable...)
The Editor code is not yet released as an open source project.
@jimkring Is this in the cards?


#776 Add Rust course to interactive Learning

Issue:  #776
Add https://dapp-world.com/course/master-rust-smart-contract-development-on-stellar to Interactive Learning section

Still reviewing course.  Will convert to READY FOR REVIEW after course review is complete.

Preview is available here:http://developers-pr833.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr833.previews.kube001.services.stellar-ops.com


Installation not working

Policy and info

Maintainers will close issues that have been stale for 14 days if they contain relevant answers.
Adding the label "sweep" will automatically turn the issue into a coded pull request. Works best for mechanical tasks. More info/syntax at: https://docs.sweep.dev/

Expected Behavior
Please describe the behavior you are expecting.
I wanted to install gpt-engineer and run it in a folder but it did not work.
Current Behavior
What is the current behavior?
Failure Information
Information about the failure, including environment details, such as LLM used.
Steps to Reproduce
If possible, provide detailed steps for reproducing the issue.

step 1
step 2
you get it...

Failure Logs
Any relevant log snippets or files here.

Hi @PierrunoYT - can you ensure that you are using Python 3.10/3.11? From your screenshot it seems like you are using 3.12?

Yeah I'm using 3.12

You can close it


Add an API to access the packed size of a given type.

This makes it easier to write generic code that uses gopack under the hood without having to hardcode the size of the structs being used by the generic code.
This change also removes a layer of indirection in makePackerWrapper, by having the code check the input buffer is big enough directly in Pack() rather than via an extra anonymous func, which improves most encoding benchmarks by 4 to 6% (the gains are really modest in absolute terms, on the order of a few cycles, as all it's doing really is just removing an extra indirect function call).

I'll have to look over it more closely, but at a first pass it looks mostly good. There's one small issue: we need to keep the anonymous function which panics. The reason is that, even for types which panic, it allows us to only perform the work of generating the packer for that type once (if it immediately, nothing is stored in the cache; the anonymous function can be cached). It's definitely an unlikely edge case, but if it happens, it means difference in performance of multiple orders of magnitude.
Also, I'm not positive that the code that manages locking and unlocking caches would work properly in the event of a panic from a subcall? I can look at it more closely, but since nothing was ever supposed to panic except for at the top-level, I didn't bother thinking about; we should just make sure that's the case even if we decide to stick with panicking immediately rather than the anonymous function.

Generating a packer than panics and caching it: micro-optimization that is useless.  99.9% of the time the caller will not be able to recover from a gopack being unable to generate a packer for a given type.  Also calling panic() is fairly expensive, so caching what calls panic vs doing some useless work before calling panic doesn't make a significant difference.
Regarding your second point: panic will unwind the stack and cause any deferred statement to run.  If a lock is unlocked via defer, it'll get unlocked properly.  In this case, however, we don't even hold the lock when in makePackerWrapper, so we're fine.

OK, I figured that the defers would save us, but I hadn't looked closely
enough to tell.
On Tue, Oct 20, 2015 at 2:29 PM, Benoit Sigoure<EMAIL_ADDRESS>wrote:

Generating a packer than panics and caching it: micro-optimization that is
useless. 99.9% of the time the caller will not be able to recover from a
gopack being unable to generate a packer for a given type. Also calling
panic() is fairly expensive, so caching what calls panic vs doing some
useless work before calling panic doesn't make a significant difference.

Regarding your second point: panic will unwind the stack and cause any
deferred statement to run. If a lock is unlocked via defer, it'll get
unlocked properly. In this case, however, we don't even hold the lock when
in makePackerWrapper, so we're fine.
—
Reply to this email directly or view it on GitHub
https://github.com/synful/gopack/pull/7#issuecomment-149657500.


Unexpected token error with simplest typescript code

Hi, guys!
I am trying to use Angular2 typescript plugins with my Angular2/Dart web app, but I've got some errors even with hello world simplest example.
For example such code
class Greeter {
    constructor(public greeting: string) { }
    greet() {
        return "
" + this.greeting + "
";
    }
};

var greeter = new Greeter("Hello, world!");

produces error
[12:08:59] Error: ts2dart-test/hello-world-simple.ts:1:1: Could not format because the source could not be parsed:

line 1, column 2: Unexpected token 'class'
 class Greeter { String greeting ; Greeter ( this . greeting ) { } greet ( ) { return "
" + this . greeting + "
" ; } } ; var greeter = new Greeter ( "Hello, world!" ) ;
 ^^^^^
    at Transpiler.reportError (/source/main.ts:280:38)
    at Transpiler.formatCode (/source/main.ts:214:12)
    at Transpiler.translate (/source/main.ts:208:17)
    at /source/main.ts:137:31
    at Array.forEach (native)
    at Transpiler.transpile (/source/main.ts:136:10)
    at Gulp.<anonymous> (/.../gulpfile.js:15:16)
    at module.exports (/.../node_modules/orchestrator/lib/runTask.js:34:7)
    at Gulp.Orchestrator._runTask (/.../node_modules/orchestrator/index.js:273:3)
    at Gulp.Orchestrator._runStep (/.../node_modules/orchestrator/index.js:214:10)

I'm using<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>and gulp to run transpilation:
const ts2dart = require('ts2dart');

gulp.task('ttt', () => {
    const transpiler = new ts2dart.Transpiler({
        generateLibraryName: false,
        failFast: true
    });
    transpiler.transpile([
        'ts2dart-test/hello-world-simple.ts'
    ], './temp');
});

Could you please help me to figure out what is wrong?

Sorry that the error is this obscure (dartfmt is not very resilient). The problem is that the var greeter = new Greeter("Hello, world!"); part of your code cannot be translated to valid Dart. You cannot initialize variables in the global scope like that, they must be in some function body.
Let me know if that helps, and please reopen if it doesn't!

Hi, mprobst!
Thanks for your answer. I've tried your suggestion, but still have the error.
ts file:
class Greeter {
    constructor(public greeting: string) { }
    greet() {
        return "
" + this.greeting + "
";
    }
};

function test() {
    var greeter = new Greeter("Hello, world!");
}

error:
$ gulp compile-ts
[13:29:54] Using gulpfile ~/dev/test-gulp-ts2dart/gulpfile.js
[13:29:54] Starting 'compile-ts'...
[13:29:54] Finished 'compile-ts' after 10 ms
[13:29:54] test.ts:1:1: Could not format because the source could not be parsed:

line 1, column 17: Unexpected token 'class'
 library test ; class Greeter { String greeting ; Greeter ( this . greeting ) { } greet ( ) { return "
" + this . greeting + "
" ; } } ; test ( ) { var greeter = new Greeter ( "Hello, world!" ) ; }
                ^^^^^
[13:29:54] ts2dart transpilation failed.

.../dev/test-gulp-ts2dart/node_modules/gulp-ts2dart/index.js:48
          throw new Error('ts2dart transpilation failed.');
                ^
Error: ts2dart transpilation failed.
    at DestroyableTransform.finished [as _flush] (.../dev/test-gulp-ts2dart/node_modules/gulp-ts2dart/index.js:48:17)
    at DestroyableTransform.<anonymous> (.../dev/test-gulp-ts2dart/node_modules/gulp-ts2dart/node_modules/readable-stream/lib/_stream_transform.js:135:12)
    at DestroyableTransform.g (events.js:260:16)
    at emitNone (events.js:72:20)
    at DestroyableTransform.emit (events.js:166:7)
    at finishMaybe (.../dev/test-gulp-ts2dart/node_modules/gulp-ts2dart/node_modules/readable-stream/lib/_stream_writable.js:371:12)
    at endWritable (.../dev/test-gulp-ts2dart/node_modules/gulp-ts2dart/node_modules/readable-stream/lib/_stream_writable.js:378:3)
    at DestroyableTransform.Writable.end (.../dev/test-gulp-ts2dart/node_modules/gulp-ts2dart/node_modules/readable-stream/lib/_stream_writable.js:356:5)
    at DestroyableTransform.onend (.../dev/test-gulp-ts2dart/node_modules/vinyl-fs/node_modules/readable-stream/lib/_stream_readable.js:523:10)
    at DestroyableTransform.g (events.js:260:16)

What should I try now?

The problem is the trailing semicolon after the class statement, Dart doesn't allow empty statements in the body of a Dart file. You can debug this more easily by taking the code from the error message and pasting it into e.g. dartpad – apparently most other Dart parsers have better error reporting.
On a more general note, ts2dart was built as a specialized tool to translate the Angular 2 code base. It's not designed to work on arbitrary TypeScript code bases, and I suspect you'll run into lots of issues trying to use it. I wouldn't recommend it as a way to consume arbitrary TypeScript libraries from Dart. On the other hand, I don't know of any other way.

The problem is the trailing semicolon after the class statement, Dart doesn't allow empty statements in the body of a Dart file. You can debug this more easily by taking the code from the error message and pasting it into e.g. dartpad – apparently most other Dart parsers have better error reporting.

It works now after removing the semicolon!
Also dartpad reports the error correctly. Thanks!

On a more general note, ts2dart was built as a specialized tool to translate the Angular 2 code base. It's not designed to work on arbitrary TypeScript code bases, and I suspect you'll run into lots of issues trying to use it. I wouldn't recommend it as a way to consume arbitrary TypeScript libraries from Dart. On the other hand, I don't know of any other way.

Yes, it was my goal to make some TypeScript plugins (ng2-select, ng2-modal, angular2-hot-loader) to work with Angular2-Dart application. Instead I will try to use ports of these plugins.
@mprobst , thanks a lot!


v0.96.8

Q
A

Needs wipe?
no

Description:
Pull Request for the next update.
Currently in draft state as a "not binding ToDo list".
We might add or remove some points in the following list during the work on that update.
Content:

[ ] (Re)organize the structure of the files/folders (scripts folder and reorganize functions from functions folder)
[ ] Equalize names of global variables in the whole framework
[x] Add proper headers to presets
[ ] Refactor scripts to functions/FSMs and optimize the code

[ ] Sector handling (spawn, capture, despawn)
[ ] Logistic (manage_logistics.sqf)
[ ] Production (manage_resources.sqf)
[ ] Battlegroups/Reinforcements (server/battlegroup folder)
[ ] Random patrols in enemy territory (server/patrols folder)
[ ] Building dialog functions (open_build_menu, do_build, etc.)
[ ] POW / Informant
[ ] Mission initialization (init, init_server, init_client, init_shared)

[ ] Add config check like already implemented in the 0.97.0 rewrite. Example from 0.97.0
[ ] Move preset initialization to server and just publish sorted preset arrays etc. to clients. Example from 0.97.0
[ ] (Don't know if added, just a thought) Merge player/enemy preset files to a universal pattern like in 0.97.0, to just have to create one preset file to use it for both sides. Example from 0.97.0
[ ] Add AA/Mortar spawning to enemy bases as constant units (maybe with dynamic simulation) to create a more immersive threat in enemy territory. And to have a strategic approach to take towers and bases before attacking a city (which would cause mortar support from bases and angry civs for destroyed buildings)
[ ] Add a carry function as replacement for the crate pushing (enabled, if no ace is loaded)
[ ] Add a similar function to escort prisoners. Example from 0.97.0
[ ] Add KP cratefiller for the "Salvage Depot" and rename it to a logistic station or similar
[ ] Improve JIP remoteExec by not using objects as jipID as it's not stackable
[ ] Ensure reinizialization of Zeus modules and/or create zeus completely via scripts like in 0.97.0.
[ ] Remove trigger zones in SQMs, play music via script and open respawn dialog via EH.

[ ]  Add AA/Mortar spawning to enemy bases as constant units (maybe with dynamic simulation) to create a more immersive threat in enemy territory. And to have a strategic approach to take towers and bases before attacking a city (which would cause mortar support from bases and angry civs for destroyed buildings)

Maybe instead of Dynamic Simulation, what if these units were "made virtual"? (for example what ALiVE mod does)
I could see these units being all spawned on the first sector activation of the military base, and when said base sector deactivates save "all static units are still alive". When the sector is activated once again then spawn back only the alive units once more.
I also suggest to NOT save the state of "constant units" between server restarts.


An Iranian ventilator (couldn't find repo link at first glance)

https://spectrum.ieee.org/the-institute/ieee-member-news/engineers-iran-open-source-ventilator?utm_source=techalert&utm_medium=email&utm_campaign=techalert-04-16-20&mkt_tok=eyJpIjoiTm1WaU1XVTROVFJoWlRNeSIsInQiOiJPRmpObVdieDVSMVcyeTlicnBJRXlqNjVUR1wvcGtjWEdoQit2S25neGVkVVpOeXl2WTJmazc0Yzh3KytPTkRMVGdyem11WGZXUDlTTDhOcjNlMEV0VDhUVmlpOHdrNndSUjl4ZSt6cWF1S1pLK3JmUVFBa0F0VjNyVm4wSTRZRDIifQ%3D%3D

Dr. Read,
It seems this isn't open as of yet.
"We are aiming to make the design open source and will let other IEEE regions know. Since building this machine involves a combination of electrical, mechanical, and bioengineering, it is important to have members from all these field work together to build it."


LightSwitch is Not Compatible with Azure SDK 2.7.1

Updated the known issues list for the Azure SDK 2.7.1 release notes to indicate that LightSwitch is not compatible with this release.

Hi @OptikSage, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;


GH-41310: [C++] S3FS Read file using the version id obtained with HEAD call

Rationale for this change
GET calls to s3 is not made using the version id (if available) from the HEAD call (which is used to get the content-length). If the version is available, using that makes the GET call more reliably return the file data without running in to race condition, if a write was intercepted between the HEAD and GET.
What changes are included in this PR?
s3fs.cc
The changes include capturing the version if available from the HEAD call and using it subsequently in the GET requests.
This doesnt need any specific tests. The existing tests should cover for regression.

GitHub Issue: #41310

Would it be possible to write a test that reproduces the issue? Possibly by writing a first version, then spawn a thread that writes a second version while reading the first version from the main thread.

@gitfy Would you like to answer the question above? For the record, I think this is a good improvement.


GLKit bug fixes and new unit tests

GLKit bug fixes for several issues. Specifically:
Issue #547
Issue #548
Issue #549
Issue #550
Issue #552
Also updated unit tests to:

Provide extra coverage of GLKit functions
Include golden outputs of GLKit functions running on iOS
Include test cases which compared golden iOS outputs against WinObjC outputs

Hi @shahmeeresmail, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

@jaredh is added to the review. #Closed

@jaredhms is added to the review. #Closed

@bdrlamb-ms is added to the review. #Closed

:shipit:

samples/KVOPerf/KVOPerf.vsimporter/KVOPerf-WinStore10/Assets.zip should not be in this change.  could you please remove that and submit? looks good otherwise.

also samples/WOCCatalog/WOCCatalog.vsimporter/WOCCatalog-WinStore10/Assets.zip dont seem necessary as you have the extracted files?

In reply to: 227616427 [](ancestors = 227616427)

in CI build.


Translate blockly mutator in "if" and "create array with" blocks

The "if", "else if", "else" and "value" blocks in the blockly mutators are not translated.

Closing this as moving to v1 we don't have mutators.


undefined local variable or method `cama_forgot_path'

When I tried to reset password for admin on local machine and copied link from console to browser I get:
undefined local variable or method `cama_forgot_path' for #CamaleonCms::Admin::SessionsController:0x00007f3a9ad21d48 Did you mean? cama_root_path cama_robots_path cama_post_path cama_error_path

Fixed!


🚀 Feature Request: Add wrangler support for ARM 32/64

Describe the solution
I run wrangler on a Raspberry Pi to collect logs, though I had some trouble figuring out which version I needed. Someone in the community forum helped me discover that I needed wrangler version 2, not 1 or 3.
I was advised to open an issue here and request that wrangler 3 support ARM 32/64 devices the Pi's OS. Mine says "Raspbian GNU/Linux 9.13 (stretch)". I'm sure that's outdated but I also understand wrangler 3 currently only supports a limited set of the latest operating systems, and it doesn't look like the Pi's OS is in that set.

Closing in favour of https://github.com/cloudflare/workers-sdk/issues/3457


blank background after update react-native

it's blank background what ever I set backgroundColor,I just think it's woring with my code,but when I create a new react-native project and use the demo code,it's blank background too. any one meet the same problem?

Hi @inu1255, which version of react-native are you using and can you post some screenshots of this? It might help placing the backgroundColor as a style prop instead of placing it in the chart config object.

it's great,thank you,I set style prop of LineChart like this style={{backgroundColor:"#fff"}} and it become a white background.
this is the black background,I use the demo code and edit nothing.

package.json
{
  "name": "Test",
  "version": "0.0.1",
  "private": true,
  "scripts": {
    "start": "node node_modules/react-native/local-cli/cli.js start"
  },
  "dependencies": {
    "react": "^0.14.8",
    "react-native": "^0.24.1",
    "react-native-ios-charts": "^0.2.3"
  }
}

Hello.
I'm using React Native 0.25 and when I run the examples I get a blank box where the chart should be displayed. Is anyone else running into this issue?
Thanks,
Christian

Just tested on react-native 0.24.1 & 0.25.1 and the ChartsExplorer worked fine for me.
React Native 0.24.1

React Native 0.25.1


Exception: Calling thread cannot access this object because a different thread owns it

Error log: https://gist.github.com/enzy/2bf4bff2741ba808898b
This happen every time Wox start.
Wox 1.1.1
Win 8.1 x64 (and also in Windows 10 preview build)

Strange, I tested on Win8.1 and didn't has this error.

try v1.2beta

I'm on 1.2 beta for some time and haven't encounter this bug yet. Thanks


Should "All Contacts" be renamed to "All People"?

This will make it clearer to the user that the main focus is on people and relationships, and not contacting them, even if it is still an address book in effect.
Same with Found Contacts.

not necessary


how to execute a function when validations for a field are completed

Versions:

VueJs: 2.5.9
Vee-Validate: 2.0.0-rc.26

Description:
I may be missing something, but i can't find a way to have a function called when validations for a field are completed, with the validation result as a parameter. These validations are triggered by vee-validate (input, blur with delay), not programmatically by my code. A validation event maybe something like @afterValidate ?
It would be also very useful to have all the results of all the validators (chain of validators) in an array as the parameter.  And not only a boolean variable result, but entire objects, for example a custom validator i created not only returns true or false, but some extra data also.
thanks in advance

This is my use case:
When user inserts a valid post code in a field, i want to populate a dropdown with areas corresponding to this post code.
Right now, i have an async custom validator for postCode, which also returns all the areas for this postCode (single request to the server).
The only solution i can think of, without the feature i asked, is to directly mutate the state (vuex) from inside the custom validator! Which is far from ideal, as it's not validator's responsibility to mess with the state. And if i wan't something similar with a built-in validator, then i wouldn't have any choice.

@nkwinder Having the same struggle. How did you solve this for now? I'm re-using the same validator for different screens and want to set a BIC when the IBAN is valid. Would love to hear some implementations ideas :)

Thanks for the response @nkwinder


Adding check for setting file in image on render

- What I did
Added a check on render that if a file passed with -f somefile.yml exists in the app dir, it will use that file and if not, pass it as is to the LoadFiles function for processing.
fixes #332
- How I did it
Added an for range and an os.Stat into the render function for passed settings files.
- How to verify it
$ docker-app render -f somefile.yml - for a file that exists within the dockerapp folder.
- Description for the changelog

Adds a check for files included in the .dockerapp folder so files included in images can be referenced and included.
- A picture of a cute animal (not mandatory but encouraged)

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "settings-from-tmp" git@github.com:ecwillis/app.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

@ecwillis thanks for the proposal :hugs: I do like the idea of being able to load settings other than the default from the image, I'm just a bit worried about the magic trick of it as proposed here.
Maybe we want to way to specify that we want to read those particular setting files from the image, with something like -f image:prod.yml or something.
cc @thaJeztah @garethr

Don't this feature should also be added to the docker-app deploy command?

@vdemeester That could work and would be easy to implement. Though the magic is not really that magic. It simply checks to see if the file exists inside the temporary working dir, and if it doesn't assumes that it must be a regular file path - the only problem I see with that logic is potential filename collisions ie. if someone wants to use a file prod.yml in the directory they are working from but there is also a prod.yml in that temp working dir it would not load both.

@silvin-lubecki - the docker-app deploy command uses render.Render to do the actual rendering of the compose file including loading the -f arguments.

@silvin-lubecki - the docker-app deploy command uses render.Render to do the actual rendering of the compose file including loading the -f arguments.

Oh right 😅

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "settings-from-tmp" git@github.com:ecwillis/app.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842362793392
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

@chris-crone @garethr what is the status of this PR ?


Requests flagged and blocked

I was testing this API since OpenAI removed the free credits for DALL-E. I implemented a request using the generateImagesLinks method and it fails with the following response.
"This prompt has been blocked by Bing. Bing's system flagged this prompt because it may conflict with their content policy. More policy violations may lead to automatic suspension of your access."
I'm not sure if Bing is blocking this request because its determine the origin is not from the browser or something else. The prompt itself was very innocuous so there shouldn't be any restrictions based upon what was requested.
I set the massive cookie in my .env file but still no luck. Any ideas?

I managed to get around this by adding "unsafe-hashes" and "unsafe-inline" to the prompt, but I still get random blocks. It would be nice if the author would take the time to comment. Cheers!

Have you tried directly copying and pasting the same prompt into the web version? Sometimes Bing rejects some fairly innocuous prompts (see here #1)

I'm having the same issue lately with innocent prompts like "Illustration showing a man eating a red apple, white background".  It seems that everything starts working after I send at least one prompt through the bing site (even though the cookie value hasn't changed for weeks). From that point it works through bimg even from a different IP.


Configure webpack build

Changes Unknown when pulling 1eaa10297dd9ebcb2c0eedeb59c54f5cc2d24b2f on setup-project into ** on master**.

fixes #1