Netflix audio and screengrab are blocked

When trying to mine a subtitle, the audio file created does not contain any sound and the screenshot does not contain the video image.

www.netflix.com-1697776716671.log

I've seen others have this issue due to hardware acceleration settings. Have you tried messing with those?

I see. I haven't touched the default settings before but turning the hardware acceleration off seems to make asbplayer work again


style(elements|ino-segment-button): update design

Closes #690
Proposed Changes

update the style of the ino-segment-button & ino-segment-group

remove depricated css properties: --ino-segment-button-checked-shadow & --ino-segment-button-hover-shadow
add 2 new css properties to accomodate new design: --ino-segment-button-focus-border & --ino-segment-button-pressed-color
TODO:

[x] wrap a border around the children of the ino-segment-group
[x] delete comments & redundant css variables ($left-segment-border...)

According to the design, the default segment button should be of 44px height


Request Denied

My excel sheet is saying request denied on Geocoding, can u suggest any solution!

Yeah, looks like either the free limit is removed or the API has changed.
Let me take a look this weekend Prashant
If you have any ideas, would be very glad for inputs please. Thanks!
On Thu, 2 May 2019 at 12:44 PM, PrakharPrashantSingh <
<EMAIL_ADDRESS>wrote:

My excel sheet is saying request denied on Geocoding, can u suggest any
solution!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/gramener/geocode-excel/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAPR5W67FFNLJJEBVGACJDPTKIGNANCNFSM4HJ3432A
.

Okay, This is a very fast reply, i wasn't expecting this fast!
Thank you sir

Any update on this?

@zkenstein As with all weekends, that one flew past too quickly, and I wasn't able to solve the problem.
I WAS able to identify it. The API has changed. This will require 2-4 hours of time to solve, which is proving a little difficult.
However, I find that live-coding sessions help me prioritize. So if you or @PrakharPrashantSingh would like to join me for an hour, we can fix it together :-)

Hi @sanand0  sir, i didn't understood what you mean by " I find that live-coding sessions help me prioritize." but yes if there is anything in which i can contribute, i am ready!
Let me know what can i do....

I've updated the function to accept a Google API key. It can also now use the Nominatim API which is free. Hope this helps!
Closed via 338b24b4


[Doc Improvement][Collaborative stage view image update][3173133]

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/tabs/tabs-link-unfurling.md
98
2
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.


Add -- to git log call

Closes #94

Coverage remained the same at 63.344% when pulling 374981fd60c0116e861d598eec763cc2b8165189 on git-log-upgrade into<PHONE_NUMBER>0289c0df4f2b3f01ad6251b07da962 on master.


oneOf array of simple types bug - propose fix

