Add an API to get a terminal's selection

This is would be useful information for Copilot to know about.

I'll wait for this to go through the API sync before doing a test plan item.

Proposal:
https://github.com/microsoft/vscode/blob/ea490e5545df658e05d0332bfb2a04435a0ef7f0/src/vscode-dts/vscode.proposed.terminalSelection.d.ts#L6-L17
If we want a change event:
export namespace window {
	export const onDidChangeTerminalSelection: Event<TerminalSelectionChangeEvent>;
}

export interface TerminalSelectionChangeEvent {
	readonly terminal: Terminal;
	readonly selection: string | undefined;
	// No kind like TextEditorSelectionChangeKind
}

Consider to have Terminal#selection which allows to retrieve the selection outside of an event
Also wondering about the missing "coordinates" (range, offset, etc) that would allow to locate and set selections

@jrieken the part of the proposal already in main is Terminal#selection. For coordinates, we don't have these anywhere in the terminal and you wouldn't be able to use them if we did.

For coordinates, we don't have these anywhere in the terminal and you wouldn't be able to use them if we did.

Does that mean one isn't ever able to set a selection? How does this work internally. Does xterm not have this or do you simply not want to expose terminal rows/columns?

You can select in xterm.js, here are the apis:
select(column: number, row: number, length: number): void;
selectAll(): void;
selectLines(start: number, end: number): void;

When considering whether we want to expose such an API it's useful to understand how the buffer works. It's essentially a big grid of characters of size <cols> x <rows> + <scrollback>, since there grid has fixed dimensions it comes with several complications/differences with the editor:

Wrapped lines are indicated by boolean flags on a line indicating whether the previous line wraps into it.
Resizing the terminal to make it more wide or narrow will "reflow" the terminal, which attempts to get natural wrapping behavior by pushing cells to different lines, potentially deleting or creating lines in the process.
The buffer has a fixed height which means when the buffer is full, each new line will trim the top line off. Internally this uses a circular buffer where the top row's index is 0, and the bottom row's index is <rows> + <scrollback> - 1.
The suggestion of a more stable line indexing system has come up before, but it gets kind of complicated. We could keep a running tally the amount of trimmed lines and just add that value to the current buffer lines, but that means if rows get added or removed we would need to change the index due to inserted or deleted lines, all lines below the changed ones.
Exposing terminal coordinates in the API wouldn't be useful at all without the ability to query the value in them. Following existing APIs and the guidelines would mean these should be synchronous, this works in the editor because the buffers are synchronized across processes. This would be a bad idea in the terminal imo, consider what happens when running ls -lR /; a tonne of data will come in and immediately be discarded as it's trimmed off the top of the buffer. Trying to keep these in sync on the extension host would be a waste of CPU time, memory and like lead to misbehaving extensions as they don't understand the intricacies of how the buffer works.

To bring this all together, we could of course expose xterm.js-like APIs in the exthost with the ability to pull and select arbitrary cell data, but it would have to be asynchronous and it would differ in subtle ways from the editor's APIs. Ultimately, I think this would just add APIs that don't add that much value but are easy to misuse, and the amount of value they add wouldn't be worth the effort to implement it anyway.

Feedback:

Should it be named selectedText instead of selection in case we expose terminal buffer coordinates in the future?

Should it be named selectedText instead of selection in case we expose terminal buffer coordinates in the future?

I'd say that for the scope a proposed API this doesn't really matter. But for finalisation I wouldn't consider either, finalising an API so that it can be made obsolete in the future without hassle isn't a goal.

Ultimately, I think this would just add APIs that don't add that much value but are easy to misuse, and the amount of value they add wouldn't be worth the effort to implement it anyway.

I don't know about this but my worry is that we do add more and more concepts to the terminal API, like selection or shell but that we don't think out load about the individual parts. We need to make sure that we don't miss the point at which we separate concerns and introduce types for buffer, buffer-coords, shell and more. I am not saying all needs to happen at once but for new API we must kept this in our mind and make sure we don't block any of this happening.

This would be a bad idea in the terminal imo, consider what happens when running ls -lR /; a tonne of data will come in and immediately be discarded as it's trimmed off the top of the buffer.

Agreeing and you can simulate that with cut-paste cycles of very many lines in a document. Tho, terminal-sync can be debounced to reduce waste.
Again, not saying all of this must happen before proposing a simple selection but my worry is that we are getting closer to this inflection point at which we should have better decoupled bits.

I'll keep this one on hold for now, at least until the idea of a Terminal.shell API has been explored in https://github.com/microsoft/vscode/issues/145234

Hey! Is there a workaround for this? Maybe executing the "copy" command and then reading clipboard?

I would like to get the selected Text from Active Terminal in vscode extension
Please let me know the options that we have for the same currently to build extensions

@poornimas-iyer this isn't possible until there's more movement here and/or in https://github.com/microsoft/vscode/issues/207504

I am hoping the same so that we can get selected text from active terminal. Looks like Copilot has already able to access this. Hoping there is something API can be exposed.

@yuqizhou77 yeah when this went in for finalization it was put on hold along with some similar APIs as what we're ultimately moving towards is full buffer access in https://github.com/microsoft/vscode/issues/207504, and we don't want to finalize an API that we don't want to keep as we generally don't break APIs.


feat: define processing time column

Is your feature request related to a problem? Please describe.
Provide a column with event ingesting time, in case there is no appropriate time column in user data.
Describe the solution you'd like
Based on #6952, we can introduce proc_time as a column in the source, which is same with Flink: https://nightlies.apache.org/flink/flink-docs-release-1.16/docs/dev/table/concepts/time_attributes/#processing-time
CREATE SOURCE user_actions (
  user_name STRING,
  data STRING,
  user_action_time AS PROCTIME() -- declare an additional field as a processing time attribute
) WITH (
  ...
);

SELECT TUMBLE_START(user_action_time, INTERVAL '10' MINUTE), COUNT(DISTINCT user_name)
FROM user_actions
GROUP BY TUMBLE(user_action_time, INTERVAL '10' MINUTE);

Describe alternatives you've considered
1. Hidden column.
The problem of this approach is that it doesn't explicitly tell the users the column is associated with source/table. For example:
This is clear:
select _proc_time, id, value from t where k = 2;

This is ambiguous:
select _proc_time, id, value from a join b where a.bid = b.id where k = 1
-- ERROR: _proc_time should be either a._proc_time or b._proc_time

2. System function
This is actually a different thing. It is apparent that functions are evaluated when executing it, rather than data injected.
It doesn't matter for this simple query
select _proc_time, id, value from t where k = 2;

But will be a problem for more complex ones:
select proc_time(), id, value from a join b where a.bid = b.id where k = 1
-- When is proc_time() evaluated? Intuitively, it should be after the two events joined.

We had better call it now() or something else to distinguish from our topic here.
Additional context
No response

prefer the Hidden column, it should be a record's properties but not a impure function

prefer the Hidden column, it should be a record's properties but not a impure function

My concern is that the word "hidden column" is unheard-of for users, especially for Postgres users. For example, he/she may feel confused when seeing a.proc_time but there is no proc_time column in describe table or show create table.

Ok, just allow PROCTIME() when the user defines the Source or Table LGTM.
Another question, should we support more kinds of DDL on the source? e.g.

add the PROCTIME column on the source
even more... change the watermark column of a source?

How we handle this in batch query? The time when it's scaned into our system?

I am wondering if it also makes sense to add a time column when being materialized instead of being just read from the source? I suppose proc_time() is added to the row when it is just read.

I am wondering if it also makes sense to add a time column when being materialized instead of being just read from the source? I suppose proc_time() is added to the row when it is just read.

For a table (previously materialized source) - Yes, the processing time should be persisted in that table/MV as well, just like any other column.
For a source (i.e. not materialized) - No, I suppose the only thing we can do is using the read time as processing time, as @liurenjie1024 commented above. To avoid that, users should define an event time e.g. Kafka timestamp


Speed Opt

https://docs.espressif.com/projects/esp-idf/en/latest/esp32c6/api-guides/performance/speed.html
profile using JTAG
Interrupt based profiler (TODO github link)

https://github.com/Carbon225/esp32-perfmon

Semihosting profiler has been added.
To configure, create /littlefs/conf/pprof.conf:
sprofiler_hz=100

The firmeware will start profiling at start-up if it detects a debugger connected with esp_cpu_dbgr_is_attached()
see https://github.com/fl4p/fugu-mppt-firmware/blob/main/doc/dev-notes/performance profiling.md

https://github.com/espressif/idf-extra-components/tree/master/iqmath
convert float math to fixed-point


How to get final position&rotation&scale

Hi. author. we can set init value position&rotation&scale by pass attribute to it, but when position&rotation&scale behavior by mouse, can we get final data by any API?  ths

Hi. author. we can set init value position&rotation&scale by pass attribute to it, but when position&rotation&scale behavior by mouse, can we get final data by any API? ths

No such API available now.


balance(ce"Cl2 + IO3{-} = IO4{-}") fails.

The balanced equation for the skeleton eq. Cl2 + IO3{-} = IO4{-} in alkaline solution is:
IO3 + 2Cl2 + 2OH{-} = IO4{-} + 2Cl{-} + H2O
I tried balance(ce"Cl2 + IO3{-} = IO4{-}"), which fails.
I tried giving all reactants and products like:
balance(ce"Cl2 + IO3{-} + OH{-} = IO4{-} + Cl{-}")
but can't be balanced.

Due to the current simplicity, equation completion is not supported yet. That means you have to write all reactants out:
julia> balance(ce"IO3 + Cl2 + OH{-} = IO4{-} + Cl{-} + H2O")
2 IO3 + Cl2 + 4 OH{-} = 2 IO4{-} + 2 Cl{-} + 2 H2O

Your second example is missing H{+} for complete ionic reaction:
julia> balance(ce"Cl2 + IO3{-} + OH{-} = IO4{-} + Cl{-} + H{+}")
Cl2 + IO3{-} + OH{-} = IO4{-} + 2 Cl{-} + H{+}

I plan to support equation completion (like webqc.org does), and if you have some ideas how to do that, we can try that.

Oh I see. I played with it to solve some more redox equations and it worked well. How wonderful algorithm it is! and I am so surprised that redox eq can be balanced without knowledge of oxidation numbers.

@eisthf Thanks! It's actually something you can do on paper as well (i.e. solve a system of linear equations), but in this package it's automated.


Vim doesn't meet PEP 451

