Add Client Analytics Pageview Tracking

What changed? Why?
I added pageview tracking to base.org and docs.base.org so the team can gain insight into which pages/docs are visited most. I'm using Client Analytics loaded via a script tag following this onchainsummer example:

Script tags
Init code

Notes to reviewers
Base Web - CCA is loaded with a Next Script in _app.tsx and initialized via that component's onLoad prop to prevent issues with async loading.
Base Docs - CCA is loaded with an async script tag and initialized in a client module, both configured in docusaurus.config.js.
How has it been tested?
I ran both apps locally, navigated to many pages of each site, and reviewed the data generated in debug logs and posted to Amplitude.

I'm not sure but U can test it by running yarn workspace @app/base-docs build if that doesn't error then the production build should be fine
(refer to the Dockerfile to understand how production works).
On Mon, 2 Oct 2023 at 19:49, jacob-moore-cb @.***>
wrote:

@.**** commented on this pull request.
In apps/base-docs/docusaurus.config.js
https://github.com/base-org/web/pull/29#discussion_r1343263604:

@@ -1,3 +1,7 @@
+const dotenv = require('dotenv');
+dotenv.config();
+dotenv.config({ path: '.env.local', override: true });

Will using process.env.NODE_ENV cause errors in docs prod or just come up
as undefined? If it doesn't cause any errors, I think I can just remove the
dotenv implementation. With that removed, I could go back to referencing
process.env in the docs analytics init code as long as I provide values for
prod to fallback on when they're undefined. So it'd be like this:

const isDevelopment = process.env.NODE_ENV === 'development'
isProd: !isDevelopment
showDebugLogging: isDevelopment
amplitudeApiKey: isDevelopment ? 'development-key' : 'prod-key'

—
Reply to this email directly, view it on GitHub
https://github.com/base-org/web/pull/29#discussion_r1343263604, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJ42D243ONUDMM323EJNSTX5NHH7AVCNFSM6AAAAAA5LUZ6UGVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMNJTG4ZTKOBQG4
.
You are receiving this because your review was requested.Message ID:
@.***>

--
Matthew Bunday
http://zencephalon.com

Running a build should empirically answer the question
On Mon, Oct 2, 2023 at 8:07 PM jacob-moore-cb @.***>
wrote:

@.**** commented on this pull request.
In apps/base-docs/docusaurus.config.js
https://github.com/base-org/web/pull/29#discussion_r1343271523:

@@ -1,3 +1,7 @@
+const dotenv = require('dotenv');
+dotenv.config();
+dotenv.config({ path: '.env.local', override: true });

Will using env variables via docusaurus customFields cause errors in prod
or just be undefined? Assuming they're just undefined, I have a solution
that I can commit in the morning. Otherwise, I'll take another pass at this
then.
—
Reply to this email directly, view it on GitHub
https://github.com/base-org/web/pull/29#discussion_r1343271523, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJ42D2XOVUJW2RQR6MQB7TX5NJKNAVCNFSM6AAAAAA5LUZ6UGVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMNJTG44TSMZWHA
.
You are receiving this because your review was requested.Message ID:
@.***>

I locally created a build with the current changes, and it looks good. A couple warnings, but those are okay, right?

Yeah, those warnings seem fine, sounds like it'll work


Delete doesn't delete records

I am trying to add an index to table/class with existing data. I am trying to run a delete statement prior to creation of index. When executed from studio, the statement deletes duplicate records. But when executed under java, the index creation fails stating duplicate records present
Here's my code...
OObjectDatabaseTx odb = //init;
            odb.command(
                    new OCommandSQL("delete from RE where @rid in (" +
                            "select rid from (select name, min(@rid) as rid from RE where name in ( " +
                            "select entityId, name from (select entityId, name, count(*) " +
                            "from RE group by  entityId, name) where count > 1) " +
                            "group by name))")).execute();
//Code to add unique index
OClass entityClass = odb.getMetadata().getSchema().getClass("RE");
entityClass.createIndex("RE.entityId_ruleName", OClass.INDEX_TYPE.UNIQUE_HASH_INDEX, "entity", "name");

I get exception at createIndex.
NOTE: Table doesn't have much data - only 72 records.

Have you tried running the inner query in studio to see the results?

Any update on this?


Click handler for treemap

I am hoping to use react-d3 in an updated implementation of http://www.avlbudget.org and, in particular, to use the treemap for http://www.avlbudget.org/expenses. To do it, I need to pass in a click handler that will let me know when the user clicks on a treemap cell (and of course some indication of which cell). The handler will just dispatch an event that will change the starting point in the hierarchy, which will then cause the component that uses the treemap to be re-rendered.
The project is https://github.com/DemocracyApps/GBE. The React front end is here: https://github.com/DemocracyApps/GBE/tree/master/gbe/resources/assets/js/components.
I'm on a tight deadline (May 26, 2015) - happy to do the work & issue a pull request, but could use a little guidance in doing it.

I wonder if it might be worth wrapping all event handlers into some type of master handler, and passing that down to all charts via the mixin - that way you could pass custom functions for click, hover, etc.
Anyways, I like this idea a lot. It makes the graphs much more interactive.
What type of guidance do you need that we can give to help you do this!?

I need to pass in a click handler that will let me know when the user clicks on a treemap cell

Hey @ejaxon , actually if you clone the latest master you will get see the treemap is actually implemented with animation. If you look at here, you can see how we handle  onMouseOver and onMouseLeave event.
Hope that this clears thing out

@yang-wei he wants to have his code listen for clicks on the treemap. The current implementation just runs our fairly crappy animation code - we should make it so that you can override all event call backs if desired, or at least, allow for event callbacks to hit the user's code before we run the animation.

