bug: increment helios version to fix hurl tests

Pull Request type

Please check the type of change your PR introduces:

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Testing
[ ] Other (please describe):

What is the current behavior?
When running hurl tests, helios panics and makes beerus crash.
Issue Number: #367
What is the new behavior?
The hurl tests pass, and all tests run with cargo are in green.
Does this introduce a breaking change?

[ ] Yes
[X] No

Other information
This was fixed recently by the new release of helios in branch master commit: 1ae57f1b009efd33365d8fdbb61fde13354fa00b.

Important to mention that we must also increment ethers. As we refer to it explicitly, there are types conflicts if only helios is upgrade to 0.4.0.

When running beerus i now have a big amount of logs:

Is that something introduced by your pr?

Also getting this new warning:

Since update

Hurl are passing and tests are passing 👍

When running beerus i now have a big amount of logs:

Is that something introduced by your pr?

I am not sure because I never run a hurl test before, but it looks like the common behavior. Perhaps we have to revise the log level for some output.
And the warning was already present in the Helios codebase. With the PR I made to Helios, this warning will only be seen at the first launch of beerus.

When running beerus i now have a big amount of logs:

Is that something introduced by your pr?

I am not sure because I never run a hurl test before, but it looks like the common behavior. Perhaps we have to revise the log level for some output.
And the warning was already present in the Helios codebase. With the PR I made to Helios, this warning will only be seen at the first launch of beerus.

Try running the main, you will see that this is not normal behavior hence my analysis is that logs + warning message comes from the helios update, do you agree?

Remove this print statement to get rid of that verbose block data https://github.com/keep-starknet-strange/beerus/blob/main/crates/beerus-core/src/lightclient/beerus.rs#L160

Comment it out. what do you think @drspacemn it's your pr. Surprising that i didn't see these logs before.

Remove this print statement to get rid of that verbose block data https://github.com/keep-starknet-strange/beerus/blob/main/crates/beerus-core/src/lightclient/beerus.rs#L160

Comment it out. what do you think @drspacemn it's your pr. Surprising that i didn't see these logs before.

I switch from println to debug! to keep the possibility of dumping the block if needed for debug purposes.

I'm still seeing "Empty reply from server" running hurl starknet_getClassAt.hurl even with the update to helios and ethers.

Imo this is not related to helios or ethers version. Possibly wrong params or research needed

lgtm, merging


[SPARK-22093][TESTS] Fixes assume in UtilsSuite and HiveDDLSuite

What changes were proposed in this pull request?
This PR proposes to remove assume in Utils.resolveURIs and replace assume to assert in Utils.resolveURI in the test cases in UtilsSuite.
It looks Utils.resolveURIs supports multiple but also single paths as input. So, it looks not meaningful to check if the input has ,.
For the test for Utils.resolveURI, I replaced it to assert because it looks taking single path and in order to prevent future mistakes when adding more tests here.
For assume in HiveDDLSuite, it looks it should be assert to test at the last
How was this patch tested?
Fixed unit tests.

@vanzin, Could you take a look when you are available please?

Test build #82114 has started for PR 19332 at commit beaedb0.

Will check other assumes too soon and be back.

I checked other instances of org.scalatest.Assertions.assume. I am quite sure of the ones here I added. I didn't add ones I am less sure of.

Test build #82117 has started for PR 19332 at commit c32f65b.

Test build #82114 has finished for PR 19332 at commit beaedb0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82114/
Test PASSed.

Test build #82117 has finished for PR 19332 at commit c32f65b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82117/
Test PASSed.

Merged to master.
Thank you @srowen and @dongjoon-hyun.


Add autocomplete

Something like:

As I understand, this is different from the spellcheck module which fixes search input lexical mistakes, but does not suggest keywords/sentences from the indexed text corpus.

Just to clarify:
You are interested in "keyword matching" autocomplete, i.e. because the word huggingface is indexed, typing hugg should suggest it. This is something that could make use of the inverted index in Weaviate.
The reason I'm asking this is because the alternative would be using a neural model to predict the next word. E.g. something like typing Where can I buy and the model would predict umbrellas next. This is something that would require fine-tuning a model to the user's dataset first as it would act purely on training data, not necessarily on what the user has imported.

Both solutions look interesting. Of course, predictions are far more impressive. Can the training be somehow be "pre-integrated", like by calling some auto-ML solution (huggingface or other)? Perhaps some new modules "xxx fine-tuning"?

You are interested in "keyword matching" autocomplete, i.e. because the word huggingface is indexed, typing hugg should suggest it.

This.
The prediction request could be a separate issue. Though, umbrellas should also be indexed. No need to suggest terms which will have 0 results.

Keep open. This will be on the soon-to-be-released roadmap.


JsonSerializer the GetUserTimelineAsync

Hi,
i am new to tweetinvi..
How can I JsonSerializer the content of GetUserTimelineAsync?
Also do you have a sample class of the fields?
Thanks,
Jassim

What i am trying is to put the output of all the tweets of an account e.g. ("Microsoft") inside a ListView like this:
List<MicrosoftTweets> data = JsonSerializer.Deserialize<List<MicrosoftTweets>>(result);
ListViewTweets.ItemsSource = data;

This is documented here: https://linvi.github.io/tweetinvi/dist/twitter-client/json.html
Hope this helps.


Flask plugin breaks DeclarativeBase

The plugin breaks the DeclarativeBase, e.g.:
from typing import Any

from flask_sqlalchemy import SQLAlchemy
from sqlalchemy.orm import DeclarativeBase
from sqlalchemy.types import JSON

class Base(DeclarativeBase):
    type_annotation_map = {dict[str, Any]: JSON}
db = SQLAlchemy(model_class=Base)

Gives the error:
sqlalchemy.exc.InvalidRequestError: Declarative base class has both a 'registry' attribute and a type_annotation_map entry.  Per-base type_annotation_maps are not supported.  Please apply the type_annotation_map to this registry directly.
However, type_annotation_map is supported on the base, but by subclassing the base class (something which seems to be happening inside the plugin), it now detects both the registry created from the base class with type_annotation_map and the type_annotation_map. This doesn't happen without the Flask plugin.
Environment:

Python version: 3.11.6
Flask-SQLAlchemy version: 3.1.1
SQLAlchemy version: 2.0.31

Yep, I get the same - unsure what the right way around this is :(

Are you able to use Flask-SQLAlchemy-Lite for your use case? https://flask-sqlalchemy-lite.readthedocs.io/en/latest/

Theoretically yes - but if that's the official advice it's probably be good to hear that and see this deprecated? If this is still being maintained then it feels like there should be an answer to this question.
Switching to FSL is probably not a hard migration, but it's a chunk of work obviously still. And FSL is a very young lib in comparison.

Hello everyone. I stumbled upon this issue myself, and found the following workaround (snippet from my codebase):
db = SQLAlchemy()
db.Model.registry.update_type_annotation_map(
    {
        datetime: sa.DateTime(timezone=True),
        dict[str, Any]: JSONB().with_variant(sa.JSON, "sqlite"),
        uuid.UUID: UUID(as_uuid=True).with_variant(SQLiteUUID(), "sqlite"),
    }
)

This seems to solve the issue for me, hope it will help if anyone else is having this issue :)


ARM5 function return not recognized

Binary Ninja Version: 2.0.2153-dev Personal, b5661dc1
Platform: Windows 10 Version 1909
ARM5 disassembly:
defopsys:
0000acc4  0dc0a0e1   mov     r12, sp {arg_0}
0000acc8  00d82de9   push    {r11, r12, lr, pc} {var_4} {var_8} {var_c} {arg_0} {__saved_r11}
0000accc  04b04ce2   sub     r11, r12, #0x4
0000acd0  04309fe5   ldr     r3, data_acdc  {0x13d6c, "Linux"}
0000acd4  0300a0e1   mov     r0, r3  {0x13d6c, "Linux"}
0000acd8  00a89de8   ldm     sp, {r11, sp, pc} {__saved_r11} {__saved_r11}

ARM5 MLIL:
defopsys:
int32_t var_4 = pc
int32_t var_8 = lr
void* var_c = &arg_0
int32_t r0 = 0x13d6c  {"Linux"}
void* sp = var_c
int32_t temp0 = [sp].d
void* sp = sp + 4
jump(temp0)

ARM6 disassembly:
defopsys:
0000ac80  0dc0a0e1   mov     r12, sp {arg_0}
0000ac84  00d82de9   push    {r11, r12, lr, pc} {var_4} {var_c} {arg_0} {__saved_lr} {__saved_r11}
0000ac88  04b04ce2   sub     r11, r12, #0x4
0000ac8c  0c309fe5   ldr     r3, data_aca0  {0x1751c, "Linux"}
0000ac90  0300a0e1   mov     r0, r3  {0x1751c, "Linux"}
0000ac94  0cd04be2   sub     sp, r11, #0xc
0000ac98  00689de8   ldm     sp, {r11, sp, lr} {__saved_r11} {__saved_r11} {__saved_lr} {__saved_lr}
0000ac9c  1eff2fe1   bx      lr

ARM6 MLIL:
defopsys
int32_t var_4 = pc
void* var_c = &arg_0
int32_t r0 = 0x1751c  {"Linux"}
void* sp = var_c
return 0x1751c  {"Linux"}

Another ARM5 return instruction not recognized, pop {.., pc}:
ARM5 disassembly:
__GI_getgid:
0000e47c  10402de9   push    {r4, lr} {var_4} {var_8}
0000e480  c80090ef   svc     #0x9000c8
0000e484  010a70e3   cmn     r0, #0x1000
0000e488  0040a0e1   mov     r4, r0
0000e48c  0300009a   bls     0xe4a0
0000e490  81f8ffeb   bl      0xc69c
0000e494  003064e2   rsb     r3, r4, #0
0000e498  003080e5   str     r3, [r0]
0000e49c  0040e0e3   mvn     r4, #0
0000e4a0  0400a0e1   mov     r0, r4
0000e4a4  1080bde8   pop     {r4, pc}

ARM5 MLIL:
__GI_getgid:
int32_t var_4 = lr
int32_t var_8 = r4
undefined

Seems to me like ldm sp, {..., pc}  and pop {.., pc} are not recognized as return instructions?
ARM5: https://www.virustotal.com/gui/file/73fb82db761da98edbf3e62e625f2ba17aea64b3e1de5b33238e08abdcc5b67a/detection
ARM6: https://www.virustotal.com/gui/file/9d9fdf0a8ee11c4cda3d9af842ef992aa7dff9d30054de61c79cdca387b3ffc9/detection
I can provide the samples if needed.
Thanks :)
EDIT: added another ARM5 return function not recognized

I think we actually handled it correctly. Since we are terminating the function there which indicates that we do treat it as a function return.
I am closing this for now. Please feel free to re-open it if I am wrong.