There are a few issues with vim's Python "module" implementation. Most specifically, it doesn't meet PEP 451. This means that Google's Protobuf can't be imported within a Vim plugin.
The vim "module" is used in sys.path_hooks. It is intended to be a finder, specifically a path entry finder. This means that it should at least duck-type as an importlib.abc.PathEntryFinder. Unfortunately, it doesn't support find_module, which is the reason for the error when importing Protobuf.
See https://github.com/vim/vim/commit/b999ba2778b6c02fdd100e498a9ba5b2e7036f7d, https://github.com/pypa/setuptools/pull/1563#issuecomment-463810046, and https://github.com/vim/vim/pull/3954#issuecomment-463708190.

More than that, this issue is severely impacting python-mode vim plugin, we have been trying to work around this issue for several months, but it always get back again (off course, we just worked around it).

PEP 451 title is "PEP 451 -- A ModuleSpec Type for the Import System" and it's a very long document.
This issue is very vague and unclear about what the actual problem is or how it could be fixed.
It's unlikely anything is going to happen unless that is cleared up.

I left all of the information that's needed in order to fix the issue. The vim module needs to implement what's specified for a PathEntryFinder, which is an abstract base class. https://docs.python.org/3/library/importlib.html#importlib.abc.PathEntryFinder Specifically, it needs to implement find_spec, find_loader, find_module, and invalidate_caches. It does not implement find_module.
There's been some confusion in the past about deprecation. In order to work, all deprecated methods must be implemented. The reason is that these deprecated methods are part of the contract for being a PathEntryFinder. Those using a PathEntryFinder should use the newer methods, but they may use the deprecated API.

Joel Frederico wrote:

I left all of the information that's needed in order to fix the issue.
The vim module needs to implement what's specified for a
PathEntryFinder, which is an abstract base class.
https://docs.python.org/3/library/importlib.html#importlib.abc.PathEntryFinder
Specifically, it needs to implement find_spec, find_loader,
find_module, and invalidate_caches. It does not implement find_module.
There's been some confusion in the past about deprecation. In order to
work, all deprecated methods must be implemented. The reason is that
these deprecated methods are part of the contract for being a
PathEntryFinder. Those using a PathEntryFinder should use the
newer methods, but they may use the deprecated API.

I suppose you are talking about what is at the end of if_py_both.h.
That was changed not too long ago.  It still isn't right?
--
SECOND SOLDIER: It could be carried by an African swallow!
FIRST SOLDIER:  Oh  yes! An African swallow maybe ... but not a European
swallow. that's my point.
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Sorry for the delayed response. No, it still doesn't work. The changes didn't make it into the vim module. You can check by opening vim and running:
:python3 import vim; vim.find_module
It says there's no attribute find_module.

I edited a bit and I got find_module to appear, but there's a segfault. I'd like to run MacVim through the XCode debugger so I can do breakpoints, etc., but I can't find documentation on how to do that?


Applying styles on particular rows based on data in it

Hi,
Is there any way of applying style on row based on the data in that.

I have the same question ! Will follow on this..

I did:
settings['rowClassFunction'] = (row) => {         if (row.data.type === '(-)') {           return 'score negative'; // Color from row with negative in score         } else if (row.data.type === '(+)') {           return 'score positive';         }         return '';       };
row.data is the same, you have to change type with the object property you have.
In the return, you return a class which should be defined in .css

Thanks a lot !
I didn't manage to get it works when my class was specified in the style sheet of the component itself but it worked like charm when I specified it in the general styles.css..
Also I didn't find rowClassFunction property in the doc..

rowClassFunction  was added in a PR.
If  you have the last version of the library it will work

Hello :)
is it possible to pass a parameter from the controller to settings (to the rowClassFunction)?
I want to do:
rowClassFunction : function(row){
if(row.data.id == this.scenarioId)
return "scenarioToClone";
return "";
}
this.scenarioId in rowClassFunction return 'undefined'.
thank you,

@HaifaGhidhaoui
It is not possible, because the this inside the function, is not the same this of the controller. It dependes in the context you are.

Hope someone can clarify the use of classRowFunction here to change a specific cell's background color. I have the function that checks if a value is true or false. if it's false the cell's background color should be red. Any help is appreciated.

Hi @lopenchi
I am trying to give different colors to table data based on their value.
Is there any way to do it.
I tried the rowClassFunction method, but it's giving me errors

@anandhuajayakumar can you put your code here? This way I can help you

@lopenchi I am using the component in a angular 4 application.The  following is the settings for the directive
settings = {     columns: {       user_id: {         title: 'User ID',         filter: true,         editable: false       },       full_name: {         title: 'Full Name',         filter: true,       },       email: {         title: 'Email',         filter: true,       },       password: {         title: 'Password',         filter: true,         type: 'password'       },       role: {         title: 'Role',         filter: true,       },     },     add: {       confirmCreate: true,       addButtonContent: 'Add'     },     delete: {       confirmDelete: true     },     edit: {       confirmSave: true     },     actions: {       position: 'right'     },     color: {       type: 'html',       title: 'color',       filter: false,       rowClassFunction: (row) => {         if (row.data.type === '(-)') {           return 'score negative';         } else if (row.data.type === '(+)') {           return 'score positive';         }         return '';       }     }   };

@anandhuajayakumar
in css define your style classes
.good {
    color: #0ef506;
  }
  .warning {
    color: #ffa500;
  }

in component create a function
getHtmlForCell(value: string) {
         if(value === '(-)') {
             return `<a class="${ warning }">score negative</a>`;
          }
         else{
              return `<a class="${ good }">score positive </a>`;
     }
}

in your table settings define a column like so
color: {
         type: 'html',
         title: 'color',
         filter: false,
          rowClassFunction: (row) => {
             return this.getHtmlForCell(row.data.type)
          }

}

@jarredszabadi when you say row.data does it mean the whole row or just a certain field

check the edit. good catch

@anandhuajayakumar The problem is that rowClassFunction is a property of settings , not a property of column.
So your code should be:
settings = {
  columns: {
    user_id: {
      title: 'User ID',
      filter: true,
      editable: false
    },
    full_name: {
      title: 'Full Name',
      filter: true,
    },
    email: {
      title: 'Email',
      filter: true,
    },
    password: {
      title: 'Password',
      filter: true,
      type: 'password'
    },
    role: {
      title: 'Role',
      filter: true,
    },
  },
  add: {
    confirmCreate: true,
    addButtonContent: 'Add'
  },
  delete: {
    confirmDelete: true
  },
  edit: {
    confirmSave: true
  },
  actions: {
    position: 'right'
  },
  color: {
    type: 'html',
    title: 'color',
    filter: false
  },
  rowClassFunction: (row) => {
    if (row.data.type === '(-)') {
      return 'score negative';
    } else if (row.data.type === '(+)') {
      return 'score positive';
    }
    return '';
  }
};

You must change type of row.data for the name of column you want to analyze. If I understand you, it should be color

@lopenchi - Hi, am facing a similar issue. I added rowClassFunction as per your suggestion. And I added a css class as well. But it does not show a different font color for the rows. When I inspect in chrome, it still shows only a div, with no class attributes.
Here is my code:
  settings = {
    columns: {
      userName: {
        title: 'User', editable: false, sort: false, sortDirection: 'asc', filter: false, width: '40%'
      },
      count: {
        title: 'Count', editable: false, sort: false, sortDirection: 'desc', filter: false, width: '60%'
      },
    },
    actions: {
      add: false,
      edit: false,
      delete: false
    },
    rowClassFunction: (row) => {
      //console.log("row.data.userID:: " + row.data.userID + ", this.loggedInUserId:: " + this.loggedInUserId);
      if (row.data.userID === this.loggedInUserId) {
        return 'highlightOwner';
      }
      return '';
    }
  };

CSS:
.highlightOwner{
    color: red;
}

Can you pls guide me on what am I doing wrong? Appreciate the help.

@iosnewbie2016  I think that you dont have value of variable : this.loggedInUserId.
WHen you initializa settings maybe you dont have it. Try to add rowClassFunction in settings AFTER data es completed

Hello @lopenchi
I just did what you suggest, but the row is not being colored.
rowClassFunction: (row) => {                 if (row.data.GestionFacturacion === 'Observado') {                     return 'highlightObservado';                 }
row.data.GestionFacturacion is actually 'Observado', it goes inside the "if" but the row doesn't change to red (the color i want)
.highlightObservado{   color: red; }

the css is applied only to first column i want it to be applied on full row

the css is applied only to first column i want it to be applied on full row

You have to put the code in every column at configurations object.

css class is not applying to all rows that meet the condition.. only showing for certain
row

@lopenchi    Can u please guide me to apply css style to (noDataMessage: 'Loading, Please wait...',..) I am able to handle dynamically but I would like to change the fontstyle and text-alignt to center....
settings = {
mode: "external",
columns: {
}
actions: {
add: false,
delete: false,
position: 'right',
edit: true,
},
****noDataMessage: 'Loading, Please wait...',****

};

could anyone help on the above post...
Appreciate the help

I have no idea how to add styles to noDataMessage


Format validation for the --env flag values

The "build" and "profile" command have a flag to pass environment variables and the flag values are passed as-is to the downstream container runtime API calls. There's no format validation for the provided values. The values are expected to have the standard environmental variable declaration format (example: ENV_VAR_NAME=ENV_VAR_VALUE)
It'll be good to validate the --env flag values to make sure they follow the expected format. If they don't follow the expected format we need to exclude the malformed values from the list of the env vars used later on in the command processing printing to the screen (using the 'execution context' in the command) that the malformed environment variable is discarded/ignored (we should not fail the command execution just because one of the env var values is malformed).

@kcq , thanks I'm looking into it. When outputting the notice that a key=variable is to be excluded, do you want to use "Error" or "Prompt" or "Info"?

@kcq , will create a PR for you to review, it's a WIP as there are some that might need some clarification.

the enhancement is done... closing the ticket


Replaced hipstershop to oteldemo

Fix #3179.
The hipstershop name was removed from the demo some time ago, and the docs were not updated.
This PR syncs the docs with the changes done in the demo ShippingService(Rust) and FeatureFlagService(Elixir).

How can I add oteldemo as a known word?
I'd like to avoid the following error:
Warning: Unknown word (oteldemo)

2. npm run fix:dict

npm run fix:dict

> fix:dict
> find content layouts -name "*.md" -print0 | xargs -0 ./scripts/normalize-cspell-front-matter.pl

Can't locate JSON.pm in @INC (you may need to install the JSON module) (@INC contains: /etc/perl /usr/local/lib/x86_64-linux-gnu/perl/5.34.0 /usr/local/share/perl/5.34.0 /usr/lib/x86_64-linux-gnu/perl5/5.34 /usr/share/perl5 /usr/lib/x86_64-linux-gnu/perl-base /usr/lib/x86_64-linux-gnu/perl/5.34 /usr/share/perl/5.34 /usr/local/lib/site_perl) at ./scripts/normalize-cspell-front-matter.pl line 5.
BEGIN failed--compilation aborted at ./scripts/normalize-cspell-front-matter.pl line 5.

I manually tweaked the page-local dictionaries, and all checks are passing now. I opened an issue for the problem you reported (thanks):

#3187


feat: Adding files support for file sending

About
Title.
Checklist

[x] The pre-commit code linter has been run over all edited files to ensure the code is linted.
[x] I've ensured the change(s) work on 3.8.6 and higher.

I've made this pull request: (check all that apply)

[ ] For the documentation
[x] To add a new feature
[ ] As a general enhancement
[ ] As a refactor of the library/the library's code
[ ] To fix an existing bug
[ ] To resolve #ISSUENUMBER

This is:

[ ] A breaking change

Docs should be updated

Oh it.


vnStat interface filter not working

I have multiple interfaces on my linux system, bot really only want to see stats on one - eth0. In the interface box I have manually entered eth0m but I still see tunl0 (another interface) in the vnStat view

This is intended. The interface is just for the speed nothing else, if you wish to filter more you can edit vnstat config.

I'm all about clutter... I hate it in my interfaces :P
I'll look into filtering it out, then updating the wiki. might be a couple days, my TODO list is huge.

Np, I'm happy for all the help I can get

fix https://github.com/Hellowlol/HTPC-Manager/blob/master2/modules/vnstat.py#L37

@EZarnosky Can you test if the filter works now?

Should work now, feel free to reopen if that isnt true


iPad Pro

iPad Pro screenshot's export?

+1

+1

+1
On Tuesday, December 15, 2015, KoichiroKAMADA<EMAIL_ADDRESS>wrote:

+1
—
Reply to this email directly or view it on GitHub
https://github.com/LaunchKit/SketchToAppStore/issues/17#issuecomment-164996880
.

:+1:

ipad pro would be great.

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

👍

Check recent commit.
https://github.com/LaunchKit/SketchToAppStore/commit/465e0a8ee6f147cf083f0605cf554b8c095b62c4

That was.....fast.

@ecjs No reason to be sarcastic like that.
Thanks @bmull

.I actually was serious, if you look at my 👍  it was 2 hours before the fix was committed.

@ecjs So sorry about this, didn't notice the 👍 couple hours before 😄

No problem, these things happen ;)  I probably did a poor job conveying my gratitude :)


