text:
Please reactivate removed repo (orocrm/composer)

Hi,
we are still using oro/platform@v1.9.3 which has a dependency on composer/composer@v1.0.0-p1 (here).
Unfortunately, you removed the repository.
Could you please reactivate this repo? Is it possible to use one of the official newer composer versions?
We would host it on our github. Perhaps you can send us the code?
Best regards

Hi @sbani
Do you mean this repo https://github.com/oroinc/composer ?

Hi @sbani
Repository was restored, 1.9.3 is very old version of the platform, please consider upgrade to a newer one.

Thank you @DimaSoroka. I know, but we are not able to upgrade for now.

I think this still might be an issue. Our composer still fails on install:
- Installing composer/composer (1.0.0-p1): Downloading (failed)    Failed to download composer/composer from dist: The "https://api.github.com/repos/orocrm/composer/zipball/833ce984264204e7d6576ab082660105c7d8f04c" file could not be downloaded (HTTP/1.1 404 Not Found)
   Now trying to download from source
 - Installing composer/composer (1.0.0-p1): Cloning 833ce98426

 [RuntimeException]
 Failed to execute git clone --no-checkout 'https://***:***@github.com/orocrm/composer.git' '/var/www/sites/ca1/www/vendor/composer/composer' && cd '/var/
 www/sites/ca1/www/vendor/composer/composer' && git remote add composer 'https://***:***@github.com/orocrm/composer.git' && git fetch composer
 Cloning into '/var/www/sites/ca1/www/vendor/composer/composer'...
 remote: Repository not found.
 fatal: repository 'https://***:***@github.com/orocrm/composer.git/' not found

It looks like https://packagist.orocrm.com/packages/composer/composer is still referencing the old repo: github.com/orocrm/composer.git

@danbentley, add this to you composer.json:
    "repositories": [
...
        {
            "type": "vcs",
            "url": "https://github.com/oroinc/composer.git"
        }
...

Thanks @sbani.
For anyone else with the same issue, it appears the order of entries in the "repositories" array is significant. This fixed it for us:
…
"repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/oroinc/composer"
        },
        {
            "type": "composer",
            "url": "https://packagist.orocrm.com"
        }
    ],
…

I have the same problem and tried all of the above to no avail... is there anything else to do about this?

@dave08  what version of OroPlatform do you have?

@mbessolov I have 1.9.3, but don't have time to update to 1.10... I also just tried running composer update and it gives me:
oro/platform 1.9.3 requires sensio/distribution-bundle 4.0.3 -> satisfiable by sensio/distribution-bundle[v4.0.3] but these conflict with your requirements or minimum-stability.

