text:
Make landing section of the site mobile friendly

What was the feature/problem?
fixes #99 - Primary way the kids are going to interact with this site is via a tablet
How does this PR address the above feature/problem?

Additional fixes

Removed the sliding "hello, hola..." animation as it was distracting and helped with the text overlap issue on mobile screen.
Have made the login, registration, and wish list landing pages mobile-friendly.
Fixed few accessibility issues and HTML5 semantic issues (hover pointer on wish type and the section on landing header)
Removed the use of the CSS grid as it is not fully supported in all browsers.

Check lists

[x] Test passed
[x] Coding style (indentation, etc)

Codecov Report

Merging #105 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #105   +/-   ##
=======================================
  Coverage   95.39%   95.39%
=======================================
  Files          28       28
  Lines         391      391
  Branches       36       36
=======================================
  Hits          373      373
  Misses         15       15
  Partials        3        3

Flag
Coverage Δ

#api
97.12% <ø> (ø)
:arrow_up:

#ui
94% <ø> (ø)
:arrow_up:

Impacted Files
Coverage Δ

ui/src/header/index.js
100% <ø> (ø)
:arrow_up:

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 f1e1799...8d39bfc. Read the comment docs.

Wow thanks so much for the contribution! I will sit with our designer on this and walk them through the experience and either accept or leave feedback as soon as I can.

@crstamps2 Do I need to create a new issue to make all the text black, as well as update the logos to the black versions or is that part of #98 ??? Not sure if they are pointing to same gfx. Just LMK.

@hellonoosh I think we are going to keep the design updates in a separate issue/PR, since someone else is working on those already. This PR will just be for making the app mobile responsive.

@chekkan when I boot up the app and use chrome dev tools to force it to be mobile sized, this is what I get.

And this is what I get if it's an iPad Pro

I will look into those issues and get it fixed 😁 any other feedback? Are you happy with me getting rid of the hello, hola animation?

@crstamps2 those issues should be solved now. Please let me know if I missed something.

@chekkan Taking a look here shortly. Concerning the hello animation, while it is a little gimmicky and very 90s-esque animation style, some of the decision makers like it. I think it's ok to not have it in a mobile phone view, but on the standard desktop view, I believe we are going to keep it. Will get back to you on that.

@chekkan can you update with master and resolve the conflicts and then I will review and merge.

@crstamps2 any update on this?

source: gharchive/pull-request

authors: chekkan codecov-io crstamps2 hellonoosh

repo: homedepot/infinite-wish-board

license_type: permissive

license_source: github-api


text:
Update selected slice path when manipulating the diagram

Certain manipulations such as attachment and homotopies should change the path of the currently selected slice.

I remember finding this exceptionally tricky, and in fact I think it's still not right in the JS implementation. In particular imagine you're deep inside a slice of a slice of a slice etc of your diagram, and you perform a mouse action to insert a half-braid. What this really does is inserts a "half braid creation, half braid removal bubble" in the middle of your diagram. But since the user has performed the mouse action to create a half-braid, what she wants to see is the result of that, i.e. the half-braid, so you need to make sure to update the slices to navigate to that. For some reason, I found it difficult to compute in general what those slices are.
Extra complexities for this will be added when we have invertible generators, since that requires a new ability to insert "creation-annihilation bubbles" for every generator.

Calin is working on this now.

source: gharchive/issue

authors: jamievicary zrho

repo: homotopy-io/homotopy-rs


text:
Slight adjustment to control points for 2d renderer

I would like us to make some minor tweaks to the bezier control parameters, Calin maybe we can do this together some time in person.

Sure, what kind of tweaks did you have in mind?
The nice thing is whatever changes we make will also be reflected in the Tikz export.

Yes I agree that is a nice feature. I haven't collected my thoughts on this
yet, I will do that soon and we can discuss in person.
On Thu, May 5, 2022 at 10:36 AM Calin Tataru @.***>
wrote:

Sure, what kind of tweaks did you have in mind?
The nice thing is whatever changes we make will also be reflected in the
Tikz export.
—
Reply to this email directly, view it on GitHub
https://github.com/homotopy-io/homotopy-rs/issues/473#issuecomment-1118358816,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACQ4OHVSZBGISJEKTUWBUVLVIOJC3ANCNFSM5VEPAOEQ
.
You are receiving this because you were assigned.Message ID:
@.***>

Hi Chiara thanks for this. I loaded the example you sent in the version hosted by the PR, I agree it looks nice, but I'm not sure what it looked like before. Also, Calin and I already worked on this a couple of weeks ago and I thought we fixed it, if there are further issues could you attach screenshots please?

The main different between this PR and master is that full braids now have a nearly horizontal tangent to distinguish them from algebraic points.

The current version:

The new version:

I can make it less sloped, but that is precisely the change.

How can it be more horizontal than fully horizontal?? Sorry I am totally confused...

How can it be more horizontal than fully horizontal?? Sorry I am totally confused...

Haha sorry I mean the slope is closer to being a horizontal line than a vertical one (i.e. the angle is less than 45 degrees).

Base example:

90% horizontal:

80% horizontal:

Old one was 60% horizontal. Opinions? I'm veering for a 85% horizontal solution.

I think I might prefer 95% ... so maybe we can compromise and go for 90%?

I am happy with 90%.

90 looks really cool!

90% it is.

Thank you all glad to see such strong engagement with this critical issue :)

Chiara can you make sure it works correctly on this example:

homotopy_io_export.hom.zip
The two wires should have the same tangent, otherwise they will not join up correctly.

Oh sorry about that, should be a quick fix.

So, turns out it was not a quick fix, and the idea of using sloped tangents is fundamentally flawed.
The reason is that we may make braids arbitrarily wide, and to keep the tangent angle fixed, it will eventually cause one of the control points to be above the endpoint. This is the visual result:

We discussed this with @calintat, and the solution we will adopt is to using horizontal tangents, but make the horizontal control point closer than the vertical one:

The above approach does not produce any meaningful difference in the output, compared to the current beta. I'm not changing the current version and closing the issue.

authors: calintat doctorn jamievicary regular-citizen


text:
fix: Handle the content not found gracefully

Pull Request Submission Checklist
Please confirm that you have done the following before requesting reviews:

[x] I have confirmed that the PR type is appropriate for the change I am making according to the Honest Pull Request and Commit Message Naming Conventions.
[x] I have typed an adequate description that explains why I am making this change.
[x] I have installed and run standard pre-commit hooks that lints and validates my code.

Description
The lib panics when it couldn't find the content under the queried path, since it always assumes that the returned object is non-null.
This PR changes the lib to return a nil for caller to handle gracefully in such cases.

This change is

Found another issue. Fixing...

repo: honestbank/event-driver

license: MIT


text:
How can i get the demo input about human（smpl）?

Since smpl is a direct estimation of 3D coordinates, how do I get 2D joint coordinates from the image

Thanks for your reply, but can OpenPose get a 2D detection of the 24 joints?

I misunderstood your question.
I think there’s no off-the-shelf 2d smpl pose detectors. You may have to train on your own.

authors: hongsukchoi naivate


text:
[Feature Request] - Return whoosh result object along with each result

It would be useful to have an option to return the whoosh result object along with each result.
One use case is so we can then at a later stage highlight the matched terms in each result.
Whoosh Documentation - “Did you mean... ?” Correcting errors in user queries

For convenience, flask-msearch always return a Model , if you want to use origin whoosh syntax, you can use
search = Search()

....
search.msearch(Post, "apple", fields=None, limit=None, or_=True)

Apologies, I'm interested in the whoosh.searching.Results and whoosh.searching.Hit objects so I can view which search terms matched and highlight the matching terms in the results.
References:
Which terms from my query matched?
How to create highlighted search result excerpts

authors: honmaple veeshi

repo: honmaple/flask-msearch

license_source: bigquery


text:
Cannot set property duplex of #<_Request> which has only a getter

What version of Hono are you using?
4.0.4
What runtime/platform is your app running on?
node
What steps can reproduce the bug?
app.mount('/',(req)=>new Response(new ReadableStream()))

curl http://0.0.0.0:8787/xyz -X OPTIONS

