Infer shape of source space from target space

Hi,
I'm wondering if it's possible to allow transformations of coordinates when the target space has a shape but the source space doesn't?

Hi @vigji,
Thanks for the response! In my workflow I save the coordinates of points in the "sal" space and sometimes I need to conversion the coordinates to the space of the mpin_zfish_1um atlas. When I initialize the source space for the conversion I need to calculate the shape of the atlas stack after transforming to the source space. I think it'd be convenient to have this new feature since the atlas space already contains the shape information. I also see why it could be dangerous, so I think that having a keyword argument for this feature and making it not the default behavior would be a good option.

I've added a test with axes flipping and reordering.

Awwwwwesome!  Thank you very much @kclamar !

will release a new version with this


cargo test fails with archive from crates.io

---- src/lib.rs -  (line 38) stdout ----
thread 'src/lib.rs -  (line 38)' panicked at 'test executable failed:

thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 2, kind: NotFound, message: "No such file or directory" }', src/libcore/result.rs:997:5
note: Run with `RUST_BACKTRACE=1` environment variable to display a backtrace.

', src/librustdoc/test.rs:356:17
note: Run with `RUST_BACKTRACE=1` environment variable to display a backtrace.

---- src/lib.rs -  (line 18) stdout ----
thread 'src/lib.rs -  (line 18)' panicked at 'test executable failed:

thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Os { code: 2, kind: NotFound, message: "No such file or directory" }', src/libcore/result.rs:997:5
note: Run with `RUST_BACKTRACE=1` environment variable to display a backtrace.

', src/librustdoc/test.rs:356:17

I'm packaging gif and we always run cargo test to ensure that library works fine in distribution. But unfortunately gif one fails (I assume because you exclude test data). Could you either include it or make it to not fail in case data is missing? Thanks.

This reminded me, I've just opened a PR on image that could ensure that tests can be ran with the archives from crates. You might want to have a look, and I'll see when I get around to porting it to gif for additional data points of usage.

Maybe do the opposite, and remove tests from the release tarball?
Cargo doesn't support testing of packaged crates nor crates.io dependencies. IMHO the published tarballs are not meant to run tests, and stripping tests will save majority of users a bit of bandwidth and storage.

I've moved some tests out, but that wasn't sufficient. There are still tests inline in .rs files that are published, so this is WONTFIX.


Fixing page title

https://github.com/MicrosoftDocs/office-docs-powershell/issues/4815

Docs Build status updates of commit 9e176ba:
:clock10: Full build: the average full build time is 12 min(s) 3 sec(s), this is based on the last 1000 build(s) for this repository.

https://github.com/MicrosoftDocs/office-docs-powershell/issues/4816
https://github.com/MicrosoftDocs/office-docs-powershell/issues/4817

Docs Build status updates of commit 9e176ba:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Get-CsOnlineVoiceRoute.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Get-CsOnlineVoiceRoutingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

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.

Docs Build status updates of commit d017c14:
:clock10: Full build: the average full build time is 12 min(s) 3 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit d017c14:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Get-CsOnlinePstnUsage.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Get-CsOnlineVoiceRoute.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Get-CsOnlineVoiceRoutingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

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.

Docs Build status updates of commit f315710:
:clock10: Full build: the average full build time is 12 min(s) 5 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit f315710:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Get-CsOnlinePstnUsage.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Get-CsOnlineVoiceRoute.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Get-CsOnlineVoiceRoutingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

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.

Docs Build status updates of commit e6d5491:
:clock10: Full build: the average full build time is 12 min(s) 5 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit e6d5491:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Get-CsOnlinePstnUsage.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Get-CsOnlineVoiceRoute.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Get-CsOnlineVoiceRoutingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

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.

Docs Build status updates of commit 16a55ec:
:clock10: Full build: the average full build time is 12 min(s) 5 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 16a55ec:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Get-CsOnlinePstnUsage.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Get-CsOnlineVoiceRoute.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Get-CsOnlineVoiceRoutingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

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.


[main] Upgrade to latest dependencies

