See the entire 40x22 cave

If I want to see the entire 40x22 cave on the screen, is it possible to expand the cave view to full screen/window?
Motivation:
I believe a 40x22 cave would be fully visible with a scaling factor of 1 or even 2 on my screen.
I'm forced to set the scaling factor to 5 to make the visible box close to the size of my screen, but all the elements look quite huge with these settings.

In early versions of Gdash this was the case /versions until 2010/, later Cirix eliminated this possibility. All in all, maybe it's a good thing, the current view is similar to that of the commodore 64.

I installed the 2010 version of Gdash to remind myself what it looked like, that is, this large cave view.
And I admit now that as an option it would be an interesting solution.
Could it be done?

Where can I get the 2010 version? Or can you make screenshots (game and settings)?

I tried to send you the 2010 version after win, but I got a message that the message is too large .

Well then just attach the screenshots here.

As a quick solution you can set the visible area here: GAME_RENDERER_SCREEN_SIZE_X and GAME_RENDERER_SCREEN_SIZE_Y
The scaling factor still has to be suitable for your monitor.
Unfortunately I don't know how to add this setting yet without ruining the code...

I assume you mean playing a cave without scrolling? Because in the editor you see the full cave on the screen.
But what sense does it make to play the game without scrolling? His would look weird.

And it's also some kind of cheating! ;-)

Playing the game almost without scrolling:
https://www.youtube.com/watch?v=DPD6NlFFuSQ
Cheating?

Somehow this wouldn’t seem like Boulder Dash anymore for me

@cwscws Don't worry, we will keep C64 resolution as default for connoisseurs of the original Boulder Dash. )

New release for Windows: Preferences -> Game graphics -> Full screen view

This option has a lot of potential in itself.
In 1920x1080 resolution I normally play on scale 5 and it's ok.
The new option on a scale of 5 or even 4 is unplayable, I can't see Rockford, it's out of view of the monitor.
Only changing the scale to 3 shows me the whole cave.
The idea is good.
On higher scales from 4 onwards, the image should center on the Rockford as in the non-fullscreen version.

Just for comparison:
Scroll view:
https://youtu.be/zalm1LKr5ZY
Full cave view (without the title):
https://youtu.be/_fPfWIpQjYM
Playing in full cave view was definitely easier.

Some fractional scaling factor is required, to accommodate a header that has 85px height.
For instance, with scaling factor 2.75 a cave 40x22 will occupy 1760x968 and enough space (112px) for a title on a 1920x1080 screen.

At a scale of 4 and above, the cave behaves like "Intermission". Surely on a 2k monitor it will work, but I have to reduce it on a scale of 3.
Question:
The editor has its own scaling level
The Commodore64 style game has a level of scaling,
Can the fullscreen option have its own C64 independent scaling level?

It's interesting to play that way. But Boulder Dash also lives from the unknown lurking around the next screen it scrolls to... ;) So I prefer regular cave view.

My last checkin allows now a double value for the scaling factor, so you can use your complete screen.
I had to touch a lot of code, hope I haven't destroyed too much...
The title screen does not like double values... looks odd, so I forced an integer for it. Not sure if there is a better solution.
New release is out.

Not bad at all.
Looks good for me with scaling factor 2.9
Just as minor issue, maybe the better step is .125 ?
If the original size of cell is 16x16, the result of scaling with step .125 should always lead to some integer sizes:
1.125 -> 18x18 -> 720x396
1.250 -> 20x20 -> 800x440
1.375 -> 22x22 -> 880x484
1.500 -> 24x24 -> 960x528
...
2.875 -> 46x46 -> 1840x1012
3.000 -> 48x48 -> 1920x1056
and so on.
I don't know, what is a size of header, probably it is also subjected to scaling...

This is now great for the cave test in the editor.
I set the scaling to 2.4 in the editor and in the game in full screen, and ... it's a revelation.
Thank you

Can the fullscreen option have its own C64 independent scaling level?

Yet another scaling level that is only relevant for full screen mode? IMHO we already have enough options.

Looks like OpenGL does not like double values: Full screen game is not centered. Right now I don't know how to fix that...

It seems to me full screen game was never centered with OpenGL engine, even when there was integer scaling factor.
Try some older GDash-export version with OpenGL engine, scaling factor 1, full screen, the visible area is aligned to left bottom corner. I can make a screenshot, if you want.

Oh, good to know. Thought I have something destroyed...

The size of app after cold start appropriate to the Scaling factor 5:

Make some changes, for example, switch on Particle effects.
The size of application after that becomes appropriate to the "perfect scaling factor for full cave view"

Each time after app start I am foced to change something in the Settings to adjust app size to the "perfect scaling factor for full cave view".

Are you sure you are using the latest version?
Now if you change the "Full cave view" settings the GDash gets restarted. I added the restart flag.

Why don't I change the Scaling factor to 2.875 (as a perfect for full cave view)?
Bacuse I am forced to switch off Full cave view and enable full screen mode for recording Intermissions.
A scaling factor for perfect for full cave view is calculated for 40x22 caves and don't change for Intermissions, so the header is fr from Intermission cave, and that is not suitable for video recording:

When I turn off Full cave view for Intermission, Scaling factor 5 is good for video recording:

Are you sure you are using the latest version? Now if you change the "Full cave view" setting GDash gets restarted. I added the restart flag.

Yes, GDash gets restarted after changing "Full cave view", it's true, but I say about a cold start.
The preserved scaling factor (5) is used instead of "the perfect scaling factor for full cave view" (2.875) after a cold start.

In fact, another inconvenience also appears after the implementation of Full cave view feature:
Every time I run a new cave from the Start screen, the size of app window resizes, so OBS Studio needs some time (~ 2-10 sec) to catch the new window to be able to record it.
I have no idea how to fix it. I understand that window resize is required as a result of Full cave view scaling factor ajustment.
So I'm forced to wait 2-10 sec for OBS Studio catch the window before start a recording.
With "Full cave view" off window size remains the same from the Start screen through all caves, so it's convenient to immediately video recording without interruptions, but then there is no "Full cave view"...

Ok, I see that after a cold start there the window size is different compared with right after changing something in the settings.

@renyxadarox Thanks for reporting. SDL/OpenGL initalization is different than GTK.
Your OBS concerns:  Yeah, the title screen resolution is a different story. I'm on.
And yes intermissions don't look good in full cave view. I'm on, too.

@revvv
regarding Intermissions and full cave view I see 2 options:

calculated optimal scaling factor before each cave start and resize the window. Probably not good case, because there also bypass effect between previous and current caves.
align cave header to the top of cave visible area. Something like that:

It's not a big deal, to crop video in video editor.

Released a new version. Windows seems OK, Mac has problems, but did only a quick test. Therefore marked as pre-release.

Looks weird after a cold start, even after the second cold start

(Full cave view on)

Sorry, I messed it up. I fear I can't do more today...

I'd rather return back to 1.8.6

My problem was I tested with SDL, look find, but OpenGL looks terrible.
Then I always started a game after startup, but if I enter settings immediately you notice a weird font...
So if anybody out there wants to do an endless trial and error session... join me ;-)

New release is out. Title screen and cave now have always the same size.

@revvv, nice. The OBS issue has been resolved now.

Great to hear that.
Unfortunately I was not aware that you can create caves bigger than 40x22. In this case a full cave view makes no sense... I'll try to fix that...

I think the title screen should be enlarged to the window size - otherwise it just looks weird.

I think the title screen should be enlarged to the window size - otherwise it just looks weird.

That's not so easy...

Intermissions look very good in Full cave view with Gdashexport-1.9-fork!
@revvv A lot-lot-lot-lot-lot thanks to you for that briliant enhancement!
I have no idea what needs be done to made the Full cave view further better.
I think you can close the issue. Or do you have other ideas of what else can be done?


还是想加一个remove_if_exists

直接修改原函数的返回值，确实破坏了API的一致性原则导致不兼容，那么增加函数是否可以呢？主要是在使用过程中会有很多
if set.contains(b){
set.remove(b)
.....//执行其他代码
}
有不少这类逻辑的，实际上remove本身就判定了是否存在，如果这种代码多，相当于执行性能就要降低一半，故建议增加
remove_if_exists这种函数，将contains跟remove能合并的，同时增加一个add_if_noexists，将contains和add合并

Pull Request Test Coverage Report for Build 3868
Details

