using hash router, getting the parameters from the url is incorrect

Situation:
When the router is set to hashmode the parameters are not correctly parsed from the url.
and you get the error that the state is not found.
Analysing:

UrlUtils.readParams expects an unhashed routed url

easiest way yo fix:
const unHashUrl = url.replace('#/', '');
const response = new SigninResponse(UrlUtils.readParams(unHashUrl, this.settings.response_mode));
and
const unHashUrl = url.replace('#/', '');
const response = new SignoutResponse(UrlUtils.readParams(unHashUrl, this.settings.response_mode));

When i look at
export class UrlUtils {
    public static readParams(url: string, responseMode: "query" | "fragment" = "query"): URLSearchParams {
        if (!url) throw new TypeError("Invalid URL");
        const parsedUrl = new URL(url, window.location.origin);
        const params = parsedUrl[responseMode === "fragment" ? "hash" : "search"];
        return new URLSearchParams(params.slice(1));
    }
}

UrlUtils.readParams does support hash, see optional parameter responseMode. What is your this.settings.response_mode?

The response mode is mend to be for the stsservice urls. That is different then the urls for the callbacks (our site uses a hash routing /#/path?a=foo) fragment is used for hash parameters only. Like #a=foo. So for the stsmanager it needs query parameters and for the site too. The function tries to get the state and code parameters from the query part but when using hash routing the query parameters in the location = empty.

@frontendplace thanks for clarification
The js library did this for that case: https://github.com/IdentityModel/oidc-client-js/blob/dev/src/UrlUtility.js#L34-L40
Does something like this work too (untested)?
export class UrlUtils {
    public static readParams(url: string, responseMode: "query" | "fragment" = "query"): URLSearchParams {
        if (!url) throw new TypeError("Invalid URL");

        if (responseMode == "query") {
            // if we're doing query, then strip off hash fragment before we parse
            idx = url.indexOf("#");
            if (idx >= 0) {
                url = url.substr(0, idx);
            }
        }

        const parsedUrl = new URL(url, window.location.origin);
        const params = parsedUrl[responseMode === "fragment" ? "hash" : "search"];
        return new URLSearchParams(params.slice(1));
    }
}

You can try it in the unit test. With url http://foo/#/bar?baz=param1
Removing the hash. You get http://foo//bar?baz=param1
Then the search params are correct. Yes

@frontendplace ok lets use remove #/
Please adapt your merge request, as such that the the replacement is done within the UrlUtils.readParams with a comment why this is needed and a unit test...

I will do that

I have changed it an tested with
        if (responseMode == "query") {             const hashIdx = url.indexOf("#");             if (hashIdx >= 0) {                 url = url.substr(hashIdx + 1);             }         }

Follow up here for further discussion on my comments on the merge request https://github.com/authts/oidc-client-ts/pull/736#discussion_r1010189305
To sum it up, the described case seems invalid as the URL in the example at https://github.com/authts/oidc-client-ts/issues/734#issuecomment-1295241647 does not have a query parameter and imo it is not the job of this library to fix special client side URLs.
See https://www.rfc-editor.org/rfc/rfc3986#section-4.1 for reference.

The url with /#/ notation is a regular used url in hash routed angular apps. So the url should be corrected to a kind of url mentioned in your comment prior to getting the parameters. Like it was also dealed with in the previous js version.

Yes - the hash based url routing is common. I am just saying that the URL having ? after the hash # start is client specific and thus needs to be dealt with on the client code which expects to retrieve such and not in a generic library.
If you receive callback requests from an IdP which appends query mode parameters 'at the end' instead of at the correct place for query parameters before the hash, I suggest you open an issue there too.
I am open to discuss possible solutions which avoid side effects when an URL is correctly generated as imo we should support the correct order first and foremost.

Maybe I do not understand. Do y mean the fix must be outside the urlUtils. Or outside the oidc-client.ts library as a whole?  The previous version supported (normal) angular syntax with first hash then a route then ? & separated parameters. The oid-clientts library takes care or redirecting with callback urls.

Yes, i mean outside of oidc-client-ts as a whole. To my knowlege oidc-client-ts does not do anything Angular specific. Thus the actual integration into routing is done "outside". There a "fix" can also be made but also that fix is very specific to wrong behavior of the IdP you seem to be using.
To make an example with the URL http://foo/#/bar?baz=param1 from above.
Let's assume you register/provide http://foo/#/bar as callback URL (which is configured via application specific URL routing to trigger the oidc-client-ts authorize callback) to the IdP.
If the IdP redirects back to http://foo/#/bar?bar=param1 then the IdP simply has "appended" the query. This is wrong and it means that the IdP does not support hash based URL routing as it generates the redirect URL wrong.
Instead a correct IdP would generate http://foo/?bar=param1#/bar which works prefectly fine as the query is now at the correct place and will be processed correctly while before there was just some additional application specific data inside the hash.
oidc-client-ts has no idea what and how to process this application specific hash data and thus imo nothing should be added for it, even if this behavior might be common.
Instead i suggest that whatever angular code you are using to trigger the oidc-client-ts callback, there you can modify the URL to the application specific needs.

Ok sorry to hear y don't want to support the angular way of hash routing like the previous version did and other libraries do.
http://foo/?bar=param1#/bar you suggest won't be parsed as a correct url for angular routing.
Is it possible you add to your readme.md that y don't support hash routing urls with parameters anymore? That will prevent a lot of time wasted for other developers that want to change from the previous version to this ts version.

@frontendplace i think we need to know more to find a suitable solution which does not complicate using this library while at the same time does not introduce any specific hacks to the code base.
Regarding the router.implementation, can you share details or a link to some example with angular url routing? I find it hard to believe that the angular has based url routing would not support query parameters. Maybe play around with the location of the slash(es)?

For completeness, i also suggest to investigate if you can use response_mode fragment (https://authts.github.io/oidc-client-ts/interfaces/OidcClientSettings.html#response_mode) to see if you can avoid the issue altogether and potentially avoid the issue that the IdP generates wrong callback urls in query mode.
And last not least, in the oidc-client-ts Angular example application https://github.com/authts/sample-angular-oidc-client-ts/blob/c9b21715c19fd68af0a9b4d4b9fedaa815e52592/src/app/core/components/signin-callback.component.ts#L14 this is the place where you would need "fix up" any invalid/wrong callback URL. The reason why the callback URL might need fixing is a combination of URL routing needs and potentially wrongly generated URLs by the IdP in question (especially when the configured callback URL contains a hash).

This is part of documentation of angular that explains the routing strategy. https://angular.io/api/common/HashLocationStrategy
A path is placed after the hash.
The url that we get back from the stsserver is domain/apppath#/signin-callback?code=foo&status=bar.

This is a presentation
https://codecraft.tv/courses/angular/routing/routing-strategies/

This is part of documentation of angular that explains the routing strategy. https://angular.io/api/common/HashLocationStrategy

Yes, but that documentation is lacking. It does not say anything about query support or what if someone wants to use an additional set of hash values.

The url that we get back from the stsserver is domain/apppath#/signin-callback?code=foo&status=bar.

OK, this is wrong. I don't know what stsserver is but it obviously does not expect the callback URL to end with a hash and adds the query parameter at the wrong place.
If you want to support this in your specific application it can work without problem. Simply process the URL before passing it along to signinCallback.

Oh I think there is a solution for us. I see that calling the signinRedirectCallback function I can pass a modified url.
Default the window location href is used and that includes the hash

@longsleep Thanks for your excellent help here!

You are very welcome - closing this.

Oh I think there is a solution for us. I see that calling the signinRedirectCallback function I can pass a modified url. Default the window location href is used and that includes the hash

thanks! When use Vue.js or React.js's HashMode Use the following code work fine.
mgr.signinRedirectCallback(window.location.href.replace('/#/','/'));


Hide action button on realtime only collection

Hide Browse and Create a document button if the collection is realtime only
fix #291 #290

Codecov Report

Merging #299 into 2.x will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              2.x     #299   +/-   ##
=======================================
  Coverage   76.67%   76.67%
=======================================
  Files          39       39
  Lines         763      763
=======================================
  Hits          585      585
  Misses        178      178

Continue to review full report at Codecov.

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


Error in phantom.js

When loading swellrt in phantom.js, the following erros appears:
2016-10-02T09:47:58.913Z getting https://staging.teem.works/teems/cHJvdG90eXBlLnAycHZhbHVlLmV1L3MrbTNYVXBxNTVEZ0E
Atmosphere callback 200
Atmosphere callback 200
TypeError: Function.caller used to retrieve strict caller

  https://swellrt.staging.teem.works/swellrt/5DD6799DBD533E8AD639C0D7159FD847.cache.js:13148 in collect
  https://swellrt.staging.teem.works/swellrt/5DD6799DBD533E8AD639C0D7159FD847.cache.js:13032 in captureStackTrace_0
  https://swellrt.staging.teem.works/swellrt/5DD6799DBD533E8AD639C0D7159FD847.cache.js:21870 in toJava
  https://swellrt.staging.teem.works/swellrt/5DD6799DBD533E8AD639C0D7159FD847.cache.js:47237 in onMessage_1
  https://swellrt.staging.teem.works/swellrt/5DD6799DBD533E8AD639C0D7159FD847.cache.js:47003 in onMessage_0
  https://swellrt.staging.teem.works/swellrt/5DD6799DBD533E8AD639C0D7159FD847.cache.js:47123 in onMessage
  https://swellrt.staging.teem.works/atmosphere/atmosphere.js:2454 in _f
  https://swellrt.staging.teem.works/atmosphere/atmosphere.js:2444 in _invokeFunction
  https://swellrt.staging.teem.works/atmosphere/atmosphere.js:2540 in _invokeCallback
  https://swellrt.staging.teem.works/atmosphere/atmosphere.js:1278 in onmessage
Atmosphere callback 200
Atmosphere callback 200

Would you mind, please, walking through the steps to replicate this issue?
Environments, process, steps taken... these are important so that we can help.
Thanks!

It seems it's related with JS strict mode. @atd is there any JS module involved in the execution? e.g a test library?

Not sure.. We are using https://github.com/prerender/prerender with the script at https://github.com/P2Pvalue/docker-prerender/blob/master/server.js

This seems to be one of the offending functions

I have searched arguments.callee.caller and cannot find the string in SwellRT code
Seems to be related with https://github.com/gwtproject/gwt/issues/9132


Adding new level to MultiIndex raisese error

Describe the bug
Adding a new level to MultiIndex using set_index raises error.

[x] I have checked that this issue has not already been reported.
[x] I have confirmed this bug exists on the latest version of pandera.
[ ] (optional) I have confirmed this bug exists on the master branch of pandera.

Note: Please read this guide detailing how to provide the necessary information for us to reproduce your bug.
Code Sample, a copy-pastable example
       import pandera as pa
        example_schema = pa.DataFrameSchema(
        {
        "key_2": pa.Column(str),
        "key_3": pa.Column(int)
        },
        index = pa.Index(name="key_1", dtype=int)
        )
        new_schema_1 = example_schema.set_index(["key_2"], append=True)
        print(new_schema_1)
        # this raise error
        new_schema_2 = new_schema_1.set_index(["key_3"], append=True)
        print(new_schema_2)

Expected behavior
A new level should be added without error and it yields a new MultiIndex object with an additional level.

fixed by #751


Resource Landing Page: Add Readme Functionality

On the resource landing page, add the ability to automatically render a readme file directly on the page. If a user has uploaded a readme.txt file or a readme.md file, it should automatically be displayed in the readme area on the page. If no readme.txt or readme.md file exists, the readme viewer should not appear on the page. Only readme files in the root content directory will be supported.
The readme content area should be styled/treated the same as the file content area. It should have a minimum and maximum vertical height on the page, but should be draggable to make it larger and scrollable if the content doesn't fit within the maximum height.
If a readme.txt or readme.md file are present in the resource (even if it is only discoverable), the readme file should get rendered on the page.
If both a readme.txt and a readme.md file are uploaded to a resource, the default display will be readme.md.
Need to add the following text to the "Uploading" information box that shows up below the Content area in Edit Mode:

"If you upload a readme.txt or readme.md file at the root level in your resource, it will be rendered automatically on the page. Markdown formatting is allowed in readme.md. Learn more about Markdown."

The text "Learn more about Markdown" should link to the following page:
https://guides.github.com/features/mastering-markdown/#syntax
See the mockup of the page for what it should look like and the location on the page:
<EMAIL_ADDRESS>

Live and functional on www. Closing this one, but see #3176, which is a new issue about rendering readme.txt.


Application Insights profiler install instructions very unclear, near impossible to follow

I've spent the last two days trying to convince Application Insights to collect profiler traces from an Azure virtual machine, a scenario that "should just work".
I'm following the instructions in this article with no success.
Some notes:

There's a requirement to install an unrelated monitoring agent. Why?
Then there's a step for manually editing that agent's config file. What about ARM template deployments?
The documentation helpfully provides a JSON sample for what is an XML config file in the (now outdated) ARM template samples.
The provided snippet in the documentation is out-of-context. Should I just put it anywhere?
Apparently the SDK integration is also required. What about codeless monitoring? That works in App Service!
App Insights supports an instrumentationKeyMap setting, so that multiple virtual apps can send telemetry to different instances of App Insights. However, the WADCfg has only a single sink for "ApplicationInsightsProfiler"... so... this can't ever work for multi-app servers.

To summarize, after days of effort, following these instructions I could not enable a basic feature that should just work out of the box, just like it does with New Relic. The process is completely impractical within ARM templates, and will never work in quite ordinary scenarios.

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

ID: 02cd947d-30c3-2551-4dd6-9762db6869f7
Version Independent ID: 4e5a974c-686c-2005-c036-55dac3107573
Content: Monitor performance on Azure VMs - Azure Application Insights - Azure Monitor
Content Source: articles/azure-monitor/app/azure-vm-vmss-apps.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @lgayhardt
Microsoft Alias: lagayhar

@peter-bertok Thanks for your comment. We will review the issue and get back to you shortly.

Hi @peter-bertok, Have you checkout our profiler documentation? There is one for Enabling Profiler for an Azure VM.  I think this might be more of what you are looking for.

I'm looking for an ARM Template, PowerShell is for some things, but it's
not a viable solution for many scenarios.
On Tue, Mar 30, 2021 at 9:40 AM Lauryn Gayhardt @.***>
wrote:

Hi @peter-bertok https://github.com/peter-bertok, Have you checkout our
profiler documentation? There is one for Enabling Profiler for an Azure VM
https://docs.microsoft.com/en-us/azure/azure-monitor/app/profiler-vm. I
think this might be more of what you are looking for.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/72812#issuecomment-809759386,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADSLGW7GQ26PO3DY4CDJNVDTGD6WXANCNFSM4Z2RQDHA
.

Closing this issue for now, @peter-bertok let us know if you have any additional questions. Thank you!
#please-close


Brittleness in translator relation tests

Change these to not rely on fixed indexing of prover polynomials.

How do you think we can do this?

For completeness: can with an index variable and stash when e.g. &z_perm == &all_entities[i] then z_perm_index = i, then use that index. This should get better once ProverPolynomials owns its memory


Deletion of every item set

Unlike the runes, the itemsets are being all deleted instead of the DFF name one

This shouldn't be too hard to fix, but right now I'm a bit busy with work. Ill try to take a look at it once I get more free time.


Example app at user limit

Attempting to run the auth example shows: https://www.dropbox.com/s/faarnfl83mlu051/Screenshot 2016-07-26 15.27.31.png?dl=0
Presumably Dropbox's example app should allow new users.

@braincore can you fix this?

Switched the example app back to yours.


Uncaught Error: This socket has been ended by the other party

[Enter steps to reproduce:]

...
...

Atom: 1.15.0 x64
Electron: 1.3.13
OS: Unknown Windows version
Thrown From: atom-typescript package 11.0.1
Stack Trace
Uncaught Error: This socket has been ended by the other party
At events.js:160

Error: This socket has been ended by the other party
    at Socket.writeAfterFIN [as write] (net.js:286:12)
    at Immediate.setImmediate (/packages/atom-typescript/dist/client/client.js:139:22)
    at runCallback (timers.js:574:20)
    at tryOnImmediate (timers.js:554:5)
    at processImmediate [as _immediateCallback] (timers.js:533:5)

Commands
  2x -2:52.9.0 editor:newline (input.hidden-input)
     -2:51.8.0 core:paste (input.hidden-input)
     -2:44.9.0 core:save (input.hidden-input)
     -2:28.4.0 core:backspace (input.hidden-input)
     -2:23.8.0 core:save (input.hidden-input)
  2x -1:53.7.0 core:backspace (input.hidden-input)
  4x -1:45 core:save (input.hidden-input)
 12x -0:49.1.0 core:backspace (input.hidden-input)
  2x -0:46.7.0 core:undo (input.hidden-input)
  4x -0:43.2.0 core:backspace (input.hidden-input)
     -0:40.2.0 core:save (input.hidden-input)
  2x -0:39.2.0 editor:newline (input.hidden-input)
     -0:36.8.0 core:backspace (input.hidden-input)
     -0:36 editor:newline (input.hidden-input)
  3x -0:35.3.0 core:backspace (input.hidden-input)
     -0:33.8.0 core:save (input.hidden-input)

Non-Core Packages
atom-beautify 0.29.17
atom-typescript 11.0.1
autoclose-html 0.23.0
autocomplete-java 1.2.5
autocomplete-paths 1.0.5
emmet 2.4.3
linter 2.1.2
monokai-dark 1.3.0

I have the same issue, any solution?

Restarting Atom usually fixes this, at least until tsserver crashes again. I have a fix in the works to handle this sort of thing gracefully, but it might be some time until I can publish it.


Elastic-cache-module

elastic cache single module to create Redis cluster as well as the Memcached cluster.

Elastic cache module for Redis and Mamcached


As a non-confrontational (i.e. shy) student, I can push certain tags

... (e.g. “Assistance required!”, “Unable to complete :(”) to the application, so that I can gather my professor’s attention without direct confrontation.

Implemented in #137.


Archive files delete right order  [wip]

unit test to confirm bug #847

bug fixed. Code of FileTarget isn't in a good condition. Need some refactoring


Filter IsAny not wokring in Int Type

Hi,
Query Filter 'IsAny' not working in integer datatype.
The below query is not working nad it throws error
query {
clients(filter: "Code.isAny[5,2,3,10,11,12]") {
items {
Code
}
}
}
{
"errors": [
{
"message": "Field 'isAny' not found on type 'Int'"
}
]
}

You need to call it like a method Code.isAny([5,2,3,10,11,12])" note the brackets

It is not working even with the brackets. It throws below error
Wat is the limitation of isAny? Does it support all data types? why it throws below error?
{
"errors": [
{
"message": "Method 'isAny' not found on current context 'Nullable`1'"
}
]
}


Feature/sponsorpg

Sponsor page content. Also changed default  element font size to 1em and fixed a nested tag error in resources.js (ul nested inside p that was throwing a warning)

Going to create a new PR rebased ontop of main....


build schemas before parameters for immediate linking

@iscai-msft , I think it is better to link issue or add description for track in the future ^ ^

will do, thanks!


"antialiasing" of active toggle-button in Chrome

@comerc please give us more details about the browser/OS. This is how we see them on Chrome v50 MacBook Retina

Best,
Alex

Chrome: Version 50.0.2661.102 (64-bit)
OS: Linux Mint


Support for internal visibility like gitlab?

(Followings are from docker image f23a2efe3e3f)

Gogs version (or commit ref):  <IP_ADDRESS>6
Git version: 2.11.3
Operating system: $ uname -a
 > Linux 2fa12c5c7d12 4.15.7-1.el7.elrepo.x86_64 #1 SMP Wed Feb 28 14:38:13 EST 2018 x86_64 Linux

Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Description
Internal visibility like gitlab ：

I'd like to demonstrate some of my projects to the public while others are restricted to my organization members only. So one doesn't have to suffer from selecting all users as a team.

I think it's same as https://github.com/gogits/gogs/issues/2308.


Release date for 1.6.0?

Just wondering when 1.6.0 and its assumeNearbyYear option will be officially released. Thanks!

+1

My plan is to have 1.6.0 released by the end of the month, but we will work with some beta/rc releases first. So I hope you guys can give those RC's a test before the final release!
I will ping this issues when the beta/rc is out! (and the stable release)

@jeremyschultz @tclindner I've just release v1.6.0-alpha please test this release and report any issues! Then we should have a stable release (if no major issues) by the end of the month!

@jeremyschultz @tclindner 1.6.0 was just released! Check the release page for the changelog!

Awesome, nice work! Thank you, @acrobat


Kubernetes deletion is not working

Description
Unable to delete a kubernetes cluster.
When i tried to delete a k8s cluster it loaded for only a couple of seconds and when the deployment list appeared again the cluster also appeared and nothing was deleted.
Screencast from 21-05-23 16:38:45.webm

Verified, Devnet
Deployed and delete 2 kubernetes master with/without a worker and both were succesfully deleted.


Does this code need large GPU memory？

Hello! During the training of cifar10 dataset, do you encounter that when the batchsize is set to 2048, you can't run on the dual card nvidia3090? Display memory overflow.
So I changed the batch size to 256, which is still a memory overflow.
Finally, I had no choice but to change it to 128 to run.
However, compared with simclr and swav codes, the batch size that can be set under the same device is not so small. I can generally run 2048 or 1024. Is this normal?
My device is nvidia3090, dual card, with 48g of running video memory. The training data set is cifar10
If you can easily answer, I will be very happy!

Hi,
Unfortunately, I can't tell what is the issue as I don't have that much gpu memory  to test it with a  such large batch size. However, in the paper it is mentioned that VICReg consumes less memory  than SwAV.


Add leaderboard data using API

This pull request adds functionality to the leaderboard with scores fetched from the API. It includes several new module functions that creates the game, fetch the score, render page that is triggered when the page finishes loading and calls the render function to display the leaderboard.
Changes made

[x] Removed dynamic scores earlier used
[x] Create game with a unique ID
[x] Function to fetch game scores from API
[x] Function to create score data
[x] Function to submit game data
[x] Function to render leader board data on load
[x] Refresh & Submit button functionality Added
[x] Tested the functionality and sorting behavior.

Please kindly review this and let me know if further improvement are needed.
Thank you.

Thank you @rloterh


App not quiting

pressing quit on app since 3 hours

Please also add your environment and logs. sigh


build.zig: Add ability to install a shared library

Initial build.zig only installed a static library. This is fine for Zig itself but not sufficient for other ecosystems.
Current patch doesn't break backwards compatibility but does a little refactoring where the old function addRaylib uses the newly added function addRaylibTo, which is more flexible. addRaylib still generates a static library, whereas the new addRaylibTo allows to choose the desired artifact and only applies necessary build steps to that artifact.
Additionally, since a dynamic library is also a common artifact, build.zig also installs it to a default location of zig-out/lib/libraylib.dll.
Example code how to generate a shared library artifact in Zig:
const libraylib = b.addSharedLibrary(.{
    .name = "raylib",
    .target = target,
    .optimize = optimize,
});
raylib.addRaylibTo(b, libraylib, target, .{});

Shouldn't this be defining BUILD_LIBTYPE_SHARED? or is that done elsewhere?

I didn't know about this constant, let me rework this a bit. Thanks!

I didn't know about this constant, let me rework this a bit. Thanks!
It's required to build a usable DLL.
Also, anything consuming the shared lib should be defining USE_LIBTYPE_SHARED, though usually you can get away without it. I think the function calls have to go through some sort of "trampoline" instead and are less efficient though.

@greenfork It seems this is a considerable redesign of the build.zig, adding an extra level of complexity and maintenance cost. Is it really required? Would this change improve the experience for most of the users requiring this script?
I'm not using neither maintaining this script but I think other users should review this PR.

@Peter0x44 I have addressed your comment and also ported -fvisibility=hidden from the Makefile:
https://github.com/raysan5/raylib/blob/e46b6147fc30e69dd300bbda90ff0a62bf8a4df0/src/Makefile#L386-L397
In order to port -fvisibility=hidden I also had to change from static flag concatenation to dynamically allocated array or static strings.

@greenfork It seems this is a considerable redesign of the build.zig, adding an extra level of complexity and maintenance cost. Is it really required? Would this change improve the experience for most of the users requiring this script?
I'm not using neither maintaining this script but I think other users should review this PR.

It does change a lot of lines, but conceptually it is not so scary. The patch essentially does these things:

addRaylib is split into two functions addRaylib and addRaylibTo.
addSharedRaylib function is added that uses addRaylibTo.
raylib_flags now use allocated array instead of a static array.

The result is that Zig can now build a shared library and, more importantly, cross-compile it to different systems. Users of Raylib can now develop on Linux and cross-compile a dynamic library for Windows. For me this is a very essential functionality because my language of choice can only use shared C libraries and I would like to do cross-compilation of my games from Linux to Windows.
PS: The cross-build for MacOS is broken at the moment both on master and on this branch, this will require some input from MacOS users.

I'm not using neither maintaining this script but I think other users should review this PR.

@raysan5 should I find someone or there's someone already and I should just wait?

Also, I have a question about the goal of build.zig. Does it strive to be a fully featured build system like existing Make and CMake or is this just a tool for some specific purpose? @raysan5 when you say that added complexity may be not worth it, does it mean that you don't consider adding a shared library generation by Zig or that you would like to achieve it with a lesser cost?

@greenfork The goal of build.zig was providing a simple alternative to default Makefile. By default raylib is compiled as a static library and that's the recommended usage... but I'm aware that at this point there are many raylib bindings that require a shared library.
I will ask in raylib Discord for any Zig experts that can review and evaluate this change.

Line 5: To make a proper alias, you can just do
pub const addRaylib = addStaticRaylib;

Lines 25 and 46: Should use catch instead of if statement for error handling
Line 76: Not sure if this flag is needed, if you throw it away, you can throw away all the arena allocation since all other flags are comptime and you can merge them with ++ like it was done originally. Or could roll it into common flags since I don't think it does anything when building a static library. Then you can also get rid of the error handling altogether in the wrapping functions.
Lines 267+: I don't think you need to install headers for each library type since the headers are the same. Can get rid of the whole loop and just replace it with one extra installArtifact instead.

As far as I'm concerned. I think adding a basic build script for Zig that can create a shared library is alright but I don't think it should go beyond just basic functionality. I use the provided build script for quick and dirty, get things setup fast and then migrate to writing my own customized build scripts as my projects get out of prototype state.
Since the code is already written, and could be cleaned up with a few steps then it could be fine to accept it but as a Zig user, I wouldn't need this functionality myself. Zig can very well use the library produced by make script with no issues, and fewer things to maintain is better.

Line 76: Not sure if this flag is needed, if you throw it away, you can throw away all the arena allocation since all other flags are comptime and you can merge them with ++ like it was done originally. Or could roll it into common flags since I don't think it does anything when building a static library. Then you can also get rid of the error handling altogether in the wrapping functions.

I have read about the -fvisibility=hidden flag and I'm not sure how it will interact with Zig, needs experimentation. GCC says that it supports it on Windows too and for clang I didn't find any information. I honestly don't know if this flag will work as expected on Windows when used from Zig.

Zig can very well use the library produced by make script with no issues

My main reason for this was easy cross-compilation. Zig allows for that with ease.

Unfortunately I failed to create bindings for Pony because of some strange interaction between the Pony runtime and Raylib. This patch requires more work to do and I will leave it for the next volunteer, closing this one.

My understanding of -fvisibility=hidden is that it doesn't affect windows at all
With windows, only the functions marked __declspec(dllexport) can be called.
An so by default exports everything. -fvisibility=hidden effectively gives you the windows behavior, except you need to use __attribute__((visibility("default"))).
I believe macos dylibs work the same way, but I'm not 100% on that, and I can't test it.


Revert "Keras grouped convolutions rollback"

This PR resubmits the changes from #36773 which were rolled back in dd2ea875d92eeb83e81b1cb92e29e61d488e98b2.
I couldn't reproduce the failure mentioned in https://github.com/tensorflow/tensorflow/pull/36773#issuecomment-627079900 locally with or without XLA, so I am resubmitting the changes to see if CI is happy now.
@tanzhenyu could you take a look?

@tanzhenyu Any chance you could approve this so CI can run? Since this is a git revert of the commit that rolled back the merge of #36773, there should be no need for an indepth review again.

I rebased the PR and resolved the merge conflicts. @gbaned is there anything that still blocks this?

I'm not sure why it didn't show up in external CI. But here's what we got:
  File "/tensorflow/python/keras/layers/convolutional_test.py", line 296, in test_conv3d
    self._run_test(kwargs, expected_output_shape)
  File "/tensorflow/python/keras/layers/convolutional_test.py", line 265, in _run_test
    expected_output_shape=expected_output_shape)
  File "/tensorflow/python/framework/test_util.py", line 1717, in decorated
    result = f(self, *args, **kwargs)
  File "/tensorflow/python/keras/testing_utils.py", line 227, in layer_test
    model.train_on_batch(input_data, actual_output)
  File "/tensorflow/python/keras/engine/training.py", line 1476, in train_on_batch
    logs = train_function(iterator)
  File "/tensorflow/python/eager/def_function.py", line 766, in __call__
    result = self._call(*args, **kwds)
  File "/tensorflow/python/eager/def_function.py", line 826, in _call
    return self._stateless_fn(*args, **kwds)
  File "/tensorflow/python/eager/function.py", line 2812, in __call__
    return graph_function._filtered_call(args, kwargs)  # pylint: disable=protected-access
  File "/tensorflow/python/eager/function.py", line 1838, in _filtered_call
    cancellation_manager=cancellation_manager)
  File "/tensorflow/python/eager/function.py", line 1915, in _call_flat
    ctx, args, cancellation_manager=cancellation_manager))
  File "/tensorflow/python/eager/function.py", line 549, in call
    ctx=ctx)
  File "/tensorflow/python/eager/execute.py", line 60, in quick_execute
    inputs, attrs, num_outputs)
