utf8 is not a function

Hello,
I've updated react native to 0.46 from 0.44 for my company project. But problem occurred , when i compiled release version.
** I initiated a new 0.46 react native project, the problem also exists.**

The package file content is :
{
"name": "smartpark",
"version": "0.0.1",
"private": true,
"scripts": {
"start": "node node_modules/react-native/local-cli/cli.js start",
"test": "jest"
},
"dependencies": {
"babel-core": "^6.25.0",
"react": "16.0.0-alpha.12",
"react-native": "0.46.0",
"react-native-image-crop-picker": "^0.14.1",
"react-native-progress": "^3.3.0",
"react-native-splash-screen": "^2.1.0",
"react-native-tab-navigator": "^0.3.3",
"react-navigation": "^1.0.0-beta.11",
"react-redux": "^5.0.5",
"redux": "^3.7.1",
"redux-persist": "^4.8.2",
"redux-thunk": "^2.2.0",
"start": "^5.1.0"
},
"devDependencies": {
"babel-jest": "20.0.3",
"babel-preset-react-native": "2.1.0",
"jest": "20.0.4",
"react-test-renderer": "16.0.0-alpha.12",
"redux-devtools": "^3.4.0"
},
"jest": {
"preset": "react-native"
},
"main": "index.android.js",
"repository": {
"type": "git",
"url": "ssh://git-gogs@git.hanyunsh.com:10022/zgc2537/SmartPark.git"
},
"author": "",
"license": "ISC",
"description": ""
}
thx

"react-native run-android" is ok

The bug report is missing some necessary details. Please read the contributing guide and open a new issue, thanks!

@dgoudy Thx, it works for me.

@dgoudy i have same problem. can you explain step detail for it?. thanks

@roofiah just update version of nodejs of system to the 6.11.1, and it will be ok.

I had the same issue and after upgrading node everything was fine again.
Kudos to @zgc2537

thk ,update node is working

This sounds like it might still be an issue. Package.json says RN supports Node 4+
https://github.com/facebook/react-native/blob/master/package.json#L11


Full implementation of to-do list web app for code review

This commit adds a fully complete to-do list web app for code review by a coding partner at Microverse.

Thank you very much for your time @Efiamotu-1, I will begin work on these issues as soon as possible.


[sdl2-image] Updated to 2.0.5

I generated the hash by running CertUtil on the official release zip. Hope that's the proper way to get it.

 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.:x: Vultraz sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

I think most people get the new hash by updating the version and then cut and paste from the hash mismatch error message.

Thanks for the update!  It looks like the only failure was due to an unrelated flaky port so this is clean.


Synchronous execution of SUBSCRIBE and PUBSUB CHANNELS

ioredis version: 4.6.2
I'm trying to achieve synchronous execution of SUBSCRIBE and PUBSUB CHANNELS commands; so PUBSUB CHANNELS command should receive channel i subscribed to in the SUBSCRIBE command. As for now, sometimes I get an empty reply, sometimes - subscribed channels.
Here is my code:
const Redis = require('ioredis');

(async () => {
  const cluster = new Redis.Cluster(
    [
      'redis://<IP_ADDRESS>:7000',
      'redis://<IP_ADDRESS>:7001',
      'redis://<IP_ADDRESS>:7002'
    ],
    { enableOfflineQueue: false }
  );

  cluster.on('ready', async () => {
    await cluster.subscribe('ch');
    const reply = await cluster.pubsub('CHANNELS');

    console.log(reply);
  });
})();

Output:
~/application $ node ./src/redis-cluster.js
[]
~/application $ node ./src/redis-cluster.js
[]
~/application $ node ./src/redis-cluster.js
[ 'ch' ]
~/application $ node ./src/redis-cluster.js
[]

Could you please advice me on how to change my code to always get the list of subscribed channels?