Issue #2094 Added Windows Path Matching for Version Replace

Please see:
https://github.com/ractivejs/ractive/issues/2094

@ISNIT0 This doesn't merge well with current dev branch, there are nested conflicts that are really confusing. Do mind updating and pushing again? Then I'll pull and check it out on mac

@martypdx Hmm, no idea where that came from, it was all fresh... I'll do it when I get into work tomorrow.

Thanks, have resolved the conflicts and merged this


Basic: adjust the reported subprocess id

Adjust the reported error to use the first string in the command line.
This is needed to ensure that we use the correct value on Windows.  THis
allows the Windows build to get further.

@swift-ci please smoke test

@swift-ci please smoke test


feat: add expiry notices to connection cards

I've addressed @bumi's feedback and tried to optimize the UI a bit.
Great refactoring @rolznz, thanks for splitting up the components! :+1:


clear doesn't clear anything

Hello
When I call clear (on map resize to recreate the chart), the chart is not really removed (the  and its content persist, only the path is removed).
Léo

Hi,
that´s the intention. To clear the displayed profile.
BR
Felix


ExecBytes

Any chance we could make Schema.Exec accept byte slices instead of just strings with a new method to avoid unnecessarily copying the query string when it's a []byte?
func (s *Schema) ExecBytes(
  ctx context.Context,
  queryString []byte,
  operationName []byte,
  variables map[string]interface{}
) *Response

The performance gains aren't very high (only 200 ns/op in the worst case on x86_64 @ ~3.8Ghz) but it'd still be convenient I suppose? Not sure how difficult it'd be and whether a few hundred nanoseconds are worth the effort.

I even wonder why not a io.ReadCloser

Since this is only executed once at app start time, this isn't really a hight priority. I'm closing the issue for now. This would be a breaking change with existing library if we change it and I do not want to introduce more than one of invoking this.
If we introduce more than one API method for doing the same thing, then people will start using it and we'll have to maintain it. Smaller API surface is always better.


Sign in form can't take mobile numbers as login username

Describe the bug
In Login Form while creating a new account there is a option of entering either Mobile Number or email

But we can't enter the mobile number as a login credential because it only accepts email id as a credential

Expected behaviour
Option 1:-
Either We can simply write to enter Email id in login form so that it only accept for email id.
Option 2:-
We can manipulate the code such that it can accept both number as well as email id for login Credential.
Screenshots
No response
Additional context
No response

Hey @Kanika637 can you assign this issue to me as a hactoberfest contribution.

thanks for opening the issue @mecskyverse, Go ahead🚀

Hey @Kanika637 I was going through the feature to add phone number authentication but as this project's authentication is supported by firebase and when I researched about it I get to know I need to login in firebase account for which I need to login in google account of yours and which is not appropriate. To add phone number authentication in this project. I wanted your help to know if there is any other way or should I remove sentence of entering phone number something like this.


MatSelectionList should update option selection state when list of options changes

What is the expected behavior?
MatSelectionList should update option selection state when list of options changes
What is the current behavior?
If you change the list of options after the view has been initialised, MatSelectionList does not update the model of the new options to reflect that they are selected in the model.
What are the steps to reproduce?
https://stackblitz.com/edit/angular-material2-issue-ptk4tc?file=app/app.component.ts
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Latest StackBlitz template
Is there anything else we should know?
MatSelectionList should listen to options.changes on the options QueryList: https://github.com/angular/components/blob/fc0b971509d0ec0b4a227aa403c31a5f5b0e6f26/src/material/list/selection-list.ts#L323
This is how MatSelect works:
https://github.com/angular/components/blob/9eeb4b5aa0473c0815a73bfe3c2ed6164d86a3b3/src/material/select/select.ts#L550

One problem is that MatSelectionList does not internally keep track of its selection state - instead it delegates this to each corresponding option item (except for _tempValues which is only used for initialisation). I think that to fix this bug we would need to introduce a longer-lived version of _tempValues that can be consulted whenever an option item is added.


test: fix fuzz test

Codecov Report

Merging #30 (52c5e64) into develop (e63b289) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop      #30      +/-   ##
===========================================
+ Coverage    78.73%   78.76%   +0.02%
===========================================
  Files           12       12
  Lines         2370     2373       +3
===========================================
+ Hits          1866     1869       +3
  Misses         504      504

Flag
Coverage Δ

unittests
78.76% <100.00%> (+0.02%)
:arrow_up:

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

Impacted Files
Coverage Δ

src/utils.c
90.90% <100.00%> (+0.11%)
:arrow_up:

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


Moveable: Update all elements together after group actions.

Summary
Adds group actions of Moveable to the history as one entry.

Relevant Technical Choices
Uses updateElementsById with an updater to update all the elements at the same time instead of using it inside loop.

To-do
Integration test would be good in the future.

User-facing changes
When the user resizes/drags/rotates a group and then clicks Undo/Redo, the Undo/Redo will be applied to all the elements together now instead of one by one.

Testing Instructions

Add a few elements.
Multi-select.
Drag these elements around
Use Undo-Redo
Verify that Undo-Redo applies to all the elements at the same time in the multi-selection (as opposed to first Undoing the action for one element, then for the second, etc.)
Verify the same for Resizing.
Verify the same for Rotation.

Fixes #1657

There are already some Jest tests for this in assets/src/edit-story/components/canvas/test/multiSelectionMovable.js. They now need to be updated as they currently expect multiple updateElementsById calls.

They now need to be updated as they currently expect multiple updateElementsById calls.

Yep, I'm actually not sure how to test this now, looking into it.

At some point we can have tests that actually verify if one history record is created or if it works as expected in the UI.

The easiest way would be to click the undo button after multi-element transformation - if both are returned to old transform, it works.

@merapi Note that this issue had already been moved to Ready for QA previously. Per new QA process once a PR is approved it doesn't get merged right away but just moves to "Ready for QA".
It's currently failing Travis now, could you please fix Travis and consent to Google to make CLA sign positive, too?

@merapi Note that this issue had already been moved to Ready for QA previously. Per new QA process once a PR is approved it doesn't get merged right away but just moves to "Ready for QA".
It's currently failing Travis now, could you please fix Travis and consent to Google to make CLA sign positive, too?

Sorry I missed the status. On the bright side, it doesn't affect QA work (if it was already started). I will clean up the tests.

@googlebot I consent.

Sorry I missed the status (Update: I guess it was changed after my push?). On the bright side, it doesn't affect QA work (if it was already started). I will clean up the tests.

It was moved to QA yesterday already, however, for some reason only the issue updated and not the PR so it wasn't visible on the PR. (Did some additional changes to update it on the PR later, after your push) Oh, well... Next time.


Should we allow for 'function' in place of 'transform'?

function is used more in programming and transform may not be obvious to remember when writing these.
Current:
set foo to transform
    return "bar"
end

Allow:
set foo to function
    return "bar"
end

Added in upcoming commit


MB-11915 customer select closeout office

Jira ticket for this change
Summary
This PR adds functionality to allow Army and Air Force users to search for, select, and save a closeout office to a move in the internal/customer app. This functionality is included when both creating and editing a PPM shipment.
Setup to Run Your Code

💻 You will need to use three separate terminals to test this locally.
Terminal 1
Start the Storybook locally.
make storybook

Terminal 2
Start the UI locally.
make client_run

Terminal 3
Start the Go server locally.
make server_run

Validation/Testing
The most straightforward way to test this PR is to create two new users in the internal/customer app, one of branch Army or Air-Force, the other of any other branch.  Continue through the profile and orders setup until you are able to add and update PPM shipments.
Army/Air-Force users should be able to:

[ ] View the Closeout Office section in the PPM shipment form
[ ] Search for a Closeout Office
[ ] Select a Closeout Office
[ ] Save Closeout Office by submitting the form when all required fields are populated
[ ] Closeout Office is required to allow submission of the PPM shipment form
[ ] Update the Closeout Office of a previously created PPM shipment that can still be edited
[ ] Update Closeout Office on a shipment applies to all shipments for the current move

All other branch users should:

[ ] Not see the Closeout Office sections/select
[ ] Have validation and submission behavior unchanged

Videos
Army/Air-Force user can search for, select, and save a new closeout office:
https://user-images.githubusercontent.com/62263329/212374192-d4c8230b-ad2e-4c47-afff-4012104febec.mov
Navy user does not see closeout office section, validation and submission work as before:
https://user-images.githubusercontent.com/62263329/212374229-ea0d619f-4819-402b-8b45-5296c00d2e03.mov

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

Messages

:book:
:link: MB-11915

  Generated by :no_entry_sign: dangerJS against 926c45d99b97edba0cfdba7247a0f20782b307b2

PR is functionally ready to go so it should be all set for a design review. I still have a few tests in the works that will be coming soon.


validation for cohort members to be filled

The backend code assumes that the cohort is saved with atleast one member and i think it makes sense.
I have handled it in the frontend by making the field to be mandatory and now at the backend - It throws an exception that internal error has occured instead of showing the stacktrace.

A better solution would be to disable the save button unless there is at least one member in the box. This would be consistent with the job submit page.

On Jul 15, 2016, at 8:30 AM, Akshatha Pai<EMAIL_ADDRESS>wrote:
The backend code assumes that the cohort is saved with atleast one member and i think it makes sense.
I have handled it in the frontend by making the field to be mandatory and now at the backend - It throws an exception that internal error has occured instead of showing the stacktrace.
You can view, comment on, or merge this pull request online at:
https://github.com/eurekaclinical/eureka/pull/108
Commit Summary
validation for cohort members to be filled
File Changes
M eureka-webapp/src/main/webapp/assets/js/eureka.cohort.js (18)
Patch Links:
https://github.com/eurekaclinical/eureka/pull/108.patch
https://github.com/eurekaclinical/eureka/pull/108.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

i initially thought of jquery validation which we have in user management pages is the best solution. But that would not be consistent with phenotypes page. Hence the pop up. Disabling the button like you mentioned can be another solution. i think it makes sense to disable the button until both name and members field is filled out. if that is the case we would not need a pop up for the name field as well. i will work on implementing it this way. Let me know if you think otherwise.

or do you want me to leave the name field validation the way it is now?

Let's use your popup implementation for now. This page will be converted to angular eventually. We'll reimplement the validation then.
Andrew
Sent from my iPhone

On Jul 15, 2016, at 12:40 PM, Akshatha Pai<EMAIL_ADDRESS>wrote:
i initially thought of jquery validation which we have in user management pages is the best solution. But that would not be consistent with phenotypes page. Hence the pop up. Disabling the button like you mentioned can be another solution. i think it makes sense to disable the button until both name and members field is filled out. if that is the case we would not need a pop up for the name field as well. i will work on implementing it this way. Let me know if you think otherwise.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Fix chromeLauncher import

