Using browser language to determine country-specific features is flawed

I don't know if anyone reads comments on already closed issues, so please see https://github.com/nrenner/brouter-web/issues/358#issuecomment-762730701

Thanks for reporting. I find it better to not re-open implemented feature requests, but to open a separate issue for a bug like you did here.

Thanks for reporting. I find it better to not re-open implemented feature requests, but to open a separate issue for a bug like you did here.

from #358 (comment) by @Negativland:
Nice feature, but it has a little bug: when the browser ist set to another language than german (e. g. english) it draws a buffer zone of only 1 km around the home location; see screenshot attached.

P. S.: the "1 km go-to-zone" can be set worldwide which is also a bit odd.

from #358 (comment) by @Negativland:
Nice feature, but it has a little bug: when the browser ist set to another language than german (e. g. english) it draws a buffer zone of only 1 km around the home location; see screenshot attached.

P. S.: the "1 km go-to-zone" can be set worldwide which is also a bit odd.

We currently use the browser language setting to determine if the French or German radius rules apply or if the button should appear at all. I already got another report, so the language setting is less reliable than I hoped.
As a workaround you can override the language by passing it in the URL:
https://brouter.de/brouter-web/?lng=de-DE
https://brouter.de/brouter-web/?lng=de-DE#map=10/50.0977/8.6641/osm-mapnik-german_style&ringgo=8.677826,50.111332,15000

P. S.: the "1 km go-to-zone" can be set worldwide which is also a bit odd.

This is an additional bug, the check to not add the button outside FR/DE does not work.

We currently use the browser language setting to determine if the French or German radius rules apply or if the button should appear at all. I already got another report, so the language setting is less reliable than I hoped.
As a workaround you can override the language by passing it in the URL:
https://brouter.de/brouter-web/?lng=de-DE
https://brouter.de/brouter-web/?lng=de-DE#map=10/50.0977/8.6641/osm-mapnik-german_style&ringgo=8.677826,50.111332,15000

P. S.: the "1 km go-to-zone" can be set worldwide which is also a bit odd.

This is an additional bug, the check to not add the button outside FR/DE does not work.

My only idea right now is to check on every map move if the map center is inside the country boundary of FR or DE, either in addition or as a replacement for the browser language check.

My only idea right now is to check on every map move if the map center is inside the country boundary of FR or DE, either in addition or as a replacement for the browser language check.

language setting is less reliable than I hoped.
(…)
This is an additional bug, the check to not add the button outside FR/DE does not work.

From the user perspective it's the same, because the buffer zone depends on the location you're in, not the language you speak.
Possible solutions:

Change the function to "set buffer zone (where applicable)". If the user then sets his home location in germany, the 15 km rule is used; if e. g. in the UK it just says "you're lucky – no limitation"
… make it a general "choose your activity radius". So by default, one can choose to move no further than n km from home. Wherever limitations apply accordingly, these are used (aos of today in FR and DE).

language setting is less reliable than I hoped.
(…)
This is an additional bug, the check to not add the button outside FR/DE does not work.

From the user perspective it's the same, because the buffer zone depends on the location you're in, not the language you speak.
Possible solutions:

Change the function to "set buffer zone (where applicable)". If the user then sets his home location in germany, the 15 km rule is used; if e. g. in the UK it just says "you're lucky – no limitation"
… make it a general "choose your activity radius". So by default, one can choose to move no further than n km from home. Wherever limitations apply accordingly, these are used (aos of today in FR and DE).

You're right. Implemented 1. (sort of) in #362.
I'm trying to keep the number of buttons on the screen as minimal as possible, especially didn't want to show this one for users that don't need it. But hiding doesn't really make sense here, as when zoomed far out we would need to show it anyway.
Not sure about a general solution 2, I don't really see the use case and personally don't want to spend time on it.

You're right. Implemented 1. (sort of) in #362.
I'm trying to keep the number of buttons on the screen as minimal as possible, especially didn't want to show this one for users that don't need it. But hiding doesn't really make sense here, as when zoomed far out we would need to show it anyway.
Not sure about a general solution 2, I don't really see the use case and personally don't want to spend time on it.

Good choice. Keep it simple und useability and GUI.
Btw, to make the situation even more complicated, some german states (like Hessia and Rhineland-Palatinate) just dropped the 15 km rule…

Good choice. Keep it simple und useability and GUI.
Btw, to make the situation even more complicated, some german states (like Hessia and Rhineland-Palatinate) just dropped the 15 km rule…

some german states (like Hessia and Rhineland-Palatinate) just dropped the 15 km rule…

Thanks for the info, I updated Hessia in the wiki. For Rhineland-Palatinate I haven't found a mention in the new regulation either, do you have a source? Somehow I missed this in the news, even though I'm also following the spiegel.de Corona-News.
But generally this doesn't affect the code, it doesn't tell you if the rule applies or not, it only implements the different definitions of the rule.

some german states (like Hessia and Rhineland-Palatinate) just dropped the 15 km rule…

Thanks for the info, I updated Hessia in the wiki. For Rhineland-Palatinate I haven't found a mention in the new regulation either, do you have a source? Somehow I missed this in the news, even though I'm also following the spiegel.de Corona-News.
But generally this doesn't affect the code, it doesn't tell you if the rule applies or not, it only implements the different definitions of the rule.

I've bookmarked https://www.bundesregierung.de/breg-de/themen/coronavirus/corona-bundeslaender-1745198 as a directory to the corona rules of the states.
Hessia clearly says that the 15 km rule was dropped. RLP just does not mention that rule any more; from that I conclude that they've also dropped that rule.
I don't know if other states also dropped the rule. In these times I don't cycle further than RLP…

I've bookmarked https://www.bundesregierung.de/breg-de/themen/coronavirus/corona-bundeslaender-1745198 as a directory to the corona rules of the states.
Hessia clearly says that the 15 km rule was dropped. RLP just does not mention that rule any more; from that I conclude that they've also dropped that rule.
I don't know if other states also dropped the rule. In these times I don't cycle further than RLP…

Thanks.

Thanks.


Portal-only mode

REVIEW (d321f3d)
I'm happy to discuss the design decisions in a voice call.

REVIEW (06feae6)


SlideNavigationController not called as initial controller

I would like to have the SlideNavigationController called from a button rather than as the initial storyboard item.
I have changed the demo code around so the singletonInstance is initilize from the button correctly and only once, everything come up looking correct but the left and right menu do not respond to touches. Do have a clue as to what I am doing wrong. I do have images of the menu items not responeding but thought better about sending them.
Nick

Duplicate https://github.com/aryaxt/iOS-Slide-Menu/issues/147#issuecomment-104016689


As a server admin, images can be hosted on my server.

5/10

6/10

3/10

6

4/10

4.8 story points total


[QUESTION] How store file as pdf ?

[x] Have read and understood: https://github.com/Maatwebsite/Laravel-Excel/blob/3.1/.github/SUPPORT.md
[x] Checked if your Laravel Excel version is still supported: https://docs.laravel-excel.com/3.1/getting-started/support.html#supported-versions
[x] Checked that your question isn't already asked before.
[x] Filled in the entire issue template

Versions

PHP version: 7.3
Laravel version: 6.0.2
Package version: 3.1

Description
How to save report as pdf on storage
I try with
Excel::store(new \App\Exports\PointsExport($resultado), 'report.pdf',\Maatwebsite\Excel\Excel::DOMPDF, 'storage');
but i have a error
No writer found for type storage
Someone can help me?

Please refer to docs for correct usage of ::store() method: https://docs.laravel-excel.com/3.1/exports/store.html#custom-disks You are passing them in the wrong order.


Kpi is smaller than spend on response curve

Hi,
Response curves' KPI is way smaller than the X axis which is spend but it should be the other way around considering that ROI and contribution are on point. This is really weird that I am spending more than I'm getting. The model i am using is hill_adstock

My code: plot.plot_response_curves(steps=52, media_mix_model=mmm, media_scaler=media_scaler, target_scaler=target_scaler, figure_size=(15,10))
I have 5 features and more than 20 extra features.
This is the ROI

the problem is somewhat similar to Issue #51
The author suggested an alternative solution, i am not sure if that solution would work for my case, tried, but failed to implement it.

@MuhammedTech
Hi~ Can I ask you something if you don't mind? You said you put in 20 extra features, but I wonder what you put in. If the target is order, can I also include data such as visit, revenue, and cart add?


Multipart requests to support multipart-mixed type

At present, multipart requests can only have String part and File part. There was a request in #4 for having multipart-mixed type like:
Content-Type: multipart/form-data; boundary=AaB03x

--AaB03x
Content-Disposition: form-data; name="submit-name"

Larry
--AaB03x
Content-Disposition: form-data; name="files"
Content-Type: multipart/mixed; boundary=BbC04y

--BbC04y
Content-Disposition: file; filename="file1.txt"
Content-Type: text/plain

... contents of file1.txt ...
--BbC04y
Content-Disposition: file; filename="file2.gif"
Content-Type: image/gif
Content-Transfer-Encoding: binary

...contents of file2.gif...
--BbC04y--
--AaB03x--

Source: http://www.w3.org/TR/html401/interact/forms.html#h-<IP_ADDRESS>

Hi;
I need to be able to set (at least) a different Content-Disposition header for each File part of a Multipart request.  The UI REST client only allows setting the Content-Type header for each File part.
Each File part should be allowed to be sent with it's own headers, not just Content-Type.
Do you have a workaround I could use right now ?
Content-Type: multipart/mixed; boundary=AaB03x
--AaB03x
Content-Type: application/xml
Content-Disposition: metadata; filename="1.xml"
... contents of file1.xml ...
--AaB03x
Content-Type: application/octet-stream
Content-Disposition: content; filename="1"
... contents of file1 ...