It's a limitation on the redis server side. When subscribing a channel, ioredis will only subscribe on a single node of the cluster. Redis server won't sync the subscription to other nodes, so when running .pubsub, the command may be sent to a different node, in which case an empty array will be returned.
As a workaround, there's a private variable that recording the subscribed channels on the client side:
await cluster.subscribe('ch');
console.log(cluster.subscriber.subscriber.condition.subscriber.set)`

But this may change in the later version.

Thank you for your detailed explanation.


Error span causes refactoring suggestions

TypeScript Version:  nightly (2.6.0-dev.20170915)
Code
function f(): number {
    return {
        x: 1,
    }
}

The entire return statement has a red squiggly line under it.
Click anywhere in the error span and there will be a suggestion to refactor to an inner function.
Remove : number and there are no refactoring suggestions.
(Tested in vscode.)
Expected behavior:
Presence of a compile error doesn't cause a refactoring to be suggested.
Especially because it looks exactly like a code fix, but extracting to an inner function won't fix the compile error.
Actual behavior:
It does.
In fact, hovering over the light bulb shows "Show Fixes", but this is a refactoring, not a fix.

Moved to Microsoft/vscode#34481


Update composer.json

I think PHP-CS-Fixer has out grown the script-label and it doesn't fix the Symfony Coding Standard but PHP code style issues to a selected standard.

:+1:

:+1:

Thank you @SpacePossum.


Problem searching for effective in Observation

Hi,
I trying to search for a observation by effective (DateTime).
I have tried this:
http://spark.furore.com/fhir/Observation?subject.identifier=12345&effectiveDateTime=2012-09-17
This does not give me any result.
Could you please help me how the best way is to search for effective(period og dateTime)?
Thanks

For me, Spark returns 16 resources on this search. All of these are observations where the identifier of the subject is 12345.
effectiveDateTime is not a valid search parameter, and Spark will ignore that, so I guess the result Spark sends is not what you were looking for. For valid search parameters, check the list on the bottom of the resource page. For the effective date on Observation, you need to use 'date':  Obtained date/time. If the obtained element is a period, a date that falls in the period	Observation.effective[x]
So your search would be:
http://spark.furore.com/fhir/Observation?subject.identifier=12345&date=2012-09-17


Resize-able column

Table columns doesn't look good and support only a long schema.
It would help to have a resize-able column to adjust to suit wider audience

rolled-out in 0.1.101


Add ActivityIndicator's color support

When using the following code, you need to change the color of the ActivityIndicator in order to see it
AlertToast(displayMode: .alert, type: .loading, style: .style(     backgroundColor: Color.black ))
Now, you can use this:
AlertToast(displayMode: .alert, type: .loading, style: .style(     backgroundColor: Color.black,     indicatorColor: Color.white ))

This feature already pushed. Please check the latest commit and see if it suits your needs.


Add some site search enhancements

like,

remember the search string between searches
add lunr search help
it seems the lunr search strings are not handled as described in the lunr searching help (e.g. - prefix does not exclude the given search string)

Solved by #20


Add TOC in post

Hello,
I'm trying to add a Table of Content (TOC) in posts on my blog following the official guide provided by Ghost : https://ghost.org/docs/tutorials/adding-a-table-of-contents/#adding-a-table-of-contents-to-your-theme
It'w working perfectly with code injection or modifying theme if I add it inside content.
The best should be to have this TOC on the side of the post content, before the share/top buttons.
I've added them at this place, but I don't succeed to have a good presentation with colums in 100% of browser display, and I also don't know how to not display in mobile view.
Here is how it renders on my blog :
Normal view, we don't see the entire TOC :

View with unzoom at 80%

Mobile view:

Do you think you can guide me on how to implement this on best way ? Or include this into your great theme and let user enable it with a parameter if he wants, like it's done for fade animation or search key (if possible).
Thanks in advance.
Guillaume

Hi,
I'm afraid that I don't have plans to implement this feature. I can't also provide a full description on how to implement it in a good way since that will require deep changes in the current layout, sorry.
What I can tell you is that you will probably have to remove/reset all the styles for desktop on the classes m-share, m-share__content and m-icon-button in-share so the share bar is always fixed at the bottom, just like it happens on mobile, that way you will have enough space on the right side.
Then you can add the following styles:

Then add your TOC right after the div with the class l-post-content. The code might look like this:

Of course that's just a very rough implementation, you still have to adjust more styles and probably fix some issues, but at least I think it can gives you an idea of how it can be implemented properly.

Thanks for your answer, I will have a look at your suggestions and try to set it up.

I will close this issue for now, please feel free to reopen it if needed.


Duplicate props shown in the List Item Tag playground code snippet

Describe the bug / expected behavior
On List Item Tag's playground page, there is a duplicate prop backgroundColor that should have been fontColor instead.
What are the steps to reproduce?

Go to https://brightlayer-ui-components.github.io/react/components/list-item-tag/playground
Observe that there are two backgroundColor prop being shown in the code snippet area at the bottom of the screen.

Screenshots / Screen recording

Your environment information
Chrome, MacOS
Suggested fix
https://github.com/etn-ccis/blui-react-component-library/blob/db563fc5664f3fdb5295128b37a7c13ce3a9e06a/docs/src/componentDocs/ListItemTag/playground/PreviewComponent.tsx#L20
This line, replace all backgroundColor with fontColor should do the trick.
Anything else to add?
Please leave this for someone new to React or new to this project to fix.

Fixed in the migration to shared tooling


refresh materialized view concurrently

@TheExGenesis what does this do? what's it used for?

It looks like it's a follow up to this PR https://github.com/TheExGenesis/community-archive/pull/115#issuecomment-2410737427
which creates a new table/view to store threads:

The materialized view 'main_thread_view' contains only the tweets in the main thread

and this PR is making a change to make this faster/more performant?

It looks like it's a follow up to this PR #115 (comment)
which creates a new table/view to store threads:

The materialized view 'main_thread_view' contains only the tweets in the main thread

and this PR is making a change to make this faster/more performant?

it's not a followup to that, it just refreshes these views concurrently so selects on those views won't block


Unsaved Changes (Timeout Error)

My colleagues and I are collaborating on a DrawIO project. A couple of us are experiencing issues with the project automatically saving and get the following timeout errors:

Others are not experiencing any issues at all when working in the same project. Any help on resolving this issue would be great.

So, two of you get timeout issues on that specific diagram and others don't? What are the browsers/versions/OSes involved?
What browser extensions do you have? Does it work private/incognito?

Hello @davidjgraph, I'm not sure if I can reopen this issue but I am currently having the same problem where I can open all of my diagrams but one. I always use Chrome (currently Versión 97.0.4692.99) and just tried with Safari but I am getting the same message. I am using MacOS BigSur 11.5.2 After I click on "Try again" I get a second error "Access Denied". I have also tried in Incognito with the same results.

@maichomper Please open a new issue and file as a bug report.


Handle issue where DB secret already exists by requeueing and restarting the flow

Description
Before this PR, when a Secret already existed, the operator would report an error and retry. Now, it only retries, as this can happen under normal circumstances.

@orishoshan bump

@orishoshan bump

@amitlicht thank you!


Integrate Surcharges in oscar

Implements surcharges in Oscar.
A surcharge, also known as checkout fee, is an extra fee charged by a merchant when receiving a payment by cheque, credit card, charge card or debit card (but not cash) which at least covers the cost to the merchant of accepting that means of payment, such as the merchant service fee imposed by a credit card company.
So this surcharge can be used as a payment fee but also as any other fee that you want to have.
For example service costs.
You can use the calculate method on a Surcharge to do all sorts of calculations.
Surcharge can even be implemented into a model if you want.
Overwrite the oscar.apps.checkout.applicator.SurchargeApplicator.get_surcharges to implement some surcharges.
After that you can also overwrite get_applicable and based on the basket return if it's applicable to that person or whatever.

rebase please

It's rebased now, please review.

It will definitely need at least some documentation.

I added a new how to to the docs called surcharges.

Hey @solarissmoke thanks for reviewing. I hope to process these on sunday!

@solarissmoke all your reviews are processed.


pull error: error copying from layer download progress reader: download cancelled

A user reported the error message: error copying from layer download progress reader: download cancelled while attempting a pull.
This is a horrible error message as it has nothing to do with copying download progress and seems to be that there is an issue actually pulling the image/layer.
This error is produced @ https://github.com/docker/docker/blob/v1.8.1/graph/pull_v2.go#L225
Which comes from the defer beneath it: https://github.com/docker/docker/blob/v1.8.1/graph/pull_v2.go#L228-L236
It seems that any error produced by this function is being masked (as far as the user can see) the errors.New call. The real error should be visible in some way to the user.

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

This bad error message is being removed by https://github.com/docker/docker/pull/15955
We may need some additional work to expose a useful error message.

this issue has been gone? Seems graph dir is missing...

Yes, I think this has since been fixed. The error message should now pass through the network-level error that caused the pull to fail.
@cpuguy83: OK to close?


[select] dropdown auto width

About styles 🛠
If you have text that overflows in the select options they are hidden, there should be a prop to enable us to set the containing .dropdown because currently the dropdownClassName and dropdownStyle props gets sent to the .select-dropdown and not the parent container, so you cant use them to set the .dropdown width to auto
This is without the auto width on .dropdown:

This is with the auto width on the .dropdown:

This requires a prop to control whether Option follows Select's width, this is common feature, I'll do it.

@PaulPCIO
now v1.6.0 is out, the new feature is already available.


Compiler warnings/errors on VS 2010 (_CRT_SECURE_NO_WARNINGS)

On Windows 7 with Visual Studio 2010 I am seeing the following warnings (which is being treated as an error).
C:\<path removed>\cpputest-3.7.2\src\Platforms\VisualCpp\UtestPlatform.cpp(91) : error C2220: warning treated as error - no 'object' file generated
C:\<path removed>\cpputest-3.7.2\src\Platforms\VisualCpp\UtestPlatform.cpp(91) : warning C4996: 'localtime': This function or variable may be unsafe. Consider using localtime_s instead. To disable deprecation, use _CRT_SECURE_NO_WA RNINGS. See online help for details.
C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\INCLUDE\time.inl(112) : see declaration of 'localtime'
C:\<path removed>\cpputest-3.7.2\src\Platforms\VisualCpp\UtestPlatform.cpp(105) : warning C4996: '_vsnprintf': This function or variable may be unsafe. Consider using _vsnprintf_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.
C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\INCLUDE\stdio.h(363) : see declaration of '_vsnprintf'
C:\<path removed>\cpputest-3.7.2\src\Platforms\VisualCpp\UtestPlatform.cpp(113) : warning C4996: '_vsnprintf': This function or variable may be unsafe. Consider using _vsnprintf_s instead. To disable deprecation, use _CRT_SECURE_NO _WARNINGS. See online help for details.
C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\INCLUDE\stdio.h(363) : see declaration of '_vsnprintf'
C:\<path removed>\cpputest-3.7.2\src\Platforms\VisualCpp\UtestPlatform.cpp(127) : warning C4996: 'fopen': This function or variable may be unsafe. Consider using fopen_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.
C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\INCLUDE\stdio.h(234) : see declaration of 'fopen'

Basically _vsnprintf, fopen, and localtime are deprecated/unsafe and the compiler recommends replacing the usage with the _s variants. Alternatively if you add /D_CRT_SECURE_NO_WARNINGS it builds fine.
For reference I was attempting to build with cmake with an nmake generator after pulling in the vs2010 x64 toolchain. I was able to get around the issue by altering the cmake cache to include the necessary define to not display these warnings.

I assume these are fixed now, at least I fixed them on the gcc-dialect build.


Can't connect to cloud config server, from inside docker compose

Describe the bug
So, I try to connect my spring app to the cloud config server, but it doesn't work when I try to access it after I deploy my app from inside the docker.
networks:
  vd-network:

services:
  config:
    image: hyness/spring-cloud-config-server:3.1.3-jdk11
    deploy:
      labels:
        description: "Spring Cloud Config Server"
      mode: replicated
      restart_policy:
        condition: on-failure
    networks:
      - vd-network
    environment:
      - SPRING_APPLICATION_NAME=config
      - SERVER_PORT=8888
      - SPRING_CLOUD_CONFIG_SERVER_GIT_URI=<URI>
      - SPRING_CLOUD_CONFIG_SERVER_GIT_REFRESH-RATE=3
      - SPRING_CLOUD_CONFIG_SERVER_GIT_USERNAME=<NAME>
      - SPRING_CLOUD_CONFIG_SERVER_GIT_PASSWORD=<PASSWORD>
      - SPRING_CLOUD_CONFIG_SERVER_GIT_DEFAULT-LABEL=main
      - SPRING_CLOUD_CONFIG_SERVER_GIT_SKIPSSLVALIDATION=true
      - ENCRYPT_KEY=<KEY>
    ports:
      - "8888:8888"
  app:
    image: my-image
    ports:
      - "9000:9000"

bootstrap-docker.yaml
  application:
    name: ${name:config}
  cloud:
    config:
      failFast: true
      name: ${name:config}
      profile: ${profile:local}
      debug: true
      uri: ${config.host:http://config:8888}
      label: main
  config:
    import: optional:configserver:${config.host:http://config:8888}

This doesnt work, and If I change it to localhost, it works only when deploying the app directly, but not when running through docker.
What is the reason for the inability to connect. when running through docker?

Can you provide a complete, minimal, verifiable sample that reproduces the problem? It should be available as a GitHub (or similar) project or attached to this issue as a zip file.

Problem was they were not on the same network, and after that I had another problem of the app starting before the cloud config server and requesting the data from it from an endpoint, but as it was not fully started it failed. I fixed it by adding:
deploy:
  restart_policy:
    condition: on-failure

Now it fails until the server starts and then it is able to connect. I have tried using healthchecks and conditionals, but the cloud server config image does not allow execution from terminal, I tried with wait-for-it.sh, with which I had some other problems. So I solved it with this.
So now my app looks something like:
my-app:
  image: my-app-image
  deploy:
    mode: replicated
    restart_policy:
      condition: on-failure
  depends_on:
      - config
  networks:
    - vd-network
  ports:
    - "9000:9000"

Thanks for the update!

No, problem. The less people have to do this, the better.


Is static_assert in intrusive_ptr required?

https://github.com/DymOK93/KTL/blob/master/include/smart_pointer.hpp#L1357
Often this compile-time assertion gets in the way of finding the real cause of a compilation error.
Example:
template <class ConcreteContext>
class ContextBase {...};

template <class ConcreteContext>
void intrusive_ptr_add_ref(ContextBase<ConcreteContext>* ptr) {...};

template <class ConcreteContext>
void intrusive_ptr_release(ContextBase<ConcreteContext>* ptr) noexcept {...};

class SomeContext : ContextBase<SomeContext> {...}; // Oops... Private inheritance due to a typo

In the case of a large project, it becomes almost impossible to debug this compilation error.
What do you think, @GeorgyFirsov?

I think, the more errors can be expressed in more readable form, the better.
But you should consider one limitation of static assertions:
template<size_t N>
class Foo {
  static_assert(N != 0, "error");
};

using Boo = Foo<0>; // OK

I.e. static_assert doesn't raise a compilation error until you try to create an object of the "forbidden" type.
If it is critical, you have to use enable_if (assume C++17)


chore: cleanup readme, remove .gitmodules, add author

Proposed changes
Small cleanup of readme, author and .gitmodules
Fixes

Types of changes
What types of changes does your code introduce? (A breaking change is a fix or feature that would cause existing functionality and APIs to not work as expected.)
Put an x in the box that applies

[ ] Non-breaking fix (non-breaking change which fixes an issue)
[ ] Breaking fix (breaking change which fixes an issue)
[ ] Non-breaking feature (non-breaking change which adds functionality)
[ ] Breaking feature (breaking change which adds functionality)
[ ] Refactor (non-breaking change which changes implementation)
[ ] Messy (mixture of the above - requires explanation!)

Checklist
Put an x in the boxes that apply.

[ ] I have read the CONTRIBUTING doc
[ ] I am making a pull request against the main branch (left side). Also you should start your branch off our main.
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have locally run services that could be impacted and they do not present failures derived from my changes

Further comments

aah I was too eager, didn't think about the spell check
        13 | * Ardian Abazi<EMAIL_ADDRESS>       13 | * Ardian Abazi<EMAIL_ADDRESS>[0x
       13 | zi<EMAIL_ADDRESS>[0xArdi](https://github.com/0xArdi)


Get hash of file, returning object and not a string

Context

node version:  v10.10.0
module (formidable) version: 1.2.1

What are you trying to achieve or the steps to reproduce?
I'd like to use the option hash to rename my file using the a sha1 string.
form = formidable.IncomingForm({
        hash: 'sha1'
      });

  form.on('fileBegin', function(name, file) {
    if (file.name !== '') {
      var ext = file.name.match(/\.(jpg|jpeg|png|pdf)$/i)[0];
      file.path = './public/uploads/' + file.hash + ext;
    }
  });

form.on('file', function(name, file) {
    var resizeImageOptions;
    if (file.name !== '') {
      resizeImageOptions = _.find(fileFields, { field: name }).resizeImageOptions;
      if (file.name.match(/.jpeg|.jpg|.png|.gif/g)) {
        if (resizeImageOptions) {
          ImageHelper.resize(file, resizeImageOptions.imageWidth || 0, resizeImageOptions.imageHeight || 0, resizeImageOptions.forceResize || false);
        } else {
          ImageHelper.resize(file, 500, 500, true);
        }
      }
    }
  });

form.parse(request, function(err, fields, files) {
    var item = _.clone(fields);
// here I save the data in my database
});

What was the result you got?
The file is rename by [Object object].png.
What result did you expect?
The name of the geenrated sha1 string with the correct extension.

True. It seems like a bug and that this object is actually the return of crypto.createHash. Or maybe it's intentional. And actually it's not the hash OF the file, it's just random hash I guess.
If you want the filename to be the hash of the file, use the the newly introduced in #591 options.filename which is a function.

Thank you for your response.
Until the feature will be added I found an other solution.
I'm using crypto in the EVENT fileBegin to generate my hash and change the name of the file. And on the EVENT parse, to save the correct name in the database, I'm using a regex on the new path file to get the true name.

I don't see why you're doing it that way. Just use the options.filename.
Something like that should work?
const hasha = require('hasha'); // or `crypto`

const form = formidable({
  keepExtensions: true,
  uploadDir: `${__dirname}/public`,
});

const filename = (part, form) => {
  const hash = hasha.fromFileSync(part.filename, { algorithm: 'sha1' });

  return `${form.uploadDir}/${hash}`
}

And just noticed, please don't use formidable.IncomingForm, just formidable() or new Formidable().

file.hash is calculated at the end , so not available on 'fileBegin'
see src/PersistentFile.js end method

file.hash is calculated at the end , so not available on 'fileBegin'
see src/PersistentFile.js end method

@tunnckoCore it is not a bug, but it is reusing the same variable name, which can be confusing
@anthonycluse you are now using a hash of the filename

@tunnckoCore it is not a bug, but it is reusing the same variable name, which can be confusing
@anthonycluse you are now using a hash of the filename


Menu add more subcat ...

currently we have :
** Category > (menu_type = 2 & parent_id = NULL)

link (menu_type = 1 & parent_id = 3)
link (menu_type = 1 & parent_id = 3)

do you think we could have :
** Category > (menu_type = 2 & parent_id = NULL)

SubCategory > (menu_type = 2 & parent_id = 3)

link (menu_type = 1 & parent_id = 4)
link (menu_type = 1 & parent_id = 4)

etc ... whatever

Thank you for suggestion, but we are not planning to make more than two-level menus at the moment.

All right !
If I have the time I will do it and share it after ^^


Multi-uav simulation with qgc

I have successfully built 3 uavs in gazebo, but when I start qgroundcontrol, it can only connect with iris1 and iris2. I don't know whether there are some parameters I missed to change. Do you have any suggesstions?

Did you ever find any solution to this? I'm am having the exact same problem.


expanding the mix task to include Phoenix

hi there
this is a stub, am trying to work out how to go from a fully designed database, say an ER or other visualization of a complex database into ecto and then into Phoenix
https://github.com/parkerl/fishing_spot/issues/2
looks like this mix task generates the models for ecto
https://github.com/alexandrubagu/ecto_generator/blob/master/lib/mix/tasks/ecto.dump.schema.ex
but what about Phoenix integration?
I assume the answer would be to fork this repo and merge it with a Phoenix generator for HTML or ?
https://github.com/phoenixframework/phoenix/blob/master/lib/mix/tasks/phoenix.gen.html.ex
leaving this as an open question for myself and others that my to come to this same issue

looking at this more, the gen model for phoenix is closer
https://github.com/phoenixframework/phoenix/blob/master/lib/mix/tasks/phoenix.gen.model.ex
am thinking that maybe a Phoenix.Gen.HTML could be modified to have a switch like "--use-schema-db=dbfile.sql" instead of the usual "Post posts title body"
and that there would be a conditional in the phoenix.gen.model or phoenix.gen.html to call this file instead
apologies if my terminology is poor, am new to Elixir / Phoenix

Hi there,
When I start this project I was thinking that will be merged into ecto, here's the pull request: https://github.com/elixir-ecto/ecto/pull/1722
Basically you need to add this projects into you mix.exs deps, run mix deps.get and then you can ecto.dump.schema. Sometime you will need to edit generated files because sometime you can have indexes, other primary keys etc.
If you have better ideas pleasse open a pull request and I will be happy to discuss or implement new features. 👍  😄

ok, I am working on something inspired by this


much cheaper plasma

About the PR
8500 is a literal scam considering atmos prices have been nerfed lately
i think 4000 is still somewhat of a scam but should be fine
Media

no

[ ] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

tweak: Plasma gas canisters are now much cheaper.

it was 8500 before??!? gosh..

test fail not my fault
by the way, if this is merged now and server downloads the update, the 24-hour round should receive the change as it's an YML change


"Overwrite existing properties" appends

I have "Overwrite existing properties" toggle on, but it still appends instead of overwriting what is already there.

This should be fixed in 0.10.0.


Use electron's getPath() API to get the user's app data directory

Docs: https://github.com/electron/electron/blob/master/docs/api/app.md#appgetpathname
Closes: https://github.com/zeit/hyper/pull/1960
Fixes: https://github.com/zeit/hyper/issues/1944
A comment at https://github.com/zeit/hyper/pull/170/files#r83043056 suggests
(process.env.XDG_CONFIG_HOME !== undefined)
    ? join(process.env.XDG_CONFIG_HOME, 'hyper')
    : app.getPath('userData')

but I have no clue how relevant that still is. Detailed discussion at https://github.com/zeit/hyper/pull/1595

also solves https://github.com/zeit/hyper/issues/137

Merging now, we can revisit the use of XDG_CONFIG_HOME later


Strange stretched images at top of features

fixes  #183
I added height: auto; to the image css class. This
StackOverflow question indicates that this will resize the
image's height in order to keep the native image aspect ratio.
Testing:

checked that http://localhost:3000/1900/09/11/border looks better
checked that http://localhost:3000/2017/01/09/under-no-management
was un-affected

Before

After

Great job! Thanks, @astronaut-wannabe! 🌟⭐🌟⭐


Add tracelogs  for rtcp

@jinleileiking What information would you like to be logged? Most errors are returned from the library so there is no need to log.

Trace rtcp packet type etc.

I'm going to move this to pion/webrtc.
I think it makes more sense to implement an RTCP trace there so we don't add an extra logging dependency on rtcp.


repeat() is not working

let animation: Box<dyn Animated<f32, Instant>> = Box::new(keyframes(0.0)
      .ease_to(1.0, Duration::from_secs_f32(1.0), Easing::Linear)
      .repeat()
      .run(Instant::now()));
...
animation.get(Instant::now())

The animation only plays once and that's it, I checked with repeat_n() and it works correctly so I think it's a bug.

Thank you for the report!
The problem was fixed here - https://github.com/monkin/glissade/pull/2.
The fixed version is already available at https://crates.io/crates/glissade/0.1.37.


Replyf function as in obsolete tbot

I noticed it is quite hard to make replies when user triggered any handleMessage requests.
Is there a Replyf function as in obsolete golang tbot library for making stateful go tbots?
thank you!
https://tutorials.botsfloor.com/develop-your-own-telegram-bot-with-golang-and-tbot-de726883b83c

There is no more Replyf function, but you can get same result by using plain SendMessage() with fmt.Sprintf() like so:
func replyHandler(m *tbot.Message) {
	c.SendMessage(m.Chat.ID, fmt.Sprintf("formatted %s", "text"))
}

Thank you for your explanation!
One more thing to ask, is there any function equivalent to register_next_step handler in PyTelegramBotAPI?
https://github.com/eternnoir/pyTelegramBotAPI/blob/master/examples/step_example.py
If not, I am thinking of using HandleFunc with regexp "." for handling step states, is that a good idea?
Thanks!

There is no equivalent to register_next_step in tbot so you need to implement it by hand using the persistence layer of your choice: file or some database.
I've done something similar in my small game, feel free to use it as an example (a bunch of old code, but still) https://github.com/yanzay/picasso/blob/master/login.go

Nice! I will try that out later.
Hopefully it works out and I will share the code.
Big thank you!


Training Codebase

Great work!
Do you have any plan to release your training code?
We want to evaluate on our custom dataset.

Hi,
Thank you for your interest in our work. We will release the training code once the paper revision is complete.

Hi,
Thank you for your interest in our work. We have released the code and processed data for training.


[UI] Settings-->Metrics: Support GraphQL subscription

Current Behavior
When MeshSync discovers a Grafana or Prometheus instance, they will be listed in the dropdown fields in the Settings page under Metrics if the user refreshes their browser to GET those entries.
Desired Behavior
The Settings-->Metrics-->Grafana page and the Prometheus pages are in need of using a subscription in order to expose discovered Grafana and Prometheus endpoints for selection without requiring that the user refresh their browser.
Implementation
Create GraphQL Subscription for these connections.
Acceptance Tests
The list of discovered Graf and Prom instances populate without page refresh.

Contributor Guides and Handbook

🎨 Wireframes and designs for Meshery UI in Figma
🖥 Contributing to Meshery UI
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Can i work on this one ??

I've been working on GraphQL, I can take a look and see if I can make something out of this.

Was there an open PR for this? Not sure I saw one, but I haven't made headway with this item since I've been working on the Next.js upgrade. You can take this one on, @MUzairS15

No, there hasn't been a PR on this item, yet. Still an open need.

Can i work on this one ??

Hi @Vaani-pathariya! Thanks for asking. I would like for the answer to be "yes". Just before we get this issue (or another issue) assigned, will you take a moment to confirm that you've be able to build and run Meshery Server and Meshery UI? Let's set you up for success. As you confirm this, will you also mention your thinking in terms of the approach to solving this issue?
Again, we would love for you to take this one (or if not this one another one). Let's get you ramped up and ready to go. Signal when ready.


Lab 11 2.3 Screenshot of finished code doesnt match to the code Blocks of text being asked to enter

Lab/Demo: 11
Task: 2.3
Step: 04
Description of issue:
This is one instance of where the Code within the instructions for the user to paste within Visual Studio does NOT align with the screenshots provided. Multiple places within labs 7-10 as well have experienced issues like this.
Code for student to enter:

Screenshot provided in instructions as solution following that step:

Checked the Github repo
https://microsoftlearning.github.io/PL-400_Microsoft-Power-Platform-Developer/Instructions/Labs/LAB[PL-400]_Lab11_Azure.html#exercise-2-create-azure-function-for-inspection-routing
That screenshot is after step 6 and it looks correct there so it must be something in the ALH copy of the repo


Issues merging optional values

Consider my main .rs file:
use figment::{Figment, providers::{Serialized}};
use structopt::StructOpt;

mod config;

use config::Config;

fn main() {
    let cli_config = Config::from_args();
    println!("{:#?}", cli_config);
    let app_config: Config = Figment::from(Serialized::defaults(Config::default())) /// theme is "glorious"
        .merge(Serialized::from(cli_config, "default"))
        .extract().unwrap();
    println!("{:#?}", app_config); /// theme becomes None if --theme is not set
}

Now consider my default implementation in the config.rs file:
use serde::{Deserialize, Serialize};
use structopt::StructOpt;

#[derive(Debug, Deserialize, Serialize, StructOpt)]
#[structopt(name = "xwebgreet")]
pub struct Config {
    #[structopt(short, long)]
    theme: Option<String>,
}

impl Default for Config {
    fn default() -> Config {
        Config {
            theme: Some("glorious".to_string()),
        }
    }
}

The issue is when I run the program and do not pass --theme as argument then cli_config has theme as None, and when merging it changes the value of theme to None even though a default value exists.
Can I somehow prevent this behavior ? When I have an Option type variable and if it's value is None, then while merging I don't want the value to be changed to None if it already exists.

This can be solved with #[serde(skip_serializing_if = "Option::is_none")]. Or if you want it to apply for the whole struct instead of having to annotate each field: skip_serializing_none from serde_with crate

This can be solved with #[serde(skip_serializing_if = "Option::is_none")]. Or if you want it to apply for the whole struct instead of having to annotate each field: skip_serializing_none from serde_with crate

Glad you found the solution!

Glad you found the solution!

It took me some time to wrap my head around how to use Figment with cli parsers. The examples are good, but they didn't follow the order many people use in their apps. I ran into this exact same issue and used the exact same solution as you. I wrote about it help others out. I just posted this in another issue, I really don't mean to spam, I truly think it's helpful when using cli args with Figment.
https://steezeburger.com/2023/03/rust-hierarchical-configuration/


buildNumber is not propagated in dev mode

buildNumber is not propagated in dev mode and DEV UI shows app version as null in the top right corner of http://localhost:8080/q/dev-ui/ and in http://localhost:8080/q/dev-ui/welcome

This behaviour is caused by https://github.com/quarkusio/quarkus-github-bot/pull/231 changes which sets the version based on the hash, buildNumber property is set to null in dev mode:

Hash works fine when running app in JVM production mode, here is the message from dev mode vs. production jvm mode:
2024-06-19 14:23:01,533 INFO  [io.quarkus] (Quarkus Main Thread) quarkus-bot on JVM (powered by Quarkus 3.11.2) started in 1.267s. Listening on: http://localhost:8080
    vs.
2024-06-19 14:30:02,608 INFO  [io.quarkus] (main) quarkus-bot 77f2357 on JVM (powered by Quarkus 3.11.2) started in 0.499s. Listening on: http://<IP_ADDRESS>:8080

I tried to propagate revision property same way as in https://github.com/quarkusio/quarkus-github-bot/commit/88d76b2b5a38be56981922403967a73a73ff5705 and expected value was visible in configuration editor.
Seems this all goes to buildnumber-maven-plugin and how it sets the buildNumber property. @gastaldi any tips on this?

That doesn't seem to be propagated as a ConfigSource during dev mode. As a workaround, you can do the same as I did in registry.quarkus.io: add buildNumber=999-SNAPSHOT to your local .env, which will display the version as 999-SNAPSHOT when in dev mode/test


Requesting: Non member sign up workshop with library staff

What is the problem to solve?
Fully understanding the language, needs and limitations around providing requesting to those not already registered as library members.
What stage of research is it? (discovery/general usability testing/post release)
Discovery
What are the objectives?
To ascertain if there are things we need to take note of when we build the non member requesting workflow
What are the assumptions?
What research technique is required?
A workshop with the library staff members
Optimal number of participants
5
Acceptance criteria
Deadline for results/dependencies:

Emailed David langrish to get some library staff members to join our workshop


eikonomiya-tool todo list

This issue serves as a list of things to consider and ideas when the front tool will be made.

tool is now linked to the project
for simplicity purposes, i'll need to merge the julia server into eikonomiya-front (that will be renamed eikonomiya later) in nextjs code (sadly)
api will change a bit to reflect that
also it will shrink the docker image size quite a bit.
i still need to look on how to run passive code on the server.
Maybe map every processing operation in the /api/refresh endpoint ? Then run a cron service or something in the docker image ?


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_2a509509-fd39-44aa-aafc-abcca8a8f774/pull/1 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target_2a509509-fd39-44aa-aafc-abcca8a8f774.git

Docs Build status updates of commit 31ebeb2:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Release_Archive_master_2020-04-07-09-48-10 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Release_Archive_master_2020-04-07-09-48-10

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


Support qualified names for tables

See http://groups.google.com/group/scalaquery/browse_thread/thread/12eae34c368e96bd

If you are in a hurry, it's available in my fork at https://github.com/plalloni/scala-query

This was merged into master in dfd6597de4ae2c622099aaecbf8beb23dfe3a7ac on 2011-09-10

So it should be in the next release, I guess...  Any ETA on that?  Thanks!


Update google-cloud-pubsub to 1.133.0

About this PR
📦 Updates com.google.cloud:google-cloud-pubsub from 1.116.4 to 1.133.0
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.cloud", artifactId = "google-cloud-pubsub" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.google.cloud", artifactId = "google-cloud-pubsub" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #361.


DefaultInterfaces for unix* are not default for Ubuntu

Originally opened at rfratto/ckit#5

Yo!
DefaultInterfaces are hardcoded to try eth0 and en0 and advertised by comment for unix-like platforms, but these will never be found on Ubuntu and probably many other linux flavors that use predictable network interface names.
Eg, ethernet is enp4s0f1 for me on Ubuntu 20.04 LTS. examining net.Interfaces() might be a better bet

Originally posted at https://github.com/rfratto/ckit/issues/5#issuecomment-1105469802

Hmm, I'll need to give this a bit of thought. I inherited the list from dskit, where the default list of interfaces is more of a hint than trying to be inclusive of all target platforms.


FR: include strict mode that does not accept partial expressions

This feature would help with validation of cron expressions

Can you elaborate and provide a few examples?

Sorry for the late reply. Here is an example:
0 * * * 1-5 appears as valid however the full cron should be 0 * * ? * 1-5.
I understand that the library "Supports 5, 6 (w/ seconds or year), or 7 (w/ seconds and year) part cron expressions", as per the Readme, however it would be nice to have a flag that would only accept strict quartz crons for 6/7 part cron expressions.

I don't think I fully understand.  0 * * * 1-5 is a 5 part expression that happens "every hour" (* hour, at 0 minute) and 0 * * ? * 1-5 is a 6 part expression (includes seconds) that happens "every minute" (* hour, * minute, at 0 second).  Both are valid, the second one is just a 6 part cron that specifies seconds.  If you want to ensure it only expects expressions with a certain number of parts (6,7) I would think you could do that from outside the library using expression.split(" ").length > 5 or something of the like.  I'm going to close this one out but if I am still misunderstanding and you still think there is a use case, I would welcome a PR>


chore(dev): Move to tox to simplify test/CI setup

Had this in a branch for a bit. I think it's working...

All coverage losses are a result of using branch tracing so that ifs are only considered fully covered if both the True and False paths are hit.

Any objections? Otherwise will merge today.


appliedSpans includes spans that didn't meet filter condition

When a filter condition is set, it's expected that spans that failed to meet the filter condition would be included in identifiedSpans but not included in appliedSpans.
Here's a test case:
https://github.com/resurfaceio/phileas-benchmark/blob/41b07620dd4b21cba9715d392154[…]1/src/test/java/ai/philterd/phileas/benchmark/RedactorTest.java
ps. The actual string redaction in these cases is correct, it's only an issue if you're inspecting FilterResponse.explanation() directly

Thanks @RobDickinson. I can reproduce this.


Wrong Language C++

Problem Description
My code base is C++ and Cmake and the repo is being marked as objective-c. The repo in questions is BeyondCrisisEngine
URL of the affected repository:
Last modified on:
2018-02-28
Expected language:
C++
Detected language:
Objective-C

This is happening because https://github.com/Impulse21/BeyondCrisisEngine/blob/master/include/BeyondCrisis/ThirdParty/Catch2/catch.h is far bigger than all other code combined at 425kb and header files are incredibly difficult to determine the language they belong to as they look very similar for all the C-family languages. As it is over 384kb it won't appear in the search results when you click the language after clicking the language bar.
As this appears to be a third party header file, the best thing to do is implement an override marking it or the ThirdParty directory as vendored.

Hi lildude,
Thank you for the quick response. I have updated the .gitattributes file as such
include/ThirdParty/Catch2/* linguist-vendored
but I am still seeing it marked as objective-C. Is there something I am doing wrong here? Does it take time of the change to reflect on the github's website?
Note: I have also tried the following:
include/ThirdParty/Catch2/catch.h linguist-vendored
include/ThirdParty/Catch2/catch.h linguist-language=Cpp
include/ThirdParty/Catch2/catch.h linguist-language=C++

Your file is in include/BeyondCrisis/ThirdParty/Catch2/ not include/ThirdParty/Catch2/. Could that be the cause?

Well that is embarrassing!! It is.
Thank you very much for the help!!!


chore(cherry-pick): For PR 89...92, 94, 98

Why is this PR required? What issue does it fix?:
What this PR does?:
This PR cherry-picks commits from branch develop to v0.6.x.
Followings PRs are added for cherry-picking:
https://github.com/openebs/dynamic-nfs-provisioner/pull/89
https://github.com/openebs/dynamic-nfs-provisioner/pull/90
https://github.com/openebs/dynamic-nfs-provisioner/pull/91
https://github.com/openebs/dynamic-nfs-provisioner/pull/92
https://github.com/openebs/dynamic-nfs-provisioner/pull/94
https://github.com/openebs/dynamic-nfs-provisioner/pull/98
Does this PR require any upgrade changes?:
If the changes in this PR are manually verified, list down the scenarios covered::
Any additional information for your reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Checklist:

[ ] Fixes #
[x] PR Title follows the convention of  <type>(<scope>): <subject>
[ ] Has the change log section been updated?
[ ] Commit has unit tests
[ ] Commit has integration tests
[ ] (Optional) Does this PR change require updating NFS-Provisioner Chart? If yes, mention the Helm Chart PR #
[ ] (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
[ ] (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

Codecov Report

Merging #99 (ab20fd2) into v0.6.x (bab5c4e) will increase coverage by 0.11%.
The diff coverage is 37.75%.

@@            Coverage Diff             @@
##           v0.6.x      #99      +/-   ##
==========================================
+ Coverage   46.85%   46.97%   +0.11%
==========================================
  Files          29       29
  Lines        2356     2378      +22
==========================================
+ Hits         1104     1117      +13
- Misses       1174     1182       +8
- Partials       78       79       +1

Impacted Files
Coverage Δ

...kg/kubernetes/api/apps/v1/deployment/kubernetes.go
0.00% <0.00%> (ø)

provisioner/controller.go
0.00% <0.00%> (ø)

provisioner/provisioner.go
0.00% <0.00%> (ø)

provisioner/provisioner_kernel_nfs_server.go
0.00% <0.00%> (ø)

provisioner/signal_handler.go
0.00% <0.00%> (ø)

...es/api/core/v1/persistentvolumeclaim/kubernetes.go
67.44% <10.00%> (ø)

...ernetes/api/core/v1/persistentvolume/kubernetes.go
65.07% <14.28%> (ø)

pkg/kubernetes/api/core/v1/namespace/kubernetes.go
72.54% <20.00%> (ø)

provisioner/config.go
12.19% <37.50%> (+12.19%)
:arrow_up:

provisioner/helper_kernel_nfs_server.go
79.63% <90.00%> (-0.44%)
:arrow_down:

... and 2 more

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 bab5c4e...ab20fd2. Read the comment docs.


Handle text 'the_geom' columns as desired in #117

@rafatower for your review

:+1: looks good to me


Why getInsertedItemsWithIDs, getInsertedItems and getInsertedIDs removed?

Hi,
It's written in the Changelog of v0.5.0 that "Removing getInsertedItems, getInsertedIDs API. You have to collect them on your own.". Why were these great API removed?

I don't want to mix data in the inserted item and maybe the data is no ID column.
I think it's better to let you do it on your own by configure the insertTpl option.


[AutoDiff] SILGen differentiability witnesses.

Generate SIL differentiability witnesses from AST @differentiable attributes,
using lowered parameter indices, result indices (currently with capacity 1 and
set index 0), and derivative generic signature.
Resolves TF-869.

The TF-866 master issue tracks all retroactive derivative registration tasks.
Todos related to this patch:

TF-835: lower @differentiating attribute to SIL [differentiable] attribute instead of creating implicit @differentiable attributes. This is blocked by full retroactive derivative registration, see issue comment for details.
TF-919: explore emitting serialized differentiability witnesses. Currently, SILGen never creates serialized differentiability witnesses to avoid deserialization issues.

@swift-ci Please test tensorflow

CI failed:
swift: /home/swiftci/jenkins/workspace/swift-PR-TensorFlow-Linux/llvm/include/llvm/ADT/SmallVector.h:155: llvm::SmallVectorTemplateCommon::reference llvm::SmallVectorTemplateCommon<unsigned int, void>::operator[](llvm::SmallVectorTemplateCommon::size_type) [T = unsigned int]: Assertion `idx < size()' failed.
Stack dump:
0.	Program arguments: /home/swiftci/jenkins/workspace/swift-PR-TensorFlow-Linux/Ninja-ReleaseAssert+stdlib-Release/swift-linux-x86_64/bin/swift -frontend -emit-module -filelist /tmp/sources-0fc1a7 -supplementary-output-file-map /tmp/supplementaryOutputs-7871dc -disable-objc-attr-requires-foundation-module -target x86_64-unknown-linux-gnu -disable-objc-interop -sdk / -I /home/swiftci/jenkins/workspace/swift-PR-TensorFlow-Linux/Ninja-ReleaseAssert+stdlib-Release/swift-linux-x86_64/./lib/swift/linux/x86_64 -warn-swift3-objc-inference-complete -warn-implicit-overrides -enable-library-evolution -module-cache-path /home/swiftci/jenkins/workspace/swift-PR-TensorFlow-Linux/Ninja-ReleaseAssert+stdlib-Release/swift-linux-x86_64/./module-cache -module-link-name swiftCore -nostdimport -parse-stdlib -resource-dir /home/swiftci/jenkins/workspace/swift-PR-TensorFlow-Linux/Ninja-ReleaseAssert+stdlib-Release/swift-linux-x86_64/./lib/swift -swift-version 5 -O -enforce-exclusivity=unchecked -group-info-path /home/swiftci/jenkins/workspace/swift-PR-TensorFlow-Linux/swift/stdlib/public/core/GroupInfo.json -verify-syntax-tree -Xllvm -sil-inline-generics -Xllvm -sil-partial-specialization -Xcc -DswiftCore_EXPORTS -parse-as-library -module-name Swift -o /home/swiftci/jenkins/workspace/swift-PR-TensorFlow-Linux/Ninja-ReleaseAssert+stdlib-Release/swift-linux-x86_64/./lib/swift/linux/x86_64/Swift.swiftmodule -runtime-compatibility-version none -disable-autolinking-runtime-compatibility-dynamic-replacements
1.	Swift version 5.1-dev (LLVM c5340df2d1, Swift 11dcca990d)
2.	Contents of /tmp/sources-0fc1a7:
3.	While emitting SIL for 'differentiableFunction(from:)' (at /home/swiftci/jenkins/workspace/swift-PR-TensorFlow-Linux/swift/stdlib/public/core/DifferentiationSupport.swift:214:8)
4.	While silgen emitFunction SIL function "@$ss22differentiableFunction4fromq_xXGq_5value_13TangentVectorQzADQy_c8pullbacktxc_ts15_DifferentiableRzsAHR_r0_lF".
 for 'differentiableFunction(from:)' (at /home/swiftci/jenkins/workspace/swift-PR-TensorFlow-Linux/swift/stdlib/public/core/DifferentiationSupport.swift:214:8)