The current code seems to be working perfectly, why is this change needed?

Sorry, see #15. I got an error spatie/lighthouse-php 1.0.5<EMAIL_ADDRESS><EMAIL_ADDRESS>See https://github.com/GoogleChrome/chrome-launcher/releases/tag/v1.0.0, chromeLauncher is now published as an ESM.

Thanks, works great @Pluuk
@freekmurze this fixes incompability with ChromeLauncher v1.0.0, which gives the following error message in main:
Could not run Lighthouse. Error : `/usr/src/app/vendor/spatie/lighthouse-php/bin/lighthouse.js:1
const chromeLauncher = require('chrome-launcher');
                       ^

Error [ERR_REQUIRE_ESM]: require() of ES Module /usr/src/app/node_modules/chrome-launcher/dist/index.js from /usr/src/app/vendor/spatie/lighthouse-php/bin/lighthouse.js not supported.
Instead change the require of index.js in /usr/src/app/vendor/spatie/lighthouse-php/bin/lighthouse.js to a dynamic import() which is available in all CommonJS modules.
    at Object.<anonymous> (/usr/src/app/vendor/spatie/lighthouse-php/bin/lighthouse.js:1:24) {
  code: 'ERR_REQUIRE_ESM'
}

Node.js v20.5.0
`

Thanks!


AWS Athena usage?

I am doing a bake-off between AWS Athena and Google BigQuery for a client.  I am wondering about customer usage of Athena.  Is anyone out these using it in production yet?  If so, should I start a small section on Athena for this og-aws?

@lynnlangit Thanks for noting the absence of Athena. Another member of my team recently used Athena for ad-hoc analysis of ELB logs and the experience seemed like it was good. I'd definitely be supportive of your creating a section in the guide.

Absolutely. Any chance your bake-off produced anything we can include / link against, @lynnlangit?


Using a reference (e.g. YoY) throws a NotImplementedError for interval

Using the references feature (http://fireant.readthedocs.io/en/latest/2_slicer.html#comparing-data-to-previous-values) throws a NotImplementedError on the interval function. I understand that I need to override the interval function in my subclass of Database, but what exactly is the function supposed to do?

@spinningarrow See https://github.com/kayak/fireant/blob/master/fireant/database/vertica.py as an example. This method should return an instantiated pypika's term/function, which could be different from the one implemented above in case you're trying to add support to another database engine (FYI fireant currently supports only vertica). That's necessary for intervals and date truncs cause not all databases comply with strict iso guidelines. We tried to make fireant as engine independent as possible, but those two cases were found to be different in mysql and vertica for instance.

Hey @spinningarrow did the answer from @x8lucas8x resolve your issue?

@twheys @x8lucas8x thanks for the response! It does help; now I need to figure out how to apply this to SQL Server. Regarding only supporting Vertica, it seems to work fine with SQL Server so far (though I haven't gotten around to overriding trunc and interval so I guess fireant isn't quite being used to its full potential!)

@spinningarrow Interesting, thanks for the feedback. If you run into any issues with FireAnt on SQL Server please do let us know or feel free to highlight it in an issue!


Update Slim.php

Refactor view method if else

It is unlikely that this will merged as we are only accepting security fixes to the 2.x branch.

Thank you for your contribution but unfortunately I wont be able to merge it for reasons stated above, we are currently only accepting security fixes for 2.x branch.


Update outdatedbrowser.js

Tolerate absence of btnClose and btnUpdate, when using custom content.

@schonarth thanks 👍
But why do you want to use the plugin without the option to have the button to update?
Cheers

My client is very specific in how they want the message bar to appear,
especially the content. They don't want the prominent button, but rather a
link in the text, which is what I did and this small change allowed.
While I was at it, I also covered the same condition for the Close button,
although this is not my case right now.
Thanks for considering my small contribution =]
Cheers
Gus
Em ter, 26 de abr de 2016 às 05:54, Alexandre Gomes <
<EMAIL_ADDRESS>escreveu:

@schonarth https://github.com/schonarth thanks 👍
But why do you want to use the plugin without the option to have the
button to update?
Cheers
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/burocratik/outdated-browser/pull/232#issuecomment-214672934

@schonarth Thank you 👍
I am giving you my personal opinion: why add an extra if in the js, if we can do that with css? I know it's just an extra if, and if we keep the buttons we have extra html to load, but even so I prefer the CSS way:
.btn{
 position: absolute;
 left: -5555px;
 top: -5555px;
}

We know that js is more prone to errors then css, and this way I know I don't need to test for older browsers (IE6, etc) because I am really sure it will work.
Personally I would do this approach, so maybe we will wait for the merge if more ppl wants your approach.
Thanks

It works, though it changes the base logic. My point was to make the change as small as possible, just to avoid exceptions in my particular case.
By all means, do as you prefer =]

@schonarth let see what is the feedback of the community.
Once again thank you man 👍

@alexandre-works do we received any feedback from the community about this subject?

@brunoamorim no. We had no suggestions or requests regarding this feature. Still thing that the CSS approach is the best solution for this.

I just think that a null check is never too much for a more robust piece of
software. I did it on my copy. I think others might benefit from it too.
Vc q sabe.
Em ter, 11 de out de 2016 às 17:29, Alexandre Gomes <
<EMAIL_ADDRESS>escreveu:

@brunoamorim https://github.com/brunoamorim no. We had no suggestions
or requests regarding this feature. Still thing that the CSS approach is
the best solution for this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/burocratik/outdated-browser/pull/232#issuecomment-253036040,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMhyx_hUsZJoGoT86TvY4YVWlM6qCWmks5qy_G5gaJpZM4IPbJ0
.


Add ability to exclude tiles by implementing a C++ or Blueprints interface

This is the Unreal equivalent of CesiumGS/cesium-unity#248.
In Unreal C++ code, users can implement the cesium-native ITileExcluder interface directly. So the first step is to simply allow users to use C++ to specify an ITileExcluder that will be passed through to cesium-native.
The next step is to allow this to be done from Blueprints. This will look a bit like the C# wrapper in Cesium for Unity, involving a Blueprints-accesible wrapper for the cesium-native Tile class that provides access to a minimal set of properties, like the tile's bounding box. I suspect that calling into Blueprints for every visited tile on every frame is likely to cause a major performance problem, though. If so, we can solve this with caching. Maintain a map of Tile pointer to exclusion state, and only ask Blueprints for the exclusive state once. Provide a way to invalidate the cache, so that the per-tile exclusion state can be changed when needed.

Fixed in #1194.


OS.get_unique_ID only implemented on Android and iOS

Bugsquad edit: This issue is actually not specific to HTML5, but OS.get_unique_ID is only implemented on Android and iOS; on all other platforms it returns an empty string and an error message.
See https://github.com/godotengine/godot/search?utf8=✓&q=get_unique_ID

Seen while testing Minilens in Firefox 64-bit on Linux, Godot 2.0 Beta:
close data.pck flags 1
close data.pck flags 1
close data.pck flags 1
ERROR OPENING FILE: icon.png
close data.pck flags 1
ERROR: get_unique_ID: Method/Function Failed, returning: ""
   At: core/os/os.cpp:459.
close data.pck flags 1
close /userfs/options.txt flags 2
close /userfs/options.txt flags 1
[SUTjoystick]: mouse emulation enabled
close data.pck flags 1
close data.pck flags 1
elapsed 8 tts 4992
elapsed 287 tts 4705
elapsed 89 tts 4616

I don't know if the get_unique_ID error is related to the previous error about icon.png, so in doubt I give a good chunk of the log.

Further down the log, after DOING SYNCH!:
[SUTjoystick]: mouse emulation disabled
close /userfs/options.txt flags 1
close data.pck flags 1
close data.pck flags 1
close data.pck flags 1
close data.pck flags 1
close data.pck flags 1
close data.pck flags 1
close data.pck flags 1
close data.pck flags 1
close data.pck flags 1
close data.pck flags 1
ERROR: get_unique_ID: Method/Function Failed, returning: ""
   At: core/os/os.cpp:459.
ERROR: get_unique_ID: Method/Function Failed, returning: ""
   At: core/os/os.cpp:459.
close /userfs/savedata.bin flags 2
ERROR: get_unique_ID: Method/Function Failed, returning: ""
   At: core/os/os.cpp:459.
MAGIC:<PHONE_NUMBER>
MODE: 1
close /userfs/savedata.bin flags 1
ERROR: get_unique_ID: Method/Function Failed, returning: ""
   At: core/os/os.cpp:459.
close /userfs/savedata.bin flags 2
elapsed 43 tts 4957
[SUTjoystick]: mouse emulation enabled
elapsed 75 tts 4882

and several other occurences. It seems to happen often while trying to handle stuff in the user filesystem.

Also it shouldn't probably print the ASCII control characters.

Also it shouldn't probably print the ASCII control characters.

Yeah that's a bit weird as IIRC those should only be printed on unix. Does it consider the javascript platform as unix, or does it detected my desktop's platform?

asm.js emulates linux as its "host" AFAIK.

I can confirm on linux. Prints:
ERROR: get_unique_ID: Method/Function Failed, returning: ""
   At: core/os/os.cpp:459.

(bump)
A quick search through the code uncovers that get_unique_ID is implemented only on android and ios... which means that the "This will make the file unreadable to users, but will still not avoid them to share savefiles" point from the encrypting save games tutorial is invalid on those platforms

This issue is actually a duplicate of #196, but the latter was closed as "won't fix". I'm not happy with this resolution though, as a method named OS.get_unique_ID() that only works on mobile platforms is quite misleading.
At the very least the documentation should be written and state clearly where it works and where it doesn't (and it shouldn't be mentioned in the Encrypting save games tutorials if it works only on mobile where 98% of users won't even know how to browse their filesystem :)
The best would be to find a way to implement this on desktop platforms too of course :)

I don't know if this helps or if it is some kind of overkill, but it is just me trying to help.