tensorflow.python.framework.errors_impl.UnimplementedError:  Hit a case for convolution that is not implemented on GPU.
	 [[{{node cluster_131_1/xla_compile}}]] [Op:__inference_train_function_7135]

@tanzhenyu Thanks for taking another look, very much appreciated.
After some digging I was able to run the tests with XLA on a cloud VM and could reproduce the failure locally. It looks like currently train_on_batch fails for 3D grouped convolutions when XLA is enabled.
Since the underlying failure is not related to this PR and would also appear when using grouped convolutions with tf.nn.conv3d on master, I disbled the relevant part of the test in 215161ff7a7cd136feac27686561bcedbdaef493. Please let me know if that works for now.

Thanks for approving, looks like CI is also happy this time 💚

Thanks for approving, looks like CI is also happy this time 💚

Great, thanks for fixing!


Support x64 build mode on Windows with DMD v2.076.0.

This request is used to update the last pull request #1263 using latest thrift.
Running make check failed  because of missing some third-party libraries.
I have run all the test codes by hand in Visual Studio Code with DMD 2.076.0 on Windows 10 and Linux . All the tests passed.
The auto-test tool for D need improved. Maybe it's better to use CMake or DUB (a build tool for D).

To address the last sentence in your comment above: We're standardizing on cmake and moving everything towards that, albeit slowly.  Could you tell me if this makes older D versions unusable on Windows, and if so which D version on Windows would be the earliest supported?