5.	While emitting SIL for 'original(_:)' (at /home/swiftci/jenkins/workspace/swift-PR-TensorFlow-Linux/swift/stdlib/public/core/DifferentiationSupport.swift:218:3)
6.	While silgen emitFunction SIL function "@$ss22differentiableFunction4fromq_xXGq_5value_13TangentVectorQzADQy_c8pullbacktxc_ts15_DifferentiableRzsAHR_r0_lF8originalL_yq_xsAHRzsAHR_r0_lF".
 for 'original(_:)' (at /home/swiftci/jenkins/workspace/swift-PR-TensorFlow-Linux/swift/stdlib/public/core/DifferentiationSupport.swift:218:3)

I didn't encounter this error until pulling from apple/tensorflow, investigating now.

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow

CI fails due to differentiability witness deserialization failures (see occurrences of UNKNOWN):
  Block ID #14 (SIL_BLOCK):
      Num Instances: 1
         Total Size: 12694b/1586.75B/396W
    Percent of file: 4.9487%
      Num SubBlocks: 0
        Num Abbrevs: 46
        Num Records: 56
    Percent Abbrevs: 100.0000%

	Record Histogram:
		  Count    # Bits     b/Rec   % Abv  Record Kind
		     16      3040     190.0  100.00  SIL_FUNCTION
		     10      2221     222.1  100.00  UnknownCode35
		      8       640      80.0  100.00  SIL_ONE_OPERAND
		      6      1144     190.7  100.00  SIL_INST_APPLY
		      4       452     113.0  100.00  SIL_ONE_TYPE_ONE_OPERAND
		      4       196      49.0  100.00  SIL_ONE_TYPE
		      4       792     198.0  100.00  SIL_BASIC_BLOCK
		      2       198      99.0  100.00  decls_block::PROTOCOL_CONFORMANCE_XREF
		      2       534     267.0  100.00  SIL_INST_WITNESS_METHOD

  Block ID #15 (SIL_INDEX_BLOCK):
      Num Instances: 1
         Total Size: 5686b/710.75B/177W
    Percent of file: 2.2167%
      Num SubBlocks: 0
        Num Abbrevs: 2
        Num Records: 4
    Percent Abbrevs: 100.0000%

	Record Histogram:
		  Count    # Bits     b/Rec   % Abv  Record Kind
		      1       324            100.00  SIL_DIFFERENTIABILITY_WITNESS_OFFSETS
		      1      1762            100.00  UnknownCode12
		      1       510            100.00  SIL_FUNC_OFFSETS
		      1      2936            100.00  SIL_FUNC_NAMES

