Changed "/" to DIRECTORY_SEPARATOR

Makes path helpers usable in Windows servers. Fixes issue #8913.

:-1: from me.

Makes path helpers usable in Windows servers.

If you're doing this, all you're doing is breaking your code that used to work on linux.

If you're doing the things that were described in that other issue.

If you're doing this, all you're doing is breaking your code that used to work on linux.

Please support with some examples/facts.
You can thumb-down it all you want, it doesn't create any problems on Linux and Taylor has already acknowledged the issue is an actual issue. So let's focus on getting the issue fixed, not outright rejecting everything you don't see as fit.

DIRECTORY_SEPARATOR in linux env will evaluate to / so I don't see how this would change anything on linux.

Also, let's be clear with what can happen now. On Windows environments you can easily explode() the output returned by the helpers, before you would need to explode, then do it again to the parts for the other separator.
This change allows for far better coding to be done. Any challenges to it should be met with facts/test cases. That way they can be addressed and solved. Plenty of facts exist as to why this constant should be used, none as to why "/" should be other than "it works in most situations most of the time so it isn't a problem."

On the note of storing paths being an edge-case, it really depends on how people have built their systems. A lot of the time, things like image/file attachment to models will store some path in the DB. If they store the path pulled from one of these helpers as a starting point and are under Windows, then there would be a problem.
That is the most likely point of concern (other than temp caching of paths in redis or something.) But, once again, all Windows focused which is already a small subset of users, especially in production.

:-1: from me, because @Garbee. Just kidding mate :-)

Dude stop sounding like Flyn. It will only bring pain and anguish.
On Jul 15, 2015 5:38 PM, "Samuel Georges"<EMAIL_ADDRESS>wrote:

[image: :-1:] from me, because @Garbee https://github.com/Garbee. Just
kidding mate :-)
—
Reply to this email directly or view it on GitHub
https://github.com/laravel/framework/pull/8916#issuecomment-121757747.

:thumbsdown:

This stuff isn't changing again, while I don't really like it either, that's what we have now.

@GrahamCampbell Why don't you like it exactly? It seems odd a framework that prides itself on consistency and ease of use has a collaborator who evangelizes things that make it inconsistent and harder to use.
You also never supported your claims on it breaking code that worked on Linux boxes. So if you could revisit that as well it would be great. I'd love to know how it would break things, yet has been released to stable and we haven't heard a word of it.

It seems odd a framework that prides itself on consistency and ease of use has a collaborator who evangelizes things that make it inconsistent and harder to use.

I think it's more consistent to always use linux style separators.

I think it is more consistent to always use the constants PHP provides for a reason.
/imbackbitches

Hi Rob. :0

Hi Rob. :)


Error when reading OpanApi 3.1.0 document

Is your feature request related to a problem? Please describe.
I parsed an OpenApi 3.1.0 document with the Readers and got this error:
[Exception] One or more errors occurred. (OpenAPI specification version '3.1.0' is not supported.)

Describe the solution you'd like
When will it support 3.1.0?

@elize-vdr this work is currently ongoing and will go live when we release v2.0.

Any indication when release v2.0 will be released please? Will it still be in 2023 or 2024?

Hi @elize-vdr , this work has been planned for CY23H1

Thank you so much!


Load default roots.pem in PHP via grpc_set_ssl_roots_override_callback

Attempts to fixs this #5711.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

Can you please send this PR without all of the irrelevant commits? All of the changes for this PR are in the second-to-last commit.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok to test.

@murgatroid99 I have cleaned up all of the extra commits.

LGTM @murgatroid99 is the user of gpr_malloc appropriate here?

I think so. This change is copied pretty much verbatim from the corresponding Node change.


New version: Nvidia.app.Beta version <IP_ADDRESS>

Checklist for Pull Requests

[X] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

[Policy] Blocking-Issue


suddenly unable to read .h5ad file

there were no problem reading .h5ad file but all of the sudden, this error came up and unable to read .h5ad files,,
someone have any idea??
Thank you!!

It was probably written by a newer anndata version. Try updating anndata, maybe that helps.
We have a PR that improves error messages in this regard. @ivirshup could you please merge or adapt and then merge #734?

Thanks for your help! I have updated anndata and h5py and re-start my jupyter noteobok and it works!!


API docs outdated

/docs/api.md is outdated.
Actual endpoints
defer wg.Done()
	router := mux.NewRouter().StrictSlash(true)
	router.HandleFunc("/", func(rw http.ResponseWriter, r *http.Request) { rw.Write([]byte("Hello")) })
	router.HandleFunc("/v1/dashboard", a.handleDashboard).Queries("filter", "{filter}")
	router.HandleFunc("/v1/dashboard", a.handleDashboard)
	fmt.Println("Start serving on port 10000")
	http.ListenAndServe(":10000", router)

Example output
{"clients":[{"name":"eth2","count":505},{"name":"geth","count":327},{"name":"tmp","count":320},{"name":"go-opera","count":33},{"name":"bor","count":9},{"name":"quai","count":7},{"name":"besu","count":7},{"name":"erigon","count":5},{"name":"coregeth","count":5},{"name":"go-photon","count":4},{"name":"getd","count":2},{"name":"thor","count":1},{"name":"ronin","count":1},{"name":"multigeth","count":1},{"name":"gubiq","count":1},{"name":"godin","count":1},{"name":"gexp","count":1},{"name":"drkchain","count":1}],"languages":[{"name":"","count":825},{"name":"go","count":399},{"name":"java","count":7}],"operatingSystems":[{"name":"","count":825},{"name":"linux","count":403},{"name":"windows","count":3}],"versions":null,"countries":[{"name":"United States","count":360},{"name":"Germany","count":273},{"name":"","count":119},{"name":"Finland","count":103},{"name":"Netherlands","count":66},{"name":"Canada","count":43},{"name":"France","count":42},{"name":"Singapore","count":27},{"name":"United Kingdom","count":25},{"name":"Russia","count":25},{"name":"Japan","count":24},{"name":"Hong Kong","count":16},{"name":"Ireland","count":13},{"name":"Australia","count":13},{"name":"Austria","count":9},{"name":"South Korea","count":8},{"name":"Switzerland","count":6},{"name":"Ukraine","count":5},{"name":"China","count":5},{"name":"Turkey","count":4},{"name":"Brazil","count":4},{"name":"Vietnam","count":3},{"name":"Sweden","count":3},{"name":"Norway","count":3},{"name":"Italy","count":3},{"name":"Bulgaria","count":3},{"name":"United Arab Emirates","count":2},{"name":"Thailand","count":2},{"name":"Taiwan","count":2},{"name":"Spain","count":2},{"name":"Philippines","count":2},{"name":"New Zealand","count":2},{"name":"Chile","count":2},{"name":"South Africa","count":1},{"name":"Seychelles","count":1},{"name":"Portugal","count":1},{"name":"Malaysia","count":1},{"name":"Lithuania","count":1},{"name":"Kyrgyzstan","count":1},{"name":"India","count":1},{"name":"Egypt","count":1},{"name":"Czechia","count":1},{"name":"Croatia","count":1},{"name":"Belgium","count":1},{"name":"Argentina","count":1}]}

That document was done for a different reason, which we decided to drop, we should definitely update it with the existing api, sorry for this


Added no compare neg zero lint rule

Ref https://github.com/facebookexperimental/rome/issues/94

Thank you! This is great.


Intermittent Error "Response code is 200"

Hi, since about 24 hours, all my scripts using this scraper are giving the error "Response code is 200" about once every 4 tries. I'm using residential rotating proxies, and even if I manully set my script to use one single proxy for testing, it will work about 3-4 times in a row, than throw this error, and if I try again 2 sec later with the same proxy it's gonna work again, so it's not because the proxies are banned.
Anyone had this issue before ?
Here's the full error output:
InstagramScraper\Exception\InstagramException: Response code is 200. Body:
Something went wrong. Please report issue. in /home/Instagram-Scrapper/vendor/raiym/instagram-php-scraper/src/InstagramScraper/Instagram.php:385 Stack trace: #0 /var/www/vhosts/XXXXXX.com/httpdocs/test3.php(34): InstagramScraper\Instagram->getAccount() #1 {main}

Which residential proxies are u using?

Does it matter ?

Same problem here.

same issue here

Same issue.

any update?

Same issue using ProtonVPN and not the built-in proxy support.

Same issue here with or without proxy...

Same issue here

Same issue there.
I have 2 web site for analyze IG stats, and starting from today no a single successfully request is done, no matter if I use proxies (like I always do) or not.
waiting for solution!

It has probably something to do with a change that broke the /?__a=1 jquery variant of Instagram pages. It still works within the browser but not if you try to get the contents of that page via curl or a simple file_get_contents(). It redirects you to the login page. So my guess it has something to do with cookies or sessions...

Same issue.

here too