@MatthewHerbst, @yang-wei,
Well, one mystery solved anyway. I installed via npm, but was looking at the latest master and so I was confused by the fact that I couldn't turn on the animation. Only occurred to me this morning that the reason it wasn't working was that the npm version doesn't have that yet. Duh.
Anyway, yes, I did see the implementation of onMouseOver and onMouseLeave. It should be easy just to add another property to pass down with callbacks for the various events. I started to implement for onClick, but you're right @MatthewHerbst  that one might as well generalize to all events. There are a few questions for a fully general implementation, esp. how do you indicate what was clicked, ideally in terms of the data rather than the visualization object. I'm happy to contribute to the general problem, but over the next week or two will have to focus just on getting my specific case done (this is one of many things that I need to get done over the next week).
As to guidance, I'm probably mostly just dealing with my own incompetence. I cloned the repo and seemed to successfully run npm install and gulp build, but got errors when I tried to use (errors that  that I did NOT get when I used via the npm installation). When I get to the office this AM I'll start over and try to do it a bit more systematically. I've only recently started working with Node and React (about a month ago) and I'm hardly an expert with Javascript either, so I'm prone to silly errors and gaps, especially with my usual "first implement then read the documentation" approach. If anyone is around today or tomorrow for questions, I'd appreciate it (I'm U.S. eastern time zone).

we should make it so that you can override all event call backs if desired, or at least, allow for event callbacks to hit the user's code before we run the animation

@MatthewHerbst  Yes regarding to this I want to keep the simplicity of current implementation. It's a good idea but  our main focus at the time is to provide an api to render data using react and d3 quickly. If you need more, you can change the source.
Of course this is considerably in the future. Would like know more the use case of other events than obMouseLeave on treemap component. Will check @ejaxon project out to see.

@yang-wei There are a couple standard use cases for treemaps. You can see one of them here: http://avlbudget.org/expenses. When the data is part of a hierarchy, clicking on a cell in the treemap should allow you to descend down to the next level. The second standard use case is to have a tooltip with details of the item you are hovering over. For example, if I hover over China in the sample treemap, you might show the actual value of the population number plus other statistics.

The tooltip example applies to many visualizations. The hierarchy descent is more specialized, but certainly can be of interest for, e.g., pie charts.
Understand the focus on simplicity, but I would highly recommend considering a general way of allowing interactivity. The advantage of allowing users to pass in handlers is that you retain the internal simplicity while creating a nice mechanism for highly interactive visualizations. This will work especially well in the React/Flux paradigm.

@ejaxon Yes tooltip will be supported soon once I have time to work on this.
The clicking on cell example is a good one ! I am also planning to do such thing in my project gh-top. Sure it would be great if our library has this kind of power. What's your expectation of the api provided will be.

Let me think on that a bit @yang-wei. Passing an object with eventType: callback pairs seems a simple way to go about it, but the key question is what context I get with the callback. Again, it's trivial with the treemap since there's a clear, discrete piece of data associated with any click - just provide a reference to the data item, which allows the user to get back any information they might have included in the structure. Not sure that's always the case.

Let me know if there are particular visualizations that I should include in my thinking

I.e., look more closely at the implementation ... I'll think about all of them, of course.

What I am thinking about is instead of giving user freedom to set their callback when it's clicked, we will just set the callback to be zoom in if prop (i.e zoomOpt.zoomable is set). Something like this:
var zoomOpt = {
  zoomable: [bool]          // zoomable or not
  groupBy: [group name],    // e.g continent name correspond to our homepage treemap example
  level: [number],          // e.g 1 , 2
 ...                        // some other props needs
}
<Treemap zoomOpt={zoomOpt}  ... />

I expect we need to do some nesting or grouping to the data prop by using d3.nest, then in the children component,
<Cell
  // props.group is a reference to data item
  onClick={ props.zoomable ? props.zoom(props.group) : null  }
/>

prop.zoom will be pass to the top parent, data changed and treemap is rerendered.

zoomable doesn't feel like the right level of abstraction to me. I would be happier with the lower level constructs (like onClick) that enable users to build custom interactions.

Do you mean that we just need to provide interface like
<Treemap cellOnClickHandler={customMethod} />

Hmm... let me just write out the code and figure out how to do this. Thanks @parshap

@yang-wei yes that's the kind of API I was thinking about. I'm not sure if this is the best solution for everybody, but is how I feel I could make best use of react-d3 personally.

@yang-wei correct me if I'm wrong, but the goal of the library is to emulate d3 on React. d3 allows full customization of all the features of a graph, including overriding and setting events. Thus I honestly don't think it should be the job of this library to do any type of animation or event handling. Rather, we just need to support allowing the users to do those things if they wish. @esbullington I think your comments are needed on this thread for what your vision is.

IMHO, I think a good way for us to go about this would be to have an eventHandlerMixin that gets implemented by all charts. Each chart would call the appropriate methods from the mixin. That way users could safely override any event handler method simply by passing it into the chart definition as a prop.

As my contribution to the discussion, here's my quick-and-dirty solution for my own needs: https://github.com/DemocracyApps/react-d3/tree/master/src/treemap. The example use is here: https://github.com/DemocracyApps/GBE/blob/master/gbe/resources/assets/js/components/SimpleTreemap.js. I only implemented for the treemap for now, but implemented the slight generalization I mentioned above of passing in an eventHandlers hash rather than doing each one individually.
I'm not submitting a pull request since I think there's zero chance I've done it the way you will actually want to. Happy also to take suggestions (esp. on a better way to return context back to the user about where the event occurred). I don't really know D3 at all.

Incidentally, the implementation works nicely now for traversing down through the hierarchy. I'll probably add in the ability to pass in the mouseover-related event handlers so that I can do a tooltip for cells where the label doesn't fit (see issue #217 ).

the goal of the library is to emulate d3 on React. d3 allows full customization of all the features of a graph, including overriding and setting events.

@MatthewHerbst Sounds true but when we start earlier, what we want to achieve firstly is a graph library to help users build graph as quick as possible (e.g metrics graphics).
However since we had so many contributors and different opinion on this, we might change. And the custom event handler is really a good suggestion but it does bring in complexity for our user. (i.e they have to figure out how to do this and parent-child relationship will make callback of the events difficult)
What I can say by now until Eric is back (he's busy with his job) is to keep current implementation. We do have a lot of task stacking

animation for all graph
tooltips
documentation

Just a quick update. I ended up not using this library, but rather porting the treemap code from our site last year (http://avlbudget.org/expenses). That not only gives me the hierarchy navigation (which was trivial to add here), it also does nice animation while zooming in or out. While that adds nothing to the substance and usefulness of the visualization, that kind of 'sizzle' is unfortunately really important to how it is perceived by users.
In this case I'm wrapping the D3 code underneath a React component, but after I get past this deadline, I will look again at whether to finish converting what I have to purer React or coming back to React-D3.
As a general comment, I recommend being careful about distinguishing between ease of use and power of the library. It's perfectly possible to build a very powerful library with lots of possibilities while retaining the ability for a new user to get a great visualization with almost no effort. As long as you retain the "does something reasonable when all you do is give it a dataset and a container" quality, the complexity of using more advanced features remains something the user can do whenever they need to get the additional functionality and are willing to invest a little more time.

@ejaxon Will keep your comment as a reference. Thanks for input and good luck in your project !


/differential.createrawdiff can't handle binary files

If I have a diff with something like this
diff --git a/hphp/test/slow/archive_handler/tar_archive.tar b/hphp/test/slow/archive_handler/tar_archive.tar
new file mode 100644
index<PHONE_NUMBER>000000000000000000000000000000..e51f4b8b8e8ebdf50f96e2113d1073ff97fef968
GIT binary patch
literal 10240
zcmeIyy$*sf5C`x+3QvG7AgsDGIJh`9XQ)a<FxMKzrx)T!#6c%W)c*~X_F8Cu&D9_I
zqe)to>ohO7qMwjW#8D*Lwu#qPs&R53yXKXOWh}h3pY3$Nda#bz^8{Cw>t?#WPx<cb
zuj#FrcIH^|R=Z8k*`C<d`fSQfJEqbW<cwy@3tQ8!&_%vv;#$uixoJjwXZU%wT`g{N
z_4Rc|Yz_PF^cMJQ$bM?0L!5tG)bHxQy%SUrCMryVFbafh9+48B976if>wgGh`~w0I
wfB*y_009U<00Izz00bZa0SG_<0uX=z1Rwwb2tWV=5P$##AOHafK%hT?3!w*j_y7O^

literal 0
HcmV?d00001

If I call createrawdiff with that, then the thing I get back when calling differential.getrawdiff is
diff --git a/hphp/test/slow/archive_handler/tar_archive.tar b/hphp/test/slow/archive_handler/tar_archive.tar
new file mode 100644
index<PHONE_NUMBER>000000000000000000000000000000..0000000000000000000000000000000000000000
GIT binary patch
literal 0
Hc$@<O00001

literal 0
Hc$@<O00001

How can I get this binary file into phabricator?

We no longer accepts bug reports and feature requests on GitHub. Please take the time to:

Update your install to HEAD.
Verify issue still persists.
File task on https://secure.phabricator.com/ with steps to overly complete steps to reproduce.

More information on filing bug reports:
https://secure.phabricator.com/book/phabcontrib/article/bug_reports/
More information on filing feature requests:
https://secure.phabricator.com/book/phabcontrib/article/feature_requests/
Thank you for your interest in Phabricator.


Can not open database file

I have a data ingestion code written in python and today morning raised an error as follows:
07-Oct-24 09:28:47 - Exception occurred:
07-Oct-24 09:28:47 - Error: FATAL Error: Failed to create checkpoint because of error: fsync failed!
07-Oct-24 09:28:51 - DB con. closed
I tried to relaunch the program but it throw an exception as:
07-Oct-24 10:48:19 - Exception occurred:
07-Oct-24 10:48:19 - Error: IO Error: No more data remaining in MetadataReader
I tried to open the database file from the duckdb cli but also shows same error message.
I am worried if the database file is corrupted, since the filesystem where the file was stored was restarted for urgent maintenance.
How to solve this problem and is there. away to restore or import this data?

Error: unable to open database "/home/alqudami/scratch/fs_db/backup.db": IO Error: No more data remaining in MetadataReader

@nasserHHDE is it possible you're running low on disk space? See this issue: https://github.com/duckdb/duckdb/issues/9667

Yes, the filesystem cluster was shutdown for urgent maintenance, but since then the database file is corrupted and couldn't be open anymore. I was wondering if there any tool in duckdb to import/recover the data?

@nasserHHDE, sorry I am not sure, I imagine that what happened is that your database file got in a corrupted state because of the filesystem restart and some data got lost. I would follow up in the above issue and see if anyone has advice for you. I'm going to close this issue for now, let us know if you figure out a way to recover your data!


How do you have cipher suites marked on the certificate when you generate it?

Doc says "Then have cipher suites marked on the certificate when you generate it". How would a customer do this? Can you provide an example?

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

ID: 2454e304-bfe8-9e27-981d-0b4b37f12d07
Version Independent ID: a40c3bab-025b-4d7a-e5ac-faf8caada56b
Content: End-to-end TLS with Azure Front Door
Content Source: articles/frontdoor/end-to-end-tls.md
Service: frontdoor
GitHub Login: @duongau
Microsoft Alias: duau

@DanielLarsenNZ Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

@DanielLarsenNZ We decided to remove this part from the documentation as it was originally added by mistake. I apologize for the inconvenience. The documentation will reflect later today with that section removed. #please-close


Recent user activity does not show deleted items

Describe the bug
I cannot see items that I deleted on my user profile.
Instead of showing the delete action, it removes the action of my created page, like it never existed.
Steps To Reproduce
Steps to reproduce the behavior:

Create book
Check history
Delete book
Check history
Not showing delete action

Expected behavior
Show delete action on my user profile.
Screenshots

Your Configuration (please complete the following information):

Exact BookStack Version (Found in settings): BookStack v0.29.2

Additional context
I tried it with bookshelves, books, pages, chapters. All delete actions not showed.

Hi @Venkinovec,
This is kind of by design. Since the items are no longer there we can no longer calculate permissions on them so we don't show those activities, to avoid exposing potentially private information.
The activity is still within the database though. More detail about this can be found in #1167. Since this is effectively a duplicate of #1167 I will close this.


connected notes and added fix associated with it #29

Description
Realised JPA and hibernate automatically creates table for entity within database but still configured it in data.sql as well as fix an issue regarding the notes description going out of board.
Closes #29
Type of Change

[ ] Bug fix
[X] New/Update feature
[ ] New/Update Documentation

Approved and looks good


DAP eventually fails to reconnect

Hello! I have a minor issue I'd like to report: after several hours left unattended, DAP can fail to stay connected to a voice channel. When this happens, selecting a different voice channel & back didn't seem to work, or at least not fully: the bot account was in the channel, but audio didn't go through until a full program restart.
Errors log:
Disconnected from voice... Reconnecting in 1.18s.
Traceback (most recent call last):
  File "discord\voice_client.py", line 383, in poll_voice_ws
  File "discord\gateway.py", line 889, in poll_event
discord.errors.ConnectionClosed: Shard ID None WebSocket closed with 1006
Attempting a reconnect in 0.61s
Traceback (most recent call last):
  File "discord\client.py", line 551, in connect
  File "discord\gateway.py", line 578, in poll_event
discord.errors.ConnectionClosed: Shard ID None WebSocket closed with 1000
Attempting a reconnect in 1.14s
Traceback (most recent call last):
  File "discord\client.py", line 551, in connect
  File "discord\gateway.py", line 578, in poll_event
discord.errors.ConnectionClosed: Shard ID None WebSocket closed with 1000
Attempting a reconnect in 0.87s
Traceback (most recent call last):
  File "discord\client.py", line 551, in connect
  File "discord\gateway.py", line 578, in poll_event
discord.errors.ConnectionClosed: Shard ID None WebSocket closed with 1000
Disconnected from voice... Reconnecting in 0.41s.
Traceback (most recent call last):
  File "discord\voice_client.py", line 383, in poll_voice_ws
  File "discord\gateway.py", line 889, in poll_event
discord.errors.ConnectionClosed: Shard ID None WebSocket closed with 1001

I believe part of this failure to auto-reconnect may be witnessed by manually changing a server's voice region while connected; the "jump" in sessions can then result in these slightly different errors:
Disconnected from voice... Reconnecting in 1.27s.
Traceback (most recent call last):
  File "discord\voice_client.py", line 383, in poll_voice_ws
  File "discord\gateway.py", line 889, in poll_event
discord.errors.ConnectionClosed: Shard ID None WebSocket closed with 4000
Exception in voice thread Thread-20
Traceback (most recent call last):
  File "discord\player.py", line 598, in run
  File "discord\player.py", line 591, in _do_run
  File "discord\voice_client.py", line 598, in send_audio_packet
TypeError: str, bytes or bytearray expected, not NoneType
Disconnected from voice... Reconnecting in 1.77s.
Traceback (most recent call last):
  File "discord\voice_client.py", line 383, in poll_voice_ws
  File "discord\gateway.py", line 889, in poll_event
discord.errors.ConnectionClosed: Shard ID None WebSocket closed with 4000

In this specific case, selecting a different voice channel & back worked.
Again, this probably isn't a very important issue as I reckon this program is mainly aimed at active usage, but if you wanted to look into improving its long-term resiliency, I'd be happy to help try out fixes or improvements.
Cheers! (and by the way, the new UI looks great 😃)

Thanks! Network issues can be difficult to reproduce so I appreciate the reports. It is strange that it fails to reconnect automatically, I'll have to investigate.
Also, did you try switching the channel to None and back? That performs a hard reconnect to voice while switching to another room will simply move the bot.

Just had another session end in a stranger way, after much longer (roughly 2 whole days)... DAP.exe was nowhere to be found. Only one error, the same as before (line 383 & 889) was in the log file.
As for switching the channel to None and back, I can confirm it works to get voice "unstuck"! 🙂

Coming in with another report that exhibited a slightly different behaviour: the bot still showed in the channel, but doing the "switch to none & back" trick did not work to get voice through again, neither on Channel nor Server. A full app restart worked.
This one's pretty long:
(a bunch of line 383/889 tracebacks)
Traceback (most recent call last):
  File "discord\voice_client.py", line 383, in poll_voice_ws
  File "discord\gateway.py", line 889, in poll_event
discord.errors.ConnectionClosed: Shard ID None WebSocket closed with 1006
Attempting a reconnect in 1.63s
Traceback (most recent call last):
  File "discord\client.py", line 551, in connect
  File "discord\gateway.py", line 578, in poll_event
discord.errors.ConnectionClosed: Shard ID None WebSocket closed with 1000
Attempting a reconnect in 0.75s
Traceback (most recent call last):
  File "aiohttp\connector.py", line 967, in _create_direct_connection
  File "aiohttp\connector.py", line 830, in _resolve_host
  File "aiohttp\resolver.py", line 30, in resolve
  File "asyncio\base_events.py", line 792, in getaddrinfo
  File "concurrent\futures\thread.py", line 57, in run
  File "socket.py", line 752, in getaddrinfo
socket.gaierror: [Errno 11001] getaddrinfo failed

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "discord\client.py", line 548, in connect
  File "asyncio\tasks.py", line 442, in wait_for
  File "discord\gateway.py", line 296, in from_client
  File "discord\http.py", line 910, in get_gateway
  File "discord\http.py", line 185, in request
  File "aiohttp\client.py", line 1012, in __aenter__
  File "aiohttp\client.py", line 483, in _request
  File "aiohttp\connector.py", line 523, in connect
  File "aiohttp\connector.py", line 859, in _create_connection
  File "aiohttp\connector.py", line 971, in _create_direct_connection
aiohttp.client_exceptions.ClientConnectorError: Cannot connect to host discord.com:443 ssl:default [getaddrinfo failed]
Attempting a reconnect in 3.98s
Traceback (most recent call last):
  File "aiohttp\connector.py", line 967, in _create_direct_connection
  File "aiohttp\connector.py", line 830, in _resolve_host
  File "aiohttp\resolver.py", line 30, in resolve
  File "asyncio\base_events.py", line 792, in getaddrinfo
  File "concurrent\futures\thread.py", line 57, in run
  File "socket.py", line 752, in getaddrinfo
socket.gaierror: [Errno 11001] getaddrinfo failed

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "discord\client.py", line 548, in connect
  File "asyncio\tasks.py", line 442, in wait_for
  File "discord\gateway.py", line 296, in from_client
  File "discord\http.py", line 910, in get_gateway
  File "discord\http.py", line 185, in request
  File "aiohttp\client.py", line 1012, in __aenter__
  File "aiohttp\client.py", line 483, in _request
  File "aiohttp\connector.py", line 523, in connect
  File "aiohttp\connector.py", line 859, in _create_connection
  File "aiohttp\connector.py", line 971, in _create_direct_connection
aiohttp.client_exceptions.ClientConnectorError: Cannot connect to host discord.com:443 ssl:default [getaddrinfo failed]
Attempting a reconnect in 0.74s
Traceback (most recent call last):
  File "discord\client.py", line 551, in connect
  File "discord\gateway.py", line 578, in poll_event
discord.errors.ConnectionClosed: Shard ID None WebSocket closed with 1000
Attempting a reconnect in 1.22s
Traceback (most recent call last):
  File "discord\client.py", line 551, in connect
  File "discord\gateway.py", line 578, in poll_event
discord.errors.ConnectionClosed: Shard ID None WebSocket closed with 1000
Error on change_channel
Traceback (most recent call last):
  File "discord\voice_client.py", line 329, in connect
  File "discord\utils.py", line 352, in sane_wait_for
concurrent.futures._base.TimeoutError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "gui.py", line 277, in change_channel
  File "discord\abc.py", line 1122, in connect
  File "discord\voice_client.py", line 331, in connect
  File "discord\voice_client.py", line 423, in disconnect
AttributeError: 'VoiceClient' object has no attribute 'ws'
Error on change_channel
Traceback (most recent call last):
  File "discord\voice_client.py", line 329, in connect
  File "discord\utils.py", line 352, in sane_wait_for
concurrent.futures._base.TimeoutError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "gui.py", line 277, in change_channel
  File "discord\abc.py", line 1122, in connect
  File "discord\voice_client.py", line 331, in connect
  File "discord\voice_client.py", line 423, in disconnect
AttributeError: 'VoiceClient' object has no attribute 'ws'
Error on change_channel
Traceback (most recent call last):
  File "discord\voice_client.py", line 329, in connect
  File "discord\utils.py", line 352, in sane_wait_for
concurrent.futures._base.TimeoutError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "gui.py", line 277, in change_channel
  File "discord\abc.py", line 1122, in connect
  File "discord\voice_client.py", line 331, in connect
  File "discord\voice_client.py", line 423, in disconnect
AttributeError: 'VoiceClient' object has no attribute 'ws'
Task was destroyed but it is pending!
task: <Task pending coro=<Event.wait() done, defined at asyncio\locks.py:280> wait_for=<Future pending cb=[<TaskWakeupMethWrapper object at 0x00000249FD24A258>()]>>
Task was destroyed but it is pending!
task: <Task pending coro=<Event.wait() done, defined at asyncio\locks.py:280> wait_for=<Future pending cb=[<TaskWakeupMethWrapper object at 0x00000249FD24A3A8>()]>>
Task was destroyed but it is pending!
task: <Task pending coro=<Event.wait() done, defined at asyncio\locks.py:280> wait_for=<Future pending cb=[<TaskWakeupMethWrapper object at 0x00000249FCE0C3D8>()]>>

While I hope these logs may prove useful in helping improve the software, I can't help but think they could help even more if they had timestamps on each line—right now we don't know what is directly linked to what because of this...

I was able to reproduce some of the behavior by sleeping my computer for a few hours while leaving the program running, however the None toggle trick still worked for me. You are right about the the timestamps, I'll add that to the program along with some options to get more detailed logs.

The 2.0 exe should have timestamps for errors now. For verbose logging, you'll have to launch the program from CMD or PowerShell with .\dap.exe --verbose, and it will create a discord.log in the same dir.

So, the bot eventually got itself stuck in a "reconnected, but not actually transmitting voice" state, here's the log. I trimmed the first 1000 lines or so, since they seemed like they contained potentially sensitive information (server/user IDs? shrug)
1009_discord.log
The most intriguing thing here is that there is nothing more is being logged after 22:19:52, including dis/connections to the channel, which did show up previously... except for ONE error that is not in discord.log, but dap_errors.log still... and close to 45 minutes later:
2020-10-09 23:05:06 Exception in voice thread Thread-3
Traceback (most recent call last):
  File "discord\player.py", line 598, in run
  File "discord\player.py", line 591, in _do_run
  File "discord\voice_client.py", line 598, in send_audio_packet
TypeError: str, bytes or bytearray expected, not NoneType

Hope that helps!

Looking through the logs it looks like everything is fine before it stopped recording logs. I do see a bunch of packet loss but the program managed to survive through those events.
The fact that the program did not crash but stopped producing logs is extremely confusing. In my own testing, I did get the bot to fail to reconnect to voice api.
Disconnected from voice... Reconnecting in 1016.26s.

But I think your bot is somehow getting completely disconnected from discord.
Anyways thanks for the reports, I think we are getting closer to the issue.  I'm looking to collect enough information so that I can create a decent bug report to discord.py and hopefully get these issues solved.

I had the same problem this past weekend when running a game, you can see attached all the exceptions that happened.  It kept disconnecting quite often, but my internet service provider really sucks so they drop packets now and again.  So if this is a persistent connection type thing, might need to write a catch around this and just reconnect automatically because people with bad internet (or using this during peak times on their cable modems when the rest of the neighborhood is using up all the bandwidth) will have the same issues.
https://pastebin.com/HqVhSbhd


Does Self-hosted Linux agents support "Oracle Linux 8"?

[Enter feedback here]
Does Self-hosted Linux agents support "Oracle Linux 8"?
I could confirm that Oracle Linux 7 is supported, but Oracle Linux 8 is not mentioned in the documentation(https://learn.microsoft.com/en-us/azure/devops/pipelines/agents/v2-linux?view=azure-devops).

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

ID: f64ca92c-b7d5-3142-71fb-306d32a37281
Version Independent ID: dfef40cf-2f50-4c03-bc97-bafa6c305108
Content: Deploy an Azure Pipelines agent on Linux - Azure Pipelines
Content Source: docs/pipelines/agents/v2-linux.md
Service: azure-devops-pipelines
Sub-service: azure-devops-pipelines-agents
GitHub Login: @steved0x
Microsoft Alias: sdanie

I am looking for same information

@miezure, @mohitora -- thank you for your question.  Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

@steved0x -- please look into this potential documentation issue.

Related to https://github.com/MicrosoftDocs/azure-devops-docs/issues/10469.


Call for maintainers

Anyone interested in helping maintain this project? Unfortunately, I don't have the time to dedicate to it :]

I'm willing to continue to help maintain this project. I think the only thing I don't have access to is publish rights on npm.

@JoeDoyle23 what is your npm username?

joedoyle23

@JoeDoyle23 added publish rights. Thanks!


DOC Add Expr background and supported expressions

There is some great info in the src/app_model/expressions/_expressions.py module docstring, which is unfortunately not included in the documentation. Would you consider including this / is there a way to easily do this in the current documentation system?
Also would a list of supported comparisons / ops be useful?

sure, happy to have it included

Would you just add to docs/expressions.md or is there a better way?

Let's move whatever you want from the module docstring  into the body of the Expr class docstring (which I think is already documented there), that way we can avoid duplication.


Return periods

Coverage increased (+0.4%) to 83.388% when pulling 956f8ab1c6f353e35443609f7811359d83e0882d on spence97:return_periods into 023608ce47b9e27e3ae451bceb1c8615b8485607 on erdc-cm:master.


ci/docs: split linkcheck

What does this PR do?
the link check seems to be failing, but as it was an optional step, no one was paying attention to it so lets make it as regular check but not required

Before submitting

Was this discussed/agreed via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
Did you make sure to update the documentation with your changes? (if necessary)
Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
Did you list all the breaking changes introduced by this pull request?
Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

Reviewer checklist

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

cc @borda @carmocca

Because of the edit of group-check, we will need a force merge - @lantiga


[Spike] Run one off task with Celery 4.x to test if it can use ECS assigned execution role

As a notification developer, I want to use the most recent celery version so that we can verify updating celery is required to use the existing deploy/task execution role
Acceptance criteria:
WHEN sample celery fargate task is run
AND notification_ecs_task_execution role is used
THEN new sample SQS queue is created
Checklist:

[x] create or use docker image to include the latest celery - the image should have a simple code so it creates one new SQS queue
[x] create new task definition to use the image
[ ] run the task and confirm the queue is made

Additional info:

can use the AmazonSQSFullAccess policy
#30 is related to this user story

We created a task definition called celery-test and activated it.  The task was not able to create SQS but showed the error message [2020-05-27 18:22:52,163: CRITICAL/MainProcess] Unrecoverable error: ClientError('An error occurred (AccessDenied) when calling the GetQueueAttributes operation: Access to the resource https://us-east-2.queue.amazonaws.com/ is denied.',).
This proves that upgrading to the latest celery allows a task to authenticate using the AWS metadata.

Used another role and the task created the SQS queue. So we should upgrade celery.

Dockerfile:
FROM python:3.6-alpine

ARG DEBIAN_FRONTEND=noninteractive
ENV PYTHONIOENCODING UTF-8
ENV PYTHONDONTWRITEBYTECODE 1
RUN apk add --no-cache bash build-base git gcc musl-dev g++ make libffi-dev libmagic curl-dev libressl-dev && rm -rf /var/cache/apk/*
RUN python -m pip install wheel
RUN set -ex && mkdir /app

COPY requirements.txt /app/requirements.txt
WORKDIR /app
RUN pip install -r requirements.txt

COPY . /app

CMD ["sh", "-c", "celery -A celery_run worker --loglevel=info"]

celery_run.py:
from celery import Celery

app = Celery('tasks', broker='sqs://')
app.conf.task_serializer = 'json'

@app.task
def consume(msg):
    print(msg)

requirements.txt:
celery[sqs]==4.3.0


Dark Mode Website and Documentation

First, I want to thank you for Flask.  It's incredible.
I am requesting  a dark mode version of the website and documentation. It's not about my personal style preference. It's a usability issue since I am presently dealing with cataracts.
The white background of the flask websites and documentation are painful to read.  Dark mode provides relief by reducing the light noise while also improving contrast.   An alternate solution would be having the ability to change the style or generate a PDF version via Read the Docs.

I'm sure a contribution to https://github.com/pallets/pallets-sphinx-themes that adds a dark mode would be welcome :)

I don't personally use dark mode much, I find it harder to read paragraphs in (code is a different story, but the docs and website are mostly prose). If someone wants to contribute a dark theme, that's fine, but it needs to integrate with the system preference as well as be toggleable (with the preference stored in local storage).
local storage user choice > system preference > light mode default


project doesn't generate with workbox

I get this error on npm run generate
Error: workbox-sw is needed by the service worker and could not be copied over to your new site. Error: EPERM: operation not permitted, open 'c:\zzz\nuxt\app\static\workbox-sw.prod.v1.1.0.js.map'
    at Promise.all.then.catch (c:\zzz\nuxt\node_modules\workbox-build\build\lib\utils\copy-workbox-sw.js:17:13)
    at <anonymous>

Hi. This issue is sometimes happening with workbox during generate and is not by us. Currently, you may simply run it again and it works. I'll investigate more finding root cause.

Hi, I'm also seeing this when running nuxt build in addition to nuxt generate. As you said @pi0, re-running the command seems to do the trick, which is great when working locally or generating and hosting the static site somewhere. Unfortunately, I'm deploying a Nuxt app to Now and am running the build step at deploy-time, so it's causing the deployment to fail. Any chance you have some more details on this, or a link to the issue on workbox? I'm happy to look into this more and provide a fix if possible, as it's a bit of a critical issue for me and I've loved working with Nuxt and the PWA plugin. Thanks!

This should be fixed with @nuxtjs/workbox:^1.1.5 please upgrade :)


iOS 9 Support?

Hey,
It's not clear out of my perspective if this will support AirPlay Mirroring from iOS 9 because they changed a few things in the Protocol and even decryption from iOS8 to iOS9
cheers and great work!

Would be really interested too. I'm digging into this topic, but it seems a lot of changed in iOS 9, then iOS 9.3 and iOS 10..

i  have made iOS9 -  iOS11.3 to work.

Good new @notedit do you have some code to share ?


showPrompt() not working, no error being logged

We're setting up Armchair in our app using the very simple 1-step process. Our App ID is 100% correct, I've tried filling out additional fields, and I've tried setting debugEnabled(true) as well.
Calling showPrompt() provokes no response. onWillPresentModalView and shouldPromptClosure are both not firing either.
We have no way of seeing what error is being thrown or what is going on. We'd really love to use your SDK and we're trying to release ASAP, but if we can't figure this out we might need to turn to another product in the meantime until we can get this sorted.

What are the settings you are using to setup Armchair? Have you turned on debug mode or tried stepping through the code to see at what point the logic is skipping the rating screen? Start by placing a breakpoint here and stepping through


[GLUTEN-1524][CH]support function to_date/to_timestamp/dayofweek/weekday

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
(Fixes: #1524)

@rui-mo maybe we need to append to_date/to_timestamp to velox's blacklist before they are supported by velox.

@rui-mo maybe we need to append to_date/to_timestamp to velox's blacklist before they are supported by velox. I'm not sure, can you confirm it?

OK, let's wait for the CI.

Velox failed uts.  cc @rui-mo
2023-05-05T10:42:05.2000116Z E0505 18:42:05.199209 48497 Exceptions.h:68] Line: ../../velox/substrait/SubstraitParser.cpp:136, Function:parseType, Expression:  Parsing for Substrait type not supported: timestamp {
2023-05-05T10:42:05.2001968Z Type is not supported due to:Parsing for Substrait type not supported: timestamp {
2023-05-05T10:42:05.2002960Z   nullability: NULLABILITY_NULLABLE
2023-05-05T10:42:05.2003617Z }
2023-05-05T10:42:05.2004237Z   nullability: NULLABILITY_NULLABLE
2023-05-05T10:42:05.2004867Z }
2023-05-05T10:42:05.2005175Z
2023-05-05T10:42:05.2005560Z Validation failed for input types in ProjectRel.
2023-05-05T10:42:05.2006453Z , Source: RUNTIME, ErrorCode: NOT_IMPLEMENTED

I found the new adding trim functions introduced by (https://github.com/Kyligence/ClickHouse/pull/287) has bugs, and they are fixed in this pr.

2023-05-08T02:22:07.5509959Z [10/53] Performing download step (download, verify and extract) for 'zlib_ep'
2023-05-08T02:22:07.5512095Z FAILED: zlib_ep-prefix/src/zlib_ep-stamp/zlib_ep-download /opt/gluten/ep/build-arrow/build/arrow_ep/cpp/build/orc_ep-prefix/src/orc_ep-build/zlib_ep-prefix/src/zlib_ep-stamp/zlib_ep-download
2023-05-08T02:22:07.5516103Z cd /opt/gluten/ep/build-arrow/build/arrow_ep/cpp/build/orc_ep-prefix/src/orc_ep-build/zlib_ep-prefix/src && /usr/bin/cmake -P /opt/gluten/ep/build-arrow/build/arrow_ep/cpp/build/orc_ep-prefix/src/orc_ep-build/zlib_ep-prefix/src/zlib_ep-stamp/zlib_ep-download-RELEASE.cmake && /usr/bin/cmake -E touch /opt/gluten/ep/build-arrow/build/arrow_ep/cpp/build/orc_ep-prefix/src/orc_ep-build/zlib_ep-prefix/src/zlib_ep-stamp/zlib_ep-download
2023-05-08T02:22:07.5519128Z CMake Error at zlib_ep-stamp/zlib_ep-download-RELEASE.cmake:49 (message):
2023-05-08T02:22:07.5519987Z   Command failed: 1
2023-05-08T02:22:07.5520363Z
2023-05-08T02:22:07.5522580Z    '/usr/bin/cmake' '-Dmake=' '-Dconfig=' '-P' '/opt/gluten/ep/build-arrow/build/arrow_ep/cpp/build/orc_ep-prefix/src/orc_ep-build/zlib_ep-prefix/src/zlib_ep-stamp/zlib_ep-download-RELEASE-impl.cmake'
2023-05-08T02:22:07.5523880Z
2023-05-08T02:22:07.5524092Z   See also
2023-05-08T02:22:07.5524423Z
2023-05-08T02:22:07.5525633Z     /opt/gluten/ep/build-arrow/build/arrow_ep/cpp/build/orc_ep-prefix/src/orc_ep-build/zlib_ep-prefix/src/zlib_ep-stamp/zlib_ep-download-*.log
2023-05-08T02:22:07.5526481Z
2023-05-08T02:22:07.5526496Z

Velox compile failed, but seems not related to this pr  cc @rui-mo @zzcclp

@taiyang-li All checks were passed.


Add the profile name being used in the workspace settings

Hi,
I use the same workspace in different computers, but I'm using my Github account to sign in and use the same VS Code profiles in all those vs code instances, in every computer.
Problem is, when I open the same workspace in different computers in a local network (since the workspace is stored on a NAS shared folder), the same profile is not automatically activated when I open the workspace on a new computer, other profile (usually it inherits the profile the last opened project was using on the same vs code instance on that particular computer).
Is there some way in which the profile name could be stored in the workspace settings, so that the same workspace opened by different computers would automatically open the same profile (of course, this is a feature that would work only for one user) ?
Thanks for your attention, best regards,

As an example, you could export the profile to a .code-profile file, and in the .work-space settings add the ability to specify the path to the profile file to use. As was recently done with local workspace extensions

Hi,
thanks for your answer.
Are you talking about this ?
https://code.visualstudio.com/updates/v1_88#_preview-features ?
Exported the profile to an external .vscode-profile file, but I can't find
any key name in the .code-workspace file suitable to specify the
.vscode-profile in there ? Am I missing something ?
Best regards,
sias32 @.***> escreveu (domingo, 19/05/2024 à(s) 18:20):

As an example, you could export the profile to a .code-profile file, and
in the .work-space settings add the ability to specify the path to the
profile file to use. As was recently done with local workspace extensions
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/213036#issuecomment-2119305981,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFPCVVGBFQMBYMD5PRHTFLZDDNPRAVCNFSM6AAAAABH6PJMF2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJZGMYDKOJYGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Sorry, i apologize for misleading you. I wrote an example of how this idea can be implemented, it is not a solution.

Anyway, I questioned the Copilot bot about writing an extension specifically for this use case, that's what it answered:
I see, you're asking about the Visual Studio Code's internal API for changing the running profile. Unfortunately, as of now, there is no direct API in VS Code that allows you to switch between different running profiles programmatically.

However, you can programmatically modify the settings.json file which is essentially what the profiles are. You can use the `vscode.workspace.getConfiguration` and `update` methods to get and update settings respectively.

Here's a basic example:

```javascript
let config = vscode.workspace.getConfiguration('yourConfiguration');

