Multiple no-data values

I have GRIB files from NOAA MRMS QPE, which can have the following values:
No-Data:
-1: missing value
-3: no coverage
Valid values: >= 0 (in mm)
I'm trying to describe this with STAC extensions, but are somewhat failing:

raster extension: Doesn't support multiple no-data values
classification extension: Doesn't support ranges (or greater than / lower than)
file extension (legacy version 1.0.0): It seems I need to fall back to this old version of the file extension to describe two no-data values (and specify the unit for the remaining values).

This is unfortunate, because using an old extension version that has no upgrade path seems unfortunate.
What do the extension authors think? How/where could we address this? @drwelby @pjhartzell @emmanuelmathot
For me it seems like allowing an array of no-data values in raster extension as an upgrade path from the old file extension like the best fit.

@drwelby That how I'd understand it, yes.
I'm not as critical regarding the no-data values array in raster. Raster should not be GeoTiff specific so if there's a format out there that can have multiple no-data values, then it should be covered by the extension. In the end, you need to make good decisions anyway and as such the metadata should just reflect what is in the file. And you also don't always to change the original data as STAC is often just an addition to what is already out there.
An alternative would indeed be to introduce somethline likes a "role(s)" string or array, where we could pre-define a no-data role.

And you also don't always to change the original data as STAC is often just an addition to what is already out there.

Yeah, agreed on this. There should be a way to describe (with STAC) the data in its native format (GRIB, HDF, etc).

Here's an example for the usecase above:
      "raster:bands": [
        {
          "spatial_resolution": 1000,
          "data_type": "float64",
          "unit": "mm",
          "classification:classes": [
            {
              "value": -1,
              "roles": ["nodata"],
              "description": "Missing value"
            },
            {
              "value": -3,
              "roles": ["nodata"],
              "description": "No coverage"
            },
            {
              "value": [0, null],
              "roles": ["data"],
              "description": "No coverage"
            }
          ]
        }
      ],

I'm not sure what other roles it could have. If it' just these two roles, it could also be a simple boolean flag "nodata": true instead of "roles": ["nodata"].

