[WIP] Provide instance to give a change to implement delegate methods of NSURLSession

Users of APIKit may want to implement delegate methods of NSURLSession to hook events or to handle authentication challenge, so I think we have to provide instances for each subclasses of API and re-implement wrapper of NSURLSession using delegate methods (not block). Currently, an instance of API subclass is provided as a singleton because I would like to keep interface simple and multiple clients are not necessary for one web API.
@ikesyo how do you think?

I agree that this is a required feature, and the users could implement the delegate methods in API subclasses is quite simple usage.

I added some minor notes. :eyeglasses: Altogether, looks good :sparkles:

Thank you for your reviewing! I will fix code and develop to support NSURLSessionDelegate.

Oops, neilpa/Assertions is no longer available :skull:  I replace it with ikesyo/Assertions swift-1.1.

Thanks for picking it! I didn't report that :sweat:

No problem. I commit @3dec41cb78ea2e78b534147bab15e2a2565c3321 on master too.

@ikesyo This is ready for review :sparkles:

Sorry for the late response :bow: I added some review notes. Could you check it out, and merge master into this?

Thank you for reviewing! I will merge this soon.

Oops. I overlooked 1 note. I'll fix it soon.

:tada:


Missing available preset documentation

Reason/Context
We have quite a few presets available to use in different languages, but no way of expressing this to the end-user.
Related to https://github.com/asyncapi/modelina/issues/287

Closing as we don't list all the presets, we rather guide the user through use-cases.


your tests/build is failing

do you plan to correct?

https://travis-ci.org/kevintweber/KtwDatabaseMenuBundle

Thanks for reminding me.  I've updated the tests with the latest push.

thanks :+1:


Submit CSV to CQL

Is it possible to submit CSV text along with delimiter and other options with Stargate REST?
Couldn't find anything in the docs.

This isn't something currently supported but we do have https://github.com/stargate/stargate/issues/821 which is a similar issue. Also, any such batch operations would most likely be implemented as json rather than csv.

Also, any such batch operations would most likely be implemented as json rather than csv.

Very sound logic :)


Added missing dependency to<EMAIL_ADDRESS>

Without this, npm run dev shows error:
ERROR in /Users/.../typescript-react-redux-starter/typings/tsd.d.ts
(6,1): error TS6053: File '/

Initially tried<EMAIL_ADDRESS>but didn't work, looks like<EMAIL_ADDRESS>is the intended version?

@bertrandk : comments?

Seems like definition-header isn't needed after the move to typings from tsd. Closing for now.


prisma pull specific tables only

Problem
Our database has hundreds of tables. In the project we know that we don't want most of the schemas. When we do npx prisma db pull
It creates hundreds of those schemas which we don't need.
Suggested solution
Have a way to provide a list of tables that prisma should create schemas for.
For example:
datasource db {
  provider = "postgresql"
  url      = env("DATABASE_URL")
  focusedTables = ['users', 'books']
}

If no focusedTables defined, then by default it should sync all the tables.

Looking for this capability also.


Bump java versions

Bump MS OpenJDK Java versions. JDK 8 was left unchanged since we are waiting for Temurin to drop the installers. We can include those in this change, if that is what we desire.

We can pick up 8 when they're ready.


New version: LearnBase v0.5.0

Registering package: LearnBase
Repository: https://github.com/JuliaML/LearnBase.jl
Created by: @CarloLucibello
Version: v0.5.0
Commit: aae4cbd00ae9de77f3be19db0119c3ae2e1f4589
Reviewed by: @CarloLucibello
Reference: https://github.com/JuliaML/LearnBase.jl/commit/aae4cbd00ae9de77f3be19db0119c3ae2e1f4589#commitcomment-54046748

stop auto-merging


Loading the Sample Databases

In the book, it states "To install the database on PostgreSQL, copy the data.dump file from the Datasets folder in the GitHub repository of the book.......Then, load the data.dump file from a command line using the command:
psql < data.dump"
Can someone please provide clarification on how this is done??  I've had the same issue as someone else when clicking on file that it brings up the code in a separate window.  The book did not do a good job at explaining much of anything in terms of downloading software or loading the data.

Had the same issue on Mac
The issue I encountered was due to a later version of OpenSSL I had: v1.1.1.dylib
I needed to point to an earlier version: v1.0.0.dylib
When I executed the psql command psql I got the following error:
"dyld: Library not loaded: /usr/local/opt/openssl/lib/libssl.1.0.0.dylib
Referenced from: /usr/local/lib/libpq.5.dylib
Reason: image not found
Abort trap: 6"
The resolution I used:

brew uninstall --ignore-dependencies openssl brew install https://github.com/tebelorg/Tump/releases/download/v1.0.0/openssl.rb

brew switch openssl ls /usr/local/cellar/openssl | sed 's/\///g'

psql -U user -d database < /Users/user/directory/data.dump

Hope this helps

Obviously, it's probably a little late to help out OP, but hopefully this helps other students who are stumped by this as well.
Here is the way I figured it out:

Open the SQL Shell (psql), that came with the PostgreSQL installation package.
Connect to the server/database/user that you would like to use. If you are just using the default settings, you can just press enter until it prompts you for the password.
Type in the password you set for that user and press Enter
On the command line, change your directory to the location of the data.dump file. Using this command:
dbname=#\cd C:/Users/..../SQL-for-Data-Analytics-master/Datasets (Make sure you pay attention to the direction of the slash marks. If you copy and paste the location from the File Properties in Windows, the slashes will be wrong)
Now that you are in the correct directory, you can use a psql command to load the dump file:
dbname=# \i data.dump
After psql has finished loading the file, I had to exit and restart the psql terminal before my database would recognize the new tables. I don't know if that is 100% necessary, or if a COMMIT statement would do the same thing.

Best of luck!

Obviously, it's probably a little late to help out OP, but hopefully this helps other students who are stumped by this as well.
Here is the way I figured it out:

Open the SQL Shell (psql), that came with the PostgreSQL installation package.
Connect to the server/database/user that you would like to use. If you are just using the default settings, you can just press enter until it prompts you for the password.
Type in the password you set for that user and press Enter
On the command line, change your directory to the location of the data.dump file. Using this command:
dbname=#\cd C:/Users/..../SQL-for-Data-Analytics-master/Datasets (Make sure you pay attention to the direction of the slash marks. If you copy and paste the location from the File Properties in Windows, the slashes will be wrong)
Now that you are in the correct directory, you can use a psql command to load the dump file:
dbname=# \i data.dump
After psql has finished loading the file, I had to exit and restart the psql terminal before my database would recognize the new tables. I don't know if that is 100% necessary, or if a COMMIT statement would do the same thing.

Best of luck!

Mine asks for a username, not a password, and then ignores all commands and exits after I press any key.
Maybe my brain is too fried but I cannot figure out what to do to upload the databases.
I've done the command line, I've tried simply creating the database within but run into the data error when it comes inserting the data into the tables for smalljoins, I've tried simply doing it step by step. I'm totally lost and my instructor is busy so slow to reply unfortunately. I've been stuck with this for the last week or so racking my brain and just feeling defeated and frustrated at this point. I'm sure I know what to do but cannot figure it out lol


IE11 Bug - Breakpoint Overlap at 640px

How can we reproduce this bug?

Step one
Visit the following plunk in IE11 and size your window to exactly 640px
https://run.plnkr.co/plunks/20e8IP6FQji9rwqThqXh/

You will see both the "show-for-medium" element AND the "hide-for-medium" element visible at the same time.
Note that this issue occurs on foundation.zurb.com as well.
What did you expect to happen?
There should be no window size possible that renders both of those elements at the same time. This appears to be specific to IE11
What happened instead?
Both elements were presented when the window is exactly 640px
Test case:
https://run.plnkr.co/plunks/20e8IP6FQji9rwqThqXh/

I imagine that IE11 is having issues distinguishing between 39.9375em and 40em.
It seems like you have the opposite of the gap effect here, you actually have an overlapping issue.
I won't be on my Windows machine until tomorrow, but I'll try to look at thing more closely then. Could you tell me if you have any settings on your browser other than the default please? Font sizes and zoom are both important to know here.

I have it on good authority that the em value should be at least 0.1 difference between breakpoints. I'll try to look at this tomorrow and offer a PR if it hasn't already been done.

I think this is related to #8811.

Closing in favor of #8811, which has a few ideas for how to fix this issue.


Response header does not reflect the correct version for HTTP2 requests

Issue
The response header does not reflect the correct version for HTTP2 requests. Even though, wiremock replies in h2 but the version in the response header shows "HTTP/1.1 200". This is caused by the hardcoded value in the toString() method of Response.java class
    @Override
    public String toString() {
        StringBuilder sb = new StringBuilder();
        **sb.append("HTTP/1.1 ")**.append(status).append("\n");
        sb.append(headers).append("\n");
        // no longer printing body
        return sb.toString();
    }

Environment

[ ] Which version of WireMock you're using -
2.31.0 standalone

[ ] How you're starting and configuring WireMock, including configuration or CLI the command line
Starting wiremock in standalone mode inside a docker container

Steps to reproduce

Insert some request-response data in wiremock
Try to make a request with h2
See the response on the console. You get the response like below for the h2 requests

Response:
HTTP/1.1 200
Content-Type: [application/json]
Matched-Stub-Id: [fc793a71-3651-4c9b-bafd-9678c6265950]

Is this fixed in 2.32.0? I tried and below is the reponse
curl --http2 -s -I -X GET http://localhost:8080/api/test

HTTP/1.1 101 Switching Protocols

HTTP/2 200
matched-stub-id: 908d8b29-fdf2-4d13-9435-240d804d4f02
vary: Accept-Encoding, User-Agent


Add more docker image name validation rules based on specification, Fix #69

Added rules

