ARMA workshop

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
same changes from #1017
Closes #1208
What are the significant changes in functionality due to this change request?

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[x] 1. Review all computer code.
[x] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[x] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[x] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[x] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[x] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[x] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[x] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[x] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

Job Test linux on 048cae4 : invalidated by @alfoa
try to invalidate?

PR has been reviewed in  #1017
Checklist passed...
PR can be merged...


bug: incorrect stack order in PR

When adding a commit to the beginning of the stack it shows up in the correct order in the CLI, but not in the PR description update (add tmp2 file is the beginning of the stack but was created later).

Interestingly it worked on the 2nd git spr update after I added a 4th commit to the end.

thank you for reporting, looks like the pr stack description doesn't handle reordering of commits correctly.


Unable to Beautify Ruby files

Description
The results of beautification are not what I expect for Ruby files:
Although I can see the beautification pop up appear, when I run the Atom Beautify: Beautify Editor command, the contents of the file do not change.
Also, I can beautify other files (e.g. HTML) but can't get it to work with Ruby files (using Rubocop).
Expected Results
The beautified code should have looked like:
  def start_all(agents_file)
    Log.info('Starting agents...') # <- this line should have been indented like this!
    ...
  end

Steps to Reproduce

Add code to Atom editor
Run command Atom Beautify: Beautify Editor
This beautified code does not look right!

Debug
Here is a link to the debug.md Gist: https://gist.github.com/eddbb0dedd19fb715448986633382ca0
Checklist

[x] I have tried uninstalling and reinstalling Atom Beautify to ensure it installed properly
[x] I have reloaded (or restarted) Atom to ensure it is not a caching issue
[x] Searched for existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues
so I know this is not a duplicate issue
[x] Generated debugging information and added link for debug.md Gist to this issue