Yeah that's a solution, though it's hard to maintain something on all platforms that would rely on parsing shell commands, as the command output can change in later versions, etc.
A better solution might be to use existing C++ or C libraries for that: boost-uuid (we don't want to link boost, so bad idea), ossp-uuid (cross-platform?), libuuid (util-linux), UuidCreate (Windows)

Generating unique IDs is used in more places than just savegame encryption, so I believe we can have this feature on any OS.
This library could help: https://github.com/graeme-hill/crossguid
Most of the time it's as easy as calling an OS-specific function.

As mentioned before, I don't think this makes much sense on PC, we could
try to find some way to generate an ID from stuff like hardware serial
numbers, but if the user changes hardware and the game depends on this ID
(for example, as salt to encrypt a savegame, so it can't be shared with
other phones which is one of the most common use cases), that would be a
disaster.
On Wed, Jun 22, 2016 at 9:56 AM, Marc<EMAIL_ADDRESS>wrote:

Generating unique IDs is used in more places than just savegame
encryption, so I believe we can have this feature on any OS.
This library could help: https://github.com/graeme-hill/crossguid
Most of the time it's as easy as calling an OS-specific function.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/3628#issuecomment-227734118,
or mute the thread
https://github.com/notifications/unsubscribe/AF-Z25xBLWqmIt10_yDqax4Hji-6fOWgks5qOTDggaJpZM4HV14g
.

Microsoft's OEM license is not affected if you change parts of the computer, except the mainboard. Maybe only take mainboard serial numbers?

Well we don't have to reinvent the wheel, there are already a lot of people who worked on UUIDs and even wrote standards: https://en.wikipedia.org/wiki/Universally_unique_identifier#RFC_4122_Variant

we could try to find some way to generate an ID from stuff like hardware serial
numbers, but if the user changes hardware and the game depends on this ID
(for example, as salt to encrypt a savegame, so it can't be shared with
other phones which is one of the most common use cases), that would be a
disaster.

I wouldn't say a "disaster". An UUID would typically be used to check that the encrypted savegame was made on the same machine, and if the UUID changes after a hardware change, it would just make the save broken for this user. So if you change your hardware on desktop, it won't allow you to share your saved game with other machines, it will just prevent you from using it further.
Moreover, crackers will always find a way to crack stuff. This is more about providing an easy way to get a "unique for 99.99% of users" ID, instead of providing an empty string...

Then of course if we implement such a UUID we should not advertise it as a fully secure solution. But there are no fully secure solutions anyway, and I'm pretty sure the UUIDs we serve on mobile platforms can be broken too.

Why not just generate a random number and put it on a global place, then
use that number subsequently? It works as a unique id, and lets the user
take control of their privacy if they want. Also I think that's what we do
on certain versions of iOS.
On 22 June 2016 at 11:05, Rémi Verschelde<EMAIL_ADDRESS>wrote:

Then of course if we implement such a UUID we should not advertise it as a
fully secure solution. But there are no fully secure solutions anyway, and
I'm pretty sure the UUIDs we serve on mobile platforms can be broken too.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/3628#issuecomment-227753813,
or mute the thread
https://github.com/notifications/unsubscribe/AGVmPVixqjZvnAcWX1pV5Bg5OY4uXJhwks5qOUEYgaJpZM4HV14g
.

Well, what is the goal of this function? What is it supposed to be bound to? The hardware? What hardware? What if it changes? There is no one simple solution. We either do something like what @punto- says and sacrifice real security or generate it from some hardware ids/MAC and risk the user changing some components and breaking stuff.
It really matters on what the purpose is. Maybe it will be best to provide both. OS.get_id() for an id that is generated on the first run of Godot and stored somewhere and OS.get_hardware_id() that is generated from hardware. That way the choice is on the developer and his/her intent.

Hang on, so far this function is undocumented and I need to clarify something:
Is OS.get_unique_id() supposed to return the same ID on every call for the same machine? Or does it generates a new one?
Also, as I said, a unique IDs (or should I say GUID? Is that the same thing?) can be generated in case your game runs in a distributed environment, if you want to identify something (a player? a user-generated content? A game session? anything). If the reason it's only present on iOS and Android is because of savegames and unmodifiable hardware, it misses other usages IMO.

Hardware modifications of smartphones are possible, you just need to have a professional do it. But yes, they are very very uncommon. Its even more uncommon to modify them in a way the ID changes. What is farly more common is that you want to migrate devices. E.g. you have an old phone, and want to put your savegame on a new phone. What now? Then this unique id feature fails as well.
Also, how many people change their motherboards? Not many.  So from that its only a small step to emailing the developers of the game as well. Its no service the game developers have to offer, they simply can chose to not use the function!
This feature is not about pleasing the player: its precisely the opposite. Its there to force things upon the players, and stand against the player's will. Well, godot's users are not the players but the game creators.  Its like if godot was GPL licensed: The game creators wouldn't applaud "oh great more freedom for the users", they would simply not use godot. So if the game creators want something that may or may not annoy some players, then they should get it. Its their decision after all if they want to use that function or not.
Maybe the docs can include a hint that if the motherboard is changed, the ID changes as well.

Not critical for the upcoming 2.1, so moving to the next milestone.

Can't you use RFC-4122 or one of its version variants?

Here's my hacktastic workaround (and it's not RFC-4122 complaint).  It can probably be made better but it should work across platforms and generate reasonably non-collided IDs.  If you need internet-scoped uniqueness then the random numbers should be switched for a 48-bit hash of a hardware (ie: unspoofed) MAC address.  This still isn't a total guarantee but works for most use cases.  I don't need internet-wide uniqueness in my case (it's identifying serverside objects) hence the pseudo-random generation for the 48 MAC address bits.
func julian(date):
	var jd=0
	for mo in range(date['month']-1):
		if mo in [0,2,4,6,7,9,11]:
			jd+=31
		if mo in [3,5,8,10]:
			jd+=30
		if mo in [1]:
			jd+=29
	jd+=date['day']
	return jd

var guid_state={
	'serial':0,
	'mac':[int(65535.0*randf()),int(65535.0*randf()),int(65535.0*randf())]
}
func generate_guid():
	#
	#  GUID generation for game objects
	#  NB: This is not an RFC-compliant guid generator!
	#
	var uid="%X%X-%X-F" % guid_state['mac']
	var now=OS.get_datetime()
	var ms=OS.get_ticks_msec()
	var yr=now['year']-1582
	var jd=julian(now)
	uid+="%03X-%04X-" % [yr,0x8000+(jd*24)+now['hour']]
	var msm=now['minute']
	msm=(msm*60)+now['second']
	msm=(msm*1000)+ms
	uid+="%06X" % msm
	uid+="%06X" % guid_state['serial']
	guid_state['serial']=(guid_state['serial']+1)%0x1000000
	return uid

On Windows you can get motherboard UUID easly from API (and changing your mobo would onvalidate your Windoes license - bigger problem than game). On OS X you can ge serial number of that Mac device. But dunno what for Linux.

@Marqin I believe a mainboard change only invalidates an OEM Windows license.  I wonder if the mobo UUID would be accessible via ACPI layer in linux?

This app has ability to get motherboard Serial Number. May be helpful.

Just use a random number, and store it somewhere in user:// or the engine
configuration, use it on subsequent runs. It'll be portable, and nobody
actually needs a fingerprint of the hardware for this. If that is actually
needed, we'll look into it at the time.
On 4 August 2016 at 06:51, Hubert Jarosz<EMAIL_ADDRESS>wrote:

This http://git.savannah.gnu.org/cgit/dmidecode.git/tree/ app has
ability to get motherboard Serial Number. May be helpful.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/3628#issuecomment-237507121,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGVmPbS73POYLhqPr8jLSPe1eCapXenCks5qcbY8gaJpZM4HV14g
.

I doubt its accessible to any non-root app.

Just my 2 cents: systemd (which most Linux distributions use nowadays) generates a unique 128-bit machine id (/etc/machine-id) during installation.
The machine ID does not change based on user configuration or when
hardware is replaced.

http://man7.org/linux/man-pages/man5/machine-id.5.html
Sounds like the right thing to me?

Would be fun to have godot to depend on systemd xD

@est31 the other problem with platform-specific alternatives is the id's may then clash with those from Godot's other platform.  There may then be a vector to attack salted savegames to make them shareable across platforms.  Something the DLC monetizers definitely don't want happening.


Add status code to compile and test

Replaces the previous mechanism, whereby build tools implementing BSP
would use a BSP protocol error to signal failure in compilation or
testing. Now, this new mechanism will simplify the handling on the
client side.
Fixes #41.

Addressed.


go get with -ldflags build flag fails

Please answer these questions before submitting your issue. Thanks!
What did you do?
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.
go get -u -ldflags github.com/hyperledger/fabric-ca/lib/metdata.Version=1.1.0 github.com/hyperledger/fabric-ca/cmd/...
What did you expect to see?
Install fabric-ca-server and fabric-ca-client binaries
What did you see instead?
#github.com/hyperledger/fabric-ca/cmd/fabric-ca-server
usage: link [options] main.o
  -B note
    	add an ELF NT_GNU_BUILD_ID note when using ELF
  -D address
    	set data segment address (default -1)
  -E entry
    	set entry symbol name
  -H type
    	set header type
  -I linker
    	use linker as ELF dynamic linker
  -L directory
    	add specified directory to library path
  -R quantum
    	set address rounding quantum (default -1)
  -T address
    	set text segment address (default -1)
  -V	print version and exit
  -X definition
    	add string value definition of the form importpath.name=value
  -a	disassemble output
  -buildid id
    	record id as Go toolchain build id
  -buildmode mode
    	set build mode
  -c	dump call graph
  -cpuprofile file
    	write cpu profile to file
  -d	disable dynamic executable
  -debugtramp int
    	debug trampolines
  -dumpdep
    	dump symbol dependency graph
  -extar string
    	archive program for buildmode=c-archive
  -extld linker
    	use linker when linking in external mode
  -extldflags flags
    	pass flags to external linker
  -f	ignore version mismatch
  -g	disable go package data checks
  -h	halt on error
  -importcfg file
    	read import configuration from file
  -installsuffix suffix
    	set package directory suffix
  -k symbol
    	set field tracking symbol
  -libgcc string
    	compiler support lib for internal linking; use "none" to disable
  -linkmode mode
    	set link mode
  -linkshared
    	link against installed Go shared libraries
  -memprofile file
    	write memory profile to file
  -memprofilerate rate
    	set runtime.MemProfileRate to rate
  -msan
    	enable MSan interface
  -n	dump symbol table
  -o file
    	write output to file
  -pluginpath string
    	full path name for plugin
  -r path
    	set the ELF dynamic linker search path to dir1:dir2:...
  -race
    	enable race detector
  -s	disable symbol table
  -tmpdir directory
    	use directory for temporary files
  -u	reject unsafe packages
  -v	print link trace
  -w	disable DWARF generation
# github.com/hyperledger/fabric-ca/cmd/fabric-ca-client
usage: link [options] main.o
  -B note
    	add an ELF NT_GNU_BUILD_ID note when using ELF
  -D address
    	set data segment address (default -1)
  -E entry
    	set entry symbol name
  -H type
    	set header type
  -I linker
    	use linker as ELF dynamic linker
  -L directory
    	add specified directory to library path
  -R quantum
    	set address rounding quantum (default -1)
  -T address
    	set text segment address (default -1)
  -V	print version and exit
  -X definition
    	add string value definition of the form importpath.name=value
  -a	disassemble output
  -buildid id
    	record id as Go toolchain build id
  -buildmode mode
    	set build mode
  -c	dump call graph
  -cpuprofile file
    	write cpu profile to file
  -d	disable dynamic executable
  -debugtramp int
    	debug trampolines
  -dumpdep
    	dump symbol dependency graph
  -extar string
    	archive program for buildmode=c-archive
  -extld linker
    	use linker when linking in external mode
  -extldflags flags
    	pass flags to external linker
  -f	ignore version mismatch
  -g	disable go package data checks
  -h	halt on error
  -importcfg file
    	read import configuration from file
  -installsuffix suffix
    	set package directory suffix
  -k symbol
    	set field tracking symbol
  -libgcc string
    	compiler support lib for internal linking; use "none" to disable
  -linkmode mode
    	set link mode
  -linkshared
    	link against installed Go shared libraries
  -memprofile file
    	write memory profile to file
  -memprofilerate rate
    	set runtime.MemProfileRate to rate
  -msan
    	enable MSan interface
  -n	dump symbol table
  -o file
    	write output to file
  -pluginpath string
    	full path name for plugin
  -r path
    	set the ELF dynamic linker search path to dir1:dir2:...
  -race
    	enable race detector
  -s	disable symbol table
  -tmpdir directory
    	use directory for temporary files
  -u	reject unsafe packages
  -v	print link trace
  -w	disable DWARF generation

Does this issue reproduce with the latest release (go1.10.1)?
i don't know, i have not tried with go 1.10.1
System details
go version go1.9 darwin/amd64
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/tmp/fabric"
GORACE=""
GOROOT="/usr/local/Cellar/go/1.9/libexec"
GOTOOLDIR="/usr/local/Cellar/go/1.9/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/r1/jplnn1z139x5d6jqj_62dn0h0000gn/T/go-build221356682=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOROOT/bin/go version: go version go1.9 darwin/amd64
GOROOT/bin/go tool compile -V: compile version go1.9
uname -v: Darwin Kernel Version 17.4.0: Sun Dec 17 09:19:54 PST 2017; root:xnu-4570.41.2~1/RELEASE_X86_64
ProductName:	Mac OS X
ProductVersion:	10.13.3
BuildVersion:	17D47
lldb --version: lldb-9<IP_ADDRESS>
  Swift-4.1

You're forgetting -X
go get -u -ldflags -X github.com/hyperledger/fabric-ca/lib/metdata.Version=1.1.0 github.com/hyperledger/fabric-ca/cmd/...

@AlexRouSg Thank you for the response...
go get -u -ldflags "-X github.com/hyperledger/fabric-ca/lib/metdata.Version=1.1.0" github.com/hyperledger/fabric-ca/cmd/... works


Support for Firefox 112.0 with flatpak

The issue is already reported or known issue?

[x] I have searched the existing issues on github
[x] I am using the latest version

Current behavior
None of the keymaps works
Expected behavior
Extension works
Steps to reproduce

Install firefox 112.0 with flatpak

Environment

OS: Ubuntu 22.04.02
Browser: Firefox 112.0 (flatpka)
Vimmatic version: 0.2.0

Works for me on Firefox 113.0 with flatpak. You need to go to extension settings and manually give it all the permission it is asking for


Introduce TransactionSimulatorService

Introduce TransactionSimulatorService so plugins can simulate transactions on their own

Thanks @macfarla for the review, I merged this to unlock the Linea release in testnet, but feel free to submit other comments if needed, that I will address later.


[video_player] - 0.10.8+1, freezes the application with the latest dev channel

I use the video_player pub, and everything worked fine, but since the latest flutter upgrade, sometimes the video does not start, or stops during playback.
Sometimes it starts, but when  I push the app in background, and open again, it is also freezing.
Another weird thing: if the app freezes, and I push the app to background and reactivate, it will work again smoothly...
When I changed back the channel to "beta", everything works fine. But at 'master', 'stable', 'dev' channels causes this weird error.
[✓] Flutter (Channel beta, v1.15.17, on Mac OS X 10.15.3 19D76, locale hu-HU)
• Flutter version 1.15.17 at /Users/janosdupai/Developer/flutter
• Framework revision 2294d75bfa (2 weeks ago), 2020-03-07 00:28:38 +0900
• Engine revision 5aff311948
• Dart version 2.8.0 (build 2.8.0-dev.12.0 9983424a3c)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
• Android SDK at /Users/janosdupai/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-29, build-tools 28.0.3
• ANDROID_HOME = /Users/janosdupai/Library/Android/sdk
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.3.1, Build version 11C504
• CocoaPods version 1.8.3
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 3.6)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 44.0.2
• Dart plugin version 192.7761
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
[✓] Connected device (4 available)
• MATCHmeeting      • 00008020-0002242E1A29002E            • ios            • iOS 13.1.2
• iPhone 11 Pro Max • D441D7C8-ACB0-47BC-8CF6-A4A245BF352A • ios            • com.apple.CoreSimulator.SimRuntime.iOS-13-3 (simulator)
• Chrome            • chrome                               • web-javascript • Google Chrome 80.0.3987.149
• Web Server        • web-server                           • web-javascript • Flutter Tools
• No issues found!