Repository component can have DNS hostname prefix(Ref #69 )
Tag values are limited to the set of characters [a-zA-Z0-9_.-], except they may not start with a . or - character.
Tags are limited to 128 characters
A hostname component may not contain _ characters
Name components may contain lowercase characters, digits, and separators
A separator is defined as a period, one or two underscores, or one or more dashes
A name component may not start or end with a separator

Ref

https://github.com/moby/moby/blob/master/image/spec/v1.2.md

@mpppk have you signed the CLA yet? (https://github.com/fnproject/fn/blob/master/CONTRIBUTING.md)

@AstromechZA Today I signed CLA, thank you for your advice!

thanks for the great PR, this is really thorough.
should we do this in the https://github.com/fnproject/fn server ? client side validation is nice as an added layer, but is brittle to maintain with server side validation checks (they can get out of sync / have bug mismatches). I have an inkling our server will probably gladly accept most of the things being gated by this patch, but I do think that the #69 case will 'just work' without validateImageName.
I guess I propose we remove validateImageName from the CLI and let the server do validation. at present, it appears that the server does no validation of the image until we attempt to run it, at which point it may fail awkwardly since we are only parsing it and not checking for any errors. It would be nice if we did image validation at create/update time (not that the image exists, just that the image specified is valid) and returned this to the user at that point in time, from the server. What do you think?

@mpppk any interest in moving this over to the server side?  Looks like a great addition.

@rdallman @treeder
Sorry for late reply and thank you for your comments.
I'll send new pull request to fnproject/fn and close this PR. Thanks! :smile:


[backport] Allow to compile using PTX with an envvar

Backport of #5622 by @emcastillo

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 391084ad1d7824fff95b25cd1a8ba677874c346b, target branch v9) succeeded!


Add Hour, Minutes, Seconds Columns and Configs

The User should be able to Select hours, minutes, and seconds and these functions should work:

minDate
maxDate
configs

Hi Ali ,
I hope you are fine
I need this config in my project ,
when do you think it will be fixed?


Drought nav does not update when edited in WP admin

@chachasikes showed me how to update the navigation menu in the WP admin (Appearance > Menus). But while the changes I made appear to be saved in WP, they don't appear on the website. This is affecting our ability to link in a new page to the nav.
See Slack thread on this: https://cadotgov.slack.com/archives/C026VJB8Q1G/p1632877211125600

Drought frontend build is pulling menu structure here https://github.com/cagov/drought.ca.gov/blob/main/src/templates/_data/menus.js#L4

Per Carter: https://github.com/cagov/drought.ca.gov/blob/main/src/templates/_data/menus.js
Need new issue on this

Issue added...
https://github.com/cagov/wordpress-to-github/issues/50

Slack thread --> https://cadotgov.slack.com/archives/C026VJB8Q1G/p1633117513186300

Quick note. Menu work within the wordpress-to-github module has been completed. In short, this set of changes to wordpress-to-github will help trigger site rebuilds whenever a change is made to a menu within Wordpress.
However, deploying this set of changes into live service is on-hold. We're working on putting processes in place around how we deploy wordpress-to-github changes. These processes will help us ensure everyone is properly notified and nothing breaks.
Once the above wordpress-to-github changes are deployed, we'll need to do some minor Drought work here to consume the new menu data. That's the next step, stay tuned.

Revisiting this. I need to add a new link to the top nav: Water-saving tips. Made the change in WP menu, but it's not live. How do we make it live? @xjensen

Per Jon: "Just takes a few minutes. The menu updates are not pushed out to the site immediately; they need to be pulled when the wordpress-to-github polling service runs every few minutes."


How to Deserialize an ActiveModel ? and a request for Serialization examples

I'm trying to implement a basic CRUD app and I'm a bit lost. I'm trying to create a REST API with Axum. I've been trying to use the Json(new_active_model). Is this not supported by Serde yet ?
 let new_task = task::ActiveModel {
        title: Set(payload.title),
        text: Set(payload.text),
        ..Default::default()
    }
    .save(conn)
    .await
    .expect("could not insert task");

    // this will be converted into a JSON response
    // with a status code of `201 Created`
    (StatusCode::CREATED, Json(new_task))

Regardless of the problem that I'm facing, It'd be nice if the examples and/or docs were updated showing how to serialize and deserialize entities/models, as well as, examples for REST APIs

Hey @VaZark, you can check web API demo in the /examples folder. Example for Actix, Rocket and Poem (#) are included

hI @billy1624, I played with the examples but it seems like the underlying issue is the trait Serialize is not implemented for task::ActiveModel.  (I generated the model with the sea-orm-cli generate entity)
So can the ActiveModel add support for Serialize? Or can I convert an ActiveModel to just a Model and just use derive Serialize on it?
I'm pretty new to rust so you can feel free to point out if I'm missing /misunderstanding something here

If you want to get back the inserted / updated model in Model instead of ActiveModel. You can use insert / update method in sea-orm 0.5.x

Thanks. Insert was what I needed to get it working. Is it not possible to make the ActiveModel support Serialize?

Thanks. Insert was what I needed to get it working. Is it not possible to make the ActiveModel support Serialize?

Is there any reason you need to Serialize ActiveModel? I guess what you need is just getting POST data (Model without primary key attribute if it's insert) from end user and responding the saved Model

Yup. That's what I ended up doing. I was trying to come up with ways to not write new types by hand for each table.

Nice! Feel free to ping us if you got any other questions :)


Fixed memory leaks found running Khronos conformance

Fixed memory leak found whilst running Khronos conformance with more …

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

LGTM.  Running internal testing on the commits now.

Changes committed with IDs: 3a0201d7ad4eb2ad2e8b336e5030fd28d482b528 and 973c2aab1f20c53f219ce9f1e3002f1bdc01046c


[#JAVA-163] This is a first step for Unix domain socket integration. It should be double checked

This is a first step for Unix domain socket integration. It should be double checked.
I made this change for a quick test so I just do the minimum to make it works. By the way performance gain was not so useful in my case (complicated environment with caches etc...). That's why I stop my work here.

Unfortunately, any work we do on Domain Sockets will need to be standalone with no dependencies required for users not interested in the Unix Socket support.

At runtime the dependency is only needed if the user use an URI triggering UDS. At this point the unix socket library will ask for native libraries. If the user don't use URI triggering UDS it should not need more dependencies at runtime.


Animation library API docs need a general tutorial section

According to the survey the animation library's biggest problem is people just don't understand how everything fits together. We should create a long API doc section on the library's API docs showing how everything fits together, and then point to it from the various main classes and so on (maybe based on which ones get the most hits).

To give a quick update: I've written up a draft for this that is currently undergoing review by our tech writers.


[Bug]: SyntaxWarning is not always captured correctly

To Dos

[X] I tested with the latest released version.
[X] I have checked the issues and think that this is not a duplicate.
[X] I added a very descriptive title to this issue.

Example Code (rst)
Example 1:

.. code-block:: python

    if mystring is "ok":
        ...

    foo(b=1,2,3) # syntax error

Example 2:

.. code-block:: python

    if mystring is "ok":
        ...

Relevant log output
Example 1:
❯ rstcheck test.rst --report-level=warning
test.rst:6: (ERROR/3) (python) positional argument follows keyword argument

Example 2:
❯ rstcheck test.rst --report-level=warning
<string>:1: SyntaxWarning: "is" with a literal. Did you mean "=="?
Success! No issues detected.

Description
It seems like handling of SyntaxWarning is not consistent.
In the first example it doesn't even fire, while in the second example it passes the Success! test.
Furthermore, the formatting of the error could be a little nicer.
Operating System
macOS
Operating System Details
No response
Python Version
Python 3.10.0
rstcheck Version
See below.
Additional Context
❯ python -c "import rstcheck; print(rstcheck.__version__)"
Traceback (most recent call last):
  File "<string>", line 1, in <module>
AttributeError: module 'rstcheck' has no attribute '__version__'

The SyntaxWarning from the second example should not be there to begin with, if I remember correctly. At least this should be nothing rstcheck's code does directly.
Python code is checked with the builtin compile() function: https://github.com/rstcheck/rstcheck/blob/main/src/rstcheck/checker.py#L537 So if a syntax error in python code is not detected it is most probably because compile() did not detect it.
I will take a look.

Good thing that you encountered this issue with version. I almost forgot about it and wanted to implement it again. Thanks.

SyntaxWarnings in general are now catched and handled like SyntaxErrors

@DanielNoord
This and the additional inline configuration are merged.
Could you now test the main branch and check if you have any other issues?

Thanks @Cielquan!
I'm don't have access to my regular computer today, but if you could push these as a new tag to the repo (rc2 perhaps) I can change the tag in this PR https://github.com/PyCQA/pylint/pull/6631 to see if our CI passes with the last changes!

@DanielNoord I made a complete rc2 release.

CI ran successfully! Seems like everything is working as expected 😄

Great!
Now I am confident that the final v6 release will be within this week, so the next pre-commit.ci autoupdate round should include it.


Extension causes high cpu load

Issue Type: Performance
Extension Name: swdc-100-days-of-code
Extension Version: 1.0.5
OS Version: Windows_NT x64 10.0.18362
VSCode version: 1.48.0

:warning: Make sure to attach this file from your home-directory:
:warning:C:\Users\abdounuru.niyigena.GTBank.<IP_ADDRESS>.004.005\softwaredotcom.swdc-100-days-of-code-unresponsive.cpuprofile.txt
Find more details here: https://github.com/microsoft/vscode/wiki/Explain-extension-causes-high-cpu-load

@AbdoulNuru can you attach your file? Webviews may cause a spike in CPU, which might be the cause, but we'll take a look.

@AbdoulNuru can you attach your file? Webviews may cause a spike in CPU, which might be the cause, but we'll take a look.

@brettmstevens7 The issue is now resolved.


Return the source documents in same order for slackbot as per the response in the UI

Add a fix so that the Slackbot should receive the source documents in the same order as they appear in the UI.

closes: #250
Tested it here:

Slack request: https://ankit-test-group.slack.com/archives/C061F4LR4KT/p1704903223475919
Web UI request: https://f9c33f8b.ask-astro.pages.dev/requests/48c82d1e-afd1-11ee-a80f-42004e494300

The PR title and description needs to be corrected however IMO.

The PR title and description needs to be corrected however IMO, the current title means opposite to what is intended. #256 (review)

Changed it Pankaj


XML to CSV crash

/Users/philip/Desktop/node_modules/converter/src/convert.js:233
		map[self.from][self.to](self.buffer, self.opts, function(err, data) {
		                       ^
TypeError: undefined is not a function
    at Transform.self.stream._flush (/Users/philip/Desktop/node_modules/converter/src/convert.js:233:26)
    at Transform.<anonymous> (_stream_transform.js:130:12)
    at Transform.g (events.js:199:16)
    at Transform.emit (events.js:104:17)
    at prefinish (_stream_writable.js:474:12)
    at finishMaybe (_stream_writable.js:482:7)
    at endWritable (_stream_writable.js:493:3)
    at Transform.Writable.end (_stream_writable.js:459:5)
    at ReadStream.onend (_stream_readable.js:505:10)
    at ReadStream.g (events.js:199:16)

What options are you passing into?
This needs more error handling to catch this type of error, thanks for pointing it out.

// setup the options for the data converter
var options = {
    from: 'xml',
    to: 'csv'
};

Supported conversions can be found in this map. We'll get the docs updated and better error handling soon.

fwiw I have an almost complete refactor done


Orion Browser and Authenticating

Hi, so I am giving Orion a go at the moment, which promises Firefox / Chrome plugin compatibility. I am using a few extensions from Firefox and Chrome store, and Hide My Email installs correctly as well, from both extension stores.
However I am running into issues with signing in to iCloud using Hide My Email. I complete the login process on iCloud.com and can access my account, but the login never affects Hide My Email. It remains signed out.  I don't get the 2FA prompt either.  Any ideas on what I could try?

Hi @kormose! I dogfooded the extension in Orion and encountered the same issue. After some debugging I figured out that Orion does not fully implement the extension API. The Chrome version of the extension (MV3) relies on declarativeNetRequests, whereas the Firefox one (MV2) relies on blocking webrequests. As far as I understand, Orion does not support any of those (possibly by design).

Thank you so much for investigating! I will flag this up with them.

I came here to ask this exact question. I was really excited to see this extension but sadly im having the same issues. Except I seem to be having an extra cosmetic issue too, see the image below.
Even if it's hacky for now, has anyone managed to get the extension working?
The cosmetic issue happens with both the Chrome and Firefox versions. Though based one the conversation above I currently have the Firefox version installed.
Even though this is technically a different issue, I put it here with the other Orion topic since it's more relevant to this specific browser than it is the extension itself.

@relink2013 thanks for reporting this. I'd suggest flagging this with the Orion folks as it could be a wider issue on how Orion renders extension popups.


remove jcenter() , fixes deprecations in the building process of servers

And Updated Gradle to 7.0.2

This will fail to build as the org.jetbrains.kotlinx:kotlinx-dom package is only available on jcenter. I am also perfectly capable of bumping the gradle wrapper myself when I deem appropriate, and have done so already on the current dev branch.


allow hpvolumes to be created at runtime and tagged accordingly

Hpvolumes must be created and labeled on the host system. The hostpath provisioner container technically runs on the host rather than in microshift container so presenting a podman volume to the microshift container doesn't actually work.

/lgtm


"Run" button should be disabled if the server is disconnected, no model is chosen or has no prompt

The existing button is located in sessions/[id]/+page.svelte
Right now it's only disabled if it has !prompt
The only server check we do is in the Settings view and the server status not globally available.
This is the current Typescript definition of Settings.
We could probably save the server status to the settingsStore so we can access it from the Sessions view.
Currently, we only check the status of the server whenever ollamaServer changes.

That means that if the server went offline after starting a new session the status would still appear as online.
This is not ideal because submitting the form to an offline server would cause an error but we should handle that behavior in #49

Should add a new test for this behavior in sessions.test.ts

:tada: This issue has been resolved in version 0.7.9 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


which algorithms (simpson, adaptive trapezoid, trapezoid) did you use

I am not sure what you mean. I used the three you mentioned.


user does not have permission to call the Discover method

HI,
I tried to run the commands but I can't move forward since I get this error:
OperationException                        Traceback (most recent call last) Cell In[13], line 1----> 1 migration.create_pqt_file(dataset = dataset_name, workspace = workspace_name) File /nfs4/pyenv-37980388-9737-41fc-8dbb-c078cd639d85/lib/python3.11/site-packages/sempy/_utils/_log.py:310, in mds_log..get_wrapper..log_decorator_wrapper(*args, **kwargs)    307     raise    309 try:--> 310     result = func(*args, **kwargs)    312     # The invocation for get_message_dict moves after the function    313     # so it can access the state after the method call    314     message.update(extractor.get_completion_message_dict(result, arg_dict)) File /nfs4/pyenv-37980388-9737-41fc-8dbb-c078cd639d85/lib/python3.11/site-packages/sempy_labs/migration/_create_pqt_file.py:52, in create_pqt_file(dataset, workspace, file_name)     49 subFolderPath = os.path.join(folderPath, "pqtnewfolder")     50 os.makedirs(subFolderPath, exist_ok=True)---> 52 with connect_semantic_model(     53     dataset=dataset, workspace=workspace, readonly=True     54 ) as tom:     55     if not any(     56         p.SourceType == TOM.PartitionSourceType.M for p in tom.all_partitions()     57     ) and not any(t.RefreshPolicy for t in tom.model.Tables):     58         print(     59             f"{icons.info} The '{dataset}' semantic model within the '{workspace}' workspace has no Power Query logic."     60         ) File ~/cluster-env/trident_env/lib/python3.11/contextlib.py:137, in _GeneratorContextManager.enter(self)    135 del self.args, self.kwds, self.func    136 try:--> 137     return next(self.gen)    138 except StopIteration:    139     raise RuntimeError("generator didn't yield") from None File /nfs4/pyenv-37980388-9737-41fc-8dbb-c078cd639d85/lib/python3.11/site-packages/sempy_labs/tom/_model.py:4579, in connect_semantic_model(dataset, readonly, workspace)   4576     workspace_id = fabric.get_workspace_id()   4577     workspace = fabric.resolve_workspace_name(workspace_id)-> 4579 tw = TOMWrapper(dataset=dataset, workspace=workspace, readonly=readonly)   4580 try:   4581     yield tw File /nfs4/pyenv-37980388-9737-41fc-8dbb-c078cd639d85/lib/python3.11/site-packages/sempy_labs/tom/_model.py:47, in TOMWrapper.init(self, dataset, workspace, readonly)     44 self._readonly = readonly     45 self._tables_added = []---> 47 self._tom_server = fabric.create_tom_server(     48     readonly=readonly, workspace=workspace     49 )     50 self.model = self._tom_server.Databases.GetByName(dataset).Model     52 self._table_map = {} File /nfs4/pyenv-37980388-9737-41fc-8dbb-c078cd639d85/lib/python3.11/site-packages/sempy/_utils/_log.py:310, in mds_log..get_wrapper..log_decorator_wrapper(*args, **kwargs)    307     raise    309 try:--> 310     result = func(*args, **kwargs)    312     # The invocation for get_message_dict moves after the function    313     # so it can access the state after the method call    314     message.update(extractor.get_completion_message_dict(result, arg_dict)) File /nfs4/pyenv-37980388-9737-41fc-8dbb-c078cd639d85/lib/python3.11/site-packages/sempy/fabric/_flat.py:1488, in create_tom_server(readonly, workspace)   1485 workspace_url = _get_workspace_url(workspace_client.get_workspace_name())   1486 connection_str = _build_adomd_connection_string(workspace_url, readonly=readonly)-> 1488 return _create_tom_server(connection_str, workspace_client.token_provider) File /nfs4/pyenv-37980388-9737-41fc-8dbb-c078cd639d85/lib/python3.11/site-packages/sempy/_utils/_log.py:310, in mds_log..get_wrapper..log_decorator_wrapper(*args, **kwargs)    307     raise    309 try:--> 310     result = func(*args, **kwargs)    312     # The invocation for get_message_dict moves after the function    313     # so it can access the state after the method call    314     message.update(extractor.get_completion_message_dict(result, arg_dict)) File /nfs4/pyenv-37980388-9737-41fc-8dbb-c078cd639d85/lib/python3.11/site-packages/sempy/fabric/_client/_utils.py:49, in _create_tom_server(connection_string, token_provider)     46 tom_server.AccessToken = get_access_token(None)     47 tom_server.OnAccessTokenExpired = FuncAccessToken, AccessToken---> 49 tom_server.Connect(connection_string)     51 return tom_server OperationException: The ' USER_NAME' user does not have permission to call the Discover method. Technical Details: RootActivityId: b0384b69-0319-4dd6-bb26-f73f21136342 Date (UTC): 11/4/2024 2:22:31 PM    at Microsoft.AnalysisServices.XmlaClient.CheckForSoapFault(XmlReader reader, XmlaResult xmlaResult, Boolean throwIfError)    at Microsoft.AnalysisServices.XmlaClient.CheckForError(XmlReader reader, XmlaResult xmlaResult, Boolean throwIfError)    at Microsoft.AnalysisServices.XmlaClient.SendMessage(Boolean endReceivalIfException, Boolean readSession, Boolean readNamespaceCompatibility)    at Microsoft.AnalysisServices.Core.AnalysisServicesClient.Discover(IMajorObject obj, ObjectExpansion expansion, JaXmlSerializer serializer)    at Microsoft.AnalysisServices.Core.Server.Refresh(IMajorObject obj, ObjectExpansion expansion)    at Microsoft.AnalysisServices.Core.Server.SendRefresh(IMajorObject obj, ObjectExpansion expansion)    at Microsoft.AnalysisServices.MajorObject.Refresh(ObjectExpansion expansion)    at Microsoft.AnalysisServices.Core.Server.Connect(String connectionString, String sessionId, ObjectExpansion expansionType)    at Microsoft.AnalysisServices.Core.Server.Connect(String connectionString, String sessionId)    at Microsoft.AnalysisServices.Core.Server.Connect(String connectionString)
The logged user is a power platform admin and an analysis service admin.
Can anyone suggest me what else to check in order to understand the reason behind this?
Thanks a lot
Lorenzo

Seems like an access issue. The strange this is that the discover objects are not used to do this.
Can you do this?
from sempy_labs.tom import connect_semantic_model

with connect_semantic_model(dataset='', workspace='', readonly=True) as tom:
    print(tom.model.Model.Name)

HI,
I got this error:

AmoException                              Traceback (most recent call last)
Cell In[13], line 3
1 from sempy_labs.tom import connect_semantic_model
----> 3 with connect_semantic_model(dataset='LinePlan', workspace='testFabric', readonly=True) as tom:
4     print(tom.model.Model.Name)
File ~/cluster-env/trident_env/lib/python3.11/contextlib.py:137, in _GeneratorContextManager.enter(self)
135 del self.args, self.kwds, self.func
136 try:
--> 137     return next(self.gen)
138 except StopIteration:
139     raise RuntimeError("generator didn't yield") from None
File /nfs4/pyenv-f02d29df-38d0-489a-ad88-9b0b9102e641/lib/python3.11/site-packages/sempy_labs/tom/_model.py:4615, in connect_semantic_model(dataset, readonly, workspace)
4612     workspace_id = fabric.get_workspace_id()
4613     workspace = fabric.resolve_workspace_name(workspace_id)
-> 4615 tw = TOMWrapper(dataset=dataset, workspace=workspace, readonly=readonly)
4616 try:
4617     yield tw
File /nfs4/pyenv-f02d29df-38d0-489a-ad88-9b0b9102e641/lib/python3.11/site-packages/sempy_labs/tom/_model.py:50, in TOMWrapper.init(self, dataset, workspace, readonly)
45 self._tables_added = []
47 self._tom_server = fabric.create_tom_server(
48     readonly=readonly, workspace=workspace
49 )
---> 50 self.model = self._tom_server.Databases.GetByName(dataset).Model
52 self._table_map = {}
53 self._column_map = {}
AmoException: The 'Database' with 'Name' = 'LinePlan' doesn't exist in the collection.
at Microsoft.AnalysisServices.NamedComponentCollection.BaseGetByName(String name, Boolean throwIfNotFound)
at Microsoft.AnalysisServices.Tabular.DatabaseCollection.GetByName(String name)
This is the workspace and dataset I’m trying to process:
@.***
Thanks in advance
Lorenzo.
Da: m-kovalsky @.>
Inviato: martedì 26 novembre 2024 17:19
A: microsoft/semantic-link-labs @.>
Cc: Lorenzo Danielli @.>; Author @.>
Oggetto: Re: [microsoft/semantic-link-labs] user does not have permission to call the Discover method (Issue #305)
Seems like an access issue. The strange this is that the discover objects are not used to do this.
Can you do this?
from sempy_labs.tom import connect_semantic_model
with connect_semantic_model(dataset='', workspace='', readonly=True) as tom:
print(tom.model.Model.Name)

—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/semantic-link-labs/issues/305#issuecomment-2501291495, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BHNT6D52XM5KRKC6FDBMGN32CSNPLAVCNFSM6AAAAABSQZZG6CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBRGI4TCNBZGU.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Is this tenant setting disabled?

Yes it was…should I enable it?
Thanks!
Da: m-kovalsky @.>
Inviato: domenica 1 dicembre 2024 10:59
A: microsoft/semantic-link-labs @.>
Cc: Lorenzo Danielli @.>; Author @.>
Oggetto: Re: [microsoft/semantic-link-labs] user does not have permission to call the Discover method (Issue #305)
Is this tenant setting disabled?
image.png (view on web)https://github.com/user-attachments/assets/3b3c2e68-718c-4fd5-b029-f49de152c69c
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/semantic-link-labs/issues/305#issuecomment-2509665503, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BHNT6D2OQP63GVFT6NBKYXT2DLMVZAVCNFSM6AAAAABSQZZG6CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBZGY3DKNJQGM.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Don't enable that. Is the model on a shared capacity?


Bash command history scroll causes crash

Windows build number: Microsoft Windows [Version 10.0.15063]

What you're doing and what's happening:  Attempting to scroll through command history (buffer) using the keyboard's up or down arrows.

What's wrong: Bash.exe crashes at end of buffer. If no buffer Bash crashes immediately (the instance seems to remain open in the background, however; the window freezes and then closes. Bash must be terminated manually via Task Manager. Launching bash again before manually terminating the application creates a new instance of Bash). The same behavior is seen when attempting tab completion (though less consistently).

What should be happening instead: Bash should continue to run as normal?

Strace of the failing command, if applicable: I followed the instructions from Jack Hammons provided here. Four .elm files have been attached for reference as lxcore_service.zip (the article says there should be three but the .cmd scripts generated four).

I am running the latest (non Insider Preview) version of the application. I have already successfully uninstalled and reinstalled the application and see the same behavior. I am happy to assist with further troubleshooting/log gathering if it is helpful.
Thank you!

Derek

I'm not able to reproduce this, can you think of any other steps that would be required?  Do you have a custom .bashrc perhaps?

Hey benhillis. Thanks for the response. :)
Pastebin of cat .bashrc here. I don't recall modifying this.
I've got two Windows 10 boxes on the same release. I can't reproduce this on computer number 2. I always encounter this issue on computer number 1. That's when I went ahead and fully blew the install away and reinstalled. I came here when the issue persisted regardless.

After an update I can no longer produce the problem. Phantom one-off issue seems to have spontaneously resolved.


add undo conversion and show original time buttons to converted time messages

suggestion by: Missilestorm

also delete message button

remove all buttons after a timeout, tell that to the user somehow.. (or dont? idk, need input)


fix: google may return empty ical

something like:
BEGIN:VCALENDAR
PRODID:-//Google Inc//Google Calendar 70.9054//EN
VERSION:2.0
CALSCALE:GREGORIAN
X-WR-CALNAME:TEST
X-WR-TIMEZONE:Asia/Shanghai
BEGIN:VTIMEZONE
TZID:Asia/Shanghai
X-LIC-LOCATION:Asia/Shanghai
BEGIN:STANDARD
TZOFFSETFROM:+0800
TZOFFSETTO:+0800
TZNAME:CST
DTSTART:19700101T000000
END:STANDARD
END:VTIMEZONE
END:VCALENDAR

Please use o.is_loaded() to check if the object has a payload or not, it's more efficient.
I remember that this issue (google returning empty objects) have been looked into a couple of times, but I don't remember the conclusion.  Will o.load() help?

I'm a bit baffled why the tests break.  I'm looking into it.

I do not like the usage of o.icalendar_component here, becuase it may involve converting the object into an object from the icalendar library.  Anyway, I can see that it's already used in the same function, after if kwargs.get("expand", False):, so then my objection is moot.  :-)  I should probably go over and refactor at a later stage.  It's still needed to figure out why the test breaks though.

@tobixen  Thanks for your review.

Will o.load() help?

It has no effect, the original data have been loaded, but have no event/todo/etc.
FYI:
this event looks like a event's recurrence item, id format is<EMAIL_ADDRESS>i cannot reproduce this event, but my user have 😞 .
The weird thing is i call search with expand=False, this recurrence item should not return standalone. If do not pass sort_keys parameter, the return result is like
<EMAIL_ADDRESS><EMAIL_ADDRESS>

I promised to look into why the test breaks - but then I got interrupted and busy.  I will try and have a look now.

Sorry for overwriting your changes.  I think it will be better this way.  I think the special logic for dealing with objects that aren't loaded or doesn't contain components is not specific to when expand=True, and hence it should be outside the if.
I consider this to be a bugfix, as it will cause consistent and expected behaviour even when the caldav server doesn't conform to the RFCs.

Could you confirm that this solves your problems?

@tobixen Yes, it's works for me.


Change of background color cannot be undone as an atomic event

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
When the background color of text is changed, a checkpoint is not added to the history of events so ctrl+z (undo) skips it.
To Reproduce
Create a text box with some text.
Change the background of the first letter.
Change the background of the second letter.
Press ctrl+z to undo.
See that all the text backgrounds are undone, not only the last.
Expected behavior
I expect to be able to undo every change of background.

draw.io version [e.g. 12.6.5]

Desktop (please complete the following information):

OS: Manjaro

Hi,
I tested this and it is working at my place with Ctrl+z and with undo arrow. I am using:

diagrams.net 14.9.8
Chrome Version 92.0.4515.159 (Official Build) (64-bit)
Win 10 Pro

What version do you see under Help in the editor?
If you change browser and test in Private mode, do you still see the same issue?
Regards,
https://user-images.githubusercontent.com/44292306/130768961-baefab85-7855-4c37-8481-5229d4a49ccd.mp4


odo log  for devfile components

/kind user-story
User Story
As a user, I want to see logs of a running application So that I can check if my application is running properly
Acceptance Criteria

[ ] odo log command should show the stdout of the "devRun" process, similarly how it does for s2i based components

/kind user-story
/area devfile
/priority medium

Could we expand this to also retrieve the build logs from the devBuild process? When I was running through the end-end scenario today, there were a couple times I thought it'd be handy to have the build logs on hand 🙂

yep, I think this would be really nice to have for both build and run commands tbh. I have stack creators ask me a couple of times how they could view their command outputs

Could we expand this to also retrieve the build logs from the devBuild process? When I was running through the end-end scenario today, there were a couple times I thought it'd be handy to have the build logs on hand 🙂
CC @maysunfaisal

We used to always show build output, but it was really noisy and most people don't care about the output if the build was successful.
If the build process fails (at least with s2i components) it dumps stdout and stderr of the build process.
Or you can use odo push --show-log to always show build logs.

@johnmcollier  At least for the machine readable case (eg Odo called by IDEs, projects like Codewind turbine container, etc) the build log output would be covered by the work described on https://github.com/openshift/odo/issues/2550 via a command like odo push -o=json (referred to as odo push --events in the comments  on 2550)

this is duplicate of https://github.com/openshift/odo/issues/3260 hence closing


fix blake8 - click lib error

Hello,
as discussed offline, I just saw that the click lib gave errors with pre-commit, see:
(spbrain) ovh@job-3d94d6d0-44eb-41ba-8972-c33856dbaa9c:~/speechbrain-aheba-contribs/recipes/TIMIT/ASR/transducer$ git commit -m "fix RNN-T loss bug - use logits instead of log_probs"
[WARNING] Unstaged files detected.
[INFO] Stashing unstaged files to /workspace/.cache/pre-commit/patch1649596825-52794.
Trim Trailing Whitespace.................................................Passed
Fix End of Files.........................................................Passed
Fix requirements.txt.................................(no files to check)Skipped
Mixed line ending........................................................Passed
Check for added large files..............................................Passed
black....................................................................Failed
- hook id: black
- exit code: 1

Traceback (most recent call last):
  File "/workspace/.cache/pre-commit/repo41p2_ikj/py_env-python3/bin/black", line 8, in <module>
    sys.exit(patched_main())
  File "/workspace/.cache/pre-commit/repo41p2_ikj/py_env-python3/lib/python3.8/site-packages/black.py", line 4134, in patched_main
    patch_click()
  File "/workspace/.cache/pre-commit/repo41p2_ikj/py_env-python3/lib/python3.8/site-packages/black.py", line 4123, in patch_click
    from click import _unicodefun  # type: ignore
ImportError: cannot import name '_unicodefun' from 'click' (/workspace/.cache/pre-commit/repo41p2_ikj/py_env-python3/lib/python3.8/site-packages/click/__init__.py)

flake8...................................................................Passed
yamllint.............................................(no files to check)Skipped
[INFO] Restored changes from /workspace/.cache/pre-commit/patch1649596825-52794.

I do an update on linter-requirement which solve the issue

Two other errors with pytest --doctest-modules speechbrain:
================================================================================== ERRORS ===================================================================================
________________________________________________________ ERROR collecting speechbrain/alignment/ctc_segmentation.py _________________________________________________________
speechbrain/alignment/ctc_segmentation.py:25: in <module>
    from ctc_segmentation import ctc_segmentation  # noqa: F401
E   ModuleNotFoundError: No module named 'ctc_segmentation'

During handling of the above exception, another exception occurred:
speechbrain/alignment/ctc_segmentation.py:36: in <module>
    raise ImportError("The ctc_segmentation module is missing.")
E   ImportError: The ctc_segmentation module is missing.
------------------------------------------------------------------------------ Captured stdout ------------------------------------------------------------------------------
ImportError: Is the ctc_segmentation module installed and in your PYTHONPATH?
_____________________________________________________ ERROR collecting speechbrain/lobes/models/huggingface_wav2vec.py ______________________________________________________
speechbrain/lobes/models/huggingface_wav2vec.py:27: in <module>
    from transformers import Wav2Vec2Model, HubertModel, WavLMModel
E   ImportError: cannot import name 'HubertModel' from 'transformers' (unknown location)

During handling of the above exception, another exception occurred:
speechbrain/lobes/models/huggingface_wav2vec.py:38: in <module>
    raise ImportError(MSG)
E   ImportError: Please install transformers from HuggingFace to use wav2vec2 / Hubert
E   E.G. run: pip install transformers

as I know, the transformers lib doesn't support the call Hubert and WavLMModel..

@TParcollet , any idea about the transformers lib errors ?,
I'm using:
>>> transformers.__version__
'4.4.0'

just update the ./run-doctest.sh and seems working..
LGTM


[Media] - remove tags from required

@amarzavery I don't know if this is the right fix honestly, but I know for sure that "tags" is not required in the "create" operation. Also RestAPI ref:
https://msdn.microsoft.com/en-us/library/azure/mt750387.aspx
Since it's indirectly inherited, maybe some objects have "tags" as required, in this case the split in objects must be re-thinked.

This change is

@BrianBlum - Can you review this PR and make sure that this is the correct change?

No modification for NodeJS

 https://github.com/Azure/azure-sdk-for-python/pull/840

 https://github.com/Azure/azure-sdk-for-ruby/pull/533


Full support for MPEGH_PROFILE_LC_LVL_4

Hello,
Amazonmusic has "ultra" quality providing mpegh streams with MPEGH_PROFILE_LC_LVL_4 which is not fully supported (limited by channel count) with libmpegh at the moment.
Since this service is quite popular I would like to ask whether support for this profile and level is planned for future.
An example of such stream is Piano Man by Billy Joe on Amazon music
thanks!
Adam

Hi Christof,
thank you for your insightful comment.
Based on my understanding if LC4 is used by stream, this check in decoder/ia_core_coder_init_config.c will fail:
   if (dec_proc_core_chans > MAX_NUM_CHANNELS_LVL3 || ref_layout_chans > MAX_NUM_CHANNELS_LVL3)
    {
      return IA_MPEGH_DEC_INIT_FATAL_STREAM_CHAN_GT_MAX;
    }

because
decoder/ia_core_coder_cnst.h:#define MAX_NUM_CHANNELS_LVL3 16

Hence why we were not able to decode the stream.
If I understand your comment correctly libmpegh may in case of LC4 and CompatibleProfileLevelSet() flag revert to BP3 and in that case in code check vs MAX_NUM_CHANNELS(24) will be used instead and the stream may potentially play.
However since this is decoder internals I assume the patch would have to be done to the libmpegh as AFAIK there is no way to produce this behavior just by using the decoder API.
Adam

Hi Adam,
You are right. I didn't check what the code does, but from the description it should be supporting that scenario:
State-of-the-art MPEG-H 3D Audio Low Complexity Profile decoders according to ISO/IEC 23008-3:2019/AMD 2:2020 also support the MPEG-H 3D Audio Baseline Profile, which is a subset of the MPEG-H 3D Audio Low Complexity Profile.
Have you been able to check the streams you are using and if they are in fact using the signalling as described?
Independent from that, I think the check should be changed according to what's standardised in AMD 2:2020 or the 23008-3:2022 version accordingly.
Thanks,
//Christof

Hi @christof-fersch, @SakethSathuvalli,
we tested decoding mpegh stream file from Amazon with "ultra" quality.
libmpegh currently does not implement CompatibleProfileLevelSet parsing and support in mp4 ext box or in decoder config as described in 2022 version of the standard.
The Amazon ultra streams that I tested do not use MHAProfileAndLevelCompatibilitySetBox or CompatibleProfileLevelSet in the mpegh3daConfigExtension. This leads me to believe even if the compatibility flag could be used, some ultra streams may not be able to be played, unless full LC LVL 4 support is implemented.
We were able to play and listen to at least one "ultra" stream when patching libmpegh to force decoding in LC LVL 3 mode.
Amazon streaming is a major use case, @SakethSathuvalli, are there any plans for implementing full LC LVL 4 decoding and the CompatibleProfileLevelSet as described by the 2022 standard?
Best Regards,
Adam

Hi @miartad,
We will be adding this support in decoder config in a week's time. Thanks for your patience!
Best Regards,
Saketh

Hi @adminitis,
Support for CompatibleProfileLevelSet landed on main branch today. Regarding Low complexity profile level 4, work is currently under progress. Support will be added in a few weeks.
Thanks,
Saketh

It looks like it is still not decoding profile level 4.

https://github.com/Fraunhofer-IIS/mpeghdec
This program supports decoding for profile level 4 and is open source. So you can refer to this.

https://github.com/Fraunhofer-IIS/mpeghdec This program supports decoding for profile level 4 and is open source. So you can refer to this.

Only with LC@L4/BL@L3 files unfortunately.

@SakethSathuvalli I've seen you have been working on this recently. What are your plans on merging this to main?

@SakethSathuvalli I've seen you have been working on this recently. What are your plans on merging this to main?

The changes are expected to be available on main branch - in the same form (enhancements maintained under build time switch) - by early June.
Thanks!

Initial version available on main branch.
Note: The default build has support only till profile level 3, in order to enable support for profile level 4 the build switch LC_LEVEL_4 has to be enabled.
Thanks!


@conda-forge-admin, please add bot automerge

@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/git-machete-feedstock#3.


Create Production Build

Create a Travis Build that automatically deploys tagged commits to production. Make sure all other commits are deployed to staging.

Getting close... Master -> Production works... still working out the kinks to push to staging.

Any commits to master will be autodeployed to production. Any commits to staging will be autodeployed to staging.


Adding Named range and Table support

Still Work In Progress for table support, but basic functionalities are currently running (creating, browsing, ...).
Corrected also some bugs with the Shared Strings items. To do so, the strategy of storing / accessing the shared string has been changed (stored in the document to avoid any copy, and access via the worksheet which are propagated through the objects).
Its a huge pull request but I'm still working on table support to add capabilities:

autofilter
formatting
more access fonctionalities

A new pull request as I am a little bit ahead with table functionality (formulas, styles, ..) I also corrected some bugs

adding functionality for table support. Missing autofilter implementation

Table support fully available


Send user_typing event?

Is there a way to send the user_typing event on Slack RTM? I can't find it anywhere!!

Hello,
I believe your should look at this PR https://github.com/howdyai/botkit/pull/73

@martip-sainsburys Did you get your question answered?


Introduce account generator

Important: https://github.com/aeternity/aeternity/pull/3978  https://github.com/aeternity/aeternity/pull/4025  is required for this to work !
Update 24.Oct.: The dependencies for generating wallets have been moved to devmode now.
Utilising a hook function, accounts which are to be prefunded are generated and saved to the AE__CHAIN__DB_PATH path, where the node checks for their presence whenever devmode is ran.
Further on, the AE__CHAIN__DB_PATH should serve as a work space, on a higher level of abstraction. Everything needed to work on some chain state is contained in the workspace. Code-wise, the only thing that changes is that the files with the prefunded accounts are now also stored inside it (that was not the case with the other accounts JSONs, bundling everything in one folder makes a project self-contained, you can easily just pass around a folder and get going where somebody else left off.)
The generating of the accounts happens in a fully new module which could be abstracted as a separate application one day or so, to be easily imported wherever needed.
For now, 10 prefunded accounts are generated. More options will follow, like passing a seed or a mnemonic.
The devmode status API puts the account data up for grabs:
curl --location --request GET 'http://localhost:3313/status' --header 'Accept: application/json' | jq -r '.prefunded_accounts'

[
  {
    "initial_balance":<PHONE_NUMBER>0000000000000000000000,
    "priv_key": "2b8d52535e0fdabb280d1e26fcb0ad2ecedf17beacb3e20d16dff597d3e103b362339c5ec6f768569f6a7d386f9aefc11dae882363848505e784750f501ac45e",
    "pub_key": "ak_kFRqSfUhDjSq285KSkAMnz2r5CNwhQzyjm7AaGUacUMbEab9p"
  },
  {
    "initial_balance":<PHONE_NUMBER>0000000000000000000000,
    "priv_key": "f0c8d6bc6633ae4a09229edbfb6bbf9c601517bcf67d916d0e5bb7973834ef116f1750986a584b91c79a546b428dcf67357ddd2e83be4c69ff2e4b1f4c56b554",
    "pub_key": "ak_qvfzs9Pr4R1rbJ8qposU5EQfkqqetmWPrnYcjnYvBNUY2gCsv"
  }
]

I will update the documentation accordingly asap, it's just unbearably hot right now.

CC @uwiger
It can be tested by building the node with https://github.com/aeternity/aeternity/pull/4025, then running
AE__SYSTEM__DEV_MODE=true \
AE__SYSTEM__PLUGINS='[{"name":"aeplugin_dev_mode"}]' \
AE__CHAIN__DB_PATH='/some/path/aeternity/_build/prod/rel/aeternity/some_empty_folder/' \
~/path/to/aeternity/_build/prod/rel/aeternity/bin/aeternity console

If no custom chain db path is provided, it's taking the default one. in any way, if there seems to be some DB present in there already, it will not generate custom accounts.

@uwiger any remarks ?


WIP: V20

This is a V20 sheet that simplifies the end-user experience and integrates well with health bars etc. I'm using jinja to generate all of the HTML code.
One of the major problems I'm finding running V20 is that it is a pain to keep track of experience and health in game, so this is really focused on fixing those two aspects. Also adding bells and whistles like support for discipline sheets.
Tasks

[X] Freebie Points Spent
[ ] Backgrounds
[ ] Willpower
[ ] BloodPool
[ ] Merits
[ ] Flaws
[x] Base Points Spent (7/5/3 etc.) (maybe want user to specify primary/ secondary attr/abil to make sure points are correctly spent?)
[x] Experience Points Spent (any points past (freebie + base = j) get summed with sum_j:n(n*X) type formula
[x] Handle case for attr/abil with no free points or base points (new disciplines etc, and cost diff)
[ ] Health Macro for Carrying Damage from Bash -> Lethal -> Aggravated
[ ] Compendium Loader (Load a user supplied file of disciplines, avoid copyright issues by not embedding it in the sheet, and making the user type it out from the book in a separate file, but at least something they can keep around and use for games) Include format in docs.

wing-it will probably tell you to consider integrating these changes to the existing sheet, as Roll20 has the policy to only have one sheet/system, except for the biggest few. If it is easy to do is a whole another matter.

@Anduh Thanks for the heads up. I'll keep working to make it functional and then see how best to merge it in with the existing sheets. There are a lot of radical changes here, so not much similarity to existing sheets.

To expand a little on what Anduh said, yes we do prefer to keep it to one sheet per system. If it is at all possible to submit this as an upgrade to the existing sheet, we'd appreciate that.
If that is what you'd like to do, you can go ahead and close this pull request, and re-submit it when you're ready to replace the old one.
Thanks!

@wing-it OK, will do. I'll close and reopen as an overwrite on the existing sheet. I'll try to pull in the cool parts of the other sheet as well.

@wing-it If you don't mind, going to leave this up until I finish my remaining tasks, then try to merge it in and open a new PR for that then, just easier for me to stay organized that way and review changes with other authors.

@wing-it @Anduh  this is now setup to replace CWOD-V20, rebased as 1 commit. Still need to finish tasks above.

Need to tie in calcs for discp/ background but layout is done.

@Anduh @wing-it One argument for making this a new sheet and leaving the old one is this:
The old sheet just records strength, so there is no way to know how you got there xp wise, which is why I have the new input fields. So I can't take the old attribute data and upgrade to the new sheet and calculate the xp of the character since I don't have the data.
For instance, strength 5 could mean (1 base point + 2 freebie points + 3 xp points) or (2 base point + 0 freebie points + 3 xp points) which would both have different impacts on the xp for the character.

@alanman @Smil3yRiley @JackNexus @Twibbit @mantonyz thoughts?

I'll start with I do not myself use this, so consider more users opinions on this if possible.
Would one way be to use tabs? first tab have the regular view, and the second would be this. Then you could use the "base" variable shared between the tabs, so the old view would have all as it used to, and anyone who want to switch to the new way could divide the points however it's supposed to be done.

@Anduh Yeah, at start I need to cycle through and make sure strength = base + free + xp, and add to base if it doesn't I guess. Is it best to do that on sheet load? On start I get a lot of warnings about undefined attributes on sheet load, so I'm not sure how to wait until it connects to the sheet properly and fixes the undefined values into zeros.

Current status.

Current status.

I don't know how it would be best to resolve that, sorry. I no experience with sheet workers. But put up a thread on the roll20 forums and people will have suggestions, maybe even someone using the sheet.

There's no way to delete attributes via sheet workers, unfortunately. Setting them to the empty string should reset them to the default, but that may interact weirdly with autocalc fields (do not use autocalc fields, ever). A good solution would be to do this instead:

Let strength (and whatever other fields this apply to) be calculated by sheet workers from strength_base etc instead. This will also improve your sheet's performance.
On sheet opening, if the version attribute is undefined, perform a conversion function that takes the old strength if present and puts it into the new strength_base. This conversion function should probably be set up in such a way that it does nothing if run on an empty sheet.
Also on sheet opening, set a version attribute (not defined in the HTML, only in sheet workers) that tells you what version of the sheet a particular character has been last opened with. If undefined, you know it's either a new sheet or an old sheet that needs to be converted, which tells you to run the conversion function in 2. If not undefined, you're good (or you can run some other upgrade stuff if you ever change attribute names between sheet versions).

Thanks @joesinghaus  I'll go ahead and finish removing all of the autocalcs then.

Working on a new layout to support dots and table style @Anduh.

Closing this for now until we get something more tested by our group.


SCARE corpus, app reviews

Paper: https://aclanthology.org/L16-1178/
Website: http://romanklinger.de/scare/
Public download: http://www.romanklinger.de/scare/scare_v1.0.0.zip

TODO

[ ] Download script for http://www.romanklinger.de/scare/scare_v1.0.0.zip
[ ] Download the actual texts/reviews???

blocked
Restricted access to text inputs. Thus, too inconvenient for automatic downloads.


How to set extension respect prettier wrap style?

Describe the bug
I use prettier to format my code, but It often happens that prettier format the code after this extension modify code. In face, I think the wrap style should be prettier's responsibility.

Thanks for the feedback!
You can set "wrappingStyle": "prettier" if you prefer Prettier's style.
More details can be found in the wiki: Line-Wrapping-Style.

Fine, thanks


How does keras handle the mask padding  if go_backward == 1?

I am not sure if I get it right. But it seems to me that keras always pads the mask on the left:
    if pad > 0:
        # left-pad in time with 0
        padding = alloc_zeros_matrix(pad, mask.shape[1], 1)
        mask = T.concatenate([padding, mask], axis=0)
    return mask.astype('int8')

But if go_backward ==1, it should pad the sequence on the right, right? or override the get_input function to return X[::-1].
Can anyone help explain what did I miss?
Thanks!
@fchollet  @EderSantana

Good question, @dbonadiman was that you who was working on go_backward? Could you check this out? Thank you!

Should be easy to configure the padding side to change depending on go_backwards. Thanks for pointing this out. @dbonadiman, thoughts?

We were talking about that for a while in other threads the mask have problems when applying the backward step or the patting is on the right side. I suggested the other person that pointed that out to write something into the new backend thread since i think that is possible to solve the problem on the  rnn API. what do you think @fchollet or @EderSantana.
Otherwise as soon as i finish some work i can try figure out how to solve this problem on each layer.

I think that a better use of mask could smoothly solve that problem:
h_t = self.activation( x_t + T.dot(h_tm1, u))
return mask * h_t + (1 - mask) * h_tm1

Note that with that proposed use of mask, we don't washout the states when the padding starts, we simply keep the previous value of the hidden state before the zero padding. This would work regardless of padding on the left or on the right. I like that solution because it will also be useful for the transferable/stateful states here https://github.com/fchollet/keras/pull/964
What do you guys think?

＠EderSantana  seems like a good solution for stateful rnn, Would you mind explaining a little bit more about how it solve the padding problem? I think you still need to adjust the padding on left or right based on go_backward. right?

paddings are zeros, we care about the mask from the padded sequence. Inside the RNN, we reset the state every-time we see a mask=0. This is why we pad on the left, so we can safely zero the state out before the actual sequence begins. If we pad on the right, we lose information. With the solution above, we never reset the hidden state, we just use the previous valid one and pass it along. This way, it doesn't matter if you have pad on the left or on the right.

Thanks for your explanations.  @EderSantana
I am not sure if I fully understand this.  But from the following code in class SimpleRNN(Recurrent), I think if the rnn is stateful, the outputs_info should be initilized with some meaniful 'values of someth rnn's hidden states' rather than 0. and in this case the mask should also be padded with 1 rather than 0.  What do you think?
    outputs, updates = theano.scan(
        self._step,  # this will be called with arguments (sequences[i], outputs[i-1], non_sequences[i])
        sequences=[x, dict(input=padded_mask, taps=[-1])],  # tensors to iterate over, inputs to _step
        # initialization of the output. Input to _step with default tap=-1.
        outputs_info=T.unbroadcast(alloc_zeros_matrix(X.shape[1], self.output_dim), 1),
        non_sequences=self.U,  # static inputs to _step
        truncate_gradient=self.truncate_gradient,
        go_backwards=self.go_backwards)


fix(wal): add sync_seqn to WAL and skip recovery when not matching.

Our sync order of operations is:

WAL + beatree shadow pages
meta update
destructive hash-table writes
WAL truncate

specifically, the WAL is for crash consistency in situations after step 2: the beatree has the new values but the hash-table buckets might be incompletely altered.
therefore, a crash before step (2) has completed should not lead to a WAL recovery. This PR fixes and tests this by adding the sync_seqn to the WAL and checking during recovery.

#638  👈 (View in Graphite)
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Merge activity

Dec 19, 6:43 AM EST: A user started a stack merge that includes this pull request via Graphite.


fix(website): fix issue with white logo text and light mode

It seems Vercel preview is still not triggered.

a237d2a

it should be fixed now. but on this branch it is not deploying because it checks the diff against only the last commit, so i'd have to force push this branch to trigger it

Maybe git diff supports a parameter to diff the last N commits.


Grabber is not working

Select your issue type: (check at least one)

[x] Bug
[ ] Question
[ ] Suggestion
[ ] Other (please describe):

Describe your issue:
I had installed grabber, and it is running, but it didn' t work on the web page.

@Kishiomi https://github.com/Zebraslive/Mass-Anime-Downloader


Configurable mocks

Summary
A user can set that loggers should be mocked or not
What is the expected correct behavior?
The user wants to forward logging events to real loggers and mocks only some loggers. There are two ways: set a set of mock loggers, otherwise set a set of real loggers.
Possible implementations
The extension's logger finder has properties:

include list
exclude list
logger implementation reference, enum or class-name

There are two ways:

Property file as default, the logger finder searches and loads this file.
Class annotation is associated with the extension, then the extension looks for the annotation and reads its fields, then the extension sets properties of the logger finder

The original idea was proposed by @markusschaefer at s4u/slf4j-mock#204

Known implementations (System Logger by Nicolas Fränkel):

Log4J
SLF4J
JUL

It cannot be implemented: the logger finder is a singleton and does not have any knowledge of the test context. When multiple tests simultaneously override the finder configuration, it leads to a concurrency race.


Add spring.cloud.kubernetes.enabled as a metadata

This was already addressed here https://github.com/spring-cloud/spring-cloud-kubernetes/commit/98c7eb52d80f09962553ad8f5b0bf6f3689f8828

right, he's changing the unused class to a simple entry in metadata.

Sorry my commit message is not clear. @spencegibb is right

@eddumelendez sorry about that, can you submit this against the 1.1.x branch and we can merge it forward?

@ryanjbaxter done!


Update Github Actions to use Julia 1.10

Purpose

To-do

Content

[ ] I have read and checked the items on the review checklist.

The required checks need to be updated in the Github repo settings, I don't have access to update them.


api: stabilise and document

[ ] check existing API functionality
[ ] add missing important API functionality
[ ] check API function signatures and parameters
[x] enhance API docstrings (param, returns, raises, versionadded)
[x] plug API functions to existing docs

@jirikuncar @lnielsen do you think the first 3 points are fine? Or do we need to improve somehow the API?

@hachreak we should check with @nharraud because of #30.


TypeError: Cannot read properties of undefined (reading 'searchPagePath') using Typesense theme

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[X] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
I am running a fresh Docusaurus installation on the latest version (2.3.0), and I've been hitting this error every time I attempt to build my site with the docusaurus-theme-search-typesense@next theme.
772f76ad2c7e:/testdocs# yarn run build
yarn run v1.22.19
$ docusaurus build
[INFO] [en] Creating an optimized production build...

✔ Client

✖ Server
  Compiled with some errors in 2.53m

[ERROR] Docusaurus server-side rendering could not render static page with path /search.

TypeError: Cannot read properties of undefined (reading 'searchPagePath')
[ERROR] Unable to build website for locale en.

● Client █████████████████████████ cache (99%) shutdown IdleFileCachePlugin
 resolve build dependencies

● Server █████████████████████████ cache (99%) shutdown IdleFileCachePlugin
 process pending cache items

[ERROR] Error: Failed to compile with errors.
    at /testdocs/node_modules/@docusaurus/core/lib/webpack/utils.js:180:24
    at /testdocs/node_modules/webpack/lib/MultiCompiler.js:554:14
    at processQueueWorker (/testdocs/node_modules/webpack/lib/MultiCompiler.js:491:6)
    at process.processTicksAndRejections (node:internal/process/task_queues:77:11)
[INFO] Docusaurus version: 2.3.0
Node version: v19.5.0
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

I have a cached older version that I tested this against, and it was able to build successfully. This seems to be an issue with 2.3.0, and I noticed #8539 mentions this searchPagePath variable.
I wasn't able to get the sandbox to work the way I needed it to, but I've included the repro steps below along with a sample docusaurus.config.js.
I've included some additional output below showing how I tested this. In the output below, I show I have<EMAIL_ADDRESS>installed, I installed typesense, and ran a build that succeeded. Then I upgraded Docusaurus, ran a build, and got the error.
~install Docusaurus~

772f76ad2c7e:/# cat testdocs/package.json
{
  "name": "test-docs",
  "version": "0.0.0",
  "private": true,
  "scripts": {
    "docusaurus": "docusaurus",
    "start": "docusaurus start",
    "build": "docusaurus build",
    "swizzle": "docusaurus swizzle",
    "deploy": "docusaurus deploy",
    "clear": "docusaurus clear",
    "serve": "docusaurus serve",
    "write-translations": "docusaurus write-translations",
    "write-heading-ids": "docusaurus write-heading-ids"
  },
  "dependencies": {
    "@docusaurus/core": "2.2.0",
    "@docusaurus/preset-classic": "2.2.0",
    "@mdx-js/react": "^1.6.22",
    "clsx": "^1.2.1",
    "prism-react-renderer": "^1.3.5",
    "react": "^17.0.2",
    "react-dom": "^17.0.2"
  },
  "devDependencies": {
    "@docusaurus/module-type-aliases": "2.2.0"
  },
  "browserslist": {
    "production": [
      ">0.5%",
      "not dead",
      "not op_mini all"
    ],
    "development": [
      "last 1 chrome version",
      "last 1 firefox version",
      "last 1 safari version"
    ]
  },
  "engines": {
    "node": ">=16.14"
  }
}
772f76ad2c7e:/testdocs# yarn add docusaurus-theme-search-typesense@next
yarn add v1.22.19
[1/5] Validating package.json...
[2/5] Resolving packages...
[3/5] Fetching packages...
[4/5] Linking dependencies...
warning "@docusaurus/core ><EMAIL_ADDRESS>has unmet peer dependency "react-loadable@*".
warning "@docusaurus/core > react-dev-utils ><EMAIL_ADDRESS>has unmet peer dependency "typescript@>= 2.7".
warning "docusaurus-theme-search-typesense ><EMAIL_ADDRESS>has unmet peer dependency "algoliasearch@>= 3.1 < 6".
warning "@docusaurus/preset-classic > @docusaurus/theme-search-algolia > @docsearch/react ><EMAIL_ADDRESS>has unmet peer dependency "@algolia/client-search@>= 4.9.1 < 6".
warning "docusaurus-theme-search-typesense ><EMAIL_ADDRESS>has unmet peer dependency "@types/react@>= 16.8.0 < 19.0.0".
warning "docusaurus-theme-search-typesense ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "docusaurus-theme-search-typesense > typesense-docsearch-react ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "docusaurus-theme-search-typesense > typesense-docsearch-react ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "docusaurus-theme-search-typesense > typesense-docsearch-react ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has incorrect peer dependency "<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>[5/5] Building fresh packages...
success Saved lockfile.
success Saved 8 new dependencies.
info Direct dependencies
└─<EMAIL_ADDRESS>info All dependencies
├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>Done in 6.61s.
yarn run v1.22.19
$ docusaurus build
[INFO] [en] Creating an optimized production build...

✔ Client

✔ Server
  Compiled successfully in 2.54m

✔ Client

● Server █████████████████████████ cache (99%) shutdown IdleFileCachePlugin
 stored

[SUCCESS] Generated static files in "build".
[INFO] Use `npm run serve` command to test your build locally.
Done in 170.50s.
772f76ad2c7e:/testdocs# rm -rf /testdocs/build/
772f76ad2c7e:/testdocs# yarn upgrade @docusaurus/core@latest @docusaurus/preset-classic@latest
yarn upgrade v1.22.19
[1/5] Validating package.json...
[2/5] Resolving packages...
[3/5] Fetching packages...
[4/5] Linking dependencies...
warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "docusaurus-theme-search-typesense ><EMAIL_ADDRESS>has unmet peer dependency "algoliasearch@>= 3.1 < 6".
warning "docusaurus-theme-search-typesense ><EMAIL_ADDRESS>has unmet peer dependency "@types/react@>= 16.8.0 < 19.0.0".
warning "docusaurus-theme-search-typesense ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "docusaurus-theme-search-typesense > typesense-docsearch-react ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "docusaurus-theme-search-typesense > typesense-docsearch-react ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "docusaurus-theme-search-typesense > typesense-docsearch-react ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>warning "@docusaurus/core ><EMAIL_ADDRESS>has unmet peer dependency "react-loadable@*".
warning "docusaurus-theme-search-typesense > @docusaurus/plugin-content-docs > @docusaurus/core > react-dev-utils ><EMAIL_ADDRESS>has unmet peer dependency "typescript@>= 2.7".
warning "@docusaurus/preset-classic > @docusaurus/theme-search-algolia > @docsearch/react ><EMAIL_ADDRESS>has unmet peer dependency "@algolia/client-search@>= 4.9.1 < 6".
[5/5] Rebuilding all packages...
success Saved lockfile.
success Saved 10 new dependencies.
info Direct dependencies
└─<EMAIL_ADDRESS>info All dependencies
├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>Done in 10.55s.
772f76ad2c7e:/testdocs# yarn run build
yarn run v1.22.19
$ docusaurus build
[INFO] [en] Creating an optimized production build...

✔ Client

✖ Server
  Compiled with some errors in 2.53m

[ERROR] Docusaurus server-side rendering could not render static page with path /search.

TypeError: Cannot read properties of undefined (reading 'searchPagePath')
[ERROR] Unable to build website for locale en.

● Client █████████████████████████ cache (99%) shutdown IdleFileCachePlugin
 resolve build dependencies

● Server █████████████████████████ cache (99%) shutdown IdleFileCachePlugin
 process pending cache items

[ERROR] Error: Failed to compile with errors.
    at /testdocs/node_modules/@docusaurus/core/lib/webpack/utils.js:180:24
    at /testdocs/node_modules/webpack/lib/MultiCompiler.js:554:14
    at processQueueWorker (/testdocs/node_modules/webpack/lib/MultiCompiler.js:491:6)
    at process.processTicksAndRejections (node:internal/process/task_queues:77:11)
[INFO] Docusaurus version: 2.3.0
Node version: v19.5.0
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Reproducible demo
No response
Steps to reproduce

Install docusaurus yarn create docusaurus testdocs classic
Replace the docusaurus.config.js with the one below.
Add Typesense yarn add docusaurus-theme-search-typesense@next
cd to the testdocs directory
Run yarn run build

// @ts-check
// Note: type annotations allow type checking and IDEs autocompletion

const lightCodeTheme = require('prism-react-renderer/themes/github');
const darkCodeTheme = require('prism-react-renderer/themes/dracula');

/** @type {import('@docusaurus/types').Config} */
const config = {
  title: 'My Site',
  tagline: 'Dinosaurs are cool',
  favicon: 'img/favicon.ico',

  // Set the production url of your site here
  url: 'https://your-docusaurus-test-site.com',
  // Set the /<baseUrl>/ pathname under which your site is served
  // For GitHub pages deployment, it is often '/<projectName>/'
  baseUrl: '/',

  // GitHub pages deployment config.
  // If you aren't using GitHub pages, you don't need these.
  organizationName: 'facebook', // Usually your GitHub org/user name.
  projectName: 'docusaurus', // Usually your repo name.

  onBrokenLinks: 'throw',
  onBrokenMarkdownLinks: 'warn',

  // Even if you don't use internalization, you can use this field to set useful
  // metadata like html lang. For example, if your site is Chinese, you may want
  // to replace "en" with "zh-Hans".
  i18n: {
    defaultLocale: 'en',
    locales: ['en'],
  },

  presets: [
    [
      'classic',
      /** @type {import('@docusaurus/preset-classic').Options} */
      ({
        docs: {
          sidebarPath: require.resolve('./sidebars.js'),
          // Please change this to your repo.
          // Remove this to remove the "edit this page" links.
          editUrl:
            'https://github.com/facebook/docusaurus/tree/main/packages/create-docusaurus/templates/shared/',
        },
        blog: {
          showReadingTime: true,
          // Please change this to your repo.
          // Remove this to remove the "edit this page" links.
          editUrl:
            'https://github.com/facebook/docusaurus/tree/main/packages/create-docusaurus/templates/shared/',
        },
        theme: {
          customCss: require.resolve('./src/css/custom.css'),
        },
      }),
    ],
  ],
  themes: ['docusaurus-theme-search-typesense'],
  themeConfig:
    /** @type {import('@docusaurus/preset-classic').ThemeConfig} */
    ({
      typesense: {
        typesenseCollectionName: 'DocusaurusCollection',
        typesenseServerConfig: {
          nodes: [
            {
              host: '<IP_ADDRESS>',
              port: 8108,
              protocol: 'http',
            }
          ],
          apiKey: 'xyz'
        },
        typesenseSearchParameters: {},
        contextualSearch: true,
      },
      // Replace with your project's social card
      image: 'img/docusaurus-social-card.jpg',
      navbar: {
        title: 'My Site',
        logo: {
          alt: 'My Site Logo',
          src: 'img/logo.svg',
        },
        items: [
          {
            type: 'doc',
            docId: 'intro',
            position: 'left',
            label: 'Tutorial',
          },
          {to: '/blog', label: 'Blog', position: 'left'},
          {
            href: 'https://github.com/facebook/docusaurus',
            label: 'GitHub',
            position: 'right',
          },
        ],
      },
      footer: {
        style: 'dark',
        links: [
          {
            title: 'Docs',
            items: [
              {
                label: 'Tutorial',
                to: '/docs/intro',
              },
            ],
          },
          {
            title: 'Community',
            items: [
              {
                label: 'Stack Overflow',
                href: 'https://stackoverflow.com/questions/tagged/docusaurus',
              },
              {
                label: 'Discord',
                href: 'https://discordapp.com/invite/docusaurus',
              },
              {
                label: 'Twitter',
                href: 'https://twitter.com/docusaurus',
              },
            ],
          },
          {
            title: 'More',
            items: [
              {
                label: 'Blog',
                to: '/blog',
              },
              {
                label: 'GitHub',
                href: 'https://github.com/facebook/docusaurus',
              },
            ],
          },
        ],
        copyright: `Copyright © ${new Date().getFullYear()} My Project, Inc. Built with Docusaurus.`,
      },
      prism: {
        theme: lightCodeTheme,
        darkTheme: darkCodeTheme,
      },
    }),
};

module.exports = config;

Expected behavior
A successful build.
Actual behavior
Build failed.
Your environment

Docusaurus version used: 2.3.0
Environment name and version (e.g. Chrome 89, Node.js 16.4): Node.js 19.5.0 (containerized)
Operating system and version (e.g. Ubuntu 20.04.2 LTS): Alpine Linux 3.17.1

Self-service

[ ] I'd be willing to fix this bug myself.

Hi
Indeed it's the PR (https://github.com/facebook/docusaurus/pull/8539) that broke it by introducing Algolia-specific things to our theme-common package. Retrospectively that seems like a bad idea @Josh-Cena 😅 we'd rather not couple theme-common to anything apart from our 3 content plugins IMHO.
However this breaks because the typesense plugin is using Docusaurus internal implementation details and we can't commit to not do changes that annoy plugins. CC @jasonbosco.
If the API is marked as "internal" you should not use it in your code (in particular in plugin code). I can see our internal generateSearchPageLink() is used there:
https://github.com/typesense/docusaurus-theme-search-typesense/blob/1678d5364316bc334bbbe51490a08d5fc11d9abf/src/theme/SearchBar/index.tsx#L71
If you need this code, for now, it's safer to copy/paste it and adapt it to your own need rather than using our internal export.

Workaround that probably works: { themeConfig: { algolia: { searchPagePath: "search" } } }

@slorber Thank you for the heads-up! I essentially forked the built-in Algolia plugin, so I didn't realize these were internal methods. I've just been regularly syncing the Typesense plugin with upstream changes from the Algolia plugin all these months! 😅
I'll now refactor it to not rely on this internal method. Thank you for the pointers.


Auto-generate IAM policies

Manually merge generated definitions with existing code.
Fixes #2269
Superceeds #2384

Ok, we've got a silly problem. Python adds _ characters in the name, but some policy names only vary by a _ e.g.
{Name: "AWSCloudTrail_FullAccess", Value: "arn:aws:iam::aws:policy/AWSCloudTrail_FullAccess"},
{Name: "AWSCloudTrailFullAccess", Value: "arn:aws:iam::aws:policy/AWSCloudTrailFullAccess"},

Logged an issue around the python conflicts (https://github.com/pulumi/pulumi/issues/12314) and am just using the approach used elsewhere in the enum - to drop the leading "AWS" or "Amazon" on the new names.


Updated string for clarity

Summary
Based on feedback from the Mattermost Translation Community, updated the Remove license and download server string to clarify that server will be downgraded to Mattermost Free as a result.
Updated the **Remove license and download** text in-product to clarify that server will be downgraded to Mattermost Free as a result.

/cherry-pick cloud

Cherry pick is scheduled.

/cherry-pick release-7.6

Cherry pick is scheduled.


changes structure for original and corrected test statistics

OriginalTestStatistics now holds original_test_statistics (of type SimpleTestStatistics or EarlyStoppingTestStatistics), OriginalAndCorrectedTestStatistics holds original_test_statistics and corrected_test_statistics (same types).

Coverage increased (+0.03%) to 91.103% when pulling 00fec54330ea21cc8b351e9ca4a32520f914e916 on original_corrected_statistics into de08313c430ad97860276cc55c912721fb435ca8 on master.

Coverage increased (+0.03%) to 91.103% when pulling 00fec54330ea21cc8b351e9ca4a32520f914e916 on original_corrected_statistics into de08313c430ad97860276cc55c912721fb435ca8 on master.

Thanks Dasha! I will add the docs in the same branch..

not finished yet, do not review yet.


Feature not showing up on map when using images prop with svg

having an issue where feature doesn't show up on map.
I'm loading a custom image with images prop. confirmed via logs the images prop is being set correctly.
import marker from './marker.svg';

// tslint:disable-next-line:no-var-requires
const { token, styles } = require('./config.json');

// full options found at: https://github.com/alex3165/react-mapbox-gl/blob/master/docs/API.md
const Map = ReactMapboxGl({ accessToken: token });

const MapComponent = (props) => {

  const [images, setImages] = React.useState(null)
  React.useEffect(() => {
    const markerImage = new Image(30, 30)
      markerImage.src = marker
      setImages(['custom-marker', markerImage])
  }, [])

  // long, lat
  const center = [-83.0458, 42.3314] as [number, number];

  if (images === null) {
    return null
  }

  console.log(images)

  return (
    <Map
      style="mapbox://styles/mapbox/light-v9"
      center={center}
      zoom={[9]}
      containerStyle={{
        height: '100vh',
        width: '100vw'
      }}
    >
      {images.length > 0 ? (
        <Layer
          type="symbol"
          id="marker"
          images={images}
          layout={{ 'icon-image': 'custom-marker' }}
        >
          <Feature coordinates={center} />
        </Layer>
      ) : (

      )
    }
    </Map>
  );
}

even if i use the example from the npm page, still doesn't work
 return (
    <Map
      style="mapbox://styles/mapbox/light-v9"
      center={[-0.481747846041145, 51.3233379650232]}
      zoom={[9]}
      containerStyle={{
        height: '100vh',
        width: '100vw'
      }}
    >
      <Layer type="symbol" id="marker" layout={{ 'icon-image': 'marker-15' }}>
        <Feature coordinates={[-0.481747846041145, 51.3233379650232]} />
      </Layer>
    </Map>
  );
}

updated to doing it via the same way they do it in londonCycle example. and still doesn't work. No errors
const MapComponent = (props) => {
  // [Long, Lat]
  const center = [-83.3677, 42.4990] as [number, number];

  const [images, setImages] = React.useState(null)
  React.useEffect(() => {
    // Create an image for the Layer
    const markerImage = new Image(30,30)
    markerImage.src = 'data:image/svg+xml;charset=utf-8;base64,' + btoa(svg);
    setImages(['marker', markerImage])
  }, [])

  if (images === null) {
    console.log("null")
    return null
  }

  console.log(images)

  const layerLayout = { 'icon-image': 'marker' }

  return (

      <Map
        style="mapbox://styles/beezebrown/ckj9ivjqk2tqt19nd4briqiwq"
        center={ center }
        containerStyle={{
          position: 'relative',
          height: '50vh',
          width: '100%'
        }}
      >
        <Layer
          type="symbol"
          id="marker"
          layout={layerLayout}
          images={images}
        >
          <Feature coordinates={center} />
        </Layer>
      </Map>

  );
}

note that the img works correctly with the exact same src as what I'm providing in the images object

Same issue. Can't get features to load like the demo example here with copy pasting the code. https://alex3165.github.io/react-mapbox-gl/

Same issue. Can't get features to load like the demo example here with copy pasting the code. https://alex3165.github.io/react-mapbox-gl/

@alex3165 @daniel-hauser

@alex3165 @daniel-hauser

Any news on this @Beeze ? I have the same issue, sometimes it loads, sometimes it does not. I can see that the images are properly loaded but the features are not showing up.

const  = PointsOfInterest = [
  {
    value: "restroom",
  },
  {
    value: "men-restroom",
  },
  {
    value: "women-restroom",
  }
]

getImagesPointOfInterest() {
    return PointsOfInterest.map((poi) => {
      const image = new Image(30, 30);
      image.src = `/images/floor-mapping/icons/${[poi.value]}.png`;

      return [[poi.value], image];
    });
  }

<Layer
  id="points-of-interest"
  type="symbol"
  layout={{ "icon-image": ["get", "icon"] }}
  images={this.getImagesPointOfInterest()}>
    {pointsOfInterest.map((poi, index) => {
      return (
        <Feature
          key={poi.uuid}
          properties={poi.properties} // ex : {icon: "restroom"}
          coordinates={poi.geometry.coordinates}
        />
      );
    })}
</Layer>

HI, any solution on this? Getting the exact same issue; directly copying the examples from the github but getting no feature displayed.

I couldn’t figure it out, i added deck.gl, and used all their hooks to get this to work. Also, this use case is easier (and better supported) on google maps.
Leigh Halliday on YouTube has example code/videos with both these frameworks

Same on my side, switched to deck.gl/nebula.gl. You can do it with an IconLayer.

I ended up ditching MapBox for Leaflet after seeing that this was unresolved. Hopefully it's addressed in the future.

Hi Guys,
I think this bug has been introduced in v5.
I've installed v4.8.6 and it works (got the marker showing up)
 <Map           style="mapbox://styles/mapbox/streets-v8"           zoom={[8]}           containerStyle={{             height: '100%',             width: '500px'           }}         >           <Layer type="symbol" id="marker" layout={{ 'icon-image': 'marker-15' }}>             <Feature coordinates={[-0.481747846041145, 51.3233379650232]} />           </Layer>         </Map>

@alex3165 @daniel-hauser I understand that you guys might pretty busy but any updates on this? I think lot of guys having issues on this, which should be the first thing to fix if there's any fix in plan.
Thanks a lot~

Hey guys!!
I'm having the same issue here...
It's pretty disappointing that they aren't listening to us really.
  <Map  center={[-0,0]} zoom={[10]} style='mapbox://styles/bcastelo/ckn8pch7b01a817qw1pnjj26w' className="containerStyle" >   <Layer type="symbol" id="marker" layout={{ 'icon-image': 'map_pin' }}>     <Feature coordinates={-0,0]} />   </Layer>   </Map>

Hi Guys,
I think this bug has been introduced in v5.
I've installed v4.8.6 and it works (got the marker showing up)
 <Map style="mapbox://styles/mapbox/streets-v8" zoom={[8]} containerStyle={{ height: '100%', width: '500px' }} > <Layer type="symbol" id="marker" layout={{ 'icon-image': 'marker-15' }}> <Feature coordinates={[-0.481747846041145, 51.3233379650232]} /> </Layer> </Map>

Tried it but to no avail.
Didn't worked for me...
How can I solve this ?


ui: replace go-bindata with embed

Fixes https://github.com/thanos-io/thanos/issues/3782

[x] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes

Replace go-bindata with embed

Verification

I open web UI of thanos query and thanos tools bucket web and saw no error

Why with go-bindata, we ignored (.*\.map|bootstrap\.js|bootstrap-theme\.css|bootstrap\.css)?

One check blocked by https://github.com/prometheus/golang-builder/pull/109

Why with go-bindata, we ignored (.*\.map|bootstrap\.js|bootstrap-theme\.css|bootstrap\.css)?

Source maps increase the binary's size by a lot and I don't remember about bootstrap. IIRC it's because we use the minified versions in the final build:
pkg/ui/templates/_base.html:        <script src="{{ pathPrefix }}/static/vendor/bootstrap-4.1.3/js/bootstrap.min.js?v={{ buildVersion }}"></script>
pkg/ui/templates/_base.html:        <link type="text/css" rel="stylesheet" href="{{ pathPrefix }}/static/vendor/bootstrap-4.1.3/css/bootstrap.min.css?v={{ buildVersion }}">

@longngn Let's rebase and move forward with this as well.

Hey @longngn 👋 We're going to cut a release candidate soon, is there any chance for you to push this to the finish line?

Hey @longngn 👋 We're going to cut a release candidate soon, is there any chance for you to push this to the finish line?

I've rebased with the main branch. AFAIK there's no way to exclude certain pattern with go embed, so the resulting binary might be heavier. On my Mac the binary size is 67MB.

@longngn Thanks for the heads up.
I think we need to discuss if we really want this?
cc @bwplotka @GiedriusS @onprem

Is this something we can tell the Go goembed community? Create some issue?
I don't think 67MB is considered big, but for safety I would release Thanos 0.20 without it. WDYT?

We still have a build process that produces the build artifacts of the react app, no? What's the problem with deleting everything except the resulting artifact for release binaries?

I think we are excluding the source-maps from the build artifacts currently which contributes to the size reduction. Since we are deleting these source-maps anyways, I vote for not even generating them (this can be done setting env variable GENERATE_SOURCEMAP=false).

I just did three builds locally, thanos-bindata is current main branch, thanos-embed-sourcemap is the build from this branch which contains the sourcemaps, and the third thanos-embed binary is with a patch on top of this branch to not generate sourcemaps (modified scripts/build-react-app.sh to have PUBLIC_URL=. GENERATE_SOURCEMAP=false yarn build).
-rwxr-xr-x  1 prem prem  56M Apr 21 14:51 thanos-bindata
-rwxr-xr-x  1 prem prem  61M Apr 21 15:02 thanos-embed
-rwxr-xr-x  1 prem prem  69M Apr 21 14:58 thanos-embed-sourcemap

There's about 5M of difference between bindata (without sourcemaps) and go-embed without sourcemaps which sounds acceptable to me.

@longngn Could you please rebase?

@longngn Could you please rebase?

Done.
IMO we could delete the sourcemap and unnecessary files in release pre-build step.


Design: Path Resolver

First pass of design to get comments from the community.
Introduction of two pluggable components.

Path Resolver
Runtime Validator

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I'm going to look to add Ruby to this after we merge the Ruby build logic. Basically, the following shell one-liner will check compatibility with the ruby2.5 runtime:
ruby -e "unless RUBY_VERSION.match(/2\.5\.\d/); exit(1); end"