On the Travis build - the Ubuntu Trusty image is using D 2.070.0 (see: https://github.com/apache/thrift/tree/master/build/docker).
Currently we don't really have an official record and mechanism for minimum required versions of things.  The closest thing is a store in the backlog that I opened recently:  https://issues.apache.org/jira/browse/THRIFT-4344
It sounds like we may need to make a decision as to how old an environment we want to continue to support for D in future thrift releases.  The project has been really relaxed in support, for example the Perl library claims to require only v5.10.0 or later which is ancient (trusty has 5.18.2 for example).  What I'm finding is that the matrix for CI builds is just too wide and unruly, and we need to start making some decisions about how far back we need to support with each release.

The D compiler upgrades quite frequently. The latest version is not always compatible with the older one.
I have a CMake module for D. However it needs more time to add the module to Thrift and to reorganize the test codes. My project will use Thrift and I want to make the Thrift-D lib stable firstly.

I got the same error messages in DMD 2.070.0.

I found one reason why the make check failed.
The command line used by DMD (Failed):
dmd -g -ofunittest/debug/thrift/server/transport/ssl -unittest -w -wi -I../../lib/d/src src/thrift/server/transport/ssl.d libthriftd-event.a libthriftd-ssl.a libthriftd.a -L-levent -L-lcrypto -L-lssl unittest/emptymain.d
The underlying command line generated by DMD  (Failed):
cc unittest/debug/thrift/server/transport/ssl.o -o unittest/debug/thrift/server/transport/ssl -g -m64 -levent -lcrypto -lssl  -L/usr/lib/x86_64-linux-gnu -Xlinker --export-dynamic libthriftd-event.a libthriftd-ssl.a libthriftd.a -Xlinker -Bstatic -lphobos2 -Xlinker -Bdynamic -lpthread -lm -lrt -ldl
The correct command line reorderd by me (Succeeded):
cc unittest/debug/thrift/server/transport/ssl.o -o unittest/debug/thrift/server/transport/ssl -g -m64 -L/usr/lib/x86_64-linux-gnu -Xlinker --export-dynamic libthriftd-event.a libthriftd-ssl.a libthriftd.a -Xlinker -Bstatic -lphobos2 -Xlinker -Bdynamic -levent -lcrypto -lssl -lpthread -lm -lrt -ldl
I have no idea how to fix this by far.

It looks like you may not have static link libraries for libevent, and openssl on your system?  It also looks like automake didn't figure it out properly.

Any news?

I have updated all my patches base on the latest Thrift.
All the patches are tested with DMD 2.077.1 on  Linux Mint 18.3.

I'd like to see this squashed to one commit and rebased one more time; our CI builds are not in good shape but I can run it on my local system to prove it out and they deal with CI issues when that system is back and operating (sourceforge has been down for days and we depend on some components hosted there).

Opened THRIFT-4504 to track this.

The tutorials didn't get updated and as a result fail to build in the autotools build (build/docket/script/autotools.sh) which is what some of the Travis CI build jobs do.  I am attempting to fix this.

Yes, it still needs more works. I'll check them all.

THRIFT-3458 is also related, and the approach in THRIFT-3458 would allow us to add the apache git repo to dub directly so we should move in that direction, but still try to use dub to build in lib/d, test/d, tutorial/d.  I closed THRIFT-4504 as a duplicate of THRIFT-3458.  If we can get this PR to the point where there is a dub file at the top level of the repository that builds lib/d, that will allow us to add the apache repo to the dub registry and have it build and publish a dub package.

@Heromyth reminder this one still needs completion, if you have time..

We're prepping to push a dub package up once v0.12.0 is released.  I think those changes are in place, so this just needs to focus on getting it to build in cmake on windows.

@Heromyth I'd love to see this move to completion.  Any chance you will be able to address the remaining issues?  We've made a lot of progress with dlang in 0.12.0, we're even in dub now.  It's possible some of this work is no longer needed.  Not sure.

Going to close this, we'll end up redoing it anyway for cmake.  Thanks for the pull request though!


Ly exit with error code 0 when pressing 'Fn'

Hello,
When I press the 'Fn' key  (151 XF86WakeUp for my X server) on my Thinkpad T440s, ly simply exit with error code 0. I don't think that's a good thing for a DM ;-)