fix wrong version on main (https://cloud-native.slack.com/archives/C04LY4Y3EHF/p1729776009449449?thread_ts=1729763678.995479&cid=C04LY4Y3EHF) -ReToCode
/cc knative-extensions/eventing-writers
/assign knative-extensions/eventing-writers
Produced by: knative-extensions/knobots/actions/update-deps

/lgtm
/approve


Plant sends error message but generally works (on pl-PL locale)

(I am hosting the bot on my PC)
When I use >plant command the bot says:
"I can't tell you if the command is executed, because there was an error printing out the response. Key 'games_planted' is not properly formatted. Please report this."
But after this it says that I can pick it up with >pick
Console doesn't show any errors (everything is white)
I checked and it's present on locale pl-PL but on en-US it doesn't send this

It means the response string is missing. So games_planted string isn't in the language response string. You can add the missing string or wait until it fixed.

It means it's there but has messed up placeholders. I will take a look.

Ok, fix is applied, but it will be available in 1.4


Add trigger for tab click client event

Add a trigger for the tab click event.  This is useful in cases where elements within the tab div need redrawing or resizing after the display attribute has been changed.

How would this be implemented in markup? Say, for example, I want to re-render a calendar (e.g. fullCalendar) using a "render" method. How would I write that using your trigger?

This should do the trick:
$('ul.tabs').bind('tab_change', function(event, content) {
  if (content && content.hash) {
    console.log('now on tab ' + content.hash);
  }
});

With the placement of the trigger in tabs.js, your code has the guarantee that it will execute after the tab section itself has finished coming into view.

I'm not using jquery at all but would like to be able to supply an onClick option to M.Tabs.init. i'm new to this library but it doesn't make much since to put a handler onShow because do things beforehand and I think this would be much more helpful when working in React. Any suggestions?


Make compile shader function to use struct instead long parameter list

I think having a single struct passing to ShaderLanguage::compile and ShaderLanguage::complete is better than a long parameter list - it's easier to debug, fix and further expand.

I think it was an interesting refactoring, did you change your mind?

I thought it was forgotten and lost interest in it...


Enable Numba RuntimeLinker

Ref: https://github.com/xnd-project/rbc/issues/33

Done in https://github.com/xnd-project/rbc/pull/24


Effective Interest Rate Calculator

This Effective Annual Interest Rate Calculator to calculate the Effective Annual Interest Rate based on the input values of Interest Rate, Monthly Compounding, Quarterly Compounding, Semi-Annually Compounding and Annual Compounding Period on loan

Go ahead
Deadline 29 April,2021


Update to 1.15.1 (synced from<EMAIL_ADDRESS>

cilium connectivity test shows the same result as for the previous version.

cilium connectivity test shows the same result as for the previous version.

@mcharriere PTALA

the changelog! don't forget the changelog :)


Tabs active class

When tabs with active class loads as shown, the active applies to left aligned location by default. After clicking a tab, active applies correctly.

The issue here is because you still have s3 defined on your tabs, so the indicator is adjusting to the initial width. If you update those cols to have s4 it should work.

Doh! Thanks.


Having the same problem !!!

Having the same problem !!!
Try cmd:

echo fs.inotify.max_user_watches=524288 | sudo tee -a /etc/sysctl.conf && sudo sysctl -p

It worked for me !!!
Hope your problem would be resolved by it
Also tried it,now getting 'sudo' command is not recognized is an internal or external command.

same problem as what?
Also you shouldn't have to use sudo with npm almost ever. If you do, you should fix your install as that can be scary

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: react-scripts start
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.

Can you give more information please like versions?

@vinitsinghbaghel
npm ERR<EMAIL_ADDRESS>start: react-scripts start
have you installed the wrong package by typoing the issue?
does npx create-react-app@latest your-project-name fix it?


DateTimePicker dispaly error

Describe the bug
Set DateTimePicker's SelectedDateTime, and open the select dialog, the dialog display error datetime.
To Reproduce
Steps to reproduce the behavior:

Create a xaml <hc:DateTimePicker x:Name="dt"/>
Add code behind dt.SelectedDateTime = new DateTime(2021, 3, 20, 14, 35, 59);

Expected behavior
Show the datetime given in code behind.
Screenshots

Environment (please complete the following information):

.net: core 3.1
IDE vs 2019 latest
HC Version 3.2.0

Additional context
Add any other context about the problem here.

after 8813fddf you can use DisplayDateTime


x/build: use AWS EC2 Mac instances for macos builders

For context: https://aws.amazon.com/blogs/aws/new-use-mac-instances-to-build-test-macos-ios-ipados-tvos-and-watchos-apps/

The instances feature an 8th generation, 6-core Intel Core i7 (Coffee Lake) processor running at 3.2 GHz, with Turbo Boost up to 4.6 GHz.

EC2 Mac instances with the Apple M1 chip are already in the works, and planned for 2021.

Related: #40762, #26530, #39782.

Can you clarify why those are better then the current mac builders?

Actually, I failed to read this part of the announcement:

The instances are launched as EC2 Dedicated Hosts with a minimum tenancy of 24 hours.

This doesn't work with the way we do our builds, so I'll go ahead and close this.


#121: fixed umount error by removing duplicate umount call

This change is

Coverage decreased (-6.5%) to 54.181% when pulling d365b356c9f48a8c8d6f15f9e8848378ab64368b on feature/fix_umount into 9b68138017d040ab5a6210cfa8dcc024afca6eb9 on dev.

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable


Second required query parameter falsely identified as missing

To Reproduce

Create OAS3 spec with a path that has two required query params.
Make a request and provide both params

openapi: 3.0.0
servers:
  - url: 'http://localhost:3000'
paths:
  /path:
    get:
      responses:
        '200':
          description: OK
          content:
            application/json:
              schema:
                type: object
              examples: {}
      parameters:
        - schema:
            type: string
          in: query
          name: a
          required: true
        - schema:
            type: string
          in: query
          name: b
          required: true

curl <IP_ADDRESS>:4010/path?a=123&b=12

Expected behavior
Should return 200 OK
Actual
Returns error.
{
   "type" : "https://stoplight.io/prism/errors#UNPROCESSABLE_ENTITY",
   "validation" : [
      {
         "code" : "required",
         "severity" : "Error",
         "message" : "should have required property 'b'",
         "location" : [
            "query"
         ]
      }
   ],
   "detail" : "Your request body is not valid and no HTTP validation response was found in the spec, so Prism is generating this error for you.",
   "title" : "Invalid request body payload",
   "status" : 422
}

Screenshots
If applicable, add screenshots to help explain your problem.
Environment (remove any that are not applicable):

Library version: [e.g. 3.0.0]
OS: [e.g. Windows 7]
Browser: [e.g. Chrome 61]

Additional context
Add any other context about the problem here.

Again, when using curl you need to quote the url with the query string.


Different number of cells and bbox

Hi, thanks for sharing your work. I'm facing a problem that doesn't seem to be tackled by the library, even if kind of expected: what if the table structure extraction model outputs a different number of non-empty html cells compared to the table cell bbox detection model?
Here an example:

Extracted table:
['
',
 '
',
 '[]	',
 '[]	',
 '
',
 '
',
 '
',
 '
',
 '[]	',
 '[]	',
 '[]	',
 '
',
 '
',
 '[]	',
 '[]	',
 '[]	',
 '
',
 '
',
 '[]	',
 '[]	',
 '[]	',
 '
',
 '
',
 '[]	',
 '[]	',
 '[]	',
 '
',
 '
',
 '[]	',
 '[]	',
 '[]	',
 '
',
 '
',
 '[]	',
 '[]	',
 '[]	',
 '
',
 '
',
 '[]	',
 '[]	',
 '[]	',
 '
',
 '
',
 '[]	',
 '[]	',
 '[]	',
 '
',
 '
']

-> 26 non-empty cells
Detected bbox:

-> 27 bbox

-> html table is shifted of 1 cell
Did you encounter this problem before? Did you have some smart way to solve this problem?

is there any solution to solved this case? @valedica-core
ive exactly same case

Thank you both for the question! I believe the above example is OOD for our training set as the spanning cell text is not aligned with any column in the header. A high-quality training dataset with abundatnt tables of various styles, colors, and designs will be ideal to resolve this.
How to perfectly align the cells predicted by the bbox branch and the structure branch is an open question now. Currently, UniTable divides the table parsing as structure+bbox+cell content. I hope there's a new way to divide and parse the table so that UniTable and the new method can double-check each other's outputs. Feel free to share your thoughts on this @valedica-core @ridhoalattas !

Thanks @ShengYun-Peng for the reply and again for this library.
I saw some other authors (2023 Nam Tuan Ly et al.) have proposed multi-task models to align a bit more the sub tasks, but I'd say the beauty of this library's approach is the simplicity. I wonder then why not employing a single decoder model that outputs both html tags and bbox at the same time, is it feasible?
About how I tried to fix the current issue, I first tried to frame it as a minimalization problem: find the best assignment of bboxes to html cells -possibly by skipping cells or splitting a cell to fit more than one box- that minimized the number of misplaced bboxes, where misplaced means that left-right or above-below relationships between bboxes are broken once you assign them to cells. However, with big tables and a lot of empty cells, the problem seems untreatable.
The simpler and drastic approach I ended up with is to just use the bboxes, drawing the axes of the grid using local minima of number of intersections with bboxes. It has some problem with spanning cells or columns, but I'm not that interested in it since my goal is to have lossy markdown representation as output.

Thanks @valedica-core ! It seems like table cell alignment is a good research direction. We tried GPT-4o for spanning cells, and UniTable still did better on table structure understanding. https://x.com/RealAnthonyPeng/status/1790431978829087123


:sparkles: Implemented basic score calculation

Please check if the PR fulfills these requirements

[x] Tests for the changes have been added (for bug fixes / features)
[x] PR title follows the guidelines defined in  https://github.com/ossf/scorecard/blob/main/CONTRIBUTING.md#pr-process

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Feature

What is the current behavior? (You can also link to an open issue here)
There isn't a score after the run.

What is the new behavior (if this is a feature change)?

Implemented basic score calculation

This score will be used for generating badges.

This score calculation can be changed when tiers are done.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
None

Other information:
None

This is the foundation for generating badges #271  https://github.com/ossf/scorecard/pull/301


question while loading pretrained models

Thanks for your code @MinkaiXu  a lot : ), it's cool.
I met a problem in loading pretrained models, the error message is as follows:
Traceback (most recent call last):   File "eval_analyze.py", line 198, in <module>     main()   File "eval_analyze.py", line 127, in main     with open(join(eval_args.model_path, 'args.pickle'), 'rb') as f: FileNotFoundError: [Errno 2] No such file or directory: 'outputs/qm9_p/args.pickle'
I'm confused how to solve it. Hope you'll help me with it!

simply unzip the downloaded file will work.


Ensure that we always change options, even when unsubbed

For #374. If we go from skip -> no-skip w/ new options, we won't have a subscription, but we will have an observable that we need to update with the new options.

Hold on merging this due to further investigation needed: https://github.com/apollostack/react-apollo/pull/376#discussion_r93367673

Hmm, this is more complicated than I feared.
The problem referenced here is due to the fact that:

We unsubscribe (when skipping).
We change the variables, which triggers a fetchQuery here
Immediately, we re-subscribe, which triggers the ObservableQuery to re-fetch here

Because 2+3 happen instantaneously in sequence, both queries hit the server. I'm not sure what we could do differently; if we reversed the order I would expect the same problem, except that the re-subscription would trigger the old query.
I guess this behaviour is better than the broken behavior it is fixing, so I'll merge it, but I wonder if there's any thing we can do about it. Thoughts @helfer, @jbaxleyiii?

@tmeasday sorry for the late response on this one. I'm guessing our discussion around not fetching when setVariables is called but there's no subscriber is because of this very issue?

@helfer, yes, I should have added a note here that I opened https://github.com/apollostack/react-apollo/issues/381 to deal with this.


Capital com multiple instances

Even if  we  use  1  api with  multiple   subacounts or multiple  api  each  one  with  his  coresponding subaccount , we  enter in  a race  condition  for  prefered  account , that  could be  changed  from  anywhere  and  anytime   by  another  class instance
We   must  implement  lock   and  release   to  acquire   preferred  account,   on  order  execution  based on  subaccount  id

So to force/predefine a subaccount as self.account for that client?

My  aproach is   :
Self.accountId   set
in  a   create position  method  ,  (that require  specific  accountId ) i lock  the  account  switch ,
check  if  active account ==   to accountId
If  not   ,  switch  to  that  accountId   ,  make it  preferred
Place the order
Release  the  lock
.. not  the  optimal  way , ( it  has  network  request  , if  accounts  dont  match ) ..
Maybe  we   get some  better  response  from  capital  team ,  or they force bound   some  api  account to   certain  accountId ..
Everything  is  fine  in   a  single  api ,   single  subaccount.   Else I will check that in coming days

@Kamvia i did a quick pr and added a new settting default_account.

Nope.
We must  use  Sequence IDs and Semaphores
Sequence IDs: Labeling each event with a unique sequence ID can help you track and manage events in the correct order, ensuring that each operation is handled properly.
Semaphores: Semaphores can be used to control access to a resource, allowing a set number of threads to access a shared resource simultaneously, which helps prevent race conditions.

@Kamvia not sure to follow. Shoot a pr and we can go further.


$$html$$ output size problem

Hi there,
When trying to run the following IJavascript example
$$html$$ = "

";

I get

This is what happens if I add something in the div like:
$$html$$ = "
test
";

The next $$svg$$ example works but it looks like the canvas size is too big:

Cheers!
Jun

@lgeiger This issue isn't specific to IJavascript. Here's a snippet to reproduce it using IPython:
import IPython.display
IPython.display.HTML("

")
IPython.display.HTML("
test
")
IPython.display.HTML("<svg><rect width=80 height=80 style='fill: orange;'/></svg>")

@jmatsushita Thank you for reporting this issue. As a workaround, you can do this:
$$.html("

");

Although I haven't advertised this yet, the latest version of IJavascript provides new functionality to customise output. See here and here.

@n-riesco thanks for the IPython snippet. It's an issue with the styling of hydrogen. I'll take a closer look tomorrow.

@lgeiger Thanks the css fix works!
@n-riesco The new syntax is great. And I'm excited about $$.async() !!! I've been wondering when I'd run into a library that didn't provide synchronous versions :)

I couldn't fix the SVG example with fiddling around in the dev console. I don't know why this is happening.

@lgeiger If it isn't easy, I think there is no much point fix the SVG case now (we will have to find another solution when we use webviews).

Setting height and width on the svg element does it!
$$.svg("<svg height=80 width=80><rect width=80 height=80 style='fill: orange;'/></svg>")

@jmatsushita Thanks! That makes sense.


Pseudonetcdf engine with Mfdataset using multiple csv files

Hello,
I am attempting to open about 50 csv, each represents monthly data and I would like to open them all in xarray.
unfortunately it is not working and I have included the code and error, any help would be appreciated it
 import xarray as xr

xr.open_mfdataset('C:/Users/*.csv', concat_dim="time",
                  data_vars='minimal', coords='minimal', compat='override',engine='pseudonetcdf',
                  backend_kwargs={'format': 'csv'})

```
DtypeWarning: Columns (12) have mixed types. Specify dtype option on import or set low_memory=False.
  file, _ = self._acquire_with_cache_info(needs_lock)
```

This looks like you need to specify some more backend_kwargs. https://github.com/barronh/pseudonetcdf/issues might be a better place to ask this question.

Thank you !!!! i'll post it


what about backend storage on mysql? over handlersocket or plain sql :) ///∞

Hi @spx, I just want to let ledisdb keep simple and don't rely on outer service like MySQL.
If MySQL is down, all services will not work too and I don't think using MySQL is faster than other local backend storage like RocksDB, but we need a benchmark for it. :-)
Btw, supporting MySQL may be easy (I prefer plain sql and don't trust handlersocket) but I have no time now, sorry.


Question about scipy.interpolate.interp1d

I want to do linear scaling , for example:
There is a sequence   A=[1,2,3,4],
So the length of A is 4, I want to do linear scaling to make a new sequence B
(Compare to A ,the length of B maybe 0.5times or 1.1times  or 1.5,times   and so on....)
For example: Suppose the scaling rate is  7/4 , the length of B is  4 * ( 7/4) = 7
Using linear scaling, the result of B sequence must be B=[1, 1.5 , 2 , 2.5, 3, 3.5 ,4]
How to use scipy.interpolate.interp1d to do that ?  Or is there any other function could do that?
Input A to get output B  (with any scaling rate )
Thank s a lot!!

Hi @hsingchenlee, we are using this repository exclusively for code problems, bugs, feature requests etc. You would attract many more people if you use stackoverflow or SciPy mailing list for usage questions
Here is a related one after a 5 second googling
https://stackoverflow.com/questions/29085268/resample-a-numpy-array


numix-icon-theme-square: 22.08.15 -> 22.09.04

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for numix-icon-theme-square is: "Numix icon theme (square version)"
meta.homepage for numix-icon-theme-square is: "https://numixproject.github.io"
meta.changelog for numix-icon-theme-square is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 22.09.04 in filename of file in /nix/store/dd6rla3z1j3a2624x8adlwhqcijwhkqc-numix-icon-theme-square-22.09.04

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
numix-icon-theme-square

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/dd6rla3z1j3a2624x8adlwhqcijwhkqc-numix-icon-theme-square-22.09.04 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A numix-icon-theme-square https://github.com/r-ryantm/nixpkgs/archive/b040cb633ab701f02e9ece440206289b775210c3.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/dd6rla3z1j3a2624x8adlwhqcijwhkqc-numix-icon-theme-square-22.09.04
ls -la /nix/store/dd6rla3z1j3a2624x8adlwhqcijwhkqc-numix-icon-theme-square-22.09.04/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

numix-icon-theme-square

Maintainer pings
cc @romildo for testing.

Result of nixpkgs-review pr 190320 run on x86_64-linux 1

1 package built:

numix-icon-theme-square


shopware.cache.redis_prefix should not be used for cart

PHP Version
8.3
Shopware Version
6.6.4
Affected area / extension
Platform(Default)
Expected behaviour
Carts in Redis are not prefixed by shopware.cache.redis_prefixso that the carts are not lost after changing the prefix.
Actual behaviour
Carts in Redis are prefixed by shopware.cache.redis_prefix
How to reproduce

set redis for cart
set shopware.cache.redis_prefix to any value
add product into cart in storefront
change shopware.cache.redis_prefix
see cart is empty

[public] Shopware automated response: This issue is linked to the internal issue: NEXT-38084
[created from NEXT-38084, comment 490988]

As @shyim found out, it could be okay to just remove the prefix from RedisConnectionFactory while it is already set for caches by CompilerPass.

As analysed and discussed further with @shyim this is really intended.
Use SHOPWARE_CACHE_ID instead!


Code organization

Hi!
So currently all tclsh-wrapper commands are in bag-of-procs namespace.  There are only so many, the complexity is not insurmountable or anything, but to dip your toe into tclsh-wrapper is swim in it's pool.  There are two obvious 8.6 solutions: child namespaces and tcloo.
Although I want to justify this at length, I won't, instead when next you hear from me, I am going to present a tcloo-ified tclsh-wrapper, and we all can discuss it as a proposal.  If you prefer to use namespaces, I don't mind at all if this effort is ultimately in vain :)

Thanks.
I don't prefer namespaces.  Feel free to apply tcloo to the existing code.
I'm not familiar with tcloo, though, I don't mind learning it.

closing this


TypeError: Cannot read property 'sprop-parameter-sets' of undefined

help! when i connected to a honeywell ip camera, an error occurred as follow:
Client paused
Client started
[mse] Media source opened: open
[info] ------------- Info ---------------------
[info] License expires at : Invalid expiration time
[info] Requested domain : <IP_ADDRESS>
[info] -----------------------------------------
[client:rtsp] OPTIONS rtsp://<IP_ADDRESS>:554/ch01.264 RTSP/1.0
CSeq: 1
User-Agent: SFRtsp 0.3
[mse] Media source ended: closed
[mse] Media source closed: closed
[mse] Media source opened: open
[client:rtsp] RTSP/1.0 200 OK
CSeq: 1
Public: OPTIONS, DESCRIBE, SETUP, PLAY, GET_PARAMETER, SET_PARAMETER, TEARDOWN
[client:rtsp] DESCRIBE rtsp://<IP_ADDRESS>:554/ch01.264 RTSP/1.0
Accept: application/sdp
CSeq: 2
User-Agent: SFRtsp 0.3
[client:rtsp] RTSP/1.0 200 OK
CSeq: 2
Date: 23 May 2019 11:14:05 GMT
Content-Type: application/sdp
Content-Base: rtsp://<IP_ADDRESS>:554/ch01.264/
Content-Length: 357
Cache-Control: no-cache
v=0
o=- 8246755 8246755 IN IP4 <IP_ADDRESS>
a=control:*
a=source-filter: incl IN IP4 * <IP_ADDRESS>
s=WUHAN Ants RtspServer
a=range:npt=0-
t=0 0
m=video 0 RTP/AVP 96
c=IN IP4 <IP_ADDRESS>
a=rtpmap:96 H264/90000
a=control:ctype=video
a=recvonly
m=application 0 RTP/AVP 106
a=rtpmap:106 vnd.onvif.metadata/90000
a=control:ctype=app106
a=sendonly
[parser:sdp] Ignored unknown SDP directive: c=IN IP4 <IP_ADDRESS>
[client:rtsp] SDP contained 2 track(s). Calling SETUP for each.
[client:rtsp] setup track: video
[client:rtsp] setup track: application
[client:rtsp] SETUP rtsp://<IP_ADDRESS>:554/ch01.264/ctype=video RTSP/1.0
Transport: RTP/AVP/TCP;unicast;interleaved=0-1
Date: Thu, 23 May 2019 03:17:06 GMT
CSeq: 3
User-Agent: SFRtsp 0.3
[client:rtsp] RTSP/1.0 200 OK
CSeq: 3
Server: Ants Rtsp Server/1.0
Date: 23 May 2019 11:14:05 GMT
Session: 8246755; timeout=60
Transport: RTP/AVP/TCP;unicast;interleaved=0-1;ssrc=006A5741
TypeError: Cannot read property 'sprop-parameter-sets' of undefined
=============================================
but, it can be played in VLC media player

Can anybody help me?

Please add "fmtp" atrribute to SDP of your stream, or remove the processing from code


Add types for pandas.DataFrame.map

Added types for pandas.DataFrame.map (renamed from applymap in pandas 2.1.0 and later https://pandas.pydata.org/docs/dev/reference/api/pandas.DataFrame.map.html, applymap has been deprecated)

[ ] Closes #800

precommit is failing because you didn't format with black
See https://github.com/pandas-dev/pandas-stubs/blob/main/docs/setup.md to set up your environment.

Also CI is failing because pyright 1.1.332 is picking up stuff we need to investigate.
Can you put pyright==1.1.331 in the pyproject.toml file?

pyright seems to now enforce more checks for stub files

Behavior Change: Changed override-related checks (reportIncompatibleMethodOverride, reportIncompatibleVariableOverride and reportImplicitOverride) so they apply to stub files. Previously, these were skipped for stubs.

I will try to look into that.

Made some changes:

Removed stub and test for applymap
Downgraded pyright
Ran the precommit (it just removes some whitespace)


Permissions when using alias

When using an alias the permission for the InvokeFunction should get an addition :* after the function name
Starting
From AWS:
Currently, you can restrict the scope of a user's permissions to the Lambda function APIs by using an unqualified ARN in the Resource element of your IAM or VPC endpoint policy. Previously, when used in this way, Lambda interpreted such permissions in one of two ways depending on the API usage. Either by granting permission to requests to the function only, or granting permissions to requests to the function and all of its sub-resources. For example, when using IAM or VPC endpoint policy with resource element as unqualified ARN, arn:aws:lambda:aws-region:acct-id:function:helloworld, API Request(s) with qualified ARN as arn:aws:lambda:aws-region:acct-id:function:helloworld:42 is denied and the request using arn:aws:lambda:aws-region:acct-id:function:helloworld, Qualifier=42 is allowed. We received customer feedback that this dual interpretation was not always intuitive. To address this feedback and increase the consistency of the Lambda function APIs authorization, starting January 24, 2022, Lambda requires IAM or VPC endpoint policies to specify the fully qualified ARN in the Resource element when authorizing API calls that use an unqualified ARN with a qualifier parameter.
This change will begin on April 30, 2022, to give you ample time to update your IAM policies or permissions attached to the users and roles that call Lambda APIs before that.

Thanks @MartinRisk !
Do you have a link to that statement?
I can't find the oficial source about this.

Hi,
I received an Email about this change from AWS.
Found someone else with the same mail: https://forum.serverless.com/t/lambda-security-notification-from-aws-involving-iam-policies-need-help-determining-action-for-serverless-framework/16660
Official doc: https://docs.aws.amazon.com/lambda/latest/dg/lambda-api-permissions-ref.html
Look in the Function resource names part.
"Whether you are allowing or denying an action on your function, you must use the correct function ARN types in your policy statement to achieve the results that you expect. For example, if your policy references the unqualified ARN, Lambda accepts requests that reference the unqualified ARN but denies requests that reference a qualified ARN."
"If your policy references any ARN using *, Lambda accepts any qualified or unqualified ARN."

If you always add a * directly after the FunctionName (without a : ) it should be ok for qualified and unqualified calls

Solved in v710


build(deps): Bump grpc version to 1.57.0

Thank you for opening a Pull Request! For general contributing guidelines, please refer to contributing guide
Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> ☕

CI failed with:
Error:  Errors:
Error:    GrpcDirectStreamControllerTest.testRetryNoRaceCondition:145 » UncheckedExecution java.lang.NoSuchMethodError: java.nio.ByteBuffer.clear()Ljava/nio/ByteBuffer;
[INFO]
Error:  Tests run: 149, Failures: 0, Errors: 1, Skipped: 0

grpc release note said the release breaks Java 8.
@suztomo should we skip this version?


Missing Container?

This is what I get when running the exe and main.py:
Xbox Game Pass for PC savefile extractor
Installed supported games:

Starfield
!! Missing container "Saves/Autosave1_CD65C61F_437562616E6272656164_020225_20230918222206_27_0_4.sfs" !!
Xbox cloud save syncing might not be complete, try again later.
Extracted saves for this game might be corrupted!
Press enter to skip and continue.

The XGP save data can contain duplicate entries for same containers. Maybe the syncing is still in progress (I've seen some reports that the Xbox save syncing can take 30 minutes or more) or something else went wrong at some point, 🤷.
You can hit enter and the extracted saves will most likely work and maybe even include the indicated file. In worst case, you might have missing savefiles in the export.

Yea I figured it there might be an issue with sync. I left it open all night to see if it would finish, but no dice. I tried hitting enter, but it just keeps repeating the same message with no zip file output.

Got it!!!
Just keep hitting enter until I get the output text.
Now I can finally downsize from 4 pet rocks lol.
Thank you for the amazing tool, this saved my playthrough.


JSAPI docs

For #1429

These are all new JSAPI docs (Deck, View, Controller RFC)  that will undergo review before release. Merging to get structure in place.


Lock-to-Landscape/Lock-to-Portrait Support

Support getActivity().getRequestedOrientation() for previews, pictures (i.e., lock-to-landscape). Include support for both in-manifest and dynamic (setRequestedOrientation()) orientation.

Also check out https://github.com/commonsguy/cwac-camera/issues/241 for ideas.


requests are made not accordingly to Asterisk documentation

So far I've tried Channels::createChannel, which expects the body to be passed as a Json object, but the library passes urlencoded values. As a result, variables are not set.
This works:
$response = $this->client->getEndpoint()->post($uri, [
                    \GuzzleHttp\RequestOptions::JSON => [
                        'endpoint' => $endpoint,
                        'extension' => $extension,
                        'context' => $context,
                        'priority' => $priority,
                        'label' => $label,
                        'app' => $app,
                        'appArgs' => $appArgs,
                        'callerId' => $callerId,
                        'timeout' => $timeout,
                        'channelId' => $channelId,
                        'otherChannelId' => $otherChannelId,
                        'formats' => $formats,
                        'variables' => array_map('strval', $variables)
                    ]
            ]);

Another thing is for instance the Channels::setVariable method, which by Asterisk doc does not return a variable, but just a HTTP status. As a result, setting a variable with the API generates a json decoding exception.

I'm very busy on another project at the moment.  Would you be able to submit a fix for this?

I have a fork I'm working with which contains the fixes, I have not yet submitted the fix to the master repo because I suspect that there are more, and I want to be done with the fixes I need for my project before doing that. But eventually, I will.

@wormling why hasn't the fix made by @pietrobaricco still been merged?


opendata: ValueError possibly due to an incorrect date

ggugan@Ala:~/covizu/covizu$ wget https://data.nextstrain.org/files/ncov/open/sequences.fasta.xz
ggugan@Ala:~/covizu/covizu$ wget https://data.nextstrain.org/files/ncov/open/metadata.tsv.gz
ggugan@Ala:~/covizu/covizu$ nohup python3 convert.py sequences.fasta.xz metadata.tsv.gz --xz --mgz --region region --division division --outfile opendata.json.xz &
ggugan@Ala:~/covizu/covizu$ cat nohup.out
Traceback (most recent call last):
  File "convert.py", line 214, in <module>
    for record in feed:
  File "convert.py", line 89, in combine
    dt = seq_utils.fromisoformat(coldate)
  File "/home/ggugan/covizu/covizu/covizu/utils/seq_utils.py", line 125, in fromisoformat
    return date(year, month, day)
ValueError: month must be in 1..12

Can you catch the exception so that we can determine which record is causing this issue?  Then we can report it to the Nextstrain team

Hm, that's in the original Genbank record.  We're have to write a catch for that edge case.


[web][SIMS-#90] Integrate Vuetify 3.0.alpha

Integrate Vuetify 3.0.alpha with current web app as main UI layout framework.
Observation:

application main view is has some added padding in left/right side.
The logo is not loading

Vuetify enable

Previous

Adjusted BC Logo and left/right spaces.


Delete unused _dbName property in CursorApiClientTest.

A minor commit to get my first contribution going.
This resolves a warning in the build caused by a an unused property in tests. #9

bug/ doesn't feel very appropriate for this branch name, but we don't have a clear convention yet. I'm open to other suggestions.


pull request for e2e test at 11/14/2016 04:45:09

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

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.


add emi calculator in c++

🚀 Feature
want a add a program for calculating emi
Have you read the Contributing Guidelines on Pull Requests?
Yes
PS: Please assign this issue to me. I'll start working on it as soon as it gets assigned to me.

All the best!

one ( @bhav09 ) is saying okay you can add and the other ( @HarshCasper ) is not
what is this ?


Using new Prometheus native histograms for Cilium metrics

Summary of changes
I have observed that Cilium is still using traditional native histograms to instrument most of its metrics, in this PR I am enabling native histogram for Cilium instrumentation. More info on native histograms in the official Prometheus talk.
For ops teams, this change should bring no difference since native histograms are only scraped if a feature flag is enabled on Prometheus side, that is, if they are not using the feature their metrics will still be exporter with the same name and using the same bucket configuration of the existing metrics.
If the feature is enabled in the Prometheus scraping Cilium, the metric names will change, users won't see the metrics with suffixes like _count, _sum, _bucket anymore.
The main benefits of using native histogram are that the metrics become "lighter" in cardinality (since it removes the "fake label" le) and also becomes much more precise.

[x] For first time contributors, read Submitting a pull request
[x] All code is covered by unit and/or runtime tests where feasible.
[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[x] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[x] Provide a title or release-note blurb suitable for the release notes.
[ ] Are you a user of Cilium? Please add yourself to the Users doc
[ ] Thanks for contributing!

Fixes: #issue-number
Enabling native histograms for Cilium related metrics to reduce cardinality and improve precision

Thanks for the contribution 🙏 😄
Just trying to understand any possible impact of this better.
I am a bit concerned about the stability of this feature, the documentation mentions that this feature is still quite experimental. Notably, it mentions possibly with including breaking changes for the TSDM format.
I think this would be worth discussing in one of our community meetings

Also, FYI, there's some more Cilium Operator specific metrics living in operator/metrics/ and pkg/ipam 😄

Thanks for the contribution 🙏 😄
Just trying to understand any possible impact of this better. I am a bit concerned about the stability of this feature, the documentation  mentions that this feature is still quite experimental. Notably, it mentions possibly with including breaking changes for the TSDM format.
Might it be premature to be adding this?
I think this would be worth discussing in one of our community meetings

This feature is currently hidden behind a feature flag in Prometheus, so users of Cilium that are still not using the native histogram feature, are still going to see the normal histograms as they were seeing before.
If they opt-in though, to native histograms they will see the native histogram version of the metric.
The only problem here is that native histogram metrics change their names, for example, where you had agent_boot_time_seconds and metrics like: agent_boot_time_seconds_bucket, agent_boot_time_seconds_sum, agent_boot_time_seconds_count. Now you only have the agent_boot_time_seconds and you will need to apply new functions prefixed with histogram, to find out the sum/count of the histogram. Example: histogram_count(rate(agent_boot_time_seconds[2m])).

I see that you put this PR up for discussion on the next call, so I will try to drop by the meeting and see what folks have to say.

Thanks @pedro-stanaka for the update.
I know it will be discussed in more details in the community meeting, but if it's a go then the feature should be documented somewhere here I think. And as @tommyp1ckles mentioned - there are Cilium Operator specific histograms, it would be good to have them consistent with Cilium Agent metrics. For Hubble metrics I don't think it's as necessary, they're used differently (although maybe something to consider too).

As discussed in the community meeting, it would be good to have a flag and a Helm value to explicitly enable native histograms.
This means more configuration for users who want to use them: users will need to enable native histograms in Prometheus and additionally in Cilium (and in any other metrics exporter that will follow such approach). However, native histograms are highly experimental and bring non-trivial underlying changes: switch to gRPC for scraping, non-stable TSDB format, resources usage in particular in the case when you need to scrape both old and new. Because of that, I imagine users might want to enable native histograms in Prometheus, but only for one particular service, not necessarily Cilium.

This looks really useful, and thanks for enabling this. Of course, native histograms haven't settled yet, but they certainly are very compelling from a server operations perspective.
One thought about interacting with multiple versions of Prometheus: should we add a rule in our ServiceMonitor to drop native histograms, and a corresponding Helm option? (I'm not sure if that's possible, but maybe?) What happens if we expose native histograms to a version of Prometheus that doesn't support it? Will it cause problems and/or metric spam?
In other words, does this need to be behind a feature flag? If so, maybe we can avoid plumbing this all through the codebase and just drop at the ServiceMonitor instead.
Reading the docs, there's nothing explicitly needed for newer versions of Prometheus. In other words, it seems like client-side buckets coexist peacefully with native histograms. If this understanding is incorrect, do let me know.

This looks really useful, and thanks for enabling this. Of course, native histograms haven't settled yet, but they certainly are very compelling from a server operations perspective.
One thought about interacting with multiple versions of Prometheus: should we add a rule in our ServiceMonitor to drop native histograms, and a corresponding Helm option? (I'm not sure if that's possible, but maybe?) What happens if we expose native histograms to a version of Prometheus that doesn't support it? Will it cause problems and/or metric spam?

This would be cool, but it is not possible. If the Prometheus administrator enabled the feature flag "enable-native-histograms" the new metric version will be scraped directly from the target.

In other words, does this need to be behind a feature flag? If so, maybe we can avoid plumbing this all through the codebase and just drop at the ServiceMonitor instead.

As @lambdanis said, the agreement reached in the community call was that we needed a feature flag from Cillium side.
Something I wonder here is how a Cilium user that can't control the initialization options from Cilium can enable this feature flag? For example, if I want to use Control Plane V2 from Google's GKE. I would have to wait anyway until we decide to remove this from a feature flag.

Reading the docs, there's nothing explicitly needed for newer versions of Prometheus. In other words, it seems like client-side buckets coexist peacefully with native histograms. If this understanding is incorrect, do let me know.

You are correct.


integration: extend request timeout

Extend request timeout to give etcd cluster enough time to return
response.

I am not sure if this is to aggressive since it is just for travis... Other than that, LGTM

Its impaction is only to wait longer when test fails, so it is fine IMO.


Dates are not being parsed after upgrade to 0.14

Hi, after upgrading from 0.13.5 to 0.14 we noticed that dates are not being parsed any more.
Was there a breaking change around this we need to implement for 0.14? Here's an example of where we are seeing this.
reader.onload = (e) => {
  var data = e.target.result
  var workbook = XLSX.read(data, { type: 'binary' })
  let mySheet = workbook.Sheets['MyData']

  let rawList = XLSX.utils.sheet_to_json(mySheet, { header: columns, dateNF: 'MM/DD/YYYY', cellText: false })
  console.log(rawList)
  // dates in rawList now look like 43374 where as before they looked like "10/1/18"
}

Cool - that worked, thank you!


Fix for #334

@begriffs Please merge this into master then if possible merge it to v3 so that v3 is the latest and greatest

Nice compact fix.
How about a quick changelog entry and lint fix?


feat(graphql_flutter): make first version of demo app

Continuing to split https://github.com/zino-hofmann/graphql-flutter/pull/1190
Fixes https://github.com/zino-hofmann/graphql-flutter/issues/1162
Fixes https://github.com/zino-hofmann/graphql-flutter/issues/1155
The common mistake that I make also during this demo is how we declare the link for the subscription, and I think in this case the link offers a bad API.
BTW the correct way to declare the Link in order to work with Subscriptions is the following one
  final Link link = Link.split(
    (request) => request.isSubscription,
    wsLink,
    httpLink,
  );

Ah I was forgetting, this PR introduce also a new demo that work with the following API https://api.chat.graphql-flutter.dev based on the repository https://github.com/vincenzopalazzo/keep-safe-graphql that will be offer a lot of servers to provide integration testing server for graphql library

Why are you deleting the tests? Also, are you sure that your example project structure actually works with pub.

Good catch the test are just a refused because I was trying to implement integration testing, and accidentally I removed the test dir, I will reintroduce it and also make the suggestion that you had!


JSON composite Job and RemoteProperties

Hey @weburnit and thanks.  Couple points:

Please provide a description in your pull request.  That way I know exactly what you're thinking and intending with the change.  I know that this is based on your last pull request, but it's still best practice to give some context when you make a new PR.
It seems like you want to change the format of job.Job.  I said this in the last PR, but I don't really want to change that structure, because it will break compatibility with existing Kala users.
You mentioned that the dashboard/frontend is broken.  If that's the case, then that's my fault because I coded up the dashboard pretty quick and probably didn't test it as thoroughly as I should have.  But then, I think the correct solution is to simply fix the dashboard.

I hope that makes sense to you.  If you feel capable of fixing the dashboard, I would really appreciate a PR on it, otherwise I can have a look at it as soon as I have a little more time.

I will ask my team-mate to fix Dashboard issue soon


in embedded terminal, vscode can't find file when clicking on file path that starts with ./ and ends with line/col numbers

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.95.3
OS Version: Linux x64 6.1.0-27-amd64

Steps to Reproduce:

open embedded terminal
run (mkdir -p dir; cd dir; touch file.txt; echo "./file.txt:1:1")
ctrl+left-click the ./file.txt:1:1 in the terminal
observe vscode can't find the file until you delete the leading . in the path.

Rust has started generating paths that start with ./ in backtraces, which makes it very annoying to go to files in the backtrace because you have to manually edit the path after clicking on it to remove the leading . before vscode can find the file.

Hey @programmerjake, thanks for reporting this.
I could not reproduce this on stable 1.96.0 on Mac OSx 15.1 zsh and stable 1.95.3 on Ubuntu 24(6.8.0-50) bash, so we need more information. What kind of terminal are you using? Anything else you can think of to help us to repro the issue?
I also noticed in your instructions that if you delete the leading . in the path, the path becomes absolute, therefore are you certain that you perform ctrl+left click you've changed-dir to the correct location? Note that the (mkdir -p dir; cd dir; touch file.txt; echo "./file.txt:1:1") does not cd into dir automatically, so the correct command might be (mkdir -p dir; cd dir; touch file.txt; echo "./dir/file.txt:1:1")?

Hey @programmerjake, thanks for reporting this.
I could not reproduce this on stable 1.96.0 on Mac OSx 15.1 zsh and stable 1.95.3 on Ubuntu 24(6.8.0-50) bash, so we need more information. What kind of terminal are you using? Anything else you can think of to help us to repro the issue?

I'm using bash on Debian 12 and vscodium 1.95.3 installed with apt from https://vscodium.com/#deb (i don't expect vscodium to behave differently than vscode here...)

I also noticed in your instructions that if you delete the leading . in the path, the path becomes absolute, therefore are you certain that you perform ctrl+left click you've changed-dir to the correct location? Note that the (mkdir -p dir; cd dir; touch file.txt; echo "./file.txt:1:1") does not cd into dir automatically,

no, "./file.txt:1:1" is correct here, I encounter paths like this when using Rust with a Cargo workspace:
Run RUST_BACKTRACE=1 cargo test in /tmp/demo-crate with the following files:
/tmp/demo-crate/Cargo.toml:
[workspace]
resolver = "2"
members = ["crates/*"]

/tmp/demo-crate/crates/demo-crate/Cargo.toml:
[package]
name = "demo-crate"
workspace = "../.."
version = "0.1.0"
edition = "2021"

/tmp/demo-crate/crates/demo-crate/src/lib.rs:
#[cfg(test)]
#[test]
fn test_a() {
    panic!(); // print backtrace
}

despite running in /tmp/demo-crate it outputs a backtrace containing:
   5: demo_crate::test_a
             at ./src/lib.rs:4:5
   6: demo_crate::test_a::{{closure}}
             at ./src/lib.rs:3:12

I just created a new user (so there aren't any non-default vscodium config files) and was able to reproduce it there.

looks like this is caused by calling AnythingQuickAccessProvider.doGetFileSearchResults with "./file.txt" which returns no results (on vscodium <IP_ADDRESS>47 which corresponds to 138f619c86f1199955d53b4166bef66ef252935c):
https://github.com/microsoft/vscode/blob/138f619c86f1199955d53b4166bef66ef252935c/src/vs/workbench/contrib/search/browser/anythingQuickAccess.ts#L672-L685
I tried debugging further but got stuck on the RPC since idk how to debug the other side of the RPC.
Is this the right spot to add a check for filePattern starting with ./ and removing that? if so I can make a PR to fix that.

Right, that is the behavior of the RUST_BACKTRACE=1 flag, - it outputs paths relative to the test file, so in your example the ./src/lib.rs:4:5 path is incorrect because you are in the /tmp/demo-crate folder, hence clicking on it will try to resolve file at /tmp/demo-crate/src/lib.rs:4:5 which does not exist 🤷
You can use RUST_BACKTRACE=full instead which outputs absolute paths in the traces hence it will work like a charm (the output is much more verbose though). You can follow related https://github.com/rust-lang/rust/issues/113805 issue on the official Rust repo for more info.
Another option is to use the --remap-path-prefix flag when you run the tests like so:
RUSTFLAGS="--remap-path-prefix $(pwd)=." cargo test

This will produce paths relative to your current working directory, hence the ctrl+click will work as you'd expect. If you are using nightly, you can also set this up in the test profile of your .cargo/config file (please note that this config option is still unstable).
Hope this helps 🤗

I think this is still a bug in vscode since you're clicking on a relative path so it should give search results for all files that the relative path could be referring to. this is what vscode successfully does if the path doesn't have a leading ., imo it's a bug that vscode fails to find anything because it doesn't understand the . path component.
Just because this is also a bug in rust doesn't mean this isn't a bug in vscode.
@legomushroom can this be reopened?


Does bloxstrap compatible with Roshade?

Just a question, cause I tried combining roshade with bloxstrap and it's doesn't work.

Roblox anti-cheat dosent like roshade or reshade, just uninstall it.
If you wanna use them, no it's not possible.

Roshade hasn't worked with Roblox for the past six months because of Roblox's anticheat.


Implement "Least-Requests" load balancing

Load balance to the backend which has the fewest on-going requests. This requires some state be tracked about the currently load to each backend (though @Tratcher indicated we already have that state).

EndpointInfo.ConcurrencyCounter
https://github.com/microsoft/reverse-proxy/blob/be96d64cb0fb732e5a31aebb91e0b9350798d62a/src/ReverseProxy.Core/Service/RuntimeModel/EndpointInfo.cs#L42-L45


Using .value in an input task should result in a compilation error

steps
Use .value for an input task, instead of the correct .evaluated.
problem
/data/rw/code/scala/sbt-bintray/build.sbt:24: warning: `value` is deprecated for an input task. Use `evaluated` or `inputTaskValue`.
See http://www.scala-sbt.org/0.13/docs/Input-Tasks.html for more details.
    scripted := Sbt1M6Scripted.scripted.value,

expectation
A compilation error. .value should not be possible.
notes
sbt version: 1.0.0-M6.
Note: Is there actually a meaningful use of .value instead of .evaluated for input tasks?

I've implemented a Restligeist macro for InputKey in https://github.com/sbt/sbt/pull/2710
[info] Loading project definition from /Users/eed3si9n/work/hellotest/project

hellotest/build.sbt:23: error: `value` is removed from input tasks. Use `evaluated` or `inputTaskValue`.
See http://www.scala-sbt.org/1.0/docs/Input-Tasks.html for more details.
  val one = (run in Compile).value
                             ^
hellotest/build.sbt:25: error: `value` is removed from input tasks. Use `evaluated` or `inputTaskValue`.
See http://www.scala-sbt.org/1.0/docs/Input-Tasks.html for more details.
  val two = (run in Compile).value
                             ^
[error] Type error in expression

Project loading failed: (r)etry, (q)uit, (l)ast, or (i)gnore?


Possible Regression using two or more regex

Be clear about the software, hardware and version you are running
For example:

With version 20.2.2 of the Mycroft software (latest version as of Apr 24, 2020)
With the standard Wake Word
With Jarvis keyword

Try to provide steps that we can use to replicate the Issue
Given the following intent methods:
@intent_handler(IntentBuilder('SearchChild').require('SearchKeyword').require("Title").require("ParentTitle"))
def handle_search_confluence_child_page(self, message):
    print("\/\/\/\/\/ stuf happened!")

@intent_handler(IntentBuilder('SearchTitle').require('SearchKeyword').require("Title"))
def handle_search_confluence_title(self, message):
    print(message.data)

I have tried the following regex in find.title.rx
search confluence for (?P<Title>.*)

With these regexes (all tested one at a time) in find.child.rx
.* under (?P<ParentTitle>.*) page

and
under (?P<ParentTitle>.*) page

and
\b^.* under (?P<ParentTitle>.*) page

Using the trigger:
search confluence for chicken under breakfast page

The output of message.data happens every time to be this:
~~~~st', 'key': 'chicken under breakfast', 'from_context': False}], 'utterance': 'search confluence for chicken under breakfast page', 'SearchKeyword': 'search confluence', 'Title': 'chicken under breakfast page'}

I believe this to be a regression because using a similar method in a different skill which I have not changed no longer works.
This intent no longer gets called
@intent_handler(IntentBuilder('').require('CreateItemKeyword').require("Food")
                    .require("ShoppingList").optionally("Category"))`

With the regex file1
add (?P<Food>.*) (to my) (?P<ShoppingList>.*) list
add (?P<Food>.*) (to the) (?P<ShoppingList>.*) list

and regex file 2
under (?P<Category>.*)

Output of message.data
~~~~tems', 'key': 'steves items', 'from_context': False}], 'utterance': 'add something to the steves items list under ai', 'CreateItemKeyword': 'add', 'Food': 'something', 'ShoppingList': 'steves items'}

This was known working on Apr 11 when I committed this to my master branch

Hey @stratus-ss , thanks for bringing this up. Regexes are particularly dicey in Adapt; they have the ability to collide with each other in weird ways, and tend not to behave the way you expect when there are multiple in play.
I suspect that this is not a regression, as there have been no material changes (docstrings aside) to adapt since this commit in May 2019. If you updated your version of adapt since your most recent change, that may point to a latent regression. If you only updated mycroft, this issue might be more appropriate for that forum.
I would recommend using git-bisect to try to track down the exact change that broke your flow.

I appreciate the reply but git bisect isnt going to help. Nothing related to either of those functions or the regex changed.
I may have to investigate building the other intent.
If you don't think that this is a valid report please feel free to close it and clear up your issues :)

Hey @stratus-ss , sorry for the long silence here. I suspect that this is a collision of regexes interacting with each other, either via code changes in mycroft, addition/removal of skills or vocabulary, introduction of padatious, or other downstream changes. Adapt regexes are known to be flaky, especially when multiple skills come into play.
If you have not already resolved this issue, I would recommend looking for assistance debugging mycroft-core with those channels. I'm going to close this issue now, but please feel free to re-open if you have a stable reproduction (outside of mycroft-core). Thanks!


master: Controller nodes never come up when Calico is enabled

Aug 21 04:21:24 ip-10-0-0-172.ap-northeast-1.compute.internal docker[30414]:         "k8s_api_root": "https://__KUBERNETES_SERVICE_HOST__:__KUBERNETES_SERVICE_PORT__",
Aug 21 04:21:24 ip-10-0-0-172.ap-northeast-1.compute.internal docker[30414]: /install-cni.sh: line 116: KUBERNETES_SERVICE_HOST: parameter not set
Aug 21 04:21:24 ip-10-0-0-172.ap-northeast-1.compute.internal systemd[1]: kubelet.service: Control process exited, code=exited status=2
Aug 21 04:21:24 ip-10-0-0-172.ap-northeast-1.compute.internal systemd[1]: Failed to start kubelet.service.

This is possibly due to https://github.com/projectcalico/cni-plugin/pull/350
In nutshell, kube-aws does a work-around which runs install-cni in a docker container to avoid the chicken-and-egg problem encountered in #494.
The work-around broken due to the recent improvement on install-cni made in https://github.com/projectcalico/cni-plugin/pull/350.
It is kube-aws' fault because it is kube-aws who uses install-cni in an untended way - it uses the script not for writing cni config files but only for extracting calico binaries.

We can deal with the "improvement" by slightly modifying cloud-config-controller for now.
In longer term, we should revisit #494 and find a way to use install-cni also for writing cni configs.


Generate code targeting v3.0 in top of existing 3.1 SDK

Generate code
Try running some endpoints and see what we get

Generated code done here: https://github.com/maririos/azure-sdk-for-net/tree/fr2and3/sdk/formrecognizer/Azure.AI.FormRecognizer/src

This has been done

running V2.1 and v3 side by side works. clossing issue


Added Hack The Box site in Web section

I remember when I bumped into this site. I had a lot of fun tampering with it and I learned a lot that is why I added it to the list!

Thanks!

Glad I could contribute xD
Regards,
Karalis, Atrestis
Software Engineer @ Accenture S.A.
On Fri, Mar 30, 2018 at 11:57 AM, Taehoon Kim<EMAIL_ADDRESS>wrote:

Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/carpedm20/awesome-hacking/pull/45#issuecomment-377482068,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADi-EKjr4lSpLvuJ5SptW6yMjh88a7lgks5tjfNxgaJpZM4TBdia
.


[Publishing] Create Gcloud commands and rest API for below templates. Cassandra, Kafka, Mongo, PubSub to Target GCS

Java - CassandraToGCS
Java - KafkaToGCS
Java - PubSubToGCS
Python - MongoToGCS
Add the commands in : go/go/dataproc-gcp-documentation-internal document also contains commands for reference from other templates

commands added to doc

Pending review


[WIP] Minor adjustments to aon wake behavior

Instead of using notidle to transition to AwkWokenUon, use !trigger instead.
This ensures that we are seeing usbdev's view of resume handling instead
of the asynchronous pin state.

Add pinmux's low power indication to usbdev_aon_wake detection.  Low power
is a system wide indication that we have entered low and the usbdev module
can no longer responsd to a "resume" pin state. This signal may remove
the need for usb_out_of_rst, but that is kept for now.

Move usb_suspend_o output to cio_suspend_o.  This allows the software to
arbitrarily override the value of suspend for testing purposes.

Signed-off-by: Timothy Chen<EMAIL_ADDRESS>

hmmm? sorry which sleep enable signal that comes from the usbdev directly?
and yeah there is a cdc crossing, so this one does introduce a sync.  If
the trigger suddenly drops away, the logic here "should" gracefully handle
itself, because firmware will come back and assert the ack.
On Thu, Feb 25, 2021 at 4:57 PM Michael Schaffner<EMAIL_ADDRESS>wrote:

@msfschaffner commented on this pull request.
In hw/ip/pinmux/rtl/pinmux.sv
https://github.com/lowRISC/opentitan/pull/5380#discussion_r583310045:

@@ -139,6 +139,12 @@ module pinmux
.clk_aon_i,
.rst_aon_ni,

// input signals for usb entering low power
// This will be different per system integration.
// It specifically refers to situations where the system
// containing usb has gone to low power.
.low_power_i(sleep_en_i),

Oh I was just thinking that we could connect this to the sleep enable
signal that comes from the usbdev directly to keep this consistent (sorry I
didn't mean that we should add a dedicated pinmux CSR for this, since all
the control signals are coming from usbdev anyways).
Also, this would introduce a CDC crossing on the sleep enable signal (see
discussion on #5359 (comment)
https://github.com/lowRISC/opentitan/pull/5359#discussion_r583291042).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/5380#discussion_r583310045,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSRXLWHC53UH76H5LQDTA3WXBANCNFSM4YHJZSOA
.

the other thing (i'll need to check with Mark) and maybe this is what
you're saying. It would be better for usbdev to explicitly have a point of
no return.
Ie, once software sets a bit, it will no longer attempt to handle resume
and instead assumes the usbdev_aon_wake module will handle it instead.
That will probably get rid of a lot of the gray area.  That would need to
change a bit how this module is structured though .
On Thu, Feb 25, 2021 at 4:59 PM Timothy Chen<EMAIL_ADDRESS>wrote:

hmmm? sorry which sleep enable signal that comes from the usbdev directly?
and yeah there is a cdc crossing, so this one does introduce a sync.  If
the trigger suddenly drops away, the logic here "should" gracefully handle
itself, because firmware will come back and assert the ack.
On Thu, Feb 25, 2021 at 4:57 PM Michael Schaffner <
<EMAIL_ADDRESS>wrote:

@msfschaffner commented on this pull request.
In hw/ip/pinmux/rtl/pinmux.sv
https://github.com/lowRISC/opentitan/pull/5380#discussion_r583310045:

@@ -139,6 +139,12 @@ module pinmux
.clk_aon_i,
.rst_aon_ni,

// input signals for usb entering low power
// This will be different per system integration.
// It specifically refers to situations where the system
// containing usb has gone to low power.
.low_power_i(sleep_en_i),

Oh I was just thinking that we could connect this to the sleep enable
signal that comes from the usbdev directly to keep this consistent (sorry I
didn't mean that we should add a dedicated pinmux CSR for this, since all
the control signals are coming from usbdev anyways).
Also, this would introduce a CDC crossing on the sleep enable signal (see
discussion on #5359 (comment)
https://github.com/lowRISC/opentitan/pull/5359#discussion_r583291042).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/5380#discussion_r583310045,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSRXLWHC53UH76H5LQDTA3WXBANCNFSM4YHJZSOA
.

@mdhayter thinking about this a bit more.. i'm thinking about removing a couple of states in here.
When we get the software wake trigger, i think it's safe to just move directly to wait for the wakeup indication. If the usbdev actually detected a resume in time, it would break the software out of any IRQ loop it's in and eventually trigger wake_ack.   The notidle would be captured anyways.
This would kind of make it behave more like a generic pad wakeup.  I'll think about it a bit more.

I think I added the extra states based on an example in the one-pager where the software will set the aon_wake as part of configuring rather than having to respond to a suspend event. The simpler states could be used if we forced the software model.

I think I added the extra states based on an example in the one-pager where the software will set the aon_wake as part of configuring rather than having to respond to a suspend event. The simpler states could be used if we forced the software model.

ooooo... maybe i missed this part. Let me go back and re-examine based on this context.

okay all (especially @mdhayter ) i went over this again with an eye on keeping the enable set all the time as a static function. I believe what's in this PR should work now as long as we don't need to support someone turning enable on, then later suddenly deciding to turn it off (I should add handling for that).

@vogelpi / @tomroberts-lowrisc can you guys have a look also?

yeah i thought the same thing.. but i think it doesn't hurt to keep it
there for now.
On Wed, Mar 24, 2021 at 12:33 PM mdhayter @.***> wrote:

@.**** commented on this pull request.
Looked again, may be options to simplify more but would be ok as it is.
In hw/ip/usbdev/rtl/usbdev_aon_wake.sv
https://github.com/lowRISC/opentitan/pull/5380#discussion_r600706567:

@@ -21,6 +21,7 @@ module usbdev_aon_wake import usbdev_pkg::*;(
input  logic usb_dnpullup_en_alw_i,

// Register signals from IP

input  logic usb_out_of_rst_upwr_i,

There may be an argument for removing this. It was added to ensure that we
wouldn't get garbage signals from the power-domain crossing when the usbdev
is powered down. If the power-domain crossing is guaranteed to park signals
low then we don't need this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/5380#pullrequestreview-620022778,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSTJCZVWFQO73KGNKV3TFI5CLANCNFSM4YHJZSOA
.


Do you have the full list of functions we can use?

maybe you can check this file
https://github.com/paquettg/php-html-parser/blob/master/src/PHPHtmlParser/Dom.php

Ok. :)


[Feature][Python] Migrate exists method call in PythonGateway to corresponding service

Purpose of the pull request
Migrate exists method call in PythonGateway to corresponding service
fix:#10989

Codecov Report

Merging #11188 (1d27d1c) into dev (81fa096) will increase coverage by 0.01%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##                dev   #11188      +/-   ##
============================================
+ Coverage     40.22%   40.24%   +0.01%
- Complexity     4922     4925       +3
============================================
  Files           982      982
  Lines         37562    37572      +10
  Branches       4127     4129       +2
============================================
+ Hits          15109    15120      +11
+ Misses        20917    20912       -5
- Partials       1536     1540       +4

Impacted Files
Coverage Δ

...heduler/api/service/impl/ResourcesServiceImpl.java
52.42% <63.63%> (+1.13%)
:arrow_up:

...che/dolphinscheduler/api/python/PythonGateway.java
17.82% <100.00%> (-1.99%)
:arrow_down:

...ler/plugin/task/api/loop/BaseLoopTaskExecutor.java
0.00% <0.00%> (ø)

...loop/template/http/HttpLoopTaskInstanceStatus.java
0.00% <0.00%> (ø)

...r/plugin/task/sqoop/parameter/SqoopParameters.java
56.41% <0.00%> (+1.28%)
:arrow_up:

...er/master/dispatch/host/assign/RandomSelector.java
83.33% <0.00%> (+5.55%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us.

Currently, we only call method queryResourcesFileInfo from pyton gateway service in https://github.com/apache/dolphinscheduler/blob/797057336529d3077c42ec689416c08c2e8d14cf/dolphinscheduler-python/pydolphinscheduler/src/pydolphinscheduler/core/task.py#L299-L304 for python side, maybe we should migrate this code to Resource class of Python API side, to make our code more sense


Can the arm architecture be used?

How do I need to compile successfully in the arm environment?

Hi! The PoC should be architecture-agnostic. If you've tried to build it on an ARM machine and it hasn't worked, can you provide some logs of it that point out possible errors?


Updated IPv6InterfaceId to WeaveNodoId Conversion Functions.

Historically Weave NodeIds have always been universal (globally unique) identifiers.
Therefore, it was sufficient to set or clear the Universal/Local (U/L) bit when converting
WeaveNodoId to IPv6InterfaceId and vise versa.
In preparation for supporting also local identifiers this implementation has
been updated to invert U/L bit when converting, per RFC-3513.

Codecov Report

Merging #327 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #327      +/-   ##
==========================================
+ Coverage   54.22%   54.23%   +<.01%
==========================================
  Files         332      332
  Lines       56407    56407
==========================================
+ Hits        30589    30592       +3
+ Misses      25818    25815       -3

Impacted Files
Coverage Δ

src/lib/core/WeaveMessageLayer.h
25% <0%> (ø)
:arrow_up:

src/test-apps/TestCASE.cpp
96.1% <0%> (+0.22%)
:arrow_up:

src/lib/profiles/security/WeaveCert.cpp
84.41% <0%> (+0.64%)
: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 48c3236...6cb6c44. Read the comment docs.


[Suppressions] Support request by Cameron Taggart: need suppression review

Email:
Subject Re: AVS 2023-03-01 Suppression
Recipient Swagger and Tools Team

Advised Cameron over email to reach out to assigned PR reviewer, Crystal Yu. PR:

https://github.com/Azure/azure-rest-api-specs-pr/pull/9349

Also pointed out to related guidance on suppressions.

The relevant PR still requires suppression review:
https://github.com/Azure/azure-rest-api-specs-pr/pull/9349
I have advised Cameron over the email to reach out to the PR assignee for the suppression review and to follow these instructions:
https://dev.azure.com/azure-sdk/internal/_wiki/wikis/internal.wiki/85/Swagger-Suppression-Process?anchor=obtaining-suppression

Closing, this was solved weeks ago.


Decide on the usage of this page within b.io

Description:
This is a publicly accessible page even though this is not linked from b.io atm.
So decide on from where to link this and then improve the page a bit to have consistent look and feel with the b.io pages. Ex : headers, footers , menus etc
Describe your task(s)
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

@anuruddhal We can take @dinushan20 's help on improving the look and feel of the pages.


materialize: 0.7.1 -> 0.8.0

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 126574 run on x86_64-darwin 1

1 package built:

materialize

Thank you!


DISTINCT ON (distinct on partial columns)

Background: https://airtable.com/tblD3oZPLJgGhCmch/viwINAgXA5on9pr2T/recuVUy1ggv9TutAt
PRs:

https://github.com/cockroachdb/cockroach/pull/20463

From https://github.com/cockroachdb/cockroach/pull/20463:
Release notes (feature): the DISTINCT ON clause was introduced for
SELECT statements.
Syntax
SELECT DISTINCT ON (<distinct-on-exprs>) <select-exprs> FROM x ...

Which will retrieve an arbitrary row of  for every distinct
combination of .
If the ORDER BY clause is specified in the SELECT statement, DISTINCT ON
will respect that ordering when choosing the "arbitrary row" (that is:
it will choose to first row based on the ordering).
For more examples, see logic_test/distinct_on and logic_test/distsql_distinct_on.

Depends on being able to generally clean up the SELECT diagram: https://github.com/cockroachdb/docs/issues/2719

The diagram will show up properly in #2762.

Thanks, @knz! Do we also need to update the description under Parameters? This is what postgres says:

SELECT DISTINCT eliminates duplicate rows from the result. SELECT DISTINCT ON eliminates rows that match on all the specified expressions. SELECT ALL (the default) will return all candidate rows, including duplicates.

yes, indeed. I added a paragraph accordingly, which I believe will close this issue.


V4.0.0 is no longer useful for Xamarin.Forms apps

Hey guys,
maybe it is not a bad idea to make sure that when upgrading the dependancies, we would not turn down some of the technologies that could be utilizing the Azure Client. v4 made it unable for Xamarin.Forms apps to be using it by the introduced .NET Standard dependency.
Package Microsoft.Azure.Mobile.Client 4.0.0 is not compatible with portable45-net45+win8+wpa81 (.NETPortable,Version=v4.5,Profile=Profile111).
 Ultimately your best bet is to update your project to .NET Standard 1.4 or later. You can easily maintain backwards compatibility with libraries that haven't updated to .NET Standard as shown below.
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>NETSTANDARD_LIBRARY_VERSION</TargetFramework>
    <PackageTargetFallback>$(PackageTargetFallback);portable-net45+win8+wpa81+wp8</PackageTargetFallback>
    <DebugType>full</DebugType>
  </PropertyGroup>

</Project>

@dansiegel, what do you mean by this?

You can easily maintain backwards compatibility with libraries that haven't updated to .NET Standard as shown below.

I tried to add my .NET Standard 1.4 Library to a PCL (for Windows 8.1, Windows Phone 8.1 and Windows 10 UWP as well), but it did not allow me to, even with the PackageTargetFallback directive in the .NET Standard Library project file, as you advised.

@stefanov-stefan We are going to support PCL in .NET SDK for Mobile Apps. I don't have a timeline yet but it will take us a few weeks to release the new SDK.

@artemious7 you can't force netstandard into a PCL, but you can maintain compatibility by updating your project to netstandard1.4 and following my guidance. This will allow you to continue using PCL dependencies your project may have while taking advantage of netstandard. It really is the better way to go as netstandard1.4 gives you much more API surface to work with in your Apps. PCL has always been very limiting as the lowest common denominator. You may notice the sample I gave you showed using netstandard for the target framework not PCL.

@stefanov-stefan We released the latest version of Zumo .NET client SDK v4.0.2 today where we support PCL based Xamarin.Forms project, please find more information here.


Move _server_cert_config_fetcher_wrapper to credentials.pyx.pxi

credentials.pyx.pxi is the only place it's used. Letting it go where it belongs.

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,952,584      Total (=)      1,952,584

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,665,706      Total (<)     10,665,707

 No significant differences in binary sizes

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,952,584      Total (=)      1,952,584

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,665,705      Total (>)     10,665,701

 No significant differences in binary sizes


Project Scope Approval

Required: Have instructors approve the scope of the project.
Project name: Dota 2 Counter-Picker
Possible wireframes:
Features:

@myjohnson23 @luczaki114 this is awesome. I'm not a huge MOBA player but I think you should market this as much as you can to newbie players. You'll knock it out of the park with the Mechanize library!
Approved!


Logging in no longer grabs active streams.

Hey bro, probably due to a recent Firefox update, Twitch-Live is now always showing "There are no live streams." in the active streams window. Tried different accounts / reinstall / fresh Firefox, but it just doesn't grab any live streams anymore. It was working perfectly few weeks back. Thanks for your time!

It might be related to Twitch API, I didn't updated this app lately and Twitch is in transition to API v5. I'll investigate the source of this problem. Thanks for reporting it! I need to update this add-on but there is a problem. Mozilla Add-ons website doesn't accept apps with Angular 1.x library anymore due to security issues. I need to rewrite front-end with React first. So it may take some time to solve and publish this :/

Fixed this problem with last commit. But like I said, still I need to rewrite front-end part with React to be able to publish. But if you don't want to wait, you can build your own add-on with build instructions at the home page. Thanks for reporting it again!

Thanks man, appreciate your work!

Chiming in to say that I just installed the extension from AMO, and from what I can tell, the "login" function isn't working anymore. Nothing shows up on my live feed.

@garoto You're right. I still need to re-write some parts in React instead of Angular 1.X to be able to push new version to AMO. I'll try to do it as soon as possible.
Until that time you can build your own Twitch Live add-on from this source code if you want.
Thanks for your feedback!


GEODE-6846: Clean up PureJavaMode class

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[x] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[x] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[n/a] Have you written or updated unit tests to verify your changes?

[n/a] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>Extra info
Clean up of PureJavaMode class and related code taking for granted Geode run always in that mode.

The org.apache.geode.management.internal.cli.commands.VersionCommandJUnitTest is now broken. It is looking for the native version string in the banner

Just fix the broken test. I'm also okay with PureJavaMode being completely removed. We just need to cleanup the external dependencies before merging this PR.

Ok, so Im going to include the removal of the class then.

Just fix the broken test. I'm also okay with PureJavaMode being completely removed. We just need to cleanup the external dependencies before merging this PR.

Ok, then I will remove the class


Hyundai Ioniq EV 2019

Adds fingerprint for Hyundai Ioniq EV 2019

Make your pull request to the master branch instead. Currently it's trying to merge in all changes from 0.7.11


Patched bot.js to fix issue #269

Works great for me. Please accept the PR!

@ekmartin can you review, merge this PR, and make a new release?


Retry upload on some types of upload failures

The error Segments do not add up to provided total file size. sometimes happens. This just seems to be a basic upload failure. I should just be able to catch this and just redo the upload without waiting for the next time the script gets run.

Made postTweet give returns instead of just dying on failure, so it's possible to run in a loop now.
This will help for simple one-time problems. #6 will make sure actually bad problems don't prevent it from posting for hours on end.
Monitoring for any problems that come up.

Everything has been good up to now.


scion-pki: Add support for auto-completing scion-pki commands

Currently the support is limited to zsh and bash

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

go/tools/scion-pki/README.md, line 137 at r1 (raw file):

`scion-pki certs gen 1-4_294_967_022`

## Autocompleting scion-pki commands

Would be nice if scion-pki autocomplete -h would also output something like this.

go/tools/scion-pki/internal/cmd/cmd.go, line 43 at r1 (raw file):

)

var autoCompleteCmd = &cobra.Command{

This should not generate both auto-completion scripts by default. I suggest you add a -zsh flag and by default only generate the bash auto-completion file, since that is more common.

Comments from Reviewable

Review status: 1 of 2 files reviewed at latest revision, 2 unresolved discussions.

go/tools/scion-pki/README.md, line 137 at r1 (raw file):
Previously, shitz wrote…
Would be nice if scion-pki autocomplete -h would also output something like this.

Done.

go/tools/scion-pki/internal/cmd/cmd.go, line 43 at r1 (raw file):
Previously, shitz wrote…
This should not generate both auto-completion scripts by default. I suggest you add a -zsh flag and by default only generate the bash auto-completion file, since that is more common.

Done.

Comments from Reviewable

Review status: 0 of 2 files reviewed at latest revision, 3 unresolved discussions.

go/tools/scion-pki/internal/cmd/cmd.go, line 49 at r2 (raw file):
Previously, shitz wrote…
There is no -bash flag

Done.

go/tools/scion-pki/internal/cmd/cmd.go, line 61 at r2 (raw file):
Previously, shitz wrote…
scion-pki autocomplete should output which files it has generated and also what steps the user should take to enable autocompletion. Actually, I'd say you can remove the Long help description and just show either the bash part or the zsh part on the console after every run of scion-pki autocomplete

Done.

Comments from Reviewable

Review status: 0 of 2 files reviewed at latest revision, 3 unresolved discussions.

go/tools/scion-pki/README.md, line 150 at r3 (raw file):

./bin/scion-pki autocomplete

--zsh flag missing here

go/tools/scion-pki/internal/cmd/cmd.go, line 61 at r2 (raw file):
Previously, jinankjain (Jinank Jain) wrote…
Done.

Move the instructions to the top level const block.

Comments from Reviewable

Review status: 0 of 2 files reviewed at latest revision, 4 unresolved discussions.

go/tools/scion-pki/README.md, line 150 at r3 (raw file):
Previously, shitz wrote…
--zsh flag missing here

Done.

go/tools/scion-pki/internal/cmd/cmd.go, line 45 at r2 (raw file):
Previously, shitz wrote…
Generate autocompletion files for bash and zsh.

Done.

go/tools/scion-pki/internal/cmd/cmd.go, line 61 at r2 (raw file):
Previously, shitz wrote…
Move the instructions to the top level const block.

Done.

Comments from Reviewable


Cors issue

having some weird cors issues with this plugin.
Any way around it guys?
news:1 Fetch API cannot load https://api.instagram.com/v1/users/self/media/recent/?count=2&access_token=TOKEN. No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:3000' is therefore not allowed access. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.

Hey, are you trying to use this module in a browser directly?


Duplicate required properties returns missing property error

Using AJV 6.1.1
If I have the following schema:
var loginSchema = {
  "$schema": "http://json-schema.org/draft-06/schema#",
  "id": "AuthenticationRequest",
  "title": "Authentication Request",
  "description": "Request body for an authentication request",
  "type": "object",
  "properties": {
    "provider": { "type": "string", "minLength": 2, "maxLength": 50 },
    "username": { "type": "string", "minLength": 2, "maxLength": 50 },
    "password": { "type": "string", "minLength": 7, "maxLength": 100 }
  },
  "required": ["provider", "username", "password"],
  "additionalProperties": false
};
var validateLoginSchema = ajv.compile(loginSchema);

if I send a body of the request with duplicate required properties:
{
    "provider": "local",
    "username": "User A",
    "username": "User B",
    "username": "User C",
    "username": "User D",
    "username": "User E",
    "username": "User 1",
    "username": "User F",
    "username": "User G",
    "password": "password1"
}

The errors ajv gives back is:
{
    "errors": [
        {
            "keyword": "required",
            "dataPath": "",
            "schemaPath": "#/required",
            "params": {
                "missingProperty": "provider"
            },
            "message": "should have required property 'provider'"
        },
        {
            "keyword": "required",
            "dataPath": "",
            "schemaPath": "#/required",
            "params": {
                "missingProperty": "username"
            },
            "message": "should have required property 'username'"
        },
        {
            "keyword": "required",
            "dataPath": "",
            "schemaPath": "#/required",
            "params": {
                "missingProperty": "password"
            },
            "message": "should have required property 'password'"
        }
    ]
}

I would expect an error stating something like duplicate required properties rather than missing properties as all the required properties actually are specified.

There is no such thing as the duplicate properties in JS objects, later ones simply override previous property definitions. I think you may not be passing the object above to validateLoginSchema function, you may be passing something else. Could you please submit a working code sample to runkit.com?

Seems you are correct.  On runkit it works fine when I define the object passed to validate.  The only difference I see is that in my code I am passing req.body to the validate method that is coming on the post.  Thanks for the quick response.

req.body can be an empty object - do you have a parser?

Should have done more investigating prior to posting here.  That specific API call mistakenly specified the body content as text instead of JSON.  That was the problem.  Sorry to waste your time man.


DispHTMLDOMTextNode Object doesn't support this property or method

Affects: IE8 Angular v1.2.28
I have found which seems an historical bug, I'm not clear the origin (but I know perfectly how to reproduce) , I didn't found any previous issue like this.
This happens only in IE8.
Problematic code: ( http://jsfiddle.net/yfg1kjhf/1/ )

            Bar: -{{vm.bar}}-

            <input ng-model="vm.bar" />

Test template is "

".
Reporte error: Object doesn't support this property or method. References to test template element.
The problem is the space between <input> and
.
How to reproduce:

open http://jsfiddle.net/yfg1kjhf/1/embedded/result/ with IE10 in IE8 compatibility mode (it should fail)

Contraexample which works (does not have the space):

http://jsfiddle.net/yfg1kjhf/embedded/result/ (it should work)

Where seems to be originated?
While debugging I found the function jqLiteExpandoStore which has as element a DispHTMLDOMTextNode and it tries to assign a new property for that object: element.ng339 = value; but IE8 does not allows to add/modify ng339 property. In this point fails.
What I do not know why only fails in this case. And not sure that it is because ng-transclude also.

I am not sure if this is fixable in Angular 1.2.x (nor do I remember the exact cause of this), but happens under the following circumstances:

When there is white-space before and/or after (can't recall) certain elements:

<input> is affected for sure
<select> might be affected also

--
You can see a version of your fiddle working in IE8 by removing the white-space around <input> here.
(Of course, this is just a workaround, not a proper solution.)

Yep,
this workaround is just the same that I have apported.
The problem is that the error is difficult to understand, and of course, it is IE8/10, difficult to debug. May be a try {} catch with better error message can be added.

I am gonna close this, as 1.2.x is a legacy branch (the last one to support IE8) and only receives critical bug fixes.


How to draw the Average feature maps from your paper?

Hi, I am wondering which tools/library are you using to draw the average feature maps.

Hi, I am wondering which tools/library are you using to draw the average feature maps.

Hi, bro. I have the same problem with you. Have you solved it?


Make Image Sequence... virtual stack work properly

When opening an Image Sequence with SCIFIO, it opens as a CellImg automatically if needed - but it is still fully iterated over via FolderOpener.
Opening an Image Sequence with the virtual stack option enabled currently bypasses the LegacyOpener mechanism.
So we should figure out how to get the virtual stack option to open with SCIFIO, and as a CellImg.

Note that running Image Sequence... with SCIFIO enabled right now simply failed to work for me (trying to open a sequence of Blobs written as text images)


CIF-1680 - Add htl-maven-plugin to CIF ui.apps

only enabled the validation of scripts, not the generation and compilation of scripts in java (IMHO it doesn't bring much).

Merged, there is nothing to verify here.