config.update('yourSetting', 'newValue', vscode.ConfigurationTarget.Workspace);

In this example, 'yourConfiguration' is the section of the configuration (like 'editor', 'files', etc.), 'yourSetting' is the specific setting you want to change, and 'newValue' is the value you want to set.
Remember that this will only work if the extension is activated and has the necessary permissions to modify workspace settings.`
In this way, I could store the profile being used in the workspace, but there is no way to automatically retrieve its value through the vscode internal API in its current state.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Adding winzip artifact support

Signed-off-by: Atul-source<EMAIL_ADDRESS>

This PR is for     Support artifact archiving in windows zip format #35. issue


🤷🏻♂ Fix CodeQL scanny thing

It doesnt work maybe bc .net 8
Consider adding editorconfig

Done - CodeQL not supported yet and so removed CodeQL from workflows


Add LoongArch basic support

Dear maintainers,
I'd like to add one new architecture support for LoongArch.
The document and code we've already supported list as follow:
https://loongson.github.io/LoongArch-Documentation/README-EN.html
https://github.com/loongson
And we tested libcds on  3A6000 cpu with firebird links follow,
https://github.com/FirebirdSQL/firebird/pull/7531
Waiting for your reply, thanks

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

这是来自QQ邮箱的假期自动回复邮件。   您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

Dear maintainers, how about this PR? We've already tested on Loongson-3A6000 with all of the test passed as mentioned in
https://github.com/FirebirdSQL/firebird/pull/7531
thanks.

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。


Broken Link

this link is broken . please fix it
https://docs.substrate.io/tutorials/work-with-pallets/contracts-pallet/

@kia9372 This tutorial has been removed from the documentation for the time being because of compatibility issues. Please use the substrate-contracts-node to work with smart contracts for now you can find a link to the repo for it on the https://docs.substrate.ui/tutorials/ page. HTH.


Test S3A hadoopfs multipart upload to lakeFS

Fixes #1927 by adding a multipart-from-S3A test.  (#2556 attempted and failed to do this through Spark, so use a program that directly runs S3A.)

Reviewer: Please review!  The test is currently failing only because it cannot get logs.  It might be related to docker/compose#2067.  I will take a closer look and try to reproduce on my machine, but this should not interfere with the review.
More generally, if this is a recurring rough spot, we might want to find another way to verify the test actually used MPU.  Or we could skip that part -- it only verifies internal correctness of the test, which should remain...


GdipDrawString produces incorrect text

I need to draw multiline text on the bitmap.
if graphics has Pixel unit it works fine.
but i need to prepare bitmap for the exporting/printing, so i need to use the "Document" units.
unfortunately the text is cropped.
static void Main() {
    const string outputFile = "output.png";
    const float PxDpi = 96f;
    const float DocumentDpi = 300f;
    using(var bitmap = new Bitmap(300, 300))
    using(var graphics = Graphics.FromImage(bitmap))
    using(var font = new Font("DejaVu Sans", 18f))
    using(var stringFormat = new StringFormat()) {
        bitmap.SetResolution(PxDpi, PxDpi);
        graphics.FillRectangle(Brushes.White, 0, 0, bitmap.Width, bitmap.Height);
        graphics.PageUnit = GraphicsUnit.Document;
        var pixelBounds = new RectangleF(50, 50, 150, 250);
        var bounds = new RectangleF(
            Convert(pixelBounds.X, PxDpi, DocumentDpi),
            Convert(pixelBounds.Y, PxDpi, DocumentDpi),
            Convert(pixelBounds.Width, PxDpi, DocumentDpi),
            Convert(pixelBounds.Height, PxDpi, DocumentDpi));
        graphics.DrawString("HELLO WORLD", font, Brushes.Black, bounds, stringFormat);
        bitmap.Save(outputFile, ImageFormat.Png);
    }
}

static float Convert(float value, float fromDpi, float toDpi) {
    double scale = (double)toDpi / fromDpi;
    double result = value * scale;
    return (float)result;
}

on the windows platform I see the
Hello
World

on the png.
But on the linux I see the cropped text
H
W

on the output png.
The good side is a location of text start is right.
but the bound's size is too small.
Transfered from: https://github.com/dotnet/corefx/issues/33540
cc: @hughbe @akoeplinger @filipnavara

I would also like to solve this problem


Create a User Interface with CAP (SAP HANA Cloud)

Tutorials: https://developers.sap.com/tutorials/hana-cloud-cap-create-ui.html
When we try to create a SAP Fiori Worklist Application using Fiori: Open Application Generator, it does not create index.html file in webapp folder.

Yes there does appear to have been a change in the wizard very recently. Unfortunately this week I'm 100% consumed supporting SAP TechEd on site in Germany.  I will adjust this tutorial first thing when I return home next week.

I ran through this and the earlier tutorials in the mission. I updated screenshots and adjusted for these changes in the wizard. The source code of the index.html file is now in tutorial with a manual step to create it until I can figure out what the plan is for the wizard and why this feature was removed.


Optimization improvements

Implementation of the work started in #2214. Work in progress. Needs

More and better tests.
A general clean-up.
Docstrings all over.
Taking care of the TODOs littered around.

@torfjelde
Closes https://github.com/TuringLang/Turing.jl/pull/2227

This is at a stage where getting a review would be good. Please be pedantic and show no mercy, I need to learn the ways of the project. Some questions I've been wondering about myself:

Functions names, especially exported ones. estimate_mode and link were two that I hesitated on.
What's the convention for when to use implicit and explicit imports? I see a lot of source files mixing the two.
Do we want to keep using ModeResult? That's the type that OptimInterface.jl returns its results in. I made Optimization.jl return the same type for compatibility, but would probably have designed the return type differently if given free rein.
I've copied over tests from OptimInterface.jl's tests with minimal adaptation, resulting in some overlap. If the whole Optim.jl interface is on its way out though, might be worth leaving the overlap there, so that one can delete the OptimInterface.jl tests and not lose anything significant.
Depending on what we do with ModeResult and OptimInterface.jl tests, I might split off some of the tests to a new file of tests just for OptimizationCore.jl.

Having a look now:)

We can put all of it in estimate_mode, yes, though I would at least keep some subfunctions to separate concerns. I do find the abstraction clarifies it for me

I can see that; as I said, I think they make sense. But it does add quite a bit of an overhead for a uninitiated reader who just wants to see what's up.

he main reason for the struct was that whether the objective function and the initial value (and correspondingly the returned solution) have been transformed to unconstrained space or not need to go hand in hand.

Makes a lot of sense! But I think after one of the changes suggested, this shouldn't really be a worry anymore.
Specifically, you can just do the following once:
    ld = p.log_density
    varinfo = DynamicPPL.link(
        DynamicPPL.unflatten(ld.varinfo, copy(p.init_value))
        ld.model
    )
    ld = Accessors.@set ld.varinfo = varinfo

if we need to work in unconstrained space, and then use
    varinfo_new = DynamicPPL.unflatten(varinfo, solution.u)
    # `getparams` performs the invlinking, if needed, etc.
    vns_vals_iter = getparams(prob.model, varinfo_new)
    syms = map(Symbol ∘ first, vns_vals_iter)
    vals = map(last, vns_vals_iter)
    return ModeResult(
        NamedArray(vals, syms),
        solution,
        -solution.objective,
        prob.log_density
    )

(copy-paste from inline comments above) once you need to convert to a ModeResult. With this, there shouldn't be any other handling of unconstrained, etc.
Does that make sense?

Fair point about copy-paste, sorry about that. I would usually make moving and editing separate commits, but this time the code went through a liquefy-and-reassemble stage in the beginning, and the first few commits do a bit of everything.

No worries:) I also should have mentioned this before, and I every much know what you mean:)

I think I've addressed all the points you raised @torfjelde. The big code-move where OptimisationCore.jl got merged into Optimisation.jl is in the first commit of the new batch, if you want to skip it in the diff. With VarInfo handling converting between constrained/unconstrained the way you pointed out above I agree that the ModeEstimationProblem struct became overkill, so I did remove it.
Let me know if you have any other comments. Before we merge, I'll merge main into this branch and add the := test you added for the Optim.jl interface.

Actually, I'm gonna sort out the merge conflicts now, because otherwise CI isn't running, and it was failing earlier. Some of the failures seemed unrelated to this PR, but I need to investigate. If you don't want the merge commit in your diff, there's a button in the Files changed tab that allows you to select any range of commits to view the diff for.

Awesome @mhauru :) Having a look now!

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 120 (0.0%)  changed or added relevant lines in 2 files are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage remained the same at 0.0%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

ext/TuringOptimExt.jl
0
20
0.0%

src/optimisation/Optimisation.jl
0
100
0.0%

Files with Coverage Reduction
New Missed Lines
%

ext/TuringOptimExt.jl
2
0.0%

src/optimisation/Optimisation.jl
3
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
0

Relevant Lines:
1501

💛  - Coveralls


Does someone from core want to submit a talk to CfgMgmtCamp?

Hi, I'm part of the organizing team for the upcoming https://cfgmgmtcamp.eu/ghent2020/ in Ghent. If someone would like to submit a very technical cue talk, we'd love to have it! Let me know ASAP =D
Thanks!

That will almost certainly work for me. I'm not far away and Belgium is awesome. :)
Feel free to contact me directly.

Great!
@mpvl Best thing to do is just submit to the CFP listed above, and ping me when done. Any questions, also feel free to ping. I'm purpleidea on Freenode IRC or email in git log.
Cheers!

Okay, will do tomorrow or latest in the weekend, if okay. Intense day tomorrow!
Thanks!

This issue has been migrated to https://github.com/cue-lang/cue/issues/212.
For more details about CUE's migration to a new home, please see https://github.com/cue-lang/cue/issues/1078.


Link to spectrum chat in readme doesn't work

Thanks for report ❤
Sorry for the delay in answering, we are a bit busy with other activities 🙍 😿

@amilajack problem solved #77 😸


opennti_input-oc gives RPC error

I am attempting to use the input-oc method. Starting the opennti_input-oc results in the following logs:
docker logs:
2018-01-19T17:34:25Z I! Loaded outputs: influxdb
2018-01-19T17:34:25Z I! Loaded inputs: inputs.jti_openconfig_telemetry
2018-01-19T17:34:25Z I! Tags enabled: host=deda41e2fcc6
2018-01-19T17:34:25Z I! Agent Config: Interval:10s, Quiet:false, Hostname:"deda41e2fcc6", Flush Interval:5s
2018-01-19T17:34:25Z I! Started JTI OpenConfig Telemetry plugin
2018-01-19T17:34:25Z E! Failed to read: rpc error: code = 14 desc = JGrpcServer: Unknown RPC /telemetry.OpenConfigTelemetry/telemetrySubscribe `received