Are you building ly using the lastest commit ?

I'm using the latest (and only) release: v0.0.1. Was this bug fixed in the meantime ?

I've personally submitted a fix for a similar problem ( #45 ), and it has been merged only after the release v0.0.1 .
I suggest to try building ly from the very latest commit, and see if it solves the problem.

Yep, that's it. Thank you!


Update DG with sort feature

LGTM, do remember to upload the sort sequence and activity diagrams too!


Add language support for Vietnamese language

This PR follows the issue #223

Here is some screenshot under Vietnamese UI

@fmaclen I changed it and we are good to go

:tada: This PR is included in version 0.20.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


goroutine leak

millrun goroutine leak.
I use zap to wrap lumberjack, my code is following
w := zapcore.AddSync(&lumberjack.Logger{
  Filename:   "/var/log/myapp/foo.log",
  MaxSize:    500, // megabytes
  MaxBackups: 3,
  MaxAge:     28, // days
})
core := zapcore.NewCore(
  zapcore.NewJSONEncoder(zap.NewProductionEncoderConfig()),
  w,
  zap.InfoLevel,
)
logger := zap.New(core)

when I do testing, I got the following error:
Too many goroutines running after all test(s).
1 instances of:

**/vendor/gopkg.in/natefinch/lumberjack%2ev2.(*Logger).millRun(...)
**/vendor/gopkg.in/natefinch/lumberjack.v2/lumberjack.go:379 +0x58
created by **/vendor/gopkg.in/natefinch/lumberjack%2ev2.(*Logger).mill.func1
***/vendor/gopkg.in/natefinch/lumberjack.v2/lumberjack.go:390 +0x7e
ERROR: *manager.**TestSuite is failed by leak goroutine check.

I tried to used Close() method, but it still failed.
I read source code and fount that nowhere to close the logger.millCh, so this goroutine will live forever.

You're right that we do have a Close method, and that close method could stop the mill goroutine fairly trivially.  I'll get that fix out sometime soon, or you're welcome to send in a PR :)

one PR, please review it, Thanks

I am still facing same issue in v2
13 @ 0x1037dab 0x1037e63 0x1007f1e 0x1007d0b 0x15658c7 0x1069101
0x15658c6 tptest/vendor/gopkg.in/natefinch/lumberjack%2ev2.(*Logger).millRun+0x46 tptest/vendor/gopkg.in/natefinch/lumberjack.v2/lumberjack.go:379


[error]unknown flag: --disable-content-trust

When I try to run this build pipeline, I get
##[error]unknown flag: --disable-content-trust

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

ID: 971cc718-07a5-eaf3-f0bc-f85590309faf
Version Independent ID: c10c998c-e2cb-6946-3513-4efcad549fed
Content: Docker Content Trust in Azure Pipelines - Azure Pipelines
Content Source: docs/pipelines/ecosystems/containers/content-trust.md
Service: azure-devops
Sub-service: azure-devops-ecosystem
GitHub Login: @azooinmyluggage
Microsoft Alias: atulmal

We have the same issue.
We can no long build images with content-trust enabled

@noqcks  -- thank you for your question. This board is for documentation questions. Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

I am closing this issue now. You are welcome to @ mention me for any followup. We hope to hear from you again.


Changing No.# of classes

Hi,
For a custom dataset,

I changed the number of classes to 7  in the deeplabv3.py file (my data has 8 classes).
Eliminated the class_wieghts file requirement in the train.py file
Put my data & labels into the same folder structure

I get the following error in the line  loss = loss_fn(outputs, label_imgs):
IndexError: Target 78 is out of bounds.
Can someone point out where I am going wrong?
Thanks

Any luck @moizashfaque ?

Yes @vikasnr , The pixel values of the label image was, in my case, not same as the class. i.e. class 1 had pixel values of 78, and so on. Changing pixel values of class-1 to '1', class-2 o '2' ... class-7 to '7' in the label images resolved my error.

@moizashfaque I altered the trainIDs in before preprocessing to match with the classes I wanted. Basically keep some number <255 or just 255 for all irrelevant classes.

@vikasnr Yes, the number has to <255, and also <number of classes.
Closing the issue.


Icon request: Goodreads

Describe the requested icon
A Goodreads icon added to your brands category to link to my profile without having to use other icon fonts for it. I think Phosphoricons deserves to have one!
Screenshots

Added goodreads-logo in v2.0.0


Update FileInfo when the file is committed

Fixes https://github.com/Alluxio/alluxio/issues/13128
When this guard is added the issue does not happen anymore.
The trade-off is, when we getFileInfo on a file that is currently being written, the number of BlockId and FileBlockInfo we get will mismatch. I don't think that is a problem.

Codecov Report

Merging #13131 (959bcd2) into master (2c32656) will decrease coverage by 26.55%.
The diff coverage is 0.00%.

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

I should probably add an integration test

@apc999 One potential problem is, we might have been relying on this rehydration to commit files that are not properly closed. When a FileOutStream is not closed (and the process just dies), on the next getStatus the file is implicitly committed and we don't experience an issue (we are not aware this happened at all).
Now with this change, a FileOutStream that is not closed will have the file in the completed status forever? Is that a problem?
In my option, we should not rely on the rehydration logic here to handle incomplete files anyway. So I'm in favor of this change.

alluxio-bot, merge this please

Failed test is irrelevant to this PR
04:06:35.995 [INFO] Running alluxio.server.ft.journal.MultiMasterJournalTest
Error: 7.137 [ERROR] Aborted (core dumped)
04:07:07.796 [INFO] Running alluxio.server.ft.journal.MultiProcessCheckpointTest
04:08:06.265 [INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 58.464 s - in alluxio.server.ft.journal.MultiProcessCheckpointTest
04:08:08.498 [INFO] Running alluxio.server.ft.journal.TableMasterJournalIntegrationTest
Error: 0.192 [ERROR] WARNING: An illegal reflective access operation has occurred
Error: 0.202 [ERROR] WARNING: Illegal reflective access by org.powermock.reflect.internal.WhiteboxImpl (file:/home/jenkins/.m2/repository/org/powermock/powermock-reflect/2.0.7/powermock-reflect-2.0.7.jar) to method java.lang.Object.clone()
Error: 0.209 [ERROR] WARNING: Please consider reporting this to the maintainers of org.powermock.reflect.internal.WhiteboxImpl
Error: 0.210 [ERROR] WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
Error: 0.220 [ERROR] WARNING: All illegal access operations will be denied in a future release

To be cherry-picked to 2.4 and 2.5 after the test issue is resolved.

The test error mysteriously disappeared on the 6th or 7th run.

alluxio-bot, cherry-pick this to branch-2.4 please

alluxio-bot, cherry-pick this to branch-2.5 please


Query view crash

We are seeing the following stack trace:
2015-05-01 18:57:11,599 - werkzeug - INFO - <IP_ADDRESS> - - [01/May/2015 18:57:11] "GET /admin/queryview/ HTTP/1.1" 500 -
Traceback (most recent call last):
  File "./env/lib/python2.7/site-packages/flask/app.py", line 1836, in __call__
    return self.wsgi_app(environ, start_response)
  File "./env/lib/python2.7/site-packages/flask/app.py", line 1820, in wsgi_app
    response = self.make_response(self.handle_exception(e))
  File "./env/lib/python2.7/site-packages/flask/app.py", line 1403, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "./env/lib/python2.7/site-packages/flask/app.py", line 1817, in wsgi_app
    response = self.full_dispatch_request()
  File "./env/lib/python2.7/site-packages/flask/app.py", line 1477, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "./env/lib/python2.7/site-packages/flask/app.py", line 1381, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "./env/lib/python2.7/site-packages/flask/app.py", line 1475, in full_dispatch_request
    rv = self.dispatch_request()
  File "./env/lib/python2.7/site-packages/flask/app.py", line 1461, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "./env/lib/python2.7/site-packages/flask_admin/base.py", line 68, in inner
    return self._run_view(f, *args, **kwargs)
  File "./env/lib/python2.7/site-packages/flask_admin/base.py", line 354, in _run_view
    return fn(self, *args, **kwargs)
  File "./env/lib/python2.7/site-packages/airflow/www/utils.py", line 52, in view_func
    return f(*args, **kwargs)
  File "./env/lib/python2.7/site-packages/airflow/www/app.py", line 1248, in query
    db_choices = [(db.conn_id, db.conn_id) for db in dbs if db.get_hook()]
  File "./env/lib/python2.7/site-packages/sqlalchemy/orm/attributes.py", line 237, in __get__
    return self.impl.get(instance_state(instance), dict_)
  File "./env/lib/python2.7/site-packages/sqlalchemy/orm/attributes.py", line 573, in get
    value = state._load_expired(state, passive)
  File "./env/lib/python2.7/site-packages/sqlalchemy/orm/state.py", line 480, in _load_expired
    self.manager.deferred_scalar_loader(self, toload)
  File "./env/lib/python2.7/site-packages/sqlalchemy/orm/loading.py", line 608, in load_scalar_attributes
    (state_str(state)))