At present we do not support :( I will take this as a requirement.

The last two commits add support for multipart/mixed Content-type. I saw the list of Content-Disposition values here. @igiguere , in this list I don't see the values you mentioned content and metadata.

@subwiz : The values "content" and "metadata" are defined in our code, to take different action based on those values.  That's why the request sender needs control over the value of Content-Disposition.

Thanks for the input. Will work on this when I have available time.

Had a look at this. there seems to be more than just multipart/form-data and multipart/mixed Content-types. A tool like RESTClient must implement flexibility, and the current UI I designed may not be sufficient.

This issue has three requests:

Support for multipart/mixed Content-Type. This is implemented in for the top-http header now.
Support for multipart/mixed Content-Type for sub-parts. This may not be supported by the tool in immediate future.
Changing the Content-Disposition header. The backend API I am currently using is not supporting this. I will raise this as a request in that project and wait.

http://stackoverflow.com/questions/29211730/multipart-requests-in-httpclient-4-4-custom-content-disposition-header-for-each

Oleg has given an answer in the above SO question:
FormBodyPart bodyPart = FormBodyPartBuilder.create()
        .setName("blah")
        .addField("Content-Disposition", "blah")
        .setBody(new StringBody("blah", ContentType.TEXT_PLAIN))
        .build();
HttpEntity entity = MultipartEntityBuilder.create()
        .addPart(bodyPart)
        .build();
entity.writeTo(System.out);

Need to develop the UI and implement this.

Self-note, multipart-part content with fields:
<file>
  <content>/path/to/file</content>
  <fields>
    <field>
      <name>name</name>
      <value>value</value>
    </field>
  </fields>
</file>


Setup CI platforms

This revision includes:

Setup CI platforms (#2)

Appveyor
Azure Pipelines
Travis CI

Add header generator script
Add yahtzeemaster image to Docker Hub

The cause of Travis CI/dockercloud error is absence of Dockerfile. It will be fixed after PR is merged.


Register Entities created by scripts

Similar to Entity.CreatedByTheCallingPlugin in Ragehook, registers entities that are created via a script so devs who only want to work on entities spawned by the game can simply check this property.

If some entities or pickups are created by directly calling native functions, Entity.CreatedByScript in this PR doesn't do the job. Also, I doubt if this PR is worth merging, since you could create some wrapper functions in your scripts to check if some entities or pickups are created by your scripts.

I don't think this PR is worth merging and I am closing this. Entity.CreatedByTheCallingPlugin of RPH is flawed anyway and no one ever asks about Entity.CreatedByTheCallingPlugin in public forums or the RPH Discord.
RPH does not bother to track if new entities are created and CreatedByTheCallingPlugin will return false on entities that are created when directly invoked some native functions. The CreatedByTheCallingPlugin will return true on entities that are created with the constructors of Object, Vehicle, Ped, Weapon, and Camera (just view inside of rockstarhelper.dll or  nethelper.dll that RPH creates).
You are responsible for keeping track of entities that are created by some calls on your scripts.


versioncmp error on PE2015.2

Using Puppet 4.2.1 on PE2015.2 using this module throws an error, complaining that it found a number instead of a string. This is a quick fix.

Hi,
Thank you for your fork of my module (https://github.com/echoes-tech/puppet-gitolite).
I merged your work on my own branch and I will publish a new release today.
Thanks a lot for your contribution.
Regards,


introduced a simple query extraction mechanism for single-file multi …

This pr aims to address this issue : https://github.com/graphql-rust/graphql-client/issues/433
If there are multiple queries in the same file, they are all concatenated into one string which requires manual adjustments after generation. This function extracts the relevant query string. It assumes that all queries start with "query operation_name" and are followed by another "query".
If the extraction fails due to some reason, the original query string is returned, so the original behavior is not modified. It also handles the case of just one query in a single file.

Would be nice to support mutation and subscription too?

We would at least need tests to make sure nothing breaks. I also think the string matching isn't robust enough. We do parse the queries with graphql-parser, so we could use the AST for that.
I see one reason not to do this: validation errors from the server. They will return file positions in the query the server received. It's confusing for users if they get an error with a position in a document they did not write but that was constructed from their queries.

@tomhoule
Thanks for the quick reply. I am happy to spend sometime on this and get this pr merged eventually. I plan to:

Use parse_query (as used here) to format the the extracted query string, to make sure that its extracted correctly. Question: This doesnt actually help with extracting individual queries, so i will keep the extraction mechanism the same but using parse_query will make sure that they are valid and are properly formatted, is this correct ?
Write some tests for the cli with different query files. I think the function i am modifying is used else where as well but i assume those parts have their tests accordingly which would break if there is an issue.
I didnt have to use mutation and subscription myself, but i would take a look at it once i am done with this. @Sytten

About validation errors, i understand the concern and this might be a problem if you have a lot of large queries in a single file. But to my experience if thats the case, you already need to manually modify the query_string variable in the generated code which would cause the same issue (the file position not matching the actual query file). Another idea would be splitting the large query file into multiple files with a single query in them, but this beats the purpose of containing related queries into the same query file (which what got me going in the first place)

Use parse_query (as used here) to format the the extracted query string, to make sure that its extracted correctly. Question: This doesnt actually help with extracting individual queries, so i will keep the extraction mechanism the same but using parse_query will make sure that they are valid and are properly formatted, is this correct ?

What I had in mind would have been using the Position on the parsed operation to locate the query within the file. String matching is not very reliable.

@tomhoule
Following your suggestion, i have modified the pr to use positions to extract query strings. It indeed became much more robust and now a mixed query file with subscriptions, mutations and queries is also parsed correctly. I have tested the changes locally by using the the following query file.
query StarWarsQuery($episodeForHero: Episode!) {
  hero(episode: $episodeForHero) {
    name
    __typename
  }
}
mutation CreateReviewForANewHope {
  createReview(episode: NEWHOPE, review: {
    stars: 5,
    commentary: "A classic that started it all!",
    favorite_color: {
      red: 255,
      green: 255,
      blue: 0
    }
  }) {
    stars
    commentary
  }
}
subscription OnReviewAdded {
  reviewAdded(episode: NEWHOPE) {
    stars
    commentary
    episode
  }
}
query StarWarsQuery2($episodeForHero: Episode!) {
  hero(episode: $episodeForHero) {
    name
    __typename
  }
}

i have tried to not touch any of the query.rs functionality, so the rest of the codebase should remain unaffected. I can add more tests if its needed.


swlkr.com

My personal website, homepage is 334kb including one image, all stylesheets and goat counter analytics I believe

Added!


Add warning for when clampToGround is ignored in KMLDataSource

This came up in https://github.com/AnalyticalGraphicsInc/cesium/issues/8404. The clampToGround option is ignored in KMLDataSource for lines that don't have tesselate to true. This isn't document and no warning is shown which makes this hard to figure out.
Here's a localhost Sandcastle to test with. Notice the line clamps fine, but if you change the tesselate in the KML doc to <tesselate>0</tesselate> you'll see the line doesn't clamp and, in master, no warning or anything is shown.
We could also document or warn about the case where a line that "can be extruded" are also not clamped, so it looks like the line just can't have an altitude mode of absolute, relativeToGround, or relativeToSeaFloor.

Thanks for the pull request @OmarShehata!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

I'm not sure this warning makes sense.  It's working exactly as expected by the spec, isn't it? If the behavior is different than Google Earth, we should fix that, but I don't think it is in this case.

It's working exactly as expected by the spec, isn't it?

I have limited experience here, but I don't think it is. All I can find in the spec is that tesselate won't work unless altitudeMode is clamp, not the other way around:

To enable tessellation, the value for  must be clampToGround or clampToSeaFloor.

I also just tested these two KML files in Google Earth, one has tesselate on and one has it off, both are set to clamp. Both do clamp in Earth but only the tesselated one clamps in CesiumJS. So it sounds like we should be changing the behavior here?
two_kml_lines.zip
I actually didn't realize that clampToGround inside the KML file meant the same thing as the clampToGround option you pass in to KMLDataSource.load (I thought the KML value just pushed all heights down instead of actually draping the geometry). Why do we have this option instead of just using the clamped value defined for each KML element?

Why do we have this option instead of just using the clamped value defined for each KML element?

Because there are too many caveats for us to just always use GroundPrimitive.  Some of them are:

depthTestAgainstTerrain is required to be true for GroundPrimitive  to work properly.
depthTestAgainstTerrain is required to be false for Primitive to work properly when draped on the ellipsoid.
GroundPrimitive outlines don't work
GroundPrimitive is considerably slower than Primitive.

It used to be worse and the goal is to do what you suggest, but we have a while to go.  Some of this is captured in https://github.com/AnalyticalGraphicsInc/cesium/issues/5942 (since the real goal is terrain on by default)

Because there are too many caveats for us to just always use GroundPrimitive

That's fair, thanks for the explanation.

It's working exactly as expected by the spec, isn't it? If the behavior is different than Google Earth, we should fix that, but I don't think it is in this case.

The behavior is indeed different from Google Earth here, which can be verified with the two KMLs I posted in https://github.com/AnalyticalGraphicsInc/cesium/pull/8428#issuecomment-560399926. It sounds to me like we should have CesiumJS always clamp regardless of whether tesselate is true or false, which would be a breaking change, but would match the behavior of Google Earth.

Thanks again for your contribution @OmarShehata!
No one has commented on this pull request in 30 days. Maintainers, can you review, merge or close to keep things tidy?
I'm going to re-bump this in 30 days. If you'd like me to stop, just comment with @cesium-concierge stop. If you want me to start again, just delete the comment.

@OmarShehata something weird happened with the commits in this branch.  Can you try resetting it or merging in master?

@hpinkos sorry about that, not sure how that happened. Fixed now.

@mramato could you give this one a quick look?

While there's obviously room for improvement all around, the warning about current behavior looks fine to me.  Thanks @OmarShehata
If you think it's worth it, please write up an issue for us to be able to match Google Earth, either exactly or in a way that makes more sense.

It just came up on the forum again so I opened an issue to document my notes on this consider revisiting it: https://github.com/CesiumGS/cesium/issues/8760.


issue with v6.0.0 in lsposed

After installing v6.0.0 from github, lsposed shows available update to older v5.0.0

ignore it, i have published it on lsposed repo yet.


Simple oauth integration

It includes previous discarded work from https://github.com/CartoDB/cra-template-carto/pull/24 and it adds:

Simple use of new scope: datasets:r:* (to avoid double oauth process)
Simpler dataset loading (restricted to just 1 layer)

I have to make a small change regarding to callback redirection

Note: if testing from vercel, beware of blocked popup for OAuth
chrome example:

After our conversation, and while the performance of global scopes is improved, we have decided to bring back the double oauth check, on a dataset by dataset interaction

This is now ready for a second review. I have included all suggestions (except the use of a external lib for debounce) and I have adapted it to the new folder structure.
The most notable change is the use of one by one OAuth authorization when loading datasets


CKEditor wont load imgs and toolbar isnt displaying properly on firefox

When I click on a plugin it works but its icon is not displaying on Firefox v57.0 -

Also imgs I uploaded are not displaying?

NOTE: Both issues above work fine on Chrome - using latest release v02.01.11

Which theme (skin) for the editor did you use?

'moono-lisa' -

both issue i cannot reproduce with firefox 57. Are you getting any issues in the browser console regarding the icons?

any update?


Prerequisites for AMQP port - outbound

In the Prerequisites it says "Make sure the network that your development machine is connected to permits Advanced Message Queueing Protocol (AMQP) over port 5671. This setup enables Azure IoT Tools to communicate with Azure IoT Hub."
It is not clear that this is outbound traffic. Being outbound, most of the times you don't have to do anything since it is already open. That would not be the case if it was inbound.
Suggest to reinforce it's outbound traffic.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8c6611e1-23a3-48b0-d7e9-b7034bc3be9e
Version Independent ID: b05e4e17-70e7-59eb-3c75-d54efea6aa75
Content: Get started with Live Video Analytics on IoT Edge - Azure - Live Video Analytics on IoT Edge
Content Source: articles/media-services/live-video-analytics-edge/get-started-detect-motion-emit-events-quickstart.md
Service: media-services
Sub-service: lva-edge
GitHub Login: @Juliako
Microsoft Alias: juliako

@tiagonmas Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

#please-close see https://docs.microsoft.com/en-us/azure/azure-video-analyzer/video-analyzer-docs/transition-from-video-analyzer


Bugs in cydia

While the installation screen is showing tapping on status bar takes it to top thus breaking the large-small tile on top.
DPKG locked issues(add inbuilt command for fix)
hash mismatch for cydia.lists
Uicache not working

Might be fixed, in the meantime, don’t tap the status bar?
DPKG_locked means that another process is using dpkg and therefore has “locked” it. If you can’t find which process this is, reboot or use ldrestart
Hash sum mismatch means that the maintainer of whatever repo it’s saying has the issue needs to update their Packages.bz2 to match their deb files. Please contact them instead
uicache is disabled except when needed intentionally as a performance enhancement. If cydia doesn’t automatically run uicache when it’s need it, run it yourself.

is also an intended feature, when you reach the top of any page, the title is supposed to change to be the iOS 11 style title


fest(registry):Add support for private registry

[Current coverage][cc-pull] is 85.09% (diff: 44.00%)

Merging [#904][cc-pull] into [master][cc-base-branch] will decrease coverage by 0.60%

@@             master       #904   diff @@
==========================================
  Files            30         30
  Lines          3008       3026    +18
  Methods           0          0
  Messages          0          0
  Branches        502        507     +5
==========================================
- Hits           2578       2575     -3
- Misses          287        304    +17
- Partials        143        147     +4

Powered by Codecov. Last update [4053e51...b17b2bc][cc-compare]
[cc-base-branch]: https://codecov.io/gh/deis/controller/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/deis/controller/compare/4053e51bccf242002f50bd2df16f502653cbcef0...b17b2bcb3f538cef9727640f83dcbc6be25d2713?src=pr
[cc-pull]: https://codecov.io/gh/deis/controller/pull/904?src=pr

I would like to see some tests in this PR, ideally if you develop some for https://github.com/deis/controller/pull/920 then you can bring them over here as this PR has a fair bit of redness in it (when it comes to new lines and lack of coverage)

I agree. We should be holding the controller to a higher standard now that we are out of pre-production.


<EMAIL_ADDRESS>+ "avoidOptionals: true" generating "| undefined"

Describe the bug
Starting in<EMAIL_ADDRESS>avoidOptionals: true is appending | undefined; The | undefined should not be present with avoidOptionals: true.
This was not happening in<EMAIL_ADDRESS>In 2.2.0, the package correctly generates x | null.
To Reproduce
Steps to reproduce the behavior:
Live example
https://codesandbox.io/s/icy-browser-u7p8q?file=/codegen.yml:180-201

My GraphQL schema:

type Query {
    user(id: ID!): User!
}

type User {
    id: ID!
    username: String!
    email: String
}

My GraphQL operations:

query user {
    user(id: 1) {
        id
        username
        email
    }
}

My codegen.yml config file:

schema: schema.graphql
documents: document.graphql
generates:
  types.ts:
    config:
      avoidOptionals: true
    plugins:
      - typescript
      - typescript-operations

Expected behavior

Maybe types generate x | null, not x | null | undefined.
Environment:

OS:
@graphql-codegen/...:
NodeJS:

Additional context

avoidOptionals IIRC prevents generating ? in field?: .... But something has changed and | undefined is being added now. Did not find a way to disable it yet.

The thing happened here https://github.com/dotansimha/graphql-code-generator/pull/6951

Ok I just found myself again already solving this :/ https://github.com/dotansimha/graphql-code-generator/pull/7104

@simPod I updated #7104, and it'll be merged later today.
meanwhile, you can use the alphas

Thank you for pushing this forward!

meanwhile, you can use the alphas

I guess few more hours of waiting won't kill me now 😜

Verified that this is fixed, thank you


Update index.yml

Updated linked URL for serverless landing page

@intrepidsoldier : Thanks for your contribution! The author, @, has been notified to review your proposed change.

@ggailey777 - Would you review the proposed changes? Thanks


Cannot discover bridge

Hello!
I've created an arbitrary device in my bridge, and am now attempting to discover the bridge with my Echo. However, it simply repeats that it cannot find any new devices on the network. The bridge is being hosted on an Ubuntu-based headless server within the same network. I've opened the firewall on the server entirely but I still cannot discover it from the echo. I can access the Web GUI on port 8080 without issue.
Thank you in advance for your assistance. Let me know if I can do anything additional to inform you.

Turn on trace upnp in the bridge config page and then post the log so we can see if it is finding it.

It turns out this was an issue with networking being run inside a docker container, which I stupidly neglected to mention. It works as intended when the jar is executed at the system level.
You can close this and I'll determine if it's worth opening another issue regarding docker. My guess is there's a workaround on the docker side to enable generic UPnP discovery and you don't need to make any changes.
Thanks!


Masked the API key for Predibase LLM

Description: Added the masking of the API key for the Predibase LLM using SecretStr
Issue: #12165
Dependencies: N/A
Tag maintainer: @eyurtsev

resolved in #15119


Command not found: Spleeter

Any ideas? I followed all the instructions.

Hi @chrisgauthier9 , please use the full issue template with info on your setup so that we can help.

This bug is back


Cape And Torch Resistance Changes: improve tooltip compatibility with other mods

The current method for generating the tooltip works fine as long as the mod is the only interacting with GetDamageModifiersTooltipString, however it is not robust in case other mods are also editing tooltips as they are prone to picking up the Water and Cold custom mods, resulting in visual issues where the mods appear twice and/or without label.
To fix this, we filter the Water and Cold custom mods out of the list passed to the original call, and integrate the missing tooltip information after the original call has concluded.

thank you for the PR. that is a sensible idea and a principle that I have also applied in other mods before. I will try to review and test the changes in the coming days


Update deps (cargo update)

Closes #72

Thanks a lot!
I am merging this for an immediate fix but hope soon there will be a v0.6 of jwalk to which dua can then immediately upgrade to not depend on a hopefully soon to be yanked version.

Thanks a lot!
I am merging this for an immediate fix but hope soon there will be a v0.6 of jwalk to which dua can then immediately upgrade to not depend on a hopefully soon to be yanked version.

dua v2.10.8 has just been released.

dua v2.10.8 has just been released.


issue using github for feedback

when you click the github link for feedback it drops you into github making it difficult to get back to the page you were looking at.  Giving feedback should split off to a separate page, not act as a roach motel.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 073882cd-36d0-b95a-77b2-aadd06fa34a5
Version Independent ID: 463f2f77-3fb0-8078-5b49-878747f2842b
Content: Responding to a Compromised Email Account - Office 365
Content Source: microsoft-365/security/office-365-security/responding-to-a-compromised-email-account.md
Product: m365-security
Technology: mdo
GitHub Login: @chrisda
Microsoft Alias: chrisda

Hello @rsgmodelworks. Thank you for posting your feedback here. We will pass it to our GitHub team. As this isn't a specific documentation issue, I am going to close this issue for now. However, please know that your feedback will be shared with the correct team.


Headless?

I built the project on OS X and ran the following png export command
~> bin/plotly-graph-exporter.js '{"data":[{"y": [3, 2, 5, 3]}],"layout":{}}' -o out.png

The out.png image is generated successfully, but I noticed that the electron window pops up in front of my other windows.
Is there any way to prevent the popup? I'm concerned this would be a bit disruptive for eventual python/R users.
cross reference: #21, plotly/plotly.py#880
cc: @cpsievert @etpinard @jackparmer

@n-riesco @JamesCropcho @etpinard  Any thoughts here? Do we need https://github.com/plotly/image-exporter/pull/63 to prevent the popup?

I guess that adding show: false here and here could do the trick, but I haven't tested it.

@jackparmer I don't believe I'll win any popularity contests with my thoughts here, but I'll share them as you have requested.
I do not believe that a window popping up for half a second (or even many such windows) in an application targeting computer-savvy types (e.g. Plotly users) is substantially "disruptive" to warrant significant effort from Plotly to remedy it.

We've heard that pop-ups with current offline PNG generation
cause issues when generating many images in series on a server without a
display. For example, we have customers who would like to generate
thousands of plot images as part of a nightly batch process.  So, yep, this
should be resolved.
On Sun, Apr 8, 2018 at 11:25 AM, James Cropcho<EMAIL_ADDRESS>wrote:

@jackparmer https://github.com/jackparmer I don't believe I'll win any
popularity contests with my thoughts here, but I'll share them as you have
requested.
I do not believe that a window popping up for half a second (or even many
such windows) in an application targeting computer-savvy types (e.g. Plotly
users) is substantially "disruptive" to warrant significant effort from
Plotly to remedy it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/plotly/image-exporter/issues/65#issuecomment-379558687,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABx4ajus71Lb0h_ch0DEs3s2ikuKkpmgks5tmiv5gaJpZM4TK2lY
.

@jackparmer Ah, okay. That is a different use-case than @jmmease described.
I don't have any experience with this particular issue.

I guess that adding show: false here and here could do the trick, but I haven't tested it.

@n-riesco's suggestion above sounds like as a good an initial experiment/step, as any. More broadly, an experimental approach is probably needed, i.e. trying various ideas which seem promising and seeing what works.

Do we need #63 to prevent the popup?

@jackparmer As the author of #63, I am pretty sure that #63 would neither fix this, nor exacerbate it.

This issue is essentially a duplicate of https://github.com/plotly/image-exporter/issues/22
This repo won't attempt to do anything fancy (e.g. xvfb) to wrap electron in a headless executable. We'll just wait until electron supports chromium's --headless flag natively.

What does show: false do exactly? Does it still create a window and simply doesn't focus on it on creation?

Would this be a reasonable command line option eventually?

Definitely. The longterm vision was to to package this with plotly.py so we stop getting complaints like this: https://github.com/plotly/plotly.py/issues/880#issuecomment-377812417

Interesting, it seems that the show: false option suppresses the window for a png export
bin/plotly-graph-exporter.js '{"data":[{"y": [3, 2, 5, 3]}],"layout":{}}' -o out --format png

But when I do pdf export
bin/plotly-graph-exporter.js '{"data":[{"y": [3, 2, 5, 3]}],"layout":{}}' -o out --format pdf

There are actually two popup windows, and only the first one is hidden by the show option

what if you add show: false to this block:
https://github.com/plotly/image-exporter/blob/1c164bb2a4cb69f67a0f842aaf371bcb7721ca55/src/component/plotly-graph/render.js#L155-L158

@etpinard The options are documented here

@etpinard Awesome, adding show: false to image-exporter/src/component/plotly-graph/render.js took care of the second window too.

@JamesCropcho - Please go ahead and make a PR for this. Afterwards we can merge https://github.com/plotly/image-exporter/pull/63 and put out the first release! 🍾

@jmmease Are you able to see https://github.com/plotly/image-exporter/pull/66? If so, is it identical to the changes you made, which worked as desired?

Closed via https://github.com/plotly/image-exporter/pull/72


service: ensure files cannot be modified on a published record

The DraftFilesComponent, suggest it might be possible to change files on a published record. Make sure this is not possible.
https://github.com/inveniosoftware/invenio-drafts-resources/blob/master/invenio_drafts_resources/services/records/components.py#L53-L56
It should probably be handled via permissions management, so that if an instance desires, they could allow editing files after the upload.

replaced by inveniosoftware/invenio-rdm-records#498


The parser must throw an exception with all the errors encountered

For example:
string env = @"
       This is an error.
       KEY1=VAL
       =VAL
       KEY2=${KEY3}
";
new EnvParser().Parse(env);

There are three errors in the data source:

The line: This is an error. does not have a key-value pair format.
The line =VAL has a key with empty string.
The line KEY2=${KEY3} has an embedded variable that does not exist in the current process.

The parser should throw an exception with the three errors found instead of doing it one at a time.

This issue is resolved by PR #18


Feature command printer

Implements a pretty printer for the command stream. Should improve debuggability.

I like the idea of this, but I really don't think this belongs into Commands.js. What do you think about adding a separate file, e.g. famous-dev.js that uses UMD to expose prettyPrint? That way it could be embedded via a simple script tag during development - or - if people really want to have this in their production codebase - can be required using CommonJS.

landed as 4fe6dfc

@michaelobriena/ @DnMllr  Could you elaborate on why you merged this/ respond to the issue(s) raised in the comments above, please? I still think this is something that should not have been merged in this form.

@michaelobriena I think this should be removed. This doesn't belong in core, but should be a separate utility used during development. It isn't more useful than simply swapping out Commands.js with string values IMO.


Missing sites required for Exercise 4

Module: 01
Lab: Exercise 4 - Trainable classifiers
Task: 2
Step: 12
Description of issue
Repro steps:
Task 2 – Create a trainable classifier
10. Select Next.

Select Choose sites to open the right side pane.

Select the following SharePoint sites:
Communication site
News @ Contoso
Contoso Web 1
Brand
Digital Initiative Public Relations
Work @ Contoso
Sales and Marketing
Contoso Landings
Mark 8 Project Team
HR
Operations
Retail
PointPublishing Hub Site
Team Site
Leadership Team
Community
Give @ Contoso
Benefits @ Contoso
Learn @ Contoso
Campaigns - Events
While followign the following steps:

I found that the following sites are no longer available on new tenants:
- PointPublishing Hub Site
- Team Site
- Community
- Give @ Contoso
- Benefits @ Contoso
- Learn @ Contoso
- Campaigns - Events

Can we confirm if these can be added?, otherwise I will adjust the instructions and delete them from the instructions as it can be misleading to learners.

Hi Cristian,
I've noticed you need to be logged in as Jodi Sherman in the tenant to see those SharePoint sites.  I just fetched a classroom tenant last week for testing and it has those sites, but they only appear when signed in as Jodi not Admin.  See image below.


[storage-extract-image-text] Invalid link appears on extension details

Expected Behavior
Extension details link should link to a valid page showing the extensions details.
Actual Behavior
It directs to an invalid link similar to https://console.firebase.google.com/project/extensions-testing/extensions/instances/null.
Steps to Reproduce the Problem

Install the latest storage-extract-image-text extension.
Select the extensions from your list of extensions.
Click Extension Details as shown below...

￼

Specifications

Extension name: storage-extract-image-text
Version: 0.0.1

closing as i believe this is fixed in the GCP repo


Wmmc88/hex tol test

Same Pins (7.95mm diameter). Left hole starts with 0.01mm surface to surface clearance (7.97mm diameter hole). clearance steps up by 10 microns each hole. The last hole has 190 microns of clearance and is 8.33 mm diameter.
^referring to circles but hex has same pattern
from previous testing, 200 microns of clearance was a very loose fit and 150 micron was interference(tho could have been due to print issues)

Waiting for petg to print and merge

Closes #58

@matthew-reynolds as discussed, we "tight" fits of around 10 microns for hex and 30 microns for circle. can size slip and interference around that


Processing vector of block-action pairs

Replaced with https://github.com/clemahieu/raiblocks/commit/cf32e1677e51b2d1e712483c5f7df75712b5aa96


Gun Still Not Appearing

Update 1.1.5 was supposed to have "Fixed base skin gun not appearing?? (why tf does it keep doing this)" however in the lobby the gun still takes time to appear. I am playing pre sots btw

I dont update for pre sots


Encryption support

I'm using https://github.com/Willena/sqlite-jdbc-crypt to secure connection to the database. Is it possible to support that in this Grafana datasource plugin?

I am not actually aware how this tool works. If you create/use a database via this sqlite-jdbc-crypt tool, can you open the database with any other program, too?

I have only ever used that library to open it. I would think that you could with the same cipher settings and key but that's not my area of expertise either.

I don't have the setup to test this Java tool. If you could try out opening a database created with the java tool with the SQLite CLI command, then it could tell me if I can support that or not.
It might be also worth asking in the github repo of that encryption, tool, though how other tools are supposed to work with it (that don't use Java for example)


[ENHANCEMENT]

What is your issue or question?
Hello @Muskansahuincredible, I would like to add hover effect on Log In and Sign Up buttons on Navbar. It would enhance the look of the website even more. Kindly assign me this issue.
Checklist

[X] I have checked the existing issues.
[X] I have read the Contribution Guidelines.
[X] I have read the Code Of Conduct.
[X] The changes don't break the code.

sorry no need of this now. so, I closing this issue please work on another issue


WFLY-4512 Upgrade Weld to 2.2.11.Final

Linux Build 6292 is now running using a merge of 90de55fe184948d49ec16e875e6e57d3540c0eb8

Windows Build 1399 is now running using a merge of 90de55fe184948d49ec16e875e6e57d3540c0eb8

Linux with security manager Build 1257 is now running using a merge of 90de55fe184948d49ec16e875e6e57d3540c0eb8

Linux with security manager Build 1257 outcome was SUCCESS using a merge of 90de55fe184948d49ec16e875e6e57d3540c0eb8
Summary: Tests passed: 823, ignored: 356 Build time: 0:05:04

Linux Build 6292 outcome was SUCCESS using a merge of 90de55fe184948d49ec16e875e6e57d3540c0eb8
Summary: Tests passed: 2847, ignored: 412 Build time: 0:46:07

Windows Build 1399 outcome was SUCCESS using a merge of 90de55fe184948d49ec16e875e6e57d3540c0eb8
Summary: Tests passed: 2847, ignored: 412 Build time: 0:49:50


Cannot release new version of the package. CI is stuck

Build fails. GH Action generates changes in git. They are different from local PC.
https://github.com/pulumiverse/pulumi-astra/runs/7524235454?check_suite_focus=true

@vavsab the main reason for this "worktree is not clean" is that you are running a different Go version than on CI. It seems the code generation is slightly different between Go versions, leading to this difference.

@ringods the problem was in the missing plugins here
https://github.com/pulumiverse/pulumi-astra/blob/19feeabe8e17dd60928f428780b0c9b6964468c3/Makefile#L110
After several months of thinking I finally switched to devcontainers locally and reproduced the issue.


Github Actions

Needs a github actions work as travis is no longer operational see.
https://github.com/davenverse/mapref/pull/10

Blocks release of 0.5.0

Fixed by #222


Enhancement: Gotify priorities

Thanks for releasing with the Gotify PR included so soon!
Makes it easier a lot - I had used Webhook to NodeRed and from there relayed to my Gotify as a workaround in the meantime.
As commented on the Commit already, I'd really love to see priorities being supported. I use different notification classes in the Gotify Android App. This way I could react on critical failures fast, while not getting woken up by minor things in the night (just an example ;))
Thx!

good suggestion, uptime-kume itself might require some more internal enhancements tho. cc @louislam 🧐
uptime-kuma currently only checks for available/unavailable states afaik.
For this functionality it would make sense to check status codes, response times, minimum counters/ threshholds (like statping does)
In the meantime, I implemented basic priority customization in PR #52 .


Why does Shipit try to use rm command to delete file on windows?

IMO it should use del command. How can I change that?
Running "mkdir -p /var/www/<hidden>/releases/20201021130855" on host "<hidden>".
Release path created.
Copy project to remote servers.
"copy" method is deprecated, please use "copyToRemote", "copyFromRemote", "scpCopyToRemote" or "scpCopyFromRemote". It will break in v5.0.0.
Running "mkdir -p /var/www/<hidden>/releases/20201021130855" on host "<hidden>".
@<hidden>-err 'rm' is not recognized as an internal or external command,
@<hidden>-err operable program or batch file.
'deploy:update' errored after 5.69 s
Error: Command failed: cd C:\Users\<hidden>\AppData\Local\Temp && C: && rm tmp-33708lcvMxMo00Qtf.tar.gz
'rm' is not recognized as an internal or external command,
operable program or batch file.

    at ChildProcess.exithandler (child_process.js:294:12)
    at ChildProcess.emit (events.js:198:13)
    at maybeClose (internal/child_process.js:982:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:259:5)

Monkey patch for Windows users:

Add a the following task:

shipit.blTask('fix-win', () => {
    shipit.pool.connections.forEach((connection) => {
        const { runLocally } = connection;
        connection.runLocally = function(cmd, options) {
            if(!cmd.startsWith('ssh')) {
                cmd = cmd.replace(/ rm /g, ' del /s /q ');
            }
            return runLocally.call(this, cmd, options);
        }
    });
});

Make sure patch is applied when you call shipit env deploy

shipit.on('deploy', () => {
    return shipit.start(['fix-win']);
});


⚠ Samsung has degraded performance

In 0d31915, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2752 ms

Resolved: Samsung performance has improved in c4e07fb.


do_not_track_routes working?

do_not_track_routes track routes if:
'log_enabled' => true,

?
I need to lock the track in admin.* routes, but it is not working.
'do_not_track_routes' => [
		'tracker.stats.*',
		'admin.*',
	],

Works for me. If it still doesn't work for you, please ask to reopen.

it does not work for me too?
how to solve it?


Applescript document method not recognized

Steps to reproduce

Create an Applescript that opens MacVim and calls document 1.  Eg:

tell application MacVim
    activate
    set my_path to get path of document 1
end tell

Execute script with osascript script_name.
Find the error: Can’t get document 1. Invalid index. (-1719)

See more details here.
Expected behaviour
get path of document 1 should return the file path of the first document being currently displayed.  I believe it's also called the 'front document'.
Version of Vim and architecture
9.0.65 (173)
Environment
Operating System: Bug Sur version 11.6.8 (20G730)
Terminal: Using MacVim GUI, not a terminal.  It's version shows the Vim version 9.0.65 (173).  It also says c2007 Bjorn Winckler.
Value of $TERM: xterm-256color
How MacVim was installed
brew install macvim
Logs and stack traces
Don't find any log or other error besides what's displayed.  Maybe I don't know how to turn on verbose logging?

Vim configuration where issue is reproducable
No response
Issue has been tested with given configuration

[x] by running MacVim.app from GUI macOS interface
[x] by running vim/gvim/etc installed by MacVim
[ ] by running other versions of vim (e.g. /usr/bin/vim)

Issue has been tested with no configuration

[X] by running mvim --clean (or gvim, supplied by MacVim distribution)
[X] by running vim --clean (in terminal, supplied by MacVim distribution)
[ ] by running vim --clean (in terminal, other suppliers, e.g. /usr/bin/vim)

Other conditions

[ ] The both Homebrew packages "vim" and "macvim" are installed

The problem is not in MacVim, but in the script file itself as some conditions hasn't been met. This error is quite common as you Google it

This would be a useful thing to support but I think it requires MacVim to be a document-based program. It's a little more tricky than it seems because the actual files opened are being managed by Vim. We could just expose the currently edited document but that wouldn't be really supporting the full document API.
Marking this as a feature request for now. It's doable, but just not very high priority.
Just curious though @cPMarco what are you trying to do here? What do you want to accomplish by writing that script?

We could just expose the currently edited document but that wouldn't be really supporting the full document API.

I'd vote for that - I'm only interested in this one functionality.

Marking this as a feature request for now.

Makes sense.

Just curious though @cPMarco what are you trying to do here?

I often open/take quick notes with TextEdit, while Vim is used for development.  I'd like to be able to toggle those quick notes into MacVim, and back out into TextEdit if I need.

I see. And you are only interested in the file path right? Can you show me what an example script would look like and how you would use the path you get from MacVim to send to TextEdit? AppleScript is always one of those things that I didn't feel like learning too much of. As for TextEdit -> MacVim, I would imagine the easiest way is just to do mvim:// protocol.
I kind of need to think through more how to expose MacVim functionality to other apps anyway. There are kind of too many ways to do this right now and Apple seems keen on pushing more and more without supporting each one fully. For example, I was going to suggest doing this via the Shortcuts app instead, but it doesn't seem like TextEdit even supports it (correct me if I'm wrong).  There are Services, Apple Script, Share, Shortcuts w/ SiriKit Intents and App Intents and Action extensions, and more, and they all have overlapping functionality. Anyway I digress.

Yes, the file path is what I need.  The full script I'm using is here.

Paste that script into a file, example: ~/toggle-text-editor.apple.scpt
Open a file in MacVim
From Terminal, run osascript ~/toggle-text-editor.apple.scpt e

If I had to do any setup to install Applescript in Mac osacript, I don't remember any.

Ok, I can take a look. Judging from current features down the line, I am guessing this will be investigated around r176. AppleScript support is kind of one of those things that haven't been touched in a while and I want to take a cohesive look into what kind of functionality MacVim should expose to other applications via Shortcuts, intents, and whatnot as well.

Just don't listen to that StackOverflow comment about switching to Emacs :)


Search doesn't work with --web flag, without it, or on the website, and I'm freaking out

[x] I've read and understood the *CONTRIBUTING guidelines and have done my best effort to follow.

Command
/usr/local/bin/pod search PeerKit --web

Report

What did you do?
I searched a Pod, as I wasn't sure if it was something that was its' own spec or not.

What did you expect to happen?
I haven't used CLI search before, so I don't know; probably something searchy.

What happened instead?
I got this. Additionally, this doesn't work on the website either.

Stack
   CocoaPods : 1.5.0
        Ruby : ruby 2.3.3p222 (2016-11-21 revision 56859) [universal.x86_64-darwin17]
    RubyGems : 2.5.2
        Host : Mac OS X 10.13.4 (17E199)
       Xcode : 9.3 (9E145)
         Git : git version 2.15.1
Ruby lib dir : /System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib
Repositories : KittenCommonModules - https://github.com/coffeeboo/KittenCommonModules @ f50c046a44729c16740891d806782df52db46779
               master - https://github.com/CocoaPods/Specs.git @ dc738293f3d7341050af7736385d5c94fa39cfb2

Plugins
claide-plugins        : 0.9.2
cocoapods-deintegrate : 1.0.2
cocoapods-playgrounds : 1.2.2
cocoapods-plugins     : 1.0.0
cocoapods-search      : 1.0.0
cocoapods-stats       : 1.0.0
cocoapods-trunk       : 1.3.0
cocoapods-try         : 1.1.0

Error
NoMethodError - undefined method `open!' for #<Pod::Command::Search:0x007fb11023f078>
Did you mean?  open
/Library/Ruby/Gems/2.3.0/gems/cocoapods-search-1.0.0/lib/cocoapods-search/command/search.rb:79:in `web_search'
/Library/Ruby/Gems/2.3.0/gems/cocoapods-search-1.0.0/lib/cocoapods-search/command/search.rb:61:in `run'
/Library/Ruby/Gems/2.3.0/gems/claide-1.0.2/lib/claide/command.rb:334:in `run'
/Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.0/lib/cocoapods/command.rb:52:in `run'
/Library/Ruby/Gems/2.3.0/gems/cocoapods-1.5.0/bin/pod:55:in `<top (required)>'
/usr/local/bin/pod:22:in `load'
/usr/local/bin/pod:22:in `<main>'

Other issues
“pod search --web «search text»” Command Pukes

https://github.com/CocoaPods/cocoapods-search/pull/26 should fix the exception, but please be aware that cocoapods web search is currently experiencing an outage, you can follow status.cocoapods.org for updates on that


Docker Load API not working

Description
While using the docker client to load the image with the following code
final File imageFile = new File("/Users/xxxxx/postgres.tar");
			final String image = "postgres-test";
			logger.info("Loading Image");
			try {
				InputStream imagePayload = new BufferedInputStream(new FileInputStream(imageFile));
				dockerClient.load(image, imagePayload);
				logger.info("Loaded Image");
			} catch (Exception e) {
				logger.info("Exception in loading Image");
				e.printStackTrace();
			}

[Add feature/bug description here]
Exception in loading Image
java.lang.IllegalStateException: Could not acquire image ID following load
   at com.google.common.base.Preconditions.checkState(Preconditions.java:173)
   at com.spotify.docker.client.DefaultDockerClient$LoadProgressHandler.getImageId(DefaultDockerClient.java:142)
   at com.spotify.docker.client.DefaultDockerClient$LoadProgressHandler.access$700(DefaultDockerClient.java:129)
   at com.spotify.docker.client.DefaultDockerClient.load(DefaultDockerClient.java:732)
   at com.spotify.docker.client.DefaultDockerClient.load(DefaultDockerClient.java:696)
   at com.ge.postgres.service.PostgresDocker.createPostgresDocker(PostgresDocker.java:77)

[Add steps on how to reproduce this issue]
What do you expect
Image to be loaded and listed in the docker images
What happened instead
Exception is thrown
[Describe the actual results]
Software:

docker version: [client: 1.12.0 and server : 1.12.0]
spotify docker-client version: [3.5.12]

We just released 6.0.0. Does this problem still exist for you?


Prevent Events in specific instances

I would like to prevent some events from occurring depending on the context.
An example: a User should not be able to deactivate a node by clicking on it, but the node should still deactivate if the User selects a different node.
Another example:  a User should not be able to toggle the root node, but they should be able to toggle any other nodes (provided they have children of course).
Is there a way I can achieve this?    I am using v1.1.18 (stuck at Angular2 rc5 for the moment).  If so, does it require an upgrade to the newest version of angular2-tree-component?
Thanks for your help/hard work on this fantastic library!

Hi!
You can achieve this using actionMapping.mouse.click (available in 1.1.18):
https://angular2-tree.readme.io/v1.1.18/docs/action-mapping
The default is TREE_ACTIONS.TOGGLE_SELECTED, which does this:
(tree:TreeModel, node:TreeNode, $event:any) => node.toggleActivated()
You can specify a different method that does something like:
(tree:TreeModel, node:TreeNode, $event:any) => {
  if (node.isRoot) return;
  node.setIsActive(true);
}

And I strongly recommend you to advance to the release version of Angular 2.
Most open source out there has already advanced, and you'll be stuck behind
All the best!

Thanks for the response!  An Angular2 upgrade is definitely on the list, unfortunately we have a couple of other priorities to get to before then...


[Security][Question] Is there a possibility to disable exec in templates?

Hi :slightly_smiling_face:
I wanted to use Helmfile with ArgoCD and I was looking at this plugin https://github.com/travisghansen/argo-cd-helmfile
Then a security notice drew my attention that helmfile has insecure functionalities that allows execution of any scripts.
Background:

ArgoCD executes Helmfiles with privileges of argocd-repo-server
I would like to achieve an architecture where CI produces artifacts and does not have access to cluster, after merging to GIT - ArgoCD can apply those changes to separate access to cluster from CI pipelines which can be modified by development teams (https://github.com/jenkins-x/jx/issues/8060)

My question is
Is it possible to block exec and any other possible security-breaking features by e.g. setting an environment variable?
Thanks for your time, thanks for this fantastic project :blush:

@keskad It's not possible today! But I'm very eager to include some feature flag to disable the functionality (perhaps something like DISABLE_INSECURE_FEATURES=true as a starter) if anyone could contribute one 👍
Probably the list of things to be disabled for security would include:

template functions (exec, readFile + relative path)
go-getter support for remote helmfile fetching)
helmfile hooks (allows hooking arbitrary shell commands on helmfile execution)

I see @itscaro already started working on this, nice! Thank you! :smile:


Not recognizing exact letter

Description of the new feature / enhancement
To enhance the extracting of text from the video. Its not able to get exact letter from the video. i had select 10 lines of code from the video but instead of "disabled" word it returns me disab1ed. But all the letters was selected correct from that video.
please tell the Ai to work properly.
Scenario when this would be used?
Asap
Supporting information
No response

Please use Snipping Tool's new OCR system which is hands down better than text extractor.

Unfortunately new Snipping Tool with OCR is only available on Win11, and my organization hasn't yet migrated me off of Win10. Hopefully they do so in the next year or two I guess.

Fwiw here's a screenshot where OCR failed for me (it's originally rotated vertically, I also put it in an image editor and rotated horizontally which slightly improved results but still barely usable).

OCR:
O
O
1-0
o
0

OCR:
+08500
+08500 002-PSOI
CLEAÄ


Arbitrum support?

Hi! I was wondering if you guys were planning to add Arbitrum support?

https://github.com/uniswap-python/uniswap-python/discussions/176


Class "SoapClient" does not exist at compile time

I am planning to use zephir to create a php extension which needs to use SOAP REST API's.
Kindly let me know if I can use php standard classes in zephir-lang.

can I link php_soap.so library while compiling this zephir extension?

You can provide prototype. In the configuration file:
{
    "prototype-dir": {
        "soap": "prototypes/soap"
    }
}

Then you will need to create the SoapClient stub.
Another way is to provide header files. Also see Extension Configuration File.

Hi, I'm getting an error.
I try everyhing. What am I missing?
Platform: Windows 10.
PHP: 7.3.7
Zephir: 0.12.0
Error:
PHP Fatal error:  Uncaught SoapFault exception: [Client] Error finding "uri" property in D:\Projects\admin\public\index.php:35 Stack trace: #0 [internal function]: SoapClient->__call('__construct', Array) #1 D:\Projects\admin\public\index.php(35): Common\WindowsApi->__construct('http://localhos...', Array) #2 {main}   thrown in D:\Projects\admin\public\index.php on line 35
windowsapi.zep:
`namespace PxPanel\Common;
class WindowsApi
{
private uri;
private client;
public function __construct(url, soapOptions){
    var sclient = "\\SoapClient";
    var_dump(url);
    var_dump(soapOptions);
    //var_dump(this->client);
    let this->client = new {sclient}(url,soapOptions); //error line
    // let this->client = new \SoapClient(url,soapOptions); //same error
    var_dump(this->client);
}

public function CallSoapApi(soapMethod, soapMethodParams) {
    /*var returnValue;

    let returnValue = this->client->__soapCall(soapMethod, ["parameters": soapMethodParams]);

*/
return this->client;
}
}`
php code:
use Common\WindowsApi; $sc = new WindowsApi('http://localhost/pxpanelapi/WebServerApi.svc?wsdl',['soap_version' => SOAP_1_1]); var_dump($sc->CallSoapApi('AddWebsite',$params));


standard testing of benchmark and model additions

To ensure that new benchmarks and models actually run through, we should add our own tests that provide a dummy model or dummy benchmark and simulate an entire benchmark or model run respectively. Otherwise we might merge new plugins that don't actually run properly.
We might need to do this separately for each domain, i.e. dummy model+benchmark for language, and the same for vision (and all future domains).
@kvfairchild

Core infrastructure put in place by https://github.com/brain-score/core/pull/72
Solved for vision models; issues created in corresponding domain repositories for vision benchmarks, language models, and language benchmarks.


RFM - dropit

I DON'T Want To Become The Maintainer

[x] I have followed the Package Triage Process and I do NOT want to become maintainer of the package;
[x] There is no existing open maintainer request for this package;

Checklist

[x] Issue title starts with 'RFM - '

Existing Package Details

Package URL: https://chocolatey.org/packages/dropit
Package source URL: https://github.com/dtgm/chocolatey-packages/tree/master/automatic/dropit/

Date the maintainer was contacted (in YYYY-MM-DD): 2021-12-13
How the maintainer was contacted: email

this request covers the packages dropit.install and dropit.portable as well

Hello,
This isn't updated since 2018, and the latest update is the same version as on the community repository.
So i don't think it's interesting to push something new.


APE-688|  modify xml-http-request span name

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@853fcb9). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #1   +/-   ##
=======================================
  Coverage        ?   89.37%
=======================================
  Files           ?       11
  Lines           ?      320
  Branches        ?       46
=======================================
  Hits            ?      286
  Misses          ?       34
  Partials        ?        0


parse json response into a dict before returning

One of two solutions to issue #47. I prefer this one, because even though it breaks existing apps, it's now consistent with rtm_read() returning dicts.

Obsoleted by #74, which fixes both #47 and #73.


Fix broken links

Original report on BitBucket by John Shepherdson (GitHub: john-shepherdson).

Remaining "broken" links are to non-public CESSDA repos, thus Bitbucket returns 404

This blocks #61.

Fixed in https://github.com/cessda/cessda.guidelines.public/commit/6b2479746c0390bd4b61050b338380417b7dbbff


Support setting UA

Per the contributor instructions, just wanted to open an issue for this instead of directly sending PR.
Currently the UA is a hard coded string from a Macintosh computer:
event_user_agent = 'Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:121.0) Gecko/20100101 Firefox/121.0'
Which renders like this on Umami dashboard:

Since there is already a pretty low fidelity of information for the server side data, I think it would be helpful to allow forwarding the header information. Not everyone will have this, but some may.
One thing to be wary of is:
https://github.com/umami-software/umami/discussions/1833

This is needed to prevent bot detection check that prevents the request from being tracked in case we don't pass a user agent
You can also set DISABLE_BOT_CHECK=true in your umami environment to disable the bot check entirely:
https://github.com/umami-software/umami/blob/7a3443cd06772f3cde37bdbb0bf38eabf4515561/pages/api/collect.js#L13

We will also need a similar warning either way.

Thanks @ddxv I'm not sure how to handle this one, which is why I punted and embedded a random Firefox user agent. You can see there is a proper UA describing the library and its environment that is usually passed. However, when I passed it to the event api, it was getting blocked (I guess due to this bot thing).
One option would be to allow users to pass the UA for each call. That might be nice because it would be higher fidelity when it is caused directly by a user-action. What do you think?

Yeah, I think it should be default the hardcored UA (same as now) and
optional to forward one. Definitely we should include the note so that
other people don't waste time trying to figure out why they are blocked if
they end up forwarding a UA from a software library.
Let me know if this is something you'd appreciate a PR or want to add. I'll
be busy today but could get to it this weekend for sure.
On Thu, Jan 25, 2024, 23:21 Michael Kennedy @.***>
wrote:

Thanks @ddxv https://github.com/ddxv I'm not sure how to handle this
one, which is why I punted and embedded a random Firefox user agent. You
can see there is a proper UA describing the library and its environment
that is usually passed. However, when I passed it to the event api, it was
getting blocked (I guess due to this bot thing).
One option would be to allow users to pass the UA for each call. That
might be nice because it would be higher fidelity when it is caused
directly by a user-action. What do you think?
—
Reply to this email directly, view it on GitHub
https://github.com/mikeckennedy/umami-python/issues/5#issuecomment-1910425373,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZ72K47LKSU2MLQCNOP3ETYQJZ6FAVCNFSM6AAAAABCJ2YCNKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJQGQZDKMZXGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey @ddxv I'd definitely be up for a PR. Go ahead and put together something and we can get it in there.
It might be nice to have a more unusual but not blocked default user agent. If you have something in mind there, go ahead and try that too. I don't think it matters much for events. But it will matter a lot for new_page_view events that report browser usage.

This is fixed thank's to @ddxv. Published as 0.1.12 on PyPI.


Enabled index-header under experimental flag.

Depends on: https://github.com/thanos-io/thanos/pull/1952
Signed-off-by: Bartlomiej Plotka<EMAIL_ADDRESS>

FYI @squat  This is chained PR, chained against https://github.com/thanos-io/thanos/pull/1952 which has to be reviewed first to be merged (:
I think for the experimental flag it's fine to not have CHANGELOG. Doc was added to https://github.com/thanos-io/thanos/pull/1952, let me know there if something is missing...

Yes but I mean the generated document for the cli flags. That has to be merged simultaneously with this change, no?

.. or after.
It's hidden flag, so CI is not crying for lack of generated docs.

But still, some integration tests are failing, need to investigate after holidays (in week time) (:
Anyway, the https://github.com/thanos-io/thanos/pull/1952 should be ready for review.

Actually let's merge one into another, will be easier to merge and see integration tests results.


Added an event for default rule "Container Drift Detected (open+create)"

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind documentation

/kind tests

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area commands

/area pkg

/area events

What this PR does / why we need it:
Added an event for default rule "Container Drift Detected (open+create)" as mentioned here
Which issue(s) this PR fixes:

Fixes #181
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: GLVSKiriti
Once this PR has been reviewed and has the lgtm label, please assign fededp for approval. For more information see the Kubernetes Code Review Process.
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

@FedeDP  Similarly I executed this code in a go container and it triggers the rule

@leogr I updated the code now. The first log is triggered when go is building executable and 2nd log is triggered
when running the event

LGTM label has been added.  Git tree hash: a24c105f9d3cf422ec2cfeadb309ceb858fe847c

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: GLVSKiriti, leogr
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 [leogr]

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


Component naming

see https://github.com/vuejs/docs/commit/3a328661cc5aa0ce0bab64f9bb41995b38044e1d
related with README.md#Component-Naming
I don't know why...

closed by #173


Repl tool not functioning

plugin version: 0.65
Description
Thanks for releasing the repl tool.  Looks to have a bug which prevents me from using this as I have trouble clicking on the Ocaml Repl template in run configurations. (let me know if you prefer a separate issue). Then when I can click on it, it has the options of the last clicked template prior to that.  Couple of examples below:
(v 0.65 of plugin and
PyCharm 2018.2.4 (Professional Edition)
Build #PY-182.4505.26, built on September 19, 2018
JRE: 1.8.0_152-release-1248-b8 x86_64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
macOS 10.14)

REPL needs modules. Can you / do you have configured module ? see the wiki for that.
I don't know if pycharm accepts modules

Hmm thanks, not sure what you mean by configured module, or which wiki you are referring to?
I have the reasonml plugin working and installed and that all works ok, just not the REPL which was recently added.

Sorry, REPL is only available in IDEA (community and ultimate).
I updated the config to hide it in other versions.

Trying to use this in Intellij ultimate now.  However get an SDK error, (even though the bucklesript compiler etc... al works correctly)

@yunti you need to define a SDK before using it, see https://github.com/reasonml-editor/reasonml-idea-plugin/wiki/Configure-an-ocaml-project

@yunti yes, ocaml need to be installed.

@giraud Thanks installed via brew on macos but I'm not clear if that can be picked up by intellij properly.

Hmm still can't find a way to get this unfortunately.....

@yunti did you look at the link I gave you ? it doesn't seem like you are referencing the .opam directory, in the screenshot you only have symlinks to ocaml binaries.
Maybe you expect something particular that is not yet implemented ? it is a very rough implementation.

@giraud yes thanks I took a look at that.  Couldn't figure out at first how to get it to point to the right place.  In the end got it to work by pointing to the opam directory directly, not attempting to use the symlinks that homebrew uses on mac.  Pointed it to /usr/local/Cellar/ocaml/4.07.1

However as it's 4.07.1 it doesn't work (won't let me type)

Homebrew only let's you install the latest version so I need to figure out how best to install the older version of ocaml which matches...
Probably best in the end if we can just use the ocaml version included with reason so it's easier for people to setup (only need to install reason/ bsb platform and reason-cli) and don't also need to install ocaml and make the version match up).
Appreciate that this is just a rough implementation and that you are out here on your own making this plugin - which is amazing by the way. (Thanks for all your work on it).

Done thanks


Revert "Use acloud ownership label. (#24)"

This reverts commit de5bbbbfe4d7ba197bcf5bd80b6ef37a41ea29ff.

Not the acloud ownership label.

This doesn't look like a complete revert of that PR, maybe a different name should be better


Doesn't work with compound filetypes.

I have a really niche use case here. Both Vim and Neovim have a handy feature that allows a file to be recognized as multiple filetypes, then various scripts and autogroup actions can be ran for each of the filetypes that applies. As an example I'm using the Vimwiki plugin to edit markdown files, some types of markdown can also be considered a pandoc file as well.  In neovim if I open a markdown file and issue the command :set filetype it shows the current filetype as vimwiki.markdown.pandoc as expected. However, this plugin doesn't recognize this as a list of filetypes. Instead, as far as I can tell, it recognizes a single filetype vimwiki.markdown.pandoc and doesn't load until I issue the command set filetype=markdown. At any rate, thanks again for this rad plugin!

So, embarrasing. Just posted to the wrong page.


[Bug]: I'm unable to install the plugin

Bug Description
The plugin does not show up in the community plugins section, after unzipping in the release file and moving it to the plugins folder in my vault, or by installing it with BRAT (by pasting the repo URL there).
Relevant Screenshot
No response
Text before pasting it into Obsidian.
N/A
Text after pasting it into Obsidian.
N/A
Where the text was copied from
N/A
To Reproduce
No response
Obsidian Version
1.1.9
Which editor are you using?

[X] New Editor
[ ] Legacy Editor

Checklist

[X] I updated to the latest version of the plugin.

oh sorry, I forgot to archive this repo. The complete functionality of this plugin has already been sherlocked (implemented) by the Linter plugin. You can install it, and find the respective features under the pasting section


Handle changing tags

It seems like users quite frequently change tags, which is how they expect automation to work.
For example: having a 0.0.1-SNAPSHOT tag (or latest), and just changing that on dockerhub to point to the new thing.

For the record, this is a big no-no, and expressly verboten by folks like Kelsey. The right thing to do here is education, not acquiescence.

I agree with Peter on this one, if it's a terrible practice (and it's easy to see that it is) we ought to educate, rather than encourage.

and it's easy to see that it is
Not convinced of that, but fine. That's all a reasonable argument.


exoscale-cli: 1.12.0 -> 1.14.0

Motivation for this change
Update to latest release of the exoscale cli
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

/marvin opt-in
/status needs_reviewer

Binary executes.

/status needs_work

@timokau What am I doing wrong here?

The needs_work status doesn't exist anymore since https://github.com/timokau/marvin-mk2/issues/29. The closest equivalent is awaiting_changes. Most of the time you should just let Marvin do its thing though, you don't have to set the state explicitly. It sets to awaiting_changes when you leave a change-requesting review. Its has recently become a bit more conservative with switching to that state (https://github.com/timokau/marvin-mk2/pull/58), since it doesn't hurt much to have a PR in awaiting_reviewer by mistake but it may hurt a lot to have it in awaiting_changes by mistake.
Marvin should of course have linked the usage document on opt-in, it failed to do so (https://github.com/timokau/marvin-mk2/pull/62).

@timokau Thanks for your detailed explanations.
/status awaiting_changes

@turion  Err... what needs to be changed here?

@turion Err... what needs to be changed here?

As I said in https://github.com/NixOS/nixpkgs/pull/92705#discussion_r451515738 it would be nice to split into two commits where one contains your refactoring and the other does the version bump :)

Result of nixpkgs-review pr 92705 1

1 package built:
- exoscale-cli

Awesome!
/status needs_merger


Celo verification deep links not clickable on some android phones

Expected Behavior
Link in SMS is clickable on all (or at least most) phones
Current Behavior
Reproducible on our Huawei + Honor phones, the links are in the SMS but don't get parsed into something clickable. There may not be anything we can do about this but we should investigate.

@i1skn FYI just made this to track


drools-pmml: don't print unnecessary info to sysout during tests

Agree it should not go to sysout.
The test should make some assertions though:

validate against the xsd schema
check that the NN parameters in the XML matches the expected content

@sotty I agree about the assertions, but I am afraid I don't have the knowledge to add those (at least not without some non-trivial time spent studying the pmml module). I merely tracked down the source of those messages and disabled them. Would it be possible for you to take a look at adding those assertions?

I have created a PR with an improved version of this test

Thanks @sotty, I am closing this PR in favor of #484


[otbn] Use RND/URND in software

In OTBN software, e.g. in the p256 library, we currently explicitly set blinding parameters and random numbers. These should come from RND and URND instead, which are now hooked up.

Nonce for p256 and p384 (RND)
Potentially for the blinding constant for ECDSA (URND or RND)

Update the software to use these WSRs.

estimate 4

Prep work going on in #7374

Another prep PR in https://github.com/lowRISC/opentitan/pull/7466.
I have a patch on top of that that uses RND and URND, but it't not yet completely working, likely to to some software verification error on my end. I'll continue debugging this, but I don't expect that much is left.


Refactor in #104 broke #108 keycode handling fix.

#104 was applied after #108, reverting its fix in some way.
   // Won't fire after #104
    use_hotkeys!(("control+keyk") => move |_| {
        tracing::info!("PRESSED!");
    });

Commit: ad8fa5774c6bfe254b128d2358b706e570c87389 (#104) broken
Commit: e135560ccdcfbb66888a219e6bcb95f02f068048 (commit prior) working
Ever since #108 I'd been using that specific git hash (waiting for a release), just updated to the new release and realised.

@zakstucke thanks!

@zakstucke can I get your discord? You can add me matteobacalao

Closed via #117


[AppCheck]: Pub get fails after share_plus releases new versions

Is there an existing issue for this?

[X] I have searched the existing issues.

Which plugins are affected?
App Check
Which platforms are affected?
Android, iOS, macOS, Web, Windows
Description
After the new releases of share_plus, 10.0.1 and 10.0.2, app_check don't allow pub get, forcing us to override the dependency version.
Because no versions of firebase_app_check match >0.3.0+4 <0.4.0 and firebase_app_check 0.3.0+4 depends on firebase_app_check_web ^0.1.2+12, firebase_app_check ^0.3.0+4 requires firebase_app_check_web ^0.1.2+12.
And because no versions of firebase_app_check_web match >0.1.2+12 <0.2.0, firebase_app_check ^0.3.0+4 requires firebase_app_check_web 0.1.2+12.
And because firebase_app_check_web 0.1.2+12 depends on web ^0.5.1 and share_plus >=10.0.1 depends on web ^1.0.0, firebase_app_check ^0.3.0+4 is incompatible with share_plus >=10.0.1.
So, because loterias depends on both share_plus ^10.0.2 and firebase_app_check ^0.3.0+4, version solving failed.

You can try the following suggestion to make the pubspec resolve:
* Consider downgrading your constraint on share_plus: flutter pub add share_plus:^10.0.0

Note that it says: firebase_app_check ^0.3.0+4 is incompatible with share_plus >=10.0.1
The last workable version is 10.0.0
10.0.1 and 10.0.2 results in the same problem, forcing override dependencies to fix pub get.
Reproducing the issue
Uses share_plus at new versions, under 10.0.0
Firebase Core version
3.3.0
Flutter Version
3.24.0
Relevant Log Output
Because no versions of firebase_app_check match >0.3.0+4 <0.4.0 and firebase_app_check 0.3.0+4 depends on firebase_app_check_web ^0.1.2+12, firebase_app_check ^0.3.0+4 requires firebase_app_check_web ^0.1.2+12.
And because no versions of firebase_app_check_web match >0.1.2+12 <0.2.0, firebase_app_check ^0.3.0+4 requires firebase_app_check_web 0.1.2+12.
And because firebase_app_check_web 0.1.2+12 depends on web ^0.5.1 and share_plus >=10.0.1 depends on web ^1.0.0, firebase_app_check ^0.3.0+4 is incompatible with share_plus >=10.0.1.
So, because loterias depends on both share_plus ^10.0.2 and firebase_app_check ^0.3.0+4, version solving failed.

You can try the following suggestion to make the pubspec resolve:
* Consider downgrading your constraint on share_plus: flutter pub add share_plus:^10.0.0

Flutter dependencies

Expand Flutter dependencies snippet

  share_plus: ^10.0.2
  firebase_core: ^3.3.0
  firebase_app_check: ^0.3.0+4

Additional context and comments
No response

Does it work without mentioning a certain version of share plus.
I faced similar issue, and left the version blank.

Duplicates of https://github.com/firebase/flutterfire/issues/13190, check solution at https://github.com/firebase/flutterfire/issues/13190#issuecomment-2295927416


Jericho OXF

Incorrect entries have been added to the Jerico entry - church name only and not following the greater to smaller rule.
Please move the search records for both St Barnabas entries to Jericho St Barnabas and remove the incorrect entries.
Geoff

Done


Syntax highlighting doesn't work

let
  pkgs = import (builtins.fetchTarball {
    # nixos unstable
    url = "https://github.com/nixos/nixpkgs-channels/tarball/a31736120c5de6e632f5a0ba1ed34e53fc1c1b00";
    sha256 = "0xfjizw6w84w1fj47hxzw2vwgjlszzmsjb8k8cgqhb379vmkxjfl";
  }) {config = {}; overlays = [];};
  vscode-nix-ide = builtins.fetchGit {
    url = "https://github.com/nix-community/vscode-nix-ide";
    rev = "699c803e056d9ef080061c211ace0518dbcb4fc4";
  };
  codium-nix = pkgs.writeScriptBin "codium-nix" ''
    #!/usr/bin/env bash
    ${pkgs.vscodium}/bin/codium --extensions-dir ${
      pkgs.runCommand "extensions-dir" { buildInputs = [ vscode-nix-ide ]; } ''
        mkdir $out
        cp -r ${vscode-nix-ide} $out/vscode-nix-ide
      ''
    } --user-data-dir $(pwd)/.data $@
  '';
in
pkgs.mkShell {
  buildInputs = [ codium-nix ];
}

Inside this shell, run codium-nix. The extension is enabled in the settings, but doesn't seem to do anything

the extension needs to be compiled. It is in Typescript and also the syntax file need to be compiled to json. please check the package.json script commands

https://github.com/nix-community/vscode-nix-ide#development
yarn build

is a build step needed. Or you could directly use the vix file from vscode extension store.

Thanks a lot!!
Currently I'm trying to figure out how to get a permanent of the vsix extension file.
https://marketplace.visualstudio.com/_apis/public/gallery/publishers/jnoortheen/vsextensions/nix-ide/0.1.3/vspackage
raises error 429 when downlaoded from cmdline

The issue is related to build steps rather than the code itself. I am closing it. Feel free to ask any issues encountered building with nix.


Update ECDSA signature

Pdo should move from secp256k1 to 384. secp256k1 support could remain when used with Sawtooth. Other ledgers such as CCF should be synced to use the upgraded one.

Fixed in #366 and #369


Improvement: DO-1959: Support setting metadata in extra attributes

Motivation and Context
Currenlty, if we set any metadata when updating an edge/node, it will overwrite all other transformations in the serializers. This PR fixes this inconsistency.
Implementation Description

Any new dependencies Introduced

How Has This Been Tested?
Locally
PR Checklist:

[x] I have implemented all requirements? (see JIRA, project documentation).
[x] I am not affecting someone else's work, If I am, they are included as a reviewer.
[x] I have added relevant tests (unit, integration or regression).
[x] I have added comments to all the bits that are hard to follow.
[x] I have added/updated Documentation.
[x] I have updated the appropriate changelog with a line for my changes.

Screenshots (if appropriate):

Hi, could you add a test for the extras with meta scenario?


Some of our pages could be made more clear by showing the data model the operation is working on and the result of the query

For instance .changeAt() - http://rethinkdb.com/api/javascript/change_at/
I think it should something like this down below the ReQL query just to be completely explicit and clear. A lot of other pages could benefit from this.
["Iron Man", "Hulk", "Spider-Man"]

List forthcoming, but wanted to put this down.

Been digging into the documentation recently and what I'm trying to understand is that given a table or two tables, certain operations/queries produce a certain result.
The process of assembling a mental model of how all the pieces work together would be greatly aided by a single complex data structure that I could reference. The various table and model structures - blogs, marvel heroes, users, etc - just add another layer of complexity. One data structure able to be referenced from any query/transformation code sample would help a lot.

I think this is a good idea, but it's also a lot of work. We could start doing this one command at a time, and build up the reference data set as we go.
I think we need to address some other things in the documentation first (especially in the Horizon documentation), but let's keep this in mind for later.


New layer adder and layout changer

More intuitive layer adder with filtering
Layout manager for

layouts (single column, square, etc.)
Custom data sources
Change projection
Bounding box selection
preset loading and saving

Top!!


No such file or directory in Unknown

Hello,
the project is https://github.com/qkqpttgf/OneManager-php ,
deploy: https://scfonedrive.github.io/Vercel/Deploy.html ,
my app: https://verceltest-ysun.vercel.app/ .
I deploy the project, it works at that time,
but,
atfer a few hours, (not visit for one night), it show `
Warning: Unknown: Failed to open stream: No such file or directory in Unknown on line 0
Fatal error: Failed opening required '/var/task/user/index.php' (include_path='.:/opt/remi/php80/root/usr/share/pear:/opt/remi/php80/root/usr/share/php:/usr/share/pear:/usr/share/php') in Unknown on line 0
the vercel seethat is not supported by Vercel's support team`,
what can I do now？

Can you test redeploy?

Can you test redeploy?

tested, it work after deploy, but error after some hours( I dont know how many hours)

I will reach Vercel team and we will see.

I will reach Vercel team and we will see.

the demo in #180 is error now, after some hours,
so, help.
thank you.

Hello?

i have a same error as you. do you have a solution at now?

Does it still an issue? I've tested it again and it works seems to me.

Does it still an issue? I've tested it again and it works seems to me.

It looks work now, how it happen, and what can I do?

It looks like vercel team fixed runtime API/workflow.


Bug 1794376: config: disable IPv6DualStack feature flag

We are not going to ship dual stack support in 4.4 and enabling this flag bricks the clusters as the service object ip family field is being defaulted (operators with old clients are affected).

/cherry-pick release-4.4

oops, now i see there's no 4.4 branch, and it's just other repos that need to get an openshift/api update in 4.4 ...

(There is a 4.4 branch, it's just that it already has this commit on it.)


Better error handling for requests to Bugsnag

I was having problems with big reports throwing the agent into an endless loop trying to send the same report over and over again. Since there was no delay this continued until someone restarted the agent, creating huge network spikes.
This commit solved two problems:
Remove report for client errors (4xx):
Actually I see now in the docs that a 408 could be return (on timeout), so perhaps the removal of the report should only happen on a 400 status.
Also sleep for 5 sec before enqueuing  (useless otherwise because of threading)

Thank you, @bix0r! I've merged the changes as well as ensured timeouts get retried.


[Tock OS/Test Rom] Entry point with an absolute address / load error

Hey!
I'm trying to get Tock OS working on the latest master of OpenTitan. However, I've come across  this issue where at this point of the test_rom
https://github.com/lowRISC/opentitan/blob/b05a44082aadb68759622caa3d819296a9327042/sw/device/lib/testing/test_rom/test_rom.c#L142
we return manifest + manifest->entry_point.
When Tock is built, it has an absolute address for the entry point, so manifest + manifest->entry_point turns out to be something like 0x20000000 + 0x20000400 (in the test rom) which is invalid. This is the current linker for Tock.
MEMORY
{
  rom   (rx)  : ORIGIN = 0x20000000, LENGTH = 0x30000
  /* Support up to 0x2009_0000 for apps
   * and 0x2009_0000 to 0x2010_0000 is for flash storage.
   */
  prog  (rx)  : ORIGIN = 0x20030000, LENGTH = 0x60000
  /* The first 0x650 bytes of RAM are reserved for the boot
   * ROM, so we have to ignore that space.
   * See https://github.com/lowRISC/opentitan/blob/master/sw/device/silicon_creator/lib/base/static_critical.ld
   * for details
   */
  ram   (!rx) : ORIGIN = 0x10000650, LENGTH = 0x10000 - 0x650
}

MPU_MIN_ALIGN = 1K;
SECTIONS {
    .manifest ORIGIN(rom):
    {
      _manifest = .;
      /* sw/device/silicon_creator/lib/manifest.h */
      . = . + 384; /* signature */
      . = . + 4;   /* usage_constraints.selector_bits */
      . = . + 32;  /* usage_constraints.device_id */
      . = . + 4;   /* usage_constraints.manuf_state_creator */
      . = . + 4;   /* usage_constraints.manuf_state_owner */
      . = . + 4;   /* usage_constraints.life_cycle_state */
      . = . + 384; /* modulus */
      . = . + 4;   /* address_translation */
      . = . + 4;   /* identifier */
      . = . + 4;   /* length */
      . = . + 4;   /* version_major */
      . = . + 4;   /* version_minor */
      . = . + 4;   /* security_version */
      . = . + 8;   /* timestamp */
      . = . + 32;  /* binding_value */
      . = . + 4;   /* max_key_version */
      . = . + 4;   /* code_start */
      . = . + 4;   /* code_end */
      LONG(_stext); /* entry_point */
      . = . + 128; /* padding */
      /* size = 1024 bytes */
    } > rom
}

INCLUDE ../../kernel_layout.ld

Is there an easy solution to this for getting a relative address offset for the entry point (something like position independent code)?  Currently I have Tock running on master with the test_rom modified to just return manifest->entry_point (instead of the +manifest). Hacky workaround though.
Thanks,

Fixed with  LONG(_stext); /* entry_point */ -> SHORT(_stext);


Description

Description
All methods in TWString.h have visibility set to default using  __attribute__((visibility("default"))). This allows dart ffi (for flutter applications) to find the methods. When the attribute is not set dart ffi throws lookup errors.
Fixes #2013
Testing instructions
No new tests needed. Run normal android and iOS tests
Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist

[x] Prefix PR title with [WIP] if necessary.
[x] Add tests to cover changes as needed.
[x] Update documentation as needed.
[x] I have read the guidelines for adding a new blockchain
[x] If there is a related Issue, mention it in the description (e.g. Fixes #<issue_number> ).

Originally posted by @hnord-vdx in https://github.com/trustwallet/wallet-core/pull/2033

@arslan-raza-143

I> ## Description

All methods in TWString.h have visibility set to default using  __attribute__((visibility("default"))). This allows dart ffi (for flutter applications) to find the methods. When the attribute is not set dart ffi throws lookup errors.
Fixes #2013
Testing instructions
No new tests needed. Run normal android and iOS tests
Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist

[x] Prefix PR title with [WIP] if necessary.

[x] Add tests to cover changes as needed.

[x] Update documentation as needed.

[x] I have read the guidelines for adding a new blockchain

[x] If there is a related Issue, mention it in the description (e.g. Fixes #<issue_number> ).

Originally posted by @hnord-vdx in https://github.com/trustwallet/wallet-core/pull/2033


Fixed ggt synching to production when it's a fresh start

This PR in conjunction with https://github.com/gadget-inc/gadget/pull/10083
Should fix ggt ever touching production again and mucking things.

🙌
Can we add a test for this inside spec/services/filesync/sync-json.spec.ts

Added two since it felt important to test both cases. Let me know how you feel about em.

:chef-kiss:

sorry @scott-rc I added the subdomain being correct when multi-environment is on, if you wanna take another look.