Remove vim configurations from source code.

Source and header files contains commented vim configurations.
Is there a reason why these configurations are directly in source code and not in a local vim configuration file?

I use vim's modeline feature to set specific settings only for specific source code in specific paths.
Do you know how to achieve this functionality using a local configuration file?
Something that works with both vim and neovim would be preferable.

I use vim's modeline feature to set specific settings only for specific source code in specific paths.
Do you know how to achieve this functionality using a local configuration file?
Something that works with both vim and neovim would be preferable.

Thank you for your consideration.
Yes it should be possible.
As I'm at NEC today and tomorrow, I'll provide you with an example by the end of this week.

@tunefs sorry for the delay on this PR.
I'm not sure whatever you use this configuration in other directories aswell but by adding the following to your .vimrc file, the shift indention will automatically apply to path specified.
Since the cfg is only available under lexxpluss_apps/src, you could do something like this:

autocmd BufRead,BufNewFile /repo/path/lexxpluss_apps/src/* set expandtab shiftwidth=4

It will not work unless the paths to the git repository and the vim local configuration match.
The git repository changes depending on the development environment.

It will not work unless the paths to the git repository and the vim local configuration match.
The git repository changes depending on the development environment.
Sorry I didn't quite understand this point but I'll close this PR.
Thank you for your consideration.


Portuguese Create User Test

Overview
All our testing is currently in english and we want to ensure our other site languages are working. This need came from an incident where a bug happened on all browser languages except for english and the cypress tests did not catch it.
Action Items

[ ] Get Cypress tests up and running. See Cypress Readme
[ ] Create your own branch
[ ] Take a look at this german example of the test
[ ] Duplicate this test in Portuguese and name it create-user.pt.cy.tsx. Note that you will have to update some of the test assertions to match the Portuguese version of the site. Take a look at bloom.chayn.co/pt for the Portuguese version of the site to find the equivalent text.
[ ] See notes below for helpers
[ ] Commit and push your changes and create a PR.
[ ] Tag @eleanorreem  for a review

Resources/Instructions

Cypress readme
Ensure you create a custom visitPortuguesePage that replicates the behaviour of visitGermanPage in this file
You will also have to update the type file

Hey @MichelleMello, absolutely 😄  I have just assigned you.

@eleanorreem can I work on this?

Hi @bhiwagade-rahul thank you for your interest! We are currently running a volunteer program and need to keep beginner issues open for them. Please check out our other issues or check back in a month for more beginner issues. Thanks!

Hi Can I be assigned this?

Hey @nsem7 As a volunteer, please let us know if you need more time, have any questions, or need guidance. Otherwise we will open this issue back up to volunteers. Thanks!

@nsem7 Understood. Feel free to open up a pull request in the mean time. Are you having issues configuring tests or failing tests?

@nsem7 any updates?
Also you may have run into issues with Cypress tests because we updated where the environment variables are stored. Instead of a dedicated env file for Cypress variables, they are now stored in the same env file as all the other variables. See updated directions in README.


In the app, line thickness and icon size can be set to 0 and below

The issue is in the title. This is not that critical but is still a bad thing to have in a public project :D

And, yeah, do change the color value of inputs and stuff if you change the background color or do neither of that, so people with dark desktop themes don't bleed their eyes out.

This was made just as a small MVP / proof of concept so obviously, there's plenty of things that could be added or done better :) For, example atm there's just no validation on form inputs at all, not only for negative values. We'll do our best to make one in the future!
Good call on the inputs' color property, I overlooked it. Thanks again for your help 🙌🏼

Both issues should be fixed now! Now, as you reach zero, the buttons get disabled.
Although, you still can insert negative values manually (same as any other string) but I would consider this acceptable. We will try to add some validation in the future.


[gtk] update to 4.10.5

Fixes https://github.com/microsoft/vcpkg/issues/33500
All feature tested pass in the following triplets:

x64-windows
x86-windows

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[ ] The "supports" clause reflects platforms that may be fixed by this new version
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Thanks for the update!


[Bug]: When creating an index for a table without change permission, consider the index name as the table name

ODC version
odc432
OB version

What happened?

What did you expect to happen?
Just need the change permission of the original table
How can we reproduce it (as minimally and precisely as possible)?
Create a table without change permission and create an index
Anything else we need to know?
No response

pass


Add style header and file checking

process_upload() in https://meta.svn.wordpress.org/sites/trunk/wordpress.org/public_html/wp-content/plugins/theme-directory/class-wporg-themes-upload.php is doing a bunch of checks on the style.css header which is not in the theme check. We should create a separate function that would be included in the theme check. It would be easier to build this check in the theme check plugin then in PHPCS.
We could also check that the text domain is set right.
Could include the checking of the style header.
https://github.com/WordPress/theme-check/blob/master/checks/style_needed.php
https://github.com/WordPress/theme-check/blob/master/checks/style_suggested.php
https://github.com/WordPress/theme-check/blob/master/checks/style_tags.php

I have started on a function that will do the job but I am not sure how best to output the data nicely. https://github.com/grappler/ns-theme-check/commit/1bb62ecceaa25d0ca42c6efd1a8847d18ce21550


How do you translate in an API route (/pages/api/...) ?

Hi,
I am currently evaluating this lib, and it looks great!
I have few newbies questions:

how can we get the t function inside an api route ?

Inside an api route, how can we translate to another language (changeLanguage) ?
use case: a user click a Like button ie: /pages/api/like.js and inside the api route we notify the recipient which can use another language

Thanks

Right now there is no way, but if you are interested we could implement this feature. We usually advise that the translations are better used inside React, while outside (in the api or inside methods like getStaticProps) it is better to use the translation keys.

@aralroca Thanks for your quick reply :)
I will keep migrating our pages from next-i18next and will come back to you if I still have issues with /api routes.

@fillon I have made a PR adding getT, with this you can use translations directly in api route. https://github.com/vinissimus/next-translate/pull/383

@fillon I pre-released the change under 1.0.0-canary.8. I would appreciate it if you could try it out on your project and give me feedback that everything is working well for you 😊
If you wonder how to migrate to version 1.0, here is a guide:

https://github.com/vinissimus/next-translate/blob/1.0.0-canary.8/docs/migration-guide-1.0.0.md

Any doubt, tell me. Thank you very much for reporting it 🙏 I guess that version 1.0 will be released soon.

@aralroca Everything worked with @canary.7
@canary.8
I am getting a ReferenceError: globalThis is not defined on a page when using loadNamespaces
export async function getStaticProps(ctx) {
  return {
    props: await loadNamespaces ({
      ...ctx,
      pathname: '/hello',
    }),
  };
}

i18n.js
module.exports = {
  defaultLocale: 'en',
  loader: false,
  locales: [
    'en',
...
    'vi',
    'zh',
  ],
  // localesPath: 'locales',
  loadLocaleFrom: (lang, ns) =>
    import (`./locales/${lang}/${ns}.json`).then (m => m.default),
};

next.config.js
// const { locales, defaultLocale } = require ('./i18n.js');
const nextTranslate = require ('next-translate');

module.exports = nextTranslate (
  withMDX ({
    pageExtensions: [ 'js', 'jsx', 'md', 'mdx' ],
  })
);

_app.js
I was using <I18nProvider lang={router.locale} namespaces={pageProps._ns}>
I removed it.

_app.js
...

// used with canary.7
// export const loadNamespaces = async (namespaces, lang) => {
//  let res = {};
//  for (let ns of namespaces) {
//    res[ns] = await import (`../../locales/${lang}/${ns}.json`).then (
//      m => m.default
//    );
//  }
//  return res;
// };

MyApp.propTypes = {
  Component: PropTypes.elementType.isRequired,
  pageProps: PropTypes.object.isRequired,
};

export default function MyApp(props) {
  const { Component, pageProps } = props;

  return (
    <>
      <ThemeProvider theme={theme}>
        <AuthProvider>
            <CssBaseline />
...
            <Component {...pageProps} />
        </AuthProvider>
        <ReactQueryDevtools initialIsOpen={false} />
      </ThemeProvider>
    </>
  );
}

my locales are in the root folder
/locales
/en
common.json
...
/src
/pages
_app.js
...
i18n.js
next.config.js

@fillon are you using a Node version lower than 12? https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/globalThis

@aralroca node v10
let me upgrade it and come back to you :)

@fillon  globalThis is supported on Node +12, I hope it works for you after migrating. I'm also thinking about changing it to justglobal to give more support so you can use it on Node 10. Although all the tests we have are from Node 12 onwards...

@fillon I did another prerelease, 1.0.0-canary.9, replacing globalThis to global, now it should work with Node 10 👍  😊

@aralroca with 1.0.0-canary.9 it's compiling again
I still have an error, it doesn't seem to loadnamespaces
next-translate - compiled page: /hello - locale: fr - namespaces:  - used loader: -
i18n.js
module.exports = {
  defaultLocale: 'en',
  loader: false, // This deactivate the webpack loader that loads the namespaces
  // When loader === false, then loadLocaleFrom is required
  loadLocaleFrom: (lang, ns) =>
    import (`./locales/${lang}/${ns}.json`).then (m => m.default),
  locales: [
    'ar',
    'da',
    'de',
    'en',
    'es',
...
  ],
  pages: {
    '*': ['common'],
    // '/hello': ['common'],
  },
};

/pages/hello.js
export async function getStaticProps(ctx) {
  return {
    props: await loadNamespaces ({
      ...ctx,
      pathname: '/hello',
    }),
  };
}

@aralroca  Do we still need to wrap MyApp with appWithI18n like in the canary examples?

In your configuration locales fr is missing. But the detected locale is fr.

@aralroca fr was defined.. it's working ok in the pages
For /api endpoint:
const t = await getT ('fr', 'common'); if i specify the locale.. translation is working great
how does req.query.__nextLocale get initialized ?  I always get 'en'
Fetching like this on the client side:
await fetch ('/api/v3/ping', {
      method: 'POST',
      headers: {
        'Accept-Language': router.locale,
        'Content-Type': 'application/json',
      },
      body: JSON.stringify ({
...
      }),
    });

This req.query.__nextLocale is introduced by Next.js. If is returning always English is because you are requesting to /api/endpoint instead of /fr/api/endpoint. Another way you can archive this is by using a cookie named NEXT_LOCALE, this way also is saving the preferred user language. This is introduced by Next.js i18n routing. I hope it helps

@aralroca it helps a lot 👍
I will finally be able to uninstall i18next and in-house i18middleware.
Great work!! Love this lib. Simple straight to the point.


Locking idioms are incomplete

The user spec provides this guidance:

We recommend the use of the AMO Swap idiom shown above for both lock acquire and release
to simplify the implementation of speculative lock elision [25].

I've been puzzling over this for months and I don't currently think it's possible to implement pthread_mutex_lock using a single amoswap.w.aq and amoswap.w.rl in the uncontended path while supporting existing futex/WaitOnAddress APIs, PTHREAD_PROCESS_SHARED, and fairness.
(In particular, if a mutex is released with a single AMOSWAP, then it goes directly from "contended" to "unlocked", which allows another process to acquire the mutex before the wait list can be processed.)

(Just providing context:) glibc's low-level locking code seems to use CAS to acquire locks and swap to release them.

I think all there is to say in the spec is that some more complex locks will be implemented with LR/SC.  The only interesting thing about those sequences is how to set the AQ/RL bits on the two operations; while I think it will suffice to set AQ on a lock-acquiring SC and RL on a lock-releasing SC, we'd rather wait til the memory model discussion is more complete before writing that up.

I think that lock elision isn't compatible with LR/SC mechanism. Elided store wouldn't invalidate reservations held by other harts after LC instruction and SC would succeed. It would made LR/SC suffer from ABA problem.

There are a lot of misconceptions about ABA problem.  LR/SC only has an expressiveness advantage over CAS if unrelated memory accesses are allowed to occur between the LR and the SC; RISC-V forbids this so RISC-V LR/SC is already precisely equivalent to CAS..

Regarding ABA - in the case where there is a single word being accessed, then LR/SC will abort if a second hart rewrites the same value as is currently in memory to the single word under reservation by a first hart.   CAS will not detect this. This is admittedly a limited case but does mean LR/SC is not equivalent to CAS for ABA.


Tenant name "public" confuses everything

Describe the bug
Creating and especially deleting a tenant named "public" confuses the entire Saltcorn application.
To Reproduce
Steps to reproduce the behavior:

Enable multi-tenant support
Create tenant named "public"
Observe error message schema "public" already exists
Delete the tenant "public"
Observe next error message relation "public._sc_tenants" does not exist
Saltcorn is no longer responding

Expected behavior
Display error message when attempting to create a tenant with a name that conflicts with internal structures.

This is fixed now.


set clipping and nonwear thresholds outside of loops

in g.getmeta.R: set clipping and nonwear thresholds before the loop for chunksize as well as the loop for ws2 and the loop for accel channel. Code efficiency savings.

Codecov Report

Merging #218 into master will increase coverage by 0.11%.
The diff coverage is 73.07%.

@@            Coverage Diff             @@
##           master     #218      +/-   ##
==========================================
+ Coverage   75.34%   75.46%   +0.11%
==========================================
  Files          63       63
  Lines        7878     7878
==========================================
+ Hits         5936     5945       +9
+ Misses       1942     1933       -9

Impacted Files
Coverage Δ

R/g.getmeta.R
63.84% <73.07%> (+1.44%)
:arrow_up:

Continue to review full report at Codecov.

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

thanks Matt


feat: support Option<fn()> in WrapperApi

use dlopen2::wrapper::{Container, WrapperApi};
use std::os::raw::c_int;

struct Api<'a> {
    c_fun: unsafe extern "C" fn(),
    // A function may not exist in the library.
    c_option_fun: Option<unsafe extern "C" fn() -> c_int>,
}

fn main(){
    let mut cont: Container<Api> =
        unsafe { Container::load("libexample.so") }.expect("Could not open library or load symbols");
    unsafe{ cont.c_fun() };

    if cont.has_c_option_fun() {
      // do something
    }
    // option function returns Option<fn_return_type>, it's Option<c_int> here.
    unsafe{ cont.c_option_fun().map(|i| i == 0) };
}

We use dlopen2 to load multiple versions of libraries. dlopen2_allow_null is helpful, but fn pointer is a bit more complex to use. Currently we use Option fn and extract each fn mannually with dlopen2 raw api. It's better to support Option types in WrapperApi derive impl.
Hope it can be accepted. Please have a look @OpenByteDev .

Published in 0.6.0


Fix screen orientation flip-flop when screen opens

Change-Id: I7cf33778ae2fc0f08ba1e178d2d3c80d1b1f4026

To test, create a project where Screen1 has landscape orientation. Possibly add more screens with other orientations. Compile the APK and install it on a device. Hold the device in portrait mode and launch the app. It will start in portrait and then rotate to landscape in production. With this PR applied, the app should start in the designated orientation and not rotate.


Media type application/javascript is outdated. Should be text/javascript

See https://developer.mozilla.org/en-US/docs/Web/HTTP/Basics_of_HTTP/MIME_types#textjavascript

Fixed in 0.3.1


[bond] build failure

D:\opt\vcpkg>vcpkg install bond
Computing installation plan...
The following packages will be built and installed:
bond[core]:x64-windows

boost-algorithm[core]:x64-windows

boost-assign[core]:x64-windows

boost-atomic[core]:x64-windows

boost-build[core]:x64-windows

boost-chrono[core]:x64-windows

boost-container[core]:x64-windows

boost-date-time[core]:x64-windows

boost-exception[core]:x64-windows

boost-filesystem[core]:x64-windows

boost-foreach[core]:x64-windows

boost-iostreams[core]:x64-windows

boost-lexical-cast[core]:x64-windows

boost-locale[core]:x64-windows

boost-math[core]:x64-windows

boost-phoenix[core]:x64-windows

boost-proto[core]:x64-windows

boost-ptr-container[core]:x64-windows

boost-range[core]:x64-windows

boost-regex[core]:x64-windows

boost-serialization[core]:x64-windows

boost-spirit[core]:x64-windows

boost-system[core]:x64-windows

boost-thread[core]:x64-windows

boost-unordered[core]:x64-windows
Additional packages (*) will be modified to complete this operation.
Starting package 1/25: boost-build:x64-windows
Building package boost-build[core]:x64-windows...
-- Using cached D:/opt/vcpkg/downloads/boostorg-build-boost-1.73.0.tar.gz
-- Using source at D:/opt/vcpkg/buildtrees/boost-build/src/ost-1.73.0-018fb008dd
-- Using cached D:/opt/vcpkg/downloads/boost_LICENSE_1_0.txt
-- Downloading https://raw.githubusercontent.com/boostorg/boost/boost-1.73.0/boostcpp.jam...
-- Downloading https://raw.githubusercontent.com/boostorg/boost/boost-1.73.0/boostcpp.jam... Failed. Status: 5;"Couldn't resolve proxy name"
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:175 (message):
Failed to download file.
If you use a proxy, please set the HTTPS_PROXY and HTTP_PROXY environment
variables to "https://user:password@your-proxy-ip-address:port/".
Otherwise, please submit an issue at https://github.com/Microsoft/vcpkg/issues

Call Stack (most recent call first):
ports/boost-build/portfile.cmake:27 (vcpkg_download_distfile)
scripts/ports.cmake:90 (include)
Error: Building package boost-build:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: boost-build:x64-windows
Vcpkg version: 2020.02.04-nohash
Additionally, attach any relevant sections from the log files above.
D:\opt\vcpkg>set HTTPS_PROXY=
D:\opt\vcpkg>set HTTP_PROXY=
D:\opt\vcpkg>vcpkg install bond
Computing installation plan...
The following packages will be built and installed:
bond[core]:x64-windows

boost-algorithm[core]:x64-windows

boost-assign[core]:x64-windows

boost-atomic[core]:x64-windows

boost-build[core]:x64-windows

boost-chrono[core]:x64-windows

boost-container[core]:x64-windows

boost-date-time[core]:x64-windows

boost-exception[core]:x64-windows

boost-filesystem[core]:x64-windows

boost-foreach[core]:x64-windows

boost-iostreams[core]:x64-windows

boost-lexical-cast[core]:x64-windows

boost-locale[core]:x64-windows

boost-math[core]:x64-windows

boost-phoenix[core]:x64-windows

boost-proto[core]:x64-windows

boost-ptr-container[core]:x64-windows

boost-range[core]:x64-windows

boost-regex[core]:x64-windows

boost-serialization[core]:x64-windows

boost-spirit[core]:x64-windows

boost-system[core]:x64-windows

boost-thread[core]:x64-windows

boost-unordered[core]:x64-windows
Additional packages (*) will be modified to complete this operation.
Starting package 1/25: boost-build:x64-windows
Building package boost-build[core]:x64-windows...
-- Using cached D:/opt/vcpkg/downloads/boostorg-build-boost-1.73.0.tar.gz
-- Using source at D:/opt/vcpkg/buildtrees/boost-build/src/ost-1.73.0-018fb008dd
-- Using cached D:/opt/vcpkg/downloads/boost_LICENSE_1_0.txt
-- Downloading https://raw.githubusercontent.com/boostorg/boost/boost-1.73.0/boostcpp.jam...
-- Downloading https://raw.githubusercontent.com/boostorg/boost/boost-1.73.0/boostcpp.jam... Failed. Status: 6;"Couldn't resolve host name"
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:175 (message):
Failed to download file.
If you use a proxy, please set the HTTPS_PROXY and HTTP_PROXY environment
variables to "https://user:password@your-proxy-ip-address:port/".
Otherwise, please submit an issue at https://github.com/Microsoft/vcpkg/issues

Call Stack (most recent call first):
ports/boost-build/portfile.cmake:27 (vcpkg_download_distfile)
scripts/ports.cmake:90 (include)
Error: Building package boost-build:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: boost-build:x64-windows
Vcpkg version: 2020.02.04-nohash
Additionally, attach any relevant sections from the log files above.

Thanks for your issue. Could you work around this issue as following steps:

Download https://raw.githubusercontent.com/boostorg/boost/boost-1.73.0/boostcpp.jam manually.
Put it to .\vcpkg\downloads\
Try to build again.

Pinging @sinozope

Thanks for posting this issue. Please reopen this issue if this is still a problem for you.


Include DeletionPolicy entries for all resources.

Importing this JSON example fails because the resources are missing DeletionPolicy entries.  This change will fix that.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks for the feedback and suggestion. There might be some SCP or controls in your environment that prevents the creating of the resources without delete policy. By default, all resources in CFN have the deletepoicy set to true. Thanks!


Add sample ID to robot

Fixes #499
Instructions to reviewer on how to test:

Run dodal connect i03 and dodal connect i04 confirm that the robot connects happily for both

Checks for reviewer

[ ] Would the PR title make sense to a scientist on a set of release notes
[ ] If a new device has been added does it follow the standards
[ ] If changing the API for a pre-existing device, ensure that any beamlines using this device have updated their Bluesky plans accordingly
[ ] Have the connection tests for the relevant beamline(s) been run via dodal connect ${BEAMLINE}

Good spot, yes we should!


Default mutating webhook prevent resource to be deleted.

When deleting a policy server the policies are deleted. In this process the webhook register to mutate the clusteradmissionpolicies  receive a request. The function used to handle this request is the default one which adds a kubewarden finalizer in the resource preventing the policy to be removed and consequently the policy server is not deleted as well. To work around this the default webhook code checks if the resource has a deletion timestamp which indicates that the resource should be deleted and does not add the default finalizer.
Closes #100

It makes sense, but the kubebuilder markers only list the create verb (not the update, or delete):
(line 42 of policyserver_webhook.go for example)
//+kubebuilder:webhook:path=/mutate-policies-kubewarden-io-v1alpha2-policyserver,mutating=true,failurePolicy=fail,sideEffects=None,groups=policies.kubewarden.io,resources=policyservers,verbs=create,versions=v1alpha2,name=mpolicyserver.kb.io,admissionReviewVersions={v1,v1beta1}

So I wonder if we even run this code on delete.

It should be called just on create, not on delete. I can't reproduce it. I'm installing kubewarden using the helm chart. I see you are using make install and make run. https://github.com/kubewarden/kubewarden-controller/issues/100#issuecomment-944283124
Can you also reproduce the error if you use the helm chart? I'm thinking there might be something wrong with make install or make run?

@ereslibre Great job! you found the problem :) That's why I could not reproduce it using the helm chart. When using the helm chart webhook is executed just on create https://github.com/kubewarden/helm-charts/blob/main/charts/kubewarden-controller/templates/webhooks.yaml#L25
I think the webhook should just be executed on create. This webhook is just adding the kubewarden finalizer. In my opinion it does not make sense to also execute it on update and check if the deletion timestamp is there. It would be executed on all updates, but it won't do anything as the finalizer is already there.
I would just remove https://github.com/kubewarden/kubewarden-controller/blob/main/main.go#L162 and https://github.com/kubewarden/kubewarden-controller/blob/main/main.go#L143


eventsbilling and events storage api.

@rajesh-rajagopal Good job.

Thank you sir.
On Thu, Jun 23, 2016 at 3:04 PM, Raj<EMAIL_ADDRESS>wrote:

Merged #373 https://github.com/megamsys/vertice_gateway/pull/373.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/megamsys/vertice_gateway/pull/373#event-701632734,
or mute the thread
https://github.com/notifications/unsubscribe/ARyd4_yoZeQYxm15NfUfBOVdtYWDEnNoks5qOlMhgaJpZM4I8nk6
.

:+1:


Edit history contents overlap with timestamps

When I view the edit history of messages, the timestamps overlap with the messages. I'm seeing this on a couple different messages with Firefox on Ubuntu and OSX. Sample picture attached; messages in question are around https://chat.zulip.org/#narrow/near/193783/stream/general/topic/edit.20history.

See #3731 for related notes.


Issue #618 Change order of filtering to get correct list of vulnerabilities

This PR attempts to resolve Issue 618, where filtering by severity and allowed ghsas does not seem to work as expected.
Currently filtering by allowed advisory first then by severity, yields different (incorrect) output. Changing the order of filtering --> filter by severity first then by allowed advisory seems to resolve the issue. Added 2 test cases to reproduce the scenario.
[Needs review] "Deny packages" check and "Invalid License" check to run against the unfiltered changes. This is to ensure that it does not have incorrect output from the trimmed/filtered list. Also, these 2 checks should run independently of vulnerability/severity checks and not on a subset of the changes, since they are kind of unrelated. For example, a filtered change might have an incompatible/restrictive license issue

@virangdoshi thanks for this PR! After looking closer at this issue, I have noticed that the current behavior of the Action when dealing with GHSAs is to entirely drop the dependency change, instead of dropping the specific vulnerability.
Given this example from your test repo:
{
    "change_type": "added",
    "manifest": "package.json",
    "ecosystem": "npm",
    "name": "lodash",
    "version": "4.17.0",
    "package_url"<EMAIL_ADDRESS>    "license": "MIT",
    "source_repository_url": "https://github.com/lodash/lodash",
    "scope": "runtime",
    "vulnerabilities": [
      {
        "severity": "critical",
        "advisory_ghsa_id": "GHSA-jf85-cpcp-j695",
        "advisory_summary": "Prototype Pollution in lodash",
        "advisory_url": "https://github.com/advisories/GHSA-jf85-cpcp-j695"
      },
      {
        "severity": "high",
        "advisory_ghsa_id": "GHSA-4xc9-xhrj-v574",
        "advisory_summary": "Prototype Pollution in lodash",
        "advisory_url": "https://github.com/advisories/GHSA-4xc9-xhrj-v574"
      },
      {
        "severity": "high",
        "advisory_ghsa_id": "GHSA-35jh-r3h4-6jhm",
        "advisory_summary": "Command Injection in lodash",
        "advisory_url": "https://github.com/advisories/GHSA-35jh-r3h4-6jhm"
      },
      {
        "severity": "high",
        "advisory_ghsa_id": "GHSA-p6mc-m468-83gw",
        "advisory_summary": "Prototype Pollution in lodash",
        "advisory_url": "https://github.com/advisories/GHSA-p6mc-m468-83gw"
      },
      {
        "severity": "moderate",
        "advisory_ghsa_id": "GHSA-x5rq-j2xg-h7qm",
        "advisory_summary": "Regular Expression Denial of Service (ReDoS) in lodash",
        "advisory_url": "https://github.com/advisories/GHSA-x5rq-j2xg-h7qm"
      },
      {
        "severity": "moderate",
        "advisory_ghsa_id": "GHSA-29mw-wpgm-hmr9",
        "advisory_summary": "Regular Expression Denial of Service (ReDoS) in lodash",
        "advisory_url": "https://github.com/advisories/GHSA-29mw-wpgm-hmr9"
      },
      {
        "severity": "low",
        "advisory_ghsa_id": "GHSA-fvqr-27wr-82fm",
        "advisory_summary": "Prototype Pollution in lodash",
        "advisory_url": "https://github.com/advisories/GHSA-fvqr-27wr-82fm"
      }

If we add GHSA-fvqr-27wr-82fm to allow_ghsas (a low severity vulnerability), the entire lodash dependency will be dropped from the list of changes. This behavior should be fixed regardless of the order of the operations (although it also fixes this bug). I've opened a PR in https://github.com/actions/dependency-review-action/pull/623/ and would appreciate your feedback.

Re:

"Deny packages" check and "Invalid License" check to run against the unfiltered changes. This is to ensure that it does not have incorrect output from the trimmed/filtered list. Also, these 2 checks should run independently of vulnerability/severity checks and not on a subset of the changes, since they are kind of unrelated.

Agreed! Same with license issues (GHSAs have nothing to do with those). Feel free to create an issue, or better yet, submit a PR :)

Closing (fixed in main: https://github.com/actions/dependency-review-action/releases/tag/v3.1.4)


Pytest override session dependency

When using the fastapi.testclient.TestClient with pytest, the app doesn't use the session fixture resulting in writes to the database.
To solve this problem, pytest needs to tell the app to use the same session as used for testing. This can be done by using dependency_overrides.
@pytest.fixture(autouse=True)
def _app_session(app, session):
    app.dependency_overrides[fastapi_sqla.sqla.default_session_dep] = lambda: session

At least, this is how I solved the problem. Is there a beter approach or should I send a PR to document this?

My app is a session scoped fixture, changing it to function scoped and adding session as dependency raises on https://github.com/dialoguemd/fastapi-sqla/blob/4fc7c755a7c14c82845d8a6424e03f1dc67256e2/fastapi_sqla/sqla.py#L62
with
AttributeError: 'Connection' object has no attribute 'connect'. Did you mean: 'connection'?


Implement Array.Copy()

There doesn't seem to be any standard ways of copying data between arrays in Javascript, and if you're doing any significant manipulation in .Net, System.Array.Copy() would be the natural choice.
Also note that as this is a static method, it can't be simulated with an extension method.

Any update on this pull request?

Sorry, I forgot

I've removed the null checks, especially as there will probably be a pure JS error when the arrays are ultimately accessed.  But I'm uncomfortable at removing the bounds checks as it will make it really hard to diagnose bugs with ArrayCopy arguments.

Thanks for merging this and the other pull requests! :)


export --format md --output dist/ produces dist/.md (and --output dist/slides.md fails)

Describe the bug
I would like to Export PNGs and Markdown... primarily so that I can send a PR to https://github.com/slidevjs/docs/blob/main/.vitepress/showcases.ts with a cover: ... 😄
In order for e.g. that Cover PNG to be available, I want it to be in my dist/, along side the HTML app/site, and the PDF which I'm also putting there.
I've tried export --format md --output dist/ - but that produces a dist/.md - note the "empty" filename, which is a bit weird, agreed?
Using export --dark --format md --output dist/slides.md fails, like this:
●■▲
Slidev  v0.49.29
theme       @slidev/theme-seriph
css engine  unocss
entry       /workspaces/LearningMachineLearning/docs/prez/sli.dev/slides.md
▲ rendering...[Error: EISDIR: illegal operation on a directory, open 'dist/slides.md'] {
errno: -21,
code: 'EISDIR',
syscall: 'open',
path: 'dist/slides.md'
}
Environment

Slidev version: 0.49.29

Proposal
Perhaps you would like consider permitting --output dist/slides.md and using that instead of defaulting to dist/.md ?

Oh... wait; so: Using export --format md --output dist/png works! (The difference being it not ending in a trailing slash.)
That produces dist/png.md and dist/png/*.png - which makes sense, actually.
Perhaps you would consider handling --output with trailing slash, and automagically chopping it? Just an idea.
Or do you want a PR with an update for a note about this in the doc?

Using export --format md --output dist/png works!

Actually, nope, that doesn't really "work" - the links in the MD are broken, they point to ./dist/png/1.png (instead of just png/1.png).
I'm working around this like this.


no amalgamtes/crow_all.h?

Hi, so in the readme.md it says to clone amalgamates/crow_all.h, But Im not finding it. Im only seeing merge_all.py, which is a python file (from the amalgamets dir) but Im not sure, any thoughts?

You have to build the library locally on your machine, then only you are going to find that file.
``

git clone git@github.com:ipkn/crow.git
cd crow && mkdir build && cd build && cmake .. && make && sudo make install
cd amalgamate
ls

You will find `crow_all.h` in the directory.

In case you don't want to build the library locally then you can use [https://gist.github.com/Shravan40/56208dd932591a997158835e923a891a](this)

the other link is missing the boost/optional.hpp file, but thank you for letting me know how to use it!


proposal: net/http: add ErrSchemeMismatch

What version of Go are you using (go version)?

$ go version
go version go1.16 linux/amd64

Does this issue reproduce with the latest release?
Yes
What did you expect to see?
I'd like to see the http: server gave HTTP response to HTTPS client error in the net/http package to be exposed as a constant that can be compared with errors.Is
https://github.com/golang/go/blob/f21be2fdc6f1becdbed1592ea0b245cdeedc5ac8/src/net/http/client.go#L262
Context
I'm working on a clone of Docker called nerdctl (contaiNERD ctl).
I want to compare http: server gave HTTP response to HTTPS client with errors.Is, for ease of reimplementing dockerd --insecure-registries, which attempts HTTPS first and then falls back to HTTP.
What did you see instead?
The http: server gave HTTP response to HTTPS client error is not exposed.

/cc @bradfitz @rsc @empijei @neild

This seems reasonable to me, especially given that this error is replacing a more-informative (to programs) tls.RecordHeaderError.
Perhaps the error returned here should wrap the tls.RecordHeaderError rather than replacing it.

Can we wrap (2) in a function like func IsNotHTTPS(err error) bool?

I'm fine with (2). Or (1) if we name it, say, ErrSchemeMismatch ("HTTP"  in "ErrHTTPResponse" sounds too generic, as "HTTP" often also means "HTTPS").
I wouldn't want to see a new function. That's too heavy in godoc for how fringe this is.

Hi @bradfitz  @rsc @empijei @neild any solution regarding this error message?  @AkihiroSuda did you found the solution?
I tried to post and I got the same error message "http server gave http response to https client"
Thank you

PR: https://github.com/golang/go/pull/50939

@bradfitz Any chance to get the PR https://github.com/golang/go/pull/50939 merged? 🙏

This is a proposed API change, so adding to the proposal process.
Specific proposal:
Add http.ErrSchemeMismatch, a  distinguishable error that indicates an HTTPS server received what looks like an HTTP request. This converts an existing error to an exported, distinguishable symbol. See https://go.dev/cl/382117.

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

Sounds like everyone is in favor of this. Have all concerns been addressed?

Sounds like everyone is in favor of this. Have all concerns been addressed?

I don't see any remaining concern

Based on the discussion above, this proposal seems like a likely accept.
— rsc for the proposal review group


Add an alternative to sc-network-gossip for parachains

Address points (1) and (2) of https://github.com/paritytech/polkadot/issues/1348
We can add a crate similar to sc-network-gossip, except that it keeps track of the number of valid/invalid messages for each peer and pulls messages at a slower rate for peers whose ratio is too low.
The slower pulling rate should in turn apply back-pressure on the connection, and force the sending side to slow down the sending rate.
"Stops processing messages" requires some modifications to the API of sc-network in addition to https://github.com/paritytech/substrate/issues/6403. https://github.com/paritytech/substrate/issues/6403 itself depends on https://github.com/paritytech/substrate/pull/5938.
Not strictly-speaking concerned by this issue, but related: #5481 will cause issues on the sending side if we back-pressure the connection.

Not all of the protocols for Parachains networking are gossip protocols. And actually, many that are currently gossip protocols, we would prefer not to be as there would be major efficiency gains to be made by having validators communicate via direct messaging.
substrate-network-gossip is also not async-friendly, so for Parachains networking (https://github.com/paritytech/polkadot/pull/1344 & https://github.com/paritytech/polkadot/pull/1326 ) we have just been rolling our own components that do handle asynchrony & backpressure, although not in a particularly intelligent way.
Because of that, all of the backpressure work on networking can happen behind the scenes, from the Parachains codebase perspective. It would make most sense to me to focus on refactoring the network service to make write_notification an async fn. The safest assumption to me right now is then to just assume that there are no messages that we can block, and then pause any networking task on awaiting this function.
Behind the scenes one layer down in libp2p we would want to be dropping peers who are not accepting messages fast enough, which, as I understand it, has not actually been a problem in practice.

I'm going to close this issue as it's unclear what exactly to do.
#6403 is still relevant, though.


Updates to the code to allow it working on Terraform version 1.1.9

The goal is to make these terraform scripts part of schematics. However,
schematics does not support Terraform 1.2.x. Hence this commit removes the
features that would demand Terraform 1.2.x, there by enabling this product
to be deployable via schematics.
Also, use Ubuntu:20.04 as the OS version since 22.04 has new openssl algorithms that Terraform can't connect to.
When Schematics moves to 1.2.x in the future(Q3: 22 is tentative), we
can enable it back on.
 Resources: 20 added, 0 changed, 0 destroyed.

cos://eu-gb/custom-image-gb/vishwa-custom-image-ubuntu-20-04.qcow2


C40660: Removing comment and fixing links to avoid broken content on localization process

Hello, @RohitSharma-pnp,
Localization team has reported source content issue that causes localized version to have broken/different format compared to en-us version.
Description: Format comment is causing the strings to be left unblocked for localization when they should not be.
Please review and merge the proposed file change to fix to target versions. If you make related fix in another PR  then share your PR number so we can confirm and close this PR.
Many thanks in advance.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference-architectures/hybrid-networking/vpn.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference-architectures/hybrid-networking/vpn.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fix float type of matrix size in Python 3

In Python 3 division operator "/" produces "float" type output.
Use either conversion to 'int' type or '//' operator to get correct matrix size

Thanks for contributing.


[React Native] on mutation promise is not fullfilled

I'm trying to use urql to make a mutation but no result is returned
urql version : 2.1.1
Steps to reproduce
I have made a small example to reproduce the issue here : https://github.com/mlecoq/urql-issue
It is a simple login form that calls my login mutation to log in
Expected behavior
I expect to have a result after calling it
Actual behavior
I see with flipper that the network call is made but the promise never returns

I got the same issue after upgrading @urql/core to 2.4.0 in a Svelte app too. The following promise does not resolve or reject, although the request was actually sent and was completed (I saw that in the dev tools Network tab).
const { data, error } = await client
    .mutation(LoginDocument, { input })
    .toPromise()

Also, I've noticed this happens only when used with either cacheExchange or authExchange for some reason. Removing both makes the promise resolve (I use basically every other officially supported exchange in my setup, but none of them cause this issue).

Hey. we've identified the issue and will get a fix out asap

Thanks a lot for your speed !

No worries :+1: This should be resolved in<EMAIL_ADDRESS>Sorry for the regression 😄


Feature/#134027897 anticipated departure

Given that operations for a room that's a occupied by a "stayover" and a
room that's had it's reservation departed, it's useful to know in advance
which is happening.
Therefore the anticipated departure of the occupying reservation should be
stored along with the reference to the occupying reservation. This gets
set when the reservation occupies a room, and is maintained by the FRP
amendment rabbit event.
This is in order to fulfill https://www.pivotaltracker.com/story/show/134027897 because we have separate "stayover" and "checking out" categories of occupation. I suspect this will be useful in housekeeping as well.

Upon further reflection, I don't think this is a good idea. There's a lot of complexity in how we get the date, and it has some correctness concerns. I think the better approach might be to just grab the summaries of occupying reservations after we query the rooms, and let the client sort it out.


Add additional maintainers to sigstore-sig-clients

Right now, if I make a PR nobody can merge it :)
CC @bdehamer @woodruffw who were the attendees at the last meeting who were already in this file.

Summary

Release Note

Documentation

CC @sigstore/core-team


fix: update no-const-assign to error

eslint rule no-const-assign should be set to error ,due to this will be a runtime error.
refer: airbnb/javascript#559

@ianschmitz can you help me to review this pull request ?

Any responses? This option is profitable for preventing bugs


Unity launch arguments

It could be nice to have an option to specify launch arguments for different projects.
An use case could be having both a Pro and Personal projects on one machine and have Unity run the latter with '-force-free' argument

yeah, possible to add.. just need to think of some nice way to do it.
perhaps extra column after LastUpdated in the grid, with editable cell to type any arguments there?

Yeah why not - that could work. The only thing Im afraid of is that it will clutter the view a bit. Maybe a simple text input would be enough?
Regarding visuals, there is an option of replacing/moving the 'Modified' column or maybe adding an extra view (a popup) after right clicking/double clicking on a project?

ok have one idea for this now that we can resize window and columns..
Save launcharguments.txt (or similar) file under ProjectSettings/ folder,
Benefits of that:

otherwise we would end up orphaned project settings in memory (if the arguments are saved within launcher, and then project disappears from recent list, or would need to do house keeping to clear old ones)
launch arguments would be transferred with git/copying project
can build your own scripts to change launcharguments.txt if want to

now available in latest build,
Select project row, press F2 to edit arguments, then can launch with those
https://github.com/unitycoder/UnityLauncher/releases


C-style function calls

I want the syntax to look and feel more like C, JavaScript et all.
(add x y) -> add(x, y)
That's kind of a big deal, because it means handling left-recursive productions (like #22).

Done in #44


Hierarchy of tours in tour tree

What if we implement some way for certain tours to be "starred" by default or shown as higher importance? Or, a tagging/categorization mechanism? That way, a new dev onboarding to the codebase will know right away what is most important to view first.

We’ve already got #50 tracking the ability to group/categorized tours into directories. Should we make this specific to being able to star tours? If so, is the “Star” set by the owner of the repo, or is that something that the individual dev can do?

What if we implement default "stars" and then ability for personal customizations?

I just renamed this issue, since I think we should use it to track the ability to mark a tour in a codebase as being the “main” tour. This would have two key benefits:

When a new user opens the codebase, and are presented with the option to take a tour, they’ll be immediately guided to the the “main” tour as opposed to displaying the entire list of tours.

The “main” tour will be displayed uniquely in the  CodeTour tree (e.g. a unique icon)

also, don't let them open the other code tours till they are done with the main one. just kidding

Support for this has now been shipped. You can simply right-click a tour in the CodeTour tree and select Make Primary.


Hiring Dev to help me Setup Peatio on Azure

Hi,
I am looking for help setting up Peatio (muhammednagy fork with Ethereum) on Azure.
Additionally it would be nice if you could explain to me how to customize Peatio. I am a software developer myself, but I need an Expert to show me all the pitfalls and things I need to consider,
and help me hit the ground running.
The plan is to set up the basic framework within a weekend. After that you could possibly join as a consultant and I would of course pay you for your time and expertise.
Cheers

Can I email you?

Would love to hear from you!
Am 13.09.2017 23:15 schrieb "MaffeProg"<EMAIL_ADDRESS>
Can I email you?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/peatio/peatio/issues/604#issuecomment-329299061, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AeV_BgsY8sR62VYxlefwUC1T_lbi5Njhks5siEXVgaJpZM4PWPbC
.

If you have any Questions, shoot. Does not have to be Azure. Just something that scales well for the future.
Best Regards.

How can I send you a message?

Contact me at<EMAIL_ADDRESS>Thanks!
Am 14.09.2017 20:07 schrieb "MaffeProg"<EMAIL_ADDRESS>
How can I send you a message?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/peatio/peatio/issues/604#issuecomment-329563548, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AeV_BvX33k_v1N74N-fuj6RngYF9ETWSks5siWtUgaJpZM4PWPbC
.


All commands (especially hooks) should print logs before they start

When there are pre-connect and post-connect commands that run, the user does not get any indication of what things are running and when. Before each command runs, the program should print relevant log information so the user understands what's running. Child process outputs should generally not be printed, but it's important for users to know what was passed to exec, in case:

It's slow
It does something unexpected/malicious

Example output might be:
Running pre-connect hook (id=foo): echo foo bar baz

Rich text rendering could help this display clearly, e.g. bold/colored text. Or quotes/punctuation.

As these are diagnostic info and not output, they should go to stderr

Alternatively/additionally, commands could be allowed to specify a name to be displayed to user, similar to GHA.


feature request: .net standard

Any chance of the SDK becoming .net standard compatible?
I'd like to have the option of using it in .net core applications.

Thank you for your feature request. At the moment, there is no plan to publish .NET Standard compatible libraries for the SDK. But I'll definitely mention this to the team and if there is more interest regarding this, we will put this in the backlog.

We would also appreciate .Net Standard support.

Another vote for this, .net standard support would be very much appreciated

Another vote for .NET Standard would be great.
I'll have to use vmware/govmomi instead because I need xplat support.

As the future of .NET is Core, I'd like to mention that .NET 4.8 is the last version of .NET based on "Framework".
.NET 5 will be based on the .NET Core lineage.
https://devblogs.microsoft.com/dotnet/net-core-is-the-future-of-net/
I too am having to adopt govmomi because of the lack of .NET Core support.

this is not currently on the radar

how do we get it on the radar?

Just ran ApiPort.exe to verify compatibility. It seems as if at least vapi-runtime is almost .NET standard compliant:

Target type
Target member
Header for assembly name entries
.NET Core + Platform Extensions
.NET Core
.NET Standard + Platform Extensions
Recommended changes

T:System.Web.Script.Serialization.JavaScriptSerializer
T:System.Web.Script.Serialization.JavaScriptSerializer
vapi-runtime
Not supported
Not supported
Not supported
Use   JSON.NET instead.

T:System.Web.Script.Serialization.JavaScriptConverter
T:System.Web.Script.Serialization.JavaScriptConverter
vapi-runtime
Not supported
Not supported
Not supported

T:System.Web.Script.Serialization.JavaScriptConverter
M:System.Web.Script.Serialization.JavaScriptConverter.#ctor
vapi-runtime
Not supported
Not supported
Not supported

I've just tried to write a small program listing VMs in a .NET core 3 application.
Referencing the VAPI is simply done by referencing the prebuilt DLL files in this repo.
When starting the app, it complained about not being able to find System.Configuration.ConfigurationManager. This is no problem; that's easily installed via Nuget, and is available as a .NET standard compatible library.
The next error is harder though:

'Could not load file or assembly 'System.Web.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'. The system cannot find the file specified.'

System.Web.Extensions is not available for .NET core. So that is the part that will need to be rewritten.
Didn't really expect this to be a problem when logging in though. I was expecting some calls not to work correctly, but honestly, I wasn't expecting logging in to be a problem.

+1

@Rainmaker52 I have the same issue, did you solve it?

I have been able to reference the binaries (eg. VmWare.Vim.dll, VimService65.dll, vapi-client-bindings.dll, etc) in a .NET Core 3.1 project using Visual Studio 2019.  How well they work remains to be seen, but the project builds so I am optimistic.


Add jsdoc annotations and TypeScript type file

Adds JSDoc annotations and a .tsconfig that allows TypeScript to be run in it's "allowJS" mode and apply type and sanity checking to JavaScript code also. See Type Checking JavaScript Files.
I've been using TypeScript a lot lately and finding it very beneficial and wanted to get  those same benefits here but without converting the whole project to TypeScript.  It was rough at the beginning but now that this is finished I think it's about 80%-90% of the benefits without any of the TS compilation pipeline. The big difference in being JSDoc for adding typing information vs inline types with TypeScript.
Should be super helpful for maintainers using an editor with tight TypeScript integration and the improved docs/comments should help everyone else.

[x] Adds types/index.d.ts to NPM build (should be useful for TypeScript peeps)
[x] Improves documentation of many functions
[x] Adds JSDoc annotations to almost all functions
[x] Adds JSDoc type annotations to variables that can't be inferred
[x] Refactors a few smaller things to allow the TypeScript compiler to better infer what is happening (and usually also made the code clearer)

Other than the small bits of refactoring this is mostly a docs/comments update.

Apologies for coming in late on this one, but yes! :heart: thank you for this PR! This will make my life so much easier when reading through the code ☺

@allejo Glad to help.  It's even more amazing when writing it in a TS enabled editor. :)

Just a note that because the types refer to HTMLElement they don't compile out of the box in Node.
The community-maintained typings over at DefinitelyTyped work around this by defining an empty interface Node { }

I'm not sure exactly where we'd add that in our type file or how we'd export it... I'm still confused on the whole ambient module stuff at the moment.

why not "converting the whole project to TypeScript" ?

why not "converting the whole project to TypeScript" ?

JS is more popular (for those who sometimes contribute) and simply converting the whole project is not necessary.


include the event year in the JSON response

@PurpurTentakel97 requested this feature here: https://github.com/TechStreamConference/website-frontend/pull/42#discussion_r1771952688

corresponding issue https://github.com/TechStreamConference/website-frontend/issues/45


LoL crashes after patch 11.23

Apologies if this is not appropriate I was just wondering if it's on my end or the skin changer has not been updated yet. It crashes the game when injecting before and after the game starts, which it has not done before the most recent patch. Playing on EUNE live launcher.

It crashes if you try inject before game has loaded. Wait for fountain spawn first

On RU server works fine

I have since reinstalled the game which seems to have fixed the issue of crashing and the program works flawlessly now.


Somebody else is on my

Type: Feature Request
Somebody else is on my account using my stuff
VS Code version: Code - Insiders 1.92.0-insider (89690b9d76a88fab676d26651134a1ffa4562732, 2024-07-19T05:03:57.939Z)
OS version:
Modes:

Hi @kevindelgado2467,
Thank you for contacting the VS Marketplace Team. To assist you effectively, could you kindly provide additional details about your concern? This will help us investigate thoroughly.
Regards,
VS Marketplace Team

Yes, somebody that wasn’t me went in and reported my code spaces. They went
in and changed everything. Somebody basically hacking into my account and
reported all the spaces and text messages. Then there were somehow able to
sync on my settings and change all of my hard drives and they somehow
logged into my iCloud from there. It was a completely different device. I
never turned on sync settings.
On Mon, Jul 22, 2024 at 6:20 AM Neelima @.***> wrote:

Hi @kevindelgado2467 https://github.com/kevindelgado2467,
Thank you for contacting the VS Marketplace Team. To assist you
effectively, could you kindly provide additional details about your
concern? This will help us investigate thoroughly.
Regards,
VS Marketplace Team
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vsmarketplace/issues/966#issuecomment-2242945904,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJ7JKZK5BGBSSIF2GMK2GCTZNUBKRAVCNFSM6AAAAABLGUDREGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBSHE2DKOJQGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @kevindelgado2467,
Thanks for sharing the details, could you please reach out to us through our support alias<EMAIL_ADDRESS>to address this issue promptly.
Regards
VS Marketplace Team


user check method with rails self

bool 类型字段，自带 method?

Current coverage is 96.66% (diff: 100%)

Merging #740 into master will not change coverage

@@             master       #740   diff @@
==========================================
  Files           136        136
  Lines          5215       5215
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           5041       5041
  Misses          174        174
  Partials          0          0

Powered by Codecov. Last update ee989fe...53490d9


Update Pages “index”

Automatically generated by Netlify CMS

👷 Deploy Preview for nervous-engelbart-a15395 processing.
🔨 Explore the source changes: da07bb78ea82d5f259586ef99a9bb5aaeb10cbc5
🔍 Inspect the deploy log: https://app.netlify.com/sites/nervous-engelbart-a15395/deploys/61aac4303c13bf00074372e7


Per-folder remote configuration in workspace

Would it be possible to support mixed remote authorities in a workspace? I'd like to have a single, multi-root workspace with one folder from WSL and one from SSH, for example.

No that's not possible.
Right a remote window connects to one remote server. It would be major architectural change to run multiple remote extension hosts.

I know, just didn't know if we are tracking it somewhere. I don't think we are considering this to be in scope though.


Fix Compiler Warnings

Fixing the warnings generated from yarn compile

Pull Request Test Coverage Report for Build 2912

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 2907:
0.0%

Covered Lines:
633

Relevant Lines:
633

💛  - Coveralls


🐛 BUG: (macOS dev beta) ERR_RUNTIME_FAILURE error when using wrangler dev

FYI, I already made a post in discord but no one was answering
Which Cloudflare product(s) does this pertain to?
Wrangler
What version(s) of the tool(s) are you using?
3.64.0 [Wrangler]
What version of Node are you using?
22.0.0(now 22.4.1 since i upgraded it, but not fixed), but prefer bun 1.1.20
What operating system and version are you using?
macOS Sequoia 15 dev beta 3
Describe the Bug
Observed behavior
I get ERR_RUNTIME_FAILURE error when using wrangler dev

Expected behavior
Works without any error, ex) apply db, open dev server, etc.
Steps to reproduce

A minimal working subset of your worker code

import { Hono, type Context } from 'hono'
import { authHandler, initAuthConfig, type AuthConfig } from '@hono/auth-js'
import { PrismaAdapter } from '@auth/prisma-adapter'
import { PrismaClient } from '@prisma/client'
import { PrismaD1 } from '@prisma/adapter-d1'
import type { D1Database } from '@cloudflare/workers-types'
import users from './routes/users'

declare let env: {
  DB: D1Database
}

const app = new Hono()

const adapter = new PrismaD1(env.DB)
const prisma = new PrismaClient({ adapter })

function getAuthConfig(c: Context): AuthConfig {
  return {
    // auth config...
  }
}

app.use('*', initAuthConfig(getAuthConfig))
app.use('/api/auth/*', authHandler())
app.route('/api/users', users)

export default app

A minimal working subset of your wrangler.toml

name = "[REDACTED]"
main = "api/index.ts"
compatibility_date = "2024-03-20"
compatibility_flags = ["nodejs_compat"]

[[d1_databases]]
binding = "DB"
database_name = "[REDACTED]"
database_id = "[REDACTED]"
preview_database_id = "DB"

Commands used to start your local dev server, including custom env and cli args

bunx wrangler dev # or npx wrangler dev, results are same

Steps to be performed in the browser, curl commands, or a test we can run that reliably fails (at least a percent of the time)

use the command wrangler dev, that's it

A git repo we can clone and run a test suite on, or which has a README with step-by-step instructions, is even better. In this case, please use the field below to provide a link to the minimal repro.
Please provide a link to a minimal reproduction
No response
Please provide any relevant error logs
# when using wrangler dev

 ⛅ wrangler 3.64.0
-------------------

Using vars defined in .dev.vars
Your worker has access to the following bindings:
- D1 Databases:
  - DB: [REDACTED] (DB), Preview: (DB)
- Vars:
  - GOOGLE_ID: "(hidden)"
  - GOOGLE_SECRET: "(hidden)"
  - AUTH_SECRET: "(hidden)"
  - TEBI_ACCESS_ID: "(hidden)"
  - TEBI_ACCESS_SECRET: "(hidden)"
⎔ Starting local server...
╭────────────────────────────────────────────╮
│ [ open a[d open D[ turn off [ clear  [ to  │
│ ]browser, vtools,]local mode]console,]exit │
│                                            │
╰────────────────────────────────────────────╯
/Users/helloyunho/Projects/[REDACTED]/node_modules/wrangler/wrangler-dist/cli.js:29765
            throw a;
            ^

MiniflareCoreError [ERR_RUNTIME_FAILURE]: The Workers runtime failed to start. There is likely additional logging output above.
    at #assembleAndUpdateConfig (/Users/helloyunho/Projects/[REDACTED]/node_modules/miniflare/dist/src/index.js:9178:13)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Mutex.runWith (/Users/helloyunho/Projects/[REDACTED]/node_modules/miniflare/dist/src/index.js:3521:16) {
  code: 'ERR_RUNTIME_FAILURE',
  cause: undefined
}

Node.js v22.4.1

Still persists on macOS 15 dev beta 6...

Hello brother, if you still have this issue on the Windows system, please refer to this link for a solution. I have already solved the problem of not being able to run npm run dev or wrangler dev using this method. If you also get a solution as a result, I will be very happy for you. We can become good friends.
link: https://github.com/cloudflare/workers-sdk/issues/6170#issuecomment-2239513454
summarize: Update To Microsoft Visual C++2015-2022 Redistributable (x64) -14.40.33810


Installing subject-package with invenioRDM crashes

1, Create invenio-Instanz
mkdir inv7
cd inv7
python3 -m venv venv
source venv/bin/activate
python -m pip install --upgrade pip
pip install invenio-cli
pip install Flask-Debugtoolbar
invenio-cli --version
invenio-cli check-requirements
invenio-cli init rdm -c v7.0

Add invenio-subjects-gnd package in Pipfile

vi Pipfile

...

[packages]
invenio-app-rdm = {extras = ["postgresql", "elasticsearch7"], version = "~=7.0.0"}
uwsgi = ">=2.0"
uwsgitop = ">=0.11"
uwsgi-tools = ">=1.1.1"

# Subjects gnd.
invenio-subjects-gnd = "*"

...

Creates (in step 3) the following Pipfile.lock entry:
        "invenio-subjects-gnd": {
            "hashes": [
                "sha256:93842d6837f47a590917961607928d79bc301021534db4b30df5d4817958c25d"
            ],
            "index": "pypi",
            "version": "==2022.1.31"
        },

Container install

invenio-cli packages lock
invenio-cli containers build
invenio-cli containers setup
invenio-cli containers start

Failes at invenio-cli containers setup

(venv) 13:47:31 chriz@mr09202018n:~/rz/code/invenio/inv7/inv7$ invenio-cli containers setup
Setting up services with force False, demo data True and stop after setup False...
Making sure containers are up...
redis up and running!
postgresql up and running!
es up and running!
Containers started and healthy.
Creating database...
Creating database postgresql+psycopg2://inv7:***@db/inv7
Creating all tables!
INFO  [alembic.runtime.migration] Context impl PostgresqlImpl.
INFO  [alembic.runtime.migration] Will assume transactional DDL.
Created all tables!
Creating files location...
Location default-location /opt/invenio/var/instance/data as default True stored in database
Creating admin role...
Role "admin" created successfully.
Assigning superuser access to admin role...
Creating indices...
Creating indexes...
requests  [####################################]  100%
Putting templates...
Updating service setup status (True)...
Service setup status updated (new value True).
Creating fixtures...
Failed to setup services.
Output: Creating required fixtures...
Traceback (most recent call last):
  File "/usr/local/bin/invenio", line 8, in <module>
    sys.exit(cli())
  File "/usr/local/lib/python3.8/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/flask/cli.py", line 600, in main
    return super().main(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.8/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python3.8/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python3.8/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python3.8/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/click/decorators.py", line 21, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/flask/cli.py", line 444, in decorator
    return __ctx.invoke(f, *args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python3.8/site-packages/invenio_rdm_records/cli.py", line 48, in create_fixtures
    FixturesEngine(system_identity).run()
  File "/usr/local/lib/python3.8/site-packages/invenio_rdm_records/fixtures/__init__.py", line 32, in run
    PrioritizedVocabulariesFixtures(
  File "/usr/local/lib/python3.8/site-packages/invenio_rdm_records/fixtures/vocabularies.py", line 202, in load
    self.load_from_extensions()
  File "/usr/local/lib/python3.8/site-packages/invenio_rdm_records/fixtures/vocabularies.py", line 233, in load_from_extensions
    self.load_vocabularies(filepath)
  File "/usr/local/lib/python3.8/site-packages/invenio_rdm_records/fixtures/vocabularies.py", line 253, in load_vocabularies
    self._loaded_vocabularies = fixture.load(
  File "/usr/local/lib/python3.8/site-packages/invenio_rdm_records/fixtures/vocabularies.py", line 313, in load
    entry.load(self._identity, ignore=ids, delay=self._delay)
  File "/usr/local/lib/python3.8/site-packages/invenio_rdm_records/fixtures/vocabularies.py", line 333, in load
    for data in self.iterate(ignore=ignore):
  File "/usr/local/lib/python3.8/site-packages/invenio_rdm_records/fixtures/vocabularies.py", line 403, in iterate
    yield from create_iterator(filepath)
  File "/usr/local/lib/python3.8/site-packages/invenio_rdm_records/fixtures/vocabularies.py", line 47, in __iter__
    data = yaml.safe_load(fp) or []
  File "/usr/local/lib64/python3.8/site-packages/yaml/__init__.py", line 125, in safe_load
    return load(stream, SafeLoader)
  File "/usr/local/lib64/python3.8/site-packages/yaml/__init__.py", line 81, in load
    return loader.get_single_data()
  File "/usr/local/lib64/python3.8/site-packages/yaml/constructor.py", line 49, in get_single_data
    node = self.get_single_node()
  File "/usr/local/lib64/python3.8/site-packages/yaml/composer.py", line 36, in get_single_node
    document = self.compose_document()
  File "/usr/local/lib64/python3.8/site-packages/yaml/composer.py", line 55, in compose_document
    node = self.compose_node(None, None)
  File "/usr/local/lib64/python3.8/site-packages/yaml/composer.py", line 82, in compose_node
    node = self.compose_sequence_node(anchor)
  File "/usr/local/lib64/python3.8/site-packages/yaml/composer.py", line 111, in compose_sequence_node
    node.value.append(self.compose_node(node, index))
  File "/usr/local/lib64/python3.8/site-packages/yaml/composer.py", line 84, in compose_node
    node = self.compose_mapping_node(anchor)
  File "/usr/local/lib64/python3.8/site-packages/yaml/composer.py", line 127, in compose_mapping_node
    while not self.check_event(MappingEndEvent):
  File "/usr/local/lib64/python3.8/site-packages/yaml/parser.py", line 98, in check_event
    self.current_event = self.state()
  File "/usr/local/lib64/python3.8/site-packages/yaml/parser.py", line 438, in parse_block_mapping_key
    raise ParserError("while parsing a block mapping", self.marks[-1],
yaml.parser.ParserError: while parsing a block mapping
  in "/usr/local/lib/python3.8/site-packages/invenio_subjects_gnd/vocabularies/subjects_gnd.yaml", line 7804, column 3
expected <block end>, but found '<scalar>'
  in "/usr/local/lib/python3.8/site-packages/invenio_subjects_gnd/vocabularies/subjects_gnd.yaml", line 7806, column 32

Updated the file and tested the complete recipe locally. All working now!