DetachedInstanceError: Instance <Connection at 0x7f8331cf0850> is not bound to a Session; attribute refresh operation cannot proceed

Looks good, I'm not sure why it doesn't create problems on our side though.


test: Adds common tests (couchdb, mysql)

This PR adds common tests for storage package. Tests are covering CouchDB and MySQL implementation.
Signed-off-by: Andrii Soluk<EMAIL_ADDRESS>

Codecov Report

Merging #10 into main will increase coverage by 1.59%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
+ Coverage   75.72%   77.31%   +1.59%
==========================================
  Files           1        2       +1
  Lines         173      291     +118
==========================================
+ Hits          131      225      +94
- Misses         25       39      +14
- Partials       17       27      +10

Impacted Files
Coverage Δ

component/storage/mysql/store.go
79.66% <0.00%> (ø)

Continue to review full report at Codecov.

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


Adapt the new Signal Service in Bridge UI/Backend

Now the MK proof for signals/message are stored in the SignalService address, not the Bridge contract address. We need to change the bridge relayer logics to adapt this change.

This can be done once we have an alpha-2 devnet

Completed in https://github.com/taikoxyz/taiko-mono/pull/13170


hs100 stable

Tested and proofed or important bugfix?

Looks OK.

Ok, but then "versionDate" looks weird


[filter demo] - Fix demo, looks like number filter is broken

In https://www.ag-grid.com/example.php
Equal and Not Equal filter not working on Number columns(Monthly Breakdown -> Jan).

They actually do work, just not in the way that you would expect.
The jan column shows a number, and a currency. That number is a round formatted number, so for example if you try looking for 81081 which is shown on the table, you will find nothing, because this is just a cell rendered value, while if you look for the real value - 81081.49 you will find it.

thanks @AmitMY

@ceolter I think closing this issue is a mistake.
I think the issue shows a problem with the main example.
Either there should be a custom filter there, or a text filter, but not the number one, as it will not "really" work for people who are trying the grid for the first time - and I remember that the example page is what sold me on it, so I would not take it lightly

tracked by AG-178
https://www.ag-grid.com/ag-grid-pipeline/

This should be fixed now


QueueBusy event dispatched when there's 0 active jobs, and 100 delayed jobs

Laravel Version
11.29
PHP Version
8.3.12
Database Driver & Version
mongodb (    "mongodb/laravel-mongodb": "^4.0",)
Description
After dispatching 100jobs, with delay 1000
\Illuminate\Queue\Events\QueueBusy::class is fired.
IMO it should consider that job has delay, and it's not busy at all at this moment. Thoughts?
Steps To Reproduce
for($i=0;$i<100;$i++){ EmptyJob::dispatch()->delay(1000); }

Hey! 👋
Yep, QueueBusy gets triggered even with delayed jobs, since Laravel doesn’t check if they’re actually ready to run. It just counts them all as “busy.”
Quick Workaround with Example
You could make a custom listener to skip jobs with a delay. Here’s a rough example:
php
Event::listen(\Illuminate\Queue\Events\QueueBusy::class, function ($event) {
$pendingJobs = Queue::getPendingJobs();
// Filter out delayed jobs
$nonDelayedJobs = $pendingJobs->filter(function ($job) {
    return $job->delay === 0; // Only count jobs with no delay
});

if ($nonDelayedJobs->count() > 0) {
    // Handle "busy" condition for non-delayed jobs
    \Log::info('Queue is actually busy with non-delayed jobs.');
}

});
This way, QueueBusy only considers jobs without a delay as “busy.” Hope it helps! 😊

Hey there,
While this may be a legitimate issue, can you first try posting your problem or question on one of the support channels below? If this issue can be definitively identified as a bug, feel free to open up a new issue with a link to the original one and we'll gladly help you out.

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks!