These surfaced only after differentiability witness SILGen. Investigating now.

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow


Problem with breadcrumb on Runs page

This is how to reproduce this:

Go to the Services page of a repository (first screenshot)
Click one of the repositories in the Repository colum (second screenshot)
Click Runs (last screenshot)

https://cl2.aydo2.com/ays81/ays81/Services?ayspath=/optvar/cockpit_repos/yves

https://cl2.aydo2.com/ays81/repo?repo=/optvar/cockpit_repos/yves

https://cl2.aydo2.com/ays81/ays81/Runs?repo=$$repopath


All add-ons containing a manifest.json are considered WebExtensions

I submitted my add-on Mail Redirect and it was automatically accepted and didn't need manual reviewing, because it contained a manifest.json.
The manifest.json file however, did contain a "legacy" property, indicating that the add-on was in fact a legacy add-on that uses Thunderbird's overlay loader to load the XUL/JavaScript.
This type of add-on shouldn't get automatically accepted, but stay in the review queue.

Legacy add-ons using manifest.json are a weird Thunderbird hack so yeah, addons-server was never designed for these.
I don't know how easy it is to change this but I'll look into it eventually. If @darktrojan knows where to change this or how difficult it is I'd welcome a patch.

Sorry, I missed this. It should be reasonably easy to fix.