I just started picking up Flutter last week and was trying to implement video_player  and also found many issues resuming a backgrounded video.
Coming from a native iOS background and digging into FLTVideoPlayer, I suspect that some of these issues might be related to not using an AVPlayerLayer in the implementation.
Note:  Though I don't know the root reason, I have also found that resetting the AVPlayer in the AVPlayerLayer fixes a lot of issues when resuming playback from background.

I have exactly the same behaviour in iOS. I have noticed, that it is enough to load the video w/o showing it to get the app freeze. Is there any fix or a workaround for this issue?

For me the only solution was to switch to another package, which does not rely on AVPlayer.

cc @amirh for plugin triage visibility.

Anu update in that case?

2020-07-23 15:31:45.159345+0200 Runner[28300:413281] *** Terminating app due to uncaught exception 'NSRangeException', reason: 'Cannot remove an observer <FLTVideoPlayer 0x600003d56800> for the key path "status" from <AVPlayerItem 0x60000114cfd0> because it is not registered as an observer.'
*** First throw call stack:
(
0   CoreFoundation                      0x00007fff23e3de6e __exceptionPreprocess + 350
1   libobjc.A.dylib                     0x00007fff512539b2 objc_exception_throw + 48
2   Foundation                          0x00007fff259114ff -[NSObject(NSKeyValueObserverRegistration) _removeObserver:forProperty:] + 601
3   Foundation                          0x00007fff2591190d -[NSObject(NSKeyValueObserverRegistration) removeObserver:forKeyPath:] + 74
4   Foundation                          0x00007fff2591183a -[NSObject(NSKeyValueObserverRegistration) removeObserver:forKeyPath:context:] + 177
5   better_player                       0x0000000105864cd2 -[FLTVideoPlayer disposeSansEventChannel] + 130
6   better_player                       0x0000000105864ec4 -[FLTVideoPlayer dispose] + 36
7   better_player                       0x0000000105866efb __48-[FLTVideoPlayerPlugin handleMethodCall:result:]_block_invoke + 75
8   libdispatch.dylib                   0x00000001068bee8e _dispatch_client_callout + 8
9   libdispatch.dylib                   0x00000001068c17a3 _dispatch_continuation_pop + 552
10  libdispatch.dylib                   0x00000001068d4b2c _dispatch_source_invoke + 2205
11  libdispatch.dylib                   0x00000001068ccbce _dispatch_main_queue_callback_4CF + 692
12  CoreFoundation                      0x00007fff23da1869 CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE + 9
13  CoreFoundation                      0x00007fff23d9c3b9 __CFRunLoopRun + 2041
14  CoreFoundation                      0x00007fff23d9b8a4 CFRunLoopRunSpecific + 404
15  GraphicsServices                    0x00007fff38c39bbe GSEventRunModal + 139
16  UIKitCore                           0x00007fff49325968 UIApplicationMain + 1605
17  Runner                              0x0000000102d9fe3b main + 75
18  libdyld.dylib                       0x00007fff520ce1fd start + 1
19  ???                                 0x0000000000000001 0x0 + 1
)
libc++abi.dylib: terminating with uncaught exception of type NSException
(lldb)

@TahaTesser Do you folks have a repro case for this coded up? If you do, would you mind retesting? If not, I wouldn't take the time to retest. I'm just wondering if we've fixed it along the way without knowing it, or if it's still an issue. Thanks!

Hi  @janosdupai
Can you please provide your flutter doctor -v, your pubspec.yaml and flutter run --verbose logs, and a minimal complete reproducible code sample?
Thank you
@kf6gpe
I don't believe we have rerpo case, thanks for bringing attention to this

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


fix: adding cond/endcond to mp_ds2cards

When generating doxygen docs on source file with a large number of datalines, processing can take some time.
To prevent processing, the mp_ds2cards() macro will now automatically add the cond/endcond tags to prevent doxygen processing of the resultant files.
Doxygen docs: https://www.doxygen.nl/manual/commands.html#cmdcond

:tada: This PR is included in version 4.18.13 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


PYTEST Fixtures

We set up two fixtures in conftest.py. However, they are not called @pytest.mark.usefixtures('fresh_directory', 'set_seed') is missing for our test classes. See here for the doc https://docs.pytest.org/en/latest/fixture.html and here for use in respy https://github.com/restudToolbox/package/blob/master/respy/tests/test_integration.py, line 19
Otherwise, we do not have full reproducibilty as the seeds might be different.

I added the calls to our test files.


Bugfix : missing 'await' in the rendering of the html template

Hello,
I really like your fork of the archived repo sunflowerdeath/webfonts-generator because you're using async/await everywhere.
But one await is missing in the html rendering, ending up with a [Object Promise] in the style tag
For example, if you run test/manual.js, the generated html temp/fontName.html is containing :

This bug does not appear in your other repo furkot/icon-fonts because it uses its own custom handlebars template.
I'm hoping I'm clear enough (english is not my native language) and I hope it helps :heart:

poke @pirxpilot

Thanks for the fix. I somehow managed to miss the notification when you opened the PR. Sorry about that.

No problem, I'm happy to contribute
Thanks for your time  :heart:


GPU Detection/PROBLEM

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version

=2.2

Custom code
Yes
OS platform and distribution
UBUNTU 24.4
Mobile device
No response
Python version