@ismaildasci thank you, I've tried but no such method "getPendingJobs" for me.
This is what I've ended up using. (use at own risk)
    public function busyCount(QueueBusy $event): int
    {
        if($event->queue != 'default'){
            Log::error('Unknown queue in QueueBusyListener: ' . $event->queue);
        }

        if($event->connection != 'redis'){
            Log::error('Unknown connection in QueueBusyListener: ' . $event->connection);
        }

        $default = $event->queue;
        $totalCount = Queue::size($default);
        $delayedCount = Queue::getRedis()
            ->connection(null)
            ->zcard('queues:'.$default.':delayed');

        return $totalCount - $delayedCount;
    }
    ```


ModuleNotFoundError: No module named 'nacl._sodium' (caused by paramiko/pysftp)

I am using Python 3.7.6 on Windows 10 64-bit (1909).  I get the problem with Nuitka 0.6.8 (not tried earlier versions) :
>python -m nuitka --version
0.6.8
Python: 3.7.6 (tags/v3.7.6:43364a7ae0, Dec 19 2019, 00:42:30) [MSC v.1916 64 bit (AMD64)]
Executable: C:\Program Files\Python37\python.exe
OS: Windows
Arch: x86_64

Nuitka was installed with PIP.  No VEnv used.  Python was installed with "python-3.7.6-amd64.exe".
The paramiko module was installed by PIP as a dependency of pysftp.  I suspect the PyNaCl module was installed as a dependency of paramiko.
This is the code (nuitkaTest.py) to reproduce the problem:
import paramiko
#import nacl
#import nacl._sodium
print("OK")

Although it also happens with programs that import "pysftp", since that depends on paramiko.
This is how I compiled the code:
python -m nuitka --recurse-all --remove-output --show-progress --show-modules --show-scons nuitkaTest.py

And when I run the resulting executable, this is what it outputs:
>nuitkaTest.exe
Traceback (most recent call last):
  File "......\nuitkaTest.py", line 1, in <module>
    import paramiko
  File "C:\Program Files\Python37\lib\site-packages\paramiko\__init__.py", line 22, in <module paramiko>
    from paramiko.transport import SecurityOptions, Transport
  File "C:\Program Files\Python37\lib\site-packages\paramiko\transport.py", line 90, in <module paramiko.transport>
    from paramiko.ed25519key import Ed25519Key
  File "C:\Program Files\Python37\lib\site-packages\paramiko\ed25519key.py", line 22, in <module paramiko.ed25519key>
    import nacl.signing
  File "C:\Program Files\Python37\lib\site-packages\nacl\signing.py", line 17, in <module nacl.signing>
    import nacl.bindings
  File "C:\Program Files\Python37\lib\site-packages\nacl\bindings\__init__.py", line 17, in <module nacl.bindings>
    from nacl.bindings.crypto_aead import (
  File "C:\Program Files\Python37\lib\site-packages\nacl\bindings\crypto_aead.py", line 18, in <module nacl.bindings.crypto_aead>
    from nacl._sodium import ffi, lib
ModuleNotFoundError: No module named 'nacl._sodium'

I tried tricking Nuitka to import the module (see the commented-out import lines), without success.  I also tried using command line parameters (--include-package & --recurse-to) without success.
And I was also able to reproduce the same problem using this code, which has a much smaller dependency list:
import nacl.signing
#import nacl
#import nacl._sodium
print("OK")

But for some reason the problem does not happen if you ONLY import "nacl".
For what it's worth, I believe the paramiko & nacl modules are correctly installed:
>python
Python 3.7.6 (tags/v3.7.6:43364a7ae0, Dec 19 2019, 00:42:30) [MSC v.1916 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import paramiko
>>> import nacl
>>> import nacl.signing
>>>

And my programs work fine with the Python interpreter.
I have the latest version of paramiko installed (2.7.1 according to pip list).
I also have the latest version of PyNaCl installed (1.3.0 according to pip list).
Any work-around to this apparent bug would be great...

@cshandley-uk, i used Nuitka 0.6.7
Try compiling the code as below:
python -m nuitka --follow-imports --standalone nuitkaTest.py

Try compiling the code as below:
python -m nuitka --follow-imports --standalone nuitkaTest.py

Unfortunately that made no difference for me.
Did you get my problem when compiling use the same options as me?

Try compiling the code as below:
python -m nuitka --follow-imports --standalone nuitkaTest.py

Unfortunately that made no difference for me.
Did you get my problem when compiling use the same options as me?

@cshandley-uk, yes i got the same error when using the same options as you.
After adding --standalone, i was able to compile it and it gave no error
NOTE: what was different in my case - ubuntu 18.04, Nuitka 0.6.7

Thanks, I've done more experimenting, and using --standalone does allow it to work... At the expense of needing a ton of ".pyd" files in a "nuitkaTest.dist" folder (which is useless for me as I need a self-contained executable).  I hadn't realised it was putting the executable inside that folder, and so I had been inadvertently testing an older executable.
So it seems the problem is a Nuitka limitation - it's unable to create an executable that doesn't need some or all of the following files present:
nacl/_sodium.pyd
pyexpat.pyd
select.pyd
unicodedata.pyd
_asyncio.pyd
_bz2.pyd
_cffi_backend.pyd
_ctypes.pyd
_decimal.pyd
_elementtree.pyd
_hashlib.pyd
_lzma.pyd
_msi.pyd
_multiprocessing.pyd
_overlapped.pyd
_queue.pyd
_socket.pyd
_sqlite3.pyd
_ssl.pyd
_tkinter.pyd
This is a great shame, and I hope the issue can be rectified eventually.

I should add that "pyd" files are apparently Windows DLLs.  It looks like these might be part of the relevant Python packages, rather than generated by Nuitka, in which case it's difficult for Nuitka to make them part of the executable...
... It looks like there may be a sneaky work-around to this, at least on Windows, but I don't know if such a solution is on Nuitka's roadmap:
https://www.drdobbs.com/packing-dlls-in-your-exe/184416443

The trap of accelerated binary and standalone binary being in a different directory is very real. Not sure though, if Nuitka should feel entitled to remove both. But an error would probably do good.
For #575 there is #230 but no progress. So being on a roadmap is not quite right. But maybe some day, there is not just simply somebody who wants it, but also somebody who creates it. For all I can tell, that person is not me until a company pays me good daily rate to do it.


fix: import statement init.lua

There was a bug in require:
nvim-spider/lua/spider/init.lua:2: module 'lua.spider.pattern-variants' not found:

I just noticed the issue myself and have already fixed it a second ago 🙈
Thanks though!


fix: Improve submit button styling

Changed height to a fixed value for consistency.
Ensured proper text alignment using flexbox for vertical and horizontal centering.
Removed duplicate width declaration.
Added a hover effect to enhance user interaction.
Overall improvements to the usability of the signup page.

What does this PR do?
This pull request (PR) improves the styling of the submit button on the signup page. It focuses on fixing text alignment, ensuring the button is visually appealing and user-friendly. Key changes include setting a fixed height for the button, applying flexbox for proper text centering, removing redundant width declarations, and adding a hover effect for better interactivity.
Fixes #(issue)
Before

After

Type of change

Bug fix (non-breaking change which fixes an issue)
This is because the changes you made correct the alignment and styling issues with the submit button without affecting any existing functionality negatively.

How should this be tested?

[x] Test A
[ ] Test B

Mandatory Tasks

[x] Make sure you have self-reviewed the code. A decent-sized PR without self-review might be rejected.

please add the hacktoberfest-added label also

hi @YadavAkhileshh , can you add hactoberfest-accepted label


Improvement download operation for priority and some protect

New Pull Request Checklist

[x] I have read and understood the CONTRIBUTING guide

[x] I have read the Documentation

[x] I have searched for a similar pull request in the project and found none

[x] I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)

[x] I have added the required tests to prove the fix/feature I am adding

[x] I have updated the documentation (if necessary)

[x] I have run the tests and they pass

[x] I have run the lint and it passes (pod lib lint)

This merge request fixes / reffers to the following issues: ...
Pull Request Description
This just a little improvement to the SDWebImageDownloadOperation. Including these:

Add a cache check for 304 response when using NSURLCache
Remove the extra cost to nil the imageData
Fix download operation may not marked as finished when data task create failed
Adopt the priority options to change URLSessionTask's priority

Codecov Report

Merging #2208 into master will decrease coverage by 0.1%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master    #2208      +/-   ##
==========================================
- Coverage   78.35%   78.24%   -0.11%
==========================================
  Files          36       36
  Lines        3654     3659       +5
  Branches      336      338       +2
==========================================
  Hits         2863     2863
- Misses        769      773       +4
- Partials       22       23       +1

Impacted Files
Coverage Δ

SDWebImage/SDWebImageDownloaderOperation.m
90.84% <75%> (-1.57%)
:arrow_down:

Continue to review full report at Codecov.

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

Looks good


language model estimation

language_model needs the ability to estimate from a corpus instead of requiring a .arpa file

We should consider this formulation (scroll down for the actual paper): http://homepages.inf.ed.ac.uk/s0562315/progs/#pldlm
Kenneth even mentions it in his thesis as future work. It looks like it isn't much harder than modified interpolated knesser-ney while giving better perplexity.

Once we implement this, we should have some way of saving the tokenization setup to ensure queries using the LM are tokenized the same way.


Added OTA, Languages, and international charracters

Added support for OTA ESP8266 web based firmware updater, see settings
Added language files, EN and SK are available now
Changed default font to OpenSans
Added function to transliterate special characters as extended latin or cyrilica

Lukas, I'm sorry this has been sitting idle for so long but we can't merge it as-is. Please prepare individual and isolated change sets for individual features. We may want to accept some but maybe not all. Smaller PRs are much easier to reason about and they stand a much better change for timely review and merge.


CNN encoder

Codecov Report

Merging #336 into master will decrease coverage by 1.53%.
The diff coverage is 89.06%.

@@            Coverage Diff             @@
##           master     #336      +/-   ##
==========================================
- Coverage    70.7%   69.16%   -1.54%
==========================================
  Files          68       70       +2
  Lines        5676     5637      -39
==========================================
- Hits         4013     3899     -114
- Misses       1663     1738      +75

Impacted Files
Coverage Δ

onmt/data/Batch.lua
70.45% <100%> (ø)
:arrow_up:

onmt/modules/init.lua
100% <100%> (ø)
:arrow_up:

onmt/modules/PositionEmbedding.lua
100% <100%> (ø)

onmt/Factory.lua
53.41% <41.66%> (-6.46%)
:arrow_down:

onmt/modules/CNNEncoder.lua
91.56% <91.56%> (ø)

onmt/Constants.lua
66.66% <0%> (-33.34%)
:arrow_down:

onmt/Seq2Seq.lua
49.63% <0%> (-11.86%)
:arrow_down:

onmt/utils/CrayonLogger.lua
44.44% <0%> (-10.11%)
:arrow_down:

onmt/SeqTagger.lua
38.88% <0%> (-10.07%)
:arrow_down:

onmt/train/Optim.lua
47.01% <0%> (-8.54%)
:arrow_down:

... and 29 more

Continue to review full report at Codecov.

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

Hi @natsegal , cc @guillaumekln
I get this error:
[07/07/17 17:27:57 INFO]    - structure: cnn_kernel = 3; cnn_layers = 2; cnn_size = 500;
[07/07/17 17:27:57 INFO]  * Decoder:
[07/07/17 17:27:59 INFO]    - word embeddings size: 500
[07/07/17 17:27:59 INFO]    - attention: global (general)
[07/07/17 17:27:59 INFO]    - structure: cell = LSTM; layers = 2; rnn_size = 500; dropout = 0.2 (naive)
[07/07/17 17:27:59 INFO]  * Bridge: copy
/home/moses/torch/install/bin/luajit: ./onmt/modules/Bridge.lua:26: with the copy bridge type, encoder and decoder must have the same number of layers and hidden size
misusing it ?

Hi @vince62s , you cannot use CNN encoder with the default copy bridge, you need to use either  "none" or "dense". Also, you should probably use it either with adam or with a much smaller learning rate than the usual rnn encoder, I had trouble making it converge otherwise.

great thanks. Did Adam work well for you in some other configs ? I had very hard time to use it with rnn ....
I'll keep you posted.

BTW, paper seems to suggest that POS is mandatory to get competitive results. Do you confirm ?

With CNN encoder adam actually worked better for me than simple SGD. You may also add -dropout_input option for better convergence.
There are still some issues left that need to be resolved, for instance the results with padded batch are still a little worse than with -batch_size 1. I continue working on it.
And yes, positions do help, at least it my settings.

I think we can get competitive results, we just need to find the right configuration for optimization. So far CNN is still 2 BLEU points lower in my settings...

@natsegal : for your constraint on 'copy' bridge - you can use 'depends' cmdline - to validate that one does not select illegal option combination - see https://github.com/OpenNMT/OpenNMT/blob/master/onmt/data/SampledDataset.lua#L44-L51


Site down?

It looks like bip174.org is unreachable

Yes, I'm currently doing some maintenance. It should be up tonight (EU time) or tomorrow, sorry for the inconvenience

No worries, just sharing in case you didn't know. Cheers

Should be back up now, sorry once again!


Fix: Resubmitting a create conversion project form

Changes
We had a bug in which resubmitting a create conversion project form, after it had failed validation, raised a 500 error.
To reproduce:

Fill in either an involuntary or voluntary create project form, making sure the form will fail validation
Click submit
Correct any validation errors
Click submit

The app will then render a 500 error.
This is because the Conversion::Voluntary/Involuntary::ProjectsController create method is expecting params with a key of conversion_project. When the form is first submitted, the principal parameter key will be conversion_project because the form is initialised with a Conversion::Project object.
However, once the form fails validation and re-renders, the form will be initialised with a Conversion::Involuntary::CreateProjectForm or a Conversion::Voluntary::CreateProjectForm, and the parameter key on re-submission will be either :conversion_involuntary_create_project_form or :conversion_voluntary_create_project_form
To remedy this we have:

Moved the new and project_params methods into the individual controllers. This means the forms are initialised with either a Conversion::Voluntary::CreateProjectForm or
Conversion::Involuntary::CreateProjectForm depending on which project is being created, and project_params can then expect the correct key

Explicitly added the text for the form labels. Previously these were being added via Rails/I18n magic and assumed the key would contain conversion_project because that was the key to the form. As the form is now named either conversion_involuntary_create_project_form or
conversion_voluntary_create_project_form we needed to either change the keys (and have two sets of identical keys for each form type) or explicity add the labels. We have gone for the latter option to  reduce duplication.

Checklist

[x] Attach this pull request to the appropriate card in Trello.
[x] Update the CHANGELOG.md if needed.

 Bug: Resubmitting an invalid conversion project form results in an error


I have launched one AKS but i need to launch it with RHEL 8 as Host.

I have launched one AKS but i need to launch it with RHEL 8 as Host. is that possible ?

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

ID: d501ff67-9208-8bcb-e8df-f09e9b16a802
Version Independent ID: e3fc3851-ddc4-d26c-251d-6ae53d714081
Content: Frequently asked questions for Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/faq.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

@JAI2436 Thank you for the feedback. We are actively investigating and will get back to you soon.

@JAI2436 No, choosing the Host OS to RHEL is not possible as AKS is a fully managed service.

Hope the provided information is helpful.
We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.


GraphQL 0.11 : Can only create NonNull of a Nullable GraphQLType but got: MongoID

One of the drawbacks of pre-v1 software is instability and GraphQL does not escapes that rule :smile:
Starting from GraphQL v.0.11.0 and later, the app crashes when building the schema. Relavant stack trace:
/home/morgan/.../node_modules/graphql/jsutils/invariant.js:19
    throw new Error(message);
          ^
Error: Can only create NonNull of a Nullable GraphQLType but got: MongoID.
    at invariant (/home/morgan/.../node_modules/graphql/jsutils/invariant.js:19:11)
    at new GraphQLNonNull (/home/morgan/.../node_modules/graphql/type/definition.js:780:84)

I assume that a breaking change in GQL breaks graphql-compose-mongoose.
(Workaround: downgrade to<EMAIL_ADDRESS>

In my app all works perfectly with<EMAIL_ADDRESS>It looks like that you have different graphql versions in your node_modules. Try to update all packages that use graphql (eg graphql-express, react-relay) and reinstall all packages from scratch. If it does not help, try to remove graphql folders in graphql-compose/node_modules/graphql, graphql-express/node_modules/graphql, maybe_any_other_pkg/node_modules/graphql. All packages that use graphql should use the same graphql package, because they all use checks via instanceof (that's why graphql-compose has graphql as peer-dependency for reducing package versions hell between different libs).

Well, you were right, I've upgraded graphql again, then deleted node_modules and reinstalled everything from the lockfile, and that works. Using yarn, I'm not used to that type of problems anymore...
Sorry sorry :)


Converts all of the views inside AdminSite to mixins

Something I've been wanting to do and @joncotton's recent changes had me working in that space anyhow.

FYI, I believe this is ready -- going to test after lunch, but assuming it looks good to everyone else I'll merge it and release the code this afternoon.


Annotate Action<T> as a target for lambdas with implicit receiver

So it becomes unnecessary (illegal, in fact) in Kotlin build scripts
within lambda expressions passed as Action<T> arguments.

See gradle/gradle-script-kotlin#155


Can't compile with musl-libc?

../../../aws-c-common/libaws-c-common.so.1.0.0: undefined reference to `backtrace_symbols'
../../../aws-c-common/libaws-c-common.so.1.0.0: undefined reference to `backtrace'

Why #if defined(AWS_HAVE_EXECINFO) not wokring?

Can you provide more details about your environment and how you are trying to build this repo when you run into the above error?


Custom rotations for mages no matter the talent tree

I like to be able to make custom rotations on everything, I can't really make an arcane mage that's using FFB as primary.
I would like to be able to create a semi custom rotation. For exemple the arcane rotation is cool but I would like to be able to select my primary (eg. FFB here with arcane missile on procs and PoM + Pyro on CD). Thanks
I would learn how to do it myself but I'm way too far and don't have much time atm :D
I know I'm a bit of a mad scientist when it comes to talent trees and I like to be able to sim everything. Thing is usually what I do is a bit weird so it's not something that's accounted for when these kind of tools are created which I'm not mad about but I really would like to see what It can do.
Many thanks

You're free to fork the repo and define whatever mage rotation you'd like here: https://github.com/wowsims/wotlk/blob/master/sim/mage/rotations.go

@PoicoKronk
As a Le Wagon student and a future alumni, you should maybe try to do it by yourself.
Also it could be a good idea to pitch this as your project idea ?
Maybe you could freshen up the website, making it a bit more user friendly. I'm pretty sure you could do a good job.

Marking as duplicate of #2173 because it's not worth taking the time to implement this individually.


[SPARK-45884][BUILD][3.4] Upgrade ORC to 1.8.6

What changes were proposed in this pull request?
This PR aims to upgrade ORC to 1.8.6 for Apache Spark 3.4.2.
Why are the changes needed?
To bring the latest maintenance releases as a part of Apache Spark 3.4.2 release

https://github.com/apache/orc/releases/tag/v1.8.6

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass the CIs.
Was this patch authored or co-authored using generative AI tooling?
No.

Thank you, @yaooqinn . Merged to branch-3.4


website shortcuts broken in Gnome on Fedora

OS/Platform
Fedora and CentOS
Installed
Flatpak
Version
Version 125.0.6422.112 (Official Build, ungoogled-chromium) (64-bit)
Have you tested that this is not an upstream issue or an issue with your configuration?

[ ] I have tried reproducing this issue in Chrome and it could not be reproduced there
[X] I have tried reproducing this issue in vanilla Chromium and it could not be reproduced there
[X] I have tried reproducing this issue in ungoogled-chromium with a new and empty profile using --user-data-dir command line argument and it could not be reproduced there

Description
website shortcuts don't appear in gnome applications
How to Reproduce?

visit a website
menu - Save and share - Create shortcut
save shortcut
open Gnome overview
type shortcut name

Actual behaviour
shortcut not present in the menu
Expected behaviour
shortcut gets created
Relevant log output
LaunchProcess: failed to execvp:
update-desktop-database

(the same log output is present with a fresh user profile that is not affected by the issue)

Additional context

I can't reproduce the problem with a fresh profile.
All the shortcuts disappeared during flatpak update approximately month ago.
tested on Fedora 39 and 40

Something with the sandbox maybe? CC: @rany2

Maybe you need to manually edit the desktop files to point to the new app ID?

Something with the sandbox maybe? CC: @rany2

It works fine in the a fresh profile, so I think that it might be migration related.


Cut off negative values for padAlongDim in computing convInfo.padInfo

This fixes https://github.com/tensorflow/tfjs/issues/1270
Reference: https://www.tensorflow.org/api_guides/python/nn#Convolution

This change is

Thank you!!!


fix(authentication): userInfoEndpoint should be optional for token auth providers

Currently token auth providers without a userInfoEndpoint will always result in an undefined user in the wundergraph context object.

Checklist

[x] run make test
[x] PR title must follow conventional-commit-standard
[ ] tests and/or benchmarks are included
[ ] documentation is changed or added
[x] commit message and code follows the Code of conduct

@StarpTech please verify and merge. Looks good to me.


Fix index comparison for certificate chain export

This fixes the certificate export logic to allow all chains to be included in the export to pkcs12 pfx file.

This PR would resolve the issue #141. Great work.

hello! any ETA on the PR? Thanks!


[EPIC] semantic-ui integration with invenio v3.4.0a1

Documentation:

RFCs (what is this madness?)

Templating/assets building: Multi theme support
React-Searchkit integration: Overridable customization

Getting started

Module migration guide
Developer quickstart

Steps:

[x] 1. Update cookiecutter-invenio-rdm to support Python 3.7 and 3.8. https://github.com/inveniosoftware/cookiecutter-invenio-rdm/issues/71

[x] Quick win: fix https://github.com/inveniosoftware/cookiecutter-invenio-rdm/issues/64 as part of the work.

[x] 2. Integrate Invenio v3.4.0a1. Set APP_THEME=['bootstrap3']. InvenioRDM should stay as is now, no UI should be broken. https://github.com/inveniosoftware/cookiecutter-invenio-rdm/issues/72

[x] Update dependencies in `invenio-rdm-records: https://github.com/inveniosoftware/invenio-rdm-records/issues/102
[x] Update dependencies in invenio-records-permissions: https://github.com/inveniosoftware/invenio-records-permissions/issues/47
[x] Using an alpha release will incur in using --pre in invenio-cli. Crosscheck how this affect 3rd party dependencies. Is there any dependency breaking code? --> NO