@darktrojan could you please look into this again, if it /should/ be reasonably easy to fix? last week another add-on was automatically accepted, while it should have been reviewed.


Refactor parseFunctionArgs and parseArgs

          This would be a great place to handle the opening parenthesis, if it's present, _before_ calling `parseArgs()` (and then to handle the closing parenthesis, after it returns)...like it says in the function header comment.  😉

Originally posted by @webbnh in https://github.com/arcalot/arcaflow-expressions/pull/23#discussion_r1471930600

Unfortunately, the hyperlink above to the comment is not working.  I think this one does, though.  However, the comment's link to the "earlier comment" doesn't work either; try this one instead.  (The reasons why are a long story, the short of which is that I've reported this to GitHub as a bug.)
In any case, here's what the second comment says [slightly edited]:

I have a suggestion for reworking this function, which I think will make it simpler and easier to understand:

before entering the loop, look for and remove the opening parenthesis -- it's supposed to be there, and it's an error if it's not;
then, if the current token is not a closing paren, enter the loop (i.e., don't enter the loop if there are no arguments in the list); inside the loop, parse the argument; then exit the loop if the next token is not a comma; inside the loop, continue to remove the comma and iterate;
after the loop, look for and remove the closing parenthesis -- it's supposed to be there, and it's an error if it's not.
This saves you from having to have a loop iterator and from having to have special cases for the first iteration of the loop. The only conditional is the exit condition in the middle of the loop if you encounter a closing paren instead of a comma after an argument.

If we were writing out the detailed grammar, it would look like this:
<function_call> := <identifier> <argument_list>
<argument_list> := "(" [ <arguments> ] ")"
<arguments> := <argument> [ "," <argument_list> ]

in specific, the opening and closing parens are part of the argument_list production but not part of the arguments production, and so they should be parsed independently of the arguments. (E.g., the loop should arguably be in a separate function.)

So, to summarize, it would be better if parseFunctionArgs() managed the opening and closing parentheses and parseArgs() managed only the sequence of zero or more arguments separated by commas and left everything else to its caller.


GFS: appendFile() and deleteFile() have small bug related to query

appendFile() currently modifies no object because there is no query passed in to find the object.
Currently in lib/extensions/gfs/gfs.js:169:
				return FSObject.findOneAndUpdate(node._id, {

It should be:
				return FSObject.findOneAndUpdate( { _id: node._id }, {

This is the same case for deleteFile() in lib/extensions/gfs/gfs.js:238.

Fixed

Btw, I removed deleteFile in ef9eee73afd6461040a17290a32a9a5a763049ed as there is no such function. Reference: https://nodejs.org/docs/latest-v8.x/api/fs.html

Yes but our GFS is currently not the exact map of fs. It was a
fucntionalities the GFS provides so a user can delete the file. Maybe just
keep it for now, but your choice.
On Thu, 11 Oct, 2018, 4:31 PM Kumseok Jung<EMAIL_ADDRESS>wrote:

Btw, I removed deleteFile in ef9eee7
https://github.com/DependableSystemsLab/ThingsJS/commit/ef9eee73afd6461040a17290a32a9a5a763049ed
as there is no such function. Reference:
https://nodejs.org/docs/latest-v8.x/api/fs.html
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/DependableSystemsLab/ThingsJS/issues/33#issuecomment-429155617,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJtt-J_meLSpuGyXCAas2N238A98jROhks5uj9TNgaJpZM4XAvOP
.


feat: add datasets-metadata endpoint

ref #268
@jpmckinney I'm prioritizing this one so that we can use this endpoint as part of our "download Data from the Registry" notebooks and download the data more easily (without having to copy and paste the URL from the registry or query the database for downloading all the datasets)
I've added the columns we need to automatically download the JSON files, plus others mentioned in the issue. But happy to add more if needed.

Can you paste the JSON response (just the first array entry)?

[
  {
    "id": 8,
    "title": "Argentina",
    "country": "Argentina",
    "last_retrieved": "2023-08-30T00:00:00",
    "retrieval_frequency": "",
    "update_frequency": "",
    "frozen": false,
    "source_id": "argentina_vialidad",
    "source_url": "",
    "region": "LA",
    "language": "",
    "data_from": "2023-01-01",
    "data_to": "2023-08-30"
  }
]

Do the translatable fields (title, etc.) get translated?

Yes

We can also add a test.

Sure

@jpmckinney Also, I'm not adding this endpoint anywhere in the front end yet, so external users won't be aware of it yet. We can decide where to put it (with Sebastian, maybe) later. (That is why I'm not closing the related issue with this PR yet)


Manually killing one POD container restarts all pods

Observed in testing somehting unrelated - @dchen1107 prio
We need a test that captures this - we can't let this slip through,  Maybe same as #5732?

I just reproduced this with newly built cluster from HEAD. This is definitely a regression introduced lately. Will look into it in detail.


Syncing Spatial Camera

How can I properly sync Spatial Camera which is a child of a kinematic body?
This is what my Player.tscn looks like:

The issue:

I tried doing a few things like adding "camera" into the unrealiable-sync field (cuz I reference it in my player controller script), also tried adjusting node hierarchy for player to make it work somehow but I ended up with a bunch of errors if I move the Sync anywhere else in the Players.tscn

Hey thanks for the very quick response, much appreciated!
Yes they both in-fact do move which is the desired behavior, however whats not the desired behavior is that the Camera is the same for both clients, both instances Camera is following the same cube which is incorrect, I'm not sure how to fix that
I do have transform and rotation in sync list

Ah got it. In that case, try putting
func _process(delta):
    $Camera.current = is_network_master()

in your Player.gd, to ensure that the global camera is the one of the current client's. This should hopefully work in any case. If it works, you can consider placing it instead in _ready or _network_ready so that it's only run once.


BGE_M3/modeling.py: Replace print() with either (already used) logger.info or at least print(file=sys.stderr)

Consider current code that loads the model
Right now if I want to make a script that outputs pure JSON to stdout, I need to work around the loader by temporary disabling print with globals()["__builtins__"].print = lambda *args, **kwargs: None as otherwise the model loader prints to the stdout that weights exist, breaking JSON.
Expected behavior:  stdout not being touched.
Proposed solution: logger.info doesn't outputs data to stdout and it's already used

Thanks for your suggestion! We have updated the source code.

Thanks!


Support Lumen 6.x branch

Changes proposed in this pull request:

Update composer to support Lumen 6.x

https://packagist.org/packages/nordsoftware/lumen-newrelic#2.0.2

Awesome thanks!
In case someone finds this having trouble updating, https://github.com/digiaonline/lumen-chained-exception-handler/pull/4 probably blocks actually using this just yet. I hope to sort out the CI breakage for them quickly so they can merge it.


pull request for e2e test at 11/12/2016 18:45:09

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

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


Glob support in optimizeDeps.include

Clear and concise description of the problem
In some libraries, files in different paths need to be loaded frequently. The page will be refreshed every pre-build. So it is necessary to pre-build all the files that may be used at startup.
So I think glob should be supported to easily add files that may be needed.

Suggested solution
support glob in optimizeDeps.include/exclude
Alternative
No response
Additional context
Related issue #699 #1545
Related pr #1571
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

I have a need of this before for packages with many sub-exports, e.g. my-pkg/some-module and many other modules that could be known ahead of time. But Vite currently only understands what you import and dynamically prebundle thereafter.
I think globbing would be tricky to work with exports since only a subset of files are allowed to be imported, I wonder if the better solution is to have Vite auto optimize all sub-exports if the exports field exists.

We have non-blocking optimization #8280, and it seems that this feature becomes unimportant.

Interesting. I think we could try Blu's proposal here, or maybe to avoid surprises only support something like include: ['my-pkg/*'] to include all the exports of that package. Or maybe a generic glob over all the names of dependencies in the package JSON plus their exports? Maybe it is fast enough.

I think this feature is still needed.
https://user-images.githubusercontent.com/6481596/175541563-20d133ab-f4d8-4a17-90e8-93d36e2c1665.mp4

Reproduction repo: https://github.com/sxzz/vite-optimize-deps
Reproduction steps:

clear all cache, rm -fr node_modules/.vite
run vite --force
Once the page is loaded you can see these messages on the terminal.

9:01:28 p.m. [vite] ✨ new dependencies optimized: element-plus/es, element-plus/es/components/card/style/css
9:01:28 p.m. [vite] ✨ optimized dependencies changed. reloading

Click About, vite will refresh the page. It means all states will be reseted, and the terminal will display the messages belows.

9:01:35 p.m. [vite] ✨ new dependencies optimized: element-plus/es/components/button/style/css
9:01:35 p.m. [vite] ✨ optimized dependencies changed. reloading

Click About again, it will work finally.

@sxzz I found that * could be used actually (it is not documented in Vite so it may change in future though). https://esbuild.github.io/api/#external
Setting ['./node_modules/element-plus/es/components/*/style/css2.mjs'] to optimizeDeps.exclude reduces reload.
Also for your specific case, ['./node_modules/element-plus/es/components/base/style/css2.mjs'] will work too.

I think we should add support for glob imports, at least document that * works. Thanks for taking the time to check this @sapphi-red, the second one is a nice trick to for element-plus.
@sxzz let us know if this works for you, better to wait to beta.3 though to test as there are a few other PRs to improve dev server perf about to be merged

@sapphi-red do you have a repro of this working with that exclude? Trying it now and I still see the reloads (that could be related to something different)

@sapphi-red Hmmmm seems that it can't work for me.

It worked for me with npm and yarn. But it didn't work with pnpm.
It seems symlink is affecting this.

This worked with pnpm. So symlinks does not work well.
import { defineConfig, normalizePath } from 'vite'
import vue from '@vitejs/plugin-vue'
import fs from 'fs/promises'
import path from 'path'
import url from 'url'

const exclude = './' + normalizePath(
  path.relative(
    url.fileURLToPath(new URL('./', import.meta.url)),
    path.join(
      await fs.realpath('./node_modules/element-plus/'),
      'es/components/*/style/css2.mjs'
    )
  )
)

// https://vitejs.dev/config/
export default defineConfig({
  plugins: [vue()],
  optimizeDeps: {
    exclude: [exclude]
  }
})

Stills can't work for me.

The first reload is expected. Does it reload two times?

Something odd is going on. I'll check it out. The first reload shouldnt be there either

@sapphi-red Yes,

@sxzz Your screenshot shows reloading is happening once and pre-bundle is happening twice.

Oh, just found it can avoid refresh page now, and I had a better experience now as the same as 2.x.

But it seems to be complicated for users to write such a config.
Actually, I wrote a config to pre-optimize deps to fix this issue before.
optimizeDeps: {
  include: await glob(['element-plus/es/components/*/style/css.mjs'], {
    cwd: $r('node_modules'),
    onlyFiles: true,
  }),
},

Suggestion: Third-party dep can provide a metafile to help Vite consider which files be included or excluded to be optimized. Out of the box for Vite users.
By the way, I have a question: Why some times it is possible to optimize dep without refreshing the page, and sometimes it is necessary to refresh the page.

@sxzz see this comment https://github.com/vitejs/vite/discussions/8721#discussioncomment-3002791

@sxzz as a summary of the new strategy:

By default, dynamic routes are not crawled because it looks at this point like a better default.
You can add the most common visited routes to optimizeDeps.entries and there wouldn't be any reloads. Adding this to the config works well in beta.4:

optimizeDeps: {
    entries: ['src/App.vue', 'src/components/Footer.vue'],
    ...

You could also achieve the same with optimizeDeps.include of some key deps, which leads to a faster cold start.

We need more data to see if not crawling dynamic routes is the best default here. An alternative could be to crawl by default leading to no reloads but slower cold start and let the user opt-out of certain dynamic imports:
optimizeDeps: {
    scanDynamicImports: string | Regex | (string | Regex)[] | boolean

By default it would be scanDynamicImports: true

@sxzz the idea above is now implemented in:

https://github.com/vitejs/vite/pull/8869

I think that 'dynamic-scan' should be the best for vue-router based projects, at the expense of keeping the complexity of the esbuild scanner around.
This for out-of-the-box cold start, the best possible perf is still 'lazy' + including some deps in optimizeDeps.include (I'm going to work on DX around hinting wich ones should be included)
Let me know if you test the PR.

pre-scan is great, but the users have to add some esbuild plugins to optimizeDeps.esbuildOptions.plugins. If some plugins is not supported on esbuild, then some files cannot be found and optimized.

  optimizeDeps: {
    devStrategy: 'dynamic-scan',
    esbuildOptions: {
      plugins: [
        Transform({
          plugins: [
            VueEsbuild(),
            ComponentsEsbuild({
              resolvers: [ElementPlusResolver()],
            }),
          ],
        }),
      ],
    },
  },

eager is better. Without any optimizeDeps.esbuildOptions.plugins. The page can be opened directly without any refreshes.


Two deposited items are in limbo in Argo

Greta has identified an issue with two H2 deposits when she tries to do the usual cataloging on them in Argo post-deposit.
Both hg666vd0263 and gj834gp4151 are showing both the "Close version" and "Open for modification" options grayed out in Argo, so she can't do anything.

From Greta:

I think these are one-time issues related to still open Hydrus workflows left behind by the migration. Since these druids were migrated and detached from Hydrus, the Hydrus workflows were never marked as completed. This blocks versioning because versioning requires the hydrusAssemblyWF to be complete.
I've marked the Hydrus workflow as complete for both objects and the buttons are no longer grayed out. We'll need to identify this specific situation for other migrated objects. I think what will be safe to do is mark hydrusAssemblyWF steps complete for each successfully migrated druid. This situation will druids that are Registered or Opened at the time of migration. Accessioned druids should be ok.

@amyehodge is there more for us to do on this ticket or should it be closed?

@jcoyne Sounds like the work @andrewjbtw  noted to mark hydrusAssemblyWF as complete for the subset of migrated druids in Registered or Open states is something that needs to be done now. And we need to add this into the workflow for the full migration in a few weeks. Who will do those tasks, and do they need tickets?
The two items called out in this ticket have been taken care of.

@amyehodge will we be able to just mark all of the hydrusAssemblyWF processes as complete at that time?

@jcoyne That's a question for @andrewjbtw.

That should be ok. Marking the steps complete won't run anything and since they'll never be in Hydrus again it should be safe.

@amyehodge I updated #1718 to add that step, so I believe this can be closed.


Remove categories, use standard Interfaces instead

Depends on removal of children language feature (#212)

Mostly done, probably want to error now if an instance has a category declaration.
Should probably also add hhconfig options for these


Fix usage code in README

Usage code does not work.
Authenticate function has a few arguments. It has not two arguments but three arguments.
It is hard to understand "your-username".  Mastodon account has a username and a email. It is correct to specify an email when authenticating.
timeline variable is not used, so go build will fail.
I fixed above.

Coverage remained the same at 81.291% when pulling 82ee148846ab6bd16bb3de716352f2e5d06b19f6 on suzukaze:fix-usage-code-in-readme.md into b1ec2978a2bded916668c5042217374fff530854 on mattn:master.

:+1:


shields: Modify overlay file order

Shield overlay file has to be taken into account after board overlay file.
ex: When a new node need to be created for a specific board (in
boards/shields/shield_name/boards/board_name/.overlay) and used by the
shield (in boards/shields/.overlay), the order of the overlay files is
important
Signed-off-by: Ismael Fillonneau<EMAIL_ADDRESS>

@mbolivar-nordic @galak Changing the order looks reasonable to me, but could you review/check there are no hidden side effects on the DTS processing itself.

@tejlmand @mbolivar As per #29074, this does seem to have broken support for at least the Mikroe click shield.

@ifillonneau this PR broke existing behaviour in non-logical way, as discussed #29017 .
Therefore this PR has to be reverted, see #29824 , and we need to discuss a proper way to handle your specific use-case.

Was there ever a soluton on handling this specific use-case?
Maybe a possible solution is to use three overlay files: project-specific board overlay, then shield overlay, then shield-specific board overlay.
As of right now, the shield overlay seems to come before the project-specific board overlay.

@tejlmand Was there ever a soluton on handling this specific use-case?

No, it seems @ifillonneau went silent, and he was the one having a use-case.
@Embla-Flatlandsmo if you can describe a proper use-case of your need, then we can take a look at it.
You are also more than welcome to propose a PR that can be used for further discussion.

@tejlmand My use case is as follows:
I want to build samples/subsys/display/lvgl for the nRF9160 DK using the st7789v waveshare 240x240 arduino shield. I do this by calling west build -b nrf9160dk_nrf9160ns -- -DSHIELD="st7789v_waveshare_240x240. This results in the following error message:
Error: nrf9160dk_nrf9160ns.dts.pre.tmp:599.1-13 Label or path arduino_spi not found
For this reason, I would like to be able to create an overlay file that is between the board device tree and the shield overlay. This overlay file would contain: arduino_spi: &spi3: {}.
This feature would also allow a user to specify which spi/uart/i2c node to use with their shields.
I will look into making a PR on this, but it will likely not happen in the next 3 weeks (at minimum).

@tejlmand ,  on my side, I lost my example so I am not able to reproduce the pbm, but I think that @Embla-Flatlandsmo have the same problem that I encountered( when I read my commit message, it seems to be the pbm).

Thanks, @ifillonneau  and @Embla-Flatlandsmo .
If I understand correctly, you can already today achieve what you want manually, by using -DDTC_OVERLAY_FILE=<my-file>.overlay, but you would like this to be automated, so that, for example, placing a:
shields/<board>_<shield>.overlay

such as: shields/nrf9160dk_nrf9160ns_st7789v_waveshare_240x240.overlay then that overlay would be applied automatically when building that specific board together with the particular shield.
Is that correctly understood ?
Couple of things we need to consider, before having a solution:

If user specifies two shields, like: -DSHIELD="st7789v_waveshare_240x240;shield2", should shields/nrf9160dk_nrf9160ns_st7789v_waveshare_240x240.overlay still be applied ?
Probably it should, but would like more opinions on this.
we already use _ for separating revision and build type, and this adds one more layer.
This can start to become hard to maintain, and predict.
For example, nRF9160dk uses revisions, so should we also have: nrf9160dk_nrf9160ns_0_14_0_st7789v_waveshare_240x240.overlay, and what if both nrf9160dk_nrf9160ns_0_14_0_st7789v_waveshare_240x240.overlay and nrf9160dk_nrf9160ns_st7789v_waveshare_240x240.overlay are present?
They should probably be merged in that case, in order to follow the principle here: https://docs.zephyrproject.org/latest/guides/dts/howtos.html#set-devicetree-overlays

@tejlmand

you can already today achieve what you want manually, by using -DDTC_OVERLAY_FILE=<my-file>.overlay

This is true, but when using -DSHIELD=st7789v_waveshare_240x240, some shield-specific necessary project configurations are also pulled into the project. My current workaround is to use -DDTC_OVERLAY_FILE=st7789v_waveshare_240x240 and extract all the applicable configurations from the defconfig file of the shield.

If user specifies two shields, like: -DSHIELD="st7789v_waveshare_240x240;shield2", should shields/nrf9160dk_nrf9160ns_st7789v_waveshare_240x240.overlay still be applied ?
Probably it should, but would like more opinions on this.

I think it should!

we already use _ for separating revision and build type, and this adds one more layer.

When looking at the ssd1306 shield, it seems to have a folder for board overlays. Not sure how these are used, but to me, it would make sense to have these "glue" overlay files here. If we were to build for the st7789v_waveshare_240x240, perhaps the overlay order could be: device tree → shields/st7789v_generic/boards/nrf9160dk_nrf9160ns.overlay → shields/st7789v_generic/st7789v_waveshare_240x240.overlay → nrf9160dk_nrf9160ns.overlay

perhaps the overlay order could be: device tree → shields/st7789v_generic/boards/nrf9160dk_nrf9160ns.overlay → shields/st7789v_generic/st7789v_waveshare_240x240.overlay → nrf9160dk_nrf9160ns.overlay

but that is already the case today.
Have you tried to add a nrf9160dk_nrf9160ns.overlay board specific overlay to a https://github.com/zephyrproject-rtos/zephyr/tree/main/boards/shields/st7789v_generic/boards/ folder ?
The board specific overlay for the shield will be applied last, and thus allows to overrule any setting defined be the shield itself.


Version 0.0.6

[x] tests and linter show no problems (npm t)
[x] code is properly formatted (npm run fmt)
[x] description of changes is added in CHANGELOG.md

Landed in https://github.com/metarhia/noroutine/commit/2e7aa66f25ac9e77868e748a780f15f02947afa2


Cached dataset not loaded

Describe the bug
I have a large dataset (common_voice, english) where I use several map and filter functions.
Sometimes my cached datasets after specific functions are not loaded.
I always use the same arguments, same functions, no seed…
Steps to reproduce the bug
def filter_by_duration(batch):
    return (
        batch["duration"] <= 10
        and batch["duration"] >= 1
        and len(batch["target_text"]) > 5
    )

def prepare_dataset(batch):
    batch["input_values"] = processor(
        batch["speech"], sampling_rate=batch["sampling_rate"][0]
    ).input_values
    with processor.as_target_processor():
        batch["labels"] = processor(batch["target_text"]).input_ids
    return batch

train_dataset = train_dataset.filter(
    filter_by_duration,
    remove_columns=["duration"],
    num_proc=data_args.preprocessing_num_workers,
)

# PROBLEM HERE -> below function is reexecuted and cache is not loaded
train_dataset = train_dataset.map(
    prepare_dataset,
    remove_columns=train_dataset.column_names,
    batch_size=training_args.per_device_train_batch_size,
    batched=True,
    num_proc=data_args.preprocessing_num_workers,
)

# Later in script
set_caching_enabled(False)
# apply map on trained model to eval/test sets

Expected results
The cached dataset should always be reloaded.
Actual results
The function is reexecuted.
I have access to cached files cache-xxxxx.arrow.
Is there a way I can somehow load manually 2 versions and see how the hash was created for debug purposes (to know if it's an issue with dataset or function)?
Environment info

datasets version: 1.6.2
Platform: Linux-5.8.0-45-generic-x86_64-with-glibc2.29
Python version: 3.8.5
PyTorch version (GPU?): 1.8.1+cu102 (True)
Tensorflow version (GPU?): not installed (NA)
Using GPU in script?: Yes
Using distributed or parallel set-up in script?: No

It actually seems to happen all the time in above configuration:

the function filter_by_duration correctly loads cached processed dataset
the function prepare_dataset is always reexecuted

Hi ! The hash used for caching map is the fingerprint of the resulting dataset. It is computed using three things:

the old fingerprint of the dataset
the hash of the function
the hash of the other parameters passed to map

You can compute the hash of your function (or any python object) with
from datasets.fingerprint import Hasher

my_func = lambda x: x + 1
print(Hasher.hash(my_func))

If prepare_dataset is always executed, maybe this is because your processor has a different hash each time you want to execute it.

If prepare_dataset is always executed, maybe this is because your processor has a different hash each time you want to execute it.

Yes I think that was the issue.
For the hash of the function:

does it consider just the name or the actual code of the function
does it consider variables that are not passed explicitly as parameters to the functions (such as the processor here)

does it consider just the name or the actual code of the function

It looks at the name and the actual code and all variables such as recursively. It uses dill to do so, which is based on pickle.
Basically the hash is computed using the pickle bytes of your function (computed using dill to support most python objects).

does it consider variables that are not passed explicitly as parameters to the functions (such as the processor here)

Yes it does thanks to recursive pickling.

Thanks for these explanations. I'm closing the issue.


[Request] Throwing out few ideas

Been spending last few days working with omnichain to test things out and see where it goes.
I noticed few things that I feel could be added/changed, hope it's fine to do feature/change requests!
Ideas

PickXArrayItem - I feel it should accept -1 as an index value for getting last element of an array.
Chat/TextCompletion - It'd be nice to be able to use common model and settings for multiple of such nodes, perhaps new Model and/or Settings nodes?
New node for replacing substrings in text, with regex version. Parameters for limiting count of replacements done and a starting index.

Those are all good suggestions.

Negative indices aren't a thing in JS array access, so I hadn't considered it, but I should be able to modify the code for the existing nodes to support this without breaking existing workflows.
I could add an override property for each type of completion node. If I make it accept an arbitrary JSON object, it would even allow for workflows where a model modifies its own settings. I'll see if this can be added in-place without breaking existing workflows, hopefully it's fine.
Adding a replace node (+ regex version) is pretty straightforward with the settings you mentioned, so I'll definitely do that.

In regards to settings reuse (the 2nd idea), have you tried using the Module nodes? Those technically allow you to reuse settings as-is because you can use the same subchain of nodes multiple times in the main chain. I haven't added any examples yet, but you can see me using them in that silly video I recently posted (which got way more views than the not-silly videos before it).
The way modules would work in this case is that when you set up one of the completion nodes a certain way, you just connect it into a ModuleIO node and you reuse that same node (+ any others attached to it) via a Module node as many times as you want, so you don't need what you described just for this type of reuse.

Ah! Yeah module nodes seem to do what I want for 2nd idea, still learning the system :sweat_smile:
I've recently coded only in Go so that's why I was looking for easy -1 as index, if you can do that and try the others, I'd appreciate it!

The negative indices are now supported on the array picker nodes. Remembered I had to modify the labels on OpenAI nodes to make it obvious they output arrays (people were getting confused despite the visibly different color of the connection) so I bundled those changes together in another minor update.

Added TextReplace and TextReplaceRegex nodes with start index and limit options.
I'm planning on adding that override capability as a global enable-on-demand feature for each node, where the dev making the node (hence me and/or anybody using the custom nodes feature) could optionally specify an override map of Field -> PropertyName.
If this map is provided by the dev, a string input named override would automatically get added to each node in order to receive a JSON object with overrides for each field, and the execution engine would automatically use the input (if connected) and the map to correctly replace values on the node's controls.
The override map would also be used to generate more documentation that tells users what fields they can pass in via the external JSON object, whether it's LLM-generated or not.
Such an implementation would avoid 3 huge nuisances in regards to controls (settings):

Renaming labels on controls to be the same as the underlying properties to let people know what to put into the JSON object
Making people uber-confused because said properties are meant for devs and are not necessarily as descriptive as the labels
Being a target of epic spam when OmniChain gets more adoption, by people eventually pointing out that ComfyUI has the ability to convert controls into inputs

For now I have no idea whether some random technical detail that I forgot about could stop me from implementing this. Hopefully not, because the increase in workflow flexibility would be massive.
Your opinion on this would be greatly appreciated btw, it's an evolution of your idea after all.

Thanks for the replacement nodes!
As for your implementation thoughts on overrides, I don't see anything immediately popout that would be an issue :thinking: - so it looks good to me :+1:

The override feature is live. I think that's it for now, so I'm probably closing this thread tomorrow.
Everything went according to plan, just had to add a separate helper method in the execution context that nodes receive because touching the old one would break compatibility for any custom nodes people have built for themselves and I don't want to be that guy.
Had to make another update anyway thanks to CodeMirror (the editor in the large modal popup when you click the expand button on text nodes) somehow disabling the keyboard on Chromium-based browsers even after it was closed. Now using react-simple-code-editor.
Dang, who knows how many people got turned off the app because of CodeMirror, right when I'm looking for investment.
I wonder who the lead dev of CodeMirror is.
"I just wanna talk to him."


[rotations] add test for idempotent rotations

@rickwierenga this fails for ResourceStack curious how you want to handle that

why does it fail

my guess is that because ResourceStack doesn't use the automatic location calculation in assign_child_resources that PlateCarrierSite does. Wondering if this should be added to a parent class somewhere instead of playing whack a mole.

yes, i guess in that case we should also automatically find the best location of the plate, similar to what we do with CarrierSite

bumping this


Change "invite coworkers to this Zulip" to something more appropriate

Yeah, it'd be great to change this to something like "Invite users to this Zulip organization"


test: Script updates for flaky tests

Description

This PR includes cypress scripts fixes for stable CI runs - fixing flaky fixes & splitting long Postgres CRUD & MySQL specs into smaller logical specs

Type of change

Script fixes

How Has This Been Tested?

CI runs

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes

/ok-to-test sha=a7c6b63

/ok-to-test sha=50ca2f7

/ok-to-test sha=3c87058

/ok-to-test sha=6839416

/ok-to-test sha=74d300a

/ok-to-test sha=8da0880

/ok-to-test sha=ceaf9c2

/ok-to-test sha=09afd23

/ok-to-test sha=64aefdf

/ok-to-test sha=e59259b

/ok-to-test sha=8c38c64


add license file?

I'll put MIT, you will be able to do almost whatever with the code as long as you mention the provenance.


Improve card sizing and add more cards

Current Behaviour:  In the Blogs & Articles page we have used Bootstrap cards. The size of each card is not the same.
You can Refer: https://skpandey885.github.io/HealhUb/pages/cards.html

Expected Behaviour: Make all the cards of the same size and add more cards(with different article links and photos).

I want to work on this issue

Hi, I am interested in working on this issue under JWOC.

Interested in working on this issue under JWOC

I want to work on this issue under JWOC


Nested Drawer drag close also closes the parent drawer.

Describe the bug
When we use nested drawer and we use drag close on nested drawer it also closes parent drawer.
Issue: https://stackblitz.com/edit/vaul-svelte-nested-drawers?file=src%2Froutes%2F%2Bpage.svelte - Svelte Version
https://github.com/user-attachments/assets/06ece19c-a3bd-4f0e-9311-96fa271178b0
Expected Behaviour: https://codesandbox.io/p/devbox/drawer-non-dismissable-forked-5z2r3j?file=%2Fapp%2Fmy-drawer.tsx - React version
https://github.com/user-attachments/assets/e7e8df98-a7a5-4f08-b340-d10f4371c62b
Reproduction
Use https://www.vaul-svelte.com/examples - nested drawer here and you will see when closing nested drawer by drag also closes parent drawer.
Logs
No response
System Info
System:
    OS: macOS 14.3
    CPU: (10) arm64 Apple M2 Pro
    Memory: 81.92 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.11.0 - ~/.nvm/versions/node/v20.11.0/bin/node
    npm: 10.4.0 - ~/.nvm/versions/node/v20.11.0/bin/npm
    pnpm: 9.4.0 - ~/Library/pnpm/pnpm
    bun: 1.1.7 - ~/.bun/bin/bun
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 126.0.6478.183
    Safari: 17.3
    Safari Technology Preview: 17.4
  npmPackages:
    @sveltejs/kit: ^2.5.18 => 2.5.18
    bits-ui: ^0.21.12 => 0.21.12
    svelte: ^4.2.18 => 4.2.18
    typescript: ^5.5.4 => 5.5.4
    vaul-svelte: ^0.3.2 => 0.3.2

- Tested on chrome only.

Severity
blocking all usage of vaul-svelte

I too found this behavior odd. Awesome component btw!


Various minor tweaks v7 features forward.

https://github.com/apache/incubator-kie-kogito-runtimes/pull/3556


[Node.js] Memory leak

There appears to be a memory leak when using this library with Node.js. When a Service object is instantiated (I believe this is also creating a "Channel"), memory usage goes up around 1 MB. Even if we don't retain a reference to the created service object or make a call on that channel, the memory doesn't appear to be collected-- confirmed using Passenger and monitoring the memory usage after up to 20 minutes, as well as by forcing garbage collection.
See the original report on gcloud-node by @pierre-b (gRPC-related conversation starts here).

I don't think this is an actual memory leak. I set up a loop that creates these Client objects and then discards the reference. If some of the memory involved in the construction of those objects was leaking, I would expect unbounded memory usage growth, but top indicates that memory usage actually quickly levels off. I actually see the resident memory grow slowly and then drop when garbage is collected, all within fixed bounds, which is exactly what I would expect to see if everything is getting cleaned up correctly.
What you are seeing may simply be an artifact of the C or C++ level memory allocator.

Sounds good to me. I haven't been able to reproduce outside of the sample repo.
@pierre-b, I'll close this issue, but if you can track this down in a grpc-only environment, feel free to reply and we'll keep working on it. And thank you for the report to begin with. It led to some great discussions!

@stephenplusplus thanks for your time!


add steph to member list

Added name, links and photo to site

Looks good! :shipit:

Yay you are live: codehbsclub.com

YAY! Thank you!!!


include steering message

this would improve the accuracy of the x-y position (currently we only drive in x)
https://github.com/AutoRally/autorally/wiki/Wheel-Odometry
examples:
https://github.com/AutoRally/autorally/blob/master/autorally_core/src/WheelOdometry/wheel_odometry.cpp
https://github.com/stevendaniluk/ghost/blob/master/src/odometry_integration.cpp

calculate theta based on differences in wheel speeds


Issue Generating Name for Windows VM

Describe the bug
When trying to generate a name for a Windows VM it always generates a name for a Linux VM so the delimiter is present when it shouldn't be in the name.  See attached pic.
To Reproduce
Steps to reproduce the behavior:

Go to Generated
Select Compute for category
Select Compute/virtualMachines - Windows (vm)
Fill out all fields
Click Generate

Expected behavior
When you generate a name for a Windows VM it should generate a name for a Windows (not Linux) VM.
Screenshots
If applicable, add screenshots to help explain your problem.
Installation Method
Docker Container
Additional context
Version 3.02

Hi @rich872,
Thank you for your feedback!  This issue is due to an issue with the auto-complete filed on the Generate page.  This will be resolved in our next release.

Bryan

Thanks Bryan.  Any idea when next release is due to come out?

V3.1.0 will most likely release next week some time. I have implemented the changes (along with other enhancements) but am still testing.  If you like, you can view the new functionality in the Pre-Lease version here:
v3.1.0 Pre-Release

Bryan

Fantastic, I'll check it out.  BTW - great work on this tool.  We are just starting to build out our Azure space to begin moving workloads and have been struggling with naming conventions.  We found this tool and it has made things so much simpler for us.  Excellent work!  :)

Thank you for the feedback! We love hearing that the tool is helping, but also we want to implement improvements where we can.
If you had a moment, we always appreciate feedback here:
Feedback Form

This has been resolved in V3.1.0, which is now released!
-Bryan


[FEATURE] MP3 playment?

Right now the music player only supports WAV, even though it lets users select MP3s (my fault. oops.)
MP3s should be playable or just be removed from the usable list. Would be nice to play them though
Assigned brad - he made the wav player and i have no clue how it works

It might be better just to remove the playback feature, it's not like media player exists or anything

completely removing it would be a better idea

@prometheus1337 the most recent update makes it useful by making it that the louder the audio is the faster the triangles move so i'd prefer to keep it and work on it to make it better


mfa/forget identity button should be easier to find

when there's a single service (or a handful) it's nbd. forget identity is 'right there' (though i assert it's kinda ugly/offputting when you have NO services (as shown)

If you have MANY services.. it's way at the bottom of your list. I don't think it should be way down there. you can see i had to scroll a long way to find the 'forget' button. same for 'mfa'

I think this is suppose to stick to the bottom. Its a bug