9 of 14 (64.29%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 80.114%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

builtin/linked_hash_set.mbt
9
14
64.29%

Totals

Change from base Build 3867:
0.05%

Covered Lines:
4363

Relevant Lines:
5446

💛  - Coveralls

@suiyunonghen this looks good to me. How about the API: add_and_check  and remove_and_check. It would be nice to phrase the commit message in English so that it is approachable to more users.

@suiyunonghen this looks good to me. How about the API: add_and_check and remove_and_check. It would be nice to phrase the commit message in English so that it is approachable to more users.

Ok, I'll make some changes

merged in #1227 with reworded commit message


Is there any kind of persistence support?

New issue checklist

[X] I have reviewed the README and documentation
[X] I have searched existing issues and this is not a duplicate

General information
I downloaded and played with the sample code, and read docs about this library, but I don't see anything about persistence.
Evidently Instagram has it, because when I open it in airplane mode, I can see different cells.
Any doc or clue about persistence when using this library?
Thanks a lot.

@Ricardo1980 within Instagram we mostly just use NSCoding for everything. IGListKit doesn't have any part in persistence layers for data. Instead I'd recommend loading cached content from disk on startup, then replace it with network data when it comes in.
IGListKit makes this easy because you don't have to differentiate between cached and network content, and when data changes just call adapter.performUpdates(animated: false).


⚠ ClamAV has degraded performance

In 27ab064, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 452 ms

Resolved: ClamAV performance has improved in 7091d89.


Add new file extensions

Add extensions for:

Cylc-flow workflow scheduler
Fortran (fixed-form and free-form)
Julia

please open separate pull requests with references and make sure the new externsions are unambiguous


type bind channel guide files

This PR includes some changes to Guides as well since the previous PR was already merged. The primary goal of this PR is to:

explore places where Guide changes affect in bindChannel
convert files to TS where necessary
update to es6 syntax with imports, consts, etc where possible
update to immutable record syntax with typing

off to a great start!

Rebased in @jonathanzong's branch so that I could work off the latest changes.

Merged in changes from vega version upgrade PR - so order of merges should be Jonathan's PR, then vega version upgrade, then this.


Finish Dockerfile

We need a working Dockerfile that has all the proper OS, GPU, and Python setup, that can easily work with our GCP project.
The last I looked at it, it was unclear:

what the base image should be
the best way to run GCP VMs using it

this seems to be the best base image for our purposes as it has cuda 11.7 cudnn 8.5 and ubuntu 20.04 that we already use on our existing VMs
https://gitlab.com/nvidia/container-images/cuda/-/blob/master/dist/11.7.0/ubuntu2004/runtime/cudnn8/Dockerfile

docker command to pull above image : docker pull nvidia/cuda:11.7.0-cudnn8-runtime-ubuntu20.04


Add Faker::Number.within

I was working with Faker today and I noticed that the Number class had a method 'between' that takes a max and a min, but no corressponding method that could do the same thing with a range. So I added it! Hope it helps others!

My pleasure!


Retain search query in search box

What do you need?
To remind the user which search term they used on the search results page.

One way to do this would be to retain the search query in the box ala 18F.

Which entry/section is this related to?
Any search result

Now being tracked in Trello


JBPM-5045 "Try-catch-finally"-zed transaction handling.

PR for this issue: https://issues.jboss.org/browse/JBPM-5045
Master PR: https://github.com/droolsjbpm/jbpm/pull/432
Please merge this. It is needed for 6.3 product testing on supported DBs.

jenkins retest this

test failure not related to the changes, +1


Merge to publish

Docs Build status updates of commit 3f2ac21:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/actionable-messages/index.md
:white_check_mark:Succeeded
View

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


AdSizeCons SmartBannerPortrait throws Arg_NullReferenceException

Hi,
I have migrated my Xamarin iOS native app to .NET iOS 7 - all build and working fine on an iOS simulator but on a physical device,  I'm seeing this exception trying to access the constant ad size SmartBannerPortrait.
Same error when using AdSizeCons.Banner.
Google.MobileAds.AdSizeCons.get_SmartBannerPortrait()

Is AdSizeCons still supported on the .NET iOS framework?

Happening the same error on .Net iOS 8


Feature: Support for spread attributes

Hi,
Currently, spread attributes don't seem to be supported.
<hr
  disabled
  { attrs... }
/>

produces an AST of
    (element ; [4:2 - 7:3]
      (self_closing_tag ; [4:2 - 7:3]
        name: (element_identifier) ; [4:3 - 4]
        (attribute ; [5:3 - 10]
          name: (attribute_name)) ; [5:3 - 10]
        (attribute ; [6:3 - 3]
          name: (attribute_name)) ; [6:3 - 3]
        (attribute ; [6:5 - 12]
          name: (attribute_name)) ; [6:5 - 12]
        (attribute ; [6:14 - 14]
          name: (attribute_name)) ; [6:14 - 14]

It would be helpful if it was treated as a single attribute node.
Thanks for all the work you do on this!

Hi,
thanks for reporting this issue.
The multiple attributes that you get here is actually the parser getting confused, it produces an attribute node for {, attrs..., } because it has no clue about spread attributes.
I didn't implement parsing for this yet (mainly because I wasn't using it).
I'll look into it.

Thank you! I am making a plugin to sort the attribute nodes within elements and it would be helpful if spread attributes were either a single node of type attribute. Even if they were their own specific node type, that would work as well.

@JonnyLoughlin please take a look at #32, it should do what you want.

Looks like it should perfectly solve my issue, thanks!


NUC Ethernet connection setup

mbed Ethernet works on osx environment, fails to connect on NUC. Requires network env change.

Works if disable Wi-Fi


Added logo to definitions search field

This will work in both Homepage and Workspace:

Fixes #636

These. Look. Amazing!

I'm gettting errors trying to select the items in the list - can you take a look?

@dabutvin thanks, i was getting error on e2e tests. Now it should be fine.

it's working now, but you have to click directly on the text. Can we let users click anywhere on the bar?

@dabutvin i've updated the code, apparently my previous implementation doesn't allow the whole row to be clicked. Thanks!

this test did eventually pass, but did not report back correctly it seems

SWEET!


Visual Studio Code won't save any file

Issue Type: Bug
Working with Visual Studio Code 1.27.2 on a Mac OS High Sierra 10.13.6.
For the last 30 minutes, I've been unable to save any file in VSC.
If I reboot, the unsaved file has persisted in VSC, and I can save that file.
However, if I open a different file and edit that file, the symptom reappears, and a reboot will be required.
I've not installed any VSC plugins, extra apps, or system updates since yesterday when VSC was working fine.
Help appreciated.
VS Code version: Code 1.27.2 (f46c4c469d6e6d8c46f268d1553c5dc4b475840f, 2018-09-12T07:04:11.329Z)
OS version: Darwin x64 17.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4790K CPU @ 4.00GHz (8 x 4000)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: unavailable_softwarevideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
1, 2, 3

Memory (System)
16.00GB (3.69GB free)

Process Argv
/Applications/Visual Studio Code.app/Contents/MacOS/Electron -psn_0_49164

Screen Reader
no

VM
40%

Extensions: none

Please see if you can find any logged output that could help us find the source of the issue:

run code --verbose from the command line and attach the output
in Code select Help | Toggle Developer Tools and select the Console
attach any output you see in the Console you can find

Thanks!

@insight47 it should work if you do this:

Thanks again. Where do I type >install in ? I don't see a field to type that into.

That's great, thanks Dick.
@bpasero I receive the following when running "code --verbose" in the Mac terminal:
[main 14:28:38] Sending env to running instance...
[main 14:28:38] Sent env to running instance. Terminating...
[main 14:28:38] Lifecycle#kill()

And when you have VSCode open and you open the developer tools?
Help -> Toggle Developer tools
Switch to the "Console" tab
When you have the issue, do you see any weird output?

Hi @DickvdBrink, yes in developer tools console, I see:
INFO no standard startup: not just one text editor /Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:276
[Extension Host] Git installation not found.
t.log @ /Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:276
/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:291
[node.js fs] fdatasync is now disabled for this session because it failed:  Error: EBADF: bad file descriptor, fdatasync
(anonymous) @ /Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/workbench.main.js:291

@insight47 interesting, so the error seems to be this: EBADF: bad file descriptor, fdatasync
Is there anything special about your environment? Is there a network drive involved maybe?

Hi @bpasero yes there is a network drive involved.
The folder I am working with is saved on the network drive.
There is no problem opening files though, and I have read/write access on the network drive.

@insight47 would you be willing to conduct a small experiment and install node.js (8.x) on your system to run some code to test if you can save with node.js (that is the library VSCode is using)?

Yes, that's fine @bpasero

@insight47 cool, so install node.js from here, then:

create a text file on that network drive so that you can write to it and remember the path
open a terminal
run node
type fs.writeFileSync(<pathToFile>, "Hello World")

Does that work or produce errors?

Hi @bpasero, I receive "undefined".

fs.writeFileSync("/Volumes/DESIGN WIP/Steve's WIP @ 6.8.18/test.txt", "Hello World")
undefined

@insight47 and did the contents of the file change, can you check?

Yes they did @bpasero

Weird, so it seems like it is possible to save, just not with that VSCode version.
Can you try to reproduce with our nightly insider builds? You can give our preview releases a try from: https://code.visualstudio.com/insiders/

Hi @bpasero , the nightly build has the same problem. Cheers.

@insight47 ok need some more things to try, can you:

open vscode
open the console (F12 > Toggle Developer Tools)
open a file from that remote disk
save it by writing some content to it
I would now expect you to see the error console
open the file with another window to see if your changes have been written to it
save it again
I would not expect to see another error in the console

Hi @bpasero , I was able to open the remote text file, edit it, and save it without trouble, and with no console errors.
I followed the same process with a CSS file, and received the following in the development console:

extfs.ts:453 [node.js fs] fdatasync is now disabled for this session because it failed:  Error: EBADF: bad file

If I open the same file in Dreamweaver, make a change, and save it, I receive "access denied".
If I try to save it a second time, it saves okay.

@insight47 then it seems like this is something VSCode cannot fix if other tools have the same issue. I suggest you check the file permissions. I will close this issue.

@bpasero if the file permissions were not correct, I wouldn't be able to save the file on the 2nd attempt. I've checked the same file again, and there is no file permission issue.


Hide pre_script from logs and save space when no newline in the script

Lint report for commit 1c6d5fb795ea3a11ddb6bf08f0967a32e66b074c:
Everything seems all right from lint perspective :smile: .
Commit inspection report:
No commit flaws detected.

CFME QE Bot


Installing extension from VSIX after installing from location deletes original directory

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

VS Code Version: 1.80.1
OS Version: Mac OS 13.4.1

Steps to Reproduce:

Set up extension
Use Developer: Install Extension From Location to install the extension from the root path of the extension (the one with package.json etc)
Package the extension into vsix
Use Developer: Install Extension From VSIX and select the newly created vsix (this file can be anywhere)
The extension directory gets deleted

This is not fixed completely after reverting the fix (https://github.com/microsoft/vscode/pull/190074).
Hence reopened it.

Set up extension
Use Developer: Install Extension From Location to install the extension from the root path of the extension (the one with package.json etc)
Open extensions.json file from extensions installation folder (Use command F1> Open Extensions Folder) and make sure the installed extension location is the location of the extension.
Package the extension into a vsix and move the VSIX to a location other than the extension location.
Use Developer: Install Extension From VSIX and select the newly created vsix
Open extensions.json file from extensions installation folder (Use command F1> Open Extensions Folder) and make sure the installed extension location is the location of the extracted VSIX in the extensions installation folder.
Restart VS Code and make sure source location of the extension in the disk is not deleted.


Minor fixes

Add support for cell other than lstm, which uses state_is_tuple=True. Remove of unused state 'c' and 'h' in feed.

@lns This PR has merge conflicts.


Remove recompose dependency

Fixes #45 and #52

@LoicMahieu Can we plz get this change deployed? This fixes a vulnerability.

@LoicMahieu Can we plz get this change deployed? This fixes a vulnerability.


tools:replace specified at line:14 for attribute android:allowBackup, but no new value specified,,,,,

it says add tools:replace="android:allowBackup" in your manifest, and after adding this to manifest i got this err

can u send manifest file......paste complete code

[image: image.png]
On Tue, Jan 25, 2022 at 2:17 PM evileie @.***> wrote:

can u send manifest file......paste complete code
—
Reply to this email directly, view it on GitHub
https://github.com/evilrat/flutter_zoom_sdk/issues/56#issuecomment-1020929602,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN6BAYVZWCNKX7LVZPNHQATUXZN3VANCNFSM5MS7OEAQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Sir this is my AndroidManifest.xml file i am unable to fix this issue i
Hope you will help to fix this . thank you so much.
On Tue, Jan 25, 2022 at 5:09 PM santosh thapa @.***>
wrote:

[image: image.png]
On Tue, Jan 25, 2022 at 2:17 PM evileie @.***> wrote:

can u send manifest file......paste complete code
—
Reply to this email directly, view it on GitHub
https://github.com/evilrat/flutter_zoom_sdk/issues/56#issuecomment-1020929602,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN6BAYVZWCNKX7LVZPNHQATUXZN3VANCNFSM5MS7OEAQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

image

where is image

[image: image.png]
…
On Tue, Jan 25, 2022 at 2:17 PM evileie @.> wrote: can u send manifest file......paste complete code — Reply to this email directly, view it on GitHub <#56 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AN6BAYVZWCNKX7LVZPNHQATUXZN3VANCNFSM5MS7OEAQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you authored the thread.Message ID: @.>

no image....please send code

tools:replace="android:allowBackup

and also send error log....

<manifest xmlns:android="http://schemas.android.com/apk/res/android"
package="com.eacademynepal"
xmlns:tools="http://schemas.android.com/tools"

<application
android:name="io.flutter.app.FlutterApplication"
@./app_name"
@./Theme.AppCompat.Light.NoActionBar"
@.***/ic_launcher">

<meta-data
android:name="android.support.FILE_PROVIDER_PATHS"
@.***/provider_paths"/>

<activity
android:name="com.eacademynepal.MainActivity"
android:launchMode="singleTop"
android:exported="true"
@.***/LaunchTheme"
android:configChanges="
orientation|keyboardHidden|keyboard|screenSize|smallestScreenSize|locale|layoutDirection|fontScale|screenLayout|density|uiMode
"
android:hardwareAccelerated="true"
android:windowSoftInputMode="adjustResize">

<meta-data
android:name="io.flutter.embedding.android.NormalTheme"
@.***/NormalTheme"
/>

<meta-data
android:name="io.flutter.embedding.android.SplashScreenDrawable"
@.***/launch_background"
/>

On Tue, Jan 25, 2022 at 8:38 PM EvilRATT. @.***> wrote:

tools:replace="android:allowBackup
and also send error log....
—
Reply to this email directly, view it on GitHub
https://github.com/evilrat/flutter_zoom_sdk/issues/56#issuecomment-1021264804,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN6BAYR5YSSN75HEY6MABJDUX22OTANCNFSM5MS7OEAQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

On Tue, Jan 25, 2022 at 10:18 PM santosh thapa @.***>
wrote:

<manifest xmlns:android="http://schemas.android.com/apk/res/android"
package="com.eacademynepal"
xmlns:tools="http://schemas.android.com/tools"

<application
android:name="io.flutter.app.FlutterApplication"
@./app_name"
@./Theme.AppCompat.Light.NoActionBar"
@.***/ic_launcher">

<meta-data
android:name="android.support.FILE_PROVIDER_PATHS"
@.***/provider_paths"/>

<activity
android:name="com.eacademynepal.MainActivity"
android:launchMode="singleTop"
android:exported="true"
@.***/LaunchTheme"
android:configChanges="
orientation|keyboardHidden|keyboard|screenSize|smallestScreenSize|locale|layoutDirection|fontScale|screenLayout|density|uiMode
"
android:hardwareAccelerated="true"
android:windowSoftInputMode="adjustResize">

<meta-data
android:name="io.flutter.embedding.android.NormalTheme"
@.***/NormalTheme"
/>

<meta-data
android:name="io.flutter.embedding.android.SplashScreenDrawable"
@.***/launch_background"
/>

On Tue, Jan 25, 2022 at 8:38 PM EvilRATT. @.***>
wrote:

tools:replace="android:allowBackup
and also send error log....
—
Reply to this email directly, view it on GitHub
https://github.com/evilrat/flutter_zoom_sdk/issues/56#issuecomment-1021264804,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN6BAYR5YSSN75HEY6MABJDUX22OTANCNFSM5MS7OEAQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

I have been in the same problem as I was using "android:usesCleartextTraffic="true"" in androidManifest, the error says it is also being used in the zoom plugin, So.
Just remove => android:usesCleartextTraffic="true" from android/app/src/main/AndroidManifest.xml
no need to use tools:replace.

this is error log
Attribute application@allowBackup value=(true) from [eSewaSdk.aar] AndroidManifest.xml:14:18-44
is also present at [:mobilertc:] AndroidManifest.xml:82:9-36 value=(false).
Suggestion: add 'tools:replace="android:allowBackup"' to  element at AndroidManifest.xml:14:4-85:19 to override.

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:processEacademyDebugMainManifest'.

Manifest merger failed : Attribute application@allowBackup value=(true) from [eSewaSdk.aar] AndroidManifest.xml:14:18-44
is also present at [:mobilertc:] AndroidManifest.xml:82:9-36 value=(false).
Suggestion: add 'tools:replace="android:allowBackup"' to  element at AndroidManifest.xml:14:4-85:19 to override.

I have not used that .

I have been in the same problem as I was using "android:usesCleartextTraffic="true"" in androidManifest, the error says it is also being used in the zoom plugin, So.
Just remove => android:usesCleartextTraffic="true" from android/app/src/main/AndroidManifest.xml no need to use tools:replace.

/// I did not use this in my AndroidManifest.xml

eSewaSdk

you are using eSewaSdk which is causing this issue......
did u try to add
tools:replace="android:allowBackup" on  AndroidManifest.xml application tag

eSewaSdk

you are using eSewaSdk which is causing this issue......
did u try to add
tools:replace="android:allowBackup" on AndroidManifest.xml application tag

///
I just removed eSewaSdk and also added tools:replace="android:allowBackup" in AndroidManifest.xml but i still another error which are :
--->  tools:replace specified at line:14 for attribute android:allowBackup, but no new value specified.
---->    Manifest merger failed with multiple errors, see logs

eSewaSdk

you are using eSewaSdk which is causing this issue......
did u try to add
tools:replace="android:allowBackup" on AndroidManifest.xml application tag

/// I just removed eSewaSdk and also added tools:replace="android:allowBackup" in AndroidManifest.xml but i still another error which are : ---> tools:replace specified at line:14 for attribute android:allowBackup, but no new value specified. ----> Manifest merger failed with multiple errors, see logs

android:allowBackup="true"


bug-frame-charge-94

Closes #94 Fix

In this part of create_frame_dataset.py
            gaussian_atom = gaussian_matrix[:, :, :, atom_idx]
            # Add at position:
            frame = add_gaussian_at_position(
                main_matrix=frame,
                secondary_matrix=gaussian_atom,
                atom_coord=indices,
                atom_idx=atom_idx,
            )
            if res_property != 0:
                gaussian_atom = gaussian_matrix[:, :, :, atom_idx] * float(res_property)
                # Add at position:
                frame = add_gaussian_at_position(
                    main_matrix=frame,
                    secondary_matrix=gaussian_atom,
                    atom_coord=indices,
                    atom_idx=5,
                    normalize=False,
                )

              ```

           Does it appear that we are declaring the  gaussian_atom variable and running add_gaussian_at_position function redundantly? In that case, does this one make sense:

           ```
                if res_property != 0:
                gaussian_atom = gaussian_matrix[:, :, :, atom_idx] * float(res_property)
                # Add at position:
                frame = add_gaussian_at_position(
                    main_matrix=frame,
                    secondary_matrix=gaussian_atom,
                    atom_coord=indices,
                    atom_idx=5,
                    normalize=False,
                  else:
                   gaussian_atom = gaussian_matrix[:, :, :, atom_idx]
            # Add at position:
            frame = add_gaussian_at_position(
                main_matrix=frame,
                secondary_matrix=gaussian_atom,
                atom_coord=indices,
                atom_idx=atom_idx,
            )
                )
                )

                ```

@sunal1996 in the case of a charged residue (say lysine), the gaussian needs to be added in the atom index (C, N, O, Ca, Cb) AND in the charge index (ie. the fifth).
I think in your if/else logic all the atoms of a charged residue (C, N, O, Ca, Cb) would end up in the 5th channel and there would be no atoms in the C, N, O, Ca, Cb channels


Skip mbed-os import in subdirs

The change is rather straightforward - if we come across an mbed-os.lib file in a library imported by an app, skip it. My test set up is as follows:

A library that has an mbed-os.lib in it.
An app that imports that library as well as mbed-os (e.g., has two *.lib files).

mbed import of both yields the desired results:

Importing the library alone gets me an mbed-os to work with
Importing the app gets me the library and mbed-os in the top level directory with no redundant mbed-os import

My knowledge of mbed-cli.py is limited, however, so the above may not be sufficient. Please review - I will not be at all offended by rejection.
Fixes #472 - mbed-cli can clone mbed-os.lib more than once.

NOTE: this can be generalized by keeping track of all the repos imported at the cwd_root level. A solution for this could be to keep a global set that holds the names of all the *.lib files that have been used to import so far. This would still be isolated to Repo.getlibs.
Alternatively, one could use a similar global set, but go a little deeper: save the actual URLs that have been used. This would be managed in Repo.fromurl. This would be a little awkward, however, in that Repo.fromurl would return None when a redundant import is found. I assume we'd want some action() call stating why something was skipped.

@wdwalker firstly thanks for the PR.
The implementation as it is creates ambiguity about what version will be used in case of conflicting dependencies. As an example a library may have mbed-os.lib at rev #ABC, but the program may have mbed-os.lib at rev #XYZ. The user end will end up with mbed-os#XYZ if they import the program, which is different if they import the library (which would give them #ABC otherwise).
Also we'd like to encourage a clean development model where a library is NOT the root of the tree. Here's some reasoning behind it:
#1. A library doesn't contain the main application code, otherwise it would really be a program...
#2. A library should live side-by-side mbed-os and handle it's own dependencies and sub-libraries
#3. mbed-os is NOT a sub-library of a library (it's an OS) and lives at the root of the tree (again at program level, not library)
#4. The program defines which mbed-os revision is used, not a library (the requirement for mbed-os to be at the root of the program is not by accident).
Do you feel that a development tree like below would work for you?
example_program
 |- library-dev
 |- mbed-os
 |- main.cpp
 |- other-stuff.h
 `- other-stuff.cpp

@screamerbg - many thanks for the response. I think the development tree looks fine and it is what I've been working with in my own sandbox. What I'm looking to do is have my library contain its own TESTS and such; those need mbed-os headers and sources to compile against. This will more easily enable library development independently of an app and automated CI.
I definitely agree about the ambiguity issue and also agree that the app should dictate the mbed-os version. One might argue that knowing which version of the mbed-os sources the library was developed against is useful info, though. Some sort of compatibility framework might be nice (though out of scope for this PR).
Today, I do an 'mbed add mbed-os' whenever I work with my library directly and I also do the same in the library's .travis.yml. I was hoping to be able to avoid this via the patch proposed in this PR, but I'm OK with continuing to do what I do today.
Happy to close this PR if you think it's a dead end. Please let me know.

@wdwalker I think we should rather show a good visible warning when duplicate clones are detected. This will help the user understand that there might be underlying problem and duplicate code.


Support only re-creating session venv if poetry.lock has changed

nox-poetry currently by default always recreates the virtual environment on every invocation of the session. This has the advantage of ensuring that the environment is always up to date, but at the cost that environments will be recreated unnecessarily when there are no changes to the locked dependencies.
Consider this small example project:
[tool.poetry]
name = "poetry-test"
version = "0.1.0"
description = ""
authors = []

[tool.poetry.dependencies]
python = "^3.10, <3.12"

[tool.poetry.group.dev.dependencies]
nox-poetry = "*"

[tool.poetry.group.lint.dependencies]
flake8 = "*"
flake8-bugbear = "*"
flake8-broken-line = "*"
flake8-comprehensions = "*"
pep8-naming = "*"
flake8-pyproject = "*"

[build-system]
requires = ["poetry-core>=1.0.0"]
build-backend = "poetry.core.masonry.api"

# noxfile.py
from nox_poetry import Session, session

@session
def lint(s: Session) -> None:
    s.install(
        "flake8",
        "flake8-bugbear",
        "flake8-broken-line",
        "flake8-comprehensions",
        "pep8-naming",
        "flake8-pyproject"
    )
    s.run("flake8")

Running a simple lint invocation recreates the environment every time. Even when the wheels are cached on the user's system, the recreation can take significant time and add delay into the developer's workflow:
$ time nox -s lint
nox > Running session lint
nox > Creating virtual environment (virtualenv) using python in .nox/lint
nox > poetry export --format=requirements.txt --dev --without-hashes
The `--dev` option is deprecated, use the `--with dev` notation instead.
nox > python -m pip install --constraint=.nox/lint/tmp/requirements.txt flake8 flake8-bugbear flake8-broken-line flake8-comprehensions pep8-naming flake8-pyproject
nox > flake8
nox > Session lint was successful.
nox -s lint  16.02s user 2.58s system 90% cpu 20.528 total

On a Macbook Pro, it took 16 seconds to recreate this relatively small venv.
If we were to reuse it:
time nox --reuse-existing-virtualenvs -s lint
nox > Running session lint
nox > Re-using existing virtual environment at .nox/lint.
nox > python -m pip install --constraint=.nox/lint/tmp/requirements.txt flake8 flake8-bugbear flake8-broken-line flake8-comprehensions pep8-naming flake8-pyproject
nox > flake8
nox > Session lint was successful.
nox --reuse-existing-virtualenvs -s lint  0.90s user 0.29s system 96% cpu 1.227 total

Only 0.9 seconds.
nox-poetry has an advantage over Nox in that we know that the user is using Poetry. Poetry includes a helpful hash of the entire lockfile and important Poetry-related metadata from pyproject.toml.
# poetry.lock
[metadata]
lock-version = "2.0"
python-versions = "^3.10, <3.12"
content-hash = "f8d0beddbcf539e1dcd2e67603593a3c1b19a31df9b295b066d498110aa03b51"

If the content-hash were read using tomlkit (which is already a dependency of nox-poetry) and stored with each venv, then nox-poetry could very quickly check if a virtual environment needed to be recreated or not. This would speed up local development using nox-poetry greatly.
One open question could be, do we want to also try to handle the case when there are changes to the session itself? For example if different packages were passed to Session.install()? Perhaps there would be a simple way to add that with the lockfile hash and then it would be possible to skip the entire install step.
This can be demonstrated with this edit to the session:
from nox_poetry import Session, session

@session(venv_backend="none")
def lint(s: Session) -> None:
    s.run("flake8")

Avoiding the install entirely speeds up the command even more (because we are skipping the pip install --contraint... step which still needs to validate that the venv has all of the needed requirements.
time nox -s lint
nox > Running session lint
nox > flake8
nox > Session lint was successful.
nox -s lint  0.30s user 0.12s system 96% cpu 0.436 total

Upon closer inspection, content-hash only covers the changes in pyproject.toml

https://github.com/python-poetry/poetry/blob/33b8b5a480de739e29938503209e404e0fa2a2a8/src/poetry/packages/locker.py#L276

So for nox-poetry, we'd want to hash the full poetry.lock file and store/compare that as any changes to the locked dependencies also need to accounted for. What's convenient is that because the pyproject.toml content-hash exists, we don't need to worry about pyproject.toml directly as we should be able to assume the user is keeping their lockfile fresh (using something like poetry lock --check) or else locking isn't defined any longer.

I'm not sure I understand why you don't just use nox -r? If you want to avoid even the install commands, use nox -R instead.
Is your suggestion that nox-poetry keep track of the content-hash and imply one of those flags if that hash doesn't change? I would argue that "explicit is better than implicit". I would also consider this out of scope for nox-poetry--we're just the minimal glue code for Nox to make the installations respect Poetry's lock files.

Is your suggestion that nox-poetry keep track of the content-hash and imply one of those flags if that hash doesn't change?

Yes.
Let me try to explain the motivation for this:

Make running repeated Nox sessions as fast possible (by avoiding rebuilding venvs on every invocation)
Avoid accidentally running a Nox session with a stale venv that doesn't have the correct dependencies installed

In the base Nox configuration, these two goals are at odds. You can set nox.options.reuse_existing_virtualenvs = True so that the commands run instantly and skip the install step, but this has the risk that if your lock dependencies change, you could be running with an incorrect/stale venv.
In practice, this happens on projects with large teams. It's not always obvious to know/remember to use -r or -R if other team members are merging PRs that might change dependencies. You could also be working on 5-10 different Poetry/Nox projects.
The rationale for the speed, we use Nox to provide a simple way to run tools such as black, isort, flake8, mypy, etc. The time for Nox to recreate a venv and install dependencies into it can often times dwarf the time of actually running the tools themselves, leading to a less-than-ideal UX for the developer.
With Poetry, we have at least the ability to do what Tox does and smartly recreate the venv only when needed, based on hashing poetry.lock. The Tox team added this feature for similar reasons listed above.

I would also consider this out of scope for nox-poetry

I can certainly understand/appreciate this! Perhaps a new/different Nox/poetry tool could take this on.

You can set nox.options.reuse_existing_virtualenvs = True so that the commands run instantly and skip the install step, but this has the risk that if your lock dependencies change, you could be running with an incorrect/stale venv.

Nox doesn't skip the install step if you set this option. You need to pass --no-install or -R for that.
I don't recommend defaulting to venv reuse in the noxfile; it should be an explicit decision. Tox has a different model here AFAIU--they cache by default.
Generally, I'd recommend avoiding reuse in CI, as well as locally when you start new work or sync with origin. Then, as you iterate on your work locally, you can choose to speed up things with nox -R--you'll know if you've touched the dependencies.
If you want automation to enforce correct practice across large teams, how about doing the work yourself in the noxfile.py? You could read the current and previous content-hash from poetry.lock and a local cache, and set reuse_existing_virtualenvs based on the outcome.

I can certainly understand/appreciate this! Perhaps a new/different Nox/poetry tool could take this on.

As nox-poetry author, I have zero issues with this. From the point of view of Nox, it's not ideal though. It would be better if people used the flexibility of noxfile.py before creating more inofficial wrappers. If people really need more flexibility than that, maybe Nox can add official extension points. Also, I'd hope that eventually a PyPA lockfile standard could improve this whole situation, and we'll no longer need tool-specific workarounds.


The frontal_face_detector takes extremely long time to handle a picture.

Hi, I use the example code ( face_detection_ex.cpp ) to detect face in a 640*480 picture. but it takes almost 23+ seconds every time.  am I compile it in a wrong way?  My computer basic info :  win7-64bit, Intel i5-6500, 8GB RAM.

Compiler params affect the performance much. Read the FAQ first
This is not a bug, should be closed

Thanks.


Fix: incorrect transform jsx-runtime in development

Motivation 🔥
Fix: #2656
Currently, swc does not transform JSX correctly with new JSX Transform in development mode. It is importing jsx from jsx-runtime when it should be importing jsxDEV from jsx-dev-runtime. (The same applies to jsxs).
I fixed this.

Oh! Sorry, I overlooked it :pray:
I updated code with passing isStaticValidation, source, self (this) to jsxDEV, and not set __source and __self as props with "runtime": "automatic" config.

Code changes look fine to me, but can you add some more tests?
Actually this feature was once implemented by https://github.com/swc-project/swc/pull/2542, but I reverted it because it broke next.js

Thank you!
I've added some tests on this, but I am wondering what more tests to add.
Can you share what happened or error logs when Next.js was broken? (I searched for the Next.js issue but couldn't find it)

@Shinyaigeek https://github.com/vercel/next.js/pull/30302#issuecomment-952042021
This was the issue, and thank you!

I’m sorry for the late response, I didn’t have time due to my graduation thesis.
Based on the report you shared, I investigated the contexts and test cases we missed and found the cause of the "next / dynamic'dev mode' should render dynamic server rendered values on client mount" error. This is that isStaticChildren would always be undefined (cast false) in React.Fragment.
Another, "Invalid hrefs ›dev mode ›should not show error for function component with forwardRef" The cause of these errors is not well understood, but next / link should have only one child element. , I'm guessing that an error has occurred there. I will continue to investigate it a bit more

I’m sorry I did not understand why #2542 broke next.js (https://github.com/vercel/next.js/pull/30302#issuecomment-952042021). I think there are enough test cases for this change, but I am also worried that this change will break next.js as https://github.com/vercel/next.js/pull/30302#issuecomment-952042021 . Is there any documentation for running Next.js tests with the local version of swc? (I rewrote Cargo.toml of next-swc with {path = "..."}, I get a compile error ...)

@Shinyaigeek Same here. I don't understand the reason.
I think in this case it will be fine, as you added tests for all input. But if you want to try it, you can patch all swc dependencies in https://github.com/vercel/next.js/tree/c004322d138e1949437efb0b80e0973f3a3775cf/packages/next-swc/crates to use git or path dependency targeting your own branch and create a PR.

But I really think it will be fine in this case, as we tested all input.

But I really think it will be fine in this case, as we tested all input.

@kdy1 I see. It seems to be a tough task to test with next-swc, so is it okay to get a review as it is, and send a follow up issue & PR if there is any problem?

Yeah, I think it is better. Thanks for the pr again and I'll review it soon


v2.3 claims to add top & bottom options via side property but errors if anything other than left or right

I tried to make use of the added functionality in v2.3 that states added options to open drawer from top or bottom but when I set the "side" property to "top" or bottom" I get an warning error saying expecting either "left" or "right":

There is even documentation stating how to use it:  but doesn't actually work:
side (String left|right|top|bottom) left - which side the drawer should be on.

also having this issue, would be really nice to have working top drawer

+1 for the bottom

Still not working for me


AMPM matches non-time text

Hi!
I've been using these rules and they have greatly improved my text. However, I came across something unexpected. The AMPM rule is being activated by text like "5 ampoules". Looking at the rule I can see that this text would match the last REGEX, '\d{1,2} ?[aApP]\.[mM]\.'.
Is this the expected behaviour?
Best regards, Andrew.

This should be fixed in the next release.

Brilliant. Thanks for sorting so quickly!


Question about the "envoy_cluster_update_failure" metric

Hi,
Setup is envoy is used as CONNECT terminator and routing to upstream cluster per domain.
When using envoy cluster type STRICT_DNS with 15s update interval we had an event when for one upstream from 200 it changed to 503 for some period of time :

the 503 are also retried 3 times with no success as well which points to the upstream having network issue and not responding.
The question is why the envoy_cluster_update_failure is incremented - the Docs say that this is:
Total failed cluster membership updates by service discovery
Confirmed from the logs the IP of the upstream_host is resolved and the response_code_details are upstream_reset_before_response_started{connection_failure} UF,URX which is normal for upstream connectivity issue.
Is there a chance it was something wrong with local Envoy and service discovery not updating the cluster somehow?
I don't fully understanding the stats from the Service Discovery graph in that case ..
BR,
Stoyan

cc @jmarantz

update_failure for STRICT_DNS cluster means, the periodic dns resolution failed to upstream DNS server.

Hi, thanks for looking into that.
I agree if periodic dns resolution fails in my case this explains the serviceDiscovery grafana panel but then why the request has an upsteram ip selected and actually connects to it. I would assume if dns tesolution failed the upstream ip will be null in the logs.


There is Issue with Starting and closing with php

Which is? Any errors you can share?

this issue showing on close and start

Change the port you're running Ratchet on. Another server is already bound to port 8015, you can't have two services listening on the same port.

how to close this port by php ... i know Its started i need to stop by php method

As @WyriHaximus said in #871 look into signal handling.


Flow caches failures across processes (despite killall)

System info:
Linux machine 4.14.11-1-ARCH #1 SMP PREEMPT Wed Jan 3 07:02:42 UTC 2018 x86_64 GNU/Linux
Flow info:
Flow, a static type checker for JavaScript, version 0.59.0
Logging this separately to #4555 as the issue description in that one currently specifies that the flow (no pun intended) to get that bug is that it requires a killall - in my case, killall doesn't solve the problem (this is only mentioned in a comment on that issue).
I frequently get cached errors pointing towards lines that have clearly changed even using killall, and have to subsequently remove my $HOME/.cache/yarn contents to clear the error. E.g.
Error: <fiilename>:<filenumber>
101:     thing => thing.foo || null,
                            ^^^^^^^^^^^^^^^^ property `bar`. Property not found in
101:     context => context.foo || null,
                    ^^^^^^^ object type

This is still an issue. Can someone explain how to clear cached state? I've tried stopping and starting flow but problem persists.

The issue is likely resolved a long time ago


[BUG] Unable to use Google APIs in a deno slack app

The deno-slack versions
    "deno-slack-sdk/"<EMAIL_ADDRESS>    "deno-slack-api/"<EMAIL_ADDRESS>
Deno runtime version
deno 2.0.2 (stable, release, aarch64-apple-darwin)
v8 <IP_ADDRESS>-rusty
typescript 5.6.2

OS info
ProductName:		macOS
ProductVersion:		15.1
BuildVersion:		24B83
Darwin Kernel Version 24.1.0: Thu Oct 10 21:02:26 PDT 2024; root:xnu-11215.41.3~2/RELEASE_ARM64_T8122

Describe the bug
Unable to make use of Google deno packages.
I'm attempting to import the Google sheets API using
import { Sheets, auth as GoogleAuth } from 'https://googleapis.deno.dev/v1/sheets:v4.ts';

This was working earlier in the year and I have a deployed application that utilizes this, but now when I use slack run I get the following error
error: Uncaught (in promise) TypeError: Requires import access to "googleapis.deno.dev:443", run again with the --allow-import flag

I've searched high and low and tried add --allow-import to get-hooks in slack.json but haven't be able to find a way to convince deno to allow the import of the google APIs. I reached out to Slack support and they directed me here.
Steps to reproduce

Clone the deno starter template https://github.com/slack-samples/deno-starter-template
In functions/sample_function.ts, add

import { Sheets, auth as GoogleAuth } from 'https://googleapis.deno.dev/v1/sheets:v4.ts';

Run the project using slack run

Expected result
Expect that I'm able to import google deno packages.
Actual result
It fails with the following error
error: Uncaught (in promise) TypeError: Requires import access to "googleapis.deno.dev:443", run again with the --allow-import flag

Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

Try adding googleapis.deno.dev to the outgoingDomains array of your app manifest.
https://api.slack.com/automation/manifest

I get the same thing when attempting to follow the slack announcement bot tutorial. Running step 6 where you create the trigger it fails with the same message. I. My case it fails trying to reach cdn.skypack.dev.  I checked and the demo apps manifest includes the outgoing domains section and lists the location so I don’t think that’s it.


Bot isn't deleting entry/kick messages

I didn't comment this on #52 to avoid polluting the issue with something possibly unrelated.
Even with /deleteEntryOnKick we are still seeing both messages (joined and removed) in our channels:

We've already tried to set the bot's permissions with as much permission as possible, but without success:

Is there anything else that we need to set or try, please?

Is the group Private ?

They are public groups.
Originally they were created as super groups but it seems that now there isn't any kind of distinction anymore.

Thanks, I'll investigate

Yep, same for me

@naoliv and I diagnosed the problem in the said group, there was confusion between what the /deleteEntryOnKick and /deleteEntryMessage commands would do.
I guess better Commands Labels could have prevented this issue.
Do you have suggestions for better Brazilian translations Naoliv ?

It seems that the bot's behavior is a little erratic.
For example, today we saw again the messages in another group:

/deleteEntryOnKick was already enabled (and you can see that after sending /deleteEntryOnKick the bot said it wouldn't remove the messages anymore)
Per the bot help, /deleteEntryOnKick is the right command to have the entry messages deleted:
/deleteEntryOnKick — whether to delete entry messages for users who failed captcha
Regardless if /deleteEntryMessages is enabled or not, it seems that /deleteEntryOnKick is still failing to delete some messages.

Just tested, everything works fine. With /deleteEntryMessages Shieldy deletes all entry and left messages. With /deleteEntryOnKick Shieldy deletes entry and left messages for people who failed captcha.

Found a bug, fixed it, behavior should not be erratic anymore. Thank you for the report!


(TravisCI) Output VM memory stats after the tests

Looks to be giving the output we need. Assigning to @simong for review.

Note that the intent of this is to try and do post-mortem analysis on a regular test failure where the logs indicate that cassandra and redis both stopped responding. This could be a result of a resource issue on the VM.


Add docs for how to test your project using Kotter

Basically, how to use the kotterx.test library.

The super quick version is, check the tests under kotter/kotter/src/main/test. They use testSession and then the terminal parameter it provides to verify the state of the console.

This is now done here:
https://github.com/varabyte/kotter/blob/main/kotterx/kotter-test-support/README.md


Corruption/artifacts?

I'm not sure what's going on here, but we've been noticing mapnik outputting some strange artifacts. The problem is intermittent – a re-render of the same area makes it go away. The node-mapnik version is the latest one: 3.3.0. Any ideas? Other notes:

This is with tilelive-mapnik
It seems to have to do with rendering of rasters (the artifact color matches our hillshade color)
PNG output

4x4 metatile (the cyan lines are the tile boundaries)

Thanks for the details. We've not yet tagged a new node-mapnik since #433 was fixed, so it might be that. Also, to clarify: is the blue box the artifact or is that just the browser highlight (the the artifacts are the faint semi-transparent blue bands on the top 1/3 of those tiles)?

@springmeyer The blue box is the browser highlight – sorry for not clarifying! The artifact is the pixel'y blue band at the top 1/3.

Okay, thanks. I worry this is the same issue as #437. Are you seeing any logs like https://github.com/mapnik/node-mapnik/issues/437#issuecomment-101592842 at the same times as you see these artifacts?

Are you seeing any logs like #437 (comment) at the same times as you see these artifacts?

bump @brianreavis ^^

I think we can hold off on this until #437 is fixed, I think. I unfortunately haven't watched the logs closely enough to tell if they happen at the same time. This one's a bugger. I'll keep this thread updated with any more information I can find.
You're probably right – there's a good chance that it's correlated to (or the same as) the other issue. We can close / re-open if necessary to cut down on Issue noise if you'd like

okay, thanks. I'm happy to keep open. I'm headed on vacation next week so trying to tie together ideas as much as possible. Hopefully when I'm back the solution will be clear :) At the least hope to have time to replicate.

Haha, sounds great. Enjoy the vacation!

@brianreavis - Lots of action over at #437. Yesterday @mojodna and I worked to run his testcase against node-mapnik master + mapnik master + gdal 2.0.0 on OS X. My summary is:

@mojodna has not been seeing image corruption, mainly TIFF errors and occasional crashes.
It is looking like using GDAL 2.0.0 fixes his problem overall
So, I'm starting to wonder again if your problem (corrupted images) might be different

Next actions I see here are:

I'll create some alternative node-mapnik binaries against mapnik master + gdal master
@brianreavis sees if the bug still replicates with those binaries
If that fixes this problem then we sync up about moving on https://github.com/naturalatlas/node-gdal/issues/110 so that the next release of node-mapnik and node-gdal could both use the same version.

Sound good?
/cc @brandonreavis

Sounds awesome. You guys and @mojodna have been killing it over there. It's been hard to keep up! Let me know what to try out and I'll give it a whirl.

@brianreavis - okay, you can give the binaries a spin like:
"mapnik": "https://github.com/mapnik/node-mapnik/tarball/gdal2",

And set export VRT_SHARED_SOURCE=0
The binaries will be available once https://travis-ci.org/mapnik/node-mapnik/builds/69403464 is all green.

Perfect. Just put that in place! I'll watch the logs for #437 and keep an eye out for any new instances of the artifacts.

Thank you. Hoping for the best.

closing this as it sounds like the 2.0 update resolved! https://github.com/mapnik/node-mapnik/issues/437#issuecomment-118929702


Feature request - add c7n-org

Greetings,
Thanks for making this.
Would it possible t add c7n-org to this?
Thanks

Added in #3

Hey, is it me, or does c7n-org not work? It installed fine, but I cant run it?
The logs:
[....]
Successfully installed c7n-org-0.6.15 click-8.1.2 typing-extensions-4.2.0
1s
0s
Run for policy in policies/daily/*
#############################################
Policy: policies/daily/ebs-[1](https://github.com/elseu/sdu-cloud-custodian/runs/6886767628?check_suite_focus=true#step:5:1)y-old.yaml:
#############################################
/home/runner/work/_temp/7f9e050d-6e78-4c2a-8b04-eff0db498d[19](https://github.com/elseu/sdu-cloud-custodian/runs/6886767628?check_suite_focus=true#step:5:20).sh: line 7: c7n-org: command not found
Error: Process completed with exit code 127.

My workflow:
name: daily-checks
on:
  workflow_dispatch:
  schedule:
    - cron: "30 7 * * *"

jobs:
  c7norg:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2
      - uses<EMAIL_ADDRESS>        with:
          include-c7n-org: true

      - run: |
          for policy in policies/daily/*
          do
            rm -rf output
            echo "#############################################"
            echo "Policy: ${policy}:"
            echo "#############################################"
            c7n-org run --metrics --metrics-uri "aws://master?namespace=c7n" -c accounts-all.yaml -u ${policy} -s output -r eu-west-1
          done

@sbkg0002 You were correct. c7n-org is being installed under the same virtualenv as the c7n executable so I created a wrapper script for it as well. Also added a test to the test workflow.
Should be fixed in #4 and v1.2.1.


Github Actions error about no environment variables

There are no environmental variables when launching GitHub Actions. PHPUnit showing error while checking required $_ENV entries (from .env, from system settings etc.).

Made workaround to pass the tests by creating temporary .env file with GH environmental variables.


Amazon Cognito Invalid+ProviderName/Username+combination

When a linked user tries to sign in with configured open idp following error appear
error_description=Error+in+SAML+response+processing%3A+Invalid+ProviderName%2FUsername+combination.+Please+check+again+&error=server_error
Steps to reproduce the behavior:

Signup on Cognito login page
Configure OIDC provider in Cognito console
Attribute mapping
 email: email
given_name: given_name
 family_name: family_name
sub: username
Sign in with previously configured oidc provider on Cognito login page
At this state there are two users in Cognito one with userStatus = CONFIRMED and another one with userStatus = EXTERNAL_PROVIDER
Link these two users with
params = { DestinationUser: { ProviderAttributeValue: 'b1644fb1-2fbe-4f24-a6e4-767d0b8598fe', ProviderName: 'Cognito' }, SourceUser: { ProviderAttributeName: 'Cognito_Subject', ProviderAttributeValue: 'jzqdntn76tlxlaeib-hhetosqidt2vy87syqe7ev7bq', ProviderName: '0mock' }, UserPoolId: 'eu-central-1_hoxUCGPVc' }
CognitoIdentityServiceProvider().adminLinkProviderForUser(params)

After users are successfully linked the userStatus for both is EXTERNAL_PROVIDER which is not a case when linking Google and Facebook not sure if this could be the problem

Try to sign in again with the linked user with oidc on Cognito Login page
Following error is thrown error_description=Error+in+SAML+response+processing%3A+Invalid+ProviderName%2FUsername+combination.+Please+check+again+&error=server_error

Expected behavior
Linked users should be able to sign in with configured OIDC

I will close the issue since we found what is the problem. The problem was that we were linking the user with ProviderAttributeValue to lower case, but in the sub claim from the OIDC provider there were capital letters, and that's where the whole confusion was coming from. If the user is linked with the original value provided from the sub claim everything works fine


[6.x] Fix mime validation for jpeg files

These changes fixes a regression introduced by recent changes in Symfony and is a follow up to https://github.com/laravel/framework/pull/35419. It makes sure that if either mimes:jpeg or mimes:jpg is used that it doesn't matter wether the file is a .jpg or .jpeg file. This was the previous behaviour.
I've also cleaned up some incorrect casing for the image rule and cleaned up the tests for the mimetypes rule which were a bit out of date it seems.
As a side note it would be much better if these tests were integration tests against actual files but that's out of the scope for this PR.
See https://github.com/laravel/framework/pull/35419#issuecomment-739707706

I think that the same ‘problem’ happens with heic and heif files...

I don't know anything about those file types sorry.

I don't know anything about those file types sorry.

yeah its ios default setting for saving photos nowadays... well my workaround works so I'll be fine :-)

@it-can I had to do this to get it working
mimetypes:image/heic,image/heif,image/heif-sequence,image/heic-sequence
And of course install the imagick on the server plus using something like intervention/image package to get it working.


Markdown rendering error when HelpTextField value is None

>>> from formidable.forms import fields, FormidableForm
>>> class FullForm(FormidableForm):
...       help_text = fields.HelpTextField(text="This form has every field available")
>>> f = FullForm()
>>> print(f)
AttributeError: 'NoneType' object has no attribute 'strip'

It looks like calling markdown(None) results in an error, because you can't strip what's not a string.

sorry, wrong procedure.


Add Softplus activation function @open sesame 04/15 10:41

Now, user can use Softplus activation function like torch or tensor flow.
Furthermore, we can use this function to build Mish or other activation functions

Self evaluation:

Build test:     [X]Passed [ ]Failed [ ]Skipped
Run test:     [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: heka1024<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2545. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @heka1024, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2545-202404150909200.1524920463562-f83a0b89ae343d65d820be010f18fe5dcc54599a/.

Please rebase:
$ git checkout main
$ git pull ${FILL_upstream_name} main
$ git checkout activation-softplus
$ git rebase main
$ git push ${FILL_your_fork_name} HEAD -f

I found this PR raises error when building with
meson build -Denable-fp16=true
@heka1024
Could you please take a look?

@heka1024 Nope it's build fail, not unittest fail.
 error: converting to ‘_Float16’ from ‘float’ with greater conversion rank [-Werror]
  364 |     return static_cast<T>(log(1 + exp_util<T>(beta * x)) / beta);

 error: converting to ‘_Float16’ from ‘float’ with greater conversion rank [-Werror]
  374 |     return sigmoid<T>(beta * x);
      |            ~~~~~~~~~~^~~~~~~~~~
../nntrainer/layers/acti_func.h:269:52: note:   initializing argument 1 of ‘static T nntrainer::ActiFunc::sigmoid(T) [with T = _Float16]’
  269 |   template <typename T = float> static T sigmoid(T x) {
      |

Try below to generate the error above:
cd ~/nntrainer/
meson build -Denable-fp16=true

Think your commit needs some fixes. Could you create a new PR to fix this issue?

@heka1024
This issue is resolved via #2557
I would like to personally recommend you to check for fp16 build case as well next time!


Unit test for notifications

What type of PR is this?

/kind test
What this PR does / why we need it:
This PR adds unit tests for notifications package.
Which issue(s) this PR fixes:

This pull request addresses part of the issue described in #131
Does this PR introduce a user-facing change?:

NONE

/cc @LiorLieberman @mlavacca

/kind test

/cc


KAFKA-5874: TopicCommand should check at least one parameter is given...

When altering topics, TopicCommand should ensure that at least one of parameters in partitions, config or delete-config must be specified.

@ijuma Please take some time to review this minor patch. Thanks.


Test cross-compiling to build arm binaries

I suspect there is a permissions issue. The CI never runs for my PRs (also see #60 and #65)

For now I've registered my fork with my personal Azure account for testing

/AzurePipelines run

/AzurePipelines run

/AzurePipelines run

Closing now that PR #82 was merged


This Pr is to Add copyright to deprecated_label.go

Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

fixes #XXXX
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Kudos, SonarCloud Quality Gate passed!
  0 Bugs   0 Vulnerabilities   0 Security Hotspots   0 Code Smells
 No Coverage information  0.0% Duplication

/assign cheyang

/assign @cheyang

/assign TrafalgarZZZ


Remove Docker

Containers were not being used by the team to run the application in development, the docker environment appears to have fallen behind as it fails to start due to errors - I got as far as govuk-frontend dependency errors
Containers are now no longer being used in staging or production since the switch from AWS ECS to GOV.UK PaaS which doesn't support containers
There is an instinct to keep hold of this code as it may be valuable in the future if the infrastructure moves back to containers. With the principle of YAGNI in mind, I believe that the likelihood of us actually needing this code again and it being valuable is less than the value we get from removing it from view for the foreseeable future so we can concentrate on building and deploying without containers.

Pause this until we have decided to either go all in on Docker for development in order to reduce set up costs, or to use something like Make to automate the set up without Docker.


big integer library for c++

This Pull Request resolves Issue #
Description:
In this Pull Request we have done the following additions/edits for the code:

Are there any updates here or should we close the PR and label it as spam?

I will make a new pull request later today. Currently I don't have the required file. So I have closed this pull request as of now.

Alright not a problem!


Gesture detection conflicts with "Zoom Page WE" extension shortcut causing mouse wheel lock to page zooming after gesture

"Zoom Page WE" extension has the following option enabled by default:

Enable mouse Right+Wheel to zoom in/out

Due to the way this extension works, it suppresses some mouse events after the right mouse key has triggered a gesture. This causes the shortcut not work properly and to 'lock' to zooming in and out after a gesture was recognized, until the mouse right button is clicked again.
Users of both extensions: It is recommended to disable the above shortcut in "Zoom Page WE"'s options interface.

This has been bugging me for so long. Finally I tracked it down to the Zoom Page WE extension and found your solution. Thank you!
For anyone interested, this bug triggers for me when I want to open a link in a new tab, and I do that by holding down right-click on the link, then releasing the click on "Open Link in New Tab". After that, mouse scrolling actually zooms, until I right click again, or switch tab and back, to reset it.
That is completely unrelated to NanoGestures of course. But it's the same bug described in this issue, so very useful to find!


Matching the fonts to our brand (marketing website)

Done in himaxwell/himaxwell.github.io#6 and himaxwell/himaxwell.github.io#7


Can't open pages for "Create", "Account" and "View Requests" options in new tabs (bug) -

Testing Environment:
Windows 10, Google Chrome 73.0.3683.86 (64bit), 15.6", 1920х1080
Steps to reproduce:

Navigate to http://rfai.singularitynet.io.s3-website-us-east-1.amazonaws.com/
Try to open "Create" page in new tab (using "Create" button)
Try to open "Account" page in new tab (using "Account" button)
Try to open "View Requests" page in new tab (using "View Requests" button)
Observe the result

EXPECTED RESULT:
We can see three tabs

The first one with opened "Create Request" page (with steps "Overview", "Evaluation", "Process", "Submit Request")
The second one with opened "Account" page (with "Your Account Details" and "Manage Your RFAI Account")
The third one with visible "Requests for AI Services" list

ACTUAL RESULT:

The first one with Home Page
The second one with Home Page
The third one with Home Page. We can actually see "Requests for AI Services" list here but we can always see it on the Home Page so I suppose that proper opening in the new tabs isn't implemented for all these buttons ("Create", "Account" and "View Requests")

Please see the video below

That is as per the expected behaviour. Those needs to be opened in the current browser instance. If this functionality is needed we can enable the same and I dont this it is required for now.

Well, on the one hand I am not sure that it's critical to provide such functionality asap indeed but on the other hand it doesn't really look as nice user experience that we show user the same Home Page in case of trying to open specific options (Create, Account etc) in the other tab. So I still think that it's better to make it possible to open "Create", "Account" and "View Requests" in the other tab.

Added this functionality by adding Routes in the ReactJS SPA App.
Issue is fixed.

You are welcome. Fix is verified, thanks.


python3Packages.folium: fix build and tests

python3Packages.folium: fix build and tests
python3Packages.folium: make geospatial team maintainer

Description of changes
This fixes the build and deactivate some failing tests due to write permission to a cache folder (no way to customize this because of the way geodatasets uses pooch).
This also makes geospatial team maintainer.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@imincik what do you think about adopting the package in the geospatial team?

Result of nixpkgs-review pr 328500 run on x86_64-linux 1

8 packages built:

python311Packages.folium
python311Packages.folium.dist
python311Packages.osmnx
python311Packages.osmnx.dist
python312Packages.folium
python312Packages.folium.dist
python312Packages.osmnx
python312Packages.osmnx.dist


Error while installing the plug in on Windows

Hi All,
We are getting the log file watcher plugin error.
After installing the plugin.while running the service we are getting the below error.
D:\NewrelicLogwatcher>ruby newrelic_logwatcher_agent.rb
Agent Logwatcher is at version 0.0.2
Error occurred in poll cycle: No such file or directory - test -e D:/NewrelicLogwatcher/LogFileTest.log
Gathered 0 statistics
Error occurred in poll cycle: No such file or directory - test -e D:/NewrelicLogwatcher/LogFileTest.log
Gathered 0 statistics
Error occurred in poll cycle: No such file or directory - test -e D:/NewrelicLogwatcher/LogFileTest.log
Gathered 0 statistics
Error occurred in poll cycle: No such file or directory - test -e D:/NewrelicLogwatcher/LogFileTest.log
Gathered 0 statistics
Error occurred in poll cycle: No such file or directory - test -e D:/NewrelicLogwatcher/LogFileTest.log
Gathered 0 statistics
Error occurred in poll cycle: No such file or directory - test -e D:/NewrelicLogwatcher/LogFileTest.log
Gathered 0 statistics
Error occurred in poll cycle: No such file or directory - test -e D:/NewrelicLogwatcher/LogFileTest.log
Gathered 0 statistics

Hello Team,
Can you please help on the above error, we are seeing while starting the watcher on windows machine?  Any one else have ever faced the error and?
Thanks and Regards,
Mounika

Can someone suggest on the above mentioned issue.
Regards,
Mounika

Hi @MounikaPearl this plugin is intended to work on *nix operating systems where test is a standard part of the system. You could try to set up on your Windows machine and make sure that it appears in the PATH, so it's resolvable as test from command line.

Hi @scream3 we have put the log file path (which we are trying to monitor) in the PATH variable.
We have also defined a variable "test" and its value is the log file path.
But we are still getting the same error.

@MounikaPearl test is a name of command line program (https://en.wikipedia.org/wiki/Test_(Unix)) . It should NOT be defined as variable. This should be present as executable file on target computer and directory with it should appear in PATH.


feat(media): added svg support to glide

Description
Currently, when using Glide as Media Library Image Service, I am unable to display uploaded SVGs, since Glide does not support SVGs.
There is a workaround to this problem, you can set the environment variable GLIDE_DRIVER to imagick. This “solves” the problem, meaning the SVG will be converted into a JPEG and thus loosing image quality and transparency.
The workaround, however, is not perfect, and I'd prefer getting my unmodified SVG back instead. This is what this PR solves.

This may result in a breaking change, and it might make more sense to add an option in the config that allows the original SVG to be returned, such as twill.glide.use_original_svg with its default value being set to false.
Open to ideas and feedback

Hi @Keimeno,
That's cool! I've also used a (hacky) homegrown helper to get the actual raw image when working with GIFs :
function safe_image_array($object, $role, $crop, $params=[], $media=null)
{
    if (! $media) {
        $media = $object->imageObject($role, $crop);
    }

    if (! $media) {
       return [];
    }

    $data = $object->imageAsArray($role, $crop, $params, $media);

    if (preg_match('/.gif$/', $media->filename)) {
        $data['src'] = '/storage/uploads/' . $media->uuid;
    }

    return $data;
}

I see the value of having a more robust solution in Twill core. Also very curious to know what others have been doing for this :)

To make things more robust, I've added support for Azure & S3 as well.
There are still some open questions, though.

Should the helper function getMediaUrl be made available for all Media Libraries? This would mean, that we make the function protected and move it into the ImageServiceDefaults.php file.
Is it enough if we offer this functionality for getRawUrl, or should this be added to other functions as well (getUrl, etc.)?
And since in your use case @pboi20 you need to return the original .gif file, it probably makes most sense if we add a configuration option, that allows us to specify an array of file endings. If a file ending matches, then we would return the original file.

@Keimeno Thanks for taking the time to refine this!

As far as I can tell, the fix is not needed for the Local service. I haven't used Imgix, so I'm not sure about the behavior of getRawUrl in this context. I imagine that it's comparable to Glide and could benefit from this fix.

I don't think this should extend to getUrl.

A config option to toggle this behavior by file extension makes sense. Defaulting to an empty list would also avoid a breaking change, as you mentionned in your PR description.

Do you think that this feature could live in it's own helper method? Something in the spirit of getRealRawUrl. Perhaps getOriginalFileUrl?

I'm unsure if getOriginalFileUrl should be the way to go. Wouldn't this mean that the method always returns the original URL, so the user has to check for the file extension himself? Also, we would have some inconsistencies in the public API between each media library service, so changing media drivers could become problematic, as long as we don't add this method to the ImageServiceDefaults trait.
Besides, after further consideration regarding getRealUrl, I think we should extend it to getUrl, since currently, when trying to access an SVG with the API described above we'd have to write something like the following (inside a blade file):
\A17\Twill\Services\MediaLibrary\ImageService::getRawUrl($block->imageObject('role')->uuid)

And since the HasMedias trait uses methods, which internally call the getUrl method, the following syntax would work as well, if we were to extend the getUrl method as well.
$block->image('role')

Hey @Keimeno, thanks for the feedback! I see that getOriginalFileUrl may not be as desirable as I initially thought. Let's scratch that one :)
From my perspective (ie. my use case), I think the main issue remains that getRawUrl could behave better for certain file extensions. Your solution adresses this very cleanly.
I'm still unsure about changing the behavior of getUrl. Would there be any side effects in the CMS (Media Library or medias field)?

Thanks a million for the input @pboi20!
I don't think that it would cause any side effects for getUrl, since it only adds parameters to the URL, and these parameters would be ignored if specifying the original media URL.
Also, I kept the name getOriginalMediaUrl, think its the better naming choice than getMediaUrl

Thanks, @ifox, for the feedback on the PR, the requested changes have been made. Also found a small bug in my code, should have returned null instead of false in the getOriginalMediaUrl method.


possibility of including commit count in .gituprevision?

Per a recent discussion about SA text on the freebsd-stable mailing list we have a desire to include commit counts to easily compare uname against a bugfix commit, to know if it is included or not.
newvers.sh generates it like so:
         if [ "$($git_cmd rev-parse --is-shallow-repository)" = false ] ; then
                git_cnt=$($git_cmd rev-list --first-parent --count
HEAD 2>/dev/null)
                if [ -n "$git_cnt" ] ; then
                        git="n${git_cnt}-${git}"
                fi
        fi

i.e., assuming not a shallow clone, the number of commits following only the first parent for merges. How feasible is it to calculate this and add to .gituprevision?

I remember from a previous ticket that John said it is not possible because gitup only memorizes only one revision, thus a shallow clone. A remote count is not possible, IIRC. /var/db/gitup/... stores only one commit hash per object/file.

I am tantalizingly close on this one -- I believe I've figured out the cryptic command syntax to get a full clone of just commit objects and it works for smaller repos (like freebsd-ci and pkg) but fails on the larger ones like freebsd-src (it's including a ref-delta object that points to an object that neither exist in the full commit pack file nor the shallow file data pack).


As a developer, I want to do synchronisation

There should be a convenient way to do synchronisation of a file.
How to manage the sync state?
Is synchronisation out of the scope of the SDK?

What do you mean by synchronization? Syncing only changes between a local file and a remote file?

Yes, syncing between a local and remote file.
When I put a file into gaia storage I want to know whether it has changed from a different device. And I have to fetch it first and merge the local changes before putting the file.
For Android's content provider and PWA's local storage this is a common use case.

The proposal API in https://github.com/blockstack/blockstack.js/issues/702 will help here.


Import Issues from Jira

Feature Description
It should be possible to migrate issues from jira to gitea.
Currently its possible to import issues from Jira into gitlab and migrate it afterwards into gitea. As of now this unfortunatley does neither migrate comments or attachments into gitlab, thus they cannot be migrated to gitea in turn.
Gitlab mitigates this a bit by linking the original jira issue. This is useful in cases where you have the jira instance left on the network for a while at least.
Thus I propose two things:

Migrate links from gitlab issue titles into gitlab as well
add jira issue import directly into gitea

Screenshots
No response

I think Jira could have so many different customizations that it would be not feasible to implement that into gitea, that should probably be done as external tool to import using gitea api

Please find attached SQL queries to directly migrate issues from Jira to Gitea, including comments, attachments and string based custom fields.
Warning: It deletes all issues, comments and attachments from Gitea. Please read all comments carefully.
migrate_from_jira_to_gitea.txt

Here is simple and very dirty tool which I used to import jira issues into gitea's SQL Server database. Should be easy to port other DB servers though.

Using @achikhv tool, I modified it to work with MySQL, I will be publishing it as one of my repositories in the near future. There also is some errors in achikhvs tool I fixed.

Once it is published, please add your repo to https://gitea.com/gitea/awesome-gitea#migration.

Using @achikhv tool, I modified it to work with MySQL, I will be publishing it as one of my repositories in the near future. There also is some errors in achikhvs tool I fixed.

Would love to see PR's with fixes. Could be useful to others.

Here you have the code, it works fine and documentation is better, hope it helps! https://github.com/juangarcia06/jira2giteaMySql
Was quite a lot of work. I didn't get attachments working. If anyone can get them to work and can make a pull request I would appreciate it a lot since this took me about 2 weeks of work.

Trying out @juangarcia06's importer, but I'm getting 401's (bad request) when it tries to fetch the data from Jira. I'm certain I have the correct login/password in launchSettings.json. However, I am trying to import from a local Jira instance (JiraServer url is http://<IP_ADDRESS>:9876). Has anyone else gotten this to work? Any ideas?

Also for anyone interested: the script can be tweaked to work with sqlite fairly easily:

Add reference to Microsoft.Data.Sqlite
In Program.cs, create the connection like var sqlc = new SqliteConnection("Data Source=path_to_your_db.db");
In importer.cs, replace MySqlConnection with SqliteConnection
And the ON DUPLICATE KEY UPDATE query with insert or replace into issue_index (group_id,max_index) values ({_repoId},{maxIndex})
And LAST_INSERT_ID() with last_insert_rowid()
And insertCmd.LastInsertedId with extension method from here

Oh and I just realized: this is importing comments, but not comment counts. i.e. although the comments are actually there, all the issues show a count of 0 comments.


Tagging spot instances with launch templates

I'm submitting a...

[x] feature request
[x] kudos, thank you, warm fuzzy

What is the current behavior?
Currently there is a way to pass tags to launch templates, but it defines those only for volumes.
My use case is deny IAM policy, which doesn't allow to call ec2:RunInstances without specifying certain tags for instance. I know there is way for ASG to pass tags to instance, but AWS Documentation states that:

In most cases, when an Auto Scaling group launches instances, it adds tags to the instances during resource creation rather than after the resource is created.

The exception is when you use a launch configuration to launch Spot Instances. For this scenario, your Auto Scaling group adds tags while the instances are in the Pending lifecycle state

if you need the Auto Scaling group to add tags to instances as part of the same API call that launches the Spot Instances, consider migrating to launch templates.

But currently there is no way to pass tags for instances down to launch templates.
In my case debugging of that took a lot of time because of terraform showing this error:
Error: Error creating AutoScaling Group: AccessDenied: You are not authorized to use launch template: lt-009a3ea50f5e2c231
    status code: 403, request id: 2f09ef93-17a6-4b41-950d-03b3f194411d
      on .terraform/modules/eks/terraform-aws-modules-terraform-aws-eks-dd06c59/workers_launch_template.tf line 3, in resource "aws_autoscaling_group" "workers_launch_template":
       3: resource "aws_autoscaling_group" "workers_launch_template" {

So error was unclear, but cause of error was that missing instance tag in launch template.
What's the expected behavior?
One could specify instance tags via launch template

Thanks!

This is now implemented in https://github.com/terraform-aws-modules/terraform-aws-eks/releases/tag/v11.0.0.
See #762 #822 for more info.


fixed some errors encountered while testing with Lua

multiboars in lua do not accept null values, so added a workaround
lua checks member access, fixed a related NPE in ClosureTimers

I hope the closureEvents change doesn't break anything :D
Thanks 👍


Add support for Google Cloud Composer / KubernetesPodOperator

It would be helpful to be able to run cosmos in composer.
There doesn't seem to be a way to do that now, because creating a virtualenv in composer is complex and/or impossible, and even if one gets created, installing the basic package conflicts with the packages that come with composer: https://cloud.google.com/composer/docs/concepts/versioning/composer-versions#images

results from adding the dbt.bigquery package:
+ python3 -m pip check
google-cloud-bigquery 3.5.0 has requirement protobuf!=3.20.0,!=3.20.1,!=4.21.0,!=4.21.1,!=4.21.2,!=4.21.3,!=4.21.4,!=4.21.5,<5.0.0dev,>=3.19.5, but you have protobuf 3.20.0.
google-cloud-aiplatform 1.16.1 has requirement google-cloud-bigquery<3.0.0dev,>=1.15.0, but you have google-cloud-bigquery 3.5.0.
dbt-bigquery 1.4.0 has requirement dbt-core~=1.4.0, but you have dbt-core 1.3.1.
The command '/bin/sh -c bash installer.sh $COMPOSER_PYTHON_VERSION  fail' returned a non-zero code: 1
ERROR
ERROR: build step 0 "gcr.io/cloud-builders/docker" failed: step exited with non-zero status: 1

To separate environments, KubernetesPodOperator seems to be the most natural fit.  It could take a really simple python image.  I'm not sure how feasible this is, but currently the virtualenv creation in managed airflow seems like a no-go.

I also think that this feature is necessary. I tried to make a PR to open the discussion. The PR is not perfect at all but atleast we can discuss around it.

PR is merged, so closing this!


TypeScript implement custom resolver

To ensure that bundle dependencies are normalized correctly, it is necessary to have a custom resolver to pass into TypeScript.
@vladima you mentioned previously this was the plan for custom resolutions? Please keep me posted on how that goes.

@guybedford sure. Currently we plan to expose custom resolution via CompilerHost.resolveModuleName so host can implement any resolution policy it needs. Can you please tell if it will cover your scenarios?

@vladima thanks yes that looks exactly right. In SystemJS we can always relate every module to an absolute URL or filepath, but it may be worth considering relaxing this constraint and allowing virtual naming as well. For example, we use the @empty module in SystemJS as an output for this for nulled out modules (an edge case certainly, but to illustrate).

@guybedford  this PR on the TypeScript side should allow to pass map (originalName -> newName, effectively load.depMap) to transpiler. Can you please check if it will cover your scenarios?

It works excellently, thanks @vladima for your work on this.

Released in 0.13.2.


Disable xdebug for composer install to speedup build

We dont use xdebug features in travis, therefore disabling it which speeds up build considerably.

Will sqash the commits when back on a real wokrstation

Hmm this offical way to disable the extension has problems with hhvm..
Need to adjust this ( and all the other PRs)

ready to land