[x] 3. Release modules:

[x] Release rdm-records: https://github.com/inveniosoftware/invenio-rdm-records/issues/108
[x] Release records-permissions: https://github.com/inveniosoftware/invenio-records-permissions/issues/49

[x] 4. Migrate to Semantic-UI. Set APP_THEME=['semantic-ui'].

[x] Migrate templates in invenio-app-rdm. https://github.com/inveniosoftware/invenio-app-rdm/issues/125
[x] Migrate templates in invenio-rdm-records. https://github.com/inveniosoftware/invenio-rdm-records/issues/100
[x] Update cookiecutter-invenio-rdm to build the new UI bundles. See https://github.com/inveniosoftware/cookiecutter-invenio-instance/pull/227 for reference.

[x] 5. Release modules:

[x] Release rdm-records: https://github.com/inveniosoftware/invenio-rdm-records/issues/108
[x] Release app-rdm

All steps have been implemented. Closing issue.
Next step theming https://github.com/inveniosoftware/cookiecutter-invenio-rdm/issues/75


Switch DSSE provider to go-securesystemslib

Signed-off-by: Aditya Sirish<EMAIL_ADDRESS>
Summary

Switches from using github.com/in-toto/in-toto-golang/pkg/ssl to github.com/secure-systems-lab/go-securesystemslib/dsse.
Ticket Link

N/A.
See: https://github.com/in-toto/in-toto-golang/pull/122
Release Note

Uses new provider for DSSE.

I should note that we're planning to deprecate in_toto.NewSSLSigner in favour of in_toto.NewDSSESigner. Happy to submit a patch when that's live. :smile:

I should note that we're planning to deprecate in_toto.NewSSLSigner in favour of in_toto.NewDSSESigner. Happy to submit a patch when that's live. smile

yes please do


Reproduce Issue

I'm trying to reproduce this repo, however I'm facing issue while tried to execute the first query in "Tour" session. Also get "0" while executing "match (n) return count (n)" in your YouTube tutorial. May i ask what package or database or anything am I missing?

It looks like you've copy/pasted python code into the Neo4j Browser.  The Neo4j broswer only processes cypher queries -- these are the steps with words like "MATCH" in it.  Any python code you see you should run in a python environment.  In the video tutorial I use a jupyter notebook.

Thanks for the tips. May i also ask how to import "testtransactions.yaml" into neo4j database? I've tried to run "knowledge_graph.ipynb" but it ends up with error "ImportError: cannot import name 'notebook2script' from 'nbdev.export' (/home/student_helper/anaconda3/lib/python3.11/site-packages/nbdev/export.py)". I've tried to install nbdev, but it didn't work. Sorry for dump question. Please feel free to contact me through this email<EMAIL_ADDRESS>just to not polluting this repo's issue page.


[Suggestion] Add thumbnails

Would it be possible to include pictures/thumbnails for shows in the Mediathek? As I understand, you're scraping the websites for info, so I would naïvely assume thumbnails are possible too ;)
I'm suggesting this because the Mediathek listing feels a bit cluttered without any visual cues. I'm imagining having thumbnails left of each episode listing, with the current info (title, duration, channel, date of publication) to the right of the image.
If thumbnails aren't possible at all, maybe even just adding icons (in color in light mode, black/white in dark mode) of the corresponding channel in place of the text (e.g. instead of the letters ZDF underneath the title, the logo is used) would add some easy visual orientation.
If thumbnails are mostly possible, with some failures due to the nature of scraping, we could fallback to filling the thumbnail square with a placeholder, possibly the channel logos as described above.
Combining this with #356 should make for an even more awesome user experience (I'm assuming that the most intuitive option, series -> episode, isn't implemented because it's not possible to group episodes reliably using scraping).
Thank you for contributing free software!

As I understand, you're scraping the websites for info, so I would naïvely assume thumbnails are possible too ;)

AFAIK the infos are read from MediathekViewWEB (MVW) ..
https://github.com/mediathekview/zapp/blob/36919ba4aa8024213214f23a34cdf4bf0562a9d7/app/src/main/res/values/config.xml#L5
curl 'https://mediathekviewweb.de/api/query' -H 'content-type: text/plain' --data '{"queries":[{"fields":["topic"],"query":"#Livestream"},{"fields":["title"],"query":"+Livestream"}],"sortOrder":"desc","future":true,"offset":0,"size":2}' | json_pp
...