vMX logs:
Jan 19 09:34:25 JGrpcServerStart: Proto Desriptor object not found in proto map for RPC /telemetry.OpenConfigTelemetry/telemetrySubscribe for peer ipv6:::ffff:<IP_ADDRESS>:44872
Jan 19 09:34:25 AllocCallMem:GRPC Server Call Completion queue created successfully
Jan 19 09:34:25 AllocCallMem:GRPC Server Call Details initialized successfully
Jan 19 09:34:25 TerminateClientThreads: Number of grpc clients connected: 0
Jan 19 09:34:25 JGrpcServerStart: Proto Desriptor object not found in proto map for RPC /telemetry.OpenConfigTelemetry/telemetrySubscribe for peer ipv6:::ffff:<IP_ADDRESS>:44872
Jan 19 09:34:25 AllocCallMem:GRPC Server Call Completion queue created successfully
Jan 19 09:34:25 AllocCallMem:GRPC Server Call Details initialized successfully
Jan 19 09:34:25 TerminateClientThreads: Number of grpc clients connected: 0
Jan 19 09:34:25 JGrpcServerStart: Proto Desriptor object not found in proto map for RPC /telemetry.OpenConfigTelemetry/telemetrySubscribe for peer ipv6:::ffff:<IP_ADDRESS>:44872
Jan 19 09:34:25 AllocCallMem:GRPC Server Call Completion queue created successfully
Jan 19 09:34:25 AllocCallMem:GRPC Server Call Details initialized successfully
Jan 19 09:34:25 TerminateClientThreads: Number of grpc clients connected: 0
Jan 19 09:34:25 JGrpcServerStart: Proto Desriptor object not found in proto map for RPC /telemetry.OpenConfigTelemetry/telemetrySubscribe for peer ipv6:::ffff:<IP_ADDRESS>:44872
Jan 19 09:34:25 AllocCallMem:GRPC Server Call Completion queue created successfully
Jan 19 09:34:25 AllocCallMem:GRPC Server Call Details initialized successfully