My Jenkins pipeline (while generating docker image) gives me (I'm using a github token...):
  - Installing composer/composer (1.0.0-p1)

    Downloading: Connecting...    Downloading: Failed

    Failed to download composer/composer from dist: The "https://api.github.com/repos/orocrm/composer/zipball/833ce984264204e7d6576ab082660105c7d8f04c" file could not be downloaded (HTTP/1.1 404 Not Found)

    Now trying to download from source

  - Installing composer/composer (1.0.0-p1)

    Cloning 833ce984264204e7d6576ab082660105c7d8f04c

  [RuntimeException]

  Failed to clone https://github.com/orocrm/composer.git via https, ssh protocols, aborting.

  - https://github.com/orocrm/composer.git

    Cloning into '/app/vendor/composer/composer'...

    remote: Invalid username or password.

    fatal: Authentication failed for 'https://github.com/orocrm/composer.git/'

  - git@github.com:orocrm/composer.git

    Cloning into '/app/vendor/composer/composer'...

    Host key verification failed.

    fatal: Could not read from remote repository.

    Please make sure you have the correct access rights

    and the repository exists.

@dave08 try to change the URL to https://github.com/oroinc/composer
orocrm org has been renamed to oroinc quite a while ago and apparently GitHub does not maintain redirects that long.

@mbessolov This is my repositories:
    "repositories": [
        {
            "type": "vcs",
            "no-api": true,
            "url": "https://github.com/oroinc/composer.git"
        },
        {
            "type": "vcs",
            "no-api": true,
            "url": "https://github.com/oroinc/composer"
        },
        {
            "type": "composer",
            "url": "https://packagist.orocrm.com"
        },
        {
            "type": "composer",
            "url": "https://packagist.orocrm.com"
        }
    ],

@mbessolov @DimaSoroka could we transfer this repo to orocrm organization and then return to oroinc? It creates all required redirects.

@aivus It will wipe the repository permissions and somebody will have to spend a few hours to return them back to the original state, which is too much for an unsupported version. And it is not necessary, as the application installs with a simple modification to composer.lock.
@dave08 what you posted looks like a fragment of composer.json, and this not the same composer.json that is included in oroinc/platform-application 1.9.3 (original composer.json)
You may not need any of those new repositories to you composer.json
You need to modify 2 lines (537, 542) in composer.lock. Replace /orocrm/composer/ with /oroinc/composer/ and run composer install again. Please note that the line numbers may be different, so just look for all references to orocrm/composer in your composer.lock

authors: DimaSoroka aivus danbentley dave08 mbessolov sbani

repo: oroinc/platform


text:
Setting :tvos => true turns off support for :ios

As seen in the segmentio subspec. I removed :tvos => true and it works. Otherwise I get:
[!] The platform of the target `Paddle8` (iOS 8.0) is not compatible with `ARAnalytics/Segmentio (3.8.0)`, which does not support `ios`.

More CocoaPods 0.39 fun?

I think this got fixed in the latest master!

source: gharchive/issue

authors: orta sgtsquiggs

repo: orta/ARAnalytics

license: mit

license_type: permissive

license_source: bigquery


text:
README.MD is extremely out of date

Examples given in the README.MD no longer work and require editing afterwards. As someone who's new to Swift, it makes it difficult to adopt and use the framework. Although I'm grateful for all the hard work you put into this 👍

I agree.  I'm inheriting a project which used the old style and am clueless about how to proceed.

authors: AlexanderBollbach Saracevas

repo: ortuman/SwiftForms


text:
Make C# API COM visible

Makes loading the library in Excel via VBA possible

I've never done this but it looks pretty easy. Have you ever done this @bstabler? It looks like you just put a little preamble before each class declaration, like this:
[ComVisible(true)]
[Guid("7884998B-0504-4CBE-9FF9-6BBAA9776188")]
[ClassInterface(ClassInterfaceType.None)]
[ProgId("MyNamespace.MyClass")]
public class MyClass : IMyClass
{
}
from
http://www.codeproject.com/Articles/612604/Best-Practice-in-Writing-a-COM-Visible-Assembly-Cs
I don't really know how to test it either.

I have not built COM functionality in C#, but I have done lots of VBA and COM programming in Excel.  I think you would test this by writing VBA in Excel using your COM library.  To start, turn on the Developer Ribbon in Excel.  Then click Visual Basic to get the VBA Editor.  Then you add COM libraries via Tools + References.  Finally, write a little VBA program to test it out.

Okay, I looked at this again and seems like the code should be com visible because in AssemblyInfo.cs, we have it set like this [assembly: ComVisible(true)]
I started to try to test this by writing a vb script. I was able to add the dlls needed CSharpOMX.dll and HDF5DotNet.dll.  Then I tried to start writing code and realized I haven't written VB in ten years so it would take me too long to try it out! So I think I need to pass this off to someone else.

Two updates:

In order to add the type library in the VBA editor, you first have to create a type library file.  You can do this as follows:

RegAsm.exe C:...\CSharpOMX.dll /tlb:omx.tlb
This creates an omx.tlb that you can add via Tools + References

Unfortunately I don't believe the C# API will work in COM since COM does not support static methods.  I tested this by trying to call the static read OMX matrix method in VBA, but it doesn't work.  So, @Ennazus (or @mmilkovits or me) needs to make some changes to get this to work.  Here is a link to this issue.

http://stackoverflow.com/questions/24193183/can-i-call-a-static-method-of-a-c-sharp-class-from-vba-via-com

That's too bad about the static methods - I'm a bit buried now, but I can take a look later on this month.  I'll touch base before I do.

I'm closing this since we should move this to https://github.com/osPlanning/omx-csharp

authors: Ennazus bstabler mmilkovits

repo: osPlanning/omx


text:
v1: Enable rhel-edge-container

[x] https://github.com/osbuild/osbuild-composer/pull/2035

Hmm... Does the edge team want these containers? I thought they only wanted the simplified installer?

I'm not sure actually, @achilleas-k  do you remember? I'll open a PR in the meantime with the extra types for rhel-8.5 because those are important (and unrelated).

Hmm... Does the edge team want these containers? I thought they only wanted the simplified installer?

I'm not sure actually, @achilleas-k do you remember? I'll open a PR in the meantime with the extra types for rhel-8.5 because those are important (and unrelated).

I don't remember if they asked for them either.  I think we decided to add it anyway since it's an image we build and it's useful.  The general plan is to add every image type to the backend eventually, isn't it?

I think I may have been the source of the confusion :-/

rhel-edge-container is

source: gharchive/pull-request

authors: Gundersanne achilleas-k teg

repo: osbuild/image-builder

license_source: github-api


text:
Add additional ignore entries

This repo is no longer being maintained. In order to get at least some testing for new nightlies I'm adding all currently failing entries to the db-ignore file. Most of them are ostree/edge which should not work here in the first place.

Considering this repo is basically in deprecation mode we agreed with @lavocatt that makes sense to just add all the currently failing test cases to the db-ignore to get at least some testing on the new nightlies before this can be taken over by the images repository. Most of them are edge type images which should be there in the first place, the only one that really stood out to me is ova image type. I just hope it won't create a false sense of "all is good" but since we're testing all the important image types in integration tests in osbuild-composer repository I think we should be good. The pipeline passed here https://gitlab.com/redhat/services/products/image-builder/ci/manifest-db/-/pipelines/1100165687 when I tried it manually before opening this MR and Schutzbot even created a PR to update the db: https://github.com/osbuild/manifest-db/pull/99
What do you think about this @atodorov @achilleas-k ?

repo: osbuild/manifest-db


text:
Update RHEL and Fedora test cases

This pull request includes:

[x] adequate testing for the new functionality or fixed issue
[x] adequate documentation informing people about the change

Regenerated manifests for test cases for RHEL 8.3, RHEL 8.4, Fedora 32, and Fedora 33.  Changes in osbuild affect the image info, specifically the output of rpm --verify.  In RHEL images, the /etc/machine-id file no longer appears as modified, but in Fedora images it does.  This has been causing CI failures in recent osbuild builds.
For images that haven't been updated in a while, the RT repo was also added by the test case generator.
Schutzfile changed to use current main for osbuild: https://github.com/osbuild/osbuild/commit/6c68e45dbe6b8e7c7a7c40eed1ff66ee4932b050

Converting to draft until all changes are covered.

This is ready but we'll have to wait for CI to make sure it's all good.  Marking as ready.

Ouch, this now needs a rebase. :(

Not sure what to do with this PR.  It'll be needed if we reintroduce the machine-id truncation, which was reverted, but there might be other changes in the manifests until then.
I'm going to close it and we can reopen it (or remake it) when the time comes.

authors: achilleas-k ondrejbudai

repo: osbuild/osbuild-composer


text:
koji: log unsuccessful requests only once

This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change such as

[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

@diaasami the changes look good, but I would like to ask for more details on the intention of this change. E.g. why we do not want to log each retry any more? Happy to approve afterwards.

@diaasami the changes look good, but I would like to ask for more details on the intention of this change. E.g. why we do not want to log each retry any more? Happy to approve afterwards.

The motivation for this is 1. not to log successful requests, which are not retried and 2. to not log twice.
we log retries, but this happens inside retryable-http so it's not visible here, what we are logging here is the decision to retry or not (which is based on the library function rh.DefaultRetryPolicy), if it's a yes we will see the logs of retrying, but if it's a no, we will not so we need to see that decision was taken and may be later change the retry policy.

authors: diaasami thozza


text:
Test: make partitioning tools stage tests pass on RHEL-8 + add unit test for sfdisk stage

The stage tests for partitioning tools use sfdisk --json to inspect the produced image partition table and compare the result with the expected JSON output. However, the RHEL-8 version of sfdisk does not include sectorsize in its output, which made tests fail on RHEL-8.
As a workaround, delete the sectorsize key from the expected JSON if the key is not present in the sfdisk --json output.
Also found that sfdisk stage had no test along the way, so add that one as well.

Added a new commit refactoring the implementation of unit test for partitioning stages + rebased on top of main.

repo: osbuild/osbuild


text:
Parse PHP comments on the line before

This aims to address #161 by simply adding all comments on the line before to the current function.
It seems to work fine, except that the Twig parser creates files like this:
/*  */
class __TwigTemplate_79a7d3b32ac8bc226e4f26ef9125e6a65dde2b398ad080a22197a3f33e8538b5 extends Twig_Template
{
    public function __construct(Twig_Environment $env)
    {
        parent::__construct($env);

        $this->parent = false;

        $this->blocks = array(
        );
    }

    protected function doDisplay(array $context, array $blocks = array())
    {
        // line 1
        echo "<header>
    <h1>";
        // line 2
        echo gettext("text 1");
        echo "</h1>
</header"...

For some reason, these line comments are now being picked up as translator comments and let the tests fail.

Now, all these comments are extracted, so I suggest to provide a way to configure whether the phpcode parser should extract the comments or not, in order to disable it on handle php returned by template compilers like twig or blade.

I think it might be even simpler than that.
In tests/bootstrap.php there's this line:
Gettext\Extractors\PhpCode::$options['extractComments'] = '';

This tells all extractors to extract comments by default, no matter what prefix they have. However, according to http://twig-extensions.readthedocs.io/en/latest/i18n.html, Twig templates will always have the notes: prefix in the translator comments, no matter what.
Thus, I'd suggest not to use $options['extractComments'] = '' for the Twig extractor and instead use $options['extractComments'] = 'notes:'.
See c91726a346a704ca83046ec50abf82d734050556 for an example. Instead of putting that into bootstrap.php I'd probably enforce that in \Gettext\Extractors\Twig::fromString().

Ok, that's great.
Yeah, I think putting this option as default here https://github.com/oscarotero/Gettext/blob/master/src/Extractors/Twig.php#L17 should be enought.

Added now, although that shouldn't really make any difference :-)

Thank you very much!

authors: oscarotero swissspidy

repo: oscarotero/Gettext


text:
Input casting/type

The invoke function casts falsy values to undefined. This means values like 0 and '' turn into undefined.
api.invoke(key, input || undefined);

nvm, this doesn't have any effect since the input type only accepts null | {...}

repo: oscartbeaumont/tauri-specta

license: MIT


text:
Svelte TypeScript example

This is exactly the project I was hoping to see. Really hope it takes off 🙌
It isn't immediately clear how to use it to bundle a Svelte app. Would you be able to add an example?

Thank you for kind words @ebeloded. Svelte Compiler is authored and intended to be used with JS build tools. I want to get to it once the go api is flushed out and sorvor is little more stable. I will keep this issue open to track the efforts and to hear ideas about the integration from anyone interested.

authors: ebeloded osdevisnot

repo: osdevisnot/sorvor


text:
docker compose ppolicy problems

I am trying to add the ppolicy module and overlay to my server with an initial ldif file. with the file I am able to add users and create groups and even add the ppolicy itself. But when I try to add the ppolicy module and the over lay
Entry 22: cn=module{0},cn=config
dn: cn=module{0},cn=config
changetype: add
add: olcModuleLoad
olcModuleLoad: ppolicy
dn: olcOverlay=ppolicy,olcDatabase={1}mdb,cn=config
changetype: add
objectClass: olcPPolicyConfig
objectClass: olcOverlayConfig
olcOverlay: ppolicyolcPPolicy
Default: cn=normalPolicy,ou=Policies,dc=aipadmin,dc=comolcPPolicy
HashCleartext: TRUE
olcPPolicyUseLockout: TRUE
and this is my docker-compose file
services:
openldap:
build:
context: .
dockerfile: ldapdockerfile
container_name: openldap
environment:
LDAP_ORGANISATION: my server
LDAP_DOMAIN: server
LDAP_BASE_DN: admin
LDAP_ADMIN_PASSWORD: admin
tty: true
stdin_open: true
command: --copy-service
volumes:
- /var/lib/ldap
- /etc/ldap/slapd.d
ports:
- "389:389"
I have been trying to find anything on having ldapmodify commands run on start up with the docker-compose but I haven't found anything.

Create volume:
docker volume create ldap-policy
Create new file with your entry and save to location volume e.g. /var/lib/docker/volumes/ldap-policy/_data/00-ppolicy.ldif
In docker-compose:
service:
volumes: - ldap-policy:/container/service/slapd/assets/config/bootstrap/ldif/custom
at the end:
volumes:         ldap-policy:                 external: true
Done.

@wolacinio sorry, i am not really understand this command ldap-policy: external: true, is it mean ```
volumes: ldap-policy
external: true
but raise error `external` is invalid .

You must use indentation for external: true

You must use spaces:
volumes:
  ldap-policy:
    external: true

Docs https://docs.docker.com/compose/compose-file/#external

it seem not work, raise error
services.openldap.volumes contains an invalid type, it should be an array

You must use spaces:
volumes:
  ldap-policy:
    external: true

Docs https://docs.docker.com/compose/compose-file/#external

there no error again, but it seems not work, i use keycloak to update ldap user password, ldap user password still show password with clear type, not sha type that i wanted

use keycloak update ldap user password:

ldap password:

Solved it by following issue #208

authors: Vickey-Wu ppolicherla wolacinio

repo: osixia/docker-openldap


text:
User "Lycée Louis Pasteur de Bogota" not identified

Hi, I have configured the Welcome tool for Bogota, and for Colombia, and in parallel I have the Pascal Neis tool to report new users by RSS. However, the user "Lycée Louis Pasteur de Bogota" has not been identified in the Bogota region, and it has already done many changes.
The RSS is http://resultmaps.neis-one.org/newestosmfeed.php?lon=-74.109993&lat=4.6487085&deg=0.38 and the user started on March 7th.
This is the list of users from Bogota https://welcome.osm.be/bogota/list
What could be causing this difference? thank you

Probably related indeed.
I fully rely on OSMCha API for mappers detection and ResultMap probably uses a different way of detecting new mappers.
I'll need to investigate further.

This issue is now most likely fixed (see #801) !
There was an issue with the daily update if one of the new mapper detected by OSMCha deleted his/her account meanwhile.
There was another issue if there were too many new mappers ; the request to get their details was truncated.

authors: angoca jbelien

repo: osmbe/osm-welcome-tool

license: ISC


text:
Multipolygons with touching inner rings

As stated in http://wiki.openstreetmap.org/wiki/Relation:multipolygon#Touching_inner_rings this is the OSM exception from standard OGC Simple Features. I've noticed that a lot of multipolygons use this feature. And every day more and more of them appear (mostly forests and other very complicated areas). libosmium considers them as invalid. This results in maps created with the help of libosmium look not correctly.

Osmium can usually handle this case. If you have specific examples that libosmium doesn't handle the way you think they should be handled, tell us. Also there were a lot of changes regarding multipolygon handling recently and there are different ways to create multipolyons with libosmium (different Assembler, Collector and Manager classes) and the version of libosmium you are using is also relevant.

I'm using pyosmium, have now updated to 2.12.4 (installed from Debian unstable packages because pip3 install failed).
I use wkbFactory.create_multipolygon(o) to construct polygons. Retried my scripts after update and still get invalid area exceptions. Example relations:
https://www.openstreetmap.org/relation/7356538 - the simplest one
https://www.openstreetmap.org/relation/7360840
https://www.openstreetmap.org/relation/7401189
https://www.openstreetmap.org/relation/7411548
(No, I'm not the author of these changesets but I've checked with JOSM and iD editors: they all validate and are correctly rendered by OSM renderer.)

This looks like cases of this here: https://github.com/osmlab/fixing-polygons-in-osm/issues/31 . Libosmium currently can't handle these and I am not sure what to do, I haven't thought through this completely. I encourage everybody to not map something like this. Even if this might be technically correct (and I am not convinced yet that we can have an iron-clad specification that explains how these cases are to be handled), this is extremly difficult for a mapper to understand. I think it would be better to split this up into smaller multipolygons.

It's mostly that case but not exactly. In my examples ? area is also defined as inner ring so there is no any fake hole.
As I understand inner rings are joined if they are touching each other. I my example all three inner rings should collapse into one.

Just another complex multipolygon that produces error but renders nicely by CartoCSS:
http://www.openstreetmap.org/relation/6677807

authors: andreynovikov joto

repo: osmcode/libosmium


text:
Custom projection using 6.0.1

I'm aware that osmdroid is designed to use the Web Mercator projection. However, it was trivial to use another projection in v5.6.5 simply by subclassing the Projection class, overriding few methods and MapView.getProjection(). This is not true anymore, for myriad of reasons.
This is mainly due to heavy usage of private instance methods with complex getters/logic. MapView.getProjection() is a good example of this problem (new Projection(this) should probably be in a separate protected method. Perhaps the same with new MapController()). The constructor for the Projection class is another example (mOffsetX/Y & TileSystem).
Sadly, it's quite hard to write a new class from scratch, because interfaces are not used much - for example observe how IGeoPoint is cast to GeoPoint in MapView.setExpectedCenter(). Projection, MapView, MapController etc are all tightly connected. Overall, it's much easier to add customisation hooks to key points.

I agree. I'll see what we can do. If you want to open a PR we'll gladly accept it

I'll have a look over the weekend

In Projection there are now just a couple of methods actually dealing with the geo projection. The other methods are needed to handle map scrolling, world wrapping and so on.
Maybe a new GeoProjection interface should be created, dealing only with the toPixel / fromPixel aspects. That would make things easier when dealing with alternate projections.

TileSystem/Projection parts need further refactoring. I created a new ITileSystem interface. I also created a small demo
public interface ITileSystem {

    BoundingBox getGeographicBoundingBox();

    double getMapSize(double pZoomLevel);

    double getTileSize(double pZoomLevel);

    PointL toPixels(double pLatitude, double pLongitude, double pMapSize, PointL pReuse);

    GeoPoint fromPixels(long pX, long pY, double pMapSize, GeoPoint pReuse);
}

feature/#971

@janekp Sorry for the late reply: I'll send you comments on your commit within a week.

Great!

@janekp As you saw I commented your commit. An additional thought: there are many places in the osmdroid code where you can see TileSystem.MinLatitude and so on. I don't know how that could be impacted too. Could you tell us a bit more about your specific implementation of the abstract TileSystem methods (cf. my comments on your code), just to check how much impact we're talking about?

TileSystem.MinLatitude is used for checking GeoPoint bounds and assumes WGS84. Ideally one could use their own bounds/system, but this is not a big problem. I can convert input coordinates to WGS84 and areas are always smaller subsets of TileSystem bounds.

@janekp I just PR'ed #1029.
Feel free to:

extend your own TileSystem (like what I did in TileSystemWebMercator)
use MapView.setTileSystem to make this implementation active
and tell me if if works or if something is missing

Nice! I added couple of comments.

PR looks good.

merged

authors: janekp monsieurtanuki spyhunter99

repo: osmdroid/osmdroid


text:
unable to login, using chromium on linux

When trying to view the website using Chromium, on Linux, I only see the throbber.

This is on Chromium version 58.0.3029.96 Built on Ubuntu , running on Ubuntu 16.04 (64-bit).
On this same machine, using Firefox, the website works fine.

Hey @govvin, are you seeing any errors in the browser console?

@ajithranka  Please see attached.
By the way, now I couldn't log-in with Firefox and Chromium
osmlab.github.io-1495532192568.log.txt

This should be fixed after to-fix-backed/issues/96.
@govvin Let me know if you are able to login now.

Yes, that's confirmed.  I noticed that earlier today. Thank you!
However, I couldn't get iD to launch for editing.

authors: ajithranka govvin

repo: osmlab/to-fix


text:
[ValSet-Pref] Module Wired up and created MsgSetValidatorSetPreference

Part of : #2579
What is the purpose of the change
This PR does the following;

[x] Setup folder/file structure for the ValSet module
[x] Setup KVStore for the module
[x] Setup Keeper/Msg_server test suite
[x] Create MsgSetValidatorSetPreference that lets users to create their validator-set
[x] Make MsgSetValidatorSetPreference module so that it can handle create + update

Brief Changelog
n/a
Testing and Verifying

Added tdd tests for MsgSetValidatorSetPreference

Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? (yes / no)
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? (yes / no)
How is the feature or change documented? (not applicable   /   specification (x/<module>/spec/)  /  Osmosis docs repo   /   not documented)

Have a few changes, one in particular that is a must. Please lmk if you have any questions!
As a side note kind of unrelated to this particular PR, this concept of storing account valset preferences on chain is kind of disturbing to me. I could be wrong / misunderstanding, but wouldn't it be extremely simple for someone to spam create accounts and set validator preferences to preference all 150 validators? I don't even think the account would need to have any balance as it stands right? I think we need to rethink the implementation if this is the case
cc: @p0mvn @mattverse

Thank you @czarcas7ic  for the feedback! Addressed everything here: bb8fb72
I do think we should have an upper limit for the # of validator we can have in a set. Yea the account donot need to have any balance to create a validator set. I can change it based on any design decision we end up making

I am going to tag @ValarDragon for his opinion on potential attack surface this would provide. I am not concerned about the number of validators an account can set. Even if it was two, if someone could create unlimited accounts with two validator preferences, this would bloat state significantly.

I am going to tag @ValarDragon for his opinion on potential attack surface this would provide. I am not concerned about the number of validators an account can set. Even if it was two, if someone could create unlimited accounts with two validator preferences, this would bloat state significantly.

My stance is that I'm not pretty much concerned about this, since we have the val set creation fee

Yes, as long as fee is implemented I am also not concerned

Left second round of reviews, overall looking good, I think we just need test cases for the community pool funding for val set creation fee

Thank you @mattverse for the feedback. Resolved all you feedback except replacing DeepEqual here 0c6ece9
I want to see if there is a much easier way of doing this other than sorting and comparing, maybe some built in function or osmo util function that i'm not aware of

Left second round of reviews, overall looking good, I think we just need test cases for the community pool funding for val set creation fee

Thank you @mattverse for the feedback. Resolved all you feedback except replacing DeepEqual here 0c6ece9
I want to see if there is a much easier way of doing this other than sorting and comparing, maybe some built in function or osmo util function that i'm not aware of

Can you assign me for review once r4r?

Ready for Review @mattverse 🌮  224bbd9

Thank you @mattverse for feedback. Resolved all of it 3458de7

Feel free to re-request for review once ur in a comfortable state for the review

Thank you @alexanderbez for your feedback! implemented all of it here c64682a

@alexanderbez are you able to check if your required change was addressed so this can get merged?

LGTM! Did we decide to limit the number of validators one can split delegate to? I dont have any preference for or against, I just remember this being talked about and I didn't see anything in ref to this in the code

Dev seem to not care much about it so i didn't include it. If this discussion opens up in the future i will add it

Thank you @alexanderbez resolved all your comments here a3a55cc

Merging, thanks again @stackman27 !

repo: osmosis-labs/osmosis


text:
Polar standard library

This is an external tracking issue to:
Gauge interest from the community for this feature
Learn about what you'd want to see out of it if we worked on it.
So please:
Upvote the issue if it's important to you, and
Comment with any relevant info on your requirements use cases, etc.
Thanks!
PS We do all our internal engineering issue tracking separately in Notion, so you won't necessarily see regular updates to the project status here even once we begin work.

What sorts of things would you like to see in the Polar standard library? So far my biggest wishes are.

More builtin list methods

Indexing
Ranges
Appending

More builtin Math

sin / cos / tan
min / max
exponential

What sorts of things would you like to see in the Polar standard library? So far my biggest wishes are.

More builtin list methods

Indexing
Ranges
Appending

More builtin Math

sin / cos / tan
min / max
exponential

repo: osohq/oso


text:
System preferences icon for each sub preference [WIP]

End result

JFYI there is also the hard-way which is extracting each pane icon and putting them in the app, it is ofc work, but will deliver the best results:
https://www.macobserver.com/tmo/article/os-x-using-icons-from-system-preferences
:)

Ah that is even better to be fair!

just FYI I'm refactoring a lot of code to make it easier for you (and others) to contribute to the project... most of the code I threw in really quickly and as I went adding features it became messy. :)

Cool, think it's ready now

repo: ospfranco/sol


text:
Add a generic model of robots for testing purposes

As the title says. I think it should be added here. https://github.com/osrf/rmf_demos/blob/814a1204389c0d5c969db067a58216a7bba8a0fa/rmf_demo_maps/maps/office/office.building.yaml#L174

I tried to do this just in case (without any context of course =) ), but without success.
[1790.467, 1326.218, 0, magni2_charger, {
 is_charger: [4, true],
 is_parking_spot: [4, true],
 spawn_robot_name: [1, magni2],
 spawn_robot_model: [1, generic],
 spawn_robot_type: [1, Magni]}]

Already discussed with @Yadunund. Closing this!


text:
default value initialization to avoid redundant variable assignment

Sub-Tasks

[x] Modify wavefield.cc to initialize the wavefield parameter with default values and update them only when a user passes a value for it in the SDF file.
[x] Modify all world sdf files in order only to have PMS model wave parameters, namely -- direction, steepness, gain, and period(will do it after the first review)

Summary
The modification avoids repeated initialization of some wavefield variables, which always have the same value across all world files. Therefore now, the publisher allows the user to indicate solely the variable they wish to modify, leaving the remaining variables to be managed automatically.
Test it
To test this, one can use the sydney_regatta.sdf. The world file currently only publishes direction, steepness, gain, and period.

For now, please review this using the testing instructions above to make sure the sydney_regatta.sdf is working properly.

@tejalbarnwal This seems to be working fine for syndey_regatta.sdf

@tejalbarnwal This seems to be working fine for syndey_regatta.sdf

great, I will go ahead and push the updated world files

update: I have updated all the world files

@tejalbarnwal Is this PR also intended to address issue #684 (which we discussed in our meeting but I only just wrote up today)? If so, we should wait to merge it until we figure out what the new parameters should be. Otherwise we can merge this now and create a new PR for #684. What would you prefer?

@tejalbarnwal Is this PR also intended to address issue #684 (which we discussed in our meeting but I only just wrote up today)? If so, we should wait to merge it until we figure out what the new parameters should be. Otherwise, we can merge this now and create a new PR for #684. What would you prefer?

We could merge this and create a new PR to address #684; this way, we could have a clean commit log for the repo. I was also thinking if I should squash all the commits made to this branch to just one commit?

@tejalbarnwal Ok, in that case I'll make a new PR soon with the new wavefield values. Could you review it today?
I don't usually squash commits but I don't think it's a problem if you prefer that.

@tejalbarnwal Ok, in that case I'll make a new PR soon with the new wavefield values. Could you review it today?
I don't usually squash commits but I don't think it's a problem if you prefer that.
Edit: Also, since this is approved and we aren't modifying it further and we're wrapping up for the release today, it would help if you could merge it soon. Thanks!

Yes, I will test out the parameters specified wave model envelope in #684. I will let you know in a few minutes if any warnings or error is raised on my system
Oh, I wasn't aware that I could merge it. I am used to reviewers merging it in other repos. I will merge it now.

@tejalbarnwal Thanks and no problem. We let the creator of the PR perform the merge (if you have the permissions, which you do).
You are welcome to play around with the wave parameters, but I will also soon have a PR for you to review with specific values for the various practice worlds.

@tejalbarnwal Thanks and no problem. We let the creator of the PR perform the merge (if you have the permissions, which you do).
You are welcome to play around with the wave parameters, but I will also soon have a PR for you to review with specific values for the various practice worlds.

yup, I will review it!

repo: osrf/vrx


text:
Analyzer com.sun:tools:jar Dependency SystemPath Error

I've decided to run the whole ORT process on my old repositry from the start using latest ORT master branch and I've  encountered this error again in multiple places regarding other packages during the analyze step
14:59:51.390 [Analyzer-1] WARN  org.ossreviewtoolkit.analyzer.managers.utils.MavenLogger - Failed to build parent project for org.glassfish.jaxb:txw2:jar:2.2.11
org.apache.maven.project.ProjectBuildingException: Some problems were encountered while processing the POMs:
[WARNING] Reporting configuration should be done in <reporting> section, not in maven-site-plugin <configuration> as reportPlugins parameter. @
[ERROR] 'dependencyManagement.dependencies.dependency.systemPath' for com.sun:tools:jar must specify an absolute path but is ${tools.jar} @ com.sun.xml.bind.mvn:jaxb-parent:2.2.11, /root/.gradle/caches/modules-2/files-2.1/com.sun.xml.bind.mvn/jaxb-parent/2.2.11/9496f6747e6d5ab5da869941dea4143969122b91/jaxb-parent-2.2.11.pom, line 188, column 29
...

At the button of my analyzer-result.yml file I have has_issues: true, with older ORT code state (from 04.03.2020) it was always has_issues: false without the error above. I don't see any JAVA related errors in the .yml file that have caused has_issues: true so, maybe there are some other issues that set this values or maybe I'm not searching correctly.
Nevertheless, the only thing that has changed is ORT's code. I'm still using the same JDK12.0.2 and project repository snapshot as before. I've even deleted the .gradle/cache/ content to be sure.

The "[...] for com.sun:tools:jar must specify an absolute path but is [...]" is a generic error cause by using Java >= 11 with older JAXB versions. There's nothing ORT could do about it, but you (or the dependency using JAXB) should upgrade to at least JAXB 2.3.0.

So basically, I'll need to update every package where the problem occurs and see if the errors pop up again in the analyze step. If those errors are gone then in this case the has_issues: true conditions is harmless and I can ignore it with current package versions.

That's one way to do it, yes, but actually the same error should occur with a regular Maven build of the project when using Java 12.

I've checked out an old ORT commit e8369c3dcd57d41bd3cfa4f3961bc4757a65b420 (04.03.2020) and left everything else as is. The error was still present and I think it was always there but I've forgot about it. Nevertheless the issue flag reports:  has_issues: false, did anything change in ORT that could have caused this behavior?

Yes, as mentioned before there was a bug in ORT back then which caused has_issues to be set to false although there were issues, see https://github.com/oss-review-toolkit/ort/pull/2405.

authors: CleanHit sschuberth

repo: oss-review-toolkit/ort


text:
Android specific Google Maven Repository dependencies warning in ORT: Could not find artifact

Hello ORT Team,
while testing ORT on different Android Projects, I have noticed that I get warnings and issues with androidx dependencies.
It seems that specific repositories (example Google Maven) in the build.gradle buildscript are not recognized correctly by ORT.
This happens specially with dependencies which are on the androidx google maven repository.
For example, an Android project with a build.gradle ex:
buildScript {
repositories {
google()
maven { url "https://plugins.gradle.org/m2/" }
}
}
and with an app/build.gradle
dependencies {
implementation "androidx.appcompat:appcompat:1.3.0"
}
When running the analyzer, the following issues / warns are reported:
This is a part of the log using the --info option of analyzer:
16:33:01.835 [DefaultDispatcher-worker-1] WARN  org.ossreviewtoolkit.analyzer.managers.utils.MavenSupport - Could not find artifact androidx.appcompat:appcompat::1.3.1 in any of [https://dl.google.com/dl/android/maven2/androidx/appcompat/appcompat/1.3.1/appcompat-1.3.1, https://plugins.gradle.org/m2/androidx/appcompat/appcompat/1.3.1/appcompat-1.3.1].
16:33:02.448 [DefaultDispatcher-worker-1] WARN  org.ossreviewtoolkit.analyzer.managers.utils.MavenSupport - Could not find artifact androidx.activity:activity::1.2.4 in any of [https://dl.google.com/dl/android/maven2/androidx/activity/activity/1.2.4/activity-1.2.4, https://plugins.gradle.org/m2/androidx/activity/activity/1.2.4/activity-1.2.4].
16:33:03.059 [DefaultDispatcher-worker-1] WARN  org.ossreviewtoolkit.analyzer.managers.utils.MavenSupport - Could not find artifact androidx.fragment:fragment::1.3.6 in any of [https://dl.google.com/dl/android/maven2/androidx/fragment/fragment/1.3.6/fragment-1.3.6, https://plugins.gradle.org/m2/androidx/fragment/fragment/1.3.6/fragment-1.3.6].
Is there any specific configuration or parameter I might be missing, so that the Google Maven Repository and androix dependencies get correctly identified?
Thanks and best regards
Felix

Yes, we're aware of several Android-specific issues when analyzing Gradle projects, and I'm working on a solution as part of https://github.com/oss-review-toolkit/ort/issues/4694, which will very like also fix this issue.

Thanks for the quick response Sebastian.
Best regards

authors: felix-leyva sschuberth


text:
ExperimentalScanner: Plugin support

Add support for plugins for the experimental scanner, and for using multiple scanners at once. See the commit messages for details.

PS: If you use GitHub's new "Update with rebase" on this, existing approvals do not get lost.

PS: If you use GitHub's new "Update with rebase" on this, existing approvals do not get lost.

Nice!

authors: mnonnenmacher sschuberth


text:
model: Fix the Set contract for data classes that implement Comparable

Kotlin's data classes have auto-generated equals() functions that take all primary properties into account. When implementing the Comparable interface for those classes, i.e. to give them a defined order in a SortedSet, special attention needs to be put on compareTo() to return 0 if and only if equals() returns true. This is required because "a sorted set performs all element comparisons using its compareTo method, so two elements that are deemed equal by this method are, from the standpoint of the sorted set, equal" 1. This means that sorted sets are considered equal if for all their respective elements compareTo() returns 0. So if compareTo() is implemented to not consider all properties for an equality check, this can lead to unexpected results, for example when using Kotest's data class matchers.
Fix this and obey the general contract of the Set interface by taking all properties into account if the properties to sort on are equal. As the ordering does not matter if not all properties are equal but the properties to sort on are, arbitrarily return 1 from compareTo() in that case.
Signed-off-by: Sebastian Schuberth sschuberth@gmail.com

I'm closing this in favor of https://github.com/oss-review-toolkit/ort/pull/6244.


text:
VulnerableCode: Uses aliases instead of internal vulnerability IDs

While older versions of VulnerableCode used to return official identifiers (like CVEs) for vulnerabilities, this is no longer the case. Instead, internal IDs are used. There is, however, information about aliases which contains such official identifiers. This PR evaluates this information and tries to generate more meaningful IDs for vulnerabilities to be displayed in reports.
Note: The presence of alias information in query results is a rather new feature which is not yet part of the latest release of VulnerableCode. There is a related issue [1]. So, there may still be changes in the final release version. The results used by the tests are obtained from the public VulnerableCode instance which already includes this information. The changes in the PR are compatible with older versions, since alias information is optional.
[1] https://github.com/nexB/vulnerablecode/issues/1090

The listing of aliases in VulnerableCode query results is not yet released. From my discussions with nexB, I expect that this will become available soon. Do you think this PR can already been merged (since it should not break existing users), or should we wait for the next VC release?

In the latest RC of VulnerableCode [1] the aliases are contained in the responses of the bulk queries we use. I have asked for confirmation whether the format is expected to be stable; but I think, it should be pretty safe to merge this PR. I have verified again locally that the implementation works against an older deployment of VulnerableCode.
[1] https://github.com/nexB/vulnerablecode/releases/tag/v32.0.0rc3

So, are there any objections against merging this?

Not from my side.

Hm, I might have to rebase again to pick up the changes related to the failing OSV test. Or could anybody with the appropriate power merge this directly?

Hm, I might have to rebase again to pick up the changes related to the failing OSV test. Or could anybody with the appropriate power merge this directly?

Merging despite the unrelated OsvServiceFunTest failure.

authors: oheger-bosch sschuberth


text:
feat(cyclonedx): Set basic supplier information

While at it, also set author information for package components.
See 1 for context.

Merging prematurely for the SBOM Harmonization Plugfest 2024.


text:
Add CRAN+Bioconductor ecosystems

Closes #175.
I didn't quite understand "Source URL" vs. "OSV Formatted URL" so I took a guess.
cc @tylfin as upstream maintainer. I marked this as "unofficial" despite R Consortium backing, please CMIIW.

I think it's okay to mark this as unofficial pending widespread adoption. I'll be working on organizing a working group (or similar) to document a process for triaging vulnerabilities and documenting a formal approval process.

authors: MichaelChirico tylfin

repo: ossf/osv-schema


text:
OSS-Fuzz client download attempt even though the disabled in policy

Is your feature request related to a problem? Please describe.

Hi folks, I am trying to get the scorecard to run on a self-hosted runner and I keep running into the error for the OSS Fuzz client:
Error: GetClients: getting OSS-Fuzz repo client: error during InitRepo: repo unreachable: GET https://api.github.com/repos/google/oss-fuzz: 401 Bad credentials []
This has 2 causes:

Our private runners don't have internet access
I want to run this against a private repo on GitHub Enterprise Server, so the token I give it does not have access to api.github.com

In my opinion, these are valid reasons to only download the OSS-Fuzz client if it is enabled in the policy (I have it disabled), or else I should be able to give it an alternative token to use for downloading the client.
Describe the solution you'd like
Do not attempt to download the OSS Fuzz client if the policy is disabled. That setting should be used in the GetClients call here.
Describe alternatives you've considered
Having a different token for downloading the clients, or perhaps even download the clients from a different url (I can host it on our GHES server itself for example.
Additional context
Needed for any user running on GHES with locked down private runners.
Linked to this issue that tried to solve this (and succeeded for their intent):
Remove download of unnecessary tarball
#1697

As of #2719 this behavior should no longer occur, although it hasn't made its way to a release yet.
The Fuzzing check now reads projects from the OSS-Fuzz build log at https://oss-fuzz-build-logs.storage.googleapis.com/status.json (For more details see https://github.com/google/oss-fuzz/issues/9782). This download only occurs when the Fuzzing check is run.

perhaps even download the clients from a different url

there's no support for this currently, but it would be an easy fix. However sounds like you just want to prevent the download which should be fixed already. Feel free to re-open.

@spencerschrock should we cut a release? Its been a while anyways.

Thanks! Eagerly awaiting that release 👍.

authors: azeemshaikh38 rajbos spencerschrock

repo: ossf/scorecard


text:
Add Ashley Madison Story Time

Add Ashley Madison Story Time about using MD5 for passwords.
The Ashley Madison data breach
story is more complicated, e.g., the mere fact that someone was in the database was a problem, and people could create accounts for others' email address without verification: https://en.wikipedia.org/wiki/Ashley_Madison_data_breach
However, if we focus on just the use of MD5, that part of the story is much easier to explain, and it's a helpful example that got a lot of media attention. It also helps clarify that "storing in both secure AND insecure formats" is a bad idea.
Signed-off-by: David A. Wheeler dwheeler@dwheeler.com

LGTM! I agree this is an interesting example to add. I don't think the nature of the service should play a role in our selection process. If anything listing them here may not really be to their advantage even though the old saying is that there is no such thing as bad publicity...

Thanks! As I mentioned on the mailing list, I'm no fan of their business model, but I think it's a really good example of the specific issue we want to illustrate.. so I think we should include it.

I posted the idea on Nov 18 (4 days ago), and this specific proposed change 2 days ago. No one has complained about the concept, so I will merge it. Thank you everyone!

authors: david-a-wheeler lehors

repo: ossf/secure-sw-dev-fundamentals


text:
AV1/VP9 via SRT, out as MPEG-TS? Or AV1 via RTMP?

What is the business background? Please provide a description.
No business, private use.
Is your feature request related to a problem? Please describe.
This is twofold.
Nowadays AV1 and VP9 get more and more widely adopted. SRS still seems to focus on HEVC, though, at least in version 6.
As far as I know the only way to ingest a stream with AV1 video with SRS is to use SRT or WebRTC.
Describe the solution you'd like
The Enhanced RTMP spec allows for other codecs apart from HEVC, too, so it would be great if SRS could support this.
Otherwise: is it possible to ingest a stream containing AV1 using SRT and converting it to MPEG-TS/HTTP-FLV, so it can be used with mpegts.js, which now supports those newer codecs?
Thank you so much for your work!

Is it actually needed that SRT still needs to be converted to RTMP before converting it to MPEG-TS/HTTP-FLV? Why not directly remuxing it from SRT to MPEG-TS. Then codecs are not a problem?
I don't see any benefits of using SRT, because it still gets converted to RTMP so it makes no sense to use it.

Is that the case right now and that's possible already with SRS? Or are you saying it should be the way.

No not possible yet. Would still be nice to have, as MPEG2-TS supports more codecs / audio channels then RTMP.

As far as i know, AV1 is not yet supported with SRT (only WebRTC and eRTMP), because SRT uses MPEG-TS and AV1 is not yet implemented for transport stream usage.

repo: ossrs/srs


text:
improve timestamp type: int -> uint.

when I check the timestamp type from RTC -> RTMP.
RTC(uint32_t) -> RTMP (uint64_t).
Some timestamp type can be improved.
e.g. int64_t convert to int32_t.

close this PR, because introduced timestamp bug.


text:
Create new top-level section "Foundations" for housing non-CS prerequisites?

As I described in #438, I am thinking about pulling most non-CS topics out of Intro/Core/Advanced CS and putting them in a separate section called "Foundations". Courses from this section would only be required when someone doesn't have the relevant background, and the curriculum would still begin with Intro CS. Students would only take what they need from it, when they need it. The prerequisites section of each course would then contain a hyperlink to the relevant section in Foundations.
Rough outline of how that would play out, assuming we were to adopt other changes I've suggested recently:
Foundations
Complete only those sections as directed by the prerequisites of a CS course you're about to take.
K-8 math
Select the appropriate grade level from Khan Academy Math.
High school math

Depends on: K-8 Math
Khan Academy: High school math (all sections)
For extra practice / refreshers: ASU College Algebra and ASU Precalculus
Getting help: /r/learnmath

Single-variable calculus

Depends on: High school math
Full courses: MIT Calculus 1A 1B 1C
For extra practice / refreshers: Expii Calculus 1
Getting help: /r/learnmath

Multivariable calculus

Depends on: Single-variable calculus
Full course: Multivariable Calculus
For extra practice / refreshers: Khan Academy Multivariable Calculus
Getting help: /r/learnmath

Probability

Depends on: Multivariable calculus
Full course: Introduction to Probability
For extra practice / refreshers: Khan Academy Statistics and probability

Electricity

Depends on: Single-variable calculus
Full course sequence:

Introduction to Mechanics, Part 1
Electricity and Magnetism, Part 1
Electricity and Magnetism, Part 2

For extra practice / review: Expii Mechanics and Expii Electricity, Magenetism, and Optics

Modern technology

Stanford CS101 (see the discussion in #530)

Intro CS

Depends on: Foundations#Modern technology
Python prep for those who have never programmed in their life: Computing in Python 1-3
MIT Intro to CS and Programming

Core CS
Core math
Linear algebra

Depends on: Foundations#High school math
Textbook supplement
Course sequence:

Essence of Linear Algebra
Linear Algebra Foundations to Frontiers

Discrete mathematics

Introductory reading: Notes on discrete math
Prep for those who need more exercises: Discrete Math Specialization
Full course: MIT Math for CS courseware and textbook

(etc.)
Advanced CS
Advanced machine systems

Depends on: Foundations#Electricity
Full course sequence: MIT Computation Structures

(etc.)

I like this idea in general. A future goal would be to have an assessment that would let students know if they can skip a course. (CLEP has an Algebra, Precal and Calculus class) If we are listing electricity, we should list high school physics.
On math, I suspect linear algebra should sit alongside calculus and probability in Foundations math, with only discrete math in the required math courses.
Python programming should receive more (not less) emphasis than Stanford's CS101.
We'll help students evaluate if they should take a Foundations course by not only listing its pre-reqs, but also what courses it prepares students for.

Whether to put Calculus and up in Foundations or Core depends on whether the goal is for Core to be everything in a cs degree besides for gened or if it's to only include cs specific courses. If it's the first, most cs degrees assume you already know precal and start with Calculus so that would be where ossu starts from. If it's the second then Calculus would be in Foundations.
Sidenote: I don't know how practical this is, but you could consider making something like University of Buffalo's [flowcharts] (https://engineering.buffalo.edu/home/academics/undergrad/advisement/flowsheets.html?plan=CSEBS-2018). They let you mouse over courses to see prerequisites and courses that use it as a prereq.

Hello!
I just want to say that I like this idea a lot. I was looking for guidance on the best resource to learn/refresh the necessary high school math without knowing what is relevant and what is not. Much beacause I was not sure what the American curriculum demands. I am following this site, https://www.mathsisfun.com/links/curriculum-high-school-algebra.html, but then I am like, is it enough? Is it too much? So, having some guidance is really useful.
Also, Khan Academy is great, but bandwidth is really expensive where I live, so whenever I can, I avoid videos.
Thank you!

I love the idea of the Foundations page.
Back in the 70's, I studied Engineering at Georgia Tech. Dropped out. In the 80's, I studied Computer Science at Rutgers. Dropped out again. At both, calculus was a required course. I think the Core Math section should section should include Single Variable Calculus and Multivariable Calculus to reflect the both structure of a degree program, the importance of these courses to our studies and the effort needed to understand and pass them. Personally, I hated Calculus but loved Linear Algebra.
Perhaps we could link each course to a page or pop-up that lists things like alternatives, testbooks, sites for practice and places to go for help to keep the main page and the foundations page cleaner?

Thanks for everyone's feedback!
@waciumawanjohi

On math, I suspect linear algebra should sit alongside calculus and probability in Foundations math, with only discrete math in the required math courses.

It should, yes, but currently we use LAFF which uses a lot of programming exercises written in a style similar to how one writes linear algebra programs in C, etc. Linear algebra could go in Foundations if we used a more abstract resource with no to little programming (Strang's materials probably are like this).

Python programming should receive more (not less) emphasis than Stanford's CS101.

I'm not sure what you're saying here. Are you suggesting we don't use Stanford CS101? Or add more Python courses? What's your rationale? (Not challenging, just asking for clarity)

We'll help students evaluate if they should take a Foundations course by not only listing its pre-reqs, but also what courses it prepares students for.

Reverse dependencies are a good idea!
@Alaharon123

Whether to put Calculus and up in Foundations or Core depends on whether the goal is for Core to be everything in a cs degree besides for gened or if it's to only include cs specific courses. If it's the first, most cs degrees assume you already know precal and start with Calculus so that would be where ossu starts from. If it's the second then Calculus would be in Foundations.

@CamLV

I think the Core Math section should section should include Single Variable Calculus and Multivariable Calculus to reflect the both structure of a degree program, the importance of these courses to our studies and the effort needed to understand and pass them.

Thanks for your input. I understand your perspectives, but I want to point out that even though I'm proposing to move Calculus, et al., off the Core CS page, it's still required for everyone who hasn't already taken it and would be clearly marked as such in the relevant sections.
Regarding geneds: I'm trying to solve a short-term problem specific to OSSU CS, but general education requirements (as commonly, and rightly, required by CS standards committees) isn't just an OSSU CS problem; it's an OSSU problem. Geneds are, by definition, required of everyone at a university, so it should be solved at the OSSU level. Maybe someone else can step up and create a general education curriculum that we could then refer to.
@chinguelessi

Also, Khan Academy is great, but bandwidth is really expensive where I live, so whenever I can, I avoid videos.

Not having an answer for this situation is an oversight of OSSU and I apologize for that! But as you can probably tell from above, I want to start including text-based resources alongside videos for people who prefer (or need) this format.

On math, I suspect linear algebra should sit alongside calculus and probability in Foundations math, with only discrete math in the required math courses.

It should, yes, but currently we use LAFF which uses a lot of programming exercises written in a style similar to how one writes linear algebra programs in C

If Linear Algebra is in the core curriculum only because the only acceptable course uses programming, we should ensure that students know which courses rely on LinAlg and that their selected path through the curriculum may not require the course.

I'm not sure what you're saying here.

The wording sounded to me like CS101 was being emphasized more than CS1301. Not a big deal.

@chinguelessi https://courses.edx.org/courses/course-v1:ASUx+MAT117x+1T2016/course/ is really good especially for returning students. It tests your knowledge and then only teaches you what you need to know. It also checks every so often to make sure you've mastered topics so you don't get left behind with something you don't really understand. It starts off with a knowledge check. If you start after that with less than 15% in your pie, then you're not ready for college algebra yet. If you start with more than 85%, you can skip to Pre-Calculus. It's all text and pictures so it should take less bandwidth than videos. Of course, this is only relevant if you're ready for College Algebra. If you get less than 15%, then you still have the same problem. I've heard the OpenStax textbooks, which start all the way from pre-algebra, are good, but I haven't used them so ymmv. You could probably take the quizzes on Khan Academy to figure out where you're holding and download a pdf (free) or buy a physical copy (paid) of the appropriate textbook.
@hanjiexi gened for ossu is definitely doable. Using the SUNY requirements as an example since I'm familiar with them, Mathematics is already covered, there are plenty of online courses for learning a language, social sciences can be some course on Computers in Society (that's what it is in my college), Arts can be an introductory html/css course (I'm taking that right now), I think there's such a thing as an online physics lab course. The hardest part, other than actually choosing the courses (requires knowledge of courses from a variety of fields so that's actually pretty hard), would probably be finding a college level online english course. Not sure how that would even work.

@waciumawanjohi

If Linear Algebra is in the core curriculum only because the only acceptable course uses programming, we should ensure that students know which courses rely on LinAlg and that their selected path through the curriculum may not require the course.

That's a good point. It will be very inconvenient to have a section in Core Math that isn't actually needed except in some very specific domains. From a brief glance and trying to keep all the proposed changes in my head, linear algebra is only needed for one course in Advanced CS (LAFF Programming for Correctness) and several courses that I proposed to move to Domains (e.g. graphics, machine learning).
I have some invariants I hope to maintain for the top-level sections, but without mapping all the proposed changes I don't know yet how this will play out:

No top-level section should contain forward-dependencies.

Example 1: a course in Foundations can depend on another Foundations course, but it shouldn't depend on material from Core CS or above.
Example 2: a course in Core CS can depend on material from Core CS and Foundations, but not on material in Advanced CS.

No top-level section except Domains (our endpoint) should contain unused dependencies. If there is no way to trace the reverse dependencies of material in Foundations, Core, and Advanced all the way to some Domain / application area, then the material belongs in Extras.

Example: Logic programming (fulfilled via Prolog) currently isn't used anywhere beyond Core Programming. It should be moved to Extras, unless we add a Domain that uses it or depends on material that uses it.

Every section in Advanced CS should have a corresponding section in Core CS. If there is no way to justify adding a corresponding section in Core CS, then such section belongs in Domains.

This is relevant to our discussion in #528.

There needs to be at least one section of Advanced CS that only depends on Foundations or Core CS, not on other sections of Advanced CS.

For this to be true of all sections is a "nice to have" but it might be too hard to maintain.

My basic idea is to model a four-year undergraduate CS degree, where this progression is possible:

Year 1 begins with Intro CS, and the remainder of the time is spent fulfilling prerequisites in Foundations (possibly in geneds if we have them), with any remaining time on early Core CS coursework.
Year 2 is for finish the rest of Core CS, having already done everything needed in Foundations.
Year 3 is spent on one selected section of Advanced CS, which would have to be one that doesn't depend on any other section in Advanced CS (see invariant 4).
Year 4 is spent either on one Domain or a second section of Advanced CS.

So with the above design (for which I, like everything else, hope to get feedback), that means Linear Algebra would likely end up in Foundations or Advanced CS. If it goes in Foundations, the resource we use can't depend on advanced programming knowledge. If it goes in Advanced Math, we would need to show that Advanced Math is a viable subject in and of itself for year 3 or 4, and that such material is depended on by at least one Domain.
(I might be overreaching on the amount of work that is viable to do before v9 lands. I want v9 to arrive by the end of the year.)

I have used our wiki to create a draft for what the new Foundations page could look like: https://github.com/ossu/computer-science/wiki/Foundations-draft
Notes:

No narrative written yet for any individual sections (and usually should be self-explanatory)
The Prerequisites and "Needed by" sections don't yet have hyperlinks, but they will when everything is finalized
The "Use cases" headings are important in the table where I list Resources:

When it says learn, that means we are recommending this resource as the one to use when learning it for the first time.
If it says review, this means that you should use it if you have already learned the material but need to review it, perhaps because you learned it a long time ago.
If it says practice, this means it's a resource meant for when you have already learned it but need more exercises.
(Currently, I always list the same resource as both review and practice, but we should pick better individual resources for these two activities.)
If it says prepare, this means the material isn't dense enough to really learn from (it may just be video(s)), but it helps to prepare the student for whatever comes next. This is the case with Essence of Linear Algebra.
If it says help, that means this is the resource we're recommending for those who need help with the material.

I originally didn't have a section labeled Physics, but did have one named Electricity. @waciumawanjohi recommended I make a separate physics section, but I looked closer at the courses under Electricity and found that those are the very first courses a student would take in physics and don't appear to assume any prior physics knowledge. So I basically just renamed "Electricity" to "Physics". I wish we could find a course like "Physics for CS" as I'm sure much of the material is not relevant.
Of note about Linear Algebra: currently I'm listing two tracks, one is LAFF and the other is Gilbert Strang's course. Although LAFF has programming assignments, they are optional, so I think it's fine to include it in Foundations (and they don't assume any prior programming knowledge anyway). Strang's course is listed as a more challenging and rigorous alternative — I didn't want to leave it out since it is probably the very best course on the subject — but it requires MATLAB and Multivariable Calculus, so not an option for most students.

I will proceed to format the rest of the curriculum pages in this way unless there is anything seriously wrong with it.

Looks good. Should probably also have a flowchart to give an overarching view of everything.

Looks good, but it's unclear why Applied Linear Algebra is here rather than in extras/readings. Why would one want to reference this incomplete text?

I want to ensure that there are text-based resources listed alongside video-based resources for people whose bandwidth is expensive. I don't know how this will play out yet, but I'm thinking that the separate extras/readings page might be removed in favor of keeping all high-quality materials grouped under the relevant topic, with explicit recommendations given about which resources are worth using under which conditions.
Applied Linear Algebra, incomplete though it may be, is geared towards beginners with only a high school math background. The other free text, by Jim Hefferson, assumes college level calculus just like the Strang course. So both groups of resources are grouped according to their prerequisites. I do probably need to add a note about the incompleteness of Applied Linear Algebra compared to the course it sits alongside.
At some point I also need to figure out just how much of linear algebra exactly is required for which later courses. It may be that some courses need a lot more and some need much less, so the tracks could be re-organized later according to their reverse dependencies. But that's not going to happen anytime soon as I don't have the time.

Ok I made a mistake. I saw Immersive Linear Algebra in the text and then Applied Linear Algebra in the table and didn't realize they weren't the same thing. What I meant to ask was why Immersive Linear Algebra is put there if that's something for OSSU to keep an eye on rather than for students to keep an eye on. However, I am now realizing that this is because you are using the wiki to have notes for yourself and are probably planning on transferring this all to the readme when 9.0 is ready.
I see that you added a note afterwards stating that Applied Linear Algebra does not cover everything it should. If that's because of my comment, I was mixing it up with Immersive Linear Algebra (if that's a thing anyway then I guess I'm glad that my mistaken comment led to something positive). Sorry for the confusion. The original text was clear, just Applied and Immersive look similar enough that I had a brain fart.

authors: Alaharon123 CamLV chinguelessi hanjiexi waciumawanjohi

repo: ossu/computer-science


text:
Subdomain extraction & Ordinary coordinate transforms

This PR is linked to https://github.com/osyris-project/osyris/issues/52 and is in anticipation of an upcoming PR that will handle cartopy projections of surfaces (https://github.com/osyris-project/osyris/issues/53).
A spatial.py file containing common coordinate transforms alongside matrix rotations has been added. I am still open for discussion on how these transformations should be handled/called, but for the moment the current routines are generic enough for us to easily modify in the future.
Git should handle the diff if #81 and #85 are merged.

Trying to get this going again:
After doing some more thinking about this, about how it would be used, I think what you would want is to be able to

rotate your entire domain into a new basis (could be to align to angular momentum)
some mechanism to obtain spherical or cylindrical components of vectors (position or other)

That's probably what it boils down to (I think?).
I suggest the following:

Make a function that would rotate your domain, supplying a normal vector (we could use the top and side orientations like for the maps, or make the function that computes the angular momentum vector more accessible, or both)
Make .r, .theta, .phi properties on the vectors that would each compute and return a new Array (you can always store those in your Datagroup if you don't want to compute them over and over again)
If the .theta, .phi is requested on a non-position vector, get the corresponding position vector from the .parent reference to compute the components (for hydro velocities, we would have to go a look into the .parent.parent['amr'], but we are already doing this for plotting maps)
If needed, make wrapper functions that can convert a whole vector to spherical or cylindrical (which calls the .r, .theta, .phi internally)

We could try this as a first step, see how usable it is, and make wrapper functions that convert an entire DataGroup in a later step, if we feel that is also needed?

PR is indeed trying to do two things, but this keeps things simpler for us to review the code together.
Checklist:

[X] Subdomain extraction using spatial criterion (cubic & spherical)
[ ] Numpy indexing for subdomain extraction
[X] Spherical coordinate transforms
[X] Wrappers in Vector for spherical coordinate transforms
[ ] Cylindrical coordinate transforms
[X] Change of origins
[X] Change of basis using rotations
[ ] Add matmul operator overload for Vector to make code cleaner and more user friendly (#85)
[X]  Spatial toolkit to be accessed by users with osyris.spatial

Current workflow looks like this:
# data load
data = osyris.Dataset(8,path=path).load()

ind = np.argmax(data["hydro"]["density"])
center = data["amr"]["position"][ind.values] # (or data["sink"]["position"])

#proceed to subdomain extraction
subdomain = osyris.Subdomain(data, selection="spherical", dr=1e3*osyris.units("au"), origin=center, basis="top", dr_L=5e2*osyris.units("au"))

#do a histogram
fig,ax = plt.subplots()
osyris.histogram2d(subdomain["amr"]["position"].r.to("au"), subdomain["hydro"]["velocity"].r.to("km/s"),
                   norm="log", logx=True, logy=False, cmap="viridis", ax = ax)
fig.show()

Basis can be changed with spatial.change_basis, and the origin with spatial.change_origin.
Accessing spherical components is fast and easy in subdomains, with the .r, .theta and .phi components using a rotation matrix, allowing for any arbitrary cartesian vector to be converted into its spherical counterpart.

This is however messing up in slices. I never had this problem in my own simulations, presumably because they're much higher resolution, but I'm getting some weird effects as can be seen here:
osyris.map({"data": subdomain["hydro"]["density"], "norm":"log"},
                     {"data": subdomain["sink"]["position"], "mode": "scatter", "c": "white"},
             dx=2e3 * osyris.units("au"),
             direction="z",plot=True, resolution=256)

This doesn't seem to occur when the chosen basis for the subdomain is `[0,1,0]` or `[1,0,0]`:

I'm still trying to track the issue down, but I'm certain from the spherical components that things are working as intended. Maybe there's something I've misunderstood with slices.

This is however messing up in slices. I never had this problem in my own simulations, presumably because they're much higher resolution, but I'm getting some weird effects as can be seen here:

The effect on maps is indeed weird. You can actually see the little squares from the cells that are rotated (around the edges).
I suspect it comes from the fact that when making a map, we try to find for each image pixel the cell that contains it. And there must be somewhere in there an assumption on the orientation of the grid.

Have a look at the changes in #102 , maybe if we get that merged it might make some things easier/more compact here?

Have a look at the changes in #102 , maybe if we get that merged it might make some things easier/more compact here?

It should reduce the number of .values calls.

Thanks for all your comments. I'll work on this again and push the changes sometime this week.

PR now simply extracts subdomains. Ordinary coordinate transforms will be added after this PR is merged.

authors: Adnan-Ali-Ahmad nvaytet

repo: osyris-project/osyris


text:
None of the tutorial steps HTML have a link to deploy action anymore

When I select any tutorial now the Prepare tab does not have a button to deploy the tutorial BAR anymore!

@ekharlamov Hi Eugene - we removed these after feedback from Alasdair. All these tutorials use the Flow exerciser which deploys the artifacts anyway, so it seemed redundant to make the user manually deploy them first. The deploy action is now optional and can be included for other tutorials if needed.

authors: cathyyoung ekharlamov


text:
[WIP] Enable sort function by creation time

Related Issue: https://github.com/clear-code/redmine_full_text_search/issues/115
Related PR: https://github.com/clear-code/redmine_full_text_search/pull/118
What I did

Added sort function by creation time

User can sort each item by creation time on the search page.

ref: http://localhost:3000/search

Before
After

It's already been implemented in upstream.


text:
Kubernetes monitoring

Выполнено ДЗ №

[X] Основное ДЗ
[ ] Задание со *

В процессе сделано:

Созданы манифесты для сервис мониторинга

Как запустить проект:
kubectl -f ./kubernetes-monitoring
PR checklist:

[X] Выставлен label с темой домашнего задания

Добрый день, дз принято


text:
Make the depth limit and the input bandwidth limit hard limits

Rather than the current soft limits

Depth limit is more important than bandwidth

I think that not having the depth limit enforced has become more confusing now that we've introduced MIB and now that the trees are shorter. See https://ought-inc.slack.com/archives/CMVHEDXAQ/p1569151835045200

repo: oughtinc/mosaic


text:
Add TokenId to redeemInstructionsHashIsAllowed for Redeem Contracts

Adding tokenId to redeemInstructionsHashIsAllowed in order to set redeem permissions to be token specific rather than across the collection

looks good!

authors: IsabellaSmallcombe oveddan


text:
[BUG] - list index out of range

After completing all the install steps, I try to add the integration but get the error "An unknown error occured. See logs for details." The Tuya & localtuya integration are both installed. I did disable the vac in the localtuya for trobuleshooting. It does appear in the tuya integration as a vacuum and can be triggered from HA. The vacuum is setup in the android Tuya Smart app, with partitions labeled/named, also created overlapping areas. Sweeping Robot Open Service is enabled.
I'm not sure if this is related to #8
To Reproduce
Adding the integration with all appropriate IDs.
Version:

HA Docker: 2024.3.1
Robot: Proscenic M9
Map extractor v1.2.1

Log entry
After enabling debug logging in tuya, the two errors are below:

This error originated from a custom integration.

Logger: custom_components.tuya_cloud_map_extractor.config_flow
Source: custom_components/tuya_cloud_map_extractor/config_flow.py:57
integration: Tuya Cloud Map Extractor (documentation, issues)
First occurred: 11:18:43 AM (3 occurrences)
Last logged: 11:32:55 AM

list index out of range
Traceback (most recent call last):
  File "/config/custom_components/tuya_cloud_map_extractor/config_flow.py", line 57, in async_step_user
    headers, image = await validate(self.hass, user_input)
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/tuya_cloud_map_extractor/config_flow.py", line 214, in validate
    return await hass.async_add_executor_job(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/tuya_cloud_map_extractor/tuya_vacuum_map_extractor/main.py", line 131, in get_map
    raise e
  File "/config/custom_components/tuya_cloud_map_extractor/tuya_vacuum_map_extractor/main.py", line 127, in get_map
    map_link = link["result"][0]["map_url"]
               ~~~~~~~~~~~~~~^^^
IndexError: list index out of range

This error originated from a custom integration.

Logger: custom_components.tuya_cloud_map_extractor.tuya_vacuum_map_extractor.main
Source: custom_components/tuya_cloud_map_extractor/tuya_vacuum_map_extractor/main.py:130
integration: Tuya Cloud Map Extractor (documentation, issues)
First occurred: 11:18:43 AM (4 occurrences)
Last logged: 11:37:10 AM

Encountered an error, please include the following data in your github issue: b'eyJyZXN1bHQiOiBbXSwgInN1Y2Nlc3MiOiB0cnVlLCAidCI6IDE3MTE0NDQ3MjM1NjIsICJ0aWQiOiAiZDc3YzkzYTVlYjUxMTFlZTkxYzAzMjIxZGQ4NDFkOWMifQ=='
Encountered an error, please include the following data in your github issue: b'eyJyZXN1bHQiOiBbXSwgInN1Y2Nlc3MiOiB0cnVlLCAidCI6IDE3MTE0NDU1MjYzODQsICJ0aWQiOiAiYjVmYjhjMmFlYjUzMTFlZTljNjdkMjk2Y2NmNWMwOTEifQ=='
Encountered an error, please include the following data in your github issue: b'eyJyZXN1bHQiOiBbXSwgInN1Y2Nlc3MiOiB0cnVlLCAidCI6IDE3MTE0NDU1NzUxMjcsICJ0aWQiOiAiZDMwZTRjNGNlYjUzMTFlZTkyYWU0NjMwNTdmZTlhMmUifQ=='
Encountered an error, please include the following data in your github issue: b'eyJyZXN1bHQiOiBbXSwgInN1Y2Nlc3MiOiB0cnVlLCAidCI6IDE3MTE0NDU4MzAyMjcsICJ0aWQiOiAiNmIxOTE0MTNlYjU0MTFlZTg0Y2I1MmVkYzJhMWI0YTMifQ=='

Thank you :-)

Same here

Same here

This error results from tuya not offering any map downloads for your vacuum. Nothing can be done until local communitcations are figured out. Will add error handling for this.

Is there no way to fix this?


text:
getMessage获取到的数据有误

我用的环境
PHP 版本：7.2
overtrue/wechat 版本：4.0
是否使用了框架？框架名称：laravel
问题及现象
我这边直接按照demo来, 接收到客服请求直接push一个消息, 就这么简单
现象:    小程序端 没有收到信息
代码截图:

然后我debug了半天, 最终发现的可能原因就是$app->server->getMessage()方法获取到的并不是传参的数组, 而是一个数组key是0, value是get方式的传参字符串, 这样导致后面很多地方以getMessage获取当数组用的地方取出来的数据都是空的截图:

不知道这是什么原因, 是哪里需要配置? 请教下

应该是这边的问题?  获取到的是 & 拼接的参数,  这边的操作方法并不能获取到参数

上面是本地模拟微信post请求, 日志也打印出错误的参数了

这个是线上的日志显示, 打印出来数据为空

我估计可能的原因, 已开始本地试post/form-data  也是空的, 后来用的是post/x-www-form-urlencoded就可以打印出 上上面那张错误的信息了
首先确认一点, 微信发我服务器的请求跟参数都是正确的

所以目前我这边只能归咎于是扩展包的问题, 想请教下 这边到底怎么弄呢?

我不明白你在说什么，你自己在模拟微信推消息？然后取不到？

小程序客服功能,  微信后台设置ok的, 服务器能收到微信的请求,   按照easywechat的文档写的一个demo, 就是一个push, 但是小程序收不到内容

我不明白你在说什么，你自己在模拟微信推消息？然后取不到？

小程序客服功能, 微信后台设置ok的, 服务器能收到微信的请求, 按照easywechat的文档写的一个demo, 就是一个push, 但是小程序收不到内容

我不明白你在说什么，你自己在模拟微信推消息？然后取不到？

发现返回xml的ToUserName跟FromUserName都是空的,   然后我就本地单步调试下, 发现赋值的地方

这边取getMessage里面的参数来进行赋值, 但是getMessage返回的并不是一个参数数组, 而可能是我上面截图的字符串参数, 所以这边取都没取到

我不明白你在说什么，你自己在模拟微信推消息？然后取不到？

然后按照文档, 看下easywechat的错误日志, 如上图所示, content为空, 所以toUserName信息为空也是可以理解, 但是我打印出来了, 接口获得的参数是有的呀, 感觉有点矛盾

请问下是哪里出了问题呢

我不明白你在说什么，你自己在模拟微信推消息？然后取不到？

然后按照文档, 看下easywechat的错误日志, 如上图所示, content为空, 所以toUserName信息为空也是可以理解, 但是我打印出来了, 接口获得的参数是有的呀, 感觉有点矛盾

请教下, 这边是哪里出了问题了呢

我不明白你在说什么，你自己在模拟微信推消息？然后取不到？

然后按照文档, 看下easywechat的错误日志, 如上图所示, content为空, 所以toUserName信息为空也是可以理解, 但是我打印出来了, 接口获得的参数是有的呀, 感觉有点矛盾

请教下, 这边是哪里出了问题了呢

微信发过来的应该是post请求吧, 为啥你这边日志里面写的是get呢, 是我哪里弄错了?

你的请求哪里来的？请求内容本来就是空的啊

你的请求哪里来的？请求内容本来就是空的啊

服务器上,我打印出来的日志, 绿色字体的

你的请求哪里来的？请求内容本来就是空的啊

以下是服务器上的日志信息, 是微信发过来的请求, 我直接打印请求参数, 里面有内容

但是你的日志里的打印信息却是

看你的日志是获取的是get方式的参数, 所以获取都为空?  可是微信发过来的客服消息是post请求

首先，这个功能已经上线使用两年多了，不可能今天才出问题，要么是你用错了地方，要么是你的环境或者框架哪里有问题
ohmygd notifications@github.com于2019年7月11日 周四上午9:21写道：

你的请求哪里来的？请求内容本来就是空的啊
以下是服务器上的日志信息, 是微信发过来的请求, 我直接打印请求参数, 里面有内容
[image: image]
https://user-images.githubusercontent.com/19659483/61015177-18b25e80-a3bd-11e9-81a0-53d8c5e40718.png
但是你的日志里的打印信息却是
[image: image]
https://user-images.githubusercontent.com/19659483/61015225-2ec01f00-a3bd-11e9-96ab-894f1ed57a33.png
看你的日志是获取的是get方式的参数, 所以获取都为空? 可是微信发过来的客服消息是post请求
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/overtrue/wechat/issues/1597?email_source=notifications&email_token=AALHOYAFJYI7UMP5NRXHSLLP62DLDA5CNFSM4H7M2KV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZVGGZY#issuecomment-510288743,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALHOYFLOZNV4ZMRQ5L4COLP62DLDANCNFSM4H7M2KVQ
.

你的请求哪里来的？请求内容本来就是空的啊

以下是服务器上的日志信息, 是微信发过来的请求, 我直接打印请求参数, 里面有内容

但是你的日志里的打印信息却是

看你的日志是获取的是get方式的参数, 所以获取都为空? 可是微信发过来的客服消息是post请求

既然是POST变成GET 那你先检查一下你的HTTP server（nginx或者Apache这类）的配置

首先，这个功能已经上线使用两年多了，不可能今天才出问题，要么是你用错了地方，要么是你的环境或者框架哪里有问题 ohmygd notifications@github.com于2019年7月11日 周四上午9:21写道：
…
你的请求哪里来的？请求内容本来就是空的啊 以下是服务器上的日志信息, 是微信发过来的请求, 我直接打印请求参数, 里面有内容 [image: image] https://user-images.githubusercontent.com/19659483/61015177-18b25e80-a3bd-11e9-81a0-53d8c5e40718.png 但是你的日志里的打印信息却是 [image: image] https://user-images.githubusercontent.com/19659483/61015225-2ec01f00-a3bd-11e9-96ab-894f1ed57a33.png 看你的日志是获取的是get方式的参数, 所以获取都为空? 可是微信发过来的客服消息是post请求 — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#1597?email_source=notifications&email_token=AALHOYAFJYI7UMP5NRXHSLLP62DLDA5CNFSM4H7M2KV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZVGGZY#issuecomment-510288743>, or mute the thread https://github.com/notifications/unsubscribe-auth/AALHOYFLOZNV4ZMRQ5L4COLP62DLDANCNFSM4H7M2KVQ .

我也这么觉得, 所以我一直说 是我哪里错了吗? 可是这个不太可能是我这边问题呀, 我就干了两件事

把微信发过来的请求打印了下,  发现post请求, 小程序端发过来的客服消息的参数 都有
把你的扩展包的错误日志打开, 发现里面写的是GET方式, 获取的内容都是空
所以  给我的感觉就可能是 这个包的问题, 或者是其他什么问题?

首先，这个功能已经上线使用两年多了，不可能今天才出问题，要么是你用错了地方，要么是你的环境或者框架哪里有问题 ohmygd notifications@github.com于2019年7月11日 周四上午9:21写道：
…
你的请求哪里来的？请求内容本来就是空的啊 以下是服务器上的日志信息, 是微信发过来的请求, 我直接打印请求参数, 里面有内容 [image: image] https://user-images.githubusercontent.com/19659483/61015177-18b25e80-a3bd-11e9-81a0-53d8c5e40718.png 但是你的日志里的打印信息却是 [image: image] https://user-images.githubusercontent.com/19659483/61015225-2ec01f00-a3bd-11e9-96ab-894f1ed57a33.png 看你的日志是获取的是get方式的参数, 所以获取都为空? 可是微信发过来的客服消息是post请求 — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#1597?email_source=notifications&email_token=AALHOYAFJYI7UMP5NRXHSLLP62DLDA5CNFSM4H7M2KV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZVGGZY#issuecomment-510288743>, or mute the thread https://github.com/notifications/unsubscribe-auth/AALHOYFLOZNV4ZMRQ5L4COLP62DLDANCNFSM4H7M2KVQ .

首先，这个功能已经上线使用两年多了，不可能今天才出问题，要么是你用错了地方，要么是你的环境或者框架哪里有问题 ohmygd notifications@github.com于2019年7月11日 周四上午9:21写道：
…
你的请求哪里来的？请求内容本来就是空的啊 以下是服务器上的日志信息, 是微信发过来的请求, 我直接打印请求参数, 里面有内容 [image: image] https://user-images.githubusercontent.com/19659483/61015177-18b25e80-a3bd-11e9-81a0-53d8c5e40718.png 但是你的日志里的打印信息却是 [image: image] https://user-images.githubusercontent.com/19659483/61015225-2ec01f00-a3bd-11e9-96ab-894f1ed57a33.png 看你的日志是获取的是get方式的参数, 所以获取都为空? 可是微信发过来的客服消息是post请求 — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#1597?email_source=notifications&email_token=AALHOYAFJYI7UMP5NRXHSLLP62DLDA5CNFSM4H7M2KV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZVGGZY#issuecomment-510288743>, or mute the thread https://github.com/notifications/unsubscribe-auth/AALHOYFLOZNV4ZMRQ5L4COLP62DLDANCNFSM4H7M2KVQ .

首先，这个功能已经上线使用两年多了，不可能今天才出问题，要么是你用错了地方，要么是你的环境或者框架哪里有问题 ohmygd notifications@github.com于2019年7月11日 周四上午9:21写道：
…
你的请求哪里来的？请求内容本来就是空的啊 以下是服务器上的日志信息, 是微信发过来的请求, 我直接打印请求参数, 里面有内容 [image: image] https://user-images.githubusercontent.com/19659483/61015177-18b25e80-a3bd-11e9-81a0-53d8c5e40718.png 但是你的日志里的打印信息却是 [image: image] https://user-images.githubusercontent.com/19659483/61015225-2ec01f00-a3bd-11e9-96ab-894f1ed57a33.png 看你的日志是获取的是get方式的参数, 所以获取都为空? 可是微信发过来的客服消息是post请求 — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#1597?email_source=notifications&email_token=AALHOYAFJYI7UMP5NRXHSLLP62DLDA5CNFSM4H7M2KV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZVGGZY#issuecomment-510288743>, or mute the thread https://github.com/notifications/unsubscribe-auth/AALHOYFLOZNV4ZMRQ5L4COLP62DLDANCNFSM4H7M2KVQ .

我也这么觉得, 所以我一直说 是我哪里错了吗? 可是这个不太可能是我这边问题呀, 我就干了两件事

把微信发过来的请求打印了下,  发现post请求, 小程序端发过来的客服消息的参数 都有
把你的扩展包的错误日志打开, 发现里面写的是GET方式, 获取的内容都是空
所以  给我的感觉就可能是 这个包的问题, 或者是其他什么问题?

你的请求哪里来的？请求内容本来就是空的啊

以下是服务器上的日志信息, 是微信发过来的请求, 我直接打印请求参数, 里面有内容

但是你的日志里的打印信息却是

看你的日志是获取的是get方式的参数, 所以获取都为空? 可是微信发过来的客服消息是post请求

既然是POST变成GET 那你先检查一下你的HTTP server（nginx或者Apache这类）的配置

我这边是php, nginx, 难道是我孤陋寡闻了, nginx还可能将请求类型post转换成get?

我的controller里面打印出来了, method方法跟请求参数, 发现微信发过来的请求是没问题的
可是扩展包的日志,打印出来的, 应该就是这边

这边 获取的都是get请求?

你的请求哪里来的？请求内容本来就是空的啊

以下是服务器上的日志信息, 是微信发过来的请求, 我直接打印请求参数, 里面有内容

但是你的日志里的打印信息却是

看你的日志是获取的是get方式的参数, 所以获取都为空? 可是微信发过来的客服消息是post请求

既然是POST变成GET 那你先检查一下你的HTTP server（nginx或者Apache这类）的配置

我这边是php, nginx, 难道是我孤陋寡闻了, nginx还可能将请求类型post转换成get?

我的controller里面打印出来了, method方法跟请求参数, 发现微信发过来的请求是没问题的
可是扩展包的日志,打印出来的, 应该就是这边

这边 获取的都是get请求?

如果controller打印出来是post请求，那我猜错了。可nginx确实可能会把post转成get

小程序服务端的功能和公众号并不相同，并不能回复消息，只有验证 echostr 和 回复 success, 你在代码里回复的内容并不是合法的内容
你需要详情阅读文档：https://developers.weixin.qq.com/miniprogram/dev/framework/server-ability/message-push.html

你在小程序服务端用公众号的用法。。。。能行就奇怪了

你在小程序服务端用公众号的用法。。。。能行就奇怪了

你说的这些我都看过, 因为这文档写的挺好的, 所以我就直接看小程序客服消息, 里面写的很简单

所以我就去看公众号的客服消息, 以为customer_service这个变一下就好了, 那么请问下你的扩展包这边小程序客服消息应该如何接入呢, 如果我就返回success, 那么如何实现用户跟客服间的交流呢
文档中 感觉没有说的地方
天气炎热, 请老铁喝杯茶, 钱已到账

你在小程序服务端用公众号的用法。。。。能行就奇怪了

目前到这边发现可以回复个死信息了, 但是如何跟客服建立聊天呢? 应该是要把消息转发给客服, 或者 建个会话什么的, 这边是我发现最有可能的地方

可是却没有这个属性...

你在小程序服务端用公众号的用法。。。。能行就奇怪了

小程序客服消息转发, 文档中貌似没有写到, 公众号的倒有

https://developers.weixin.qq.com/miniprogram/dev/api-backend/open-api/customer-message/customerServiceMessage.send.html
没写就是不支持呗，我也没办法哈

https://developers.weixin.qq.com/miniprogram/dev/api-backend/open-api/customer-message/customerServiceMessage.send.html
没写就是不支持呗，我也没办法哈

**customerServiceMessage.send | 微信开放文档**微信开发者平台文档

尴尬, 好吧

authors: almas1992 ohmygd overtrue

repo: overtrue/wechat


text:
Add akashctl events command

Fixes #687

@boz , events are printing out in real time. But I am not able to get event name in subscriber event details. I am just getting attributes in event. Output of bid-create event when running akashctl events command:
{
  "ID": {
    "owner": "akash16q6s0tauc3cks5us7f57wds8c8lqg4jqs0qtaf",
    "dseq": "5039",
    "gseq": 1,
    "oseq": 1,
    "provider": "akash1zsgzee6vvx942c4c69vl859w9azn77j8uhduug"
  },
  "Price": {
    "denom": "stake",
    "amount": "20"
  }
}

Cool, thanks @akhilkumarpilli!  We can iterate on formatting them differently.

authors: akhilkumarpilli boz

repo: ovrclk/akash


text:
Deploy a new non-snapshot release

This commit https://github.com/owainlewis/yaml/commit/7341c5a6a293707a4858c9c23522130b343aad84 fixes CVE-2017-18640
It would be good to deploy a new non-snapshot release that contains this fix.
Many thanks.

Please? 🙏

Bump?

authors: RickMoynihan kiranshila vemv

repo: owainlewis/yaml


text:
[QA] OCM - images (& thumbnails) are completely broken with OCM

Describe the bug
A user who received a shared folder via federation cannot view images. The image viewer hangs
Steps to reproduce
Setup two federated machines with e.g.
env OCIS_DNSNAME=dog-DATE OCIS_VERSION=daily ./deploy_ocis_bare_metal.sh
env OCIS_DNSNAME=cat-DATE OCIS_VERSION=daily ./deploy_ocis_bare_metal.sh
ocm_trust.sh dog-20241015.jw-qa.owncloud.works cat-20241015.jw-qa.owncloud.works

admin adds the instances to each other's ocmproviders.json .
user alice at cat visits the sciencemesh app and create an invite token for user bob at dog.
user bob at dog enters the token, and shares a folder smallpics (continaing some png and jpg images) with alice ad cat
user alice clicks Shares in the left sidebar, and clicks smallpics ( see #10271 )
user alice sees no thumbnails in the folder view: BAD

user alice clicks on one of the images. It does not disaplay in the viewer: BAD

user alice tries to download an image. A login prompt appears: BAD

after entering the correct credentials there, the download works. OKish

Image viewer still does not. BAD

txt and md files work perfectly fine.

Expected behavior

thumbnails appear for federated folders too,
images can be viewed and downloaded.

Seen with both ocis-6.5.0 and daily 2024-10-09

There are 2 underlying issues:

When sharing a single image file, the file.mimetype property of the driveItem (returned by the server when querying graph/v1beta1/me/drive/sharedWithMe) is application/octet-stream. So Web is not recognizing the file as an image.

When sharing a folder that includes an image, the GET request to the image fails with a 404. It looks correct to me though: https://localhost:9200/remote.php/dav/spaces/{remoteItemId}/image.jpg

IMO this is a p2, images are completely broken in OCM. This does not only affect thumbnails, but also viewing images via the preview app. @micbar agree?

Scope

Previews and Thumbnails can be broken -> Known Issue
Downloads and Archiver need to work.

[ ] TODO: create apiTest for downloading file/folder/tar
[ ] TODO: create e2e test for Previews and Thumbnails after https://github.com/owncloud/web/pull/11693 will be done

@butonic @kwdrago @kobergj @rhafer
After talking today, we came to the conclusion that this is fixable in the scope of the 7.0.0 release. OCM has a storageprovider implemented with a fixed providerID, so the thumbnail and archiver services should be able to use the default implementation by doing an InitiateFileDownloadrequest to the gateway. We have a bug in the implementation maybe, but no "missing" piece.

After talking today, we came to the conclusion that this is fixable in the scope of the 7.0.0 release.

If you think so ...
{"level":"error","service":"thumbnails","time":"2024-10-30T15:37:01Z","line":"github.com/owncloud/ocis/v2/services/thumbnails/pkg/service/grpc/v0/service.go:311","message":"resource info is missing checksum"}

The first issue seems to be that the ocm storage provider does not return checksums in its StatResponse. Still trying to figure out why exactly that is...

Fix was merged.

Confirmed fixed in 7.0.0-rc.3 + web 11.0.4

authors: JammingBen ScharfViktor jnweiger micbar rhafer

repo: owncloud/ocis


text:
thumbnails: extract pictures from audio files

Description
This PR uses https://github.com/dhowden/tag to extract pictures from audio files in the thumbnails service.
Related Issue
Motivation and Context
It would be nice to show appropriate cover images for audio files in the Web UI.
The main problem for now (which can totally be solved in another iteration is that not every audio file has embedded cover artwork, so on a page with a bunch of audio files, we would likely trigger a lot of status 500 requests.
This is my pet peeve, no company task - while reviews are personally appreciated, there is no priority.
How Has This Been Tested?

test environment:
test case 1:
test case 2:
...

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

Followup tasks
It would be nice to know in advance whether a thumbnail is available or not - via propfind or graph api. For that we likely need to store thumbnail information in the metadata - but who would be responsible for that? thumbnails service, search service, ...?
AFAICT tika does not extract image information from audio files - so we would need to duplicate some of the functionality here in the search service to write thumbnail information to the metadata. The thumbnail service does not generate images before they are requested, so we would have the information only on the second directly listing, so search service seems like a better fit actually...

Updated and CI fixed

Nice, thanks!


text:
Create contribution.md

Closing Issue/Referencing Issue
issue reference (#)
Changes Made In Pull Request

Please merge

repo: ows-ali/Hacktoberfest


text:
Bidirectional arrow (<-->) results in "The content of elements must consist of well-formed character data or markup".

I have an XSLT that converts these bidirectional arrows into <-->. Using the encoding I can skip the XML parsing error. However, with encoded <--> the HTML transformation fails to the render SVG image.
If I use unidirectional arrow, it works. ( afsvm-->afs)
Excerpt Dita file
     <foreign outputclass="embed-plant-uml">
        @startuml
        left to right direction
        agent apc as "PC"
        agent afsvm as "VM"
        agent afs as "FS"
        apc-->afsvm
        afsvm<-->afs
        @enduml </foreign>

@DivyaRao23 can you try to re-download and re-install the plugin? I made a change to the "extensions.jar" library.
In addition I think you also need to install this extra application: https://plantuml.com/graphviz-dot to have the PlantUML libraries properly be able to create your diagram.

@raducoravu, Thank you for the reply.

I downloaded the latest plugin and reinstalled (I did use, dita --install to reinstall after copying the plugin). Still observe the same error.

 [xslt] Error evaluating (fn:parse-xml(...)) in xsl:copy-of/@select on line 21 column 141 of xhtmlSVG.xsl:
 [xslt]   FODC0006: First argument to parse-xml() is not a well-formed and namespace-well-formed XML
 [xslt]   document. XML parser reported: org.xml.sax.SAXParseException; systemId:
 [xslt]   file:/dita-ot-3.6.1/plugins/com.oxygenxml.diagrams.svg/xsl/xhtmlSVG.xsl; lineNumber: 30; columnNumber: 26; The content of elements must consist of well-formed character data or markup.

I have the following version of Dot installed.

java -jar plantuml.jar -testdot
The environment variable GRAPHVIZ_DOT has been set to /usr/local/bin/dot
Dot executable is /usr/local/bin/dot
Dot version: dot - graphviz version 2.49.0 (20210828.1703)
Installation seems OK. File generation OK

I'm using dita-ot-3.6.1 and my Java version: 17.
With the flowers project, I do not get any SAXParseException error reported in the console. However, the SVG image doesn't render completely. flowers-html5.zip

@DivyaRao23 I think I made this work, maybe you can re-download the plugin and try again.

@raducoravu , I downloaded the plugin again and reinstalled. It is now working. Thank you so much for the help.

@DivyaRao23 I'm glad I could help.

repo: oxygenxml/dita-ot-diagrams-plugin


text:
Can you add please Spring5 support?

import spring4 cause error.

Hi @itskovichanton.
The releases of this library are bind to the releases to the latest Spring Boot release. Currently we have 1.5.2 that uses Spring 4.x.
Unfortunately, I'm currently working alone, thus I cannot handle a compatibility matrix with several branches.

Closed for now since out of scope.

authors: itskovichanton robertotru

repo: ozimov/spring-boot-email-tools


text:
targets/basic_routing

When the first installation,I follow the README.
cd p4factory/targets/basic_routing/ make bm sudo ./behavioral-model
It is successful.
But today,I want to do the same thing,use these code
sudo ./behavioral-mode
error
`wpq@wpq-VirtualBox:~/p4factory/targets/basic_routing$ sudo ./behavioral-model
No PD RPC server address specified, using 127.0.0.1:9090
No listener specified, switch will run in standalone mode
P4 Program:  basic_routing
Starting RPC server on port 9090
bmi_port_interface_add(port_mgr, veth_name, n_veth, pcap_filename): ERROR -1 at main.c:434wpq@wpq-VirtualBox:~/p4factory/targets/basic_routing$
`
thx

Every time you restart your machine, you need to create the veth interfaces again using the script under tools. Otherwise you get the error you are seeing because the switch cannot connect to any interface.

authors: antoninbas peiqiaoWang