Sadly MVW does not scrap thumbnails for the items:
{
   "err" : null,
   "result" : {
      "queryInfo" : {
         "filmlisteTimestamp" : "1692700200",
         "resultCount" : 2,
         "searchEngineTime" : "2.05",
         "totalResults" : 45
      },
      "results" : [
         {
            "channel" : "DW",
            "description" : "",
            "duration" : "",
            "filmlisteTimestamp" : "1671312060",
            "id" : "8+BoYmnJMAnL6xM+o6ksM5QlbVe+p9K5KNoqI2mq6q8=",
            "size" : null,
            "timestamp" : 0,
            "title" : "DW Livestream",
            "topic" : "Livestream",
            "url_subtitle" : "",
            "url_video" : "https://dwamdstream111.akamaized.net/hls/live/2017972/dwstream111/index.m3u8",
            "url_video_hd" : "",
            "url_video_low" : "",
            "url_website" : "https://www.dw.com"
         },
         {
            "channel" : "HR",
            "description" : "",
            "duration" : "",
            "filmlisteTimestamp" : "1671312060",
            "id" : "z1ygSQCgSHQsw+nNXmfs+hzy3/6qT9/KMKjofz8kul0=",
            "size" : null,
            "timestamp" : 0,
            "title" : "HR Livestream",
            "topic" : "Livestream",
            "url_subtitle" : "",
            "url_video" : "https://hrhls.akamaized.net/hls/live/2024525/hrhls/master.m3u8",
            "url_video_hd" : "",
            "url_video_low" : "",
            "url_website" : "https://www.ardmediathek.de/live/Y3JpZDovL2hyLmRlL0xpdmVzdHJlYW0tSFI/"
         }
      ]
   }
}

Aww, that's a shame. I just checked and it looks like the devs weren't interested in implementing it until now. Have asked for an update.

Thank you very much for the suggestion!
Thumbnails would be very nice, but currently are not provided by the mediathekviewweb api. Also, displaying them in every client would cause quite a performance impact on the mediathek servers - so this is a feature which most likely will not be implemented.

If thumbnails aren't possible at all, maybe even just adding icons (in color in light mode, black/white in dark mode) of the corresponding channel in place of the text (e.g. instead of the letters ZDF underneath the title, the logo is used) would add some easy visual orientation.

I already tried this ;) It adds a aweful lot of visual clutter with minimal benefit for the user.


No Highlighting on VS2015 (v.<IP_ADDRESS>)

For some reason my highlighting is gone... I have fiddled around with opening / closing VS with different settings.  I cannot find a file extension that asm is associated with.. so I don't think that is the issue.
I see there was an update a couple days ago..  Is it related?  Next step is to uninstall / re-install the plugin.

Uninstalled / Reinstalled extension... No effect...
I wish I could chase down this bug but I just don't know enough on how these extensions are coded or how they are loaded into VS.

have you checked https://github.com/HJLebbink/asm-dude/issues/43

Yes... When I download the source code asm-dude-vsix is not compatible with my current version of VS... Could that be an issue?

It is not a discernable file association issue.

is vsix the extension platform?  like the VS Extension Development items?

The source code is c#7, the project is a vs2017 project, but it project creates an installer for vs2015 and vs2017. You are probably using vs2015 and the plugin should work perfectly. Yes vsix is the plugin.
I have no idea why the .asm file is not associated with the AsmDude extension, which is what probably has happend. Have you upgraded via the Visual Studio or have you downloaded the vsix from github?

I just cloned the repository and tried to open it... I'm upgrading my VS to 2017 now..

dang... went to a new computer and tried to use it without updating... but some how the update automatically got rechecked and when I rebooted my computer it updated the asm-dude extension and lost the syntax highligting..

New Version works in VS Community 2017 though just fine

Thanks for the info. I'll check on VS2015 why the 'crosscompile' was messed up (again).

I have the same problem, when I switched from asm to windows C++ then back to asm, the highlight is gone. (VS2015)

@Tiiberiu Could you try to revert to <IP_ADDRESS>, (uninstall v<IP_ADDRESS> and download <IP_ADDRESS> from github) to see whether that solves the issue. In the meantime I will waste some time figuring out why VS2015 complains...

That solved the issue, thanks.

I've created a new release v<IP_ADDRESS> that should fix this issue. If someone still has problems, please let me know.


Stability of service: Not rendering anything anymore.

Hi there,
I've deployed it to a Google Cloud instance, but we noticed that after several days the pages aren't rendered any more.. It seem the same issue is currently occuring on this environment: https://render-tron.appspot.com/

Actually I noticed a possible memory leak issue in the docker version of rendertron 2 days. But I didn't have time to investigate it further.  Maybe it's the cause?

Hmm, I'm not seeing that issue on rendertron-appspot.com right now and I haven't touched anything.

@hekod777: not sure I understand around the clearTimeout(). It's already on https://github.com/GoogleChrome/rendertron/blob/master/src/renderer.js#L94 which is where the timeout will no longer trigger. In loadEventFired(), there are still are a number of checks before its returned.

@paales: I have an outdated version running since Aug 9, which is still running fine. The render-tron-appspot.com one is slightly outdated (I'll update it now), but its been running since Oct 5 and it appears to be rendering pages for me right now.

@samuelli It seems to be rendering something. It does appear to be dog-slow for me, is that because the service is being hammered or because of the demo setup?
Doesn't render:
https://render-tron.appspot.com/screenshot/https://reach-digital.appspot.com/
https://render-tron.appspot.com/screenshot/https://tweakers.net/ (did render just render, but can't get it to work anymore?)

So a couple of things:

I accidentally disabled the cache on render-tron. Fixing now.
I'm looking at reach-digital. I'm seeing lots of variation in times so I looked at the logs. It appears to be hitting the 10 second limit because of an outstanding network request. I'd recommend looking at #126, which is where we changed the explicit flag behavior. The explicit flag should help let rendertron know exactly when you're ready.

@samuelli Thanks, will take a look at #126, see if that solves the issue.

So I upgraded to the latest version version of this repo, but can't seem to get it to work:
Debug mode doesn't seem to be working (never got it working)? I have configured the config.json to enable the debug mode. I expect that when I run npm start to see all the console logs of a page it is trying to render. Am I doing somthing wrong?

I can't get the explicit render flag to work:

Sometimes the network is done earlier than this flag and sometimes later. How is it supposed to work, I expected it to be fully manual and I need to set the flag?
If I call the following code it works fine in my browser:
console.log('initial render complete: ', document.querySelector('title').innerText);
window.renderComplete = true

Debug allows console messages from the rendered page to be piped through. Someone else had also requested better debug messages. Filed #131 to track this.
It is fully manual and what you have looks correct. Can you possibly deploy that to some URL for me to test? Or send me any repro instructions?

I have one possible suspect. Can you retry without the debug flag?

@paales: nvm, looks like a bug on my side. I'll send out a fix hopefully soon.

#133 should hopefully fix this.

@samuelli We're a step further!


Ignore backend msgs in connection closed state

Current coverage is 0.00% (diff: 0.00%)

Merging #75 into master will not change coverage

@@           master    #75   diff @@
====================================
  Files         100    101     +1
  Lines        1286   1288     +2
  Methods      1216   1219     +3
  Messages        0      0
  Branches       70     69     -1
====================================
  Hits            0      0
- Misses       1286   1288     +2
  Partials        0      0

Powered by Codecov. Last update 9555ad7...fd58359


Mention slack channels to join in the contributor quickstart

As discussed with @iamtrask & team in #team_marketing slack channel
https://openmined.slack.com/archives/C6G0ZA0HJ/p1502823797000163
Open to suggestions! I was trying to focus on "topics" as opposed to what each
project does, because I think that is redundant with the rest of the
documentation already, and this was more about getting people to
follow and join what is discussed in each channel.

Hey @swaroopch - I've actually already done this in my PR: https://github.com/OpenMined/Docs/pull/35
Would we want to merge our efforts?

Yeah! Let's combine forces between both lists and pick the elements we like. Hopefully our new work on teams will help us be able to plan this collaboration pre-PR instead of post-PR. Sorry for the hassle on this one... but we're fixing it!

All good.  I'll put in a change on my PR to update the slack channels if we want to close this one?

@cereallarceny yep, sounds good to me!


Fix the API for reload segments

Fix the issue where return value is not a valid JSON String
Deprecate the GET method, add POST APIs
Update the integration test to send POST request for reloading

Codecov Report

Merging #2464 into master will decrease coverage by <.01%.
The diff coverage is 28.57%.

@@            Coverage Diff             @@
##           master    #2464      +/-   ##
==========================================
- Coverage   69.59%   69.59%   -0.01%
==========================================
  Files         840      840
  Lines       39216    39221       +5
  Branches     5278     5278
==========================================
  Hits        27294    27294
- Misses      10155    10161       +6
+ Partials     1767     1766       -1

Impacted Files
Coverage Δ

...ler/api/resources/PinotSegmentRestletResource.java
26.7% <28.57%> (-0.79%)
:arrow_down:

...ore/query/scheduler/resources/ResourceManager.java
81.81% <0%> (-13.64%)
:arrow_down:

...pinot/core/operator/docidsets/OrBlockDocIdSet.java
84.9% <0%> (-9.44%)
:arrow_down:

...e/impl/dictionary/LongOnHeapMutableDictionary.java
88.88% <0%> (-6.67%)
:arrow_down:

.../pinot/core/query/scheduler/PriorityScheduler.java
77.46% <0%> (-5.64%)
:arrow_down:

.../impl/dictionary/LongOffHeapMutableDictionary.java
81.81% <0%> (-5.46%)
:arrow_down:

...not/core/query/pruner/DataSchemaSegmentPruner.java
53.65% <0%> (-4.88%)
:arrow_down:

...impl/dictionary/FloatOffHeapMutableDictionary.java
87.27% <0%> (-3.64%)
:arrow_down:

...e/operator/dociditerators/BitmapDocIdIterator.java
60.71% <0%> (-3.58%)
:arrow_down:

...om/linkedin/pinot/transport/netty/NettyServer.java
81.05% <0%> (-2.11%)
:arrow_down:

... and 17 more

Continue to review full report at Codecov.

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


[BUG]: Delete method on KVStore is broken

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The delete implementation on the KVStore is:
pub fn delete(&mut self, k: &[u8]) -> Option<Vec<u8>> {
        let tx_value = self.tx_cache.remove(k);
        let block_value = self.block_cache.remove(k);
        let persisted_value = self.persistent_store.remove(k);

        tx_value.or(block_value).or(persisted_value)
    }

This deletes from the block cache and persisted store even if the Tx fails. The block cache and persisted store should only be written to when write_then_clear_tx_cache is called.
Gears version
main
Steps to reproduce?
No response

@joneskm Should we close this issue as outdated?


Getting error on some pages in ios - reported by @thesahindia

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

Action Performed:

Create NewDot account, verify
Create Workspace
Start add VBA flow
In Company information step, type something in the "Company Address" field

Expected Result:
No error should appear
Actual Result:
User sees the error
Workaround:
unknown
Platform:

Where is this issue occurring?

iOS

Version Number: 1.1.36-0
Reproducible in staging?:
Reproducible in production?:
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43995119/152916593-63c38259-f45a-4990-90f9-6c08c6da06dc.mov
Expensify/Expensify Issue URL:
Issue reported by: @thesahindia
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1642611684186800
View all open jobs on GitHub

Triggered auto assignment to @Christinadobrzyn (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

Duplicate of https://github.com/Expensify/App/issues/7547


add create(id) method to TrackerFactory class

this is inconsistent with other beanFactory classes in the library.

added