=9

Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
12.6 /   9.5
GPU model and memory
NVIDIA GeForce RTX 4060
Current behavior?
Hello, everyone!
TensorFlow managed to detect my GPU, an NVIDIA GeForce RTX 4060 Laptop GPU, which is a good sign as the hardware is working correctly.
However, I'm facing an issue: TensorFlow only detects the GPU when I install version 2.2. With newer versions, it doesn't detect the GPU. I've tried reinstalling several times, but nothing worked. Does anyone know what's going on?
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2024 NVIDIA Corporation
Built on Thu_Sep_12_02:18:05_PDT_2024
Cuda compilation tools, release 12.6, V12.6.77
Build cuda_12.6.r12.6/compiler.34841621_0
Standalone code to reproduce the issue
import tensorflow as tf

print("Num GPUs Available: ", len(tf.config.list_physical_devices('GPU')))

Relevant log output
/home/nzakiese/PycharmProjects/pythonProject/.venv/bin/python /home/nzakiese/PycharmProjects/pythonProject/TESTT.py
2024-10-22 07:52:02.196951: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 AVX_VNNI FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2024-10-22 07:52:02.258352: I tensorflow/core/util/port.cc:104] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable TF_ENABLE_ONEDNN_OPTS=0.
2024-10-22 07:52:02.260071: W tensorflow/compiler/xla/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcudart.so.11.0'; dlerror: libcudart.so.11.0: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /usr/local/cuda/lib64:/usr/lib/x86_64-linux-gnu
2024-10-22 07:52:02.260079: I tensorflow/compiler/xla/stream_executor/cuda/cudart_stub.cc:29] Ignore above cudart dlerror if you do not have a GPU set up on your machine.
2024-10-22 07:52:02.589600: W tensorflow/compiler/xla/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer.so.7'; dlerror: libnvinfer.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /usr/local/cuda/lib64:/usr/lib/x86_64-linux-gnu
2024-10-22 07:52:02.589629: W tensorflow/compiler/xla/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer_plugin.so.7'; dlerror: libnvinfer_plugin.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /usr/local/cuda/lib64:/usr/lib/x86_64-linux-gnu
2024-10-22 07:52:02.589632: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Cannot dlopen some TensorRT libraries. If you would like to use Nvidia GPU with TensorRT, please make sure the missing libraries mentioned above are installed properly.
[]
2024-10-22 07:52:04.743596: E tensorflow/compiler/xla/stream_executor/cuda/cuda_driver.cc:267] failed call to cuInit: CUDA_ERROR_NO_DEVICE: no CUDA-capable device is detected
2024-10-22 07:52:04.743615: I tensorflow/compiler/xla/stream_executor/cuda/cuda_diagnostics.cc:169] retrieving CUDA diagnostic information for host: nzakiese
2024-10-22 07:52:04.743618: I tensorflow/compiler/xla/stream_executor/cuda/cuda_diagnostics.cc:176] hostname: nzakiese
2024-10-22 07:52:04.743686: I tensorflow/compiler/xla/stream_executor/cuda/cuda_diagnostics.cc:200] libcuda reported version is: 560.35.3
2024-10-22 07:52:04.743698: I tensorflow/compiler/xla/stream_executor/cuda/cuda_diagnostics.cc:204] kernel reported version is: 560.35.3
2024-10-22 07:52:04.743701: I tensorflow/compiler/xla/stream_executor/cuda/cuda_diagnostics.cc:310] kernel version seems to match DSO: 560.35.3

@Villjoie,
Every TensorFlow release is compatible with a certain version, for more information please take a look at the tested build configurations. In this case, you are trying to install TensorFlow v2.2 with CUDA 12.6 and cuDNN 9.5 which is not compatible. Also the current Tensorflow v2.17, and v2.2 is not supported.
https://www.tensorflow.org/install/source#gpu
So I request to upgrade to the latest Tensorflow where most of the bugs will be resolved. Thank you!


Update join-centos-linux-vm.md

Making sure all these guides have the note about hostname character length:
Unique Linux VM names that are a maximum of 15 characters to avoid truncated names that might cause conflicts in Active Directory.

@mabicca : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Potentially outdated guidance in public docs?

Howdy! I was reviewing the Datadog public docs for all Ruby-related references (to see if anything related to dd-trace-rb was outdated and needed to be fixed) and noticed that in https://docs.datadoghq.com/developers/dogstatsd/?code-lang=ruby# in the Ruby section we state

By default, Ruby DogStatsD client instances cannot be shared across processes but are thread-safe. Because of this, the parent process and each child process must create their own instances of the client or the buffering must be explicitly disabled by setting single_thread to true. See the dogstatsd-ruby repo on GitHub for more details.

This... seems like outdated guidance, right? I believe this no longer applies in recent releases?

https://docs.datadoghq.com/developers/dogstatsd/high_throughput/ also has a similar warning.

https://github.com/DataDog/documentation/pull/13899


Fixed #32603 -- add transaction to changelist_view

Ticket :- https://code.djangoproject.com/ticket/32603

I guess the most difficult part of this patch would be to write a regression test!

I guess the most difficult part of this patch would be to write a regression test!

Yes, I am working on it. Will update the PR soon.

@claudep I have written a test for the patch, can you please review it and let me know if its okay?

You should uncheck the Needs tests box on the ticket, then your patch will be re-queued in the patch reviewing list.

Is the CI failing due to my changes?

No, the CI failure is unrelated, don't worry.


--report: Category is prone to ... stuff

Snakemake version
5.74
Describe the bug
If category contains characters like '.',  '(' or ')' (and probably some other chars), the results will be hidden from view
Logs
NA
Minimal example
rule one:
       output: report(touch("oneDone"), category="Module 1.1")
       run: print (output)

snakemake -f one; snakemake --report
Additional context
The category link will be visible under the Results section of the html file.  Alas, clicking on it will not make it expand to show results (normal behaviour).
Obviously not important - but it could be documented, or could hide something more serious.

First of all, sorry for the late response. After many new features, Snakemake is approaching feature completeness (at least for my personal agenda and at the moment). Hence, now is the time to fix or close all the open bugs.
This has been recently fixed with the latest release, providing a complete report rewrite. I have just checked your example and it now works fine.


Regenerate preview images for lectures

This PR enables:

Generation of thumbnails on demand for a given stream via the admin interface. This includes video section and timeline thumbnails.
An API endpoint that triggers regeneration of thumbnails for all VoDs. For each VoD path a request is sent to a worker. This has the advantage that we can split the regeneration tasks to different workers.
A helper program that traverses all streams with VoDs and regenerates thumbnails for them.

Yeah, looks great now! 🎉

Ah, just one thing. Have you tested video sections previews?

Let's see what happens :D

Ah, just one thing. Have you tested video sections previews?

I have not. How do they relate to this PR? Should they be generated as well?

Yes, they should also get generated. At least there is still code left for that in RegenerateThumb.


Add SinSpirit Strategy

Fixes # .
Changes proposed in this pull request:

This only works with one address as argument
I know that some specifications are required to do the pull request but this works, you can check it
@acrylicfiddle

So sir i have update this:
if (contract_code != '0x') {
highest_block = search_block;
} else if (contract_code == '0x') {
lowest_block = search_block;
}
To only one line.
Can you see now?

Sir have you seen what you send me?
So you are giving me a solution that is to change the on smart contract but guess what?
The contract of this specific token (inspirit) is immutable and cannot be change
@acrylicfiddle

@chimpydev @ChaituVR Yes the inSPIRIT contract is immutable and cannot be upgraded or deployed again, thats our governance token.

@chimpydev what is the use case for this?
  const owner: string = await find_contract_creator(addresses[0],provider)

why do you need a contract creator if the contract won't be changed in the future? can you hard-code the address?
This is where the problem is, there is a limit on the number of requests a strategy can send https://github.com/snapshot-labs/snapshot-strategies#checklist-for-adding-a-new-strategy

Hello again sir,
So the use case is:
Inspirit holders can vote on emissions, but that is a problem, major part of the holders is smart contracts, so they cannot vote, and i want to do is, if that smart contract have balance, so lets get his creator and put the balance equal to the smart contract balance. If this people can vote with the wallet that create the smart contract

major part of the holders is smart contracts, so they cannot vote, and i want to do is, if that smart contract have balance, so lets get his creator and put the balance equal to the smart contract balance. If this people can vote with the wallet that create the smart contract

Are these smart contracts change in the future? I mean will they add up or be removed in the future?
If they are not gonna change better to have an IPFS file here (with all the mappings of smart contracts to the owner) and fetch it when required)

@ChaituVR There could be more contracts added in the future so maintaining an ipfs file will not be the correct approach as it will constantly change. What we want to do as an end goal is that there are some contracts which hold our inSPIRIT tokens which is not transferrable, so we need to enable a way for the owner to vote with the same balance in the contract.
So when a user logs in, if that user has a contract deployed which holds inSPIRIT then the user should be able to vote with that balance. in future we are gonna have NFTs which will also hold inSPIRIT within it, so we would need the same solution as this as we will need to find the balance of token in the ERC-721 contract and the owner should be able to vote with it. Lmk if you have more questions!

Closing this for now, feel free to reopen if you find a workaround for this without so many requests


Adrian

fixed some of the errors but still unable to get the put request to dynamically update the front end card

need to fix a  bug first


Preload :address in divisions list

I wasn't able to test my change, but I think it should fix it
Fixes https://github.com/edenlabllc/ehealth.api/issues/3167

@AlexKovalevych good catch, I fixed it, hope the tests will pass

According to this commit the issue was fixed a month ago https://github.com/edenlabllc/ehealth.api/commit/8c60441c38e991e87fb017faa69553fe8047679d
Could you please confirm (maybe by improving the test) that it works as expected?


Initial setup of app for hangman game

Initial setup looks good. Consider looking into hmr for webpack 2 so that I don't continue to deprecate to webpack 1.14


Preview not showing until disabling double side

This is on iOS 10, and I'm trying to print out an image (https absolute URL in an img tag).
Preview is not showing when the printing dialog opens up, just a blank page. If I disable double side, which is enabled by default and I haven't been able to change it programmatically, then the preview refreshes and it shows the image, but not always. If I cancel the printing, and open it up again, then the double side disabling workaround won't work.
Thanks!

This is what it logs out when I tap on the Double-sided toggle:
The relevant UICollectionViewFlowLayout instance is <UICollectionViewFlowLayout: 0x100fbf0b0>, and it is attached to <UICollectionView: 0x10130e400; frame = (0 0; 375 250); clipsToBounds = YES; opaque = NO; autoresize = W+H; gestureRecognizers = <NSArray: 0x170658420>; layer = <CALayer: 0x170420d40>; contentOffset: {0, 0}; contentSize: {0, 0}> collection view layout: <UICollectionViewFlowLayout: 0x100fbf0b0>.
2016-10-08 09:23:30.092845 MyApp[867:266198] Make a symbolic breakpoint at UICollectionViewFlowLayoutBreakForInvalidSizes to catch this in the debugger.
2016-10-08 09:23:30.985816 MyApp[867:266198] The behavior of the UICollectionViewFlowLayout is not defined because:
2016-10-08 09:23:30.985886 MyApp[867:266198] the item height must be less than the height of the UICollectionView minus the section insets top and bottom values, minus the content insets top and bottom values.