telegraf.tmlp:
[[inputs.jti_openconfig_telemetry]]
  server = "<IP_ADDRESS>:50051"
  sampleFrequency = 15000
  sensors = [
   "/junos/system/linecard/interface/",
   "/junos/system/cpu/memory/",
   "/junos/system/linecard/packet/usage/",
   "/junos/system/linecard/fabric/"
  ]
  data_format = "influx"```

@mcaulifn , I am facing the same issue using an MX80 router running 19.3. How did you actually resolve that issue, just by restarting the router?


Configurable HttpCompletionOption for generated c# Client

We have a problem with file uploads when using .net core and docker.
The response content cannot be read when using file upload because of
https://github.com/dotnet/corefx/issues/16259
We should be able to change the HttpCompletionOption to ResponseContentRead
in client_.SendAsync(...).
I think this might be usefull in other scenarios too.

Oh, this is bad... Will this be fixed?

I hope so...
I think a easy solution could be to prevent the generation of a client operation if
it is allredy implemented in the base class or via an attribute.
Therefore we are aple provide custom implementations for certain methods.

You can hide methods with the ProtectedMethods setting

ah ok thanks


PowerTab does not work on Powershell Core on *Nix

work/tut/ni-sshj-test via ☕ v11.0.8 on ☁  eu-west-1 took 1m15s
❯ pwsh
PowerShell 7.0.3
Copyright (c) Microsoft Corporation. All rights reserved.

https://aka.ms/powershell
Type 'help' to get help.

MethodInvocationException: /Users/pavel/.local/share/powershell/Modules/PowerTab/1.1.0/PowerTab.psm1:10
Line |
  10 |      [Void][System.Reflection.Assembly]::LoadWithPartialName("System.W …
     |      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Exception calling "LoadWithPartialName" with "1" argument(s): "Could not load file or assembly
     | 'System.Windows.Forms, Culture=neutral, PublicKeyToken=null'. Operation is not supported. (0x80131515)"

Launch the setup wizard to create a PowerTab configuration file and database?
PowerTab can be setup manually without the setup wizard.
[N] No  [Y] Yes  [?] Help (default is "Y"):

Where should the PowerTab configuration file and database be saved?
Any existing PowerTab configuration will be overwritten.
[P] Profile Directory  [I] Installation Directory  [A] Application Data Directory  [S] Isolated Storage  [O] Other Directory
[?] Help(default is "P"):

Update current profile to automatically import PowerTab?
To manually update the current profile or another profile, select 'No'.
[N] No  [Y] Yes  [?] Help (default is "Y"):
Get-Content: /Users/pavel/.local/share/powershell/Modules/PowerTab/1.1.0/TabExpansionUtil.ps1:701
Line |
 701 |          [Byte[]]$byte = Get-Content -Encoding Byte -ReadCount 4 -Tota …
     |                                                ~~~~
     | Cannot process argument transformation on parameter 'Encoding'. 'Byte' is not a supported encoding name. For
     | information on defining a custom encoding, see the documentation for the Encoding.RegisterProvider method.
     | (Parameter 'name')

Update .NET type list in tab completion database from currently loaded types?
Loading .NET types.
[Y] Yes  [A] Yes to All  [N] No  [L] No to All  [S] Suspend  [?] Help (default is "Y"):
                                                                                Update WMI class list in tab completion database?                                                                                Loading WMI classes.                                                                                [Y] Yes  [A] Yes to All  [N] No  [L] No to All  [S] Suspend  [?] Help (default is "Y"):                                               New-Object: /Users/pavel/.local/share/powershell/Modules/PowerTab/1.1.0/TabExpansionLib.ps1:693                                       Line |                                                                                693 |          $Options = New-Object System.Management.EnumerationOptions                                                                 |                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~                                                                 | Exception calling ".ctor" with "0" argument(s): "System.Management currently is only supported for Windows desktop                  | applications."                                                                                InvalidOperation: /Users/pavel/.local/share/powershell/Modules/PowerTab/1.1.0/TabExpansionLib.ps1:694                                 Line |                                                                                694 |          $Options.EnumerateDeep = $true                                                                                |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~                                                                                | The property 'EnumerateDeep' cannot be found on this object. Verify that the property exists and can be set.                                                                                InvalidOperation: /Users/pavel/.local/share/powershell/Modules/PowerTab/1.1.0/TabExpansionLib.ps1:695                                 Line |                                                                                695 |          $Options.UseAmendedQualifiers = $true                                                                                |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~                                                                                | The property 'UseAmendedQualifiers' cannot be found on this object. Verify that the property exists and can be set.                                                                                InvalidOperation: /Users/pavel/.local/share/powershell/Modules/PowerTab/1.1.0/TabExpansionLib.ps1:698                                 Line |                                                                                698 |          foreach ($Class in (([WmiClass]'').PSBase.GetSubclasses($Opti …                                                            |                                ~~~~~~~~                                                                                | Unable to find type [WmiClass].                                                                                Update COM class list in tab completion database?                                                                                Loading COM classes.                                                                                [Y] Yes  [A] Yes to All  [N] No  [L] No to All  [S] Suspend  [?] Help (default is "Y"):
Get-CimInstance: /Users/pavel/.local/share/powershell/Modules/PowerTab/1.1.0/TabExpansionLib.ps1:718                                  Line |                                                                                718 |          foreach ($Class in (Get-CimInstance -ClassName "Win32_Classic …                                                            |                              ~~~~~~~~~~~~~~~                                                                                | The term 'Get-CimInstance' is not recognized as the name of a cmdlet, function, script file, or operable program.                   | Check the spelling of the name, or if a path was included, verify that the path is correct and try again.                                                                                Update computer list in tab completion database from 'net view'?                                                                      Loading computer names.                                                                                [Y] Yes  [A] Yes to All  [N] No  [L] No to All  [S] Suspend  [?] Help (default is "Y"):

Launch the setup wizard to create a PowerTab configuration file and database?
PowerTab can be setup manually without the setup wizard.
[N] No  [Y] Yes  [?] Help (default is "Y"):

Where should the PowerTab configuration file and database be saved?
Any existing PowerTab configuration will be overwritten.
[P] Profile Directory  [I] Installation Directory  [A] Application Data Directory  [S] Isolated Storage  [O] Other Directory
[?] Help(default is "P"):

Update .NET type list in tab completion database from currently loaded types?
Loading .NET types.
[Y] Yes  [A] Yes to All  [N] No  [L] No to All  [S] Suspend  [?] Help (default is "Y"):
                                                                                Update WMI class list in tab completion database?                                                                                Loading WMI classes.                                                                                [Y] Yes  [A] Yes to All  [N] No  [L] No to All  [S] Suspend  [?] Help (default is "Y"):                                                                                Update COM class list in tab completion database?                                                                                Loading COM classes.                                                                                [Y] Yes  [A] Yes to All  [N] No  [L] No to All  [S] Suspend  [?] Help (default is "Y"):                                                                                Update computer list in tab completion database from 'net view'?                                                                      Loading computer names.                                                                                [Y] Yes  [A] Yes to All  [N] No  [L] No to All  [S] Suspend  [?] Help (default is "Y"):

PowerTab version 1.1.0 PowerShell TabExpansion Library
Host: ConsoleHost
PowerTab Enabled: True
Add-TabExpansionComputer: /Users/pavel/.local/share/powershell/Modules/PowerTab/1.1.0/TabExpansionLib.ps1:548
Line |
 548 |              Add-TabExpansionComputer -NetView
     |              ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | The term 'net' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the
     | spelling of the name, or if a path was included, verify that the path is correct and try again.

PowerTab version 1.1.0 PowerShell TabExpansion Library
Host: ConsoleHost
PowerTab Enabled: True
Loading personal and system profiles took 47824ms.

It seems the main PowerTab module is using Forms (Windows only, I believe):
https://github.com/jasonmarcher/PowerTab/blob/0dc5377530c3f146beb91257a6da442c338ebd65/modules/powertab/src/PowerTab.psm1#L8-L11
You could try commenting out this line in your installed version, or (probably better) see if the PowerTabArgumentCompleters (PTAG) module works without it—I just skimmed the completions, they seem to be using the native Register-ArgumentCompleter function which should be compatible with core (also possibly TabExpansionPlusPlus, if you haven't tried it out yet)


no need for the div

this will provide react-native support for example.

This duplicates #10

Thanks for this PR, it's a duplicate of #10 so I'm closing this one.


code search with Bleve is broken after upgrading to 1.19.0 from 1.18.5

Description
code search with Bleve is broken after upgrading to 1.19.0 from 1.18.5,
web of code search always returns: No source code matching your search term found.
the startup and requests log looks normal:
2023/07/13 01:23:18 ...exer/code/indexer.go:270:func4() [I] [64af5206-5] Repository Indexer Initialization took 314.492001ms
2023/07/13 01:23:18 ...exer/code/indexer.go:302:populateRepoIndexer() [I] Populating the repo indexer with existing repositories
2023/07/13 01:23:18 ...er/issues/indexer.go:269:func3() [I] [64af5206-3] Issue Indexer Initialization took 314.970288ms
2023/07/13 01:23:18 ...exer/code/indexer.go:354:populateRepoIndexer() [I] Done (re)populating the repo indexer with existing repositories

router: completed GET /**/search?q=df for **:0, 200 OK in 57.0ms @ repo/search.go:18(repo.Search)

app.in
ISSUE_INDEXER_PATH   = /data/gitea/indexers/issues.bleve
REPO_INDEXER_ENABLED = true
REPO_INDEXER_PATH    = /data/gitea/indexers/repos.bleve
# UPDATE_BUFFER_LEN    = 20
MAX_FILE_SIZE        = 1048576
REPO_INDEXER_INCLUDE =
REPO_INDEXER_EXCLUDE = resources/bin/**

then I upgrade to 1.19.4, and it doesn't work
Gitea Version
1.19.0
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
docker
Database
PostgreSQL

update app.ini:
ISSUE_INDEXER_PATH   = indexers/issues.bleve
REPO_INDEXER_ENABLED = true
REPO_INDEXER_PATH    = indexers/repos.bleve

and it works now


remove unused code from services.xml

please adjust your services.xml file like in your docs https://docs.shopware.com/en/shopware-platform-dev-en/internals/plugins/plugin-quick-start#the-services-xml
because is a confuse

Hello,
thank you for creating this issue.
I have opened a ticket on our Issue Tracker for you. See the ticket link: https://issues.shopware.com/issues/NEXT-10794
Please use this ticket to track the state of your issue.


Pass mocks object to global config

Hi,
I have a feature request/idea.
I work in a non English speaking country, so we are using vue-i18n to create a bilingual app. It adds a global $t function. I have to do:
mocks: {
  $t: jest.fn()
}

in every test, or the test fails with $t is undefined, since almost all components have some form of text.
Should this be something handled by a test running? Not sure - if not, maybe we can have some way to set some default options for all tests, or all shallow renders or something.

Exact same issue as @lmiller1990. Also things like ElementUI's $alert,$confirm or own global helpers we have can be easily mocked everywhere :)

Yes please - big problem :(

A simple work around is to make a helper with all your commonly used mocks, for example:
// Let's call it mocks.js
import { shallow } from '@vue/test-utils'

const mocks = {
  $t: () => {}
}

const shallowWithMocks = (component, otherData, otherMocks) => shallow(component, {
  mocks: {
    ..mocks, ...otherMocks
  },
  ...otherData
})

export shallowWithMocks

import { shallowWithMocks } from './mocks.js'

const wrapper = shallowWithMocks(component, {
  propsData: { /* */ }
})

Note: I didn't test this, but I have something similar in a project somewhere. It probably has some typos.
I still think a config file would be good. I'd like to work on it, but not sure where to start. Maybe we can start by fleshing out an API? I'm thinking something along the lines of a test-utils.config.js that could be something like this:
export default {
  mocks: {
    // mocks to use in all tests
    // they should be merged with any mocks passed in `shallow` or `mount`
    // the ones from `shallow`, `mount` should take priority and override the ones in this config file
  },
  stubs: {
    /* I often find myself wanting to stub certain components a lot as well */
  }
}

You should be able to provide defaults for anything you can pass in the options object to shallow and mount.
I'd like to hear thoughts of everyone else too before starting hacking.

I think we can go ahead and add mocks to the global config object. We can open a separate issue for adding a config file.
@lmiller1990 would you like to create a PR that adds the ability to pass mocks to the config object?

@eddyerburgh Sure. I'll look at the existing way we do it with stubs and mimic that.
Good idea, we should make another issue for config. I can do that this evening, and add a summary of the discussion in here.

➕ for separated js config file

Here is issue for discussion around a separate config file: https://github.com/vuejs/vue-test-utils/issues/523

@eddyerburgh I had a go at implementing this just now. I hadn't used the config object before. I did the following:

new project with vue-cli v3, using jest
Create a TestStub.vue, containing <template>
TestStub
</template>
In, HelloWorld.vue, import TestStub.vue and render
In HelloWorld.spec.js, stub TestStub.vue with by doing

const wrapper = shallow(HelloWorld, {
  stubs: {
    'TestStub: '
T
'
  }
})

console.log(wrapper.html()) yields:

T

So far so good. Next, I decided to try using the config object. In HelloWorld.spec.js, this works:
import { shallow } from '@vue/test-utils'
import HelloWorld from<EMAIL_ADDRESS>const VueTestUtils = require('@vue/test-utils')

VueTestUtils.config.stubs['TestStub'] = '
T
'

describe('HelloWorld.vue', () => {
  it('renders props.msg when passed', () => {
    const wrapper = shallow(HelloWorld)

    console.log(wrapper.html())
  })
})

Now my question: where should I be able to define VueTestUtils.config to make it applicable in every unit test? I tried this in jest.config.js:
require('jsdom-global')()

const VueTestUtils = require('@vue/test-utils')

VueTestUtils.config.stubs['TestStub'] = '
A
'

But doing so had no impact in my tests - all components became  instead of inheriting the config as expected. I added require('jsdom-global')() since otherwise VueTestUtils throws errors.
After I understand where I should be able to do global mocks, I will be able to add mocks to the config, then start on #523 .
Thanks for this great lib!

You can use the Jest setupFiles option to run a file before the tests.


not good on Chinese words audio

Hi, I try the model on a piece of audio with the words of "广大党员干部正在积极学习。。。"
But the result is not good. The images below are from test_sample3_fake_audio_B_0_0.png to test_sample3_fake_audio_B_0_17.png

I think the sequence do not denote "广大", I do not know where is the problem.

And the mouth will have some slight open and close during two words. Even if I use the test_sample3.jpg and my audio to generate the sequence, the result is not as good as what you show on the github.

I mean the change of mouth shape is not smooth enough. What should I do to make the video as smooth as what you shown on github.

The model is only trained on English data, so I am afraid we cannot guarantee its performance with Chinese inputs.

I mean the change of mouth shape is not smooth enough. What should I do to make the video as smooth as what you shown on github.

Have you ever trained in Chinese dataset?

I mean the change of mouth shape is not smooth enough. What should I do to make the video as smooth as what you shown on github.

I have tested the project on windows with python=3.6 and pytorch=1.0. But the results are so terrible. So what is your test environment?


Enhancement: extend the 'spo site apppermission list' command to show the roles

We should enhance the spo site apppermission list command to show the roles granted to the permissions. The discussion orginated from https://github.com/pnp/cli-microsoft365/issues/2409

Hi @waldekmastykarz
I can work on this.

Awesome! I appreciate your help 👏

Hi @waldekmastykarz
Unfortunately, roles property cannot be expanded.
Error: Property 'roles' on type 'microsoft.graph.permission' is not a navigation property or complex property. Only navigation properties can be expanded.
The Graph API tried was as follows:
https://graph.microsoft.com/v1.0/sites/contoso.sharepoint.com,772dcab3-5d0c-4f5d-ada0-ed1e8c6273c4,c0f71fdc-624f-44b1-ae3e-4bc403ed4339/permissions?$expand=roles

We could consider an additional call for each permission to retrieve the assigned roles. What do you think?

Yes, I was thinking of same. I will implement accordingly. Thank you :)


Please assist using langchain RetrievalQA

Hi
I am need of some assistance please:
I am using AutoModelForCausalLM.from_pretrained so that I can use GPU,
I am trying to use langchains RetrievalQA to query pdfs.
Do you have some code that can assist please?
If I use RetrievalQA.from_llm then I get:
ValidationError: 1 validation error for LLMChain
llm
value is not a valid dict (type=type_error.dict)
Many Thanks
Garth Waters

Hi, in LangChain, you shouldn't use AutoModelForCausalLM.from_pretrained. You should use CTransformers class to create llm object:
from langchain.llms import CTransformers

llm = CTransformers(model='...', model_type='...')

qa = RetrievalQA.from_chain_type(llm=llm, ...)

See LangChain docs.
Also see:

https://github.com/marella/chatdocs
https://github.com/marella/chatdocs/blob/2927aa21ab12f815dbad75e5340e57b22daad3f4/chatdocs/llms.py#L70-L73
https://github.com/marella/chatdocs/blob/main/chatdocs/chains.py


Fix "surprising queries" samples.

We were setting the filter twice (overwriting the first) rather than
using a composite filter.  This was causing the query results to be even
more surprising than we intending.
PTAL @halgrimur @shun-fan

Argh - Good catch!
We probably should ask Arie to review the whole thing? @aozarov

Looks good to me.

BTW, maybe adding this reference in the test would help with understanding why these queries behave that way.

+1 on adding links to the documentation. We should probably get into the habit of doing this for any code we include in the docs.


typo in sklearn requirement

The sklearn library is installed via scikit-learn as opposed to sklearn (eg. pip install scikit-learn). Similarly dateutil is installed via pip install python-dateutil.

Thanks so much for this, @vzhong!


Support using Self or projections inside an RPIT/async fn #103491

Summary
Discuss @cjgillot's PR https://github.com/rust-lang/rust/pull/103491 " Support using Self or projections inside an RPIT/async fn "
Rationale

Scheduled for Nov 11 -- but 2 hours later than normal! Calendar event

We held this meeting.

https://rust-lang.zulipchat.com/#narrow/stream/326132-t-types.2Fmeetings/topic/2022.E2.80.9011-11.20PR.20review/near/309197876


feat(automate release)

via PAT https://code.visualstudio.com/api/working-with-extensions/publishing-extension#get-a-personal-access-token

Fixes https://github.com/prisma-labs/vscode-graphql/issues/45


Adding documentation to wrap.sh, added ability to retain env vars

This does two primary things:

Better documentation of wrap.sh. Hopefully as nix moves forward these bash incantations will start to get documented so that contributors aren't as confused as I was :grimacing:

Adds ability to add arbitrary environment variables to be stored in the wrapping script. Rather than just PATH and PYTHONPATH, you can specify any environment variable. This is especially useful when a package will read some environment variables (e.g. configuration) at runtime, and you want to be sure it will read the right ones.

Example:
with import <nixpkgs> {};

buildPythonPackage {
  name = "foo-0.1.0";
  src = ./.;
  setEnvVars = ["NUM_WORKERS" "CONFIG_FILE_PATH"];
  NUM_WORKERS = "4";
  CONFIG_FILE_PATH = "./config/cfg.json";
}

Then assuming that the setup.py for foo builds some executable, the derivation will produce a wrapped executable with something like
export PYTHONPATH=/nix/store/....
export PATH=/nix/store/.....
export NUM_WORKERS=4
export CONFIG_FILE_PATH=./config/cfg.json
exec /nix/store/.../bin/.foo-wrapped "${extraFlagsArray[@]}" "$@"

@domenkozar @chaoflow

Would be really nice to separate this work in two PRs:

documentation and semantic variables, big :+1:

setEnvVar support -> I think we should just expose something like pythonWrapperArgs = [ "--set MANPATH /"] as a general solution to pass flags to the wrapper scripts.

@domenkozar I can separate into two PRs. I'm not sure I love the pythonWrapperArgs idea because it relies on the specific way the makeWrapper function works, as opposed to being more high-level. Also, makeWrapper is another undocumented function.

@adnelson but wrapPython is using makeWrapper, hiding that away does more trouble than good. I'd really be happy to see docs about makeWrapper arguments

@domenkozar that's true, but the idea is that passing literal arguments to makeWrapper means that (a) developers have to go look at how makeWrapper works to see how to supply arguments, and (b) if the functionality of makeWrapper were to change at some point in the future, it would break python derivations. I'm not completely set, and I think it doesn't make a huge difference either way, but I prefer the higher level approach which doesn't have these issues.

On Sun, May 24 2015, Allen Nelson wrote:

@domenkozar that's true, but the idea is that passing literal arguments to makeWrapper means that (a) developers have to go look at how makeWrapper works to see how to supply arguments, and (b) if the functionality of makeWrapper were to change at some point in the future, it would break python derivations. I'm not completely set, and I think it doesn't make a huge difference either way, but I prefer the higher level approach which doesn't have these issues.

makeWrapper is the standard way of making wrappers. If that were to
change, a lot in nixpkgs would need to be changed.
--
Florian Friesdorf<EMAIL_ADDRESS>GPG FPR: 7A13 5EEE 1421 9FC2 108D  BAAF 38F8 99A3 0C45 F083

@domenkozar @chaoflow updated

@thinkpad20 thanks! Could you also add documentation to buildInputPackage function at https://github.com/NixOS/nixpkgs/blob/master/doc/language-support.xml#L357?

Documentation added.

@adnelson thank you, this is a well needed contribution! (someone has to go through python-packages.nix someday and make current makeWrapper calls use the new argument)


Custom CSS with remote_theme

Hello and thanks for building this theme!
Is there a simple way to add some custom CSS when using this with remote_theme? I don't want to override any of the theme's style files.
If I understand, the imports work like
/assets/css/style.css is built by assets/css/style.scss, which includes _sass/portfolYOU.scss, which includes _sass/{variables,base,navbar,projects,articles,autumn,skills,timeline,404}.scss.
What about importing an empty _sass/custom.scss file as well, which can safely be overriden?

Try overriding assets/css/style.scss and add your custom style there.

This does not seem to work for the assets directory. If I try to override assets/css/style.scss,

The original assets/css/style.scss is compiled into _site/assets/css/style.css
My file is copied into _site/assets/css/style.scss without being compiled

Hey @webmaster128, I have added the required file, please test it and tell me if it really solves the issue.
Thanks in advance.

Hi @YoussefRaafatNasry, thank you!
Unfortunately overriding partials inside of _sass does not work. This matches what is documented here:

Note: To make more extensive changes and customize the Sass partials bundled in the gem. You will need to copy the complete contents of the _sass directory to <your_project> due to the way Jekyll currently reads those files.

https://mmistakes.github.io/minimal-mistakes/docs/stylesheets/#customizing
and

Note that the sass_dir becomes the load path for Sass imports, nothing more. This means that Jekyll does not know about these files directly.

https://jekyllrb.com/docs/assets/
For this reason I'd revert 9d883d557010083a64c137eada38ca7f1a9a4e02.

However, reading some more documentation I realized that your first suggestion works if I include the empty ---/--- block. It was not clear to me that this is crutial for the Jekyll override mechanism.
What I tried before (not working):
@import "portfolYOU";

body {
  background-color: greenyellow;
}

What actually works 🎉 :
---
---
@import "portfolYOU";

body {
  background-color: greenyellow;
}

The great thing is that it allows you to append and prepend to the portfolYOU style.

Thanks @webmaster128 for your efforts!
I've forced push to exclude 9d883d5.


Allow clippy::manual_map

We manually expand these to reduce the amount of LLVM IR generated. (#183)

@bors r+

:pushpin: Commit 24e866117cddc10e1cb0911ca315ee1308f1faf6 has been approved by Amanieu

:sunny: Test successful - checks-travis
Approved by: Amanieu
Pushing ae5eaa7ddac1ef0a6531bc0f48425c1236de999a to master...


Re consider the search order

Right now the following searches are applied in order,

Exact match Qnode/Pnode search if the input string matches the pattern
Exact match label search, ordered by pagerank
textlike search, ordered by score*pagerank
textmatch search, ordered by score*pagerank

As discussed in the KGTK meeting on Dec 7th, 2021, this order should be revised, here is the proposed order,

Exact match Qnode/Pnode search if the input string matches the pattern
textmatch search, ordered by score*pagerank, only if the input string's length is >= 3
textlike search, ordered by score*pagerank, only if there are too many or too few results from 2
Exact match label search, ordered by pagerank

Me and Pedro agreed to remove the text like search

Removed text like search. Reorder as described


Design Tokens: Update format for Android

Summary

Update space tokens to use dimen format with dp
Links

[Jira](link to Jira ticket(s))
[TDD](link to Paper doc)
[Figma](link to Figma file)

Checklist

[ ] Added unit and Flow Tests
[ ] Added documentation + accessibility tests
[ ] Verified accessibility: keyboard & screen reader interaction
[ ] Checked dark mode, responsiveness, and right-to-left support
[ ] Checked stakeholder feedback (e.g. Gestalt designers)

@ayeshakmaz Could you add a little bit more context to this change? So we can follow along?


Add Identity types

Add identity types as in the paper and prove univalence expressed using Id instead of Path.
@simhu : Didn't you do this at some point?

See the branch equiv; AFAIR, the proof that univalence holds for Id is not complete but it should be really easy the way univalence is stated in the paper.  (I unfortunately lost some local files during my HD crash..)

There is also something in the branch defeq.

Err, sorry, I meant defeq of course--thanks Bas!

Resolved by https://github.com/mortberg/cubicaltt/pull/49


Issue with jetty-server dependency upgrade

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
Maven
Package manager version
3.6.3
Language version
Java 8
Manifest location and content before the Dependabot update
No response
dependabot.yml content
No response
Updated dependency
org.eclipse.jetty:jetty-server from 9.4.44.v20210927 to 9.4.51.v20230217
What you expected to see, versus what you actually saw
There seems to be an issue with upgrading org.eclipse.jetty:jetty-server dependency:
Previously, we had version 9.4.44.v20210927 in pom.xml.
A few days ago dependabot has created a PR to automatically upgrade dependency to 10.0.14 version, which is currently unsuitable for us.
At the same time, dependabot alert says the following:
Patched in Jetty versions

9.4.51.v20230217 - via PR #9345
10.0.14 - via PR #9344
11.0.14 - via PR #9344

So, we have applied 9.4.51.v20230217 version.
However, dependabot alert does not go away and PR with an upgrade to 10.0.14 version is still open.
Also, dependabot fails to create PR with the proper version:
Dependabot cancelled the update as there is already an existing pull request for the fixed version
There is already an existing pull request for the fixed version<EMAIL_ADDRESS>
Closing PR or using @dependabot recreate command does not solve the issue.
Could you please suggest any solution?
Native package manager behavior
No response
Images of the diff or a link to the PR, issue, or logs
No response
Smallest manifest that reproduces the issue
No response

In another component we have faced the following situation:
Automatically created dependabot PR was patched manually to use jetty-server version 9.4.51.v20230217 and merged.
However, after that dependabot created a new PR to upgrade jetty-server from 9.4.51.v20230217 to 10.0.14, which is unsuitable for us.
Security alerts: OutOfMemoryError for large multipart without filename in Eclipse Jetty, Eclipse Jetty's cookie parsing of quoted values can exfiltrate values from other cookies.
Both alerts say that the patched version is 9.4.51 (which is already used) but dependabot creates PR for version 10.0.14.


Git Reports Issue

Submitter: Anthony Bartomeo
Game
I noticed some of the cards in the deck builder do not contain any artwork.  Is the artwork for the cards being uploaded on a regular basis?

Will need more details. Were the cards completely black only with text? Or they just did not have the art/image at the center?
Because if it was completely black, it is probably because of the incorrectly cached images. You could try to clear the cache and load again.


feat: YouTube controller design layout for embed videos

Pre-Flight checklist

[x] Screenshots and design review for any changes that affect layout or styling

[x] Desktop screenshots
[x] Mobile width screenshots

[x] Testing

[ ] Code is tested
[x] Changes have been manually tested

What are the relevant tickets?
Closes #1103 and https://github.com/mitodl/hq/issues/1196
What's this PR do?
This PR updates design layout for embed videos. It also fixes the bug issued in https://github.com/mitodl/hq/issues/1196 by updating youtubeParams.
How should this be manually tested?
For any of these changes (start time, end time, or general layout related changes), sometimes they can take a little time even after clearing cache and using incognito window. If it still does not work, try clearing cache a couple of times, give it a minute or two, and then try again.
Firstly, follow the testing steps on this PR. They will be repeated exactly as they are, but you will be checking out to this branch instead.
Next, we will have to test a sample course to test for embed videos. And we will also be testing that start_time and end_time of videos are functioning properly with the new changes.
Clear cache, and use new incognito window.
Use this course: 18.02sc-fall-2010

Checkout to this branch
Make sure you update the above course in your .env
Run yarn start course
Go to this link: http://localhost:3000/pages/2.-partial-derivatives/part-a-functions-of-two-variables-tangent-approximation-and-optimization/session-27-approximation-formula/
Note: You can do the next step (5) in combination of step (7) to save time.
For the recitation video (embed video), Make sure the YouTube controls are working + "VIEW VIDEO PAGE" is in black tab bar (clickable) + Download icon is there with "Download video" and "Download transcript" links. The look should match the InVision designs in these links:
https://projects.invisionapp.com/d/main?origin=v7#/console/21537330/472347461/preview
https://projects.invisionapp.com/d/main?origin=v7#/console/21537330/472347462/preview
We will also make sure our single video YouTube controller is also working on embed video courses: Go to this link http://localhost:3000/resources/clip-approximation-formula/ and make sure the changes you tested in the above PR are also prevalent on this link.
Now we will edit start_time and end_time for all their possible combinations for the recitation video in step 4. Values for all combinations can be: {(ST: 5, ET: 9), (ST: 5, ET: ''), (ST: '', ET: 9), (ST: '', ET: '')}
Go to this markdown file for the recitation video in your local ocw-content-rc directory: 18.02sc-fall-2010/content/resources/tangent-plane-approximation.md
You will see that currently the start_time and end_time are set to empty in the markdown file. Test the video in step 4 for this case. Similarly test for the remaining 3 test cases in (7) step by editing the video's markdown file and saving it. Terminate the project and run yarn start course for each of the test case (along with clearing cache measures). Make sure that the video starts to the given start_time, and automatically ends at the given end_time (if you force this behavior through keyboard arrow keys, the video will not end at the given end time. So let the video end naturally at the given end time mentioned in the file)

Screenshots (if appropriate)

I'd really like to get this merged asap, but I have to ask -- does our test course for playwrite testing include this type of markdown data?

@pdpinch @ibrahimjaved12  re

I'd really like to get this merged asap, but I have to ask -- does our test course for playwright testing include this type of markdown data?

If it doesn't, maybe that can be an issue for later.

It does not, but it should.
We're a little limited in terms of what we can actually test with the youtube player, since I doubt playwright can make any meaningful assertions about what is inside the iframe. But we can certainly test things like

transcript start/end times are correct
iframe has the expected start/end/controls query parameters.

I'll make an issue for this.

Looks good!
@ibrahimjaved12 BTW, if you have not found this yet, you can start a particular course with its shortid. For example:
yarn start course 18.02sc-fall-2010

Thanks @ChristopherChudzicki. I did know of this but hadn't tried it yet, had gotten used to the other approach. I'll try this!