basicSchemaDefn = """{
'$schema':'http://json-schema.org/draft-04/schema#',
'title':'Test',
'properties':{
'SimpleArrayOfNumberOrString':{
'$ref':'#/definitions/simparray'
}
},
'required':[
'SimpleArrayOfNumberOrString'
],
'type':'object',
'definitions':{
'simparray':{
'oneOf':[
{
'type':'array',
'items':{
'type':'number'
}
},
{
'type':'array',
'items':{
'type':'string'
}
}
]
}
}
}"""
basicSchema = json.loads(basicSchemaDefn.replace(''', '"'))
builder = pjs.ObjectBuilder(basicSchema)
ns = builder.build_classes(no_extra_parameters = True)
ns.Test().from_json('{"SimpleArrayOfNumberOrString" : [0, 1]}')

Proposed fix in classbuilder.py validate:
elif isinstance(val, list):
if len(val) and hasattr(val[0], "validate"):
for subval in val:
subval.validate()
else:
handle basic types
setattr(self, prop, val)

Hi.
Not sure you have the right repo here.
If you do, can you update your description to be runnable?
Thanks!

Hi,
Sorry I missed the imports at the top, otherwise it is runnable. Just paste into a python script.
The failure is against your latest release.
Thanks.
Yours,
Graham

That is 3.0.0a3, and the title ' handle basic types above' in the proposed solution is of course a comment mangled by the wiki

I am deeply sorry - you are quite right, I am using 3.0.0a3 in conjunction with python jsonschema object and the bug is in their library. Many thanks for your prompt reply, and thank you for the high quality of your library !

No worries, and thank you!
On Mon, Oct 8, 2018, 19:35 GrahamJB1<EMAIL_ADDRESS>wrote:

Closed #476 https://github.com/Julian/jsonschema/issues/476.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Julian/jsonschema/issues/476#event-1891510855, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAUIXqaXeeggOJGKoIYigirAN3XmDwZ-ks5ui-FOgaJpZM4XNJy5
.


Adding support for FileSystem

I've added a filesystem class and linked it to the Console.
This is currently just a Proof-Of-Concept, a basis for future development. A complete linux file structure is therefor not created.
In the filesystem class I've added some commands to intereact with the files and directories.
cat /etc/passwd should ouput the root user.
The filesystem is modelled after how linux and windows operate, with linked-list. A inode/mft entry list, and a data entry list.
Currently there is no way to add content to files. To replicate linux, the console needs to support StdIn, StdOut, and StdErr; 0, 1, & 2.
If we can implement some concepts of that, it's easier then to add piping ( "|", "<", ">"), and add content to files.

This is really neat!

What you've done for me seems more like an in-memory fs (or fs emulator), correct me if I'm wrong. Because you could just pass commands like ls/cd/... to OS.execute.

Another thing is, this probably won't make into default console configuration.
Q: Why?
A: I want to keep it as simple and flexible as possible, so that on the start you get only basic stuff but it can do a lot.
Q: What is default configuration?
A: This is what you get as soon as you install console and run your game. But that doesn't mean your FS wont be available in the console git repo (if you still would want it to). At some point in time I would love and need to introduce plugins (see issue #24). This would allow greater flexibility and smaller default configuration. Everything that isn't a core functionality could be moved to a plugin.

To replicate linux, the console needs to support StdIn, StdOut, and StdErr

Totally agree, I'll create and issue for that. If I would have time I would do that my self.

Thanks, this is an awesome POC, would be cool see it developed further.
Best.


Add system health support

Add support for https://www.home-assistant.io/integrations/system_health/

@mickemartinsson Can you have a check the new system_health translation in sv.json? I used google translate...

That translation works great!!


feat(opendataset): add dataloader for SVHN dataset

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5340

Relevant Lines:
6442

💛  - Coveralls


Context Guide: mention that machine.context is the same as machine.in…

…itialState.context

Not sure if this is helpful. .initialState.context is more explicit of what context that is.

@davidkpiano makes sense.
But I think it would be nice if you could mention somewhere that machine.initialState.context is the same as machine.context.
In the Machines Guide we have the following snippet:
const testLightMachine = lightMachine.withContext({
  // merge with original context
  ...lightMachine.context,
  elapsed: 1000
});

So after reading both guides (Context and Machines), I was wondering if machine.initialState.context and machine.context they were the same thing.

Good point. I'll add it to the docs.


fix: CameraPostionBack to LLCameraPositionRear

fix: CameraPostionBack to LLCameraPositionRear in description example

This is already fixed, thanks!


dynamic import not work ?

Description
Hi, I use webpack4 and loadable-components for code splitting and dynamic import , I see the webpack code splitting success , but dynamic import not work , here is my code
// router.js
import loadable from 'loadable-components'
export const Home = loadable(() => import('./Home'))
export const About = loadable(() => import('./About'))
export const Topics = loadable(() => import('./Topics'))

// RouterLink.js
<BrowserRouter>
    <Route exact path="/" component={Routes.Home} />
    <Route path="/about" component={Routes.About} />
    <Route path="/topics" component={Routes.Topics} />
</BrowserRouter>

// App.js
class App extends Component {
    render(){
      <RouterLink />
    }
}

DId the loadable-components need config babel ? I try to config loadable-components/babel  in .babelrc ,  but it not work , here is my .barbelrc
{
  "presets": [
    [
      "env",
      {
        "loose" : true,
        "modules": false
      }
    ],
    "stage-0",
    "react"
  ],
  "plugins": [
    "loadable-components/babel",
    "react-hot-loader/babel"
  ]
}

Environment
node: 8.9.0
react-hot-loader: 4.1.1
webpack :4.6.0
webpack-dev-server: 3.1.3
loadable-components: 1.4.0

You don't need Babel plugin if you are not doing server-side rendering. What is the error?

@neoziro not error , just dynamic import not work. I just config like Get Start in README. Do I need config other ?

If you are using react-hot-loader this is the expected behaviour.

@neoziro Sorry for a delay . I use react-hot-loader for hot reload ,  it means if I use this , all components are loaded , i need add .load()?
I use react-router-config to Static Routes , code like this , how to configure to implementation dynamic import ?


Constrain docks to window size in single window mode

Fixes: https://github.com/godotengine/godot/issues/44439
I created a separate node which listens for the window_input event. If the editor is in single_window_mode it will prevent the window from being positioned outside of the editor window.
https://user-images.githubusercontent.com/6378569/111897681-a3df5680-89de-11eb-8b8d-a8bc16629f08.mp4
Note: You'll notice an unrelated bug. Dragging a window after dragging a different window will move that window to the previous windows position.

Closing this for now in favor of using the clamp_to_embedder property.


Chroot Shortcut

Is there a way to launch my Chroot from a shortcut on my Apps menu?
Much thanks!

Nope, see https://github.com/dnschneid/crouton/issues/1468#issuecomment-77066112


Name shows "null null" or empty when no set name on Quotes

When a customer or admin has no set names the quote details page shows "null null" or sometimes blank which confuses the process.

Easy fix by just going to the details quote component and just adjusting the logic there no need for this my bad :).


for some reason the control center doesn't show notification

control center doesn't show notification i tested it with notify-send it just wont work even getting a notification it doesn't show up, any clue what's going on?

can u do this and check
So you just go into ~/.config/eww/Control-Center/scripts and run the logger.zsh file
Can u tell me if there is any error?

here's when I run the logger.zsh

That means that the file is working correctly..

Can u try to clear the notifications and then re-send one?

sorry for the weird setup I was using my other hard drive

Well, if you are using my dunstrc, then all you need to do is just press the clear all button (the one to the right of Notifications), if you are not using my dunstrc, then u need to append 3 lines to yours so that it executes the script everytime a new notification is received.
[logger]
summary = "*"
script = "~/.config/eww/Control-Center/scripts/logger.zsh"

Then press the clear all button and try to send a notification 😄

Oh that's why, ok Ill try if this works

sorry for the long reply, I tried it but it still won't work but checking the logs it seems the it doesn't recognize scroll as a widget for some reason

I'll try doing some test if there's something missing, the script works it's just eww

just figured it out that im using a old version of eww

Oh, I guess upgrading it might do it

lol thanks for the help, you can close this issue

Sure :)


Failed to compile with "sqlx = 0.8.0"

My bin crate uses both apalis and sqlx, when I upgraded sqlx to v0.8.0,  it fails to  compile.

https://github.com/geofmureithi/apalis/pull/380

Oh, it's duplicate, will track there.

Resolved by #384 Please update to v0.6 to use sqlx 0.8.

:tada:


[SUPPORT]The default behavior of spark sql insert into has changed from upsert to insert?

I looked into the source code of Hudi 0.15, and find the option hoodie.sql.insert.mode is deprecated.
  @Deprecated
  val SQL_INSERT_MODE: ConfigProperty[String] = ConfigProperty
    .key("hoodie.sql.insert.mode")
    .defaultValue("upsert")
    .markAdvanced()
    .deprecatedAfter("0.14.0")
    .withDocumentation("Insert mode when insert data to pk-table. The optional modes are: upsert, strict and non-strict." +
      "For upsert mode, insert statement do the upsert operation for the pk-table which will update the duplicate record." +
      "For strict mode, insert statement will keep the primary key uniqueness constraint which do not allow duplicate record." +
      "While for non-strict mode, hudi just do the insert operation for the pk-table. This config is deprecated as of 0.14.0. Please use " +
      "hoodie.spark.sql.insert.into.operation and hoodie.datasource.insert.dup.policy as you see fit.")

Prior to hudi 0.14.0 ,the default behavior for spark sql insert into statement is doing upsert which will not introduce duplicates.In the doc ,it says that this option is replaced with two options: hoodie.spark.sql.insert.into.operation and  hoodie.datasource.insert.dup.policy.
The definition for hoodie.spark.sql.insert.into.operation is as follows, the default value for the spark.sql.insert.into.operation has been changed to insert.
  val SPARK_SQL_INSERT_INTO_OPERATION: ConfigProperty[String] = ConfigProperty
    .key("hoodie.spark.sql.insert.into.operation")
    .defaultValue(WriteOperationType.INSERT.value())
    .withValidValues(WriteOperationType.BULK_INSERT.value(), WriteOperationType.INSERT.value(), WriteOperationType.UPSERT.value())
    .markAdvanced()
    .sinceVersion("0.14.0")
    .withDocumentation("Sql write operation to use with INSERT_INTO spark sql command. This comes with 3 possible values, bulk_insert, " +
      "insert and upsert. bulk_insert is generally meant for initial loads and is known to be performant compared to insert. But bulk_insert may not " +
      "do small file management. If you prefer hudi to automatically manage small files, then you can go with \"insert\". There is no precombine " +
      "(if there are duplicates within the same batch being ingested, same dups will be ingested) with bulk_insert and insert and there is no index " +
      "look up as well. If you may use INSERT_INTO for mutable dataset, then you may have to set this config value to \"upsert\". With upsert, you will " +
      "get both precombine and updates to existing records on storage is also honored. If not, you may see duplicates. ")

The  definition for hoodie.datasource.insert.dup.policy is as follows, the default value is none
  val INSERT_DUP_POLICY: ConfigProperty[String] = ConfigProperty
    .key("hoodie.datasource.insert.dup.policy")
    .defaultValue(NONE_INSERT_DUP_POLICY)
    .withValidValues(NONE_INSERT_DUP_POLICY, DROP_INSERT_DUP_POLICY, FAIL_INSERT_DUP_POLICY)
    .markAdvanced()
    .sinceVersion("0.14.0")
    .withDocumentation("**Note** This is only applicable to Spark SQL writing.<br />When operation type is set to \"insert\", users can optionally enforce a dedup policy. This policy will be employed "
      + " when records being ingested already exists in storage. Default policy is none and no action will be taken. Another option is to choose " +
      " \"drop\", on which matching records from incoming will be dropped and the rest will be ingested. Third option is \"fail\" which will " +
      "fail the write operation when same records are re-ingested. In other words, a given record as deduced by the key generation policy " +
      "can be ingested only once to the target table of interest.")

With default value for the above two options, it means that the default behavior of spark sql insert into statement has been changed for upsert to insert（may introduce duplicates）?
I am not sure whether I have understood correctly. If i am correct, then this change is a broken change, some people using older version are using spark sql insert into to do upsert which will not introduce duplicates, but after upgrading to 0.14.0+, the default behavior is using insert which may introduce duplicates?

yeah, if there is no record key definition it will fall back to pk-less, whereas it's INSERT.

yeah, if there is no record key definition it will fall back to pk-less, whereas it's INSERT.

Thanks @danny0405 for the reply, but I don't get what you said.
With the two default options:  hoodie.datasource.insert.dup.policy=none and hoodie.spark.sql.insert.into.operation=insert,
what is the behavior if record key definitions have been defined, insert or upsert?

The operation is taken the highest priority if it is set up explicitly, so it is still INSERT instead of default upsert.

@bithw1 Closing this. Please reopen in case of any more queries.


Harden security headers

The goal of this PR is to harden security headers on the app as the DevSecOps team officially onboards it.
One of the goals is to come up with a strict content security policy, which required some changes in the code:

removing FontAwesome as it doesn't seem to be used and the JS file we load dynamically injects a <style> element with all the CSS rules inlined
tweaking create-react-app through environment variables to prevent things from being inlined (see official docs)

IMAGE_INLINE_SIZE_LIMIT=0 to ensure no images are inlined during the build; and
INLINE_RUNTIME_CHUNK=false to prevent the bit of webpack runtime to be inlined in the HTML file

Quick edit: right now the CSP policy is in "report only" mode, so it gives us some time to analyse if we missed anything before switching to the "enforce" mode
Hopefully the rest of the commits are self-explanatory.
Holler out if it's not the case 👍

Dummy comment to trigger the PR build in Azure DevOps

Closing to open a new one to get the Azure DevOps build to run.


Redundancies

Hi.
I copy the host rules into Redundant checker and I ended up with 43678 redundancies and 341 warnings.
Is it normal?

Hi @Quorum75 the raw amalgamated lists we publish are supposed to be de-duplicated.  What redundancies are you seeing?

This is a bum steer.  Thank you @Quorum75.  Closing.

Do you mean I should not use it? What's bad about this program?

For starters, @Quorum75 I have no idea what you are referring to.  I have no idea what you mean by 'redundancies'.
We have zero duplicates.
I suspect you posted your question in the wrong repo.  This has nothing to do with us.

@Quorum75 That tool is for adblock lists. This is not an adblock list.
This script checks if there are any redundant rules in your Adblock Plus filter list. A rule is redundant if the content it blocks, hides or excludes is also blocked, hidden or excluded by another rule.
There is no such thing as a "redundant rule" in a host file. Blocking google.com will not automatically block maps.google.com for example.

We have zero duplicates.

KADHosts sometimes duplicate blocked domain when:

CERT Polska
Private Company "Legalnie w sieci"

banned same fake/fraud "shop" in same time (intervals):

11:00 - 14:59 (4h - "Europe" day)
15:00 - 18:59 (4h - "Europe" day)
19:00 - 10:59 (16h - "Europe" night)

KADHosts no have GNU/Linux .sh script to detect these situation and now is allways manual fixed after minimum 4 hours - few days by human.


add script to download all open access and foaf:page in corpus.jsonld

Added a script to download all resource files.
In this script, we assume that all publication open access uri returns pdf files, and dataset foaf:page returns either html or pdf file.
To run using default corpus file (corpus.jsonld) and default output_dir (resources/):
python download_corpus_resources.py

Known issue when running this script on v0.1.5 corpus file:

unable to download publication pdf files embedded in an HTML page (for example: on epdf links in onlinelibrary.wiley.com, reader.elsevier.com html file)

Great! Many thanks


Detect if app is served over HTTPS and adjust WS protocol accordingly

When sparklint is exposed through a proxy using HTTPS, modern navigators refuse a direct WS connection and require WSS.
This patch adds a protocol detection test to handle this case when building WebSocket URL.

Thanks for reporting! This update looks simple and risk free to me. Merging it and bumping up version...


Correct the README and the Repo Description

The README states:

The repository contains all the documentation found on docs.aion.network.

The Repo description states:

This is where all the docs for the Aion Network are kept! https://docs.aion.network

This should be corrected, as this repo affects the documentation beta site (see #7)

The repository description has been changed to reflect the current state and purpose of this repository.


TypeError: $auth.isAuthenticated is not a function at redirectToLogin

I'm submitting a ...

[X] bug report

Provide information & explanation

Hi, i'm trying the framework and i have found a bug in the routes.run.js file.
If implements: "data: {auth: true}" in the routes.config.js in theory the event "redirectToLogin" must be fired, but when arrive to the funcion "$auth.isAuthenticated()" it crashed.
Attach a IMG: https://s32.postimg.org/s9gmn42th/Captura_de_pantalla_2016_07_12_a_las_16_25_27.png
I've change this code:
export function RoutesRun($state, $transitions) {
    'ngInject';

    let requiresAuthCriteria = {
        to: ($state) => $state.data && $state.data.auth
    };

    let redirectToLogin = ($auth) => {
        'ngInject';
        if (!$auth.isAuthenticated()) {
            return $state.target('app.login', undefined, {location: false});
        }
    };

    $transitions.onBefore(requiresAuthCriteria, redirectToLogin, {priority:10});

}

By this:
export function RoutesRun($state, $auth, $transitions) {
    'ngInject';

    let requiresAuthCriteria = {
        to: ($state) => $state.data && $state.data.auth
    };

    let redirectToLogin = () => {
        'ngInject';
        if (!$auth.isAuthenticated()) {
            return $state.target('app.login', undefined, {location: false});
        }
    };

    $transitions.onBefore(requiresAuthCriteria, redirectToLogin, {priority:10});

}

I've change the $auth var to inject instead the "redirectToLogin".
And now (with my modifications) it works perfectly.

Laravel & Angular version: 3.2.1

Thanks @jeslomo
I'll fix it soon

Hi @jeslomo  when i'm trying your code i got this error.
angular.js:68Uncaught Error: [$injector:unpr] Unknown provider: $transitionsProvider <- $transitions

Hi @proxnoob06
This error is the unique error in javascript console?
I think you should have more errors or problems in the code that are preventing you inject this library
Could you give us more information?

Hello @jeslomo,
in my index.modules config
angular.module('app.run', ['ui.router'])
my  Routerun config
I tried to inject $transitions
RoutesRun($rootScope, $state, $auth, $transitions)
and i got this error
angular.js:68Uncaught Error: [$injector:unpr] Unknown provider: $transitionsProvider <- $transition
https://s31.postimg.org/6rnfrs9qj/transitions.png

Hi,
I see the problem.
In my index.modules.js i've this:
`angular.module('app', [
'app.run',
'app.filters',
'app.services',
'app.components',
'app.directives',
'app.routes',
'app.config',
'app.partials'
]);
angular.module('app.run', []);
angular.module('app.routes', []);
angular.module('app.filters', []);
angular.module('app.services', []);
angular.module('app.config', []);
angular.module('app.directives', []);
angular.module('app.components', [
'ui.router', 'ngMaterial', 'angular-loading-bar',
'restangular', 'ngStorage', 'satellizer'
]);
`
Witch version are using?

the second one..
angular.module('app.run', []);
angular.module('app.routes', []);
angular.module('app.filters', []);
angular.module('app.services', []);
angular.module('app.config', []);
angular.module('app.directives', []);
angular.module('app.components', [
'ui.router', 'ngMaterial', 'angular-loading-bar',
'restangular', 'ngStorage', 'satellizer'
]);

version of laravel angular or ui router?
laravel angular: lastest version 3.2
ui router: ~0.2.15 already tried ~0.3.1 same error

Version of laravel angular, and witch branch has use?

That cause we are Using Ui-Router 1.0.0-alpha3
https://github.com/jadjoubran/laravel5-angular-material-starter/pull/283
https://github.com/jadjoubran/laravel5-angular-material-starter/pull/296

Hello @flick36 ,
Thank you very much.
i already update into 1.0.0-alpha3 version but i got several issues into $state with params and notify.
code:
this.$state.go('dashboard.test',{param1:param1},{notify:false})
in my example above my current state become disabled and unclickable but when i use the old version '0.2.15' it works fine.

That's because notify will broadcast $stateChangeStart and $stateChangeSuccess events. And those events stateChange* as you can read on https://github.com/angular-ui/ui-router/releases/tag/1.0.0alpha0

All state events, (i.e. $stateChange* and friends) are deprecated and disabled by default; re-enable them by including the ng1/stateEvents.js file, and depend on the ui.router.state.events angular module in your application.

So, notify won't work, altough i encourage you to update to the new beta you might have a hard time triyin yo figure out how to make things work, because there's no documentation yet, but it will be a must-have changue when the it reaches stable version. So if you don't want to learn all over again, you can just rollback the changues in https://github.com/jadjoubran/laravel5-angular-material-starter/pull/283 and use the 0.2.15 version that you are most used to...

@jeslomo thanks again for reporting this
I wanted to fix it in 3.3.0 but I just noticed that it has already been fixed before
@flick36 thanks as well for your help 😄

Original issue is happening on fresh install.
This happends once I set auth:true
.state('app', {
   abstract: true,
   data: {auth: true},
   views: {
     header: {
        templateUrl: getView('header')
    },
...

That's because you don't put auth:true on the abstract state... you put auth in any child state for example the landing page
https://github.com/jadjoubran/laravel5-angular-material-starter/issues/346#issuecomment-228129322

@cyberkiko updated comment in routes.config.js, because it used to be in the abstract state

@jadjoubran & @flick36 thanks that works, but also I did update the routes.run.js file as was suggested in the first comment, else wouldn't work.

Yes that's because this has been probably merged after you installed your version of this starter

I am getting this error because for the most obvious of reasons: $auth.isAuthenticated is not a function.
I changed routes.run.js on line 10 to:
if (!$auth.isAuthenticated) { .. }
And all good after that,
Thanks Jad btw!


advise to support decompress gzip response body

because some server will send compressed data back, so i very strongly to advise rawhttp to support decompress data automatically

@Nyx2022 please recreate the issue with examples and more details.


[BUG] Marketplace is balnk

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to ' localhost3000'
Click on '.marketplace'
Scroll down to '....'
See error blank page :

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Flow
If applicable, add exported flow in order to help replicating the problem.
Setup

Installation [e.g. docker, npx flowise start, pnpm start]
Flowise Version [e.g. 1.2.11]
OS: [e.g. macOS, Windows, Linux]
Browser [e.g. chrome, safari]

Additional context
Add any other context about the problem here.

Which version of Flowise are you using? How are you installing it?
Refer to this for fix: https://github.com/FlowiseAI/Flowise/issues/3583


Support bulk modifications in the fixer with the XML API's "edit"

https://docs.paloaltonetworks.com/pan-os/9-1/pan-os-panorama-api/pan-os-xml-api-request-types/configuration-api/edit-configuration.html

https://github.com/PaloAltoNetworks/pan-os-python/blob/develop/panos/base.py#L1840

WONTFIX. Using PA commands prevents deleting objects or the like that are in use. Bulk modifications run a risk of creating an inconsistent XML file.


Add conditional installation of sphinx to build_locally.sh

I could also try and check if sphinx is installed with
test -x `which sphinx-build` || pip install sphinx==1.8.5
I am specifically using pip so that I can ensure we are using the same version as ROS1, but if you would prefer we use apt we just need to double check which version gets installed.

@JafarAbdi would you mind taking at look at this and we can get it merged?


Add missing include of DecodableList in AttributeCache

For users using Decode with DecodableList, it fails to compile as the include is missing.
Problem

For users using Decode with DecodableList, it fails to compile as the include is missing.

Change overview
Add missing include of DecodableList
Testing
How was this tested? (at least one bullet point required)

Compile and build on local environment.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Fixed windows_share resource is not idempotent

Signed-off-by: sanga17<EMAIL_ADDRESS>
Description
windows_share is not idempotent when using local groups
Related Issue
Fixed: 1) https://github.com/chef/chef/issues/9242
2) https://github.com/chef/chef/issues/10226
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[x] All commits have been signed-off for the Developer Certificate of Origin.

review comment are addressed for related PR: https://github.com/chef/chef/pull/9796

This really needs at least unit tests (if not figuring out how to do real functional tests, but i don't know how you could setup a 'test' SMB share and destroy it reliably on every test run).  For unit tests the powershell command can be mocked with the output that it would normally produce.  It should test the cases where the chef user requests the same user on multiple rights, and should test the domain mangling.

@tas50 and @lamont-granquist, I have rebased the PR so please have a look, Thanks!!

@tas50 and @lamont-granquist  Please have a look into this PR. Thanks!!

@tas50 @lamont-granquist @phiggins @marcparadise Please have look into the PR. Thanks!!

@sanga1794 can you please rebase this and push it again?

We just added some important changes and updates to the code base in the last 24 hours. Can you please rebase and push one final time? I just approved your PR

Topic for review for 2022-12-13 PR meeting


POST request not correct work in admin

Hi everyone
I need your help with sending data from form by POST method in admin
Sometimes form submit work as reload without POST data
Routes
Route::middleware('verify.shopify')->group(function (){
	Route::get('/products', [ProductController::class, 'index'])->name('products');
	Route::match(['get', 'post'], '/products/create', [ProductController::class, 'create'])->name('create_product');
});

ProductController create
$data = array();

if(!empty($_POST))
{
    $validator = Validator::make($request->post(), [
        'title' => 'required|max:255',
        'subtitle' => 'required|max:255',
        'description' => 'required'
    ]);

	$data['errors'] = $validator->errors()->all();

    if(empty($data['errors']))
    {
    	// Saving product

    	redirect(route('create_product', ['submit' => 'success']))->send();
	}
}

if($request->get('submit') == 'success'){
	$data['success'] = 'Product created successfully';
}

return view('create_product', $data);

What is wrong with my code?
I tried to make a separate route for the post request, but it also does not work for how much shopify makes a request with GET host,shop,token data
Please help

Hi @airarm
What version of laravel-shopify are you using?

@inaryu
laravel/framework => 8.0
osiset/laravel-shopify => 17.1
php => 7.3

@airarm
If you are using the above version, a Session Token is required when making a request.
https://github.com/osiset/laravel-shopify/wiki/Authentication-Process#non-spa


Support Iceberg Partition

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I would like to ask how the tablePartition in OM supports Iceberg's Partition? For Iceberg, there is no case where an interval corresponds to multiple columns. On the contrary, a table corresponds to multiple intervals, such as hour, day and identity, etc.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Support Iceberg Partition. Maybe like this:
{
  "tablePartitions": [
    {
      "column": "",
      "interval": "",
      "invervalType": ""
    },
    {
      "column": "",
      "interval": "",
      "invervalType": ""
    }
  ]
}

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

hey @TheFu527 thank you for opening an issue. Could you elaborate on what you are looking for regarding this? Thank you.


favorite: saga-ize add/ignore + generating promise helpers in rpc layer

@keybase/react-hackers this is ready to look at

👍 % C


Equivalent of AmazonSQSBufferedAsyncClient Java based implementation in dotnet

I would like to implement simple request batching using client-side buffering—calls made from the client are first buffered and then sent as a batch request to Amazon SQS. It seems Java already has an implementation for such a use case as described here:
https://docs.aws.amazon.com/AWSSimpleQueueService/latest/SQSDeveloperGuide/sqs-client-side-buffering-request-batching.html
Unfortunately, I couldn't find an equivalent of AmazonSQSBufferedAsyncClient in aws-sdk-net.
Does it exist or are there any alternatives?

This is a :question: general question

Hi @niklr,
As per documentation/example, the Java SDK class AmazonSQSBufferedAsyncClient allows for simple request batching using client-side buffering. Client-side buffering allows up to 10 requests to be buffered and sent as a batch request, decreasing your cost of using Amazon SQS and reducing the number of sent requests.
While I could not find a similar named class in .NET SDK, if you refer the documentation Sending an Amazon SQS Message Batch, the  SendMessageBatch() method of AmazonSQSClient class offers somewhat similar functionality. Please let me know if it works for your scenario.
Thanks,
Ashish

Hi @ashishdhingra
Unfortunately this does not work for my scenario. The buffering is the key part. I would like to delay SendMessageBatch() until the buffer contains 10 messages or a certain amount of time has passed (e.g. 5 seconds).

Hi @niklr,
Thanks for your response. Unfortunately, I do not see an equivalent class in .NET for AmazonSQSBufferedAsyncClient. The SendMessageBatch() is the closest match, but looks like it doesn't support buffering.
I was looking at the code for Java SDK's AmazonSQSBufferredAsyncClient class and it appears that it maintains a buffer of messages. While sending the message, it still uses realSQS.sendMessageBatchAsync(sendMessageBatchRequest). Also this class is just an extension of AmazonSQSAsync class. May be you could use the similar idea in .NET and try extending AmazonSQSClient class to implement custom buffered implementation.
Hope this helps. Please let me know if we could close this issue if this provides some guidance.
Thanks,
Ashish

I would love to see this feature implemented in .NET as well. Feel free to close it if not worthy to put on the backlog/roadmap.
In any case, thanks for the guidance!

Hi @niklr,
I think this would be good to have feature in .NET SDK. Hence I have converted this guidance issue into feature-request. I'm not sure how sooner it would be implemented, but I will have a plan to discuss with development team to put them in their backlog.
Thanks,
Ashish

Hi @ashishdhingra!
Please reopen the issue, I also interested in such feature in .Net

@ashishdhingra do you know which team owns this feature ? else I can take a look how to enable this... we also have one internal use case...


Mute remote participant

Is there a way to mute/unmute remote participant in Azure Communication Services video calling? If there is such ability - does it work the same way like in MS Teams? As a host participant I would like to mute other participants during the call.

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

ID: 45723fb2-21e8-8125-5c0d-1a5fba357fe3
Version Independent ID: f8208916-5b63-1379-8197-6dc497af1434
Content: Quickstart - Use the Azure Communication Services calling client library - An Azure Communication Services quickstart
Content Source: articles/communication-services/quickstarts/voice-video-calling/calling-client-samples.md
Service: azure-communication-services
GitHub Login: @mikben
Microsoft Alias: mikben

Hi @andriyovchar - thanks for submitting this issue. This feature isn't yet available but muting remote participants will be possible in a future release. Unmuting remote participants is unlikely to be supported given the privacy concerns that are associated with this feature. Don't hesitate to reach out with any other questions and thanks for using Azure Communication Services :).

Thanks for your comment, Mikben.
andriyovchar, We will close this issue out, but if you have any questions please just let us know.  Thanks again for your feedback and using Azure Communication Services.

Hi @mikben mikben, is this feature available now?

Hi @andriyovchar - thanks for submitting this issue. This feature isn't yet available but muting remote participants will be possible in a future release. Unmuting remote participants is unlikely to be supported given the privacy concerns that are associated with this feature. Don't hesitate to reach out with any other questions and thanks for using Azure Communication Services :).

Hi is this available now? Thank you!


Add a pop up dialog box for subscribing to newsletter done ! #265

Hello, PR
Add new feature - #265
Add a pop up dialog box for subscribing to newsletter done properly -

Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have created a helpful and easy-to-understand README.md if it's a new page/tech stack.
[x] I have gone through the contributing.md file before contributing.

[x] I'm a GSSOC-EXT contributor
[x] I'm a HACKTOBERFEST contributor

@Anuj3553 done, please merge my PR with level 3, If you check other projects, they are also giving level 3 for this.. i add this new feature, You can check my code also...
Thank You 🙏🏻

I can only give level2 for this.

Well done @Subhajit-2023-44

@Anuj3553 done, please merge my PR with level 3, If you check other projects, they are also giving level 3 for this.. i add this new feature, You can check my code also...
Thank You 🙏🏻

I can only give level2 for this.

Okay, Thank You

@Anuj3553 done, please merge my PR with level 3, If you check other projects, they are also giving level 3 for this.. i add this new feature, You can check my code also...
Thank You 🙏🏻

I can only give level2 for this.

@Anuj3553 okay 😢Thank You
I would be very grateful if you could give me level 3 for this.. because If you check other projects, they are also giving level 3 for this...

Hello, PR Add new feature - #265 Add a pop up dialog box for subscribing to newsletter done properly -

Checklist:

[x]  I have mentioned the issue number in my Pull Request.

[x]  I have commented my code, particularly in hard-to-understand areas.

[x]  I have created a helpful and easy-to-understand README.md if it's a new page/tech stack.

[x]  I have gone through the contributing.md file before contributing.

[x]  I'm a GSSOC-EXT contributor

[x]  I'm a HACKTOBERFEST contributor

When i can saw this popup.

Please send the video of your task.


rebranding

Status
Ready/In Progress/In Hold (Reason for hold)
In Hold until Products dropdown works
Related Issues
Rebranding
Description
Rebranding to Cortex XSOAR
Screenshots
Paste here any images that will help the reviewer

@csestito  can we close this one?


add positional argument with iomux configuration to bitstream_to_* scripts

This PR slightly modifies bitstream_to_* bitstream conversion scripts. It adds optional input arguments for passing iomux configuration paths. If this option is not used, then script acts as previously (inferring path to iomux configuration from bitstream path). This change is required to use those scripts in new F4PGA build flow (see https://github.com/chipsalliance/f4pga/pull/577, f4pga build requires explicit input arguments to run external scripts in F4PGA flow).
Signed-off-by: Pawel Czarnecki<EMAIL_ADDRESS>

Hi @tpagarani, please take a look at this. It would be nice to have this in main repo so that we can use it in F4PGA.

lgtm


Utilise PCH to reduce compilation times and includes

Basically as the title says, need to add includes to the PCH

Won't fix as we will continue on a IWYU model


[BUG] curve-leverage-lending-bot: withdrawal when health < threshold doesn't happen

~- [ ] This bot 0x420BA68F74FAf559e702138093e044fDE77C172C (curve squeeze leverage) is below the health threshold, but the withdrawal doesn't happen, even though it seems to try to process this bot~  current status of curve-squeeze is captured here https://github.com/VolumeFi/paloma/issues/1682

[ ] Same for 0xEC3c1940d88B8875b39b41e6D023026da500D4bB (curve leverage lending)

blocked until cw code fixed


Shadows wobble when looked at

Godot version:
3.0
OS/device including version:
Windows + Linux GTX970
Issue description:
I noticed quite a bit of issues with shadows "wobbling" around in my test scene and was surprised when moving the camera around that the shadows would start becoming super sharp and steady, just like I wanted them to. Is this a misconfiguration on my part, a bug or is this intended behaviour somehow?
Issues #12148 and #13575 might be related.
Edit: Shadow Filtering in project settings is disabled.
Lookin good:

Lookin meh:

Steps to reproduce:

Check out https://github.com/Worgames/superspacearcade_godot
Open "scenes/TestScene"
Move the camera around near a tunnel element

If you set the shadow depth range to Optimized, the engine will find the best possible "layout" for shadow rendering, which will make shadows look sharper at the expense of visible "fuzzing" when the camear is moving (it's noticeable around shadow edges). In some scenes, it may also cause shadow quality to vary depending on the camera's position.
The default setting, Stable, uses a constant approach; shadows won't look as sharp, but won't have a "fuzzy" appearance when the camera is moving.
See Lights and Shadows in the documentation.

Wow, this looks much better for what I intended, thanks @Calinou - I also set Mode to "PSSM 4 Split" and now it's just brilliant - don't know the performance hit yet but the visuals are great.


Fix trigger for Swagger UI update

Motivation
The trigger for creating the swagger definition should be on push of a tag.
Description
Fix the trigger for the workflow. I also added workflow_dispatch to allow the manual trigger of the workflow.
Test Plan

n/a

TODO

[x] Explain motivation or link existing issue(s)
[x] Test changes and document test plan
[x] Update documentation as needed
[x] Update changelog as needed

bors merge

test timeout
bors merge

flaky Test_Checkpoint
bors merge

bors merge

bors merge

@fasmat I wanna try to get the systest resilience PR in so we could have the timeout set on the workflow through the go test runner to see why it hangs. It could be that some cleanup on a test isn't finishing correctly.

bors merge


Change the character for works in bower

Must be change the name to "jquery-hoverIntent" because bower can't be found "jquery-hoverintent"

Closing, since it turns out to not be an issue.


fixing street name field capitalization in Brevard County, FL

street name was missing, changing /SiteStreetName/SiteStreetname/

Good catch!


glip-v2

Hi genius!
I am so interested in your work, great job!
looking forward to your code for glip-v2
do you have a plane?

I hope so


Plans to implement more paymentmethods?

Are there plans to implement mor paymentmethods? I'd like to have Bancontact and Giropay also.
Or can I do this myself?

You can submit a PR if you like :)

Will do, I first have to figure out the Git system all together ;-)


add feature for search/replace dialog same as visual studio

can you add support for search/replace dialog option to select scope of search/replace or is there any plugin that can add this behaviour?

/duplicate https://github.com/microsoft/vscode/issues/20530


docs(documentation): reference the rxjs-doc repository

ReactiveX/rxjs-docs#249
hey @benlesh and @ladyleet, like discussed here is the pr to reference to documentation project. If there is another file where I should add a reminder, please let me know :)

@jwo719 rocking our socks off! thx! we super appreciate it. we hope you can join us for some of the rxjs learning team meetings soon! :) ping me on slack! <3

@JWO719 Nicee 💯

Coverage remained the same at 97.506% when pulling 94d4f29a4d109110017f05b1e69b7aef9b8eaa4a on JWO719:reference-documentation-repository into a24146f1ad09acd62ea92ee0f8d983d5ab5cf835 on ReactiveX:master.

omg yay! @JWO719 congrats on your first merged PR to the RxJS project! 🎉

Thank you. Definetly not the last one :blush:


Refactored AdafruitMotorHat and AdafruitPwm constructors to allow stacking of other hats

Refactored constructor methods of AdafruitPwm and AdafruitMotorHat classes so they can receive the i2c device name and i2c slave device address instead of having them hardcoded.
With the changes I'm commiting and a new driver I've written (inspired by Antonio's Motor Hat driver), I was able to stack the Adafruit 16-Channel PWM Servo Hat that I'm using to control the pan/tilt bracket's servos my camera module is mounted on, on top of the Adafruit DC and Stepper Motor HAT.

@RacZo This is great, particularly now that we're working on https://github.com/zugaldia/android-robocar/pull/40 (though I haven't been able to make it work yet). I'll review it soon!

@hsalameh it's white space changes – if you append "?w=1" to the files URL, e.g. https://github.com/zugaldia/android-robocar/pull/45/files?w=1 you'll see that the number of lines decreases by 300


Next hackathon we're taking part in (constellation - chainlink)

I was wondering if we could take part in the next chainlink constellation hackathon.
Reading their FAQ, the rules are as follows:

The event will take place from 11/08 to 12/10.
The team can be 1-5 people.
The project has to be developed after 08/11 but it is possible to update the project to participate in the hackathon but we have to contact them in advance to check with them.
We can continue to work on the project after the deadline.

After these points, do you think it's worth taking part?

Looking at our roadmap, we think it's best not to take part in this hackathon.


Messed up the 0.10.0 release :(

Codecov Report

Merging #106 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #106   +/-   ##
=======================================
  Coverage   99.44%   99.44%
=======================================
  Files          19       19
  Lines        1788     1788
=======================================
  Hits         1778     1778
  Misses         10       10

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 5b99ac6...f48a50e. Read the comment docs.


perhaps fn verified_by() need more comments

https://github.com/datrs/tree-index/blob/82e189ef4eb6b4dba4f9003e946f3601eeecb149/src/lib.rs#L236
It's hard to understand the function of fn verified_by() by reading the test code and the current comment.

Ah yeah, good question! We should write a better version for this for the book, but here's some notes on how verification works with flat-tree (and in turn tree-index).
A PR for a better description based on ^ would be very welcome! -- also happy to answer any questions you might have around this, as I think I have a good grasp on it nowadays :D

verification is a hash of hashes -- kind of like with a blockchain, it signs the whole history. A signature is an Ed25519 cryptographic signature. It takes data (e.g. the hash), and signs it to ensure this hash was created by the person with the private key.
Ummm, I don't know if my explanation makes much sense here! -- I always struggle explaining this stuff back -- and that's not a good sign. I need to do better here; sorry if things are not clear!

Produced with print-flat-tree.
Before we go in, it's important to know that we have 2 trees with the exact same layout. In .dat/ they're stored as .dat/data and ./dat/signatures.
In the viz above, you can see numbers in yellow, white, blue and green.

yellow: leaf node. (src) Holds the hash of a piece of data in the .dat/data tree. In the .dat/signatures tree this holds a signed hash of all previous nodes in the tree (see green for how this works).
white: parent node (src). Combines the hashes of the nodes below it. This is what makes this a "merkle tree". E.g. this becomes a hash of hashes.
blue: root node. A parent node that has a hash of all data under it -- e.g. the tree under the node is complete. This is otherwise the same as a "parent node".
green: head (src). The last node added to the tree. It holds a combined hash of all previous root nodes. So we use the data stored in this node to verify all previous parts of the tree.

Hope this somewhat makes sense? So when you call .verified_by() you try and find the green node that corresponds to holding the signatures for a blue node.
All of this hasn't been reviewed btw, it's just a quick draft of my best-effort understanding of how things work. I want to write some more about it for the guide, and let some people review it to make sure it's 100% accurate. Hope this is somewhat helpful though!

Thanks! Very helpful.

it's important to know that we have 2 trees with the exact same layout. In .dat/ they're stored as .dat/data and ./dat/signatures.

But look at hypercore/feed's fn append(), It only writes the merkle-tree data to the tree storage, signatures storage and data storage are not tree:
self.storage.write_data(self.byte_length + offset, &data)?;
// ...
self.storage.put_signature(index, signature)?;

for node in self.merkle.nodes() {
  self.storage.put_node(node)?;
}

The last node added to the tree. It holds a combined hash of all previous root nodes.

Yeah! I see it:
let hash = Hash::from_roots(self.merkle.roots());
let index = self.length;
let signature = sign(&self.public_key, key, hash.as_bytes());
self.storage.put_signature(index, signature)?;


Labels overflow into form fields

This appears to happen with both text input and select menus.

https://secure3.convio.net/jdrf3dev/site/Donation2;jsessionid=00000000.app363a?idb=974189056&df_id=18647&FR_ID=6617&mfc_pref=T&PROXY_ID=10804197&PROXY_TYPE=20&18647.donation=form1&NONCE_TOKEN=1992F565FFF3383C864EA627D51BF075

Append a Red asterisk for required or "(Optional)" text at the end of the label instead of floating the text at the end of the field. With this approach, no stacking is needed on mobile.
Zeplin for exact specs of the asterisk and optional text: https://zpl.io/ZfxOcw
NOTE: The DS HTML has the label element below the field and moved into place with absolute positioning. The order of the HTML can not change for this fix because 2 large projects are already underway using this order.


Kafka code share - umbrella ticket

Problem
We would like to increase code share in Kafka components and also make them consistent.
This is just an umbrella ticket to hold subtasks.
This is the meeting notes that happened in Eventing Delivery group about this initiative: https://docs.google.com/document/d/1f556J18aaG4NUQFawWAcnMAV6_EL_eLCflTyPWn2Yz0/edit
Subtasks:

[x] https://github.com/knative-sandbox/eventing-kafka/issues/268: Common util for SaramaConfig
[x] https://github.com/knative-sandbox/eventing-kafka/issues/298: Use Sarama field of config-kafka in consolidated channel and in source
[ ] https://github.com/knative-sandbox/eventing-kafka/issues/405: Leverage eventing-kafka field of config-kafka in consolidated channel and in source

/close
Duplicate of https://github.com/knative-sandbox/eventing-kafka/issues/386


[chip-test, dv] Enhance chip_sw_alert_handler_lpg_clkoff

Per https://github.com/lowRISC/opentitan/pull/16305#issuecomment-1314452844,
Enhance the test to check if the ping mechanism works correctly when multiple clocks are turned off at the same time.

estimate 4

Per the discussion in https://github.com/lowRISC/opentitan/pull/16305#issuecomment-1315763731,
Add an SV check to verify that ping mechanism is active when an IP is clock gated.

Removing sriyerg from assignee list since no longer on this project

@andreaskurth - I think this is a nice to have - so pushing to P4 unless you think the priority should be increased.
@moidx
--> P4


GT4Py programs as DaCe SDFGConvertibles

This feature is needed for implementing DaCe orchestration, e.g. in ICON4Py.

Just a general comment, I am wondering if DaCe offers alternatively the option to register the conversion functions, instead of having to make them part of the object. (E.g. like jax pytrees work) If only the current option is available I am wondering if we should go with a monkey patching approach to remove the dace import from unrelated modules. But this is for @egparedes to answer. (And we could think of the other option as a contribution to productice DaCe.)

My first impression is that @havogt is right and the best option would be to enhance DaCe with support for registering SDFG convertible functions defined outside of the object. It would improve both the Gt4Py and DaCe codebases and would make this PR much cleaner. Another option, even simpler to implement in the DaCe side, would be to remove the requirement to inherit from SDFGConvertible (which is not even an ABC) and implement it as a typing Protocol using runtime_checkable or a custom typeguard for isinstance() checks.
If we want to merge this PR in the short term before waiting for changes in DaCe, I think it'd be better to add the required DaCe methods and bases conditionally to avoid adding DaCe as a non-optional requirement, as @edopao and @havogt suggested. For example:
class Program:
    ....

if dace:
    class Program(Program, SDFGConvertible):
        def __sdfg__(self, *args, **kwargs) -> Optional[Any]:
             ....

Please

add a few more details to the PR description (which will become the commit message)
add yourself to the Authors file

Please

add a few more details to the PR description (which will become the commit message)
add yourself to the Authors file

Thank you Hannes! It's done.

cscs-ci run

cscs-ci run default

cscs-ci run

cscs-ci run default


[hf inference] Implement HuggingFaceImage2TextRemoteInference

[hf inference] Implement HuggingFaceImage2TextRemoteInference
[hf inference] Implement HuggingFaceImage2TextRemoteInference
Implement HuggingFaceImage2TextRemoteInference for running hf image-to-text models via inference API for gradio.  The api takes in an image of various supported types: Union[str, Path, bytes, BinaryIO]. For now, just implementing support for path and uri since it's not needed for gradio and my python skills aren't great.

Testing:
Build/install the local hugging face package with these changes
(hf) ryanholinshead@Ryans-MacBook-Pro aiconfig % cd extensions/HuggingFace
pip3 install build && cd python && python -m build && pip3 install dist/*.whl
pip3 install -e .

Register these parsers in /Users/ryanholinshead/Projects/aiconfig/cookbooks/Gradio/hf_model_parsers.py then run aiconfig editor with the local parsers and gradio config:
aiconfig_path=/Users/ryanholinshead/Projects/aiconfig/cookbooks/Gradio/huggingface.aiconfig.json

(hf) ryanholinshead@Ryans-MacBook-Pro aiconfig % parsers_path=/Users/ryanholinshead/Projects/aiconfig/cookbooks/Gradio/hf_model_parsers.py

(hf) ryanholinshead@Ryans-MacBook-Pro aiconfig % aiconfig edit --aiconfig-path=$aiconfig_path --server-mode=debug_servers --parsers-module-path=$parsers_path

Ensure the model works in the config. Ensure changing settings works & is persisted to aiconfig.
Test setting fake model ('test') propagates expected error
Test setting invalid api_token in the InferenceOptions in run server-side propagates expected error
Set to my own key and ensure execution works

Stack created with Sapling. Best reviewed with ReviewStack.

#1019
-> #1018

Changes from review:

remove copy-pasted comment about translation api
raise ValueError if output data type isn't string, instead of silently failing by dumping to json


reverse commit, which produces a bug

the latest commit to master caused a problem with raw queries. syntax error at or near ")"
It changed the query
SELECT translation
    , translatable.technical_name as translatable
    , language.iso_639_1 as language
    , locale.full_locale as locale
    , translatable.id as translatable_id
FROM translation
JOIN translatable ON translatable_id = translatable.id
LEFT JOIN language ON language_id = language.id
LEFT JOIN locale ON locale_id = locale.id
WHERE (language.iso_639_1 = $1 OR locale.full_locale = $1)
AND translatable.%s = $2`

to
SELECT translation
    , translatable.technical_name as translatable
    , language.iso_639_1 as language
    , locale.full_locale as locale
    , translatable.id as translatable_id
FROM translation
JOIN translatable ON translatable_id = translatable.id
LEFT JOIN language ON language_id = language.id
LEFT JOIN locale ON locale_id = locale.id
WHERE (language.iso_639_1 = $1 OR locale.full_locale = $1)
AND translatable.id = $2) LIMIT 1

Does change TrimRight to TrimSuffix works for you?

Push the change to master, let me know if it still cause error.


Graphql subqueries crash the query node server

example:
{
  nodes(limit:1){
    publicConfig{
      gw4
    }
  }
}

We are using reverse lookups for the nested objects, https://docs.subsquid.io/v/v3/schema-spec/entity-relationship#reverse-lookups

Fixed when upgrading to subsquid v4!


Wrapping the init function inside jax.jit

I'm currently doing a runtime analysis of the attention matrix of a transformer. Specifically, I'd like to know how the time complexity behaves w.r.t. to the size of the attention matrix.
def model(x):
    net = factory(hidden_dim, num_layers=1)
    return net(*x)

def inference(fn, rng, params, x, mode="runtime"):
    start = time.timeit()
    _ = jax.block_until_ready(fn)(params, x)
    end = time.timeit()
    print(f"{mode}: {end - start} s")
    return end - start

def main():
    if args.single == 0:
        nb_nodes = np.arange(args.max_nb_node) + 1
    else:
        nb_nodes = [args.max_nb_node]
    runtimes = []
    rng = jax.random.PRNGKey(42)
    for nb_node in nb_nodes:
        print(f"Number of node: {nb_node}")
        net = hk.without_apply_rng(hk.transform(model))
        node_fts = jax.random.normal(rng, (batch_size, nb_node, hidden_dim))
        edge_fts = jax.random.normal(rng, (batch_size, nb_node, nb_node, hidden_dim))
        x = (node_fts, edge_fts)
        params = net.init(rng, x)
        apply_fn = jax.jit(net.apply)

        # compile time
        _ = inference(apply_fn, rng, params, x, mode="compile time")

        # execution time
        runtime = inference(apply_fn, rng, params, x, mode="execution time")
        runtimes.append(runtime)

For nb_node = 414 I'm getting an OOM error in the initialization (when performing the Einstein summation), which looks as follows:
jaxlib.xla_extension.XlaRuntimeError: RESOURCE_EXHAUSTED: Out of memory while trying to allocate<PHONE_NUMBER>6 bytes.
BufferAssignment OOM Debugging.
BufferAssignment stats:
             parameter allocation:  249.86MiB
              constant allocation:         0B
        maybe_live_out allocation:   50.39GiB
     preallocated temp allocation:         0B
                 total allocation:   50.63GiB
              total fragmentation:         0B (0.00%)

This is quite strange, since I'm using A100 80GB.
However, when one jit the init funtion, there is no OOM error even after nb_node = 500.
My question is, would this be a correct workaround given in this case?

tl;dr - You should jit the init function to get a version of it that (1) uses as little memory as possible (2) runs quickly.
To add a bit more detail, jax.jit (through XLA) applies a number of optimizations to your program. Some of these might reduce the overall peak memory footprint required by the init program. For example, one optimization that XLA does is limiting the live range of arrays.
Lets consider the following JAX program:
def fn():
  a = some_big_array()
  b = other_big_array()
  c = a + b
  d = yet_another_big_array()
  e = c + d
  return e

XLA would be able to notice that a and b can be safely deleted (and as such their GPU memory woudl be freed) before you compute d:
def fn():
  a = some_big_array()
  b = other_big_array()
  c = a + b

  # XLA knows a/b aren't used again so it can release GPU memory for them.
  free_gpu_memory(a)
  free_gpu_memory(b)

  d = yet_another_big_array()
  e = c + d
  return e

If you wanted to debug further and figure out which arrays were still hanging around causing the 50GB allocation to fail, then JAX allows you to see a traceback for where arrays were created, which might help you understand this in more depth. To help you get started, something like the following might work:
import traceback

def print_live_arrays():
  for array in jax.live_arrays():
    print(array.shape, array.dtype)
    traceback.print_tb(array.traceback.as_python_traceback())
    print()

try:
  benchmark()
except RuntimeError as e:
  if 'RESOURCE_EXHAUSTED' in e:
    print_live_arrays()
  raise e

That said, even if you knew the root cause (e.g. which arrays were hanging around) the recommended fix is the same: use jax.jit and let XLA optimize this for you.


Added in patch which supports PNC

Describe your changes
Hello everyone, I am apart of the team over at NREL who has been working on the SIL demos in everest/everest-demo. Recently, @shankari went to the CharIN Testival and showcased EVerest by running a custom demo that was tailored to have PnC functioning, found here. To get PnC to work, she had to apply a handful of patches to different parts of EVerest, all which can be found here. We now would like to slowly integrate these patches into EVerest so that it can fully support PnC.
This patch in particular involves adding in support for a Contract payment method to PyEvJosev.
Testing Done
As mentioned above, this patch is one of many, so it has only been tested in conjunction with the other patches in the SIL demo. The particular demo which has PnC working can be found linked above. To run this demo, I would recommend...

Clone repo and check out branch.git clone https://github.com/US-JOET/everest-demo.git
git checkout -b test-demo origin/charin-e2e-demo

Run the software simulation demo.bash demo-iso15118-2-ac-plus-ocpp.sh -r $(pwd) -b test-demo -3

Wait until the EVerest software boots up and you see the following.2024-06-18 18:23:22.836788 [INFO] ocpp:OCPP201     :: Received BootNotificationResponse: {
    "currentTime": "2024-06-18T18:23:22.000Z",
    "interval": 300,
    "status": "Accepted"
}

Navigate to http://localhost:1880/ui/, choose from the Car Simulation dropdown menu AC ISO15118-2 Plug&Charge, then plug the car in and observe it authenticate on its own.

Issue ticket number and link
The issue can be found here in everest-demo.
Checklist before requesting a review

[x] I have performed a self-review of my code
[ ] I have made corresponding changes to the documentation
[x] I read the contribution documentation and made sure that my changes meet its requirements

Hello :)
thank you very much for your contribution. However, after looking through it, I have decided not to accept this PR and close it in a few days.
Because, the changes this PR describes, I deleted from everest-core a few months ago. The payment option in the iso_start_v2g_session cmd had no effect on Josev at that time.
I have seen that there is another PR in the Josev fork and that the option selected here is actually also used by Josev.
However, I find this approach less realistic and less effective than the current situation.
At the moment, the Josev EV side itself decides whether to select PnC. Certain conditions must be met: TLS must be active and the EVSE must also offer PnC as a payment option. PnC only works in the SIL if these conditions are met. The config also makes it possible to test certain scenarios, such as TLS + EIM.
This PR can lead to strange behavior on the EVCC side if, for example, PnC is selected and the charger only supports EIM. This cannot happen with the current status. For me, this means that another potential source of error is being added here when executing the SIL.
I'm sorry, but I see no reason why the payment option should be added again.

After a brief discussion with @corneliusclaussen, I can understand why you would want to select PnC or EIM via node-red when EVerest is running. And don't always want to restart everest with a different config.
I still don't like the possibility of setting the payment option directly for the car sim.
As an alternative, however, I could imagine that you can set a payment option priority using the iso_start_v2g_session cmd.
A priority list could be defined in Josev. E.g. EIM: 1, PnC: 2 (small number -> higher priority) The same principle applies to the SupportAppProtocol message.
I.e. if Josev can select both options (EIM and PnC), Josev selects the relevant payment option depending on the priority. The cmd can be used to change the priority. E.g. iso_start_ DC, PnC -> EIM: 2, PnC: 1
This has the advantage that if the priority is set to PnC but the Charger can only do EIM, PnC is ignored by Josev and EIM is selected. And you can "change" the payment options during running EVerest via node-red.
The options for not sending a payment option via the cmd do not work. The interface system does not allow optional arguments. Therefore the iso15118_ev interface should also be adapted here. All node-red yaml files in the config folder would also have to be adapted.
What do you think of the suggestion?

@louisg1337 Any thoughts on my suggestion?

Is this still being worked on or should we close it?

I'll close this next friday if nobody answers

So I close this one. If there is a need, then please reopen a comment on my proposal.


Add compute-image-package build and test workflow for debian-9

/ok-to-test

I'm still working out how this will best serve us. Right now I see two scenarios we want to test:
Test prerelease packages, this PR doesnt help much there and I think this is the bigger thing we would like to test, we do this well with windows right now but our linux build process needs to be updated to handle this. Auto building packages is the best way to handle this, then we just need to build packages updates into the existing tests via a flag. Or autobuild images based on prerelease packages, this is probably much simpler from a test perspective.
Test packages based on a specific PR or repo, this is what I see this PR focusing on. In this case I don't think these tests need to run automatically at all, we just need a simple command line to run

Hi @zmarano @adjackura,
As we discussed before, the idea here is to have a workflow where we can test a given PR for compute-image-packages, so we can add a CI in compute-image-packages to trigger this when a PR happens.
It will be hard to test a PR against every distro, but at least against Debian is a start, What do you think?


🐛 Bug Report: Stripe calling home for unknown reasons

👟 Reproduction steps
Self-host Appwrite console constantly sends tracking to Stripe. It should NOT.
👍 Expected behavior
Not send stuff to other websites.
Not even include stripe if needed to avoid that.
👎 Actual Behavior
Sending metrics or whatever to Stripe
🎲 Appwrite version
Version 1.5.x
💻 Operating system
Linux
🧱 Your Environment
Version 1.5.7
Installed through Coolify
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

We do actually have a check for that in place here https://github.com/appwrite/console/blob/main/src/routes/console/%2Blayout.svelte#L243-L248 - I'll do some digging to see why its doing that on self-hosted 👍🏻

@ttrinidad I can't seem to find any evidence we are initializing stripe nor sending any requests on self-hosted 👍🏻
Can you tell me where this is happening to you? 🙂

nevermind, just was able to reproduce it - this is definitely a bug and not intended!

Found the reason why I couldn't reproduce it at first. The fix for it got merged last month, but hasn't been released yet.
https://github.com/appwrite/console/pull/1111
It will be fixed once 1.5.8 is released 🙏🏻


Update to BYSETPOS support

In August 2016, Nicolas Marlier added BYSETPOS support (#349).  This PR updates that PR with a few small changes to run in modern Ruby (use Integer instead of Fixnum) and a more modern rspec.

I can confirm this works as expected, this is fantastic! 👍

Hello @davidstosik, @NicolasMarlier provided this original PR.  I simply updated it to work with modern versions of Ruby.

@nehresma what's the status of this PR? I'd very much like to use the setpos functionality.

@dimerman It has been merged but I do not believe a new version of the gem has been released.  I've pointed my Gemfile to the latest git ref on master for the time being.    I just realized it has NOT been merged.  I checked my Gemfile and I'm pointing to the ref with BYSETPOS support out on my fork -- not master on seejohnrun/ice_cube.
Sorry for the confusion.  I should have double checked before trusting my memory.

@nehresma no worries - I've been pointing my gemfile to your branch/repo. thanks for that!
However, I found the following case is not working right:
rule_a=FREQ=MONTHLY;COUNT=12;BYDAY=MO,TU,WE,TH,FR;BYSETPOS=-1`
rule_b=FREQ=MONTHLY;COUNT=12;BYDAY=MO,TU,WE,TH,FR;BYMONTHDAY=13,14,15;BYSETPOS=-1

when I create a IceCube::Schedule and add each rule individually, the occurrences are accurate. But if I create a schedule with both these rules, I get 18 occurrences instead of the expected 24.
I thought you might want to know. 🙌

What's still holding this back — the PR feedback seems pretty minor? If that gets cleaned up can we get this merged?

@seejohnrun It seems like several people need this and are using forks to get around it being missing. I'd be more than happy to help clean this up to avoid using a fork myself if it is otherwise merge-able.

@nehresma / @mattbooks / @ovinix - If you can get this PR cleaned up and resolve the feedback and conflicts with master, I'll give it a proper review.

Hello, looks like the gem already supporting what BYTSETPOS does but in different ical format. Example: If we want recurring for every 2 months 3rd Wednesday, ical format w.r.t the BYSETPOS is FREQ=MONTHLY;INTERVAL=1;BYDAY=MO;BYSETPOST=3 which is same as FREQ=MONTHLY;INTERVAL=1;BYDAY=3MO, the latter one is already supported by the gem, so not sure why we still wants this PR.

not sure why we still wants this PR

I don’t use the gem anymore, but you might want to consider that some people use the gem to parse strings produced by another piece of software they don’t have control on, and this would require the whole specification to be supported. 😉

@avit Anything we can do to help get this merged?

@davidstosik what gem do you use instead?

Nothing. 😬
My need to handle schedules has vanished. 🙇🏻♂

It's been over 4 years since I created this PR and over 7 years since Nicolas Marlier's initial commit with this support.  I have rebased it against current master and I've applied changes per the feedback received above.
If someone has time to review and merge, that would be most appreciated.  BYSETPOS is part of RFC 5545 and it would be great if this gem could finally get support merged in rather than just the "noop" that it currently is (https://github.com/ice-cube-ruby/ice_cube/blob/master/lib/ice_cube/parsers/ical_parser.rb#L77-L78).

I'll take a look. Am going to try to dedicate sometime this holiday season to get things tidied up and in better shape.

Trying this out for monthly frequencies with an interval > 1 seems to get stuck an infinite loop. I used the monthly test case from this PR and ran the following code, which defaults to an interval of 1, and everything works fine:
schedule = IceCube::Schedule.from_ical "RRULE:FREQ=MONTHLY;COUNT=4;BYDAY=WE;BYSETPOS=4"
schedule.start_time = Time.new(2015, 5, 28, 12, 0, 0)
schedule.next_occurrence(Time.new(2015, 01, 01))
=> 2015-06-24 12:00:00 -0400

However, if I add an interval value > 1 then next_occurrence never returns:
schedule = IceCube::Schedule.from_ical "RRULE:FREQ=MONTHLY;COUNT=4;BYDAY=WE;BYSETPOS=4;INTERVAL=2"
schedule.start_time = Time.new(2015, 5, 28, 12, 0, 0)
schedule.next_occurrence(Time.new(2015, 01, 01))
# method hangs

If I add a puts statement in MonthlyBySetPos#Validation#validate to log step_time and run the code above again I can see it just going infinitely into the future.
The same thing happens for yearly frequencies with an interval > 1.

I don't think this PR is sufficient to fully support BYSETPOS. It only deals with monthly and yearly frequencies. However,  playing around in the rrule.js demo site, BYSETPOS can have an effect on shorter frequencies too. For example these rule pairs each return different results with and without BYSETPOS:
RRULE:FREQ=WEEKLY;COUNT=15;INTERVAL=2;WKST=MO;BYDAY=MO,TU
RRULE:FREQ=WEEKLY;COUNT=15;INTERVAL=2;WKST=MO;BYDAY=MO,TU;BYSETPOS=1

RRULE:FREQ=DAILY;COUNT=15;INTERVAL=2;WKST=MO;BYHOUR=1,2
RRULE:FREQ=DAILY;COUNT=15;INTERVAL=2;WKST=MO;BYHOUR=1,2;BYSETPOS=1

RRULE:FREQ=HOURLY;COUNT=15;INTERVAL=2;WKST=MO;BYMINUTE=1,2
RRULE:FREQ=HOURLY;COUNT=15;INTERVAL=2;WKST=MO;BYMINUTE=1,2;BYSETPOS=1

RRULE:FREQ=MINUTELY;COUNT=15;INTERVAL=2;WKST=MO;BYSECOND=1,2
RRULE:FREQ=MINUTELY;COUNT=15;INTERVAL=2;WKST=MO;BYSECOND=1,2;BYSETPOS=1

Hey @jkehres I see what you mean regarding the infinite loop when combining interval and bysetpos.  I'm looking into it.
Also, you're right regarding the use of bysetpos and other intervals.  I'll look into adding support for those as well.
@NicolasMarlier is the original author of this work.  I've been shepherding it through bitrot issues over the years and applying feedback.
Thanks for your time evaluating this.

@jkehres I've fixed the interval use for month and year frequencies with bysetpos.  I ran out of time for adding support for the other frequencies yet.  I'll add support for those after Christmas.

I've pushed bysetpos support for weekly frequencies too.  I don't think bysetpos makes sense with frequencies of daily, hourly, minutely, or secondly.  RFC 5545 makes no mention of them, and indeed all of it's examples are with monthly and weekly frequencies.  If someone can think of an example where using bysetpos with one of these other frequencies makes sense, let me know.

I've pushed bysetpos support for weekly frequencies too. I don't think bysetpos makes sense with frequencies of daily, hourly, minutely, or secondly. RFC 5545 makes no mention of them, and indeed all of it's examples are with monthly and weekly frequencies. If someone can think of an example where using bysetpos with one of these other frequencies makes sense, let me know.

I believe BYSETPOS support is still needed for daily, hourly, and minutely frequencies to be compliant with the RFC, which I think should be the goal of this PR. As I mentioned in my comment here, you can see BYSETPOS having an effect for these other frequencies using rrule.js - another library that implements RFC 5545 (itself a port of the python-dateutil library). If ice_cube and these other libraries all properly implement the spec, they should all give the same answer for the same RRULE. We could use that fact to aid in testing this PR to ensure we have a compliant implementation of the RFC. We could us one of these other libraries to generate a more exhaustive set of test inputs and outputs for RRULEs that use BYSETPOS and, thus, increase the number of tests for this new code along with our confidence in it.
Secondly, even though the RFC gives no examples for these other frequencies with BYSETPOS, I think you can derive that they are necessary from the details in the spec.
From the description of BYSETPOS (emphasis mine):

The BYSETPOS rule part specifies a COMMA-separated list of values that corresponds to the nth occurrence within the set of recurrence instances specified by the rule.  BYSETPOS operates on a set of recurrence instances in one interval of the recurrence rule.

My understanding of the spec is that "interval" in this context means one unit of the frequency regardless of the INTERVAL rule part so one year for yearly, one month for monthly, etc. If there are multiple recurrences instances within one interval (e.g. year, month, etc.), then BYSETPOS has an effect and selects the nth one. Normally, there is only a single recurrence instance within an interval. The way you get more than one recurrence instance in an interval is by adding another BYxxx rule part.
Again, from the spec (emphasis mine):

BYxxx rule parts modify the recurrence in some manner.  BYxxx rule parts for a period of time that is the same or greater than the frequency generally reduce or limit the number of occurrences of the recurrence generated.  For example, "FREQ=DAILY;BYMONTH=1" reduces the number of recurrence instances from all days (if BYMONTH rule part is not present) to all days in January. BYxxx rule parts for a period of time less than the frequency generally increase or expand the number of occurrences of the recurrence. For example, "FREQ=YEARLY;BYMONTH=1,2" increases the number of days within the yearly recurrence set from 1 (if BYMONTH rule part is not present) to 2.

Furthermore, this table from the spec shows how each BYxxx rule part interacts with a given frequency and whether it expands or limits the number of recurrences instances (emphasis mine):
   +----------+--------+--------+-------+-------+------+-------+------+
   |          |SECONDLY|MINUTELY|HOURLY |DAILY  |WEEKLY|MONTHLY|YEARLY|
   +----------+--------+--------+-------+-------+------+-------+------+
   |BYMONTH   |Limit   |Limit   |Limit  |Limit  |Limit |Limit  |Expand|
   +----------+--------+--------+-------+-------+------+-------+------+
   |BYWEEKNO  |N/A     |N/A     |N/A    |N/A    |N/A   |N/A    |Expand|
   +----------+--------+--------+-------+-------+------+-------+------+
   |BYYEARDAY |Limit   |Limit   |Limit  |N/A    |N/A   |N/A    |Expand|
   +----------+--------+--------+-------+-------+------+-------+------+
   |BYMONTHDAY|Limit   |Limit   |Limit  |Limit  |N/A   |Expand |Expand|
   +----------+--------+--------+-------+-------+------+-------+------+
   |BYDAY     |Limit   |Limit   |Limit  |Limit  |Expand|Note 1 |Note 2|
   +----------+--------+--------+-------+-------+------+-------+------+
   |BYHOUR    |Limit   |Limit   |Limit  |Expand |Expand|Expand |Expand|
   +----------+--------+--------+-------+-------+------+-------+------+
   |BYMINUTE  |Limit   |Limit   |Expand |Expand |Expand|Expand |Expand|
   +----------+--------+--------+-------+-------+------+-------+------+
   |BYSECOND  |Limit   |Expand  |Expand |Expand |Expand|Expand |Expand|
   +----------+--------+--------+-------+-------+------+-------+------+
   |BYSETPOS  |Limit   |Limit   |Limit  |Limit  |Limit |Limit  |Limit |
   +----------+--------+--------+-------+-------+------+-------+------+

For secondly, you can see that no BYxxx rule part will expand the set of recurrence instances so BYSETPOS won't have an effect. However, for minutely, hourly, and daily frequencies you can see that some BYxxx rule parts will expand the set of recurrences, which would allow BYSETPOS to select the nth one in an interval and thus effect the final set of recurrence instances.

Using BYSECOND with MONTHLY does not work. I tried this:
ical = "RRULE:FREQ=MONTHLY;COUNT=4;BYSECOND=1,2,3,4;BYSETPOS=2"
schedule = IceCube::Schedule.from_ical(ical)
schedule.start_time = Time.new(2015, 5, 28, 12, 0, 0)
schedule.occurrences_between(Time.new(2015, 01, 01), Time.new(2017, 01, 01))

and got an empty array. I expected to get:
2015-05-28 12:00:02
2015-06-28 12:00:02
2015-07-28 12:00:02
2015-08-28 12:00:02

I suspect there is an assumption in the code that you will only use BYDAY with MONTHLY and not other BYxxx rule parts.

First I want to thank @nehresma @jkehres and others for all the work on this.

I believe BYSETPOS support is still needed for daily, hourly, and minutely frequencies to be compliant with the RFC, which I think should be the goal of this PR.

The current code already doesn't implement the RFC, and it fails to do so silently (just spent over an hour finding the this as the source of a bug). I think partial RFC compliance is better than none. And the users who use daily/hourly/minutely frequencies would continue to see the current unimplemented behavior. I know I wouldn't be alone in appreciating getting this work merged.

Thanks for the review @jkehres. Can we get the tests running/passing?

Thank you so much for your work here. I'd love if we can get this merged. Apple Calendar feeds use BYSETPOS when selecting a day of the month. And there is no direct equivalent for the negative values (starting from the end). So this would be super handy.

@jorgemanrubia - Since nothing has moved on this PR for a long time ... it might be easiest if you can fork it, get the tests passing and then we can move forwards? I have struggled for time recently otherwise would have done this myself.

Monthly/Yearly would be more than enough for me.


”Muntenii de Sus”, ”Satu Nou” are listed in the drop-down menu

Description:
When user writes ”Muntenii de Sus”, ”Satu Nou” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Muntenii de Sus"

Expected results
The location appears in the drop-down menu as "Sat Muntenii de Sus, VASLUI (Muntenii de Sus)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Satu Nou"

Expected results
The location appears in the drop-down menu as "Sat Satu Nou, VASLUI (Muntenii de Sus)".


Recalibrate 11q line D sweetspots after QM hard reboot

We had to hard reboot QM due to an issue with the controllers used in qw5q_platinum. I do not yet understand why, but this changed the sweetspot values that I had calibrated for 11q line D. Here the sweetspots have been recalibrated and fortunately we can still address all qubits with similar assignment fidelities as before:

Qubit flux: http://login.qrccluster.com:9000/_LCbtU2lQgC5KnZVzK-5fQ==
Single shot: http://login.qrccluster.com:9000/Vq7uF6HnSgS013NvchHIpQ==
Rabi: http://login.qrccluster.com:9000/4jXvtyl-QI2QCNejzzc-7A==

Thanks @andrea-pasquale. Yes, feel free to push. Indeed, when I first created the runcard I put no parameters in the characterization section and then started adding incrementally. Unfortunately, I think we cannot copy the runcard generated by qibocal, due to the issue with string qubit names, so we have to manually copy paste the parameters.

Thanks @andrea-pasquale. Yes, feel free to push. Indeed, when I first created the runcard I put no parameters in the characterization section and then started adding incrementally. Unfortunately, I think we cannot copy the runcard generated by qibocal, due to the issue with string qubit names, so we have to manually copy paste the parameters.

Ok, I pushed in https://github.com/qiboteam/qibolab_platforms_qrc/pull/148/commits/ef7824e1d64852ce77022867733011668a57da72. Regarding strings for qubit name, do we really have to use them? I know from @alecandido that you guys are planning to drop them soon from qibolab. We could try to push people to already use integers here to simplify the calibration procedure.
We are ready to merge from my side.

Ok, I pushed in ef7824e. Regarding strings for qubit name, do we really have to use them? I know from @alecandido that you guys are planning to drop them soon from qibolab. We could try to push people to already use integers here to simplify the calibration procedure.

I believe this is mostly a purpose from my side. But this will be a non-breaking change: my intention is to preserve them in the runcards, but avoid them inside.
The QubitId will be either an index in a list of names hold by the platform (at that point, names could be whatever is JSON serializable, I don't care). Internally, it will be the only name used by Qibolab. But from the outside, you will get translations.
The exact same mechanism could be used in Qibocal. It is not yet clear to me if this should be inherited from Qibolab, or it is better to repeat the translation independently (and I'm leaning towards this second option).
Btw, the implementation of the translation will be trivial:
QubitId = typing.NewType("QubitId", int)

class Platform:
    qubits: list

    def id_to_name(self, id: QubitId) -> Any:
        return self.qubits[id]

    def name_to_id(self, name: Any):
        id = self.qubits.index(name)
        if id < 0:
            raise
        return QubitId(id)

Thanks for the reviews.
Regarding string names, personally I also find them useful in some cases such as this one. Now we are using line A and D, so the natural way to convert to integers is A1, A2, ... A6 -> 0, 1, ..., 5 and D1, ..., D5 -> 6, ..., 10. However, soon we may decide to connect line C and in that case we either map C to 11-15, which is counterintuitive (since C should come before D) or we need to shift all Ds. Either way is confusing for the user. Using strings here also makes it easier to identify which qubits share readout lines and is also in line with the manufacturer labels, so it is easier to write connectivity etc.
This is about the external level only, so basically the platform. Internally in the drivers it is perfectly fine to translate to integers as @alecandido is proposing.
Regarding the issue that we cannot copy the qibocal updated runcard, it may actually be caused by qibolab's serialize module and not qibocal. I will also try to have a look.


[]: React 19 + Next 15

Describe the bug
TypeError: Cannot read properties of undefined (reading 'ReactCurrentDispatcher')
"use client"; import { TonConnectUIProvider } from "delab-tonconnect-ui-react"; export function Providers({ children }: { children: React.ReactNode }) {   return (     <TonConnectUIProvider manifestUrl="https://web.example.com/">       {children}     </TonConnectUIProvider>   ); }
Expected behavior
works :D
Current behavior

Steps to Reproduce
pnpx create-next-app@rc --empty + install ton connect react
Environment
next 15, react 19, macos
Additional context
No response

we need to upgrade jsx transformer perhaps

Yeah, i'm having the exact problem in my projejct

SAME ISSUE HERE

maybe the best solution would be downgrading into next14 and react18

I guess so, i actually downgraded the project to Next 14. I think this would be resolved in a later update

Still same+

same

@thekiba bro, mind taking a look?

yo guys, any update?

@thekiba can be fixed by correctly not bundling react by setting the external option in the vite.config to:
external: ["react", /^react\/.*/, "react-dom", /react-dom\/.*/],

I just changed my dependency from @tonconnect/ui-react to @tonconnect/ui, went to https://github.com/ton-connect/sdk/tree/main/packages/ui-react and copied all files from there, since they are just relatively simple wrappers around @tonconnect/ui library
Then I just imported components from copied files and it works fine so far, I don't think it will cause any issues in a future but I will make update of this comment if something will brake

Is there any chance this will be fixed in any upcoming releases now that react 19.0.0 is stable also? Thanks!


Undocumented settings

I'm struggling to make this integration at least being configured.
Trying to setup latest <IP_ADDRESS>, the installation doesn't match the documentation:

What is the alarm checkbox supposed to do?
What code value should be set? Keeping the default value of 4 throw an error showing code is supposed to be a string. What string exactly??

I'm a bit lost. What am I doing wrong?
Any link to documentation would be appreciated.

Code: As u installed the Alarm together with Securitas (Hardware), u specified a code with Securitas (Code words as well). This code (digits, not characters --> string) u use for the Home panel normally. This code u have to enter here.
User and password are not enough to access your Alarm for deactivation.
When installed, u will have a specific card in HA in which u have to enter the code to disarm. Arming is possible without this code.
Please see and post info from the log file in HA (Settings --> System --> Logs) always.
Apart HA core version would be interesting.
Did u implemented in config.yaml anything related to securitas?
BR

You are right. Alarm doesn't do anything. Let me remove that configuration.


Added ppc64le support

Signed-off-by: Artem Ryabov<EMAIL_ADDRESS>

@mike-dubman for some reasons I cannot add reviewers - could you please grant me permissions?

@mike-dubman for some reasons I cannot add reviewers - could you please grant me permissions?

plz try now

@mike-dubman for some reasons I cannot add reviewers - could you please grant me permissions?

plz try now

Still not available.

@mike-dubman for some reasons I cannot add reviewers - could you please grant me permissions?

plz try now

Still not available.

and now?

@mike-dubman for some reasons I cannot add reviewers - could you please grant me permissions?

plz try now

Still not available.

and now?

Thanks, works fine for me. Can we also add others (now I cannot add other teammates as reviewers)?
CC @MrBr-github (it seems you've created a group for us, haven't you?)

CC @vasily-v-ryabov @aod314

@mike-dubman for some reasons I cannot add reviewers - could you please grant me permissions?

plz try now

Still not available.

and now?

Thanks, works fine for me. Can we also add others (now I cannot add other teammates as reviewers)?
CC @MrBr-github (it seems you've created a group for us, haven't you?)
UPD: cannot add only Andrey and Vasily.

you have full admin rights on this repo - plz add more folks as needed.
Also, you can create github group with all and add group only.


Code Quality: Refactor Status Center code

Feature: Display transfer speed in status center
Motivation & Context

Removed IStatusCenterAction interface used to access viewmodel because we now have Dependency injection.
Removed PostedStatusBanner class and merged it to StatusBanner class which was renamed to StatusCenterItem
Renamed some classes to use same words
Updated UI

Validation
How did you test these changes?

[x] Did you build the app and test your changes?
[x] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[ ] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[ ] Are there any other steps that were used to validate these changes?

PR TO-DO

[x] Remove InfoBar
[x] Add color visual states

Concepts

Screenshots

@0x5bfa we haven't finalized any plans around this flyout.

I'd like to get a speed graph in there sooner than later, but we need to spec this out.

I can wait. how long will it take?
I thought the spec was already formulated when you mentioned this concepts and I raised my hand.

The concepts were done like a year or two ago.  There was going to be a task UI "Status centre" and it was at a different time.
I was not asked to look into it any further since, so I guess we can work on this aspect and maybe implement it later, when other bits are figured out.

Do we want this flyout UI to be always accessible?
Do we automatically remove completed tasks?
Do we want notifications when apps are in the background? (Adaptive Cards?)
Do we want the icon for this UI to reflect progress or number of tasks in operation?

Here is a rough idea of how we could transition from a Progress control to a Progress Chart - how possible is something like this @0x5bfa
https://github.com/files-community/Files/assets/7389110/7908fbd2-5ad8-41c1-8e8d-d18230ac73df

You think refactoring is more impertant than redesigning the UI?

It's mainly a reflection that this is preparation for the larger design changes in phase 2.

Ready for review for me.

Are there any other steps that were used to validate these changes?

Can you share some testing steps?

omg i'm sorry i will add soon (i'm in bed so it will be tomorrow)

It looks good code wise.

Done

Open StatusCenter

Does this work for you? The app crashes when I try opening the status center.

Last time I build successfully is https://github.com/files-community/Files/pull/13276/commits/1a120fe202e46175aaf8017917fafd17f6c1b903


Add latest policy images to VHD

Add latest policy images to VHD
Gatekeeper image: v3.4.1
Policy images: prod_20210713.1

@robbiezhang , @feruilob, @nreisch - can you please review this PR?


Incorrect handling of plural forms for translation

Description
Hi!
The current handling of strings with multiple plural forms by mapping them to key-value pairs in ini file is incorrect. It allows using only two plural forms like in English (singular for one only). There's also no problems that have only one plural form. But there are many languages with three and more plural forms (you may find a review here for example). Correct translation to such languages is currently impossible.
Please consider changing the translation framework to handle pluralization properly.
Gitea Version
1.19.0
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
n/a
Database
None

See also https://github.com/go-gitea/gitea/pull/19916.

What do you think about this approach #23933 ?

For en: There $[is, are] %d $[item, items]
For lv: There $[is-0, is-1, is-other] %d $[item-0, item-1, items]
For ar: There ... %d $[item-0, item-1, item-2, item-few, item0-many, items]

(I don't understand lv or ar, so use English words for demo)
Because each language has standard defined Plural Forms ( https://github.com/unicode-org/cldr/blob/main/common/supplemental/plurals.xml )
With this approach, translators just need to fill the words for these forms.
I think we need a crowdin-compatible and ini-compatible syntax, because Gitea is using these systems.
And we need a translator-friendly syntax, otherwise the strings could get broken frequently (I have found a lot of broken translation strings recently ... )

@wxiaoguang, this approach will not work for different languages. E. g. in Russian we usually don't use the verb (is/are) in sentences like that. In other languages some additional words may be needed. The common practice is to allow translators deal with the whole string, not separate words, good examples are ngettext and gotext plurals.
Probably the simplest way is to implement the ICU message format support. It is supported by Crowdin and there is a Go module for parsing it.

Thank you, then I think ICU message format  is the answer.

@mikhirev thank you for those details and your research into libraries. ICU format looks good. We could probably force ICU format into ini, and normally I'd be against changing format of config files, but maybe this is an opportunity to look into getting away from ini (even if only for translations).