the same issue ( any news?

Does anybody here use the instagram-php-scraper with phpAnalyzer - Social Media Analytics Statistics Tool from CodeCanyon like me?

+1

it seems like it works only with login and password

Any news?

+1

Same issue here, any news?
Fatal error: Uncaught InstagramScraper\Exception\InstagramException: Response code is 200. Body: <!DOCTYPE html> <html lang="en" class="no-js not-logged-in client-root touch"> <head> <meta charset="utf-8"> <meta http-equiv="X-UA-Compatible" content="IE=edge"> <title> Login • Instagram </title> <meta name="robots" content="noimageindex, noarchive"> <meta name="apple-mobile-web-app-status-bar-style" content="default"> <meta name="mobile-web-app-capable" content="yes"> <meta name="theme-color" content="#ffffff"> <meta id="viewport" name="viewport" content="width=device-width, initial-scale=1, minimum-scale=1, maximum-scale=1, viewport-fit=cover"> <link rel="manifest" href="/data/manifest.json"> <link rel="preload" href="/static/bundles/es6/ConsumerUICommons.css/26b4bcda12c8.css" as="style" type="text/css" crossorigin="anonymous" /> <link rel="preload" href="/static/bundles/es6/ConsumerAsyncCommons.css/638f1bd337c8.css" as="style in /var/www/html/vendor/raiym/instagram-php-scraper/src/InstagramScraper/Instagram.php on line 431

Same problem, any news ?

No-one able to solve this issue?
Is there a possible solution, or this script is dead?

Hello?! Is there anybody out there?
(cit. Pink Floyd)

Same issue :(
In local works fine, but in server does not work

instagtam redirecting some IP's to login page, and rather than getting profile page you get another one.
Sent from Blue
On 11 Jun 2020, 04:06, at 04:06, Xavier Ojeda<EMAIL_ADDRESS>wrote:

Same issue :(
In local works fine, but in server does not work
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/postaddictme/instagram-php-scraper/issues/685#issuecomment-642377927

Tested in various servers and it seems o be a general problem, in localhost it works perfectly, but in server environments doesn't work at all, even with proxies, the majority of them are getting the wrong response from the cURL @raiym you should change the scraping url because Instagram is redirecting the requests to the login page, so the scraper is scraping the login page and not the user page, maybe try to change the user agent or the mobile url. or maybe if the cUrl gets the wrong scrap try to make the try again until you get the page. In theory your scrapper is working just in local environment, because Instagram recognize that the call is local.


Add more information regarding the references

We've had some questions about the reference origins.  We should probably add some links to file and bash scripts where applicable with regards to reference acquisition.

Resolved with #2


refactor: extract UI components and make it reusable to other use cases

I found this buffer UI and the way to interact with the UI is very desired for other scenarios like docker or any other CLI tools.
Hope there's standalone version of this ui components which can easily adapt to other use cases, and be used by others plugin authors

Interesting idea, haven't thought along those lines but that should be feasible.
The code already is a bit split up into different responsibilities and it should handle any json data.

i think it is possible but there are already plugins that provide some of the UI principles in this one.
as this is open source code, anyone can copy the logic freely, no @Ramilito ?

Yeah some of the UI things are already solved, what I took from this though is basically extract the table.lua and some of the buffer.lua stuff.
Being able to take json data and just get out a column based table basically.


The kind 'Template' is not recognized by oc-4.17.0

This is caution message until the issue in oc 4.17.0 is fixed. https://redhat-external.slack.com/archives/C075PHEFZKQ/p1728656947447139

This is docs so merging, but next time create PR on branch of main repo not of fork.


Cannot access member XXX of type "object"

Environment data

Language Server version: pylance 2021.1.2
OS and version: macOS 22.1
Python version (& distribution if applicable, e.g. Anaconda): pyenv 3.9.1

Actual behaviour
The following snippet of code
done_commands = await asyncio.gather(*commands)
if not all([cmd.succeeded() for cmd in done_commands]):
    for cmd in done_commands:
        if not cmd.succeeded():
            raise ArchonError(f"Failed sending line {cmd.raw!r}")

raises errors Cannot access member XXX of type "object" on .raw and .succeeded(). commands is a List[ArchonCommand], and done_commands is correctly assigned a type Tuple[ArchonCommand]. In the loop, cmd is correctly assigned type ArchonCommand. The class ArchonCommand does have the members .raw and .succeeded.

Could you please provide a full, self-contained source sample? I don't know where the symbol ArchonCommand is being imported from. Also, please specify whether you're using type stubs for this package.

Could you please provide a full, self-contained source sample? I don't know where the symbol ArchonCommand is being imported from. Also, please specify whether you're using type stubs for this package.

This seems to be a complete example that causes the problem.
import asyncio
import sys

if sys.version_info >= (3, 9):
    FutureType = asyncio.Future["ArchonCommand"]
else:
    FutureType = asyncio.Future

class ArchonCommand(FutureType):
    def __init__(self, cmd_str: str):
        super().__init__()
        self.cmd_str = cmd_str
        self.set_result(self)

    @property
    def raw(self):
        return self.cmd_str

    def succeeded(self):
        return True

async def main():
    commands: list[ArchonCommand] = []
    commands += [ArchonCommand("hello"), ArchonCommand("world")]
    done_commands = await asyncio.gather(*commands)
    if not all([cmd.succeeded() for cmd in done_commands]):
        for cmd in done_commands:
            if not cmd.succeeded():
                raise ValueError(f"Failed sending line {cmd.raw!r}")

Apparently the issue is with
FutureType = asyncio.Future["ArchonCommand"]

If I do class ArchonCommand(asyncio.Future["ArchonCommand"]): then pyright doesn't issue any error.

This seems to be a complete example that causes the problem.
import asyncio
import sys

if sys.version_info >= (3, 9):
    FutureType = asyncio.Future["ArchonCommand"]
else:
    FutureType = asyncio.Future

class ArchonCommand(FutureType):
    def __init__(self, cmd_str: str):
        super().__init__()
        self.cmd_str = cmd_str
        self.set_result(self)

    @property
    def raw(self):
        return self.cmd_str

    def succeeded(self):
        return True

async def main():
    commands: list[ArchonCommand] = []
    commands += [ArchonCommand("hello"), ArchonCommand("world")]
    done_commands = await asyncio.gather(*commands)
    if not all([cmd.succeeded() for cmd in done_commands]):
        for cmd in done_commands:
            if not cmd.succeeded():
                raise ValueError(f"Failed sending line {cmd.raw!r}")

Apparently the issue is with
FutureType = asyncio.Future["ArchonCommand"]

If I do class ArchonCommand(asyncio.Future["ArchonCommand"]): then pyright doesn't issue any error.

The problem here is that the type alias FutureType refers to the type ArchonCommand which is defined in terms of FutureType. You have a cyclical definition that pyright is not able to resolve. I confirmed that mypy (another type checker) also emits an error in this case.
Pyright does have some special-case logic to deal with cyclical definitions in a limited form — in particular, if the cycle is contained within the class declaration itself. That's why it works when you do class ArchonCommand(asyncio.Future["ArchonCommand"]):.
It's unfortunate that versions of Python prior to 3.9 generate a runtime exception when you subscript the asyncio.Future type.
Here are some other potential workarounds:

Use asyncio.Future[Any]
Consider ways to refactor your code so ArchonCommand doesn't derive from a future of itself

The problem here is that the type alias FutureType refers to the type ArchonCommand which is defined in terms of FutureType. You have a cyclical definition that pyright is not able to resolve. I confirmed that mypy (another type checker) also emits an error in this case.
Pyright does have some special-case logic to deal with cyclical definitions in a limited form — in particular, if the cycle is contained within the class declaration itself. That's why it works when you do class ArchonCommand(asyncio.Future["ArchonCommand"]):.
It's unfortunate that versions of Python prior to 3.9 generate a runtime exception when you subscript the asyncio.Future type.
Here are some other potential workarounds:

Use asyncio.Future[Any]
Consider ways to refactor your code so ArchonCommand doesn't derive from a future of itself

Hi @erictraut,
Thanks for the explanation. Using asyncio.Future[Any] certainly does work and it seems it doesn't prevent me from annotating output variables with tuple[ArchonCommand]. So, while it breaks the automatic type inference, it's a workaround until I can restrict the version to 3.9+.

Hi @erictraut,
Thanks for the explanation. Using asyncio.Future[Any] certainly does work and it seems it doesn't prevent me from annotating output variables with tuple[ArchonCommand]. So, while it breaks the automatic type inference, it's a workaround until I can restrict the version to 3.9+.

This issue has been open with the "waiting for info" label for over a month, so I'm going to close it. I don't know if there's any work to be done here.

Yes, sorry, this could have been closed then, it was already answered.


Issues since update - 1.3.1

I ran NPM update to update the plugin to v1.3.1. This throws an error when I run npm start, see below:

<EMAIL_ADDRESS>start /Users/mike/Documents/Design Work/Design work/Fort Productions/Website/build/templates
gulp
module.js:557
throw err;
^
Error: Cannot find module 'vinyl'
at Function.Module._resolveFilename (module.js:555:15)
at Function.Module._load (module.js:482:25)
at Module.require (module.js:604:17)
at require (internal/module.js:11:18)
at Object. (/usr/local/lib/node_modules/gulp/node_modules/gulp-util/index.js:2:9)
at Module._compile (module.js:660:30)
at Object.Module._extensions..js (module.js:671:10)
at Module.load (module.js:573:32)
at tryModuleLoad (module.js:513:12)
at Function.Module._load (module.js:505:3)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: gulp
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      /Users/mike/.npm/_logs/2018-03-07T21_07_47_992Z-debug.log`

I have a live version of the site running v1.0 of the plugin and that has no issues.

This looks like Gulp/npm error. Try to delete all node modules and reinstall them. A command sequence like this would do:
npm cache clean
rm -Rf node_modules/
npm install
Also check this and this solution.


Sync ZCC db with ZVM after release fcps.

After deleting BFV VMs, their consumed fcps are unreserved in ZCC db, however, the state of those fcps are still 'Active'. Now calling a sync_db_with_zvm() to ensure the state to be updated to 'free'.

@LiJunBJZhu a reminder to fix the PEP8 error.

@LiJunBJZhu a reminder to fix the PEP8 error.

Resolved.


WIP feat: Stepper

Closes #805.
Description TBA after finishing up component.

@epr3  Thanks for working on this. However I'm not sure if this component is really needed. From what I can see it's closely resembles the Tab component, and can be easily build with Tab.

@zernonia Yes, I agree. I've seen most implementations follow a Tab-like structure.
I was thinking of making this component different than the Tabs component so:

adding state data attributes to permit a more granular styling (e.g. for inactive, active and completed states)
a linear property to limit navigation and focus and a Title and Description component following the implementation of the Dialog component for accessibility (aka describing the current step)
this component would not have had a Content component

I referenced this and this for the features.
I would have used a nav element and marked it with aria-label="progress" which in turn would have wrapped an ul if linear: false or an ol tag if true.
Let me know your thoughts! You can close the PR if the component is overlapping the Tabs component in usage.
Thank you! 👍🏻

@epr3 Thanks for your amazing work ☺
@zernonia I Think that the approche of @epr3 is great for steps than hack Tabs to look like Steps ? Even Nuxt UI 3 wait for steppers to Land in Radix-vue
Thank you all for your great work

Thanks for the explanation @epr3 ! If that's the case let's continue on with this component! 🚀
Thanks @Celibioux  for the input, yeah I can see some demands for this component 🤩

@epr3 Great Work 😎

Thanks for create working on this component @epr3 !! Just suggestion regarding some implementation.
Also, as there's no WAI-ARIA spec for Stepper, I think we need to think about how to inform user properly (Screen reader/Voiceover mode). Currently the a11y is not well covered yet, especially informing how many steps there were.

@zernonia Thank you very much for the feedback! I can think of adding a visibility hidden div with aria-live="polite" to the root with the text of Step 1 of 5 or something like that for announcing the total number of steps and the current step. Could you let me know if there any other elements that should be added?

Thanks for create working on this component @epr3 !! Just suggestion regarding some implementation.
Also, as there's no WAI-ARIA spec for Stepper, I think we need to think about how to inform user properly (Screen reader/Voiceover mode). Currently the a11y is not well covered yet, especially informing how many steps there were.

@zernonia Thank you very much for the feedback! I can think of adding a visibility hidden div with aria-live="polite" to the root with the text of Step 1 of 5 or something like that for announcing the total number of steps and the current step. Could you let me know if there are any other elements that should be added?

This is not a bad suggestion! I was looking into the having role tablist, tab, but afraid that without proper tabpanel it would causes bad a11y. Let's go with aria-live announcer 😁


[Keyvault] Fix code coverage for node tests

Followup PR of https://github.com/Azure/azure-sdk-for-js/pull/6609#issuecomment-569863330

@HarshaNalluru There is an overlap between this PR and #6719
Please rebase/merge after #6719 is merged before merging this PR


Minor changes to filenames

Main.html didn't mean anything
Replaced .HTML with .html because linux is case sensitive and I am habituated to use .html rather than .HTML. It also solves the inconsistency problem - some files have .html extention and some have .HTML

Need to fix Register.html not found message


fix minor issues in the documentation in v2.0.0

This test PR is to merge the fixes for the documentation into v2.0.0-beta branch

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


fix: handle 'Integrated Security' token with both Npgsql 7.x and 8.x

Close #695

Codecov Report
Attention: 7 lines in your changes are missing coverage. Please review.

Comparison is base (2c06a40) 67.59% compared to head (e48d5fc) 23.37%.

Files
Patch %
Lines

...ore/Rewriting/Implementation/PostgresqlRewriter.cs
0.00%
4 Missing :warning:

DubUrl.Core/Rewriting/Tokening/Specificator.cs
50.00%
1 Missing :warning:

...rl.Core/Rewriting/Tokening/SpecificatorStraight.cs
0.00%
1 Missing :warning:

...l.Core/Rewriting/Tokening/SpecificatorUnchecked.cs
0.00%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #696       +/-   ##
===========================================
- Coverage   67.59%   23.37%   -44.22%
===========================================
  Files         238      238
  Lines        3027     3029        +2
  Branches      526      527        +1
===========================================
- Hits         2046      708     -1338
- Misses        849     2263     +1414
+ Partials      132       58       -74

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


hooks.js handle not called in in run preview, only run dev

Describe the bug
in the documentation it states the the handle hook will be called before each call to an end point.
when running an app usiing npm run dev, the handle function is called as expected. Once the same app is npm run build and then npm run preview, the site that is generated does not call the handle hook when loading a page with a page endpoint.
Sometimes the site doesnt appear to make any network calls at all, when calling a page with a page endpoint. Sometimes it does.
So to be clear, I have made a repo that shows that pageendpoint is not being called in "preview" mode at all. If this is desired behaviour, then I will continue to work on the repo to try and produce the behaviour where __data.json is being called without hooks/handle being called first
Reproduction
https://gitlab.com/christopherreay/sveltekit_testHookPageEndpoint/-/commit/6a99222100825eb0080639b75ae55332ee90997a
Logs
## `npm run dev`

HOOKS: handle:  Request {
  size: 0,
  ... AS EXPECTED
}
ENDPOINT /index.js, get: {
  request: Request {
    size: 0,
    ... AS EXPECTED
  }
}

## `npm run preview`
bash
> Using @sveltejs/adapter-auto
  Could not detect a supported production environment. See https://kit.svelte.dev/docs/adapters to learn how to configure your app to run on the platform of your choosing
  ✔ done
lancaster_app_user@mcsSustenance:~/context/lancasterApp/sveltekit_testHookPreview$ npm run preview

><EMAIL_ADDRESS>preview
> svelte-kit preview --port=60000

  SvelteKit v1.0.0-next.314

  local:   http://localhost:60000
  network: not exposed
  network: not exposed
  network: not exposed

  Use --host to expose server to other devices on this network

When localhost:60000/ is loaded in the browser window (which is indeed very very fast :) ), there is no output in the console. Either in the browser console, or on the server side console. I would expect there to be output on the server console. In fact the front end does not make any network calls at all. Sometimes my app does make calls to __data.json, but it does not make calls to hooks/handle

### System Info

```shell
System:
    OS: Linux 5.4 Ubuntu 20.04.1 LTS (Focal Fossa)
    CPU: (1) x64 Intel(R) Xeon(R) CPU E5-2630L v2 @ 2.40GHz
    Memory: 147.52 MB / 981.37 MB
    Container: Yes
    Shell: 5.0.17 - /bin/bash
  Binaries:
    Node: 17.8.0 - ~/.nvm/versions/node/v17.8.0/bin/node
    npm: 8.5.5 - ~/.nvm/versions/node/v17.8.0/bin/npm
  npmPackages:
    @sveltejs/adapter-auto: next => 1.0.0-next.34
    @sveltejs/kit: next => 1.0.0-next.314
    svelte: ^3.46.0 => 3.47.0

Severity
blocking all usage of SvelteKit
Additional Information
Sorry for the high priority, but I am unable to deploy my app, and I have tried speaking to people in the discord. No one has heard of anything like it.
I have tried using the -node adapter and building and running as production, same results.
When npm run build is called, the end points and the handle hook are called for each page, during SSR... however... given the documentation, there seems to be no reason why an endpoint wouldnt be called during page load of an already built site. In fact, I dont understand why the SSR pages are built during npm build
I can understand that, but it doesnt it at all into what I think that sveltekit is supposed to be doing.
I have read the source code, and I do not see how it is possible for __data.json to be called for a page endpoint without hooks/handle being called as well, which is what is happening on my full app.
If all this behaviour in this repo is doing what you expect it to (which will be very interesting to me), then I will work in more detail on what is happening in my full app, where __data.json is being called without hooks/handle being called before it

It sounds like you're hitting pre-rendered versions of those pages/endpoints, which will not result in handle being called at runtime, as described in https://kit.svelte.dev/docs/hooks#handle

So that would make sense if:
pageendpoints are a special kind of endpoint that are only called during SSR - PreRender
unless, the pageEndpoint is dynamic? in which case they must be called and memoised for each specific URL?

in the documentation you linked to it says:

It receives an event object representing the request and a function called resolve, which invokes SvelteKit's router and generates a response (rendering a page, or invoking an endpoint) accordingly.

In fact, in the pageEndpoints documentation, it is stated clearly that fetch is called on the endpoint:

https://kit.svelte.dev/docs/routing#endpoints-page-endpoints
If an endpoint has the same filename as a page (except for the extension), the page gets its props from the endpoint — via fetch during client-side navigation, or via direct function call during SSR.

Ok, yes, that is the case. Thank you very much.
I would suggest that the documentation on pageendpoints points to this documentation here: https://kit.svelte.dev/docs/page-options#prerender
would make things very clear
tytyty
Also, where do we donate to this project?

And one last question. Would it be reasonable to pre-render a page, and then have it collect information using a standalone enpoint, and update the contents of the page?
This would affect the SEO of the page, but would it be faster?
I very much thought that SvelteKit was rending pages on the server side before sending them to the browser, and then hydrating them with data.
Are there two kinds of SSR then? one is SSR-Prerender, and the other is SSR every page load?

Also, what is the meaning of this: https://kit.svelte.dev/docs/loading
within this context?

So aye, in the documentation the phrase "Server Side Rendering" and "Pre-Rendering" have been difficult for me to grasp. I was working from a page that was prerender true, from the skeleton.
It all makes sense, but may I suggest a PR to make it clear to people when pre-rendering affects SSR

Closing this as the prerendering documentation has since been updated, and includes details on how +server.js routes are handled https://kit.svelte.dev/docs/page-options#prerender-prerendering-server-routes


ionic for android device $cookies does not work!

description of the problem:
i use ngCookies store data that store 30 days!
it works on ios and browser,but does not work on android device!

Which Ionic Version? Ionic, v1.2.4  and  ngCookies, v1.4.3

Thanks for opening an issue with us, we just need a little more info from you so that we can better help you with your issue. What version of Android are you testing on? Also would you be able to provide a plunkr, codepen or repo that we could use to repro the issue on our devices?

@jgw96 the version of  my emulation  is 4.4.4 and  my device is also 4.4.4! the code is very simple. i use ngCookies module and use method  $cookies.putObject("key",{id:1}) and $cookies.getObject("key")

@gluttonman After doing some testing of my own on a Nexus 7 running Android 6.0.1, a Nexus 6 running the Android N dev preview and a Moto X 2014 running 5.1 ngCookies does appear to have some issues on android in Cordova apps that i will need to look further into. Is there a specific reason that you are using cookies and not something like localStorage?

Hello! Thanks for opening an issue with us! Since it has been a little while since there was any activity on this issue i will be closing it, but feel free to respond back if your still having the issue! Thanks again!

@jgw96 Thank you for help me! the cookies can store data by expire date but localStorage can not do it . so  i choose to use cookies.

Hi, I also got this problem when I try to use $cookies on android version 5.1+, it's not working.
When I run it on the browser (with ionic serve) it's working but on android it isn't.
Do you have any ideas ?
Tnx!

@chen1203  it is because that chrome broswer not support  cookies that load local file. the app fix on device is like the broswer load local file, so the cookies is disable.

@gluttonman - So what is the solution ? What is the best/recommended way to store some data when the app is close? (for example - to login)

I used localStorage and it works both on browser and android.
@gluttonman - thanks.


Allow for customizable instance IDs ... and a few fixes

The main purpose of this PR is to add the ability to use custom instance IDs. This is needed in order to register multiple instances of the same app in a PAAS like Cloud Foundry. In such an environment the host name is the same for multiple instances of an app. Thus, using the host name as the instance ID will cause collisions in the registry. Beside the PAAS-case there are other use cases that require this feature, e.g. docker containers running on the same host.
This functionality has been added to the Java client a while ago, for details see https://github.com/Netflix/eureka/pull/85.
The rest of the commits in this PR were needed to (a) get the Vagrant boxes up and running and (b) have all tests pass.
Let me know if you've got any questions. Thanks!

Thanks for the contribution @st3v. This looks good to me.
@tysonstewart or @llaughlin could I get another set of eyes on this before I merge. It shouldn't affect our usage of fargo since we don't currently set the UniqueId() function anywhere.

@jcox92 I'll take a look at this before the end of the week.

Looks good to me, :100:

Thanks for merging this!

Definitely! Thanks for making fargo better!


If imported spreadsheet creates a new item, and title/point values are modified, original values are used instead

In import/export, if a new item is detected you are taken to the create screen.
The title and point values are prepopulated. However if they are edited, the next screen shows the original values. But when the assignment is actually created, it uses the edited value.

@kyleblythe what workflow would you like here? For discussion.

This is a good catch, Steve! The confirmation screen should definitely show the edited title/point value.

This will be done as part of the work in #1356

Yup! Looks like this is now working correctly.


Update Operation won't catch full Data and Before Object in Kafka Message

Hi,
There is a case where the Update Operation only displaying properties data which being updated, but not all the data. For example, this is one of the SQL Statement:
update CUSTOMER.NCS_JOB_INFORMATION
set JOB_STATUS = 'FINISHED', JOB_FINISH_TIME = TIMESTAMP ' 2020-04-24 23:02:54'
where
JOB_STATUS = 'RUNNING' and JOB_FINISH_TIME IS NULL;

will only displaying data properties which are updated, but not with the one which is not updated:
  "payload": {
    "SCN":<PHONE_NUMBER>492,
    "SEG_OWNER": "CUSTOMER",
    "TABLE_NAME": "NCS_JOB_INFORMATION",
    "TIMESTAMP":<PHONE_NUMBER>000,
    "SQL_REDO": "update \"CUSTOMER\".\"NCS_JOB_INFORMATION\" set \"JOB_STATUS\" \u003d \u0027FINISHED\u0027, \"JOB_FINISH_TIME\" \u003d TIMESTAMP \u0027 2020-04-27 10:08:54\u0027 where \"JOB_STATUS\" \u003d \u0027FINISHED\u0027 and \"JOB_FINISH_TIME\" \u003d TIMESTAMP \u0027 2020-04-24 23:02:54\u0027",
    "OPERATION": "UPDATE",
    "data": {
      "JOB_NAME": null,
      "JOB_DESC": null,
      "JOB_PARAMETER": null,
      "JOB_STATUS": "FINISHED",
      "JOB_START_DATE": null,
      "JOB_FINISH_TIME":<PHONE_NUMBER>000
    },
    "before": {
      "JOB_NAME": null,
      "JOB_DESC": null,
      "JOB_PARAMETER": null,
      "JOB_STATUS": "FINISHED",
      "JOB_START_DATE": null,
      "JOB_FINISH_TIME":<PHONE_NUMBER>000
    }
  }

Unfortunately, we need all data properties to be filled (even the one which aren't updated). Do you have any idea on how to do this? Thanks.

Hi,
Firstly could you please check supplemental logging from your v$database view ?
select v."SUPPLEMENTAL_LOG_DATA_MIN",v."SUPPLEMENTAL_LOG_DATA_PK",v."SUPPLEMENTAL_LOG_DATA_UI",
       v."SUPPLEMENTAL_LOG_DATA_ALL"
 from v$database v ```

Hi,
This is the result from the above select statement:
SUPPLEME SUP SUP SUP
-------- --- --- ---
YES      NO  NO  NO

Hi,
Could you please execute following command mentioned in Readme in order to capture all fields of data and retest the issue ?
alter database add supplemental log data (all) columns;
Thanks

Hi,
Sorry for late reply. I have confirmed that this solved my problems. Many thanks for the great support @erdemcer! :thumbsup:


Custom buttons under bot response

Is your feature request related to a problem? Please describe.
I'd like to be able to include custom buttons under the response from a bot in order to collect feedback or take other actions, similar to how ChatGPT does it

Describe the solution you'd like
If this is already possible, can you please help me understand how to do it and maybe update the docs to be more explicit about this feature?
If not, can you please create the ability to set some buttons and actions to accompany a bot's response?
Describe alternatives you've considered
I have considered not having this feature and I have considered not using copilotkit so that I can have this feature.
Additional context
Here's a screenshot of what ChatGPT's buttons look like under a bot response.

Hey @ogoldberg , thanks your description of alternatives, made me laugh!
As for custom buttons below the assistant message, CopilotKit allows you to swap out the components for UI elements so you can provide your own.
In this case, you can customize the ResponseButton to get what you want.
For example:
const CustomResponseButton = ({ onClick, inProgress }: ResponseButtonProps) => {
  return (
    <div style={{ display: "flex", marginTop: "10px" }}>
      <button
        style={{ flex: 1, border: "1px solid #eee", borderRadius: "4px", marginRight: "5px" }}
      >
        👍
      </button>
      <button style={{ flex: 1, border: "1px solid #eee", borderRadius: "4px" }}>👎</button>
    </div>
  );
};

And then passing this to CopilotSidebar:
<CopilotSidebar ResponseButton={CustomResponseButton}>
...
</CopilotSidebar>

Would give you this result:

Let me know if this helps.

Thank you! Very helpful!


inline MathJax is over left text ( width of stack span to small )

Setting Scale of HTML-CSS output Jax in config/default.js to 85 to match font size of surrounding text my inline math block is shorther than the contents, so it overlaps with the text on its right. The em value is bit too small.
Problem exists only in Google Chrome (19.0.1084.56 m)

Hey! I have a problem with overlapping elements, although it's a bit different.
How can I target the rendered MathJax formulas with CSS, because they overlap on one another if they div containing them is smaller than the text that will be rendered.
What I do is write a few divs with some dimensions (say width:50px; height:50;), in each div I write a math formula (of different sizes). If the formula rendered is bigger than the div then it overlaps on the next div's formula. I could set the div a larger width, but then when the formula is short the text will be too far from one another.
I want to set something like "position: relative" so they would not overlap (as they would be relative to one another wouldn't they?). Will that work? or is there another solution.
Anyway... I need to target them directly. But there are so many classes and spans and div, I can't figure out which one represents the rendered text. I have this problem for several weeks now, and still couldn't fix that.
Please help!

Please don't double post. Thanks.


Fully automate dev setup with Gitpod

This commit implements a fully-automated development setup using Gitpod.io, an
online IDE for GitHub and GitLab that enables Dev-Environments-As-Code.
This makes it easy for anyone to get a ready-to-code workspace for any branch,
issue or pull request almost instantly with a single click.

Thanks for contributing! ð

LINKED ISSUES:
Does this PR close/fix an existing issue? Write something like Closes #10

TEST URLS:
Add some test URLs

SCREENSHOT:
Add a screenshot here if your PR makes visual changes

Thanks for the suggestion, but we are going to pass on this.


Cache Index CRCs verification during login logic allows bypassing

https://github.com/rsmod/rsmod/blob/43d42a54601ac2b2e94318c6252c2cd70ba64dc3/plugins/api/src/main/kotlin/org/rsmod/plugins/api/protocol/codec/login/LoginDecoder.kt#L249
Kyle E pointed this out to me last night so I don't know if he brought it to your attention or not, but I do believe CRCs can be negative and while the check if greater than 0 allows for not checking index 16 (as it is 0) means any client could write all CRCs as zero or negative and completely bypass the verification no?
I would submit that the greater than zero check adds more overhead than to simply just check that index 16 is 0 like it should be (20 checks if the value is greater than zero as opposed to just a single check for each CRC received matching).
These are the CRC that the 193 client sent as proof they can be negative (note this is printed client side - not after being read/after being written etc):
[-1173228978, -1157589397, -1212584219, -570258262,<PHONE_NUMBER>, 268483285, 187074646, -353829414, -1052908534, -1247884930, -597431892, -608311250, -274006329, -1116086727, 153718440, -445574398, 0, -1481854411, -2029010792, 342059222,<PHONE_NUMBER>]
In rsmod1 I use the RL cache libs to retrieve the CRCs as so:
val crcs = filestore.indexes.map { it.crc }.toIntArray()
And that resulted in matching CRCs just fine for me. While CRCs themselves are not inherently signed, the int data type of the variable they are residing in within the client is inherently signed and if you read using the same encoding the client sends then you should also be looking at negative values server side.
The attached code snippet should sufficiently verify CRCs for you barring all the above is true:
patch.txt

Addressed in commit https://github.com/rsmod/rsmod/commit/089dede92254ed70d828e2c4d18cf2d5441b3f2a


Liquibase 4.x Compatibility

Overview
Liquibase 3.6 as well as 4.0 introduced API compatibility issues with the current log4j-liquibase codebase. See liquibase/liquibase#1777
This updates the code to be compatible with 4.0+ while still supporting 3.5 and 3.6+ users.

With the new Log4j2Logger implementation, the messages coming through in the test class look like  liquibase.ext.logging.log4j2.Log4j2LoggerTest liquibase.logging.core.AbstractLogger FATAL Severe message and I'm not sure where that "AbractLogger" part is coming from.
I think I'm setting up the ExtendedLogger the same as it was in the old code, but that extra piece is not showing up there.
Any ideas why?

From Liquibase's standpoint, as a library we try hard not to introduce dependencies on external libraries such as log4j. Therefore, we build off the java.util.Logger but do have a lightweight extension point where 3rd parties can plug in alternatives instead.
This code is really just plugging into that liquibase extension point to bypass JUL and go directly to log4j. An alternative is certainly to use the log4j-jul adapter as well. I didn't write this extension in the code here, just fixing it :)
If you are against continuing to have this extension as part of log4j, we could look at bringing it into the liquibase codebase directly as well. I don't think we could have it automatically enabled if log4j is found, since many users end up with log4j in their classpath even though it's not their "real" logging system. But, have some configuration settings we could support.

I perfectly understand your conservative attitude for adding dependencies to your project and share them. This said, dependencies increase re-usability and independent evolution. Hence, I ~strongly~ kindly advice you to consider adopting log4j-api, if possible.
Even though I am a PMC member, I cannot speak for the rest of the board. I will raise this issue in the dev mailing list. If we decide to exclude log4j-liquibase from 3.x onward, I see two possible paths:

Liquibase adopting log4j-api (recommended)
Introducing liquibase-log4j module (I already see a org.liquibase.ext:liquibase-javalogger module)

For the records, would you mind sharing this with the rest of Liquibase crew and let us know what do you think, please? Is adopting log4j-api really a no-go for you?

Yes, I will keep the rest of the team up on this discussion.
I think that making log4j-api a required part of our API or standard logging implementation is a no-go. It would mean having to change a stable API, and impacts users embedding liquibase but not wanting to introduce log4j-api as a dependency into their software.
But, there are ways we can include it in liquibase so log4j isn't required but still would work when people already have it as a dependency.

I don't necessarily object to Log4j providing integrations with third party components. But if something like liquibase chooses to provide its own logging API then it should also supply the bindings to the various implementations its users want to use. That binding could certainly be hosted inside the Log4j project, but it really shouldn't be part of the main Log4j release. We can't possibly support every such integration in that way.

I close this, since Liquibase will publish their own plugin (cf. #1193).
@nvoxland, thank You for your contribution.


[BEAM-8233] [BEAM-8214] [BEAM-8232] Document environment_type flag

We've gotten a lot of questions about this on the mailing list, particularly from new users, so hopefully this makes things a bit easier.

Use loopback for getting started; don't require a docker build.
Explain the differences between the different values for environment_type.
Remove runner-specific wordcount instructions from the portability page. The runner documentation pages should be the source of truth for those.

R: @soyrice
I saw you were working on container documentation, so I wanted to sync up on this.

Python docs failure is unrelated, should be fixed by #9620
Leaving BEAM-8273 as a follow-up.

Thanks for working on this/ I would have liked to review this PR,  but did not even know it existed..
Couple issues:

We should probably show how to run the wordcount example on the runner page (it was removed from the portability page)
There is still a gradle command left. Doesn't the Flink runner wrapper eliminate the need to have that gradle build step?

@tweise These are great follows ups. Want to open a PR? The Gradle command is still required to build from master. For releases it is not necessary.

@mxm these are instructions for users. We can mention what people can do with master as well, but first we need to cover the case of a user working with a release. That is also why I think changes to these docs will benefit from more eyes.

@ibzib should the JIRAs be resolved? Regardless of this PR, Beam has a problem with JIRA status. Users get confused to find tickets open for which changes have been merged long ago. It is time to bring back the topic of the committer merging a PR also being responsible for resolving the ticket.

Yes I agree but props for Kyle's initiative to start with that. There is a balance to strike in terms of the total number of reviewers when requesting reviews from people. The current status is already much better and we can iterate further on it using your feedback.

@mxm there is actually no harm tagging more people on a PR. Broader review is good and it may help with faster feedback as well. Not everyone needs to respond. It is sufficient to have a single approval.

Slightly diverging from the topic here. I don't think it is as easy as tagging the entire set of Beam committers on every PR. Choosing the right audience for a PR is vital. You want to avoid swamping people with PRs, but you also want a good review. Tagging someone means taking time from that person and it also implies leaving some time for the person to comment, as well as replying to the comments.

And leaving some time is a good idea regardless who is tagged on the PR :)
When I look for reviewers for my PRs I try to pick folks that I know are most knowledgeable with the area or have expressed interest in the topic. I have never come across a case that required to tag all committers so far :)


Fixing grammatical issue

@thyeggman : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Full MaterialPropertyType and Quantity support to other properties and units

[ ] Right now, not all MaterialPropertyType values are supported (there are many properties inside SpaceClaim, Discovery that are not currently supported). Expand this list to the available properties.
[ ] Same happens for units. The unit naming convention coming from the products is not handled properly by Pint. We should look into this.

Related to #949 and #947


Added convenient method to generate container url

Codecov Report

Merging #404 into dev will increase coverage by 0.01%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##              dev   #404      +/-   ##
========================================
+ Coverage   85.98%    86%   +0.01%
========================================
  Files          56     56
  Lines        4367   4372       +5
  Branches      467    468       +1
========================================
+ Hits         3755   3760       +5
  Misses        462    462
  Partials      150    150

Impacted Files
Coverage Δ

...storage-blob/azure/storage/blob/baseblobservice.py
94.56% <100%> (+0.05%)
: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 5061fca...6885b6c. Read the comment docs.


[pt] Localize content/en/docs/languages/_index.md

Issue to localize content/en/docs/languages/_index.md to pt.

Hey! I want to work on it, could you assign it to me?


chore: Adjusted quality settings, pp effects and render features

What does this PR change?

Postprocessing volume of the scene has been removed
Postprocessing effects are now tied to quality settings
Moved bloom and color adjustment to quality pp
Quality settings now have different values for low medium and high
Render features are now adjusted and disables / enabled based on quality settings

How to test the changes?

Open the debug menu and check your performance
Open the settings menu and change between Quality settings
Compare performance between them
Check any visual glitch that wasnt ther before

Works really well for me!


Produce a more complete union of string literal template interpolations when the same const appears multiple times

Bug Report
When using a constant union of two types, using the value below will always infer it to still be either or both types although as a const it can only be one.

🔎 Search Terms
duplicate key union

🕗 Version & Regression Information
Seen in most recent version of TS (4.6.2) and every version available on the playground.

This is a crash
This changed between versions ______ and _______
This is the behavior in every version I tried, and I reviewed the FAQ for entries about Unions
I was unable to test this on prior versions because _______

⏯ Playground Link

Playground link with relevant code
💻 Code

const key = 'guest' as 'guest'| 'service';

const obj = {
    guestguest: 'hello',
    serviceservice: 'goobdye'
}

obj[`${key}${key}`]

🙁 Actual behavior
Typescript throws error
Element implicitly has an 'any' type because expression of type '"guestguest" | "guestservice" | "serviceguest" | "serviceservice"' can't be used to index type '{ guestguest: string; serviceservice: string; }'.   Property 'guestservice' does not exist on type '{ guestguest: string; serviceservice: string; }'.(7053)

as this is a const, it is impossible for it to still be an either/or, so multiple instances of the variable should all be equal.
In this case, only guestguest or serviceservice should be possible
🙂 Expected behavior
Typescript should correctly infer the key options to be only a combination of one of the options, and not any possible combination.

How did this happen in real code?

trying to get the names of an i18n json that was called something like guestpanelguestoption or servicepanelserviceoption depending on what feature is used. the name is typed based on the keyof typeof the file.
const string = t(`${key}panel${key}option`)
the option only exists for a single case, not a mixed option. ie. always guest-guest or service-service
the original key (from the example) was a string that only expected guest or service as the response.

Any updates on this? I believe we're running into a similar problem as shown in the example below.
const obj = {
  a: '1',
  b: '2',
} as const;

type Obj = typeof obj;
type ObjKeys = keyof Obj;
type ObjPaths = { [K in ObjKeys]: `${K} | ${Obj[K]}`}[ObjKeys];

const getObjPath = <K extends ObjKeys, V extends Obj[K]>(k: K, v: V) => {
  return `${k} | ${v}` satisfies ObjPaths; // unable to correctly infer
}

getObjPath('a', '2'); // correctly infers

Current workaround:
// use this instead of `${key}${key}`
type Repeat2<T> = T extends T ? `${T}${T}` : never
type Result = Repeat2<'guest'| 'service'> // 'guestguest'| 'serviceservice'


Improve error handling with OpenAI

First of all, this is a very exciting project. Thank you for sharing.
I'm running into this issue when I try my first screenshot conversion:
ERROR:    Exception in ASGI application
ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: certificate is not yet valid (_ssl.c:1007)
(There are many error lines but I believe these are the most important)
Edit: this could be an issue with openai itself
https://status.openai.com/

Can you post your full logs? Is this happening when you trying to connect to OpenAI?

I'm sorry I dont have the log anymore. Yesterday openai was down for a few minutes and that was when I experienced the error (I only knew about that when I checked their website later). I think you can place a try catch when you attempt to connect to openai and returns an error message if the connection fails.

Nice, so it's working for you now?


Update vcpkg-tool to 2024-09-18.

https://github.com/microsoft/vcpkg-tool/releases/tag/2024-09-18
Fixes https://github.com/microsoft/vcpkg/issues/34095
Fixes https://github.com/microsoft/vcpkg/issues/24084

VS smoke test passed.


gemspec: bump dependency on puppet-strings

This is required so that projects that depend on bolt can make use of the more recent puppet-strings release.
I ran the doc:all rake task and there where no changes

Hi @b4ldr is there a particular feature in puppet-strings you are looking to take advantage of?

Yes the @deprecated flag

Thanks! Looking in to a bunch of gem bumps as we work on getting bolt ready to be compatible with puppet 8. I will make sure this gets looked at as part of that.

any update on this, its preventing any project that depends on bolt from making use of the recent improvements in puppet-strings

Are you managing bolt's gem dependencies in your workflow or are you depending on the gems we ship with bolt system packages? Trying to get a sense of whether a gem release with this relaxed dependency will unblock you or if you need us to ship the new puppet-strings in bolt packages?

Are you managing bolt's gem dependencies in your workflow or are you depending on the gems we ship with bolt system packages?

My main motivation is to start using the deprecated tag in  the puppetlabs-stdlib repo but the same flow applies for any puppet module that use bolt or litmnus for testing.  stdlib is bringing this dependency in via puppet_litmus from its Gemfile
We have had to do similar patches in the past

Cool. This should work for your use case. We merged this change in https://github.com/puppetlabs/bolt/pull/3216 . I would expect this to eventually make it in to bolt system packages, but for now if you are using bolt as a gem, we can get this released and that should help.

great thanks <3


Hide "Watched"

I am pretty sure there is no way to do this (because it is connecting to live website), but could there be a way to hide "watched" programs in this addon? Right now it puts a check mark beside them (very nice), but does not hide them.
thanks

Hey, sorry for the late reply. Kodi has this feature when browsing Movies and TV shows (in the left side menu), but apparently not in plugins. I'm pretty sure the plugin could check which files are watched and hide them, but I think this feature is up to the Kodi team to add.


[P0] Memory efficient version of LoReFT

Descriptions:
When use LoReFT in practice, the orthogonalization process of torch takes up the majority of memory overhead during training. If we get rid of this constraint, then it is no longer a pure LoReFT - it makes it Non-linear Low-rank ReFT (NLoReFT). There is some trade-off in memory efficiency and performance. One should feel free to explore ideas like NLoReFT to see the trade-off if there is one.

I may trying writing a triton kernel for this similar to unsloth


Generate create operation deterministically from mnemonic

Construct a new ION DID from a mnemonic allowing transfer between mobile and desktop to facilitate verifiable presentations.

Next steps:

Merge 117-mnemonic-create (TODO: open PR)
Merge https://github.com/alan-turing-institute/trustchain/pull/122

Related to https://github.com/alan-turing-institute/trustchain-mobile/issues/23.


Configuration if Nginx also running in container

Can you give an example of the docker startup command or the docker-compose.yml if the nginx instance also running in a docker container.
How can we link both containers?

Hi, thx for the nice UI!
For me it would be also helpfull that I can run nginx-ui out of docker-compose.yml.
I tried to map the volume  ./nginx/sites-available:/etc/nginx/conf.d. But it does not recongnize the subfolder/files in there. In /nginx/sites-available I have the different .conf files in subfolders like 1_dev/site1.conf, 2_test/site1.conf, 3_prod/site1.conf.
Would be nice if this would work with your tool

Can you give an example of the docker startup command or the docker-compose.yml if the nginx instance also running in a docker container.
How can we link both containers?

Can you try the docker-compose.yml snippet : https://github.com/schenkd/nginx-ui#docker
You might have to use the above snippet inside an existing docker-compose.yml with nginx and use along side.

Hello @JoLander, an example is already in the project for docker-compose. It is important that you mount the directory of nginx (/etc/nginx) in the same place as in nginx-ui (/etc/nginx) . Does this help you?
version: '3'
services:
  nginx-ui:
    container_name: nginx-ui
    build: .
    image: nginx-ui:latest
    ports:
      - 8080:8080
    volumes:
      - nginx:/etc/nginx

  nginx:
    container_name: nginx
    image: nginx:1.18.0-alpine
    ports:
      - 80:80
    volumes:
      - nginx:/etc/nginx

volumes:
  nginx:


feat: Provide support for upgrading HTTP requests

This lays the groundwork for upgrading HTTP requests from query calls to update calls.
It applies the changes from https://github.com/dfinity/agent-rs/pull/195 that are specific to ic-utils so that they can be used downstream in icx-proxy.

This is the downstream PR: https://github.com/dfinity/icx-proxy/pull/6


Wake word detection responses in Greek not working..

Wake word detection responses in Greek not working..

You must also set the TTS service, TTS language to be the same as the ones you use in the pipeline. I tried this setting and it works.

Thank you for your response.
For some reason i have 2 Home assistant cloud options in TTS but no edge TTS option.

When i did the guide i just used one of the 2 home assistant cloud TTS option (the second one). I will try the Top one now and check. The strange thing is that the replies work correctly in Greek. Its just the wake work detection responses that were not working.

It's good, for TTS service write tts.home_assistant_cloud, and for TTS language, everything after language=, el-Gr&voice=AthinaNeural

I had tts.home_assistant_cloud but its not working. If the change i made still doesnt work i will try with edge tts

Hi, i managed to get this working last time with edge_tts but now its not working again.
Any chance you can make your configuration page to select tts service and tts language from a list?
That would be so much easier and eliminate errors. Also i cannot find a way to troubleshoot this as going to voice assistants debug, nothing shows up for the wake word responses..
Thank you


Correct bom entry for netty-transport-native-unix-common

Motivation:
The entry for the netty-transport-native-unix-common module in the bom
was using the wrong artifact ID and version.
Modifications:
Correct the artifact ID for the netty-transport-native-unix-common
module in the bom.
Result:
Fixes [#6849]

@wilkinsona thanks a lot!
Can you please sign our icla and let me know once its done ?
https://netty.io/s/icla

@normanmaurer I've signed the ICLA

@wilkinsona thanks a lot!
Cherry-picked into 4.1 (ea2af3593cbe5ab70b2a7433b07cc96780d88124)


Look into PrismJS replacement

Prism is nice and I like its plugins, but its regex breaks in more complex (bash) scripts.

Time to look into the Hugo docs for syntax highlights.

Also, maybe i can tweak the way Asciidoctor uses Rouge, but i don't have high hopes. If I could, I would have done so instead of switching to prism.

Highlight.js looks promising, but I have to look into how to get the downloaded package into a workable form for hugo


Refactor output exports

Checklist

[ ] I've read and followed the Contribution Guidelines
[ ] step.yml and README.md is updated with the changes (if needed)

Version

Requires no version update
Context
Clean up the code that exports step outputs and gradually replace v1 go utils with the v2 equivalents.
Changes

Move output export logic to a separate output.go file. It's mostly copy-paste, but I made some small style improvements with new structs and renames.
Make the error message more accurate that is printed when output export is not possible

Investigation details

Decisions

@hisaac good idea, I added some tests for the main business logic. Originally I wanted to test the whole function, but I had soo much trouble mocking all the dependencies that I gave up. I think ultimately it's cleaner to only test the business logic, so I don't mind it anymore, but let me know what you think.


js-multistream

js-multistream
https://github.com/diasdavid/js-multistream
Follows https://github.com/jbenet/multistream/tree/master/multistream spec
How you can help

make code coverage go up to 100%

✔


google.org/personfinder/global/tos should be internationalized

It looks google.org/personfinder/global/tos always redirects to the English version:
http://www.google.com/intl/en/policies/terms/
It should link to the URL which shows localized version automatically, or change the URL based on the lang URL parameter.

The issue is not happening in current production, but I realized that the tos page in latest code shows 500 internal server error during local test with the log below:
：
File "/Users/kmiki/pf/app/tos.py", line 73, in get
tos_url=url)
UnboundLocalError: local variable 'url' referenced before assignment
INFO     2016-10-24 01:39:44,286 module.py:788] default: "GET /global/tos HTTP/1.1" 500 228
：
To fix this issue, I added the initial value=None when no config url value exists.

The original bug no longer reproduces and another bug mentioned above was fixed.


[ci] More explicit message on how to keep issues not staled

Noticed some issues not being marked as Stale which were dated 29 Nov 2019 and before

Good eye ;) This is expected as this is the very beginning of the automated clean-up and there is a max number of items to be inspected by run (30 or 100 IIRC).
You can check the run:
https://github.com/cloudera/hue/actions/workflows/autoclose-stale.yml
Reached max number of operations to process. Exiting. |

Oh I see!
After 5 days cycle, they will be marked too eventually.


RF4.0.1: robot.libdoc no longer processes resource files unless full pathname is given

With the RF 4.0.1 upgrade, robot.libdoc is no longer able to process resource files which are not accessible directly, but can  be found using PYTHONPATH. This used to work in 4.0.0 and earlier, and could be a consequence of fixing #3919
Deails:
(lm) OBOEHMER-M-C1EC:~ oboehmer$ robot --version
Robot Framework 4.0.1 (Python 3.7.9 on darwin)
(lm) OBOEHMER-M-C1EC:~ oboehmer$ pwd
/Users/oboehmer
(lm) OBOEHMER-M-C1EC:~ oboehmer$ cat /tmp/resource.robot
*** Keywords ***
Test Kw 1
    No Operation

(lm) OBOEHMER-M-C1EC:~ oboehmer$ PYTHONPATH=/tmp python -m robot.libdoc resource.robot list
Importing library 'resource.robot' failed: Module 'resource' does not contain 'robot'.

Try --help for usage information.

it works when I specify the full pathname to the resource file:
(lm) OBOEHMER-M-C1EC:~ oboehmer$ PYTHONPATH=/tmp python -m robot.libdoc /tmp/resource.robot list
Test Kw 1
(lm) OBOEHMER-M-C1EC:~ oboehmer$

This used to work in RF 4.0.0
(lm) OBOEHMER-M-C1EC:~ oboehmer$ robot --version
Robot Framework 4.0 (Python 3.7.9 on darwin)
(lm) OBOEHMER-M-C1EC:~ oboehmer$ PYTHONPATH=/tmp python -m robot.libdoc resource.robot list
Test Kw 1
(lm) OBOEHMER-M-C1EC:~ oboehmer$

Yes, this is a regression caused by the fix for #3919. Needs to be fixed on RF 4.0.2. Is that release being planned for May 11 a problem for you?

Yes, this is a regression caused by the fix for #3919. Needs to be fixed on RF 4.0.2. Is that release being planned for May 11 a problem for you?

4.0.2 is ok, we can workaround by specifying the full path for now...

Another pretty severe regression found. Probably need to release 4.0.1 sooner than originally planned.


Permission Admin Panel

Description:

As [CLARISA admin user], I want [manage and asign permission to users to access APIs] so that [We need to create an admin panel to manage roles to API endpoints]


Set headers for expected error (throw error(..))

Describe the problem
Right now, when throwing an "expected" (expected as in described in the docs) error using throw error(...) on the server, you can only modify the body. If, for example, we are doing a broad check in the handle function in hooks.server.ts, this might originate from both an API request or from a page load. In the case of the API request, we might want to include some headers, e.g. when required by a spec (for example, technically, responding 401 Unauthorized requires setting WWW-Authenticate headers).
Of course, we could craft the response ourselves. However, if we also want to do this check when it comes from a page load, it would require quite a lot of work to ensure the +error.svelte is still rendered. The error helper exists to make it easy to send a status code for a broad range of cases, why not also let you include a header?
Describe the proposed solution
Add an additional parameter to the error helper, e.g. headers, where you pass in a Headers object. This would then add these headers to the response.
Alternatives considered
As mentioned above, it's possible to make your own custom response. That does mean you can no longer use the error helper, which is very useful (as in the case of for instance, an API request that only accepts JSON it will send a JSON response, while in the case of a page load it will send the nearest +error.svelte page) and also makes it easy to see for others when reading code that this is where an error is thrown.
Importance
would make my life easier
Additional Information
This is very closely related to #9188, although in that case it concerns unexpected errors, which are handled through the handleError. Somewhat related is also #8393, because even if this is solved, the error helper isn't as useful in hooks.server.ts as this only renders the static error.html page. If that issue is solved, this also becomes more useful.
I can make a PR for this if the Svelte team accepts that this is useful.

Somewhat related: it feels a bit unusual to throw an error for a valid server response just because it has a non-successful HTTP status code for its response. Returning a response object seems more intuitive. throw feels more semantically correct for a runtime error.


file not found error not reported when building mime container

if you call SVG("file-does-no-exist") then you just get

[Fatal Error] :1:1: Content is not allowed in prolog.
but the error should be "file not found".
ditto for Image().

the errors are still not reported correctly.
need https://github.com/twosigma/beakerx/issues/5095
should not be catching the exn at all in the runtime,
the kernel should catch and report.

fixed in #5095

i am still seeing these errors reported to stderr, not as error outputs.


CWMS Decode

The messages being sent from the datalogger need to be set up in the cwms system and decoded, to ensure it is correct.

Campbell datalogger should send information as Big Endian

E-mailed Greg for help setting up this unit in CWMS on 6/22.

Reached out to Greg on 6/22 to help set up a new station in CWMS.

Manual decode of .sbd file from 6/24

Updated in CWMS ~June 2021


Can we define a maximum height of grid?

I know we can define a rowHeight and in GridItem limit maxH.
So for example i want grid with maximum height of 300px. I set rowHeight to 100px and maxH to 3.
Thats the only thing I can do.
Now I can resize a gridItem only to 300px height, but unfortunately I can place another gridItem under the first gridItem and in consequences the overall height will be bigger then 300px.
Is there any way to prevent it, so I can have grid with fixed height?

have you got the answer? or have you managed it anyhow?
I am also struggling to do same.

I think you can use maxRows https://github.com/STRML/react-grid-layout/issues/30

maxRows do not prevent from dragging second element under the first one

You could likely implement it yourself using the onDragStop and onResizeStop handlers.

Do you know way how to use callbacks like onDragStart,onResizeStart ?

I know how to use them, but you can't stop user from dragging one element below the other.

I am also struggling in same ploberm
I think if we can get x,y position at dragStop and resizeStop than we can check y offset of layout and stop propogation if y >3.
I will post solution if I could get it done or let me know if you done .

do you know go to stop dragging?
I mean when user starts to drag an element, do you know how to stop him from placing an element and  bring the element to the start position?

you can try below code
onDrag = (layout, oldItem, newItem, placeholder, e, element) => {
const grid = document.getElementsByClassName('react-grid-layout')[0];
const translateXMaxValue = grid.offsetWidth - element.offsetWidth;
const translateValues = window.getComputedStyle(element).transform.split(',');
let translateX = parseInt(translateValues[translateValues.length - 2]);
let translateY = 0;
if (translateX > translateXMaxValue) {
translateX = translateXMaxValue;
}
if (translateX < 0) {
translateX = 0;
}
element.style.transform = translate(${translateX}px, ${translateY}px);
};

@madnai Have you figure it out?

No but you can try below code
I am also trying to achieve the same but did not get success yet
onDrag = (layout, oldItem, newItem, placeholder, e, element) => {
const grid = document.getElementsByClassName('react-grid-layout')[0];
const translateXMaxValue = grid.offsetWidth - element.offsetWidth;
const translateValues = window.getComputedStyle(element).transform.split(',');
let translateX = parseInt(translateValues[translateValues.length - 2]);
let translateY = 0;
if (translateX > translateXMaxValue) {
translateX = translateXMaxValue;
}
if (translateX < 0) {
translateX = 0;
}
element.style.transform = translate(${translateX}px, ${translateY}px);
};

its not working :(

Hello,
Reviving this old thread. I ran into the need of setting a maximum height for the grid. I found a hacky solution that worked for me but it still hasn't been well tested.
To ensure that the grid doesn't exceed the predefined max value, the idea is as follows :
on onResizeStart and onDragStart callbacks, we save the current layouts (supposed to respect the height limit) somewhere.
Once the resize or drag is done, the onLayoutChange is called. At its end, we call resetOldLayoutIfNewHeightTooBig   which will do the following:
get the new grid height which is computed automatically if autoSize is true (this is the default currently)
if the new  height exceeds the predefined max, we reset the layouts to the value we saved in onResizeStart or onDragStart i.e. the one that respects the height limit.
This of course requires the use of some sort of state to save the layouts for the methods setLayoutsInState and getLayoutsFromState using React state or Redux for instance.
this.maxAllowedGridHeight is to be defined based on the max height requirement.
I noticed that it is necessary to let the layouts state change to the new forbidden value before resetting the old correct one.

  onLayoutChange(currentLayout, allLayouts) {
    this.setLayoutsInState(allLayouts);
    this.resetOldLayoutIfNewHeightTooBig();
  }
  resetOldLayoutIfNewHeightTooBig = () => {
    const newHeight = document.querySelector(".react-grid-layout").offsetHeight;
    if (newHeight > this.maxAllowedGridHeight) {
      this.setLayoutsInState( this.lastLayoutsThatRespectMaxHeight );
    }
  };

  saveCurrentLayouts = (layout, oldItem, newItem, placeholder, e, element) => {
    this.lastLayoutsThatRespectMaxHeight= { ...getLayoutsFromState() };
  };

  render() {
    var layouts = getLayoutsFromState();
    return (
      <ResponsiveGridLayout
           className="layout" layouts={layouts}
           breakpoints={{lg: 1200, md: 996, sm: 768, xs: 480, xxs: 0}}
           cols={{lg: 12, md: 10, sm: 6, xs: 4, xxs: 2}}
           onResizeStart={this.saveCurrentLayouts}
           onDragStart={this.saveCurrentLayouts}>
      {this.generateGridItems()}
      </ResponsiveGridLayout>
    )
  }

Hello, Reviving this old thread. I ran into the need of setting a maximum height for the grid. I found a hacky solution that worked for me but it still hasn't been well tested.
To ensure that the grid doesn't exceed the predefined max value, the idea is as follows :
on onResizeStart and onDragStart callbacks, we save the current layouts (supposed to respect the height limit) somewhere. Once the resize or drag is done, the onLayoutChange is called. At its end, we call resetOldLayoutIfNewHeightTooBig  which will do the following: get the new grid height which is computed automatically if autoSize is true (this is the default currently) if the new height exceeds the predefined max, we reset the layouts to the value we saved in onResizeStart or onDragStart i.e. the one that respects the height limit.
This of course requires the use of some sort of state to save the layouts for the methods setLayoutsInState and getLayoutsFromState using React state or Redux for instance.
this.maxAllowedGridHeight is to be defined based on the max height requirement.
I noticed that it is necessary to let the layouts state change to the new forbidden value before resetting the old correct one.
  onLayoutChange(currentLayout, allLayouts) {
    this.setLayoutsInState(allLayouts);
    this.resetOldLayoutIfNewHeightTooBig();
  }
  resetOldLayoutIfNewHeightTooBig = () => {
    const newHeight = document.querySelector(".react-grid-layout").offsetHeight;
    if (newHeight > this.maxAllowedGridHeight) {
      this.setLayoutsInState( this.lastLayoutsThatRespectMaxHeight );
    }
  };

  saveCurrentLayouts = (layout, oldItem, newItem, placeholder, e, element) => {
    this.lastLayoutsThatRespectMaxHeight= { ...getLayoutsFromState() };
  };

  render() {
    var layouts = getLayoutsFromState();
    return (
      <ResponsiveGridLayout
           className="layout" layouts={layouts}
           breakpoints={{lg: 1200, md: 996, sm: 768, xs: 480, xxs: 0}}
           cols={{lg: 12, md: 10, sm: 6, xs: 4, xxs: 2}}
           onResizeStart={this.saveCurrentLayouts}
           onDragStart={this.saveCurrentLayouts}>
      {this.generateGridItems()}
      </ResponsiveGridLayout>
    )
  }

Curious how this has worked for everyone? I see 9 likes, and curious if there was any implementation based on this... cheers!

@spxis It works.

Just commenting here in case it can help someone. This is what I did referring to @CefBoud solution. I am using functional component in my work so I converted it to functional component. Just sharing it here in case anyone wants to refer. I hope it helps.
`const ResponsiveGridLayout = WidthProvider(Responsive);
const maxAllowedGridHeight = 6;
const DashboardComp = (props: any) => {
const [currentLayouts, setCurrentLayouts] = useState<Layout[]>([
{ i: 'item1', x: 0, y: 0, w: 2, h: 3, minW: 2, minH: 3 },
{ i: ''item2', x: 2, y: 0, w: 3, h: 4, minW: 2, minH: 3 },
{ i: ''item3', x: 5, y: 1, w: 2, h: 3, minW: 2, minH: 3 }
]);
const lastLayoutsThatRespectMaxHeight = useRef<Layout[]>([]);
const saveCurrentLayouts = (layout: Layout[], oldItem: Layout, newItem: Layout, placeholder: Layout, e: MouseEvent, element: HTMLElement) => {
    lastLayoutsThatRespectMaxHeight.current = layout;
};

const resetOldLayoutIfNewHeightTooBig = async (currentLayout: ReactGridLayout.Layout[]) => {

//FInding all any elements that exceed max height
currentLayout.forEach((element) => {
if (element.y + element.h > maxAllowedGridHeight) {
setCurrentLayouts(lastLayoutsThatRespectMaxHeight.current);
return true;
}
});
return false;
};
useEffect(() => {
    //Delay is set here as sometimes if the item is dragged too fast, it does not reset to the old layout even if it exceeds the max height
    setTimeout(() => {
        resetOldLayoutIfNewHeightTooBig(currentLayouts);
    }, 50);
}, [currentLayouts]);

return (
    <>
        <ResponsiveGridLayout
            className="layout"
            layouts={{ lg: currentLayouts }}
            breakpoints={{ lg: 768, sm: 0 }}
            cols={{ lg: 12, sm: 4 }}
            rowHeight={80}
            onDragStart={saveCurrentLayouts}
            onResizeStart={saveCurrentLayouts}
            onLayoutChange={(currentLayout: ReactGridLayout.Layout[], allLayouts: ReactGridLayout.Layouts) => {
                setCurrentLayouts(currentLayout);
            }}
            compactType={null}
        >

            ))}
        </ResponsiveGridLayout>
    </>
);

};`

For what's worth, there's a "maxRows" prop now that does the job.

But it will not work, when we drop from outside.


refac: CardView

Refactor CardView, to functional component.
Fixed eslint in some files.

@yuritoledo I saw you translated components labels and placeholders to Portuguese. I think we should keep them in English and then add internationalisation to these components.

@wagnerdelima about the translate, I added it in portuguese, because I'm using this repo in a project. I will back it to english and then make a internationalization flag.

@yuritoledo this is good news. Thanks for the update.

I am sending you an email with some question if you don't mind.

@wagnerdelima today I'll update the placeholders!
I already answered your email :)


Added Cloudmersive Document Conversion connector

Added Cloudmersive Document Conversion connector

This checklist is used to make sure that common guidelines for a pull request are followed.

[ X ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: Cloudmersive sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the submission @Cloudmersive Can you please add a readme.md like the following connector?
https://github.com/microsoft/PowerPlatformConnectors/tree/master/connectors/AzureAD

@mamurshe We went ahead and added a README.md


cocomac: Watch sadupsoft.com

cocomac requests the watch of the watch_keyword sadupsoft\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

sadupsoft\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by cigien in Charcoal HQ


Upgrade to stable futures 0.3

Feature Request
Motivation
Currently, Tonic is not semver compatible with stable upstream futures >= 0.3.0 because the manifest pins the version to futures-core-preview = "=0.3.0-alpha.19". This means that conversion traits such as IntoRequest and IntoStreamingRequest do not work as expected in projects that depend on upstream futures 0.3.1, which results in confusing and misleading compiler error messages. For example:
  --> my-project/src/foo.rs:38:45
   |
38 |         let response = self.0.send_progress(stream).await?;
   |                                             ^^^^^^ the trait `futures_core::stream::Stream` is not implemented for `S`
   |
   = help: consider adding a `where S: futures_core::stream::Stream` bound
   = note: required because of the requirements on the impl of `tonic::request::IntoStreamingRequest` for `S`

This is despite the fact that the type parameter S already has a valid Stream<Item = MyRequest> bound in the code.
Proposal
Fixing this should be as simple as replacing dependencies on the unstable futures-core-preview and futures-util-preview crates with the stable futures-core and futures-util crates and releasing a new version of Tonic on Crates.io.
Unfortunately, we seem to be blocked on Hyper, which also depends on a version-pinned futures-preview crate instead of the stable futures crate (hyperium/hyper#2013). That project will need to be updated first before Tonic itself can be updated.
Alternatives
None that I can consider. Since futures 0.3 is now out of alpha, Tonic should make use of it to allow the ecosystem to move forward.

@ebkalderon yeah we should update, we should also move to a git version of hyper and tokio while everything upgrade but I am catuious about breaking things. Next tokio release should be next week so I'll follow up then. Sorry about things being a bit in flux :(

Any update on when this will happen?

Yes, I am actively working on this right now. We have a couple of blockers. I am hopping to get a release out this week but it really depends on the other deps and issues we run into.

#159
#160
#161
and hyper 0.13 release

Thanks for the fast response!

This is done on master! #163 Thanks for the patience everyone <3


vim-mode should dispose all its subscriptions

If I understand it correctly, this patch should be appropriate. If not, at least I'll improve my understanding. 8-)

This is right.


In ActiveAdmin 1.2.0 NoMethodError: undefined method `inheritance_column 'for

Hi!
If activeadmin is set to 1.2.0
NoMethodError: undefined method` inheritance_column 'for MyModel: class

I am  embarrassed...(´・ω・`)
Does anyone know what to fix by solving it?

@aimerald what kind of model do you use? ActiveRecord or ActiveModel ?

@aimerald this might help https://github.com/activeadmin/activeadmin/pull/5253#discussion_r155525109

@aimerald Any luck with @Fivell's suggestion?  I haven't yet used the gem with ActiveAdmin 1.2, I intend to do so in the next week or so.


Integration with Slack Chat - Low Level Design

Background
C4SG web application will interface with Slack to provide chat room functionality.

Requirement

Read the requirement and technical approach document attached.
Research on Slack's Java API
Define Rest API endpoints, and the supporting Slack API calls.
Create development tasks, one for each API endpoint. The new tasks will be distributed to volunteer developers.

Document
Vision - Online Chat.docx
Use Case Specification - Online Chat.docx
Chat Technical Design and Slack Integration.docx

GREAT :)

This issue was moved to Code4SocialGood/c4sg-services#3


Does the Capslock key only work in insert mode?

The caps key in normal mode doesn't seem to work?

Probably duplicates with #40.


Add base ASP.NET Core 3.1 MVC website project

My First PR
Please be gentle. I’m eager to help. I’ve created an ASP.NET Core 3.1 MVC website.

e Looks great!.


Not all fonts are loading

Bug Description
Currently only 1000 fonts of the 1011 fonts are loading in the REST API request for fonts.
Expected Behaviour
All fonts are loaded.
Steps to Reproduce

Screenshots

Additional Context

Plugin Version:
Operating System:
Browser:

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

Why is that? Is this an issue/limit with the REST API? Something else?

There is a limit of 1000.


Support for bazel C++ - adding compilation rules (BUILD.bazel)

I Would like to have an option for easy compilation of the Drogon in my Bazel project, something  like:
WORKSPACE.bazel
load("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive")
http_archive(
    name = "drogon",
    urls = ["https://github.com/drogonframework/drogon/archive/refs/heads/master.zip"],
    strip_prefix = "drogon-master",
)

BUILD.bazel
load("@rules_cc//cc:defs.bzl", "cc_library")
cc_library(
    name="drogon",
    deps = [
        "@drogon//:drogon"
    ]
)

An excellent example of this is in one of the Drogon dependencies (https://github.com/open-source-parsers/jsoncpp) which you can compile with Bazel via:
WORKSPACE.bazel
load("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive")
http_archive(
    name="jsoncpp",
    urls=["https://github.com/open-source-parsers/jsoncpp/archive/refs/heads/master.zip"],
    strip_prefix="jsoncpp-master"
)

BUILD.bazel
load("@rules_cc//cc:defs.bzl", "cc_library")
cc_library(
    name = "jsoncpp",
    deps = [
        "@jsoncpp//:jsoncpp"
    ]
)

This is possible because in the repo jsoncpp there is a library target specified (BUILD.bazel): jsoncpp*
If I want to use Drogon right now, I need to run make via Bazel which is not ideal and not straightforward.
This change would allow me to have very clear access to this library and my project would be less prone to linking
errors, because the whole process would be way easier to debug
For invoking bazel in all of this cases I am using bazel build --config=gcc :all command, where config is defined in .bazelrc file that contains:
build -c opt
build:gcc --cxxopt=-std=c++17 --color=auto

I am using GCC 9.4.0

An extra cool feature for this would be also to define WORKSPACE.bazel file with all the Drogon dependencies, but since not every dependency have a BUILD.bazel file specified, this would be hard to achieve. This would allow bazel to automatically fetch, build and link all the dependency that is needed in order to build Drogon repo.

Thanks for the feature request. I think this is a good idea. I added the enhancement label to this issue. Every PR in this regard is welcome.


Log connection error message

Logs the error to aid troubleshooting.
e.g.
ts=2019-04-23T10:31:03Z caller=main.go:121 Error="Unable to connect to Database" DSN="username@someserver:password@tcp(someserver.database.azure.com:3306)/socksdb?allowNativePasswords=true&tls=true" ERR="tls: either ServerName or InsecureSkipVerify must be specified in the tls.Config"

(The error shown above is another bug, I think fixed by https://github.com/go-sql-driver/mysql/pull/564/files but not included in the current build)

Coverage remained the same at 25.926% when pulling bfc0f6d518077b6ad2c2eccafc07f8d3ffd5892c on PriceChild:log-db-error into 97545f4ae068250190491a970a8940c44f1cd2a5 on microservices-demo:master.


remove keepPaperKey option from PaperProvision, always do it

Get rid of the --keep-paper-key flag for PaperProvisionEngine. Default to true, always store the paper key. Also store it before signing in the new device (makeDeviceKeysWithSigner). This is motivated by enabling per-user-key support. I will need the paperkey to encrypt per-user-key boxes. Another option would be to pass it down through arguments and not affect this flag. This seemed cleaner to me.
cc @jzila

FYI this will fail CI until the flag is removed in kbfs-server. Maybe leave the flag itself in but make it do nothing, wait a few days (until people's rebases catch up), then remove it from kbfs-server, then remove it from client.

Also IIRC this paperprovision command was introduced purely for testing (in kbfs-server). What is the new use of it that is needed?

Yeah thanks for the heads up I was in denial about that. ;)
There is no new use of it. Instead there is new code in DeviceKeygen which is called which needs access to the paperkey for per-user-keys.


The same method attach_payment_method is twice in code

The same method was introduced twice. Apparently a merge issue:
https://github.com/rebelidealist/stripe-ruby-mock/blob/master/lib/stripe_mock/request_handlers/payment_methods.rb#L59
and
https://github.com/rebelidealist/stripe-ruby-mock/blob/master/lib/stripe_mock/request_handlers/payment_methods.rb#L77

Also an empty implementation of detach_payment_method: https://github.com/rebelidealist/stripe-ruby-mock/blob/master/lib/stripe_mock/request_handlers/payment_methods.rb#L66


Use simplified xproperty

Uses https://github.com/QuantStack/xproperty/pull/41

This slightly reduces the binary size but not so much 6330000 -> 6182704.


Version 3.3.x

modify Dialog/Tbody/TreeViewItem component to support<EMAIL_ADDRESS>fix Table component multiple selection problem(hack way);
modify some test case.

Coverage increased (+3.0%) to 94.558% when pulling 26326be6670bc4a68460f0a3122d2422cc799190 on ssdna:version-3.3.x into b60af13691a76a2813dc2a2a268aaf0869ac1e65 on ecomfe:version-3.3.x.


ViroScene - not capturing any touches, clicks, ... and camera is not used even if it is active

Hi,
In your react-viro lib within a Viro3DNavigator containing a ViroScene I'm not able to detect any clicks, touches, ... on an object within that scene.
I expected that a normal click would be triggered if the user clicks on an Object within that scene.
Is it true that the camera is also not working in an ViroScene? My rotation and position is totaly ignored.
Thank you in advance

Hi @dominicrico ,
Thanks for reaching out. Would you mind sharing your code around how you are handling clicks, touches, etc.? I just tested internally, and all click / drag events seem to be working as expected in v2.5.1 for Viro3DSceneNavigator. Let us know and we can take a deeper look at why these click events aren't working for you.
And regarding setting camera rotation and position in a ViroScene used in a Viro3DSceneNavigator - this seems to be a legit bug in ViroReact. Great catch on the bug! Sorry for the trouble though. We are looking into fixing this for an upcoming release.
Thanks!

Hi @dominicrico ,
Were you able to unblock yourself wrt. touch events and camera? We did some more digging with the camera bug, turns out was a miss on our part when we tested it 5 days ago. It is in fact working as expected. Let us know if this is still not working for you. If not, then getting some code snippets from you should help in diagnosing why are you seeing these issues.
Good luck!

Closing this issue as can't repro. Please reopen if you are still experiencing issues with the requested additional info and we can look into it further.


For Image-To-Text Message::$content property must allow array data type

According to OpenAI, their message's content property can also be an array. Are there any plans for handling different data types for this property? I must initiate a new instance without the wrapper to access the functionality as a workaround.
response = client.chat.completions.create(
  model="gpt-4o-mini",
  messages=[
    {
      "role": "user",
      "content": [
        {"type": "text", "text": "What’s in this image?"},
        {
          "type": "image_url",
          "image_url": {
            "url": "https://upload.wikimedia.org/wikipedia/commons/thumb/d/dd/Gfp-wisconsin-madison-the-nature-boardwalk.jpg/2560px-Gfp-wisconsin-madison-the-nature-boardwalk.jpg",
          },
        },
      ],
    }
  ],
  max_tokens=300,
)

Hey @prykris, we should add it yes. Do you want to work on it?

@MaximeThoonsen I think this issue can be closed


Add code coverage

There are some options to add code coverage:

https://coveralls.io
https://emberobserver.com/addons/ember-cli-code-coverage
https://emberobserver.com/addons/ember-cli-blanket

We have ember tests and node tests so we need to cover both but I think the ember part is the most important one.

ember-cli-code-coverage added support for coverage in addons in v0.3.0.
See  kategengler/ember-cli-code-coverage#31


Collapse.js not working when on IOS and data-toggle element is not an a element

Hi,
when using following code example

then nothing happens when the data-toggle is touched on IOS devices. The handler in the collapse data-api is not called.
I could fix the issue replacing line 199 of collapse.js with following
$(document).on('click.bs.collapse.data-api touchstart.bs.collapse.data-api', '[data-toggle="collapse"]', function (e) {

just added
cursor:pointer

on the div

Yeah, it's a browser quirk that's not Bootstrap-specific; see https://developer.mozilla.org/en-US/docs/Web/Events/click#Safari_Mobile

@cvrebert didn't we recently add cursor:pointer to some more places in the CSS? worth doing it for all [data-toggle="collapse"] ?

@theluk as a side note, using a generic
 as the toggle will make your collapse inaccessible to keyboard users (which is why the examples in http://getbootstrap.com/javascript/#collapse use focusable elements like  and <button>, which incidentally don't need the extra cursor:pointer fix)

@patrickhlauke thats true unless you make The collapsable for mobiles only and want to make an accordion like view. When using a elements, they could semantically make no sense when on desktop.

"mobile only" also - at least on Android - also has the scenario of users with a paired keyboard (in which case the same focusability problem applies). whether or not a link which links to a panel immediately following it (as in the examples in the docs) is semantic or not is more of a philosophical discussion (generally, i'd say a <button> is the cleaner way to go)


Don't show days of service for trips when there is no variation in a timetable

Using the GTFS here (http://data.trilliumtransit.com/gtfs/yosemite-ca-us/ last modified 2015-09-14 21:02) I am seeing an issue where days of week are shown for each trip, even though it is consistent in the timetable. In FresnoHWY41_HWY41_0.html. Screenshot below.

This is solved. Screenshot below.

I did open issue #4 as an open thread on how to represent service variations.


general comment / suggestion

I've been messing around with the GenerateUnitFileForMVCFramework. It looks like it uses the MVC Framework mostly for support of things like attributes, I'd like the part that generates the final .pas unit to incorporate the ability to add further code-gen mechanisms to fill in the contents of the swagger class methods before it spits out the entire unit.

@thetoolwiz this feature was incorporated into the DMVC.


Remaining SACP Preprocessings

Complete BPE Preprocessings for SACP

@danielrcardenas  New File for SACP was generated with ALL BPEs PReprocessings
Name: [sacp-python-common-all-corpus-1609224778.517111].csv
Find the file in this path : /data/cisco/sacp_data/


2-10 & 2-10 alpine builds failing

The Docker hub build logs here: https://hub.docker.com/r/starefossen/ruby-node/builds/ report the 2-10 and 2-10-alpine images as having failed to build with a "Build Cancelled" message. Was that a timeout or a real cancel? Anything I can do to get them building?
Thanks!

Ah, I think #27 will fix this.