@jacobo3d how about trying self.automaticallyAdjustsScrollViewInsets = NO; in your view controller? I know some cases it helps.

If I disable double side, which is enabled by default and I haven't been able to change it programmatically

To disable the duplex mode at all, just set duplex: 'none' when invoking the print method.

Please re-open if needed.


Code blocks need bottom margin

If you create a code block in markdown in the guidance for a component or pattern, it's missing a bottom margin, like this:

you solved that by wrapping it in a paragraph right?

I did indeed 'solve' it :-)
It actually looks better too.
On Tue, 12 Jun 2018 at 11:56, Jani Kraner<EMAIL_ADDRESS>wrote:

you solved that by wrapping it in a paragraph right?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/alphagov/govuk-design-system/issues/299#issuecomment-396549553,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABhFTG7I-gJc05F11UE6ixb1UxE3tH34ks5t755jgaJpZM4UZZnV
.

good, close this?

There are two ways to do code blocks in Markdown - using a single backtick (`) creates inline code snippets - e.g. if you're mentioning a method name.
Using three backticks (```) will create an actual code block which is what you should use in the example above.
It's the difference between this and:
this

There should be no need to wrap any code blocks in paragraphs.

Sorry Tim, I closed this prematurely – it looks like you were using the triple backticks and switched to single backticks because it didn't have a bottom margin?
My bad! I'll re-open this.


Mathjax conversion doesn't work properly for some formulas.

Issue Summary
Mathjax conversion doesn't work properly for some formulas.
Steps to Reproduce:

I'm building a github page built using Jekyll. (and Minimal-mistakes theme)
I'm trying to enter formulas using Mathjax, but I'm having a problem with some formulas not being converted.
After checking them one by one, I found that formulas containing curly brackets were not converted properly.

For example,
\mathbf{w}

prints fine, but like
\mathbf{w}_{N-1}

don't.
I don't know what's causing the problem, could you tell me how to fix it?
Technical details:

MathJax Version: 2.7.5
Client OS: Windows 10 21H2 (19044.2604)
Browser: Chrome 110.0.5481.178 (64 bit)

I am using the following MathJax configuration:
<script type="text/javascript" async
	src="https://cdnjs.cloudflare.com/ajax/libs/mathjax/2.7.5/latest.js?config=TeX-MML-AM_CHTML">
</script>

and loading MathJax via
<script type="text/x-mathjax-config">
   MathJax.Hub.Config({
     extensions: ["tex2jax.js"],
     jax: ["input/TeX", "output/HTML-CSS"],
     tex2jax: {
       inlineMath: [ ['$','$'], ["\\(","\\)"] ],
       displayMath: [ ['$$','$$'], ["\\[","\\]"] ],
       processEscapes: true
     },
     "HTML-CSS": { availableFonts: ["TeX"] }
   });
</script>

Additional information (Screenshots and links)
https://joonsuryu.github.io/studies/theory-of-generalization/

https://joonsuryu.github.io/studies/bias-variance-tradeoff/

It's not the braces that are the problem it is the underscores.  Markdown uses underscores to indicate italics (notice how the {1} ~ \mathbf{x} in your screen shot is in italics — this is the math between two underscores), and that means that your page will have <i> tags or some other markup within the mathematics.  MathJax does not process math that contains HTML tags, so that math is not being typeset.
One solution is to use \_ rather than _ in your equations, as that will prevent Markdown from processing them, and they will be sent to MathJax as plain underscores.
Because the markdown is processed before MathJax sees the page, there is nothing MathJax can do about it. It has to be handled by the markdown engine. My understanding is that kramdown (one of the Markdown engines) is supposed to properly handle this, but I don't have any experience with it myself. There are several tutorials for MathJax in kramdown, so consider this one or this one for more. There seem to be various settings to control the handling of math, so you may need to set one of those.

Thank you!
I sincerely appreciate your help in resolving this problem.


Asociatia Oportunitati si Cariere - The company's logo is missing and the company name is not written correctly

ENVIROMENT: production
COMPANY: Asociatia Oportunitati si Cariere
URL: firme.peviitor.ro
Company's Career website: Asociatia Oportunitati si Cariere official website
Browser: Google Chrome
Device: laptop
OS: MacOS
Precondition:
The site must be open in browser.
Steps:

Click on Firme from footer.
Enter Asociatia Oportunitati si Cariere in the search bar.

Expected result:
You can see the company's logo and the name of company is written correctly
Actual result:
The company's logo is missing and the company name is not written correctly.

Fixed


Keyboard shortcuts for undo and redo

I believe ctrl+y is standard for redo on Windows.
I've been reading through all the code and it is just gorgeous.
Addresses #15 but is probably not comprehensive enough to close the issue.

It would probably make sense to add shortcuts for everything else in the menu bar.

Seems like you can't override cmd+n in Chrome.
I tried to pull out the shortcuts and actions into an object like
shortcuts = {
  "command+z, ctrl+z": @_undo
  "command+shift+z, ctrl+y": @_redo
}

and loop through that, but for some reason it didn't work.

Thanks David! For the compliment and the contribution.
Your loop through an object may be not working due to the coffeescript/javascript issue that for loops do not "close" over the iterated variables. I get bitten by this all the time. For example, if you do:
shortcuts = {
  "command+z, ctrl+z": @_undo
  "command+shift+z, ctrl+y": @_redo
}

for own keyCombo, action of shortcuts
  key keyCombo, (e) =>
    return if Util.textFocus()
    e.preventDefault()
    action()

Then this will make all the key combos do _redo instead of what they're supposed to! This is because there is just one action variable that gets set as the loop iterates and the last thing it gets set to is _redo.
Check out the last paragraph of the Coffeescript docs on loops.
The solution to that problem is to include an extra function wrapper to close over the action variable. For example, you could do:
shortcuts = {
  "command+z, ctrl+z": @_undo
  "command+shift+z, ctrl+y": @_redo
}

for own keyCombo, action of shortcuts
  do (keyCombo, action) =>
    key keyCombo, (e) =>
      return if Util.textFocus()
      e.preventDefault()
      action()

Another option is to use an iteration method that takes a callback, like _.each. The callback will then close over the iterated variable.
Other things:
You only need to include the return if Util.textFocus() line for the backspace one. This is to check that the focus is not on a text field, in which case backspace should backspace a letter, not delete the selected shape.
Do you need to have e.preventDefault()? I am not sure, maybe you can check this. I guess it's safer to have it so the browser doesn't try to do its own thing when you do a key combo?
Unfortunately even after removing the return if Util.textFocus() line, cmd+z doesn't work while the text is in codemirror. This is because codemirror traps the cmd+z. I'm sure there is a way to disable this in the codemirror keymap but I haven't figured it out yet.
Anyway, this is good enough for now. I will merge it in! :+1:

Thanks for the explanation and the merge.
I figured the preventDefault would be necessary to block default browser behaviors, but I didn't test without it. As for leaving in Util.textFocus(), that was intentional: I thought it might be strange for the keyboard shortcuts to work while typing. On the same note, I don't think it's a problem that Codemirror captures cmd+z.


feat: make commit status update configurable

Add a new application configuration to disable the commit status update
for all repositories in case permissions are not yet given.
Closes #306.

ah damn, we forgot the docs: https://github.com/MediaMarktSaturn/technolinator/blob/main/docs/Runtime_Config.md
but let's do it on the next PR


Handle condition where skip_spaces could put the index out of range.

ref: https://github.com/react-native-community/react-native-svg/issues/1083

Good job, man. I hope this gets merged asap.

@msand no mean to disrespect by tagging you but this PR fixes something really important, it's literally fixing this bug that breaks the library from being used. Hope we can merge it as soon as possible. Thanks in advance.

Published v9.8.2
@msqar In the future, no need to rush these thing, you're able to use forks/specific commits without them being in the master branch / published to npm.

@msand oh thanks, didn't know. :) thanks for your time. And sorry to bother!

@eahrold Oops, lint complaining about a missing ";" in the java file. Do you disable lint?

@msqar you're absolutely right needs the ;
My mistake. I ran the npm run lint but it doesn't catch native code.
> react-native-svg $ npm run lint

><EMAIL_ADDRESS>lint ../Code/react-native-svg
> eslint ./

../Code/react-native-svg/lib/extract/extractColor.js
  153:12  warning  'name' is defined but never used  no-unused-vars

../Code/react-native-svg/lib/util.js
  3:14  warning  'key' is defined but never used  no-unused-vars

✖ 2 problems (0 errors, 2 warnings)

Those warnings are a bug in eslint. If the native code compiles it should be enough. Assumed you had done that.

Those warnings are a bug in eslint. If the native code compiles it should be enough. Assumed you had done that.

Unfortunately, I did not.  I threw the changes together late last night without even intending to do a PR, and they were so small I thought "why not", and then did the pr to develop.
@msand It was careless of me, and I apologize. I hope it didn't bum out your Sunday too much.

No worries, takes seconds to cut a release anyway, and noone else seems to have experienced the issue, no hard feelings :) Thanks for the fix!

Thanks for the fix guys :)

Btw, this was certainly on me, I should've added continous integration with unit and end to end tests long ago, or tried using the changes at the very least. I set up a e2e testing prototype this spring, and this reminds me I need finish that up.

I am getting the same error of ''Error while updating property 'd' of a View managed by: RNSVGPath' while I create an array of String data


Android - Disk Usage HealthMetricProvider

Add onto the Disk Usage tool the ability for it to be a HealthMetricProvider, with the following information:
isHealthy is true if the sum of ditto_store and ditto_replication is <= 2 GB<PHONE_NUMBER> bytes), false otherwise
details includes the full size list as shown in the UI

@texasRanger09 based on the conversation in https://github.com/getditto/DittoAndroidTools/pull/79#issuecomment-2135661374, I'm assigning this to you


[Bug]: When the video is on full screen and the playback bar disappears, the subtitles also disappear

What happened?
If I turn on the video and select subtitles from the menu and put full screen and then wait for the playback bar to disappear, the subtitles disappear too. I have to have my mouse on the playback bar to see the subtitles throughout the show.
My player setting:
What browsers are you seeing the problem on?
Other (tell us in input box below)
Steps to reproduce?
No response
Other relevant information
I use Brave Browser Version 1.67.123 Chromium: 126.0.6478.126

Hi, do you have a video demonstrating the issue? Do you also have a rough idea on how you reproduced this? I've been looking at this issue for some time (as this was also reported in the Discord) and I could not seem to reproduce it.