What is the expected behavior?
Works as expected
What do you see instead?
TypeError: Cannot set property duplex of #<_Request> which has only a getter
    at new Request (file:///app/node_modules/.pnpm/@hono+node-server@1.8.1/node_modules/@hono/node-server/dist/index.mjs:15:22)
    at handler (file:///app/node_modules/.pnpm/hono@4.0.4/node_modules/hono/dist/hono-base.js:169:9)
    at dispatch (file:///app/node_modules/.pnpm/hono@4.0.4/node_modules/hono/dist/compose.js:29:23)
    at file:///app/node_modules/.pnpm/hono@4.0.4/node_modules/hono/dist/compose.js:30:20
    at logger2 (file:///app/node_modules/.pnpm/hono@4.0.4/node_modules/hono/dist/middleware/logger/index.js:35:11)
    at dispatch (file:///app/node_modules/.pnpm/hono@4.0.4/node_modules/hono/dist/compose.js:29:23)
    at file:///app/node_modules/.pnpm/hono@4.0.4/node_modules/hono/dist/compose.js:6:12
    at file:///app/node_modules/.pnpm/hono@4.0.4/node_modules/hono/dist/hono-base.js:226:31
    at Hono.dispatch (file:///app/node_modules/.pnpm/hono@4.0.4/node_modules/hono/dist/hono-base.js:236:7)
    at fetch (file:///app/node_modules/.pnpm/hono@4.0.4/node_modules/hono/dist/hono-base.js:60:19)

Additional information
Seems triggered by
https://github.com/honojs/node-server/blob/826b77bbf0d10c103fd03e66956bbfd1f01c41ea/src/request.ts#L14-L18
But this is not allowed see https://github.com/sindresorhus/ky/issues/467

xref https://github.com/honojs/node-server/pull/149

repo: honojs/hono


text:
Can't get working 404 handler with honojs and CloudFlare Pages

Trying simple CloudFlare Pages example
import { Hono } from 'hono';
const app = new Hono();

app.get('*', async ctx => await ctx.env.ASSETS.fetch(ctx.req));

app.notFound(ctx => {
    return ctx.text('Custom 404 Message', 404);
})

export default app;

but don't have 404 message on missing assets

Hi @caracal7 !
In Cloudflare Pages, we can't handle 404 not found in the route returns c.env.ASSETS.fetch() . But you can put the 404.html in the directory, and it will be shown.

Working!
@yusukebe thanks a lot :)

Working!
@yusukebe thanks a lot :)

Great! I'll close this issue.


text:
Issue/92

Fixed issue 92:
https://github.com/hoodiehq/hoodie-server-account/issues/92

@alinebastos I've rebased your commits for you over at https://github.com/hoodiehq/hoodie-server-account/pull/110 :D

authors: alinebastos nickcolley

repo: hoodiehq/hoodie-server-account


text:
extend chairman so it can looks at the problem and decides who should attend.

will then need to create the list of participants and create prompts for each of the people that it wants to attend

Should chairman create custom executives with expertise and concerns or pick from predefined list?

chaircman defines them.
so he first looks at the refined problem
then choose a group of people for the meeting
then iterates through and write a prompt suitable to tell gpt to act like that persona

repo: hookla/DreamTeamGPT


text:
🛑 FWAw is down

In b4120aa, FWAw ($FWA_SITE) was down:

HTTP code: 0
Response time: 0 ms

Resolved: FWAw is back up in 94043be after 12 minutes.

repo: hoopybe/uptime


text:
Multiple Columns - Tenancy

Is it possible to allow tenancy that is based on more than 1 column ? This will significantly enhance capability

Could you offer more details about the multiple columns which can help me to dig deeper?

Let's say we want tenancy established based on two attributes tenantID and clientID (assume scenario where each tenant may have multiple clients within). Ability to optionally allow scope to include more than one column can significantly enhance security

Let's say we want tenancy established based on two attributes tenantID and clientID (assume scenario where each tenant may have multiple clients within). Ability to optionally allow scope to include more than one column can significantly enhance security

This looks like a nest tenant(Tenant and Sub-tenant). May I ask which should the current_tenant represent to? The Tenant or the Client?
Sorry, I'm not quit sure how to start, maybe you could give  more suggestions? @bazooka720

What if it's dependent on the user role? If the user is "client" type then current_tenant refers to company(subdomain)+client, else its just company

@hoppergee thoughts?

authors: bazooka720 hoppergee vcknorket

repo: hoppergee/multi-tenant-support


text:
fix race condition in TestLocalResourcesTrackerImpl.testLocalResourceCache

add sleep time after call to handle to let the time to all the events triggered  by the handle call to be processed before the test run any verification.
Resolve #11

The test is still failing, I have to make more corrections:

repo: hopshadoop/hops


text:
CB-21044 Patch FreeIPA constants lib to support hostnames longer than…

… 64 chars
RedHat claims that hostname must be the same as FQDN thus have the limit 64 char for the hostname/FQDN. As we don't set the hostname to FQDN, it's possible the FQDN is longer than 64 char, but this limit would cause failures on client side when ipa-client-install is run. This change is introduced in RH8, Centos7 works fine. Commit causing the validation error: https://github.com/freeipa/freeipa/commit/6662e99e173a16059271528e1fa58e37a3028924
The limit on FreeIPA server side is also increased: https://github.com/hortonworks/cloudbreak/commit/18a867c6b47fc6efb39dd50c3f5557f3940f25bb

@akanto I've replaced file.replace with file.patch which would fail if the patch could not be done for any reason. Tested with moving the line and deleting it

repo: hortonworks/cloudbreak-images


text:
2.54 to master

See detailed description in the commit message.

https://github.com/hortonworks/cloudbreak/pull/12443

repo: hortonworks/cloudbreak


text:
Revert "Revert "CB-18490 Modify proxy config e2e test""

This reverts commit f119772bbc633b9e823d678f504e542da03b74a0.
This commit does not contain changes to the e2e test, but new parameters were added to the test runs that will hopefully fix the original issue:

https://github.com/hortonworks/cloudbreak-ansible-playbooks/commit/30a3f1655c2df4b77feb8e1772d9742eaee130ff
https://github.com/hortonworks/cloudbreak-ansible-playbooks/commit/31409ccc50c695632a145bd3cb9e18aca3e32e26

retest this please


text:
CB-10173 CLI flag for Public Gateway Endpoint

This adds two new parameters to the EnvironmentRequest for the feature called "semi-private
networks" also known as "public endpoint access gateway". Details of the feature can be
found in CRB-1318.
Since all of the network related and enableTunnel related parameters are at the environment
level (rather than per-datalake or per-datahub) I added these parameters at the environment-api
rather than datalake-api. Also logically it seemed appropriate to bundle these together in
the EnvironmentNetworkRequest since all network related configuration is there.
Testing done:

./gradlew build

See detailed description in the commit message.

This PR probably needs more work to add converters, etc. Just putting out there to see whether the API change here makes sense.

@enis I think you've got it right, and these are where we'll ultimately have to make the changes. I'll start poking around and see what updates we'll need to make to persisted the new fields to the database, and propagate the information to the appropriate places.

Currently, this looks fine.

@hreeve-cloudera has incorporated these changes in #9620. Let me close this one.

authors: enis hreeve-cloudera orngejaket


text:
Ajoute l'installation de postgresql dans le readme

Nécessaire pour faire tourner l'env de dev
J'ai aussi installé Postgres.app (brew install --cask postgres) mais ne sais pas si c'est une bonne pratique

Nécessaire pour faire tourner l'env de dev
J'ai aussi installé Postgres.app (brew install --cask postgres) mais ne sais pas si c'est une bonne pratique

On part du principe que les potentiels contributeurs connaissent postresql et savent l'installer, mais c'est toujours mieux de le préciser! Merci

Il me semble que si tu installes postgres.app, pas besoin de brew install postgres, c'est postgres.app qui gère son installation de postgres

authors: mathieuripert mininao yannbertrand

repo: hostolab/covidliste


text:
Turbo 7.3 Update Bug

I recently update to Turbo 7.3 and found that some links I am adding to the page are suddenly returning the full page html when clicked even though they have the data-turbo-frame attribute set on them. I am trying to just update a modal which does in fact get updated but the page underneath the modal suddenly becomes blank. This works perfectly on Turbo 7.2.
I will note that no Turbo-Frame header is being sent in the network request on Turbo 7.3 but it is on Turbo 7.2.
This might be something that is hard to reproduce but I am suspecting it has something to do with how these links are being added to the page. They are added after a turbo_stream request updates a div on the page within the turbo-frame. Again, this works perfectly if I switch back to Turbo 7.2 and there are no console errors being reported.
I can try to recreate the issue in a test example but thought I would first report it to see if this is a known issue or something obvious might be going so that I am not spinning my wheels trying to distill it.
Thanks for your time and consideration and please let me know how I can help.

Hi @jonsgreen. This isn't a known issue as far as I'm aware. Thanks for reporting it.
To make sure I'm following, is this the sequence of events that you're seeing:

A turbo stream response updates a div which is inside a <turbo-frame>, and that response includes links.
When those links are clicked, Turbo is not sending the expected Turbo-Frame header with the request.
And when the response is returned, content outside of the frame is being updated.

If you are able to recreate the issue in a test example, that would be great. That should help us investigate what's going on.

@kevinmcconnell so far I cannot create a minimal reproduction of the bug. I definitely have the problem on my app but the pattern works fine with a vanilla rails app. For a moment I thought maybe it could have something to do with the fact that the turbo stream request is happening from a stimulus controller using request.js but I tried to do the same thing in the minimal example and the added links still worked fine requesting turbo frames with the Turbo-Frame header passed along.
I might play with it some more to see if I can figure out anything else but otherwise we may just want to close this issue. Perhaps someone else will have a similar problem and we can triangulate.

@kevinmcconnell I have one more little clue that might be helpful. I just realized that not only is the link request not requesting a Turbo-Frame, it ends up changing the url on the page. I don't really know much about debugging javascript in the browser but the request follows the following chain gotten from the network initiator in Chrome (I believe it is in reverse order):
| perform | @ | application-49895f6….js:11157
-- | -- | -- | --
  | await in perform (async) |   |
  | issueRequest | @ | application-49895f6….js:12389
  | visitStarted | @ | application-49895f6….js:12634
  | start | @ | application-49895f6….js:12345
  | startVisit | @ | application-49895f6….js:12902
  | visitProposedToLocation | @ | application-49895f6….js:12629
  | visitProposedToLocation | @ | application-49895f6….js:13590
  | proposeVisit | @ | application-49895f6….js:12893
  | visit | @ | application-49895f6….js:13530
  | followedLinkToLocation | @ | application-49895f6….js:13583
  | LinkClickObserver.clickBubbled | @ | application-49895f6….js:11758
Meanwhile successful turbo-frame request have the following:
| perform | @ | application-49895f6….js:11157
-- | -- | -- | --
  | await in perform (async) |   |
  | (anonymous) | @ | application-49895f6….js:14117
  | visit | @ | application-49895f6….js:14110
  | loadSourceURL | @ | application-49895f6….js:13961
  | sourceURLChanged | @ | application-49895f6….js:13934
  | attributeChangedCallback | @ | application-49895f6….js:10793
  | set src | @ | application-49895f6….js:10803
  | (anonymous) | @ | application-49895f6….js:14124
  | withCurrentNavigationElement | @ | application-49895f6….js:14296
  | navigateFrame | @ | application-49895f6….js:14123
  | linkClickIntercepted | @ | application-49895f6….js:14003
  | LinkInterceptor.linkClicked | @ | application-49895f6….js:11717
  | dispatch | @ | application-49895f6….js:10985
  | notifyApplicationAfterClickingLinkToLocation | @ | application-49895f6….js:13670
  | applicationAllowsFollowingLinkToLocation | @ | application-49895f6….js:13662
  | willFollowLinkToLocation | @ | application-49895f6….js:13578
  | LinkClickObserver.clickBubbled | @ | application-49895f6….js:11756
It seems like there is a deviation right in the beginning where it ends up calling followedLinkToLocation.
I don't know anything about this lower level code but perhaps it would suggest something to someone who is more familiar?

I finally figured out the difference that is causing the problem. The links that are added are inside a custom autocomplete component that is backed by a stimulus controller. I have a click@window for closing the autocomplete results (which contains the links). If I remove that click handler then everything works fine. Somehow this seems to make the event.defaultPrevented set to true and interferes with the whole turbo frame flow.
Note again that this only started happening on Turbo 7.3.

@jonsgreen that's helpful, thanks. Can I ask what version you were using before that worked? Also how is your Stimulus controller interacting with the event -- is it calling event.preventDefault() or is there more to it?

There is no event.preventDefault() in the Stimulus controller.
Everything worked fine on Turbo 7.2.0
Now that I better understand the issue I will see if I can reproduce it in the minimal example I started. The whole turbo-stream was a red herring in the end.

@kevinmcconnell So I was just able to determine the bigger source of the issue. While the stimulus controller does not call event.preventDefault it does set the innerHTML to '' which effectively removes the link from the page. I am not sure that Turbo should be responsible for dealing with that kind of twisted behavior so I would be fine if you close this issue. I can hopefully find a different way to deal with the situation. If you disagree then you can let me know.
Thanks so much for your incredibly prompt responses to my seemingly spurious issue.

@jonsgreen ah interesting! Glad to hear there was an explanation. It's curious that it worked differently with the previous Turbo version, but it does sound like an unusual edge case. I wonder if changing that controller to call event.preventDefault() would help then, if the intended behaviour is for your Stimulus controller to hijack the click completely? There are also the turbo:* events that may be useful if you want it to intercept the clicks in some other way.
I'll close this issue then, if you're happy that this is an application issue. But feel free to reach out if it does turn out to be something unexpected on the Turbo side after all!

authors: jonsgreen kevinmcconnell

repo: hotwired/turbo


text:
Add disable-with Behavior to Turbo

Following some discussion in the GoRails Discord a while back, it looks like there's some popular sentiment that the old Rails UJS behaviors for confirm and disable-with should be rolled into Turbo. Seeing that #379 implements the confirm behavior, I wanted to get the ball rolling on this.
As of right now, the behavior works in the most baseline fashion possible. On a form submission, the submit tag is disabled. Unfortunately, I'm unfamiliar with Intern, so I had some trouble getting the test to work correctly. For some reason I couldn't seem to query for the button fast enough before the page changed.
Additionally, as I see it right now, there's 3 major items that still need to be addressed:

Getting the buttons to be un-disabled when the form is a permanent element on the page.
Getting the buttons to be un-disabled when the form is targeting a different Turbo frame.
Making sure that the buttons are not disabled when navigating using the back button after a full redirect.

I have an idea of how to handle the first two using the FormSubmissionDelegate interface, but I'm not sure about the 3rd. I think it involves the way history is tracked.
So if anyone can offer some insight on getting tests working and on how we'd make this work with how Turbo manages history, it'd be a huge help.

Thanks for working on this! I do like to see this come to Turbo. Does feel like a natural place.
Can you cleanup the PR so it doesn't have style changes? Then it's easier to review.

Of course! This is still very much WIP, just wanted to bookmark this to see if anyone else had ideas. I’ll likely have time to clean this up this weekend.
Do you see styles changing on the element or are you referring to code style?

No sweat. I just meant the style changes that are showing up in the diff unrelated to your changes:

Given the need for this behavior, and Given that #384 relies on a new attribute (data-turbo-disable-with) and won't be a drop in replacement of data-disable-with, could it be worth considering an alternative like https://github.com/hotwired/turbo/pull/386 instead of re-implementing the Rails UJS behavior?

Yeah, I really like the simplicity of just always disabling the submitter until the request is finished. Should just be standard. We even moved away from using extra submission wording in HEY as well. Just wasn't required. I dig that, @seanpdoyle 👍

Let's proceed with #386. Thanks for kicking this off @jacobdaddario 🙏

authors: dhh jacobdaddario seanpdoyle


text:
Blake updates

I added two .py files.  The first, sensitivity_tools.py, sets up and performs the sensitivity analysis for a generic model.  The second, data_processing.py, contains a variety of functions for reading all our various data files and processing them, then making pandas dataframes from them.

@swapilpaliwal can you comment on the line where you think there is a typo?

line 24
On Tue, Feb 23, 2016 at 7:46 PM, houghb notifications@github.com wrote:

@swapilpaliwal https://github.com/swapilpaliwal can you comment on the
line where you think there is a typo?
—
Reply to this email directly or view it on GitHub
https://github.com/houghb/HDSAviz/pull/1#issuecomment-188049292.

Line 24 is just a bit of the function docstring so I'm not sure what you're referring to.  Can you go here: https://github.com/houghb/HDSAviz/pull/1/files and click the + sign next to whatever line you want to comment on then type your comment?

Hey sorry,
i was commenting on this
https://github.com/houghb/HDSAviz/commit/7a5d239d9ffd4c6a52beacb67e85b3f87556a325
its line 33 in this document  https://github.com/houghb/HDSAviz/pull/1/files
On Tue, Feb 23, 2016 at 8:04 PM, houghb notifications@github.com wrote:

Line 24 is just a bit of the function docstring so I'm not sure what
you're referring to. Can you go here:
https://github.com/houghb/HDSAviz/pull/1/files and click the + sign next
to whatever line you want to comment on then type your comment?
—
Reply to this email directly or view it on GitHub
https://github.com/houghb/HDSAviz/pull/1#issuecomment-188053743.

I've documented the specific changes that I made in this branch in my commit messages and in my responses to comments above.  We've decided it is ready to merge with the master branch.

authors: houghb swapilpaliwal

repo: houghb/HDSAviz


text:
syncup指定路径时--include-regex参数失效

使用命令:
bypy --include-regex='^(?!testfolder1|.testfile3).*$' -v syncup /home/user/test/testbypy /backup/test
文件全部上传到了/apps/bypy/backup/test/, 无法过滤.
但是不指定路径, 命令:
bypy --include-regex='^(?!testfolder1|.testfile3).*$' -v syncup
却能过滤掉(但是会创建一个空的testfolder1文件夹)

谢谢反馈。等有时间时（不定哪天。。）我看一下，或者你想提交PR也行。

authors: houtianze theLSA

repo: houtianze/bypy

license: mit


text:
add auth API

增加get_auth_link, set_auth_code两个API，用以支持API方式授权，减少交互

感谢PR，没看明白到底增加了什么功能？


text:
visibleOnFullScreen

 breakWinLocal.setVisibleOnAllWorkspaces(true, { visibleOnFullScreen: true })
visibleOnFullScreen is currently deprecated. Once it's removed, what will be the effects?

As I understand it, on macOS, when users have multiple workspaces, Stretchly won't be visible on those workspaces that have app in fullscreen mode ;/

But it looks like it might have never worked anyway: https://github.com/electron/electron/issues/19674

authors: hovancik pjebs

repo: hovancik/stretchly


text:
Change style for checkbox

Checkbox has a different color than the standardized colors. Need to change to a more consistent color with regards to the current website.

The issue here is that the checkbox defaults to checked color secondary instead of primary. We currently use Material UI v4.
Material UI v4 docs: default color is secondary
https://v4.mui.com/api/checkbox/#props
Material UI v5 docs: default color is primary
https://mui.com/components/checkboxes/#color
So this will be fixed once we update to material v5 (another reason to address #290)

authors: MorganFH erikttv

repo: hovedstyret/indok-web


text:
Bildduplikate müssen ausfindig gemacht werden

Zu manchen Suchbegriffen/ in manchen Kategorien findet man viele Bildduplikate, die ein und dasselbe Motiv immer wieder aus der gleicehn Perspektive zeigen. Bevor wir eine Lösung dafür umsetzen können, müssen wir erstmal in der Lage sein diese Duplikate ausfindig zu machen.

Relevant: #324

authors: MariusDoe baerlach-git


text:
Feat/reload classes

closes #190

The only test that is not running yet is testEnableAfterInstall in BOALGTest, because it installs Algernon from the development branch and doesn't find the new message yet. This will most likely be fixed with the merge of this PR into development

@bewee

Although that's not explicitly stated in the issue, I think that - since the intent was not having to manually reset Algernon during programming - reloading packages and methods is part of this issue? I also feel like this should be pretty easy to implement now, isn't it? (even the tests, I think a lot from testCreateAndDeleteClass could be reused)? Or do you think that's rather another issue?

We favoured packing that into new user stories as only reloading classes was requested (tbh the issue originally didn't even took reloading when classes get deleted into consideration). As you noted a lot of our code can be reused and e.g. when taking categories and methods into consideration, those kinds just need to be added into the itemKinds in subscribeToSystemChangeNotifier and there you go.
What i am trying to say: those new user stories won't produce much effort (but we could pose like that for the customer? 😄)

@FranzSw Olli and I talked and decided that it's fine for now. But feel free to have a look at it if you still want to😅

authors: Olliwehr bewee dieknolle3333

repo: hpi-swa-teaching/Algernon-Launcher


text:
关于训练损失函数

作者，你好，我看到代码里面损失函数部分如下：
def loss_func(pred):
soft_loss = -pred[0]pred[0].log()
mask = torch.zeros(pred[1:].shape).to(pred[1:].device)
mask[...,1] = 1
hard_loss = -maskpred[1:].log()-(1-mask)pred[0].log()
soft_loss = soft_loss.sum(-1).mean()
hard_loss = hard_loss.sum(-1).mean()
#alpha=0.9
return 1soft_loss+0.5*hard_loss
而论文里面的损失函数写作如下图所示：

感觉难以对应起来，怀疑代码这里写的有点问题，能解释一下吗？

Hello, there are some mistakes in our paper, we will update a revised version later. Thanks.

We have deduced the equations again, and there is no mistake in the equation. Both of the objective functions can be treated as cross-entropy, and we omit the y(mask)=0 item in the function. If there is any question, please let me know.

repo: hq-deng/AnoVL


text:
subfolder exceptions no longer work

In my WordPress site .gitignore prior to v1.7 this worked to ignore all the wp-content folders and then specify some sub folders as exceptions. Using PHPStorm 2016.3.2
!wp-content/
wp-content/*
!wp-content/mu-plugins
!wp-content/themes/zeropointdevelopment

As of v1.7 all of wp-content is ignored and it doesn't seem to pick up any sub folder exceptions.

@DeveloperWil
Thanks for reporting. I've fixed it and it will be available in the next release this morning

authors: DeveloperWil hsz

repo: hsz/idea-gitignore


text:
Proxy websockets

Proxy websockets when --websocket is given.
Relevant issues
Fixes issue https://github.com/http-party/http-server/issues/142
I took the PR from @fabioimpe at https://github.com/http-party/http-server/pull/528, removed style changes (and made sure not to add any of my own), and removed the changes to the changeOrigin parameter.
And of course, I tested whether it worked, which it does.

Is there any way to test this? This should have tests before I merge it, if a unit test for this is complex I'll do a manual test later.

Well, it relies on another server working with websockets. A unit test could be done, but would require quite a bit of work.

authors: KernelDeimos pft

repo: http-party/http-server


text:
Add code examples to API docs

The existing API docs were written quickly and don't have examples yet. Let's get some added!

I'd like to take on this one. Seems like a great way to get to know the project.

How come this is open, do we need more code examples?

@zacck that was the idea. But we already have quite a few docs, so there's not too much use in keeping this open. Perhaps we should do a docs push at some point though, but we can open a new issue for that.

authors: aturon pbvie yoshuawuyts zacck

repo: http-rs/tide


text:
Flaky test: SSL Handshake WRAP produced 0 bytes.

02:19:49.930 [InnocuousThread-4] WARN  o.h.blaze.pipeline.stages.SSLStage - SSL Handshake WRAP produced 0 bytes.
Status = OK HandshakeStatus = NEED_WRAP
bytesConsumed = 0 bytesProduced = 0
02:19:49.930 [InnocuousThread-4] WARN  o.h.blaze.pipeline.stages.SSLStage - SSL Handshake WRAP produced 0 bytes.
Status = OK HandshakeStatus = NEED_WRAP
bytesConsumed = 0 bytesProduced = 0
02:19:49.931 [InnocuousThread-4] WARN  o.h.blaze.pipeline.stages.SSLStage - SSL Handshake WRAP produced 0 bytes.
Status = OK HandshakeStatus = NEED_WRAP
bytesConsumed = 0 bytesProduced = 0
02:19:49.931 [InnocuousThread-4] WARN  o.h.blaze.pipeline.stages.SSLStage - SSL Handshake WRAP produced 0 bytes.

This happened on an openjdk11 build, which makes me fear that http4s/blaze#233 is sticking our heads in the sand.

Again: https://travis-ci.org/http4s/http4s/jobs/444236484

I expect this is fixed by #2204.  Will reopen if it's not.

:fire:

https://travis-ci.org/http4s/http4s/jobs/447372379#L3243

We just got hit by that in production, right after we switched to JDK11 😢

We also started seeing this after upgrading to jdk 11.

Same here. It happens after several minutes, client idle.
AdoptOpenJDK (build 11.0.4+11) / http4s v0.21.0-M5

We're working on releases of 0.20.x and 0.21.x, but in the meantime, this should be fixed by explicitly upgrading to blaze-0.14.11.

authors: coltfred ghostdogpr rossabaker sderosiaux


text:
Upgrade to scalaz-stream-0.7a.

Fixes #237.

Do you mind if I push a fix for the broken test?

Go for it. I haven't had a chance to look.

I think this is good to go, but someone should double check my two commits. Sorry there were two, but I had an epiphany and felt compeled to do a second commit.

Oops, that's what I get for not cleaning. :+1:

authors: bryce-anderson rossabaker


text:
Update jackson-databind to 2.12.3

Updates com.fasterxml.jackson.core:jackson-databind from 2.12.2 to 2.12.3.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (2.12.2).
You might want to review and update them manually.
website/src/hugo/content/changelog.md

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.fasterxml.jackson.core", artifactId = "jackson-databind" } ]

labels: library-update, semver-patch, old-version-remains

We can do this on 0.22.

authors: rossabaker scala-steward


text:
Small typo in readme

clietn => client

Thanks!

authors: tarcieri thorn

repo: httprb/http


text:
🛑 TW591 is down

In 81ece87, TW591 (https://www.591.com.tw) was down:

HTTP code: 504
Response time: 20194 ms

Resolved: TW591 is back up in f8d5fb0 after 48 minutes.


text:
[Bug]文件夹解析错误

url: https://katetseng.lanzoue.com/b00r4c8ah
password: 9jw9
部署方式:vercel

https://lanzou-straight-chain-analysis.vercel.app/api/?url=https://katetseng.lanzoux.com/b00r4c8ah&9jw9&pass=9jw9

解析大佬readme中的文件夹正常，自己新创建的文件夹返回203，大佬的示例api返回一样结果
改成lanxzoux也无法正常解析
单文件解析/id均正常

奇怪，我api示例我看可以请求

但是你的不行

找到问题了，pgs参数正则没匹配到

fixed on commits:
https://github.com/huankong233/lanzou_url/commit/354d5d8a9c13181257269628f20b76884147e092
https://github.com/huankong233/lanzou_url/commit/1d472879fe02f3597fd6ba27ee7d927df4c7bb50


text:
Meta file to watch for scenario changes

Is your feature request related to a problem? Please describe.
The base issue here is that we currently require scl scenario --clean in most cases to rebuild scenarios. This could be greatly sped up by skipping cases where things do not need to be rebuilt.
Describe the solution you'd like
I suggest a meta file that would maintain a hash of base scenario resource files:

Map (e.g. map.xml)
Scenario (e.g. scenario.py)
requirements (e.g. requirements.txt)

When such files are noted to have changed it would trigger a rebuild of dependent scenario resources. This would be useful especially for very large maps and complex scenarios that take a long time to build.

Closed by #1764

authors: Adaickalavan Gamenot

repo: huawei-noah/SMARTS


text:
Zooming in/out menu behavior in Google Chrome

The top navigation options reappear when zooming out:

@LibbyUX fixed in pr #468

Thank you!

authors: LibbyUX bhushankhope

repo: hubmapconsortium/hra-ui


text:
Release v1.3.2 to forge?

I see the metadata.json was updated to version 1.3.2 after all the updates I submitted got merged. That was nearly a month ago now. Is this going to be released to the forge anytime soon?

@pranav, @kenbreeman The force account for hubspot is in SS.  You'll need to review the process for building and submitting.  I haven't done it in awhile.

Unless you're using something to do the build / submit cycle for you the process is
From an up to date checkout, in the root of the project

puppet module build
login to forge
select publish from the links next to your avatar
press the choose file button and select the file
press upload

;)

v1.3.2 has been released to Forge!
https://forge.puppetlabs.com/hubspot/nexus/readme

Thanks!

Could you please also tag version 1.3.2?

Done

authors: kenbreeman sgerhards tmclaugh tykeal

repo: hubspotdevops/puppet-nexus


text:
🛑 Akkoma is down

In 98bc79b, Akkoma (https://akkoma.chez.chat) was down:

HTTP code: 404
Response time: 194 ms

Resolved: Akkoma is back up in edcf89e after 6 minutes.

repo: hufman/upptime


text:
Не помогает с twitter

Так и должно быть или надо искать корректные настройки?

Так и должно быть или надо искать корректные настройки?

ciadpi -i 127.0.0.1 -p 10801 -d 1
и настроить DoH, например, так:
https://help.keenetic.com/hc/ru/articles/360007687159-Прокси-серверы-DNS-over-TLS-и-DNS-over-HTTPS-для-шифрования-DNS-запросов
у меня так работают твитор, мордокнига и инста без всяких hosts

Facebook, twitter заблокированы также и по ip.
Поэтому в дополнение надо прописывать в hosts незаблокированные ip адреса.

Фейсбук работает, abs-0.twimg.com — нет. Куда копать?

authors: EvgenyKV artem-burashnikov tkzv

repo: hufrea/byedpi


text:
Fix TPU training in example

We are using accelerator.distributed_type in our collate_fn but the accelerator is defined later.

Ah thanks a lot for fixing!

authors: sgugger thomwolf

repo: huggingface/accelerate


text:
Add ZEST: ZEroShot learning from Task descriptions

Adds the ZEST dataset on zero-shot learning from task descriptions from AI2.

Webpage: https://allenai.org/data/zest
Paper: https://arxiv.org/abs/2011.08115

The nature of this dataset made the supported task tags tricky if you wouldn't mind giving any feedback @yjernite. Also let me know if you think we should have a other-task-generalization or something like that...

Looks good to me, we can ping the authors for more info later. And yes apply other-task labels liberally, we can sort them out later :)
Looks ready to merge when you're ready @joeddav

authors: joeddav yjernite

repo: huggingface/datasets


text:
Fix test_delete_from_hub on Windows

Fix test_delete_from_hub on Windows by passing explicit encoding.
Fix #6856.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


text:
Improve error message when push to hub fails because it is not a git repo

Is your feature request related to a problem? Please describe.
I can't figure out which folder I need to modify to get a push_to_hub to work. I would like to improve the already excellent error message:
/usr/lib/python3.8/site-packages/huggingface_hub/repository.py in clone_from(self, repo_url, token)
    760                 # Check if the folder is the root of a git repository
    761                 if not is_git_repo(self.local_dir):
--> 762                     raise EnvironmentError(
    763                         "Tried to clone a repository in a non-empty folder that isn't a"
    764                         " git repository. If you really want to do this, do it"

OSError: Tried to clone a repository in a non-empty folder that isn't a git repository. If you really want to do this, do it manually:
git init && git remote add origin && git pull origin main
 or clone repo to a new folder and move your existing files there afterwards.
OSError: Tried to clone a repository in a non-empty folder that isn't a git repository. If you really want to do this, do it manually:
git init && git remote add origin && git pull origin main
 or clone repo to a new folder and move your existing files there afterwards.

Describe the solution you'd like
I would like the self.local_dir variable to be interpolated into the error message:
"Tried to clone a repository in a non-empty folder that isn't a"
f" git repository ('{self.local_dir}'). If you really want to do this, do it"

Describe alternatives you've considered
None
Additional context
None

Happy to implement the suggested fix

Hi @payoto, I'm completely fine with the idea as it would for sure improve user experience. Feel free to implement it and thank you in advance! :)

Out of curiosity, in which context are you using the Repository class in your projects ? Question is unrelated to the issue, it's just to get to know more about the usage of this module.

Great, I'll do that then.
It's used as part of the optimum Graphcore library (the implementation of optimum for Graphcore's IPU). The error shows up during a call to push_to_hub. I've been running quite a lot of models and tinkering with moving/sharing caches between containers and I suspect a .git folder got lost in the process.

Ok, thanks for the input :)
I approved the PR. Have a nice weekend.

authors: Wauplin payoto


text:
Fix precedence of default tasks over custom ones in registry

Fix precedence of default tasks over custom ones in registry.
Currently, custom tasks are overwriting default ones.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

I am wondering why default tasks have precedence over custom ones: I would expect that if a user creates a custom task with the same name as a default one, they would expect to overwrite the default one.

wait, not sure I follow, from the code It seems like custom tasks are overwriting the default ones right ?

Yes, and the docs say the opposite:

Defaults tasks should overwrite custom tasks

You are right custom tasks should overwrite default ones I think

I am fixing the comment/warning instead.

repo: huggingface/lighteval


text:
Sarkar/transformers 4.43 fixes

What does this PR do?
Fixing test errors
Focussing on one model file, hopefully changes will help fix others:
RUN_SLOW=true  GAUDI2_CI=1 python -m pytest tests/transformers/tests/models/t5/test_modeling_t5.py -v -s
main: 3 failed, 112 passed, 37 skipped, 20 warnings
regiss branch: 15 failed, 100 passed, 37 skipped, 10 warnings
sarkar/transformers_4.43_fixes: 4 failed, 107 passed, 37 skipped, 4 xfailed, 10 warnings
Of the 4 failures, 2 are present in main (the other fail from main is xfail)
The remaining 2 are of the same type: running sample generate twice gives different results (which is fine if seed is not set, but if seed is set, we expect them to be same)
Sampling run twice should gen same result if seed is set
FAILED tests/transformers/tests/models/t5/test_modeling_t5.py::T5ModelTest::test_sample_generate - AssertionError: Lists differ: [[0, 0, 3, 3], [0, 0, 3, 3]] != [[0, 82, 91, 39], [0, 82, 91, 63]]
FAILED tests/transformers/tests/models/t5/test_modeling_t5.py::T5ModelTest::test_sample_generate_dict_output - AssertionError: tensor([[ 0,  0, 62, 62],
Failures from main
FAILED tests/transformers/tests/models/t5/test_modeling_t5.py::T5ModelIntegrationTests::test_summarization - AssertionError: Lists differ: ['pro[360 chars]n in the occupied Palestinian territory . as m[703 chars]2 .'] != ['pro[360 chars]n in Palestinian territories...
FAILED tests/transformers/tests/models/t5/test_modeling_t5.py::T5ModelIntegrationTests::test_torch_quant - AttributeError: 'Linear' object has no attribute 'custom_name'

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

LGTM! Is it ready to be merged?

new PR. deleting unnecesary tests: https://github.com/huggingface/optimum-habana/pull/1200

repo: huggingface/optimum-habana


text:
VLLM Supported?

I wonder is quanto-quantized model available using vllm?

quanto models are pytorch models, so I don't see a reason why they would not be compatible with any other tool running pytorch models.
However, since vLLM modifies the graph of transformers models, the model should be quantized inside vLLM, and not inside transformers.

authors: RanchiZhao dacorvo

repo: huggingface/optimum-quanto


text:
FIX: Small regression in BNB LoRA output

Our regression tests reveal that the 8bit LoRA BNB regression test is failing. To reproduce, run:
pytest tests/regression/test_regression.py -s --regression -k test_lora_8bit
The regression was introduced in #2122. We didn't notice this earlier because of other failing tests in the nightly CI.
The cause of the error is subtle. In the original code, we would calculate the LoRA output, convert the dtype if necessary, then add it to the base output. After the mentioned PR, we calculate the LoRA output, add it to the base output, then convert the dtype if necessary (code). The difference is very small on a per layer basis, but it can accumulate over the layers, leading to a significant difference in outputs, as witnessed by the regression test.
This PR rolls back this specific part of the PR (both for 8bit and 4bit) while leaving the main change of that PR intact.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

authors: BenjaminBossan HuggingFaceDocBuilderDev

repo: huggingface/peft


text:
[docs] Prompt tuning for CLM

Move the prompt tuning for clm notebook to the docs.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

authors: HuggingFaceDocBuilderDev stevhliu


text:
loading saved model when n_classes != 2

Required to for: Assertion t >= 0 && t < n_classes failed,  if your default number of classes is not 2.

This problem is discussed is #135 and I don't think that this is the right way to patch this problem. The saved model contains the num_labels information.

authors: SinghJasdeep rodgzilla

repo: huggingface/pytorch-pretrained-BERT


text:
Ability to cancel generation requests while they're underway

Feature request
Kind of like continuous batching, but for removing items from the batch in real-time, I guess? Not sure how technically difficult this is. Edit: Although now that I think about it, it seems like it should be pretty easy - it's just "emulating" the situation where the LLM produces a stop token - and all the code for that is obviously already written.
Motivation
The ability to stop an inference is useful in situations where the initial text being generated is clearly not what the user wanted, or where they suddenly changed their mind. Otherwise they have to wait up 20 seconds or so (in my case) for the previous generation to finish before they can submit another request.
Design
Ideally I'd just be able to send a simple HTTP request with an ID that I passed to the original generation request, or something like that.
For example, starting the request:
curl http://localhost:8080 \
        -X POST \
        -d '{"inputs": "Once upon a time", "id":1234}' \
        -H "Content-Type: application/json"

and to cancel it:
curl http://localhost:8080/stop \
        -X POST \
        -d '{"id":1234}' \
        -H "Content-Type: application/json"

But maybe there's a better way - anything that's roughly as ergonomic as that would suit me.
Your contribution
I likely won't be able to contribute to this unless there's an JS work that needs to be done.

to cancel it: curl http://localhost:8080/stop

Usually this works with streaming on. You can hit Ctrl+C that stops curl, server in turn sees socket close (or HTTP/2 cancel in more serious applications). There is no need for a /stop with id.
To test this, you can try several other HTTP servers, FastAPI streaming hello world (cancel works even through ngnix for FastAPI), openai API.
(by streaming I mean "stream": true in the request, also known as server-side events)

Indeed it' s already implemented on connection closed. So it also handles rogue connection cuts etc..

Indeed it' s already implemented on connection closed. So it also handles rogue connection cuts etc..

I am using HuggingfaceEndpoint by Langchain, how do I implement this?

authors: Narsil SMAntony josephrocca olegklimov

repo: huggingface/text-generation-inference


text:
Standardize dataset naming

What does this PR do?
Small standardisation PR
ds, raw_dataset etc -> dataset
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a GitHub issue? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

authors: HuggingFaceDocBuilderDev qgallouedec

repo: huggingface/trl


text:
TOC setting bug

Describe the bug
Cannot turn off TOC for a blog post when global setting of TOC is on.
Steps to reproduce

set global TOC setting as true

[params]
  toc = true # set to false to disable table of contents 'globally'

Turn off TOC setting for some post.

+++
title = "About"
toc = false
+++

Expected behavior
Should overwrite global config

Thanks for reporting. I will implement a fix for this.

Please see https://hba.sid.one/posts/table-of-content/ for documentation regarding disabling ToC on certain blog posts.
Feel free to reopen this issue if you encounter any problem with Toc.
If this theme has helped you consider supporting it.

repo: hugo-sid/hugo-blog-awesome


text:
consent property 'flag' ambigious

Description
With ConsentCookie.get("<app id>") you will get an object that represents a consent.
This object will have a property called flag. Like stated in the documentation the value 0 represents a disabled (rejected?) consent and the value 1 represents an enabled (granted?) consent.
I think the term flag is ambiguous.
Expected Behavior
In my opinion, it would be better to have this flag property only for internal use and backward compatibility.
Instead, functions like:

ConsentCookie.isGranted("<app id">)
ConsentCookie.isRejected("<app id">)
ConsentCookie.get("<app id>").isGranted()
ConsentCookie.get("<app id>").isRejected()
will be more intuitive to use.

@rockxwre
With the 0.7.0 version the current ConsentCookie.get is set deprecated.
Instead 2 API functions are added:

ConsentCookie.getConsent
ConsentCookie.getConsents

ConsentCookie.getConsent works as the the deprecated ConsentCookie.get with some minor changes

When calling the function and instance of Consent is always returned. The instance has the following getters for easy checking which flag is set:

isAccepted() // If the user has accepted the consent
isAlwaysOn() // If the consent is always on (and cannot be changed by the user)
isEnabled() // If the consent is accepted by the user or is always on
isRejected() // If the consent has not been accepted by the user

When calling the function with an id, a instance of Consent is always given. If no consent with the given id exist, a consent with the flag:null is returned. The getters will all return false when called because no state was set.
When calling the function without an id all configured consents are returned as an Array

ConsentCookie.getConsents is new and is based on calling the deprecated ConsentCookie.get without an id arguement.

The consents are now a private variable. The return of the ConsentCookie.consent will give the Consents instance with the following getters:

get($id) // Works the same as calling ConsentCookie.get
getConsentMap // Will give a map of all configured consents with the key the id of the consent and the value a Consent object
getCookieValues // The ConsentCookie values not filtered by the configuration mapped by key = id and value = consent flag

Could you comment on this update related to the requested changes.

Test results
I tested it using the browsers Console. Everything works as described, which is great!
However, I'm struggling with some method names. What I did was executing each method on a specific consent. I did not read your explanation of the method but tried to explain the methods myself. These are the results:
isAccepted
I read some GDPR documentation and in this documentation the verb 'grant' is used in this context. You grant or reject a consent. So maybe isManuallyGranted() is better (please read further).
isAlwaysOn
In line with my previous comment (isAccepted), a consent cannot be 'on' or 'off'. In this case you have no choice. You are forced to grant the consent. Maybe something like isForcedGranted() or isAutoGranted()?
isEnabled
This one confused me the most. Since consent toggles can be immutable, my first guess was that if isEnabled() === true the toggle was enabled. But what it means is that the user granted the consent explicitly or the user had no choice to grant the consent because it was granted automatically. So maybe isGranted()?
isRejected
This one is in line with my previous comments.

authors: StevenChoo rockxwre

repo: humanswitch/consentcookie


text:
fix/ Fix account balance not reflect correctly + Improve websocket connection

Before submitting this PR, please make sure:

[x] Your code builds clean without any errors or warnings
[x] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
Fixed:

Account balance in status not updated (https://github.com/hummingbot/gateway/pull/128#issuecomment-1810831230)
Websocket connection to XRPL RCP node not stable for long time bot running

Tests performed by the developer:

Run PMM for SOLO/XRP on Ripple DEX mainnet for 1 hour, no major error. Made 5 trades, all trades and order fills are tracked correctly.
Link to the market: https://sologenic.org/trade?network=mainnet&market=534F4C4F00000000000000000000000000000000%2BrsoLo2S1kiGeCcn6hCUXVrCpGMWLrRrLZz%2FXRP

Tips for QA testing:

You would need an activated ripple account to trade on Ripple mainnet DEX
This PR has a counterpart in client: https://github.com/hummingbot/hummingbot/pull/6708

Merged to development and part of release version 1.23.0

authors: mlguys rapcmia

repo: hummingbot/gateway


text:
International calendars that the @internationalized/date supported

Describe the feature
As you know the @internationalized/date Support for 13 calendar systems used around the world, including Gregorian, Buddhist, Islamic, Persian, and more.
can we have them in shadcn-svelte calendar component?

Closing as already open in bits

authors: blue-range huntabyte

repo: huntabyte/shadcn-svelte


text:
🛑 chirpy is down

In 31aa397, chirpy (https://chirpy.thekor.eu/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: chirpy is back up in 82e92f4 after 9 minutes.

repo: hupratt/upptime


text:
About using custom datasets

请问您能提供一个训练所需数据格式吗？看readme好像只需要images就好了，是这样吗？用nerfstudio处理数据失败了。

ok，我知道了，我的问题在于我的数据集是多个摄像头固定拍摄，每个摄像头也就是每个固定视角有一个video file。而nerfstudio的视频样例都是一个视频里是摄像头围绕一个场景拍摄。请问您对于我这种格式的数据集需要怎么做有什么建议吗？

您好，你可以把每个摄像头的第一帧提取出来组成一个数据集算pose，这样每个pose就对应一个video了

感谢您的回复！成功使用nerfstudio获得了pose，但是然后我该怎么组织格式让我的视频数据集符合您的代码需求呢？如果能给一个具体文件组织tree将不胜感激！

您好，我觉得您需要将nerfstudio的相机坐标系转换为llff的相机坐标系，这个您需要自己了解一下不同坐标系的xyz轴朝向写一个脚本进行转换，最后保存为poses_bounds.npy文件，然后按照dynerf的数据集组织视频格式即可

您好，我觉得您需要将nerfstudio的相机坐标系转换为llff的相机坐标系，这个您需要自己了解一下不同坐标系的xyz轴朝向写一个脚本进行转换，最后保存为poses_bounds.npy文件，然后按照dynerf的数据集组织视频格式即可

请问这一步包含在最新的pr里面了吗？还是仍然需要自己手动生成这个npy文件？

自己实现了这部分。 不需要了。

authors: guanjunwu rexainn yangqing-yq


text:
[Discussion] Introducing a new network engine

Well... it would be a rant if I call it's an engine :laughing:
I developed a new networking library - lit, the main idea is to replace Socket.IO (for this game)
lit was built on top of Engine.IO, the framework that built Socket.IO as well.
It focuses on transmitting binary data instead of string data (what Socket.IO is doing right now), I know Socket.IO is already supported binary data but it's not the reason to keep me from building another one :trollface:
The main reason is #5, #349 and #190, there are so many people email me and ask if this server is good for develop production-ready games or not, and I keep saying that it's not ready. Now, it's time to make it production-ready (okay, I know I'm a bit ambitious :D)
Honestly, I didn't have time to test or benchmark it yet, I think it's better to introduce it here, so everybody can join with me and see/test/benchmark/integrate it.
Let's discuss to see if we can use it and replace it with Socket.IO for this game server in the future or not :D
Thank you so much!
Huy

How far along are you in integrating this new alternative? I would be very interested in seeing how well this improves performance!

I'm so sorry. I'm busy with some other stuff recently and don't have enough time for the integrating process. I will try to make a small demo to benchmark this library, maybe in the incoming long weekend.

Update: I made a demo with this engine, seems good, will do more testing and see if we can replace Socket.IO with this or not.
https://github.com/huytd/lit-demo/

Great! I actually have some time on my hands, so I may be able to help you out on this one!

@huytd Why use engine.io - its also incredibly slow. Use uWS (https://github.com/uNetworking/uWebSockets).


text:
Google Service (Drive v3 or Excel V4) added as a parameter

Modify GoogleDriveLoader so that it can accept a Google Service instead of relying on a path to credential files.
I am deploying LangChain in serverless environment where I use Redis for chat memory and security token store. This pull request allows to pass a Google Drive or a Google Sheet Service instead of a path to local files.
I would also consider moving _load_sheet_from_id into a different class than GoogleDriveLoader since it is not based on Google Drive API.
Dear @eyurtsev, please advise,

II have further improved this Pool request to add support to Slides and allow load_file_from_ids to check for the appropriate loader.

cross-referencing: https://github.com/hwchase17/langchain/pull/5135

The pull request 5135 resolve this.

#9999 resolves it.

authors: Yvelo eyurtsev leo-gan pprados


text:
Fixed a similarity score calculation bug in Chroma module

The _results_to_docs_and_scores() function should have returned the similarity score (higher values indicating more similarity), but erroneously returned the distance (higher values indicating less similarity).
@rlancemartin, @eyurtsev

I think most of the vector stores "scores" by default are distances, though it sometimes depends on how you configure the underlying index. This is why the relevance scores method was created, as an api that enforces an interpretation of what score means

I agree with you that most of the vector stores "scores" by default are distance.
If we want to use distance as the score as well, then we should modify the comparison operation  in VectorStore.similarity_search_with_relevance_scores() from
if similarity >= score_threshold to if distance_score <= score_threshold.

I think most of the vector stores "scores" by default are distances, though it sometimes depends on how you configure the underlying index. This is why the relevance scores method was created, as an api that enforces an interpretation of what score means

I agree with you that relevance scores method was created to enforce an interpretation of what score means.
At the same time, I also think that since VectorStore.similarity_search_with_relevance_scores() treats score uniformly as similarity, the underlying implementation (Chroma in this case) should also return similarity instead of distance.
https://github.com/hwchase17/langchain/blob/64039b9f112653d686bbcdd980fa90ad2f3eb9fc/langchain/vectorstores/base.py#L168-L174

I think most of the vector stores "scores" by default are distances, though it sometimes depends on how you configure the underlying index. This is why the relevance scores method was created, as an api that enforces an interpretation of what score means

Yes, https://github.com/hwchase17/langchain/pull/6570 has a fix for this.

@boxcounter please confirm that #6570 implements what you need. Will work with @raymond-yuan to get this in.

Great, that's what I need. And it's a much more complete implementation. Nice work!

Fixed in v0.0.230. Close this PR.

authors: boxcounter hinthornw rlancemartin


text:
Adds thrown error when using TimeWeightedVectorStoreRetriever on unsupported vector stores

Builds off of #911. Also adds docs and another test.

Cherry-picked onto original PR


text:
CSI Provisioner/storageclass support for hyperscalers

Replaced existing values-$OCPVER-hub.yaml with more specific values-$…
This is to support the new csi provisioners and storageclass in the big3 public clouds.
Removed the channel parameter from the values-hub.yaml file so that operator subscriptions will use the default channel for the given operator.

Awesome, thanks for the quick fix!

repo: hybrid-cloud-patterns/medical-diagnosis


text:
Publications Feature Tracker

About
Publication pages adds an easy way to create custom no-code page types, with support using a custom front matter schema and Blade templates. This feature is in development at the HydePHP develop monorepository, https://github.com/hydephp/develop/pull/685, and will be published as an official HydePHP extension tied to HydePHP v1.0.

Awesome!

authors: caendesilva rgasch

repo: hydephp/publications


text:
[BUG] Downloaded Games do not launch

Describe the bug
Since update 3.1.4, everytime I click on the "play" button, the game does not start.
Steps to Reproduce

Update Hydra to version 3.1.4
Install any game
Run any game
See error

Expected behavior
In any previous version, pressing the button would launch the game, but unfortunately, since the last update, it has not happened anymore.
Additional information and data
error.txt
Operating System
Windows 10
Hydra Version
3.1.4
Before opening this Issue

[X] I have searched the issues of this repository and believe that this is not a duplicate.
[X] I am aware that Hydra team does not offer any support or help regarding the downloaded games.
[x] I have read the Frequently Asked Questions (FAQ).

I assume it is happening for any game you try.
Did the same games with the same exact executables worked before 3.1.4?
The logs says Hydra could not run the executable because of permission denied (EACCES)
Any chance you did anything different with Hydra since 3.1.4 update?
Could you try running Hydra with admin mode and try launching a game?

It indeed happens with any game, and I've been using the same executables for a long time, but they only stopped working since the update. No configs were changed since the update, and I have already tried running it as admin. Thanks for the attention. I also found that downgrading Hydra doesn't fix the issue, so something happened when I updated, that botched some kind of authorization the application had.

Issue was solved after downgrading to 3.1.3, removing permissions from the Hydra executable, giving them back, and updating back to 3.1.4.

@BruninDoMine could you just explain better what steps you did to remove e readd hydra permissions?

authors: BruninDoMine zamitto

repo: hydralauncher/hydra


text:
[BUG] Downloads não disponíveis em nenhum jogo

Describe the bug
Todos os jogos da plataforma não possuem opções de downloads disponíveis.
Steps to Reproduce

Clique em qualquer jogo para instalar

Expected behavior
Quando entrar na página do jogo, não aparecerá nenhum link para download disponível. Espero que isso seja concertado.
Screenshots

Operating System
Windows 10
Hydra Version
v2.0.3 "Leviticus"
Additional Information
No response

O meu ta da mesma forma. Desintalei e instalei novamente e deu no mesmo.

Boa rapaziada coloca a fonte, coloca em ajustes e fonte de downloads
https://hydralinks.cloud
so copiar e colar desse site e clica para sincronizar acredito ser isso

Eu fiz isso e estava dando 1.2kb/s ia baixar em 36 anos kkkk E esta normal minha inthernet coloquei outro jogo para baixar na epic e foi rapidinho

Então, se não for via real debrid, voce vai depender do tanto de seeds disponiveis no torrent do game que esta baixando, isso que vai mandar na velocidade de download ! pode testar por exemplo com o dbz kakarot ele vai bem rapido por conta de ter mais seeds

Não entendi... tem discord pra suporte?

authors: M4riotto Pudk slanow vicxe


text:
Add capability to copy a resource

I think we need a button to allow a user to copy a resource that is shareable. This would allow them to copy the content then start working on it, say if they want to adjust it for inputs to their model or workflow.
Note that this is different from versioning, in that versioning is available only to resource owners and specifically holds forwards and backwards version pointers that establish and obsolescence chain.
When a copy is made, a relationship "Copied from" should be added to point back to the original resource.
This functionality would add to the existing resource manipulation functionality that we have, comprised of Create, Edit, Delete, and Create a New Version Version.

@dtarb - this is described by the workflow for creating derived resources in our Google doc. The Google doc specifies a "Derive Copy" button of some sort. I suggested in the comments using the term "isDerivedFrom" to capture the relationship between the source resource and the newer copy.

@horsburgh @dtarb I only see section 3.5 with a very brief (5 bullets) description for creating a derived resource for a published resource in google doc here: https://docs.google.com/document/d/1ixma0MlIiUjEEQ807kKLM9PfcMlqN7Yqv2AK1HEHlP4/edit. Is there another document I can reference for resource copy implementation? A specific question to clarify: Is the criterion for a user to be able to copy a resource is that the user has at least view permission to the resource and the resource is shareable? I believe if a resource is public, it must be shareable, right? So the condition for checking whether a resource can be copied could be resource is shareable and user has privilege of view or above over the resource. I will use "isDerivedFrom" relation metadata element for the copied resource.

A resource that is public is not necessarily shareable.  I think we should consider two options.

A user can copy any resource they can see.  This is the simplest and least restrictive.  It is de facto possible anyway because a user can always download the files then upload them again (breaking metadata) so allowing copying is not giving anything more, but it retains metadata better.
2  A user can copy any resource they can see that is shareable.  This overloads/extends the role of the shareable flag to include copy.  There is a logic to this, in that it does not make it easy for copying that an owner has not permitted by making the resource shareable, but we run the risk of disabling copying by mistake where owners have unset sharing and do not realize that this impacts copying.  The owner could always be contacted to ask to set the sharing flag, but that builds additional friction in the system.  Also if the resource is published then the owner can no longer change the sharing flag.  If we go with this option, I think we would also want to enable copying of published resources (regardless of sharing).

Given that option 2 seems to add a number of complexities, I favor option 1.
I agree that the isDerivedFrom relation should be used.

@dtarb I favor option 1 as well. So I'll go with option 1 for now.

@hyi and @dtarb - what is to keep someone from copying a resource, changing the author information, and then re-publishing?
I know there's use cases for copying resources, but I'm not sure we've fully thought this through.

@horsburgh Good point. Probably we need some kind of "agreement" language to at least inform users of our policy for copying a resource from others such as keeping original authors of the resource, etc. At a minimum, we have this "isDerivedFrom" source metadata element that is attached to the copied resource, so we could trace these kinds of misuse of copied resources if we need to. For now, I'll just implement it in the way that allows anybody to copy a resource he or she can see with an informational message to inform users of our copy policy. @horsburgh @dtarb I'll need you to come up with a formal policy to display to the users when they try to make a copy of a resource, though. Also let me know if you think I should hold up on implementation of this copy functionality until we spend some more time to think it through.

@hyi @horsburgh lets be efficient and try not hold up implementation by extended debate or complexity or generation of a new policy.
I recognize that it is almost impossible to protect against unethical copying.  It is possible for a user to download the files and do with them what they like, including re-uploading them into a new resource, losing isDerivedFrom metadata unless they specifically create a relationship.
Our policy is that we expect users to respect the license agreements that other users set for resources when they share them.  I would prefer not to have to have a new policy on copying, and hope that it is already covered by our terms of use https://www.hydroshare.org/terms-of-use/ that require ethical and legal behavior.  Copying something counter to a license agreement is unethical and illegal so if we find abuse we could act according to our current policy.
When a user asks to make copy of a resource, I suggest a pop up that says: "This resource has been provided with a .  In copying this resource you are ethically bound to respect the terms of this license."  If it is a creative commons license (which most will be as that is the default), then provide a link to the creative commons website describing the terms.  If it is one of the creative commons no derivs licenses (CC BY-ND or CC BY-NC-ND), the ND indicating no derivatives, then automated HydroShare copying should be disallowed.  The best implementation of this would be for the copy button to never be enabled for resources with these licenses.  Alternatively, the pop up should point out that the license on this resource does not allow derivative works, so HydroShare will not make a copy.  For all other licenses, allow a copy after a user has clicked Agree to seeing the license terms.
In making the copy all science metadata properties of the resource are copied (authors, abstract, keywords, resource specific metadata etc).  An isDerivedFrom relationship is created.  System metadata (comments, ratings, owners and access control information) is not copied.  Instead the resource defaults to private (just as any resource creation action), and the user performing the copy becomes the owner.  The user/owner is then responsible for further changes/sharing of the resource.
We also need a help page that explains copying @christinabandaragoda.  This should explain these rules.

@dtarb - by defining the "rules," you defined the policy. My comment above was simply to address the fact that we should not program a system that allows users to violate licenses specified when other users share resources. I think your ideas above cover it.
I personally like the idea of having a pop up that explains why something can't be copied because it would hopefully serve to increase awareness of these issues within the community of users.

I agree with @horsburgh that it has merit to show a pop up that explains why something cannot be copied even though this would involve a bit of more work on my part as opposed to not showing copy button at all if the resource's license does not allow derivation. @dtarb Want to confirm with you again which way you prefer. I have implemented not showing copy button at all if the resource cannot be copied, but if you agree it is better to have a popup that explains why it cannot be copied, I will change the user interface to achieve that, and if so, could you also suggest a language that explains why it cannot be copied in the popup?

I am willing to be practical in the implementation, but for me the ideal solution is to have the copy button greyed out/inactive when a user is not allowed to make a copy, with a hover over that explains that "The license for this resource does not permit copying."  This gets the message across, while at the same time not giving them a button followed by "You can not copy because the license for this resource does not permit copying."  Having a button that is active that results in a warning/error is not something I like.  This is probably more work than the solution you had in mind, but you did ask.

@dtarb OK, have implemented it in the way you describe above with a minor tweak: instead of show the hover over tooltip The license for this resource does not permit copying, it shows the tooltip as You cannot copy because the license for this resource does not permit copying.

Super.  Thanks.

@dtarb @horsburgh I have deployed this to playground for your testing. Please test resource copy functionality on playground and let me know any changes you want to make. I will finish adding unit tests and create a PR tomorrow.

@hyi - I tested on playground. It mostly looks OK. I made a copy of a resource. That worked fine. The "Source" element gets set correctly to point at the resource that the copy was derived from. I then changed the license to the "no derivitives" license, and I cannot make a copy. When I hover over the copy button, I get the correct message. This all looks good.
My only complaint is that when I can't make a copy, the copy button looks just like it did when it was active. It just doesn't do anything when I click on it.  And, if I don't hover over it just right or for the right amount of time, I don't get the hover text. So, can we have a gray version of the same icon that is displayed when copying is not an option to indicate that it is disabled?

@horsburgh Have updated playground to make the copy icon look grayed out when copy action is not allowed. Can you test on playground again to see if it looks good now?

If I am the owner of a resource that has a ND license, I should still be able to copy it.
I would prefer to soften the language that appears when a resource is copied to:
"Your new copy of the resource has been created. A Derived From Source metadata element has been added to this resource in the Related Resources tab below that links to the original resource. Please respect the terms of the license of the original resource and recognize the original authors as authors or contributors as appropriate in this derived resource. Note that this new copy is created as a private resource by default. If you want the new resource to be discoverable, you need to make it public or discoverable or publish it. "
The rationale may be that the changes may be sufficiently big that it is no longer appropriate or needed to retain the original author.  One use case is that I share a resource that is a DEM for a watershed.  I list myself as the author and indicate it was derived from the national elevation dataset.  I then share this with students in my class and ask them to delineate the watershed.  They make a copy then open the resource with CyberGIS that results in a large number of watershed delineation files in the resource.  I am not an author of any of these files, rather I just contributed the original DEM and they did the delineation work.  The best practice would be for them to leave me as a contributor and to enter their own names as authors.

@dtarb @horsburgh Playground has been updated with latest code changes that addressed all your testing comments. Feel free to take a look and let me know if you have any further comments to be addressed.

Testing today on candidate release 1.9.4 beta this looks to be all satisfactory, so closing.

authors: dtarb horsburgh hyi

repo: hydroshare/hydroshare


text:
Certain download buttons lack action/info when either filesize or resource size > 1GB

I happened upon the bug(s) regarding attempting to download resource bag files and/or individual resource files. There are two resources that will help me illustrate the bug(s). Here is the first:
https://www.hydroshare.org/resource/126bb28f05224636a8bc8b3d1bdad6b5/
This resource is storing one file of 1.6GB.  If I click the "Download All Contents as Zipped BagIt Archive" a page immediately loads with the message: 'File larger than 1GB cannot be downloaded directly via HTTP.Please download the large file via iRODS clients.' This is great; no problem here. However, if I try to click the download button next to the the individual file, then nothing very indicative happens for a while... By that I mean that 1) the informative page mentioned before never comes up, and 2) the browser does show a loading icon in place of the hydroshare icon in the tab, yet that is hardly noticeable. It just seemed like nothing was happening. No message, no alerts, no popups. It made me just want to click the button again... and again, just to make sure. That is a problem.
The same problem mentioned above can also be seen with the following resource, however I'm going to highlight the other problem with it as well:
https://www.hydroshare.org/resource/31f35199a2e548e29786182fbb3e2941/
I would expect, since the cummulative size of this resource is well over 1GB, that when clicking the "Download All Contents as Zipped BagIt Archive" button, I would get the same page immediately loading to inform me that the BagIt archive is over 1GB. However, that never happens. I did get a loading bar in the browser tab, but it remained for a long time until finally a page came up informing me of a '504 Gateway Time-out'. That is a problem. Another odd thing is that even if trying to simply download the 243.8 KB sized file, it takes a very long time (minutes). Luckily, I did not get the 504 error, but still.
I have no idea who to assign to this, so I'll leave that up to someone else.

@shawncrawley Thanks for reporting this. I am assigning it to myself and will take a look at this issue.

@dtarb @mjstealey @rayi113 I have added a popup "Download Alert" message box being displayed when the user clicks on Download button trying to download a file larger than 1GB as shown below:

Let me know if you want to modify the alert message to make it more informational for users so that I can incorporate changes before getting the fix into our code develop branch.

Let me know if you want to modify the alert message to make it more informational

@hyi - I think once we have proper pages.hydroshare.org documentation for exactly how to use iRODS clients we'll be able to update the text you have with links to the how-to information. Otherwise I think it looks fine.
For how-to's:

How to create an iRODS user account from your profile page
How to use Cyberduck to connect to your iRODS user account

For the "How to use Cyberduck to connect to your (HydroShare) iRODS user account"  how-to, there's a start here: https://pages.hydroshare.org/accessing-hydroshare-irods-from-a-windows-pc-or-mac/

@mjstealey @rayi113 @dtarb Have added the how-to link for Cyberduck and icommands link, so the popup notification looks like the following now:

After we have the how-to link for how to create iRODS user account in user space, I will add that link then. Let me know if there are any further suggestions. Will create a PR soon to address this issue as well as other large file download issues.

@shawncrawley The fix to this issue has been deployed to www. Could you test it out and see whether this issue has been fixed? If so, please close this issue, and if not, please add details on the issues and I will look into them further.

It looks good to me. Great work.

authors: hyi mjstealey rayi113 shawncrawley


text:
Direct file downloads in folder interface upgrade

On the pre-folder interface deployment it was possible to hover over a file download button and see the URL at the bottom of the browser and right click to  copy link location to build direct URL's to files in resources.  This functionality I used in building a web page for my class that used direct links to students work - see http://hydrology.usu.edu/dtarb/giswr/2016/students.html.  In the new folder interface currently deployed on beta (and scheduled for roll out next week) this functionality is no longer there.  One has to right click on a file to download, but cannot see what the direct download link is or copy the link location using browser functionality.  The links to direct fiels still work.  There is just no way to discover this capability.  I would prefer a simpler download button or link to click that retained this standard browser copy link and showing the URL functionality.

@dtarb, how about a right click menu item "Copy file URL"

I am OK with that, but it does seem that there used to be a more common web approach that handled things naturally as html links the way browsers commonly work.  Now you are rewriting some of this to work differently.  If either the icon used to show the file, or its name was just made into html hyperlink text with the URL linked you would not have to do the work of building a new right click item.

@dtarb - If there is a right click required already to get the URL, I'm not sure how this is really different. When I hover over the download link in the pre-folder version (the little circle with the down arrow), I do see the URL at the bottom of my browser. That would be the only thing that has been lost. You have to right click on the download button to get the link anyway.
I think if we added a "Copy Link" or "Copy file URL" on the right click on a file it would actually be better on the new interface because the right click would be on the file icon rather than the download button.

I am OK with either of these as a solution.  I do worry though that this is functionality that the browser used to do and now we are taking on the burden of having to do it ourselves for all browsers we support.

Checked on beta.

authors: dtarb horsburgh mauriel133


text:
Additional metdata can't be added in IE11

On the resource page, the additional metadata (+) button does not work.
010.zip

This has been fixed and can be tested at https://beta.hydroshare.org

authors: Maurier hydrocheck


text:
Add support for bulk spawn.

We can spawn multiple computation in parallel, at the same time.

The first question that popped in my mind while starting my review is: why do we need a special type to represent a collection of futures? Is there a way we could use Array or Buffer?

The first question that popped in my mind while starting my review is: why do we need a special type to represent a collection of futures? Is there a way we could use Array or Buffer?

We can, but that would be inefficient. We would essentially be needing escaping futures, and thus we have one memory allocation for each computation. Using bulk_spawn we only need one memory allocation for the entire frame.

authors: kyouko-taiga lucteo

repo: hylo-lang/hylo


text:
moving loadPixels() to after beginDraw() should resolve #87

I tested with 3.0, default renderer and P2D.

Fixes #87

@shiffman thanks again... this in fact sorted it. I owe High-Fives

authors: christophertino hype shiffman


text:
chore: add egg.json to packages and remove package.json

This PR adds egg.json to and removes package.json from all Deno packages, in prep for publishing to nest.land and deno.land.
TODO:

[ ]  Add automation for bumping versions
[ ] Add automation for changelog generation
[ ] Add automation to create git tags
[ ] Add quality checks as part of version bumping (git hooks and Makefiles?)

Closing because adapters are being split out in separate repos based on decision from #231 . egg.json will be added to each of those repos.


text:
[BUG] Hyperf\Coroutine\Exception\ChannelClosedException" not found

Execute the command and paste the result below.
Warning: Class "Hyperf\Coroutine\Exception\ChannelClosedException" not found in /home/asdf/DEV/swoole/hyperf-skeleton/vendor/hyperf/utils/src/Exception/ChannelClosedException.php on line 14
swoole bin/hyperf.php start
Warning: Class "Hyperf\Coroutine\Exception\ChannelClosedException" not found in /home/asdf/DEV/swoole/hyperf-skeleton/vendor/hyperf/utils/src/Exception/ChannelClosedException.php on line 14

Fatal error: Uncaught Error: Class "Hyperf\Coroutine\Exception\ChannelClosedException" not found in /home/asdf/DEV/swoole/hyperf-skeleton/vendor/hyperf/utils/src/Exception/ChannelClosedException.php:20
Stack trace:
#0 /home/asdf/DEV/swoole/hyperf-skeleton/vendor/composer/ClassLoader.php(571): include()
#1 /home/asdf/DEV/swoole/hyperf-skeleton/vendor/composer/ClassLoader.php(428): Composer\Autoload\includeFile('/home/asdf/DEV/...')
#2 [internal function]: Composer\Autoload\ClassLoader->loadClass('Hyperf\\Utils\\Ex...')
#3 /home/asdf/DEV/swoole/hyperf-skeleton/vendor/hyperf/utils/src/Functions.php(541): class_exists('Hyperf\\Utils\\Ex...')
#4 /home/asdf/DEV/swoole/hyperf-skeleton/vendor/composer/autoload_real.php(55): require('/home/asdf/DEV/...')
#5 /home/asdf/DEV/swoole/hyperf-skeleton/vendor/composer/autoload_real.php(38): composerRequire9f630ee78015a795c6afb270c0c614e2('bbeb7603826cb92...', '/home/asdf/DEV/...')
#6 /home/asdf/DEV/swoole/hyperf-skeleton/vendor/autoload.php(12): ComposerAutoloaderInit9f630ee78015a795c6afb270c0c614e2::getLoader()
#7 /home/asdf/DEV/swoole/hyperf-skeleton/bin/hyperf.php(20): require('/home/asdf/DEV/...')
#8 {main}
  thrown in /home/asdf/DEV/swoole/hyperf-skeleton/vendor/hyperf/utils/src/Exception/ChannelClosedException.php on line 20

按照官网教程安装 使用swool-cli 5.0

参考这个 issue 中的解决办法：https://github.com/hyperf/hyperf/issues/5677#issuecomment-1524500969

authors: her-cat nullNullNone

repo: hyperf/hyperf


text:
如何使用WEBSOCKET_OPCODE_BINARY传输二进制？

如何才能使用WEBSOCKET_OPCODE_BINARY，我会通过二进制的再gzip压缩我的消息进行传输

我记得Swoole可以直接配置吧

我记得Swoole可以直接配置吧

$server->push((int)$v, gzencode($message),WEBSOCKET_OPCODE_BINARY);
hyperf的WebsocketSever  我是这样用的，socket.io 没找到类似方式
如何配置？

authors: limingxinleo nicett

repo: hyperf/socketio-server


text:
Use godep to manage dependencies

https://github.com/tools/godep

Hi @achanda, Good idea, but looks you committed your whole local workspace

@gnawux I think that's how it works. Godep saves a snapshot of the deps in the Godep folder. This can be committed in git and distributed. Thus, even if the deps change and it breaks the app, we always have a local working copy in source control.

Hello @achanda,
Since there is also a third-party library in the code, I will not merge your pull request.  However, your idea is really great to build this dependency.  I did the work based on your idea to use godep to make this work.
Thanks again.
Regards,
-Simon

authors: achanda carmark gnawux

repo: hyperhq/hyper


text:
Uri::path() and and Uri::path_and_query() have confusing semantics

path() returns &str which means it is never missing. Then why does path_and_query() return Option<&str> instead of &str? In other words, if the path is never missing then how can path_and_query() ever be missing? Under what circumstances would path_and_query() return None?
path() returns * when the URI is *. This seems really error prone. Regardless, what does path_and_query() return when path() returns *? This should be documented.
I would expect that url.path_and_query().unwrap().starts_with(url.path()) never panics and is always true, and that url.path_and_query().unwrap() == url.path() + url.query().map(|q| "?" + q) never panics and is always true. That is, I would expect that url.path_and_query() never returns None.

If the return type of Uri::path_and_query is intended to explicitly indicate when the query part is not present, I might expect it to return something like (&str, Option<&str>), where the path part is always returned and the query part is optional?

@briansmith you're not wrong...
The various APIs evolved over time. There should definitely be a re-evaluation of how everything works.
The problem is the HTTP uri component always includes a path except when * or for CONNECT requests. This means that for 99% of the use cases, there will always be a path. As such, having uri.path() return Option is going to be pretty annoying.
The question is if there is a way to have an "easy" API for the 99% cases while still exposing the subtleties when you care.
One option (without an opinion of quality of the option) would be to switch Uri::path to panic if there isn't a path component present.
Also, currently path_and_query does not return a &str, but a ref to a PathAndQuery. It is None in the two cases above and Some otherwise.

A question arises in https://github.com/carllerche/h2/pull/228#issuecomment-370584403:
Should a URI that does not include a path default to /?

authors: briansmith carllerche hawkw

repo: hyperium/http


text:
feat(service): add adapter for tower services

This adds

TowerToHyperService which converts a tower service into a hyper service.
Builder::serve_connection_tower and Builder::serve_connection_with_upgrades_tower for serving a tower service directly. The adapter will be applied internally.
An example showing how to use it since there was previously only a client example.

I wanna do some more thinking about how to best do this. So I'll close this PR for now and submit PRs with smaller pieces.

repo: hyperium/hyper-util


text:
fix(http): handle axios error code UNABLE_TO_VERIFY_LEAF_SIGNATURE

Monika Pull Request (PR)
This PR resolves #1266
What feature/issue does this PR add

Handle axios error code UNABLE_TO_VERIFY_LEAF_SIGNATURE

How did you implement / how did you fix it

Map error code UNABLE_TO_VERIFY_LEAF_SIGNATURE to error code 27
Map error code UNABLE_TO_VERIFY_LEAF_SIGNATURE to error codename ELEAFSIGNATURE
Add documentation for error code 27

How to test

Create monika.yml config below

probes:
  - id: 'Example'
    requests:
      - url: https://kemendesa.go.id/

npm run start -- -r 1
On 5th failure, expect to see warning log id:Example ERROR: EINVALIDLEAFSIGN: Unable to verify the first / leaf certificate., NOTIF: Service probably down

CMIIW, is the UNABLE_TO_VERIFY_LEAF_SIGNATURE the same as the CERT_HAS_EXPIRED?
If by chance it is different, maybe you could add CERT_HAS_EXPIRED as well.

On second thought, the CERT_HAS_EXPIRED happens if you don't use --ignoreInvalidTLS, but UNABLE_TO_VERIFY_LEAF_SIGNATURE still happens. Carry on then

authors: dennypradipta syamsudotdev

repo: hyperjumptech/monika


text:
feature suggestion:  configurable extra option for docker run command

current latest minifabric can set additional environment variable to docker containers via settings section in spec.yaml
but it doesn't have a way to set additional option to docker run command.
then, I cannot specify '--restart" nor "--log-opt" policies  even I want to use minifabric  in long run use cases.
first I think adding  '--restart=always'  by hard coding style into playbook, but logging policy may be different in each case.
then, I think it is good to have configurable extra docker option in spec.yaml as described bellow snippet.
it may be single(choice 1, same option to all containers), or multiple (choice 2, different option for each type).
I'm not sure which choice is preferable for minifabric maintainers. any suggestion?
# ansible playbooks
- name: Start xxx
  command: >-
    docker run -d ... --name {{item.fullname}} ... {{ extra_docker_option }} ... # append extra docker option.

# spec.yaml
# choice 1: single;  add the same option to all containers(ca,peer,orderer)
fabric:
  ca:
  peers:
  orderers:
  settings:
  extra_docker_option: " --restart=always  --log-opt max_size=Xmb --log-opt max_file=N " # new key

# choice 2: different option in each type, same style as 'settings( for environment variable)'.
fabric:
  :
  settings:
  extra_docker_option:
     ca:       " --restart=always  --log-opt max_size=3mb  --log-opt max_file=3"
     peers:    " --restart=always  --log-opt max_size=10mb --log-opt max_file=10"
     orderers: " --restart=always  --log-opt max_size=5mb  --log-opt max_file=5"

Either is fine, prefer choice1. Thanks.

Not sure we need to call it extra docker option, can we call it container options?

@litong01 ok, I take choice 1, and change name to container_options.

repo: hyperledger-labs/minifabric


text:
Repo Management Script

This contains a script for creating scaffolding for a new plugin, managing global dependencies and configuration and common development files.
It's quite a bit simpler than my first attempt. It still contains a new folder where the global files are stored plugin_globals, but it doesn't require any additional files from the plugins themselves. It will combine the dependencies and conflicts will be overridden by the global ones. Config sections will just be overriden by the globals (no merging).
There isn't support for plugins to have extra sections currently. They will just be removed. I'm not sure if we want to let plugins be able to do this anyway. Will make things harder to maintain.
Files in the existing plugins got adjusted slightly. I think these changes could be safely ignored.

I moved main to a function and also added more documentation and tried to improve maintainability with some code changes.

repo: hyperledger/aries-acapy-plugins


text:
feat: Presentation Exchange Format / PPV2

This PR puts the PEX PPV2 code into 0.3.0-pre.
Note that there is still some development to do but this is the correct time to merge

I will reopen this PR when the JsonLd ICV2 branch is merged as there are many duplicate commits across the two PRs.

repo: hyperledger/aries-framework-javascript


text:
docs: updated macos install script

Libindy should now be installable for macOS users by using brew. This should make the setup easier for beginners :).
Signed-off-by: Berend Sliedrecht berend@animo.id

Codecov Report

Merging #328 (b63fbb1) into main (9859db1) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #328   +/-   ##
=======================================
  Coverage   88.28%   88.28%
=======================================
  Files         220      220
  Lines        4004     4005    +1
  Branches      465      448   -17
=======================================
+ Hits         3535     3536    +1
  Misses        469      469

Impacted Files
Coverage Δ

src/modules/proofs/index.ts
100.00% <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 9859db1...b63fbb1. Read the comment docs.