x-select.js:151 Uncaught TypeError: Cannot read property 'is' of undefined

Version of x-select: 2.2.2
Ember Version / Ember CLI Version:
Ember Version: 2.7.2
Ember CLI Version:
Jquery version: 2.2.4
Expected Behavior
In my template i have:
{{#x-select value=(mut (get modelRelationsValues key)) class="chosen-select" as |xs|}}
    {{#xs.option value=''}}{{/xs.option}}
    {{#each choices as |option|}}
        {{#xs.option value=(get (json2obj option.value) 'id')}} {{option.display_name}} {{/xs.option}}
    {{/each}}
{{/x-select}}

Actual Behavior
On select any item throws this error (Cannot read property 'is' of undefined)
Ember Twiddle / Example repo / Failing test:
Please create a reproduction of your bug with this ember-twiddle. This is extremely helpful and speeds up the debugging process.

Hi @arthur-s! Is there anyway you can make an ember twiddle for this? I tried plugging in what you have in your template and I can't reproduce. I'm going to guess something is going on with the value of json2obj being null or undefined

might be related https://github.com/thefrontside/emberx-select/pull/148

@Robdel12, I cannot recreate my situation on Twiddle, becouse when i select latest Ember.js or Ember Data, in the debug console i always see
DEBUG: Ember      : 2.5.1
VM135 ember.debug.js:6440 DEBUG: Ember Data : 2.5.2
VM135 ember.debug.js:6440 DEBUG: jQuery     : 1.11.3

Not the latest version.

Now i tried to use x-select (installed from git repository, dev version), and this error occured when I navigated to page, containing select box:
x-option.js:77 Uncaught TypeError: _this.get(...) is not a function

@arthur-s current master is basically x-select 3.0, so there's a lot of breaking changes there. Could you push an example repo and I can take a look at that?

I've received the same error, I'm trying to reproduce it but unsuccessfully until now.
My template is
{{#x-select value=model.country as |xs|}}
  {{#each countryList as |country|}}
    {{#xs.option value=country.id}}
      {{ country.country }}
    {{/xs.option}}
  {{/each}}
{{/x-select}}

I'm pretty sure that model.country was null, countryList is an array of ember objects containing ~250 elements like this:
{ id: "US", country: "USA" }

I've received ~250 error notifications, one for each option in the select but only for one user. Other users were able to use the select without problems.
Currently I'm using:
emberx-select: 2.2.2
ember-data: ^2.8.0
ember.js: ~2.8.0

I think this has been fixed in v2.2.3. I'll reopen if that's not the case.

I am running "emberx-select": "^3.0.0", but still have this error.  If you update your HBS to this does it still throw the same error?
{{#x-select action='selectAppType' id="app-type-select" value=model.AppType class="form-control" as |xs|}}
  {{#each model.AppTypes as |appType|}} {{log appType}}
    {{#xs.option value=appType}}
      {{appType}}
    {{/xs.option}}
  {{/each}}
{{/x-select}}

Thanks @Robdel12 , this seems to solve the issue but can you please explain what |xs| does or point me documentation for the same? It seems to have magically fixed the issue :-).

Absolutely! Think of it as a component "wrapping" a component. x-option will always be used as a child of x-select. Since that will always be true we can do things like automatically pass the instance of x-select down to x-option. We also register x-option with x-select.
Here's a couple links:

https://emberjs.com/blog/2015/11/16/ember-2-2-released.html#toc_contextual-components
https://guides.emberjs.com/v2.5.0/components/wrapping-content-in-a-component/


Add BNI VA on PaymentType

Hi, @rizdaprasetya and Midtrans team!
Based on Snap API official documents, I proposed to add bni_va as a PaymentType. Please take a look 🙏
Thanks,
Harits Fahreza

Thanks for submitting PR, will review on monday! :)

Looks good 👍
Thanks for another contribution :)


consider obsoleting 'disease causes feature' property

we have ~30 usages of the 'disease causes feature' but it is so infrequently that we wonder if this should still be used or obsoleted?
cc @matentzn

Query: https://api.triplydb.com/s/qB0mhat1q
I think we should remove this relation, or otherwise document its use case and complete it.


fix/test(CL Swaps): in given out

Closes: #4423
What is the purpose of the change
This PR is a follow-up to: https://github.com/osmosis-labs/osmosis/pull/4500
It fixes two problems with swap in given out:

Determining whether a swap is zero for one or one for zero

Using the correct swap step implementation that correctly computes next square root price by swapping the amount out in the appropriate direction

Note that the inverse relationship test will be updated later in: https://github.com/osmosis-labs/osmosis/issues/4475
Brief Changelog

switch swap in given out to use the correct compute swap step logic from #4500
remove legacy ComputeSwapStepLegacy
update logic for detecting "zeroForOne" vs "oneForZero" swaps
update all test cases

Testing and Verifying
The tests are updated to reflect the new changes.
For each relevant test updated, there is a short Python script in comments that can be used to validate the expected numbers
Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? no
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? no
How is the feature or change documented? not applicable

Closed and reopened to re-run CI

Thanks for the reviews, everyone. As there are no more action items in this PR, I'm planning on merging it once CI passes


the "play bloxstrap" opens up the browser of roblox discovery not the roblox desktop app.

so heres my problem. i think i screwed up something into roblox and now the play bloxstrap opens up the browser not the desktop app. i tried all of my ideas and nothing worked. bloxstrap appear for some seconds and dissapears after the loading is done.

oh wait. nevermind. problem fixed my itself. uhh.. akward.


GitHub Codespaces?

I'm been trying out codespaces to see what the experience is like and now I'm wondering if it would be worth it to setup a "Open in codespaces" button in our repos (at least in the main one, website, and sscce)
If we can configure it to be usable out of the box, i.e. that the dev wouldn't need to do any further configuration work, it could reduce the barrier for contributing to the project

I've been using GitHub Codespaces exclusively for the past year (or 2 maybe even). It's great for not having to worry about cleaning up databases or whatever.
I have my VSCode settings synced so it automatically downloads certain extensions as well.
Adding a 'default' devcontainers config with the eslint and prettier extensions might not be a bad idea. And maybe SQL tools


Blank line after/before license

It would be nice to have an option to add a blank line before the license block or / and after the license block.

Yes, and please make this optional.
For example, I would want something like this on a JSP file:
<%--

    Licensed to Apereo under one or more contributor license
    agreements. See the NOTICE file distributed with this work
    for additional information regarding copyright ownership.
    Apereo licenses this file to you under the Apache License,
    Version 2.0 (the "License"); you may not use this file
    except in compliance with the License.  You may obtain a
    copy of the License at the following location:

      http://www.apache.org/licenses/LICENSE-2.0

    Unless required by applicable law or agreed to in writing,
    software distributed under the License is distributed on an
    "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
    KIND, either express or implied.  See the License for the
    specific language governing permissions and limitations
    under the License.

--%><%= "xyz" %>

Rather than:
<%--

    Licensed to Apereo under one or more contributor license
    agreements. See the NOTICE file distributed with this work
    for additional information regarding copyright ownership.
    Apereo licenses this file to you under the Apache License,
    Version 2.0 (the "License"); you may not use this file
    except in compliance with the License.  You may obtain a
    copy of the License at the following location:

      http://www.apache.org/licenses/LICENSE-2.0

    Unless required by applicable law or agreed to in writing,
    software distributed under the License is distributed on an
    "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
    KIND, either express or implied.  See the License for the
    specific language governing permissions and limitations
    under the License.

--%>
<%= "xyz" %>

Additionally, it would be nice to say: one blank line, but none above if the licence block is the first thing in the file, and never more than one blank line (i.e. don’t add one if one exists, or even collapse them all into one empty line).

@bot see https://github.com/mathieucarbou/license-maven-plugin/issues/78#issuecomment-240091273

@rnc @mmoayyed @mirabilos : Stable Bot automatically mark and closes stale issues. If nobody resurrects or open a PR for that issue, it will just be closed. I understand the feature (nice to have IMO), but as an OSS projects and not having any bandwidth for such nice to have thing, people in need also are welcomed to contribute.
Thanks :-)

@mathieucarbou I perfectly, understand. From my perspective, you're welcome to close this issue. We are no longer using this particular feature or plugin, and have moved on for about 6 years.

stale[bot] dixit:

Is this still relevant? If so, what is blocking it? Is there anything

Probably, but I’m not using the plugin anywhere right now (working
on C projects), so I don’t know for sure.
bye,
//mirabilos
Infrastrukturexperte • tarent solutions GmbH
Am Dickobskreuz 10, D-53121 Bonn • http://www.tarent.de/
Telephon +49 228 54881-393 • Fax: +49 228 54881-235
HRB AG Bonn 5168 • USt-ID (VAT): DE122264941
Geschäftsführer: Dr. Stefan Barth, Kai Ebenrett, Boris Esser, Alexander Steeg

stale[bot] dixit:

Is this still relevant? If so, what is blocking it? Is there anything
you can do to help move it forward? This issue has been automatically
marked as stale because it has not had recent activity. It will be
closed if no further activity occurs.

🤡

Yay for bots…

stale[bot] dixit:

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward? This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

yes, as always

stale[bot] dixit:

Is this still relevant? If so, what is blocking it?

-- Henry Nelson, March 1999

I hate the “stalebot” concept on principle (there recently was a good article about that on Fediverse, too).

Two sides to that.
First, I have this thread bookmarked as an example of what not to do with a stale bot. :-)  If a repo is not really actively maintained, the bot is noise.
Second, on the opposite end, if a repo is actively maintained, there are often bug report/issues where the maintainer asks a question and it just sits there... forever.  And having a bot close these old, useless issues is sometimes helpful.
Third, this is open source.  Maintainers aren't paid for their work, and often end up with competing priorities on their time. This is part of the design of open source.  If an issue is important to you, feel free to contribute a fix for it!

The project is actively maintained but new features are not actively added for the reasons mentioned above.
And I am baffled to see how many people would find something useful, but none is courageous enough to send a complete PR.
As such, the bot serves the right purpose here: if a feature is asked by 2-3 people but no one implement it, then it is not an important feature, or these people can work around it. Otherwise, if this is really a no-brainer feature, it would have been implemented because it would have bothered more people.
In the case of this issue, I think only a few people find this interesting, and amongst them, none dares contribute. That's why it is really low priority.

Daniel Widdis dixit:

Second, on the opposite end, if a repo is actively maintained, there
are often bug report/issues where the maintainer asks a question and it
just sits there... forever. And having a bot close these old, useless
issues is sometimes helpful.

Perhaps, but I’d rather a maintainer do that, not a bot, as they can
see when a moreinfo thing lacks responses.

Third, this is open source.  Maintainers aren't paid for their work

Of course this argument comes. Do you know how much Open Source I
maintain? sigh… and almost all of it in whatever spare time I can
manage.
bye,
//mirabilos
 you introduced a merge commit        │ % g rebase -i HEAD^^
 sorry, no idea and rebasing just fscked │ Segmentation
 should have cloned into a clean repo      │  fault (core dumped)
 if I rebase that now, it's really ugh     │mika:#grml wuahhhhhh

Mathieu Carbou dixit:

And I am baffled to see how many people would find something useful,
but none is courageous enough to send a complete PR.

Can you imagine people being experienced programmers but in
completely different areas and not having the headspace to
begin to learn developing a Maven plugin in Java (especially
when they’re not experienced in the latter)?
Give me an i386 bootloader problem in assembly rather…
and most of the things I do nowadays are in Korn Shell or C.
bye,
//mirabilos
„Cool, /usr/share/doc/mksh/examples/uhr.gz ist ja ein Grund,
mksh auf jedem System zu installieren.“
-- XTaran auf der OpenRheinRuhr, ganz begeistert
(EN: “[…]uhr.gz is a reason to install mksh on every system.”)

Can you imagine people being experienced programmers but in completely different areas and not having the headspace to begin to learn developing a Maven plugin in Java (especially when they’re not experienced in the latter)?

Totally :-) But my point still remains true ;-)
This is an OSS project. Either use it the way it works, or contribute to make it better, or do not use it at all. But don't expect maintainers or contributors to work on your needs.

still waiting in license-gradle-plugin 😆

We are no longer using this particular feature or plugin, and have moved on for about 6 years.
@mmoayyed Hello, which one do you use instead?

Not using maven anymore.

@mmoayyed Thanks. I was wondering how to resolve this issue while using license-gradle-plugin whose core logic is bassed on this plugin.
Do you use gradle? If so, it would be grateful to know which one you're using to avoid this issue :)

I think that I have worked around this, at least for my needs with Java. I have added a header definitions line:
<headerDefinitions>C:/<mydirectory>/headerDefs.xml</headerDefinitions>
I then overrode the SLASHSTAR_STYLE as follows in the header definitions file:
<additionalHeaders>
  <slashstar_style>
    <firstLine>/*</firstLine>
    <beforeEachLine>  </beforeEachLine>
    <endLine>*/EOL </endLine>
    <afterEachLine> </afterEachLine>
    <!--skipLine></skipLine-->
    <firstLineDetectionPattern>(\s|\t)*/\*.*$</firstLineDetectionPattern>
    <lastLineDetectionPattern>.*\*/(\s|\t)*$</lastLineDetectionPattern>
    <allowBlankLines>true</allowBlankLines>
    <multiLine>true</multiLine>
    <padLines>false</padLines>
  </slashstar_style>
</additionalHeaders>

The key point here is the addition of EOL to <endLine> to skip a line at the end of the header. I suspect that if you really want to have a blank line at the top, the same workaround would apply to <firstLine>.

@iansergeant I have applied that…
tglase@tglase:~/Projekte/mvnparent $ cat src/main/ancillary/myc-javapkg.xml
<?xml version="1.0" encoding="UTF-8"?>
<additionalHeaders>
        <JavaPackage_style>
                <firstLine><![CDATA[EOL/*-]]></firstLine>
                <beforeEachLine><![CDATA[ * ]]></beforeEachLine>
                <endLine><![CDATA[ */EOL]]></endLine>
                <skipLine><![CDATA[^package [a-z]+(\.[a-z][a-z0-9]*)*;$]]></skipLine>
                <firstLineDetectionPattern><![CDATA[(EOL)*(\s|\t)*/\*.*$]]></firstLineDetectionPattern>
                <lastLineDetectionPattern><![CDATA[.*\*/(\s|\t)*(EOL)*]]></lastLineDetectionPattern>
                <allowBlankLines>false</allowBlankLines>
                <isMultiline>true</isMultiline>
                <padLines>false</padLines>
        </JavaPackage_style>
</additionalHeaders>

… but it sometimes leads to an extra blank line when updating the licence headers.


Schedule list command: CallbackEvent with custom description is ignored.

Laravel Version: 9.8.1
PHP Version: 8.1

Description:
Before 0256484f71a9100405c2ae6f9bc9f850e6d640cf  commit we were able to see set description for CallbackEvent.
Now we are able too see only  Closure at: vendor/laravel/framework/src/Illuminate/Console/Scheduling/Schedule.php:144.
Would be great to leave the description / command if the description is set?
Thank you
Steps To Reproduce:

This is expected. You can show the description with the verbose flag.

thanks for the answer, I wonder if it should be the opposite but OK. have a great day


Release 2.0.23 offers yii-advanced-app-2.0.22.tgz (filename correct?)

Just a small observation:
Under https://github.com/yiisoft/yii2/releases I found the new release 2.0.23. But at least for me unexpectedly the according file yii-advanced-app-2.0.22.tgz has the previous version in its file name (while yii-basic-app-2.0.23.tgz the new current version).
Additionally the member file advanced/vendor/yiisoft/yii2/CHANGELOG.md of yii-advanced-app-2.0.22.tgz contains change informations about version 2.0.23

Thank you for noticing it. Fixed.


Fix Overlay of "JOIN THE PODCAST" Button

Description
This PR addresses an issue where the "JOIN THE PODCAST" button is overlapping the last header items, which impacts the user experience and visual layout of the header.
-fixes #75
Type of change :

bug fix

Screen Recording :
Screencast from 2024-10-18 11-29-26.webm
Screencast from 2024-10-18 11-28-55.webm

hey @Sanskriti1102 ,KIndly check my this PR and merge it.

@Sanskriti1102 , Please look into my PR.

@Sanskriti1102 , Please merge it.


Don't select on @bazel_tools//src/conditions. Only use @platforms//os.

Don't select on @bazel_tools//src/conditions. Only use @platforms//os.

Solved by https://github.com/Vertexwahn/rules_qt6/commit/7814568b95dc6c1be420a5a844843404c793e316


Microsoft.Graph.Applications Version Number

🐛 Summary
Required Versions is looking for up to 1.99 of Microsoft.Graph.Applications, but latest version is 2.2
To reproduce
Steps to reproduce the behavior:

Install-Module microsoft.graph.authentications
Run Scuba

Expected behavior
What did you expect to happen that didn't?
Failed to complete, module won't import
Any helpful log output or screenshots
Paste the results here:
Import-Module : No acceptable installed version found for module: Microsoft.Graph.Applications
        Required Min Version: 1.14.0 | Max Version: 1.99.99999
        Run Get-InstalledModule to see a list of currently installed modules
        Run SetUp.ps1 or Install-Module Microsoft.Graph.Applications -Force -MaximumVersion 1.99.99999 to install the
latest acceptable version of Microsoft.Graph.Applications

Add any screenshots of the problem here.

Thank you for the feedback.  We will look at updating the tool to support version 2 of this module in a future release.  We need to determine backwards compatibility first, but hopefully it will be a minor lift and we can get it into our next release along with the baseline updates.
You should be able to install a 1.x version of the module explicitly using Install-Module -MaximumVersion 1.99.9999

I took a look.  Obstacles to upgrading to version 2 of Graph Powershell:

Change to behavior of "beta" cmdlets (no more Get-MgProfile/Select-MgProfile).  Now the beta cmdlets are in a different set of "beta" packages.  Its easy to handle this by catching/ignoring the failure to select "beta" BUT
We appear to use "beta" cmdlets in at least one place (Get-MgDirectorySetting in AAD 2.7).  This is now Get-MgBetaDirectorySetting.  We will need to identify a way to handle this, either by setting minimum version of ScubaGear to Graph 2.x and changing how we call it or by handling both cases to preserve backwards compatibility.  This is likely a topic for once the baseline updates are finalized but before the code is updated.

This issue is also responsible for all the PowerShell and Smoke test failures.

Closed with #514


Earth Engine initialization error: authentication token not getting detected from Streamlit app's Secrets

First off, thank you so much for this excellent free and open resource! Your efforts here are much appreciated! I am currently looking to build a simple Earth Engine app and am using this repo to learn how to do so.
The issue
I am getting a "Earth Engine client library not initialized. Run ee.Initialize()" error on my web app.
Although I have generated and saved the Earth Engine authentication token in the app's "Secrets", as you have described in your blog post, the app seems to not pick it up from there.
How do I fix this?
What I have done
I began with trying to replicate this multi-app (https://streamlit.geemap.org/) on my Streamlit account, so I forked  this repo and published it as-is on my Streamlit.
Error as seen on the web app
But when I select an app working with Earth Engine (eg. Google Earth Engine (GEE), Awesome GEE Community Datasets), I get the following error.
ee.ee_exception.EEException: This app has encountered an error. The original error message is redacted to prevent data leaks. Full error details have been recorded in the logs (if you're on Streamlit Cloud, click on 'Manage app' in the lower right of your app).
Traceback:

File "/home/appuser/venv/lib/python3.7/site-packages/streamlit/scriptrunner/script_runner.py", line 475, in _run_script
    exec(code, module.__dict__)
File "/app/streamlit-geospatial/app.py", line 47, in <module>
    apps.run()
File "/app/streamlit-geospatial/multiapp.py", line 64, in run
    functions[titles.index(title)]()
File "/app/streamlit-geospatial/apps/gee.py", line 121, in app
    nlcd()
File "/app/streamlit-geospatial/apps/gee.py", line 37, in nlcd
    Map.addLayer(getNLCD(year), {}, "NLCD " + year)
File "/app/streamlit-geospatial/apps/gee.py", line 22, in getNLCD
    dataset = ee.ImageCollection("USGS/NLCD_RELEASES/2016_REL")
File "/home/appuser/venv/lib/python3.7/site-packages/ee/computedobject.py", line 32, in __call__
    return type.__call__(cls, *args, **kwargs)
File "/home/appuser/venv/lib/python3.7/site-packages/ee/imagecollection.py", line 38, in __init__
    self.initialize()
File "/home/appuser/venv/lib/python3.7/site-packages/ee/imagecollection.py", line 72, in initialize
    super(ImageCollection, cls).initialize()
File "/home/appuser/venv/lib/python3.7/site-packages/ee/collection.py", line 32, in initialize
    apifunction.ApiFunction.importApi(cls, 'Collection', 'Collection')
File "/home/appuser/venv/lib/python3.7/site-packages/ee/apifunction.py", line 183, in importApi
    cls.initialize()
File "/home/appuser/venv/lib/python3.7/site-packages/ee/apifunction.py", line 154, in initialize
    signatures = data.getAlgorithms()
File "/home/appuser/venv/lib/python3.7/site-packages/ee/data.py", line 1055, in getAlgorithms
    call = _get_cloud_api_resource().projects().algorithms().list(
File "/home/appuser/venv/lib/python3.7/site-packages/ee/data.py", line 285, in _get_cloud_api_resource
    'Earth Engine client library not initialized. Run `ee.Initialize()`')

Error as seen in "Manage App" log, from inside my Streamlit account
The most recent log in "Manage App" ends with the following—I'm guessing the app is not finding the authentication token eventhough I have generated and save it in the app's "Secrets".
(I have purposely truncated the authentication web link below.)
Installing collected packages: click
  Attempting uninstall: click
    Found existing installation: click 8.0.4
    Uninstalling click-8.0.4:
      Successfully uninstalled click-8.0.4
Successfully installed click-8.0.0
WARNING: You are using pip version 22.0.3; however, version 22.1 is available.
You should consider upgrading via the '/home/appuser/venv/bin/python -m pip install --upgrade pip' command.
[manager] Python dependencies were installed from /app/streamlit-geospatial/requirements.txt using pip.
Check if streamlit is installed
Streamlit is already installed
[manager] Processed dependencies!
To authorize access needed by Earth Engine, open the following URL in a web browser and follow the instructions. If the web browser does not start automatically, please manually browse the URL below.
    https://code.earthengine.google.com/client-auth?scopes=https%3A//www.googleapis.com/auth/earthengine%20https%3A//www.googleapis.com/auth/ ...
The authorization workflow will generate a code, which you should paste in the box below.
2022-05-19 10:54:04.502 Uncaught app exception
Traceback (most recent call last):
  File "/home/appuser/venv/lib/python3.7/site-packages/streamlit/scriptrunner/script_runner.py", line 475, in _run_script
    exec(code, module.__dict__)
  File "/app/streamlit-geospatial/app.py", line 47, in <module>
    apps.run()
  File "/app/streamlit-geospatial/multiapp.py", line 64, in run
    functions[titles.index(title)]()
  File "/app/streamlit-geospatial/apps/gee_datasets.py", line 182, in app
    eval(datasets[category][dataset])
  File "<string>", line 1, in <module>
  File "/app/streamlit-geospatial/apps/gee_datasets.py", line 75, in global_mangrove_watch
    gmw2007 = ee.FeatureCollection("projects/sat-io/open-datasets/GMW/GMW_2007_v2")
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/computedobject.py", line 32, in __call__
    return type.__call__(cls, *args, **kwargs)
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/featurecollection.py", line 44, in __init__
    self.initialize()
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/featurecollection.py", line 91, in initialize
    super(FeatureCollection, cls).initialize()
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/collection.py", line 32, in initialize
    apifunction.ApiFunction.importApi(cls, 'Collection', 'Collection')
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/apifunction.py", line 183, in importApi
    cls.initialize()
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/apifunction.py", line 154, in initialize
    signatures = data.getAlgorithms()
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/data.py", line 1055, in getAlgorithms
    call = _get_cloud_api_resource().projects().algorithms().list(
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/data.py", line 285, in _get_cloud_api_resource
    'Earth Engine client library not initialized. Run `ee.Initialize()`')
ee.ee_exception.EEException: Earth Engine client library not initialized. Run `ee.Initialize()`
2022-05-19 10:54:05.079 Uncaught app exception
Traceback (most recent call last):
  File "/home/appuser/venv/lib/python3.7/site-packages/streamlit/scriptrunner/script_runner.py", line 475, in _run_script
    exec(code, module.__dict__)
  File "/app/streamlit-geospatial/app.py", line 47, in <module>
    apps.run()
  File "/app/streamlit-geospatial/multiapp.py", line 64, in run
    functions[titles.index(title)]()
  File "/app/streamlit-geospatial/apps/gee_datasets.py", line 182, in app
    eval(datasets[category][dataset])
  File "<string>", line 1, in <module>
  File "/app/streamlit-geospatial/apps/gee_datasets.py", line 75, in global_mangrove_watch
    gmw2007 = ee.FeatureCollection("projects/sat-io/open-datasets/GMW/GMW_2007_v2")
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/computedobject.py", line 32, in __call__
    return type.__call__(cls, *args, **kwargs)
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/featurecollection.py", line 44, in __init__
    self.initialize()
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/featurecollection.py", line 91, in initialize
    super(FeatureCollection, cls).initialize()
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/collection.py", line 32, in initialize
    apifunction.ApiFunction.importApi(cls, 'Collection', 'Collection')
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/apifunction.py", line 183, in importApi
    cls.initialize()
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/apifunction.py", line 154, in initialize
    signatures = data.getAlgorithms()
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/data.py", line 1055, in getAlgorithms
    call = _get_cloud_api_resource().projects().algorithms().list(
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/data.py", line 285, in _get_cloud_api_resource
    'Earth Engine client library not initialized. Run `ee.Initialize()`')
ee.ee_exception.EEException: Earth Engine client library not initialized. Run `ee.Initialize()`

Were you able to test the app locally?

I have not tried that, I went straight from forking to publishing on Streamlit. I'm new to this, I didn't think that step was necessary.
Will now try that step and report back.

If you did not it try it locally, how did you get the Earth Engine token? Make sure the app works fine locally before deploying it to the cloud.

I tried it locally, and it worked (screenshot of the local URL on a browser tab below).
[I think I got the app to work on the cloud—by using the token stored on my local machine instead of the one generated by following the link the build log on Streamlit takes me to. But, strangely, "client not initialized" error in the build log persists. More details below.]

Before trying it locally, I got the Earth Engine token by following the authorization link that came up in the "Manage app" build log for the app in my Streamlit. This is in the log I pasted in my message above, see screenshot below where I have highlighted it.
It did not give me the box to paste the token into, like it said it would. I went ahead and saved the token into the app's "Secrets". I thought it'll get picked up on rebooting the app, but that never happened, I get shown a (new) authorization link each time in the build log and I get the same errors.

On seeing that the app runs fine locally, I also took the token in my local machine (in my .config/earthengine/credentials) and saved that into my "Secrets" on Streamlit (they were not the same). With this, the app has started working.
But, strangely, the build log in "Manage app" on Streamlit hasn't stopped saying "Earth Engine client library not initialized. Run ee.Initialize()"
With the app now started to work, perhaps I can ignore this message and move on—or would that be inadvisible, any suggestions?
The last few lines of the log copied over, are as follows:
WARNING: You are using pip version 22.0.3; however, version 22.1 is available.
You should consider upgrading via the '/home/appuser/venv/bin/python -m pip install --upgrade pip' command.
[manager] Python dependencies were installed from /app/streamlit-geospatial/requirements.txt using pip.
Check if streamlit is installed
Streamlit is already installed
[manager] Processed dependencies!
2022-05-20 08:40:56.344 Uncaught app exception
Traceback (most recent call last):
  File "/home/appuser/venv/lib/python3.7/site-packages/streamlit/scriptrunner/script_runner.py", line 475, in _run_script
    exec(code, module.__dict__)
  File "/app/streamlit-geospatial/app.py", line 47, in <module>
    apps.run()
  File "/app/streamlit-geospatial/multiapp.py", line 64, in run
    functions[titles.index(title)]()
  File "/app/streamlit-geospatial/apps/gee_datasets.py", line 182, in app
    eval(datasets[category][dataset])
  File "<string>", line 1, in <module>
  File "/app/streamlit-geospatial/apps/gee_datasets.py", line 75, in global_mangrove_watch
    gmw2007 = ee.FeatureCollection("projects/sat-io/open-datasets/GMW/GMW_2007_v2")
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/computedobject.py", line 32, in __call__
    return type.__call__(cls, *args, **kwargs)
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/featurecollection.py", line 44, in __init__
    self.initialize()
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/featurecollection.py", line 91, in initialize
    super(FeatureCollection, cls).initialize()
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/collection.py", line 32, in initialize
    apifunction.ApiFunction.importApi(cls, 'Collection', 'Collection')
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/apifunction.py", line 183, in importApi
    cls.initialize()
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/apifunction.py", line 154, in initialize
    signatures = data.getAlgorithms()
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/data.py", line 1055, in getAlgorithms
    call = _get_cloud_api_resource().projects().algorithms().list(
  File "/home/appuser/venv/lib/python3.7/site-packages/ee/data.py", line 285, in _get_cloud_api_resource
    'Earth Engine client library not initialized. Run `ee.Initialize()`')
ee.ee_exception.EEException: Earth Engine client library not initialized. Run `ee.Initialize()`

Yes, you need to use the token from .config/earthengine/credentials. Do NOT use the token you copy from the browser. It will not work.
Also, use geemap.ee_initialize() or geemap.Map() to initialize Earth Engine. Never use ee.Initialize() with a streamlit web app. It can't detect the secret and the initialization will fail.
As long as the web app is working properly, you can ignore the warning message.

After a gap of ~18 months, I sync'd with the source repo's master branch to pick up updates/changes. I am now seeing a  ee.ee_exception.EEException: This app has encountered an error. error. Before this sync, I was getting a ERROR: THESE PACKAGES DO NOT MATCH THE HASHES FROM THE REQUIREMENTS FILE. in the "Manage App" logs on Streamlit, but thankfully the sync has made that go away, and I suspect the current issue has something to do with authentication again, so I'm coming back and reopening this issue..
I have checked that the app runs locally as well as when I run it in a GitHub Codespace (I have added my EARTHENGINE_TOKEN to the Codespace). I am unable to tell why the EE exception error is happening only on Streamlit Cloud. Help, please?
Links:

Branch of my fork of this repo, for my app
app on deployed on Streamlit Cloud

Screenchots to show app is running fine for me locally and on Codespaces:

The authentication method has never changed.

If possible, it would be very helpful if could you suggest where I could look for why my app is not working on the cloud (Streamlit) alone. I'm unable to tell, and I'm not very proficient at this.
The most recent logs on "Manage App" on Streamlit is as follows. It is essentially the same as what appears in the map panel in the cloud app.

The line 106 it points to used to work, neither it nor the asset has changed.
It asks to run ee.Initialize(), but I had learnt (from earlier in this thread) that as long as the app runs I can ignore this.

ee.ee_exception.EEException: Earth Engine client library not initialized. Run `ee.Initialize()`
2023-10-31 11:40:14.944 Uncaught app exception
Traceback (most recent call last):
  File "/home/adminuser/venv/lib/python3.8/site-packages/streamlit/runtime/scriptrunner/script_runner.py", line 534, in _run_script
    exec(code, module.__dict__)
  File "/mount/src/streamlit-geospatial/app.py", line 131, in <module>
    app()
  File "/mount/src/streamlit-geospatial/app.py", line 106, in app
    oneLabelsCollapsedForViz = ee.Image("projects/ee-open-natural-ecosystems/assets/publish/onesWith7Classes/labelWithUncertAppliedONEStates")
  File "/home/adminuser/venv/lib/python3.8/site-packages/ee/computedobject.py", line 28, in __call__
    return type.__call__(cls, *args, **kwargs)
  File "/home/adminuser/venv/lib/python3.8/site-packages/ee/image.py", line 52, in __init__
    self.initialize()
  File "/home/adminuser/venv/lib/python3.8/site-packages/ee/image.py", line 98, in initialize
    apifunction.ApiFunction.importApi(cls, 'Image', 'Image')
  File "/home/adminuser/venv/lib/python3.8/site-packages/ee/apifunction.py", line 195, in importApi
    cls.initialize()
  File "/home/adminuser/venv/lib/python3.8/site-packages/ee/apifunction.py", line 160, in initialize
    signatures = data.getAlgorithms()
  File "/home/adminuser/venv/lib/python3.8/site-packages/ee/data.py", line 1388, in getAlgorithms
    _get_cloud_projects()
  File "/home/adminuser/venv/lib/python3.8/site-packages/ee/data.py", line 297, in _get_cloud_projects
    raise ee_exception.EEException(
ee.ee_exception.EEException: Earth Engine client library not initialized. Run `ee.Initialize()`

Thank you!


CC_28_MetaBalls_p5.js

I have re-implemented the MetaBalls coding challenge in p5.js, as in issue #267. Unfortunately, it is quite slow (due to the modification of individual pixels being slow in p5), so I downscaled the canvas (down to 400x200). Let me know I you know a way of optimizing it !

Wonderful thank you!


Change image processor

Ditch oily_png for the more versatile rmagick, thus unshackling the dependence on PNG I/O files.

It's been a while since I checked up on this project and the Ruby ecosystem of image manipulation. If there's a better library out there post here.
Minimum capabilities:

Pixel-by-pixel reading in a standard color format (RGB, RGBA, HSL, HSLA, etc.)
Pixel-by-pixel setting in a standard color format

Options that justify ditching oily_png:

Wicked fast pixel-by-pixel operations or general performance increase
Multiple image formats (PNG, JPEG, GIF)

Unrealistic goals:

No external dependencies like ImageMagick

If there are siloed gems for the various image formats (sorta like how chunky_png is for PNG) and they can be switched between in a sane and realistic way, that would be OK but might be a lot of work.
If a different dependency is not viable, should at least upgrade chunky/oily_png.


Extract All only gives "unk_#####" filenames

doing "extract all" option doesn't spit the files out properly? Only gives me files named "unk_###" with no proper filenames or directories that's shown in the filelist.
edit:
did extract option to point to a file, still gives "unk" filename.

new release up


workflows: Disable builds for targets that aren't generally needed

@hjpotter92 Should we merge this change it?


Add Chrome notifications

This change adds notifications to the Chrome extension. These
system-wide notifications will display when the status of an environment
changes.
Clicking on a notification will take the user to the corresponding
Icinga page.
Notifications can be configured for each environment, for each status
level in the extension settings.

Looking forward to trying this this week :100: 🎉


Add setting to switch to nightly releases

Description
#730 already set up the nightly pipeline. This PR will add a new "Use nightly releases" toggle to switch the electron-updater to the nightly repository.
It will also show a popup warning about nightly releases before switching so unexperienced users won't accidentally switch.
Checklist:

[x] My pull request is properly named
[x] The changes respect the code style of the project ($ npm run lint)
[x] I tested/previewed my changes locally

Thank you for taking care of this @vantezzen 🙏
I have been a bit confused to see the updater working since I have switched to the nightlies manually - see https://github.com/getferdi/ferdi/issues/329#issuecomment-626537218. I am not sure how, given setFeedURL for getferdi/nightlies is made available in this pull request only. Could you please help me clarify this?

I also don't know why the updater was working - it shouldn't even look in the nightlies repository (or have you changes something manually there?)

I was looking at the wrong documentation.
We are using autoUpdater from electron-updater, not from electron:

Do not call setFeedURL. electron-builder automatically creates app-update.yml file for you on build in the resources (this file is internal, you don’t need to be aware of it).

I think there is a small issue with the UX; after choosing Activate or Cancel in the modal, the user isn't redirected properly to the update tab which is a little bit confusing:

It seems the settings screen is being closed when the modal is shown. What do you think of moving the confirmation part below the ToggleRaw?

There is a bigger issue which I am not sure we should address in this pull request or not; I think with electron-updater, calling setFeedURL has no effect, so maybe going back to using autoUpdater from electron and explicitly calling it for every cases would be a solution? I am not sure why we're using the one from electron-updater in the first place.

We're using electron-updater because Franz uses it, too - I don't think there are any other reasons behind this.
It seems like Electron has a good method for updating, too (https://www.electronjs.org/docs/tutorial/updates). Should we switch to that instead? Maybe that will also fix the update problems.

@vantezzen Let's address the auto-updater issue (if any) in another pull request.
We can improve the UX via https://github.com/getferdi/ferdi/pull/977 if needed.

Yes, that sounds good.


Current Project State?

Hey all,
I'd like to get some more features added to the Grafana Agent which wraps around this exporter, however the number of outstanding PR's and issues is making me question if this project is still being actively developed?
The additional metrics I would like to see are:

Issue state (at the moment, it only returns "open" issues. We'd like to track open vs closed vs reopened over time)
Contributors, ideally with the number of PR's they've approved on the repo etc
Enhanced PR data such as checks passed, approvals required vs approvals granted, and merged vs awaiting merge

Given that the last commit to master was 248088c nearly 12 months ago, can someone let me know the status of the project so the Grafana team can work out where they want to go with this?
Thanks in advance!

@Rucknar / @iamrare - forgive the direct tags, however you seem to be the top contributors to this project.
Given that this project appears to be relatively low-priority when it comes to adding new features etc, would you both be willing to "donate" it to the wider Prometheus Community to see if it's possible to get it maintained there instead?
I have confirmation from the Prom Community IRC channel that it would require opening an issue at https://github.com/prometheus-community/community/issues and then updating the CI/CD tooling to get it to work with their own infrastructure, however I am happy to do both of these things, I just need your permission to move it and probably some of your time to approve the move as well.
Let me know your thoughts :)

@proffalken Happy to donate, apologies for the slow delay in responding.

@Rucknar no worries, and thanks!


lack of entropy

after boot finished
$ docker info
crypto/rand: blocked for 60 seconds waiting to read random data from the kernel

I tried to add a virtio-rng device like this
    <rng model='virtio'>
      <rate bytes='1024' period='2000'/>
      <backend model='random'>/dev/random</backend>
      <address type='pci' domain='0x0000' bus='0x00' slot='0x08' function='0x0'/>
    </rng>

but
$ cat /sys/devices/virtual/misc/hw_random/rng_current
none
$ cat /sys/devices/virtual/misc/hw_random/rng_available

I configured CPUID like this and I'm pretty sure it has rdrand feature flag
  <cpu mode='host-model' check='partial'>
    <model fallback='allow'/>
  </cpu>

but /proc/sys/kernel/random/entropy_avail is still increasing very slowly at ~3/second.
If I intentionally disable rdrand like this
  <cpu mode='host-model' check='partial'>
    <model fallback='allow'/>
    <feature policy='disable' name='rdrand'/>
  </cpu>

I have haveged and a lot of entropy.

We do include haveged inside the ISO for generating entropy, but we try to be very conservative in when we'll start it, especially given that it could have unintended impacts on the users of the system:
https://github.com/boot2docker/boot2docker/blob/e61c69392a4a145f299bc3fd76e1c70c446a4384/files/init.d/haveged#L53-L73

Yes, I understand and agree with the behavior of boot2docker/files/init.d/haveged. I just wonder why virtio-rng & rdrand cannot provide enough entropy in my setup. Is this a configuration problem on my side? Or we need to enable some kernel config or include some extra kmods?

@tianon I think kernel/drivers/char/hw_random/virtio-rng.ko should be included in boot2docker.

RANDOM_TRUST_CPU is also not not set in boot2docker and defaults to n which renders rdrand useless.

Definitely agree on the virtio RNG driver, although we're still on Linux 4.14 and CONFIG_RANDOM_TRUST_CPU wasn't introduced until 4.19+ so that shouldn't be relevant.

Thanks. virtio-rng now works out-of-box.


Bug: TypeError: util.inherits is not a function

Describe the bug
I see the following error in the console when I click on the coinbase connector
Steps

Expected behavior
The coinbase wallet connector modal should open where I can scan the QR code to login.
Version
v3.0.11
Additional info
Vite, Vue 3, typescript
Desktop
No response
Smartphone
No response

It’ll work fine in any working node module bundler.


Avoid unnecessary includes

Hello,
when MBED_CLOUD_CLIENT_SUPPORT_UPDATE is not defined, UpdateClientResources.h and update_client_public.h should probably not be included in ServiceClient.cpp to avoid unnecessary dependencies towards update client hub.
https://github.com/PelionIoT/mbed-cloud-client/blob/d7edc529ed3722c811ff401440ef58ea980bf543/source/ServiceClient.cpp#L28-L31
Proposal:
#if !defined(MBED_CLOUD_CLIENT_FOTA_ENABLE)
#if defined(MBED_CLOUD_CLIENT_SUPPORT_UPDATE)
#include "include/UpdateClientResources.h"
#include "update-client-hub/update_client_public.h"
#endif
#else

Thanks for the feedback @ghseb. Looks sensible.

https://armiot.atlassian.net/browse/IOTCLT-5026

Should be fixed now with 4.11.0 release.

Thanks!


add support to Log Message breakpoint

#426  @WebFreak001, thanks for your advice, i modified the code, makes LogMessage var and function in a class.

cool! can you add a sample, maybe a sample screenshot/gif in the README or other docs file to show off this feature?

btw you can just push to the new branch that you made and the PR will automatically update on github, you don't have to create a new one when you change things

btw you can just push to the new branch that you made and the PR will automatically update on github, you don't have to create a new one when you change things

thank you. i fix the lint error, add a gif in readme


[Feature] Support Function [[targets]]

WHY are these changes introduced?
In #gsd:34464, we're breaking down Function APIs into multiple targets. The exact TOML is still being debated, but this should be reasonably close, and not be used in production until we roll out unified deployment.
Closes https://github.com/Shopify/script-service/issues/6912
WHAT is this pull request doing?
Add support for a TOML that looks like:
name = "network-validation"
type = "cart_checkout_validation"
api_version = "unstable"

[[targets]]
target = "purchase.validation.one"
input_query = "path/to/one.graphql"

[[targets]]
target = "purchase.validation.two"
input_query = "path/to/two.graphql"

Map that targets array to a new targets field in the config, of type {handle: string, input_query: string | undefined}[].
Without the unified deployment beta flag, having the [[targets]] array won't error, but it won't do anything either (the non-unified-deployment path does not read the targets array). That makes the implementation simpler and feels like a reasonable tradeoff: unified deployment will be fully rolled out soon-ish.
How to test your changes?
It's going to be difficult, but here's how:
Setup:

Checkout the branch from Shopify/shopify#428889
Add a target for target = "purchase.validation.perform" in your function.extension.toml.
Enable the unified deployment beta on your app
Create a dev store
Install the app on your dev store

Action:

Deploy the app (app deploy)
Create a deployment in the UI
Setup a checkout validation in your store

Outcome:
The function should've ran with the input query from that was specified in purchase.validation.perform.
Checklist

[x] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[ ] I've considered possible documentation changes
=> not yet, we're not rolling out the feature yet.

Some changes since I opened the PR:

Added an export field, because we're switching from a WASI-args based approach to named export.
Renamed [[targets]] array to [[targeting]] because that's the leading proposal for the TOML structure. The resulting extension config is still an array of targets because I think internally the name makes more sense (i.e. the data structure field should not be targeting: Target[], but instead targets: Target[].
The extension building code moved to a different file, so I moved the additions too.

I'm working on related PRs so the above WASI-args-to-export works, thus haven't yet tried the end-to-end solution. Will report back once that's done, but feel free to give this another review if you'd like.


[Function] Refactor validation of address

User story:

this function helps validate the address of the constituent before sending as a lettet
Select lines 57-80 in lob.js within server > routes/api
We're going to now make

Copilot Prompts

Type into chat:

what does this function do
how can I make this more readable
how can I refactor this

Talk to your mentor if it makes sense to update this file through whats given:

const { line1, city, state, zip } = address;

if (!line1?.trim() || (!zip?.trim() && (!city?.trim() || !state?.trim()))) {
	throw new Error('Address object must contain a primary line (line1) and either a ZIP code or both city and state');
}

if (zip && typeof zip !== 'string') {
	throw new Error('Address object must contain a string-based ZIP code');
}

if (zip && !VALID_US_ZIP_CODE_MATCH.test(zip.trim())) {
	throw new Error(`Address object contained an invalid ZIP code: ${zip.trim()}`);
}

Branch: issue-592


Wrong instructions for rehype plugins

📝 Issue Description
Hey guys,
I would like to report a problem with documentation.
It is about markdown.rehypePlugins settings.
import rehypeMinifyHtml from 'rehype-minify';
This line is not valid anymore since there is no npm package called rehype-minify
📋 On which page(s) it occurs
https://docs.astro.build/en/reference/configuration-reference/#markdownrehypeplugins
🤔 Expected Behavior
The documentation should provide a working PoC code/configuration.
👀 Current Behavior
The configuration of documentation doesn't work since there is no package called rehype-minify
🖥 Browser
Chrome X
📄 Additional Information
No response

Thanks for spotting this and pointing it out @kristijan97 ! We did update our docs content on the Markdown/MDX page when this was discovered before ( https://docs.astro.build/en/guides/markdown-content/#markdown-plugins ), but it looks like we did not also update the code example on the configuration reference page!
This issue would require a PR to the main astro repo at this location: https://github.com/withastro/astro/blob/fca6892f8d6a30ceb1e04213be2414dd4cb4d389/packages/astro/src/%40types/astro.ts#L1169
The example could be updated to show any valid rehype plugin, as we have done on the Markdown page.
Labelling this as help wanted so anyone can feel free to make this update!  🚀


CT: Missing step for communicating with user about application status

There's a step missing here that we have. After a user has applied, we send a letter (ew) to tell them that their application is valid and 'successful', and then the actual pass arrives in due course from the supplier. Here the confirmation page only states that "if your application passes manual checks, you'll get it in 7 days", but never gives the user an update on whether those manual checks have been passed - they only find out once they pass arrives, weeks later (= status anxiety)

@samuelculley can we not use the Home Office photo checker? It might need to be made a bit less strict but it should be open source to steal and adapt right? https://www.passport.service.gov.uk/photo/upload
Passport renewals still do a manual check against the old passport photo to make sure it's the same person, but we wouldn't need to do this. Then the notifications would become irrelevant as we'd be able to send stuff off straight away. Notifications would only be relevant when manual checks are required.

Makes sense, I think there might be some discrepancy where the user's pass still has some checks to go through after they finish their online journey. I know at least one supplier that plans to check there aren't any duplicates only after the user has finished the application. I'm not sure how that would work, and I don't know if it's something we should include in the service pattern.

That supplier/LA shouldn't be doing things in that way if possible. We should be failing users fast, so it would be better to use an API to check this while the user is going through the service.
Do you know why they plan to do it in this way?

Their current pass printing setup also does their duplicate checking. They aren't sure how they could separate the two without having a lot of issues with suppliers. I don't have much knowledge about this though, it was raised as a concern when we did the service design workshop so things may have happened since then. I'm speaking to them later today so I'll find out what's up.

Cool. Either way I'd stay away from incorporating this into the pattern, it
sounds like a local idiosyncrasy and it's not something we'd want to
recommend.
K
On 25 January 2017 at 15:16, samuelculley<EMAIL_ADDRESS>wrote:

Their current pass printing setup also does their duplicate checking. They
aren't sure how they could separate the two without having a lot of issues
with suppliers. I don't have much knowledge about this though, it was
raised as a concern when we did the service design workshop so things may
have happened since then. I'm speaking to them later today so I'll find out
what's up.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/alphagov/verify-local-patterns/issues/97#issuecomment-275134860,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATw9IhAyyhMx6OUZZ_rBFyUliFx60gNDks5rV2digaJpZM4LtZ6F
.

--
Kate Ivey-Williams | Service Designer for GOV.UK

@samuelculley @kateiw Ok to close this issue?

yep I think so
On 6 February 2017 at 14:35, Sanjay Poyzer<EMAIL_ADDRESS>wrote:

@samuelculley https://github.com/samuelculley @kateiw
https://github.com/kateiw Ok to close this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alphagov/verify-local-patterns/issues/97#issuecomment-277699717,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATw9IhmiYK7ff5z28AFuXErpvqe3ZtIZks5rZy-pgaJpZM4LtZ6F
.

--
Kate Ivey-Williams | Service Designer for GOV.UK


Configure font-size via query-string

This can be merged, but it's still not fixed for single-number metrics


Ignore pattern is not distinguish file and directory

sometimes it will confuse.
for example, '.d' file is generated by gcc and nobody is glad to edit them. but many directories in /etc are end with '.d' too.
I suggest use two patterns to deal with this case: '.d$' for files, and '.d/$' for directories.

Just pushed a commit to handle this.
See :help NERDTreeIgnore for details, but long story short:
if you have '.d$[[dir]]' it will match only dir nodes, and 'foobar[[file]]' will match only file nodes.


Getter methods on web plugin return wrong types

Describe the bug
Web version getter methods return different type (e.g. Promise<string|int|boolean>) than native versions (e.g. Promise<RCReturnData>).
To Reproduce

Call FirebaseRemoteConfig.getString on web version => returns Promise<string>
Call FirebaseRemoteConfig.getString on iOS version => returns Promise<RCReturnData>

Expected behavior
Return the same type regardless the platform.

FirebaseRemoteConfig.getString should return Promise on all platform.

It should return a Promise<type> with a consistent type and not Promise<string> on one platform and Promise<RCReturnData> on another...

@mic345 Sorry for my first msg. It was badly formatted, and gh removed the <string> part...
But yes, that's what I'm saying, but specificaly: the Promise<type>, should NOT be Promise<RCReturnData> when the method is getString...
=> It should be Promise<String> for getString, and Promise<boolean> for getBoolean, etc...
Even getValue  could use generic to return the type.
getValue<T>(key: string): Promise<T>;
Also, there is no use for passing an object in getter. Instead of getBoolean({key: string}): Promise<boolean>;, method should be getBoolean(key: string): Promise<boolean>; (same for others getter)
And you are right, it should be consistent for every platform, be it WEB or Android, or IOS.

Sorry for my first msg.

No worries 😊


Conflicted Nodes (1)  ShowText|pysssss [ComfyUI-Universal-Styler]

【pythongosssss/ComfyUI-Custom-Scripts】Conflicted Nodes (1)
ShowText|pysssss [ComfyUI-Universal-Styler]

This is an issue with the other repo copying the node and not giving it a different ID, nothing I can do about it if you could please log an issue there, thanks.


ADS1015 returning 16 bits?

Hej!
I am using the ads1015 in single ended mode and getting 16 bits when I ask for the value. If it is allegedly a 12 bits ADC shouldn't I get 4096 as the maximum value? Am I missing something?
Thanks a lot,
Joan

Are you sure it's an ads1015 and not a ads1115 you have?

Hej! I am pretty sure it is the ads1015, I am using the version from SparkFun Qwiic (12-bit) ADC, which only exists fot the 12 bits (and it is specified in the board).
Here is the snippet of the code I use to check the value:
from time import sleep
import board
import busio
from adafruit_ads1x15.analog_in import AnalogIn
import adafruit_ads1x15.ads1015 as ADS

i2c = busio.I2C(board.SCL, board.SDA)

gain = 16
address = 72
channel = 0

ads = ADS.ADS1015(i2c,gain = gain, address = address)

ch = AnalogIn(ads, 0)

while True:
    try:
        value, voltage = ch.value, ch.voltage

        print(f'\rvalue = {value: 6d}, voltage = {voltage:.4f} V', end = '')
        sleep(0.1)

    except KeyboardInterrupt:
        print('\nProgram terminated')
        break
    except Exception as e:
        print(e)
        print('\nAn exception ocurred.')
        break

thanks!

The returned value gets stretched to be 16 bits even if it's coming from 12-bit resolution.  I assume that this is so it can be used as a drop in solution for onboard ADCs that also return 16 bit values.
https://github.com/adafruit/Adafruit_CircuitPython_ADS1x15/blob/1554279746863bf07b7e9f770cd9f62c33d027df/adafruit_ads1x15/analog_in.py#L53-L55

Oh! I see, I missed that, I didn't get at first this bit shifting operator.
Problem solved then. Thanks!

Just commenting to confirm the shifting is intentional and is simply a CircuitPython specific design decision. Main motivation is to have a consistent behavior for the value property regardless of ADC being used:
https://docs.circuitpython.org/en/latest/shared-bindings/analogio/index.html#analogio.AnalogIn.value
This is similar to Arduino's analogRead() returning 10 bit values by default.


google-cloud-datastore Java library exposes too little of the information in query results

In particular, this makes it impossible to implement a more efficient implementation of counting query results, as that requires the values from the skipped_results and more_results fields.
See https://groups.google.com/forum/#!topic/gcd-discuss/wH8lVOA-a8Y for an in-depth discussion of what's needed from query results to implement that.
Generally, skipped_results, more_results and snapshot_version should be available in QueryResults
(https://github.com/googleapis/googleapis/blob/master/google/datastore/v1/query.proto#L261)

@wsh Could you take a look?

Sure, though I'm not sure what you'd like me to look for/at. QueryResultsImpl already uses the more_results field. I think this is a question of taking the info from the existing proto and adding an elegant encapsulation for the interface (or potentially exposing runQueryResponsePb itself).


[Feature] Validate object-store-memory before creating the cluster

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
User could create the cluster and specify object-store-memory with a large number that overs container memory. it's better for us to do some more validation.
Test cases:
rayStartParams:
  ...
  object-store-memory: '10240000000'
resources:
  limits:
    cpu: "1"
    memory: "8G"
  requests:
    cpu: "500m"
    memory: "8G"

ValueError: After taking into account object store and redis memory usage,
the amount of memory on this node available for tasks and actors (-2.86 GB) is less than -35% of total.
You can adjust these settings with ray.init(memory=<bytes>, object_store_memory=<bytes>).

Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

/cc @scarlet25151

Hey @Jeffwan 👋🏼
Just checking if this is still a relevant issue? Thanks!

@MadhavJivrajani
not a problem now. It can be closed. Thanks for the follow up.


Text selected in editor while dragging debug toolbar

Test item #7815

VSCode Version: Code - Insiders 1.3.0-insider (798b41ab1cf3636a27042020a54ddfb4cfb8d23c, 2016-06-29T05:07:11.483Z)
OS Version: Windows_NT ia32 6.3.9600

Steps to Reproduce:

Enable showTabs
Open two files side-by-side
Start debugging
Drag the debugging toolbar left and right with the mouse passing over the editor tabs
-> Sometimes all text in the left editor is selected and then deselected again

@isidorn Do you call e.preventDefault() ?

@alexandrudima no, let me do that - thanks!

Looks like not yet part of the build. So untagging

Yes - this was just fixed 8 hours ago and we did not have an update yet - tomorrow morning should be good.


Fix quantity formatter initialization

Ensure quantity formatter is initialized after IModelApp.startup is awaited.

@mergifyio backport release/2.11.x

@mergifyio backport release/2.11.x


VPN: OpenVPN: Connection Status: migrate to MVC

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Is your feature request related to a problem? Please describe.
Static pages need to be removed eventually. Services page offers opportunity for it and enables API handling of the functionality.
Describe the solution you like
Probably a good idea to start with in order to make room for a new mvc implementation of OpenVPN. The backend part is already finished.
Describe alternatives you considered
N/A
Additional context
N/A

@AdSchellevis there seems to be a small leftover:
# git grep status_openvpn
src/www/widgets/include/openvpn.inc:$openvpn_title_link = 'status_openvpn.php';
src/www/widgets/widgets/openvpn.widget.php:            $.post('/status_openvpn.php', {action: 'kill', port:port, remipp:ip}, function(data) {


typo in find_and_run.sh for linux / macos backup / restore

here's a quick patch:
 for i in $(sh -c "$list_fat"); do
     if [ -d "$(printf $i)/Nintendo 3DS" ]; then
         case "$act" in
             inject)
                 $save3ds --sd "$(printf $i)" extdata/$tid --import --sdext $tid --boot9 resources/boot9.bin --movable resources/movable.sed
                 ;;
             backup)
-                $save3ds --sd "$(printf $i)" $tid --extract --sdext $tid --boot9 resources/boot9.bin --movable resources/movable.sed
+                $save3ds --sd "$(printf $i)" extdata/$tid --extract --sdext $tid --boot9 resources/boot9.bin --movable resources/movable.sed
                 ;;
             restore)
-                $save3ds --sd "$(printf $i)" $tid --import --sdext $tid --boot9 resources/boot9.bin --movable resources/movable.sed
+                $save3ds --sd "$(printf $i)" extdata/$tid --import --sdext $tid --boot9 resources/boot9.bin --movable resources/movable.sed
                 ;;
         esac
     fi

no, extdata is where the exploit data stored, what you did simply ruin it
but yeah, it should be inside backup, which is already fixed (not uploaded yet)

suit yourself -- I added it to the backup command where it was missing and without it I was getting the cryptic error messages from https://github.com/wwylele/save3ds/issues/14
thought I'd be helpful and post here but it seems it is not welcome -- have a good day


[Windows] Fix TTS events arriving out of order.

Fixes https://github.com/godotengine/godot/issues/99166
Ensures TTS query is updated only after processing pending TTS events (processed by hidden window message loop).

Thanks!


Error during install with empty DB

When I try install with empty DB, I always get this error:
There is no column with name 'status_id' on table 'orocrm_sales_opportunity'.
If run command again, error does not occur.

Hi @scapital
It is a known bug. Thank you for reporting it.

the same here

Fixed in 2.2
4325369e31a2c4f7de5fb5edc7a3fd3f8aad0221


Gjoranv/ensure same host

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Yes, but that should typically take less than a minute, so in most cases I don't think we will lose any metrics at all.

Closing, as 'restart' means to restart all services on a node.


Captions for premium users are 2048, not 2024

As written in the title, the captions for premium users can contain up to 2048 characters, not 2024!
https://github.com/tginfo/Telegram-Limits/blob/46c1e03190d4447f4f67c79ddb32f2c85a498d1a/localization/en/data.json#L273

As written in the title, the captions for premium users can contain up to 2048 characters, not 2024!
https://github.com/tginfo/Telegram-Limits/blob/46c1e03190d4447f4f67c79ddb32f2c85a498d1a/localization/en/data.json#L273

Thanks, yeah it was fixed already.


does back-end（web & collector）support cluster？

We hope the back-end  can load balancing
if can，how？
Thank you
Sorry, my English is very poor

sorry :sob:
we do not support function of load balancing.
and we have no plan to make this function yet.

Can use haproxy to implement ？
and what is the function of ‘package com.navercorp.pinpoint.collector.cluster’ ？

Yes. you can haproxy.
'com.navercorp.pinpoint.collector.cluster' package do connect web to agent.
It's largely irrelevant function of load balancing .

ok，Thank you very much


Fix path of 0..* variable-length relationships and align it with Neo4j

Sample case for Kuzu:
CREATE NODE TABLE T(id SERIAL, PRIMARY KEY(id));
CREATE REL TABLE CHILD_OF(FROM T TO T);
CREATE (t:T);

kuzu> MATCH p = (t1:T)<-[:CHILD_OF*0..]-(t0:T) return properties(nodes(p),'id');
---------------------------
| PROPERTIES(NODES(p),id) |
---------------------------
| [0,0]                   |
---------------------------
(1 tuple)

Sample case for Neo4j:
CREATE (t:T);
MATCH p = (t1:T)<-[:CHILD_OF*0..]-(t0:T) return [i in nodes(p) | id(i)]
╒═══════════════════════╕
│[i in nodes(p) | id(i)]│
╞═══════════════════════╡
│[0]                    │
└───────────────────────┘

Interpretation
A variable-length relationship 0..* can be interpreted as matching a path with 0 to n hops between nodes.
Taking the special case 0..0 for this bug report, it can be regarded as: "match a node without hops/relationships in-between". Following this logic, we get node with id 0 as result here, despite having no existing CHILD_OF relationship. In fact the relationship does not matter at all and we could write (t1:T)<-[:NOT_EXISTING_REL*0..]-(t0:T) in Neo4j and still retrieve node 0,
I would expect [0] as output for the matched path, which is Neo4j's result.
Output of Kuzu currently is [0,0], which does not seem to reflect above quantifier semantics.
Note: 0..0 is probably not really useful. But this behavior also has influence on results of 0.. relations.

Solved in #2412


Integration to QuickAdminPanel

Hello,
Can this starter be integrated to a project created through QuickAdminPanel? I tried with mine (QAP jQuery, laravel 9, bootstrap 4), but it's giving me error

Vite manifest not found

@ScriptBe3 hi, no, those have totally different codebases and are incompatible at the moment.


Added default name based on the queue name(s)

Added implementation of generating a default name from the specified queue name(s) to fix issue #4

Fixes #4
Thank you for your contribution!

I tweaked it a little bit: https://github.com/SRWieZ/queue-size-health-check/commit/dad185252aba3e984414084efe7c43e10340e913


reqIs does not correctly parse the Content-Type header

Describe the bug
reqIs does not separate the content type from the charset
To Reproduce
( related issue #142 )
Running this test results in an error, since reqIs does not properly separate the actual content from the charset info.
it('should ignore charset', async () => {
    const app = runServer((req, res) => {
      req.headers["content-type"] = 'text/html'
      expect(reqIs(req)('text/html')).toBe('text/html')
      res.end()
    })
    await makeFetch(app)('/', {
      headers: {
        'Content-Type': 'text/html; charset=UTF-8',
      },
    })
    expect.assertions(1)
  })

Expected behavior
It should separate the text/html part of the Content-Type header from ; charset=UTF-8
Versions

node: 14.13.0
@tinyhttp/app: 0.4.3

Anyone can submit a bugfix :D


Replace obsolete --disable-geoReplication with new --type switch

The documentation 'storage account create/set' with still referring to the obsolete --disable-geoReplication parameter.

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

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

TTYL, AZPRBOT;

@tomconte Thanks for the contribution!  I've sent this change to the author for review.

@tomconte Good update, thanks! Looks fine to me. @deneha Please merge


doc: Some improvements to KinematicBody docs

I have looked over any verbs that ends with -ing and replaced them with active verbs, according to the documentation writing guidelines.

idk, even by my own standards, this feels a bit too pendatic, i don't really care too much about grammar, or weird standards, as long as the comment easily explains what the source code is supposed to do in a clear way

@RaTcHeT302 Godot is great, because many people contribute in many different ways. Some create features. Others fix bugs. Some create documentation. Others translate the documentation. Godot automatically generates class documentation, and the people that create the original features, classes and functions are often not native English speakers. Sometimes the documentation needs to be corrected. What's important is not what contributions people make, but that feedback is relevant and constructive.

@madmiraal What do you think of the current state of this PR?
There is still one instance of "it calls" being added, not sure if that's desired or not.

Rebased, further edited to address remaining feedback, and applied same changes to KinematicBody3D. No changes to -ing forms which were all correct in the end...

Thanks!

Cherry-picked for 3.2.4.


Pytest xdist

Codecov Report

Merging #1 into master will decrease coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #1      +/-   ##
==========================================
- Coverage   95.14%   95.08%   -0.07%
==========================================
  Files          24       24
  Lines        3072     3072
==========================================
- Hits         2923     2921       -2
- Misses        149      151       +2

Impacted Files
Coverage Δ

geopandas/io/sql.py
87.50% <0.00%> (-1.67%)
:arrow_down:

Continue to review full report at Codecov.

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


DCX - add complete instruction for building a DLL file

I am using "Visual Studio Community 2019 v16.11.3" in system "Windows 8.1 x64".
When trying to build a DLL, at least a few errors about missing files appear. Surely many have encountered similar problems and difficulties in assembling this project, and do not know how to fix it and what needs to be done:

C1083 - gitBuild.h: No such file or directory
C1083 - GSL\gsl: No such file or directory

This creates problems when testing the update in "dcxdll-master" to find bugs and quickly fix them, before the main release.
Many of the users of your project would like to receive a complete and detailed instruction on how to correctly assemble this project into a DLL file. This can be a list that should contain at least the following step-by-step instructions:

What development environment programs and their versions need to be installed.
What additional components and dependencies need to be installed.
What additional files may be needed, where to get them or how to create them manually.
Which project files should be selected and which menu items in the development environment should be clicked to start the correct DLL compilation process.
Where and to whom to contact for help and support in case of errors during the compilation of the project.

This has been added to the docs.


video version of demo

thanks for making the Gradio demo https://huggingface.co/spaces/jjeamin/ArcaneStyleTransfer, can a video version also be setup? Similar to for example, https://huggingface.co/spaces/nateraw/animegan-v2-for-videos

A video version is not yet planned.
My model needs to improve the style transfer performance of the body or background.
I'm currently looking for ways to improve performance.


Interface variance is totally unchecked, usually

A canonically contravariant interface is Consumer (left hom). Yet Flow
lets us (a) type this as a covariant interface, and (b) abuse this
unsoundness to induce a runtime error:
// @flow

// This interface is obviously not covariant, yet I can define it,
// instantiate it, _and_ unsoundly take advantage of the covariance!
interface Consumer<+T> {
  accept(t: T): void;  // no Flow error! bad!
}

class LowercasePrinter implements Consumer<string> {
  accept(x: string) {
    console.log(x.toLowerCase());
  }
}

const wat: Consumer<mixed> = new LowercasePrinter();
wat.accept(123);  // runtime error

But the error is caught if we ever store a Consumer as a field:
// @flow
interface Consumer<+T> {
  accept(t: T): void;  // raises a Flow error (good)
}
class Box {
  consumer: Consumer<string>;
}

If in this latter example we change class Box to declare class Box,
then the error is again not caught.
The converse—declaring interface Supplier<-T> { get(): T; }—exhibits
the same problem.
What gives?

This yields coerce:
// @flow
function coerce<T, U>(t: T): U {
  interface UProducer<+T> {
    apply(t: T): U;
  }
  const p: UProducer<U> = {
    apply: (u) => u,
  };
  return (p: UProducer<mixed>).apply(t);
}


Register with CC Gateway

CC Gateway (Stripe) has been configured. Keys have been added to our internal key store.


Coq proof lib

Add coq library files.

From what I can see MachineIntergers.v file is not directly part of CompCert library, so we do need it here.

From what I can see MachineIntergers.v file is not directly part of CompCert library, so we do need it here.

Interesting. So, where does it come from then? If possible, we should try to avoid code duplication.

From what I can see MachineIntergers.v file is not directly part of CompCert library, so we do need it here.

Interesting. So, where does it come from then? If possible, we should try to avoid code duplication.

Hmm it might be here https://github.com/AbsInt/CompCert/blob/master/lib/Integers.v , however it seems modifications been made to that file, as modules are used, while we just need a typeclass.

Yes, that seems like a likely source.
It's possible that we are just using an old version.
For the last yard, we've switched to the math-comp integers. It would be good to switch to the new stdlib one that's in preparation when it becomes available.

Yes, but I would like to add some testing, and structure the library a bit, but I guess those can be follow up PRs.

Why are you going back to the compcert integers? The jasmin/math-comp ones are needed for the jasmin connection.
Getting this merged would help us make a choice. Moreover, that library comes from fiat-crypto, which we also connect to with hacspec.
https://github.com/coq/coq/pull/17043

Ok. Yes, that's a good idea. Didn't we use the MC/jasmin integers for the
SSProve backend?
On Mon, Mar 25, 2024 at 3:08 PM Lasse Letager Hansen <
@.***> wrote:

We have been using CompCert integers for the Coq backend, I was just
updating them, to remove the weird extra file (MachineIntegers.v). If there
is a better or more standard implementation, I can change the
implementation to use that.
—
Reply to this email directly, view it on GitHub
https://github.com/hacspec/hax/pull/386#issuecomment-2018089075, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABTNTXNOFDWIZ3VX7GP6YTY2AVVPAVCNFSM6AAAAABALBNIJKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJYGA4DSMBXGU
.
You are receiving this because your review was requested.Message ID:
@.***>

Closed in favor of #987 and annotated core lib.


[BEAM-5537] Upgrading google-cloud-bigquery to 1.108.0

Upgrading the following dependencies:

google-cloud-bigquery 1.108.0
google-api-services-bigquery v2-rev20190917-1.30.3 (from google-cloud-bigquery-parent:1.108.0)

Linkage Checker
No change in Linakge Checker result
https://gist.github.com/suztomo/8e3aafc5564be5356184e7760c789b8b

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[x] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.


Improve handling progressbar refresh rate in Google Colab

🚀 Feature
Set progress bar refresh rate automatically in Colab (as is right now), but still allow user to override if needed.
Motivation
Want no warnings in colab, but at the same time not crash the browser and also not prevent user from overriding the value.
Pitch
Before:
Trainer(progress_bar_refresh_rate: int = 1)

Now:
Trainer(progress_bar_refresh_rate: int = None)

if colab and refresh_rate = None:
    refresh_rate = 20
elif refresh_rate = None
   refresh_rate = 1

# otherwise we keep value passed in by user

Alternatives
Currently, user can't override the value in Colab.
Additional context
Discussed in slack with @edenlightning

well just for the record, the user can override this today in colab AFAIK. please also note that the warning can be removed.

well just for the record, the user can override this today in colab AFAIK. please also note that the warning can be removed.


Fixed parameters declaration in targets command

This was triggering the following assert:
thread 'main' panicked at 'Command targets: Argument or group 'package_values_regex' specified in 'conflicts_with*' for 'package-values' does not exist', C:\Users\<username>\.cargo\registry\src\index.crates.io-6f17d22bba15001f\clap-3.2.25\src\builder\debug_asserts.rs:237:13

Thanks so much. I guess we rarely run any tests or anything with debug assertions on. I wonder if we should have at least a smoke test in debug mode to catch these kind of violations.


Feature request: Add subview rather than placeholder

The af_setImageWithURL call is fantastic and I've integrated it deeply into my software, so thank you for the good work. However, I was curious if you have any plan to implement something along the lines of a placeholderSubview parameter? The placeholderImage parameter is a very nice touch. However, when one is seeking to download an image that will replace the one previously shown, it'd be ideal to overlay an image (or preferably animation) on the previous photo. This, however, may be too complicated to integrate. Thanks again for building a solid framework.

This seems like a complicated enough use case that you'd probably just want to implement this yourself @farice. There are a few too many variables to make an elegant solution here that could satisfy everyone in AFI.
You should be able to leverage most of AFI though to make this a fairly straightforward feature to implement.
You can also use the ImageTransition enum to accomplish the animation portion of your request.
Cheers. 🍻


V4 UA string change (Jim)

This is defined in the code here and the format is ${packageName}/${packageVersion}.

Updated UA string.


io.load_image_from_dir needs documentation/support

This is a critical function for brainiak but from the documentation or comments it is unclear how it works. Our code was not working because the files had been loaded in 'out of order' because this function was using the file order system of our OS (1, 10, 11, 12, ... 19, 20, 21, ...) rather than what we wanted (1, 2, 3 ...). It would be good if there were three changes:

A way to print out the order of files that are loaded in (if it is currently possible, it could be easier or documented).
A comment in the docstring stating how the data is expected to be organized.
Encourage users in the docstring to name their subjects with zeros (e.g. 01, 02, 03)

The file names are sored lexicographically, so "10" comes before "2", as you noticed. This is how software I am familiar with works.
You can see the file names in the log as the loading happens if you set the logging level to debug.
I think extending the docstring the way you suggest would help. Looking forward to a PR.

I think adding a sort option to enable "natural" sorting ("2" before "10") also makes sense. Nevertheless, I would leave lexicographic sorting as default.

I agree that the default should be the lexicographic ordering (especially since we don't want to change the behavior for other users). I think adding a natural sorting step would be a great idea, and more documentation to say that the information is available in the log file. Still though, why not make the listed paths an attribute of 'images'?

The resulting images are Nibabel SpatialImages (as mentioned in the documentation). You can get the filename of each image using get_filename().


Auto switch l2 network

@joshklop While testing custom prefixes for #273 I realized that we'll probably want the step to add the L2 chain config to still be a button rather than having it immediately add on page load. If a user is using a custom prefix then the bech32Config that's passed to keplr will need to be updated with the expected bech32 prefixes.
We'll probably need the user to add their prefix through a text input field and then we can update the bech32 config for them whenever they click the button to add their chain to Keplr.

For visibility - Talked about this in the daily meeting and we won't include the custom prefix change in this PR. That work will be included in #282


Auth with Sentry

I have implemented your package but I can't make a login with Facebook::auth() because I'm using Sentry for user login/registration.
How to login with Sentry and LaravelFacebookSdk?

I don't use Sentry but I imagine you just need to find the user model by their Facebook ID and then use Sentry::login() to log them in. As long as your User model implements Cartalyst\Sentry\Users\UserInterface you should be able to do this:
// Find the user using the their Facebook id
$user = User::where('facebook_id', 4)->first();

// Log the user in
Sentry::login($user, false);


General: Simplified addon distribution

Brief description
Simplified ayon distribution by using ayon_api module for most of server calls.
Description
Use ayon_api to download files from server and to validated versions. Initialization of AddonInfo is not crashing if there is source with unknown type -> it causes crash of addon even if the source may not be important. Unknown source are stored to addon info. Initialization does not crash if there is none dependency package available on server. Addon code can expect type "server" to download package.
Renamed function check_addons -> make_sure_addons_are_updated and check_venv to make_sure_venv_is_updated. There will be functions in future to check addons updates but they should not download anything. Simplified argument of both functions.
Moved helper functions from start.py to openpype common as they'll be needed elsewhere. Argument data is required for downloaders but they don't contain "headers" or "endpoint" but what type of download file it is. Added autofix of http type to be converted to server. Fixed unzipping which was unzipping to wrong directory.
Testing notes:

Everything should work as did before

Task linked: OP-3739 Connect OP tray to v4 server


Automated cherry pick of #3287: huawei eip disassociate bugfix

Cherry pick of #3287 on release/2.11.
#3287: huawei eip disassociate bugfix

/approve
/lgtm


:sparkles: Adds ability to also format the code

In this pull request, I added the ability also to format the code. This allows us to automatically create a pull request using the peter-evans/create-pull-request with the required changes. I will add the test report to this pull request.  Let me know what you think and if you have any suggestions.

✅ Tests results (success)


[ListView] renderSectionHeader onPress event not working on overlay?

Hey guys, I'm having a little weird issue here not sure if anyone else has noticed this but when adding an event inside a renderSectionHeader it works when its docked to the top, when overlayed over the section it doesn't work.
Here's a screenshot scenario.
I am able to click through the touchableHighlight surrounding the image (when docked):

I am not able to click through the touchableHighlight surrounding the image when overlayed over the section:

@yamill Can you test this with 0.10.0-rc if you still have the code available? An issue with touch events in sticky headers has been fixed.


Losing asset editor work with multiple Arcade windows open

Describe the bug
I've noticed this happening occasionally where I'm editing an image within the asset editor, and then have another tab open with a different project. If I switch to that tab for a bit and then go back to my image, I will see the triple toggle switch to the blocks editor (while behind the asset editor). When I click 'Done' for my image, it won't be saved to my assets and will be lost.
I've managed to reproduce it a few times, but never on camera :(
To Reproduce
Steps to reproduce the behavior:

Open a new project, go to the Asset Editor, make a new image
Draw something
Open another arcade tab, open up a project. Let the simulator simulate, maybe hang out here a bit
Switch back to the asset editor tab
Maybe you'll see the triple toggle move from Assets to Blocks. Maybe it has already moved to Blocks. (maybe it stays in asset and doesn't repro)
Click Done - you'll be in the blocks editor. Go back to the asset editor, and your asset won't be there

Desktop (please complete the following information):

OS: Windows
Browser Chrome 87

Additional context
I will keep trying to repro it to see if I can get a gif...

Switching tabs we reload to latest. There are bugs there.

Switching tabs we reload to latest. There are bugs there.


Login with Turbolinks and remote form

I couldn't find anywhere a tutorial or some SO answer to this problem. Basically I'd like to

have form to use remote: true (so it's javascript request)
and render some JS on failure

However, when I do that - Devise session's controller responds with
You need to sign in or sign up before continuing.

And then browser tries to evaulate this as JS code.
How can I write JS for such response?

Please use StackOverflow for questions/help, where a wider community will be able to help you. We reserve the issues tracker for issues only. Thank you!


Add 0.19 manifests and remove 0.15 manifests

Proposed Changes

Add the 0.19 manifests and remove the 0.15 manifests for operator 0.19 release.

Release Note

/test pull-knative-operator-upgrade-tests

/test pull-knative-operator-upgrade-tests

I think we do not need the pre job for eventing:
https://github.com/knative/eventing/issues/4505
🤷

/test pull-knative-operator-upgrade-tests

@matzew Do we still need pre jobs for 0.18?

/test pull-knative-operator-upgrade-tests

See the PRs for
https://github.com/knative/eventing/issues/4505
we remove them on master and 0.19 release branch
/cc @lionelvillard


Updating Game Art memory error

Updating Game Art
To Reproduce
Steps to reproduce the behavior:

I chose my GameBoy Advance Directory
Selected them all
Chose Update Art
See error

Expected behavior
I am able to update no problem on version GarlicPress.0.0.2307.261
Screenshots

OS: Windows 11
GarlicOS version: 1.4.9

Can you update garlicpress?
Sent from Outlook for iOShttps://aka.ms/o0ukef

From: devofreck @.>
Sent: Tuesday, August 15, 2023 4:00:21 AM
To: prosthetichead/GarlicPress @.>
Cc: prostheticHead @.>; Assign @.>
Subject: Re: [prosthetichead/GarlicPress] Updating Game Art memory error (Issue #81)
Assigned #81https://github.com/prosthetichead/GarlicPress/issues/81 to @prostheticheadhttps://github.com/prosthetichead.
—
Reply to this email directly, view it on GitHubhttps://github.com/prosthetichead/GarlicPress/issues/81#event-10090024764, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAOYKWLLRQCEKMJ3ZTUZ75TXVJVD3ANCNFSM6AAAAAA3QCAXAQ.
You are receiving this because you were assigned.Message ID: @.***>

Sorry I should wake up more before I reply! I will check this out after work tonight.

Can you paste the text from the error box into here also?

i am so sorry for making you think there was something wrong. I must have input my password incorrectly for screescraper.fr
I entered the credentials again, and its working as expected!  I will close out the issue

All good!!! It is interesting this had that error though. I'm thinking I need to add a password check.


User stuff cache

:eyeglasses: @ajacksified
related to https://github.com/reddit/reddit-mobile/pull/341

:fish:


blackbody fit for spectra generation

could you please add a blackbody fit to xspec fits? 👉🏻👈🏻

Partially related to issue #435, the general xspec fitting capability, but will probably as blackbody is so common I'll probably implement a specific blackbody function

The last couple of things to do with this issue:

Please could you put the lower limit of energy back to 0.3 keV - I know it'll be slightly annoying to have to change it to a lower value when you want to do the fits, but thats the default of the other models (and XCS' default) so I'd rather be consistent.
Remember to change the starting temperature to something higher - maybe 1keV as thats the default of the model?  Again I know its a shame you'll have to change it to lower values, but thats something it seems like you still need to play around with anyway.
Please could you change the name of start_temperature to start_temp, to be consistent with single_temp_apec's starting temperature parameter.

I was also thinking of getting you to modify powerlaw and blackbody to support being unabsorbed, but I know you want to pull this into master and say you've finished it, so I'll make it a separate issue and either you or I can do that later.


Instantiation prevents graceful shutdown.

When setting up a new Express server, simply instantiating the ByuJWT library prevents the server from terminating gracefully when using Ctrl+C; stopping the server has to be done forcefully. This is the code I'm using (we are using imports in our project):
import ByuJWT from 'byu-jwt'

const byuJwt = ByuJWT({ development: process.env.NODE_ENV === 'development' })

Even if I use require rather than import, the result is the same. You'll also notice I'm not registering any middleware; I'm only creating the byuJwt object.
The desired outcome is that this doesn't stop the server from shutting down gracefully. Currently, I am still investigating workarounds. I will comment again if I find one.

A found a work around by handling the SIGINT event and calling process.exit(0).

A found a work around by handling the SIGINT event and calling process.exit(0).

That's probably because of these lines:
  process.on('exit', () => shutdown('exit', data)) // app is closing
  process.on('SIGINT', () => shutdown('SIGINT', data)) // catches ctrl+c event
  process.on('SIGBREAK', () => shutdown('SIGBREAK', data)) // catches Windows ctrl+c event
  process.on('SIGUSR1', () => shutdown('SIGUSR1', data)) // catches "kill pid"
  process.on('SIGUSR2', () => shutdown('SIGUSR2', data)) // catches "kill pid"

I don't know the original motivation for doing things this way instead of exposing shutdown to users and expecting them to call it as needed.
If this behavior continues to be a pain point, we can change it in a future major release.

It's not a pain point because of the workaround. If it can get on the roadmap for the future, then that'd be fine.

@GaryGSC Is there a problem exiting without clearing timeouts? I think we can remove those lines and expose a clearCache function instead of the shutdown function.

https://github.com/byu-oit/byu-jwt-nodejs/pull/78


Add clipboard support to CLI

Integrates the clipboard-rs crate to enable direct clipboard image import functionality as an alternative input method alongside existing file path support. Implementation includes proper error handling, platform-specific clipboard access, and graceful feedback whenever both clipboard and file inputs are in the arguments. This enhancement significantly improves user experience by eliminating the need for manual file path input or image saving, streamlining the workflow for users who frequently process screenshots or web images.

Hello. Thanks for the contribution. This seems like a useful feature. Since it does add significant new dependencies which may not be needed or available on some systems I would like it to be behind a crate feature (clipboard). Also I would prefer to use arboard as the dependency to read the clipboard. I can't see anything wrong with clipboard-rs from skimming the code, but since there is a lot of platform-specific / unsafe code in there, I'd prefer to have something with as many eyeballs on it as possible.

I like arboard much more as a dependency as well, I'd also like to implement outputting to the clipboard as well in the future but that may be outside the scope of this PR


Bug: Transactions stuck "Pending" despite success

Describe the bug
Transactions in my Coinbase Wallet Chrome extension are stuck in "Pending" state even with 80+ block confirmations.
Steps

Deploy a transaction
Observe transaction in Chrome extension
Click on "View on Tx Explorer" to view the transaction on a block explorer
Observe that the transaction has been processed successfully with many block confirmations
Observe that Coinbase Wallet still shows the transaction in "Pending" state.

Expected behavior
The transaction state in the extension should be marked as Completed immediately upon transaction success or after a small number of block confirmations.
Version
3.0.8
Additional info
Incongruency is leading to a lack of trust between our users and dapp and we are considering dropping support for Coinbase Wallet until the issue is resolved.
Desktop
OS: Ubuntu 20.04 Focal Fossa
Browser: Chrome
Smartphone
No response

Also, for a company valued at nearly $100 billion, can you guys do a better job policing these Github issues. Issues left unanswered for months and some useless issues need to be deleted.

I would like to add that this issue persists across multiple chains and never resolves itself. After 5 hours and 2700+ block confirmations I am still seeing the issue.

I would like to add, which I believe to be aligned with this issue.
While using block-native onboard to connect to coinbase wallet, and being connected to a custom network
logs show a PollingBlockTrackerError where it cannot update to the latest block (undefined).
Where the dapp itself IS able to fetch the latest block data.

hi. sorry for the belated response. we are marking stale issues closed for now. please try our latest version 3.6.0. feel free to open a new one if needed. thanks.