Oh, I had this issue as well I posted about it in discord, I have a video showing the problem.
never solved it but think it's by design how chromium browsers or at least edge browsers to my knowledge work as I have tried to remake this on safari and it doesn't happen, however I could be wrong I just have had the same problem

Ok weird fix that I think currently works on latest edge, turning on the enhancement thingy that comes with edge I think makes it work fine for now, although it could not always work.

Oh, I had this issue as well I posted about it in discord, I have a video showing the problem.  never solved it but think it's by design how chromium browsers or at least edge browsers to my knowledge work as I have tried to remake this on safari and it doesn't happen, however I could be wrong I just have had the same problem
I have experienced this same problem on other chromium-based browsers but currently am experiencing this on Microsoft Edge Version 126.0.2592.81 which i believe uses chromium 126
Also this could also be a problem with 2 monitors when you move the mouse to a secondary monitor

There was a video demonstrating around ~1 week ago that demonstrated this issue could happen without moving your mouse onto a second monitor.
If anyone here is reading this but is also experiencing the same issue, please feel free to reply with your Browser & version and/or a rough idea of how you can reproduce the issue.

Hello,
after a long hiatus, I just wanted to let you know that somehow, miraculously, things got fixed. I don't know what it was, but it's fine.
Thanks for the support anyway.

Hello,
after a long hiatus, I just wanted to let you know that somehow, miraculously, things got fixed. I don't know what it was, but it's fine.
Thanks for the support anyway.

Glad to hear! Very weird bug.


Error validation subscription keys: Undefined property: stdClass::$licenses

We're using Pushover and this great wrapper for our application to notify users about upcoming events. For easy handling we're using the Pushover Subscription API, so instead of user keys, we have subscription keys. Other as stated in the Pushover documentation, the response seems to not contain the licenses for the user. This leads to an error:
Undefined property: stdClass::$licenses
in serhiy/pushover/src/Client/Response/UserGroupValidationResponse.php in line 103
Steps to reproduce:
1.) Acquire subscription key via Pushover Subscription API
2.) Try to validate using Serhiy\Pushover\Api\UserGroupValidation\Validation
$application = new Application($appId);
$validation = new Validation($application);
$recipient = new Recipient($subscriptionKey);
$response = $validation->validate($recipient);
The last line throws this message
Potential fix:
Check if response contains licenses data prior to accessing it. Since PHP 7 it is possible to use null-coalesence operator ??.
Replace line 103 as follows:
$this->setLicenses($decodedCurlResponse->licenses ?? []);
Thank you for your great work!
Best regards,
Sebastian

Thanks for reporting the issue, @spetters. Let me look into it during the next few days and come back with a fix.

We are using a subscription key. As far as we understand the API documentation, user keys, group keys and also subscription keys are handled the same. So let's say, our hope was, the subscription keys can also be verified. Besides not returning information about licences, the status seems to be correct:
{     "devices": [         "device 1",         "device 2"     ],     "group": 0,     "request": "reducted",     "status": 1 }
I can create an app and subscription keys for testing if this helps.

According to API documentation (https://pushover.net/api#validate), only "the user's or group's identifier" can be validated. It does not mention subscription keys in validation section.
I played a bit more with subscriptions and it seems like I can reproduce the issue. Users, when subscribed to an Application, are added to a Delivery Group. Each user in the group gets newly generated User Key. Keys can be found under Group Members. Validating such key indeed produces a response without licenses array.
In my opinion, the array should be there and this may be an issue on Pushover side. I can produce a workaround for now, but I will contact Joshua from Pushover and ask why licenses array is not included in the response.
How urgently do you need a fix?

Thank you very much!
Since only validation is affected we can skip this in our development environment as a workaround, so this is not an urgent topic for now. It would be great to find a correct solution.
From the Pushover API docs it was not really clear if user keys and subscription keys are interchangeable everywhere. Also the Message API does not mention them. Maybe Joshua can clarify.

I have submitted a support case. Let's also ask @jcs here politely :)

Hi, the Pushover user validation API now returns a blank licenses array for subscribed user keys.

Thank you @jcs ! I can confirm, validation now works as expected. This issue is resolved.


Improved examples in docs about raw SQL queries.

Regression in 14459f80ee3a9e005989db37c26fd13bb6d2fab2.

Have you gone and rescanned the changes to the blacken-docs commit or should someone do that briefly now? I looked at the commit, it's quite huge exploding_head

I tried to check it very carefully before merging, but I couldn't notice everything :disappointed: my bad :fire: . I noticed this today by accident. We shouldn't have any more cases like this :crossed_fingers: .

Thanks y'all for reviews :+1:


Fix documentation, add more documentation

Hello,
I would like to use this project in my flake and replace my own library (https://github.com/drupol/nixos-x260/blob/1ff3ef6b7bb0253934e11356956ea4ac38a8c75a/imports/lib.nix#L44).
I'm using flake.parts too but I'm having trouble getting it running correctly so I create this issue as a reminder that it needs to be done at some point, I might not be the only one having this issue.
E.g.: How would I do to use this project in my own project at https://github.com/drupol/nixos-x260/ and replace all the calls to inputs.self.lib.mkChromiumApp with the one from this lib?

Fixed in https://github.com/drupol/nixos-x260/commit/11f40bac8e5ee251dd80ccb76a2ea792d16c6312
The issue was not related to this project, mostly a PEBKAC.


Strange error in CHYT in case of improper sorted join usage

Suppose we have a table //tmp/a sorted by (x, y) and //tmp/b sorted by (y).
Then, the following query
SELECT
    a.y
FROM
    `//tmp/a` as a
LEFT JOIN
    `//tmp/b` as b
ON a.y = b.y

fails with an error
Query 724938c1-d50bd0df-d08a8932-c69b527f failed[1]
Invalid sorted JOIN: key is empty[1]

This error is not very informative, is it possible to make it better?

I think this is some kind of a regression, previously the error was much better.


Android.App.Result Always Returns Canceled

I'm attempting the run the sample project as is, but keep getting an error on if (resultCode == Android.App.Result.Ok) as it keeps returning a result of Android.App.Result.Canceled
Any recommendation?

I am having the same problem now, does anyone know how to solve this?

It is necessary to configure authentication with google api: https://developers.google.com/fit/android/get-api-key

This sample has been deprecated/archived (check README for more information on newer samples related to this technology).
As recommended by GitHub, we are closing all issues and pull requests.


how to build extension as static

I usually place extension souce in ext direcory and then run ./buildconf
I'm not How to do same for extension developed using  PHP-CPP
please guide, how can i do this

Have a look in the documentation: http://www.php-cpp.com/documentation/your-first-extension
After building the yourextension.so there are two ways to include it in your php installation.
1. Load from php.ini
If you don't know where your ini file is:
$ php -i | grep php.ini

Then simply at the bottom at:
[myextension]
extension=/path/to/yourextension.so

2. Custom ini
Or create a custom .ini file something like yourextension.ini and link the extension there.
By running:
$ php --ini

You will find out what folder php is scanning for ini files.
Hope that helps.


Make sanitised agent name alpha-numeric only

A build agent's working dir is /usr/local/var/buildkite-agent/builds/{{name}}.
This change makes it a little safer by making sure there are no weird characters, especially emojis with sneaky : characters. Docker-compose doesn't seem to like them too much when mounting volumes.

Agent
Folder

hello world
/usr/local/var/buildkite-agent/builds/hello_world

:taco: builder
/usr/local/var/buildkite-agent/builds/_taco_builder

On the downside this will change many working dirs, so agents will need to re-download every repo... thoughts?

Thanks for this! We may need to be a smarter about this change. We should check the existence of the older named folders, then move them and use the new name. The reason being that by changing how they're named, after people upgrade their agent, they'll get hit with a slow build because it'll clone to a new folder (also leave the old one to go stale and take up space). I hope that made sense :)
I'm happy to make this change if you want!! Or you can if you're feeling adventurous :)
Thanks again for the PR!!

Yep made total sense, check+move sounds like a good option :+1: I'll give it a shot tomorrow.
In the meantime I added this to my pre-checkout hook to make docker-compose happy.

What do you think the behaviour should be if there's conflict, e.g. the folder exists with the old name, but the new name is already taken too?

@rprieto that's a good question! What do you think?

We could rm -rf the old one and move on perhaps?

Or maybe we just use the new folder and leave the old one alone?

What does the existing agent do in case of conflict, e.g. if there are 2 agents with the same name? BuildKite doesn't seem to reject same-name agents, but I assume it would cause problems if they run on the same server.

Yeah... So currently if you have 2 agents with the same name, bad things happen. We try to give people tools to avoid that (like --name "foo-agent-%n" which will assign it a unique number) but clashes sometimes still happen. That's a future problem though I think :)

So I think the safest bet is to just use the newer named folder if it exists, and leave the old one alone? Just because the likelihood of this is happening is pretty rare I think? So the code would be something like this...
if !folder_with_new_name.exists?
  if folder_with_old_name.exists?
    mv folder_with_old_name folder_with_new_name
  else
    make_new_folder
  end
end

test party!

This is probably ok to merge as is for v3.0 (this PR is against master).
Okay to merge @keithpitt?

@toolmantim yeah, after thinking about the edge case, I think it's fine to merge into master as-is.

@keithpitt cool, I reckon so too!

Thanks for that! Sorry I didn't have a chance to look at the smart rename yet. So are we 👍 about the fact agents will need to clone from scratch again?

@rprieto yep! If you wanted to backport this to 2-1-stable branch to go out in a 2.1 point release, we'd need to make sure they wouldn't re-clone.
Actually… I just released the bootstrap.sh doesn't apply to master, it has the bootstrap in go. So I just merged this into oblivion. We're removing this in #303 to avoid this confusion.
So for master we need to update:
https://github.com/buildkite/agent/blob/3c6d8ccca71fd918de9272c004187c4a66452ea0/agent/bootstrap.go#L674-L676
@rprieto We can fix the golang version. Did you want this in 2-1-stable so you can emoji on stable?

here's the change in master: https://github.com/buildkite/agent/pull/310

Thanks that made sense. I made that change to our local pre-checkout so we're OK for now. Just saw your new PR on the golang version, looks great!


sdk updates etc

Docs Build status updates of commit c6fe2f7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Advertising/bingads-13/guides/currencies.md
:white_check_mark:Succeeded

Advertising/bingads-13/guides/get-started-java.md
:white_check_mark:Succeeded

Advertising/bingads-13/guides/handle-service-errors-exceptions.md
:white_check_mark:Succeeded

Advertising/bingads-13/guides/release-notes.md
:white_check_mark:Succeeded

Advertising/bingads-13/guides/walkthrough-web-application-java.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel