Embed Cockroach U videos in docs

[ ] Identify optimal pages per video.
[ ] Establish best practice for where to embed on page and how/when to promote Cockroach U.

We should also:

[ ] Track where videos are embedded, for ease of maintenance.

Your list is correct. I think this would require:

A writer or writers to review the video content and decide what would be appropriate for cross-posting
Work with myself to create a list, by video, of the best doc page for cross-posting
Decide, by page, where and how it would be appropriate to post the content
Implement the videos on the pages
Establish a wiki page to track the location of each video

Next steps: Crossman to put together a wiki page that lists each video and the docs pages the video is linking to.

Created this page for tracking: https://cockroachlabs.atlassian.net/wiki/x/jgDaEw

Sorry for the slow response, I was on vacation. I think some of the content from Getting Started and the Python course would be good for this but needs to re-branded. I don't know if re-branding is on the table for "performance basics" @dweatherbee ?

Yes to all eventually. Re-branding of these course videos are all on the table but as a Lamont side projects after completing the core videos. Core videos require substantial content or are new videos. The rebranding videos only require green to blue/purple  rebranding. I expect most of them will be done by mid-late April.


Does not fully establish connection to Kodi device when internet is disconnected

I cannot connect the Kore app to my OpenElec system when my internet connection goes down.  It just sits there saying "Connecting..."  then eventually it says "Connecting to Downstairs (<IP_ADDRESS>)...".
The Now Playing, Remote, and Playlist tabs are all blank while it attempts to establish the connection.
Both the Kodi device and my phone are on the same LAN.
Steps to reproduce:

Connect Kodi device and Kore to the same LAN.
Connect and confirm that Kore can control the Kodi device.
Close the Kodi app, or turn your phone on standby long enough for the connection to close.
Unplug your router from the internet.
Open the Kore app and try to control the Kodi device.  It should fail to connect and be stuck on "Connecting...".

My internet has been down for 3 weeks.  For now the workaround is to get up and control Kodi using a wired keyboard like a savage.

I could've opened an issue on the forums, however this seemed a pretty generic problem.  Did you actually try to reproduce the issue?  Because I've had this problem on 2 different Android phones, and 2 different wireless networks.  I'm simply unable to connect to the device.

Wow ok I somehow missed the last sentence in your post.  Sorry.  I'll follow up on the forums.


[Story] Rename resourceTemplateParameters to properties

Description
As a UI developer
I want to be able to reference workspace properties in a fashion that makes sense
So that I can built out the UI
It is confusing the workspace properties (that might be inputs or outputs - but not necessarily all template parameters) are accessible under resourceTemplateParameters.  The resource processor selects the properties to be used as parameters prior to deployment. They should be under a more appropriate name such as properties.
Acceptance criteria

[ ] What is currently resourceTemplateParameters becomes properties

@TessFerrandez I know we discussed this, does the above make sense/is there a reason not to do this/do it another way?


ios: external map links

The built-in map view for restaurant detai pages is pretty limited. Perhaps add an option to go to the apple maps app, google maps, etc.

added apple maps, google maps, and open street map


.env fix for demo variables

Description
This PR includes the following proposed change(s):
Updated the window.env settings in efiling-demo to match that of efiling-frontend
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Refactoring / Documentation
[ ] Version change

if your change is a breaking change, please add breaking change label to this PR
How Has This Been Tested?
Running the app locally now works
Does the change impact or break the Docker build?

[ ] Yes
[x] No

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[x] New and existing unit tests pass locally with my changes

that would break docker lol

Modified per our discussion. This should no longer break the docker build, only fix when running locally: yarn start

that would break docker lol

Modified per our discussion. This should no longer break the docker build, only fix when running locally: yarn start


x/review/git-codereview: complains about _content dir in x/website

messageRE in git-codereview is used to validate the commit message title, but this doesn't play nice with the new _content/ directory in x/website.
var messageRE = regexp.MustCompile(`^(\[[a-zA-Z0-9.-]+\] )?[a-zA-Z0-9-/,. ]+: `)

cc @kevinburke @rsc


⚠ Spryr has degraded performance

In 7c6cf02, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 5474 ms

Resolved: Spryr performance has improved in b2570b9 after 30 minutes.


Boundscheck incompatibility in println

The expression &message[0] is used in println.ark to get the pointer to the underlying memory of a string. However, if we're dealing with an empty string this won't pass the bounds check. Ideally we should generalise the RawArray code from mem.ark and use this in println.ark.

Added if len <= 0 checks.


feat(@embark/solc): add embark-solc to monorepo

Updated. Tests should work now 🤞

Restarted build again as it was failing due to remote server errors.

Ok, tests passed. Windows tests were failing because of line-endings


[ADAM-1356] Make ADAMContext.getFsAndFiles and related protected visibility

Fixes #1356

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1761/
Test PASSed.

Merged! Thanks @heuermh!


Calls to dict() does not serialize the STIXdatetime objects

For more information look at the thread in mitre/cti#17

I think this is the intended behavior of the dict() function. It effectively does a shallow copy of the top-level keys, without modifying any of the values. There is some more conversation in that issue that discusses using serialize() instead.


chore(master): release 1.1.5

:robot: I have created a release beep boop
1.1.5 (2023-11-30)
Bug Fixes