Since only nodata values are classified, maybe something like:
"nodata:classes": [
            {
              "value": -1,
              "description": "Missing value"
            },
            {
              "value": -3,
              "description": "No coverage"
            }

or
 "nodata": -3 #classic interpretation of no data coming into the system, ie fill
 "nodata:classes": [
            {
              "value": -1, # no data coming out of the system
              "description": "Missing value"
            }

Why not the more general-purpose approach? I don't see the benefit of your proposal.
Also, this is the really expressive way. I'd also be happy with a simple "nodata": [-1, -3] and then use classification:classes in addition.

I'd also be happy with a simple "nodata": [-1, -3] in raster and then use classification:classes in addition.

I like this option. It keeps nodata explicitly defined in the Raster Band Object.
      "raster:bands": [
        {
          "spatial_resolution": 1000,
          "data_type": "float64",
          "nodata": [-1, -3],
          "unit": "mm",
          "classification:classes": [
            {
              "value": -1,
              "description": "Missing value"
            },
            {
              "value": -3,
              "description": "No coverage"
            },
            {
              "value": [0, null], # >= 0
              "description": "Precipitation in mm"
            }
          ]
        }
      ],

Is [0, null] form standard for >= 0? Or would we use a "range object"?

This is inspired by the open ended ranges as done for the extents in collections. I'm open to any of those options though. And we don't necessarily need to have it now.

I really don't like calling a continuous data range a "class", but we keep running into range-type objects. So maybe we need a more generic value(s)-to-concepts extension.

I really don't like calling a continuous data range a "class"

Agreed. But it was a topic when the extension was being put together a few months ago. I think the general thought was to kick the topic down the road until a use-case presented itself. Seems like we are at that moment.
Other than preventing the "abuse" of classification with range values, what other use cases do you see for a generic values:mapping extension?

Other than preventing the "abuse" of classification with range values, what other use cases do you see for a generic values:mapping extension?

Multiple keys that map to one value is the first one that comes to mind, like [3, 7, 21]: "vegetation"

Returning to this, do we need something like "flag values", for lack of a better term?

Returning to this, do we need something like "flag values", for lack of a better term?

I unfortunately have not had the time to follow carefully all the discussion but isn't it the purpose of the classification extension?

The classification extension now has the nodata flag for the Class Object, so this seems to be the way forward.


Return a collection of forms from ExtractForm and variants

Consider whether this should be pageable

@ctstone , @kristapratico , @samvaity, @jeremymeng , @mayurid, @bojunehsu

Not quite sure how we can make this pageable if the API returns the full set of results at the same time.

@bojunehsu Would you ever support processing so many forms/pages in a single input document at once that you would want to send back the results in difference pages?

We recently bumped up the PDF page limit to 2000.  I agree that it will be really nice to support streaming processing for such situations in the future.  This would require significant changes to the REST API that APIM cannot easily support today.
That said, at the SDK interface level, it would be great if ExtractForm returns IEnumerable<> or equivalent instead of IList<> even if the underlying implementation today does not support paging.  This is assuming it is just as easy for developers to use a paged result as a list result.  (IEnumerable in .NET is good.  Not sure what is the Python/Java/JS equivalent.)


New package: MaterialPointSolver v0.2.0

Registering package: MaterialPointSolver
Repository: https://github.com/LandslideSIM/MaterialPointSolver.jl
Created by: @ZenanH
Version: v0.2.0
Commit: a2f64fb5a39f717a14606fdda4306286f1626e16
Reviewed by: @ZenanH
Reference: https://github.com/LandslideSIM/MaterialPointSolver.jl/issues/1#issuecomment-2296100386
Description: 🧮 High-performance Material Point Method (MPM) Solver in Julia.
Release notes:

## 🎉 Release v0.2.0

[noblock] Can you comment on any potential connection to https://github.com/JuliaRegistries/General/pull/113418#issuecomment-2297845482


Update to pick up latest omnibus and omnibus software

From the diff it looks like the only bump that affect us is likely openssl.  There is a small config change to pcre.

Wilson ad-hoc build: http://wilson.ci.chef.co/job/chef-server-12-trigger-ad_hoc/159/downstreambuildview/

@stevendanna Yeah, the intent of this update was to pull in the latest openssl.

Merging as @mmzyk is out today


router: add scheme and port redirect actions (#3060)

Description:
Add scheme_redirect and port_redirect to RedirectAction which replace the respective
URI components when applied.
scheme_redirect and https_redirect are mutually exclusive.
It is presumed that host_redirect won't include a ":" when port_redirect is used.
Risk Level: Medium
Testing: Unit tests
Docs Changes: Adding the scheme_redirect and port_redirect doc strings
Release Notes:
Fixes #Issue: #3060
Signed-off-by: Nikolay Nikolaev<EMAIL_ADDRESS>

@junr03 all comments fixed. Thanks for the review.

looks good to me, @zuercher can you take a look?


Added Rabs AP% passive, fixed Mpen% to percent, Mpenflat to flat, rpen% to percent instead of flat

Added dependencies in requirements.txt and extended docs with installation and running.
Item changes:

Rabadon's Deathcap was missing the increased ability power percent in passive stats object, so added to passive stats
Mpenflat is missing from Sorc shoes and Ludens Tempest
Mpen% missing from Voidstaff and Blighting Jewel
Apen% is now mapped to percent instead of flat on Lord Dominiks Regard, Last Whisper, and Serylda's Grudge
AS% for regular stats is now mapped to percent instead of flat eg. Trinity Force, Kraken Slayer, etc.

Most of this looks good. These are fine:

.gitignore
adding ability power percent
changing armor pen to percent
magic pen changes

There are a couple newlines deleted in lines 608 and 609 in pull_items_wiki.py that look like they should be there.
The attack speed should be flat, not percent.
For the requirements.txt, we don't need all the packages - only the ones that are actually used. This is what I don't like about pip freeze, it prints all the packages, not just the top-level ones that are needed. We also don't need the versions. If packages get bugfixed, I want those updates to be applied. The only real way to create a good requirements.txt is to look through all the imports and find the corresponding pip packages to install.

Should I add the newlines back into pull_items_wiki.py?
Attack speed on item regular stats are percent, while item passive stats are set to flat. This is to keep consistent with the model of attack speed being a percent stat and not a percent and flat stat across the entire Item model.
The requirements.txt is just the content that is needed to run the code. I first cloned and created a virtual environment (why I added environments in .gitignore) without any packages and added packages when it threw an exception while attempting to run. When packages get updated, you can update the required package while the virtual environment is activated.

Yeah, add the newlines back to pull_items_wiki.py. Even better, the autoformatter black with linelength=120 can be used to ensure the files are formatted properly.
You're going to have to give me some more info on the attack speed stuff. I put quite a lot of time into this and while I haven't been paying much attention lately, I doubt Riot made sweep changes to how they calculate and apply attack speed modifiers during the preseason changes. I may be wrong, but I need much more info before I can merge that change.
For the requirements.txt, remove the versions and only include the packages that are imported in this library, not packages that are dependencies of other packages (like certifi).

Added the newlines back to pull_items_wiki.py with the black autoformatter and linelength=120.
Looking at the attack speed formula, items, runes, and champion ability modifiers (eg. Fiora [E] Bladework) are on an additive basis (flat) and then multiplied back with champion's base attack speed. The general formula for attack speed:
AS = AS_BASE * (1 + (StatsGrowthFormula(as_per_level, champion_level) + items + runes + champion_modifiers) / 100)

Then the item regular stats for attack speed should stay flat instead of percent, and passive stats for attack speed should be flat instead of percent. I haven't made any changes, but I'm pretty sure that's what it's supposed to be.
I brought it up since the the template for Item Wiki data has the description of attack speed as percent, so my understanding would be percent instead of flat.

Fantastic! Thanks for putting all the time and effort into this work and this PR. It looks great. I think you are dead on with the attack speed, and it's looking good now. Anyone who uses it will have to create a custom formula for it as well.


Update "Password not recently used"

For password reset, the current and recently used passwords can be used.

@justinha
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

#please-close @ms-yuukih I will submit the change on my side and update the author info.


Taxes - Error when saving 0 tax amount and it is saved when hitting Save button again

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.66-2
Reproducible in staging?: Y
Reproducible in production?: Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:
Precondition:

Collect workspace has tax enabled.

Submit an expense with tax.
Go to transaction thread.
Click Tax amount.
Change the tax amount to 0.
Click Save.
Click Save again.

Expected Result:
App will save 0 as tax amount when saving it for the first time.
Actual Result:
App shows error when saving 0 tax amount for the first time, and it is saved when hitting Save button again.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/49659a6c-a029-483b-8b22-c4dbae0448fd

View all open jobs on GitHub

We think this issue might be related to the #collect project.

No update yet, I've been focused on other GHs.

Closing - I'm unable to replicate the experience and was able to save without an error


Any plans to make this workable on Google Colab? Thank you very much!

Here is the colab example.
Just install with pip.

Maybe I'm missing something, but I can't seem to get training to work in the notebook.
Is there a tutorial on the training process that I can follow ?

Yeah I got this error

Would also be great if there are tutorial on setting up the images and caption files on the cfg_file.yaml


Correct Framebuffer error.

See https://github.com/libgdx/libgdx/issues/3227
Fixes #3227

Thanks!


Possible but emitting duplicate events

It appears that the startup sequence allows for duplicate logs to be emitted if more than 655360 events are in the logs read at startup, and if the logs are read in an order which loads them newest to oldest.

+also add info on startup about state restoration

https://github.com/spyderbat/event-forwarder/pull/13


Added rust support

Following the existing python and c language implementations as references, rust support was set-up. Additions have also been made to extend some common tags which may find use elsewhere. For example,

CLion was added to jetbrains
A generic look loop {} was added to imperative
Additions to document comments for nested and bulk doc comments.

The rust.py diff is large enough that GitHub collapses it by default. It's too large to actually see what the real changes are, since it rearranges all the action implementations.

It's hard to review this PR b/c it's hard to distinguish genuine changes from code moving around. In future PRs please try to minimize the diff with the existing code.

Unfortunately, I develop this branch prior to the merging of the current rest code and without knowledge of it. I did move around lines in order to minimize some of the conflict, especially in the talon file, however I didn't want to compromise the flow of the implementations which are in the same order as the tag definitions in the talon file.

Please check for methods you may have deleted (eg. code_operator_address_of). Likewise for deleted list items.

Two of the three methods of this tag around pointers were implemented, with the third function which has the word dereference in its name, omitted. Because the concept of the referencing in rust is itself achieved with *, I neglected to implement this tag in preference of rust specific grammar. However, if it is preferred that that tag be used, I can add this back in.

I'm not sure whether separating out macros from functions is good. I'm not a Rust programmer, though. It seems like many macros are called like functions (eg. println!) and it might be easier not to have to remember a different command for invoking them, but there are also some special ways to invoke macros. Perhaps all macros should be included in the code_functions list as well? Your opinion?

I could perhaps see merit of including them in the general functions list, however macros are quite distinct from functions in reality. As they have their own internal grammar, it's not that they can be called differently, but they get to a fine how they are called. It is for this reason that I implemented it separately, and allows three different ways for calling a macro with the most common parentheses, those being parentheses themselves for function like macros, square brackets for those that emulate listing, like vec![], and braces, for those macros, like error_chain.

I'm a bit concerned about over-reliance on the details of editor auto-formatting, especially in the function-defining commands/actions like code_private_function, and others like code_state_implements. Can you double-check these? Generally we're ok assuming that tab auto-indents the current line if it needs more indentation but I'd like to avoid assuming too much more than that if possible.

I agree, I'm afraid this is my first talon pr, and am a bit naive to the best usage. I simply relied on things I had seen elsewhere.  where superiors solutions have been pointed out, I'll make the appropriate changes. I wasn't sure which of my implementation and the conflicting rust implementation that I had previously been unaware of,  was better, and so I tried to make as few possible changes to my own branch.

I've addressed everything I gave a thumb up, and left outstanding questions for things I wasn't sure about or where I didn't know community preference. Of the things that I said I would fix that I have not, that is the auto formatting of movements keys, I would like to know what is the best way to make multiple left movements while using edit. left.
Regarding all variants on if statements, I have left them as one liner statements. This is because it is possible to have a one line if statement whose value is assigned to a variable, and I didn't feel it was a major  problem for a person to have to hit enter when they need already to define the if clause on the same line.

@RyanRuane the below is not a blocking request, but fwiw I'd be inclined to go a step further wrt the macro / function thing, and just have a single command "funk" that you use for functions, square macros, curly macros etc, and then have separate sets that you can use to indicate things that want square or curly brackets.  Something like the following:
square_bracket_macros = {
    "vec!",
}

curly_bracket_macros = {
    "foo!",
    "bar!",
}

Then in code_insert_function, do something like
    def code_insert_function(text: str, selection: str):
        if text in square_bracket_macros:
            code_insert_function_or_macro(text, selection, '[', ']')
        elif text in curly_bracket_macros:
            code_insert_function_or_macro(text, selection, '{', '}')
        else:
            code_insert_function_or_macro(text, selection, '(', ')')

This approach presumes you'll always use the same style brackets with the same macro, but if so, it's one less thing to think about while coding.  You just always say "funk foo", and don't need to worry about function vs macro / bracket style / etc.  As @rntz mentions, the less you have to think about while voice coding the better

Have moved block and inner documents comments into rust specific space.
Have included all macros into the general functions and functions gui tags.
Now automatically deciding on whether to use brackets braces or parentheses with respect to given macros based on hardcoded designations.
Have corrected control flow statements formatting to avoid issues in ides such as pycharm which attempt to insert lines and chars automatically.


docs(DTO): small typos

Pull Request Checklist

[ ] New code has 100% test coverage
[ ] (If applicable) The prose documentation has been updated to reflect the changes introduced by this PR
[ ] (If applicable) The reference documentation has been updated to reflect the changes introduced by this PR

Description

Fix small typos

Close Issue(s)

@all-contributors add @aorith for docs


maplibre doesn't show map details

I have successfully installed maplibre on both android and ios devices using react native, however when I run the test on the device, the mapview only shows the country name, other than that, nothing else

can you send a picture of one of the mobiles?

This is because the style.json only has a layer for countries and no more.
For maps with more details you need to add more specifications to the style.json. For that you can either use an existing style.json or create one yourself with e.g. maputnik or maptiler. You can also use a local style.json.
You can find tons of existing styles on github :)

I have tried with this styleURL "https://demotiles.maplibre.org/styles/osm-bright-gl-style/style.json"
But it seems something is still broken. I get a blank view when zooming too much. This is an actual device but it also happens in my simulator.
Device video:
https://github.com/maplibre/maplibre-react-native/assets/24256469/13276867-4f66-4d6a-813a-3c5a473287c0

Hey, I made the styleURL without the curly bracket "{url...}".
I'm not sure if this changes anything, react native is sometimes strange.
The problem can also come from not using absolute points for your UI.
MapLiebre React Native only works when it is in the foreground (experience report).
<MapLibreGL.MapView
      logoEnabled={false}
      rotateEnabled={false}
      style={[tw`self-stretch flex-1`, style]}
      styleURL='https://demotiles.maplibre.org/style.json'
/>

@Anwenden001 Does it work for you with these styles?
 styleURL="https://demotiles.maplibre.org/styles/osm-bright-gl-style/style.json"

ohh i know what you mean!
That's bold!
the server only hosts a certain area in switzerland.
If you want to have the world you either have to host a server yourself or use providers like maptiler from where the game comes from.
"https://www.maptiler.com/"
But there are also many other providers you would have to look where you can get the data from.

Oh I see! Thank you!

Jumping in a bit late here, but yes, the demo tiles are just that: demo tiles. You would need to either use a provider of hosted map tiles such as Stadia Maps (disclosure: I'm a cofounder) MapTiler, Mapbox, etc., OR host your own tiles (check out https://github.com/maplibre/martin or https://github.com/onthegomap/planetiler).
I'm unfortunately not sure about your ArcGIS basemaps. MapLibre expects to have a MapLibre GL JSON style as input, and I don't know what format your ArcGIS map is in. If you're using a raster basemap layer from ArcGIS and want to render vector features on top, you'll need to create a style that includes a raster source and raster style layer.
You can debug this setup with Maputnik. Go to the editor, click "open" and select empty style. Then add your source under "data sources" and then add a style layer. Once you've confirmed it works in the browser, you can download the JSON style and host it somewhere or bundle with your app.
Hope this helps!


Upgrade the pvsadm version in periodic cleanup job

This PR is to upgrade the pvsadm version used by period cleanup jobs to purge the vms.
Upgrading to latest available release version v0.1-alpha.12
https://github.ibm.com/powercloud/container-dev/issues/1367

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Rajalakshmi-Girish
To complete the pull request process, please assign mkumatag after the PR has been reviewed.
You can assign the PR to them by writing /assign @mkumatag in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/cc @mkumatag

/lgtm

/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mkumatag, Rajalakshmi-Girish
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [mkumatag]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Rajalakshmi-Girish: Updated the job-config configmap in namespace prow at cluster default using the following files:

key cleanup-k8s-periodics.yaml using file config/jobs/periodic/housekeeping/cleanup-k8s-periodics.yaml

In response to this:

This PR is to upgrade the pvsadm version used by period cleanup jobs to purge the vms.
Upgrading to latest available release version v0.1-alpha.12
https://github.ibm.com/powercloud/container-dev/issues/1367

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


It is useless for  @SagaStart timeout,and it could not be compensated under @SagaStart

I don't know what SagaStart annotation timeout use, because the timeout is not work.
In the spring cloud demo,if hotel set a wrong address or port，the car service won't be compensated
 @SagaStart(timeout = 60000) //timeout no use,can not affect compensate
  @PostMapping("/booking/{name}/{rooms}/{cars}")
  public String order(@PathVariable String name,  @PathVariable Integer rooms, @PathVariable Integer cars) {

    template.postForEntity(
        carServiceUrl + "/order/{name}/{cars}",
        null, String.class, name, cars);

   //if set a wrong hotel address,car service compensated method not be triggered
    template.postForEntity(
        hotelServiceUrl + "/order/{name}/{rooms}",
        null, String.class, name, rooms);

     postBooking();

     return name + " booking " + rooms + " rooms and " + cars + " cars OK";}

  public void postBooking(@PathVariable String name,  @PathVariable Integer rooms, @PathVariable Integer cars) { }

Thanks for pointing that out, it looks like a bug. do you mind create a JIRA
Timeout is of the event scanner to check if we can get the SagaEnd event on time.

Already created ,thanks
https://issues.apache.org/jira/browse/SCB-739

FYI, I did a quick fix in the SCB-784 branch. Now alpha will get a TxAbortEvent in your test case.
You can find the use case here


Webpack

How to use it in webpack? Is there a npm module of echarts??

We put it on the bower

+1

@pissang  when using in bower, I found that I can not use it by module, I can only import the whole file echarts-all

echarts.js and all chart modules in the chart folder are also available. If you means including the modules in the src folder. Yes you can't use them after installing from bower because the missing of zrender


Change module name

As this package is published as @suldashi/lame, declaring the module name as lame gives a TS error when importing the module:
error TS2306: File<EMAIL_ADDRESS>is not a module.
import * as lame from '@suldashi/lame';

However, when the name is changed to @suldashi/lame, the error does not appear.

I’ll just assume this is unmaintained.


WASM support

Are you expecting/planning on including a WASM target in this codebase?

hey @tmc, WASM support isn't in our roadmap at the moment. The reason is there's quite a bit of limitations of WASM when it comes to networking. In the browser we don't have direct control of the networking layer so most of the webrtc stack in the Go SDK would not be applicable.
There is a way to map WebRTC APIs to RTCPeerConnection that browsers expose, but there will be quite a bit that will be lost in translation.
If you look at our JS client SDK, there are also a (small) bit of code that deals with browser-specific compatibility issues.


Migrations: Fix bug in complex column ordering code

I knew we'd eventually find weird corner case bugs in this code.
Fixes #14027

Linux failures are weird and not normal cosmos issue it seems.

Also fixes https://github.com/aspnet/EntityFrameworkCore/issues/17773 ?

@aspnet/efteam @aspnet/build Who broke the Linux build? Is this a known issue?
error NU5128: Some target frameworks declared in the dependencies group of the nuspec and the lib/ref folder do not have exact matches in the other location. Consult the list of actions below: [src/EFCore.Analyzers/EFCore.Analyzers.csproj]
error NU5128: - Add a dependency group for .NETStandard1.3 to the nuspec [src/EFCore.Analyzers/EFCore.Analyzers.csproj]
error NU5128: Some target frameworks declared in the dependencies group of the nuspec and the lib/ref folder do not have exact matches in the other location. Consult the list of actions below: [src/dotnet-ef/dotnet-ef.csproj]
error NU5128: - Add a dependency group for .NETCoreApp2.1 to the nuspec [src/dotnet-ef/dotnet-ef.csproj]

You hit this https://github.com/aspnet/EntityFrameworkCore/pull/18214
But it does not fail for me locally in WSL.

@wtgodbe thought you said the placeholder problems resulted only in warnings? If your placeholder removal is needed here, let's get it into the 'release/3.1' branch.

@bricelam - What did you do with Linux errors? I hit the same in #18271

I ignored them. They were clearly unrelated to my changes.
@wtgodbe Should we revert something while you investigate?


ivy-prescient breaks counsel-imenu

Got following error when tried to jump to a function definition with counsel-imenu:
Wrong type argument: listp, #("Variables: ivy-prescient-sort-commands" 0 1 (face compilation-info idx 1) 1 9 (face compilation-info))

How to reproduce:

go to ivy-prescient.el
M-x counsel-imenu
choose ivy-prescient-excluded-commands and hit enter

Tried both with ivy-prescient-retain-classic-highlighting set to nil and t, same issue.

Thanks for the report.


⚠ Bofrost.fr has degraded performance

In 5da9440, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1531 ms

Resolved: Bofrost.fr performance has improved in d2faaeb after 13 minutes.


Improve article 1705.08023

Exact location of issue
the superset marks seem to have been parsed as emails. Also, the "topic" type seems not to have been parsed (and left a stray mark on the page)
Example:

The supersets seem to be parsed as parts of the email https://ar5iv.org/html/1705.08023

I appreciate the red arrows!


Add tests to harden site functionality

We should set up tests for the different functions that the site depend on
This includes things located in the various lib files
Jest?

would love to see some cypress tests too covering basic functionality, ideally things that someone wouldn't need to remove when cloning the site to avoid someone having to clean out a bunch of tests before getting started
we could theoretically make a query to WordPress and test page / post creation this way

Get the posts
check hte homepage shows the first several posts based on posts per page
etc


Revise tutorial, add howtos

The tutorial takes into account the previous comments from @aryx on #70.
I also added some howtos. This is part is a bit rushed, but better than nothing. I didn't even re-read it or tested the code samples.
PR checklist:

[x] Purpose of the code is evident to future readers
[x] Tests are included or a PR comment includes a reproducible test plan
[x] Documentation is up-to-date
[x] A changelog entry was added to CHANGES.md for any user-facing change

Check out CONTRIBUTING.md for more details.

better, thx.


didTriggerLeftUtilityButtonWithIndex always getting index of zero

when using the - (void)swipeableTableViewCell:(SWTableViewCell *)cell didTriggerLeftUtilityButtonWithIndex:(NSInteger)index method, the index is always set to zero for every cell.

+1 - seeing the same thing - any updates here ?

+1 for this. It's either 0 or 1 for me. But the cell returned is correct

how to get the current cell inex


Add %jinx hint to Vere.

Resolves proposed UIP (unnumbered).
Adds a %jinx hint to Vere which permits a computation's timeout to be specified in Urbit time.
> ~>  %jinx.[~s4]  =|(i=@ |-(?:(=(10.000.000 i) i $(i +(i)))))
10.000.000

> ~>  %jinx.[~s3|  =|(i=@ |-(?:(=(10.000.000 i) i $(i +(i)))))
recover: dig: alrm
crud: %belt event failed
call: failed

May want to improve the error message to provide guidance on why the crash takes place.

notes from discussion today, for posterity:

mil_w was a functioning instance of this mechanism in u3m_soft for certain executions.
u3m_soft → u3m_soft_top which processes trace for you
signals send you to a top-level call
we probably have to do different things depending on whether we're in a virtual computation or not.
if this only ever happens in a road, and we only run u3m_soft from inside, then we can assume jump buffer is initialized and all we need to do is reset itimer_virtual in cm_signal_deep()
in cm_signal_deep() if mil_ != 0
so reset SIGVTALRM if we hit a new %jinx hint
fn in manage.c that nock.c can call when it encounters this hint that does the same thing that cm_signal_deep() does with the interval timer, but it also needs to check somehow
need some other global state to be able to see if we are prepared to handle this signal or not
vere/serf.c → ll. 26, 28 mil=@

inner timer if computation is done
outer timer to see if still valid

could build proof of concept that doesn't worry about safety and nesting
if you call setitimer multiple times you clobber the previous value, so to nest you need global state (timer value and monotonic timestamp), then when comp finishes then restore parent with adjusted time
proof of concept:  explore hint callbacks in nock.c, whitelist in bytecode compiler, then check dynamic hint processing

    case c3__jinx: {
      if (c3y == u3a_is_cat(*clu)) {
        c3_w mil_w = *clu;
        // call fn in u3m passing it mil_w TODO
      }
      u3z(*clu);
      *clu = u3_nul;
    } break;

one to set ITIMER like cm_signal_deep, one to cancel it using zeroed itervaltimerstruct

Extremely based functionality

From discussion ~2024.8.12, some thoughts on the interface:

subsecond syntax for @dr, like ~us1000? ~ms, ~us, ~ns (mibiseconds as powers of two?)
(ms 1000) support function?
use bloq size instead?

So write a UIP to implement subsecond timing syntax?
kiki-second, miki-second, niki-second, piki-second

Probably just stick with @dr and add ++msec, ++usec arms for now.

Also urbit/urbit#7067 for the parser.

Should the nested timer be measured against the nominal outer timer or against the actual time remaining?  I'm building the stack handler for the latter case, but worth thinking about.

::  should fail (timeout)
~>  %jinx.[~s0..8000]  =|(i=@ |-(?:(=(10.000.000 i) i $(i +(i)))))

::  should succeed (normal)
~>  %jinx.[~s1]  =|(i=@ |-(?:(=(10.000.000 i) i $(i +(i)))))

::  should succeed (nested properly)
~>  %jinx.[~s2..8000]  ~>  %jinx.[~s1]  =|(i=@ |-(?:(=(10.000.000 i) i $(i +(i)))))

::  should fail (nested improperly)
~>  %jinx.[~s1..8000]  ~>  %jinx.[~s5]  =|(i=@ |-(?:(=(10.000.000 i) i $(i +(i)))))

::  should fail (nested improperly and timeout)
~>  %jinx.[~s0..8000]  ~>  %jinx.[~s1]  =|(i=@ |-(?:(=(10.000.000 i) i $(i +(i)))))

%jinx squats on the SIGVTALRM signal, and when this is caught in manage.c it always assumes that the cause is an expired timer.  While this is correct for the current usage of Vere, I highlight it in case some future runtime engineer decides to utilize that signal in some other way.

@pkova @mopfel-winrux wants me to bug you on this for 3.2


Stormlord

Hmm, is the Shocking/Thundering supposed to work for thrown weapons? It looks like currently it would.

Hmm, is the Shocking/Thundering supposed to work for thrown weapons? It looks like currently it would.

Yes, according to my complete divine it should:
"Shock Weapon (Su): Any spear or javelin used by a stormlord of 2nd level or higher is treated as a shock weapon (dealing an extra 1d6 points of electricity damage). The weapon loses this ability 1 round after leaving the hand of the stormlord."
The same holds true for Thundering

Forgot to remove the python action toggle :(
About the colour of the aura. I used the color scheme of Shocking Grasp and Shock Weapons. Unsure if a more white colour scheme would be preferred?


nixos: networking does not restart successfully

This is config i use:
      # we already have router
      networking.firewall.enable = false;
      networking.bridges = {
        br0.interfaces = [ "eth0" ];
      };
      networking.useDHCP = false;
      networking.interfaces = {
        br0 = {
          ipAddress = "<IP_ADDRESS>";
          prefixLength = 24;
        };
      };
      networking.nameservers = ["<IP_ADDRESS>" "<IP_ADDRESS>" "<IP_ADDRESS>"];
      networking.defaultGateway = "<IP_ADDRESS>";

      services.udev.extraRules = ''
        ACTION=="add", SUBSYSTEM=="net", ATTR{address}=="bc:5f:f4:9b:46:d6", NAME="eth0"
      '';

and whenever networking restarts, server gets offline and i have to manually restart it.

(triage) still a problem?

I think it is worth assuming it is solved by now, and if not, they can open it up again :)


Fix: shellquote src path

Re-work former PR https://github.com/pypa/pipenv/pull/1204 to get the fix back into the new master branch using the shellquote helper, as mentioned in https://github.com/pypa/pipenv/pull/1361#issuecomment-360797751.

@bradsbrown thanks for the PR, good catch! Travis had a transient failure so I restarted the test but this does get passed down so it makes sense to be quoting this as it will be a path and it will be passed as an argument to a subprocess call

the py3.4 Travis test really doesn't like me for some reason, it appears.


PGAN 1024x1024 availability

Is there currently plans to make the CelebaHQ-1024 available on torch hub? The 512 model works pretty well, but I think 1024 would be great for generating even larger images.

Hello,
Good point I think I could add it. Let me check if it's still somewhere on the server.

That would be amazing; thanks!

It seems that I no longer have the 1024x1024 Celeba-HQ: I need to relaunch a new training session.
In the meantime you can use StyleGAN 1024x1024 on FlickrHQ.


Multi GPU really accelerates the training?

From the code for multi gpu training, you expand the number of samples by the number of gpus (https://github.com/microsoft/Semi-supervised-learning/blob/main/semilearn/core/utils/build.py#L158), but you never decrease the number of iterations, so multi gpu can not really accelearate the training.
From the other repo, they do divide the number of batch size by the number of gpus (https://github.com/LeeDoYup/FixMatch-pytorch/blob/main/train.py#L155), do you have any idea?

We intend to fix the total number of iterations as a protocol for semi-supervised learning.

Have you finished the multi-gpu setup? @Hhhhhhao

Have you finished the multi-gpu setup? @Hhhhhhao

I have the same concern. It doesn't seems.


Camera for iOS is not working in webvieww

Hello,
I am using lastest version of webview and want to access iOS camera from within webview. Everything works great on Safari (e.g. page: https://davidwalsh.name/demo/camera.php), however, nothing happens inside webview. I granted needed permissions like camera or audio, however, it still does not work.
Any ideas?

@lukaskai Have you found the solution?

That's weird, cause I definitely can use camera for input fields. 🙁

@Titozzz
Maybe you can give some example how it works?
Add permissions to Info.plist (NSMicrophoneUsageDescription, NSCameraUsageDescription), ask permissions after load webview, but nothing
https://github.com/react-native-community/react-native-webview/issues/1045

@Titozzz I believe react native webview is blocking permission alert, even though you give permission from the app, I found a work around using the lib https://github.com/proyecto26/react-native-inappbrowser, when doesn't block the alert when you view that above web page you will get the camera permission alert which is blocked in react native webview.

@lukaskai did you find solution ?

@lukaskai did you solve this problem ?

Hey @lukaskai any update on this issue?

no solution yet? thanks

Having this issue too on ios, did you find any solutions ?


Expression math to calculate new values

Is your feature request related to a problem? Please describe.
use the math expression function to calculate new values obtained from zabbix to display

Sorry to reply on a seems closed ticket. But this is not working
Adding any expression turns the graphs blank. no matter if hidden or not.
Tried on grafana 8.x and 9.x.


Any way to get old object data on modification notification?

Is there any way to get the old object data on a modification notification? I am really not wanting to cache all the old data so I can tell how it changed.
My specific use-case is determining if a modification notification makes a element move from one uitableview section to another.

Hi @asjesset, we're tracking adding this functionality as #3520. We're also tracking exposing moves more explicitly in #3571.
Alternatively, you could use KVO if the collection you're observing is an RLMArray or a List, which would provide the old/new objects. See our docs on KVO for more information.
I'm closing this as a duplicate of #3520 & #3571. Please subscribe to those issues for any updates in this area.


TerraformTaskV3 support for workspace?

In trying to help with this question, I noticed in the V3 README that the word workspace does not exist.  Does that mean tasks like
          - task: ms-devlabs.custom-terraform-tasks.custom-terraform-release-task.TerraformTaskV3@3
            inputs:
              provider: aws
              command: custom
              customCommand: 'workspace select devstage'

should expect to see an exit code 127 (job's command can not be found or executed)?

Hi,
customCommand field only takes one keyword. in this case it would be 'worksapace'
'select devstage' sholud be in the commandOptions


Create glossary for hpa

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
For 1.9 Features: set Milestone to 1.9 and Base Branch to release-1.9
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
NOTE: Please check the “Allow edits from maintainers” box (see image below) to
allow reviewers to fix problems on your patch and speed up the review process.
Please delete this note before submitting the pull request.
NOTE: After opening the PR, please un-check and re-check the "Allow edits from maintainers" box. This is a temporary workaround to address a known issue with GitHub.

This change is

Dupe in https://github.com/kubernetes/website/pull/6585.

Hi @zhangxiaoyu-zidif, thank you for submitting this to the Kubernetes docs, and taking the time to do this during KubeCon! There was a bit of duplication that happened here, so we'll have to close this PR, but we really appreciate your help. Please let us know if you have any other questions! :)


Support vendir's githubRelease

Describe the problem/challenge you have
vendir does support githubRelease to fetch artifacts
Describe the solution you'd like
kapp-controller support for same feature as vendir to fetch githubRelease

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

Could you elaborate on your exact use case?
Without context however, it feels like it would be recommended to wrap a release into a Package in most cases 🤔
Some context would also help us figure out if there are other folks who are looking for something like this as well.

I'm trying to migrate vendir's/ytt/kbld/kapp done out of cluster into a kapp-controller's App to run it within cluster. I'm currently using vendir's githubRelease to fetch release artifacts, and if I need to use git instead I need to alter how I produce artifacts in the source repository as the release files are not available in the git repository as it is. It'll be ideal to fully support vendir's capabilities so that it's easy to move imperative provisioning to declarative in cluster GitOps like via Apps.


[builder] improve float precision reduction on normalized X layer

change precision reduction method to utilize the same method as embeddings, ie., reduce by rounding

Resolved by #884


Fix Vulkan validation layer include path

Depending on the version of the vulkan validationlayers header package,
the include path is different. Old versions put the headers into the
vulkan/ subdirectory, laters ones don't use that sub-directory.
Since we have to work with both versions, this commit adds a cmake
find-package detection script which adds a cmake target
Vulkan::ValidationLayers which conveys for the proper include path.
The vk_layer_dispatch_table.h header is not self-contained and needs a
special inclusion order. That's why this commit als adds some
clang-format descriptors which avoid reordering of the vulkan layers.
Ideally this will be fixed upstream.

Thanks. I will quickly test this on a Windows workstation - just to be sure - and then merge it.

Seems to work.


Suggest using less prescriptive paths in favor of link relations

In some places, the spec seems to use RESTful resource links and relations to allow navigation and seems to explicitly support those relations potentially crossing server boundaries, allowing more flexibility in API implementation and evolution.
In other places, the spec prescribes specific routes and operations relative to the API root (actually, language-wise, it's not 100% clear that it isn't relative to the server root, but it's my understanding that's not the intention).
Example:

The API SHALL support the HTTP GET operation at the path /collections/{coverageid}/coverage.

As written, client developers would would be crazy to not concatenate paths into a server's resources; it's easier to write such a client and requires fewer network requests for interrogation.  The result is that many of the benefits of link relations are lost.
I have three examples where this causes real challenges for us and the potential evolution of the spec:

We have metadata catalogs that could expose a features api.  They live on different endpoints from our data services that would provide coverages.  We further have coverage services that could benefit from being on different web endpoints depending on the collection / request.  As written (See spec: Table 2. Coverage API Paths) we will need to find a way to collapse all of these URLs, rather than simply being able to use link relations to direct clients to the proper resource.
(As discussed in the Bethesda sprint with some consensus) We have a notion of groups of collections, so essentially a parent/child relationship.  This could be accommodated with the addition of a single relation, "child", to the spec except that the URL structure definition precludes the possibility of having, say, /collections/{parentId}/{childId}(/{childId})*/coverage.  My concern would be that this belies a lack of expressiveness in the spec that will require frequent amendments in the future.
The term "collections" clashes with a domain concept for our (NASA EOSDIS) data.  We have usability concerns in providing an endpoint which will need to specify IDs for both collections in the URL, where the ID that is actually labeled "/collections" differs from what we refer to as a "collection" everywhere else.

Note that if we did switch some of the prescriptive URL paths to obey link relations, this wouldn't prevent a user of NASA EOSDIS data or ESA data or other data from constructing URLs in a convenient way for their known server endpoints, it would only impact general-purpose clients / libraries.  What I'm picturing here is a client specification, where we define how a conformant client needs to interpret information like link relations (actually, I think this is a good idea regardless so maybe a separate issue?).

I believe in the latest version, e.g. https://github.com/opengeospatial/ogcapi-coverages/blob/master/standard/clause_6_overview.adoc, the standard has been updated to favor link relations rather than prescriptive paths.
(NOTE: there is an incomplete sentence there "It is recommended, but not required, that these paths" ...)

2021-02-10 SWG: Chuck will review to fix the missing text, and make sure it is in line with the approach agreed in Common.

@jerstlouis This Clause 6 section no longer looks anything like Common.  A complete rewrite is needed.

SWG 2021-08-11: We reviewed the status of the specifications and agree now that the path portions defined after /coverage should actually be fixed, in addition to having well defined link relation types. The path portions beore /collections are up to the implementations. The specifications currently seem to be correct in that regard but should be verified.
The README.md should be updated to avoid hinting otherwise.
The paths which are fixed are:
.../collections/{collectionId}/coverage
.../collections/{collectionId}/coverage/rangetype
.../collections/{collectionId}/coverage/rangeset
.../collections/{collectionId}/coverage/domainset
.../collections/{collectionId}/coverage/metadata

@cmheazel will also review the language to be in sync with Common.
This issue will be closed when all this is reviewed and addressed.


Time step config

Adds a config option to use p.setTimeStep (off by default)
I found this while looking for a way to slow down the simulation and watch movement more closely. However, since I don't have a very deep understanding of pybullet yet, I don't know if it messes with simulation outside of speed, so I left it off by default.
0.001 is a good starting point if anyone is looking to test it.

Just looked at the failed check- log just says "Failed" but also that all packages built. PR builds & runs sim correctly on my local machine. Does anyone have an idea of what the issue is?

Stale; closing


Caution: Old peerswap CLN plugin processes don't always die

$ ps aux |grep peerswap
cln       729129  0.0  0.0 1173572  868 ?        Sl   Mar17   0:03 /home/cln/src/peerswap/peerswap
cln       729164  0.0  0.0 1173572 1568 ?        Sl   Mar17   0:03 /home/cln/src/peerswap/peerswap
cln      1261763  0.0  0.0 1173888  308 ?        Sl   Apr10   0:02 /home/cln/src/peerswap/peerswap
cln      1261796  0.0  0.0 1247108  824 ?        Sl   Apr10   0:02 /home/cln/src/peerswap/peerswap
cln      2233765  0.4  0.1 1401056 27684 ?       Sl   May23  20:01 /home/cln/src/peerswap/peerswap
cln      2723044  0.0  0.0 1247112  268 ?        Sl    2022   0:07 /home/cln/src/peerswap/peerswap-plugin

The above example is a server that hadn't rebooted in 1 year. CLN was upgraded a few times since then. lightningd the parent process launches peerswap (or older name peerswap-plugin). The child process is supposed to die when the parent dies but in many cases it didn't.
Maybe this isn't a big deal except the older peerswap processes might still have open file handles to the database and unexpected behavior could ensue? Or maybe not since their stdio with their long-dead parent process is gone maybe they're deadlocked. Since those older peerswap binaries are gone I'm unable to gdb backtrace to see where they are stuck. In any case zombie plugins are unable to log via stdio.
Is there a golang signal handler that can act in the event the parent process dies? Would it be safer to use such a thing to ensure it actually does die?

Not sure how helpful this is but I took the opportunity to run strace on a zombie plugin instance.
$ strace -p 37303
strace: Process 37303 attached
futex(0x15a3b08, FUTEX_WAIT_PRIVATE, 0, NULL

@nepet found out a way to reproduce the zombie processes by starting lightningd without a --log path. Also, I found if you start lightningd without bitcoind running, you can create zombie peerswap processes when lightningd crashes.
For some reason, gdb can't attach quickly enough to the peerswap process when this happens if using something like `lightningd && gdb /path/to/peerswap/binary -p $(pgrep peerswap) but I found a short script that works:
#!/bin/sh
progstr=$1
progpid=`pgrep -o $progstr`
while [ "$progpid" = "" ]; do
  progpid=`pgrep -o $progstr`
done
gdb -ex continue -p $progpid

Running this script before starting lightningd, I was able to attach to a zombie peerswap process and get a backtrace, although I don't know how useful it is.
runtime.futex () at /usr/lib/go-1.19/src/runtime/sys_linux_arm64.s:666
666             SVC
(gdb) backtrace
#0  runtime.futex () at /usr/lib/go-1.19/src/runtime/sys_linux_arm64.s:666
#1  0x00000000004352ac in runtime.futexsleep (addr=<optimized out>, val=128, ns=0) at /usr/lib/go-1.19/src/runtime/os_linux.go:69
#2  0x000000000040ebf0 in runtime.notesleep (n=0x15a3ac8 <runtime.m0+328>) at /usr/lib/go-1.19/src/runtime/lock_futex.go:160
#3  0x000000000043fa44 in runtime.mPark () at /usr/lib/go-1.19/src/runtime/proc.go:1457
#4  runtime.stopm () at /usr/lib/go-1.19/src/runtime/proc.go:2247
#5  0x00000000004412bc in runtime.findRunnable (gp=<optimized out>, inheritTime=<optimized out>, tryWakeP=<optimized out>)
    at /usr/lib/go-1.19/src/runtime/proc.go:2874
#6  0x00000000004422d8 in runtime.schedule () at /usr/lib/go-1.19/src/runtime/proc.go:3214
#7  0x000000000044329c in runtime.goexit0 (gp=0x4000227a00) at /usr/lib/go-1.19/src/runtime/proc.go:3540
#8  0x0000000000466534 in runtime.mcall () at /usr/lib/go-1.19/src/runtime/asm_arm64.s:192
#9  0x0000000000000000 in ?? ()
Backtrace stopped: previous frame identical to this frame (corrupt stack?)

$ ps aux |grep peer
btc       148296  0.0  0.0 1173016    0 pts/1    Sl   Aug03   0:00 /home/btc/src/peerswap/peerswap
btc      3776066  0.0  0.0 1246744 1148 ?        Sl   Jul16   0:01 /home/btc/src/peerswap/peerswap
btc      3776123  0.0  0.0 1099536 1256 ?        Sl   Jul16   0:01 /home/btc/src/peerswap/peerswap
btc      4092778  0.0  0.0 1246744 1212 ?        Sl   Jul27   0:00 /home/btc/src/peerswap/peerswap
btc      4092822  0.0  0.0 1172500    4 ?        Sl   Jul27   0:00 /home/btc/src/peerswap/peerswap
btc      4092868  0.0  0.0 1173012    0 pts/1    Sl   Jul27   0:00 /home/btc/src/peerswap/peerswap

I don't know how this is happening.


Support compiling with other platforms

Summary
When using this plugin in a project which targets an environment other than Android or iOS it can't be compiled because the file datetimepicker has no default entry.
Test Plan
npm install --save git+https://github.com/aikewoody/react-native-datetimepicker.git#feature-support-all-platforms
# Install react-native-web
# If you're inside an Expo project: https://docs.expo.io/versions/v33.0.0/introduction/running-in-the-browser/
expo install react-native-web react-dom
expo start --web

What's required for testing (prerequisites)?
An Expo project with the react-native-community/react-native-datetimepicker plugin installed.
What are the steps to reproduce (after prerequisites)?
npm install --save @react-native-community/datetimepicker@latest
# Install react-native-web
# If you're inside an Expo project: https://docs.expo.io/versions/v33.0.0/introduction/running-in-the-browser/
expo install react-native-web react-dom
expo start --web

The following error appears:
web  Failed to compile.
<EMAIL_ADDRESS>Module not found: Can't resolve './datetimepicker' in '/Users/.../node_modules/@react-native-community/datetimepicker/src'

Compatibility

OS
Implemented

iOS
✅

Android
✅

Web
✅

macOS
✅

Windows
✅

Checklist

[ x] I have tested this on a device and a simulator
[ ] I added the documentation in README.md
[ ] I mentioned this change in CHANGELOG.md
[ ] I updated the typed files (TS and Flow)
[ ] I added a sample use of the API in the example project (example/App.js)

expo start -- web, I also encounter this problem, how to solve it

@EvanBacon Thanks for your insight, I really like your solution by warning the user about it.

Thank you both!


Respect ISTIO_ENVOY_LOCAL in VM image builds

Currently it is ignored, which may lead to unexpected behavior

@howardjohn , can you please assign this issue to me?

not stale


Researcher maintainance

This is a pretty large but non-breaking maintenance PR, mostly related to the API of the Research App. The primary changes:

I've removed all fixtures files and package dependencies. The files have become a part of a new repo, integration, that's unrelated to the frontend apps.
I've renamed the env vars SEARCH_PLATFORM_ELASTIC_ENDPOINT_URL, SEARCH_PLATFORM_SPARQL_ENDPOINT_URL and KG_SPARQL_ENDPOINT_URL. The new names, SEARCH_ENDPOINT_URL and SPARQL_ENDPOINT_URL, are a bit simpler.
The env vars point to new URLs. These are the URLs of the new 'knowledge graph' and 'search graph' that Kinsuk and I have been working on.
I've updated the files in the apps/researcher/src/lib/api folder to reflect the data changes that have been made to the new knowledge graph and search graph.

Note: the new knowledge graph and search graph on production hold much more data than the current graphs (e.g. 471,274 objects instead of 52,267, due to Valentin's new data deliveries for NMVW). Some data issues could arise because of this (e.g. I've already seen some invalid dates of creation) that need to be fixed in the ETL.

Thanks!


upgrade Ray to 1.4.0

This looks a bit messy now.  But anyway we'll modify some of them in the remove-py4j PR, thus I'll fix it in that PR.

can you also describe what was changed from Ray 1.3 -> 1.4 in the PR description that requires all these changes, and what will be fixed in your remove-py4j PR? Won't that PR require Ray nightly build?

Ray 1.4.0 changed the signature of setJvmOptions.This function is used when java actor is created. It originally takes a String, now it takes a List[String]. When I mean messy, I mean we used to use different data structure in scala and use a mkString to concatenate them, but now we have to convert them to java.util.List using different APIs. So it just looks "messy", but no problem in function.
In the remove-py4j PR, we will need to use another way to pass options to AppMaster. I'll try to make it more tidy. runtime_env is under development, users might need to use ray nightly depending on their usage.

Thanks! LGTM.


CAMEL-10756: Mina2 Producer "hang" until timeout if the response message could not be decoded

Can you avoid using the Java 8 API so this PR can be backported to branches that still support Java 1.7, eg 2.17.x branch.

Ok i have updated the code to remain backward compatible.
In addition i slightly improve the exceptionCaught and skip closing session in case of an IOException, since this kind of error is handled by mina itself (@see  org.apache.mina.core.service.IoHandlerAdapter.exceptionCaught)


Fixes cloud firstore security

Fixed:

Users who own decks and teachers can delete decks again

Works well, decks are now deleted again


contact us: send email to info address

the form shall remain, XLS entry and then forward that data to an email info@sazerac

open points:

get sazerac some adobe io runtime, 0$ amendment
change the sendmail action to send the email to "Sazerac Consumer Affairs<EMAIL_ADDRESS>instead as of now, where it is sent to the email address entered in the form to avoid spamming sazerac
change the sendmail action to check for referrer with the proper domain, currently just must contain 99brandparty
setup sazeracs runtime io and change the form.js to send it to the new url


Update route.ts to deploy on cloudflare

I deployed this application on Cloudflare and an error occured, it need to change into edge and can fix the error

yeah this need edge instead of nodejs
also runtime edge can be used tauri app updater for client

Vercel does not allow request forwarding at the edge runtime, which would break compatibility on the Vercel platform.

Vercel does not allow request forwarding at the edge runtime, which would break compatibility on the Vercel platform.

i have depoyed on vercel with edge runtime and everything works well

wait what, how that can break compatibility.

wait what, how that can break compatibility.

The author privately replied to me, saying that CORS is used for WebDAV synchronization, and is meant to lift cross-origin restrictions, therefore it cannot be changed to 'edge'.

Bruh,rip then runtime 'nodejs' cant do anything.
unlike runtime 'edge', example can build own browser in tauri app 🤣


chore: bdd-sbhg-1585219428 to 0.0.1

chore: Promote bdd-sbhg-1585219428 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


mac: "flutter start" doesn't say how to obtain "ideviceinstaller" or "ios-deploy" if you don't have homebrew.

This is a problem because we don't claim homebrew to be a dependency.

https://github.com/flutter/flutter/issues/1907 would make this moot.

What is flutter start?  Is this still a valid issue?

This is a problem because we don't claim homebrew to be a dependency.

No, it's an problem because Homebrew is notorious for breaking things (in fact, you even have few issues open in this project where homebrew is to blame). And there are plenty of people avoiding homebrew for that reason. It's fine if you installing things you're developing or if it is a pre-release product (as you would expect some hiccups at this stage), but it's annoying at best if you only using this project to develop something of your own.
If you absolutely must support homebrew, you should still provide alternative ways to install required tools using alternative methods such as MacPorts or just manually installing these tools (even if just as an instruction). I also think Nodejs/npm or SDKMAN could be even better alternatives as they were designed to support multiple environments on the same machine in a non-conflicting way, so they don't conflict with other package managers one might use. But the main point is - keep the choice open. I, for one, like flutter project. But if the only way to use it requires wrecking my dev environment (including all other projects I work on). Then it is simply not an option.

I don't believe we have any reason to attach ourselves to homebrew other than it was the easiest way we found to install these dependencies.
If you or others wish to suggest alternative ways to get these dependencies, or write PRs to flutter_tools to teach "doctor" how to help others get them, that'd be awesome:
https://github.com/flutter/flutter/tree/master/packages/flutter_tools

We need to revist shipping these tools with our default artifacts. The homebrew installation process is bad, especially with all the work arounds need to support the latest of each tool.

Closed in 36093.


Bug in examples/04-computed-with-setter

Hi @HuckRidgeSW
Thank you for this Vue.js wrap with GopherJS.
I'm diving in hvue and found an error message in example  04-computed-with-setter :
the "Foo bar" text do not get updated, and I got the following message in console :
Uncaught Error: Unknown data slot set: fullName
    at $callDeferred (04-computed-with-setter.js:1412)
    at $panic (04-computed-with-setter.js:1451)
    at Object.$packages.github.com/huckridgesw/hvue.VM.ptr.Set (vm.go:322)
    at Object.main [as $blk] (main.go:43)
    at Object.$init [as $blk] (04-computed-with-setter.js:11835)

... comming for the following line (I guess) :
	vm.Set("fullName", "First Middle Last")

after a few tries and error : works fine when using
vm.Object.Set("fullName", "First Middle Last")

Good catch!  Thanks for the bug report.  This is fixed; 04-computed-with-setter now no longer panics and works as-is with vm.Set instead of vm.Object.Set.


Schema Type Inference

Hi!
First of all, thank you for your work developing this type generator! It makes it possible to catch errors across a large codebase much easier, especially in CI/CD workflows.
The documentation mentions some examples where you can 'cast' the provided variables to the desired type. This is reflected upstream in the typescript definitions. In my experienced with a rust based type-generator, I found it is possible within UPDATE, CREATE and other statements, to infer the type at compile time.
For example if I did UPDATE $user SET email = $email, it can infer the required type of email as a string, because the field email is defined as a string in the schema.
This would reduce a lot of boilerplate while maintaining the same if not a higher safety margin on static typing.
Let me know your thoughts, and thanks again for your work on this project.

Yes, that feature is definitely on the roadmap as noted in the README.md and definitely something we want to work on, as it is very important.
We'll try to prioritize it soon.

FYI, still have plans to work on this at some point soon, because it would be a huge win for reducing boilerplate, but most importantly, it actually prevents developer mistyping, because the developer could actually specify an invalid type, and currently that is not checked


port to Linux x86

fix the following function

parseUlimitOpt
FNV Hash

build succeede

in x86 userland but kernel is x86-64, MACHTYPE indicates x86-64 due to uname

platform detection of test directive should support x86

fix platform detection/MATCHTYPE definition in commit e42c6984e7871d5d7e5ca14167f95228363e2fa1


Technical blog post ideas

Here are some technical (not business, not cost) questions we may want to answer in our next technical blog post(s). Some of them are inspired by questions from https://news.ycombinator.com/item?id=31753345
Features

HTTP API [instead of Postgres] for use with Fastly Compute@Edge/Cloudfare Workers.

Attaching PostgREST in front of it.

Can I use %extensionname% with Neon? How do you handle security?

Architectural

How do I self-host? Can I use something instead of S3?
What is the consistency model?
Who is the cache for what?

Performance

What happens when we scale the compute up/down: memory cache works, shared memory, extensions state.
Do we have read scaling?
Do we have write scaling?
What is the cold start time? How do purged memory caches impact performance?
How are network-based Safekeepers faster than network-based EBS?
How is non-latency-efficient S3 any good for a database?
What kind of load is (un)suitable for Neon?
How does Neon compare to a local Postgres? Probably expected answer: worse, because it needs networks and coordination.
How does Neon compare to a multi-machine Postgres? Probably expected answer: better, here is our magic sauce, here are numbers.

Another one: a short post about our troubles with SNI and older clients.

Another: Product dogfooding. How we use neon free tier projects to power the test result database and some business analytics. Two big points to deliver:

Clear up misconceptions about when a serverless database (especially a developer preview database) is a good choice. This could be a separate topic on its own. Explain that it's not about whether it's a hobby project or a production service. Serious companies have real needs for auxiliary serverless databases, but instead they settle with storing json in s3 because there's no better option.
Demonstrate how easy and cheap it was to set up and maintain. Maybe show integration with postgrest

@LizardWizzard

Few others from twitter.
One:

I’d like to know if they do further optimisations that reduce effort of rebuilding state ... but the docs are rather slim on that.

Two:

Yeah, also how they avoid stale reads from those page servers. All a bit blurry right now, but I'll keep an eye on it, it's definitely an interesting development.

I would be interested in how to best leverage neon from AWS lambda. AWS missing a true relational serverless database with scale to zero and neon could nicely fill this gap.


Set up cron script to refresh images

It should pull new images regularly. Probably need to tag images a bit differently, so might as well fix that in the same breath.


Definitely didn't get carried away...

now uses slash commands
timing system hopefully a little more consistent.
now stores targeted channelID in the .env file
only drank two cups of tea while working on this
now stores video links as a Dict because Python Dicts are cool
got a little carried away with the README.md file but don't tell Joel that...

closed because I forgot to yeet in the /about slash command


splinter 0.8.0 throws NotImplementedError for browser.status_code.code

splinter version 0.8.0
python version 2.7.6
When using a check for 200 on a successful page access:

if browser.status_code.code == 200:

I am seeing, this worked in version 0.7.7:

Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python2.7/dist-packages/splinter/driver/webdriver/init.py", line 182, in status_code
raise NotImplementedError
NotImplementedError

Repro steps:
`
ubuntu@ip-xxxxxxx:~$ python
Python 2.7.6 (default, Oct 26 2016, 20:30:19)
[GCC 4.8.4] on linux2
Type "help", "copyright", "credits" or "license" for more information.

from splinter import Browser
browser = Browser('firefox')
browser.driver.set_window_size(640, 480)
browser.visit("http://www.google.com")
browser.status_code.code
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python2.7/dist-packages/splinter/driver/webdriver/init.py", line 182, in status_code
raise NotImplementedError
NotImplementedError
browser.status_code
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python2.7/dist-packages/splinter/driver/webdriver/init.py", line 182, in status_code
raise NotImplementedError
NotImplementedError
`

hi @smhall05 this is not a bug this is a behaviour.
some drivers does not return the status code after visit a page. Before 0.8.0 the splinter always returned 200 in these cases. But this is a false positive.
In 0.8.0 we changed this behaviour. Instead to do a fake return splinter raises the proper error showing that this feature was not implemented.

@andrewsmedina What is the best way then to confirm I have accessed a page successfully?


node.js istio -  Cannot read property '0' of undefined

Please answer these questions before submitting a bug report.
What version of OpenTelemetry are you using?
0.9.0
What version of Node are you using?
12.8.3
What did you do?
If possible, provide a recipe for reproducing the error.
What did you expect to see?
What did you see instead?
/home/node/app/node_modules/@opentelemetry/core/build/src/common/time.js:94
let seconds = endTime[0] - startTime[0];
^
TypeError: Cannot read property '0' of undefined
at hrTimeDuration (/home/node/app/node_modules/@opentelemetry/core/build/src/common/time.js:94:41)
Additional context
Add any other context about the problem here.
ibm cloud managed istio 1.6.8, ibm kube cluster 1.17

There is not a lot of information here to try to reproduce this bug. That time code is called all the time, rarely changes, and is very well tested so I wouldn't expect it to be broken.

@dyladan are their any guides for implementing open-telemetry in an istio environment? im thinking we have an integration issue since istio is creating and forwarding x-b3 headers which open telemetry is not expecting.


Support finding configuration information from other platforms or config structures

Right now, CloudConfiguration relies very heavily on the conventions of VCAP_SERVICES. We should allow the configuration information to come from not only a path inside VCAP_SERVICES, and instead multiple paths that could be compatible with other platforms like Heroku, Amazon AWS, and our Docker deployment.
We should still support loading files with data that has been the result of something like this:
cf env MyApp > config.json

But often times, the user will want something even easier, like:
Some examples:
I think that a typical config.yml used for a Docker deployment could look like:
- port: 8080
- services:
    - postgresql
        - username: robert
        - password: pa55w0rd
        - hostname: postgresql.bluemix.net

Heroku I believe encourages using a web service that serves JSON to get the config variables. Therefore, we can just use your URLSession-based Configuration loader, @youming-lin 👍
https://devcenter.heroku.com/articles/platform-api-reference#config-vars


External interrupts (example, question)

I wrote a simple example of GPIO-driven interrupt handling, available here. Compile with cargo build --example gpio_interrupt --features "rt stm32f303xc" On the F3 Discovery board it toggles an led when you push the user button.
I'm curious if anyone's working on–or has thoughts about–adding better support for interrupts. The gpio.rs in stm32f4xx-hal has some nice methods based on the ExtiPin trait, making it easy to set up interrupts without direct register access. For example:
let gpioc = dp.GPIOC.split();
let mut board_btn = gpioc.pc13.into_floating_input();
board_btn.make_interrupt_source(&mut dp.SYSCFG);
board_btn.enable_interrupt(&mut dp.EXTI);
board_btn.trigger_on_edge(&mut dp.EXTI, Edge::RISING);

It would be nice to get the f3/f4 HALs into closer alignment but I wanted to gauge others' thoughts before jumping in.

I'm all in for being more aligned with the f4 HAL as these chips are similar in a lot of ways anyway.
I've had no time to do this myself, so I'm happy for any PR for this :)
Seems quite useful. Using interrupts myself in a project of mine, I needed some time to configure them right. These methods would be the process much easier and more discoverable.

I'm looking forward to this eventually being fixed. I don't understand low-level/unsafe code enough to impl myself confidently, but jk - thank you for the detailed, commented example.

Have you had any luck configuring timer-based interrupts? Like a delay, but using a low-power mode? I suspect it'd be similar to your GPIO interrupt example, but use a timer to trigger the wakeup. Is this still something we'd use a WFI, or would it be a WFE? Thank you. And do you have an example of how to set it up for other GPIO pins? We presumably modify the address at the end of this line:

I've not experimented with timer based interrupts, but technically this should be possible. It really depends on your platform implementation and if you want to implement an interrupt event based system.
According to the ARM Documentation WFE is just an extension of WFI which also resumes from suspension on some more special events (multicore / mulitrhreading).

Thank you for the details. After more research and generous help from others, including this thread's author, I've made progress on the issue: It seems the best way to do this is fit the chip with a 32.768k crystal, enable the Low Frequency External oscillator, and set up the RTC Wakeup Timer ,which will count down, then fire an interrupt.
Doing this requires a number of steps, which are outlined in the Ref manual, and other ST documentation. I will post the result as a PR or external crate once complete.


[AutoPR] containerregistry/resource-manager

Created to accumulate context: containerregistry/resource-manager

(message created by the CI based on PR content)

Codecov Report

Merging #5203 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5203      +/-   ##
==========================================
- Coverage   49.16%   49.16%   -0.01%
==========================================
  Files        6110     6109       -1
  Lines      167736   167735       -1
==========================================
- Hits        82472    82471       -1
  Misses      85264    85264

Impacted Files
Coverage Δ

...osearch/azure/cognitiveservices/search/__init__.py

...mgmt-machinelearningcompute/azure/mgmt/__init__.py

...derator/azure/cognitiveservices/vision/__init__.py

...ch-videosearch/azure/cognitiveservices/__init__.py

...h-customsearch/azure/cognitiveservices/__init__.py

azure-mgmt-loganalytics/azure/mgmt/__init__.py

...msearch/azure/cognitiveservices/search/__init__.py

...-language-luis/azure/cognitiveservices/__init__.py

...ytics/azure/cognitiveservices/language/__init__.py

...zure-mgmt-managementpartner/azure/mgmt/__init__.py

... and 11 more

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 e760775...687e934. Read the comment docs.

Thanks @lmazuel 👍


SCons: Fix import for collections Mapping

Fixes #54200
Mapping and other "abstract base classes" were moved after python 3.3 from collections to collections.abc
Python 3.3 is long gone and a newer version of python won't support this code.

Cherry-picked for 3.5.


[1.18] Client crash when a creeper follows player into portal

log:  https://pastebin.com/UTs2uTwB
Seems as though a creeper chased a player into a portal and crashed the client when the creeper made contact with it.
using api:  0.0.1-beta46v2-1.18

It can also happen if a player or even a bat goes through the portal
99-fabric-crash-reports_crash-2021-12-05_19.38.59-server.txt

Players are not the problem, just mob entities going through the portals

Interesting, both cases are with the portal at the newly added below 0 portal of the world. Which the player can go through just fine, but entities cannot. This is not the case above y=0. Investigating

It was caused by a check for y>0 in the code. Interesting that this didn't effect players. I suspect lots of mods will have problems with this new <0 world gen.
Beta 47 fixes this

I made a portal at y = -52 and pushed some bats and a sheep through it, no crashes.  Thanks.

yep. i had a creeper go into a portal at the same time it exploded and it crashed my server with
[08:16:00] [Server thread/ERROR]: Encountered an unexpected exception
net.minecraft.class_148: Colliding entity with block
        at net.minecraft.server.MinecraftServer.method_3813(MinecraftServer.java:901) ~[server-intermediary.jar:?]
        at net.minecraft.class_3176.method_3813(class_3176.java:283) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.method_3748(MinecraftServer.java:824) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.handler$zlh000$carpet$modifiedRunLoop(MinecraftServer.java:12942) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.method_29741(MinecraftServer.java:650) ~[server-intermediary.jar:?]
        at net.minecraft.server.MinecraftServer.method_29739(MinecraftServer.java:265) ~[server-intermediary.jar:?]
        at java.lang.Thread.run(Thread.java:842) ~[?:?]
Caused by: java.lang.NullPointerException: Cannot invoke "Object.hashCode()" because "key" is null
        at java.util.concurrent.ConcurrentHashMap.get(ConcurrentHashMap.java:936) ~[?:?]
        at net.kyrptonaught.customportalapi.portal.linking.PortalLinkingStorage.getDestination(PortalLinkingStorage.java:50) ~[customportalapi-0.0.1-beta64-1.20-1536c58c09495c99.jar:?]
        at net.kyrptonaught.customportalapi.util.CustomTeleporter.customTPTarget(CustomTeleporter.java:65) ~[customportalapi-0.0.1-beta64-1.20-1536c58c09495c99.jar:?]
        at net.kyrptonaught.customportalapi.util.CustomTeleporter.TPToDim(CustomTeleporter.java:46) ~[customportalapi-0.0.1-beta64-1.20-1536c58c09495c99.jar:?]
        at net.kyrptonaught.customportalapi.CustomPortalBlock.method_9548(CustomPortalBlock.java:110) ~[customportalapi-0.0.1-beta64-1.20-1536c58c09495c99.jar:?]
        at net.minecraft.class_4970$class_4971.method_26178(class_4970.java:1121) ~[server-intermediary.jar:?]
        at net.minecraft.class_1297.redirect$cce000$carpet-fixes$checkBlockCollisionBetter(class_1297.java:9384) ~[server-intermediary.jar:?]
        at net.minecraft.class_1297.method_5852(class_1297.java:1137) ~[server-intermediary.jar:?]
        at net.minecraft.class_1297.redirect$cce000$carpet-fixes$onEntityCollision(class_1297.java:9352) ~[server-intermediary.jar:?]
        at net.minecraft.class_1297.method_5784(class_1297.java:799) ~[server-intermediary.jar:?]
        at net.minecraft.class_1542.method_5773(class_1542.java:136) ~[server-intermediary.jar:?]
        at net.minecraft.class_3218.method_18762(class_3218.java:739) ~[server-intermediary.jar:?]
        at net.minecraft.class_1937.method_18472(class_1937.java:480) ~[server-intermediary.jar:?]
        at net.minecraft.class_3218.method_31420(class_3218.java:385) ~[server-intermediary.jar:?]
        at net.minecraft.class_5574.method_31791(class_5574.java:54) ~[server-intermediary.jar:?]
        at net.minecraft.class_3218.method_18765(class_3218.java:353) ~[server-intermediary.jar:?]


Build tensorflow-model-server for gpu - Cannot find cuda library libcudnn.so.6

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux ip-172-30-1-83 4.4.0-1062-aws #71-Ubuntu SMP Fri Jun 15 10:07:39 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

TensorFlow installed from (source or binary): binary

TensorFlow version (use command below): 1.5.0

Python version: Python 2.7.12

Bazel version (if compiling from source):

GCC/Compiler version (if compiling from source): c++ (Ubuntu 5.4.0-6ubuntu1~16.04.10) 5.4.0 20160609

CUDA/cuDNN version:

== cuda libs  ===================================================
/usr/local/cuda-9.0/targets/x86_64-linux/lib/libcudart_static.a
/usr/local/cuda-9.0/targets/x86_64-linux/lib/libcudart.so.9.0.176
/usr/local/cuda-9.0/doc/man/man7/libcudart.so.7
/usr/local/cuda-9.0/doc/man/man7/libcudart.7
But I also have these files:
/usr/lib/x86_64-linux-gnu/libcudnn.so -> /etc/alternatives/libcudnn_so
/usr/lib/x86_64-linux-gnu/libcudnn.so.6 -> libcudnn.so.6.0.21
/usr/lib/x86_64-linux-gnu/libcudnn.so.6.0.21
/usr/lib/x86_64-linux-gnu/libcudnn.so.7 -> libcudnn.so.7.0.5
/usr/lib/x86_64-linux-gnu/libcudnn.so.7.0.5

GPU model and memory: Tesla K80

Describe the problem
I try to build tensorflow-model-server with gpu support, as I saw that the apt-get version is only for CPU.
I did:

git clone --recurse-submodules https://github.com/tensorflow/serving
bazel clean --expunge && export TF_NEED_CUDA=1
bazel query 'kind(rule, @local_config_cuda//...)'

And got:
Cuda Configuration Error: Cannot find cuda library libcudnn.so.6
When I do: bazel build -c opt --config=cuda tensorflow_serving/model_servers:tensorflow_model_server
I get the same error.

Hi. Try to set the env variable to LD_LIBRARY_PATH:
export D_LIBRARY_PATH="/usr/local/cuda-9.0/"

Hope it helps :)

You meant export D_LIBRARY_PATH="/usr/local/cuda-9.0/"
Nope... Still same error message:
Cuda Configuration Error: Cannot find cuda library libcudnn.so.6
The file libcudnn.so.6 is found on my server at /usr/lib/x86_64-linux-gnu/libcudnn.so.6
But it can't find it...

Sorry  I meant,
export LD_LIBRARY_PATH="/usr/local/cuda-9.0/"

I am not an expert I was just trying to help since I had a similar issue on ubuntu 18.04 with tensorflow-gpu. So, did you have the /usr/local/cuda-9.0/ folder? Just in case, try to echo your environment variables and see if there is LD_LIBRARY_PATH.
I hope maybe it will help

This should be filed with the tensorflow/serving repo. So I'll close this issue.
Not sure why Tensorflow Serving needs cudnn 6. In TensorFlow, currently by default we use cudnn 7.

I will file it in the other repo.


Replace outdated calendar instructions with the maintained ones.

This updates a link to the maintained document.

This change is

LGTM :+1:

/approve
Thanks @castrojo

/assign
/lgtm


Please make it clearer how to use options

Before you submit a new bug report, please check that

[X] I have read frequently asked questions and limitations

Write your question here
It is not obvious to me how to apply some of the common parameters.  The hint on project page is:
The configuration via configuration.yaml has been deprecated. If you have previously configured the integration there, it will be imported to ConfigFlow, and you should remove it.
For example, how to implement "expire_after"
I found configuration in.
/config/.storage/core.config_entries
Is this the correct place to edit?
Should it go in options, data or parameters?
            {                 "entry_id": "xxxxxxxxxxxxxxxxxxxxxxxxx",                 "version": 4.5,                 "domain": "garbage_collection",                 "title": "XXXXXXXXXXXXXXXXX",                 "data": {                     "unique_id": "xxxxxxxxxxxxxxxxxxx",                     "frequency": "even-weeks",                     "icon_normal": "mdi:trash-can",                     "icon_today": "mdi:delete-restore",                     "icon_tomorrow": "mdi:delete-circle",                     "first_month": "jan",                     "last_month": "dec",                     "collection_days": [                         "mon"                     ],                     "verbose_state": false                 },                 "options": {},                 "pref_disable_new_entities": false,                 "pref_disable_polling": false,                 "source": "user",                 "unique_id": null,                 "disabled_by": null             }

Configure it from Configuration / Devices and Services screen. Expire after is here:


raft: fast fail while dropping index reading

The read request will block the following read request until the context timeout when the read message dropped in the raft, try to fast fail in some case.
This pull request is used to fix https://github.com/coreos/etcd/issues/9566

@nolouch Have you tested it from etcd point of view? And can you add an integration test for this into etcdserver side?

ok, I will add the test.

Sorry for the delay in this PR.
In my test, this patch can only solve part of the problem. If the request coming in the follower, and before it dropping the leader while meeting the network partition, the problem also exists.
I will send a new PR that drop the old read request immediately if found the leader changed. So can prevent blocking the read cause of I mentioned above.


Mm 16694 4.2 validate urls

Before submitting, please confirm you've

[x] read and understood our Contributing Guidelines
[x] completed Mattermost Contributor Agreement
[x] executed npm run lint:js for proper code formatting

Please provide the following information:
Summary
This PR validates deeplink and user-clicked urls before handling them in the Desktop app. This is intended to prevent the desktop app from running invalidly formatted URLs that could compromise the users computer.
Issue link
https://mattermost.atlassian.net/browse/MM-16694
Test Cases
See issue link for test cases.

Definitely interested in feedback to this approach. While I was not able to get the mentioned whoami /all output the ticket mentioned, I did get the new spawned window. This PR now prevents the spawned window for me.
Note: The package I used for URL validation in master won't work in 4.2 .. doesn't like the package's use or es6 spread syntax. I forked the package and changed to Object.assign.

It's bizarre that Electron doesn't support ES6 without transpiling or anything since Node.js and browsers both support it...

Yup, fortunately it's just the 4.2 version, current master doesn't have the problem and is using the package directly.

Oh okay, so the newer version of Electron supports it fine. That's good to know

Okay, I think this is ready to review now. I ran the reported exploit step-by-step and am unable to repro through deeplink exploit with the app closed or open or when clicking the exploit via a link directly in MM.
I still have to troubleshoot the existing tests, add new tests and the CI pipeline is still not working properly.

Okay, so the important part here is that the tests are now passing correctly again in CircleCI. Unfortunately Jenkins is till broken – if CircleCI can successfully build and test, there's no reason why Jenkins shouldn't be able to. I believe @lieut-data had indicated that it looked like Jenkins had an incorrect version of Node or something?
@DSchalla, @cpanato, any chance one of you could provide some assistance?

all fixed now

@deanwhillier Problem was that the desktop pipeline currently is installing with npm install, but 4.2 still uses yarn. We temporarily changed the pipeline to use yarn again so it could succeed. Judging by Harrisons comments I don't know if you want to perform any changes, but otherwise this looks good. :tada:

That's great, thanks @cpanato and @DSchalla! No changes needed here from Harrisons comments.


dotenv breaks due to frame in /tmp; workaround: find_dotenv

Describe the bug
I tried using dotenv per documentation in a cell, after making sure I am in a directory which contains a .env file above it:
import os
from pathlib import Path

# Run this from a working directory
# python dotenv_working.py

Path('.env').write_text('A=10')
import dotenv
result = dotenv.load_dotenv(verbose=True)
print(result)
print(os.environ['A'])

Results in the expected:
# python dotenv_test.py
True
10

But when running inside a marimo cell (exact same code) the results are:
Traceback (most recent call last):
  Cell /tmp/marimo_2302030/__marimo__cell_Hbol_.py, line 9
    result = dotenv.load_dotenv(verbose=True)
  File /home/alon/project/.venv/lib/python3.12/site-packages/dotenv/main.py, line 336, in load_dotenv
    dotenv_path = find_dotenv()
  File /home/alon/project/.venv/lib/python3.12/site-packages/dotenv/main.py, line 300, in find_dotenv
    for dirname in _walk_to_root(path):
  File /home/alon/project/.venv/lib/python3.12/site-packages/dotenv/main.py, line 257, in _walk_to_root
    raise IOError('Starting path not found')
OSError: Starting path not found

When debugging this I can see the root cause: marimo is placing the code inside a frame that is in the /tmp directory, which is not below the current directory. I don't see an immediate fix, since you cannot assume there is write access to the current directory nor is it clear it would be expected that marimo would write there. But it does break dotenv.
✦2 at 09:31:57 ❯ ipython -i dotenv_failing.py
Python 3.12.1 (main, Dec 18 2023, 00:00:00) [GCC 13.2.1 20231205 (Red Hat 13.2.1-6)]
Type 'copyright', 'credits' or 'license' for more information
IPython 8.18.1 -- An enhanced Interactive Python. Type '?' for help.
---------------------------------------------------------------------------
OSError                                   Traceback (most recent call last)
File /tmp/dotenv_failing.py:30
     26     return
     29 if __name__ == "__main__":
---> 30     app.run()

File ~/project/.venv/lib/python3.12/site-packages/marimo/_ast/app.py:210, in App.run(self)
    208     cell_function = self._cell_manager.cell_data_at(cid).cell_function
    209     if cell_function is not None:
--> 210         outputs[cid] = execute_cell(cell_function.cell, glbls)
    212 # Return
    213 # - the outputs, sorted in the order that cells were added to the
    214 #   graph
    215 # - dict of defs -> values
    216 return (
    217     tuple(outputs[cid] for cid in self._cell_manager.valid_cell_ids()),
    218     # omit defs that were never defined at runtime, eg due to
   (...)
    223     {name: glbls[name] for name in self._defs if name in glbls},
    224 )

File ~/project/.venv/lib/python3.12/site-packages/marimo/_ast/cell.py:234, in execute_cell(cell, glbls)
    232     return None
    233 assert cell.last_expr is not None
--> 234 exec(cell.body, glbls)
    235 return eval(cell.last_expr, glbls)

File /tmp/marimo_2304356/__marimo__cell_Hbol_.py:9
      7 Path('.env').write_text('A=10')
      8 import dotenv
----> 9 result = dotenv.load_dotenv(verbose=True)
     10 print(result)
     11 print(os.environ['A'])

File ~/project/.venv/lib/python3.12/site-packages/dotenv/main.py:336, in load_dotenv(dotenv_path, stream, verbose, override, interpolate, encoding)
    319 """Parse a .env file and then load all the variables found as environment variables.
    320
    321 Parameters:
   (...)
    333 .env file.
    334 """
    335 if dotenv_path is None and stream is None:
--> 336     dotenv_path = find_dotenv()
    338 dotenv = DotEnv(
    339     dotenv_path=dotenv_path,
    340     stream=stream,
   (...)
    344     encoding=encoding,
    345 )
    346 return dotenv.set_as_environment_variables()

File ~/project/.venv/lib/python3.12/site-packages/dotenv/main.py:300, in find_dotenv(filename, raise_error_if_not_found, usecwd)
    297     frame_filename = frame.f_code.co_filename
    298     path = os.path.dirname(os.path.abspath(frame_filename))
--> 300 for dirname in _walk_to_root(path):
    301     check_path = os.path.join(dirname, filename)
    302     if os.path.isfile(check_path):

File ~/project/.venv/lib/python3.12/site-packages/dotenv/main.py:257, in _walk_to_root(path)
    253 """
    254 Yield directories starting from the given directory up to the root
    255 """
    256 if not os.path.exists(path):
--> 257     raise IOError('Starting path not found')
    259 if os.path.isfile(path):
    260     path = os.path.dirname(path)

OSError: Starting path not found

In [1]: debug
> /home/alon/project/.venv/lib/python3.12/site-packages/dotenv/main.py(257)_walk_to_root()
    255     """
    256     if not os.path.exists(path):
--> 257         raise IOError('Starting path not found')
    258
    259     if os.path.isfile(path):

ipdb> p path
'/tmp/marimo_2304356'
ipdb>

The workaround is to use:
load_dotenv(find_dotenv(usecwd=True))

Perhaps that should go in the Q&A
Environment
{
"marimo": "0.2.1",
"OS": "Linux",
"OS Version": "6.5.7-200.fc38.x86_64",
"Processor": "",
"Python Version": "3.12.1",
"Binaries": {
"Chrome": "--",
"Node": "v16.13.0"
},
"Requirements": {
"black": "23.12.1",
"click": "8.1.7",
"jedi": "0.19.1",
"pymdown-extensions": "10.7",
"starlette": "0.37.0",
"tomlkit": "0.12.3",
"typing_extensions": "4.9.0",
"uvicorn": "0.27.0.post1"
}
}
Code to reproduce
import marimo
__generated_with = "0.2.1"
app = marimo.App()
@app.cell
def __():
import os
from pathlib import Path
# Run this from a working directory
# python dotenv_working.py

Path('.env').write_text('A=10')
import dotenv
result = dotenv.load_dotenv(verbose=True)
print(result)
print(os.environ['A'])

return Path, dotenv, os, result

@app.cell
def __():
return
if name == "main":
app.run()

Thanks for reporting -- I'll look into this.

I posted an entry in the FAQ for now, as you suggested. Thank you for bringing this to our attention and debugging it so thoroughly!