Managed to solve this. The problem was that I was trying to use the rubocop gem that I had installed via rvm.
In the end, I just installed rubocop on the system ruby:
gem install rubocop```

Then changed the Rubocop path in atom-beautify back to the default (i.e. rubocop).


useQuery and useParams composables

Describe the feature
useQuery should return all route queries.
useParams should all return all route params.
Currently we have to first call the useRoute composable before getting the route queries and or route params.
Additional information

[X] Would you be willing to help implement this feature?
[ ] Could this feature be implemented as a module?

Final checks

[X] Read the contribution guide.
[X] Check existing discussions and issues.

I believe a reproduction isn't needed here, as it is a feature request.
To summarise, the author proposes the following shortand methods for accessing:

route's query: useRoute().query -> useQuery();
route's params: useRoute().params -> useParams().

If we decided to go ahead with the implementation, I would vote for more precise naming: useRouteQuery and useRouteParams.

@manniL @DamianGlowala  useRouteQuery and useRouteParams sounds better. Also this is a feature request. I've worked on this locally and will push if given the go ahead

I believe a reproduction isn't needed here, as it is a feature request.

My bad, misunderstood that!
I personally don't mind adding the helper composables, though the same could be implemented in the user land already today.

I believe a reproduction isn't needed here, as it is a feature request.

My bad, misunderstood that!
I personally don't mind adding the helper composables, though the same could be implemented in the user land already today.

hi, what's the user land?

hi, what's the user land?

As in "your own project". You could add these composables in the composables folder already to avoid writing useRoute().query etc. all the time ☺

I'm very interested in contributing to nuxt.  I'll be going through the issues and try fixing some. I'd appreciate any guidance too.

~> https://github.com/nuxt/nuxt/pull/24369#issuecomment-1820677093


[ListItemButton] Respect LinkComponent

[x] I have followed (at least) the PR section of the contributing guide.

Fixes #34028

@hbjORbj @siriwatknp did you have a chance to check this PR?
code change is really small and supplied with an explanatory comment. I've also added tests that were failing before the fix was applied.
looking forward for your feedback

Is there something blocking this? Anything I can do to help?

Hey, just a random bloke with options. Just ran into this myself and investigated. Came up with this exact solution without influence from this issue. Solution seems solid imo :shrug:


console.log output

console.log is somewhat hard to track.
We should provide a way to obtain the console.log output even in published apps.

This has been addressed by the '--print-app-output' switch in the CLI.

The Xcode can be used to obtain console logs of deployed app.


Separating into 2 libs

I think you should separate Dwifft.swift and Dwifft+UIKit.swift, because, as you said, maybe only Dwifft+UIKit.swift is actually useful for people. By doing that, I'd be easier to explain what the it does.
Dwifft is very useful and neat, but it has one major problem: It took me a whole paragraph to understand what could be explained with

Automatic updates of UITableView with animations in Swift :large_orange_diamond:

What are your thoughts?

I'm not going to do this, sorry - the UIKit additions depend entirely on Dwifft itself, and I don't want to maintain a second library so that people can avoid reading a paragraph of text.

Hey, no problem! What about adding that phrase as an easier way to explain Dwift+UIKit?
Also, lemme know if you need any help. Just open the issues and I'll do my best! :smiley:

I'd like to re-open this issue with the suggestion that the UIKit stuff is put into subspecs. That is really easy to do, and then people can easily include only the parts of the code they need.

Alright, no worries. Its your library, so your choice. I really want to just use Dwifft without including the UIKit extension, so it's more complicated for me, in that I now have to manually copy over your file, and then any improvements or bug fixes, I won't get.
One last thought. When using subspecs, by default, CocoaPods includes all of the subspecs.
So pod 'Dwifft' would automatically include the UIKit extension. In order to exclude it, you would need to explicitly make your Podfile read pod 'Dwifft/Core'.
Not trying to be pushy, just wanted to give you some insight into things you may not have considered.

@jack-stripe: To lend some additional context, this is also how we built the Tumblr SDK: I’m sure 99% of users just pod install TMTumblrSDK and grab the whole thing, but for those who are more particular about what they’re including, we allow them to specify more specific subspaces: https://github.com/tumblr/TMTumblrSDK/blob/master/TMTumblrSDK.podspec.json#L29
No opinion from me – I use Dwifft’s UIKit extensions – but echoing @AnthonyMDev that this won’t actually add an extra step except for those who explicitly want to go out of their way to only pull in a subset.


[222] Incorrect Grammar in Status Field

Title:
Incorrect Grammar in Status Field
Description:
'an status' is wrong, should be 'a status'
Summary:
Customer is requesting a correction in the grammar of the status field from 'an status' to 'a status'.
Info
Reported by: Guest<EMAIL_ADDRESS>- Open in Gleap
Location: KG
Type: 🚨 BUG
🔗 Open share link
🌍 Open ticket in Gleap
Metadata
browserName: Chrome(<IP_ADDRESS>)
userAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
browser: Chrome
systemName: Windows
sessionDuration: 55512
devicePixelRatio: 1.1458333730697632
screenWidth: 1676
screenHeight: 943
innerWidth: 1352
innerHeight: 771
currentUrl: https://docs.near.org/concepts/protocol/transaction-execution
language: en-US
sdkVersion: 10.1.0
sdkType: javascript
environment: prod

Internal note 📝: Customer is requesting a correction in the grammar of the status field from 'an status' to 'a status'.


User mandatory in DSN

v1.1.8
works: 'https://zuzu@localhost/LoggerSite/logging/0'
doesn't: 'https://localhost/LoggerSite/logging/0'
and I need the second.
Thank you

This was resolved with https://github.com/getsentry/raven-js/pull/361


MCDW crashes again with Beta 23

Describe the bug
Crashes on start up
Steps to reproduce
Before this connector update
it didnt crash, looks like the same problem that was fixed in Beta 21 I believe.
https://github.com/Sinytra/Connector/actions/runs/6618635008
This is the build you sent to fix it at the time
Logs
https://pastebin.com/hLUDVFMq
Additional context
No response

This also happened with Minecraft Dungeons Armor

"mcdw, mcda" conflicts with Clumps in "beta22"

"mcdw, mcda" conflicts with Clumps in "beta22"

you need clumps fabric


add Full HTML Layout Example to Bamboo.Phoenix docs

As discussed in #110. Feedback welcome. 🍻

@jerodsanto This looks really good! I have a couple suggestions, but won't have much time until Friday. I'll leave some comments then. Thank you!

@jerodsanto This looks awesome. Left just a couple comments and then this is ready to go!

@paulcsmith  cool, cool. Changes added. 💥

Thank you @jerodsanto!

🎉


add home button to every page

Proposed changes
Add missing home button to non home pages

Thanks Jeremie. I can see the Home nav on other tabs now.
The search box doesn't load when I click on other tabs. It works fine in prod. Sometimes the browser needs to be refreshed for the search box appear. Not sure if it's just me?

@usha-mandya I have the same weird behavior about the search bar. I'll try to figure out why. Do you have any scenario in mind where it should not appear?

@jdrouet Just noticed that this issue doesn't occur in https://github.com/docker/docker.github.io/pull/10815.

Also make sure to check on mobile to see if the top-navigation doesn't wrap and cover the left-hand nav (I recall we ran into issues if there were too many items in the top-nav)

@thaJeztah it seems ok on a simulated iphone. On my android, it works fine also.

It seems like this search input missing is due to a race condition. I'll try to fix it in another PR.

OK, that's good to know, Jeremie. Can I merge this one then?

LGTM (although it's my own PR...)

@jdrouet if the issue with the search is only in this PR, I think we should ideally fix it here.
Thinking now: could it be that a script looks for the "Guides" link specifically, and now that it moved, it no longer finds it?

It's not only in this PR. It does the same on master.
Sometimes, this https://github.com/docker/docker.github.io/blob/master/js/docs.js#L347 is called after this https://github.com/docker/docker.github.io/blob/master/js/archive.js#L52 being called and that's why it doesn't display. Now, it's time to rewrite this to be async.

It's not only in this PR. It does the same on master.
Sometimes, this https://github.com/docker/docker.github.io/blob/master/js/docs.js#L347 is called after this https://github.com/docker/docker.github.io/blob/master/js/archive.js#L52 being called and that's why it doesn't display. Now, it's time to rewrite this to be async.

ugh 😞
Perhaps we should put a css archive class on the main <html> or <body> element that we can remove, instead of trying to un-hide each element.
(we should probably also just exclude the whole search if we're doing an archive build, but that's likely more work)

Quick try on my phone;

Looks like the "samples" item wraps (but only visible if you expand the "hamburger menu").
The top-navigation is also hiding the first item in the TOC ("cli reference"), which we should fix (but not introduced in this PR)

Perhaps we should put a css archive class on the main <html> or <body> element that we can remove, instead of trying to un-hide each element.
(we should probably also just exclude the whole search if we're doing an archive build, but that's likely more work)

Is there any way we can fix the search bar for the landing page without rewriting archives.js?
@thaJeztah How does this impact the updates given that we'll be removing archives soon from our docs site?

How does this impact the updates given that we'll be removing archives soon from our docs site?

We should no longer need that code once we no longer host archives (I think it's only to add the "warning this is an archive" banner). Actually we could potentially even remove it already, if we won't be publishing the current version of the docs as an archive. (Existing archives still have that could, so would be showing the banner)
But we should keep generating (and hosting) the https://docs.docker.com/js/archives.json file for now, because that's used by the archives to detect if they are an archive.

The search bar works fine for me now. Tried on a different browser and that works fine too.

Could we merge this and fix the search bar on another PR ?


EventsPerLumi multiplier cannot be 0!!!

Fixes #8566
First the unit test, just to prove it fails where it fails

This src/ fix was already in in #8520

Thanks Alan for the quick fix.

For the record, this has been pushed in to both testbed and production cmsweb (base tag 1.1.12.pre3)


Add standard data object

This is probably just a dictionary of arrays right now.
Utilize standard names from CF to make things work automatically.

Might need to provide short names or something like it too

Provide unit capabilities as well
At least mimic iris API

This could also just be replaced wholesale by IRIS if that's deemed better.

Might even be a better idea to follow NetCDF4-Python API.

Should contain a getTimeAxis method to make it easy to get the time variable associated with an object. (This comes from the correct time variable bouncing from time to time1 on GRIB files. See unidata/unidata-python-workshop#43)

Sorry I hadn't made this yet. Should also add Axis getters for all the axis
types. Here is a list of Axes that the CDM recognizes:
Time
RunTime
Ensemble
GeoX
GeoY
GeoZ
Lat
Lon
Height
Pressure
Radial Azmuith
RadialDistance
RadialElevation
Spectral
https://github.com/Unidata/thredds/blob/144f779cc20058581c660719f768fe396cd1a9c2/cdm/src/main/java/ucar/nc2/constants/AxisType.java
On Wednesday, July 15, 2015, Ryan May<EMAIL_ADDRESS>wrote:

Should contain a getTimeAxis method to make it easy to get the time
variable associated with an object. (This comes from the correct time
variable bouncing from time to time1 on GRIB files. See
Unidata/unidata-python-workshop#43
https://github.com/Unidata/unidata-python-workshop/issues/43)
—
Reply to this email directly or view it on GitHub
https://github.com/metpy/MetPy/issues/10#issuecomment-121776226.


Retopology Horse (Horse_01) - Horse 3D Model

Retopology of hight poly Horse_01.
EXPECTED RESULT:
Finished a low poly of horse to be able to texturing.

UVs opened, new issue will be opened to follow its progress.


Catalog app, implement single choice workflow.

Is your feature request related to a problem? Please describe.

User click on single choice component.
It launches single choice screen with questionnaire, which demonstrate single choice uses.
click on navigation back key in action bar, it takes user back on landing page.

Design single choice questionnaire json.
Write api to fetch single choice questionnaire json file name.
Apply custom style at application side.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Would you like to work on the issue?
Please state if this issue should be assigned to you or who you think could help to solve this issue.

Datacapture, custom style does not get applied to the single choice options.
https://github.com/google/android-fhir/issues/1007

Lebal


[INFRA] Get Sub-Domain for HireFlow

Development Task 🔧
What? ❓
Speak to Tarik, and get hireflow.ubclaunchpad.com to redirect to hosted web app.

Update - Tarik will reach out tomorrow or next day to finish this up.


how to get binary data of image that we have to send in cognitive service api

Hi team
Thanks in advance
microsofComputerVision.orcImage({
"Ocp-Apim-Subscription-Key": "2b4043ad15ef444eb4b41182c6ac0dab",
"request-origin":"westcentralus",
// "content-type": "application/json",
//"url": "http://cdn.quotesgram.com/img/81/49/660235022-Random-Funny-Quotes-.jpg",
"content-type": "application/octet-stream",
"body": data,
// "url": attachment.contentUrl,
"language": "en",
"detect-orientation": true
}).then((result)=>{
session.send(result);
});
I am able to get the response data from image as text
when i pass the normal url
but when i am passing the url as msg.attachments[0].contentUrl getting error image cant be downloaded
so can you please help me in getting the binary data that i can send in body to have the text

duplicate of #3188

@nwhitmont , please add the solution to the problem instead of only marking the defect as duplicate.
I checked 3188,3225 and there is no solution on the threads.


support schema evolution for delta lake

Describe the feature
We are using DBT+Spark on Delta for incremental load. As we are getting data from various sources, one of the key features is to be able to support schema evolution. Delta lake does support it with the merge command as documented here
Can this be supported by DBT?
Describe alternatives you've considered
the current alternative is refresh all, this has too issues:

very time consuming
As part of ingesting the data, we keep the histories and some of our source data don't maintain history. the refresh all will lose this historic records

Who will this benefit?
I saw another issue #124 that were submitted for the similar case, but it was closed for some reason. supporting schema evolution with Delta will be extremely helpful for anyone who is using Delta incremental strategy.

I think it was implemented by means of CREATE OR REPLACE TABLE.
See this PR: https://github.com/fishtown-analytics/dbt-spark/pull/125

Hi @laiyuanliu
Thanks for opening this ticket. It depends on what your aim is. The MERGE INTO statement is currently supported. Please check from 17 minutes onwards: https://www.youtube.com/watch?v=zoHoIGE6tPc&t=527s
Schema evolution for full refreshes (i.e. non-incremental mode) is implemented in #125 in an atomic way. So we don't first drop the table, and then recreate the table. Otherwise, the table will be unavailable while recreating the table. However, when using the MERGE INTO statements, we're operating in incremental mode and this isn't supported yet. I see two options to fix this:

Detect a schema change, and then do the appropriate ALTER TABLE statements to get the schema in sync. This can be done using ADD COLUMNS and DROP COLUMNS. https://docs.databricks.com/spark/latest/spark-sql/language-manual/sql-ref-syntax-ddl-alter-table.html#add-columns However, this won't fill the historical values.
Detect a schema change. Drop the whole table, and rebuild it from the history. This will then be the same command, in the case the table didn't exist at all.

I would lean to the second one, this is more predictable and fits in nicely with the ELT way of thinking.

I have added the pre_hook but in incremental mode, new columns are not being added.


Publish a new version and bring it back to the package set

I think this does not need any refactoring. Current master builds well with package set 43.3.0.

See https://github.com/purescript/registry-dev/issues/662
I think we are waiting on a new ecosystem import to get correct dependencies.


Cannot read properties of null (reading 'removeEventListener')     at index.js:1

index.js:1 Uncaught TypeError: Cannot read properties of null (reading 'removeEventListener')
at index.js:1
at HTMLUnknownElement.callCallback (react-dom.development.js:3945)
at Object.invokeGuardedCallbackDev (react-dom.development.js:3994)
at invokeGuardedCallback (react-dom.development.js:4056)
at flushPassiveEffectsImpl (react-dom.development.js:23543)
at unstable_runWithPriority (scheduler.development.js:468)
at runWithPriority$1 (react-dom.development.js:11276)
at flushPassiveEffects (react-dom.development.js:23447)
at performSyncWorkOnRoot (react-dom.development.js:22269)
at react-dom.development.js:11327
at unstable_runWithPriority (scheduler.development.js:468)
at runWithPriority$1 (react-dom.development.js:11276)
at flushSyncCallbackQueueImpl (react-dom.development.js:11322)
at flushSyncCallbackQueue (react-dom.development.js:11309)
at scheduleUpdateOnFiber (react-dom.development.js:21893)
at dispatchAction (react-dom.development.js:16139)
at Header.jsx:21
this error occurs when the date picker component is removed from Dom

index.js:1 Uncaught TypeError: Cannot read properties of null (reading 'removeEventListener')
at index.js:1
at HTMLUnknownElement.callCallback (react-dom.development.js:3945)
at Object.invokeGuardedCallbackDev (react-dom.development.js:3994)
at invokeGuardedCallback (react-dom.development.js:4056)
at flushPassiveEffectsImpl (react-dom.development.js:23543)
at unstable_runWithPriority (scheduler.development.js:468)
at runWithPriority$1 (react-dom.development.js:11276)
at flushPassiveEffects (react-dom.development.js:23447)
at performSyncWorkOnRoot (react-dom.development.js:22269)
at react-dom.development.js:11327
at unstable_runWithPriority (scheduler.development.js:468)
at runWithPriority$1 (react-dom.development.js:11276)
at flushSyncCallbackQueueImpl (react-dom.development.js:11322)
at flushSyncCallbackQueue (react-dom.development.js:11309)
at scheduleUpdateOnFiber (react-dom.development.js:21893)
at dispatchAction (react-dom.development.js:16139)
at Header.jsx:21
this error occurs when the date picker component is removed from Dom

check here:
https://github.com/Kiarash-Z/react-modern-calendar-datepicker/issues/286#issuecomment-927020181

Any update on this bug? I am having the same error.

Is this package abandoned?

Hi @amir04lm26 , Your forked version is working smoothly and seems there is no bugs. do you have any plans on releasing a npm package? at least until main team fix occurred bugs and release new version of their own package

Hi Rezaabdi78, As you had requested, I've published the forked version that fixes the compatibility with react v17 and, here's the package: https://www.npmjs.com/package/@amir04lm26/react-modern-calendar-date-picker

God bless you 👍 ❤


Stabilize system tests on main

Issue-Id: #92
Definition of Done
The PR shall be merged only if all items mentioned in
CONTRIBUTING.md
have been followed. In case an item is not applicable as described, please provide a short explanation in the description.

[ ] All steps in CONTRIBUTING.md have been handled

I created a clean branch and a new PR for the changes to not get main messed up with the experimenting here:
https://github.com/eclipse-ankaios/ankaios/pull/96


ereader demo - js execution exceeded timeout

Hi,
How to reproduce
iPhone 4, iOS 4.3.5, WiFi
open http://cubiq.org/dropbox/SwipeView/demo/ereader/ and you get an error:
Javascript Error on Line 59
_http://cubiq.org/dropbox/SwipeView/demo/ereader/_
Javascript execution exceeded timeout.

should be fixed, I also added a progress bar


Unable to open webpage

0/10

I took down the app a few weeks ago since it was costing ~5 USD a month for no reason. Check out another one of my sites though like https://clickcalendars.ca/


o._shape = TensorShape(new_shape) caused error in inception_score.py

the tensorflow 1.8.0 seems to have disabled this operation, it suggest to use the set_shape function:
o._shape = tf.TensorShape(new_shape)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 434, in _shape
    "Tensor._shape cannot be assigned, use Tensor.set_shape instead.")
ValueError: Tensor._shape cannot be assigned, use Tensor.set_shape instead

but when I use the set_shape function I found the tensor shape is not changed.
I print this:
 o.set_shape(tf.TensorShape(new_shape))
 print(o.name, shape, new_shape, o.get_shape())

the output shows the tensor shape is not changed by this function:
ExpandDims/dim:0 [1] [None] (1,)
DecodeJpeg/contents:0 [] [] ()
DecodeJpeg:0 [None, None, 3] [None, None, 3] (?, ?, 3)
Cast:0 [None, None, 3] [None, None, 3] (?, ?, 3)
ExpandDims:0 [1, None, None, 3] [None, None, None, 3] (1, ?, ?, 3)
ResizeBilinear:0 [1, 299, 299, 3] [None, 299, 299, 3] (1, 299, 299, 3)

refer to this link  https://stackoverflow.com/questions/49525530/gan-training-cannot-calculate-inception-score-because-of-tensor-set-shape-error


Add a Friend System

Is your feature request related to a problem? Please describe.
Users want to be able to interact with one another but they are not able to if they cannot friend each other. This system would also let us build a chat system and other features later on.
Describe the solution you'd like
We could add in a friend system to the server by using the sequelize relationship system. That would let people friend each other and accept friend requests.
Describe alternatives you've considered
I've considered doing a system like the one that Github and Instagram has where you can follow people and then they can follow you back. I think that this system would be much simpler to implement.
Additional context
I will link some other issues here on what we need to do to accomplish this.

[x] #49
[x] #46
[x] #47
[x] #48
[x] #50
[x] #73

I think this is now implemented.
Massiles.


Implement fix for VISA USB read in "Wait Second" mode.

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?

Fixes the VISA Read when using a VISA USB device.
Currently, we are using the VISA attribute Number of Bytes at Serial Port when using the Wait Second mode. This attribute is not applicable to USB or USBTMC devices, and using the Wait Second mode for a VISA USB device will not read anything.
Implemented a check for the Interface Type used, and if it's USB, we give enough buffer for the VISA Read to get all the data available at the port.

Why should this Pull Request be merged?

In order for the Instrument Addon to be able to read messages in Wait Second mode when using VISA USB devices.

What testing has been done?

Run the Protocols build on the dev machine.
Run the applicable Unit Tests on the dev machine.
Did not test with a VISA USB device as I am missing the setup.

Mentioning Issue ni/niveristand-instrument-addon-custom-device#190


[WIP] HAC-2154:  Specify pathPrefix and wsPrefix in workspace API calls

Closing since this project is on hold.


La réunion doesn't exist in staging

/zenika

https://github.com/Zenika/skillZ/pull/294 PR associated ?

#294 PR associated ?

This PR removes the translation and uses the name directly from the Database.
We have to add La Réunion in database. How do we proceed, with seeder or with the admin console ?

Yes ! Indeed the issue don't come from the PR. La réunion was on seeds folder. I don't know why the agency migration has no effect in staging. I'm trying with admin console. let's see :hankey:


Updated NuGet package

I see some changes have been made since the most recent publication of the package on NuGet. Are you going to deploy those as an updated package?

There are problems with this library as it stands today. Notably, many of the built-in tests (ported from Java) are failing. This is due to the library lagging behind the original Java source, while the metadata keeps getting updated. (For instance, short numbers have been moved to a separate .XML file, which is being ignored by the C# code.)

Yes, I'm trying to set some time to update both the issues and the package soon.

@erezak: hey thanks!! It would be really nice if we could help out with this.
(ps. my fork of your repo restructures the usage so it doesn't rely on exceptions as part of the library's flow. This saves a bunch of performance hits, which are crucial to using the lib on Windows Phone.)

@elijajah - I'd love to use this code without the exception based flow!

@erezak it would be nice to have and update of nuget package. Thanks in advance.


[test-triage] rom_e2e_smoke

Hierarchy of regression failure
Chip Level
Failure Description
UVM_FATAL @ * us: (sw_logger_if.sv:157) [tb.u_sim_sram.u_sim_sram_if.u_sw_logger_if.parse_sw_log_file.unnamed$$*.unnamed$$*] Failed to open sw log db file rom_e2e_smoke_prog_sim_dv.logs.txt. has 3 failures:
Test rom_e2e_smoke has 3 failures.
0.rom_e2e_smoke.3118367884
Line 487, in log /container/opentitan-public/scratch/os_regression/chip_earlgrey_asic-sim-vcs/0.rom_e2e_smoke/latest/run.log
UVM_FATAL @  11.393316 us: (sw_logger_if.sv:157) [tb.u_sim_sram.u_sim_sram_if.u_sw_logger_if.parse_sw_log_file.unnamed$$_0.unnamed$$_1] Failed to open sw log db file rom_e2e_smoke_prog_sim_dv.logs.txt.
UVM_INFO @  11.393316 us: (uvm_report_catcher.svh:705) [UVM/REPORT/CATCHER]
--- UVM Report catcher Summary ---
1.rom_e2e_smoke.1132519800
Line 488, in log /container/opentitan-public/scratch/os_regression/chip_earlgrey_asic-sim-vcs/1.rom_e2e_smoke/latest/run.log
UVM_FATAL @  10.995501 us: (sw_logger_if.sv:157) [tb.u_sim_sram.u_sim_sram_if.u_sw_logger_if.parse_sw_log_file.unnamed$$_0.unnamed$$_1] Failed to open sw log db file rom_e2e_smoke_prog_sim_dv.logs.txt.
UVM_INFO @  10.995501 us: (uvm_report_catcher.svh:705) [UVM/REPORT/CATCHER]
--- UVM Report catcher Summary ---
Steps to Reproduce

Commit hash where failure was observed: 144e5112d
dvsim invocation command to reproduce the failure, inclusive of build and run seeds:
./util/dvsim/dvsim.py hw/top_earlgrey/dv/chip_sim_cfg.hjson -i rom_e2e_smoke --fixed-seed<PHONE_NUMBER> --build-seed<PHONE_NUMBER> --waves -v h

Tests with similar or related failures
No response

I have identified the issue, and created a fix in #15901.

Hmm this seems to be passing when I run it locally. Perhaps the dashboard is not reflecting the fix. Let's keep this open and see what the dashboard shows tomorrow.

dashboard still shows this failing, I will dig in.

I have confirmed, this is an internal infrastructure issue. I have fix out for this now. If it merges tonight, then tomorrow's regressions should show the results. Otherwise, we will see the results on Thursday.

passing now


Weird gem issues

Opening this for tracking. Some time between 14.0.89 and 14.0.99 my integration tests for poise-ruby broke. The culprit appears to be something funky with gem_package causing bundler to show as up-to-date even if it isn't:
...
* ruby_gem[bundler] action upgrade (up to date)
...
kitchen@2a2d85915112:~$ /usr/bin/gem2.3 list

*** LOCAL GEMS ***

bigdecimal (1.2.8)
did_you_mean (1.0.0)
io-console (0.4.5)
json (1.8.3)
minitest (5.8.4)
net-telnet (0.1.1)
power_assert (0.2.7)
psych (2.0.17)
rake (10.5.0)
rdoc (4.2.1)
test-unit (3.1.7)

We should track this down :)

Definitely something going wrong here:
# 14.0.114 (latest build)
irb(main):014:0> Chef::Provider::Package::Rubygems::AlternateGemEnvironment.new('/usr/bin/gem2.3').gem_specification.find_all_by_name('bundler', '>= 0').first.loaded_from
=> "/opt/chef/embedded/lib/ruby/gems/2.4.0/specifications/default/bundler-1.16.1.gemspec"

vs.
# 14.0.89
irb(main):003:0> Chef::Provider::Package::Rubygems::AlternateGemEnvironment.new('/usr/bin/gem2.3').gem_specification.find_all_by_name('bundler', '>= 0')
=> []

https://github.com/chef/chef/compare/v14.0.89...v14.0.99 shows all the changes in the target range, which almost certainly means its because of either the included bundler rev, the included ruygems, or appbundler.

The core of the problem:
kitchen@2a2d85915112:~$ ls /opt/chef/embedded/lib/ruby/gems/2.4.0/specifications/default/
bigdecimal-1.3.0.gemspec  bundler-1.16.1.gemspec  io-console-0.4.6.gemspec  json-2.0.4.gemspec  openssl-2.0.5.gemspec  psych-2.2.2.gemspec  rdoc-5.0.0.gemspec

vs.
root@6c7a3cf08e72:/# ls /opt/chef/embedded/lib/ruby/gems/2.4.0/specifications/default/
bigdecimal-1.3.0.gemspec  io-console-0.4.6.gemspec  json-2.0.4.gemspec  openssl-2.0.5.gemspec  psych-2.2.2.gemspec  rdoc-5.0.0.gemspec

Given our current code, any gem in the default set cannot be usefully managed via gem_package because AlternateGemEnvironment will always see Chef's version of it. We can probably filter on spec.loaded_from to make this work though.

Another possible fix would be in the build scripts, erasing the contents of that default/ folder but I have no idea what the consequences of that would be.

Related discussion here i think: https://github.com/chef/chef/issues/3363


The written delta file has corrupted structure

Environment
Delta-rs version:0.6.3
Binding: python 3.9.13
Environment:

Cloud provider: Azure
OS: Windows
Other:

Bug
What happened:
The dataframe was written in invalid format to AWS S3. The _delta_log has the next structure:

What you expected to happen:
How to reproduce it:
More details:

same issue with gcp !!!

What do you mean by corrupted? Does it cause another reader to error?
This looks like a temporary file written in a failed transaction. We create temporary files and them atomically rename them to create the final commit. It should be removed by a vacuum (but let me know if it doesn’t—-that would be a bug.)

@wjones127  same error here, https://github.com/delta-io/delta-rs/issues/878

same error with 0.6.4,  it is the same bug reported both in aws and GCP, can you guys have a look !!!

The behavior described here is not a bug. Those temporary files are created intentionally and should not interfere with any reader.
If you do encounter a bug where a temporary file breaks something, please file an issue that includes the error you encounter.

@wjones127 yes I did already, see here https://github.com/delta-io/delta-rs/issues/878 , the json is not written into the cloud storage, delta table can not be read, it is a corrupted table !!!

@wjones127 can you please reopen this bug report the issue is not solved, the json is not written at all in the delta_log folder

@djouallah if there is already a ticket open that contains the specific error, then please ping me there, not in a separate ticket.


hoppscotch: 24.3.1-2 -> 24.3.2-1

Description of changes

Update hoppscotch 24.3.1-2 -> 24.3.2-1

Changelog: https://github.com/hoppscotch/hoppscotch/releases/tag/2024.3.2

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

1 package built:

hoppscotch

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3861

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

1 package built:

hoppscotch


[AutoPR iothub/resource-manager] [Hub Generated] Review request for Microsoft.Devices to add version preview/2019-03-22-preview

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5544

This PR has been merged into https://github.com/Azure/azure-sdk-for-js/pull/2175


iOS Interstitial ad closed returns adType banner

Hello
I have a Sencha Touch/Cordova project and I'm using this plugin. It works great on android, but there seems to be a problem with iOS interstitial ads.
I call the admob.requestInterstitialAd providing it with the publisherId being the interstitial ad unit id, and the isTesting flag set to true.
When the admob.events.onAdLoaded event is fired, I checked the type of the ad in case of interstitial, and it was interstitial, however when the admob.events.onAdClosed event is fired, the ad type in the parameters is banner.
Any help with this?
Thank you :)

Hi @hannad93, thanks for the issue.
Solved in https://github.com/appfeel/admob-google-cordova/commit/8e35414c98411ff0031a3792254a3e90a85b4c2f

Thank you!


add observation wrapper

Added Observation Embedding
Extended their Simple115V by adding :

relative pose(position) among teammates and opponents
relative pose between active player and ball
sticky actions/card/tried factor status

Looks good to me!


Update Marshal method for the fields to accept Go native types

We want to support Go's native types such as string, int, etc. when we set data to the field using Marshal method
So, the idea is to implement the following:
f := field.NewString(someSpec)

err := f.Marshal("hello")
f.Value() // "hello"

err := f.Marshal(5)
f.Value() // "5"

str := "hello"
err := f.Marshal(&str)
f.Value() // "hello"

i := 5
err := f.Marshal(&i)
f.Value() // "5"

We want to be able to use:

int, *int, string, *string for field.Numeric, field.String
[]byte, string for field.Binary, field.Hex (string should be hex and if we fail to decode it, return error)

Reference implementation is in this PR: https://github.com/moov-io/iso8583/pull/273/files

closed by #289


[Feature Request] DNS Management for domain

fossbilling should add future domain management dns for domain templete to change side client to redirect owned domain to other hosting

I also don't think that this should be in fossbilling. Adding DNS Management to any platform is.. a lot of work...
Imo either use a server control panel like hestia or in the future maybe a specific DNS hosting manager might be added as a plugin. (F.eX https://atomiadns.com/ or https://www.poweradmin.org/

I will leave this open for a couple of days to see if the OP has some reason for this being in FOSSBilling that I'm missing but if not then I am going to go ahead and close the issue.

I think it will be nice to have in FOSSBiling so that not every task will need logging to the web panel Egbe it can be quickly done in the billing.

Yeah I'm going to be completely honest.
I can understand the appeal of this, but right now with the current number of maintainers we have, the amount of work that needs to be done, and the very significant number of open feature requests, I would consider this to be a very, very low priority. Assuming we even want to implement this ourselves which we likely wont.

@John-S4  this future very helpful to management DNS from billing like whmcs have cpanel manage it plus point if fossbilling have this future like @tobsowo say this help side client quickly to set it.

In my mind a much better long term option is to improve the integration with server control panels including SSO, so that the FOSSBilling would have a link to go and manage the DNS in the server control panel.
Personally I think that there should be tight integration between billing/client-management systems, and server control panels, but there are clear lines for me about which things each one should do.

like whmcs have cpanel manage it

I haven't used WHMCS (or cPanel) for a long time, but that sounds like the DNS management is handled by cPanel rather than WHMCS, which would support my point?

Adding the WHMCS feature list for reference purpose

WHMCS still depends on a 3rd party software like Cpanel to run the DNS servers.
If somebody want to create it please go ahead but for us it is currently not a priority..

People need to realize a couple of important things about FOSSBilling:

We don't have many active contributors. Generally speaking, FOSSBilling will have 1 or 2 active contributors with temporary peaks up to 3. Right now, we are at 1 (me).
We aren't paid for this. I haven't tracked it, but I estimate I've put in anywhere between 6 and 8 months of full-time development towards FOSSBilling. That's roughly $49,978.5 - $66,638 USD of completely free development from me alone.
FOSSBilling is still considered pre-production software. There are parts of it that don't work right, system that are overly complicated or convoluted to use and maintain, and just some security features that are outright missing such as MFA support.

So implementing new features is currently a very low priority and if we are going to implement a new feature, it needs to be something that almost everyone using FOSSBilling will benefit from. Unfortunately, a handful of people finding something useful or the fact that WHMCS supports it is not enough of a reason to justify putting any priority towards something.
And that's before we even mention the fact that there's currently 90 other open feature requests (91 total).


Correct the mimeType

Description
change of the mimetype because since their implementations, there are concerns about capitalization.
Reference issue
ivf recorder was not working (never fired )

Merged with 22eea024a224d426b31b2f66ace08a4c0e7fdc13 thanks @pascal-ace :D


Clipping planes demo "Show bounding volume" does nothing for Model

Run http://localhost:8080/Apps/Sandcastle/index.html?src=3D Tiles Clipping Planes.html&label=All
Select Model from the combo box
Check Show bounding volume

The bounding volume doesn't show up.  Looking at the code, it only handles tilesets, which is fine.  But it makes the example confusing.
Why is this checkbox even here, it doesn't really have anything to do with clipping planes, right?  Perhaps we just remove the option?

Duplicate of https://github.com/CesiumGS/cesium/issues/8509


size mismatch for action_out.weight: copying a param with shape torch.Size([4, 256]) from checkpoint, the shape in current model is torch.Size([6, 256]).

Hello, when I run to the Train the High-Level Synthesizer (HLS) step, I encountered an error. The error shows that the dimensions of the imported model are inconsistent with the required ones. I suspect that it is the first step Train the Low-level There is a problem with the model parameters generated by Subtask Expert (LSE), but I don’t know how to modify it. Can you please provide some help?
The error displayed on the terminal is as shown below:
Cuda: True
Loading A3C parametets ...
No of available cores : 16
<Synchronized wrapper for c_int(0)>
Cuda: True
Cuda: True
Process SpawnProcess-2:
Traceback (most recent call last):
File "/usr/lib/python3.6/multiprocessing/process.py", line 258, in _bootstrap
self.run()
File "/usr/lib/python3.6/multiprocessing/process.py", line 93, in run
self._target(*self._args, **self._kwargs)
File "/home/lni/PycharmProjects/ReinforcementLearningPick/Multi-Subtask-DRL-for-Pick-and-Place-Task/HLS/dense_rewardDDPG_HER.py", line 64, in train
actor_network_approach.load_state_dict(model_approach)
File "/home/lni/.local/lib/python3.6/site-packages/torch/nn/modules/module.py", line 830, in load_state_dict
self.class.name, "\n\t".join(error_msgs)))
RuntimeError: Error(s) in loading state_dict for actor:
size mismatch for action_out.weight: copying a param with shape torch.Size([4, 256]) from checkpoint, the shape in current model is torch.Size([6, 256]).
size mismatch for action_out.bias: copying a param with shape torch.Size([4]) from checkpoint, the shape in current model is torch.Size([6]).
Process SpawnProcess-1:
Traceback (most recent call last):
File "/usr/lib/python3.6/multiprocessing/process.py", line 258, in _bootstrap
self.run()
File "/usr/lib/python3.6/multiprocessing/process.py", line 93, in run
self._target(*self._args, **self._kwargs)
File "/home/lni/PycharmProjects/ReinforcementLearningPick/Multi-Subtask-DRL-for-Pick-and-Place-Task/HLS/dense_rewardDDPG_HER.py", line 381, in test
actor_network_approach.load_state_dict(model_approach)
File "/home/lni/.local/lib/python3.6/site-packages/torch/nn/modules/module.py", line 830, in load_state_dict
self.class.name, "\n\t".join(error_msgs)))
RuntimeError: Error(s) in loading state_dict for actor:
size mismatch for action_out.weight: copying a param with shape torch.Size([4, 256]) from checkpoint, the shape in current model is torch.Size([6, 256]).
size mismatch for action_out.bias: copying a param with shape torch.Size([4]) from checkpoint, the shape in current model is torch.Size([6]).

Hi Lygggg,
thanks for the interestest in my work! I tried the section cloning the repo and for me worked. As you notice probably there is an error caused by a wrong import of the weights, I mean it cannot find the weights. Check if in the folder "weights" you have these files: actor_params.pt,  approach.pt, manipulate.pt and retract.pt (I attached a picture

).
If you have everything in the folder, you can run the main without any problems. Hope I helped you.

Hi Lygggg,
thanks for the interestest in my work! I tried the section cloning the repo and for me worked. The problem is caused by the fact that the basic gym version 0.10.8 used has been modified to use only 4 actions, now I have imported it into the repo so you can directly use that and there should be no problem. Thank you so much for the comment.

Thanks for you reply,i have solved the problem.
Luca Marzari @.***> 于2021年5月22日周六 下午3:29写道：

Hi Lygggg,
thanks for the interestest in my work! I tried the section cloning the
repo and for me worked. The problem is caused by the fact that the basic
gym version 0.10.8 used has been modified to use only 4 actions, now I have
imported it into the repo so you can directly use that and there should be
no problem. Thank you so much for the comment.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/LM095/Multi-Subtask-DRL-for-Pick-and-Place-Task/issues/1#issuecomment-846368571,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANGJGAI6GJPOONZGP7OEYPLTO5MPJANCNFSM45KIWNJQ
.


unable to restore from command line/scripts

Issue Title
unable to restore from command line/scripts
actual
(from either PowerShell or cmd)
C:\src\MyApp> dotnet restore --configfile "NuGet.config" --interactive
C:\Program Files\dotnet\sdk\2.2.203\NuGet.targets(119,5): error : Unable to load the service index for source https://pkgs.dev.azure.com/af4jm/_packaging/af4jm/nuget/v3/index.json. [C:\src\MyApp\MySln.sln]
C:\Program Files\dotnet\sdk\2.2.203\NuGet.targets(119,5): error :   Response status code does not indicate success: 401 (Unauthorized). [C:\src\MyApp\MySln.sln]

dotnet build results in the same error, thus rendering build scripts useless
expected
either it just works (with cached credentials), or it gives me a URL to authenticate

What do you need credentials for? Some proxy?
@dagood any thoughts?

Sounds like a question for NuGet folks. The content of the NuGet.Config, output of dotnet --info, and version of NuGet being used seem to me the necessary first steps to figure it out.

@rrelyea can you help to route it please?

To support authenticated feeds via dotnet restore or msbuild /restore use 2.1.500 or 2.2.100 sdks (or later)
Then use this project
https://github.com/Microsoft/artifacts-credprovider
Some related docs:
(https://docs.microsoft.com/en-us/Nuget/reference/extensibility/nuget-cross-platform-authentication-plugin)
Cc @alexmullans

Looks like answered, closing.
Please let me know if I missed anything.


Remove redundant type annotation lint.

Since this is a subset of always_specify_types or always_declare_return_types I think we can safely remove it.
In practice, if you violate you get doubly nagged.
[lint] Type annotate public APIs. (packages/flutter_tools/lib/src/dart/runner.dart, line 5, col 1)
[lint] Declare method return types. (packages/flutter_tools/lib/src/dart/runner.dart, line 5, col 1)

One warning is probably enough to get the message across? ;)
cc @Hixie


Add emerge-upload-action

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Update singularity

Dockerfile and singularity recipes updated

This MR is still failing due to missing mpicxx.
~ Joe G.


Rendering bug: line is partly invisible at certain zoom levels (Firefox)

In at least Firefox 29.0.1 and 30.0, this PDF is rendered incorrectly.
This picture was taken at zoom level 80%:

The empty space is supposed to be a straight line. The same bug occurs at other levels including, but not limited to, 25%, 80%, 90%, 150%, 170% and 200%. The bug does not occur at 60%, 100%, 110% and 130% (and others).
Affected browser versions: Mozilla Firefox 29.0.1, 30.0
Affected PDF.js versions: 6d330250dadcca8a7a97bafffc7b2650fbd05f2d (the one published in CWS), the one included with above FF and master: bf92ed9c22c9665abc4f04b9d7f9de0578757f93, possibly others.
Not affected: Chromium 35.0.1916.114
Distribution: Arch Linux x86_64

@Lekensteyn The link appears to be dead. Do you have a new one, and does the problem still occur with the latest version of PDF.js at https://mozilla.github.io/pdf.js/web/viewer.html and the latest graphics drivers?

No response, closing (it's also tracked upstream).


Add work items to issues

In your proposal there is a plan to have CI set up around now.  I'd like to know how well you have progressed?  (This is a genuine question.  It is not a request for progress disguised as a question.)
I wonder if you can turn the work into trackable issues in this repository.  They do not need to be accurate.  They should help you better plan for the work and save you time.  What do you think?

Any update?

Sorry for late reply.
I was busy for one activity and other midterm exams for the past two weeks. So I may miss my schedule a lot. I'm still learning FFT now so I haven't add CI. I'll try my best to catch up the schedule.
As for opening issues for tracking progress, I haven't tried such things before, but it sounds useful. I'll give it a try when I start developing. Thanks!

Pay attention to your projects all the time.  You will be pulled in more directions in after leaving the campus.  Consistent progress on what you do will bring you a more joyful career.
Feel free to close the issue when you see fit.


next() callback in beforeRouteEnter stopped working with Vue 2.5.16

Version
3.0.1
Reproduction link
http://jsfiddle.net/09Lom5h4/3/
Steps to reproduce
Reproduced with Vue 2.5.16.

Open provided jsfiddle.net link;
Open browser console;
Click on 'first a' ;
Click on 'first b' ;
Click on 'second a' ;
Click on 'second b';
And click on 'first a' again;
Inspect console output.

What is expected?
Callback passed to next() call in beforeRouteEnter hook must be called at all times.
Expected console output:
beforeRouteEnter:
vue-component-3: mounted
vm.title: first_a
beforeRouteEnter:
vm.title: first_b
beforeRouteEnter:
vue-component-4: mounted
vm.title: second_a
beforeRouteEnter:
vm.title: second_b
beforeRouteEnter:
vue-component-3: mounted
vm.title: first_a

What is actually happening?
Something very peculiar.
The callback (let's call it CB) that's passed to next() only gets called for an instance of component that is mounted first. Any transitions to other instances of that component do trigger beforeRouteEnter hook, but CB doesn't get called. However, if you transition to other instances, and then back to the first one, CB gets called again.
Actual console output:
beforeRouteEnter:
vue-component-3: mounted
vm.title: first_a
beforeRouteEnter:
<-- missing output: vm.title: first_b
beforeRouteEnter:
vue-component-4: mounted
vm.title: second_a
beforeRouteEnter:
<-- missing output: vm.title: second_b
beforeRouteEnter:
vm.title: first_b <-- this is another bug, which is already reported, issue 2062
vue-component-3: mounted

This behaviour appeared somewhere between Vue versions 2.5.13 and 2.5.16, (almost) everything is fine with 2.5.13:
http://jsfiddle.net/09Lom5h4/4/
I said almost fine, because it overlaps with another bug:
https://github.com/vuejs/vue-router/issues/2062
I've actually used the JSFiddle from that issue to demonstrate this one.

I just found the same problem
sad story

Same problem when I upgraded Vue to 2.5.16.

Any update on this one? The projects on which I need to fix the vue version to 2.5.15 are piling up slowly and introduce other problems.
Is there anything we can do to help fixing this?

Thanks for reporting this, let's keep #2255 which points out another problem as well


Does not render diagram when used with mkdocstrings

Kroki diagrams within Python docstrings, parsed with mkdocstrings, do not render. Currently, the only way to render kroki diagrams is with .md files, not .py files.
Please see reference to mkdocstrings recipe:
https://mkdocstrings.github.io/recipes/

That's probably because this plugin operates on Markdown, and mkdocstrings does not generate Markdown or pass anything to other plugins, so this plugin never gets to see docstrings and operate on it. mkdocstrings works at the Markdown-to-HTML conversion layer.
There are two ways to make this work: rewrite this plugin as a Markdown extension, or hack into mkdocstrings' internals. I'll try t share a link that explains this later.

Yep, nothing we can do here.
I'm closing this one.


feat: Add S3 lambda event support to Parser utility

This schema is currently supported in the validator utility.
This issue will add the schema to the parser utility.
Please note that there's no need for an envelope in this case, just a schema class.

Launched in 1.9.0 now


Replace quickview button by see product button on customizable product

Questions
Answers

Description?
The quickview is useless on customized product because we need to customize it before adding it to cart

Type?
bug fix

BC breaks?
no

Deprecations?
no

Fixed ticket?
Fixes https://github.com/PrestaShop/PrestaShop/issues/29659.

How to test?
Go on a product list with a customizable product and see if the button is well replaced

I don't like it. All of these wording look like this should be a primary button you should click.
I don't get this quickview feature, Prestashop is the only ecommerce platform I know that has this “open in modal feature” and all my clients in history wanted to remove it. 😁

@NeOMakinG and @Hlavtox : did you agree on something ? 😅 I'm talking about @Hlavtox last message

I don't know, just a comment. 😄 For sure Valentin's PR is a good solution for now.. 👍
But, I would personally think that it would be good to make some exploration and remove the quickview feature. Why the hell I would want to open a product page in a modal anyway? 😄 It's bringing us only trouble. 🤷♂


View image

So my app can either have PDF documents or images, does this support images?

if your device supports images then yes.


Improve AppInsights logging

Codecov Report

Merging #115 into master will decrease coverage by 0.18%.
The diff coverage is 81.7%.

@@            Coverage Diff             @@
##           master     #115      +/-   ##
==========================================
- Coverage   91.37%   91.18%   -0.19%
==========================================
  Files          21       21
  Lines         858      885      +27
==========================================
+ Hits          784      807      +23
- Misses         74       78       +4

Impacted Files
Coverage Δ

opwen_email_server/services/sendgrid.py
77.41% <0%> (ø)
:arrow_up:

opwen_email_server/utils/collections.py
100% <100%> (ø)
:arrow_up:

opwen_email_server/utils/log.py
86.41% <80.85%> (-1.82%)
:arrow_down:

opwen_email_server/actions.py
95.93% <96.15%> (-0.37%)
:arrow_down:

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 5706e01...57507df. Read the comment docs.


BSP-29 : Add request validator

Improving the validator by :

being strict with the number type
adding a verification on the field "no_update_after"
extracting the data field validation
adding data validation on inventory update

The changes look good thanks! Could you rebase on top of main, I think you need to remove the commit that was just merged in.
Also, make sure that the commit message has the ticker number. thanks!


Dependencies too constrained

Issue Type

Bug report

Molecule and Ansible details
ansible-2.7.10
molecule-2.20.1

Molecule installation method (one of):

pip

Ansible installation method (one of):

pip

Detail any linters or test runners used:
Tests run on a Docker ubuntu:bionic with python3.6
The command is:
pip3 install --upgrade -r requirements.txt

the requirements.txt file contains:
ansible
ansible-lint
ansible-bender
molecule
yamllint
docker

Desired Behavior
pip dependencies should be resolved without any conflict.
Actual Behaviour
the install gives:
ERROR: molecule 2.20.1 has requirement Jinja2==2.10, but you'll have jinja2 2.10.1 which is incompatible.
ERROR: molecule 2.20.1 has requirement PyYAML==3.13, but you'll have pyyaml 5.1 which is incompatible.
ERROR: molecule 2.20.1 has requirement six==1.11.0, but you'll have six 1.12.0 which is incompatible.
ERROR: molecule 2.20.1 has requirement tabulate==0.8.2, but you'll have tabulate 0.8.3 which is incompatible.

molecule constraints its dependencies too much as dependencies with newer minor versions (Semver syntax) conflict.

See https://github.com/ansible/molecule/issues/1831 (this is probably a duplicate).

Yep.


Function Expose Table 1060

Table 1060 Payment Service Setup contains two functions SaveSet()  and LoadSet(). Could you please expose these for extension development?

The functions are local. Can you describe the scenario in which you need to directly call them from extension?

@Avanti05, How did you get to the point of being able to create a new payment service type? was there any documentation you found?

Didn't find any documentation, but like notifications there is an event publisher for OnRegisterPaymentServices where you can insert the Payment Service.
You can take a look at the MS wallet extension do get an idea of which fields to assign.


Query about Excluding Cluster scoped resources when 'exclude-namespaces' flag is used

Currently, Kubescape excludes cluster-scoped resources along with the provided namespaces when scanned with exclude-namespaces flag.
When a user uses --exclude-namespaces flag, shouldn't we only exclude the mentioned namespaces and scan all others including cluster scope resources  (especially RBAC resources)?
Consider this scenario,
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRoleBinding
metadata:
  name: democrb
subjects:
- kind: ServiceAccount
  name: demosa
  apiGroup: ""
  namespace: demons
roleRef:
  kind: ClusterRole
  name: crdemo
  apiGroup: ""

kind: ClusterRole
apiVersion: rbac.authorization.k8s.io/v1
metadata:
  name: crdemo
rules:
- apiGroups: ["*"]
  resources: ["*"]
  verbs: ["*"]

Suppose the user decided to exclude kube-sysytem & kube-public to avoid clutter. In that case, we would not be able to inform the user about the sub-optimal practice of granting all permissions.

This is a good point.
We have decided to ignore the cluster scope with the thought that a user who excludes a specific namespace does not want to see the complete picture of his cluster.
Also, the fact that the ClusterRoleBinding can have a ServiceAccount so Kubescape showed that the ServiceAccount failed confused many users, and we did not want to change ClusterRoleBinding by removing the ServiceAccount.


reinstall pkg-config

when I try to install postgresql on ubuntu 15.04 I ger
Error: ?home/me//.linuxbrew/opt/pkg-config not present or broken
Please reinstall pkg-config. Sorry :(
I tried reinstalling brew still the same Error

Try brew install pkg-config


Portarelle Client Tests moved into Bridge

Issue
Description
PASS
FAIL
Future
Excluded

Supported in Bridge 15.0
14000

Supported in Bridge Frameworks  (Queue and Stack classes )
36

Failing tests should pass once all portarelle issues have been closed: Defects (48) / Features (19)

413

#1626
IReadOnlyCollection, IReadOnlyList, IReadOnlyDictionary

79

#1623
Lazy

22

#1625
HttpUtility

12

#1631
[BindToThisParameter] Attribute

2

#1675
[CustomInitialization] Attribute

14

#1670
JsDictionary. Covered in Bridge by functionality within Object. The JsDictionary class is not required in Bridge.

34

#1667
Excluded tests as Not C# API

138

#1666
Excluded tests as Not JS API

193

Total
Original: 14943
14036
413
163
331

Additional Comments
Some tests had to be changed because the original expected result of the Test did not match the .NET expected result. Within Bridge, the expected result should match the .NET expected result. Detailed in #1668.
List of revisions made to Portarelle because the tests and functionality being tested do not reflect C# API - #1667.
List of revisions made to Portarelle because the tests and functionality being tested do not reflect JavaScript API - #1666.

Portarelle migrated into Bridge repo and located at Batch4.
Batch4 test runner html file located at portarelle.html.
36 Queue and Stack tests moved into Bridge.Frameworks repo.

We are updating the statistics in the issue description as we go and fix the Portarelle issues.
Once all the Portarelle issues are fixed we will merge the Portarelle tests (Batch4) with Batch1 tests.

Updated 2017-02-13: additional 100+ tests passed https://github.com/bridgedotnet/Bridge/issues?q=is%3Aissue+label%3Aportarelle+milestone%3A16.0+is%3Aclosed

9 String tests excluded due to non C# API - #2392

How to convert this?
public static JsDictionary<string, BindingHandler> BindingHandlers {
get {
return null;
}
}
I do not understand how to use Object in that case.

@hardhub,
Does .Net Dictionary work for your case?
    public static Dictionary<string, BindingHandler> BindingHandlers
    {
        get
        {
            return new Dictionary<string, BindingHandler>();
        }
    }

@LeonidVeriga
Yes it can be a solution...
I tried to rewrite Saltarelle bindings for Knockout. But migration from JsDictionary is never described.
So I just skipped it and left custom Knockout bindings initialization for JS side.
If dictionary will be converted to the following initialization in JS:
bindingHelpers.someKey1 = myBindingHandler1;
bindingHelpers.someKey2 = myBindingHandler2;

where bindingHelpers is dictionary, someKey is string, and myBindingHandler is BindingHandler class then it is a solution.

Four bugs fixed: #1554 #1562 #1591 #1596

#1571 implemented

#1616, #1617, #1618, #1619, #1572, #1573 implemented

#1615 fixed

Passing Saltarelle tests from merged into main test project Batch1.
Failing Saltarelle tests remain in Batch4: 116 failing assertions (should add a number of tests related to not implemented functionality - see column Future here)

#1569 fixed

#1592 fixed


Fix bode plot with Hz and multiple systems

Otherwise each subsequent system is plotted against
a difference omega range

Coverage decreased (-0.01%) to 79.15% when pulling aa243d2e1a4467672ff3cdd11c19b1e459f316ae on nzjrs:master into 90133e8ec4088ea96fbd2fb7858258b392b7d365 on python-control:master.

Coverage decreased (-0.01%) to 79.15% when pulling aa243d2e1a4467672ff3cdd11c19b1e459f316ae on nzjrs:master into 90133e8ec4088ea96fbd2fb7858258b392b7d365 on python-control:master.

Coverage decreased (-0.01%) to 79.15% when pulling aa243d2e1a4467672ff3cdd11c19b1e459f316ae on nzjrs:master into 90133e8ec4088ea96fbd2fb7858258b392b7d365 on python-control:master.


zita-resampler includes host /usr/lib64 path (target-host contamination)

I'm using your recipe here
ERROR: zita-resampler-1.6.0-r0 do_package_qa: QA Issue: zita-resampler: The compile log indicates that host include and/or library paths were used.
Please check the log '/home/softin/Work/RASP/build/tmp/work/cortexa7hf-neon-vfpv4-poky-linux-gnueabi/zita-resampler/1.6.0-r0/temp/log.do_compile' for more information. [compile-host-path]
ERROR: zita-resampler-1.6.0-r0 do_package_qa: QA run found fatal errors. Please consider fixing them.
ERROR: zita-resampler-1.6.0-r0 do_package_qa: Function failed: do_package_qa
ERROR: Logfile of failure stored in: /home/softin/Work/RASP/build/tmp/work/cortexa7hf-neon-vfpv4-poky-linux-gnueabi/zita-resampler/1.6.0-r0/temp/log.do_package_qa.28778
ERROR: Task (/home/softin/Work/RASP/meta-moddevices/recipes-sw/zita-resampler/zita-resampler_1.6.0.bb:do_package_qa) failed with exit code '1'
The log.compile then reports
...
arm-poky-linux-gnueabi-g++  -march=armv7ve -marm -mfpu=neon-vfpv4  -mfloat-abi=hard -mcpu=cortex-a7 --sysroot=/home/softin/Work/RASP/build/tmp/sysroots/raspberrypi2 -Wl,-O1 -Wl,--hash-style=gnu -Wl,--as-needed  -L/home/softin/Work/RASP/build/tmp/work/cortexa7hf-neon-vfpv4-poky-linux-gnueabi/zita-resampler/1.6.0-r0/zita-resampler-1.6.0/libs -L/usr/local/lib64 -o zretune zretune.o audiofile.o dither.o -lzita-resampler -lsndfile -lrt
/home/softin/Work/RASP/build/tmp/sysroots/x86_64-linux/usr/libexec/arm-poky-linux-gnueabi/gcc/arm-poky-linux-gnueabi/5.4.0/ld: warning: library search path "/usr/local/lib64" is unsafe for cross-compilation
/home/softin/Work/RASP/build/tmp/sysroots/x86_64-linux/usr/libexec/arm-poky-linux-gnueabi/gcc/arm-poky-linux-gnueabi/5.4.0/ld: warning: library search path "/usr/local/lib64" is unsafe for cross-compilation
DEBUG: Shell function do_compile finished

Interesting: My log.do_compile contains same but does not complain - so I can reproduce. Will look into tomorrow.

Should be fixed by cc9fc64e. Wonder what I have to change to get same errors...
If it does not fix for you please re-open this issue

Solved! Thanks a lot


Summarize model size in MegaBytes [WIP]

Fixes # (issue)
#4807
This PR adds a Summarization of the model with estimated total model size.
Estimated Total model size is  "input_size + forward/backward pass size + total params size".
e.g
VGG16 with input (1, 1, 28, 28)
Input Size (MBs) --> 0.57
Total Params Size (MBs) --> 528 (e.g downloading weights size)
Forward/Backward Size (MBs) --> 218
Total Estimated Model Size (MBs) --> 747
Testing

Test knownet (a simple convolution NN)
tests knownet model with different input shapes and batch sizes.
we test the corresponding model sizes with pre calculated model size.
Test Different Input Types:
tests a DummyNetwork with different input types.

Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together? Otherwise, we ask you to create a separate PR for every change.
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified; Bugfixes should be including in bug-fix release milestones (m.f.X) and features should be included in (m.X.b) releases.

Did you have fun?
Make sure you had fun coding 🙃

WIP please suggest how to proceed from here.

@awaelchli @rohitgr7  I'm sensing a usage of LRU cache in model summary, as calculating out and input sizes can be redundant,
thoughts?

Would you mind adding some tests ?

sure wont mind adding tests :)

Hello @kartik4949! Thanks for updating this PR.

In the file pytorch_lightning/core/memory.py:

Line 226:17: E722 do not use bare 'except'

In the file tests/core/test_memory.py:

Line 24:5: E731 do not assign a lambda expression, use a def

@justusschock can u just tests?

You could start with simple models like one or two layers, where you know the size / can calculate them by hand and then compare it to the automatically inferred size. Also you could do it once for a more complex model.

You could start with simple models like one or two layers, where you know the size / can calculate them by hand and then compare it to the automatically inferred size. Also you could do it once for a more complex model.

Thanks doing the same:)

Fixes # (issue)
#4807
This PR adds a Summarization of the model with estimated total model size.
Estimated Total model size is "input_size + forward/backward pass size + total params size".
e.g
VGG16 with input (1, 1, 28, 28)
Input Size (MBs) --> 0.57
Total Params Size (MBs) --> 528 (e.g downloading weights size)
Forward/Backward Size (MBs) --> 218
Total Estimated Model Size (MBs) --> 747
Testing

Test knownet (a simple convolution NN)
tests knownet model with different input shapes and batch sizes.
we test the corresponding model sizes with pre calculated model size.
Test Different Input Types:
tests a DummyNetwork with different input types.

Before submitting

[x]  Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x]  Did you read the contributor guideline, Pull Request section?
[x]  Did you make sure your PR does only one thing, instead of bundling different changes together? Otherwise, we ask you to create a separate PR for every change.
[ ]  Did you make sure to update the documentation with your changes?
[ ]  Did you write any new necessary tests?
[ ]  Did you verify new and existing tests pass locally with your changes?
[ ]  If you made a notable change (that affects users), did you update the CHANGELOG?

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[ ]  Is this pull request ready for review? (if not, please submit in draft mode)
[ ]  Check that all items from Before submitting are resolved
[ ]  Make sure the title is self-explanatory and the description concisely explains the PR
[ ]  Add labels and milestones (and optionally projects) to the PR so it can be classified; Bugfixes should be including in bug-fix release milestones (m.f.X) and features should be included in (m.X.b) releases.

Did you have fun?
Make sure you had fun coding

@awaelchli @ananyahjha93 @williamFalcon @Borda I think this PR is ready , need to add some complex models for testing
i.e 1. multiple input model, 2. multiple blocks 3. multi output.
then we are good to go! :)

@Borda dropped the idea of input size, we are good with example_input_array only which will act as custom input_size.

@Borda
so made new changes,

Dropped input_size, example_array will be used instead
Model size shown only in FULL mode.
Added new test which checks for nested models.

All Tests are passed :)

@awaelchli it works with nested sequential and modular models.

@awaelchli only assumptions are for input type.
And the complex code under def _total_out_params size
Is to calculate forward and backward pass!
If we will consider only total params as model size then it will be a cleaner implementation
But i suggest to show total model size with forward pass too :)

Hey @kartik4949,
Any updates on tis PR ?
Best,
T.C

Hey @kartik4949,
Any updates on tis PR ?
Best,
T.C

The implementation works with every mode i tested
May be some refactors :)
If anyone want to help :)

The implementation works with every mode i tested
May be some refactors :)
If anyone want to help :)

@awaelchli @justusschock  Hey
lets calculate model forward pass size only
as it will have lesser complication and further we can add more feature to this one.
what you say ?
if so then I can remove the complex feature.
lets conclude this PR faster

@awaelchli @justusschock  Hey
lets calculate model forward pass size only
as it will have lesser complication and further we can add more feature to this one.
what you say ?
if so then I can remove the complex feature.
lets conclude this PR faster

@kartik4949 Yes, I think that's best. And for the more complicated part, maybe it's better to do this in a separate follow up PR :)

@kartik4949 Yes, I think that's best. And for the more complicated part, maybe it's better to do this in a separate follow up PR :)

@justusschock thanks for responding will remove other sizes and conclude this one :)

@justusschock thanks for responding will remove other sizes and conclude this one :)

@justusschock hey, done with changes.
btw i did rebase on 1.15
but pr is for 1.12-dev.
idk how to interpret this :)

@justusschock hey, done with changes.
btw i did rebase on 1.15
but pr is for 1.12-dev.
idk how to interpret this :)

closing as pr is bit messy, starting fresh

closing as pr is bit messy, starting fresh


Enabler: Upgrade UAA to 3.2.1+

In order to fix vulnerabilities, enable theming, support SAML clients, and increase bus-factor on our UAA deployment, we want to upgrade our production version of UAA to 3.2.1 or higher.
Acceptance Criteria:

[ ] Production UAA is demonstrated to be running UAA 3.2.1 or higher

Notes:

There's a (hard-to-exploit) CVE we need to fix
Newer versions enable theming without any code modification, which we want
We can use the new SAML IDP functionality to authenticate our ideas portal, among other things
Having Chris run through this will help him get to grips with UAA, BOSH, and Concourse

Looking at the upstream commit log: https://github.com/cloudfoundry/uaa/commits/master
It looks like 3.3.0 is going to drop real soon, just FYI depending on timing with this, we might want to go right to that version.

As far as I'm concerned, the upgraded UAA in staging is now ready to go to production, but since it was left on an older version for so long, there's a risk it may cause problems that the staging tests didn't uncover.
Perhaps it's best to hold this release until after your IRL event in case triage is required after it's deployed?

Let's live dangerously on this one... The IRL event also means that there's
going to be a lot fewer people trying to use it for the next couple days. :)
On Tue, May 3, 2016 at 6:37 AM, Chris Nelson<EMAIL_ADDRESS>wrote:

As far as I'm concerned, the upgraded UAA in staging is now ready to go to
production, but since it was left on an older version for so long, there's
a risk it may cause problems that the staging tests didn't uncover.
Perhaps it's best to hold this release until after your IRL event in case
triage is required after it's deployed?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/18F/cg-ops/issues/6#issuecomment-216529722

This was actually updated, right? Can't remember or figure out where to look in CI to confirm, but closing on the assumption that it was, since it's awaiting acceptance. (Reopen if it's not the case.)

Yeah it was.  You can view the source of the login page to see the version and commit that's running.
--
cnelson
On May 14, 2016, at 19:00, Bret Mogilefsky <notifications<EMAIL_ADDRESS>wrote:
This was actually updated, right? Can't remember or figure out where to look in CI to confirm, but closing on the assumption that it was, since it's awaiting acceptance. (Reopen if it's not the case.)

You are receiving this because you were assigned.
Reply to this email directly or view it on GitHubhttps://github.com/18F/cg-atlas/issues/6#issuecomment-219261699

Yeah it was.  You can view the source of the login page to see the version and commit that's running
$ curl https://login.cloud.gov/login | grep title
<div class="copyright" title="Version: 3.3.0, Commit: 127ab6f, Timestamp: 2016-04-27T19:50:09+0000, UAA: https://uaa.cloud.gov">


Download speed limit not working

I tried to set download speed limit to 1024 kB/s and torrents does not start, it stay on 0/0 kB/s downloads with many seeders. Without speed limiting works fine.
Latest kodi and Ementum plugin.

http://elementum.surge.sh/contribute/log/

Thanks for your feedback.
I have submitted an issue to torrent library - https://github.com/anacrolix/torrent/issues/304

I am closing this issue as we migrate to another torrent engine. See #351


Fixed broken link for reference implementation link from contrib to bundles

As per https://github.com/apache/sling/commit/90d2ed9e42deb144a7f6e1610871e72726cd810a

Looks like this was fixed in aa459fd8ef4c333d81125155d49e15fd72f2b376 by @raducotescu, closing.


refactor: Better typing of PackageJson

I noticed that we lacked some type safety around our package.json types, so I updated the PackageJson type we use throughout build-tools to extend the PackageJson type from type-fest. This gets us a bit better type safety and some better uses of TypeScript types like Record.

Thanks for the review, @Josmithr!


Snapshot expiry datetime is returned differently to what is written

When creating a snapshot the default expiry time is Go's zero date time (0001-01-01 00:00:00+00:00), however when read back from the database it is returned as a different time:
lxc storage volume create zfs vol1
lxc storage volume snapshot zfs vol snap0

Read back from database via LXD (via Go) show the expiry date as 0000-12-31T23:58:45-00:01 (this shows as in LMT timezone if inspected via logging the returned time.Time variable). Using t.UTC() will convert it to the correct 0001-01-01 00:00:00+00:00 value. So it looks like the timezone is being stored or retrieved incorrectly.
lxd sql global 'select * from storage_volumes_snapshots'
+-----+-------------------+-------+-------------+---------------------------+
| id  | storage_volume_id | name  | description |        expiry_date        |
+-----+-------------------+-------+-------------+---------------------------+
| 548 | 546               | snap0 |             | 0000-12-31T23:58:45-00:01 |
+-----+-------------------+-------+-------------+---------------------------+

Shutting down LXD and then consulting the offline readonly DB snapshot seems to suggest it is being stored correctly in the database:
sqlite3 /var/lib/lxd/database/global/db.bin
sqlite> select * from storage_volumes_snapshots;
548|546|snap0||0001-01-01 00:00:00+00:00

Furthermore when editing the snapshot volume, because the value returned to the user is different from what is apparently stored in the DB, the resulting value stored in the DB is different from the previous value.
Whoops, looks like set doesn't work for snapshots, will open another bug, separately.
lxc storage volume set zfs vol1/snap0 expires_at 0000-12-31T23:58:45-00:01
Error: not found

OK lets try another approach:
lxc storage volume edit zfs vol1/snap0
description: ""
expires_at: 0000-12-31T23:58:45-00:01
name: snap0
config: {}
content_type: filesystem

Exit without changing the value (but will result in the record being updated).
lxc storage volume show zfs vol1/snap0
description: ""
expires_at: 0000-12-31T23:58:15-00:01
name: snap0
config: {}
content_type: filesystem

Hrm, so the displayed expiry date is not the desired 0001-01-01 00:00:00+00:00, nor 0000-12-31T23:58:45-00:01 that was previously displayed, but now its 0000-12-31T23:58:15-00:01. Its moved back in time by 30s.
Lets try that again:
lxc storage volume edit zfs vol1/snap0
description: ""
expires_at: 0000-12-31T23:58:15-00:01
name: snap0
config: {}
content_type: filesystem

lxc storage volume show zfs vol1/snap0
description: ""
expires_at: 0000-12-31T23:57:45-00:01
name: snap0
config: {}
content_type: filesystem

Yep, its moved back by another 30s.
Whilst we're at it:
lxc storage volume get zfs vol1/snap0 expires_at

Yep thats broken too. Will include as part of the earlier additional bug report.
Right lets try something else, setting an expiry time manually via lxc storage volume edit to ``
That is displayed correctly:
lxc storage volume show zfs vol1/snap0
description: ""
expires_at: 2022-01-04T14:00:00Z
name: snap0
config: {}
content_type: filesystem

But now if I specify a Zulu time in the summer of GMT, e.g. 2022-08-04T14:00:00Z I get something different:
lxc storage volume show zfs vol1/snap0
description: ""
expires_at: 2022-08-04T15:00:00+01:00
name: snap0
config: {}
content_type: filesystem

Oh so now its been shifted in the future by 1 hour (presumably because although the time was provided in GMT, its being displayed in local BST time). This is confirmed by using sqlite3 and it shows it stored as 2022-08-04 14:00:00+00:00.
Interestingly, editing this and re-saving doesn't shift the time like it did with the zero time value, suggesting that the issue here is due to incompatibilities between DST in the timezones being used in dqlite or go.

Tracked down the issue to go-dqlite:
https://github.com/canonical/go-dqlite/blob/4123028456df7f817eda59fa4f3672de725724a1/internal/protocol/message.go#L537-L550
Added some logging and we can see the zero time being retrieved from the database and then converted to local:
fmt.Printf("tomp %s %s %s\n", value, t.String(), t.In(time.Local))

tomp 0001-01-01 00:00:00+00:00 0001-01-01 00:00:00 +0000 UTC 0000-12-31 23:58:45 -0001 LMT

Commit that introduced this broken behaviour:
https://github.com/canonical/go-dqlite/commit/34867b2d19d5ecba78e4b787a0f3030a08c01f16

I think the issue here is that go-dqlite behaves like go-sqlite3 with the ?_loc=auto connection argument passed, but only in the retrieval direction and not in the write direction (go-sqlite3 also always writes out timezone info and ignores ?_loc setting since https://github.com/mattn/go-sqlite3/commit/7b0d180ce9aa2473631426dbc011e34b435d5c65).
However this results in datetimes being altered compared to what they were inputted.
I believe we should by default only use time.ParseInLocation with UTC and not attempt to convert to local time.
This way if a time has timezone info it will be returned as-is, and if not, will be treated as UTC.
https://github.com/mattn/go-sqlite3/blob/2b131e01c1cb65fbcbb56f7e73821a96ae661517/sqlite3.go#L2170-L2181
Then we could also implement the _loc connection string argument to support on-the-fly conversion of datetimes to the specified timezone or the system timezone.
What do you think @stgraber @MathieuBordere ?

Yeah, treating things as UTC in the DB is probably safest. With dqlite, we could have a mix of systems across multiple timezones, so having things rely on the local timezone is quite dangerous.


Not working with twitter

i am using deeplinking with twitter share and when i click on link on twitter app , which was shared from our app its directly move on playstore to download app but app has already store and loged in. and this functionality properly working when i shared on facebook.

Hi @arpitjoshi08 -- I'm unable to reproduce this issue with my test apps. Please review the following two sample projects: https://github.com/BranchMetrics/Android-Branch-Refer-A-Friend-Example
https://github.com/BranchMetrics/Branch-Example-Deep-Linking-Branchster-Android -- make sure you have implemented Branch correctly (check your manifest file, application class, activity). If you are still struggling, please write into<EMAIL_ADDRESS>and we can work with you 1:1. So far I've tested with a few links from native Twitter on Android and they are deep linking directly to my apps, so I'm suspecting a configuration error on your end.

Hey arpitjoshi08 -- If you are referring to mobile Twitter on Android (using Twitter through the Chrome browser for example), I have some more information for you:
Unfortunately, this is a limitation of the Chrome browser and is triggered by the way that mobile web Twitter redirects to the link and there's nothing we can do to improve it. Basically, when clicking the link in the Twitter web app, they "fake click" (set window.location in JS) the link likely to handle tracking analytics and such. Chrome only supports opening up the app from a user-triggered link click, and fake clicks don't count. Chrome used to show a big fat error message here, but they added support to redirect to the Play Store only, which we now use.
Apologies but you've bumped into an edge case in the platform that we can't patch. We'll continue to monitor this situation and fix it once there's some way to do so. Fortunately, this should be a limited use case, since a majority of mobile users will just be inside Twitter's app.

(https://github.com/BranchMetrics/Android-Branch-Refer-A-Friend-Example) this link working perfectly but getting  issue  in my app ,i am not understanding that why this occur. its happen only with twitter and working with app other like facebook, gmail etc.. and i matched my code with reference code its mostly same i think issue may be in setting on branch website.

and one more issue is getting that i need to get successfully upload listener so i am using this code
`branchUniversalObject.showShareSheet(getActivity(),
linkProperties,
shareSheetStyle,
new Branch.BranchLinkShareListener() {
@Override
public void onShareLinkDialogLaunched() {
}
                @Override
                public void onShareLinkDialogDismissed() {
                }

                @Override
                public void onLinkShareResponse(String sharedLink, String sharedChannel, BranchError error) {
                    if (error == null) {
                        Log.d(TAG, "success Data share");
                    //need to call webservice for update
                    }
                }

                @Override
                public void onChannelSelected(String channelName) {
                }
            }`

and  onLinkShareResponse listener always called when shared popoup open but there is no any guarantee that user will share or discard popup. so i need to call webservice on success of shared but i think there is no any method for that.

Hey @arpitjoshi08 -- glad the test app is working for you! Here's the dashboard settings:

As for the listener you're interested in: we can't listen for events beyond the user selecting an app to share to, as that app has now taken over inputs. If you shared to Twitter, then killed the twitter app before pressing post, we can't provide a share sheet process killer listener to capture that event. Don't get me wrong: it would be pretty cool. But it would also make Android a scary platform to support, or it would require outlandish permissions. One solution is that you could label your event here as "SHARE_CREATED" or since we're open source, clone the repo and add this functionality yourself. If you come across anything awesome, let me know and I'll review it!

Hey  twitter redirection app issue has resolved now working properly. Actually issue was in twitter app i think, there was an older version installed in my device so it was not working with that , then i updated twitter app and check, it working properly and now redirect to my app if installed else move to playstore. thanks bro.

Oh that makes me happy to hear, thanks for the update & you're welcome!


Hook for linting IaC configs?

Stumbled across repo this from a discussion in the Trivy repo regarding pre-commit hooks. Was wondering if you would be interested in adding support for the config flag for e.g. scanning Dockerfiles and other IaC? I can create a PR to add if so.

Sounds interesting, not sure I complete follow ?
You mean having another hook that only reacts on the trivy.yaml ?

If I'm understanding how everything works correctly, it would basically be another check. Something like the following would be the raw Docker command, which I think would map to pre-commit.
This should work against any repo with a Dockerfile in it if you want to try testing.
run -t --rm -v $PWD:/src -v /tmp/trivycache:/root/.cache/ aquasec/trivy conf --severity HIGH,CRITICAL /src

Potentially something like this in pre-commit?
- id: trivyconfig-docker
  name: trivyconfig-docker
  entry: aquasec/trivy:0.37.1 config --cache-dir /src/.pre-commit-trivy-cache --severity HIGH,CRITICAL
  language: docker_image
  pass_filenames: false

Sorry for responding so late. Could you create a PR?
I created a example repo as well, maybe you could create a show case there too

Sure thing, will try to get around to it in the next few days here.

https://github.com/mxab/pre-commit-trivy/pull/7.
Will get a PR going in the example repo as well.

Nice, currently without my computer. Will look at it begin of next week

Thank you :)
Created a new release

Nice, I will give it a test and update docs in the example repo.


Can Language Servers that read "local" files work with virtual workspaces?

I have a language server for a programming language that, when a workspace is open, finds the project's main file and parses it to analyze its dependencies. I'd followed the advice in #132 to find and open the file locally from disk. Given the push to support virtual workspaces, is there any way for my extension to replicate the parse-"local"-files behavior with virtual workspaces? My extension is structured along the lines of the LSP sample code.

We are working on a FS API for Language servers. Until that is available servers can't access files in a virtual workspace. What you can do in preparation is to make sure all your FS code is async since the LSP FS requests will be async as well.

We are working on a FS API for Language servers. Until that is available servers can't access files in a virtual workspace. What you can do in preparation is to make sure all your FS code is async since the LSP FS requests will be async as well.

Has there been progress on this, or is there another GitHub issue I should watch? I'm currently in the process of rewriting the Accord Project VSCode extension as a Web Extension and would love to have this feature.

There are issue and PRs in the LSP repository. But know big progress so far.


Add RBBGCMuso

Insert URLs to the project here:
https://github.com/hollorol/RBBGCMuso

[x] The projects is active, documented, open source licensed, shows usage from external parties and is directly targeting environmental sustainability. Find more details in the Contribution Guide.

All issues labeled as 'Good First Issue' of the project listed on OpenSustain.tech will be visible on ClimateTriage.com. This is a great way to welcome new community members to your project.

@hollorol @zbarcza

Thank you very much for including our project!


get basis w/o sparse matrix

New function build_basis_only_from_LO that works similarly to internal function build_sparse_matrix_from_LO, but does not compute or return the sparse matrix.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

21 of 26 (80.77%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.05%) to 89.307%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Hamiltonians/abstract.jl
21
26
80.77%

Totals

Change from base Build<PHONE_NUMBER>:
-0.05%

Covered Lines:
4268

Relevant Lines:
4779

💛  - Coveralls


question about derivatives and preprocessing

Your dataset is wonderful, thank you for your great contribution!
I notice that in the dataset paper, you said in the "fMRI preprocessing" session that "fMRI preprocessing was only done on the derivative data." I wonder if the derivative here is generated from the raw data and then further processed to generate the "preprocessed_data"? If so, could you share how you get the derivatives and the code for preprocessing?
Looking forward to your replies!

Hi there,
Sorry for any miscommunication. The data under data/preprocessed_data is the preprocessed data used in all the papers to fit models with directly. The rough outline of the preprocessing is described in the dataset descriptor paper here but we have no plans to release the preprocessing code. It was all developed in lab and is not well documented. If you are interested in processing the data yourself from the raw data I would recommend using fmriprep which is what I use now and gives very similar results by the point of interpreting models.
Best,
Amanda

Hi Amanda,

If you are interested in processing the data yourself from the raw data I would recommend using fmriprep which is what I use now and gives very similar results by the point of interpreting models.

I'm currently utilizing fmriprep as well and was curious about the specific parameters you're using with fmriprep. Additionally, I'd like to know which confounds you've selected for denoising the preprocessed fMRI data. Thank you for sharing your insights!
Best regards


The commit that will make your "project" work.

Your project had so much issues, Copyright was still showing 2018 instead of 2019, The redirect wasn't working either, code was copied so ugly
Next time use "View page source" instead of copying it using the "Inspect element" tool. :)

Thanks for your contribution. I will make some changes and merge this.

Thanks for contributing! <3


adding roles popup gets behind the user popup

ye so as you can see there the menu for adding roles gets behind the user popout after clicking the "+" button to add roles. This happens with any user popout, doesn't matter if it's on the member list or on chat.

Doesn't Happen to me tho

Closing this issue as my theme doesn't modify it, seems to be a problem on your side, either look at theme/plugins that you installed/check your quick css