change: Fix bug in setting applyingEdits variable (8f6447c)
change: Respect the selections of the editor (#1661) (4b8f242)
skip window changed when target window id is 1000 (#1667) (56ca01f)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/vscode-neovim/vscode-neovim/releases/tag/v1.1.5 :sunflower:


get error while use TLRequestMessageGetHistory

I used as below
            TLRequestMessagesGetHistory tlReq = new TLRequestMessagesGetHistory();
            TLAbsInputPeer _peer = (TLAbsInputPeer) new TLInputPeerChat();
            ((TLInputPeerChat) _peer).setChatId(update.getChannelId());
            tlReq.setPeer(_peer);
            TLAbsMessages messagesEx = kernelComm.getApi().doRpcCall(tlReq);`

but all the time i get exceptions INVALID_CHANNEL_ID or INVALID_CHAT_ID
      ` org.telegram.api.engine.RpcException: CHAT_ID_INVALID`

could you please help me to know why I get this error?

If you want to get the history from a channel, you need to use inputpeerchannel, not inputpeerchat.


Cannnot make use of Multi-Property Support

I'm having trouble using Multi-Property Support, could you give me some hints on how to fix it? There are multiple GA4 datasets in my BigQuery project, such as analytics_11111111 and analytics_22222222. I followed the instructions in this document [1] and wrote the following in dbt_project.yml.
models:
dev:
...
googleanalytics:
schema: googleanalytics
+tags:
- "googleanalytics"
vars:
ga4:
property_ids: [11111111, 22222222]
static_incremental_days: 3
project: "my-project"
dataset: "ga4"
start_date: "20230601"
frequency: "daily"

I also wrote the following in dbt/models/googleanalytics/daily_sessions.sql.

select * from {{ ref('dim_ga4__sessions_daily') }} LIMIT 1000

When I run dbt run --select tag:googleanalytics, I get the following result.

> dbt run --select tag:googleanalytics
10:53:05  Running with dbt=1.4.6
10:53:06  Unable to do partial parsing because a project config has changed
10:53:07  Found 122 models, 21 tests, 0 snapshots, 0 analyses, 474 macros, 0 operations, 1 seed file, 67 sources, 0 exposures, 0 metrics
10:53:07
10:53:14  Concurrency: 1 threads (target='dev')
10:53:14
10:53:14  1 of 1 START sql view model googleanalytics.daily_sessions ..................... [RUN]
10:53:15  1 of 1 ERROR creating sql view model googleanalytics.daily_sessions ............ [ERROR in 0.73s]
10:53:15
10:53:15  Finished running 1 view model in 0 hours 0 minutes and 7.59 seconds (7.59s).
10:53:15
10:53:15  Completed with 1 error and 0 warnings:
10:53:15
10:53:15  Runtime Error in model daily_sessions (models/googleanalytics/daily_sessions.sql)
10:53:15    404 Not found: Table my-project:None.dim_ga4__sessions_daily was not found in location my-location
10:53:15
10:53:15    Location: my-location
10:53:15    Job ID: xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
10:53:15
10:53:15
10:53:15  Done. PASS=0 WARN=0 ERROR=1 SKIP=0 TOTAL=1

What am I doing wrong?
[1]https://github.com/Velir/dbt-ga4/tree/4.0.0/#multi-property-support

Is that the actual indentation in your dbt_project.yml file? If so, the problem is with indentation. Indentation is really important for YAML files.
Presuming, indentation is not the issue, change your vars block to match the package name which is 'ga4' and not 'googleanalytics'.  You can use 'googleanalytics' as a tag, but the identifier for the models and vars block is 'ga4'.
Here's what I think it should look like with the corrected identifier and spacing.
models:
  dev:
...
  ga4:
    +schema: googleanalytics
    +tags: "googleanalytics"
vars:
  ga4:
    property_ids: [11111111, 22222222]
    static_incremental_days: 3
    project: "my-project"
    dataset: "ga4"
    start_date: "20230601"
    frequency: "daily"

The reason that it tried to run one model is that your daily_sessions.sql model is in your project (ie outside the package). The package settings aren't recognized because your using the wrong identifier.
Try these settings.

@dgitis
Thank you for your reply.
Regarding the indentation, it was a problem that occurred when I pasted it, and in my actual configuration file, the indentation is correct. I apologize for causing confusion on that point.
Taking your advice into consideration, I changed dbt_project.yml as follows and ran dbt run --select tag:googleanalytics, which resulted in the following output:

models:
  dev:
...
  ga4:
    +schema: googleanalytics
    +tags: "googleanalytics"
vars:
  ga4:
    property_ids: [11111111, 22222222]
    static_incremental_days: 3
    project: "my-project"
    dataset: "ga4"
    start_date: "20230601"
    frequency: "daily"

result

> dbt run --select tag:googleanalytics
04:09:47  Running with dbt=1.4.6
04:09:47  [WARNING]: Configuration paths exist in your dbt_project.yml file which do not apply to any resources.
There are 1 unused configuration paths:
- models.dev.ga4
04:09:47  Found 122 models, 21 tests, 0 snapshots, 0 analyses, 474 macros, 0 operations, 1 seed file, 69 sources, 0 exposures, 0 metrics
04:09:47  The selection criterion 'tag:googleanalytics' does not match any nodes
04:09:47
04:09:47  Nothing to do. Try checking your model configs and model specification args

Also, I changed the directory name of dbt/models/googleanalytics/daily_sessions.sql to dbt/models/ga4/daily_sessions.sql and ran it again, but the same error as 404 Not found: Table my-project:None.dim_ga4__sessions_daily was not found in location my-location occurred.
I searched for the query that was executed from the job id in BigQuery, and found that the following SQL was executed.

/* {"app": "dbt", "dbt_version": "1.4.6", "profile_name": "default", "target_name": "dev", "node_id": "model.dev.daily_session"} */

  create or replace view `my-project`.`googleanalytics`.`daily_session`
  OPTIONS()
  as select * from `my-project`.`None`.`dim_ga4__sessions_daily` limit 1000;

I still need your help. Could you give me any ideas?

This line in the error message:
There are 1 unused configuration paths:
- models.dev.ga4

That indicates to me that you have the nesting wrong as ga4 is nested under dev.
Jinja is very picky about nesting. dbt Cloud does some stuff to fix this, but, when things like this go wrong, be sure that you are using double space (and not tab) to nest.

@dgitis
Thank you for your response.
For the nesting issue, as you pointed out, I was entirely wrong. I've corrected my dbt_project.yml. I apologize for not fixing such easy mistakes multiple times. (Actually I'm new to dbt.)
However, the result of the dbt run execution is as follows.

> dbt run --select tag:googleanalytics
09:42:18  Running with dbt=1.4.6
09:42:18  Found 122 models, 21 tests, 0 snapshots, 0 analyses, 474 macros, 0 operations, 1 seed file, 69 sources, 0 exposures, 0 metrics
09:42:18
09:42:20  Concurrency: 1 threads (target='dev')
09:42:20
09:42:20  1 of 30 START sql incremental model googleanalytics.base_ga4__events ........... [RUN]
09:42:23  BigQuery adapter: https://console.cloud.google.com/bigquery?project=my-project&....
09:42:23  1 of 30 ERROR creating sql incremental model googleanalytics.base_ga4__events .. [ERROR in 3.17s]
09:42:23  2 of 30 SKIP relation googleanalytics.stg_ga4__events .......................... [SKIP]
09:42:23  3 of 30 SKIP relation googleanalytics.fct_ga4__sessions_daily .................. [SKIP]

...........

09:42:23  30 of 30 SKIP relation googleanalytics.fct_ga4__user_ids ....................... [SKIP]
09:42:23
09:42:23  Finished running 6 incremental models, 17 view models, 7 table models in 0 hours 0 minutes and 5.30 seconds (5.30s).
09:42:23
09:42:23  Completed with 1 error and 0 warnings:
09:42:23
09:42:23  Database Error in model base_ga4__events (models/staging/base/base_ga4__events.sql)
09:42:23    my-project:googleanalytics.events_* does not match any table.
09:42:23    compiled Code at target/run/ga4/models/staging/base/base_ga4__events.sql
09:42:23
09:42:23  Done. PASS=0 WARN=0 ERROR=1 SKIP=29 TOTAL=30

It seems that dbt is looking for a table named googleanalytics.events_* rather than analytics_11111111.events_*. and analytics_22222222.events_*.

@dgitis
Thank you for patiently replying my question.
The cause is unknown, but after trying several times, I was successful in executing dbt run --select tag:googleanalytics.
However, there is still a remaining issue. To create the ga4_source_categories table from the seed file, I placed the dbt/seeds/ga4/ga4_source_categories.csv file.
In dbt_project.yml, based on the document [1], I have made the following description so that the seed file is not shared between the ga4 project and dev:

seed-paths: ["seeds"]

......

seeds:
  ga4:
    +enabled: true
    +schema: seed_data

However, the following problem has occurred and has not yet been resolved. What could be the cause?

> dbt run --select tag:googleanalytics
11:45:57  Running with dbt=1.4.6
11:45:58  Encountered an error:
Compilation Error
  dbt found two seeds with the name "ga4_source_categories".

  Since these resources have the same name, dbt will be unable to find the correct resource
  when looking for ref("ga4_source_categories").

  To fix this, change the name of one of these resources:
  - seed.dev.ga4_source_categories (seeds/ga4/ga4_source_categories.csv)
  - seed.ga4.ga4_source_categories (seeds/ga4_source_categories.csv)

There is only the dbt/seeds/ga4/ga4_source_categories.csv file in the seeds directory.
[1]https://docs.getdbt.com/reference/seed-configs

You are over-thinking the seeds. The seed file already exists in the package.
You don't need to move it anywhere or upload it or anything. You don't need any seed settings in your dbt_project.yml file.
You literally just install the package and then use the dbt seed command.

Why don't you want to share the seed between the project and dev? It's a tiny file so cost is no issue and both the project and dev environments need it.


[RND-671] Fix Meadowlark CodeQL scan

Update CodeQL action versions.
Reorganize on pull request action to update dependencies before analyzing the code because there could be sync issues between the two.
Adding Coverage flags to unit tests run.

See the workflow change in action here: https://github.com/andonyns/Meadowlark/actions/runs/6896275766


How to get the result of the validation set during the train set

I have a question!
How to calculate the classification accuracy and positioning accuracy of the current model on the validation set for every epoch in the training process, just like YOLOv3 ?
Thanks a lot

I also face the same issue. At least I want to plot the validation loss somehow.

same issue


foundryup instructions on macOS (amd) missing note to install required dependency libusb

Component
Foundryup
Have you ensured that all of these are up to date?

[ ] Foundry
[ ] Foundryup

What version of Foundry are you on?
forge 0.2.0
What command(s) is the bug in?
No response
Operating System
macOS (amd)
Describe the bug
Got this error when installing foundryup via curl -L https://foundry.paradigm.xyz | bash:

foundryup: downloading latest forge and cast
######################################################################## 100.0%
foundryup: downloading manpages
######################################################################## 100.0%
dyld: Library not loaded: /usr/local/opt/libusb/lib/libusb-1.0.0.dylib
Referenced from: /Users/ayo/.foundry/bin/forge
Reason: image not found
foundryup: installed -

For older versions of macOS, please consider adding required dependencies to the docs: brew install libusb (if Homebrew is default package manager)

It is documented here: https://github.com/gakonst/foundry#libusb-error-when-running-forgecast
Do you have any suggestions for how to improve this?

#1170 is a potential fix


embrace CSS

@jelmedini I think it's preferable for the position attribute to be as consistent as possible.
So I added cd21616

@louis-bompart just checked, and actually the effect inside SFINT is like this, we have a color change on hover. I also validated with the UX this change. so we are good to keep the background color hover in general :)

👍 feel free to merge this PR into yours :)


Animation and Hover effect on Nav title

Related Issue
Adding text animation, and on-hover effects.
In relevance of Issue #553
Closes: #553
Describe the changes you've made
The sub text on banner is given an animation, using typewriter effect by using typed.js.
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Checklist:

[ x] My code follows the style guidelines of this project.
[x ] I have performed a self-review of my own code.
[x ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ x] New and existing unit tests pass locally with my changes.

Screenshots
Put any screenshot(s) of the project here.

@urvashi-code1255  okay will do so and make a new PR soon.

@urvashi-code1255 pls review it again and let me know if more changes are required

@991rajat @urvashi-code1255 kindly review this PR

Great Work.
Resolve the conflict.

Thanks for this awesome contribution. :star_struck:
Keep Contributing.


Specify setting CU username and password in running examples section

In the "Running and verifying all the samples" specify that the CU username and password should be set before running samples:
export HSM_USER=<cu_username>
export HSM_PASSWORD=<cu_password>
or
java -DHSM_USER=<cu_username> -DHSM_PASSWORD=<cu_password> ...
See also issue #3 and #68

Updated readme in c57ad1b . Leaving other issues open for specific sample modifications.


How to integrate Langchain's Human Tool into Streamlit

System Info
I'm using Langchain's Human Tool as part of my application. However, I am having difficulty integrating it into streamlit.
I'm mainly trying to change the input_func argument in the function:
Human_Tool =load_tools(["human"], lm=llm, input_func=get_input)
I've tried many different approaches but have not had success yet.
I want to ask what might be a get_input function for langchains' Human Tool that integrates successfully with streamlit.
Who can help?
@hwchase17
@agola11
Information

[X] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[X] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
Here are some of the things I've tried:
   import streamlit as st

    def get_input() -> str:
        st.write("Insert your text below.")

        # Updated this line to include a label and hide it using label_visibility
        user_input = st.text_area(label="Your Text", value="", height=100, key=None, help=None, on_change=None, args=None, kwargs=None, label_visibility="hidden")

        submit_button = st.button("Submit")

        if submit_button:
            return user_input.strip()  # This will return the text entered by the user

        if not submit_button:
            with st.empty():
                for remaining in range(3600, 0, -1):
                    st.write(f"You have {remaining//60} minutes and {remaining%60} seconds to enter the text.")
                    time.sleep(1)
                    st.empty()

        return ""  # Returning empty if no input after an hour or not submitted

The code above opens a streamlit box that and accepts the user's input but it does not pass the users input to the agent.
import streamlit as st
import uuid

def get_input() -> str:
    unique_key = str(uuid.uuid4())

    user_input = st.text_area(
        "Insert your text below:",
        key=f"uniqueTextAreaKey_{unique_key}",
        value="",
        height=200,
    )

    if st.button("Submit", key=f"uniqueButtonKey_{unique_key}"):
        if user_input:  # check if user input is not empty
            st.session_state['user_input'] = user_input
            st.session_state['submitted'] = True

            st.success("Input submitted successfully!")
            return user_input.strip()

    elif 'submitted' in st.session_state and st.session_state['submitted']:
        st.success("Input already submitted.")
        return st.session_state['user_input'].strip()

    else:
        st.warning("Please enter text and press submit.")

    return ""

This code also opens up a streamlit text box but gets stuck in an infinite loop.
Expected behavior
I want the human tool to integrate with streamlit.

Hi @sharifamlani, did you manage to solve this?

Worked for me but when I respond, it reloads the page and I have to start over.

Is anyone able to get this to work? I can't get the new HumanInputRun to integrate with the agent properly

Hi @sharifamlani, please post the solution if you were able to solve this :-)

Yes, got it to work. Code is here: https://github.com/Definitive-AI/Agents

@jfjeschke Still it reloads the app (change the state) when we add the input, right? I tried to implement it but didn't work.

@jfjeschke Still it reloads the app (change the state) when we add the input, right? I tried to implement it but didn't work.

Just use a single instance, like this:
streamlit_tool = StreamlitInput()
Otherwise it'll try to create multiple fields.

I tried this one. but it reloads the app and again starting from the beginning. Did you face that issue?


chore: refactor code

RunE provides a better way to report errors
also os.Exit (called by cobra.CheckErr) should be avoided outside main.go
Otherwise, code is difficult to test

pushed back because 94f7a2b09c906ced7c641df604a5da153aaea71c was pushed out of PR proccess and broke my PR

Sorry about that! Thanks for the PR!

My pleasure


go-github: add support for getting content of a repository's license

Fixes #228.
Change-Id: I43d414332baa87b430487ea03d125633f8e40bd5

License support was added in c35517a7668e464e4795d4da07e85c6b2a7a1d3e.  Looks like #228 just never got closed.

err, nevermind.  That was for a different issue.

Our normal naming convention of following the GitHub docs would say that this belongs on the LicenseService, but I actually like what you have better...
client.Repositories.License("owner", "repo")
so let's go with this. I do think this is one of the first times we've broken from that convention. :shrug:


Issue 110/appheader styling updates

Adding styling to AppHeader. AppHeader is now consolidated with the previous NavBar and changes upon login.
The current layout is based on an in-progress mobile-first design developed by the UX team. As such, it is subject to change based on their results. It contains a search bar, menu, and notifications icon, each one responsive to screen size changes.
The Home and Forms tabs have been moved into the menu to reduce clutter. The remaining icons, such as My Account, are currently not functional.
The first commit (a8dec0c) styles only the existing elements present in the Development branch and can easily be reverted to.

I feel like this one may take a discussion (especially with Design Team.)
I will talk about design here, and actual code tomorrow in new comment.
There are things I like, and things I dislike.

1. On mobile view we lose all branding in Navbar. There's no logo/text. And we are leaving Search and Notifications.

I can understand Notifications, but Search? How often do you search a full website?
Personally I think it should be app Name/Logo on left and Hamburger menu on right

2. This Search... is it searching the whole site?

If so, we should leave in Navbar.
If it's only searching the screen we are on (the displayed route/component), then it shouldn't be in Navbar. It should be in the route/component below Navbar.

3. I like the search icon, animation when clicked to open... I dislike that it moves the app title.

I think that should stay in place and search fills the space allotted to it.

4. Add button in Hamburger menu should not be in Navbar.

This button should be it's own component that gets rendered/displayed in the route/page shown below the Navbar.
This button will either be adding cases/users/people, documents, or whatever other functionalities we come up with and belongs with those things.
It's not a navigation (search could be, IF it's searching the whole site - if searching just page, should only be in page and not Navbar either.)

This was an older iteration, but example:

5. In Hamburger menu I think My Account should be under the navigation links.

I feel it should be all the routes the user may want to go to, then My Account, then Log Out
And maybe Log Out in a different color too? Maybe the red, sort of like a "danger" - more intuitive that way?
Reason for this layout (Routes, Account, Logout) is what will be commonly used. Much more common to change pages/views/tabs than to go to your account settings. A lot of people don't log out of any website accounts before closing either.

Had a look at the design from the branch. I think we should have a discussion with the UX team first before implementing this, especially since this is "in-progress" as stated from the PR. It's likely not what they want implemented at the end.
To that end, I think we should just implement what you had previously on the MUI branch as our baseline styling. The previous iteration seems like a more sensible mobile-first design.

Had a look at the design from the branch. I think we should have a discussion with the UX team first before implementing this, especially since this is "in-progress" as stated from the PR. It's likely not what they want implemented at the end.
To that end, I think we should just implement what you had previously on the MUI branch as our baseline styling. The previous iteration seems like a more sensible mobile-first design.

I have reverted back to the original, minimum amount of styling.

I also thought we're transitioning to Navbar and replacing the AppHeader after our recent discussions?

I also thought we're transitioning to Navbar and replacing the AppHeader after our recent discussions?

The code has already been consolidated. Do you mean changing the file structure and moving it into its own folder?

I also thought we're transitioning to Navbar and replacing the AppHeader after our recent discussions?

The code has already been consolidated. Do you mean changing the file structure and moving it into its own folder?

We did have a Navbar directory with three components in there, so I thought they'll be used. But I don't see it anymore. Did you remove it?

No. It was removed from I rebased from Development.

We did have a Navbar directory with three components in there, so I thought they'll be used. But I don't see it anymore. Did you remove it?

No. It was removed from I rebased from Development.

We should have rebased it such that you add Development under the MUI branch. That way we would be removing any features made in MUI.

Is this ready to be closed?


Reactive form - clear date if null/undefined

In reactive form, if date is null or undefined, it stays on previously selected date. Is it possible to clear date if set date is null or undefined? Thanks

I was about to open same issue, in my case the date field always has a default value which is equal to current date even though the date fields are shown as empty on the screen.
I confirmed it by submitting the form without selecting any value and writing form.value to console.
Another related issue is, in my case when the form is submitted I have to store the form.value in a variable in service, because when user opens the details of any item from search list I have to remove search filters from screen and add it back when the user closes the item details page. I repopulate the filters using the filters value stored in service. All other form fields are populated properly . Even the value is assigned to the date fields but its invisible on screen.

Thanks @zzurka this should be fixed in 0.0.7


Data Factory Contributor

In the "Data Factory Contributor" section it says that the role should be able to "Create and manage data factories, as well as child resources within them." However, I am not able to create an actual data factory. When I try to actually create the data factory via the UI part of the error says "xxx does not have authorization to perform action 'microsoft.datafactory/register/action over scope..." - which per the documentation the Data Factory Contributor does not actually permit the action microsoft.datafactory/register/action
I understand that on this page "https://docs.microsoft.com/en-us/azure/data-factory/concepts-roles-permissions" it clearly states that in order to create an azure data factory instance I must be a member of the contributor role, the owner role, or an administrator of the Azure subscription.
This makes the words a bit confusing. To actually create an Azure Data Factory (like the words on this page say) wouldn't we need more than just the Data Factory Contributor role?

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

ID: 99581afb-7950-6daa-7227-055d04446b7e
Version Independent ID: f1a5bf89-9e54-aede-3a9a-70788a95927b
Content: Azure built-in roles - Azure RBAC
Content Source: articles/role-based-access-control/built-in-roles.md
Service: role-based-access-control
GitHub Login: @rolyon
Microsoft Alias: rolyon

@KrystinaWoelkers Thank you for your feedback . We will investigate and update the thread.

@KrystinaWoelkers To create Data Factory instances, the user account that you use to sign in to Azure must be a member of the contributor role, the owner role, or an administrator of the Azure subscription.
Note: To view the permissions that you have in the subscription, in the Azure portal => Select your subscription => Go to Access control (IAM) => Under Check access => Click on view my access.

To create and manage child resources for Data Factory - including datasets, linked services, pipelines, triggers, and integration runtimes - the following requirements are applicable:

To create and manage child resources in the Azure portal, you must belong to the Data Factory Contributor role at the Resource Group level or above.
To create and manage child resources with PowerShell or the SDK, the contributor role at the resource level or above is sufficient.

For more details, refer to Roles and permissions for Azure Data Factory.
Hope this will help. Please let us know if any further queries.

@KrystinaWoelkers  We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


test-branch/execute-job/fork-branch/223fa40b-b381-487f-9fb2-67a22494b0a0

PR for testing 7501bac6fbe4041b4144391fee217150f8a963fb

padding /canonical/self-hosted-runners/run-workflows 7501bac6fbe4041b4144391fee217150f8a963fb padding


fix: Incorrect theme used for the AppBar title on the Product Edition page

To follow this issue #3804, the wrong theme is used on the Product Edition Page.
Instead of the textTheme, the primaryTextTheme is used in this screen.
The primaryTextTheme has a white color, so white on white gives this behavior.
Before:

After:

Codecov Report

Merging #3805 (34a6b6c) into develop (9d2db67) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff            @@
##           develop    #3805   +/-   ##
========================================
  Coverage    10.67%   10.67%
========================================
  Files          272      272
  Lines        13547    13546    -1
========================================
  Hits          1446     1446
+ Misses       12101    12100    -1

Impacted Files
Coverage Δ

...mooth_app/lib/pages/product/edit_product_page.dart
0.45% <0.00%> (+<0.01%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

thanks @g123k @AshAman999 👍

The barcode should be visible for shorter names, right?

Yes that's correct, we should open an issue for this one (really easy to fix


Conditional dependencies?

Is there any way to specify conditional dependencies?
For example, I have a Justfile with a recipe test. test requires a docker-compose of the project to be running in the background, so test is dependent on a recipe calledup, which calls docker-compose up. up is dependent on the containers being built, so up is dependent on the recipe build, which builds the containers.
All of that works great! But sometimes, I want to run the test recipe twice in a row, and I don't need to rebuild the containers and/or redeploy the docker-compose. So it would be nice if I could somehow in my Justfile specify to only run dependencies if needed (i.e., if docker-compose is already up, don't run the dependency)
I figure I could probably get a similar effect by putting conditional code in the up and build recipes, but I'm wondering if there's a way to keep those clean and put the logic in the recipe that is calling them.

Currently there isn't a way to do this. I think your best bet is to add conditional code in the up, e.g. check if the docker container is running, and only run docker-compose up if it isn't.


go: render code to render path template

The client library should no longer depend on gax.PathTemplate.
Updates googleapis/gax-go#32, since gax.PathTemplate can then
be removed and there would be nothing left to verify.
In addition to rendering gax.PathTemplate is also capable
of parsing a rendered string and returning the template "arguments".
This feature was never exposed by toolkit clients.
If we need it later, we can still easily create parsing code,
like we generate rendering code in this commit.
The Java implementation of PathTemplate escapes URL when rendering.
The Go implementation does not, and so the functionality is not
included here.
While this probably won't matter in production,
benchmarking pubsub's PublisherTopicPath shows
benchmark                    old ns/op     new ns/op     delta
BenchmarkPathTemplate-12     1052          104           -90.11%

Codecov Report

Merging #1400 into master will increase coverage by 0.02%.
The diff coverage is 100%.

Continue to review full report at Codecov.

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


[Sharing NG] Tokeninfo endpoint

The OCS API provides a tokeninfo endpoint for public links (ocs/v1.php/apps/files_sharing/api/v1/tokeninfo/unprotected/{token}) which can be queried unauthenticated. The server responds with same basic data that the Web client uses to correctly resolve a public link.
Therefore Web needs something similar with Sharing NG. The relevant information:

is the link password protected?
is the link an internal link?
the id of the resource

cc @rhafer

Hm, that's the github.com/CiscoM31/godata.TokenMatcher choking on the the ?. I am not whether that is actually a bug in go data or the ? needs any special escaping in odata. Still looking ...

To add a bit more detail on how Web uses the tokeninfo endpoint: When resolving a public link, Web needs to know if the link is internal or external to properly redirect the user to either the resolved public page or the internal location (after user login).
We decided to leave this untackled for now because we're not sure if a tokeninfo endpoint is the best approach to solve this issue.
Web will continue to use the OCS API for the tokeninfo for now.

I have a graph implementation ready, currently just the logic, no tests, no bells and whistles… we can use it as basis for discussion.
not at my desk right now but I’ll add the pr tomorrow. Thanks to everyone that was part of the brainstorming!

Apart from just re-implementing the tokeninfo logic for sharing-ng, we could try to find a cleaner way to get the same functionality.  After some some discussion we came up with this, which should not need a tokeninfo endpoint at all and could purely work with webdav. It is still somewhat rough and we would have to prove that it does actually work:
The client should basically always sent a PROPFIND to /dav/spaces/{publicstorageid}${publicspaceid}!{sharetoken}

authenticated clients accessing an internal link are redirected to the "real" resource (`/dav/spaces/{target-resource-id}
authenticated clients accessing a pubic link (password protected or not) for a resource they already have access to are also redirected to the "real" resource. (No need to supply the password)
unauthenticated clients access an internal link get a 401 returned with  WWW-Authenticate set to Bearer (so that the client knows that it need to get a token via the IDP login page.
unauthenticated clients accessing a password protected link get a 401 returned with WWW-Authenticate set to Basic to indicate the requirement for needing the links password.

This would however require some changes our ocdav service (it does not handled the above request on the publicstorageids yet) and AFAICS also in the way we handle authentication for ocdav. So it requires some more research.

Yes I like that approach more than the old one. But some parts are not really clear to me.

The client should basically always sent a PROPFIND to /dav/spaces/{publicstorageid}${publicspaceid}!{sharetoken}

Where does it get that URL from? It might only have the public link /s/opTtskfWusbUBGy

The public storage and space ids are currently hardcoded AFAIK, so those would basically come from configuration. Alternatively to /dav/spaces we could also use /dav/public-files/{sharetoken} as the entry point I guess. (The authentication issues we neede to figure out are pretty similar in both cases I guess).

authenticated clients accessing a pubic link (password protected or not) for a resource they already have access to are also redirected to the "real" resource. (No need to supply the password)

As much as that would be a smooth user experience, I am not sure we can do this. We are giving away information from a password protected link, without knowing the password. Might be better if we avoid this.

AFAIK this is already the current behavior of  the web client when resolving a password-protected link via the tokeninfo endpoint. And if the user does already have access to that resource we're not giving anymore access that the user would already have.

The public storage and space ids are currently hardcoded AFAIK, so those would basically come from configuration. Alternatively to /dav/spaces we could also use /dav/public-files/{sharetoken} as the entry point I guess. (The authentication issues we neede to figure out are pretty similar in both cases I guess).

Yes better :+1: I would like to avoid clients having to learn server id logic

AFAIK this is already the current behavior of the web client when resolving a password-protected link via the tokeninfo endpoint. And if the user does already have access to that resource we're not giving anymore access that the user would already have.

True, but that doesn't mean that the behaviour is currently secure. We do not disclose access to the file, that is true, but we are giving away information about the link to a user who might not have that information otherwise. I agree the attack vector here is very small, but we should try to avoid such cases completely.

Yeah, maybe I'm exaggerating the security impact. My point is only that I'm not happy with giving away information (in this case the fileid) to a user who has not yet provided the password.

I'm totally with you, hence my suggestion to leave the UX as it already is and not give away anything until the user has entered a valid password. Web can still redirect the user to whatever location necessary afterwards.

Password protected links always prompt the user for a password currently

@JammingBen Just for clarification (so that I can update the requirements/spec). When an authenticated user is accessing a password protected public link the user needs to enter the password for that link?

@JammingBen Just for clarification (so that I can update the requirements/spec). When an authenticated user is accessing a password protected public link the user needs to enter the password for that link?

Correct 👍

Follow Up Ticket https://github.com/owncloud/ocis/issues/8858


Can SyncDreamer render novel views with different elevations from the input view

For example, can SyncDreamer render the view with elevation 0 when the input view's elevation is 30?

It is possible but you may need to retrain the model with such a kind of data. We do not put other constraints on the rendering viewpoints.

Thanks!


Update audit-log-retention-policies.md

Made changes to the note
Fixes https://github.com/MicrosoftDocs/microsoft-365-docs/issues/11654

@JohanFreelancer9 Please copy review

@JohanFreelancer9

@msbemba

@robmazz Please merge this PR. Thanks

closing as per the comment in #11654


[FR] Keep windows tiled while moving + move window to tile with mouse

I would like to have the option of keeping windows tiled while moving them, along with the ability to move a tiled window to another tile using the mouse.

https://zeroxoneafour.github.io/polonium/faq#how-do-i-move-windows-with-my-mouse Does this solve the issue?

https://zeroxoneafour.github.io/polonium/faq#how-do-i-move-windows-with-my-mouse Does this solve the issue?

No, windows automatically become floating windows when moving them.
The expected behavior is how Bismuth behaves when the option "Untile windows by dragging" is off.

An option to change the default behavior would be great

This is not possible with the KWin tiling API, as KWin automatically untiles windows when they are moved. There is no configuration I can do to change that. By shift dragging, you should be able to achieve the end goal of moving the window back into any tile.


Add apt repository matcher

👍

Just a heads up for the future this wasn't actually wired up to work in Chefspec and the examples weren't added to cucumber so we didn't have a way to know that. I fixed it all in #752 so this matcher should be fully functional now.


Add schedule generation with future codes

What is this PR for?
Add schedule generation with future codes (e.g. JUN2017)
What type of PR is it?
Feature

There might be more elegant ways to convert the future code into a LocalDate but it should work

THX


Add Github Workflows + clippy fixes

Workflows before and after clippy fixes. No functionality changes are intended.
before:
https://github.com/Grinkers/edn-rs/actions/runs/5983591578
after:
https://github.com/Grinkers/edn-rs/actions/runs/5983672308

Codecov Report
Patch coverage: 83.33% and project coverage change: +1.18% :tada:

Comparison is base (e74e564) 69.02% compared to head (18d8045) 70.21%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #102      +/-   ##
==========================================
+ Coverage   69.02%   70.21%   +1.18%
==========================================
  Files           8        8
  Lines         946      940       -6
==========================================
+ Hits          653      660       +7
+ Misses        293      280      -13

Files Changed
Coverage Δ

src/edn/mod.rs
61.40% <50.00%> (+1.58%)
:arrow_up:

src/serialize/mod.rs
75.20% <75.00%> (+6.26%)
:arrow_up:

src/edn/utils/mod.rs
58.82% <100.00%> (-1.18%)
:arrow_down:

src/json/mod.rs
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

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

btw with pedantic and nursey, there's many pages of output. Most of them seem decent, but some are what I consider false positives like
https://rust-lang.github.io/rust-clippy/master/index.html#/must_use_candidate
I'm willing to help maintain and go through everything if #103 can get through. Without clojure.edn/read-string behavior, I can't consider using this crate at this time.

I think I can try to address the remaining issues this week. Thanks


Add missing comment hash

Add support for URLs like
// https://github.com/refined-github/shorten-repo-url/pull/33#discussion_r750069394

I moved your access via the "maintainers" team, you should still have write access, can you confirm? You can merge this PR

you should still have write access, can you confirm?

Yes :+1:


Incorrect "the type test for A cannot be checked at runtime because it's a local class" warning

Compiler version
Tested 3.4.2, 3.4.1, 3.4.0, 3.3.3, all with the same issue.
2.13.14 works fine and does not emit warning.
Minimized example
def f: Unit = {
  class A {
    override def equals(x: Any): Boolean = x match {
      case a: A => true
      case _ => false
    }
  }
}

Output Error/Warning message
1 warning found
def f: Unit
-- [E092] Pattern Match Unchecked Warning: -------------------------------------
4 |      case a: A => true
  |           ^
  |the type test for A cannot be checked at runtime because it's a local class
  |
  | longer explanation available when compiling with `-explain`

Why this Error/Warning was not helpful
The code seems correct and runs correctly, i.e. the type test executes correctly at runtime, contradictory to the warning.
Suggested improvement
No warning should be emitted.

The following with member A prints false twice
class B:
  class A {
    override def equals(x: Any): Boolean = x match {
      case a: A => true
      case _ => false
    }
  }
  def f(x: Any): Any = {
    val y = A()
    println(y == x)
    y
  }

@main def test() =
  val b0 = B()
  val f = b0.f(null)
  val b1 = B()
  b1.f(f)

but with local A, it is false (of course) then true. It's merely testing the implementation class.

Can I try my hand at this one? Sounds interesting

Something related: there are no warnings when a local case class is defined. But a local case class C's auto-generated equals method contains pattern match against C, or something equivalent. So if the warning is correct behavior, it should be emitted for local case classes as well.

I think the warning is in fact correct. Consider the following:
def f: Any = {
  class A {
    override def equals(x: Any): Boolean = x match {
      case _: A => true
      case _    => false
    }
  }
  A()
}

@main def foo(): Unit =
  val x0 = f
  val x1 = f
  println(x0 == x1)

The program output is true. But that is wrong, because each invocation of f creates a new class A (this is particularly obvious if A references f's local variables). So the match of x1 against x0's type (that is, the A from the first f invocation) should fail. But it succeeded, thus the warning is warranted.
What needs fixing is that the same warning should be emitted for case classes as well. Consider the following:
trait C:
  def ff: Int

def f(x: Int): C =
  case class A() extends C:
    def ff = x
  A()

@main def foo(): Unit =
  val x0 = f(0)
  val x1 = f(1)
  println(x0 == x1)

The output is true, but it should be false, since x0.ff != x1.ff. So definitely a warning is needed here.

My previous comment was that the warning is warranted but the words could be improved.
Good catch on case class, which is unchecked:
          override def equals(x$0: Any): Boolean =
            this.eq(x$0.$asInstanceOf[Object]).||(
              x$0 match
                {
                  case x$0 @ _:A @unchecked => this.x.==(x$0.x).&&(x$0.canEqual(this))
                  case _ => false
                }

I think the warning is in fact correct. Consider the following:
def f: Any = {
  class A {
    override def equals(x: Any): Boolean = x match {
      case _: A => true
      case _    => false
    }
  }
  A()
}

@main def foo(): Unit =
  val x0 = f
  val x1 = f
  println(x0 == x1)

The program output is true. But that is wrong, because each invocation of f creates a new class A (this is particularly obvious if A references f's local variables). So the match of x1 against x0's type (that is, the A from the first f invocation) should fail. But it succeeded, thus the warning is warranted.

Interesting. Should it not be an error then, not just warning? To me, this behavior seems very counter-intuitive.

The use of warning is more or less consistent with those cases when erased types appear in a match. Although local classes are more subtle, they are, in a sense, "half erased".
A more ideal fix would be to implement the match of local classes correctly. This is done for inner classes (as shown by @som-snytt's example). My guess is that in the inner class case, the match not only check the runtime class, but also the captured outer object. For local classes, captured lexical closures (could be empty) could be checked to achieve the correct behavior.


Improve min_version error message

Currently, when a user's version of mise does not satisfy the min_version field within .mise.toml we get the following error:
mise ERROR mise version 2024.12.3 is required, but you are using 2024.12.w
mise ERROR Run with --verbose or MISE_VERBOSE=1 for more information

The experience could be improved by instruction the user on how to update mise:
mise ERROR mise version 2024.12.4 is required, but you are using 2024.12.3
update mise to the latest version by running `mise self-update`

I would accept a PR but we should only show that message if they can actually use self-update. I believe there is a conditional in self_update.rs or maybe version.rs somewhere that does this.

In what cases can they not use self-update? If installing from another package manager?


Mobile Friendly Version of website.

Currently, our portfolio-template only supports the desktop view.
This Issue is for creating a mobile version of site.

Can I work on that?

It would be great. I'll assign you this issue.

hey
can i work your project ? i want to solve your issue

Yes sure..! Just comment on an Issue that you want to work on I'll assign it to you.

Hey, @0xPrateek I used your template and it was good,
Minor improvement: In Readme you should remove the GPL Licence as you have added the MIT one.


Updated Guild Spam Filter

We now support PK broadcasts with a GP treshold and users can now decide which collection log pages they want to remove from broadcasts

Ended up removing the package "Jackson" and just parse the collection log data myself from a giant string. The data is scraped from the wiki and stored within the CollectionLogHandler class

Ended up removing the package "Jackson" and just parse the collection log data myself from a giant string. The data is scraped from the wiki and stored within the CollectionLogHandler class

The preferred way of parsing json is using Gson fyi.

 The data is scraped from the wiki and stored within the CollectionLogHandler class

The preferred way of parsing json is using Gson fyi.

I tried using gson but it kept making my arraylist of data null but Jackson didn’t. But ultimately I ended up just handling it myself :-). Thanks for the heads up though :-D


Instructions to add from Nuget

It is not very clear how to add this module from source or from nuget as someone new to the abp and easyabp platform.  Some pointers would be helpful.
Added all from Nuget, no source:

it is easy to add easyabp.filemanagement using the abphelper or abphelper.gui however it does not add the ui only the module.  I completed this step without issue, created and ran the db migration and was able to see and add permissions to users.
to add a ui, i added easyabp.filemanagement.blazor.server AND added [DependsOn(typeof(FileManagementBlazorServerModule) to the blazor server module.  The "File" menu then showed up for users with permissions but the page is empty - message in the view instead says "Sorry, there's nothing at this address." and the address is "https://localhost:44371/FileManagement/Files/File".  I believe that the blob storage needs to get setup??  However, it is not clear how to do this from the instructions in the github doc.

Any next steps for completing setup using only nuget package would be appreciated.  Once this is working I would like to try from source but the instructions there are not clear for a newbie to abp.io framework.  One thing at a time...

@reidatlokblok Blazor UI is not developed yet or you can provide PR to improve Blazor UI

Thank you for the response.  The MVC version is working.


[Bazel] src/lib/dummy_values & deps, includes gen_values with genrule

generates and compiles a file, using genrule

!ci-build-me


Expose svd_flip for controlling sign indeterminacy

Building on https://github.com/dask/dask/pull/6591#pullrequestreview-481806234, it would be helpful if there was a scalable solution for normalizing eigenvector signs much like sklearn.svd_flip.
There is some discussion at https://github.com/dask/dask/pull/6591#issuecomment-686487526 on where it could live and how it would be applied.
This is particularly useful for library maintainers that want to compare results from different SVD implementations, though it is also necessary for users that wish to make SVD output reproducible.

[ ] Replace the svd_flip function used in the unit tests for https://github.com/dask/dask/pull/6591

Done in https://github.com/dask/dask/pull/6616


fix(Map.jsx): Change the No Super Blocks text to the black slider

Pre-Submission Checklist

[X] Your pull request targets the staging branch of freeCodeCamp.
[X] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[X] You have only one commit (if not, squash them into one commit).
[X] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[X] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[X] Tested changes locally.
[X] Addressed currently open issue (replace XXXXX with an issue no in next line)

Closes #16047 16047
Description

Changed the "No Super Blocks" text when Challenges are loading into the default black slider loader (UX/UI improvements)
BREAKING CHANGE: no

I think we would want a different skeleton here?

@anhtran1906 This issue is a bug not a UI issue. It usually best to to propose the solution and wait for feedback. The codemirror skeleton is specifically for codemirror, hence the name.
Thanks but this is not a solution to the issue.
Happy coding.


New Isolated v2 (I4v2, I5v2, I6v2) Azure App Service Plan Tiers Not Supported in CLI

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Related command

az appservice list-locations --sku I5v2
az appservice plan update --name PLAN_NAME -g PLAN_RG --sku "I4v2"
Describe the bug

I was taking a look at the new Isolated v2 SKUs announced on the appservice blog and tried to use CLI appservice commands like list-locations and update with the new SKUs (I4v2, I5v2, I6v2) but they appear to not work with the CLI even though they're referenced in the blog as being available with the latest CLI 2.43.0 and are referenced throughout the help documentation

To Reproduce

Verify CLI is on version 2.43.0 with az version
Perform az appservice list-locations --sku I5v2
See CLI returns error "Invalid sku(pricing tier), please refer to command help for valid values
" when SKUs I(4-6)v2 are passed in
If testing update/create, provision an ASEv3 in North Central US & provision a plan on it.
Perform an update az appservice plan update --name PLAN_NAME -g PLAN_RG --sku "I4v2"
See CLI returns error "Invalid sku(pricing tier), please refer to command help for valid values"

Expected behavior
Expected that the CLI existing behaviors (list-locations, update) work with the newly added Azure App Service SKUs I4v2, I5v2, I6v2.
Environment summary

CLI Version: 2.43.0
OS Version: Mac OS Monterey 12.6
Shell Type: fish

Additional context
These tiers were released recently and the other older tiers in the Isolatedv2 family appear to work with the CLI.

route to CXP team

@adchungcsc  Thanks for reaching out to us and reporting this issue. We are looking into this issue and we will provide an update.

@adchungcsc I am able to replicate this issue at my end. As a workaround, Could you please test with the below workaround and let us know if that helps?

az rest --url https://management.azure.com/subscriptions/XXX- XXX/providers/Microsoft.Web/geoRegions?api-version=2022-03-01&sku=I5V2

hi @navba-MSFT
The workaround you recommended for directly calling the geo regions API endpoint appears to work.

On a separate note, is this the correct usage of the geo regions API? It looks like you can only pass the app service family name for the sku parameter with a predetermined SkuName item ie "PremumV3" or "IsolatedV2" and you cannot pass in the tier name ie I5v2 based on the docs.
https://learn.microsoft.com/en-us/rest/api/appservice/list-geo-regions/list-geo-regions#skuname
This is also related to that endpoint. The new tiers in Isolated v2 (I4-6 v2) are in the Isolated v2 family but are only in two regions compared to the other members of the family which are available in many more. Would this mean that for example, the API would say that I4v2 is available in a region I2v2 is available in since the API appears to be considering family and not tier?
I also saw that this API call is returning back quite a few regions even though I expect it to only return the two preview regions (North Central US & West Central US). The endpoint seems to also return all regions when passing in invalid skunames like random strings.
az rest --url "https://management.azure.com/subscriptions/XXX-XXX/providers/Microsoft.Web/geoRegions?api-version=2022-03-01&sku=RANDOMSTRING"

Hi,
we are building out special geoRegion API for ASE to allow returning only the regions in which the SKU exists. It will take a few cycles before it is ready to use from the CLI.

@madsd we are running into an issue upgrading to the higher tiers (I4v2, I5v2, I6v2) in South Central US. CLI doesn't work, and suggested workaround in this Issue doesn't work.
az appservice list-locations --sku I5v2
shows
[
  {
    "name": "East US"
  },
  {
    "name": "North Europe"
  },
  {
    "name": "West Europe"
  },
  {
    "name": "Southeast Asia"
  },
  {
    "name": "East Asia"
  },
  {
    "name": "West US"
  },
  {
    "name": "Japan West"
  },
  {
    "name": "Japan East"
  },
  {
    "name": "East US 2"
  },
  {
    "name": "North Central US"
  },
  {
    "name": "South Central US"
  },
  {
    "name": "Brazil South"
  },
  {
    "name": "Australia East"
  },
  {
    "name": "Australia Southeast"
  },
  {
    "name": "Central US"
  },
  {
    "name": "Central India"
  },
  {
    "name": "West India"
  },
  {
    "name": "South India"
  },
  {
    "name": "Canada Central"
  },
  {
    "name": "Canada East"
  },
  {
    "name": "West Central US"
  },
  {
    "name": "West US 2"
  },
  {
    "name": "UK West"
  },
  {
    "name": "UK South"
  },
  {
    "name": "Korea South"
  },
  {
    "name": "Korea Central"
  },
  {
    "name": "France Central"
  },
  {
    "name": "Australia Central"
  },
  {
    "name": "South Africa North"
  },
  {
    "name": "Switzerland North"
  },
  {
    "name": "Germany West Central"
  },
  {
    "name": "UAE North"
  },
  {
    "name": "Norway East"
  },
  {
    "name": "West US 3"
  },
  {
    "name": "Sweden Central"
  },
  {
    "name": "Qatar Central"
  },
  {
    "name": "Poland Central"
  }
]

but we get an invalid SKU error when running the update
does this pertain to the geoRegion API issue you mentioned? And are the listed regions accurate?


Secure connections

So that they are stored encrypted. See https://airflow.apache.org/docs/apache-airflow/1.10.12/howto/secure-connections.html

following the instructions here, which were linked from the docs that @jermnelson linked to, i generated a new fernet key for each airflow env (from my local libsys-airflow dev env), and stored them in vault under the puppet/application/libsys_airflow/ENV/fernet_key paths (where ENV is dev, stage, or prod).
i also submitted a puppet PR to put the fernet key in a host VM env var (merged by rob this morning).
i also put in a PR against this repo (#306) to pass through the env var from the host OS to the docker containers via the docker-compose file (still needs review/merge).
i did all this from reading the docs, but i couldn't find a way to actually test it or confirm that there's a problem.  i didn't see any relevant errors or warnings when i started airflow via docker compose locally, and i didn't see any connection info at all in the airflow console in dev or prod (perhaps i don't have the necessary perms for that).  but from the docs, this seemed like the way to set this up, so hopefully i haven't been barking up the wrong tree.

TODO:

[ ] get the docker compose PR reviewed and merged (#306)
[ ] figure out whether it's necessary to follow the key rotation instructions given in the airflow docs.  it was unclear whether this procedure was needed when going from no fernet key to the first fernet key, or only when going from one non-empty fernet key to a different key.  see here.

It works!

I tried the key rotation directions locally going from unencrypted to encrypted -- it didn't work. If we feel the need to encrypt keys, the existing connections can be deleted and re-created.


feat: "npm On-Site authentication" → "npm Enterprise login"

Rebranding for fun and profit.
Fixes #31.
Does not change the package or repo name.

my brand! 🚢


Added my playlist Chill Mood

Added my playlist Chill Mood

Can you please add hacktober or hacktober-accepted tag in this repository


[SPARK-45193][PS][CONNECT][TESTS] Refactor test_mode to be compatible with Spark Connect

What changes were proposed in this pull request?
Refactor test_mode to be compatible with Spark Connect
Why are the changes needed?
for test parity
Does this PR introduce any user-facing change?
No, test-only
How was this patch tested?
CI, and manually check:
In [5]: import pandas as pd

In [6]:         def func(iterator):
   ...:             for pdf in iterator:
   ...:                 if len(pdf) > 0:
   ...:                     if pdf["partition"][0] == 3:
   ...:                         yield pd.DataFrame({"num" : ["0", "1", "2", "3", "4"] })
   ...:                     else:
   ...:                         yield pd.DataFrame({"num" : ["3", "3", "3", "3", "4"]} )
   ...:

In [7]: df = spark.range(0, 4, 1, 4).select(sf.spark_partition_id().alias("partition"))

In [8]: df.mapInPandas(func, "num string").withColumn("p", sf.spark_partition_id()).show()
/Users/ruifeng.zheng/Dev/spark/python/lib/pyspark.zip/pyspark/sql/pandas/serializers.py:239: FutureWarning: is_categorical_dtype is deprecated and will be removed in a future version. Use isinstance(dtype, CategoricalDtype) instead
/Users/ruifeng.zheng/Dev/spark/python/lib/pyspark.zip/pyspark/sql/pandas/serializers.py:239: FutureWarning: is_categorical_dtype is deprecated and will be removed in a future version. Use isinstance(dtype, CategoricalDtype) instead
/Users/ruifeng.zheng/Dev/spark/python/lib/pyspark.zip/pyspark/sql/pandas/serializers.py:239: FutureWarning: is_categorical_dtype is deprecated and will be removed in a future version. Use isinstance(dtype, CategoricalDtype) instead
/Users/ruifeng.zheng/Dev/spark/python/lib/pyspark.zip/pyspark/sql/pandas/serializers.py:239: FutureWarning: is_categorical_dtype is deprecated and will be removed in a future version. Use isinstance(dtype, CategoricalDtype) instead
+---+---+
|num|  p|
+---+---+
|  3|  0|
|  3|  0|
|  3|  0|
|  3|  0|
|  4|  0|
|  3|  1|
|  3|  1|
|  3|  1|
|  3|  1|
|  4|  1|
|  3|  2|
|  3|  2|
|  3|  2|
|  3|  2|
|  4|  2|
|  0|  3|
|  1|  3|
|  2|  3|
|  3|  3|
|  4|  3|
+---+---+

Was this patch authored or co-authored using generative AI tooling?
No

cc @itholic @HyukjinKwon

thanks, merged to master


docs/translations: add the translation of linux/external_fuzzing_network.md

Finish the translation of linux/external_fuzzing_network.md.

This PR has the same issue with #4650

@a-nogikh It's working by removing the last line of the document


[animation] Feature Request: Allow dir to alternate when looping is not enabled

It would be really cool if dir: alternate caused the animation to alternate on each startEvent when looping isn't enabled.
An example of this could be something like a lever that would toggle orientation between two positions when clicked, while only requiring the use of one animation  component.
I know there are more involved ways of achieving this, but this has the potential to make it much simpler.
Just a thought. Let me know what you think.
Thanks.

At minimum, the documentation should explicitly explain that dir: alternate only works when a loop is enabled.
Ideally this issue should be considered a bug, because <a-animation> respects direction="alternate" even when there isn't a >0 value for repeat.

I fixed for myself unwanted loops even the direction is nearly the same as the origin.
The glitch is, the angel values can be greater than 360 (and also negative) this leads sometimes to extra over 180° rotations:
		var rot = camNode.object3D.rotation,
			rad = 2 * Math.PI;
		console.log("before", rot.x, rot.y, THREE.Math.radToDeg(rot.x),  THREE.Math.radToDeg(rot.y));
		// Notmalize radiant values, because we don't want rotate over 180°
		if (Math.abs(rot.x) > rad) rot.x %= rad;
		if (Math.abs(rot.y) > rad) rot.y %= rad;
		// Make positive
		if (rot.x < 0 && tarX) rot.x += rad;
		if (rot.y < 0 && tarY) rot.y += rad;
		// Make negative
		if (rot.x > Math.PI && !tarX) rot.x -= rad;
		if (rot.y > Math.PI && !tarY) rot.y -= rad;
		console.log("to", tarX, tarY, "from", THREE.Math.radToDeg(rot.x) , THREE.Math.radToDeg(rot.y));


[BUG] With version 2.3.0 of the provider, the ressource opensearch_channel_configuration cannot be applied when assuming an iam role in the provider

What is the bug?
With version 2.3.0 of the provider,  the ressource opensearch_channel_configuration cannot be applied when assuming an iam role in the provider like so :

Do you have any screenshots?

provider.terraform-provider-opensearch_v2.3.0: Response contains error diagnostic: diagnostic_severity=ERROR tf_proto_version=5.4 tf_provider_addr=provider tf_resource_type=opensearch_channel_configuration @module=sdk.proto diagnostic_detail= tf_rpc=ApplyResourceChange @caller=github.com/hashicorp/terraform-plugin-go@v0.22.2/tfprotov5/internal/diag/diagnostics.go:58 diagnostic_summary="HTTP 403 Forbidden: Permission denied. Please ensure that the correct credentials are being used to access the cluster." tf_req_id=4c4f9019-ad7d-e488-f5e9-008d96464823 timestamp=2024-06-20T17:46:03.247Z
It works perfectly when I force version 2.2.0 of the provider.
Other ressources that also uses this provider configs with version 2.3.0 doesnt seems to be affected and works perfectly like :

The following terraform code works for me
terraform {
  required_providers {
    opensearch = {
      source = "opensearch-project/opensearch"
      version = "2.3.0"
    }
  }
}

provider "opensearch" {
  url = "url"
  healthcheck        = "false"
  aws_region          = "region"
  aws_assume_role_arn = "assume role arn"
  aws_assume_role_external_id = "external id"
  version_ping_timeout = "10"

}

resource "opensearch_index" "index" {
  name = "sample"
  number_of_replicas = "1"
  number_of_shards = "1"

}

resource "opensearch_channel_configuration" "webhook_channel_configuration" {
  body = <<EOF
{
  "id": "sample-webhook-id",
  "name": "sample-name",
  "config": {
    "name": "Sample Webhook Channel",
    "description": "Sample webhook description",
    "config_type": "webhook",
    "is_enabled": true,
    "webhook": {
      "url": "https://www.example.com"
    }
  }
}
EOF
}

Can you try using an AWS SNS channel instead of a webhook?

The following works for me
terraform {
  required_providers {
    opensearch = {
      source = "opensearch-project/opensearch"
      version = "2.3.0"
    }
  }
}

provider "opensearch" {
  url = "url"
  healthcheck        = "false"
  aws_region          = "region"
  aws_assume_role_arn = "role arn"
  aws_assume_role_external_id = "opensearch-external"
  version_ping_timeout = "10"

}

resource "opensearch_index" "index" {
  name = "sample"
  number_of_replicas = "1"
  number_of_shards = "1"

}

resource "opensearch_channel_configuration" "sns_channel_configuration" {
  body = <<EOF
{
  "id": "sample-sns-id",
  "name": "sample-name",
  "config": {
      "name": "sns-channel",
      "config_type": "sns",
        "sns": {
          "topic_arn": "topic arn",
          "role_arn": "role arn"
        }

      }
}
EOF
}

Thanks @rblcoder, closing this issue @Jaewongtongsoup please feel-free to re-open if required.


Add last_compaction_state to sys.segments table

Add last_compaction_state to sys.segments table
Description
Segments being compacted or not compacted have a big impact on query performance.
Currently it is not easy to verify questions like:

if performance of query can be improve by compaction
if a performance of query is bad, was it because segments for that interval was not compacted.
if a segment was compacted, what was it compacted with? was it compacted with the auto compaction config set?

This PR expose the lastCompactionState via the sys.segments table (so user can do query with slice/dice, group, time filter, etc) by using the lastCompactionState stored within the segment metadata. If a compactionState exist then the segment was compacted. This PR also change storing lastCompactionState as default for manual compaction in addition to auto compaction.
For example, if you want to know if the segment was compacted or not (ANY compaction), you can do WHERE last_compaction_state is not null and if you want to know if the segment was compacted with current auto compaction configs, you can do WHERE last_compaction_state == '{current_auto_compact_config}'
This PR has:

[x] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[x] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[x] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[x] been tested in a test Druid cluster.

Added Release Notes because it changed the default behavior of Tasks.DEFAULT_STORE_COMPACTION_STATE

I'll make the change with re-write to docs in a separate PR.


Input mess up with spaces

Press any key are prefixed with a "SPC" in either edit mode or evil mode. This issue happens randomly and I don't have any idea. First it happens in the input mode, when I type any a sentense like "hello", it becomes: " h e l l o". This problems still exists if I press esc and pressing "i" becomes "SPC i", pressing ctrl+g doesn't help. I have to restart emacs to fix it.
** my config**
It's a default config with some packages enabled "rust" and "js" related.
System information:

```
SYSTEM  type       darwin
        config     x86_64-apple-darwin20.1.0
        shell      /usr/local/bin/zsh
        uname      Darwin 20.2.0 Darwin Kernel Version 20.2.0: Wed Dec  2 20:39:59 PST 2020; root:xnu-7195.60.75~1/RELEASE_X86_64 x86_64
        path       (~/.cabal/bin ~/.ghcup/bin /usr/local/bin ~/.nvm/versions/node/v14.15.1/bin /usr/local/bin /usr/bin /bin /usr/sbin /sbin /Library/TeX/texbin /usr/local/share/dotnet ~/.dotnet/tools /Library/Frameworks/Mono.framework/Versions/Current/Commands ~/.cabal/bin ~/.ghcup/bin ~/.nvm/versions/node/v14.15.1/bin ~/.cargo/bin ~/.local/bin ~/development/flutter/bin ~/development/flutter/bin ~/.cargo/bin<EMAIL_ADDRESS>EMACS   dir        ~/.emacs.d/
        version    28.0.50
        build      Dec 04, 2020
        buildopts  --disable-dependency-tracking --disable-silent-rules --enable-locallisppath=/usr/local/share/emacs/site-lisp<EMAIL_ADDRESS><EMAIL_ADDRESS>--with-xml2 --with-gnutls --without-dbus --with-imagemagick --with-modules --with-rsvg --with-ns --disable-ns-self-contained
        features   JPEG TIFF PNG RSVG IMAGEMAGICK GLIB NOTIFY KQUEUE ACL GNUTLS LIBXML2 ZLIB TOOLKIT_SCROLL_BARS NS MODULES THREADS JSON PDUMPER LCMS2
        traits     (batch server-running envvar-file)
DOOM    dir        ~/.doom.d/
        version    2.0.9
        build      HEAD -> develop b03fdabe4 2021-01-13 14:07:47 -0500
        elc-files  0
        modules    (:completion company ivy :ui doom doom-dashboard doom-quit hl-todo modeline ophints (popup +defaults) treemacs vc-gutter vi-tilde-fringe workspaces :editor (evil +everywhere) file-templates fold snippets :emacs dired electric undo vc :checkers syntax :tools editorconfig (eval +overlay) lookup lsp magit :os macos :lang emacs-lisp fsharp json javascript markdown org python rust sh solidity web yaml :config (default +bindings +smartparens))
        packages   (n/a)
        unpin      (n/a)
        elpa       (n/a)
```

I'm unable to reproduce this on any test machine (NixOS, Arch, Ubuntu, or MacOS) or any version of Emacs (26.3, 27.1 or 28). I can only imagine this issue is local to your system or hardware, but have no further leads. In any case, this doesn't seem like a Doom issue.

I'm unable to reproduce this on any test machine (NixOS, Arch, Ubuntu, or MacOS) or any version of Emacs (26.3, 27.1 or 28). I can only imagine this issue is local to your system or hardware, but have no further leads. In any case, this doesn't seem like a Doom issue.

I'm not sure what's the problem because the same issue happens to spacemacs... The keyboard works normally in other windows in the same time. So I guess it might not be a hardware issue.  Let me close it since it's not doom-emacs specific.

I'm not sure what's the problem because the same issue happens to spacemacs... The keyboard works normally in other windows in the same time. So I guess it might not be a hardware issue.  Let me close it since it's not doom-emacs specific.


Getting 403 even if permissions:write is set

Hi,
In an open-source project we are using this action as follows: https://github.com/apache/santuario-xml-security-java/blob/main/.github/workflows/dependency-submission.yml
When I (who have write access) submit a PR everything works fine. However as you can see in this PR https://github.com/apache/santuario-xml-security-java/pull/240 the action fails with:
Error: HTTP Status 403 for request POST https://api.github.com/repos/apache/santuario-xml-security-java/dependency-graph/snapshots
Error: Response body:
{
  "message": "Resource not accessible by integration",
  "documentation_url": "https://docs.github.com/rest/dependency-graph/dependency-submission#create-a-snapshot-of-dependencies-for-a-repository"
}

contents: write is set as a permission for the job. What am I doing wrong here please?

The failing linked PR check workflow, shows permissions of read and not the required write here; https://github.com/apache/santuario-xml-security-java/actions/runs/6738707294/workflow

Looking a little further around your runs for that workflow, I think the issue lies in the permissions you are allowing for Pull Requests from forks of your repository. GitHub Actions treats these as risky and by default you have to opt into allowing permissions higher that write.

For additional context and help on deciding what is right for your use case, this blog post should be able to help you; https://github.blog/2020-08-03-github-actions-improvements-for-fork-and-pull-request-workflows/

I ran into the same issue and this guidance resolved it for me.  Thanks @peter-murray!


Ignore unsafe coercions on ghc-9+

Since ghc-9 unsafeCoerce x compiles to case unsafeEqualityProof of UnsafeRefl co -> x `cast` co instead of x `cast` co older versions produced. Such case would be turned into x by CoreToStg, so I suppose that ==- should treat those expressions as equal.
This PR changes eqSlice to treat case unsafeEqualityProof of ... -> e as e when 'ignore types' option is on.

Looks good to me. Is the existing test suite sufficient? Can you try adding GHC 9 to the CI configuration?

Done.

Thanks, great!


Seg fault

./cppip -D -f -e timestamp:2015-10-21:08:33:02-2015-10-21:08:33:20<PHONE_NUMBER>.cppip<PHONE_NUMBER>.pcap.gz carved-1445412783.pcap
extracting from<PHONE_NUMBER>.pcap.gz using<PHONE_NUMBER>.cppip...
DBG: first pkt ts:<PHONE_NUMBER>) 2015-10-21 07:33:02.690245
DBG: start pkt ts:<PHONE_NUMBER>) 2015-10-21 07:33:02.000000
DBG: stop pkt ts<PHONE_NUMBER>) 2015-10-21 07:33:20.000000
DBG: index level:		1 0
DBG: entered at pkt ts:		2015-10-21 07:33:02.000000
DBG: fuzzy match at iteration:	1
start ts: 2015-10-21 07:33:02.000000 not found, instead fuzzy matched on 2015-10-21 07:33:02.690245
Segmentation fault
(gdb) bt
#0  0x00007ffff7649d6b in __memcpy_ssse3_back () from /lib64/libc.so.6
#1  0x000000000040335f in extract_by_ts (c=0x60b010) at extract.c:367
./cppip -v<PHONE_NUMBER>.cppip
valid cppip index file
version:	1.4
created:	2015-11-02 18:53:49.616815
packets in pcap:1633283
indexing mode:	timestamp
index level:	0:0:0:1
record count:	60
I could very likely be doing something incorrectly, let me know if more info is needed.
Also the timestamps I have to input have to be off by and hour or I get an error that the end TS if before the start of the file.

Hi jwinters7, thanks for the bug report. Does cppip work in "packet number" mode? Have you gotten "timestamp" mode to work with different timestamp arguments? What system are you building / running cppip on? Also, any chance I can get to work the pcap in question?

No doesn't work in packet number mode either, it writes a little bit of the new.pcap in either mode though.  I have tried with different timestamps, and even different pcaps, I starting to think its a bit of a bigger problem. I've even tried it on different machines, I'm running on CentOS Linux release 7.1.1503. With 3.10.0-123.8.1.el7.x86_64 kernel.
Error in messages when trying to use timestamp method:
kernel: cppip[17893]: segfault at 7fffc9f0e000 ip 00007fe754bfc5af sp 00007fffc9f00e28 error 6 in libc-2.17.so[7fe754ab3000+1b6000]
Error in messages when trying to use packet number method:
kernel: traps: cppip-exec[18843] general protection ip:402d92 sp:7fffb921d288 error:0 in cppip-exec[400000+a000]
The pcaps I'm using are about 600MB compressed, if you think its the pcap I will see if I can create some random non live capture data and see how it handles that.
Thanks for your response and help with this.

I can't recreate the issue on Debian (wheezy) or OS X 10.11.1. If you can get me the offending pcap (dropbox?) that would help a ton. If that's too unwieldy, perhaps you can recreate with a smaller pcap?

Okay, been playing around some more. It does seem to be an issue with certain pcap files, I have one file that it seg faults on when the time window is around 40 seconds. However it works on that same file with a 5 second window.
I know you need the file to be able to really help more, I'm trying to capture something that I can hand out. However I'm currently having issues reproducing the issue outside of our live traffic.
I wanted to keep you updated though, I will post a pcap as soon as I find one I can release.

Happy to help but I need more hard data. :)

Was able to the crash (big packet, small buffer) -- please pull down the new tree and verify this fixes your issue and I'll close the issue. Thanks!

Nice work! Impressed since I gave you almost nothing to work with. Love the commit message too, thanks again.


Incorrect ConnectionString format

In the document section titled Execute, you mention that ConnectionString must be "Your .NET SDK endpoint, which you'll find in the Overview section of your Azure Cosmos DB for Gremlin database account. It's formatted as https://your-graph-database-account.documents.azure.com:443/".
This is incorrect!
The ConnectionString must look like this - AccountEndpoint=https://your-graph-database-account.documents.azure.com:443/;AccountKey=YourKey.
It took me some time to figure out that ConnectionString was at fault. The error that comes from the bulk import operation was equally cryptic Format of the initialization string does not conform to specification starting at index 0..

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

ID: aa51c0df-1a47-6121-df68-c787a97d717f
Version Independent ID: 1ab4ade5-4745-373f-da4d-765904b39edf
Content: Ingest data in bulk in Azure Cosmos DB for Gremlin by using a bulk executor library
Content Source: articles/cosmos-db/gremlin/bulk-executor-dotnet.md
Service: cosmos-db
Sub-service: apache-gremlin
GitHub Login: @manishmsfte
Microsoft Alias: mansha

@ksivasenthil Thanks for your findings,We will look into it and update as appropriate.

@ksivasenthil, you are absolutely correct. We have a change going live later today.


[mvfst ] : Add new port

Describe the pull request
mvfst (Pronounced move fast) is a client and server implementation of IETF QUIC protocol in C++ by Facebook.

What does your PR fix?
Fixes #...

Which triplets are supported/not supported? Have you updated the CI baseline?
,

Does your PR follow the maintainer guide?

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

@jiayuehua ,Thanks for your pr, After finished this review suggestion, please run command "./vcpkg x-add-version mvfst --overwrite-version" and commit again

@jiayuehua ,Thanks for your pr, After finished this review suggestion, please run command "./vcpkg x-add-version mvfst --overwrite-version" and commit again

ok, I will fix it as soon as possible

Looks good, thanks for the new port!


CLOUD-964 removed superfluous annotation, using javax.inject annotati…

…on instead of spring specific counterpart
@martonsereg pls check it

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1551/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/1554/

LGTM


Parallel kernel compilation

Refactors BEAM's parallel compilation to enable generic parallel compilation. Next step is to actually do it

I added parallel kernel compilation to realize.
WINO=1 python3 examples/hlb_cifar10.py performance on M1 Pro:
Before:
shuffling training dataset in 1011.06 ms (0)
  0 38269.06 ms run, 38268.71 ms python,    0.34 ms CL, 1199.83 loss, 0.000015 LR, 0.84 GB used,     12.94 GFLOPS,    495.18 GOPS
  1 29546.97 ms run, 29546.03 ms python,    0.94 ms CL, 1199.65 loss, 0.000030 LR, 4.34 GB used,     16.76 GFLOPS,    495.08 GOPS
  2  492.04 ms run,    7.23 ms python,  484.81 ms CL, 1191.60 loss, 0.000045 LR, 4.35 GB used,   1006.18 GFLOPS,    495.08 GOPS
  3  418.31 ms run,    8.61 ms python,  409.70 ms CL, 1173.59 loss, 0.000060 LR, 4.36 GB used,   1183.50 GFLOPS,    495.08 GOPS
  4  415.54 ms run,    8.24 ms python,  407.30 ms CL, 1160.32 loss, 0.000075 LR, 4.36 GB used,   1191.40 GFLOPS,    495.08 GOPS
...

After (with PARALLEL_COMPILATION=1):
shuffling training dataset in 879.53 ms (0)
  0 13002.57 ms run, 13002.24 ms python,    0.33 ms CL, 1199.83 loss, 0.000015 LR, 0.84 GB used,     38.11 GFLOPS,    495.58 GOPS
  1 11115.04 ms run, 11114.08 ms python,    0.96 ms CL, 1199.65 loss, 0.000030 LR, 4.36 GB used,     44.53 GFLOPS,    494.96 GOPS
  2  430.68 ms run,    5.67 ms python,  425.02 ms CL, 1191.60 loss, 0.000045 LR, 4.37 GB used,   1149.24 GFLOPS,    494.96 GOPS
  3  414.49 ms run,    5.17 ms python,  409.32 ms CL, 1173.59 loss, 0.000060 LR, 4.38 GB used,   1194.14 GFLOPS,    494.96 GOPS
  4  412.59 ms run,    5.96 ms python,  406.63 ms CL, 1160.32 loss, 0.000075 LR, 4.38 GB used,   1199.64 GFLOPS,    494.96 GOPS
...

Seems very complex looking and is +38 lines. What's the minimal way to do this? Is all this complexity needed?

Added a new section to contributing:
"If your PR looks "complex", is a big diff, or adds lots of lines, it won't be reviewed or merged. Consider breaking it up into smaller PRs that are individually clear wins. A common pattern I see is prerequisite refactors before adding new functionality. If you can (cleanly) refactor to the point that the feature is a 3 line change, this is great, and something easy for us to review."

Seems like the bounty got closed 2 days ago and I'll open it up again.
Gonna focus just on the factoring and not speeding the thing up or anything like that.
If the previous author shows up again then I'm down to discuss what all they were trying to do and we can collaborate.


yaml syntax broken

Hello
I have an issue now that is
https://gist.github.com/anonymous/904c4628fc3f9d23870a915cf0111610
because we have some files like
https://zerobin.net/?e3390f9a981a3ccd#mAvv8zX8ItzuQPs6qxIqXVPDc03d4yspKeQ1mQTeYms=
and our flasgger version is
flasgger==0.8.0
previously
flasgger==0.6.3
It was working fine until I upgraded flasgger version. (maybe I done something else, but I don't mind what)
Any idea on if I should do something or if flasgger should change something ?

Edit : I just tried to run swagger with flasgger 0.8.0 instead of 0.8.1 and it works (note that in my initial issue there is a mistake : I meant we upgraded to 0.8.1 where the breaking change seems to appear.

---
Get a single conversation from id.

The state field in conversation payload may take one of the following status

  - **normalState** : Default state for conversation

  - **waitForFirstMessage** : Message count equals 0

  - **waitingForPermission** : Message count equals 1 and current user created conversation

  - **receivedInvitation** : Message count equals 1 and current user DID NOT created conversation

  - **iAmBlocking** : Message count superior to 1 and current user blocked conversation

  - **iAmBlocked** : Message count superior to 1 and conversation other user blocked conversation

  - **weAreBothBlocked** : Message count superior to 1 and both users blocked conversation
---
tags:
  - Chat / Conversations
security:
  - APIKeyQueryParam: []
  - APIKeyHeader: []
parameters:
  - name: conversation_id
    in:  path
    required: true
    schema:
        type: integer
    description: The convesation to fetch.
responses:
  404:
    description: Conversation not found in database.
    schema:
      $ref: '#/definitions/NotFound'
  200:
    description: Return convesation dumped object.
    schema:
      $ref: '#/definitions/Conversation'
  401:
    description: Not authorized, user must be logged to perform query
    schema:
      $ref: '#/definitions/Unauthorized'

Traceback (most recent call last):
  File "/home/utopman/.local/share/virtualenvs/jaccede-api-v4-w1kPGGna/lib/python3.5/site-packages/flask/app.py", line 1997, in __call__
    return self.wsgi_app(environ, start_response)
  File "/home/utopman/.local/share/virtualenvs/jaccede-api-v4-w1kPGGna/lib/python3.5/site-packages/werkzeug/contrib/fixers.py", line 152, in __call__
    return self.app(environ, start_response)
  File "/home/utopman/.local/share/virtualenvs/jaccede-api-v4-w1kPGGna/lib/python3.5/site-packages/flask/app.py", line 1985, in wsgi_app
    response = self.handle_exception(e)
  File "/home/utopman/.local/share/virtualenvs/jaccede-api-v4-w1kPGGna/lib/python3.5/site-packages/flask/app.py", line 1540, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "/home/utopman/.local/share/virtualenvs/jaccede-api-v4-w1kPGGna/lib/python3.5/site-packages/flask/_compat.py", line 33, in reraise
    raise value
  File "/home/utopman/.local/share/virtualenvs/jaccede-api-v4-w1kPGGna/lib/python3.5/site-packages/flask/app.py", line 1982, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/utopman/.local/share/virtualenvs/jaccede-api-v4-w1kPGGna/lib/python3.5/site-packages/flask/app.py", line 1614, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/home/utopman/.local/share/virtualenvs/jaccede-api-v4-w1kPGGna/lib/python3.5/site-packages/flask/app.py", line 1517, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/home/utopman/.local/share/virtualenvs/jaccede-api-v4-w1kPGGna/lib/python3.5/site-packages/flask/_compat.py", line 33, in reraise
    raise value
  File "/home/utopman/.local/share/virtualenvs/jaccede-api-v4-w1kPGGna/lib/python3.5/site-packages/flask/app.py", line 1612, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/utopman/.local/share/virtualenvs/jaccede-api-v4-w1kPGGna/lib/python3.5/site-packages/flask_debugtoolbar/__init__.py", line 125, in dispatch_request
    return view_func(**req.view_args)
  File "/usr/lib/python3.5/cProfile.py", line 109, in runcall
    return func(*args, **kw)
  File "/home/utopman/.local/share/virtualenvs/jaccede-api-v4-w1kPGGna/lib/python3.5/site-packages/flask/views.py", line 84, in view
    return self.dispatch_request(*args, **kwargs)
  File "/home/utopman/.local/share/virtualenvs/jaccede-api-v4-w1kPGGna/lib/python3.5/site-packages/flask/views.py", line 149, in dispatch_request
    return meth(*args, **kwargs)
  File "/home/utopman/.local/share/virtualenvs/jaccede-api-v4-w1kPGGna/lib/python3.5/site-packages/flasgger/base.py", line 189, in get
    optional_fields, self.process_doc)
  File "/home/utopman/.local/share/virtualenvs/jaccede-api-v4-w1kPGGna/lib/python3.5/site-packages/flasgger/utils.py", line 131, in get_specs
    method, sanitizer, endpoint=rule.endpoint, verb=verb)
  File "/home/utopman/.local/share/virtualenvs/jaccede-api-v4-w1kPGGna/lib/python3.5/site-packages/flasgger/utils.py", line 527, in parse_docstring
    swag = yaml.load(full_doc[yaml_sep + 4:])
  File "/home/utopman/.local/share/virtualenvs/jaccede-api-v4-w1kPGGna/lib/python3.5/site-packages/yaml/__init__.py", line 72, in load
    return loader.get_single_data()
  File "/home/utopman/.local/share/virtualenvs/jaccede-api-v4-w1kPGGna/lib/python3.5/site-packages/yaml/constructor.py", line 35, in get_single_data
    node = self.get_single_node()
  File "/home/utopman/.local/share/virtualenvs/jaccede-api-v4-w1kPGGna/lib/python3.5/site-packages/yaml/composer.py", line 43, in get_single_node
    event.start_mark)
yaml.composer.ComposerError: expected a single document in the stream
  in "<unicode string>", line 1, column 1:
    Perform search on places in Jacc ...
    ^
but found another document
  in "<unicode string>", line 4, column 1:
    ---

@eregnier I'll investigate the issue

I missed to share an information : When I tried to investigate a bit on this issue I found this thread which is related in a sense, but with no more troubleshooting / experiments I am not really sure. Here is the thread I am speaking about
https://stackoverflow.com/questions/14359557/reading-yaml-in-python

My understanding is that Flasgger is tolerant of non-YAML content in a pre-document directives block, but that it still expects a single document, and that is inline with this error.  The exception could probably be caught and some additional help could be provided.
So Flasgger will tolerate:
Some documentation for the API.
It can even contain bare colons : like that.
Here we are essentially in the YAML directives block of document 1.
Next follows three dashes then the spec YAML.
---
# this is the actual Swagger spec
tags:
  - Chat / Conversations

The problem document here starts with ---, which means "start of YAML document with no directives block".  Then follows text-not-YAML.  Following that is a valid, second YAML document.  Without the first --- it should work fine.
TL;DR Flasgger needs a single YAML document, above two are present.
Is there a reason for the first --- - could it be removed?

We fixed the file on the project. I do not have this problem anymore. The problem may persist for some other people that have formatted their documentation like me, but this thread does not move a lot. Then I suppose there is not much problem remaining at the moment.
You may close the issue.
Thank you.